Add option to force VM shutdown via checkbox in shutdown confirm popup

Solves issue #522
Refactored components/VmActions:

Removed RemoveVmAction and integrated it directly into VmActions
Reordered Button attributes to be consistent across all definitions
For buttons that use complex Confirmation text, refactored to be
consistent and harmonize styles

Added data shape to components/Toolbar's match prop
Refactored components/Confirmation:

Embed string text in a <p> block and non-strings in a <div> block with
a shared style to maintain the same styling across both renders
reordered Button attributes to be consistent
removed 'button-l' className since it isn't in use
moved the optional extraButton render inline to the main JSX
refactored styles to use flex box layout so text and buttons are always
nicely centered

This change is

@sjd please always provide a screenshots to simplify the review
Other than that, while playing with it I found 2 issues. I think both issues have been there before, but without the force you realistically did not hit them. But now with the force option you hit them immediately.
1: the shutdown and force stop should be available in all this VM states: up, powering up, powering down, reboot in progress, migrating, suspended, paused, not responding. The most important one is the "powering down" because it is a common case that you first try to shutdown without force and when you run out of patience, than you force it :)
2: the confirmation re-appears. You can get to it like this:
1: go to VM detail
2: press shutdown (pick force so it will happen quickly)
3: wait until the VM shuts down
4: start the VM
When the VM turns up, the confirmation shows up by itself again.

@jelkosz  - The shutdown button's enabled/disabled logic was unchanged.  Shutdown (and force/Power Off) is currently available when:

the VM is not currently being shutdown
the VM status is:

up
migrating
reboot_in_progress
paused
powering_up

That leaves a gap for when a VM is powering_down, or not_responding.  In your opinion, would it be better to patch the extra states in the web-ui code or do it over in the ovirt-ui-components project .  I vote for migrating the /can.*/ functions out of ovirt-ui-components and back to web-ui.  That would make maintenance on that particular bit easier going forward.
As for the confirm popovers not going away, it's fun to click the suspend button and then the shutdown button.  Both popovers stay.  Similar to when you use the left navigation and switch VMs - if you open the confirm popover in one VM then switch to another, the popover doesn't hide.  I consider these issues outside of the scope of this PR and Issue as that behavior wasn't introduced by adding the force shutdown option.

@jniederm - Honestly I don't like either style.
Having the Force checkbox or having a secondary Force button on the confirmation popover is inconsistent with the experience in both webadmin and cockpit for the force shut down.
webadmin (action labeled as "Power Off"):

cockpit (action labeled as "Force Shut Down":

for reference, this is how it looks with the PR so far:

and this is how the remove button looks:

Is there a particular style that UXD approved or do we want to stay self-consistent?

Apologies @sjd78 I missed your last comment until today.
What @sjd78 said in the last comment is correct. Both the Force checkbox and buttons are bad / don't follow patternfly, and should be changed to secondary actions dropdown style like in Scott's screenshots:

@mareklibra @jelkosz agree?
Assuming yes, @sjd78 only worry about force shutdown for this issue, and I'll open one for force remove (awaiting ack from Tomas or Marek)

@sjd78 ,

I vote for migrating the /can.*/ functions out of ovirt-ui-components and back to web-ui.

Sure. Historically, there was a wave migrating the code under ovirt-ui-components due to various reasons, followed by another slower one to migrate it back since the original reasons passed.
So sure - fine for me to migrate it back to this project.

What @sjd78 said in the last comment is correct. Both the Force checkbox and buttons are bad / don't follow patternfly, and should be changed to secondary actions dropdown style like in Scott's screenshots

It makes sense to get aligned. IIRC, implementation in web-ui proceeded the one in Cockpit.
Anyway, there should be extra step added to confirm this destructive action (not needed with recent design).

I just noticed github uses the same secondary action dropdown pattern :)

Update. Per Tomas, for now let's:

go with Jakub's suggestion so it's at least consistent
eventually adopt the secondary dropdown pattern
converting to modals is #587

Force shutdown is now on the extra action button in the style of the force remove extra action button.

@mareklibra - I pulled and updated vm-status.js from ovirt-ui-components.  Would you like me to submit a PR on that project to remove vm-status.js?

@jelkosz - This is exactly what you described with the popover magically staying opened.

I pulled and updated vm-status.js from ovirt-ui-components. Would you like me to submit a PR on that project to remove vm-status.js?

Yes, please. Let's have it consistent ...

I pulled and updated vm-status.js from ovirt-ui-components. Would you like me to submit a PR on that project to remove vm-status.js?

Yes, please. Let's have it consistent ...

https://github.com/oVirt/ovirt-ui-components/pull/50 removes the file from ovirt-ui-components

Does it make sense to increase the dependency version for ovirt-ui-components to 0.3.0 in this PR

No, I'll take care of it independently. Thanks for the patches!

Review status: 0 of 6 files reviewed at latest revision, 5 unresolved discussions.

src/vm-status.js, line 6 at r4 (raw file):
Previously, jniederm wrote…
Please follow Marek's suggestion and use include(). I find it the cleanest and easiest to read solution.

If I understand the performance benchmark (http://jsben.ch/mhdZY) correctly, processing of predefined arrays was faster only because the time to evaluate the setup block is excluded from the test. Hence inlining of array definition shouldn't cause a performance penalty.

In Chrome, the times come out the same for static (array defined once outside of the function) vs inline definition (array redefined on each function execution).  In Firefox, the times for static definition are significantly faster.
In this particular file, with these particular tests it really isn't an issue.

src/vm-status.js, line 37 at r4 (raw file):
Previously, sjd78 (Scott Dickerson) wrote…
Good point.  I'll take the comment out.

Done.

src/components/Confirmation/style.css, line 28 at r1 (raw file):
Previously, sjd78 (Scott Dickerson) wrote…
Done

Done.

src/components/VmActions/index.js, line 139 at r1 (raw file):
Previously, sjd78 (Scott Dickerson) wrote…
Ok.

Done.

src/components/VmActions/index.js, line 182 at r1 (raw file):
Previously, sjd78 (Scott Dickerson) wrote…
I don't have a strong feeling one way or another on that style change.  I'll leave the first attribute on the line with the tag and drop any other attributes below.  At least it'll be self-consistent in this file.

Done.

Comments from Reviewable

Wow - Reviewable did a few things I wasn't expecting.  Live and learn I suppose.

@bond95 , @jniederm , any additional comments, please?


Scratchpad

Hi,
New to Bruno.
Is there any way to run request without saving them to a collection ?
I think Insomnia and Postman call this the Scratchpad.

Hey, @mrleblanc101 we don't currently support this feature.

This is the one reason I still have postman installed.  It’s just faster for random validation that I don’t care to persist in any way


Bump v0.1.8

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


[BEAM-4823] - Adds a Sink to write to Amazon's SNS

Adds a new Sink to write Amazon's SNS [a.k.a. Simple Notification Service]. It uses the AmazonSNS client underneath to submit requests. Adds a retry logic to re-publish requests when writes to SNS failed due to service not available.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

@iemejia - Thank you for looking at the PR. I moved SNS changes to sdks/java/io/amazon-web-services

@iemejia - Thank you for looking at the PR. I have addressed your feedback.

@iemejia - I updated the PR incorporating your feedback.

Mmm weird, tests aren't happy now.

@iemejia - I have always seen
INFO: The AWS S3 Beam extension was included in this build, but the awsRegion flag was not specified. If you don't plan to use S3, then ignore this message.

in my local logs. But in the build server this is flagged as an error.

retest this please

@iemejia - I rebased. Build is still failing & I don't see errors. Do you have any recommendations on how to figure this out?

retest this please

@jhalaria I am still puzzled let's see if this is a CI issue, I tried this locally and everything passes.

@iemejia - All checks now seem to pass. Did you do some magic?

Forget to tell you, thanks a lot for contributing this one @jhalaria !

@iemejia - Thank you for all your feedback. I learned a few things in the process that I can now use within my group at GoDaddy.


removal of all origins and powers

using a custom made origins data pack and am setting it up on my server and this keeps happening when you log into the world there is no origin selection screen, no orb of origin and using commands doesn't work
Have got this issue on both 1.19 and 1.19.2
Server thread/WARN16:20:08 Power "chill:ferrari/hands" will only be partially loaded: Failed to read fields: Error reading Operation: No value was found for name: "set_total" Server thread/ERROR16:20:08 Error loading power "chill:short/fall": Unknown registry key: origins:from_falling, At ResourceLocation: JsonObject; Error reading Operation: No value was found for name: "add_base_late" Loading for all instances of power chill:short/fall failed. Error loading power "chill:bee/speed_jump": Unknown registry key: origins:modify_velocity Loading for all instances of power chill:bee/speed_jump failed. 16:20:09 Error loading power "chill:frog/fall": Unknown registry key: origins:from_falling, At ResourceLocation: JsonObject Loading for all instances of power chill:frog/fall failed. Error loading power "chill:short/speed_jump": Unknown registry key: origins:modify_velocity Loading for all instances of power chill:short/speed_jump failed. Error loading power "chill:saga/refuel": Unknown registry key: origins:modify_inventory, At ResourceLocation: JsonObject Loading for all instances of power chill:saga/refuel failed. Error loading power "chill:tumtum/crafting": Unknown registry key: origins:drop_inventory, At ResourceLocation: JsonObject Loading for all instances of power chill:tumtum/crafting failed. Removed 4 missing powers from chill:bigm Removed 5 missing powers from origins:enderian Removed 6 missing powers from chill:short_one Removed 20 missing powers from chill:bee Removed 6 missing powers from origins:feline Removed 5 missing powers from origins:elytrian Removed 3 missing powers from chill:frog Removed 7 missing powers from origins:phantom Removed 7 missing powers from origins:blazeborn Removed 4 missing powers from chill:tallboi Removed 5 missing powers from origins:avian Removed 4 missing powers from chill:saga Removed 4 missing powers from chill:death Removed 1 missing powers from chill:possessed Removed 15 missing powers from chill:nightrunner Removed 2 missing powers from chill:support Removed 7 missing powers from chill:icy Removed 5 missing powers from origins:shulk Removed 4 missing powers from chill:theo Removed 11 missing powers from chill:tumtum Removed 7 missing powers from origins:merling Removed 4 missing powers from origins:arachnid Removed 9 missing powers from chill:banan Removed 6 missing powers from chill:ferrari

A power failed to load because it uses a feature that isn't currently implemented in forge.

I assumed was just wondering if there was anything more. Is there a list of supported and unsupported powers/actions/conditions

Thanks

A power failed to load because it uses a feature that isn't currently implemented in forge.

Does that mean that the port is incomplete? I'm confused


Optional custom json replacer/spaces per response

In some cases it is useful to be able to overwrite the json replacer at response level, in the end it is something related to the response.
This change also makes it possible to simply say: res.jsonReplacer = null; res.json(data); to prevent the jsonReplacer from kicking in.

+1. This would be very helpful when escaping output from certain requests, and not from others.


Update dependencies

Codecov Report

Merging #6 (83d2acb) into master (708f2db) will decrease coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   76.56%   76.41%   -0.15%
==========================================
  Files           6        6
  Lines         320      318       -2
==========================================
- Hits          245      243       -2
  Misses         75       75

Flag
Coverage Δ

unittests
76.41% <100.00%> (-0.15%)
:arrow_down:

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

Impacted Files
Coverage Δ

lib/is.lua
100.00% <100.00%> (ø)

metamodule.lua
77.02% <100.00%> (-0.10%)
: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 708f2db...83d2acb. Read the comment docs.


revert_PR5938 which crash all cirque integration test

Summary of Changes:
See the consistent crash when running all cirque integration test, after reverting #5938, issue goes away
for example, in echo test,
/out/debug/linux_x64_clang/chip-echo-requester <IP_ADDRESS>
./out/debug/linux_x64_clang/chip-echo-responder
CHIP:EM: Sending Standalone Ack for MsgId:00000000
Thread 8 "chip-echo-reque" received signal SIGABRT, Aborted.
[Switching to Thread 0x7fffe77fe700 (LWP 1467642)]
__GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
50 ../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
(gdb) bt
#0 __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
#1 0x00007ffff7522537 in __GI_abort () at abort.c:79
#2 0x00005555555e2b64 in chip::Messaging::ExchangeContext::SendMessageImpl(chip::Protocols::Id, unsigned char, chip::System::PacketBufferHandle, chip::BitFlags<chip::Messaging::SendMessageFlags, unsigned short> const&, chip::Transport::PeerConnectionState*)
(this=0x55555563c368 <gExchangeManager+520>, protocolId=..., msgType=16 '\020', msgBuf=..., sendFlags=..., state=0x55555563d310 <(anonymous namespace)::gSessionManager+32>)
at ../../src/messaging/ExchangeContext.cpp:119
#3 0x00005555555e2a53 in chip::Messaging::ExchangeContext::SendMessage(chip::Protocols::Id, unsigned char, chip::System::PacketBufferHandle, chip::BitFlags<chip::Messaging::SendMessageFlags, unsigned short> const&) (this=0x55555563c368 <gExchangeManager+520>, protocolId=..., msgType=16 '\020', msgBuf=..., sendFlags=...) at ../../src/messaging/ExchangeContext.cpp:107
fixes #6032

Thanks, I also noticed that the reverted changes cause the following error while commissioning a Nordic DK using the Python controller:
CHIP:EM: OnMessageReceived failed, err = CHIP Error 4112 (0x00001010): Unsolicited msg with originator bit clear

Looking into the crash. Please hold off on merging this PR.

Blocking this PR for now, we should fix the issue, not pull things out needed for TE2

I've submitted the fix in this PR https://github.com/project-chip/connectedhomeip/pull/6053
I have tested that the IM initiator/responder work with this change.

Thanks. @pan-apple just close this PR


Inconsistency of solid/reactivity rule when dealing with event handlers?

Hello, I have been using solid-js for more than two years and recently decided to use this tool.
I event recently made a small patch regarding one of the issues: https://github.com/solidjs-community/eslint-plugin-solid/pull/164
However, in my process of fixing linting issues I kept finding some cases that are inconsistent and IMHO could be handled differently.
I made a list in the playground to identify all cases: https://playground.solidjs.com/anonymous/29d817a1-ae6e-419e-af52-01ccc5287064

Besides the issues from my patch, I found:

These two should definitely trigger the solid/reactivity rule

These are the tricky ones. I understand function attributes of Components are expected to be synchronous, as they might access signals/stores, and be called arbitrarily inside the component.
However, at the same time, I believe it is not a stretch to expect attributes named like  on[A-Z][a-z_$]* to also be used as event listeners, even if passed to Components. In most cases, these would actually called as subroutines of event listeners of native elements....
Right now, there are already special rules such as static attribute prefixes. Wouldn't be valid to also consider Components' function attributes which name looks like on[A-Z][a-z_$]* to also be especial cases?

@pedro00dk Thanks again for your last patch and for doing the investigation here! These are some good points—I do think some regexes need to be adjusted to be as permissive as possible.

I'm with you that the async functions passed as props to <Button> should not warn.
For<button onclick={props.onClick} />, // WRONG - complain only about attribute casing

I think one warning is enough. Though onclick is certainly intended to be an event handler, it could just as easily be something like only or once, hence the existing rule guiding the user to make their intention explicit. See #23
For<button on-click={props.onClick} />, // WRONG - handler is only attached once (even if custom event)

I didn't even know the dash (-) syntax was legal! I'm happy for this to be treated like other event handlers.

I'd welcome a PR for these, but no obligation. I can take it as time allows if you don't get to it first. Thanks!


missing username in log file

Hi all,
I try alerta (with oauth2 + gitlab authentication config), but I don't find username when I DELETE a event.
This log:
==> /tmp/alertad.log <==
<IP_ADDRESS> - - [08/Nov/2016 10:49:51] "DELETE /alert/8100e41a-a91b-4056-8113-f414ae567d70 HTTP/1.0" 200 -

==> /var/log/nginx/access.log <==
<IP_ADDRESS> - - [08/Nov/2016:10:49:51 -0200] "DELETE /api/alert/8100e41a-a91b-4056-8113-f414ae567d70 HTTP/1.1" 200 21 "http://alerta-web.qa.intra/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.87 Safari/537.36" "-"

Nginx.conf logformat:
    log_format  main  '$remote_addr - $remote_user [$time_local] "$request" '
                      '$status $body_bytes_sent "$http_referer" '
                      '"$http_user_agent" "$http_x_forwarded_for"';

I need to log username when someone DELETE an event. How to I configure this?

To get the username you would need to log the Authorization header and decode the JWT bearer token to get the username.

You can get jwt decoders for any language, including Lua. So you might be able to decode the jwt and write the username to the log using an nginx Lua script.


cmake issue.

platform Windows 10 17763 x64
CMake 3.15.3
Visual studio 2015
Although source include binary. but it's still Win32 Platform.
SuiteSparse has been comilped.
I have no idea how to config CMake to build this project.
CMAKE_INSTALL_PREFIX SuiteSparse_AMD_DIR has been set. but it occured a lot of error.
Start to FindSuiteSparse.cmake :
find_library will search inside lib64 directory (can be changed with SuiteSparse_SEARCH_LIB_POSTFIX)
On windows, we absolutely need SuiteSparse_config.h every time for all projects : add suitesparseconfig component to look for
CMake Warning at cmake/FindSuiteSparse.cmake:171 (message):
Failed to find AMD :
SuiteSparse_AMD_INCLUDE_DIR not found.
Check if you wrote the component name correctly (case sensitive),
or set the SuiteSparse_AMD_DIR to look inside
Call Stack (most recent call first):
cmake/FindSuiteSparse.cmake:295 (SuiteSparse_FIND_COMPONENTS)
CMakeLists.txt:23 (find_package)
...

on Windows platform. this project has included in suitesparse-metis-for-windows.


[ Bug ] Modules update pop up Java errors

I did update new modules because my friend told me that in-game now can pop up xp and lvl and gold leaderboard. But since I've updated it, in-game doesn't work at all. I received like bunch of java processes pop up windows and in the engine or command line just showing that cannot read summoner's name and also showing that summoner name doesn't exist at all. Pickban works perfectly fine but as soon as my game load up everything crashes and pop ups jump up and everything freeze. Immediately League tool cannot connect to the server. So I don't know where the mistake is. As soon as I get to the computer I will try update java.
I think that I updated in-game and static or prm.

The toolkit does not use any Java. If you are seeing Java errors, they are unrelated to the toolkit.
If you receive errors in the output of the prod-toolkit please post them here so we can understand what is going on.
If the toolkit window closes, you can start the toolkit in Powershell (Shift+Right Click in the toolkit folder and type npm start) to see the output before the crash.

So do you suggested update Java or Node?

Feel free to update Java, as said above the toolkit is entirely unrelated to Java. Do not update Node, we build and test on a specific version (16.7)

I did not update the Node. I was updating the static and ingame with new module. I will send the screenshots as soon as I'll be able.

So this is picture from friday.

Can you share your LiveEvents.ini configuration file please?

Where can i find it?

Also i recieve this error - as last time. I don't know what to do.

The LiveEvents config is in your game folder, see the observer tool readme.
Can you give more details about the crash above? It seems to occur when a game is starting.

What type of game is it?
Is the observer tool running on a spectator or player?
How many players are in the game / are there any bots in the game?

Custom game. Observer tool is running on a spectator and there are 10 players in the game and no bots.

LiveEvents.txt

Issue is still persisting. I don't know what to do. Same issue as last time, i redownload it, fresh install but nothing working. Please help.

Can you try removing these lines from the LiveEvents config and check if it helps?
OnMinionKill
OnNeutralMinionKill
OnDamageGiven

These events are not used by the tool and might cause unexpected behaviour.

I will try it at Monday and let you know - thanks for now :)

Okay, i found the workaround - before going to match, if i got a error i restart the toolkit. It helped me in every case. Then just hit the reload in OBS and everything working just fine.


Steam TOTP possible?

Hi, Steam uses his own variant of a TOTP, a 5 digit code with letters in it. Does solo v2 support this? If yes, can you add it to the advanced section?

From my understanding Steam TOTP is just a standard HMAC-SHA-1 based TOTP, so it should already be supported.
The issue is that Steam doesn't expose the actual TOTP secret to the user. There seem to be some API wrappers that allow you to recover it, but I haven't used them and would consider such a wrapper here (and as such adding a whole Steam login flow) to be quite far outside of the scope of this app.

Well, I tried this and it does not work. There are methods to get the key. I added it into Bitwarden and it works fine.
Of course the whole login flow is oversized. Yubikey works fine too.

Did you set the number of digits to 5 in the advanced section when adding the TOTP secret? As far as I can tell this should be the only difference.

Here is a screenshot of the advanced section:

As an example I got this TOTP: 12123 Bitdefender shows this (both have the same secret, I must add a steam// in Bitdefender so they know that is steam): AB1AB. The chars and numbers are random, for the Steam Bitwarden TOTP can be 123AB too and so on.

I'll have to look more into how e.g. Bitwarden implements this and probably need to export a secret myself...


extra attributes

is there an attribute we can use to enable ray tracing, like how we use -s to specify the starting frame, is there any attribute we can use to decide whether ray tracing should be enabled or not?

Hi @JittoJoseph, you cannot exactly add attributes in the blender command. You would need to add the code in the Python code in the data variable.

If i understand correctly what he means by enabling ray tracing, @JittoJoseph wants to switch the render engine between evee/cycles, which, according to blender docs can be done with the -E or --engine parameter

Eevee doesn't work correctly in Google Colab


Update GHA configuration

As title.
Generated by Cimas.

@CAMOBAP problem here is all tests failing. Not because of token access.


Bug fix 3.6/round costs to remove costs

Same as:  https://github.com/arangodb/arangodb/pull/10638
Info: There is also an EE branch.

http://jenkins01:8080/view/PR/job/arangodb-matrix-pr/7661/


Updated Project Portfolio Page for Sam Xie

resolves #105

Pull Request Test Coverage Report for Build 189

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.973%

Totals

Change from base Build 188:
0.0%

Covered Lines:
2076

Relevant Lines:
2282

💛  - Coveralls


Matrix do not display properly when 2x2

Using marked and marked-katex-extension in a React application
Input:
$\mathbf{P} = \begin{pmatrix} 3 & 1 \ 2 & 4  \end{pmatrix} $
Output:

The bracket does not cover the whole matrix. What might be the cause of it?

This repo is only for the marked extension that uses katex for rendering.
You should ask this question on the katex repo.


Chain python.withPackages calls

Issue description
I've got 2 scenarii where I would like the user to chain python3.withPackages(a).withPackages(b), i.e.. append to the list of packages passed to withPackages.

for neovim, I would like the user to be able to add new packages via neovim.overrideAttrs(oa: { python3Env = oa.python3Env.withPackages(p: p.newPkg); }
in jupyter, I would also like to be able to autoadd the ipykernel module.

I dont think it's possible currently ?
Solutions I could think of:

convert for instance the neovim wrapper's python3Env parameter python3PackagesFn of the form ps: [ ps.numpy ] etc. It's then possible to add packages to the list after the fact python3PackagesFn2 = ps: python3PackagesFn ps ++ [ ps.pandas ] for instance. Problem is we might lose the ability to set the python3 interpreter.
Might be convoluted but why not leverage structuredAttrs to expose the function p: [p.numpy] in withPackages definition such that one can do what I described in solution 1:

withPackages = stdenv.mkDerivation (
__structuredAttrs = true;
pkgsFn =
);

pyEnv = python3.withPackages (p: [ p.numpy ]);
# I can now extend it
pyEnv2 = pyEnv.overrideAttrs(oa: {
   pkgsFn = ps:  [ ps.pandas ] ++ oa.pkgsFn;
});

@FRidh as python's maintainer.

neevrmind, it's already doable plus eevn with structured attrs, I dont think one can serialize functions.

I had a false eureka moment but this seems to work
with import ./. {};
let
  pyEnv = python3.withPackages(ps: [ ps.scipy ]);

  # extraLibs
  # https://ryantm.github.io/nixpkgs/using/overrides/
  pyEnv4 = pyEnv.override(prev: {
    # fastai
    extraLibs = prev.extraLibs ++ [ pyEnv.python.pkgs.accupy ];
  });
in
  pyEnv4


Add an extra text input to file input

I think it would be useful to have the possibility to add an extra text input to all entry in a multiple ImageInput (or FileInput).
I tried to write a custom ImageField by myself with a extra text input but i don't know how to pass these extra values to the form data to send on the server.
This is the code of the custom ImageField i wrote (not working), maybe would be useful:
import React from 'react';
import PropTypes from 'prop-types';
import get from 'lodash.get';
import TextField from 'material-ui/TextField';
import {FieldTitle} from "admin-on-rest";
const styles = {
    image: {
        margin: '5px',
        height: '100px',
        width: '100px',
        objectFit : 'cover',
        clear: 'both'
    },
    textInputWrap: {
        display : 'flex',
        flexDirection: 'column',
        alignSelf : 'center',
        margin: '10px'
    },
    textInputLabel: {
        color: 'rgba(0, 0, 0, 0.3)',
        alignSelf : 'flex-start'
    },
    textInput: {
        alignSelf : 'flex-end'
    }
};

export const ImageImportField = ({ elStyle = {}, record, source, src, title }) => {
    const sourceValue = get(record, source);
    if (!sourceValue) {
        return
;
    }
    const titleValue = get(record, title) || title;
    const fileHeader = record.rawFile;
    return (

                {fileHeader.name}
                <TextField
                    type="text"
                    name="filename"
                    defaultValue={fileHeader.name}
                    floatingLabelText={<FieldTitle label="Filename" />}
                    style={styles.textInput}
                />

    );
};

ImageImportField.propTypes = {
    elStyle: PropTypes.object,
    record: PropTypes.object,
    source: PropTypes.string.isRequired,
    nameSource: PropTypes.string.isRequired,
    title: PropTypes.string,
};

export default ImageImportField;

Hi, and thanks for your question. As explained in the admin-on-rest contributing guide, the right place to ask a "How To" question, get usage advice, or troubleshoot your own code, is StackOverFlow.
This makes your question easy to find by the core team, and the developer community. Unlike Github, StackOverFlow has great SEO, gamification, voting, and reputation. That's why we chose it, and decided to keep GitHub issues only for bugs and feature requests.
So I'm closing this issue, and inviting you to ask your question at:
http://stackoverflow.com/questions/tagged/admin-on-rest
And once you get a response, please continue to hang out on the admin-on-rest channel in StackOverflow. That way, you can help newcomers and share your expertise!

Hi, I'm not asking for a "how to", i'm asking for a new feature if possible. For now it's impossible to achive this scenario, so i think it's useless to ask for that on stackoverflow.

It's possible, you need to write a custom input. See the documentation


Communicate double / double array parameters with type info, explicitly cast when set from integer

Public-Facing Changes
Communicate double / double array parameters with type info, explicitly cast when set from integer
Description
Implements https://github.com/foxglove/ws-protocol/pull/519

Communicates double or double-array parameters with "type": "float64" or "type": "float64_array", respectively.
When setting parameters, casts integers to double when one of the above mentioned type info is given

In reviewing this PR I realized I should have read the spec changes more clearly. I have some questions here: https://github.com/foxglove/ws-protocol/pull/519#issuecomment-1674154969
In what direction do we need to communicate this type information? Is this from server -> client (studio)?


test

if bug count == 0
what will display in the main.php page... should output there are no bugs in the db

will display as normal with 0 columns and rows.


Styles and Treesitter integration

Hello,
Since the font I use is not well designed for italics I wanted to disable all styles, but it doesn't apply everywhere:

As you can see, "require" and "false" here are still in italic.
So far I detected via TSPlayground that this is the case for at least those elements:

use_declaration
identifier
integer litteral
true/false

In fact, it's the same thing with other themes. I probably did (or didn't do) something in the treesitter config. Sorry about that.


Add a new function to specify image size. Also some tests are added to t...

Add a function to specify image size. For example,
![image](image.png =100x200)

will be interpreted as
<img src="image.png" alt="image" width="100" height="200">

I also added some tests for this function, and confirmed the tests are passed on Travis CI.
Thanks.

@tatsy , it's better to make it as plugin, because core should follow commonmark spec. markdown-it is specially designed to allow such modifications without upstream change.

@puzrin, thank you for your reply. I understood the markdown-it design following commonmark. I'll re-implement the function as a plugin.
What I'm concerned is a code redundancy because the plugin for the size-specified image markup will be almost the same as lib/rules_inline/image.js. Could you give me a piece of advice to prevent the redundancy?

https://github.com/markdown-it/markdown-it/blob/master/lib/index.js#L197-L211
The most critical helpers are already pinned to each instance. Copy the rest, it will not increase total size too much (minified & gzipped)

Or you can just modify renderer - test title attr, extract size if exists and add to attributes. That will be much more simple than parser change.
IMHO, renderer patch should not cause conflicts.

Ah, sorry, last advice was wrong, title require quoting. That will work only with this notation: ![image](image.png "=100x200")

Oh, I missed the exports of helpers that you mentioned. It was exactly what I wanted to know. Thank you for your advice.

I'm sorry for rapid-fire questions. Is there any reason why lib/helpers/normailze_reference.js is not exported in lib/helpers/index.js ?

No prob. Ask as much as you need. Real devs are always welcome.
That file was not exported, because i was not sure, should it be in helpers or in utils. Forgot to care about it.
Created #42. Copy it temporary, i'll try to do something in next releases.

I checked the created issue. Thank you for your help.
And now, I released a plugin markdown-it-imsize at NPM.
So, please close this PR if everything is fine. Thank you.

you need to refer md from wrapper, instead of require markdown-it directly.
https://github.com/markdown-it/markdown-it-footnote/blob/master/index.js#L43

Thank you for your advice!
I revised my code so that md from wrapper is used in the other functions.
https://github.com/tatsy/markdown-it-imsize/blob/master/index.js#L207-L208

Looks correct.

Moved normalizeReference to utils. Will release soon with other minor fixes.


Enhance contract to support multiple input Tensors

This PR enhances the contract function for multiple tensors, which resolves issue #16 in EinExprs. The function utilizes reduce to perform pairwise Tensor contractions.
To ensure the robustness of these changes we added a testset that covers the new contract functionality, checking its behavior with various tensors in inputs.

The problem is that EinExprs does not return only pairwise Tensors to be contracted, and that leads to error. You can check the issue I mentioned above. I think that this extension of the contract function simply fixed those problems.

The problem is that EinExprs does not return only pairwise Tensors to be contracted, and that leads to error. You can check the issue I mentioned above. I think that this extension of the contract function simply fixed those problems.

Yep, but a optimization pass can be written such that
EinExpr(a,b,c,out=labels)

gets transformed to
EinExpr(EinExpr(a,b,out=labels_ab),c,out=labels)

Yep, but a optimization pass can be written such that
EinExpr(a,b,c,out=labels)

gets transformed to
EinExpr(EinExpr(a,b,out=labels_ab),c,out=labels)

@jofrevalles Actually, we don't need to write any other optimization pass: this should be currently done by just calling einexpr(EinExpr([a,b,c], out=labels), optimizer=Exhaustive()).

@jofrevalles Actually, we don't need to write any other optimization pass: this should be currently done by just calling einexpr(EinExpr([a,b,c], out=labels), optimizer=Exhaustive()).
I mean, just calling einexpr on the EinExpr that you want to "optimize".

It should, but this doesn't work for unconnected tensors, since einexpr does not return them two-by-two:
julia> tensor_vector = [
                  Tensor(rand(2, 2), (:i, :j)),
                  Tensor(rand(2, 2), (:k, :l)),
                  Tensor(rand(2, 2), (:m, :n))]
3-element Vector{Tensor{Float64, 2, Matrix{Float64}}}:
...

julia> einexpr(Exhaustive, EinExpr(tensor_vector)) |> contract
ERROR: MethodError: no method matching contract(::Tensor{Float64, 2, Matrix{Float64}}, ::Tensor{Float64, 2, Matrix{Float64}}, ::Tensor{Float64, 2, Matrix{Float64}}; dims::Vector{Symbol})

Closest candidates are:
  contract(::Tensor, ::Tensor; dims)
   @ Tensors ~/.julia/packages/Tensors/O9RAx/src/Numerics.jl:36
  contract(::Tensor; dims)
   @ Tensors ~/.julia/packages/Tensors/O9RAx/src/Numerics.jl:49
  contract(::Tensor, ::TensorNetwork; kwargs...)
   @ Tenet ~/git/Tenet.jl/src/TensorNetwork.jl:480
  ...

That's why I think that in this case it would be useful to extend contract.

Okay, I'm gonna approve this but keep in mind that this is subject to changes in the future.


Single page/ Multi page download options not working

Everytime I try to download pdf I am having this issue "The cloud function is running into some trouble, please try again later or use the browser print feature." And in console it shows CORS issue

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!


JaxbUnknownAdapter: return empty ArrayList when unmarshalling JaxbListWrapper without elements

In current implementation if there is unmarshalled JaxbListWrapper with null elements (happens when marshalling empty List) then Arrays.asList throws NPE, which upper layers process as null element. This fix assures that JaxbListWrapper is unmarshalled to List instance even with empty elements.

Can one of the admins verify this patch?

Would it be possible to add a new test as well?

Added tests covering changed functionality.

ok to test

@mariofusco could you please review this and possible merge?

@psiroky I'll look at this ASAP.

Sorry for the delay!!
Merged @ https://github.com/droolsjbpm/drools/commit/9a277f1c4a7386fdc6d1e76f00562900fb7f9384


access to hrtime

Hi
Via the k6 project, the following line fails
const now = process.hrtime.bigint();
ReferenceError: process is not defined\n\tat file:///tmp/socket-test.js:63:19(93)\n\tat github.com/loadimpact/k6/js/common.Bind.func1 (native)\n\tat _default (file:///tmp/socket-test.js:42:33(50))\n" executor=ramping-vus scenario=default source=stacktrace
What's the recommended way of accessing nanosecond time via goja?

process is not a part of ECMAScript standard, therefore goja does not support it. If you were working with goja directly there would be a way, however you are not, so this question is best directed to the k6 dev team.
Also note, goja does not support BigInt.


App roadmap

@o-sdn-o mentioned that he was planning to make vtm come with only few built-in widgets (apps) while others would be standalone and could be compiled separately. I am making list of what apps should be built-in by other DE "standards" (There is no standard but this is so far most popular configuration)

This list is unofficial, meaning that it doesn't mean that these "apps" will definitely be implemented and based on developer's response I will update list

[x] Terminal Emulator
[ ] File Manager and Browser
[ ] Web Browser
[ ] Settings
[x] Window Manager / Tiles
[ ] App/Widget Manager w/ "Installer"
[ ] Notifications (like nvim-notify)
[ ] Download Manager [OPTIONAL]
[ ] Archive Extractor and Archiver [OPTIONAL]
[ ] "Registry Editor"
[ ] Office Applications like Spreadsheet, Wordpad, Slides and more [OPTIONAL]

Suggestions from @AutumnMeowMeow:

[ ] Color theme editor [OPTIONAL]
[ ] Connection manager (if supporting shared sessions - maybe already here?) [OPTIONAL]
[ ] Character/emoji map/browser/picker (in another words UnicodePad) [OPTIONAL]
[ ] Built-in simple text editor (already here?) - for sticky notes and quick text files [OPTIONAL]
[ ] Screen shapshot tool (Screenshoter or Screen Capturer) [OPTIONAL]

Suggestions from @tomek-szczesny:

[ ] Image viewer
[ ] Volume Control System [OPTIONAL]
[ ] Audio Player [OPTIONAL]

What's the distinction between "Settings" and "Registry Editor"?
Some other things -- everything optional of course:

[ ] Color theme editor
[ ] Connection manager (if supporting shared sessions - maybe already here?)
[ ] Character/emoji map / browser / picker
[ ] Built-in simple text editor (already here?) - for sticky notes and quick text files
[ ] Screen shapshot tool

There is also such a list of planned applications in the imaginary Desktopio store on the demo server 🙂

This is all fictional information for fun.

What's the distinction between "Settings" and "Registry Editor"?
Some other things -- everything optional of course:

[ ]  Color theme editor
[ ]  Connection manager (if supporting shared sessions - maybe already here?)
[ ]  Character/emoji map / browser / picker
[ ]  Built-in simple text editor (already here?) - for sticky notes and quick text files
[ ]  Screen shapshot tool

In my idea settings allow graphically modify some "public"/"high-level" settings (window style, keybinds and wallpaper) while "registry" allows user to modify "low-level" settings.

The web browser would be really tricky to implement. browsh does a fantastic job at rendering websites in text mode (using firefox as its backend), but the usage is super awkward and I never bothered learning their keyboard shortcuts. Something like that, but with tabs and address bar like all the modern browsers do would be fantastic. But I know I'm asking for a lot here. browsh also has notorious problems with killing firefox after it exits.
vtm settings applet is of course welcome, and I'd keep it unified rather than broken down into many "apps". It could have a "advanced" checkbox for hiding advanced settings, but preferably everything should be available in one place. Color theme editor should also be accessible from within settings app. Don't you just hate when settings are scattered randomly in multiple applets?

Screen shapshot tool

That would be awesome, especially if a user could select a snapshot region with a mouse, like "snipping tool". I'm not sure what is the anticipated output though - raw text, text with VT100 gibberish, or a bash script that ensures proper display of the screenshot?
I'd add to the list:

Image viewer (like classic IrfanView)
Volume control (could be based on alsamixer / pulsemixer)
Music player (a simple wrapper for mpv, maybe? I use mpv in No-X environment, it handles ALSA without pulseaudio very well)

I am interested in making some of the applications in the checklist, mosly interested in the filemanager. I am also thinking about the application 'store'/manager.
I would like to add a panel with stuff like battery, time, (maybe wifi), username, etc. I am thinking about making that first (by just echoing info to the bottom line of the screen (I know, there are better ways, like implementing it directly.))
The reason I have not started yet is because of scripting. It would be nice to start a window from a python script (using os.system() is an option).
I will update here if i made something, and link the repo. (I think i will give a link to a first version of a panel this weekend)

Following my last post:
Some settings for default apps in the config would be nice, so the filemanager knows which app to use for which type of file. This is where the option to script opening windows comes in: the filemanager can then open a new window with the correct application running

The reason I have not started yet is because of scripting. It would be nice to start a window from a python script (using os.system() is an option).

You are absolutely right.
Widgets and applications for vtm can be created already now, but in C ++, and this is very laborious. Without integration with scripting languages, this would be a dead end.
I'm currently making up integration with scripting languages to make it possible to create vtm applications/components without using C++ at all. The essence is this: the application will consist of a UI layout and scripts attached to UI elements and events. The entire DOM of the application + its current configuration will be available from the scripting run space. The UI layout of the interface will be in XML format (slightly modified) with scripts embedded. It should be like WPF where XAML+CLR. Or like a web frontend - HTML+JavaScript.
All this will be possible after the following tasks are ready

Integration with scripting run space. See #393

Done: Start/stop external scripting engine.

Done: Handle output generated by the scripting engine.
Forwarding user input to the scripting engine.
Access to DOM and events from scripting runspace.

Keyboard events support in the vtm environment. Currently, there are no keyboard events at all, and all keyboard stream is just forwarded to focused windows as is. See #86.
DOM from XML runtime generator.

It would be nice to start a window from a python script (using os.system() is an option)

From the moment when internal integration with scripting languages is completed, it will be possible to interact with the environment from outside, i.e. interact with vtm environment objects directly from python3, including creating/closing windows, etc.

I am interested in making some of the applications in the checklist, mostly interested in the filemanager

I forgot to say that the DirectVT protocol has not yet been completed to create full-fledged applications in the sense of supporting output by an application in several windows. The application should be able to display multiple windows, for example, display modal windows, show window/context menu popups. At the moment, the application can only display its bitmap in one window.

but I cant test on windows (no windows machines : D )

No problem, I have the capacity to run/debug on all required operating systems.

I'm also interested in a file manager. Existing TUI file managers lose to GUI file managers. This is at least by the fact that they are locked in one window. They also do not support the Unicode standard well, in terms of grapheme clusters and character widths. This is a consequence of poor terminal support for Unicode. Terminal developers don't want to understand that the application, not the terminal, should determine the character width. The terminal can only assume a width if the application doesn't specify one. In addition, terminal developers are reluctant to accept half wide characters. Not to mention ligature fragments.

Yes, I am very interested in a console-file manager looking like a graphical file manager (for example dolphin, nautilus, windows exporer.). I am learning to use python3-prompt-toolkit to make apps that would run in the vtm build-in terminal, but makes pop-up windows (also the vtm build-in terminal with a seperate instance of the app running) that would report answers back via (?) using the scipting integration. But your idea about some XML languige sounds more interesting, and I would drop prompt_toolkit and learn it instead when it exists.
(I am currently smiling because of the idea making applications specificaly for vtm, and would scream happiness if I wasn't in a public space right now...)

I have started testing with an info panel, but it only supports linux xterm based terminals, and only a small group of them... also it glitches out how vtm draws colors (it resets the current escape code all the time, making the background of a vtm terminal disappear). Integration into vtm itself would be nice :). (It is not yet public)

I am learning to use python3-prompt-toolkit to make apps that would run in the vtm build-in terminal, but makes pop-up windows (also the vtm build-in terminal with a seperate instance of the app running) that would report answers back via (?) using the scipting integration.

python-prompt-toolkit is very much in the spirit of what I'm currently implementing. In vtm it will be possible to use not only python.
Among other things, everything done with python-prompt-toolkit should work smoothly without any issues in the built-in vtm terminal.

I can also make the store, and maybe host a main repo list on my raspberrypi. (like my already existing packager, vpm written in bash.) It would download a list of packages, with each package entry containing: name|description|git repo|build command or maybe even a a packaging system with package files containing pre-build binaries, like apt, dnf, pacman, etc. I could host the list of download urls for packages on the pi (replace git repo|build command with package url). If you would rather host the packaging list yourself, or use another host for it, I could integrate with that.

It is still difficult for me to answer in what format the applications will be distributed. Perhaps this will be SSH demo servers evenly distributed throughout the planet (Australia, Japan, EU, US West/East coast) to reduce round trip ping. By logging in to them via the password-free SSH, users will be able to play with the application in real time and, if they like the application, download it. I experimented with this approach and it turned out that vtm works very responsively via SSH from almost anywhere in the world.

Perhaps this will be SSH demo servers evenly distributed throughout the planet (Australia, Japan, EU, US West/East coast) to reduce round trip ping. By logging in to them via the password-free SSH, users will be able to play with the application in real time and, if they like the application, download it.

Thats an interesting! So how would users know to what ssh server to connect to? Are all apps on the same server? Is there a previeuw button in a sofware center that connects via ssh in a vtm terminal (Maybe the ip is in the packagelist entry)?
I guess it would be possible to make a package format, and download the packages via the same ssh server. I know you can pass a command to run after connect.
Also, I am already using vtm to manage my server via ssh! : D

Thats an interesting! So how would users know to what ssh server to connect to? Are all apps on the same server? Is there a previeuw button in a sofware center that connects via ssh in a vtm terminal (Maybe the ip is in the packagelist entry)?

A year ago I ran five servers evenly around the world (Sydney, Tokyo, Frankfurt, Richmond, Seattle).
On all five servers, a demo version of the vtm desktop was installed, available when logging in via SSH without a password.
In Frankfurt and Seattle, DNS servers were configured to respond with the ip address closest to the user when resolving the domain address of the SSH server. The servers use a GeoIP database (map IP to geographic coordinates and get the distance between addresses).
The user simply typed ssh<EMAIL_ADDRESS>in the terminal, the name netxs.online was resolved to the IP address closest to the user host, so the user logged in on the nearest demo-server where vtm was automatically launched as a shell by default. Dozens of users could connect to the same server at the same time and they saw each other's mouse cursors there and could move windows to each other.

I think the connection to the server with the app store should be clickable at the bottom right of the vtm desktop. This is where the vtm version number label is located.

Clicking the preview button in the store should open an additional SSH session with a demo of this app.

I will restore this server infrastructure back when the apps and store are ready.

Someone posted a link to the demo on https://news.ycombinator.com/item?id=24243521 and it was fun.

I think the connection to the server with the app store should be clickable at the bottom right of the vtm desktop.

On the desktop? At all time, or only when the store is open?
Also, what do you think of the idea to make a new package format and mirror them to all servers, downloadable via the ssh session automaticaly?
This could work like the shellcommand
ssh<EMAIL_ADDRESS>"cat /path/to/package.zip" >> package.zip

Or am I thinking too unix right now?
Or rather an apt-like reposystem like I mentioned earlier, where the store would download a list of packages, with each entry like this:
pkg1-name|example<EMAIL_ADDRESS>
or maybe use the download via ssh in that.
Users could then add third-party repos to that, like on apt (or any linux pkgmanager).

the server with the app store
Do I understand from this the app store runs on the ssh server, not the local vtm instance?

Do I understand from this the app store runs on the ssh server, not the local vtm instance?

Exactly.

Also, what do you think of the idea to make a new package format and mirror them to all servers, downloadable via the ssh session automatically?

A new package format is definitely needed. I have not yet thought about it and how distributing and updating packages. There may be all sorts of dependencies and version incompatibilities.
Downloading can be done via SSH or an async DirectVT binary stream.

Or am I thinking too unix right now?

ssh<EMAIL_ADDRESS>"cat /path/to/package.zip" >> package.zip

I absolutely don't mind using external operating system components like ssh or wget instead of integrating them into my code. In particular, if I need to connect via SSH, then I just create a process to run the ssh command and pass all the connection parameters to them :D.

What if the button to open the connection to the store would be an entry in the app list?
Also, how would the ssh server notify the local vtm what package to download?

There may be all sorts of dependencies and version incompatibilities.
Ah, I did not think of that yet... maybe the stlre can talk with the most popular package managers when on a unix system. (Windows does not have a packager, right?)

What if the button to open the connection to the store would be an entry in the app list?

Good idea. We will make this decision later, when a store prototype is somehow formed. There we can play in real time and choose the appropriate option.

Also, how would the ssh server notify the local vtm what package to download?

When the user clicks on the Download button, the Store running on the remote SSH server initiates the file transfer (either via an OSC sequence or via the DirectVT protocol). Next, a modal window (with file/folder picker) should automatically open with the option of where to save or run to automatically install.

Windows does not have a packager, right?

I'm no expert on the Windows ecosystem, but I think we'll figure it out as we get to this point.

the Store running on the remote SSH server initiates the file transfer (either via an OSC sequence or via the DirectVT protocol).

Ah, that goes above my knowledge. (I just learned to use tcp sockets in python like a month ago). But I guess that could work. It sounds good.

where to save

I guess this means the filemanager should learn to open packages with the local package installer then.

I guess this means the filemanager should learn to open packages with the local package installer then.

The file picker modal window is a light version of the file manager, a sort of sub module.

So the filemanager should be build-in like the terminal, so that the file picker can be a modal window?

Perhaps not necessarily. A full-fledged file manager is too heavy.

Would folder bookmarks sync between the dialog an the filemanager? that means vtm should have a bookmark system, or try to read filemanagerbookmark if possible.

Good point. It looks like there are a lot of things that need to be in sync with the environment. Keyboard shortcuts, folder bookmarks/quick access links, anything else.

something like https://github.com/fathyb/carbonyl be a possibility? it would be very nice to render anything (breaking or not) to any of terminals.

Using character matrices as demonstrated in the https://github.com/microsoft/terminal/issues/4345#issuecomment-2238952371 it will be possible to implement even cooler web rendering in terminals. I think it is quite possible to do (but it is not the main priority).

Or https://github.com/Maximus5/ConEmu/blob/master/src/ConEmu/Attach.cpp can do it by rendering to attached terminal. Is possible for implement to VTM future? Wanting for VTM be replacement desktop!

Re-adopting a console application by another terminal host is theoretically possible in vtm. I haven't delved into this yet because vtm is cross-platform and the main task for now is to make it operate in standard mode on common system platforms. This is planned.

Wanting for VTM be replacement desktop!

I'm all for it!

but maybe if VTM has built-in stock variants specific for its amazing platform!

Thanks for your feedback! At the moment, vtm has only a built-in terminal emulator. It is impossible to make the application set for the vtm environment until its internal architecture is unstable. Any changes to the internal architecture will inevitably break these applications and, accordingly, they will need to be updated and fixed. It takes some time to stabilize.(Edit

built-in stock variants specific for its amazing platform!
They won't be built-in, but I do plan on making a filemanager and maybe more for vtm as soon as I can get any information about the API (to me it's still very unclear on how to 'attach' to the vtm instance and render things, and as @o-sdn-o said, they would break every time the API changed if I managed to figure out the current api).

I see something like Total Manager as a result of crossing the functionality of Far ManagerxTotal Commander, so that it works in both TUI and GUI environments on any platform (Win32, Unix) 🙂.

I was personally thinking about something closer to KDE's dolphin, including split and tabs.

In general, all possible kinds of file managers are needed. It is very likely that these will be ported/adapted versions of existing file managers.


Feature Request: Easy to See Likes and Dislikes

Would it be possible to see if you already "thumbed up" a song by clicking on the menu bar button (to get to the menu, shown in the attached screenshot) via a check mark icon or a thumbs up icon next to the "Thumbs Up" option or via a separate entry within the menu (that would say, for example, "Thumbed Up")? This would be so much easier than having to open a new window by clicking on "Show" every time just to see if you liked a song or not, then to close the window.

OS:
Issue Descriptions:
Steps to Reproduce:

https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/pull/3010

Hi, If this is still open I'd like to pick it up.

@donth77 already implemented it, just waiting for merge.


Setup Java Flink runner PreCommit

Split Flink 1.13-1.16 tests from main Java PreCommit

Please add a meaningful description for your change here
Currently, Flink tests are exercised 5 times on Java PreCommit test suite. Some tests are known to be flaky (e.g. #21333). By only run on the lowest and highest supported flink version, it would reduce the flakiness by (1-x)^3 - (1-x)^5 where x is the flaky probability of a single flink test
Other flink versions are still exercised on the new GitHub Action test suite.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Example workflow run: https://github.com/Abacn/beam/actions/runs/5685099288/job/15409247648

Workflow run successfully on fork: https://github.com/Abacn/beam/actions/runs/5694030121/job/15434371968
PTAL

There are fixes on PreCommits yml ongoing (dispatch, trigger, etc), will integrate the fixed in this workflow also.

that's strange, same command (checkStyle) passed locally and also this workflow passed before

Anyways, now disabled spotless and checkStyle check. We actually should do this  because these are split from the main Java PreCommit, where those checks are also disabled. There is a Spotless PreCommit dedicated to run spotless and checkStyle check


chore(orchestrator): add new code owner

@kadel Could you please invite @batzionb? Thanks!

/verify-owners

/verify-owners


when EXTERNAL-IP too long

Kubernetes version (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.11", GitCommit:"637c7e288581ee40ab4ca210618a89a555b6e7e9", GitTreeState:"clean", BuildDate:"2018-11-26T14:38:32Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.6-eks-d69f1b", GitCommit:"d69f1bf3669bf00b7f4a758e978e0e7a1e3a68f7", GitTreeState:"clean", BuildDate:"2019-02-28T20:26:10Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

Environment:
Darwin PauldeMacBook-Pro.local 18.5.0 Darwin Kernel Version 18.5.0: Mon Mar 11 20:40:32 PDT 2019; root:xnu-4903.251.3~3/RELEASE_X86_64 x86_64

Install tools: zsh

What happened:
when use AKS in AWS, the ip too long ,so can not be found the ip
EX:
NAME           TYPE           CLUSTER-IP       EXTERNAL-IP        PORT(S)          AGE
redis-master     LoadBalancer   <IP_ADDRESS>     a98068ce56515...   3000:31686/TCP   36m

/remove-lifecycle rotten
/sig cli
/area kubectl
/kind bug
/priority P2

/remove-lifecycle rotten

@naihuayang Can you provide more info to reproduce?  Do you have a yaml file you used to create redis-master?  Are you using IPv6?
I created an AKS cluster and the deployment using the aks quickstart and did not observe the problem like you did:
$ k get services
NAME               TYPE           CLUSTER-IP     EXTERNAL-IP      PORT(S)        AGE
azure-vote-back    ClusterIP      <IP_ADDRESS>      <none>           6379/TCP       14m
azure-vote-front   LoadBalancer   <IP_ADDRESS>   <IP_ADDRESS>   80:31875/TCP   8m12s
kubernetes         ClusterIP      <IP_ADDRESS>       <none>           443/TCP        25m

Without being able to reproduce it, I just happened to find where it truncates the external IP to 13 bytes + "..." when it is longer than 16 bytes.
https://github.com/kubernetes/kubernetes/blob/ea5cef1c65b5890011bdd258f5c61dcf197ee5e3/pkg/printers/internalversion/printers.go#L1018
const (
	loadBalancerWidth = 16
	...
}

func loadBalancerStatusStringer(s api.LoadBalancerStatus, wide bool) string {
	...
	if !wide && len(r) > loadBalancerWidth {
		r = r[0:(loadBalancerWidth-3)] + "..."
	}
	return r
}

func getServiceExternalIP(svc *api.Service, wide bool) string {
	switch svc.Spec.Type {
	...
	case api.ServiceTypeLoadBalancer:
		lbIps := loadBalancerStatusStringer(svc.Status.LoadBalancer, wide)
	...
}

It looks like that in order to support full display of IPv6 in table format, loadBalancerWidth would need to be changed from 16 to 48

Hi,
I think can be try use
kubectl get svc -o wide

kubectl get svc -o wide

@yangnaihua does that show the entire address for you?

kubectl get svc -o wide

@yangnaihua does that show the entire address for you?

in fact, i am not sure. if not , it is a bug


hand-written 5.7'

Done a hand-written 5.7 when doing some homework. If the school releases the answer I would upload that.

Awesome!


Error compiling from Gradle

The library looks great, but I can't include it with gradle, I have the following error, any possible solutions?
Error:(69, 13) Failed to resolve: com.dlazaro66.wheelindicatorview:WheelIndicatorView:1.0.0
Show in FileShow in Project Structure dialog

That's not exactly how you would do that. You need to import the project then use the command
compile project (':libraryName')

Or try using jitpack you can look here to figure out how to do that.

i also have the same issue.. jonhiott i dont understand wat  u saying

The project isn't hosted on Maven Central, so if you want to get the project directly through Gradle without having to import the project files, you'll need to add JitPack as a repository, like this:
repositories {
        // ...
        maven { url "https://jitpack.io" }
    }

Then declare the dependency like this:
compile 'com.github.CardinalNow:Android-CircleProgressIndicator:-SNAPSHOT'


"Waiting for Docker" times out on upgrade to 0.4.5

RancherOS Version: (ros os version)
0.4.5
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
Mac OS X El Capitan
VirtualBox 5.0.20
I am trying to upgrade from 0.4.2 to 0.4.5 but I am unable to do so for this node. I successfully managed to upgrade an identical node, but this one is causing me trouble.

The image shows me trying to boot from the ISO. It gets stuck. If I remove the hard disk from the VM it boots successfully, so I guess there is some state or something on the hard disk that is causing this. Any idea what it could be? I've looked at all the logs in /var/log/ but can't find anything. I found this thread but unfortunately it did not resolve my problem.
I am able to upgrade from 0.4.2 to 0.4.3 and 0.4.4. But consistently getting this error for 0.4.5.
Any help is appreciated.

Do you still have this VM around?
You should still be able to log in.
Can you share your sudo systmem-docker logs and /var/log/docker.log?

@deniseschannon Thanks for the response. I was unable to login when this happened, "Operation timed out" when trying to SSH. Trying to reproduce this now on the same VM I don't get stuck on this part anymore, it continues but does still crash like my report in #945. I have a few more identical nodes to upgrade, so if I hit this again I will re-open. Closing for now and focusing on #945


hunter: remove

Motivation for this change
The program has been unmaintained for over a year. joshuto,
an alternative program, has been merged, so there's no reason to
keep hunter anymore.
https://github.com/NixOS/nixpkgs/issues/83554#issuecomment-946318661
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Hopefully @rabite0 is alright. Yeah joshuto seems to be the main ranger-in-rust contender right now

Looks good to me


Color vertices before canonicalization

Coloring the vertices allows us to reformulate the problem as
canonicalization of undirected graphs. Efficient implementations are
available for canonicalization of undirected graphs, see
https://arxiv.org/abs/0804.4881.

Whoops, the database needs to be rebuilt as the canonicalization algorithm has changed.


refactor micro-service

We are starting to have a number of micro-services in clojure that reuse the same basic ring application container. Copies of the code exist in all of the micro-services.
Refactor this to have a single application server package that can be required by the various micro-services, which then simply provide their own "defaults" and systemd service files.

Refactor the PRS to use the generic ring container.  Before doing so, the code needs to be cleaned up (see #884).


[Feature Request] Reverse Search

Feature Request
Prompts based on readline offer a reverse-search feature which is really handy as an input tool.
Is there any plan to add this in go-prompt?

Looks as duplicate of #91.


[FRONTEND] Add cache_version to triton.jit

Add a cache_version kwarg to triton.jit, so users can invalidate caches (for changes that do not affect the source of the function itself)
Add a __repr__ to JITFunction, so we don't include memory addresses in our cache key
Include triton version, nvcc version, ptxas version in cache name, compute capability in cache key
Don't rely on file mtimes for staleness
Use tuples instead of frozensets for order guarantee
Sort meta_key

Very nice! Thanks! Small suggestion -- should we just use version instead of cache_version. I'm wondering there are probably some cases where users will want to expose a version for other purposes than caching, so a single attribute for both would be nice.

Sounds good, made the change!


Update resources.md

update verbiage

Docs Build status updates of commit ac83ce0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/resources.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ac83ce0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/resources.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Slurp "Read the Docs" Auto Docs

https://hca.readthedocs.io/en/latest/
https://github.com/HumanCellAtlas/dcp-cli/blob/master/docs/Makefile
make docs => html
make docs text   ==> makes docs in  .rst format
is it possible to get markdown or JSON?
does support restructured text.
trent == tsmith12
Can use pandoc to convert???

Obsolete DCP/1 issue.


Bug

I installed bbrf, but whenever I try to create a new module with "bbrf new test" I get an error: "[ERROR] Expecting value: line 7 column 26 (char 258)"

Please enable debug in your ~/.bbrf/config.json file and report the output of your command to help me troubleshoot.

is correct? because with this change I didn't get an error output

Yes, that should be correct; except maybe the ignore_ssl_errors should have a lowercase true, I'm not sure.

Yes, I changed it to "true" and it returned error output

this error gave when I disabled the debug


Fix no show topics

@mghazanfar It does fetch topics always unnecessarily. I think we should improve a logic bit further. For example for community feeds we don’t need topics because we don’t show them there. Maybe cache or keep topics in redux for an hour or until hare refresh so we don’t fetch them every time?!

Sure.

@mghazanfar It does fetch topics always unnecessarily. I think we should improve a logic bit further. For example for community feeds we don’t need topics because we don’t show them there. Maybe cache or keep topics in redux for an hour or until hare refresh so we don’t fetch them every time?!

@feruzm I just went into details. Added my logic as well. But when I saw /ecency-vision/src/common/store/trending-tags/index.ts the logic is already there. If trending tags are already there then it does not refetch and recalls the API so I think we're good to go. It is already optimized.


fix of UUID library

using github.com/gofrs/uuid because github.com/satori/go.uuid is no longer maintained

Signed-off-by: Jozef Kralik<EMAIL_ADDRESS>

Thanks for the PR @jkralik.
Are you actively using goes?

Will have a look at why the build is failing later...

Maybe in future - we are evaluating eventstores. I just test it with current code and I had problem with a compilation of goes (this patch).


Multiple GPUs training failed

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System: Ubuntu 16.04 LTS and 14.04LTS
Compiler:
Package used (Python/R/Scala/Julia):
MXNet version:
Or if installed from source: Yes
MXNet commit hash (git rev-parse HEAD):
If you are using python package, please provide
Python version and distribution: python 2.7
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
[22:38:21] src/mxnet/dmlc-core/include/dmlc/./logging.h:304: [22:38:21] src/mxnet/mshadow/mshadow/./stream_gpu-inl.h:55: Check failed: e == cudaSuccess CUDA: an illegal memory access was encountered
Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
run the provided example/rcnn with multiple GPUs

Try reverting #6954 and see if it works. I've just made a comment on that commit.


Wildcards Not working at All

Working from a direct git-clone of this repo, or from the npm retrieved version, the wild-card feature does not work, either with the examples from the readme.md, or from examples I contrive.  The only way to get the wildcard event listener to be called, is by firing an event with the wildcard actually in the event string.  Firing 'foo.bar' for example, does not work, for a wild-carded foo namespace.  It is as if the asterisk is just being literally matched.  See screenshot from REPL attached, where I tried a variety of things, with no success on wildcards.  Is this something I'm doing wrong, or an actual issue that has emerged with recent changes?

So wildcards aren't enabled by default?

Confirmed, wildcards aren't enabled by default. The documentation isn't very clear on this, and I opened an issue for that. But this should be closed


Support for style-related props (not set via 'style') in React Native

Presently in styled components, there is no way to affect other style-related props in React Native.
This style-related-prop-not-defined-via-style can be found in many third party libraries (such as React Native Navbar's tintColor prop), in custom components one might write themselves, and in React Native's core components, for example:

TouchableHighlight underlayColor
ActivityIndicator color and size
Slider maximumTrackTintColor and minimumTrackTintColor
StatusBar backgroundColor
TextInput placeholderTextColor, underlineColorAndroid and keyboardAppearance
Switch onTintColor, thumbTintColor and tintColor
... and a lot more

It's of course possible to simply set these props manually, but styled-component's themes are unable to be leveraged to standardize them across the codebase.

This definitely will be a great enhancement!

Is there a list of the ones that exist somewhere? We should definitely add support for them!

@mxstbr I'm unaware of any existing list.  The only option would be to go through every available component and manually catalog them

I can volunteer to go through and get the list of those items.

That would be amazing @irakli-janiashvili! 🙏

@mxstbr @adamterlson
Here's the list of all possible attributes that can go in styles ->  https://gist.github.com/irakli-janiashvili/dac99f8ceb0c1a9bafa78aad0f19d861
I put a question marks in front of some attrs that I'm not sure we have to put in styles..

It seems like a lot to maintain manually.
Why not simply provide a way to generally set any prop desired?

That's an unnecessary escape hatch @adamterlson, since you can already set any prop desired when rendering. I think with the list @irakli-janiashvili dug up we should be fine to publish a first version with support for these. It doesn't matter if it's not 100% complete, as we can always add more to the list.
Basically, what we should do is have a separate module called react-native-style-props which exports an object. This object should be keyed by primitive and each one should have an array of props like this maybe?
// react-native-style-props/index.js

module.exports = {
  ActivityIndicator: ['color', 'size'],
}

Then from styled-components we can check at creation time if the current property is supported as a prop, and pass it to the underlying primitive. What do you think, does somebody want to go ahead and create that repo?

This is now finally possible in version two with the .attrs constructor!
const Button = styled.TouchableHighlight.attrs({
  underlayColor: 'red',
})`
  color: blue;
`


bad file descriptor

I've been seeing bad file descriptor error message sometimes when trying to read an incoming packet.
Problem:

The responder creates a net.UDPConn via net.ListenUDP
Afterwards, it calls connFd to get a file descriptor which uses the File method of the net.UDPConn.
The File method creates a duplicate:

File returns a copy of the underlying os.File. It is the caller's responsibility to close f when finished.

The Close method is never explicitly called and the GC takes care of it
This leads to the mentioned error message, trying to read from a closed fd

Solution:
There are at least 2 alternatives and I wanted to ask for opinions.

Avoid creating a duplicate fd to read packets: This implies not using net.ListenUDP, (which doesn't expose the "original" fd) and using a syscall directly to create the listening socket.

Explicitly close the duplicate file descriptor to avoid the GC closing it for us at an undesired time: This implies removing/modifying the connFd function.

Any other alternatives? Thoughts?

Explicitly close please ;)


Validate Slack field config and only allow the necessary input

Resolves the comments on the PR #1002 to update the documentation for Slack fields. Currently, any arbitrary key/value pairs can be defined; however, per the Slack documentation for defining fields [1] only the properties title, value, and short can be defined with the first two being required for each field. This updates the Slack field configuration to only allow the aforementioned three properties as well as validates that both a title and value are defined for each property in the yaml.Unmarshaler interface. Moreover, this allows the value for short to be defined on a per-field basis.

@simonpasquier I've resolved all of the comments you made. Thank you for reviewing!

@simonpasquier I resolved your latest comments. If there's anything else you see in the latest batches of changes let me know and I'll get them pushed up. Thanks!

LGTM


Multiple checkbox / radio items

Hey mate, great plugin.
What does the schema look like to handle multiple option checkboxes or radios? Example: a list of check boxes for a shopping list, Milk, Bread, Coffee, Jam ?
Thanks Mick

I just release<EMAIL_ADDRESS>with the support of multiple checkbox and grouped radio element.
Please read the release note to know more


Default PgSQL charset to utf8

This would fit 99% of use case for #9 as almost all of us doesn't pay attention to the charset when creating a database or the default just fit what we need.

collation should only be provided for mysql, and charset should be "utf8" for both pgsql and sqlserver.

The collation didn't create an issue, because it's not even used by the Laravel's connector:

MySQL uses it
PgSQL doesn't

@RomainMazB I've just adjusted this to make it explicit that the collation is only used for MySQL.


New features added

I added a function to the LiquidLine class so that if you have a floating point variable on the line you can set how many decimal places you want to display when it prints the variable.
I also added some code in the LiquidScreen print method to implement a simple way to scroll the lines so you can add more lines then the LCD has and easily scroll through them
You just need to set the max line display so that the screen knows how many lines to show before scrolling
See the J_scroll_menu example for more details.

This is a very nice update, thank you.


:feet: Add "copy from template" feature when editing a mapping

Issue:
When editing a mapping a user may want to copy an existing mapping template
Fix:
Allow a "copy from template" option, allowing a user to choose a template
when a user choose a template the current mapping will reset to use the same mappings as the template.
Notes:

Allow to use only templates that are on the same namespace or "default" namespace
Alow to use only templates defined for the same source and target providers
Once mapping is copied, a user can edit the mappings again

adding @dgur for visibitly


Release Plan

To improve the ease of use for the community, let's plan on how we gonna organize the released codebase.

Preliminary Tools

[x] Frame/optical flow extraction: already released in our dense_flow. Need to add instructions
Caffe: released with full usage guides
Scripts for building initialization models for the temporal stream

Training TSN

[x] TSN training model spec in Caffe style based on BN-Inception

segment number: 3
aggregation func: avg
[x] RGB
[x] Flow
[x] Warped flow

[x] Initialization model weights based on BN-Inception

[x] RGB
[x] Flow
[x] Warped flow

Testing TSN

[x] standard deploy model spec in Caffe style based on BN-Inception
[x] Trained model weights to reproduce reported accuracies
[x] Evaluation framework:

[x] Python: adapted from the anet repo.
[ ] Matlab: partially completed, will add whole dataset evaluation soon.

[x] Guidance

[x] Data preparation
[x] Run evaluation with trained models
[x] Train models

As of Aug. 8, we have released the basic functionalities. The MATLAB scripts and WARP-related stuff will be released very soon.

@yjxiong flow modality not work now?

@double-vane why did you say that?

@yjxiong  Like #7 ,I don't know where I can download the correct flow caffemodel

The models downloaded with get_reference_models.sh work well with our environments. This is also verified by multiple users. Please check your steps in reproducing the results.
And please do not spam this thread. It'd be better to open a new issue with more details of your problem.

Hi， Professor Xiong, in "...rgb_train_val.prototxt" file, new_segment and new_length are set to 3 and 1,  respectively, that means a snippet has only 1 frame, based on these settings, I train the model on HMDB51 and test on the entire HMDB51 with the setting of evenly sampling 25 frames per video, but the accuracy are really low, however, the validation accuracy during training can reach 53% only with RGB frames for input. I wonder what may cause such difference. Thank you!

@propaganda12
Please use the provided training and testing scripts to see whether you can reproduce the results.

@yjxiong Hi, Professor Xiong, I want to know if the picture after frame / optical flow extraction is the same as the picture I downloaded from https://github.com/feichtenhofer/twostreamfusion.  I am learning your paper, but I have a question in establishing my Caffe framework on the school's Server. Can you give me some advice? Looking forward to your reply.


Publish azure-functions-worker to PyPI

Closing since the package is now published /cc @elprans  - https://pypi.org/project/azure-functions-worker/


github: Return '' when name is None.

@timabbott, please review.

Automated message from Dropbox CLA bot
@umairwaheed, it looks like you've already signed the Dropbox CLA. Thanks!

What ends up happening with a '' name -- does that cause problems further down the line?  Might be good to have a comment explaining why that value is correct.

Will check.

Merged, thanks @umairwaheed!


tlog: User proposals page

This diff updates the user proposals page with the latest tlog changes.

@amassarwi @victorgcramos comments adressed :+1: appreciate the review

@amassarwi @victorgcramos comments adressed :+1: appreciate the review


Error after upgrading 5.1.5 to 5.2.0 on linux container

Describe the bug
We had connection string like this used in 5.1.5:
data source=my-server-domain,11222\SomeInstance;initial catalog=Database;persist security info=False;user id=my-user;password=my-pass;TrustServerCertificate=True;

After upgrading library to 5.2.0 we have experienced exception - but only when app was run on linux (from docker container using newest .NET 8 SDK image). The same code runs fine on Windows 11.
Verify that the instance name is correct and that SQL Server is configured to allow remote connections. (provider: TCP Provider, error: 26 - Error Locating Server/Instance Specified)
       ---> System.Net.Sockets.SocketException (0x80004005): Success
         at Microsoft.Data.SqlClient.SNI.SSRP.GetPortByInstanceName(String browserHostName, String instanceName, TimeoutTimer timeout, Boolean allIPsInParallel, SqlConnectionIPAddressPreference ipPreference)
         at Microsoft.Data.SqlClient.SNI.SNIProxy.CreateTcpHandle(DataSource details, TimeoutTimer timeout, Boolean parallel, SqlConnectionIPAddressPreference ipPreference, String cachedFQDN, SQLDNSInfo& pendingDNSInfo, Boolean tlsFirst, String hostNameInCertificate, String serverCertificateFilename)
         at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
         at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, SqlCommand command, Boolean callerHasConnectionLock, Boolean asyncClose)
         at Microsoft.Data.SqlClient.TdsParser.Connect(ServerInfo serverInfo, SqlInternalConnectionTds connHandler, TimeoutTimer timeout, SqlConnectionString connectionOptions, Boolean withFailover)
         at Microsoft.Data.SqlClient.SqlInternalConnectionTds.AttemptOneLogin(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, TimeoutTimer timeout, Boolean withFailover)
         at Microsoft.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, SqlCredential credential, TimeoutTimer timeout)
         at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, SqlCredential credential, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance)
         at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool, Func`3 accessTokenCallback)
         at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
         at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
         at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
         at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
         at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
         at Microsoft.Data.ProviderBase.DbConnectionPool.WaitForPendingOpen()

Downgrading to 5.1.5 solved issue.
To reproduce
From linux docker container:
await using var connection = new SqlConnection(/* conn here */);
await connection.OpenAsync();

Expected behavior
Connection opens w/o issues.
Further technical details
Microsoft.Data.SqlClient version: 5.2.0
.NET target: net8.0
SQL Server version: (e.g. SQL Server 2017): SQL Server 2019
Operating system: Docker container - mcr.microsoft.com/dotnet/aspnet:8.0

Try changing your connection string to specify port OR named instance, not both.

@pwasiewicz can you test with default instance name to see if that works? What I want to be checked is DataSource without \ in it.

Yes, it worked. We had other SQL Connection with port only and no issues here.

Try changing your connection string to specify port OR named instance, not both.

Is it for investigation purposes or workaround? That database is provided by customer, we cannot change to port or named instance "just like that". Connection to completely different sq; server works when using port only.

Is it for investigation purposes or workaround? That database is provided by customer, we cannot change to port or named instance "just like that". Connection to completely different sq; server works when using port only.

Both. It confirms that specifying both port and instance name is the scenario that is broken, when running on Linux or macOS. Specifying only one or the other is the workaround. Previous behavior was to ignore the instance name if port is specified. Instance name is just a nice way to dynamically look up the port against the SQL Browser service anyway. We will be fixing this in a hotfix to 5.2.

Addressed in #2395.


[Doc] Improvement: fix import statement for qdrant

fix import statement for qdrant
issue: https://github.com/langchain-ai/langchain/issues/28012

#28012

Thanks now looks good.


Calculate mAP only for one class

Hi, i want to know the mAP only for one class, i saw the answer in here:
https://github.com/dbolya/yolact/issues/76
but it's show the mAP for all classes, there is a way to do it?
Thanks!

Uhh, this is just a simple python question. Not sure why you need my help, but here:
You could do this in multiple ways, but here's one:
def calc_map(ap_data, the_class_i_want=INSERT_YOUR_CLASS_IDX_HERE):
    print('Calculating mAP...')
    aps = [{'box': [], 'mask': []} for _ in iou_thresholds]

    for _class in range(len(cfg.dataset.class_names)):
        for iou_idx in range(len(iou_thresholds)):
            for iou_type in ('box', 'mask'):
                ap_obj = ap_data[iou_type][iou_idx][_class]

                if not ap_obj.is_empty():
                    aps[iou_idx][iou_type].append(ap_obj.get_ap())
    ################
    ### addition ###
    ################
		if _class == the_class_i_want
			all_maps = {'box': OrderedDict(), 'mask': OrderedDict()}
			for iou_type in ('box', 'mask'):
				all_maps[iou_type]['all'] = 0  # Make this first in the ordereddict
				for i, threshold in enumerate(iou_thresholds):
					mAP = aps[i][iou_type][_class] * 100 if len(aps[i][iou_type]) > 0 else 0
					all_maps[iou_type][int(threshold * 100)] = mAP
				all_maps[iou_type]['all'] = (sum(all_maps[iou_type].values()) / (len(all_maps[iou_type].values()) - 1))
			print('#################### Class:', cfg.dataset.class_names[_class], '####################')
			print_maps(all_maps)
    ####################
    ### addition end ###
    ####################

    all_maps = {'box': OrderedDict(), 'mask': OrderedDict()}

    # Looking back at it, this code is really hard to read :/
    for iou_type in ('box', 'mask'):
        all_maps[iou_type]['all'] = 0 # Make this first in the ordereddict
        for i, threshold in enumerate(iou_thresholds):
            mAP = sum(aps[i][iou_type]) / len(aps[i][iou_type]) * 100 if len(aps[i][iou_type]) > 0 else 0
            all_maps[iou_type][int(threshold*100)] = mAP
        all_maps[iou_type]['all'] = (sum(all_maps[iou_type].values()) / (len(all_maps[iou_type].values())-1))

    print('#################### All Classes ####################')  # also added
    print_maps(all_maps)
    return all_maps

Replace INSERT_YOUR_CLASS_IDX_HERE at the top with the index of your desired class (so with coco, 0 would be person, 1 would be bicycle, etc.)

Thanks, i didn't mean you to tell me how to change you'r code, it's realy a python question.
I thought maybe you alredy did it.
Thanks so much!


Plan for a license?

Question - not an issue.
I am hoping to use this library but first wanted to see if you had plans on adding a license (and for selfish reasons hoping it is MIT :).
If you'd like I would be happy to add one via a PR.
No matter what - thanks for this repo and the detailed article. It was really helpful!

Solved in https://github.com/tasoskakour/react-use-oauth2/commit/5d65655e372fe93be5fe1db6df5112b83729e16c


Add docker provider for fetching images

Signed-off-by: Matias Pan<EMAIL_ADDRESS>

@matipan can u rebase if you have a sec? :pray:

Done!

:tada:


Adding Update Script to package.json

Hey @arunoda, great work! I was thinking, since many people may not know yeoman very well and may not feel comfortable using it. Maybe it would help to put yo react-cdk:update inside package.json as an NPM script, so people could simply type:
npm run update

Thoughts?

Hmm. I think that's too small for a NPM script.
Eventually, they'll get used to yo.
Since they've to use yo react-cdk my-comp when creating the component.
And we are planning to build a service which helps this process. See #7

Sounds good!