OptionalArgumentFactory and PostgreSQL

When the Optional value is absent, PostgreSQL always fails with an exception: org.postgresql.util.PSQLException: ERROR: could not determine data type of parameter $n.
But changing OptionalArgumentFactory from
 statement.setNull(position, Types.OTHER);

to
statement.setNull(position, Types.VARCHAR);

seems to fix this problem. Anybody else has experienced this?
Postgresql JDBC: 9.4-1200-jdbc41 and 9.3-1102-jdbc41
Dropwizard: All SNAPSHOTS and RC of 0.8.0

@kilemensi Thanks for the report. Which version of PostgreSQL and Java do you use?
Unfortunately, I can't reproduce your problem with the following test case:
create table departments(
  id bigint primary key,
  name varchar(32) not null
);

insert into departments(id, name) values (1, 'Sales'), (2,'Development'),
(3, 'Accounting'), (4, 'IT');

create table employees(
  id bigint primary key,
  first_name varchar(32) not null,
  last_name varchar(32) not null,
  department_id bigint references departments(id)
);

insert into employees(id, first_name, last_name, department_id) values
(1, 'Jim', 'Fields', 2),
(2, 'Gregory', 'Dunn', 4),
(3, 'Lisa', 'Myers', 1),
(4, 'Colin', 'Wilson', 2),
(5, 'Jim', 'Button', 3),
(6, 'Den', 'Dunn', 3),
(7, 'Lisa', 'Hoffman', 3),
(8, 'Colin', 'Button', 1)

drop table employees;
drop table departments;

public interface EmployeeDao {

    @SqlQuery("select id from employees " +
            "where first_name = :first_name " +
            "or    last_name = :last_name " +
            "or    department_id = :department_id")
    List<Long> find(@Bind("first_name") Optional<String> firstName,
                    @Bind("last_name") Optional<String> lastName,
                    @Bind("department_id") Optional<Long> departmentId);
}

public class PostgresOptionalTest {

    Handle handle;
    EmployeeDao employeeDao;

    @Before
    public void setUp() throws Exception {
        DBI dbi = new DBI("jdbc:postgresql://<IP_ADDRESS>:5432/dw_optional", "dw", "pass");
        dbi.setSQLLog(new PrintStreamLog());
        dbi.registerArgumentFactory(new OptionalArgumentFactory(null));

        handle = dbi.open();
        handle.createScript("employees.sql").execute();
        employeeDao = handle.attach(EmployeeDao.class);
    }

    @After
    public void tearDown() throws Exception {
        handle.createScript("clear.sql").execute();
        handle.close();
    }

    @Test
    public void testFindByFirstName() {
        List<Long> result = employeeDao.find(Optional.of("Lisa"), Optional.absent(), Optional.absent());
        System.out.println(result);
        Assert.assertEquals(ImmutableList.of(3L, 7L), result);
    }

    @Test
    public void testFindByDepartment() {
        List<Long> result = employeeDao.find(Optional.absent(), Optional.absent(), Optional.of(2L));
        System.out.println(result);
        Assert.assertEquals(ImmutableList.of(1L, 4L), result);
    }
}

My configuration:

dropwizard 0.8.0-rc3
jdbi 2.59
guava 18.0
postgresql 9.4-1200-jdbc41
PostgreSQL 9.3.5 on x86_64
Java HotSpot(TM) Server VM (build 25.25-b02, mixed mode)

Can you, please, provide a test case where the error is reproduced?

@arteam
Sorry if I wasn't clear, the problem occurs when you're inserting a value into the database rather than when reading. If you use any complex statement, postgres looses type information on absent Optional value.
Using your example, lets make last_name null i.e.
create table employees(
  id bigint primary key,
  first_name varchar(32) not null,
  last_name varchar(32) null,
  department_id bigint references departments(id)
);

Lets then add the a complex insert into the EmployeeDao as shown below:
    @SqlUpdate("WITH employee_info AS ("
            + "     SELECT :id AS id, :firstName AS first_name, "
            + "             :lastName AS last_name,"
            + "             :departmentId AS department_id"
            + " )"
            + " INSERT INTO employees (id, first_name, last_name,"
            + "          department_id)"
            + "     SELECT * FROM employee_info")
    int insert(@Bind("id") Long id,
            @Bind("firstName") String firstName,
            @Bind("lastName") Optional<String> lastName,
            @Bind("departmentId") Long departmentId);

Adding testInsert into PostgresOptionalTest
    @Test
    public void testInsert() {
        final Long id = 10L;
        final String firstName = "Artem";
        final Long departmentId = 3L;

        final int insertCount = employeeDao.insert(id, firstName,
                Optional.<String>absent(), departmentId);
        Assert.assertEquals(1, insertCount);
    }

will thrown the exception in question.

Aha, I now see your problem.
First, as a workaround you can specify a type excplicetely in the SQL query.
@SqlUpdate("WITH employee_info AS ("
        + "     SELECT :id AS id, :firstName AS first_name, "
        + "             cast(:lastName AS VARCHAR) last_name,"
        + "             :departmentId AS department_id"
        + " )"
        + " INSERT INTO employees (id, first_name, last_name, department_id)"
        + " SELECT * FROM employee_info")
int insert(@Bind("id") Long id,
           @Bind("firstName") String firstName,
           @Bind("lastName") Optional<String> lastName,
           @Bind("departmentId") Long departmentId);

Speaking about a generic solution of this problem, it's not so easy.
We have 2 major problems:

Resolve a type binding of an Optional argument
Figure out the correct SQL type for the corresponding Java type

The second problem can be solved by delegating it to JDBI's BuiltInArgumentFactory.
But the first is tough one. Information about a parametrized type can be get from the corresponding method through reflection. But  we don't have runtime information about a position of an
argument, so it's impossible to say which parameter of the method is currently being handled.
I guess, to the best of my knowledge, we can't do anything about that now without
assisting of JDBI.

Yeah, I thought about adding explicit casts but I'm heavily using Optional in my code and hence would require a lot of casting.
I guess for now I'll stick to using my custom OptionalArgumentFactory.

I'm not sure what has changed, but I'm using Drowpizard 1.0.2, Postgres, and JDBI for a project and the following binds work fine without needing to cast:
@Bind("a") OptionalLong a,
@Bind("b") Optional<LocalDateTime> b,
@Bind("c") String c,
@Bind("d") Optional<byte[]> d

I'll close this issue in a few weeks time if no new info is presented.

Hi @nickbabcock
The issue was when Optional is empty, dropwizard/jdbi could not figure out what type (it worked fine otherwise) of null it was. To quote @arteam:

Information about a parametrized type can be derived from the corresponding method through reflection. But we don't have runtime information about a position of an argument, so it's impossible to say which parameter of the method is currently being handled

I still think this is the case but since I'm in the home streatch of another project, I can't really test to see if the situation has really been fixed or not in the project I was working on when I reported this issue.

I'm optimistic that this may be fixed, as I can confirm that optionals that are empty are being persisted as null

I'm closing this because I have several apps in production using Dropwizard 1.0 + jdbi + postgres (9.5.5 server with 9.4-1212 driver) + empty java.util optionals, and I see empty optionals persisted as null.
If the erroneous behavior still persists for the latest versions, please post a repo for me to cross examine.


Code for real-to-sim system identification

First, thanks for sharing the code! May I ask if the optimization code for real-to-sim system identification is available somehow?

Hi! Thanks for asking! We will push the codes of real2sim soon.

Hi @Si-Lynnn! Any update on the release of this portion of the code ?


Add input types to attributes

I want to merge this change because it adds input types to new attributes.
Resolves #4254
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Here is the report for 30279ee836c5b619b25709663861200512ac116e (mirumee/saleor @ attributes/add-input-types)
Base comparison is 7cdebe3efa8f95373791756856917cfb6063613a.
No differences were found. (click me)

# api.benchmark checkout
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  add billing address to checkout     	         41	         41	             24
  add shipping to checkout            	          7	          7	              0
  checkout payment charge             	         16	         16	              0
  complete checkout                   	          6	          6	              0
  create checkout                     	         45	         45	             21

# api.benchmark homepage
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve main menu                  	          5	          5	              0
  retrieve product list               	          4	          4	              0
  retrieve secondary menu             	          5	          5	              0
  retrieve shop                       	          2	          2	              0

# api.benchmark product
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  product details                     	         15	         15	              3

# api.benchmark variant
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve variant list               	         15	         15	              6

When we create an attribute value, it looks like the admin value is the value field (which is a meta, e.g. a color code) and when the dashboard renders the returned admin value by GQL, it shows the slug.
Didn't you mean to use the slug field instead of the value field?

Not a part of this PR but noted


rust-analyzer crashes on this project

I cannot get rust-analyzer working in VS Code on this project. It seems to insist on needing selinux/selinux.h, which it doesn't seem that any package on Arch Linux provides (at least according to pacman -F, which searches for which package provides a file).
(Worth noting: I will not be installing SELinux outside the official distribution channels by hand or from the user provided packages on AUR, should that be required.)
I tried adding the following to the workspace settings, to prevent it from activating the selinux feature, but it made no difference:
    "rust-analyzer.cargo.features": [
        "unix"
    ]

The full error I get is:
[ERROR rust_analyzer::lsp_utils] failed to run build scripts:
error: failed to run custom build command for `selinux-sys v0.6.1`

Caused by:
  process didn't exit successfully: `/home/user/src/rust/coreutils/target/debug/build/selinux-sys-954ab2152425c9b2/build-script-build` (exit status: 101)
  --- stdout
  cargo:root=/home/user/src/rust/coreutils/target/debug/build/selinux-sys-b9d426a5e041f579/out
  cargo:rerun-if-env-changed=SELINUX_STATIC_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=SELINUX_STATIC_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=TARGET_SELINUX_STATIC
  cargo:rerun-if-env-changed=SELINUX_STATIC
  cargo:rerun-if-env-changed=SELINUX_PATH_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=SELINUX_PATH_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=TARGET_SELINUX_PATH
  cargo:rerun-if-env-changed=SELINUX_PATH
  cargo:rerun-if-env-changed=SELINUX_INCLUDE_DIR_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=SELINUX_INCLUDE_DIR_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=TARGET_SELINUX_INCLUDE_DIR
  cargo:rerun-if-env-changed=SELINUX_INCLUDE_DIR
  cargo:rerun-if-env-changed=SELINUX_LIB_DIR_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=SELINUX_LIB_DIR_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=TARGET_SELINUX_LIB_DIR
  cargo:rerun-if-env-changed=SELINUX_LIB_DIR
  cargo:rerun-if-env-changed=CC_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=CC_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=TARGET_CC
  cargo:rerun-if-env-changed=CC
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=TARGET_CFLAGS
  cargo:rerun-if-env-changed=CFLAGS
  OPT_LEVEL = Some("0")
  TARGET = Some("x86_64-unknown-linux-gnu")
  HOST = Some("x86_64-unknown-linux-gnu")
  cargo:rerun-if-env-changed=CC_x86_64-unknown-linux-gnu
  CC_x86_64-unknown-linux-gnu = None
  cargo:rerun-if-env-changed=CC_x86_64_unknown_linux_gnu
  CC_x86_64_unknown_linux_gnu = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-linux-gnu
  CFLAGS_x86_64-unknown-linux-gnu = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_linux_gnu
  CFLAGS_x86_64_unknown_linux_gnu = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("true")
  CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")
  OPT_LEVEL = Some("0")
  TARGET = Some("x86_64-unknown-linux-gnu")
  HOST = Some("x86_64-unknown-linux-gnu")
  cargo:rerun-if-env-changed=CC_x86_64-unknown-linux-gnu
  CC_x86_64-unknown-linux-gnu = None
  cargo:rerun-if-env-changed=CC_x86_64_unknown_linux_gnu
  CC_x86_64_unknown_linux_gnu = None
  cargo:rerun-if-env-changed=HOST_CC
  HOST_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64-unknown-linux-gnu
  CFLAGS_x86_64-unknown-linux-gnu = None
  cargo:rerun-if-env-changed=CFLAGS_x86_64_unknown_linux_gnu
  CFLAGS_x86_64_unknown_linux_gnu = None
  cargo:rerun-if-env-changed=HOST_CFLAGS
  HOST_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("true")
  CARGO_CFG_TARGET_FEATURE = Some("fxsr,sse,sse2")

  --- stderr
  thread 'main' panicked at 'selinux-sys: Failed to find 'selinux/selinux.h': Kind(NotFound)', /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/selinux-sys-0.6.1/build.rs:224:10
  stack backtrace:
     0: rust_begin_unwind
               at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/std/src/panicking.rs:575:5
     1: core::panicking::panic_fmt
               at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/panicking.rs:64:14
     2: core::result::unwrap_failed
               at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/result.rs:1791:5
     3: core::result::Result<T,E>::expect
               at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/result.rs:1070:23
     4: build_script_build::find_and_output_include_dir
               at ./build.rs:223:24
     5: build_script_build::main
               at ./build.rs:25:24
     6: core::ops::function::FnOnce::call_once
               at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/ops/function.rs:507:5
  note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

So how do I get past this so that rust-analyzer can work and I can contribute to this project?

That's unfortunate indeed. I've removed the selinux headers from my machine and narrowed the problem down to two utils: chcon and runcon. The problem is that for these utils, the selinux dependency is not optional, because that would make them useless, but the entire util is optional. However, because each util is a crate and rust-analyzer tries to analyze all crates in the workspace, rust-analyzer still fails on them. I'll get back to you once I have a solution (and if others have ideas in the meantime I'd love to hear them).

It turns out that cargo test --workspace also fails without the headers. Here's a relevant issue: https://github.com/rust-lang/cargo/issues/11526

I couldn't find a way to do this, so I opened an issue at rust-analyzer. Hopefully they can build some solution for this!

Maybe I can second a bit. As far as I can remember I solved this issue by installing the libselinux package from AUR, which includes the headers:
$ pacman -Ql libselinux
# ... a lot of stuff
libselinux /usr/include/selinux/
libselinux /usr/include/selinux/avc.h
libselinux /usr/include/selinux/context.h
libselinux /usr/include/selinux/get_context_list.h
libselinux /usr/include/selinux/get_default_type.h
libselinux /usr/include/selinux/label.h
libselinux /usr/include/selinux/restorecon.h
libselinux /usr/include/selinux/selinux.h
libselinux /usr/lib/
libselinux /usr/lib/libselinux.so
libselinux /usr/lib/libselinux.so.1
# ... and more stuff

I haven't tried but you may also get around the missing headers if you download just the headers and install them in /usr/include/selinux so you don't have all the other stuff installed with them.

This can be resolved on Ubuntu by installing libselinux1-dev

Thank you, I had the same problem with CLion (Rust Plugin) even with selinux feature disabled.


Add the ability to create a user  through an API endpoint

also added username, password, and email validation, though whether things are actually going through the username validator is questionable


dialog & form validation

Is there anywhere a sample to validate a form in the dialog control with  jquery validate

no. we don't have such example. you can use the jquery validate demos and use them inside dialog and everything should be fine.
https://jqueryvalidation.org/files/demo/


squeezeNet

'实训作业提交内附项目飞桨地址'

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

实训实现


[FLINK-12707]Close minicluster will cause memory leak when there are StreamTask closed abnormal

https://issues.apache.org/jira/browse/FLINK-12707
There are several threads in my application,and every thread will launch a flink job with LocalStreamEnvironment/MiniCluster.But when I interrupt these threads again and again, I found there are many residue threads,and that caused a memory leak.
When I debug it,this message appears "Recipient[Actorakka://flink/user/taskmanager_0#-584606215] had already been terminated. Sender[null] sent the message of type "org.apache.flink.runtime.rpc.messages.LocalRpcInvocation". So I think
when flink close minicluster,TaskExecutor will be closed in the first place,and this operation will cause akka message which will close StreamTask abnormal.So the work thread will be more and more.
I call Thread sleep in my application to avoid this problem,any good suggestions?

Thanks for opening this PR @liu-zhaokun. I'm not sure whether this solves the underlying problem. I'll close this PR. Let's discuss on the JIRA first what we want to solve.


[opencv4] build failure

Host Environment

OS: Windows7
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
./vcpkg install darknet[full]:x64-windows
Failure logs
-Cut and past the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
Computing installation plan...
The following packages will be built and installed:
    darknet[core,cuda,cudnn,full,opencv-cuda,weights,weights-train]:x64-windows -> 2020-08-30#1
  * opencv[core,cuda,dnn,ffmpeg,jpeg,opengl,png,tiff,webp]:x64-windows -> 4.3.0
  * opencv4[contrib,core,cuda,dnn,ffmpeg,jpeg,opengl,png,tiff,webp]:x64-windows -> 4.3.0#4
  * pthreads[core]:x64-windows -> 3.0.0-6
  * stb[core]:x64-windows -> 2020-09-14
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...

-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-opencv-4.3.0.tar.gz
-- Cleaning sources at C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/buildtrees/opencv4/src/4.3.0-58520c6ae3.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-opencv-4.3.0.tar.gz
-- Applying patch 0001-disable-downloading.patch
-- Applying patch 0002-install-options.patch
-- Applying patch 0003-force-package-requirements.patch
-- Applying patch 0004-fix-policy-CMP0057.patch
-- Applying patch 0006-fix-vtk9.patch
-- Applying patch 0006-jpeg2000_getref.patch
-- Applying patch 0009-fix-uwp.patch
-- Applying patch 0010-fix-interface_link_libraries.patch
-- Using source at C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/buildtrees/opencv4/src/4.3.0-58520c6ae3.clean
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/tiny_dnn/adb1c512e09ca2c7a6faef36f9c53e59-v1.0.0a3.tar.gz
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/nvidia_optical_flow/ca5acedee6cb45d0ec610a6732de5c15-79c6cee80a2df9a196f20afd6b598a9810964c32.zip
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-opencv_contrib-4.3.0.tar.gz
-- Cleaning sources at C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/buildtrees/opencv4/src/4.3.0-1c7c3cc613.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-opencv_contrib-4.3.0.tar.gz
-- Applying patch 0005-add-missing-stdexcept-include.patch
-- Applying patch 0007-fix-vtk9-contrib.patch
-- Using source at C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/buildtrees/opencv4/src/4.3.0-1c7c3cc613.clean
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/0ea90e7a8f3f7876d450e4149c97c74f-boostdesc_bgm.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/232c966b13651bd0e46a1497b0852191-boostdesc_bgm_bi.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/324426a24fa56ad9c5b8e3e0b3e5303e-boostdesc_bgm_hd.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/202e1b3e9fec871b04da31f7f016679f-boostdesc_binboost_064.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/98ea99d399965c03d555cef3ea502a0b-boostdesc_binboost_128.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/e6dcfa9f647779eb1ce446a8d759b6ea-boostdesc_binboost_256.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/0ae0675534aa318d9668f2a179c2a052-boostdesc_lbgm.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/e8d0dcd54d1bcfdc29203d011a797179-vgg_generated_48.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/7126a5d9a8884ebca5aea5d63d677225-vgg_generated_64.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/7cd47228edec52b6d82f46511af325c5-vgg_generated_80.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/151805e03568c9f490a5e3a872777b75-vgg_generated_120.i
-- Using cached C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/opencv-cache/data/7505c44ca4eb54b4ab1e4777cb96ac05-face_landmark_model.dat
-- Configuring x64-windows
-- Building x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:145 (message):
    Command failed: C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/downloads/tools/cmake-3.18.4-windows/cmake-3.18.4-win32-x86/bin/cmake.exe --build . --config Debug --target install -- -v -j9
    Working Directory: C:/Users/Unknown/Documents/AI/darknet/darknet/vcpkg/buildtrees/opencv4/x64-windows-dbg
    See logs for more information:
      C:\Users\Unknown\Documents\AI\darknet\darknet\vcpkg\buildtrees\opencv4\install-x64-windows-dbg-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_cmake.cmake:96 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_cmake.cmake:27 (vcpkg_build_cmake)
  ports/opencv4/portfile.cmake:362 (vcpkg_install_cmake)
  scripts/ports.cmake:136 (include)

Error: Building package opencv4:x64-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: opencv4:x64-windows
  Vcpkg version: 2020.11.12-nohash

Additionally, attach any relevant sections from the log files above.
[install-x64-windows-dbg-out.log](https://github.com/microsoft/vcpkg/files/5690929/install-x64-windows-dbg-out.log)

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.
Error-Log:
install-x64-windows-dbg-out.log

@Snickbrack ,Duplicate to #15055

@JonLiu1993 Hi, no this is not a duplicate as I have a different error than the guy in your mentioned issue.

@Snickbrack ,Sorry for my fault, i will reopen it, thanks

OpenCV 4.3 is not compatible with CUDNN 8
Please downgrade your CUDNN installed version. The problem is unfortunately triggered by a recent update to cudnn port, which uses what's installed on your machine instead of a fixed version by vcpkg (very bad idea imho, but still...)

But any 7.x Version will fix this?

yes, just pick one which is compatible with your cuda version

Okay, I have downloaded Cudnn with Version "v<IP_ADDRESS>" and it worked!
(But for installing Darknet I also needed to do some "dirty" fixes because of wrong hashes of downloaded files)

(But for installing Darknet I also needed to do some "dirty" fixes because of wrong hashes of downloaded files)

Please open an issue about that, I'd like to fix it if possible


feat: add router to vue context

Since the react implementation is already return the router instance on the context object, I've adapted these changes to vue.

See discussion here: #80


Is it possible to integrate with react starter kit from http://www.reactstarterkit.com/?

I'm using react starter kit from (http://www.reactstarterkit.com/) to develop my application. Is this lib compatible with the starter kit? What are all the configuration changes i'm expected to do in order to integrate it to the starter kit?

I haven't used react starter kit yet, so, I can't tell. Probably, I will write a starter kit specific to isomorphic-relay later.
Currently, I recommend you to use one of the examples as a starting point:
https://github.com/denvned/isomorphic-relay/tree/master/examples/star-wars
https://github.com/denvned/isomorphic-relay-router/tree/master/examples/todo


Rename "ucon" with a more telling name

Just a remark for the "Umstrukturierung" of the modules: "ucon" is the only module with a name that gives no clue as to its contents.

Thanks Anna, UCON is a shortcut for "Usage CONtrol", I will rename it.


use api_get method for getting report template

use the api_get method for getting report template, instead of the raw HTTParty.get method that was left there

Coverage remained the same at 96.725% when pulling d3cc1ac215e42da4479a80e0fc685deca2991e94 on fakenine:fix_proxy_for_reports into db3c0307dd763fe47b1df09c7dd11a6cefdaf96a on Cyberwatch:master.


Improve documentation

I've based this atop my other two changesets from today. Let me know if you want me to separate it out.
Among stylistic changes, additions of a lot of descriptions, and some reorganizing, some undocumented commands have been added.

Needs rebase after #96 was merged.

Rebased.

I can switch the order of this and #98 if you prefer.

Awesome improvement to the documentation, and highly appreciated.
Yes, it would be beneficial to rearrange the order, this can be merged right away as long as the new viewport commands in #98 are moved to that PR.

OK, this one is now directly on top of current master.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #100   +/-   ##
=======================================
  Coverage   53.81%   53.81%
=======================================
  Files          51       51
  Lines        1797     1797
=======================================
  Hits          967      967
  Misses        807      807
  Partials       23       23

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 e2d2e50...f35bbd0. Read the comment docs.

Just changed this a little since it's clear you prefer the *  style of lists in markdown rather than the -  style.


Feature: Allow changing display format for the file size

What feature or improvement do you think would benefit Files?
Currently, file size units are dynamically adjusted based on the file size. While this should be just fine (or even preferable) for most people, some (myself included) may prefer to see different units (e.g. kilobytes like in Windows Explorer, or even raw bytes).
Also, having unified file size units makes it easier to gauge the size of files in a folder at a quick glance, see the example below:

So it would be nice if Files had the option to choose the preferable units.
Also on the same note, it would be nice if Files could address the whole kilobytes/kibibytes situation, for example:

By showing proper abbreviation for the binary units (KiB instead of KB, MiB instead of MB, GiB instead of GB, etc), Dolphin style.
By allowing to switch to the decimal units as advised by SI (i.e. KB that equals 1000 bytes, etc), macOS style.

Requirements

Will allow users to see more precise file sizes, which in many cases adds additional context to the file
Will give more flexibility in terms of the file size units usage

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19045.2251
Windows Version
Windows 10 Pro 22H2 19045.2251
Comments
No response

Which formats would you like to see supported?


gqip (format paragraph) not working as expected

Format paragraph isn't smart about formatting only the text within the "sexy comment". Is there any way to get the expected behaviour?

This isn't a function of NERDCommenter per-se. You are using a text object that is defined by vim itself and doesn't know the difference between a comment and code if there is no line between them. You can see what the p text object is selecting by running vip to visually select the "paragraph" the cursor is in.
That being said perhaps a new text object t hat understood comment boundaries would be useful, but that's a big feature request. I'd be happy to review and merge if somebody comes up with such a thing.

The textobj-comment plugin provides roughly what is needed here: a text object for the comment block so you can target it from other commands. That might also be a good thing to review for anybody interested in implementing this locally (so the full range of file type support we have  would be available).


[Feature] 1.20.5/6 support

1.20.5 released recently and 1.20.6 is scheduled for early next week, with minimal changes. There were a decent number of changes under the hood, including but not limited to:

Java version being bumped to 21 (and with that Gradle and Loom must be updated I believe?)
Networking changes (The Fabric Networking API has switched to PacketCodecs)

A port is definitely planned, but I'm going to wait for any additional minor updates mojang decides to release. This mod will probably require a large effort to port due to all the under the hood changes, so keep in mind it might take some more time than past updates.

Ah I forgot to close this after I updated. I've released 1.10.0 with 1.20.5-6 support.


Adding AUPickerCell

Project URL

Category

Description

Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Hey @azizuysal, please change the link to https://github.com/azizuysal/AUPickerCell instead of https://github.com/azizuysal/AUPickerCell.git and please fill out the information on your PR

Closing this PR and opening a new one as I didn't keep the fork and cannot commit the change for this PR.


add 360X new codes

initial request from @vassilpeytchev via email for
urn:ihe:pcc:360x:hl7:OMG:O19:2017
and
urn:ihe:pcc:360x:hl7:OSU:O51:2017

request that a pull-request be submitted as per the readme.md instructions

There are actually 6 new format codes.
The final question before I commit the changes to the branch, should the FullURL be a UUID, or an OID that might be easier to manage (add 1 to the last piece of the OID for each entry, or have the last piece of the OID be the timestamp).


Connection errored out

There is no problem when making inferences from audios shorter than 5 or 6 seconds, however if this time is exceeded (so it will take more than 30 seconds to process), an error will appear, connection errored out and a runtime in the terminal without any type. of detail code.

Could you please provide a screenshot and the log for reference?

And consider temporarily disabling your antivirus to prevent it from terminating Applio.


APP-540 Deployment Automation to support Integration Bridge on Spring Boot

Change the default filename of user certificates
Change the YAML file to include the field 'key_manager_auth'

LGTM - @dnsymphony Could you review it please?

@dnsymphony fixed the comments. please check again


Dimension mismatch in T5LayerNorm with T5ParallelEmbedding

After fixing #163 and #169 locally and testing tensor parallel degree=2 for google/flan-T5-small and the summarization example, I run into a shape mismatch at the last multiplication in the T5LayerNorm
where self.weight.shape = [512] which is fine but
hidden_states.shape = [BS, SEQ_LENGTH, ->2048<-].
This happens at the first attention block where the dimension of hidden_states comes directly from the embedding layer (we expect the last dim of it to be 512 (d_model)). I therefore hypothesise that there is an issue with the implementation of the ParallelEmbedding.

@michaelbenayoun any updates on this? Can you confirm the bug?

Hi @bocchris-aws,
I am currently working on fixing the numerous bugs you reported on T5. But I do not manage to reproduce this one (I got OOM before reaching what you are mentioning...), do you have a command to reproduce the issue?

Great!

I was not able to reproduce. I have merged a fix for T5 related to an other issue you raised. Can you try from the main branch and tell me?

Yup, I am no longer running into this issue.


Look into splitting JobSteps from Job

We could provide different types of jobs, one that rely on performing sets of job steps and one that performs a job that conceptually only consists of one single step. In theory we could optimize and simplify these jobs

This does not really apply


remove rest api reference in docs for now

Pull Request Test Coverage Report for Build 2dc776c2b6df063590ab952d62efd0667af9d411-PR-1088

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 65.977%

Totals

Change from base Build 85359bdb11a180b43a1eec3f7487787ed40239d6:
0.0%

Covered Lines:
1148

Relevant Lines:
1740

💛  - Coveralls


Update React, MongoDB and Express to fix security and build issues

fix(README): update the Node.js port to match the one used in the client
fix: add missing alt text for MongoDB logo
fix(loadEnvironments): match the source code to the article and the readme
fix(server): update MongoDB driver to work with ES modules and promises. Update express to fix security vulnerabilities.
fix(client): update to React 18 to fix OpenSSL issue
fix(README): hardcode PORT in the server since it's hardcoded in the client

fixes #27

The code looks good.
Client and server run well with Node.js v18.16.


fix(generators): add uniqueHost property

🧭 What and Why
Monitoring API configuration is not correctly generated. We use the {appId}-dsn.algolia.net format but this host does not exist for the Monitoring API.
the spec is correct and only define the unique available host.
We need to introduce a new property uniqueHost  to avoid fallback to {appId}-dsn.algolia.net format in our configuration files.
C# example is provided.
🎟 JIRA Ticket:

that's nice good catch! do we contribute to your pr directly?

The CTS should be aware of that change too, the client test for monitoring is not correct right now

The CTS should be aware of that change too, the client test for monitoring is not correct right now

I don't think we assert hosts yet but we can add one of course

I don't think we assert hosts yet

yes we do ! in all clients ! except monitoring

I don't think we assert hosts yet

yes we do ! in all clients ! except monitoring

:ah: let's fix that once all clients have implemented the uniqueHost so we don't have to block everyone until they fix their CTS


Switched to this project New M16_lab02.md file

O'shea Anaya & Shane Masuda
New M16_lab02.md file   *250 Points Request

Fixes Issue #10 *100 points request
Fixes Issue #9 *100 points request

Fixes Issue #8 *100 points request

Added the app for the Save State Tutorial.

This pull request replaced by this one that goes into a separate oshea branch of the repo.
https://github.com/UCSB-CS56-Projects/cs56-android-getting-started/pull/17

I'm adding another comment in case you missed my updated comment
Fixes Issue #9 *100 Points request

Added the companion app for the Save State Tutorial *100 Points request (This is an additional 100 points request) -Nate suggested I request 100 for the Save State Tutorial and another 100 for the companion app for the tutorial.


Fix revenue example indentation on about-metricflow.md

What are you changing in this pull request and why?
Fixes the "revenue example" semantic model code indentation and defaults syntax on About MetricFlow.
Also adds a comment at the start of the second semantic model and makes it follow the same description-model-defaults order and commenting as the first semantic model, to make it easier for a new user to understand.
Checklist

[x] Review the Content style guide and About versioning so my content adheres to these guidelines.

Thanks, @mirnawong1!


std::string_view option

It should be possible for users to define BOOST_BEAST_USE_STD_STRING_VIEW or some such, and have beast use std::string_view (and std::basic_string_view) in <boost/beast/core/string.hpp>

Yes, that would be really great!

Hi,
Why std::string_view is not automatic detected ?
https://en.cppreference.com/w/cpp/string/basic_string_view
Standard required __cpp_lib_string_view >= 201606L


Fixed missing command in README

I fixed git submodule command's writing in README

Contribution License Agreement

[x] By placing an "x" in the box, I hereby understand, accept and agree to be bound by the terms and conditions of the Contribution License Agreement.

Thank you, @get-me-power .


Graph Analysis

I would like some way of prioritizing those issues that should be resolved sooner than others based on the number of incoming dependencies.
Ideally this prioritization includes transitive dependencies.  For example -
A -> B
C -> B
D -> B
In this example B has three incoming dependencies.
A -> C
C -> D
D -> B
In this example B also has three incoming dependencies.

Showing dependency counts (if > 0) as numbers on the nodes.


Mapping Issue

Hi,
I am trying to run cartographer_ros slam on ubuntu 16.04 and using ros noetic. I have built and installed the cartographer as well as cartographer_ros from this github and am running into some errors. To simulate the program I am using Turtlebot3 waffle pi and for the mapping I am using gazebo and rviz. Below are some images I am having as well as the rqt_tf_tree. Any and all help will be appreciated since it is for a university senior design!

Can you share your .lua and launch files?

turtlebot3_cartographer.pdf
turtlebot3_lds_2d.pdf
The first image is the launch files and the second is the .lua files. Thank you for your help

(This is the launch file)

(This is the .lua file)
-- Copyright 2016 The Cartographer Authors
-- Licensed under the Apache License, Version 2.0 (the "License");
-- you may not use this file except in compliance with the License.
-- You may obtain a copy of the License at
--      http://www.apache.org/licenses/LICENSE-2.0
-- Unless required by applicable law or agreed to in writing, software
-- distributed under the License is distributed on an "AS IS" BASIS,
-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-- See the License for the specific language governing permissions and
-- limitations under the License.
include "map_builder.lua"
include "trajectory_builder.lua"
options = {
map_builder = MAP_BUILDER,
trajectory_builder = TRAJECTORY_BUILDER,
map_frame = "map",
tracking_frame = "base_footprint", -- imu_link, If you are using gazebo, use 'base_footprint' (libgazebo_ros_imu's bug)
published_frame = "odom",
odom_frame = "odom",
provide_odom_frame = false,
publish_frame_projected_to_2d = false,
use_odometry = true,
use_nav_sat = false,
use_landmarks = false,
num_laser_scans = 1,
num_multi_echo_laser_scans = 0,
num_subdivisions_per_laser_scan = 1,
num_point_clouds = 0,
lookup_transform_timeout_sec = 0.2,
submap_publish_period_sec = 0.3,
pose_publish_period_sec = 5e-3,
trajectory_publish_period_sec = 30e-3,
rangefinder_sampling_ratio = 1.,
odometry_sampling_ratio = 0.1,
fixed_frame_pose_sampling_ratio = 1.,
imu_sampling_ratio = 0.1,
landmarks_sampling_ratio = 1.,
}
MAP_BUILDER.use_trajectory_builder_2d = true
TRAJECTORY_BUILDER_2D.min_range = 0.1
TRAJECTORY_BUILDER_2D.max_range = 3.5
TRAJECTORY_BUILDER_2D.missing_data_ray_length = 3.
TRAJECTORY_BUILDER_2D.use_imu_data = true
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
TRAJECTORY_BUILDER_2D.motion_filter.max_angle_radians = math.rad(0.1)
POSE_GRAPH.constraint_builder.min_score = 0.65
POSE_GRAPH.constraint_builder.global_localization_min_score = 0.7
return options

`<!--
Copyright 2016 The Cartographer Authors
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
   http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
-->

`

I was trying to upload the text but it kept coming out invisible

-- Copyright 2016 The Cartographer Authors
-- Licensed under the Apache License, Version 2.0 (the "License");
-- you may not use this file except in compliance with the License.
-- You may obtain a copy of the License at
--      http://www.apache.org/licenses/LICENSE-2.0
-- Unless required by applicable law or agreed to in writing, software
-- distributed under the License is distributed on an "AS IS" BASIS,
-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-- See the License for the specific language governing permissions and
-- limitations under the License.
include "map_builder.lua"
include "trajectory_builder.lua"
options = {
map_builder = MAP_BUILDER,
trajectory_builder = TRAJECTORY_BUILDER,
map_frame = "map",
tracking_frame = "base_footprint", -- imu_link, If you are using gazebo, use 'base_footprint' (libgazebo_ros_imu's bug)
published_frame = "odom",
odom_frame = "odom",
provide_odom_frame = false,
publish_frame_projected_to_2d = false,
use_odometry = true,
use_nav_sat = false,
use_landmarks = false,
num_laser_scans = 1,
num_multi_echo_laser_scans = 0,
num_subdivisions_per_laser_scan = 1,
num_point_clouds = 0,
lookup_transform_timeout_sec = 0.2,
submap_publish_period_sec = 0.3,
pose_publish_period_sec = 5e-3,
trajectory_publish_period_sec = 30e-3,
rangefinder_sampling_ratio = 1.,
odometry_sampling_ratio = 0.1,
fixed_frame_pose_sampling_ratio = 1.,
imu_sampling_ratio = 0.1,
landmarks_sampling_ratio = 1.,
}
MAP_BUILDER.use_trajectory_builder_2d = true
TRAJECTORY_BUILDER_2D.min_range = 0.1
TRAJECTORY_BUILDER_2D.max_range = 3.5
TRAJECTORY_BUILDER_2D.missing_data_ray_length = 3.
TRAJECTORY_BUILDER_2D.use_imu_data = true
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
TRAJECTORY_BUILDER_2D.motion_filter.max_angle_radians = math.rad(0.1)
POSE_GRAPH.constraint_builder.min_score = 0.65
POSE_GRAPH.constraint_builder.global_localization_min_score = 0.7
return options

They look fine yo me. Is your laser scanner seeing anything and not just returning invalid measurements (less than range_min or more than range_max in the LaserScan message? Try to visualize it in rviz or better run rostopic echo scan while your simulation is running and see which values are being sent.

When I run the command this pops up

Another thing that happens when I run the command roslaunch turtlebot3_slam turtlebot3_slam.launch slam_methods:=cartographer

Sorry for responding back so late

The laser scan message looks fine. This seems to be similar to #278. I am not using a turtlebot, so I cannot reproduce the error myself.
I have just noticed that in the file you attached, the tracking frame is set to imu_link, while in your comment it is set to base_footprint which is what it should be when using Gazebo. Make sure you are loading the correct config file.

Ok, thank you! I did not see that

Got everything working, Thank you for the help!


Cloud Adoption Plan Template NOT  Creating Project

I attempt to use the Cloud Adoption Plan Template or other templates and it reports success, but never creates the project.
I am using he Azure Devops DemoGenerator located here: https://aka.ms/adopt/plan/generator
Following steps in this article: https://learn.microsoft.com/en-us/azure/cloud-adoption-framework/plan/template#create-your-cloud-adoption-plan
I sign-in to my organizational account for work and try to use the Cloud Adoption Plan Template, I also tried a few others.  It shows progress that it's creating the project, however the project does not show up when I look in my Azure DevOps Organization.  If I click the Navigate to Project link after project creation it gives a 404 error message.
I did attempt in both DevOps Organizations I tried this in to run the Create Project prior to adjusting the two settings noted in the right of the page.  I tried it again afterwards as well.

I looked at another issue posted here and they found the problem.  I did the steps shown below and the policy created.  I also didn't realize that the creation success screen looks way different, it's got items and checkboxes when it "really works"


Upgrade typescript part2

moved the following libraries to use @types (sinon, rimraf, request, qs, opener, nock, mock-require, minimist, chai, chai-as-promised).

@shliderm,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


After installing data-diff, Invalid value for '--profiles-dir': Path '~/.dbt/' does not exist. error gets thrown despite this path existing

Describe the bug
I used pip to upgrade multiple dependencies, including dbt-databricks and dbt-core so I could install data-diff.
After installing it, dbt run and other CLI commands throw this error: Error: Invalid value for '--profiles-dir': Path '~/.dbt/' does not exist.
However, this path exists. When I run dbt run --profiles-dir ~/.dbt/ the job completes without error.
I've confirmed that my ~/.zshrc contains export DBT_PROFILES_DIR="path/to/.dbt" and restarted Terminal.
This error is thrown whether submitting a bigquery job or a databricks job.
What could be causing this? :thinking:
Describe the environment
MacOS Ventura 13.1
dbt-core vsn 1.5.0
dbt-databricks vsn 1.5.1
data-diff vsn 0.7.5

Resolved issue on my end; discussion on the dbt community forum here. Resolved by finding the line export DBT_PROFILES_DIR="~/.dbt/" in my .zshrc, deleting it, and ensuring only the line export DBT_PROFILES_DIR="/absolute/path/to/.dbt/
One or more of the following seems to have happened, not sure which:

After upgrading packages, the line export DBT_PROFILES_DIR="~/.dbt/" was added to my ~/.zshrc file
dbt lost the ability to interpret the relative path ~/.dbt. as a valid file path
dbt is now using some other account's home folder that does not have a /.dbt/ subfolder

To answer your question, I did not run dbt debug --config-dir before fixing the issue, but when running echo $DBT_PROFILES_DIR it returned ~/.dbt/. But since updating my zsh config it now returns the absolute path and works as expected. Feel free to close or ask any follow up questions!

Actually I just realized it's very easy to revert the fix and run dbt debug --config-dir. Here's what it returns when DBT_PROFILES_DIR="~/.dbt/" is re-added to the bottom of my zsh config:
23:36:35  Running with dbt=1.5.0
23:36:35  To view your profiles.yml file, run:

Here's what dbt debug returns:
23:38:43  Running with dbt=1.5.0
23:38:43  dbt version: 1.5.0
23:38:43  python version: 3.9.6
23:38:43  python path: /Library/Developer/CommandLineTools/usr/bin/python3
23:38:43  os info: macOS-13.1-arm64-arm-64bit
23:38:43  Using profiles.yml file at ~/.dbt/profiles.yml
23:38:43  Using dbt_project.yml file at /path/to/repo/dbt_project.yml
23:38:43  Configuration:
23:38:43    profiles.yml file [ERROR not found]
23:38:43    dbt_project.yml file [OK found and valid]
23:38:43  Required dependencies:
23:38:43   - git [OK found]

23:38:43  1 check failed:
23:38:43  dbt looked for a profiles.yml file in ~/.dbt/profiles.yml, but did
not find one. For more information on configuring your profile, consult the
documentation:

https://docs.getdbt.com/docs/configure-your-profile

@garfieldthesam glad you figured it out! wonder if it's related to that 1.5 core bug Doug mentioned.
I'm going to close this issue, but let us know if you run into any other issues.


refactor(logger): migrate logging to use odpf/salt/log

Signed-off-by: Wisdom Matthew<EMAIL_ADDRESS>closes #39
I ran all the tests myself on my machine.

@wisdommatt Log level can now move to server config.
https://github.com/odpf/guardian/blob/main/app/server.go#L42

can you help to update the config.yaml.example to match with the current service config as well?
Apart from that, the code LGTM

The only field in ServiceConfig struct that is not in config.yaml.example is the IAM field

@wisdommatt I think you closed this by mistake?

@wisdommatt ah my bad, I updated it in this PR only https://github.com/odpf/guardian/pull/38. I'll update mine from there, you can ignore it. Sorry.

@wisdommatt I think you closed this by mistake?

Yeah, but i just re-opened it
@ravisuhag you can now merge the PR


Build fails on Windows during prepare_for_platform(), missing strip program

In the prepare_for_platform() function of the tools/build/prepare_python.py file, the scons were interrupted due to the lack of the strip command in Windows：
subprocess.run(['strip', '-s', str(dest_dir / pathlib.Path(config.so_path).name)], check=True)

I made some modifications:
if not sys.platform == 'win32':
	subprocess.run(['strip', '-s', str(dest_dir / pathlib.Path(config.so_path).name)], check=True)

But other platforms have not been tested, hoping for improvement。

I don't actually have access to a Windows system, so I can only test by cross compiling from Linux or by using GitHub actions.
The GitHub action builds complete just fine, so I'm not sure what to make of this. I understand strip is a Unix style program, but if GitHub actions has it available perhaps there's something missing in your config?
What's the expected equivalent in the Windows world? I'm not all that familiar with that platform, so any insight you can share would help out a lot.

I plan to re-implement Python scripting using C++ modules. During the implementation process, I will leverage the code you have already implemented. I will share the progress with you once there are any developments.

Implementing Python scripting using C++ directly is not a small undertaking and comes with many caveats (which I've unfortunately not had time to document). I'm curious as to what parts exactly you intend to implement and your thinking in doing so is? My intent in the design of this project was to quickly and minimally offload as much as possible from C++ to pure Python so that ease, flexible and rapid development could follow. Is the current approach lacking somehow?
I'm very glad you will make use of my code! I'd very much like if the various Python binding efforts could somehow join and consolidate, it's such a big project, if we can all help each other I'm sure the result will be great!
Please do share any progress or ask questions. There is a (currently empty) discussion forum here, and you may also contact me in private if you desire (email in git logs).
I feel a little bad about the current situation. I had wanted a few months ago to finish some things and get this project a proper release, but I simply haven't had the time. This past week I actually pulled out some notes and began looking at things, so I seam to have a little more time to spare recently. We'll see 😊
Let's keep each other updated on our progress, I look forward to hearing from you soon!


Apply new object store: OpenDAL S3

Background: https://github.com/risingwavelabs/risingwave/issues/14321
https://github.com/risingwavelabs/risingwave/pull/14409 introduce opendal_s3 as remote object store, we tend to replace origin s3 with opendal(all in OpenDAL), thus, operator side need the corresponding change.

Currently this issue block the perf test and stability test of new s3, feel free to assign to someone else. cc @arkbriar @yufansong

Currently this issue block the perf test and stability test of new s3, feel free to assign to someone else. cc @arkbriar @yufansong

I don't think that the operator should support opendal_s3 before kernel does. Why not provide a switch based on env-var on the kernel side and then test with the existing operator? It sounds more reasonable to me. After all, the change to access S3 with OpenDAL should be a replacement to the original S3 access method, not a new feature, IMHO.

Currently this issue block the perf test and stability test of new s3, feel free to assign to someone else. cc @arkbriar @yufansong

I don't think that the operator should support opendal_s3 before kernel does. Why not provide a switch based on env-var on the kernel side and then test with the existing operator? It sounds more reasonable to me. After all, the change to access S3 with OpenDAL should be a replacement to the original S3 access method, not a new feature, IMHO.

Agree. It is better to keep the state store url unchanged and use a feature flag in config/ENV to control whether OpenDAL is used to connect S3.

Currently this issue block the perf test and stability test of new s3, feel free to assign to someone else. cc @arkbriar @yufansong

I don't think that the operator should support opendal_s3 before kernel does. Why not provide a switch based on env-var on the kernel side and then test with the existing operator? It sounds more reasonable to me. After all, the change to access S3 with OpenDAL should be a replacement to the original S3 access method, not a new feature, IMHO.

Agree. It is better to keep the state store url unchanged and use a feature flag in config/ENV to control whether OpenDAL is used to connect S3.

I see, let me change the implementation in kernel side.

I'll close the issue. Please open another issue if there's problem of setting the feature flag.


Multi-entry building process injects meaningless imports when set externals, causing tree-shaking fails

Describe the bug
I used multiple entries in library mode to build a component library. But I found that all of the dependencies referenced and chunks will be imported directly like import 'vue';, through the association path.
Look at the picture, note that all of these imports are set to externals except chunks

The comment parts are where the problem is. When I comment these code, tree-shaking works as expected.
I don't know why it happends, but when I set rollupOptions.output.preserveModules: true, it works well.
Reproduction
https://stackblitz.com/edit/vitejs-vite-9tmbdh?file=vite.config.js
Steps to reproduce
Just run yarn build, and then check the build result in dist.
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @vitejs/plugin-vue: ^4.1.0 => 4.1.0
    vite: ^4.2.1 => 4.2.1

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

See https://rollupjs.org/faqs/#why-do-additional-imports-turn-up-in-my-entry-chunks-when-code-splitting
Though, I think it's a problem on tree-shaking side and not the output, because this change doesn't affect code execution order or behaviour.

Oh, thanks for your prompt, I'll try it later.


#1019 always redirect to the Redirect.Version

cool. only a couple of test errors now ;-)

@reidev275 OK, just a few unit tests to fix as well.

Microsoft Telepathy (™)

Sorry, very new to the whole FSharp project scaffold with the build scripts.  Thanks for being understanding as always !

awesome. thx

@reidev275 excellent - that's exactly why "up-for-grabs" like this are good - you get to learn more about the project scaffold and the infrastructure behind the CI process, which is usually pretty consistent across most of fsprojects :-)

yeah, love the idea of the up for grabs issues.  That's exactly why I was
excited to jump in on this one.
On Fri, Aug 28, 2015 at 3:21 PM, Isaac Abraham<EMAIL_ADDRESS>wrote:

@reidev275 https://github.com/reidev275 excellent - that's exactly why
"up-for-grabs" like this are good - you get to learn more about the project
scaffold and the infrastructure behind the CI process, which is usually
pretty consistent across most of fsprojects :-)
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/pull/1023#issuecomment-135865565.


Common.CloneRecord fieldToReplace

Moved logic that updates parent record ID on clone into core CloneRecord logic. Fix for issue #255.

Tried this one and it works well


Default RoleSessionName causes SignatureDoesNotMatch exception when assuming role

For example:
Baiji.Kinesis.list_streams(%{}) |> Baiji.Auth.assume_role(role)

or
Baiji.Kinesis.list_streams(%{}) |> Baiji.Auth.assume_role({role, "Baiji (Elixir)"})

causes the SignatureDoesNotMatch error response from AWS.
Whereas
Baiji.Kinesis.list_streams(%{}) |> Baiji.Auth.assume_role({role, "baiji"})

does not.

Unfortunately this one is still happening for me on 0.6.9:
# This prints an exception from Baiji.Auth.assume_role/1 but it is ignored.
# - the role is not assumed
Baiji.Kinesis.list_streams(%{}) |> Baiji.Operation.debug! |> Baiji.Auth.assume_role(role)

# This works, just by changing the RoleSessionName used
Baiji.Kinesis.list_streams(%{}) |> Baiji.Operation.debug! |> Baiji.Auth.assume_role({role, "baiji"})

Ah, my mistake, I didn't change the default role name as I'd thought I did for v0.6.9. I've reopened this issue and will address it tomorrow.

v0.6.10 has been published to Hex and changes the default role session name to 'baiji'.


Short URL

@eddyod
Create a new url in the REST that takes an url ID and returns the entire link.
@Junjie-Cheng
Creates the typescript to consume the above data.

@Junjie-Cheng
I am creating a rest call that will look like this:
https://activebrainatlas.ucsd.edu/activebrainatlas/urldata?id=21
That will return all the data in the url:
#!{"dimensions":{"x":[4.25e-7,"m"],"y":[4.25e-7,"m"],"z":[0.00002,"m"]},"position":[26702.505859375,13043.029296875,217.71533203125],"crossSectionScale":51.38839541720887,"projectionScale":31692.00157498454,"layers":[{"type":"image","source":{"url":"precomputed://https://activebrainatlas.ucsd.edu/data/MD589/neuroglancer_data/C1","transform":{"outputDimensions":{"x":[4.25e-7,"m"],"y":[4.25e-7,"m"],"z":[0.00002,"m"],"c^":[1,""]},"inputDimensions":{"x":[4.25e-7,"m"],"y":[4.25e-7,"m"],"z":[0.00002,"m"],"c^":[1,""]},"operations":[0,0,0,0,0,0,1,1,1,0.00001,0.0001,0.5,5,0.01,0.1]}},"tab":"source","channelDimensions":{"c^":[1,""]},"name":"C1"}],"selectedLayer":{"layer":"C1","visible":true,"size":496},"layout":"4panel","partialViewport":[0,0,1,1]}\n

@Junjie-Cheng
I have the url problem fixed very nicely with the histogram branch. Can you merge this branch with your stuff? Make sure you get both of the css we used for the state_loader.css merged correctly.

@Junjie-Cheng
I merged the url and firebase stuff into the master branch. can you get your histogram and the CSV stuff, and the main google branch all into our master.


FindCUDAToolkit: Fix nvrtc_builtins library name

The lib is libnvrtc-builtins.so, not libnvrtc_builtins.so.
see NVIDIA NVRTC Document.
This fix is similar to nvrtc_builtins_static. (9688a8ebc20ac4b357e10f080cdfb1159c8b2725)

Thanks.  I've migrated this over to GitLab MR 8642.
For future contributions, please see CONTRIBUTING.rst and open a merge request in our GitLab instance.

For future contributions, please see CONTRIBUTING.rst and open a merge request in our GitLab instance.

excuse me.
I read CONTRIBUTING.rst while misreading GitLab as GitHub.
I will be careful next time.


Deep link context & Deep link received event send referrer as "null" string if it is absent

Describe the bug
Both deep link schema (https://github.com/snowplow/iglu-central/blob/master/schemas/com.snowplowanalytics.mobile/deep_link/jsonschema/1-0-0) and deep link received schema (https://github.com/snowplow/iglu-central/blob/master/schemas/com.snowplowanalytics.mobile/deep_link_received/jsonschema/1-0-0) define referrer as optional and its format is uri. SDK sends it as "null" string if it's absent and enricher is validating them using their schemas. So the validation for this field fails as it is expected to have uri format.
To Reproduce
Track DeeplinkReceived event without providing referrar.
Expected behavior
I would expect If referrer is null then it is not added to JSON as it is already optional
Device informatoin (please complete the following information):

Device: all devices
OS: all Android version
Version 3.1.2

Hi @Ehab4290 , thanks for reporting this issue.
I've done a quick check and from the tests and the code it seems that the tracker should strip out the null values from DeepLinkReceived and DeepLink.
I suspect that the "null" string is assigned to referrer somewhere else.
Do you create the DeepLinkReceived event with DeepLinkReceived fromIntent(@Nullable Intent intent) or fromReferrerDetails(@Nullable ReferrerDetails referrerDetails ?

Hi @AlexBenny,
I've realized that "null" was assigned from my implementation.
Thanks for your efforts and sorry for the confusion


Create Breadcrumb Navigation component for use on Product Grid and PDP

Create breadcrumb navigation component for use on the product grid and PDP pages (and potentially other pages)
The breadcrumb navigation should start with a home link which should link back to the index.
After that it should drill down into the tag category that the user is currently browsing, including the parents in the breadcrumb list if applicable.
e.g. if browsing the 'Skirts' tag within Womens and Active Wear, the breadcrumb list should render:
Home > Womens > Active Wear > Skirts
If on a PDP page, the breadcrumbs should show the breadcrumb list where the user originated followed by the product title
e.g.
Home > Womens > Active Wear > Skirts > Patagonia Women's Lithia Skirt

@rymorgan @cassytaylor If someone doesn't click in from a product grid to the PDP, what tag(s) should be shown in the breadcrumb list? The first tag probably?

@spencern I would think it would just home > product name.

@rymorgan that makes sense. Let's go with that.

Was discussing with @rymorgan about what to do with displaying breadcrumbs in different situations.
We'd like some input, but what we are thinking is:

If you navigate to a product from a tag page, the breadcrumbs display the lineage of tags taht got you there. So if you go to Tag A, or any sub-tag under Tag A's tag page, the crumbs will read Home > Tag A > SubTag-A. We think that's pretty straightforward.

However, when you go directly to a product from the default product grid, or say someone sends you a link and you go directly to a product, we were unsure what to do. Our thought is to show the Lowest level unique tag.
For example:

if a product is in Home -> Tag A -> Tag A-1, and nothing else, we'd show that.
If a tag was in Home -> Tag A -> Tag A-1 && Home -> Tag A -> Tag A-2, we'd show Home -> Tag A
We don't know what to show if a tag is in Home -> Tag A && Home -> Tag B.

@spencern @aldeed thoughts on what we could show in the second example above? Thoughts on what we are thinking for the first example?

@kieckhafer I like this solution. Green light from me

@kieckhafer Pending those discussions, I like the idea of just keeping it simple and showing the first hierarchy path that we find in the nav tree as the default behavior.
If they've clicked from a tag page, then it could definitely show that tag as the parent, but you'd have to track the ancestor tag(s) in mobx at the point of clicking in the nav if you want everything above that to be accurate. I'd say do that if it's easy but otherwise show only the parent and "Home" above that.

@kieckhafer But maybe you can add that default logic in the CatalogProduct resolver on the server? Add a new canonicalNavPath: [String]! prop and a resolver for it, and have the resolver look up the first tag path it finds and return something like ["Womens", "Active Wear", "Skirts"] (e.g., if product has "Skirts" tag) That way we can make that logic more customizable in the future but client UI won't need to change.

@nnnnat I'm trying to tap into the existing routingStore.tag object.
When we land on a tag page, it sets the tag, however the tag is never unset, say if you go back to the main product grid, the tag you were just on is still set in routingStore.tag. I'm thinking about updating that so it unsets the tag if you aren't currently on a tag page, is there any reason you can think of why it wouldn't work? https://github.com/reactioncommerce/reaction-next-starterkit/blame/feat-78-kieckhafer-breadcrumbs/src/pages/tag.js#L44


Refactor BlockBuilderStatus for usability

BlockBuilderStatus currently contains functionality for both block building and page building tracking. It also requires caller's close interaction to make the block building tracking work.
By splitting it into two, the two tracking functionalities are split, and close interaction is removed.

Since this change is in the SPI, @dain should approve it.
@dain, the code looks good to me

Looks good


Explain why the Padding widget exists

Fixes #5166

If I'm going to add more than Container property via wrapping than I usually use Container rather than a primitive because I think it makes the result a little easier to read.
LGTM


[temp-rvalueopt] Teach how to promote fix_lifetime.

In the case of copy_addr, we move it onto the source address and in the case of
a store, put it on the source object.
I just noted this pattern happening a bunch in the stdlib when I was looking
through it for ownership patterns.

This just cleans up the SIL.

@swift-ci smoke test

@swift-ci benchmark

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
6621
8745
+32.1%
0.76x (?)

Code size: -O
Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

String.data.LargeUnicode
93
108
+16.1%
0.86x (?)

Improvement
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
6770
4707
-30.5%
1.44x (?)

Code size: -Osize
Performance: -Onone
Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB


Add customized edge weight support for graphs.

Original implementation only supports long as edge weights.
This PR allows customized edge weight types while keeps default edge weight type as long/Int64.

Hi @zmy,
Thanks for your contribution. Can you please refactor the tests to reflect your changes?
Cheers,
Ahmad

Hi @zmy,
Thanks for your contribution. Can you please refactor the tests to reflect your changes?
Cheers,
Ahmad

Sure I am refactoring the tests, will add new commits soon :D

@aalhour It seems that some unit tests themselves are not well written. When running all the tests on my local machine, only the following two tests are related to graph data structure and failed:

public static class GraphsBellmanFordShortestPaths
GraphsUndirectedWeightedDenseGraphTest

Hello @zmy,
Sorry for the late reply, I was away on a long vacation. Can you maybe fix these two graphs tests?

@aalhour Sorry for the late reply. I've just fixed related tests. Please take a look.


update screenshot with new WebVR Player icon

See the Project Setup docs:

The icon was fixed in issue #112 in the last release (v1.1.0):

fixed in PR #261 😃


[Event Request] codeunit 5815 "Undo Sales Shipment Line". - Add parameter to function OnAfterInsertNewShipmentLine

Dear ALAppExtensions Team,
Is it possible to add a parameter to function OnAfterInsertNewShipmentLine in codeunit 5815?

OnAfterInsertNewShipmentLine(SalesShptLine, PostedWhseShptLine, PostedWhseShptLineFound, DocLineNo);

OnAfterInsertNewShipmentLine(SalesShptLine, PostedWhseShptLine, PostedWhseShptLineFound, DocLineNo, ItemShptEntryNo);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Translate concepts/cluster-administration/system-logs in Korean

This is a Feature Request

What would you like to be added

Translate concepts/cluster-administration/system-logs in Korean
Why is this needed
No Traslation concepts/cluster-administration/system-logs in Korean
Comments

Page to Update :
https://kubernetes.io/docs/concepts/cluster-administration/system-logs/

/language ko
/assign

@jm1223kim: Looks like a great idea - would you kindly contribute for this part translation in Korean? While I understand that still many parts are not translated into Korean, it will be great if you clarify more demand/requirement why this module should be more prioritized in details, if you are unable to contribute translation on that part.

/triage accepted
@jm1223kim Hope to share schedule of Korean localization (dev) branch milestone :)
Current dev branch for Korean localization is dev-1.21-ko.2 (https://github.com/kubernetes/website/issues/27763)
Please note that Korean localization team will create dev-1.21-ko.3 soon ~ ! (05/20~05/21)
I think dev-1.21-ko.3 would be an appropriate work base for this issue :)
If you any question ~ please visit kubernetes-docs-ko slace channel.
https://kubernetes.slack.com/archives/CA1MMR86S

FYI: https://github.com/kubernetes/website/issues/28127

/close


Version numbers in header file is outdate

https://github.com/syoyo/tinygltf/blob/f51243da480806032cfb6f8d6799a7493397cb86/tiny_gltf.h#L28
Other issues: The missing patch version number

Good find, fixed: https://github.com/syoyo/tinygltf/releases/tag/v2.8.14
I'm considering to remove version number in .h file


tour: The Go Playground

Context: https://tour.golang.org/welcome/4
In this section -fmt.Println("The time is", time.Now.nanoseconds()) is not working.
Error while compiling:
prog.go:11: time.Now.nanoseconds undefined (type func() time.Time has no field or method nanoseconds)

I think this one can be closed. The error is gone.


Add ChatType methods to MessageChannel

API | Common | Forge | Vanilla

@Minecrell could you look at merging this soon? @gabizou is attacking me for not having another PR done which depends on this :sos:

It looks fine to me. I can either merge it tomorrow or you can try to yell at someone else to do that ;)


Wireflow update: Form patterns

TODO

[ ] Draft IA to account for Forms "pattern/package/category"

Outcomes

Recommendations for how we approach high-lighting Forms content on the documentation site
Recommendations for how we approach walking someone through the component-pages for Forms
Decisions on edit to IA based on Forms content

Deliverables

Wireframe edit: component page addition, navigation edit

Can be found in forms design page in Figma Documentation site desktop


feat(p-text-input): add size props to textInput

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[X] Not that difficult

Type of Change

[ ] New feature
[ ] Bug fixes
[X] Feature improvement

[ ] These changes require a usage change

[ ] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[X] Updated Storybook documents
[ ] Tested with console(if usages are changed)

Description

add size props to textInput

Things to Talk About

Thnx a lot your comments! :)

:tada: This PR is included in version 1.1.0-beta.466 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for async/await using Babel

For proof-of-concept, this patch converts a couple of Promise returning methods to use async instead.
Please note that the generic build, based on this patch, has been successfully testing in IE11 (i.e. the viewer loads and nothing is obviously broken).
Being able to use modern JavaScript features like async/await is a huge plus, but there's one (obvious) side-effect: The size of the built files will increase slightly (unless SKIP_BABEL == true). That's unavoidable, but seems like a small price to pay in the grand scheme of things.
Finally, note that the chromium build target was changed to no longer skip Babel translation, since the Chrome extension still supports version 49 of the browser (where native async support isn't available).
Split off from PR #9944 as requested by @brendandahl.
Much more manageable diff with https://github.com/mozilla/pdf.js/pull/9977/files?w=1

/botio test

The one Windows failure is strange, since the commit included here is exactly the same as the first one in PR #9944 (where all tests ran successfully). One possible explanation is that Firefox was updated on the bot (which might not show up in the logs, since the bot uses a beta version); another explanation is that it's simply an intermittent.
/botio-windows test

/botio-windows test

/botio-windows test

I don't understand what to make of these intermittent failures (on Windows), since all tests seem to pass consistently in PR #9944 and the commit hashes are identical :confused:

Doesn't look like it's just your changes, I got it to fail over in https://github.com/mozilla/pdf.js/pull/9567#issuecomment-412937390

/botio-windows test

/botio test

/botio test


Make FHIR primitives verifying their regexp again with io-ts

Basic object models I've generated were verifying regexp provided by FHIR standard.
When migrating to io-ts, at first glance, to simplify thing, I forgot about those data checking.
It's time to make it works again.

Can't find a good validator for xhtml that not make a call on a server
Same for markdown ...

Primitives will be available in 4.0.4


[Feature request] Allow tabs to wrap to next line

It would be useful if we could optionally make the Tabs List component wrap containing Trigger items to the next line, rather than requiring a horizontal scroll.

You could set white-space: normal on TabsList. What does your design look like so we can consider whether it makes sense in the component API?

I don't have a design from our current app as we've been avoiding using Tabs for now, partly due to this.
Thanks for your suggestion re:white space.
As an example design, say you have the following:

In the mobile view, on the left, the user cannot see the "Settings" tab, and it is not clear that this is a scrollable area.
Wrapping the tabs to the next line would be preferred, from my perspective.
This follows, for example, what the tabs here do (Get Started/Styles/Components):
https://design-system.service.gov.uk/
Alternatively in the actual govuk tabs component, tabs turn in to a bulleted list when there is not enough horizontal space to fit them all, though I expect this is a manually set breakpoint rather than calculated at runtime:
https://design-system.service.gov.uk/components/tabs/

Ok we have implemented this in our designs as per your suggestion on overriding CSS on TabsList.
In the end we had to do this:
<TabsList style={{ rowGap: 'var(--space-2)', height: 'auto' }}>

Ah I see, that is reasonable. We'll accommodate that.


newsfeed, showDescription is ignored

The setting showDescription is not used in the njk file. After the recent change, the description is always shown.
https://github.com/MichMich/MagicMirror/blob/491f5aa7769a12de7f20f201de0078704c158a8d/modules/default/newsfeed/newsfeed.njk#L16

Duplicate isse


SW-6402 Return HTTP 409 on species name conflict

If the user tries to create a species whose scientific name already exists in the
organization, the server returns HTTP 409 (Conflict). But it wasn't catching the
case where a user tries to rename a species and the requested name already exists.
Return HTTP 409 in that case as well.

#2731  👈 (View in Graphite)
main

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


(MAINT) Fix invalid chars in Openstack ssh keyname

Current behaviour will result in 400 error from OpenStack. "Keypair data is invalid: Keypair name contains unsafe characters"

Can one of the admins verify this patch?

Hi @thlapin, thanks for submitting this fix for us! If you would prefix your commit with "(MAINT)", this PR will conform to our contributing standard.
@lucywyman @johnduarte, iirc, you two are some of our more practiced internal users of the Openstack hypervisor. Does everything look good here to you?

Sorry @kevpl just looked at this, lgtm once the branch conflicts are resolved :)

@thlapin looks like this needs rebasing. Once you've done that, I'm down with merging it in.

rebasing shows that this was actually fixed last week. https://github.com/puppetlabs/beaker/commit/7810ee990c34b622b53fc68f935f46032bf047c9
Closing pull request as this is no longer necessary.


utf8 match

Hi,I'm a novice to hyperscan
I have a question whether hyperscan supports matching Chinese characters?
I use your code hyperscan-java and set the expressionFlag to utf-8, but it can't match result.
Examples are as follows：
expression：测试
scan：这是一个测试
thanks.

Hi lihue36,
thanks for the bug report. I'm able to reproduce it. Seems like there's some issues with UTF-8 characters.
I'll investigate if it's a issue with the JNA wrapper or with hyperscan itself and will come back to you with a solution.
Best,
Matthias

@liuhe36 Sorry it took so long, but i've finally was able to fix the bug. In case of UTF8 characters the code was passing a wrong length of the input data.
I've pushed out version 0.1.1 which fixes the bug by using getBytes().length instead of the length property!
Please try again.

@gliwka oh,it's exciting, I will try it right now.

Let me know how it goes 😄

it works well
thanks and I will run a performance benchmark between native hyperscan and yours，so that I can integrate hyperscan-java into my project.
Do you have any advice or Did you run benchmark before?

I'm happy it works for you. Let's continue the discussion about benchmarking in the issue #1


LOG4J2-3122 fix Logging to Log4j2 MongoDb4 Appender get ERROR on trace, debug and info level.

After do some debugging, i found out that there are some part of the code that is missing in MongoDb4Provider class when creating log4j-mongodb4 module, knowing that this part of the code is exists in log4j-mongodb3 module. The code that check if the database actually requires authentication. I don't know if that is intentional or not but somehow this part of the code is what making the old log4j-mongodb3 module running without error. So i just copy the old code to the new log4j-mongodb4 module, and now it is running without error to. What do you think about this log4j2 team?
Please respond.
Thanks.

After do some debugging, i found out that there are some part of the code that is missing in MongoDb4Provider class when creating log4j-mongodb4 module, knowing that this part of the code is exists in log4j-mongodb3 module. The code that check if the database actually requires authentication. I don't know if that is intentional or not but somehow this part of the code is what making the old log4j-mongodb3 module running without error. So i just copy the old code to the new log4j-mongodb4 module, and now it is running without error to. What do you think about this log4j2 team?
Please respond.
Thanks.

Hi @redapel
Yes, it was intentional, with the version 4 driver, anything should be configurable from the connection string, unlike with the version 3 driver IIRC.
In general, you'll want to provide tests for new code. If you'd included a test with a configuration file, we could have determined if that was the code... so it still might be the case, but, no test...

Hi team,
Please have take a look to the new test case that i submitted. In that test, i choose to use real MongoDB server instead of the Flapdoodle's embedded MongoDB as the later can't be considered as "full integration testing", there are possibilities that both of them behaves differently, so it is better to use solution like docker container. By the way, i run the test using a replica set cluster with only single server act as a PRIMARY. I suggest you to run the test more than one (for example 10 times), Since there are at least 2 different error cases produced when i try to test it. Both of them i already made an assertion to fail the test. Let me know about the progress.
Thanks.

You'll want to fit in our existing test framework.

I already change to Flapdoodle's embedded MongoDB


dotnet-ef scaffold references old EF package name

Usage: dotnet ef dbcontext scaffold [arguments] [options]
Arguments:
[connection]  The connection string of the database
[provider]    The provider to use. For example, EntityFramework.MicrosoftSqlServer

Fixed with commit 93c1635.


Darwin chip tool cli clusters

Problem
chip-tool-darwin currently does not have cluster commands and missing README.
Change overview

Add Zap file to generate cluster commands using CHIP.framework
Add Zap file to generate README
Generated README and Cluster Commands.

Testing

Paired all-cluster-app
Ruan several commands.
Ran several  reads
Ran several writes

Fast tracking adding a new tool.

I think having README.md generated via ZAP is too much.
Our path should be generating files at compile time. This goes against that: more likely chances to have merge conflicts and generates large outputs in our source tree.

@andy31415 This is just following current practices re: ZAP - which do include a zap generation phase. I'm not sure how this increases any chances of merge conflicts, given this piggy backs on what exists.
Suggest this is allowed to proceed to unblock testing.

I think having README.md generated via ZAP is too much.
Our path should be generating files at compile time. This goes against that: more likely chances to have merge conflicts and generates large outputs in our source tree.

@andy31415 This is just following current practices re: ZAP - which do include a zap generation phase. I'm not sure how this increases any chances of merge conflicts, given this piggy backs on what exists.
Suggest this is allowed to proceed to unblock testing.

I strongly believe README.md should not be auto-generated.

I strongly believe README.md should not be auto-generated. I do not believe that the value of having the command list in the readme is worth the cost in maintenance and complexity.

Given how out of date our documentation is, this seems like a nice solve in the interim, using the tools we have available. I also am not sure how this will cause more conflicts. Can you please explain? This is all a single downstream item that gets regenerated, not like our others that have cross dependencies.

I think having README.md generated via ZAP is too much.
Our path should be generating files at compile time. This goes against that: more likely chances to have merge conflicts and generates large outputs in our source tree.

@andy31415 This is just following current practices re: ZAP - which do include a zap generation phase. I'm not sure how this increases any chances of merge conflicts, given this piggy backs on what exists.
Suggest this is allowed to proceed to unblock testing.

I strongly believe README.md should not be auto-generated. I do not believe that the value of having the command list in the readme is worth the cost in maintenance and complexity.

I took @vivien-apple advice and put the generated portion into the zzz_generated folder to avoid and gen conflicts.

I think having README.md generated via ZAP is too much.
Our path should be generating files at compile time. This goes against that: more likely chances to have merge conflicts and generates large outputs in our source tree.

@andy31415 This is just following current practices re: ZAP - which do include a zap generation phase. I'm not sure how this increases any chances of merge conflicts, given this piggy backs on what exists.
Suggest this is allowed to proceed to unblock testing.

I strongly believe README.md should not be auto-generated. I do not believe that the value of having the command list in the readme is worth the cost in maintenance and complexity.

I took @vivien-apple advice and put the generated portion into the zzz_generated folder to avoid and gen conflicts.

Please remove the README.md generation out of the PR completely (regardless of location), then it should be good to go.

I took @vivien-apple advice and put the generated portion into the zzz_generated folder to avoid and gen conflicts.

Please remove the README.md generation out of the PR completely (regardless of location), then it should be good to go.

I'm really not sure what policy disallows adding this? Why are we blocking changes here adding up to date documentation?

I took @vivien-apple advice and put the generated portion into the zzz_generated folder to avoid and gen conflicts.

Please remove the README.md generation out of the PR completely (regardless of location), then it should be good to go.

I'm really not sure what policy disallows adding this? Why are we blocking changes here adding up to date documentation?
Edit to add:
What policy prohibits using zap and code generation?
I understand certain frustrations here, however, this seems to be a relatively straight forward change, following normal/existing code generation for platforms and tools, and unless there's precedent here, we should let this through.

There is no policy that prohibts zap code generation usage of zap.
As a code reviewer, it is my technical opinion that auto generating README/text files into the code provides little benefit, hence my 'changes requested'. As an analogy taken to the extreme, there is no policy of "do not write C++ code" (that would be crazy), yet code reviews can reject C++ code if they feel its use is inappropriate.

I would even be ok if you make the output of --help or similar output available commands. I was assuming we already have that. If we do not, lets add it. Whenever someone builds or tries a tool, --help or -h or -? is the first thing they run.
Please don't place README.md into various places and make them autogen. I am against that approach.

As an aside, what is the difference beteween chip-tool and chip-tool-darwin? Both seem CLI. Would chip-tool not work on darwin as well? Why do both exist?

As an aside, what is the difference beteween chip-tool and chip-tool-darwin? Both seem CLI. Would chip-tool not work on darwin as well? Why do both exist?

I have the same concern, however - given maybe we can file an issue here to help things move forward?

As an aside, what is the difference beteween chip-tool and chip-tool-darwin? Both seem CLI. Would chip-tool not work on darwin as well? Why do both exist?

I have the same concern, however - given maybe we can file an issue here to help things move forward?

Filed: https://github.com/project-chip/connectedhomeip/issues/14987

As an aside, what is the difference beteween chip-tool and chip-tool-darwin? Both seem CLI. Would chip-tool not work on darwin as well? Why do both exist?

chip-tool uses the C++ API and as such it supports some APIs that we may not support in the near future on the Objc side.
chip-tool-darwin uses the Objc++ API from src/darwin/Framework and all the wrapping that happens on top of the C++ layer.
To give you a more concrete example of some difference between what the C++ layer allows that is not supported on the Objc++ side. Using the Any cluster of chip-tool let you build IM packets from JSON specified on the command line, notably for command arguments of attributes content when writing.
That said, chip-tool-darwin will also run the clusters tests that currently belongs to src/darwin/Framework under ./scripts/tests/run_test_suite.py (see https://github.com/vivien-apple/connectedhomeip-1/tree/ChipToolDarwin_GetChipToolDarwinToWorksWithTheTestSuiteRunner/src/app)
That would allow to get the test runs closer by using the same environment. If you look at the CI today, you will notice a lot of the "Build Darwin" tasks are red. It happens because #14606 has introduced some new Subscribe command to the test suites.
I suspect that it happens randomly because while those subscriptions are closed on the client side, the server waits for some timeouts to close them on its own side (that's just how this stuff works not a bug).
But now, why does it happens only for "Build Darwin" CI and not the other CI tests ? Because "Darwin CI" is running a single instance of the "all-clusters-app" for the whole set of tests - so the test environment is different and that's just more work...
And maybe I'm completely wrong, and that's not the root cause. But the fact that the test environment  could be the cause is an issue.

| For the command lists, I believe chip-tool is self-documenting and can auto-complete. I would like documentation to describe how to use that and not try to duplicate things.
I would argue that chip-tool would even be more self-documenting is we do use the information from the XML that (in theory) describes the cluster and what it does, the commands and their arguments, etc...
That would also be a nice way to see when the XML lack some documentation just by using the command line tool....

I removed the code gen for the README as requested.

@krypton36 @woody-apple  The file seems to actually not have been removed. I would have preffered to ask for a re-review instead of dismissing as stale review and merging (or double-checking that the review comment was indeed fully addressed)


ci: replace circle with github actions

replace circle with githubactions
replace golangci with reviewdog

Signed-off-by: Marko Baricevic<EMAIL_ADDRESS>

@tessr could you remove golangCI and circle-ci here but make the GitHub actions required?

I've made Test required. Lmk if I should make anything else required too!


Parser.clear() does not clear until session.commit() is executed

Description of the bug
Parser.clear() does not clear anything until session.commit() is executed elsewhere.
According to https://docs.sqlalchemy.org/en/13/orm/session_basics.html#deleting,

The Session.delete() method places an instance into the Session’s list of objects to be marked as deleted

To Reproduce
Steps to reproduce the behavior:
>>> corpus_parser.apply(doc_preprocessor)
>>> corpus_parser.clear()
>>> assert session.query(Document).count() == 0

Expected behavior
When corpus_parser.clear() above is executed, any instance of Context like Document, Sentence should be deleted.
Error Logs/Screenshots
If applicable, add error logs or screenshots to help explain your problem.
Environment (please complete the following information)

Fonduer Version: 0.8.2

Additional context
Add any other context about the problem here.
This might be related to #213

I'm trying to write a unit test that can reproduce this issue, but can't.
Close for now.


[FR] Wikipedia Links

Describe the solution you'd like
It would be nice to format the links like you do in Wikipedia or Obsidian.md by putting a | into the link and adding string afterwards.
it enables you to write sentences which are grammatically correct.

Agree. This is discussed in https://github.com/renerocksai/telekasten.nvim/discussions/56#discussioncomment-1977177
The problem is that this complicates everything quite significantly... Unfortunately, neither I nor renerocksai have the time to implement such a complex feature for now.

Hello @lhoffjann , I changed one line, so now I can navigate to wikilinks with aliases. I hope this solves at least one problem.


fix: update to patched node-sass-utils

This is needed to work with sass versions since 1.44, until this issue is addressed.
Tests completed with sass 1.38, 1.46, and 1.58.3 (latest).
Edit: this is still incompatible with the new sass api methods, compile and compileAsync; trying to add support for those.

Just fwiw, I got a version working with the new Sass API on this branch of my fork, which I'm currently using in a new project. It exports another file via require(sass-tailwind-functions/modern) that wraps index.js, converting modern Sass values to legacy ones before passing it to the original functions, and then converting their results back to non-legacy values. This is probably inefficient, but is non-breaking and keeps most of the current code intact.
If it would be welcome, I could submit that or a more substantial PR re-writing index.js to work with the new Sass api. Part of the issue is that node-sass-utils does not support the new JS api (and may not plan to), so updating to that means replacing node-sass-utils, possibly with sass-cast, which I'm currently using to do the modern => legacy conversion. Dropping node-sass-utils would avoid the issue of this PR, that the latest npm release is still broken since Sass introduced the new JS api.

I'd rather not pin to a GH commit for the dependency, if there is a newer alternative to use then it makes sense to switch to that instead

That makes sense; I just got a version working without node-sass-utils and will submit a separate PR.


feat: token rates resumable polling

Explanation
Note: This PR accompanies https://github.com/MetaMask/metamask-extension/pull/23507.  Read it for more context.
This PR changes the polling strategy for TokenRatesController.
Extension starts/stops polling as metamask is opened/closed, to avoid unnecessary requests.  But when it's re-opened, I want to "resume" polling rather than "restart".
e.g with a 3 minute polling interval:

it's been 12 hours since you last opened metamask, so fetch now.

it's been 1 minute since you last opened metamask, so schedule a fetch in 2 minutes.

I'm storing the last poll time so the interval can be maintained across starts/stops.
References

Changelog

@metamask/assets-controllers

CHANGED: TokenRatesController.start() will now resume its polling interval from the last poll time, if one exists from a previous start().

@metamask/package-b

: Your change here
: Your change here

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Closed in favor of https://github.com/MetaMask/core/pull/4128


rsync_backup: option -d needs further testing

The current implementation of option -d hasn't been tested properly and isn't expected to work 100% correctly.
This will also be a great opportunity to incorporate unit testing with the given scripts in this repository.

WONTFIX: The script was reverted to an earlier iteration. This no longer applies.