Oklab and Laba color space bounds are incorrect

This is consistent with the docs for the a and b fields, but I would argue that the docs are wrong here. In Lab, the a and b are unbounded and their practical ranges depend on the reference white. Similarly, the lightness is also only lower-bounded by 0, but technically any lightness can be expressed in Lab. Where 100 is defined as reference white, you can have brighter colors than your reference, so an upper bound does not make sense.
Some software clamp the a and b values to the -128..127 range as they would work in signed 8-bit integers, but this is technically clipping the Lab color space (note that palette uses this anyway as their bounds for a and b).
I don't know what sources the documentation used to determine the original ranges of the fields, and it's possible that they actually referred to a variant of Lab; but it stands to reason that the definition for this color type should follow the most generally used variant of the color space (that is, in its full name, L*a*b* CIE 1976), in which case the only meaningful bound to check is lightness > 0.

Originally posted by @SolarLiner in https://github.com/bevyengine/bevy/pull/12525#discussion_r1527573500
To fix this:

Update the docs to soften the language on the bounds to something like "typically within the range of...", and clarify that these values are not clamped.
Remove the incorrect clamping added in #12525.

there had already been some discussion about this on the discord:
https://discord.com/channels/691052431525675048/691052431974465548/1212607622893805578
https://discord.com/channels/691052431525675048/691052431974465548/1212616712982364160

this issue should also include Lcha and Oklcha

Just throwing it out there, even sRGB colors can be unbounded (in intensity) if they are scene-referred. My understanding is that we should only ever fully clamp when creating a new display-referred color, and then only to the gamut of the display (or the best available approximation).
I have a longer explanation of this in the works. Got nerd-sniped on photometry and colorimetry recently.

Similarly, here's a list of resources wrt. digital color:

https://www.youtube.com/watch?v=fv-wlo8yVhk
https://solarliner.dev/post/a-colorful-journey/ (disclaimer: it's my own writeup after falling down the digital color rabbit hole a few years ago)
https://www.youtube.com/watch?v=53m-17Y_zJg (in the context of Blender, but mostly general technical knowledge)

Ok, here's what it looks like to me:

In order to support wide-gamut spaces (webgpu currently just supports P3 but it's possible it may eventually support more), our LinearRgb color needs to be non-normalized. It can't ever be clamped.
XYZ is inherently display independent. We can clamp to the positive quadrant, but is otherwise we can't bound it, so why bother.
Lab and Lch are inherently display-independent. We can clamp the chroma components to the locus of visible colors but we shouldn't clamp lightness. But it's not clear what method we should use to turn imaginary colors into real ones.
OkLab and OkLch are probably the same story, but I haven't checked.
sRGB and it's various alternate encodings HWB, HSV, HSL are all inherently display-referred and should be clamped, but we don't actually need to clamp them ourselves because wgsl will do it for us when we pass them into rgba8unorm-srgb.

Conclusion: We should probably remove the ClampColor trait.


Line breaks in JSDocs/Markdown comments render inconsistent space in Intellisense boxes.

VSCode Version: 1.4.0
OS Version: Windows 10

If I have a JSDocs header like this:
/**
 * @summary Does something.
 * @description This is a long description. In code, this description can get pretty long,
 * so I will break it up into multiple lines for easier reading.
 * - Example bullet
 * - Example bullet
 *
 * @returns A value.
 *
 */

VS Code's Intellisense box renders it like this:

@summary Does something.
@description This is a long description. In code, this description can get
pretty long,
so I will break it up into multiple lines for easier reading.

Example bullet

Example bullet

@returns A value.
function ...

When I'm expecting something more like this:

Does something.
This is a long description. In code, this description can get
pretty long, so I will break it up into multiple lines for
easier reading.

Example bullet
Example bullet

@returns A value.
function ...

@jrieken Isn't this up to each language to implement? Understanding @summary and @description?

I believe this is about spacing in LI-tags

I don't think so.
@craxal If the list item spacing bothers you, #11101 is the issue to follow. If, on the other hand, You want the language server to understand @summary and @description in docs, vscode-typescript is where you should file that issue.

To clarify, I see these problems when writing in Typescript. Apologies if this should be brought to the Typescript team.
This is about line spacing more than the @summary and @description tags. The Intellisense boxes (or whatever they're called) render lots of extra whitespace. For example, it doesn't seem to distinguish paragraphs well. When I break up a long description comment into multiple lines, the box renders those breaks as-typed. It shouldn't actually do that unless an empty line is in between the breaks. Similarly, the box renders a log of space between list items.
I bring up the tags, however, because @param tags seem to be handled well enough. Hovering over a parameter name in a function definition opens up a box that renders the correct @param text. It even drops the JDocs type information. If @param is recognized, why not other tags, such as those I mentioned?

Yep, for spacing, follow #2330 (also see last comment).
For the tags, that is vscode-typescript.

Ah. Great. Good to know this is being worked on. Thanks!


Change background save file

Changed to using more pythonic way of splitting extensions Added some tests and some cosmetic clean up

Current coverage is 72.66% (diff: 100%)

Merging #147 into master will increase coverage by 0.14%

@@             master       #147   diff @@
==========================================
  Files            94         94
  Lines         10095      10109    +14
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7321       7346    +25
+ Misses         2774       2763    -11
  Partials          0          0

Powered by Codecov. Last update 596de63...2687459


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 813 ms

Resolved: Bofrost.de performance has improved in e9946f4 after 56 minutes.


Create new subjects endpoint

Preflight Checklist

[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Problem Description
The UI needs to figure out subjects from the returned data
Proposed Solution
Create a new /subjects endpoint for a meter that returns all available subjects.
Alternatives Considered
No response
Additional Information
No response

#224


[Question]

Can you give a simple example how the decoder works. I tryed it this way but
it doesn't work. The file size gets bigger but the bool var "hasMore" never changes his state.
Another question, why must i call decoder.Initialize()? Why ist it not automatically called when i create the decoder object?
            byte[] buffer = new Byte[2048];
            bool hasMore = true;

            FileStream fs = File.OpenRead(@"E:\example.hca");
            HcaDecoder decoder = new HcaDecoder(fs);
            decoder.Initialize();

            decoder.WriteWaveHeader(buffer);

            FileStream fo = new FileStream(@"E:\wharg.wav", FileMode.CreateNew);
            fo.Write(buffer, 0, buffer.Length);

            if (buffer.Length >0)
            {
                while (hasMore ==true)
                {
                    decoder.DecodeData(buffer, out hasMore);
                    fo.Write(buffer, 0, buffer.Length);
                }

            }
            fo.Close();```

Hi Clark,
For the first question, I made a demo DereTore.Application.Hca2Wav to demonstrate the decoding process. It works fine with the HCA sample. You can download the sample here.
I think that you didn't gave the decoder a proper-sized buffer. During my test, the buffer should have a size of at least 0x1000(4096) bytes to enable the decoding. There are utility functions HcaDecoder.GetWaveHeaderNeededLength() and HcaDecoder.GetWaveDataBlockNeededLength() to confirm the minimum size buffers required. I forgot to throw an exception for buffers too small, and it will now. Please check the latest commit.
For the second question, it's my bad. I am just scared of throwing exceptions in constructors. After observing classes like FileStream I decided to make the initialization automatic. Now you don't need to manually call Initialize() anymore.


Add license badge and section in README.md

Motivation
Let's users know about this project's license.

This is closes by #70.


bug: option stats is not compiling value true correctly

Bug report
stats: true is actually doing nothing
What is the current behavior?
for stats: true in webpack config, compiler.options.stats = {}. where as for stats: false it's correct compiter.options.stats = { preset: 'none'}
for stats: true -

for stats: false

If the current behavior is a bug, please provide the steps to reproduce.
set stats: true in webpack config and then check the value of stats in the compiler.options

What is the expected behavior?
stats: true should set compiler.options.stats : {preset : 'normal'}.

Other relevant information:
webpack version: 5
Node.js version: 12.16.2

I see preset normal is also missing here, https://github.com/webpack/webpack/blob/8aaff95a1bc7320914b26ac1cead96b18f59f9e2/lib/stats/DefaultStatsPresetPlugin.js#L21-L91

Good catch, let's fix it, should be easy

Seems correct to me.
An empty object {} results in all properties using the default value. So is uses the "normal" stats. There is no "normal" preset.

An empty object {} results in all properties using the default value. So is uses the "normal" stats

@sokra This makes sense, I was confused because it is showing preset normal for stats: 'normal'

@sokra it is hard for plugins/external tools, to detect alias, why do not create alias stats: true === stats: 'normal', stats: false-stats: 'none'`?

@sokra it is hard for plugins/external tools, to detect alias, why do not create alias stats: true === stats: 'normal', stats: false-stats: 'none'`?

I'm fine with that, send a PR.
So there is no bug here...

yep, not a bug we are returning {} for stats === true, we can return { preset: 'normal'} instead to avoid confusion and make it clear like done for stats === false.
https://github.com/webpack/webpack/blob/80b76711f97b7a474a38ce6d7a643eb1e5398273/lib/config/normalization.js#L282-L298


CS2030 Java Style Guide Point 9

"9. Each line is limited to 80 characters in length. You can break a long line into multiple lines to enhance readability, this is called line wrapping. When you do so, each continuation line is indented at 8 spaces (twice the normal indentation) from the original line."
Is there a way to automatically break a long line in VIM?
Thank you.

You can set the maximum width of each line by using this command on vim NORMAL MODE
:set tw=80

Alternatively, if you want to handle line wrapping yourself to ensure that it conforms to the style guide, u can use the command  :set colorcolumn=80or add set colorcolumn=80 into your .vimrc. It'll mark the 80th column in red so you'll know if your line exceeds 80 characters

You can set the maximum width of each line by using this command on vim NORMAL MODE
:set tw=80

Just note that although it can be automated, I would recommend breaking the long line manually, as it gives you more control over where you want to break up the line.
For example,
// 151 characters
System.out.println("<some very long text in front> The values are a = %d, b = %d, c = $d, d = $d", myVariableA, myVariableB, myVariableC, myVariableD);

//automatically broken up at character 80
System.out.println("<some very long text in front> The values are a = %d, b = %d
        , c = $d, d = $d", myVariableA, myVariableB, myVariableC, myVariableD);

// Broken up manually to improve readability
System.out.println("<some very long text in front> The values are a = %d, b = %d, c = $d, d = $d",
        myVariableA, myVariableB, myVariableC, myVariableD);


Proposal: Showing height status as diff, not fraction of ints

While trying to follow the blockchain status of the test device I had trouble comparing the API height and current device blockchain height (now in millions). You need to consider at least 4 or 5 digits to compare properly. The sync level doesn't help either. It generally shows as 100% but the device could be behind more than 20+ blocks.
So it would be better to show blockchain status as diff like this:
Height Status     +1 (1234431)

+1 is the diff, (device block height - API block height) and the value in parenthesis is the API height. This way we can easily spot failing devices with a glance without doing some arithmetic.
A problematic height status would be shown like this:
Height Status     -56 (1234431)

Plus, we should also add three or 4 decimals to the sync level to make it meaningful for problem tracing. But in millions level, we would need at least 6 decimals. That could be a little bit of a problem, though.
This issue is also a complementary feature for #286

Closing this as there is no longer sync


Specified key was too long; max key length is 767 bytes

Laravel Version: 5.5 -- dev
PHP Version: PHP 7.1.8-2+ubuntu17.04.1+deb.sury.org+4
Database : 10.1.25-MariaDB

Description:

[Illuminate\Database\QueryException]                                                                                SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes (SQL: alter table users add uni     que users_email_unique(email))

Steps To Reproduce:
1.Create the new app
2.Set Env values
3.Run php artisan make:auth
4.Run php artisan migrate
5.Got Error
Steps To Fix:
1.Goto app/Providers/AppServiceProvider.php
2. Add  \Illuminate\Support\Facades\Schema::defaultStringLength(191); in that boot method
is it really correct fix ?

Yes, check the documentation.

@themsaid  thanks
check out https://laravel.com/docs/master/migrations/#creating-indexes

10.2 MariaDB is now stable, any reason for not upgrading to fix this issue @kalidasweb?

I am confused .. is this fixed in current laravel versions or not?

@amenk There is no 'bug', 'fix' per se. It's because the default length is different. Either upgrade your MySQL/MariaDB versions or add the defaultStringLength(191); to boot method of AppServiceProvider.php.

I'm getting the same error in a Laravel project even after making various updates. I'm running the following versions on a local ampps server on Windows 10:
Apache 2.4.27
Php 7.1.8
MySql 5.6.35 InnoDB utf8_general_ci collation (changed from utf8mb4_general_ci)
Laravel 5.6.39
In AppServiceProvider.php near the top I've added:
use Illuminate\Support|Facades\Schema;
And in function boot I've added:
Schema::defaultStringLength(191);
In config/database.php I've changed charset tp 'utf8',collation to 'utf8_general_ci', and engine to 'InnoDB ROW_FORMAT=DYNAMIC'.
Using php artisan migrate:fresh I even still get the error if in the xyz_create_users_table.php I change $table->string('email')->inique(); to $table->string('email');. The first error says:
1   Doctrine\DBAL\Driver\PDOException::("SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes")
\vendor\doctrine\dbal\lib\Doctrine\DBAL\Driver\PDOStatement.php:144
I was wondering if the doctrine cache needed clearing but php app/console cache:clear said "could not open input file".

I'm getting the same error in a Laravel project even after making various updates. I'm running the following versions on a local xampp server on Windows 10:
Apache 2.4.34
PHP 7.2.8
MariaDB 10.1.34
Laravel  5.7.26
Illuminate\Database\QueryException  : SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'users' already exists (SQL: create table users (id int unsigned not null auto_increment primary key, name varchar(255) not null, email varchar(255) not null, email_verified_at timestamp null, password varchar(255) not null, remember_token varchar(100) null, created_at timestamp null, updated_at timestamp null) default character set utf8mb4 collate 'utf8mb4_unicode_ci')
the problem exists to the version 5.4 laravel why no one will change so that you do not have to change every time you create a new project?

The exception that you pasted is not the same as the exception at issue here.

Need at least Maria 10.2.2. I would think that any "fixes" that the framework could provide, so that this works even when using old database versions, would be a huge bc break dealing with migrations.


Create contracts for Page and PageFactory

@Gummibeer Here are the factories for the Page and PageFactory. If this changes are approved I can try my luck with the PageData class too 🙂

After you've fixed StyleCI most/all of the TravisCI errors should be resolved.

@Lloople thanks for doing this! :)


Python version bump.

Updating Python to 3.10.
This is a fixed PR of: https://github.com/dirac-institute/trailblazer/pull/42

 Bump Python version to 3.10

 Bump Python version to 3.10

 Bump Python version to 3.10


spawn Unknown system error -86 with uploader 0.8.0 (via codecov-action)

Describe the bug

This job passed yesterday (codecov-action@v3 picked up<EMAIL_ADDRESS>on macos-12)
This re-run of the same job failed today (codecov-action@v3 picked up<EMAIL_ADDRESS>
All runs of that workflow are failing similarly, since 0.8.0 was released (≈4h ago, at time of writing).
I haven't figured out the root cause, but the error is:
spawn Unknown system error -86

I'm guessing it's some sort of Intel/ARM confusion with macos-12.

Workaround (from https://github.com/single-cell-data/TileDB-SOMA/pull/2827):
 - uses: codecov/codecov-action@v3
   with:
     flags: python
     token: ${{ secrets.CODECOV_TOKEN }}
+    version: 'v0.7.3'

Hi @ryan-williams
Does updating to v4 of the Codecov Action help?

@drazisil-codecov I believe it did, thanks! I made the change in https://github.com/single-cell-data/TileDB-SOMA/pull/2856 / https://github.com/single-cell-data/TileDB-SOMA/pull/2856/commits/cdfb4d22fcbac35a16ec57eff384ab93294aed4a and it seems to be working.
Feel free to close this issue or leave it open, seems like others are running into the same error.

I'll close it, it should be findable via search.
Glad that fixed it!


[torchlib] Convolution decomp is not correct

Unsqueeze was added even though it was not expected:
https://github.com/microsoft/onnxscript/blob/justinchu/more-torchlib-fixes/onnxscript/function_libs/torch_lib/ops/core.py#L2048
import torch
from onnxscript import ir
import onnxscript
import onnx

lower = "at_conversion"
resnet18 = torchvision.models.resnet18(
    weights=torchvision.models.ResNet18_Weights.DEFAULT
).eval()
sample_input = (torch.randn(4, 3, 224, 224),)
exported = torch.export.export(resnet18, sample_input)
# Then use dynamo export on the exported program

# comparison
torch.onnx.export(resnet18, sample_input, "resnet18_torchscript.onnx")
proto = onnx.load("resnet18_torchscript.onnx")
proto = onnx.shape_inference.infer_shapes(proto)
onnx.save(proto, "resnet18_torchscript.onnx")
onnx.checker.check_model(proto, full_check=True)

cc @fatcat-z @xiaowuhu


Mockito Mocks in 4.0.0

Describe the bug
After I upgraded to 4.0.0, my tests started crashing. After some debugging I found that they fail because when GetControllers are used in them, the onStart callback is null.
The stacktrace is:
_InternalFinalCallback.call
package:get/…/src/lifecycle.dart:12
GetInstance._startController
package:get/…/src/get_instance.dart:284
GetInstance._initDependencies
package:get/…/src/get_instance.dart:239
GetInstance.find
package:get/…/src/get_instance.dart:327
GetInstance.put
package:get/…/src/get_instance.dart:101
Inst.put
package:get/…/src/extension_instance.dart:90
main.<fn>.<fn>
test\page\controls_page_test.dart:66

I create the mock as follows:
class ControlsControllerMock extends Mock implements ControlsController {}
Any idea how to make Mockito mocks of GetControllers working again?

Problem solved:
You can use Mock either by extending it, or by mixing it in.
So when I use class ControlsControllerMock extends GetxController with Mock implements ControlsController {} it works fine.


Fix broken project labels

Labels on the project only returns an empty array. The reason is the API no longer has labels directly on the project response. A separate call to projects/{project_id}/labels needs to happen.
https://www.pivotaltracker.com/help/api/rest/v5#Labels
If you view the project response, you will see that labels is no longer part of it.
https://www.pivotaltracker.com/help/api/rest/v5#Project

@wdhays2 if you want the labels to be returned with the project in one request you need to include it in the fields list since it isn't returned by default.
This will return the project with an array of full label objects in one request.
client.project(project_id, fields: ':default,labels')

This will return the project with an array of label object with just the name in one request.
client.project(project_id, fields: ':default,labels(name)')

This test passes with the current code.
  describe '.labels' do
    let(:project_with_labels) do
      VCR.use_cassette('get project with labels') do
        client.project(project_id, fields: ':default,labels')
      end
    end

    it 'gets all labels for this project' do
      VCR.use_cassette('get labels', record: :new_episodes) do
        labels = project_with_labels.labels

        labels.wont_be_empty
        label = labels.first
        label.must_be_instance_of TrackerApi::Resources::Label
      end
    end
  end

I think you can just update your new labels and 'label_ids` accessors to only make the extra request if the labels don't already exist.

See Story#tasks for an example.
https://github.com/dashofcode/tracker_api/blob/master/lib/tracker_api/resources/story.rb#L63-63

I'll merge this in and make the required changes.

@wdhays2 I had to do a bunch of refactoring so ending up just pulling in some of your code to support lazy loading of project labels.
See: https://github.com/dashofcode/tracker_api/pull/48 and https://github.com/dashofcode/tracker_api/commit/c65ed4c1fc81c7a73842e09e381e84ea9a8b982e
I'm going to close this pull now.


do doesn't run after "installing" local plugin

Plug '~/Developer/coc-metals', {'do': 'yarn install --frozen-lockfile'}

Strangely the do command there does not run after running PlugInstall. What am I missing?

VIM - Vi IMproved 8.2

Type:

[x] Bug
[ ] Enhancement
[ ] Feature Request
[x] Question

OS:

[ ] All/Other
[ ] Linux
[x] OS X
[ ] Windows

Vim:

[x] Terminal Vim
[ ] GVim
[ ] Neovim

I also have this issue, none of de do commands seem to run

We use the term "unmanaged plugin".
https://github.com/junegunn/vim-plug#example
" Unmanaged plugin (manually installed and updated)
Plug '~/my-prototype-plugin'

vim-plug does not install or update such unmanaged plugins. There is no update, so no post-"update" hook. If you do want vim-plug to "manage" such plugins in your local file system, you can write Plug 'file:///path-to/my-plugin' and vim-plug will try to clone the directory to its own plugin repository (e.g. ~/.vim/plugged/my-plugin). But when you do that, any changes in the original directory become unavailable, you'll have to manually run :PlugUpdate to synchronize the directories.

However I have this issue with managed plugins, telescope-fzf-native and markdown-preview.
Fzf has a do : make and md preview has cd app && yarn install. Neither of these run.

Please see https://github.com/junegunn/vim-plug#post-update-hooks

you can force it to run unconditionally with the bang-versions of the commands: PlugInstall! and PlugUpdate!.


[propose] create an awesome-nocode and add this to awesome list of awesome lists

https://github.com/sindresorhus/awesome

Excellent idea sir!
I'm definitely eager to discover other no-code frameworks and best practices.


[BUG] Exception occurs when using the Picker and SharedPreferences in Flutter 3.0.2

Describe the bug

There is a problem with the picker and Flutter 3.0.2.
An exception is thrown at startup if you are using SharedPreferences in your application.
How to reproduce

Steps to reproduce the behavior:

Create a new Flutter 3.0.2 project.
Modify pubspec.yaml to contain:
cupertino_icons: ^1.0.2
shared_preferences: ^2.0.15
camera: ^0.9.8+1
Modify the default main.dart to contain:
import 'package:flutter/material.dart';
import 'package:shared_preferences/shared_preferences.dart';
void main() {
WidgetsFlutterBinding.ensureInitialized();
checkPreferences();
runApp(const MyApp());
}
checkPreferences () async {
var preferences = await SharedPreferences.getInstance();
var all = preferences.getKeys();
print(all.toString());
}
... the remainder of the default app ...
Compile and run.
You'll get an exception.
Notice that the picker was not used at all.  It was just included in pubspec.yaml.
If you remove the dependency from pubspec.yaml then no error will happen.
Expected behavior

Expected the app not to crash with an exception.
Screenshots (If contains)

flutter: camera_avfoundation threw an error: Binding has not yet been initialized.
The "instance" getter on the ServicesBinding binding mixin is only available once that binding has been initialized.
Typically, this is done by calling "WidgetsFlutterBinding.ensureInitialized()" or "runApp()" (the latter calls the former). Typically this call is done in the "void main()" method. The "ensureInitialized" method is idempotent; calling it multiple times is not harmful. After calling that method, the "instance" getter will return the binding.
In a test, one can call "TestWidgetsFlutterBinding.ensureInitialized()" as the first line in the test's "main()" method to initialize the binding.
If ServicesBinding is a custom binding mixin, there must also be a custom binding class, like WidgetsFlutterBinding, but that mixes in the selected binding, and that is the class that must be constructed before using the "instance" getter.. The app may not function as expected until you remove this plugin from pubsp<…>
[VERBOSE-2:shell.cc(93)] Dart Unhandled Exception: Binding has not yet been initialized.
The "instance" getter on the ServicesBinding binding mixin is only available once that binding has been initialized.
Typically, this is done by calling "WidgetsFlutterBinding.ensureInitialized()" or "runApp()" (the latter calls the former). Typically this call is done in the "void main()" method. The "ensureInitialized" method is idempotent; calling it multiple times is not harmful. After calling that method, the "instance" getter will return the binding.
In a test, one can call "TestWidgetsFlutterBinding.ensureInitialized()" as the first line in the test's "main()" method to initialize the binding.
If ServicesBinding is a custom binding mixin, there must also be a custom binding class, like WidgetsFlutterBinding, but that mixes in the selected binding, and that is the class that must be constructed before using the "instance" getter., stack trace: #0      BindingBase.checkInstance. (package:fl<…>
[VERBOSE-2:ui_dart_state.cc(198)] Unhandled Exception: MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/shared_preferences)
Version information

Device: [e.g. iPhone X]
OS: [e.g. iOS 14.7.1]
Package Version: [e.g. 2.4.1]
Flutter Version: [e.g. v2.5.0]
[✓] Flutter (Channel stable, 3.0.2, on macOS 12.4 21F79 darwin-arm, locale en-US)
• Flutter version 3.0.2 at /Users/jos/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision cd41fdd495 (12 days ago), 2022-06-08 09:52:13 -0700
• Engine revision f15f824b57
• Dart version 2.17.3
• DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/jos/Library/Android/sdk
• Platform android-31, build-tools 31.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
[✓] Android Studio
• Android Studio at /Users/jos/Download0/Downloads/Android Studio 4.2 Preview.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.2)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 68.1.4
• Dart plugin version 221.5787.37
[✓] VS Code (version 1.61.2)
• VS Code at /Users/jos/Downloads/Visual Studio Code.app/Contents
• Flutter extension can be installed from:
🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (3 available)
• iPhone 13 (mobile) • 157CD17F-8E4D-4F52-9506-0F0982D23F43 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
• macOS (desktop)    • macos                                • darwin-arm64   • macOS 12.4 21F79 darwin-arm
• Chrome (web)       • chrome                               • web-javascript • Google Chrome 102.0.5005.115
[✓] HTTP Host Availability
• All required HTTP hosts are available
Additional context

I have this issues.

Run flutter pub upgrade and ensure these dependencies are 0.9.8+2.


Does selenide support safari browser on iPad

I am using Appium to run the test script in safari browser on real device. I am using IOSDriver provided by Appium library. Also, I found below browsers are supported in Selenide
https://github.com/codeborne/selenide/wiki/How-Selenide-creates-WebDriver
Does anyone have know how to use IOSDriver with Selenide API?

Selenide supports Safari browser on desktop, but as for mobiles - we didn't try it out yet, so there may well be problems:)
Viele Grüße
Alexei Vinogradov
Sent from mobile phone

Am 06.03.2017 um 11:20 schrieb sagarzlp<EMAIL_ADDRESS>I am using Appium to run the test script in safari browser on real device. I am using IOSDriver provided by Appium library. Also, I found below browsers are supported in Selenide
https://github.com/codeborne/selenide/wiki/How-Selenide-creates-WebDriver
Does anyone have know how to use IOSDriver with Selenide API?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@asolntsev Need your help on this issue.

@sagarzlp Yes, Selenide supports Safari browser.
You can create such class:
public class CreateIOSDriver implements WebDriverProvider {
public WebDriver createDriver(DesiredCapabilities capabilities) {
  URL appiumUrl = null;
  try {
    appiumUrl = new URL("http://<IP_ADDRESS>:4723/wd/hub");
  } catch (MalformedURLException e) {
    fail(e);
  }
  return new IOSDriver(appiumUrl, capabilities);
  }
}

And use it:
setWebDriver((WebDriver) new CreateIOSDriver(capabilities));
Configuration.timeout = 35000;


Use Option+Cmd+Left/Right for script editor history navigation on macOS

The default shortcuts were conflicting with TextEdit shortcuts on macOS.
Note that I haven't tested this. I'd like someone on macOS to test this to make sure it works as expected.
This can be trivially cherry-picked to the 3.x branch.
This closes https://github.com/godotengine/godot/issues/44858.

@Dentrax You can test this change on the 3.x branch:

Clone the 3.x branch of the Godot repository: git clone https://github.com/godotengine/godot.git --branch=3.x
cd to the Godot Git repository and run curl -L "https://github.com/godotengine/godot/pull/47618.patch" | git am.
Build the editor, following the instructions in the Compiling for macOS documentation.

Thanks! It seems correctly set on File shortcuts.
But somehow CMD + OPTION + LEFT still does not work as expected; it moves the cursor to the left.

Should I additionally have to map my ALT key to OPTION in editor settings? 🤔

It should work on on master, but on 3.X it will probably not work due to the input being swallowed up by TextEdit before it can be handled by the popup menu shortcut.

Hey @Calinou, should we merge this?

Hey @Calinou, should we merge this?

As you said before, Cmd + Option + Left still doesn't work, so I'm not sure if it's worth merging this and having just Cmd + Option + Right work.

Needs rebase to use .macos feature override instead of the removed APPLE_STYLE_KEYS, and figure out shortcuts that work.

We checked this PR in the input PR meeting. It needs an update with the new override system.
But looks good otherwise, if the shortcuts are still conflicting that's a thing to be solved.

Superseded by https://github.com/godotengine/godot/pull/91887.


New tracker API - stream managers registration summary

resolves https://github.com/twitter/heron/issues/1627
New tracker API /topologies/stmgrsregsummary?cluster=<cluster>&topology=<topology>&environ=<environment> that shows stream managers registration summary info.
Schema of the result is
{'registered_stmgrs': list of registered stmgrs names,
 'absent_stmgrs': list of absent stmgrs names}

This feature sits along with "/stats","/exceptions","/exceptionsummary" but is not metircs/exceptions stats related. It will remain in TMaster when we eventually move tmetrics-collector to MetricsCache?

@huijunw yes, it will remain in TMaster. As for the URL path, do you have any suggestion? Is /topologies/stmgrsregsummary good?

The URL looks good to me. @billonahill may have ideas on path naming though.

This is a great feature, but as implemented it seems overly specific. Could we instead extend upon /topologies/executionstate (I don't recall exactly what that returns) or introduce some other endpoint that represents topology state? This seems more extensible.
I would then expect to be able to make a single request to find the state of the topology, along with a collection of objects for all stream managers, each including their registration state.
That way the API would have a few key concepts like physical plan, logical plan and state.

ping for review @billonahill @huijunw

I have updated.

The C++ part looks good to me.

👍


Exception class names and error messages renamed from fail ro raise

In d763917c87dcf755300d64b10d32acffb4d60046 an indiscriminate search and replace appears to have been run to raise exceptions via raise instead of fail.  However, the string "fail" was not only present as fail() calls.  Exception class and error message strings were also changed:

"Container raiseed to stop" error
Chef::Exceptions::Validationraiseed class

These should probably be changed back.  The error messages, specifically 'Container raiseed to stop', do not make sense as written.

#674 will fix this


Phone App?

I adore this chrome extension, but would love love love if this was available for phones? I use some apps for school and also talk to family who deadname me on my phone, and would love this to help with my dysphoria!
Thank you very much!

iPhone or Android? I know there are some browsers and other apps that allow you to inject userscript code into them and other apps if you are on Android.

This would be super cool... No idea how it could be done though. Don't know what the process of reading and injecting into apps looks like 🤔

When I was using android, I had an adblocker called AdGuard that allowed you to inject userscripts as a premium feature.
https://kb.adguard.com/en/general/userscripts


While Converting from PyTorch to ONNX   Type Error: Type 'tensor(bool)' of input parameter (/Equal_27_output_0) of operator (CumSum) in node (/CumSum) is invalid.

hi @qbxlvnf11
i wanted to convert parseq model to onnx , like below
resnet18 = torch.hub.load('baudm/parseq', 'parseq', pretrained=True) this was added in main function
resnet18.eval().to('cuda')    this also added in the code otherwise its throwing input type error
providers = [
('CUDAExecutionProvider', {
'device_id': 0,
'arena_extend_strategy': 'kNextPowerOfTwo',
'gpu_mem_limit': 2 * 1024 * 1024 * 1024,
'cudnn_conv_algo_search': 'EXHAUSTIVE',
'do_copy_in_default_stream': True,
}),
'CPUExecutionProvider', ]
sess = rt.InferenceSession(args.output_path, providers=providers)

after adding above things in code we run our code like
python convert_pytorch_to_onnx.py --dynamic_axes True --sample_image_path roi_1.jpg  --output_path onnx_output_explicit.onnx --opset_version 14
But we are Getting some kind of error like
2023-01-02 11:51:23.380117232 [W:onnxruntime:, graph.cc:1231 Graph] Initializer onnx::MatMul_5865 appears in graph inputs and will not be treated as constant value/weight. This may prevent some of the graph optimizations, like const folding. Move it out of graph inputs if there is no need to override it, by either re-generating the model with latest exporter/converter or with the tool onnxruntime/tools/python/remove_initializer_from_input.py.
Traceback (most recent call last):
File "convert_pytorch_to_onnx.py", line 176, in
sess = rt.InferenceSession(args.output_path, providers=providers)
File "/usr/local/lib/python3.8/dist-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 347, in init
self._create_inference_session(providers, provider_options, disabled_optimizers)
File "/usr/local/lib/python3.8/dist-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 384, in _create_inference_session
sess = C.InferenceSession(session_options, self._model_path, True, self._read_config_from_model)
onnxruntime.capi.onnxruntime_pybind11_state.InvalidGraph: [ONNXRuntimeError] : 10 : INVALID_GRAPH : Load model from onnx_output_explicit.onnx failed:This is an invalid model. Type Error: Type 'tensor(bool)' of input parameter (/Equal_27_output_0) of operator (CumSum) in node (/CumSum) is invalid.
dont know what is the exact problem it is , can u please solve it and give us that code which will be helpful
thanks and regards , i hope u can solve this as much as earlier

I do not understand exactly why this error occured.
Maybe, some operators of Pytorch model do not match operators of ONNX or input data is invalid ...
Sorry to can not be help.


Search for closed duplicate PRs as part of bump-formula-pr

A detailed description of the proposed feature
Update brew bump-formula-pr to search for both open and closed PRs with titles that exactly match the auto-generated bump PR title (formulaname new.version.number). It currently only searches for open PRs with the formula name anywhere in the title.
The motivation for the feature
This will prevent opening PRs that duplicate previous formula bump attempts that were closed for whatever reason. https://github.com/Homebrew/homebrew-core/pull/56197 is a case where this happened. With this proposed feature in place, a warning of a possibly duplicate PR would be printed, but it would allow a person to override with the --force flag that exists.
How the feature would be relevant to at least 90% of Homebrew users
The feature would save time of contributors/maintainers as well as reduce strain on CI in cases where a person submitting a PR has no reason to believe it should succeed this time since they weren't aware it failed the previous time.

What alternatives to the feature have been considered

Users manually searching all PRs in GitHub prior to running brew bump-formula-pr.
Allowing the PR submission to continue and let CI catch it again.

I think this is a good idea! Looking at the PR now.

Corresponding PR has been merged.


Voice Control ideas (brainstorming)

Since this is a diffusion model, I'm wondering if would be possible to implement something like ControlNet did for StableDiffusion/Flux
Basically is it possible to build models that drive the audio generation by, for example, extracting the emotion from a voice reference (different from the reference used to "inpaint"), or the roughness, etc...
I'm not asking now to implement this but I just wanted to understand the possibilities given by a diffusion model for voices

yes, possible. the reason we are open-sourcing is exactly to make it easier for all people to take a try for any idea


Add tests for search top anomaly results API

Description
Add unit and integ tests for search top anomaly results API. Meets the jacoco coverage bar for all classes added as part of the API, removed from the exclusion list in build.gradle.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #325 (225893c) into main (ac69c3e) will increase coverage by 1.73%.
The diff coverage is 30.00%.

@@             Coverage Diff              @@
##               main     #325      +/-   ##
============================================
+ Coverage     75.87%   77.60%   +1.73%
- Complexity     3950     4014      +64
============================================
  Files           295      295
  Lines         17182    17188       +6
  Branches       1812     1816       +4
============================================
+ Hits          13036    13338     +302
+ Misses         3315     3009     -306
- Partials        831      841      +10

Flag
Coverage Δ

plugin
77.60% <30.00%> (+1.73%)
:arrow_up:

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

Impacted Files
Coverage Δ

...a/org/opensearch/ad/model/AnomalyResultBucket.java
62.85% <12.50%> (+62.85%)
:arrow_up:

...ch/ad/transport/SearchTopAnomalyResultRequest.java
90.90% <100.00%> (+90.90%)
:arrow_up:

...ansport/SearchTopAnomalyResultTransportAction.java
78.14% <100.00%> (+71.03%)
:arrow_up:

...g/opensearch/ad/model/DetectorValidationIssue.java
47.36% <0.00%> (-7.02%)
:arrow_down:

...c/main/java/org/opensearch/ad/util/ParseUtils.java
67.31% <0.00%> (-4.29%)
:arrow_down:

...va/org/opensearch/ad/feature/SearchFeatureDao.java
82.78% <0.00%> (-3.86%)
:arrow_down:

.../transport/SearchAnomalyResultTransportAction.java
20.22% <0.00%> (-1.76%)
:arrow_down:

...opensearch/ad/indices/AnomalyDetectionIndices.java
68.35% <0.00%> (-0.19%)
:arrow_down:

...rch/ad/transport/AnomalyResultTransportAction.java
80.18% <0.00%> (ø)

...ansport/handler/AnomalyResultBulkIndexHandler.java
67.74% <0.00%> (ø)

... and 9 more

Are you planning on adding any integration tests with security

Are you planning on adding any integration tests with security

Not as part of this PR. Seems there is a lot of missing API tests for security so seems it would be beneficial to do all of these together as a separate effort.


Collect event coverage

We need to bring together in a useful fashion

the collaborative notes (#135)
presentations (#75)
demos (#43)
posters (#42)
video recordings (apparently no ticket for that?)
Twitter (#87)
as well as
blog posts (e.g. http://gigasciencejournal.com/blog/force2017-in-berlin-changing-the-culture/ or http://openminted.eu/openminted-participates-force2017-conference-two-highly-attended-workshops/ )
pictures people have taken at the event (e.g. #127)
any other materials (images, audio, video, text, annotations, 3D, whatever)

Another one: https://www.force11.org/blog/new-ideas-discussed-force2017

For inspiration on how to present the information, see the FORCE2016 Final Report.docx, which sits in the Final Reports folder in our Gdocs.

This in on our action list of things to complete for FORCE2017 - leave issue open for now


.tag 里如何进行多行注释？

请问，在.tag 文件里如何进行多行注释？
我尝试用了 /*    */  但没成功，请问应该怎样进行多行注释？
谢谢

Hi @kangyiwen
You can only make single line comments using // in TagUI.

谢谢   我现在选中多行，用 ctrl+q （notepad++快捷键）进行多行注释。

Indeed, because TagUI doesn't have the logic to support /* and */
However, a workaround, if really needed is write in JavaScript
js begin
/* some long form comments
some more comments
further comments
end of comment */
js finish

Above will work. But I know is very ugly script lol 🤣

fyi @ruthtxh

还是希望 .tag 未来可以支持多行注释。

Hi @kangyiwen, and @ruthtxh, I had a close look at the TagUI codebase. Technically supporting /* */ block comments is doable. However, it slows down execution by a bit. The most reliable way to reduce chance of breaking existing user scripts is to do special parsing before expanding tagui subflows, and parsing again after.
Parsing before tagui step is important because calling a sub-flow within the comment block should not be done. Parsing after tagui step is also important because the feature should also work for comment blocks in subflows. Then change everything between /* */ to //.
However, this strategy means reading .tag files 2 times extra to do the parsing. And this comes at expense of most users who are not going to use the feature. For now, will recommend using js begin / finish workaround or IDE shortcuts. Perhaps when more users raise this again, can relook whether it makes sense to add that additional overhead.
Also, special care must be taken to make sure common XPath strings like //*are not accidentally treated as multi-line comments, perhaps only when the line begins with /*. This is just one case which can break, but there could be more situations which may break existing user code if not done carefully.

I have some time tonight and was thinking to add in the new feature to support /* */. But after reviewing, I believe it is a net negative for TagUI users in general, due to risks of possibly breaking existing scripts, and the small overhead in parsing when running the TagUI scripts. So I just leave just thoughts above for future consideration when users raise this again.


Translate the user id

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #149
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #150 into master will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #150      +/-   ##
==========================================
- Coverage   12.00%   11.98%   -0.03%
==========================================
  Files          44       44
  Lines        2091     2095       +4
==========================================
  Hits          251      251
- Misses       1804     1807       +3
- Partials       36       37       +1

Impacted Files
Coverage Δ

pkg/ddc/alluxio/transform.go
11.95% <0.00%> (-0.27%)
:arrow_down:

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 dd44502...85ace7e. Read the comment docs.


support for deferred content sizing on the initialCSS

I solved my earlier gripe about the content getting the height: 0 issue by conditionally rechecking the sticky's content node a little later. There are too many parallel things running at load time and the main directive's link() function is not waiting for the content before calculating its dimensions for the defaults.
is this part of the design? Or should I use other angular approaches to solve it? I did attempt to patch this sticky directive myself in hopes it helps.
Also redid the logic for deriving the scrolling viewport. Didn't like simple approach of taking the first one on the page.

can anyone review / test / decide to merge or not?


Changes to versions for 3.12 to work

Notes on changes

Bump numpy version to 1.26. As stated here, this version supports python 3.9-3.12.
Allowing sympy-plot-backends to extend up to before 3.0. Version 3.0 and onwards have breaking changes with dtumathtools.
Reflected changes in packages at conda-forge. When merging/changing this branch, it should be reflected there as well.
Updated version label to 2024.1.0
Added python 3.12 to workflow.
Passes all tests (in python 3.12.1). However, pytest has 1 warning related to datetime and python 3.12 in a dateutil package. Comes from sympy-plot-backends, and is thus not a direct result of this package. This is not considered major, and can be ignored.
Mayavi 4.8.2 has not yet been released. Thus to get dtumathtools onto pypi, the install in pyproject.toml needs to be removed (cannot have install based on path in pypi). We might consider simply removing support for mayavi for the spring 24 release?

All tests passed except the ones with python 3.8. This is due to python 3.8 not being supported by numpy 1.26 (which is required to support python 3.12).
"Simplest solution" would be to drop support for python 3.8.

Notes on changes
* Bump numpy version to 1.26. As stated [here](https://numpy.org/news/#numpy-1260-released), this version supports python 3.9-3.12.

I think we should stick with 3.8 as well, jumping across still supported Python versions is bad in this case IMHO.

* Allowing sympy-plot-backends to extend up to before 3.0. Version 3.0 and onwards have breaking changes with dtumathtools.

Good idea, keep it like this until autumn semester. Then we can bump it.

* Reflected changes in packages at [conda-forge](https://github.com/conda-forge/dtumathtools-feedstock/tree/package_versions_python312). When merging/changing this branch, it should be reflected there as well.

* Updated version label to 2024.1.0

* Added python 3.12 to workflow.

* Passes all tests (in python 3.12.1). However, pytest has 1 warning related to datetime and python 3.12 in a dateutil package. Comes from sympy-plot-backends, and is thus not a direct result of this package. This is not considered major, and can be ignored.

* Mayavi 4.8.2 has not yet been released. Thus to get dtumathtools onto pypi, the install in pyproject.toml needs to be removed (cannot have install based on path in pypi). We might consider simply removing support for mayavi for the spring 24 release?

It shouldn't be required to be removed from the pyproject.toml file. Students have been asked to do pip install dtumathtools which will only use the project.dependencies block. The mayavi dependency will only get installed when doing: pip install dtumathtools[mayavi].
Hence it hasn't been installed anyways.

No, I think you misunderstand what I meant. dtumathtools cannot be uploaded to PyPi with such a dependency (using a git path). Therefore, the package currently on pypi does not have mayavi supported (pip install dtumathtools[mayavi] does not install mayavi). I think we probably should reflect that change in the github, explicitly stating that mayavi is not supported. It is currently not the case, such that pip install .[mayavi] (after having downloaded the github package) still works with mayavi.
As it is not part of the course, it should not be a major problem though.

No, I think you misunderstand what I meant. dtumathtools cannot be uploaded to PyPi with such a dependency (using a git path). Therefore, the package currently on pypi does not have mayavi supported (pip install dtumathtools[mayavi] does not install mayavi). I think we probably should reflect that change in the github, explicitly stating that mayavi is not supported. It is currently not the case, such that pip install .[mayavi] (after having downloaded the github package) still works with mayavi. As it is not part of the course, it should not be a major problem though.

Ok, I would be ok with leaving out mayavi from supported backends. Could you create a new PR for this?

I have removed support for mayavi in a PR here.
Notes on this PR: I updated the description so it should reflect the current changes. Furthermore, all tests successfully passed, so it should be ready for your review, @zerothi.


mc-config.json doesn't work

As far as I can tell, the mc-config.json file is not taking effect as expected.
ddev get oblakstudio/ddev-minio
ddev restart
ddev mc alias ls

Outputs: (Notice the warning about tmp configuration, and that there is no minio key)
mc: Configuration written to `/tmp/.mc/config.json`. Please update your access credentials.
mc: Successfully created `/tmp/.mc/share`.
mc: Initialized share uploads `/tmp/.mc/share/uploads.json` file.
mc: Initialized share downloads `/tmp/.mc/share/downloads.json` file.
gcs
  URL       : https://storage.googleapis.com
  AccessKey : YOUR-ACCESS-KEY-HERE
  SecretKey : YOUR-SECRET-KEY-HERE
  API       : S3v2
  Path      : dns

local
  URL       : http://localhost:9000
  AccessKey :
  SecretKey :
  API       :
  Path      : auto

play
  URL       : https://play.min.io
  AccessKey : Q3AM3UQ867SPQQA43P2F
  SecretKey : zuf+tfteSlswRu7BJ86wekitnifILbZam1KYY3TG
  API       : S3v4
  Path      : auto

s3
  URL       : https://s3.amazonaws.com
  AccessKey : YOUR-ACCESS-KEY-HERE
  SecretKey : YOUR-SECRET-KEY-HERE
  API       : S3v4
  Path      : dns

If you create it manually:
ddev mc alias set minio  http://localhost:10101 ddevminio ddevminio
ddev mc alias ls

It will be listed as expected, and you can go on to use it (eg mc mb minio/my-bucket).
As expected, the alias and bucket will go away with a ddev restart, as they are in tmp.
It also may make sense to make the alias local, as this seems to be the default?

There was a problem with a config directory env var. I fixed that and some other minor annoyances.
Regarding the alias - probably will rename, after they bring the changes from the linux version to the docker one so I can rework the entire addon.

Thanks!


Not able to show the magnifing image

Hi,
Can Anyone Please help to figure the issue and resolution as Not able to show the Magnifying image(largeImageSrc props) on hover and also nor the main image (imgSrc prop), Maybe there is Some CSS not set properly? Please Help to figure put
Using this Code :
import {SideBySideMagnifier} from "react-image-magnifiers";
<SideBySideMagnifier
imageSrc={selectedImage.thumb_url}
imageAlt="Example"
largeImageSrc={selectedImage.url}// Optional
alwaysInPlace='false'
overlayOpacity={0.6}
switchSides={false}
fillAvailableSpace={false}
fillAlignTop={false}
fillGapLeft={0}
fillGapRig ht={10}
fillGapTop={10}
fillGapBottom={10}
zoomContainerBorder="1px solid #ccc"
cursorStyle="zoom-in"
inPlaceMinBreakpoint={641}
zoomPosition="right"
   />

Hi. I had a similar problem. That happened to me because I had:
img { max-width: 100%}
The solution is to remove it.
If you check the image, you can see it

@KitanoR thaanks man!

how to remo

@KitanoR thaanks man!

Hi. I had a similar problem. That happened to me because I had: img { max-width: 100%} The solution is to remove it. If you check the image, you can see it

How do I remove the max-width: 100% just for this image magnifier? I'm a beginner in this area, and I'm using Next.js and Tailwind.


”Roșiești” is listed in the drop-down menu

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

Step 1
Type "Roșiești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Roșiești, VASLUI".


Some devices benefit from being able to being able to use multiple commands. For example pen plotters need a dwell after off to allow the pen to move out of the way. Lines are split with a '\n'.

G-Code allows multiple multiple commands on a line. But I just like having only one command per line.


Fix Set-PnPList -MinorVersion documentation

Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
No related issues
What is in this Pull Request ?
Set-PnPList -MinorVersion affects CSOM property MajorWithMinorVersionsLimit, described by SharePoint with "Keep drafts for the following number of major versions:"
https://github.com/pnp/powershell/blob/7f498aa5ddd2d18784de026b11f4824681e973d9/src/Commands/Lists/SetList.cs#L251

The previous documentation stated "Maximum minor versions to keep" but SharePoint doesn't support to specify how many minor versions to keep, that's a fixed limit of 511. See https://support.microsoft.com/en-us/office/how-versioning-works-in-lists-and-libraries-0f6cd105-974f-44a4-aadb-43ac5bdfd247 for official documentation:

The maximum number of minor versions is 511, and the number of major versions can be adjusted.

It might be good to rename the parameter -MinorVersion to something else, maybe using the same name as CSOM -MajorWithMinorVersionsLimit, but that will break all existing scripts. For now fixing the documentation should be good enough.
Feel free to change the wording, I'm not a native english speaker :)

Thanks @jackpoz !

It might be good to rename the parameter -MinorVersion to something else, maybe using the same name as CSOM -MajorWithMinorVersionsLimit, but that will break all existing scripts. For now fixing the documentation should be good enough.

@gautamdsheth any opinion about this ? Would it be good to add -MajorWithMinorVersionsLimit as parameter alias and add a deprecation notice to -MinorVersion , mentioning that it might have a different behavior than what users have always expected ? Then maybe after enough time removing the -MinorVersion alias


What is the hardware requirement running llm models?

Nice tool and great talk! I am led there through the talk, which was really nice!
Playing with llm, I have successfully run the models such as:
llm -m  GPT4All-13B-snoozy 'Where is the captial of France?'
However, running on a centOS 7 workstation, downloading the model worked fine, but model can not be initiated. I am wondering what is the hardware requirement to run llm models? my CPU is Intel(R) Xeon(R) Gold 5222 CPU @ 3.80GHz, GPU is Quadro P4000
I know the GPU driver isn't properly configured, might that be the cause of problem (update: just configured the GPU properly, but still can not initiate the models ... on the other hand, mac M1/intel works both fine) ?

Can you share the error message you're receiving?
According to the GPT4All project, the requirements are that your CPU must support the AVX or AVX2 extension, which you can test with:
grep -o 'avx\|avx2' /proc/cpuinfo | sort | uniq

If this prints 'avx' or 'avx2' (or both), you should be supported, provided you also have enough RAM for the model. I'm not sure of a reliable way to determine if you have sufficient RAM to run an inference on a model, but I would assume you'll need more than 8GB of RAM to run that model. 16GB should be sufficient. Less will probably work.
Finally you can also try running a much smaller model to see if inference can even happen, or if there's some other issue. Of course small models tend to have worse output, so this is mostly just for testing.

Actually, I just noticed, the output of llm models includes this info:
❯ llm models
...
gpt4all: GPT4All-13B-snoozy - Snoozy, 7.58GB download, needs 16GB RAM
...

This information is gathered via the GPT4All python API, where they get that from I'm not sure.

Thanks!  Testing the CPU:
grep -o 'avx\|avx2' /proc/cpuinfo | sort | uniq
avx
avx2

so there should be no issue on this workstation. The workstation has 64GB RAM, which also should not be a problem.
But even when I try to use a smaller model, I have the following error:
llm -m orca-mini-7b 'What is the capital of France?'
100%|██████████████████████████████████████████████████████████████████████| 3.79G/3.79G [00:38<00:00, 98.9MiB/s]
Error: Unable to instantiate model

@ipstone been over a year, have you solved this? If so, please close this issue

this issue was resolved - somehow I forgot the specifics, perhaps it worked after system reboot or nvidia driver update. Thanks, closing it now.


silent fail bug

hey there, small bug :) when calling this, your lib fails silently:
   index.addObjects({}, function (err, content) {
                    if (err) {
                        // no error
                    }
                    else {
                        // so we are here
                    }
                });

as you can see we pass an object instead of an array
this lib fails silently, so I didn't know what the problem was. you can check for an array by using "obj instanceof Array"
but it shouldn't fail silently right? I fixed it by doing this, of course:
   index.addObjects([{}], function (err, content) {
                    if (err) {
                        // no error
                    }
                    else {
                        // so we are here
                    }
                });

but anyway you should pass back an error in the callback if the dev makes the first mistake, thanks!

Hi, indeed we too often silently fail, I think I will do a pass soon on some obvious mistakes we should report first.
Thanks for reporting! If you have any other silent fail findings, report them again it's good.

for sure np

Will release this today

Was released as 3.9.1 thanks for the idea


[Feature] Refactor Estimator for computing FLOPs/Params/Latency.

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Refactor Estimator for computing FLOPs/Params/Latency.
Modification

add EvaluatorLoop (from mmengine.ValLoop) to execute resources estimation.
add BaseEstimator in mmrazor.structures.
refactor flops_counter to flops_params_counter & add latency measuring.
add layer counter for each op.

TODO

[x] add UT & Docstring.

BC-breaking (Optional)
Does the modification introduce changes that break the backward compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here and update the documentation.
Checklist
Before PR:

[ ] Pre-commit or other linting tools are used to fix the potential lint issues.
[ ] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit tests to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

Now support counting flops with the specified scope. A list of scope names is required from users.


Performance issues with headless-shell compared to google-chrome-stable

What versions are you running?
chromedp/headless-shell:102.0.5005.63
previous ones behave similar
What did you do? Include clear steps.
I am fetching a (big) AngularJS website and printing that as PDF. I am using the image as FROM and am installing my go-application inside.
While I used nearly the default allocator options before, I now changed them like this while trying to figure things out and measuring performance.
opts := append(chromedp.DefaultExecAllocatorOptions[:],
		chromedp.WindowSize(screenWidth, screenHeight), // this was here before

		// these were added while doing performance tests
		chromedp.NoSandbox,
		chromedp.Flag("disable-infobars", true),
		chromedp.Flag("disable-component-update", true),
		chromedp.Flag("disable-domain-reliability", true),
	)

I tried directly inside a VM (compared by extracting the binary from the docker image), using docker on the same system, using kubernetes (k3s) and on google kubernetes engine. Installation directly inside the VM is obviously the fastest. Still all share the same problem: the headless-shell is hella slow.

Reasonable load times.
What did you see instead?
Processing times using the headless shell are roughly about 2 to 3 times as high compared to using google-chrome-stable (or chromium).

I am fetching a (big) AngularJS website and printing that as PDF.

Can you provide a minimal reproducer? And can you share how did you measure the performance?

I am fetching a (big) AngularJS website and printing that as PDF.

Can you provide a minimal reproducer? And can you share how did you measure the performance?

Sadly I cannot. Its an internal application, that is not accessible on the internet. I can try to describe in more detail, give stats about watchers (and whatever stats may help as well) and compare rendering times. I even can spit out debug logs from chrome (I've implemented loggers that can be enabled or disabled).
The rendering time measurement is easy.
t0 := time.Now()
// do pdf generation
fmt.Println("%v", time.Now().Sub(t0))

The pdf generation parts goes like this:
chromedp.ActionFunc(func(ctx context.Context) error {
			buf, _, err = page.PrintToPDF().
				WithPrintBackground(false).
				WithPreferCSSPageSize(false).
				WithScale(1).
				WithDisplayHeaderFooter(true).
				WithPaperWidth(8.27).
				WithPaperHeight(11.69).
				WithLandscape(false).
				WithMarginTop(0).
				WithMarginRight(0).
				WithMarginBottom(0).
				WithMarginLeft(0).
				WithHeaderTemplate("my header").
				WithFooterTemplate("my footer").
				Do(ctx)
			if err != nil {
				return err
			}

			return nil
		})

The render is triggered using an HTTP request to my service. I'm using Gin for that on port 9400.
Afterwards I tried rendering the exact same page multiple times on the various environments using the different container images.
docker-compose.yaml
version: "2.4"

services:
  chromiumservice:
    container_name: myservice
    image: myimage
    ports:
      - "9400:9400"
    volumes:
      - /etc/localtime:/etc/localtime:ro
    restart: "always"
    security_opt:
      - seccomp:./chrome.json

chrome.json: https://github.com/jessfraz/dotfiles/blob/master/etc/docker/seccomp/chrome.json
Inside-VM without docker or kubernetes I simply installed google-chrome-stable/chromium using apt and put the extracted the headless-shell binary into $PATH (after uninstalling google-chrome-stable/chromium of course).

Is it possible to create a reproducer against a public service? For example, https://www.ag-grid.com/example/ (Or any other UI libraries that provide demos).

I'll try my best finding an appropriate example the following days. This was actually critical for me, since the application is just about to be actually used.
That being said I'll stick with google-chrome-stable for now, but it would still be nice to resolve this and possibly make it even faster.

@boindil I'm not sure whether it will make any difference, but would you like to give https://hub.docker.com/r/zeke/headless-shell/tags a try?
And it would be helpful to compare the trace logs. See https://groups.google.com/a/chromium.org/g/headless-dev/c/J6SjKnRQGQc for how to record the trace log. The trace log can be loaded into https://ui.perfetto.dev/ (I don't know how to use yet. But I would like to learn it if you can provide the trace logs).

Hi, I‘ll do that for sure. Right now I am on like 110% load so I cannot do that as fast as I‘d like.
Is on my todo list.


Add lag fields in pg_stat_replication as seconds

Background
It would be nice to read replication lag in time from the primary DB. This is supported by postgresql as pg_stat_replication.(write_lag|flush_lag|replay_lag). postgres_exporter however does not parse the interval values of the native fields, so I forked the repo and wrote a patch to parse them as seconds. (pg_stat_replication.(write_lag_seconds|flush_lag_seconds|replay_lag_seconds) to avoid ambiguity)
Notes

I'm pushing this to our docker org as uprightproject/postgres-exporter:master. Commands to reproduce are

install golang (instructions)
make promu
ensure promu is in path. I needed to add ~/go/bin to path.
promu crossbuild -p linux/amd64 -p linux/armv7 -p linux/arm64 -p linux/ppc64le
make docker
docker tag prometheuscommunity/postgres-exporter-linux-amd64:<branch name> uprightproject/postgres-exporter:master
docker push uprightproject/postgres-exporter:master

I should probably open this PR  also on the upstream

Edit: Actually the master branch broke some other metrics, so I backported this to 0.15.0 here. I pushed that as uprightproject/postgres-exporter:0.15.0