error with cmake

The error is happened while me install bzip2 and boost!
I'am not know cmake's Grammar rules。So...you know it!
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:43 (message):
Command failed: D:/Tools/Cmake/bin/cmake.exe;D:/ThridLibraryPackages/buildtrees/bzip2/src/bzip2-1.0.6;-DBUILD_SHARED_LIBS=ON;-DVCPKG_TARGET_TRIPLET=x86-windows;-DCMAKE_CXX_FLAGS= /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP ;-DCMAKE_C_FLAGS= /DWIN32 /D_WINDOWS /W3 /utf-8 /MP ;-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON;-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON;-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON;-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE;-DCMAKE_VERBOSE_MAKEFILE=ON;-DVCPKG_APPLOCAL_DEPS=OFF;-DCMAKE_TOOLCHAIN_FILE=D:/ThridLibraryPackages/scripts/buildsystems/vcpkg.cmake;-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON;-DCMAKE_CXX_FLAGS_RELEASE=/MD /O2 /Oi /Gy /DNDEBUG /Zi ;-DCMAKE_C_FLAGS_RELEASE=/MD /O2 /Oi /Gy /DNDEBUG /Zi ;-DCMAKE_SHARED_LINKER_FLAGS_RELEASE=/DEBUG /INCREMENTAL:NO /OPT:REF /OPT:ICF ;-DCMAKE_EXE_LINKER_FLAGS_RELEASE=/DEBUG /INCREMENTAL:NO /OPT:REF /OPT:ICF ;-G;Ninja;-DCMAKE_BUILD_TYPE=Release;-DCMAKE_INSTALL_PREFIX=D:/ThridLibraryPackages/packages/bzip2_x86-windows
Working Directory: D:/ThridLibraryPackages/buildtrees/bzip2/x86-windows-rel
See logs for more information:
D:\ThridLibraryPackages\buildtrees\bzip2\config-x86-windows-rel-out.log
D:\ThridLibraryPackages\buildtrees\bzip2\config-x86-windows-rel-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:170 (vcpkg_execute_required_process)
ports/bzip2/portfile.cmake:18 (vcpkg_configure_cmake)
scripts/ports.cmake:72 (include)
Error: Building package bzip2:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: bzip2:x86-windows
Vcpkg version: 0.0.83

@God-Staff Could you upload the logs as well

I have exactly the same issue.
I have Windows 10 x64 and Visual Studio 2017 (with VS 2015 as well).
I can't install any package.
Could you help me?
Thanks in advance!
vcpkg install sqlite3
The following packages will be built and installed:
    sqlite3:x86-windows
Building package sqlite3:x86-windows...
-- CURRENT_INSTALLED_DIR=C:/tmp-maks/vcpkg/installed/x86-windows
-- DOWNLOADS=C:/tmp-maks/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=C:/tmp-maks/vcpkg/packages/sqlite3_x86-windows
-- CURRENT_BUILDTREES_DIR=C:/tmp-maks/vcpkg/buildtrees/sqlite3
-- CURRENT_PORT_DIR=C:/tmp-maks/vcpkg/ports/sqlite3/.
-- Using cached C:/tmp-maks/vcpkg/downloads/sqlite-amalgamation-3190100.zip
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Configuring x86-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:43 (message):
    Command failed: C:/tmp-maks/vcpkg/downloads/cmake-3.9.1-win32-x86/bin/cmake.exe;C:/tmp-maks/vcpkg/buildtrees/sqlite3/src/sqlite-amalgamation-3190100;-DSOURCE=C:/tmp-maks/vcpkg/buildtrees/sqlite3/src/sqlite-amalgamation-3190100;-DVCPKG_CMAKE_SYSTEM_NAME=;-DBUILD_SHARED_LIBS=ON;-DVCPKG_TARGET_TRIPLET=x86-windows;-DCMAKE_CXX_FLAGS= /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP ;-DCMAKE_C_FLAGS= /DWIN32 /D_WINDOWS /W3 /utf-8 /MP ;-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON;-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON;-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON;-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE;-DCMAKE_VERBOSE_MAKEFILE=ON;-DVCPKG_APPLOCAL_DEPS=OFF;-DCMAKE_TOOLCHAIN_FILE=C:/tmp-maks/vcpkg/scripts/buildsystems/vcpkg.cmake;-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON;-DCMAKE_CXX_FLAGS_RELEASE=/MD /O2 /Oi /Gy /DNDEBUG /Zi ;-DCMAKE_C_FLAGS_RELEASE=/MD /O2 /Oi /Gy /DNDEBUG /Zi ;-DCMAKE_SHARED_LINKER_FLAGS_RELEASE=/DEBUG /INCREMENTAL:NO /OPT:REF /OPT:ICF ;-DCMAKE_EXE_LINKER_FLAGS_RELEASE=/DEBUG /INCREMENTAL:NO /OPT:REF /OPT:ICF ;-G;Ninja;-DCMAKE_BUILD_TYPE=Release;-DCMAKE_INSTALL_PREFIX=C:/tmp-maks/vcpkg/packages/sqlite3_x86-windows
    Working Directory: C:/tmp-maks/vcpkg/buildtrees/sqlite3/x86-windows-rel
    See logs for more information:
      C:\tmp-maks\vcpkg\buildtrees\sqlite3\config-x86-windows-rel-out.log
      C:\tmp-maks\vcpkg\buildtrees\sqlite3\config-x86-windows-rel-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:170 (vcpkg_execute_required_process)
  ports/sqlite3/portfile.cmake:15 (vcpkg_configure_cmake)
  scripts/ports.cmake:72 (include)

Error: Building package sqlite3:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: sqlite3:x86-windows
  Vcpkg version: 0.0.83-cd7f4cccffd915edc2d4b625ff32dea970ec073f

Additionally, attach any relevant sections from the log files above.

@vmaks Could you rerun bootstrap?

@atkawa7 I rerun a bootstrap, but it doesn't help.
I rerun the bootstrap with the following script:
C:\src\vcpkg> .\bootstrap-vcpkg.bat

@vmaks Could you upload the logs as well?

@atkawa7 yes, I added my log files below.
config-x86-windows-rel-err.log
  The C compiler "C:/Program Files (x86)/Microsoft Visual
  Studio/2017/Community/VC/Tools/MSVC/14.10.25017/bin/HostX64/x86/cl.exe" is
  not able to compile a simple test program.

  It fails with the following output:

   Change Dir: C:/tmp-maks/vcpkg/buildtrees/sqlite3/x86-windows-rel/CMakeFiles/CMakeTmp

  Run Build
  Command:"C:/tmp-maks/vcpkg/downloads/tools/ninja/ninja-1.7.2/ninja.exe"
  "cmTC_09891"

  ninja: error: build.ninja:30: loading 'rules.ninja': The system cannot find
  the file specified.

  include rules.ninja

                     ^ near here

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:2 (project)

config-x86-windows-rel-out.log
-- The C compiler identification is MSVC 19.10.25019.0
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.10.25017/bin/HostX64/x86/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.10.25017/bin/HostX64/x86/cl.exe -- broken
-- Configuring incomplete, errors occurred!
See also "C:/tmp-maks/vcpkg/buildtrees/sqlite3/x86-windows-rel/CMakeFiles/CMakeOutput.log".
See also "C:/tmp-maks/vcpkg/buildtrees/sqlite3/x86-windows-rel/CMakeFiles/CMakeError.log".

@vmaks Could you comment this  PREFER_NINJA in you portfile. Its this line
https://github.com/Microsoft/vcpkg/blob/master/ports/sqlite3/portfile.cmake#L17

@atkawa7 yes, it helps. Thank you very much!
However, I have the same issue with libpng, liblzma, libjpeg-turbo, tiff.
Could you fix the issue with "PREFER_NINJA" completely?
Could you also tell how to compile libraries with help of Visual Studio 2015 compiler in Visual Studio 2017. How to change the compiler in vcpkg?

Create another triplet for your architecture, platform, complier and use it: vcpkg install <port>:<triplet>

Here's the documentation on triplets (including how to specify the compiler): https://github.com/Microsoft/vcpkg/blob/master/docs/users/triplets.md

@vmaks Can you tell me your console Codepage?

@fcharlie I think this is most likely due to the codepage issue as well, so I'll close this as a duplicate for now. We can open a new issue if this is still a problem after we fix our codepage handling.

@fcharlie Excuse me for delay with answer.
Could you tell me how to check console Codepage?

@vmaks Open Console, type chcp, will show console codepage

Console Codepage:
Active code page: 866

execute
chcp 65001

and try to install port

@Mixaill yes, it works, thank you very much!

Please report if the latest master fixes things for you.

@alexkaratarakis, excuse me for delay with answer.
Yes, with the latest version everything works great!


Upgrade probe.gl

Coverage decreased (-0.003%) to 83.668% when pulling c92d446940488c08818f0d74105cce68ef8030f2 on x/bump-probe into 882c59595d88533f56a16cebe72bcb4466971808 on master.


I encountered an error when I tried to build Android development

thread 'main' panicked at 'Failed to read source.properties: Os { code: 2, kind: NotFound, message: "No such file or directory" }', /home/anmingle/.cargo/registry/src/mirrors.ustc.edu.cn-61ef6e0cd06fb9b8/ndk-build-0.9.0/src/ndk.rs:89:14
Here is my running environment:
Android Debug Bridge version 1.0.41
Version 28.0.2-debian
Installed as /usr/lib/android-sdk/platform-tools/adb
NDK:--version 21.4.7075529
The system is  ubuntu

Is this when building or when running? Could you share the commands you executed? Could you share code to reproduce the issue?

Running,  I follow the tutorial on the official website and
[Download command]
sudo apt install android-sdk   android-tools-adb  android-tools-fastboot
[environment variable]
export ANDROID_SDK_ROOT=/home/anmingle/Android/Sdk
export ANDROID_NDK_ROOT=/home/anmingle/Android/Sdk/21.4.7075529
Sorry for the following error when I tried to retry:
Android SDK has no platforms installed.
I've been trying for a long time

The ndk version 25.2.9519653 sdk version 31 I used cannot run. Is there any documentation for the version I can use

Works for me with both ndk 25.2.9519653 and 24.0.8215888. The errors would point more at something in your setup...

Works for me with both ndk 25.2.9519653 and 24.0.8215888. The errors would point more at something in your setup...
After I reset the environment, the following error appears. I have no way to solve it:
ld: error: undefined symbol: getifaddrs

referenced by network_helper_nix.rs:108 (src/network_helper_nix.rs:108)
sysinfo-e87a282ac6bee898.sysinfo.ed36b560-cgu.14.rcgu.o:(sysinfo::network_helper_nix::get_interface_address::h3f7e40636380f86f) in archive D:\bevy\andor\tar
get\aarch64-linux-android\debug\deps\libsysinfo-e87a282ac6bee898.rlib
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Running, I follow the tutorial on the official website and [Download command] sudo apt install android-sdk android-tools-adb android-tools-fastboot [environment variable] export ANDROID_SDK_ROOT=/home/anmingle/Android/Sdk export ANDROID_NDK_ROOT=/home/anmingle/Android/Sdk/21.4.7075529
Sorry for the following error when I tried to retry: Android SDK has no platforms installed. I've been trying for a long time

It looks like your ANDROID_NDK_ROOT variable is missing the /ndk directory.
- ANDROID_NDK_ROOT=/home/anmingle/Android/Sdk/21.4.7075529
+ ANDROID_NDK_ROOT=/home/anmingle/Android/Sdk/ndk/21.4.7075529


Feature/reorganize recipe data

Problem

derive the composition dictionary from the structure of old recipes
change packing_priority to priority
Convert the rest of the header info in new recipes

Solution

created composition dictionary and moved ingredients into interior
changed all packing_priority to priority through all files
convert the old recipe to v2 structure in _migrate_version
added tests

with @meganrm
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

make sure to link the issues this PR addresses


Upgrade org.apache.iceberg:iceberg-spark3-runtime version in spark3

The current version specified for spark integration in spark3 project gradle.build, 0.12.1, is quite old and missing SQL support for time travel (supported from iceberg 0.14.0 / Spark 3.3 onward)

The code should be working with iceberg 1.3.0. We're testing this iceberg version for spark 3.4


misc. cleanup

The first commit just fixes some of clippy's more pedantic lints. I wanted to note that while the explicit imports are arguably uglier than the wildcard * and obviously more verbose, the wildcard imports have bitten me before due to similarly named types between various lsp-related crates.
The second just swaps "AST" for "CST" to match the verbage used in tree-sitter's docs. (This is such an ackshually change I kinda feel bad about it, but it may help prevent some future confusion. Feel free to disregard :laughing:)
Next todo...tests! :)

Awesome, thank you! :)


Disabling Columns has no effect

It does not matter what I select under columns, Skada always shows everything.
 Maybe thats part of the problem.
Looking into it

I have completely rewritten the code of the addon in order to get rid of what's unnecessary and to have a better idea who it was written. Next update will be a completely different Skada (for 3.3.5) - see some screenshots.

Ok, I think I completetely missunderstood what that setting was supposed to do.
Very nice to see this addon has active development! :)


[Intel MKL] Klocwork fix

Can we cherry-pick this into r2.1 as this is required to release TF 2.1? Here is the PR against r2.1 #35391

@alextp No unit test is failing. Klocwork is a tool that scans code for security and reliability issues. Intel cannot release code if it does not pass this scan. The change in this PR has no functional impact. It is needed just to make the tool scan pass which requires every member variable to be initialized in the constructor.


Changes default string to empty string

Adds Optional[] type hints
Changes default string to empty string
-Tests passed

Most of the IDs for the data elements (rather than code tables) will need to be UUIDs within the global climate data system. As such these need to be strings and not null / optional.

Thanks for the observation. Would it be safe to use UUID column type instead ? Then instead of str type we could use uuid type.
Also most of the foreignkeys would still be represented as optional so as to allow us to instantiate the object before attaching the necessary ids or objects. If we make the foreignkeys mandatory then we would not be able to create an instance of the class except the related object exists.


update CUDA to 3.0, correct cluster CI CUDA_HOME

bors try

bors try

bors try

bors try

bors try

bors try

bors r+


task problem count is incorrect on reconnection for windows

problem count is wrongly reported bc wrapped error lines can get stuck together

/closedWith 9c6513b4b8720d73215540e8d524b1759daa15ac

Friendly ping! Looks like this issue requires some further steps to be verified. Please provide us with the steps necessary to verify this issue.

run tasks, check the problem count, reload the window, wait until they're reconnected and problems come back, verify the count is the same as before


refactor using proper styleguide

Maybe https://google.github.io/styleguide/vimscriptguide.xml

Maybe when Vim9script is out I rewrite the whole thing.


Model Training

Thanks for your work! Any advices for model training? I performed training but got score around zero.

Model training is a difficult and time consuming part. You could start from something really simple (for example, use the sample config from the project) and then make it more complicated and vary various parameters.

ok, thanks!


What happened to block factory ?

What happened to Block Factory ?
There is no documentation about graph manager or the transition to Block Factory.

Graph manager is documented under Control Flow here.


Transformers branch: arbitrary event files to read?

BIDSEventCollection.read states that arbitrary event files can be passed in as files= and it will ignore event files in the current project. I'm assuming this means that the files being passed in do not have to be in the BIDS project directory structure (i.e. this is not to select a subject of files, but pass new replacement files in).
This current does not work as it tries to determine the entities of these new files by looking them up in self.project.files which obviously does not contain the new files.
I imagine the best way to proceed is to extract the entities from these new files rather than trying to look them up. Or by adding them to the project?

Closing, as we added functionality to read flat files in the Transformer branch.


chore: bump kcl lib to v0.10.0

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

chore: bump kcl lib to v0.10.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
4901

Relevant Lines:
13122

💛  - Coveralls


Core dump when running Chapter 1

Hi, I'm new to Rust and gamedev, and unfortunately I've stuck at the very first chapter. Both an independent project following the tutorial and the cloned repo yield the same error when I run them after building. Other rust programs and rust projects I've written build and run fine; I am not sure if it is caused by an issue with the bterm builder creating a window that my system does not like. I would greatly appreciate the help!
Finished dev [optimized + debuginfo] target(s) in 0.10s
     Running `[MY_DIRECTORY}/rustrogueliketutorial/target/debug/chapter-01-hellorust`
[xcb] Unknown sequence number while processing queue
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
chapter-01-hellorust: xcb_io.c:269: poll_for_event: Assertion `!xcb_xlib_threads_sequence_lost' failed.
Aborted (core dumped)

Journalctl yields the core dump as:
Process 24894 (chapter-01-hell) of user 1000 dumped core.
                                                Stack trace of thread 24894:
                                                #0  0x00007f32fdba8d22 raise (libc.so.6 + 0x3cd22)
                                                #1  0x00007f32fdb92862 abort (libc.so.6 + 0x26862)
                                                #2  0x00007f32fdb92747 __assert_fail_base.cold (libc.so.6 + 0x26747)
                                                #3  0x00007f32fdba1616 __assert_fail (libc.so.6 + 0x35616)
                                                #4  0x00007f32fda3ad2d n/a (libX11.so.6 + 0x40d2d)
                                                #5  0x00007f32fda3adc8 n/a (libX11.so.6 + 0x40dc8)
                                                #6  0x00007f32fda3b182 _XEventsQueued (libX11.so.6 + 0x41182)
                                                #7  0x00007f32fda3e176 _XGetRequest (libX11.so.6 + 0x44176)
                                                #8  0x00007f32fda29395 XNoOp (libX11.so.6 + 0x2f395)
                                                #9  0x00007f32fd3376e3 n/a (libGLX_mesa.so.0 + 0x306e3)
                                                #10 0x00007f32fd4cb428 n/a (libGLX.so.0 + 0x1b428)
                                                #11 0x000055a6cbdbefa5 n/a (/[MY_DIRECTORY]/rustrogueliketutorial/target/debug/chapter-01-hel>

My system is Manjaro Linux , x86_64 with i3 as the windows manager.

Same here,
i'm using Artix linux with kernel 5.15.10 and dwm window manager.
sometimes it won't start, sometimes it will start.
I don't know what is causing the error and the inconsistency


Clean the Get Started section of the docs

Description:
Need to modify the Get Started section of the docs for the latest release APIM 4.3.0.

[ ] Overview
[ ] Key Concepts
[ ] Architecture
[x] Quick Start Guide - https://github.com/wso2/docs-apim/issues/7813
[ ] About this Release

Overview

The link in API-led Integration does not work. The relevant documentation is not available in the Micro Integrator documentation.
The first 2 links under the What's next section get's redirected to latest documentation instead of 4.3.0 space.

The above issues are fixed with PRs :
4.3.0 - https://github.com/wso2/docs-apim/pull/7896
master - https://github.com/wso2/docs-apim/pull/7897

Closing the issue as all the relevant PRs are merged.


Zoom in and out slows down when parent element width is increased.

I have a site with a sidebar and a main panel with transform component. The issue is whenever the sidebar closes, the main panel width becomes 100%, which somehow renders zoom in and out speed quite slow. Below I have replicated the same behavior in the demo example.
https://youtu.be/99Z16DtxQvM

Yes I am having a similar issue where I have the TransformWrapper/Component in the middle of my application and when I expand its width, the same issue happens to me.

@confrodog have you found any solution?

@confrodog have you found any solution?

I don’t have any surefire solution but I kept playing around with divs and where the TransformWrapper/Component was on the DOM hierarchy and it doesn’t seem to do it anymore. It’s probably not what you want to hear but I’m a novice React programmer.


Add EDP support libraries for HMSS.

This change is

This PR will not be submitted until we agree on the interfaces and they are implemented. This initial pass of reviews is to agree on the desired interface.

Superseded by https://reviewable.io/reviews/world-federation-of-advertisers/cross-media-measurement/1552


1491 custom log levels can be silent

Fixed merge conflicts in https://github.com/winstonjs/winston/pull/2366 and reopening here -- credit to @ckujawa
The referenced issue is #1491

Forget my comment in the other PR, seems https://github.com/winstonjs/winston/pull/2386 is still needed on my mac at least...

@ckujawa I cleaned this up a bit but tests are failing (specifically the custom levels one), can you take a look and opine / revise / add tests as appropriate?


feat(codegen): print linked and external legal comments

part of #7050

[!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

#7060  👈
#7059
#7058
#7054
main

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

Join @Boshen and the rest of your teammates on  Graphite


Add Contributors list at bottom of README.md

@PyladiesTNR I can try this.Please assign this to me.


Do not memorize reply-to queue.

Do not memorize reply-to queue.
Add dotenv to development dependencies
Add code snippet to spec helper so specs can be run from IntelliJ.

Done commenting.


MINOR: Add 3.8.0 to system tests

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Packages are now in https://kafka-packages.s3.us-west-2.amazonaws.com/
Thanks @cmccabe

Tests in the CI pipeline are unrelated. @omkreddy Could you check that the system test pipeline works smoothly after this is merged?

Cherry-picked to 3.9


Update README.md

Checkmarx One – Scan Summary & Details – fb024f57-88db-4943-b5c9-293a6bd7b188
No New Or Fixed Issues Found


rm boilerplate

cleanup
or do we want to keep cli for one purpose?
anything else which can be deleted?

It's fine to delete the cli for now. I used it just for testing in the beginning and we might quickly add it back when we decide to work on a cli.

Would this be a good place to unify the docstrings a little? We are changing a lot between small and capital letters in the description of arguments. I think, they should be all captialized.

Ok. Go for it


Bind module.export = {Thing} with alias symbols

Fixes #41250, the last half of it.
This was actually an incredibly small change, thanks to already anticipating aliases like this pretty much everywhere. DO note that in addition to shorthand property references, we could also probably bind { Name: EntityName } references as aliases safely, too, though this PR does not add that currently.

@typescript-bot user test this

Huh, I bet this makes (at least parts of) https://github.com/microsoft/TypeScript/pull/40835 obsolete


Add genome assembly version to cell metatada

It was recently brought up to my attention that including the genome assembly used during mapping (e.g. GRCh38.XXX) as cell metadata would be valuable.
We need to consider if this is a common need from the community that we would want to fulfill. I believe it is a relatively easy lift from the curation perspective and it may provide valuable batch information to users.
Please proceed as follows:

Assess the impact that the addition of this field brings to the community. cc @BAevermann @jahilton
Provide an estimate on the reach of such addition. cc @BAevermann
Decide on including it in the schema v4.0, or a future version, or just put on hold for the time being.

cc @brianraymor

Can you provide more information on #2 specifically what how "reach" in this context is measured? Thanks!

I'm interested in specifically why it would be valuable and what the "XXX" represents in "GRCh38.XXX"

Can you provide more information on https://github.com/chanzuckerberg/single-cell-curation/issues/2 specifically what how "reach" in this context is measured?

@BAevermann, I was hoping for an informed guess on the percentage of users that would benefit from this information. For example, I'd say that supension_type likely benefit 90% of users.

I'm interested in specifically why it would be valuable and what the "XXX" represents in "GRCh38.XXX"
@jahilton The minor version number

GRCh38.p14 (current)
GRCh38.p13
GRCh38.p12
...

I was hoping for an informed guess on the percentage of users that would benefit from this information

I'm not too keen on SWAG approaches because then it's easy to make a theoretical case for any new field. That has not worked well in the past based on recent deprecations. I'd prefer to have some concrete users and use cases (like Malte and integration workshops for suspension_type).
We need to understand what "valuable" means and how the field will be used.

If reference information is being explored, I would suggest expanding the scope beyond the proposed assembly version. The information in order of general resolution:

Assembly (hg19, GRCh38, mm10)
Assembly version (GRCh38.p13)
Annotation version (Ensembl/GENCODE/RefSeq version)

This is more likely to be readily available than the patch version and a curator or submitter is probably using the annotation version to look up which patch version is used in that annotation reference. And it is more specific than the patch version (GRCh38.p13 was used for more than 10 different GENCODE versions)
n/a for non-expression assays

Any of them will hit issues when there's a custom reference used

Per today's triage, closing until there is further information about the user value for such an addition.


feat(packages/tokamak): update NPM info

I stopped the CI workflow to reduce unnecessary EC2 spin-up.


Multiple swipers conflict with Bootstrap tab-panes. Only 1st works correctly!

It's interesting that by putting each Swiper inside a tab-pane, only first one works correctly. I tried all possible approaches!
Is there any solution for this?
`

Brands
Categories

{% for brand in brands %}
{% if brand.link %}{{ brand.name }}{% else %}{% endif %}
{% endfor %}

 {% for category in categories %}
{% if category.products >= 0 %}
 {% if category.thumb %}{% endif %}{{ category.name }}
{% endif %}
{% endfor %}

You must init Swiper when it becomes visible or call its update() method when tab with it becomes visible

Many many thanks! It worked!


Admin's dashboard

@soniyaprasad77 when you say admin login, you mean regular login page?

what should i add

@soniyaprasad77 when you say admin login, you mean regular login page?

How should I improve the page for admin, is there any specifications?

Please check my comments.

sure, I will improve the indentations.

@soniyaprasad77 You named the file admin login, so I was wondering if there would be another page for regular user login?
Let's keep it simple, one login page for both, regular user and admin user.

@soniyaprasad77 You named the file admin login, so I was wondering if there would be another page for regular user login? Let's keep it simple, one login page for both, regular user and admin user.

Alright., Simplicity is essential :)

So I am closing this PR, Thank you for your suggestions.

Certainly!
I would appreciate some clarification regarding user authentication within a web application. Given that administrators typically possess enhanced features compared to regular users, it seems logical to have distinct database schemas for administrators and students. However, I am curious about the feasibility of implementing a unified login mechanism on the same page, utilizing POST requests for both administrators and students. Could you kindly shed light on how such a scenario can be effectively managed, taking into consideration the differing functionalities associated with these user roles? Your insights would be immensely valuable.

@soniyaprasad77 authentication for both admin and regular user would be same. Both go through the same process but once logged in, admin and regular user would be redirected to their respective dashboard.
Does this make sense?

@soniyaprasad77 did you make the changes I suggested earlier? Is this ready to be merged?

Actually, as you suggested, we are going for a different login page for the admins, so I started working on the admin dashboard, I am still working on it, i will update it soon.

@soniyaprasad77 sorry for the confusion, there is going to be one login for both members and admin, i.e. both would use the same login page.
After successfully logged in, the login user would be re-directed to their own dashboard i.e. member would go member dashboard and admin would go to admin dashboard.
Does this make sense?

Yes.. it's make sense, and we do already have a login page. So I am working on Admins dashboard


Test not flagged as failed whenever exceptions are caught whilst running a test

There are situations where for example a failure occurs when opening the log files (such as the one reported in #176),
Executing: -> [mytest01]
Exception caught while running test batch \Bin\Debug\Win32\i386\bin\BoostTest01.exe [mytest01] (The process cannot access the file 'C:\Users\gastart\AppData\Local\Temp\BoostTest01.exe.7096.6.test.stdout.log' because it is being used by another process.)

it seems that we are not providing any test result back to visual studio and the test remains faded  out. A user might not notice that the test actually failed to "properly execute" and might think that everything went ok.
I would suggest that in case there is some exception in BUTA we do either declare the test as failed or as undetermined (the yellow triangle).

@gastart yes should be fixed.
Not sure if should be marked as failed or undetermined? undetermined would have the advantage of be able to distinguish between defined and undefined failed?


Generating RoadPoints from existing Curve3D?

edit 2 : I dont know what happened, maybe the update from .51->.52 version changed it, but now the imported curve can be edited directly! (godot4.3). After importing scene should be saved, closed and opened again. The code below made is not a rock solid good solution, but might helps in some cases. The remaining question ; how to export just curves from the road-generator? Not just separate rp-parts?
Is it possible to generate proper RoadGenerator road from existing Curve3D?
Another question - how to get full continous curve from road-generator created road?
I can get roadpoint-curves, but those are alla separate curves and I dont know how to create proper curve from those. Only useful I can get is the transforms.origin point and create curve from those, but losss of curve information is enormous. Those transformations are a bit too complicated me to
Code (Godot 4.3, add it tool-script, give valid curve and chunk size to parse):
edit 3 - below (mostly) working code to make road from plain curve
@tool
extends Node3D

@export var road_manager : RoadManager
@export var generate_points := false :
	set(generate_points):
		generate_rp_points()
@export_category("settings")
@export var close_curve : bool = false
@export var use_start : bool = false
@export var start_m : int
@export var use_stop : bool = false
@export var stop_m : int
@export var chunk_length : int = 20
@export var density : int = 20
@export var n_mag : int = 8
@export var p_mag : int = 8
@export_file("*.tres") var curve_path : String
var rpg_process : bool = false;

func generate_rp_points():
	if rpg_process: return
	rpg_process = true
	if!curve_path: print("no curve_path!"); return
	# extrack points from track
	var curve : Curve3D = load(curve_path)

	rps_from_curve(curve,chunk_length,n_mag,p_mag,density)
	rpg_process = false

func rps_from_curve(
	curve:Curve3D,
	chunk_length : int,
	next_mag : int = 8,
	prev_mag : int= 8,
	interval : int = 20,
	container: RoadContainer = null,
	)->void:
	if!container:
		container = get_node_or_null(rps_container(road_manager))
	var transforms 	:=[]
	curve.bake_interval = interval
	var baked_tilts	:	Array = curve.get_baked_tilts()
	var amount	:	int	= baked_tilts.size()-1
	var baked_length : int = curve.get_baked_length()
	var tilt_step : float = float(baked_length) / float(amount)

	var start_from := 0
	var stop_to : int = baked_length
	if use_start:
		start_from = start_m;
	if use_stop:
		stop_to = stop_m
		if stop_m > baked_length or stop_m < start_from:
			stop_to = baked_length;

	for sample in range(start_from,stop_to,chunk_length):
		var trans = curve.sample_baked_with_rotation(sample)
		var tilt_index : int = clamp(int(sample / tilt_step), 0, amount)
		if tilt_index <= amount:
			trans = trans.rotated_local(
				Vector3(0,0,-1),(baked_tilts[tilt_index])
			)
		transforms.append(trans)

	for i in range(transforms.size()):
		var p = RoadPoint.new()
		p.transform = transforms[i]
		p.next_mag = next_mag
		p.prior_mag = prev_mag
		container.add_child(p)
		p.owner = get_tree().edited_scene_root
	var points_size = transforms.size()-1
	for i in range(transforms.size()):
		var next : int = i+1;
		if next >= points_size:
			next = 0
		var prev : int = i-1;
		if prev < 0:
			prev = points_size
		var p : RoadPoint = container.get_child(i)

		var next_point_node = container.get_child(next)
		#var prev_point = container.get_child(prev)
		var this_direction = RoadPoint.PointInit.NEXT
		var target_direction = RoadPoint.PointInit.PRIOR
		if!close_curve and next == 0:
			container.get_children()[container.get_child_count()-1].queue_free()

			break
			#points_size:
			#p.next_pt_init = container.get_child(i-1).get_path()

		#p.connect_roadpoint(this_direction, next_point, target_direction)
		p.connect_roadpoint(target_direction, next_point_node, this_direction)

func rps_container(rm : RoadManager = null)->NodePath:
	if!rm:	rm = get_node_or_null(rps_manager())
	var node : RoadContainer = RoadContainer.new()
	rm.add_child(node)
	node.owner = get_tree().edited_scene_root
	return node.get_path()

func rps_manager(target_node : Node = null)->NodePath:
	if!target_node: target_node = self
	var node : RoadManager = RoadManager.new()
	target_node.add_child(node)
	node.owner = get_tree().edited_scene_root
	return node.get_path()

Hey there! I'd be happy to help. But to start with making sure I understand the goal here, the questions:

how to export just curves from the road-generator? Not just separate rp-parts?

Do you mean, creating one overall curve that represents the entire road network? There's no single solution that could really work for this, as native curves are (by definition) continuous without breaks and non branching. In contrast, the Road Generator lets you connect and disconnect road segments, join them up to prefab pieces (e.g. intersections) and so forth.
You could, however, write a function that tries to generates a curve for one continuous stretch of road at a time, ie a non broken sequence of RoadPoints. My main question is: what is the use case for this? What are you trying to do with this single continuous stretch of road? Is this to add details along the side of the road or something like this? And, is this for a procedural in-game generation use case, or more about e.g. placing curves to decorate items along the road in the Godot editor? I'm asking these questions so I can help better advise.

Is it possible to generate proper RoadGenerator road from existing Curve3D?

So you mean going from a hand-authored curve and then automatically creating roads from that? Again, I'd be curious to hear about the workflow you are trying to have where you'd start with such curves. Are they coming from another software, or is there a reason it's not convenient enough to just draw the road curves directly with the Road Generator menu?
But nonetheless, while we don't have built in functions this, your code snippet is somewhat similar to what we are doing in Wheel Steal Game. We add a series of RoadPoints with some distance between each one, placement and orientation is sampled along the curve to create more organic feeling procedurally generated roads.

Another question - how to get full continous curve from road-generator created road?

I think this is the same as the first question right? Let me know if not.
Hope this helps!

I use generated content and use curve / curves as a baseline. Currently npc-vehicles use center curve to take direction on the road. Other approach might be more scalable and easier to use if intersections or other roads are added - maybe some day. Principle is pretty much the same - to take direction to some point and go towards. I use rigid bodies with physics, and precise point is not needed - just a guideline. Terrain data also uses road-curve. I generate road, terrain and assets before game starts, so there is currently no generating after game starts.
I continued code a bit and now it take edge_(RFC) points from given container point-ranges. I just really don get, how I should rotate / transform taken path and its curvepoints to generate proper curve.
main func to do it looks now this ( the affine inverse-thing really makes a mess, its last thing I have tried );
func points_from_curve(curve : Curve3D, trans : Transform3D,rot:Vector3)->Array:
	curve.bake_interval = 10;
	var samples : int= 5;
	var curve_length : float= curve.get_baked_length()
	var chunk_length = float(float(curve_length) / float(samples))
	var points_set :=[]
	for sample in range(0,curve_length,chunk_length):
		var point: Transform3D = curve.sample_baked_with_rotation(sample)#.rotated()
		point = point.affine_inverse() * trans
		#point.rotated_local(Vector3(1,0,0),rot.x)
		#point.rotated_local(Vector3(0,1,0),rot.y)
		#point.rotated_local(Vector3(0,0,1),rot.z)
		points_set.append(point.origin + trans.origin)
	return points_set

tool script to make curve from rp:s - far from perfect. Requires that edge curves are parsed by rg.
var making_curve_process : bool = false;

func make_curve_from_rp_points():
	if making_curve_process: return
	making_curve_process = true
	if!road_manager:
		making_curve_process = false;
		print("No road manager defined!"); return
	var process_nodes : Array = duplicate_paths()
	var paths : Array= process_nodes[0]
	var temp_node_path : NodePath = process_nodes[1]
	paths.reverse()
	process_duplicated_paths(paths)
	var temp_node = get_node_or_null(temp_node_path)
	if temp_node:
		temp_node.queue_free()
	making_curve_process = false;

func process_duplicated_paths(paths : Array):
	var center_curve := Curve3D.new()
	var pn = get_node(pathnode_container(path_node))
	for x in range(paths.size()):
		var path = get_node_or_null(paths[x]);
		if!path		:	continue
		var _curve = path.curve
		if!_curve	:	continue
		var points = points_from_curve(_curve,path)
		for pos in points:
			center_curve.add_point(pos)
	pn.curve = center_curve

func points_from_curve(curve : Curve3D, _path: Path3D)->Array:
	curve.bake_interval = 10;
	var samples : int= 5;
	var curve_length : float= curve.get_baked_length()
	var chunk_length = float(float(curve_length) / float(samples))
	var points_set :=[]
	for sample in range(0,curve_length,chunk_length):
		var point = curve.sample_baked(sample)
		var pos  = _path.to_global(point)
		points_set.append(pos)
	return points_set

func duplicate_paths()->Array:
	var points := []
	var road_containers :=[]
	var node_paths :=[]
	var temp_node := Node3D.new()
	add_child(temp_node)
	temp_node.owner = get_tree().edited_scene_root
	for ch in road_manager.get_containers():
		road_containers.append(ch)
		for obj in ch.get_roadpoints(true):
			var pt:RoadPoint = obj
			points.append(pt)
	for rp : RoadPoint in points:
		var p00 : Vector3 = rp.transform.origin
		var curve_to_get = "edge_C"
		var rp_path : Path3D
		for node in rp.get_children():
			if node.name == curve_to_get:
				rp_path = node.duplicate(true); break
		if!rp_path: continue
		temp_node.add_child(rp_path)
		rp_path.owner = get_tree().edited_scene_root
		rp_path.transform = rp.transform
		node_paths.append(rp_path.get_path())
	return [node_paths,temp_node.get_path()]


support metrics for tc

Ⅰ. Describe what this PR did
It let tc support basic metrics output to prometheus
Ⅱ. Does this pull request fix one issue?

https://github.com/alibaba/fescar/issues/384

Codecov Report

Merging #476 into feature_metrics will decrease coverage by 0.05%.
The diff coverage is 28.37%.

@@                  Coverage Diff                  @@
##             feature_metrics     #476      +/-   ##
=====================================================
- Coverage              30.63%   30.57%   -0.06%
- Complexity               664      670       +6
=====================================================
  Files                    182      184       +2
  Lines                   7068     7139      +71
  Branches                 843      848       +5
=====================================================
+ Hits                    2165     2183      +18
- Misses                  4681     4733      +52
- Partials                 222      223       +1

Impacted Files
Coverage Δ
Complexity Δ

...ava/com/alibaba/fescar/metrics/DefaultSummary.java
96% <ø> (ø)
7 <0> (?)

...in/java/com/alibaba/fescar/metrics/TimerValue.java
83.33% <ø> (ø)
7 <0> (?)

...va/com/alibaba/fescar/metrics/DefaultRegistry.java
58.33% <ø> (ø)
6 <0> (?)

...ava/com/alibaba/fescar/metrics/DefaultCounter.java
91.66% <ø> (ø)
6 <0> (?)

.../java/com/alibaba/fescar/metrics/SummaryValue.java
64.7% <ø> (ø)
5 <0> (?)

.../java/com/alibaba/fescar/metrics/DefaultGauge.java
90% <ø> (ø)
4 <0> (?)

.../fescar/server/coordinator/DefaultCoordinator.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...rc/main/java/com/alibaba/fescar/server/Server.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...libaba/fescar/server/metrics/MeterIdConstants.java
0% <0%> (ø)
0 <0> (?)

.../java/com/alibaba/fescar/metrics/DefaultTimer.java
96.55% <100%> (ø)
8 <0> (?)

... and 3 more

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 a033c81...107a464. Read the comment docs.


Generating type definitions for mixin classes with protected members

TypeScript Version:  2.4.2
Code:
I'm using mixins as described by: https://github.com/Microsoft/TypeScript/pull/13743
export type Constructor<T> = new(...args: any[]) => T;
export function Unsubscriber<T extends Constructor<{}>>(Base: T)  {
  class Unsubscriber extends Base implements OnDestroy {
    protected unsubscribe: Subject<void> = new Subject();

    ngOnDestroy() {
      this.unsubscribe.next();
      this.unsubscribe.complete();
    }
  }
  return Unsubscriber;
}

If I compile this code with "declaration": true to get type definitions for my library, I get the following error:
ERROR in (truncated)/mixins.ts (...): Return type of exported function has or is using private name 'Unsubscriber'.

One solution is to add an interface...
export interface IUnsubscriber extends OnDestroy {
  unsubscribe: Subject<void>;
}

...and have my mixin function have a return type of Constructor<IUnsubscriber>. This works, but it forces me to make the properties/methods exposed by my mixin be public even in cases where I want them to be protected.
Short of adding protected members to interfaces (which I'm not sure is the right thing to do), this seems to be a limitation of the currently supported mixin strategy.

BTW I found that you can work around this limitation to some extent by using ECMAScript symbols, which provide a non-TypeScript way to hide a member:
export type Constructor<T> = new(...args: any[]) => T;

const unsubscribe: unique symbol = Symbol('Unsubscriber.unsubscribe');

export function Unsubscriber<T extends Constructor<{}>>(Base: T)  {
  class Unsubscriber extends Base implements OnDestroy {
    public [unsubscribe]: Subject<void> = new Subject();

    ngOnDestroy() {
      this[unsubscribe].next();
      this[unsubscribe].complete();
    }
  }
  return Unsubscriber;
}

IntelliSense will only show the unsubscribe member to callers who have access to the unsubscribe symbol.

It bothers me too. Seems that this issue was moved into future milestone. No plan to solve it in the near future?

BTW I recently came up with an alternate way to represent mixins in TypeScript.  Consider this example:
// Helper that copies properties from base's prototype
// to child's prototype
function extend(child: any, base: any): void {
  for (var property in base) {
    var descriptor = Object.getOwnPropertyDescriptor(base, property) || { value: base[property] };
    Object.defineProperty(child, property, descriptor);
  }
}

// The mixin base classes
class Base1 {
    constructor() {
        console.log('Base1.constructor() called')
    }
    public method1(): void {
        console.log('method1() called')
    }
}

class Base2 {
    constructor() {
        console.log('Base2.constructor() called')
    }
    public method2(): void {
        console.log('method2() called')
    }
}

class Base3 {
    constructor() {
        console.log('Base3.constructor() called')
    }
    private _prop3 = 123;
    public get prop3(): number {
        return this._prop3;
    }
}

// Collect the base classes into an intermediary "Mixin" class
type Mixin = Base1 & Base2 & Base3;
const Mixin: { new(): Mixin } = class {
    constructor() {
        Base1.call(this);
        Base2.call(this);
        Base3.call(this);
    }
} as any;
extend(Mixin.prototype, Base1.prototype);
extend(Mixin.prototype, Base2.prototype);
extend(Mixin.prototype, Base3.prototype);

// Extend from the "Mixin" class
class Child extends Mixin {
    public method4(): void {
        console.log('method4() called')
    }
}

const child = new Child();

child.method1();
child.method2();
console.log('prop3 = ' + child.prop3);
child.method4();

The behavior is probably unsound if the base classes happen to have members with the same name.  But this issue could be detected in the extend() function.
There are some interesting advantages:

The base classes don't need to be represented specially; they are ordinary TypeScript classes
You can use private members without any trouble
The generated .d.ts file is simple and understandable

Not sure I'd recommend to do this in real code, but it's interesting to contrast with the class-expression approach to mixins.

It bothers me too. Seems that this issue was moved into future milestone. No plan to solve it in the near future?

Well me too !
@RyanCavanaugh Any update on when this could be fixed ? It's set on the "Future" backlog but as it's pretty huge, could you add some precision ?

Here is my research regarding possible workarounds for TS mixins with protected methods.
Overview
These are 3 approaches that can be used for writing mixins that need to expose protected methods.
They all allow subclasses extending such mixins to call these methods using super.
1. Symbol approach

Not using protected methods in the mixins

Using Symbol as recommended by Elix

Example: see above comment https://github.com/microsoft/TypeScript/issues/17744#issuecomment-431534647

Pros

Methods are truly protected (can't be called  from outside, e.g. by third party code)

Methods are "public" in terms of TS and can be annotated on the interface

Explicit exports potentially help with refactoring of the consumers' subclasses

Methods are excluded from the DevTools console's auto-complete list (good for user)

Avoid potential name conflicts if user extends a class with their own methods

Cons

Consumers have to import symbols if they need to override protected methods

Jump to definition is not ideal (jumps to the place where Symbol() is created)

2. Dumb class approach

Using protected methods in the mixins

Using abstract class SomeMixinClass to "annotate" them

Example: https://github.com/microsoft/TypeScript/issues/25163#issuecomment-507074489

Pros

No need to import anything (except the mixin itself) for the consumer component

Methods are easy to access in the DevTools console (good for maintainer)

Cons

Both interface (for public API) and abstract class (for protected API) are needed

The abstract class ends up in the JS output (as empty class with no methods)

The mixin function needs to confusingly accept the argument with a type cast (instead of the actual expected class) in order to pick up the annotated protected methods from it:

// with dumb class: confusing
<T extends Constructor<SlottedItemsClass>>(base: T)

// without dumb class: clean
<T extends Constructor<LitElement>>(base: T)

Need for extra super check: super._focus && super._focus() - this is needed because the abstract classes mark methods using _focus?(), i.e. "possibly undefined"

Jump to definition from the subclass goes to a dumb class

3. Public methods

Keep methods public, add them on the interface

Jump to definition works as expected

Example: https://github.com/wikibus/Alcaeus/issues/33

Pros

No extra code needed (symbols / dumb classes)

Cons

TS users would get protected methods listed in the code completion

Messing up public and protected API might need tweaks for tools

Summary
Personally, I like symbols approach more because it's cleaner and bulletproof. The "jump to definition" inconvenience is a drawback that is more of a personal preference.
On the other hand, the dumb class approach has its benefits: once the issue is resolved, we can get rid of those dumb classes and potentially keep the methods themselves unchanged.

@weswigham according to your comment at https://github.com/microsoft/TypeScript/issues/17293#issuecomment-522491710, what do you have on your mind to tackle these issues? Is there a plan, or does this still need a concrete proposal that you mentioned?

Is there a plan, or does this still need a concrete proposal that you mentioned?

Still needs a proposal.

@fr0 Also worth noting https://github.com/microsoft/TypeScript/issues/35822

We hit this issue again when we tried to factor out a base class with private and protected members into a mixin.
Because we've already provided protected methods to subcasses of our base class, we really needed to keep existing protected members protected. If the members became public, then subclasses would get errors on trying to override public members with protected.
The workarounds we've had to do to satisfy the compiler are pretty onerous. We've created a fake class outside of the mixin and cast the mixin function to return an intersection of the argument and that class. Static require another fake class. Then we need to get our build system to remove the fake class. I'm not sure this approach solved every problem yet.
@weswigham

Still needs a proposal.

Is there anything anyone not on the TS team can do to help here?

Well, we discussed it at our last design meeting as future work beyond abstract constructor types, however some team members think allowing a class-expression-like-thing in a type position (and having that refer to the static shape of the declared class) would be confusing. IMO, it makes sense since the shape stored by
type A = class {}

would be the same as the type of the const in
const A = class {}

But that's only my point of view. The disagreement within the team means we're going to need to see both a serious proposal and serious justification for why it's needed and how it makes sense before we're going to get close to agreement.

@weswigham I think e list of why it is needed is in this issue: https://github.com/microsoft/TypeScript/issues/35822
The list in the Description.

I come up with a (maybe) visually better solution based on The Dumb Class Approach.
 ConstructorOf<U>): ConstructorOf<T & U>
}

export function createMixin<TBase = {}, T = TBase>(factory: MixinFactory<T, TBase>): MixinOf<T & TBase> {
  return ((Base = Object) => factory(Base as any)) as any
}

export type MixinType<T extends MixinOf<unknown>> = T extends MixinOf<infer U> ? U : never

// Case 1. define a normal mixin
export type FooMixin = MixinType<typeof FooMixin>
export const FooMixin = createMixin(Base => class FooMixin extends Base {
  foo() {}
})

// Case 2. define a mixin with protected/private fields
declare class BarMixinBase {
  protected bar(): void
}

export type BarMixin = MixinType<typeof BarMixin>
export const BarMixin = createMixin<BarMixinBase>(Base => class BarMixin extends Base {
  protected bar() {}
})

// Usage
export class MyClass extends FooMixin(BarMixin()) {
  test() {
    this.foo()
    this.bar()
  }
}

Hope the TS team to eventually solve the issue.


Bump recast to 0.5.0

It'd be awesome if this used the latest version of jscodeshift, which includes support for TypeScript and therefore would (at least in principle) make this viable for TypeScript codebases. (I'll try to open a PR with it shortly.)

Update: that won't help; jscodeshift has the ability in principle but not at all in practice to do this.

I think that this issue will be resolved by https://github.com/rwjblue/ember-qunit-codemod/pull/88...


[Flow-Aggregator] Add template fields of stats

This PR is a follow-up fix for #1636
When bumping go-ipfix to v0.4.2, several new template fields of stats are introduced. Exporting process in flow aggregator will log errors when having sanity check since field count does not match for template and data, which causes e2e tests to fail. Adding template fields of stats will solve that problem.
Tested with @dreamtalen on this fix.

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature/flow-aggregator@ad052b2). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                     @@
##             feature/flow-aggregator    #1676   +/-   ##
==========================================================
  Coverage                           ?   40.41%
==========================================================
  Files                              ?      107
  Lines                              ?    13150
  Branches                           ?        0
==========================================================
  Hits                               ?     5315
  Misses                             ?     7357
  Partials                           ?      478

Flag
Coverage Δ

unit-tests
40.41% <0.00%> (?)

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

Yongming's e2e tests PR is blocked because of this issue. As all tests are passing and it is a straightforward fix, merging this immediately.


issue to track doc work

Adding this issue to track doc work.

[x] a tutorial on how to use runners
[x] fixing links in about section
[x] update about section to new readme.md
[ ] adding source links to each class, etc
[ ] changing mlrose to mlrose-ky

Addressed some to-dos in #4

summary of work from #1
Here's my list of tasks for this project. Task 1 is completed; tasks 2—4 are in progress.

Fix all Python warnings and errors (except for some unavoidable warnings such as "duplicate code") ✅

Add Python 3.10 type hints to all function and method definitions, plus all method properties (e.g., self.foo: str = 'bar')

Add [NumPy Doc docstrings](https://numpydoc.readthedocs.io/en/latest/example.html#example) to all functions and methods, plus at least a one-line docstring at the top of every file summarizing the file's contents

Add tests using Pytest, aiming for 100% code coverage

Search for TODO, FIXME, or other comments and resolve their respective issues

Optimize vanilla Python loops that can be vectorized with NumPy

Improve any other sub-optimal code, bugs, code quality issues, etc

Remove any commented-out code

Moving to https://github.com/nkapila6/mlrose-ky


fix URL to mavproxy documentation

I think this link is now supposed to go to http://ardupilot.github.io/MAVProxy/html/getting_started/starting.html

[ ] Um, I'm unsure why all lines are modified. Fixing.

Hmm, I think this file has different line endings from VS Code/OS X and so its editing all lines. I'll close this PR and try to submit new PR via github.com directly.


Triple Handlebars doesn't always render raw HTML

I am trying to bind an svg element from an object held in the VM's data
When binding to it in the template using triple handlebar syntax, it still renders in the template as if toString() were called on the element.
An example of this can be found here:
https://jsfiddle.net/zba31mnw/
I am fetching an svg here over AJAX and assigning the XML from the response to the data that gets bound to the template.

You are passing the triple handlebars an SVG element, not the actual markup string...
fixed: https://jsfiddle.net/yyx990803/zba31mnw/1/

thank you so much @yyx990803


Feature Request: Async Cmdlets

Summary of the new feature/enhancement
Start-ThreadJob is a great tool, although it's a bit cumbersome if you just need to invoke single cmdlet with parameters. Would be nice just to indicate this with some switch, e.g. "Async". Somewhat similar to AsJob switch, but returning task or thread job.
Example
Invoke-WebRequest -Uri $Url -Method $Method -Headers $h -Async

This should return task object, or be a shortcut to this:
Start-ThreadJob -ScriptBlock { param($Url, $Method, $h)

    Invoke-WebRequest -Uri $Url -Method $Method -Headers $h

} -ArgumentList ($Url, $Method, $h)

If not possible as general mechanism, this could be implemented for some selected cmdlets. I think this could be useful for iwr/irm,  test-connection and GetWMI-Object

You can do this with the & suffix operator:
$job = Invoke-WebRequest $Url -Method $Method -Headers $headers &
$job | Receive-Job

@vexx I believe this is shortcut for start-job. I was thinking about runspace/task mechanism.

Isn't this a dupe of https://github.com/PowerShell/PowerShell/issues/9873?

It is in intent, @SteveL-MSFT, but not in implementation.
@mikeTWC1984 , #9873 suggests a syntax that is part of the "plumbing" rather than relying on parameters supported by individual cmdlets (or even a new common parameter), which to me is the better way to go.

I believe the ask is to have ubiquitous parameter to run as a threadjob, like -AsThreadJob.  This would be more discoverable than a new background operator.  We DO currently have both -AsJob and the background operator, so I guess we COULD have both.

Making -AsThreadJob a common parameter would be... interesting. @PaulHigin, any thoughts on that?

Yes, making it a common parameters would be an alternative way to make it available to every cmdlet, but I don't think it belongs there conceptually:
It's awkward to specify  an invocation method - something that is extrinsic to the invoked cmdlet - as one of its parameters.
Note to how the existing -AsJob parameter is sensibly not a common parameters, because it only applies to cmdlets that themselves have invocation functionality, such as Invoke-Command.
Also, an operator such as the postpositional & is more comprehensive in that it also covers expressions  (to use an overly simplistic example: 1..10 & ).
As for discoverability: yes, all symbol-based functionality is not easy to discover, but the reason for choosing a symbol to begin with is that the feature is so common (or is expected to become that) that it warrants maximum concision.
With frequent use comes familiarity.
Also, to help with discoverability, the feature should be documented alongside its verbose equivalent, i.e. Start-ThreadJob, just as postpositional use of & should be documented in the Start-Job topic - but currently isn't: see https://github.com/MicrosoftDocs/PowerShell-Docs/issues/4965

There's another problem with using -AsJob as a common parameter.
In PowerShell 6.x and earlier, we have -AsJob as a parameter on some commands (most notably: Invoke-Command), and it returns a PSRemotingJob for the job that is being run in the background.
In PowerShell 7, we have -AsJob as a parameter on the new ForEach-Object -parallel parameter set, and it returns a PSTaskJob.
We also have ThreadJob as a third type for jobs that are launched using Start-ThreadJob.
(Aside: I find it quite confusing that we have ThreadJob and PSTaskJob types -- do we need two types of jobs here or can we move forward with just one?)
If we were to try going the common parameter route, do we provide a common parameter for each type of job? Or have a default job type for -AsJob and another common parameter to define the job type to use?
Plus, as @mklement0 pointed out, using common parameters here feels awkward. I'd rather we control invocation of commands using post-positional operators.
For those who are interested, I opened an RFC for this a while back when I felt this was missing and would add value to the PowerShell async story. You can read that proposal and add comments in https://github.com/PowerShell/PowerShell-RFC/pull/205.

I actually came to this from .net's async methods (like DoSmth()/DoSmthAsync()). As I understand from above there is already a mechanism to invoke cmdlet asynchronously without creating extra scopes (which was my main concern) and we just discuss implementation. I'm good with either operator or switch, latter sounds more intuitive to me, but I agree this could be awkward to see it on every  cmdlet. Probably switch appearance could be indicated inside CmdletBinding attribute.
Btw, what is a PSTaskJob? Thread job seem to create a decent overhead, unlike task that came from async methods. Wondering if PSTaskJob can improve this. Below are some dummy examples.
This eats/cloggs 100MB+  of memory on my machine (on average)
1..100 | foreach {Start-ThreadJob -ScriptBlock {sleep 1} }

This takes 1-2 MB
1..100 | foreach {([System.Net.NetworkInformation.Ping]::new()).SendPingAsync("www.example.com") }

This is all a great discussion.
Cmdlets aren't really designed to run asynchronously.  There are threading, data streaming, and state sharing issues.  Over the years PowerShell has added various ways to run script/commands asynchronously: PowerShell remoting, Start-Job, Workflows, asynchronous API.
More recently I have tried to make the async API easier to use through ThreadJobs and the new ForEach-Object -Parallel experimental feature.  This has been mostly successful but has also accentuated PowerShell's inherent limitations, in particular the large overhead required to run PowerShell scripts concurrently (see my blog post for more information: (https://devblogs.microsoft.com/powershell/powershell-foreach-object-parallel-feature/).
I am a bit ambivalent about adding syntactical ways to automatically invoke cmdlets in parallel.  The feedback we have received from the new ForEach -Parallel experimental feature makes clear that it does not meet many users expectations.  The large overhead, often slower performance, and lack of awareness of basic threading safety makes ThreadJobs and ForEach -Parallel potentially dangerous to use, or at minimum a disappointment for some users (again see blog link above for more explanation).
I would love to build something that 'just works'.  But unfortunately, due to inherent PowerShell design limitations, this is not possible and each existing concurrent model has its own benefits and side effects.

Adding -AsJob to commands that can natively support it is the right way to go. It is the existing idiom for running things asynchronously in PowerShell. And regardless of the underlying implementation, -AsJob should always return a Job2 object. PowerShell jobs are the abstraction for asynchronous operations. As long as you return a Job object, the user can use the standard *-Job cmdlets to manipulate it. If you break this abstraction, then you'll end up rewriting these cmdlets for each type of asynchronous "handle". In the OP's example Invoke-WebRequest -Uri $Url -Method $Method -Headers $h -AsJob would probably return a subclass of Job2 that wrapped a Task. Not a background job object, not a threadjob but still a job object that can be manipulated with standard patterns.

@BrucePay

Adding -AsJob to commands that can natively support it is the right way to go.

It is, but only for "meta cmdlets" that themselves invoke other commands - but that isn't the issue at hand, which is about a general-purpose, cmdlet-agnostic way to invoke any cmdlet in a separate thread.
Thanks for the background information, @PaulHigin - I think it would make sense to create a conceptual help topic based on your blog post.

I am a bit ambivalent about adding syntactical ways to automatically invoke cmdlets in parallel.

There's no way around users needing to understand the constraints and appropriate use case and burdening those that do with verbose syntax doesn't sound like the best way forward.
We already have postpositional & to make implicit Start-Job calls easy, so there's no reason not to provide the same convenience for Start-ThreadJob - see @KirkMunro's RFC at https://github.com/PowerShell/PowerShell-RFC/pull/205

Regarding documentation: I've created https://github.com/MicrosoftDocs/PowerShell-Docs/issues/4977, asking that guidance be added to about_Jobs, along with extending the topic to cover Start-ThreadJob and ForEach-Object -Parallel -AsJob too.

@PaulHigin  Thanks for the info.   So far I'm more than happy with thread jobs, it's trade-offs are quite reasonable. Question - is PSTaskJob an attempt to reduce thread overheads, or it's still something based on runspaces?

PSTaskJob is similar to ThreadJob except that it is specific to ForEach-Object -Parallel when run with the -AsJob switch.  Unlike a ThreadJob, which encompasses only one running job at a time, PSTaskJob contains child jobs for all scriptblocks invoked by ForEach -Parallel (one script block is invoked for each piped input object).  So if you run 1..10 | foreach -parallel { $_ }, there will be 10 child jobs within the PSTaskJob.  Note that throttling still applies so only 5 child jobs will run at a time if the throttle limit is set to 5.
But basically it allows you to use PowerShell job cmdlets on a foreach -parallel running asynchronously as a job.


SchemaMigrationsHistory to determine unapplied out of order migrations

This commit adds a SchemaMigrationsHistory table which:

Provides users with more information about the time and order in
which migration scripts were applied to their database. This is exposed
via the wrench migrate history command

Allows out of order migration scripts to be applied to a database by
determining what subset of migrations are yet to be applied in contrast
to comparing a single version number. This flexibility allows production
hotfixes to be applied on a diverging branch and having those changes
merged back to non production environments which may already be ahead of
production.

In addition to the new functionality around SchemaMigrationsHistory,
existing users of wrench will have their databases upgraded to use the
new tracking table by backfilling the missing history. New users of
wrench will have both SchemaMigrations and SchemaMigrationsHistory
from the start and no backfilling is necessary.
Resolves #29

Yes, please :)


New Java client and gRPC support

This pull request is mainly for adding experimental new Java client as well as gRPC support. Please be aware that APIs are subject to change any notice, until 0.4.0.
Key Changes

adaptive integration test and categorized test cases(under unit or integration group)
JPMS support(incomplete) along with multi-release jar file and JDK 17 support
use com.clickhouse as package name and maven group id
Note: clickhouse-jdbc still uses ru.yandex as before, but it will be changed starting from 0.4.0.
more features being added into clickhouse-client module:

async execution(by default)
better support of RowBinary format, and more data types like Date32, Geo types, and nested data types etc.
fluent API with builder classes
named parameter support
stream API via ClickHouseResponse.recordStream()

experimental gRPC support in clickhouse-grpc-client
renamed clickhouse-native-client to clickhouse-tcp-client (still empty there :p)

Known Issues about the gRPC client

query may hang if you're using an old version of ClickHouse - see ClickHouse/ClickHouse#27024
compression is not supported well - see ClickHouse/ClickHouse#28671
Note: you may want to disable compression for better performance.
server reports "session is locked" when executing multiple queries in one go - might be a bug at client side

TODOs - will address in separate PRs

clickhouse-client

re-write ClickHouseCluster
sort out exceptions for better error handling
primitive array support

clickhouse-grpc-client

custom registry for compression/decompression
ssl/tls support
support extremes
shaded jar with only netty(for building native image later)

clickhouse-jdbc

switch to RowBinary format

JDBC driver stays the same(except changes for test cases) so it's not risky to merge this large PR. Besides, this will unblock future development work for 0.3.2 release.


License

Hey hey. As @civil mentioned in https://github.com/go-graphite/carbonapi/pull/643, the project doesn't have the license.
According to https://choosealicense.com/no-permission/ this repo is forbidden to be used by anybody.
Please, consider adding the license.

Thank you!


Tolerate issues of the Teams API

The sync should continue in case  team service issue/unexpected response; return an empty list of team members as a fallback value. The change affects Create(), Sync() and Update()

Coverage decreased (-0.004%) to 11.45% when pulling bc8b950da43e64e33855b293ccc2cf371c44e90c on tolerate-team-service-issues into 5e1d86e31e9bf04d7069c075b10068858c292fdb on master.

:+1:


MobilePreview: Scaling issues

Hi,
in my context, when switching into mobile preview, the email template does not fit container:

It seems to be due to the scale css property approach.
As you can see, the visual editor header seems to be in a negative margin also , causing header to be trimmed.
Thanks

It's looks like a bug, please provide a reproduced example.

Sure!
StackBlitz

I didn't make the buttons header bar overflowing to top, but this seems to be related to event bubbling/focus repositioning: I've noticed in my app, when I click on the preview buttons, system scrolls the page to the preview container, but overflow issues happens only in moblie preview:

By this StackBlitz you can see also the #117 issue

As you can see by your screenshot also, the buttons header bar in mobile preview is being trimmed at top

Fixed now, please upgrade to the latest version.
   "easy-email-editor": "^4.1.10",

Just udated, but still experiencing the right empty margin issue:

The blitz I've previously shared it is updated showing the issue

Seems working fine now!
Thanks 😃


irmin: Improve cache usages in Tree 2

This PR is an alternative to #1722. It achieves the same result but with less impact on the performances.
When replaying 25k commits:

#1722: +18.6% minor words, +2.8% seconds
here: +7.6% minor words, +1.3% seconds

When replaying 25k commits with flambda:

#1722: +14.5% minor words, +3.3% seconds
here: +0.9% minor words, +0.8% seconds

@CraigFe tell me what you think

I prefer this approach, thanks. It's also possible to reify the priority order as a list, using something like this: https://github.com/CraigFe/irmin/commit/634cdf549743e5f3359dce91e7cd4bee1c6ef488. I'm not really sure if this is better.

Good point. That one can be avoided with some extra clumsiness: https://github.com/mirage/irmin/commit/111f4f8c763e8c84129ad0dc7c29d7dfb481c7dc

It's crazy to see such big impacts for so small changes

Yeah, I guess we're right in the guts of Irmin here. (Just think what our allocations will look like when we get rid of Lwt...)

Great. I've included your 2 commits and added another on top.


improve autosave reliability

clicking the square STOP button in PDE does not save the json gui state to disk
maybe using the dispose() function as described here would be more reliable?

it does not improve the behavior nor does it worsen it, but the code looks much better now, so let's keep the dispose() approach
this new implementation looks a lot easier to read:
    public static void registerAutosaveOnExitHandler() {
        GlobalReferences.app.registerMethod("dispose", singletonInstance);
    }

    public void dispose(){
        createNewAutosave();
    }

than this:
public static void registerExitHandler() {
        Runtime.getRuntime().addShutdownHook(new Thread(JsonSaveStore::createNewAutosave));
}


Long click on hyper link at bottom it display some background shadow - reported by @dharmik

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

Action Performed:

Open app
Long press on any link in the footer

Expected Result:
Long press that background shadow must not be there
Actual Result:
Long press that background shadow is there
Workaround:
unknown
Platform:

Where is this issue occurring?

iOS
Android
Mobile Web

Version Number: 1.1.38-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/154414757-413e100a-f9b7-43dc-a2fd-7555e1708aa8.mov
Expensify/Expensify Issue URL:
Issue reported by: @dharmik
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1643913495412989
View all open jobs on GitHub

Triggered auto assignment to @JmillsExpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Upwork job posted here: https://www.upwork.com/jobs/~015ff255431a5786a4

@JmillsExpensify I think a lot of process steps got skipped. This is not triaged yet.

Oh yeah I see what you mean. I'm so used to the External label for this repo but I was assigned for triage.

Triggered auto assignment to @nkuoch (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Proposal:
We just have to wrap a TouchableWithoutFeedback around it. In this way, we can add another layer to it for consistent performance.

Terms of Service

I agree I am not sure I see the bug here, looks to be working as expected?

Got it - So we should have no shadow on long press on mobile because there is no hover feedback for web? That makes sense if so!

Let's see what @shawnborton thinks before moving forward.

Just to confirm, this only happens when you long-tap? Just a simple tap doesn't launch the bg color right? I kind of think this is fine as it is. I suppose we could even actually keep the behavior but specify the background color we want.

This is super minor stuff, to be clear. That said, we also approach the links differently on OldDot versus NewDot. On desktop, OldDot changes color to provide feedback, whereas NewDot is static (no feedback). Then on mobile, long tapping the homepage link on OldDot uses the existing background, whereas on NewDot we use a grey background. So really whatever we do, we should probably make it consistent.

Another matter entirely is whether this is important enough to do anything. I originally thought this was good for consistency, but I also could see the argument that this is so minor and we have more important pursuits that we can also choose to close the issue.

I like the idea of treating long-press on mobile/tablets the same as :hover on desktop, and then aligning the desired effect between both behaviors.

Going to put a monthly on this and came back to it. It's not a priority for now.

Focused on another stuff right now.

I'm just going to close this and we can re-open down the line when details like this are more important.


s3_url is ignored in operator v0.2.5 and v0.2.6

We have a working oadp-operator configuration in OKD 4.5 with oadp-operator.v0.2.3 that uses a private minio instance.
We now tried to use that configuration on a fresh OKD 4.7 installation with the oadp-operator installed via OLM in version 0.2.5 and 0.2.6. The resulting backupstoragelocation is missing the s3Url config parameter and tries to connect to AWS. I've tried different combinations of cases, but nothing seems to work.
Our configuration:
apiVersion: v1
stringData:
  cloud: |
    [default]
    aws_access_key_id = XXXXXXXXXX
    aws_secret_access_key = XXXXXXXXXXXXXXX
kind: Secret
metadata:
  name: minio-dev-credentials
  namespace: oadp-operator
type: Opaque
---
apiVersion: konveyor.openshift.io/v1alpha1
kind: Velero
metadata:
  name: backup
spec:
  backup_storage_locations:
  - config:
      insecure_skip_tls_verify: "true"
      profile: default
      region: minio
      s3_force_path_style: "true"
      s3_url: https://10.127.XXX.XXX:9000
    credentials_secret_ref:
      name: minio-credentials
      namespace: oadp-operator
    name: default
    object_storage:
      bucket: backup-dev
      prefix: velero
    provider: aws
  default_velero_plugins:
  - aws
  - csi
  - openshift
  enable_restic: true
  olm_managed: true
  restic_resource_allocation:
    limits:
      cpu: "1"
      memory: 1Gi
    requests:
      cpu: 500m
      memory: 256Mi

Resulting BackupStorageLocation (missing s3Url).
apiVersion: velero.io/v1
kind: BackupStorageLocation
metadata:
  name: default
  namespace: oadp-operator
  ownerReferences:
    - apiVersion: konveyor.openshift.io/v1alpha1
      kind: Velero
      name: backup
      uid: 0a8f049d-6aa2-4508-971f-55dcc915394a
spec:
  accessMode: ReadWrite
  config:
    insecureSkipTLSVerify: 'true'
    profile: default
    region: minio
  objectStorage:
    bucket: backup-dev
    prefix: velero
  provider: aws

operator-log:
time="2021-07-23T12:44:31Z" level=info msg="Validating backup storage location" backup-storage-location=default controller=backup-storage-location logSource="pkg/controller/backup_storage_location_controller.go:114"

velero log:
time="2021-07-23T12:52:08Z" level=info msg="Backup storage location is invalid, marking as unavailable" backup-storage-location=default controller=backup-storage-location logSource="pkg/controller/backup_storage_location_controller.go:117"
time="2021-07-23T12:52:08Z" level=error msg="Current backup storage locations available/unavailable/unknown: 0/1/0, Backup storage location \"default\" is unavailable: rpc error: code = Unknown desc = RequestError: send request failed\ncaused by: Get \"https://backup-dev.s3.minio.amazonaws.com/?delimiter=%2F&list-type=2&prefix=velero%2F\": dial tcp: lookup backup-dev.s3.minio.amazonaws.com on <IP_ADDRESS>:53: no such host)" controller=backup-storage-location logSource="pkg/controller/backup_storage_location_controller.go:164"

Using s_3__url works. This is the variable name used here: https://github.com/openshift/oadp-operator/blob/master/roles/velero/templates/velero_bsl_aws.yml.j2#L34 but this code is identical across oadp-operator versions.
I think this is caused by something along those lines: https://github.com/operator-framework/operator-sdk/issues/2393

This is a regression. PReviously, both s3_url and s_3__url worked (although the latter wasn't something we'd expect anyone to use).

I think this is caused by the update of the ansible-operator.

We hit this issue as well with the upgraded image, here is the part that uses s_3__url on the templates role:
https://github.com/openshift/oadp-operator/blob/master/roles/velero/templates/velero_bsl_aws.yml.j2#L31-L36

This will be fixed in v0.3.0 releasing this week.


[action] add dependabot automerge workflow

Disabled automerge to master. Merge it into patches, instead.


Alignment workflow [WIP]

Full nextflow alignment workflow

[x] input param of path to folder containing one patient both tumor and normal fastq files
[ ] array for potentially intaking multiple fastq files to bwa-mem2 alignment
[x] use channels instead of pointing to paths using params
[x] auto-generate dirs needed for fastqc

Questions for future:

do we want to save all files, including middle steps (such as sorting before markduplicates) or are there only certain files we want to save in the interest of space?

added an example for a params file needed to run align.wf
reorganized workflows dir

note: because I overwrote the tool.nf files, merging this branch to main will break anything that relied on these tools previously (including the old draft align.nf).

Full nextflow alignment workflow

[x]  input param of path to folder containing one patient both tumor and normal fastq files
[ ]  array for potentially intaking multiple fastq files to bwa-mem2 alignment
[x]  use channels instead of pointing to paths using params
[x]  auto-generate dirs needed for fastqc

Questions for future:

do we want to save all files, including middle steps (such as sorting before markduplicates) or are there only certain files we want to save in the interest of space?

Let's get rid of the large intermediate files, like unsorted bams, since we just mark but don't toss out the duplicates info. Given we have a small data set, I would keep the intermediates until we/test user runs the workflow from end to end successfully (i.e., don't explicitly code in their removal just yet).


mod.json

Were is mod.json can't use mod cuz no mod.json

The mod.json is located in the assets folder, could you provide more details about why the mod.json is missing ? It's automatically included in the jars available on the releases page.

Mindustry doesn't register that u need to put it at the root of the .zip like every other mod

No you do not need to put it in the root directory as long as the folder it's in is included into the jar/zip which is the case here https://github.com/Redstonneur1256/Mindustry-ModLib/blob/7cdce7011da2fe630458453c2ef5420714b25858/Mod/build.gradle#L15

For me it showed an error

How did you import the mod


resolv.conf gets overwritten

Just setup an rpi3 with raspbian jessie from the link off your site and the install went fine but the nameserver addition for server <IP_ADDRESS>. in the /etc/resolv.conf is not there. It just has an entry for localhost and at the top of the file it says it was generated by resolv.conf. I've been trying to fix this as I have most of setup working, well not all. but the rpi cant even resolve the nodes name its connected to. it can resolve itself though . This is from a fresh installs with no errors but I dont think this file should be getting deleted.
I am running a gateway hsmm-pi install from a vmware running ubuntu 14.04. That seems to be fine, I just cant get a client thats plugged into the rpi lan (<IP_ADDRESS> net) to route to internet. but the two nodes are pinging on the 10.x net and I see packet requests going out with tcpdump but I think they are getting refused.

Disregard this,  everything seems to be working fine. I currently have two virtual machines running ubuntu  and one raspberry pi 3.  Im abel to route across all meshs and Len networks and I even updated it with your enhancements.   I never really got the port forwarding working but even with the net settings I am able to reach the land that works off of each Mash from wherever I am in the Mash so it's no big deal.  However I think it would be cool if there is no use for the services forwarding to at least put injuries in there that will do real IP forwarding to Web server Ases that might be across the mash as a nice menu kind of like how they implement SSL services over the secured appliance.


[Bug] In contact page the card is overflowing

Description
The card content in contact page is overflowing
Screenshots

i want to fix this issue under gssoc
Can you assign it to me?
Full name: Usha Rani
GitHub profile link: https://github.com/usharani222
Email ID<EMAIL_ADDRESS>linkedin: https://www.linkedin.com/in/usha-rani-a26098264/
What is your participant role?: GSSoC-2024 contributor
Can you add the label for GSSoC, please?
Thanks.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

I am a proficient front-end developer with expertise in HTML, CSS, JavaScript, and React. Assign me this issue
Thanks

Please assign me this issue @Rakesh9100

For which screen are you facing this issue @usharani222

On laptop
Assign me this issue @Rakesh9100


Remove insecure tls parameters or make them configurable

Describe the solution you'd like
We are currently running trident in a kubernetes cluster as a csi provisioner (installed using tridentctl).
Each of the pods of the trident daemonset is listening on port 34572 for https requests. This https session does however support outdated versions/ciphers. In particular:

TLS 1.0
TLS 1.1
TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA
TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA
It is however not possible to change the supported ciphers via configuration. Could trident be adjusted so that either the cipher/tls version list is configurable; or that only secure ciphers/tls versions are choosen?

Describe alternatives you've considered
None: supporting TLS 1.0 and 1.1 is not something that is needed
Additional context
Currently used version is netapp/trident:21.04.0

The HTTPS server should only be allowing a minimum of TLS 1.2, have you been able to verify that it's allowing 1.0 and 1.1?
https://github.com/NetApp/trident/blob/bed7323ba00aa774468ab85be452fc07b15fd5aa/config/config.go#L107
https://github.com/NetApp/trident/blob/bed7323ba00aa774468ab85be452fc07b15fd5aa/frontend/rest/apiserver_https.go#L37

Additionally, the node pods should only be opening a port on 17546 by default to listen for liveness probes from Kubelet, only the controller pod (running from the deployment not the daemonset) listens on port 34572.

it seems to be open on all of our nodes, so it seems to be attached to the daemonset.
The container does seem to start a /trident_orchestrator process that opens that port
xxx@k8s-worker97:~$ sudo netstat -tulpn  | grep 34572
tcp6       0      0 :::34572                :::*                    LISTEN      4929/trident_orches
xxx@k8s-worker97:~$ ps aux | grep trident_orches
root        4929  0.7  0.0<PHONE_NUMBER>6 ?        Ssl   2021 877:48 /trident_orchestrator --no_persistence --rest=false --csi_node_name=k8s-worker97 --csi_endpoint=unix://plugin/csi.sock --csi_role=node --log_format=text --node_prep=false --https_rest --https_port=34572 -debug

And i can connect to it using TLS1.0 (the strange openssl path comes from my default openssl lib not supporting the required ciphers anymore)
xxx@k8s-worker97:~$ ./bin/openssl.Linux.x86_64.static s_client -connect localhost:34572 -tls1
---snip---
New, TLSv1/SSLv3, Cipher is ECDHE-ECDSA-AES128-SHA
Server public key is 521 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1
    Cipher    : ECDHE-ECDSA-AES128-SHA
    Session-ID: xxx
    Session-ID-ctx:
    Master-Key: xxx
    Key-Arg   : None
    PSK identity: None
    PSK identity hint: None
    SRP username: None
---snip---

I was quite confused as i saw the TLS1.2 configuration as well and i'm now not sure where this comes from

So the port number being different seems to be coming from your DaemonSet settings. It's giving the following option to the trident binary --https_port=34572 which is overwriting the default 17546. If that's not causing in port conflicts then there's no real reason to change that right now. However I'm not understanding why the other protocols are being allowed. We do run a port scan as part of our release testing and ever since we updated the minimum TLS version is has not shown those protocols being available on the probe port.
I'll change this to a bug and track it so we can try to reproduce. Are you by any chance able to try upgrading to the latest trident to see if it has the same issue?

Thanks for the suggesstion.
After the upgrade to 21.10.1 the port changed to 17546 and now only tls 1.2 is supported.
However the cbc ciphers are still supported:

TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA
TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA

Thanks for verifying with the latest version. I'll have the team look into disabling the obsolete ciphers as well. Do keep in mind though that this port is only used for liveness and readiness probes so there's not really any sensitive info available on this port.

TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA is flagged as non-secure from Tenable's Nessus, perhaps others. Any plans to drop it from code?

Hi @flngen,
There isn't anything in the Trident code that specifically uses that cipher. We bumped the minimum server TLS version to 1.3 as you can see in the above referenced commit.
You can find more information about what ciphers are included for specific TLS versions in Go in the crypto package documentation.


[Unity][Training] Loss functions and AppendLoss pass

This PR adds two components to support the Relax training workflow: loss function library and AppendLoss pass.

Loss function library. A library that generates relax functions representing common loss functions, such as MSELoss and CrossEntropyLoss.

Generally, loss function will take one or more input parameters (that is outputs of the backbone of a model), one or more target parameters, and generate a scalar value denoting the loss.

AppendLoss pass. You can specify one function (the backbone function) in the IRModule, and one loss function (normally, generated by the loss function library) to AppendLoss. It will attach the loss function to the backbone function, and add the result function to the IRModule.

Co-authored-by: Chaofan Lin<EMAIL_ADDRESS>

Please resolve the conflict and we’re good to go :-)

@MasterJH5574 We can go now :)


Fixed TestDLTensorMemory.test_delete

Closes #2444
In some cases, the default memory pool might not be empty after free_all_blocks is called. This is happening because the gc has yet to collect some dead objects and free their references so the pool chunks can be actually freed.
This solves the issue in Jenkins by creating a new pool only for this test.

pfnCI, test this please.

Updated! thanks 😃

Jenkins CI test (for commit 2bbff865aee13c844fe5b9bb08a0a1d337139f6a, target branch master) succeeded!

PTAL

Thanks for all your help in the debug process!

pfnCI, test this please.

Jenkins CI test (for commit 9eeb599eae257ef3727cc20eb4a60c67fe9d6103, target branch master) succeeded!

Jenkins CI test (for commit 9eeb599eae257ef3727cc20eb4a60c67fe9d6103, target branch master) succeeded!


remove cmt

We may want to deprecate the lib rather than delete it? Don't know who else uses it online? Is there a proper way of doing this in open source?


Why does ionic io init modify ionic-core.js?

ionic-service-core is installed to www/lib/ via bower, so why does running ionic io init modify the file, rather than utilizing the credentials in .io-config.json somehow?

Greetings @angusfretwell!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


Bot return utter name as message

Describe the bug
when we use the test chat window, the bot returns the utter name instead of the text related to that utter
2020-03-24 16:29:38 DEBUG    rasa.core.processor  - Action 'utter_Zna1OsI67' ended with events '[BotUttered('utter_Zna1OsI67', {"elements": null, "quick_replies": null, "buttons": null, "attachment": null, "image": null, "custom": null}, {"metadata": null},<PHONE_NUMBER>.7234225)]'.
To Reproduce
Steps to reproduce the behavior:
we tested on botfront 0.20.3 and 0.21.3 with rasa 1.7.1
Expected behavior
show utter text instead of utter name
Screenshots

here we can see the utter has its text assign

Desktop (please complete the following information):

OS: Linux and Windows
Browser firefox and chrome

Is it the same response name when you hover the response in the story?

I found an interesting behavior when deploying in docker swarm. It seems it is not routing the dns call to the local namespace services, but somehow it is also reaching other services. The some API calls and Rasa calls were going to another botfront instance we have in the cluster.
I will close this issue because it is not related with botfront, or at least it seems not to be.
Thanks for the quick response.
all the best


Add a Content settings Icon Button

All work for the 'Create new header' milestone should be done under the new_header feature flag.
Description

The new header component has a Content settings Icon Button for selecting content types (audio/images/all) and filters on screens below md breakpoints.
API

Props

Name
Type
Description

isPressed
Boolean
Whether the button has been pressed (activated). When it is pressed, its colors are inverted, and the background is dark charcoal

areFiltersSelected
Boolean
Whether some filters have been selected. It should show a pink dot in the top right corner.

aria-controls
String
The id of the element it controls. It should be the modal on mobile; can be left blank until we implement the new mobile modals. Should have a default (as it would probably have a single value in prod)

Data and computeds
It should accept the following attrs to pass them on to the underlying button component.

Name
Type
Description

aria-label
String
Translated name of the button to be used as accessible name

aria-pressed
Boolean
Based on the isPressed prop.

Events

Name
Associated data
Description

click
-
The click event

Code samples

<VContentSettingsButton
  :is-pressed="false"
  :are-filters-selected="true"
>
</VContentSettingsButton>

Dependencies

VIconButton
The settings SVG from the Figma file.
References

Figma: https://www.figma.com/file/KxkZmBNk218V2Fke1GeYCu/New-header-component?node-id=522%3A59429

Implementation

[ ] 🙋 I would be interested in implementing this component.

Here are some screenshots from the Figma file. Please note that the screenshots are subject to change, and always consult the linked Figma file.
The Content settings IconButton inside the mobile header:

The states of the IconButton:


Port https://github.com/microsoft/FluidFramework/pull/5773 to 0.38

This change did not make it into 0.38 by one day.
After discussing it with Jade, we are running out of time to get it into hosts (including Teams) before GA on 5/7, so porting looks like responsible step, given that release likely did not make it yet to Bohemia.
Please let me know if you feel it's too risky given where we are.
Addresses #5477
It gives a lot of flexibility to developers to control loading process.
In particular, it allows Bohemia to apply cached ops before container is returned, and (optionally) allow ops from storage to continue to trickle in through initial boot / render cycle to speed up catch up process (this would need to be tested to ensure existing code behaves correctly in presence of changes).

I don't think it's integrated yet. I don't see any rush on this. I can do a patch release later tonight if needed.

Thanks! Yes, it's a port


ERC1155 item metadata

The metadata for each item looks like:
{
	"name": "Ration",
	"description": "A portable food ration",
	"external_url": "https://openseacreatures.io/3",
	"image": "https://storage.googleapis.com/opensea-prod.appspot.com/puffs/0.png"
}

Please comment or directly add the metadata for name, description, and external_url (our marketing site) in the metadata folder

Pictures will be updated later on to NFT Storage and we'll get the IPFS link


[dev] de-dent issues

Need to simplify this but
\newcommand{\fieldset}[3]{
  ~~~
  <fieldset class="#1"><legend class="#1-legend">#2</legend>
  ~~~
  #3
  ~~~
  </fieldset>
  ~~~
}

\newcommand{\showmd}[1]{
  \fieldset{md-input}{markdown}{
    `````markdown
    #1
    `````
  }
  \fieldset{md-result}{result}{
    ~~~~~~

    #1

    ~~~~~~
  }
}

and using it with something like
# A
## B

will lead to the first one being tabbed.

simplified version
s = raw"""
        \newcommand{\coma}[1]{
            #1
        }
        \newcommand{\comb}[1]{
            \coma{
                ```markdown
                #1
                ```
            }
        }

        \comb{
            X1
            X2
        }
        """

now fixed by having a more complex thing where we check the whitespaces before #... and add those in the injection after \n so that new lines are aligned.
So with this:
    @test h // "
X1\nX2
"


Default logo does not show up with Docker

Related to issue #9 . Same issue with the official Kanboard Docker container. It is based on Nginx. Don't know how to fix the issue, Every other theme I tried in the Extension section is working correctly.

Same problem over here on docker as well.
Failed to load resource: the server responded with a status of 404 ()

Hi @ptoulouse @Lennart97 :partying_face: Happy New Year! Try adding the following the configuration file is named nginx.conf and placed in the directory /usr/local/nginx/conf, /etc/nginx, or /usr/local/etc/nginx
location ~ \.(gif|jpg|png)$ {
        root /data/files/Nebula/Assets/images;
}

or
location /images/ {
        root /data/files/Nebula/Assets;
}

and let me know if it resolves.
P.S. I haven't had a chance to test but this should be the way

Hi @ptoulouse @Lennart97 :partying_face: Happy New Year! Try adding the following the configuration file is named nginx.conf and placed in the directory /usr/local/nginx/conf, /etc/nginx, or /usr/local/etc/nginx
location ~ \.(gif|jpg|png)$ {
        root /data/files/Nebula/Assets/images;
}

or
location /images/ {
        root /data/files/Nebula/Assets;
}

and let me know if it resolves.
P.S. I haven't had a chance to test but this should be the way