Weblogic Image: t3 to the admin port takes significant time if the Admin Container doesnt map the same port as the docker host

Hi,
Current setup
I am able to build the 12C(12.2.1) weblogic docker image and then create a sample domain 1221-domain:latest extending the oracle/weblogic :12.2.1-developer image. The resulting image is given below
REPOSITORY                             TAG                 IMAGE ID            CREATED             SIZE
1221-domain                            latest              595c72b4ce39        7 hours ago         1.29GB
I was also able to do port mappings to a different host port and the console was accessible on the mapped port. I then deployed one of our internal application which sets up some jms and jdbc resources and then deployed the application itself without any issues.
Note that I am only doing everything on just the AdminServer and not using the node manager and managed server domain.
The issue
The problem comes when I try to connect to the t3 protocol from a remote machine when the exposed port inside the container is different from the published port on the docker host. eg:- I spun up a sample container like this -
docker run -it --name WLS -p 8047:7001 -p 8048:8453 -e JAVA_OPTIONS="-Dweblogic.rjvm.enableprotocolswitch=true" myapp:v1
I can successfully login to the console using http://:8047/console
However, when i try to connect through t3 on 8047, it connects only after a significat time (15-20 mins). I initially thought this is something to do with my local test framework which I am using, but using Hermes JMS also returned the same behaviour.
I then spun up another container mapping exactly the same ports as the container exposes.
docker run -it --name SAMEPORT -p 7001:7001 -p 8453:8453 -e JAVA_OPTIONS="-Dweblogic.rjvm.enableprotocolswitch=true" myapp:v1
and then my test framework was able to immediately connect to through the t3 protocol to the AdminServer on 7001. I am not sure what is special about t3 ports vs the http ports. The console access works absolutely fine, but t3 needs the exact same port on the docker host as the container exposes.
my workaround
As a workaround, I am spinning up containers like below
docker run -it --name WLS -p 8047:8047 -p 8048:8453 -e JAVA_OPTIONS="-Dweblogic.rjvm.enableprotocolswitch=true" myapp:v1
and then 'exec' into the container and change the port in config.xml to 8047 and restart the container.
I am unable to pin point if this is a weblogic docker issue or something to do with the docker-proxy or my local set up.
Please help!!
Regards,
Vipin

Hi, It would be great if someone can look and provide a resolution to this issue.
regards,
Vipin

Hey @mriccell, mind to review this one please?

Its a very interesting issue and your workaround matches a known behavior when using OTD to balance t3 connections (not sure if you have seen this article, but here it goes: http://pauldone.blogspot.co.nz/2013/04/load-balancing-t3-initial-context.html) and it might be the way to go.
Just one more interested in it and wanting to follow the update from the product perspective.

The problem relates to t3 using the default channel to connect to the servers in the container where the internal server port and the external host port are different.

currently the workaround is to configure a channel and specify the external address and external port and then use the channel to make your t3 connection.

@vipinponnarath make reference to the documentation https://docs.oracle.com/cd/E72987_01/wls/WLACH/taskhelp/channels/ConfigureCustomNetworkChannelsForNonclusteredServers.html leave the address blank and only configure your port to the host port that you are mapping.  Can you please try it and let me know the results?

@mriccell - thanks for the response, I will try this and post the results here.

@mriccell I tried to configure a Custom Channel, but no luck here. I am not sure if I am doing anything wrong here. Here is a snapshot of the channel configuration. My admin server is accessible on the docker host port 9001. The container has the default port 7001.

The Channel External port must match the port you are mapping the internal 7001 port of the WLS server to host.  Example in your run -p 8047:7001, the internal port is 7001 the external port is 8047.  The listen address is correct it should be blank.

Hello @mriccell ,
If you mean, I should use 8047 instead of 9001 for the Public port in the above snapshot, i just chose this time to spin up the container to publish the internal port 7001 on the host port 9001, like -p 9001:7001.
Do you also mean that I should configure the Listen Port in the above snapshot to also 7001? Note that this doesnt work due to a port conflict. I have attached a snapshot here.

@mriccell, do you have some time to check my updates above? I appreciate the help you are providing.

Hello @mriccell
Please can you check the above update and provide a resolution?
Regards,
Vipin

Could you post the WebLogic config.xml and the related docker config regarding network setup?

Beside that we can tell the above "Channel-t3" configuration you have it listening on 7001 and the weblogic server has so called default channel which is the port the server is listening on and by default it is using 7001 also so if you did not customize it then your weblogic server already configured to listening on port 7001 and that is why you are getting the error message. So, send the above info and describe what port you want the weblogic server to listen on and what port you want those ports to map out.

Configure a t3 channel with ListenPort set to 8002 ( other than Server Listen Port) and Public Port set to any valid Port e.g. 9002.   See the snippet from config.xml file

AdminServer
8001

Channel-0
t3
8002
9002
true
false
false
true
false
false

--
Use the following Docker run command
docker run --name WLS -p 8001:8001 -p 3098:8002

@vipinponnarath have you tried this configuration?

@mriccell @shean-guangchang @anpanigr ,
I tried this configuration and it is quite similar to the workaround that I posted while raising the issue itself. It works for only that specific listen port mapping(-p 3098:8002). The moment I spin up a second container and configure a channel again with another port mapping (say -p 4098:6002) my test suite gets stuck at this t3 connection stage and then it waits and waits and after a really long time (~20mins) it proceeds successfully. Just as reported in my original issue. What is it doing these minutes(just thinking loud)
Just that we all understand the use case here. I have a small docker server that can spin up several containers from the same weblogic image template. All these different weblogic containers, runs on the same docker host and they just maps to different host ports. But with our current situation, it seems I can only spin up one container from one docker host.
Again - note that it is only a problem if I use t3. Otherwise, I can run as many weblogic containers as i want and they all map to different ports on the same host and runs tests without any problem.
the config.xml from both the containers that i used for today's test is attached herewith.
Also the docker container network inspect for one of the container is given here, actually even if I use the default network, its still the same - so not sure if this file is of any help.
Additionally, the docker info results are also attached - if that helps.
Please let me know what additonal config you'd like to check.
regards,
Vipin
Docker.zip

@vipinponnarath This is probably because you are using the 12.2.1 domain sample image where the domain is created at build time of the image.  How are you changing the configuration when you spin a new container (managed server)?
In the <IP_ADDRESS> domain sample image the domain is externalized to a volume and the domain is created at run time. Any changes such as creating a new port or channel for a managed server will be reflected in the domain configuration for all servers running in the domain.
The default channel does not work when the host  port is different from the server port, this is why you need the channel for t3.

Hello @mriccell - Not sure if you closed this because of not receiving a response from me for the question in your above email.  Would you be kind enough to reopen this?
---->How are you changing the configuration when you spin a new container (managed server)?
Ans: I am not using managed server. I am only using AdminServer for my development purpose.
I have the prebuilt 12.2.1 domain image from which i am spinning up 2 containers as below:

@mriccell
I wanted to try the <IP_ADDRESS> domain sample, but I cant find a way to extend the Dockerfile because of the fact that the domain is created at runtime. If I want to make an app deployment, i need to have a domain as pre-requisite. So I need to manually commit after creating the domain at runtime. This is not very efficient. Could you kindly reopen this issue?

@vipinponnarath
In <IP_ADDRESS> sample domain the domain_home, logs, stores, and app deployment is externalized to a volume outside of containers.  To deploy the application follow the following steps:

every container needs to have a shared application volume say /shared/apps
2)each of these directories should map to an external volume /hosts/apps
Use any of the following ways to deploy the applications
Administration console
REST call
WLST
weblogic.Deployer

@mriccell
The question was not really on how to deploy - this is very clear to me. We are already having wlst scripts to deploy our application. My goal was to use Dockerfiles to call the wlst for doing the deployments.
eg:- I want to write a dockerfile as below:(just concept- ignore the syntax)
FROM 12213-domain:latest
RUN ./startWeblogic.sh &&
./waitForAdminServer.sh &&
./deployApp.py &&
./stopWeblogic.sh
CMD ./startWeblogic.sh
Now as you can see the 12213 image is creating domain only at runtime - which means my above approach wont work. We are right now prototyping on a fully automated pipeline where there is no manual interaction (like as you suggested above - a console based deployment). Anyway this is a different topic from the original issue in this thread and I will try to raise this separately in another thread.
Can you please tell me, from your point of view, is it possible to run two AdminServer containers on a single host? If yes, how does the t3 channel behave in such a situation?

The <IP_ADDRESS> sample is intended to extend the WebLogic install image and create a domain with a cluster of managed servers. If you want to run only simple server (Admin Server ) topologies,  you need to extend the WebLogic install image in https://github.com/oracle/docker-images/tree/master/OracleWebLogic/dockerfiles/<IP_ADDRESS> either developer or generic version.  This WebLogic install image can also be pulled from Oracle Container Registry or the Docker Store.
Hope this helps.

Question about your port configuration when you say you configure a second t3 channel do you mean on the same server? Do you mind sharing your configuration?
Monica

@mriccell - Can you tell me exactly what configuration you would like to see? the weblogic config.xml? or some specific config to my docker setup?. I have already shared the two containers config xmls in one of the above discussion (Dec 19 update from me). But I can again redo all that and give you new config if you need - but please tell me exactly what you would like to have.

I would think to run  2 Admin Servers each exposing a different t3 port you would need to write WLST code to create a t3 custom channel in each where you are mapping the internal t3 port  of the server to a different host port for each server
See the snippet  from config.xml file

AdminServer1
8001

Channel-0
t3
8002
9002
true
false
false
true
false
false

AdminServer2
8001

Channel-0
t3
8002
9003
true
false
false
true
false
false

@vipinponnarath I have read with more care your problem.  As I understand it you want to use the same exact application image and just map the Admin server port to different host ports. You are trying to run many Admin servers with the application deployed and have t3 calls to be made to different ports.
You said before you are using  -e JAVA_OPTIONS="-Dweblogic.rjvm.enableprotocolswitch=true" are you sure that the servers that start have this JAVA_OPTION enabled? Can you show the container info?

I have a two server env and one server I use to run the weblogic docker image (<IP_ADDRESS>.0) and my command line is
docker run -it --name WLS -p 8047:7001  -e JAVA_OPTIONS="-Dweblogic.rjvm.enableprotocolswitch=true" weblogic-developer
an empty domain will be created as part of container startup...
Oracle WebLogic Server Auto Generated Empty Domain:
  ----> 'weblogic' admin password: xivp7VL0

Initializing WebLogic Scripting Tool (WLST) ...
Welcome to WebLogic Server Administration Scripting Shell
Type help() for help on available commands
domain_name     : [base_domain]
admin_port      : [7001]
domain_path     : [/u01/oracle/user_projects/domains/base_domain]
production_mode : [prod]
admin password  : [xivp7VL0]
admin name      : [AdminServer]
admin username  : [weblogic]
from the other machine I can make a t3 connection using "t3://machine:8047"
What I used is weblogic.WLST command and as long as you have the weblogic env then you can do
java weblogic.WLST and then it will enter an interactive mode
$ java weblogic.WLST
Initializing WebLogic Scripting Tool (WLST) ...
Welcome to WebLogic Server Administration Scripting Shell
Type help() for help on available commands
wls:/offline> connect()
Please enter your username :weblogic
weblogic
Please enter your password :
Please enter your server URL [t3://localhost:7001] :t3://adc00qxd:8047
t3://adc00qxd:8047
Connecting to t3://adc00qxd:8047 with userid weblogic ...
The CompatabilityMBeanServer is not initialized properly.
This might happen if the CompatabilityMBeanServer is
disabled via the JMXMBean.
To view the root cause exception use dumpStack()
debug patch for bug 25558644 WorkContextLocalMap for 12.1.3 generated on 2/7/2018
Apr 12, 2018 11:30:17 AM
FINE: Verifying debug logger in 25558644
Successfully connected to Admin Server "AdminServer" that belongs to domain "base_domain".

forgot to mention it took no time to establish the t3 connection.

If my example is matching what you did then it seems working for me.  I have a different version of wls image (wls ps3 release) so could you try with that release?

@vipinponnarath are you still experiencing this issue?

Closing issue due to lack of response

I also encountered the same problem, from the external use of the t3 protocol to access the weblogic application inside the container, but can not access, please let us know

@mricell, I have deployed my war file on weblogic server docker container,  does it reflect to the existing docker image for which docker container is running.

I faced a similar issue. On my side to add -Dweblogic.rjvm.enableprotocolswitch=true JVM param to the WebLogic start script solved the issue. I did not create any additional channels. The -D param was enough.

I faced a similar issue. On my side to add -Dweblogic.rjvm.enableprotocolswitch=true JVM param to the WebLogic start script solved the issue. I did not create any additional channels. The -D param was enough.


Store the order of the answers to the group membership question

Hello, @JamesChenX I would like to contribute. Can you please provide some more info on the task?

Hi, @DevPJ9 , I'm glad you want to contribute.
This ticket needs some small changes in turms-client-js/kotlin/swift/dart, and turms-service, so it requires a full-stack developer with a MacOS to implement the feature. And if you are still interested in it. I can write more implementation details for you.

Thanks for your reply @JamesChenX. I am sorry but I don't have macOS so I will focus on the other task. :)


Update description of package on NuGet

Current description and tags are from the package Verify.Http.

Not related to the PR, but I'm not able to sign the CLA, it says there is no CLA to sign for VerifyTests/Verify.MicrosoftLogging

thanks


Uncaught object "[object Object]" was thrown. Throw Error instead on t.takeScreenshot() with Node 18

What is your Scenario?
Take a screenshot when a test fails, or take a screenshot during a test, while on Node 18.2.1
What is the Current behavior?
If a screenshot is taken during a test, the error "Uncaught object "[object Object]" was thrown. Throw Error instead.
If a screenshot is taken when a test fails, the testrunner stalls indefinitely
If I roll back to my original version of Node 14.17.6, both scenarios work as intended
What is the Expected behavior?
If a screenshot is taken during the test or on a test failure, the terminal should report back the error and the screenshot should be published to the specified test-artifacts folder
What is your public website URL? (or attach your complete example)
Testcafe Demo Site
What is your TestCafe test code?
Github repo with sample test and setup
Your complete configuration file
{ 	"screenshots": { 		"path": "test-artifacts/screenshots", 		"pathPattern": "${TEST_ID}_${FILE_INDEX}_${DATE}_${TIME}_${BROWSER}${RUN_ID}.png", 		"takeOnFails": true 	} }
Your complete test report
No response
Screenshots
No response
Steps to Reproduce

Pull down the demo repository attached, make sure Node 18.2.1 is installed
Run the sample test as is to see the error on taking a screenshot during the test as described in the title of this issue, and then run it again without t.takeScreenshot in the test to see it stall on taking the screenshot when the test fails

TestCafe version
2.1.0
Node.js version
18.2.1
Command-line arguments
chrome:headless
Browser name(s) and version(s)
No response
Platform(s) and version(s)
Windows 10
Other
Remember that this issue with screenshot is only present on the latest version of Node 18.2.1, but work as intended on Node 14.17.6. I did not test any other versions of Node.

Hi @tsherkqa,
Yes, this behavior is reproduced in TestCafe v2.1.0 TestCafe. Please update TestCafe to 2.2.0-rc.1 to resolve the issue.


preprocessor mixin arguments are referenced as variables if defined

for example, in stylus, component(element) gets compiled to component(<code for "element" mixin>) if element is a mixin in stylus. This makes system unusable in some environments.
The solution to this is to compile all preprocessor mixins to a syntax that uses strings, rather than idents.

closed by #10


AP-759 UAT incosequential change to force a PR to be created

Display scope limitations, costs and levels of service from database
Link to story
Link to story
The original ticket to do this work was merged into master before being tested on UAT.
This branch / PR is to enable the code to be pushed to UAT for testing.
Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

there was an issue deploying this to UAT, i created a different release from this branch so I am deleting this PR.


Cannot share private domain

Dear author,
I'm following your steps to build a HA test demo.
I'm now successfully mapped one CPI's API with custom domain service (as step-3), but when I configure the second one, I cannot map the second API with the same domain, even I have already run the command share-private-domain, could you please help to figure this issue out?

@maxstreifeneder and @hterminasyan,
there was a change in the process for the cf custom domain since I published the blog initially if I remember correctly, right?
@AlgorithmGoddess did you verify the general domain create process? Have a look at my outdated CLI sequence here. The wildcard domain is the same across both spaces, only the routes are different.
I suspect the domain sharing part changed, hence the issue. @maxstreifeneder and @hterminasyan will update my blog too once figured out how to clarify.
KR
Martin

Hi All,
I have found my problem of this issue, it's because I was trying to create 2 map route in the same cloud foundry api endpoint. I then create another tenant with different area and can successfully do the routes and mapping.
Thanks all for help, especially thanks @MartinPankraz to ping me to help me out.
KR
Lexi


Make metrics consistent between nodes

Hosts have % memory usage, containers have it in KB (or MB etc).
We probably want a way of doing both, TBH.

Seeing both together would probably be A Good Thing.


Add ECS Task IAM role ARN

what
Add the output for ECS Task IAM role ARN
why
Needed for attaching to external service policies outside of module.

The CI on this will fail until this is merged/released: https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/pull/7


Pass by value

Fix to allow service programs with parameters defined with "value" to work.

sorry it took a while...thx for the steps you provided to clean up the PR

Great news!
On Mon, Mar 18, 2019, 2:49 PM Abdirahim Musse<EMAIL_ADDRESS>wrote:

@abmusse approved this pull request.
LGTM
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IBM/nodejs-itoolkit/pull/43#pullrequestreview-215788285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APFeQfNBW6h-_lZ-pkIQfOTw1udX9uH1ks5vX9-mgaJpZM4b3zRY
.


[REVIEW] Some fixes to dbscan demo notebook in prep for 0.10 release

@cjnolet did you update based on @JohnZed 's comments?  Waiting on that to merge :)

Nope. I will get to this tonight or early tomorrow.
Sent from my iPhone

On Oct 8, 2019, at 6:19 PM, Taurean Dyer<EMAIL_ADDRESS>wrote:

@cjnolet did you update based on @JohnZed 's comments? Waiting on that to merge :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@taureandyernv @JohnZed, implemented the changes. This is ready to go!

Aright, how about now?


Supported chip types

What are the supported chips this library supports? can I use this library with any kind of chip?

Hi @BravenxX it depends on your use case. If it's for NDEF read or write, most of the NFC chips should work well.


PR for article on self-perceptions

Author: Rinku Gupta
This is a PR for the curated content article on self perceptions mentioned in issue #348

This needs to be merged and publish immediately.

Assigning to @pagrubel to make changes and then publish

This is merged , metadata set and should be published in the next build

This is merged , metadata set and should be published in the next build


bug: high memory usage when operating with buffer

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.4
Operating system/version
Windows 10
Describe the bug
Any operation regarding Buffer would make a jump in memory usage from 50MB to 600MB and keep climbing to almost 2GB before I ran out of memory (I got 8GB laptop memory).
Happened multiple times throughout the day. I don't think it's the LSP coz the latest one happened when I'm editing notes.txt
Steps To Reproduce

Open a document (txt file would suffice).
Open the buffer explorer
It would take 10-15 seconds to load.
From there memory usage start creeping up from initially < 50MB to 600MB and then till I ran out of memory around 2GB usage.

The text document I'm editing was 10 lines long.
*Upon further monitoring, before nvim process starts to creep up, Git for Windows appeared for several seconds. Then the git process disappeared and from there neovim.exe start eating the memory.
Expected Behavior
I expect the buffer explorer to open instantly and memory usage to keep steady between 50-100MB (my normal usage prior to opening the buffer explorer).
I'm on a LazyVim fresh install with Gruvbox theme as the only modification made.
Repro
*forgot to mention, I have reinstalled LazyVim 3x and the same issue keeps appearing.

A screenshot of the Task Manager.
the memory usage will go through 2GB+ and then I ran out of memory. Note the Git for Windows processes. Memory usage back to normal post force-closed nvim.exe

What do you mean with the Buffers Explorer?
I can't reproduce this at all. Nobody ever reported something similar, so I'm pretty sure something's messed up with your setup.


Compile for both Swift 2.2 and Swift 2.3

Pull request checklist

[ ] All tests pass. Demo project builds and runs.
[x] I have resolved any merge conflicts.
[x] I have followed the coding style, and reviewed the contributing guidelines.

What's in this pull request?
By using nil-coalescing operator instead of force-unwrap, it's possible to compile for both Swift 2.2 and Swift 2.3.
It seems like you can completely skip releasing extra major version for Swift 2.3, since you can have a minor one, that works in both 2.2 and 2.3.

Thanks @DenHeadless ! 😄
There's already a swift2.3 and swift3.0 branch with this work.
I don't think tempURL ?? NSURL() is right thing to do here. Those are not equivalent.
Anyway, I'll be releasing updates for both Swift versions soon! — Once 3rd party tools stop being broken. 😊


[tool_crash] FileSystemException: Exists failed, OS Error: Permission denied, errno = 13

Command
flutter update-packages

Steps to Reproduce

...
...
...

Logs
FileSystemException: Exists failed, OS Error: Permission denied, errno = 13
#0      _Directory.existsSync (dart:io/directory_impl.dart:94:7)
#1      ForwardingFileSystemEntity.existsSync (package:file/src/forwarding/forwarding_file_system_entity.dart:45:33)
#2      ForwardingFileSystemEntity.existsSync (package:file/src/forwarding/forwarding_file_system_entity.dart:45:33)
#3      PubDependencies.isUpToDate (package:flutter_tools/src/cache.dart:654:58)
<asynchronous suspension>
#4      Cache.updateAll (package:flutter_tools/src/cache.dart:466:26)
<asynchronous suspension>
#5      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:936:27)
#6      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:844:33)
<asynchronous suspension>
#7      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart)
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#9      _rootRun (dart:async/zone.dart:1190:13)
#10     _CustomZone.run (dart:async/zone.dart:1093:19)
#11     _runZoned (dart:async/zone.dart:1630:10)
#12     runZoned (dart:async/zone.dart:1550:10)
#13     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#14     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:833:20)
#15     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#16     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:333:21)
<asynchronous suspension>
#17     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart)
#18     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#19     _rootRun (dart:async/zone.dart:1190:13)
#20     _CustomZone.run (dart:async/zone.dart:1093:19)

[✓] Flutter (Channel master, 1.22.0-10.0.pre.269, on Linux, locale en_US.UTF-8)
    • Flutter version 1.22.0-10.0.pre.269 at /opt/flutter
    • Framework revision 51fa704642 (2 days ago), 2020-09-19 11:27:04 -0700
    • Engine revision 4b8477d115
    • Dart version 2.10.0 (build 2.10.0-142.0.dev)

[✗] Android toolchain - develop for Android devices
    ✗ ANDROID_HOME = /opt/android-sdk
      but Android SDK not found at this location.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[!] Connected device
    ! No devices available

! Doctor found issues in 3 categories.

Flutter Application Metadata
No pubspec in working directory.

Cc @DanTup I think this may be related to https://github.com/flutter/flutter/issues/15198

I don't think so. Problem is that flutter dart uses /root/.pub_cache folder instead of current user's pub_cache folder. I run flutter doctor as non-root user.

In #15198, the VS Code validator was blindly calling existsSync() on folders to see if they have VS Code which could result in permissions errors and it was valid/correct to ignore them if there is no access.
Here, it seems like there's no access to the pub cache - so I don't think silently ignoring the failure is the right fix - the question is where is the cache and why is there no access to it.
@Wasth

flutter dart uses /root/.pub_cache folder instead of current user's pub_cache folder

My expectation would be that if you use the zip file version, it would use a pub cache inside Flutter's own cache folder, but otherwise fall back to Pub's standard default. Where are you seeing /root/ being used?

In exception output. I installed flutter-git package from AUR(I'm using Manjaro). I changed all permissons in dart-sdk folder, flutter folder but anyway it tries to get access in root pub cache folder. I think I should close this issue because I reinstalled flutter through IDEA(with Flutter plugin) and it's worked
Thanks for helping!


List of own boards

As a user, I want to see a list of my own boards on the start menu, so that I can easily go back to my old boards (e.g., if I lose the URL).
Maybe also a "recently visited boards" list, not just the ones I created?

Great idea 👍🏼 Should we only list boards a user created or also the boards the user participated in?

It would be nice to have both. I would envision something like the table below.
| My Boards |
-------------
| Board #1 |
| Board #2 |
-------------
| Participated Boards |
-------------
| Board #1 |
| Board #2 |

Thanks for your input @ImDevinC! I like the Idea!
Another idea similar to yours would be to show both, my boards and other participated boards, in a single list and then highlight my own boards. We could then add a filter to show only my own boards.
I don't know which of these I prefer. What do you think @ImDevinC & @brandstetterm?

I don't have a preference


React Router 4 migration

Any intentions on migrating this to React Router 4 soon?

@williamoliveira I try to migrate this or next weekends

alright, thank you

@williamoliveira But I hope I need to wait when will be the first stable version of react router, because current version is v4.0.0-alpha.5

I'll take

I'm having a lot of trouble migrating to the react-router, so if someone better able to contribute, you can stay at ease.

I'm working on this

It has a logo in /images, I think it should be used.
Sorry man, but I'm having problems with the configuration of android/ios, so if someone can contribute better, you can stay at ease.
I will share this issue among the groups I am a part of.


Unbreak Swagger docs

This got lost on an unmerged feature branch somewhere. The config for OpenAPI docs was incorrect, and is fixed here. Verify by visiting http://localhost:8899/api/v1/docs.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.89%. Comparing base (b3fe47f) to head (b921008).
Report is 1 commits behind head on trunk.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk      #50   +/-   ##
=======================================
  Coverage   96.89%   96.89%
=======================================
  Files          28       28
  Lines        1225     1225
=======================================
  Hits         1187     1187
  Misses         38       38

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


semantics - no results related to java tag?

I'm wondering if the sequence of my actions is limiting my results unintentionally, or if there are in fact no tags/content related to "java".
-Clear all filters
-Type java in tag selection box
-Select Java
-Hover over Java: No Related Tags in filtered results.
Is there a way to see if there are tags related to java in the unfiltered results? Is it searching for java AND related tag?
// @grechaw for implementation
// @yawitz for intended interaction

ill try to reproduce and get the sparql query out.  Could be permissions (the no-results you see is consistent with a permissions issue).  There were changes in 8.0-4 that affected permissions, and samplestack did special case permissions things.

Okay. FYI i get the same result whether logged in or not (though I realize you are likely looking at a separate set of permissions).

The related tags should reflect the post-search, but pre-filter results. (In other words, the tags filter shadow result.) Given that you didn't enter a search string, the related tags should be based on the full, unfiltered data set. This sounds like a bug to me.

I remember also we've had issues actually loading the RDF before.

looks like there really are no related tags for java.  Just something about the data dump that we have.
You can verify with some other searches.
Something about the original search criteria for stack overflow and/or dbpedia caused this issue.

Thanks for checking. Might be worth looking into in the future to better understand our extract, but no need to dive in now.


using workpackage filter links in project route switches to workpackages route

filter links need to be changed, but before this a proper dependency between a project and it'S workpackages need to be implemented.
therefore workpackages need to be saved with some reference to a project. maybe saving the project key would in a workpackages attribute.

tracking in trello using https://trello.com/c/4i9jbayU


Fix DataSet$EventNew.toString() NPE

Fixes #284

This won't fix the issue, in this instance it was the dataset.run being null.
I'll handle this separately, thanks for the PR, though.

It wan't intended to fix the root cause of the EventNew.dataset being null, but it does fix the issue of writing NPE's to the application log.

Yes, but it wouldn't fix the NPE either - in this case dataset would not be null but dataset.run would and you're checking only for the first one.


minYear is not working nicely

When I set a minimum date for the datepicker, and enable selectYear. When I first open the year field, I only get years upto 2006, only then I can scroll up more years.

What you can do is set selectYears: 200 for example. The default is 5 if you set it to true.
https://github.com/amsul/pickadate.js/blob/0d87d916a5d56284aa4b0d7fc9e725db1ea4bf72/lib/picker.date.js#L1095


Add thai language

Checklist
Please put "X" in the below checkboxes that apply::

[ ] If committing a bugfix, I have tested it in different browsers (Chrome, Firefox, Safari).
[ ] If committing a new feature, I have first submitted an issue (Please note: you are free to open PRs for non-issued features, but opening an issue increases your chance of a successful PR).
[x] If committing a new feature, I have also written an appropriate test suite for it.

I have tested the following:

[x] Qalendar component in month mode.
[x] Qalendar component in week mode.
[x] Qalendar component in day mode.
[x] All of the above modes on emulated mobile view.
[x] Dragging and dropping events.
[x] Resizing events in day/week modes.
[x] Clicking events to open event dialog.

This PR solves the following problem**.
insert your description.
How to test this PR**.
For example:

Feed X and Y in the events-Prop.
Click Z or A.
Confirm that B is displayed.

Hello @Janescience. Thanks for reopening this PR.
The changes look good, but your changes on src/language/index.ts are missing here. Mind adding them?


Issue getting the extension running

After following all the steps for getting started, the extension doesn't work, only getting errors:

Steps to replicate:

Clone repo
yarn & yarn dev
load as unpacked extension: dg-translation-chrome-ext/src

Can you tell me about your environment?
Operating System/Version:
Language:
Browser:

Ok
OS: Windows 11
Language: English
Browser: Chrome
On Tue, 11 Oct 2022, 17:55 BekahHW, @.***> wrote:

Can you tell me about your environment?
Operating System/Version:
Language:
Browser:
—
Reply to this email directly, view it on GitHub
https://github.com/deepgram-devs/dg-translation-chrome-ext/issues/10#issuecomment-1274997880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJVPQPWEI3UAYP6GRCHSD6DWCWLYZANCNFSM6AAAAAARCAGGJA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok, @arndom, can you try to get the latest version of the repo and then run yarn again? I think it was somehow missing the TypeScript devDependency. This should solve it.

Just to clarify, you stopped the server, ran yarn, and then ran yarn start?
And your package json file include typescript as a devdependency now right?

Yes, I have done all that and checked that typescript is present:

Ok, let's try this:
Delete .parcel-cache, dist, node_modules, yarn.lock.
Run yarn and then yarn start.
Hopefully, this fixes the problem.
Just double checking, you don't have a package-lock.json file, right? If you do, delete that as well.

@BekahHW, so...I've tried it but still get the same error.
Also, I've tried loading the unpacked extension from dist instead of src and have been getting:

I can't do it today,  but do you want to do a video call in the next couple of days and see if we can get it working together?

Sure, that would be really helpful...when would you be available?

@arndom I think I figured it out. I missed a step in the readme. I have it all updated now, but here are a couple of updates. Notice that we're loading from the dist folder now.

Run yarn to install the dependencies.
Run yarn build.
To load this extension unpacked, navigate to /dg-translation-chrome-ext/dist.

@BekahHW I tried it; here's my result:

@BekahHW I got it past the previous error by editing the generated popjs in the dist folder to use the generated content-script.....it asks to choose a tab to share audio but I cant seem to find the transcription of the playing video.

This is the error I am getting trying to get this to work:

@BekahHW I got the error not to come up consistently by changing the way the popup.ts reads the content-script, issue now is that it seems to be sharing my audio, but I can't see any transcription.

my fix:

popup.ts:

content-script.ts:

popup.html:

That's great news, @arndom. Do you want to create a PR for this and I'll merge in the fix?

@BekahHW, Here's the PR: https://github.com/deepgram-devs/dg-translation-chrome-ext/pull/18

@arndom did you delete .parcel-cache, dist, node_modules, yarn.lock, and then run yarn, yarn build, and then yarn start? I'm getting a different error

Yes I did all that, that error seems to happen when the manage extension tab is open: chrome://extensions/, I closed the tab and reloaded the extension to get it working

@arndom it's interesting bc I can console.log the transcript in showLatestTranscript() and it shows up in my console, but I'm not getting it in the popup.html. Feels like we're getting closer though

@arndom I just pushed up a fix. Should be working now!

awesome, I'll pull and try

@BekahHW It's working

@arndom


Submit crown counts via the background page

This avoids invoking the NetworkError on Firefox, while still maintaining the isolation from other user extensions that was blocking submissions from occurring when submitted from the injected script.
Closes https://github.com/DevJackSmith/mh-hunt-helper/issues/98

why did it try to build???


Events and food order

Can you advice on how i can change the items in events? Can i modify the data.JSON file?
I don't know how the firebase thing works.

You can't modify the events in my database, you would have to setup your own database (which could be firebase or something else) and load data from there.

Can you provide me the structure of the events page of your database?

You can inspect the request in the browser to view the data. It calls https://ionic-in-action-demo.firebaseio.com/events.json.


[web] Adding image in editor shows blank space

When I am adding image in editor in web, I see only blank space for image.

@rish07 it works :) I will check what is wrong my the custom code. Thanks for your help! I am closing the issue at this moment.


Numbers are rendered above item descriptions in haggling things

Thanks for the report.
This is fixed in newer (1.19) versions. We're unlikely to backport the fix unfortunately.

alright! my bad


Missing step for migration to resource manager

Found out that the Azure recovery services extension must be uninstalled for the migration validation to succeed as per the below article
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/migration-classic-resource-manager-errors

@SofianeMahdjoubi,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@SofianeMahdjoubi : Thanks for your contribution to the Azure documentation! The author, @markgalioto, has been notified to review your proposed change.

#sign-off

@SofianeMahdjoubi  thank you for pointing this out. I'm going to merge your commit.


Bungee error on login

What is the output link of /viaversion dump?
https://gist.github.com/anonymous/381e90451bbe677c06d08aa3f000330f
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
All relevant info listed in dump.
How does this error happen? login? Using an item?
Looks to happen randomly on login
Is there an error in the console? Use pastebin.com. Is there a kick message?
https://pastebin.com/sxzBcVju

I have this same error:
https://gist.github.com/pjorun/8ae28d2236db4dcffe2caa3d8879b00a

My variation of DecoderException error.
https://gist.github.com/Per-Thulin/ed7f683c1252f61b26e62828e9bf9955
On Bungee, related to bungee lockups maybe.
ViaVersion-1.2.1-SNAPSHOT, was latest version on Aug 5th

@pjorun This issue doesn't look related to this one, please make a new issue and fill out all details.
@Per-Thulin This just looks like a client sending bad data

Could you possibly try without some of your BungeeCord plugins? I'm in belief this might be down to outdated plugins messing up packets on the bungee.

Ill see what i can do about testing with less plugins, however most of them are used and this is in production

Please get back when you have more info & request reopen, I suspect this is down to outdated bungee plugins (most likely).


[Slide][material] Slide is not working after upgrade to V5

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Create a simple List component
Bring the list component within a Slide API
Now you can see the error.

Current behavior 😯
We are using Slide API for transition of the listing component from Right direction. It was working very smooth in version V4.x. Recently we upgraded to V5.x. Not it is throwing error.
The Code is :
Slide direction="right" in={true} timeout={500} mountOnEnter unmountOnExit
BoardsList /BoardsList
Slide
Error:
node.getBoundingClientRect is not a function
TypeError: node.getBoundingClientRect is not a function
at getTranslateValue (http://localhost:3000/static/js/bundle.js:87023:21)
at setTranslateValue (http://localhost:3000/static/js/bundle.js:87070:21)
at http://localhost:3000/static/js/bundle.js:87124:5
at Object.onEnter (http://localhost:3000/static/js/bundle.js:87119:9)
at Transition.performEnter (http://localhost:3000/static/js/bundle.js:168679:16)
at Transition.updateStatus (http://localhost:3000/static/js/bundle.js:168650:14)
at Transition.componentDidMount (http://localhost:3000/static/js/bundle.js:168598:10)
at commitLayoutEffectOnFiber (http://localhost:3000/static/js/bundle.js:162105:34)
at commitLayoutMountEffects_complete (http://localhost:3000/static/js/bundle.js:163272:13)
at commitLayoutEffects_begin (http://localhost:3000/static/js/bundle.js:163261:11)
Expected behavior 🤔
The Slide should transit the component Right to Left
Context 🔦
We are about to remove the Slide API from the application.

  Output from `npx @mui/envinfo` goes here.

Hey @vairaselvam, thanks for the report 😊
May I ask you to provide a codesanbox example of this issue? You can use this template as a starting point: https://codesandbox.io/s/material-ui-issue-latest-s2dsx

To provide more info,  this is the error stack :
Slide.js:18 Uncaught TypeError: node.getBoundingClientRect is not a function
at getTranslateValue (Slide.js:18:1)
at setTranslateValue (Slide.js:65:1)
at Slide.js:119:1
at Object.onEnter (Slide.js:114:1)
at Transition.performEnter (Transition.js:271:1)
at Transition.updateStatus (Transition.js:237:1)
at Transition.componentDidMount (Transition.js:173:1)
at commitLayoutEffectOnFiber (react-dom.development.js:23305:1)
at commitLayoutMountEffects_complete (react-dom.development.js:24688:1)
at commitLayoutEffects_begin (react-dom.development.js:24674:1)
getTranslateValue @ Slide.js:18
setTranslateValue @ Slide.js:65
(anonymous) @ Slide.js:119
(anonymous) @ Slide.js:114
performEnter @ Transition.js:271
updateStatus @ Transition.js:237
componentDidMount @ Transition.js:173
commitLayoutEffectOnFiber @ react-dom.development.js:23305
commitLayoutMountEffects_complete @ react-dom.development.js:24688
commitLayoutEffects_begin @ react-dom.development.js:24674
commitLayoutEffects @ react-dom.development.js:24612
commitRootImpl @ react-dom.development.js:26823
commitRoot @ react-dom.development.js:26682
finishConcurrentRender @ react-dom.development.js:25981
performConcurrentWorkOnRoot @ react-dom.development.js:25809
workLoop @ scheduler.development.js:266
flushWork @ scheduler.development.js:239
performWorkUntilDeadline @ scheduler.development.js:533

Apologize for delayed response. Please find the code box sample that reproduces the error.
https://codesandbox.io/s/interesting-kate-kpsskw?file=/src/App.tsx

Hi, Any update on the issue. Is it got resolved ?

Hi! The child of the Slide component must be able to hold a ref.
Here's a working version of your example: https://codesandbox.io/s/38141-fjx5tz?file=/src/boardlist.tsx
Let me know if this fixes your issue 😊


Fix egrep warning in /plans/install/docs

The test started to fail. Seems that egrep is obsoleted:
warning: egrep is obsolescent; using grep -E

Let's use the recommendation to get rid of the warning.
Pull Request Checklist

[x] extend the test coverage

/packit test --identifier=full

/packit test -i full


[Bug]: How to set  max_length and  max_new_tokens?

Ubuntu 22.04
Python 3.10
Job: Classificaiton
"provider": "huggingface_pipeline", "tokenzier":"google/gemma-7b",
and I got :
Error generating from LLM: Input length of input_ids is 1658, but `max_length` is set to 20. This can lead to
unexpected behavior. You should consider increasing `max_length` or, better yet, setting `max_new_tokens.
Error generating from LLM: Input length of input_ids is 1646, but `max_length` is set to 20. This can lead to
unexpected behavior. You should consider increasing `max_length` or, better yet, setting `max_new_tokens.
Error generating from LLM: Input length of input_ids is 1645, but `max_length` is set to 20. This can lead to
unexpected behavior. You should consider increasing `max_length` or, better yet, setting `max_new_tokens.

I have the same question.

        "params": {"max_new_tokens": 512,
                    "temperature": 0.1,
                  }

add params in the config file like this.


Move shape_predictor_trainer to a separate header

Moved shape_predictor_trainer to its own header in order to support the
use of shape_predictor with DLIB_ISO_CPP_ONLY defined (i.e. without
the API wrappers).

Sweet, thanks for the PR :)


Not work on machine where net framework 4 was not installed before

ConfuserEx Version: 1.7.0-alpha.7+adcd8e4a22
Target Framework: 4.0
Operating System: Windows 7 Ultimate

Steps to Reproduce:

Install obfuscated app on Windows 7, where Net Framework 4.0 is not installed.
Install Net Framework 4.0 and do reboot (our app dont work without nf4.0)
App crashes if there is something, that need nf 4.0

In logs i see, that for example
System.TypeLoadException: Could not load type 'System.Windows.Input.ICommand' from assembly 'System, Version=<IP_ADDRESS>, Culture=neutral, ...
But if i do the same steps with app without obfuscation, it starts normally
I tried with minimal ofuscatin - only with watermark and result was the same

On ConfuserEx 1.1.0 everything is ok, but in ConfuserEx 1.2.0 problem already exist


Walking improvements: Directions API, Elevations API

is there anything in your TODO list for this?
Was thinking to add a new walk logic to use the map generated by the Directions API for generating a polyline that would need to be walked with the speed that is set.
Additionally for the generated point we could fetch the alt from the Elevations API
this can improve the self.set_position(lat, lng, alt)
maybe this will make the bot walk more natural.
What are your opinions on this?

I like the idea, I was placed a google API there to simulate people walk.

`import polyline
import haversine
import time
from itertools import  chain
class PolylineWalker(object):
def __init__(self, polyline_points, speed):
    """
    :param polyline_points:
    URL = 'https://maps.googleapis.com/maps/api/directions/json?origin=Cham,CH&destination=Baar,CH&mode=walking'
    polyline_points =[x['polyline']['points'] for x in
                      requests.get(URL).json()['routes'][0]['legs'][0]['steps']]
    :param speed:
    """
    self.speed = float(speed)
    self.points = self.get_points(polyline_points)
    self.lat, self.long = self.points[0][0], self.points[0][1]
    self.polyline = self.combine_polylines(self.points)
    self._timestamp = time.time()
    self.is_paused = False
    self._last_paused_timestamp = None
    self._paused_total = 0.0

def reset_timestamps(self):
    self._timestamp = time.time()
    self.is_paused = False
    self._last_paused_timestamp = None
    self._paused_total = 0.0

def get_points(self, polyline_points):
    crd_points = []
    for points in polyline_points:
        crd_points += polyline.decode(points)
    crd_points = [x for n,x in enumerate(crd_points) if x not in crd_points[:n]]
    return crd_points

def combine_polylines(self, points):
    return polyline.encode(points)

def pause(self):
    if not self.is_paused:
        self.is_paused = True
        self._last_paused_timestamp = time.time()

def unpause(self):
    if self.is_paused:
        self.is_paused = False
        self._paused_total += time.time() - self._last_paused_timestamp
        self._last_paused_timestamp = None

def walk_steps(self):
    if self.points:
        return zip(chain([self.points[0]], self.points),
                   chain(self.points, [self.points[-1]]))
    else:
        return []

def walk(self):
    if not self.is_paused:
        walked_distance = 0.0
        time_passed = time.time()
        remaining_points = []
        time_passed_distance = self.speed * abs(time_passed - self._timestamp - self._paused_total)
        for step in self.walk_steps():
            step_distance = haversine.haversine(*step)*1000
            if walked_distance + step_distance >= time_passed_distance:
                if walked_distance > time_passed_distance:
                    percentage_walked = 0.0
                else:
                    if step_distance == 0.0:
                        percentage_walked = 1.0
                    else:
                        percentage_walked = (time_passed_distance - walked_distance)/step_distance
                remaining_points += self.calculate_coord(percentage_walked,*step)
            else:
                percentage_walked = 1.0
            walked_distance += step_distance * percentage_walked
        self.points = remaining_points
        if self.points:
            self.lat, self.long = self.points[0][0], self.points[0][1]
            self.polyline = self.combine_polylines(self.points)
        else:
            self.lat, self.long = None, None
            self.polyline = ''
        self.reset_timestamps()
    return (self.lat, self.long)

def calculate_coord(self, percentage, o, d):
    lat = o[0]+ (d[0] -o[0]) * percentage
    lon = o[1]+ (d[1] -o[1]) * percentage
    return [(lat, lon)]

`
and tester
`
import requests
import time
import haversine
from polyline_walker import PolylineWalker
URL = 'https://maps.googleapis.com/maps/api/directions/json?origin=Baar,CH&destination=Cham,CH&mode=walking'
a = PolylineWalker([x['polyline']['points'] for x in  requests.get(URL).json()['routes'][0]['legs'][0]['steps']], 2.5)
len(a.points)
while a.points:
before = a.points[1:]
now = a.walk()
print(len(a.points), len(a.points))
after = a.points[1:]
print('Remaining ', haversine.haversine(a.points[1], now)*1000, ' m. on current polyline. ', len(a.points), ' polylines remaining.')
print('Estimated completion time: ', (sum([haversine.haversine(*x)*1000 for x in a.walk_steps()]) / a.speed) , ' seconds.')
if before != after:
print('Changed polyline: ', a.polyline)
time.sleep(1.5)
`

got read of the .walk() concept the PolylineWalker() object has a .get_pos() method that returns the current position at call time on the polyline.
the are no methods anymore that update the .points content those remain the same during the life of the object with the initial points from the Google Direction API fetched points.
#667
if the polyline walk was finished the .get_pos() continues to return the position of the last point.
not sure what would be the best way to go...
any proposals/preferences what should the object return?

more improvements #697

more improvements #721

Maybe change sometime a little bit the movements speed. So the bot will look more natural

Good suggestion!
Posible solutions:

one could use the .pause() .unpause() methods to achieve a small speed modulation
I also think one should stop the counter while working on a cell

I did a PR  #835 to push some fixes to master branch

Will need to rewrite some code due to the big refactor to Stepper class #737

routing the generated point to 5 digits precision ensure we are around ~50 cm around the point proposed by the Google Direction API.
I that good enough as a jitter?

fixes in PR #990


ReactiveUI.Avalonia migration

What kind of change does this PR introduce?
Pushes ReactiveUI.Avalonia backend to this repository and updates it to match other backends more.
Additionally, IntegrationTests.Avalonia was added.
What is the current behavior?
Avalonia integration is owned by Avalonia.ReactiveUI package, maintained by Avalonia core team.
Functionally, that package is not up to date with new ReactiveUI features, as well as some existing features implemented inconsistently compared to other backends.
What is the new behavior?
New ReactiveUI.Avalonia package is going to be created, with code in this repository.
This PR also already adds some features, that were missing previously:

[x] ICreatesObservableForProperty is a low hanging fruit

And some are pending:

[ ] ICreatesCommandBinding - does it make sense to have Avalonia specific implementation? From WinForms implementation, I don't see a benefit.
[ ] IPlatformOperations - how this one is used? Avalonia 11.2 got Screen.Orientation API, which we can use, but it requires a window/toplevel to get screen from, as there can be multiple. Or we can assume main window from global statics.

What might this PR break?
Users who currently use Avalonia.ReactiveUI. NuGet package redirects should be created, and users should be notified.
Open questions

Testing: Avalonia had ReactiveUI specific tests in the core repository: https://github.com/AvaloniaUI/Avalonia/tree/master/tests/Avalonia.ReactiveUI.UnitTests. It would make sense to move them as well, or even better integrate into existing testing projects. From the first look, there is some connection in ReactiveUI.Tests project, but these only run for windows frameworks? How would I add Avalonia implementation there?
Initialization: Avalonia.ReactiveUI always had .UseReactiveUI() app builder method, it's also used in our ReactiveUI templates. It is different from other backends, which provide IWantsToRegisterStuff implementation instead. My question is, would it be preferable to do the same in Avalonia? My only concern is trimmability of library, how is it handled in other backends?

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

cc @glennawatson @ChrisPulman

how this one is used? Avalonia 11.2 got Screen.Orientation API, which we can use, but it requires a window/toplevel to get screen from, as there can be multiple. Or we can assume main window from global statics.

This is used in RoutedViewHost on mobile platforms - on Desktop it should follow what other desktop platforms do (probably return a constant); in general, don't worry about this too much

ICreatesCommandBinding

We probably don't need it, but it would be nice to get BindCommand working on Avalonia, afaik at the moment it does not.

Users who currently use Avalonia.ReactiveUI. NuGet package redirects should be created, and users should be notified.

"Avalonia.ReactiveUI is deprecated, please move to ReactiveUI.Avalonia" is a phrase no user of either of our platforms should ever have to think about because it is pants-on-head silly. Can we make ".UseReactiveUI" in Avalonia.ReactiveUI and make it depend on ReactiveUI.Avalonia?

Also nuget has an automatic redirect functionality to at least point the user.


Make Script.GetDestination() support P2PK script

This will make Script.GetDestinationAddress() success with P2PK script. The reason I need this because I need to get address from the early mining reward, which is P2PK instead of P2PKH (e.g. https://bchain.info/BTC/tx/0e3e2357e806b6cdb1f70b54c3a3a17b6714ee1f0e68bebb44a74b1efd512098).

Can you guys tell me which tests that fail on Travis? The logging is so hard to find which on is fail. Or at least give me how to locate failed tests.

Search for xUnit.
Anyway NACK for this PR, because the block explorer is wrong a P2PK does not have address, only P2PKH are.

I take a look at how decoderawtransaction works for P2PK and found out it create P2PKH from public key and use it to get the address. Do you mean the address it show in decoderawtransaction is not a valid address?

Yes, this is a bug.
If you send to this address it will not give you a P2PK scriptPubKey.

Hopefully I removed the bitcoind logs. (https://github.com/MetacoSA/NBitcoin/commit/19347c1116595a4c31c918b1b797e89a0a8f75beb)

I just improved test feedback! https://travis-ci.org/MetacoSA/NBitcoin/jobs/542814000

The commits is inaccessible for me. That does not look right for me since the address it show for P2PK output in decoderawtransaction will be effectively the same address as if that output is P2PKH since it get KeyId from the public key in the same way. That mean if somebody spend to that address the owner should be able to spend that output.

That mean if somebody spend to that address the owner should be able to spend that output.

By the same logic, 1 <pubkey> 1 MULTISIG should also have the same address?

Also, script.GetDestinationAddress().ScriptPubKey == script is a strong invariant that lot's of people depends on. Breaking this would break their code.

I will take a look more on Monday. Thanks for information.

I don't know about multi-sig but this PR will handle only P2PK script. Okay I agree with your last comment.

My point is that if you consider:

That mean if somebody spend to that address the owner should be able to spend that output.

To be a good argument, then it means that any scriptPubKeys which satisfy the same condition should show a bitcoin address. However, this is impossible to make code which can detect that condition automatically for any arbitrary scriptPubKey.

What do you mean by that? Current GetDestination support only P2SH, P2PKH and P2W so when the script is multi-sign it should return null. With this PR is will also support P2PK but not for multi-sig. But the problems is it might unexpected by user that when they call GetDestinationAddress for P2PK script but they got P2PKH address (and P2PKH script from ScriptPubKey of that address).

Yes, that's the problem. My point is that if you add support for P2PK why not adding support for 1 A 1 MULTISIG script as well?

I will take a look more for solutions when I have time. Thanks for discussion.

@ultimaweapon no no don't do it, you can't find solution. There is an infinity number of script that can be signed by the same pubkey. This was just to explain you that your argument for adding P2PK support can't hold.

What about adding a new property to tell if script is P2PK and let the caller do the rest?

You can already do this
PubKey key = NBitcoin.PayToPubkeyTemplate.Instance.ExtractScriptPubKeyParameters(scriptPubKey);
if (key != null) // This is PK
{ }

But seriously, don't spend time on this P2PK is not used anymore by anybody.

Yes I can do that but would it be better to have IsPayToPubKey due to it already have IsPayToScriptHash and IsWitness? It may not useful if using NBitcoin just for creating transactions but it useful when someone want to parse legacy blocks.

@ultimaweapon I think this is a good idea.
However, what about this instead:

Make IsPayToScriptHash and IsWitness obsolete
Add new Script.IsScriptType(ScriptType) where ScriptType is an enum with Witness, P2PKH, P2SH, P2PK, P2WPKH, P2WSH, MultiSig?

So it is easy in the future to add new types.

Look good to me. I will do it. Should done within this week.


arco init 初始化项目时提示eslint依赖错误

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info

Package Name And Version<EMAIL_ADDRESS>Browser: chrome<IP_ADDRESS>

Extra info
? Please select the framework you want to use Vue
? Please select the type of project you want to create Arco Pro Project
? Please choose the template of Arco Pro Full version (contains all pages)
✔ Get project template successfully
✔ Template content copy completed
✔ Template content adaptation completed
✖ Project dependency installation failed, you can try to manually install them later
Error: Command 「npm install」 executed failed:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/eslint
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^6.8.0 || ^7.2.0" from<EMAIL_ADDRESS>npm ERR! node_modules/eslint-config-airbnb-base
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

<EMAIL_ADDRESS>改为<EMAIL_ADDRESS>npm install 就正常了

我把package.json中的"eslint-config-airbnb-base": "^14.2.1",改为"eslint-config-airbnb-base": "^15.0.0",之后npm install就不报错了


chore: drop support for PHP 7.4

The 7.X tests are still running, Should we remove them? What is the protocol here @bshaffer

README also needs to be changed:
https://github.com/googleapis/google-api-php-client/blob/7e79f3d7be4811f760e19cc4a2c558e04196ec1d/README.md?plain=1#L28

@Hectorhammett that is happening because php7.4 is mentioned in the repo settings. It can be fixed by removing it, but only admin can see/do this change.


Search 'Alternative DNS' or 'Alternative root DNS' in chrome plugin store does not return this plugin.

Please consider modifying the plugin introduction so that more people can know the plugin's existence.

Good idea, I have added this paragraph to the plugin's description:
A doorway into alternative DNS roots.

Now Chrome Store returns BDNS if searching for alternative dns.


Enable docker profile in CI for log4j-layout-template-json-test

See #2953 for the 2.x port.

I am wondering about a couple of things:

Should we activate the docker profile by default on machines that have /var/run/docker.sock? Or maybe we should ask the reviewers of a release to activate docker in their Maven settings if they have Docker installed?
The Docker image versions in these tests are somehow dated, but unfortunately Elastic Search does not provide a latest tag. Should we add org.elastisearch:elasticsearch to the dependency management of this module, so Dependabot can update the version regularly? My guess is that the Docker images are published shortly after the elasticsearch artifact is out.

Should we activate the docker profile by default on machines that have /var/run/docker.sock?

I think we should do neither. In particular, bootstrapping an ELK stack is a pretty resource intensive process, I doubt if it will succeed without an OOM on every developer machine.

Or maybe we should ask the reviewers of a release to activate docker in their Maven settings if they have Docker installed?

I don't think we should, but I will update the review kit to reflect on the docker Maven profile – reviewers can choose for themselves.

The Docker image versions in these tests are somehow dated, but unfortunately Elastic Search does not provide a latest tag. Should we add org.elastisearch:elasticsearch to the dependency management of this module, so Dependabot can update the version regularly? My guess is that the Docker images are published shortly after the elasticsearch artifact is out.

Addressed this issue.


Create welcome-to-2nd-line.html.md

Transferring this info over from the wiki. It's had a content edit from myself, 2i from content friends and technical proofing from Kevin Dew.

Whoops, forgot to give the 3rd commit a title... sorry!


Improve the UI of the RATE US Page

The fonts and colours are not entirely appealing. The overall User Experience can be improved.
I would like to apply colour theory to propose a better page overall and use better font styles. The header can also be improved. Kindly assign this task to me so that I can work on it.

I am a frontend web developer and can design attractive UI designs. The rate us section seems to be not so appealing but I can make it catchy by implementing some good UI. So kindly assign this task to me as a part of GSSOC'24.

can you please label the issue

I want to work on this issue. Please assign me this issue.

Hey PA, I would like to work on this issue. Please assign it to me. Thankyou!

@rhea164 I have assigned you . You can Start Working
Welcome to the issue

@rhea164 I have assigned you . You can Start Working Welcome to the issue Are you contributing under GSSOC'24??

yes I am contributing under GSSOC'24. I will start working on the issue

I have submitted a PR regarding this issue.


iobroker can not install latest senec adapter

Clean fresh and updated iobroker installation. While adding senec adapter in iobroker console it seems that version 0.01 is getting installed. The config menu looks pretty old compared to the one located in the readme.
Any help is appreciated.

Suppose you got the wrong adapter.
Install via iobroker via interface using "latest" repository.

Custom install solved the issue.


Log4Shell not detected?

We have definitely a vulnerable dependency in our project:
./gradlew application:dependencies --configuration compileClassPath | grep "log4j-api" -B 10

+--- org.springframework.boot:spring-boot-starter-actuator -> 2.5.5
|  +--- org.springframework.boot:spring-boot-starter:2.5.5
|  |  +--- org.springframework.boot:spring-boot:2.5.5 (*)
|  |  +--- org.springframework.boot:spring-boot-autoconfigure:2.5.5 (*)
|  |  +--- org.springframework.boot:spring-boot-starter-logging:2.5.5
|  |  |  +--- ch.qos.logback:logback-classic:1.2.6
|  |  |  |  +--- ch.qos.logback:logback-core:1.2.6
|  |  |  |  \--- org.slf4j:slf4j-api:1.7.32
|  |  |  +--- org.apache.logging.log4j:log4j-to-slf4j:2.14.1
|  |  |  |  +--- org.slf4j:slf4j-api:1.7.25 -> 1.7.32
|  |  |  |  \--- org.apache.logging.log4j:log4j-api:2.14.1       <------ HERE

Still I get:
./gradlew dependencyCheckAnalyze
...
Found 0 vulnerabilities in project middleware

Why?

The vulnerability is in log4j-core, not log4j-api. You're using Logback for logging there.

Yep thanks, I closed the issue immediately

but I think log4j-core is not scored at the moment, so there is also no hit.

Now NIST has update the score to 10 - it should match now


2.9.0 breaks on Node.js 8

2.9.0, apparently due to #370, does not work on Node.js 8. This breaks my build on Debian 9 and Ubuntu 18.04.
node --version
v8.10.0

node
> var needle = require('needle');
undefined
> needle.get('http://www.google.com', function(error, response) {
...   if (!error && response.statusCode == 200)
...     console.log(response.body);
... });
PassThrough { ... }
> TypeError: Cannot read property 'apply' of undefined

If this was intentional, could the engines in package.json be updated and a major version bump be made?
(I am not using Needle myself, it appears to be pulled in as a dependency of sqlite3 → node-pre-gyp.)

I can confirm the bug with node v7.10.1 (in a buildsystem Yocto, so it's not easy to fix - we will need to pin a specific older version)

Confirmed, the example I tried seems to work again with node v8.10.0 on Ubuntu 18.04 with master. Thanks! Unless there are any other concerns about it, I would appreciate if this could be released as soon as possible.

Already pushed v3.0.0 to npm

Hmm, but that doesn’t help me, because it means that a package like node-pre-gyp (my sub-dependency) that requires "needle": "^2.2.1" will remain broken. Could we have a 2.9.1 or 2.10.0 that is fixed (and does not have whatever breaking change it is that is implied by the major version bump to 3.0.0)? If nothing else, simply equal to 2.8.0 or to 2.9.0 with #370 reverted?

I'm not sure if its possible to push a version number lower than the already published version, but if possible then sure, I can publish the fix as v2.9.1.

Any progress? Our builds are still failing…
(Yes, it’s our own fault that the build is that fragile and we should be pinning our dependency versions using package-lock.json or npm-shrinkwrap.json, but last time I tried that it caused other problems, and I just don’t have time right now to spend a lot of effort on our Node.js stuff, so I was hoping the problem could be solved as quickly by an upstream update as it was caused by an upstream update, without me having to divert effort from more important things.)

@cwalther Just published v2.9.1 with the fix.
Maybe you can buy me a coffee sometime. :)

@tomas - fwiw: I'm not seeing v2.9.1 in either the releases or tagged versions.
(And if you have a buy me a coffee, Patreon, or a GitHub Supportee link, I can't find it.)

I didn't push that commit to master since it would mess up the history, but you can find the version at npm.
I just pushed a v2.9.1 branch if you want to see the commit.

Success! Our builds are green again (Debian 9, Ubuntu 16.04, Ubuntu 18.04). Thank you!
I see that you now have 2.9.1 tagged as latest on npm. I think that could be changed back to 3.0.0 without breaking anything, if you prefer.
Then again, it seems like the only difference between 3.0.0 and 2.9.1 is the version number, which confuses me a little – was there no API-breaking change after all? Not that it matters to me though, as I’m not using the Needle API, I’m just a down-down-downstream consumer.

would be nice if you could tag 3.0.0 as latest on npm, since my dependency management pesters me to downgrade to the latest version 2.9.1 now.

There doesn't seem a way to update the latest tag, so I'll have to publish a new version (v3.0.1). It's the exact same code, though, so it won't make much difference if you install v2.9.1.

Actually, I just managed to update the latest tag so it points to v3.0.0.

I also just opened a page on buymeacoffee!
https://www.buymeacoffee.com/tomaspollak


Backwards improvement of docstrings for EpiModels

Goal
Bring improvements in doc string clarity into AbstractEpiModel subtype documentation.
My view is that the doc string layout for AbstractLatentModel subtypes and method dispatching on them was improved compared to #138. This PR aims to include backwards improvement in style for the structs/methods documented in #138 .

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.60%. Comparing base (3b2d479) to head (085c4b9).
Report is 8 commits behind head on 139-document-all-latent-models-and-methods-that-dispatch-on-them-plus-adding-keyword-args-as-required.

Additional details and impacted files
@@                                                            Coverage Diff                                                            @@
##           139-document-all-latent-models-and-methods-that-dispatch-on-them-plus-adding-keyword-args-as-required     #141      +/-   ##
=========================================================================================================================================
- Coverage                                                                                97.61%   97.60%   -0.02%
=========================================================================================================================================
  Files                                                                                13       13
  Lines                                                                                168      167       -1
=========================================================================================================================================
- Hits                                                                                164      163       -1
  Misses                                                                                4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Doc changes look reasonable but the outstanding is merge conflicts due to the change in submodules and git losing track of the folders on this PR.


Provider client iterator

Rebased on #45
Refactor provider client to allow consumers to iterate through pages, rather than waiting for all pages before continuing.
Seeks to maintain ProviderClient interface without substantive change.

I really like this idea, haven't dug into the code too much yet so I don't fully understand what's going on...
I just wanted to point you to #35, where we're discussing actually removing the whole "pass multiple providers into a ProviderClient" concept, in which case SingleProviderClient is a bit redundant. Do you have any thoughts on that approach?

Cool, just skimmed that issue and I believe I agree. Based on the name I would expect ProviderClient to be a client to a single provider.

I renamed the client classes in this PR

@ezheidtmann just leaving these comments here as well...
I'd like to take the approach suggested in #35, to clean up the existing class rather than renaming and introducing new classes. Did you want to refactor this PR towards this approach?

I believe that my latest push does that in ProviderClient, and also provides a migration path for consumers in MultipleProviderClient. That said, I'm happy to revise again if you can provide more details on how the latest push doesn't meet your goals.

Thank you. I think what I'm getting at is, I'd rather not introduce a new class and support a migration path so early in the game. Refactoring the existing ProviderClient, event breaking to some extent (removing support for multiple providers) is totally OK.

Got it, thanks for that direction. I'll work on a breaking change with an iterator-based API over the next few days; I pushed some early thinking in #45

Hey @ezheidtmann. The latest release has evolved the library to the point that I think this would 1) be much easier to implement, 2) need to be re-done anyway.
Closing this PR, but would welcome a new one. Thank you!


Create Simple UI for the Payment logic

"The value of a prototype is in the education it gives you, not in the code itself." - Alan Cooper

[ ] cover all integration tests (https://github.com/bitcoinjs/bitcoinjs-lib/blob/master/test/integration)

[x] clasify script type (waiting for merge of PR #1618)

[ ] ins/outs pagination vs list

[x] to Hex (+- witness)

[x] use toasty for Error message

[x] close pop-up on escape

[x] handle is coninbase

[ ] SIGHASH_* <- add support

[ ] detail hex fields (?)

add "Wrapp To P2SH" button when possible

[ ] add Copy to Clipboard icon next to ASM scripts

[x] Sequence to Hex -> on change

[ ] add info buttons to transaction fields

[x] add Network overall

[x] 3 witness fields -> find bug in bitcoinjs

browserify src/index.js --s bitcoinjs  > ui/lib/bitcoinjs-lib.js

[ ] move remove input out of transaction-input

[ ] p2sh: redeem witness  VS payment witness

[ ] why p2sh was witness fields?

sing Next/Prev

https://www.w3schools.com/howto/howto_js_navbar_slide.asp

Test TX:
010000000200000000000000000000000000000000000000000000000000000000000000000000000000ffffffff00000000000000000000000000000000000000000000000000000000000000000000000000ffffffff01e8030000000000000000000000


Bump CI images to use latest Weaviate release

Codecov Report
Patch coverage: 86.89% and project coverage change: +0.11% :tada:

Comparison is base (eb3bf16) 95.43% compared to head (5d98bda) 95.54%.
Report is 48 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #428      +/-   ##
==========================================
+ Coverage   95.43%   95.54%   +0.11%
==========================================
  Files          64       64
  Lines        8165     8304     +139
==========================================
+ Hits         7792     7934     +142
+ Misses        373      370       -3

Files Changed
Coverage Δ

test/batch/test_crud_batch.py
99.11% <ø> (-0.12%)
:arrow_down:

weaviate/batch/crud_batch.py
90.42% <57.69%> (-4.90%)
:arrow_down:

weaviate/schema/crud_schema.py
78.66% <86.36%> (+0.51%)
:arrow_up:

weaviate/gql/filter.py
92.96% <87.09%> (-1.39%)
:arrow_down:

integration/test_schema.py
97.75% <93.54%> (-2.25%)
:arrow_down:

integration/test_batch.py
98.03% <100.00%> (ø)

integration/test_cluster.py
100.00% <100.00%> (ø)

integration/test_graphql.py
94.08% <100.00%> (+0.44%)
:arrow_up:

integration/test_stress.py
87.09% <100.00%> (ø)

test/gql/test_filter.py
100.00% <100.00%> (ø)

... and 4 more

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


AP5GC docs should link to the 'single node' version of ASE deployment instructions.

Please change the links in "Order and set up your Azure Stack Edge Pro device(s)" to link specifically to the 'single-node' view, as shown in the example below.
Otherwise the user could end up viewing the "two node cluster" documentation, which is incorrect for AP5GC (and unsupported / confusing).  This happened to me today.
Example of how it should look with the single-node pivot selected:
https://learn.microsoft.com/en-us/azure/databox-online/azure-stack-edge-gpu-deploy-install?pivots=single-node
Also remove the superfluous "?tabs=azure-portal" URL parameter in the second link which I don't think is doing anything?
Thanks
Rob

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

ID: 73a16fd4-712b-11f5-88ad-b093cb9cec13
Version Independent ID: 814c890e-6d14-5059-1f0e-1988466b788f
Content: Prepare to deploy a private mobile network - Azure Private 5G Core
Content Source: articles/private-5g-core/complete-private-mobile-network-prerequisites.md
Service: private-5g-core
GitHub Login: @djrmetaswitch
Microsoft Alias: drichards

@robertstarling
Thanks for your feedback! We will investigate and update as appropriate.

#reassign:robswain

#please-close


Initial merge + fix for 1.4

This also contains changes to the Visual Studio project settings so the might have to be excluded from the merge
I'm on VS2022 and haven't used it in a while so I'm getting relearning stuff.

Thanks Deno226 for your contribution. I used your fix for the Combat Extended integration (but can't test it in 1.4) since CE hasn't been updated yet.


android: Various fixes to get closer to Android on CMake

Remove vulkaninfo from Android builds
Don't use the loader on Android
Various cleanup in cube/CMakeLists.txt

CI Vulkan-Tools build queued with queue ID 52571.

CI Vulkan-Tools build # 1173 running.

Still WIP but these changes are good as is

CI Vulkan-Tools build queued with queue ID 52585.

CI Vulkan-Tools build # 1174 running.

CI Vulkan-Tools build # 1174 failed.

CI Vulkan-Tools build queued with queue ID 52605.

CI Vulkan-Tools build # 1175 running.

CI Vulkan-Tools build # 1175 failed.

CI Vulkan-Tools build queued with queue ID 52667.

CI Vulkan-Tools build # 1176 running.


How to improve the models - cleaning the data

🚀 The feature
Hello all,
It is very likely that the data you use to train text recognition models has errors that when training on it prevent higher model performance. I know that the data on which you train the models is private and well protected. That's why I would like to offer you a script that would correct these errors by yourself. In view of the amount of important data you have at your disposal to train your models, this would even achieve higher performance than what the benchmarks of the models were made.
The principle is based on two things: the predictions of the models with their associated labels. To put it simply, if we order the predictions in descending order of score predictions and observe in the same order the labels considered incorrect, this would highlight the poorly labeled data that have the most negative impact on model learning.
So, if you agree it would be with great pleasure to work on this script to help you get better quality data.
Motivation, pitch
correction of labels in data allows better models performances
Alternatives
No response
Additional context
No response

@odulcy-mindee @charlesmindee
Looks like that's more a discussion topic


No onSendProgress callback for PUT method

  Response response = await dio.put(url,           data: Stream.fromIterable(fileData.map((e) => [e])),           options: Options(             headers: headers,           ), onSendProgress: (int sent, int total) {         print("$sent $total");       }, onReceiveProgress: (int recev, int total) {         print("$recev $total");       });
nothing happened in onSendProgress callback. any suggestion?

dio: 3.0.8
flutter: v1.12.13+hotfix.8
platform: iOS & Android

Any update on this.  It is also happening to me

Same issues to me, onSendProgress executed with Post method, but noting happed with Put method


fix iOS右滑前进再次执行动画

Please makes sure the items are checked before submitting your PR, thank you!

[ ] Rebase before creating a PR to keep commit history clear.
[ ] Only One commit
[ ] No eslint errors

Codecov Report

Merging #2963 into v2 will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##               v2    #2963      +/-   ##
==========================================
- Coverage   19.43%   19.32%   -0.11%
==========================================
  Files         226      226
  Lines        5603     5603
  Branches      710      710
==========================================
- Hits         1089     1083       -6
- Misses       4393     4400       +7
+ Partials      121      120       -1

Impacted Files
Coverage Δ

src/components/x-img/index.vue
20% <0%> (-30%)
: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 e531036...07386c5. Read the comment docs.


Use template literal type as object key

Suggestion
Use template literal type as a key in object
🔍 Search Terms
template literal string as object key
✅ Viability Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
Hello, I have a problem and maybe this thing will be a good feature. I would like to use template literal types as a key in for example interface. Let me show an example.
📃 Motivating Example
type ActionStringType<S extends string, D extends string> = string extends S
    ? ""
    : S extends ""
    ? ""
    : S extends `${infer T}${D}${infer U}`
    ? `${Lowercase<T>}${Capitalize<ActionStringType<U, D>>}`
    : Capitalize<Lowercase<S>>

const createRequestAction = <T extends string>(name: T) => {
    const actionName = name
        .split("_")
        .map((item, i) => {
            if (i === 0) {
                return item.toLowerCase()
            }
            return item.charAt(0).toUpperCase() + item.slice(1).toLowerCase()
        })
        .join("") as ActionStringType<T, "_">
    return {
        [`${actionName}Success`]: createActionCreator(`${name}_SUCCESS`),
        [`${actionName}Error`]: createActionCreator(`${name}_ERROR`),
        [`${actionName}Done`]: createActionCreator(`${name}_DONE`),
    }
}
// createActionCreator is a function from 'deox' npm package

I've made a ActionStringType and for example:
type S1 = ActionStringType<"SET_AS_FAVOURITE">
// returns "setAsFavourite" which is good

I was trying to make a action generator in redux and I found a problem where I can't type keys in object that createRequestAction returns. So this type is working fine but TS is still not seeing the names (keys of object) of action creators.
And for example now something like ...createRequestAction("SET_AS_FAVOURITE") returns:
const createRequestAction: <"SET_AS_FAVOURITE">(name: "SET_AS_FAVOURITE") => {
    [x: string]: ExactActionCreator<"SET_AS_FAVOURITE_SUCCESS", () => {
        type: "SET_AS_FAVOURITE_SUCCESS";
    }> | ExactActionCreator<"SET_AS_FAVOURITE_ERROR", () => {
        ...;
    }> | ExactActionCreator<...>;
}

I want to change this [x: string] to something like S1 type returns.
So, that is my idea, idk how to implement this, maybe it's impossible for now, but i think it will be a good feature. Let me know your thoughts, maybe I'm doing something wrong :)
💻 Use Cases

creating action generators etc.
typing keys in object

Looks like a duplicate of #13948 and/or #21030.   Your example is kind of complicated for what amounts to a demonstration of using a computed key of a template literal type, but if I use the "unionize" type from here to distribute records over unions, like this:
type DistribRecord<K extends PropertyKey, V> = { [P in K]: { [Q in P]: V } }[K]
const kv = <K extends PropertyKey, V>(k: K, v: V): DistribRecord<K, V> => ({ [k]: v } as any)

Then instead of {[foo]: bar} we can write kv(foo, bar) to get the behavior you want.  Your return value would probably look like
  return {
    ...kv(`${actionName}Success`, createActionCreator(`${name}_SUCCESS`)),
    ...kv(`${actionName}Error`, createActionCreator(`${name}_ERROR`)),
    ...kv(`${actionName}Done`, createActionCreator(`${name}_DONE`)),
  }

And then you get
const x = createRequestAction("SET_AS_FAVOURITE")
x.setAsFavouriteDone // ExactActionCreator<"SET_AS_FAVOURITE_DONE", () => { type: "SET_AS_FAVOURITE_DONE"; }>
x.setAsFavouriteError // ExactActionCreator<"SET_AS_FAVOURITE_ERROR", () => { type: "SET_AS_FAVOURITE_ERROR"; }>

Playground link

Hi, yes, that's that I'am talking about. For now I figured out this problem like this:
type Types = "Success" | "Error" | "Done"

type SnakeToCamelCase<S extends string> = S extends `${infer T}_${infer U}` ? `${Lowercase<T>}${Capitalize<SnakeToCamelCase<U>>}` : Lowercase<S>

type Output<T extends string> = {
    [key in `${SnakeToCamelCase<T>}${Types}`]: ExactActionCreator<
        T,
        () => {
            type: T
            payload: any
        }
    >
}

And function looks like this:
const createDeoxRequestAction = <T extends string>(name: T): Output<T> => {
    const actionName = name
        .split("_")
        .map((item, i) => {
            if (i === 0) {
                return item.toLowerCase()
            }
            return item.charAt(0).toUpperCase() + item.slice(1).toLowerCase()
        })
        .join("") as SnakeToCamelCase<T>
    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
    return {
        [`${actionName}Success`]: createActionCreator(`${name}_SUCCESS`),
        [`${actionName}Error`]: createActionCreator(`${name}_ERROR`),
        [`${actionName}Done`]: createActionCreator(`${name}_DONE`),
        // eslint-disable-next-line @typescript-eslint/no-explicit-any
    } as any
}

And that work properly, so if it's duplicated topic - I'm closing :)
Thanks for your help @jcalz ;)


Detox/E2E: Convert to private channel e2e tests in Gekidou

Summary

Added e2e for channels convert to private channel
Created associated zephyr test cases under Mobile V2 folder

Mobile V2/Channels:
MM-T4972 - Convert to Private Channel
NOTE: I will need to update screenshot and branch from gekidou once this PR is merged https://github.com/mattermost/mattermost-mobile/pull/6562. For now, testing will fail.
Ticket Link
https://mattermost.atlassian.net/browse/MM-46303
Screenshots
TBD
Release Note
NONE

/update-branch


 Did you mean to just keep one?

Yes, I’ll fix that, @knz. But I’m still also trying  I figure out how to get the script to recognize all unused images on one run.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b953b32ad26c080a5b61d354f44cf376c39c368c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/431bd69c2e9243aed7721d8f1381367398b7a55c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/431bd69c2e9243aed7721d8f1381367398b7a55c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/25496bcd31cd580faaff9ba97ce27ac706739438/


Bug 1922421:[release-4.6]Cherry pick 1113 to release 4.6

This PR is cherry-pick of #1113

Tested this on an IBM cluster

/ok-to-test

/hold
We might have to take this in 4.6.5 instead of 4.6.4 because of some other urgency. Will provide more details later.

But we can finish the review and keep it ready in case.

/test ocs-operator-ci

/retitle Bug 1937837:release-4.6]Cherry pick 1113 to release 4.6

/bugzilla refresh

This fix can now be merged in 4.6

/bugzilla refresh

/unhold
/lgtm


Make local installation of vcpkg use the appropriate vcpkg root

/azp run c - client - tests

This is a very old PR, is it still in progress?  Please commit or close.


BorderLine case - Repository with setup.cfg as a configuration file - rtdl

The rtdl repository has a setup.cfg, but it is not used. Should we annotated it as a library, or as script? The repo classifies itself as a library, but looking the README it behaves more a set of scripts.  Since the repo has a setup.cfg (but without an entry point), code_inspector classifies it as library.

I think it's used as a script, and hence it should be a script. We should explore if the setup.cfg can identify something to distinguish library from script.


links to notes inside quotes doesn't work

If you're reporting an UI issue, make sure you take a screenshot that shows the actual bug.

This was reported on Obsidian forum
https://forum.obsidian.md/t/doesnt-work-inside-quotes-in-some-themes/626/6
Basically link to notes doesnt work  inside the quotes

[[`
cycle of research
]]

The screenshot is a different theme but the same problem exist in Dracula :(
Obsidian 0.6.4 on Mac

thanks for reporting ,it has been fixed now.


Where can I change the Greeting message and make the reply anonymous

RTFM
v / r
Robert Klebes
Mobile:<PHONE_NUMBER>
https://keybase.io/sporkwitch
https://www.linkedin.com/in/klebes
On March 3, 2020 9:16:11 PM EST, playapotner<EMAIL_ADDRESS>wrote:

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/Dragory/modmailbot/issues/376

Where is the manual?

The manual is linked on the front page of this repository:
https://github.com/Dragory/modmailbot/blob/master/docs/configuration.md
The option you're looking for is greetingMessage and the !ar command to send anonymous replies.


[FIX] .count() ... WIP

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3348  👈 (View in Graphite)
#3342 : 3 other dependent PRs (#3344 , #3345 , #3346 )
#3343 : 2 other dependent PRs (#3309 , #3326 )
#3341
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Headings should not be indented with section body

In current realization (Asciidoctor PDF 1.5.0.beta.6 using Asciidoctor 2.0.10) indent of section is applied for subsection headings. It should only be applied on content of section and it subsections. Section titles and discrete headings should not be indented (see image for example -- "Overview" is heading of level 1, and "Product description" is heading of level 2).

Thanks for reporting!

The indent should be applied once to the body. Then the indent should be undone when laying out any section title or discrete heading.

I think this will be pretty easy to fix. Unfortunately, it means that as it is currently implemented, the section indent is quite broken. And that's the result of missing tests for the scenarios cited.

I would imagine that the indent should also be undo when placing footnotes.

I think I've managed to get this working. PR on the way.
After doing some research, I also determined that the following non-heading content should not be indented:

index body
footnotes
abstract

(If other cases come up, we can address them individually).

So, if i update it should be working?

It hasn't been released yet.


Spike: Investigate how to handle different languages and versions for chimera

If qiskit.org starts to pull in documentation pages it will also need to handle different languages and qiskit release versions like the current documentation.
I have no idea where to begin with sorting this out.
Track investigation here

We decided to not pursue Chimera for now: https://github.com/Qiskit/qiskit_sphinx_theme/issues/129#issuecomment-1490903514
We can reopen this if we change our mind.


Unable to set default environment

Describe the bug

Unable to set default environment
Steps to reproduce
https://github.com/seal-io/walrus/assets/26084756/6db949f0-3061-4b8e-88e4-f151c93f5068
Expected behavior

Environment

Walrus version: main 30ceb5ac9dcaade04505b64a3f6e6ef3bc1c80b2
UI version: dev-1305100

Additional context
Add any other context about the problem here.

Environment

Walrus version: main 7bb79389ab0436969f4c4214e38918ee6c6e9070
UI version: dev-1be55ba

Test Result: pass


TEXT type for MySQL dialect

Situation
MySQL knows the type TEXT which can hold large amounts of text. Exasol's closest match is VARCHAR. At least documents up to 2 million characters should be usable through the MySQL dialect for Exasol Virtual Schema.
Acceptance Criteria

UTF text from MySQL TEXT columns visible as VARCHAR in VS (up to maximum Exasol VARCHAR size).

Looking at the code the BaseColumnMetadataReader already handles LONGVARCHAR correctly. And that is what the MySQL JDBC driver maps TEXT fields to.
See:

https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-reference-type-conversions.html
https://dev.mysql.com/doc/refman/8.0/en/storage-requirements.html


Make type checking enabled only in debug mode by default

Type checking and the debug mode are often used for debugging. The former checks the shape and dtype of inputs to each Function, while the latter, for example, checks if NaN appears in the intermediate results (including the backward path). While the former is enabled by default, the latter is disabled, which makes the best practice of taking the trade-off between the performance and debugging capability a bit complicated. I want to align the behaviors and make type checking disabled by default and enabled by enabling the debug mode (or make it enabled by default and disabled by disabling the debug mode).

Do you have any consideration about the relation with Light-weight type checking #1692?

Close since no update is made for a long time. #2136 would be helpful.


Chapter 1-3 suggestions

I've made some suggestions that I think makes the code easier to read and a feel more swifty

Don't agree with all of these changes, so won't be merging this pull request, but I will be incorporating some of them piecemeal. Thank you for the suggestions!


WFCORE-4924: add plain-text option to security command

Issue: WFCORE-4924

@darranl , looks good to me. yes there is a community doc but is generic enough tonot have to update it: https://github.com/wildfly/wildfly/blob/master/docs/src/main/asciidoc/_admin-guide/CLI_Recipes.adoc#security-commands
We are suggesting to use the help to discover the options one can use.

Thanks you @jfdenise will see if I can include this in a second round of merges later today.


Publish release as a verified publisher

Client library is published on https://pub.dev/packages/influxdb_client under unverified uploader.
Publisher verification process described here: https://dart.dev/tools/pub/verified-publishers

Looks like we need to validate via a DNS record change that I will need to get help with creating. I was working through https://pub.dev/create-publisher

I was trying to implement in Web Localhost but it is not working throwing different error please help me out


Add quick action menu in chats and threads

Updated dropdown to quick action menu in chatrooms (long click on notes)
Added long click & quick action menu to the top note (NoteMaster) of a thread

We might need to revert this. Three people already complained about the lack of report in the quick action

Interesting. I could just add the report screen?


html compatible for script tags

fix(#83)

Current coverage is 91.59% (diff: 100%)

No coverage report found for master at 26d0bc6.

Powered by Codecov. Last update 26d0bc6...15b6e85

:+1: