Insert SCTE pid into ts

Apologies if this is noob question but I am trying to do the following:
Create a ts stream from ffmpeg and send to tsp for SCTE35 splice insert.
The challenge is that I need to add a stream type of type 0x86 to pid 500 (new pid).
Question: How do I add a pid (500 for example) with a stream type 0x86 ?
My assumption is that after I am able to add the above pid, I can use the spliceinject plugin to add splice events ?
An example will be very much appreciated and a beer at IBC guaranteed.
Currently I am able to generate a TS stream with ffmpeg and send via local udp unicast to tsp, but not able to add splice events because I dont have the proper scte35 table
Thanks

Hi @mhzbg
To create the PID, you need to

Have enough stuffing so that packets for the new PID can be inserted by replacing null packets
Have the new PID correctly declared in the PMT of the service

For point 1, if you work on a real broadcast stream, there is always some stuffing. If you work on a synthetic stream from ffmpeg, there is probably none and you need to create some. In this case, use the tsp option --add-input-stuffing which inserts null packets in the middle of the input stream. For instance, tsp --add-input-stuffing 1/10 adds 1 null packet every 10 input packets, which is more than enough for you. The inserted null packets will be used as replacement, when needed, for the new PID. At the end of the chain, before the output plugin, it is recommended to remove the remaining null packets using the plugin filter.
For point 2, the plugin pmt modifies a PMT on the fly. You can declare the new PID 500 with stream type 0x86. To fully comply with SCTE 35, it is also required to add a registration descriptor with id 0x43554549. If there are more than one service in the TS (probably not the case for you), you also need to specify the PMT PID or the service name or id.
This gives the following command skeleton:
tsp --add-input-stuffing 1/10 \
    -I ... \
    -P pmt --add-programinfo-id 0x43554549 --add-pid 500/0x86 \
    -P spliceinject ... \
    -P filter --negate --pid 0x1FFF \
    -O ...

Of course, all options are documented in the user's guide.

I also added a new example about SCTE 35 cue information insertion in the TSDuck user's guide, section 5.2.24.

Hi @mhzbg
To create the PID, you need to

Have enough stuffing so that packets for the new PID can be inserted by replacing null packets
Have the new PID correctly declared in the PMT of the service

For point 1, if you work on a real broadcast stream, there is always some stuffing. If you work on a synthetic stream from ffmpeg, there is probably none and you need to create some. In this case, use the tsp option --add-input-stuffing which inserts null packets in the middle of the input stream. For instance, tsp --add-input-stuffing 1/10 adds 1 null packet every 10 input packets, which is more than enough for you. The inserted null packets will be used as replacement, when needed, for the new PID. At the end of the chain, before the output plugin, it is recommended to remove the remaining null packets using the plugin filter.
For point 2, the plugin pmt modifies a PMT on the fly. You can declare the new PID 500 with stream type 0x86. To fully comply with SCTE 35, it is also required to add a registration descriptor with id 0x43554549. If there are more than one service in the TS (probably not the case for you), you also need to specify the PMT PID or the service name or id.
This gives the following command skeleton:
tsp --add-input-stuffing 1/10 \
    -I ... \
    -P pmt --add-programinfo-id 0x43554549 --add-pid 500/0x86 \
    -P spliceinject ... \
    -P filter --negate --pid 0x1FFF \
    -O ...

Of course, all options are documented in the user's guide.

This command add-pid in PMT table . but  how to let the pid (500), keep sending ?
avoid the TR101/290 pid error
"Referred PID does not occur for more than 5 seconds, pkt 133084"

This command add-pid in PMT table . but how to let the pid (500), keep sending ?
avoid the TR101/290 pid error
"Referred PID does not occur for more than 5 seconds, pkt 133084"

See my answer in #375


chapter 2 - training loss differet in github and in the book

Hello, about the code in chapter 2, in the training section, I wonder why the loss calculation is different from that on the book.
def train(model, optimizer, loss_fn, train_loader, val_loader, epochs=5, device="cpu"):
    for epoch in range(1, epochs+1):

[...]
from github:
        training_loss += loss.data.item() * inputs.size(0)
training_loss /= len(train_loader.dataset)

Here you multiply the loss.data.item() * input.size(0), which is the number of elements in the batch input (64 elements). That's because by default, "the losses are averaged over each loss element in the batch.", so you calculate a cumulative loss for the entire batch.
Then you divide the training_loss by the total number of the elments in the dataset (about 800 elements). So this is a medium loss for each element.
from the book:
        training_loss += loss.data.item()
training_loss /= len(train_iterator)

The book code seems wrong in this case, and github code is right. Because in the last row you had to divide by the length of the batch?
[...]
from github:
            valid_loss += loss.data.item() * inputs.size(0)
            correct = torch.eq(torch.max(F.softmax(output, dim=1), dim=1)[1], targets)
            num_correct += torch.sum(correct).item()
            num_examples += correct.shape[0]
        valid_loss /= len(val_loader.dataset)

from the book:
            valid_loss += loss.data.item()
            correct = torch.eq(torch.max(F.softmax(output), dim=1)[1],
            target).view(-1)
            num_correct += torch.sum(correct).item()
            num_examples += correct.shape[0]
            valid_loss /= len(valid_iterator)

Yes, there have been a few typos in the book release which have corrected meanwhile, e.g. train_iterator not defined. Additionaly the PyTorch API has had some changed which were leading to deprecation messages arising from missing dimension arguments in torch.max and F.softmax (https://github.com/falloutdurham/beginners-pytorch-deep-learning/commit/df4e02b4d6d1ded1fe6accccea3bd74ba5fc2960#diff-1abbb440e65745ab4ef4f92e32643594fc9af5130d03dfc6879613047f083bd9)
And we noticed that .view(-1) was redundant, so we removed it.
"The book code seems wrong in this case, and github code is right. Because in the last row you had to divide by the length of the batch?"

Yes, the github code is working. In general you are right about the part on input.size(0) multiplication (see e.g. here https://stackoverflow.com/questions/61092523/what-is-running-loss-in-pytorch-and-how-is-it-calculated) and on averaging the loss (but in the last row you divide by the length of the whole dataloader [i.e. no. items in dataset], not the length of the batch, as you were summing up the cumulative loss for all batches before). The same holds for the val_loader.


Add display mode tip

## Mode of the display (tip/popup)
displayMode: "tip"

Example
$this->player->sendTip

I will add an update for this either tonight or tomorrow morning, thank you.


When using HotModuleReplacementPlugin(HMR),  one file‘s change cause all files to emit.

Bug report

What is the current behavior?
In the increment build time, when I use HMR and change one file, all chunks will emit. It is deserve that
only chunk corresponding to the changed file should rerender and emit.
I find the problem is in  webpack/lib/JavascriptModulesPlugin.js  at line 61.
When use HMR
const useChunkHash = compilation.mainTemplate.useChunkHash(chunk);

useChunkHash is always false. So it is cannot use chunk.hash. Why?
If the current behavior is a bug, please provide the steps to reproduce.
webpack.config.js
const path = require('path');
const webpack = require('webpack');

module.exports = {
    devtool: 'cheap-module-source-map',
    mode: 'development',
    entry: {
        'dd3': './app/pages/dd3/export.js',
        'dd4': './app/pages/dd4/export.js',
        'dd5': './app/pages/dd5/export.js',
    },

    output: {
        path: path.resolve(__dirname, "webpackdist"),
        filename: `[name].js`,
        chunkFilename: `[name]_chunk.js`
    },

    devServer: {
        hot: true,
        port: 8804,
    },

    optimization: {
        splitChunks: {
            chunks: 'all',
            name: 'vendor',
            minSize: 0,
            minChunks: 2,
        },
    },

    plugins: [
        new webpack.HotModuleReplacementPlugin()
    ]
}

What is the expected behavior?

Other relevant information:
webpack version:  4.17.1
Node.js version: 8.9.4
Operating System: Mac
Additional tools:

i got the same problem


make posix_sitl_default failed -Wfatal-errors

i have tried to follow tutorial on how using mavros in this link : https://gaas.gitbook.io/guide/software-realization-build-your-own-autonomous-drone/build-your-own-autonomous-drone-e01-offboard-control-and-gazebo-simulation
however, when i was trying to run "make posix_sitl_default" inside firmware directory it returns bunch of errors which i dont understand.
also im using ubuntu 20 and using ros noetic distro.
ninja: Entering directory /home/vergil/mavros_projects/1/catkin_ws/src/Firmware/build/posix_sitl_default'
[6/325] Building CXX object src/driver...Files/drivers__tfmini.dir/tfmini.cpp.o
FAILED: src/drivers/distance_sensor/tfmini/CMakeFiles/drivers__tfmini.dir/tfmini.cpp.o
/usr/bin/ccache /usr/bin/c++  -DCONFIG_ARCH_BOARD_SITL -DMODULE_NAME=\"tfmini\" -DPX4_MAIN=tfmini_app_main -D__DF_LINUX -D__PX4_LINUX -D__PX4_POSIX -D__STDC_FORMAT_MACROS -Dnoreturn_function="__attribute__((noreturn))" -I. -Isrc -Isrc/lib -Isrc/modules -I../../src -I../../src/drivers/boards/sitl -I../../src/include -I../../src/lib -I../../src/lib/DriverFramework/framework/include -I../../src/lib/matrix -I../../src/modules -I../../src/platforms -I../../platforms/posix/include -Iexternal/Install/include -g -fno-exceptions -fno-rtti -std=gnu++11 -fno-threadsafe-statics -DCONFIG_WCHAR_BUILTIN -D__CUSTOM_FILE_IO__ -fcheck-new -Wall -Warray-bounds -Wdisabled-optimization -Werror -Wextra -Wfatal-errors -Wfloat-equal -Wformat-security -Winit-self -Wlogical-op -Wmissing-declarations -Wmissing-field-initializers -Wpointer-arith -Wshadow -Wuninitialized -Wunknown-pragmas -Wunused-variable -Wno-implicit-fallthrough -Wno-unused-parameter -Wunused-but-set-variable -Wformat=1 -Wdouble-promotion -Wno-missing-field-initializers -Wno-overloaded-virtual -Wreorder -Wno-format-truncation -fvisibility=hidden -include visibility.h -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -O2 -g -DNDEBUG   -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -Wno-sign-compare -std=gnu++11 -MD -MT src/drivers/distance_sensor/tfmini/CMakeFiles/drivers__tfmini.dir/tfmini.cpp.o -MF src/drivers/distance_sensor/tfmini/CMakeFiles/drivers__tfmini.dir/tfmini.cpp.o.d -o src/drivers/distance_sensor/tfmini/CMakeFiles/drivers__tfmini.dir/tfmini.cpp.o -c ../../src/drivers/distance_sensor/tfmini/tfmini.cpp
In file included from /usr/include/string.h:495,
                 from ../../src/drivers/distance_sensor/tfmini/tfmini.cpp:53:
In function ‘char* strncpy(char*, const char*, size_t)’,
    inlined from ‘TFMINI::TFMINI(const char*, uint8_t)’ at ../../src/drivers/distance_sensor/tfmini/tfmini.cpp:193:9:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:34: error: ‘char* __builtin_strncpy(char*, const char*, long unsigned int)’ specified bound 20 equals destination size [-Werror=stringop-truncation]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
[7/325] Building CXX object src/driver...MakeFiles/drivers__sf0x.dir/sf0x.cpp.o
FAILED: src/drivers/distance_sensor/sf0x/CMakeFiles/drivers__sf0x.dir/sf0x.cpp.o
/usr/bin/ccache /usr/bin/c++  -DCONFIG_ARCH_BOARD_SITL -DMODULE_NAME=\"sf0x\" -DPX4_MAIN=sf0x_app_main -D__DF_LINUX -D__PX4_LINUX -D__PX4_POSIX -D__STDC_FORMAT_MACROS -Dnoreturn_function="__attribute__((noreturn))" -I. -Isrc -Isrc/lib -Isrc/modules -I../../src -I../../src/drivers/boards/sitl -I../../src/include -I../../src/lib -I../../src/lib/DriverFramework/framework/include -I../../src/lib/matrix -I../../src/modules -I../../src/platforms -I../../platforms/posix/include -Iexternal/Install/include -g -fno-exceptions -fno-rtti -std=gnu++11 -fno-threadsafe-statics -DCONFIG_WCHAR_BUILTIN -D__CUSTOM_FILE_IO__ -fcheck-new -Wall -Warray-bounds -Wdisabled-optimization -Werror -Wextra -Wfatal-errors -Wfloat-equal -Wformat-security -Winit-self -Wlogical-op -Wmissing-declarations -Wmissing-field-initializers -Wpointer-arith -Wshadow -Wuninitialized -Wunknown-pragmas -Wunused-variable -Wno-implicit-fallthrough -Wno-unused-parameter -Wunused-but-set-variable -Wformat=1 -Wdouble-promotion -Wno-missing-field-initializers -Wno-overloaded-virtual -Wreorder -Wno-format-truncation -fvisibility=hidden -include visibility.h -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -O2 -g -DNDEBUG   -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -Wno-sign-compare -std=gnu++11 -MD -MT src/drivers/distance_sensor/sf0x/CMakeFiles/drivers__sf0x.dir/sf0x.cpp.o -MF src/drivers/distance_sensor/sf0x/CMakeFiles/drivers__sf0x.dir/sf0x.cpp.o.d -o src/drivers/distance_sensor/sf0x/CMakeFiles/drivers__sf0x.dir/sf0x.cpp.o -c ../../src/drivers/distance_sensor/sf0x/sf0x.cpp
In file included from /usr/include/string.h:495,
                 from ../../src/drivers/distance_sensor/sf0x/sf0x.cpp:51:
In function ‘char* strncpy(char*, const char*, size_t)’,
    inlined from ‘SF0X::SF0X(const char*, uint8_t)’ at ../../src/drivers/distance_sensor/sf0x/sf0x.cpp:196:9:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:34: error: ‘char* __builtin_strncpy(char*, const char*, long unsigned int)’ specified bound 20 equals destination size [-Werror=stringop-truncation]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
[9/325] Building CXX object src/driver.../CMakeFiles/drivers__gps.dir/gps.cpp.o
FAILED: src/drivers/gps/CMakeFiles/drivers__gps.dir/gps.cpp.o
/usr/bin/ccache /usr/bin/c++  -DCONFIG_ARCH_BOARD_SITL -DMODULE_NAME=\"gps\" -DPX4_MAIN=gps_app_main -D__DF_LINUX -D__PX4_LINUX -D__PX4_POSIX -D__STDC_FORMAT_MACROS -Dnoreturn_function="__attribute__((noreturn))" -I. -Isrc -Isrc/lib -Isrc/modules -I../../src -I../../src/drivers/boards/sitl -I../../src/include -I../../src/lib -I../../src/lib/DriverFramework/framework/include -I../../src/lib/matrix -I../../src/modules -I../../src/platforms -I../../platforms/posix/include -Iexternal/Install/include -g -fno-exceptions -fno-rtti -std=gnu++11 -fno-threadsafe-statics -DCONFIG_WCHAR_BUILTIN -D__CUSTOM_FILE_IO__ -fcheck-new -Wall -Warray-bounds -Wdisabled-optimization -Werror -Wextra -Wfatal-errors -Wfloat-equal -Wformat-security -Winit-self -Wlogical-op -Wmissing-declarations -Wmissing-field-initializers -Wpointer-arith -Wshadow -Wuninitialized -Wunknown-pragmas -Wunused-variable -Wno-implicit-fallthrough -Wno-unused-parameter -Wunused-but-set-variable -Wformat=1 -Wdouble-promotion -Wno-missing-field-initializers -Wno-overloaded-virtual -Wreorder -Wno-format-truncation -fvisibility=hidden -include visibility.h -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -O2 -g -DNDEBUG   -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -std=gnu++11 -MD -MT src/drivers/gps/CMakeFiles/drivers__gps.dir/gps.cpp.o -MF src/drivers/gps/CMakeFiles/drivers__gps.dir/gps.cpp.o.d -o src/drivers/gps/CMakeFiles/drivers__gps.dir/gps.cpp.o -c ../../src/drivers/gps/gps.cpp
In file included from /usr/include/string.h:495,
                 from ../../src/drivers/gps/gps.cpp:59:
In function ‘char* strncpy(char*, const char*, size_t)’,
    inlined from ‘GPS::GPS(const char*, gps_driver_mode_t, GPSHelper::Interface, bool, bool, GPS::Instance)’ at ../../src/drivers/gps/gps.cpp:288:9:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:34: error: ‘char* __builtin_strncpy(char*, const char*, long unsigned int)’ specified bound 20 equals destination size [-Werror=stringop-truncation]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
[11/324] Building CXX object src/drive...s/drivers__ulanding.dir/ulanding.cpp.o
FAILED: src/drivers/distance_sensor/ulanding/CMakeFiles/drivers__ulanding.dir/ulanding.cpp.o
/usr/bin/ccache /usr/bin/c++  -DCONFIG_ARCH_BOARD_SITL -DMODULE_NAME=\"ulanding_radar\" -DPX4_MAIN=ulanding_radar_app_main -D__DF_LINUX -D__PX4_LINUX -D__PX4_POSIX -D__STDC_FORMAT_MACROS -Dnoreturn_function="__attribute__((noreturn))" -I. -Isrc -Isrc/lib -Isrc/modules -I../../src -I../../src/drivers/boards/sitl -I../../src/include -I../../src/lib -I../../src/lib/DriverFramework/framework/include -I../../src/lib/matrix -I../../src/modules -I../../src/platforms -I../../platforms/posix/include -Iexternal/Install/include -g -fno-exceptions -fno-rtti -std=gnu++11 -fno-threadsafe-statics -DCONFIG_WCHAR_BUILTIN -D__CUSTOM_FILE_IO__ -fcheck-new -Wall -Warray-bounds -Wdisabled-optimization -Werror -Wextra -Wfatal-errors -Wfloat-equal -Wformat-security -Winit-self -Wlogical-op -Wmissing-declarations -Wmissing-field-initializers -Wpointer-arith -Wshadow -Wuninitialized -Wunknown-pragmas -Wunused-variable -Wno-implicit-fallthrough -Wno-unused-parameter -Wunused-but-set-variable -Wformat=1 -Wdouble-promotion -Wno-missing-field-initializers -Wno-overloaded-virtual -Wreorder -Wno-format-truncation -fvisibility=hidden -include visibility.h -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -O2 -g -DNDEBUG   -fno-strict-aliasing -fomit-frame-pointer -fno-math-errno -funsafe-math-optimizations -ffunction-sections -fdata-sections -fno-strength-reduce -fno-builtin-printf -fdiagnostics-color=always -std=gnu++11 -MD -MT src/drivers/distance_sensor/ulanding/CMakeFiles/drivers__ulanding.dir/ulanding.cpp.o -MF src/drivers/distance_sensor/ulanding/CMakeFiles/drivers__ulanding.dir/ulanding.cpp.o.d -o src/drivers/distance_sensor/ulanding/CMakeFiles/drivers__ulanding.dir/ulanding.cpp.o -c ../../src/drivers/distance_sensor/ulanding/ulanding.cpp
In file included from /usr/include/string.h:495,
                 from ../../src/platforms/px4_includes.h:61,
                 from ../../src/platforms/px4_defines.h:92,
                 from ../../src/drivers/distance_sensor/ulanding/ulanding.cpp:45:
In function ‘char* strncpy(char*, const char*, size_t)’,
    inlined from ‘ulanding::Radar::Radar(uint8_t, const char*)’ at ../../src/drivers/distance_sensor/ulanding/ulanding.cpp:151:9:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:106:34: error: ‘char* __builtin_strncpy(char*, const char*, long unsigned int)’ specified bound 20 equals destination size [-Werror=stringop-truncation]
  106 |   return __builtin___strncpy_chk (__dest, __src, __len, __bos (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated due to -Wfatal-errors.
cc1plus: all warnings being treated as errors
[15/324] Building CXX object src/syste...iles/systemcmds__param.dir/param.cpp.o
ninja: build stopped: subcommand failed.
make: *** [Makefile:148: posix_sitl_default] Error 1

what is wrong??

This appears to be quite an old version of PX4? Can you make sure you're on current master, I believe most of these string operation warnings were fixed.

@badsector998 I think if you are using latest master, the correct command would be
make px4_sitl_default

or
make px4_sitl <simulator>

which  can be either gazebo, jmavsim, jsbsim depending on what you want to run

@dagar @Jaeyoung-Lim thanks for the reply. I'll try it again tomorrow in my workplace. I'll post again for the update 😃

@dagar @Jaeyoung-Lim here's the update.. i actually already have the newest version of this firmware. But it seems really consume a lot of resources to just do the make command and wrap it to gazebo. Is it really resources consuming? Even my laptop freezes a lot when do the make command.. and i haven't completed the make yet, i canceled it due to the freezing.

What command were you using?

What command were you using?

Just like you said
make px4_sitl gazebo


Update package metadata

Background
Hello there! The Python packaging ecosystem has standardized on the interface for build backends (PEP 517/PEP 660) and the format for metadata declaration (PEP 621/PEP 631). As a result, the execution of setup.py files is now deprecated.
So, I'm spending my free time updating important projects so that they are modernized and set an example for others 😄
related: https://github.com/jazzband/help/issues/313
Summary of changes
This implements PEP 621, obviating the need for setup.py (see note), setup.cfg, and MANIFEST.in. The build backend hatchling (of which I am a maintainer in the PyPA) is now used as that is the default in the official Python packaging tutorial. Hatchling is available on all the major distribution channels such as Debian, Fedora, Arch Linux, conda-forge, Nixpkgs, Alpine Linux, FreeBSD/OpenBSD, Gentoo Linux, MacPorts, OpenEmbedded, Spack, MSYS2, etc.
Notes

The version is now sourced directly from debug_toolbar/__init__.py, removing a release step
The source distributions on PyPI are erroneously shipping a build artifact *.egg-info from python setup.py develop; this is now fixed
Let me know if I should remove setup.py, I followed https://github.com/urllib3/urllib3/blob/main/setup.py
I updated the pin for the pypa/gh-action-pypi-publish Action https://github.com/pypa/gh-action-pypi-publish#-master-branch-sunset-

I don't have a strong opinion one way or another for packaging, but it seems like hatch is mature enough for us to use. I would suggest us running a test with test.pypi.org first. I've made a request to the owner of the django debug toolbar package on test.pypi.org to add me as an owner. If I'm able to get those permissions, I'll add you as well @matthiask.

Why should we use hatchling?

Not a complete list but here's from a recent comment -
Hatchling:

Supports static analysis tools for editable installations by default whereas the new setuptools does not i.e. it requires enabling an option for IDEs to work.
Uses Git-style glob patterns rather than stdlib globs to better match user expectations.
Allows easy and explicit control of what gets shipped in the sdist target whereas with setuptools it's an objectively confusing mix of conditional wheel options and a MANIFEST.in file.

I don't know about leaving setup.py in there.

Can I remove it?

Hatchling: [...]

Thanks!

Can I remove it?

I think it probably makes more sense to copy urllib3 there as well? See https://github.com/community/community/discussions/6456

Done!

@hugovk

One suggestion: shall we add https://github.com/tox-dev/pyproject-fmt/ to pre-commit to consistently format pyproject.toml?

Can we do that in a separate PR?

@matthiask It'll take at least one more week until I have ownership of the test pypi repo. Do you want to do a release with the code before this change, then this after we test the release process on test.pypi.org? Or batch it all together?

@tim-schilling I think it's fine to batch it. We can always push another release or revert in the worst case.

another just merged https://github.com/jazzband/prettytable/pull/212

Me neither, let's merge it then. Thanks all!

Great. Can we create a recipe with steps to apply this update in other jazzband packages ?

First step: https://hatch.pypa.io/latest/intro/#existing-project

@hugovk

One suggestion: shall we add https://github.com/tox-dev/pyproject-fmt/ to pre-commit to consistently format pyproject.toml?

Can we do that in a separate PR?

Please see PR https://github.com/jazzband/django-debug-toolbar/pull/1700.


Enhancement - Fb::Page#posts with options.

Coverage decreased (-0.4%) to 99.567% when pulling e87ae3f83babb4b423fcc854e94e34b02282b913 on post_fields into 8ceabf7aeecc5ea492b2cec67bae93992f003b3c on master.

Coverage decreased (-0.4%) to 99.567% when pulling e87ae3f83babb4b423fcc854e94e34b02282b913 on post_fields into 8ceabf7aeecc5ea492b2cec67bae93992f003b3c on master.

Coverage remained the same at 100.0% when pulling ca52080bad8aa0daa4a90d4462f9375c78df0c68 on post_fields into 8ceabf7aeecc5ea492b2cec67bae93992f003b3c on master.


Allow to pass ip and tags into log() method

We write our node server access log with LogDNA. For each request to this http server, we trace ip address of the remote client, URL, status code and so on.
This forces us to create new logger for each request because ip address is bound to the instance. All these instances are hard to manage and inefficient. Ability to specify ip at log() call would allow to use single instance in cases like this.

@gbiryukov, our new node client was released yesterday.  We'd recommend upgrading to the new version, and shortly we can begin a minor release to add this requested feature!

A quick update on this feature request.  It's not going to be as simple to add as originally thought because the ip (and tags) is added at the request level, and each request contains multiple lines.  We can get it added, but it will require changes at the ingestion process.  Stand by, we won't forget about you!

If my request does not fit into logger architecture we can move client IP into meta. I am curious about what would be the impact if we will do it?
we capture IP only for audit purposes and does not have any functionality on top of it.

If my request does not fit into logger architecture we can move client IP into meta. I am curious about what would be the impact if we will do it?

Yes, that should work.  You'll want to set indexMeta: true so that the ip field (whatever you call it) is searchable.  Feel free to try this and report back if it's sufficient for you.  Thanks!


Added compatibility with mods that inject into PP-related vanilla functions

Though I just noticed that Cookie Monster would also break with kitten upgrades, checking that.

It's related to how CM also gets a static copy of Game.CalculateGains. It'll take some time, but I'll check what's useless from there. (Changing CM.Sim's to vanilla and checking if anything breaks)

Just thinking out loud here: is there a way to create a "Create Sim-copy" function that can then be called on each Game function to create one where everything is replaced?
I am thinking of one function that stores all possible replacement actions (so Game.Objects to Sim.Objects and Game.Has to Sim.Has) which is then passed functions like Game.CalculateGains() and Game.GetTieredCpsMult()

Another problem I just thought of is that using new Function will also result in errors with ESLint as it is basically an eval.
This is also the problem I ran into last time when I wanted to remove all evals.

Just thinking out loud here: is there a way to create a "Create Sim-copy" function that can then be called on each Game function to create one where everything is replaced?
I am thinking of one function that stores all possible replacement actions (so Game.Objects to Sim.Objects and Game.Has to Sim.Has) which is then passed functions like Game.CalculateGains() and Game.GetTieredCpsMult()

Not as broad as that, but you can use the same method I used to automate that.
new Function(
	`return ${Game.anyFunction.toString()
	.replaceAll(/Game/, 'CM.Sim')}`,
 )();

In an even more automated way:
const gameObjKeys = Object.keys(Game);
const gameObjValues = Object.values(Game);
const CM.Sim = {};

gameObjValues.forEach((originalValue, index) => {
 let injectedFunc;

 if (typeof originalFunc === 'function') {
  injectedFunc = new Function(
   `return ${originalValue.toString()
    .replaceAll(/Game/, 'CM.Sim.')}`,
  )();
 }

 Object.assign(CM.Sim, {
  [gameObjKeys[index]]: injectedFunc || originalValue,
 })
})

That should create a copy of Game with all Game methods changed to CM.Sim, (might need some checking to make sure nothing was changed unintentionally) though it'd probably lag a bit since that's pretty much going through the entire game.
Creating CM.Sim functions in this new Function()() way would be an easy fix for this problem which takes care of injections, especially since you'd already want to do that eventually when optimizing Cookie Monster. But you'd need to have a warning to place Cookie Monster last on loading order/setting a delay as noted by you before, so it isn't that much of a fix. (Although doable)

don't know if this is intended but I think we should still create CM.Sim..GetTieredCpsMult() otherwise the base game will start to use the data stored in CM.Sim instead of Game to calculate CPS and run the game.

I don't know what that'd affect Yeah, CpS changes some time after buying/selling buildings. It was intended, as that'd make the function work with all future injections, but it introduced that bug. Also checking a possible way to fix that.

Another problem I just thought of is that using new Function will also result in errors with ESLint as it is basically an eval.
This is also the problem I ran into last time when I wanted to remove all evals.

For that, it is best to disable no-new-func rule. You already need to disable quite a few rules to work with Orteil code and ESLint. And Are eval() and new Function() the same thing?

Should indeed be able to do this somewhat automated, although I think I prefer doing it on a per-function basis instead of the full game at once.
We could put the replacing functions into CM.Main.Loop() to circumvent the issue of CM loading before a content mod, although that feels a bit "hacky" an introduces unnecessary functions into the loop (it would only rarely actually be useful).
Perhaps doing something with checking in the loop if the number of registered mods has increased and in that case re-replace all functions might work.
Currently I am only using the recommend set of rules for ESLint. Changing to all or airbnb gave about 10k errors (mostly indentation and style). If I ever change this using no-new-func will probably be necessary indeed.

I have worked on this some more and have implemented code that recognizes content mods in #592.
Please do tell if any issues remain, but from my personal testing it all seems to work now!


Issue with evaluating the ALOV300++ dataset

When I am trying to evaluate my tracker using the ALOV300++ dataset in http://imagelab.ing.unimore.it/dsm/index.php I couldnt proceed further because in the Trackers tab, I am unable to choose any category as after selecting the ALOV300++ no category is been loaded.
Is there any work around or any other site for doing the same?

I only used ALOV300++ for training and validation (not for testing), so I didn't need to compare my tracker to other trackers on this dataset.


Review JIRA project boards elements

Review the JIRA project board element used by Sage. Motivations:

Get inspiration from another tool that is well-established
Adopt similar element to JIRA to ease communication between us and PM who uses JIRA

See the project board for the Synapse web client.

@jiaxinmachine88 Now tracking this task at the workflow level.


⚠ Game Join API Endpoint has degraded performance

In 4d6301e, Game Join API Endpoint (http://gamejoin.roblox.com/) experienced degraded performance:

HTTP code: 200
Response time: 2033 ms

Resolved: Game Join API Endpoint performance has improved in a479e4e after 12 minutes.


Unset properties in style plugins

Problem
The tailwind style plugin can't differentiate between a prop being removed from the inline style prop, and that prop never having been there at all. For example, when a flex gap is removed by dragging it in the negative direction, it goes from positive -> 0 -> being removed from the style prop. When the normalisation step kicks in, there's no gap prop in the inline style, so the tailwind plugin doesn't do anything. To make matters worse, when the gap property is removed while the interaction is in progress, the value coming from the generated tailwind class takes precedence, so the flex gap jumps back to its original size.
Fix: propertiesToUnset + cleanup
The fix for this is to create a piece of state in the editor canvas state, propertiesToUnset, which contains the props to unset in the normalization step, and contain the prop value that needs to be written to the inline style prop during an interaction to simulate that prop being removed (and preventing any other definition for that given prop to take effect).
propertiesToUnset is written to by the DeleteProperties command. Unless the command is in on-complete mode, the command records which properties are to be removed, sets them in propertiesToUnset, and re-applies them with their "zero" value. Since we need to manually define a zero value for each prop that's removed by any of the strategies, propertiesToUnset will be expanded incrementally (right now it only has gap, which is set/unset by the plugin-aware flex gap strategy)
Details
transformJSXComponentAtElementPath throws an error when it doesn't find an element to transform
For example, if an element had gap set to 0px, and then it moved under a new parent, when propertiesToUnset is cleaned up (after the child element is already under the new parent), transformJSXComponentAtElementPath will throw an error, because it can't find the child element at its old element path (which was valid when it was a sibling of the new parent). This manifested itself in breaking tests, where an element that had a property removed and then got reparented seemingly couldn't be reparented.
I removed the exception, because the function is correct without it (if there's no element to transform, we might as well regard it as already transformed).
deleteProperties is used from the inspector, so the normalization step needs to be run after inspector edits too
Concretely, since deleteProperties is used from the inspector too, it set the zero value for gaps too, which had to be cleaned up. I fixed this by creating the runNormalizationStep action and calling it in executeFirstApplicableStrategyForContinuousInteraction. This code could also live in the main dispatch flow, with the tradeoff that there it's harder to find out which elements to normalize. The implementation is a stub for now, it'll be fleshed out in a follow-up PR, when we get to adding tailwind/plugin support to the inspector

I have a feeling that runNormalizationStep should be its own distinct step in the editor.tsx outer dispatch flow, similar to the group true up step. but for now, this is good

Yeah I very strongly feel the same way. This PR introduces an extra field into the EditorState specifically for providing data to this normalisation step, and I'm wondering if actually the solution here should be to store enough data in the editor state to be able to run the normalisation step as part of the dispatch flow. I think this is worth a chat in the morning, because it's better for us to invest time in doing this correctly now rather than accepting what we have "for now"

As for the comment about transformJSXComponentAtElementPath - we have an internal function transformAtPathOptionally which that calls. It sounds like maybe we should introduce a transformJSXComponentAtElementPathOptionally specifically for the case where it's fine for the transformation to fail silently if the path no longer exists. That being said, I'm not sure I'd agree with "if there's no element to transform, we might as well regard it as already transformed" - doesn't that just mean that we'd fail to unset the property, meaning it would remain on the element?

closing in favor of https://github.com/concrete-utopia/utopia/pull/6549/


Working netkan indexing script for new host

Some Perl for you @pjf
It likely needs a few tweaks, but it should give a good start to improve the indexing.

Has logging
Shouldn't barf on any invalid data
Shouldn't commit any invalid data

It may have inadvertently had some live testing -> http://git.io/vTCEP

Unless we think it's a silly idea, I'm thinking the indexing bot should be its own thing. So I'll close this for now.


Feat/fuzz

This PR implements the Fuzz commands, with the arm, disarm and run sub-comands.

Moved here: https://github.com/ConsenSys/dili-faas-cli/pull/1


KeyError: 'document'

Traceback (most recent call last):
File "c:\users\adit\appdata\local\programs\python\python38-32\lib\runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "c:\users\adit\appdata\local\programs\python\python38-32\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\Adit\AppData\Local\Programs\Python\Python38-32\Scripts\tkdesigner.exe_main.py", line 7, in
File "c:\users\adit\appdata\local\programs\python\python38-32\lib\site-packages\tkdesigner\cli.py", line 71, in main
designer.design()
File "c:\users\adit\appdata\local\programs\python\python38-32\lib\site-packages\tkdesigner\designer.py", line 29, in design
code = self.to_code()
File "c:\users\adit\appdata\local\programs\python\python38-32\lib\site-packages\tkdesigner\designer.py", line 21, in to_code
window_data = self.file_data["document"]["children"][0]["children"][0]
KeyError: 'document'
This is the error I get when I run tkdesigner FILE_ID TOKEN

Python is added to PATH
pip install tkdesigner has already been run
pip is up to date
I have followed naming conventions (as far as I know)

File URL: https://www.figma.com/file/1Dd7aygqhMcCRYwtinm6Oi/File-Converter?node-id=2%3A2

hey @aditdesai
i see you have python installed but your running in it wrong directory mate.
open a fresh terminal or cmd and run these commands in order:
git clone https://github.com/ParthJadhav/Tkinter-Designer.git
cd tkinter-designer
pip3 install -r requirements.txt
pip3 install tkdesigner
cd tkinter-designer (optional: only if u arent still in tkinter-designer directory after the pip3 steps)
cd gui
python3 gui.py

@aditdesai did @AbdulkadirH5253 's comment help you? If yes, then you can close this issue.

Hey, I am seeing this issue as well

Same error here.

Hi,
I've followed those steps which @AbdulkadirH5253 mentioned above.
But this error occurs when I run gui.py, not sure why:
window_data = self.file_data["document"]["children"][0]["children"][0]
KeyError: 'document'

you did it?
On Sun, 14 Apr 2024 at 23:31, Adit Desai @.***> wrote:

Closed #208 https://github.com/ParthJadhav/Tkinter-Designer/issues/208
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/ParthJadhav/Tkinter-Designer/issues/208#event-12461244352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX4CHOSUDRTJUWIIZQB56KTY5LRRTAVCNFSM5SLATBG2U5DIOJSWCZC7NNSXTWQAEJEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW4OZRGI2DMMJSGQ2DGNJS
.
You are receiving this because you were mentioned.Message ID:
@.***
com>


5.4.0

Releases https://github.com/stoplightio/prism/pull/2401 in version 5.4.0

If you haven't released this yet, could you wait? We have one more PR that is going to get merged today that we'd like to release at the same time.

https://github.com/stoplightio/prism/pull/2402 specifically this is the PR we would like to have included with this release

Whoops!  I looked through the changed files and approved before I saw @chohmann and @brendarearden 's comments above.

my PR has been merged and it also fixes the lint error

I don't know what happened, but it looks like the changes for https://github.com/stoplightio/prism/pull/2398 are no longer on master.

oh gosh darn it. This repo is set up with merge commits by default instead of squash commits, so master got really messy when my PR got merged and I tried to fix it but clearly fucked things up a bit.

I'm fixing the PR settings to be in line with what platform-internal uses.
Meanwhile, I'll cherry-pick https://github.com/stoplightio/prism/pull/2398 back somehow. Or if anyone has a older copy of master and wants to force push, go ahead.

@chohmann are there any other PRs that got merged since the 5.3.2 release that are missing?

@chohmann are there any other PRs that got merged since the 5.3.2 release that are missing?

@billiegoose I think that's the only one.
The three PRs that should be released with 5.4.0 are:

https://github.com/stoplightio/prism/pull/2398
https://github.com/stoplightio/prism/pull/2401
https://github.com/stoplightio/prism/pull/2402

Alright, 🤣 hopefully it's good to go now. Sorry about the mishap folks, won't happen again (since I've disabled all other merge options mwahahaha).

@billiegoose awesome! I think everything looks good, but I see that the version as been updated lerna.json and package.json's, but the tag has not been created in github. That happens when running yarn release usually.
Are you following the release steps documented here?

No, I scanned the CONTRIBUTING.md and didn't find anything so I just did my best copy-paste.
I gotta run to a dinner, any chance someone else can carry this across the finish line?

I got it, @billiegoose !


Upload to Google Cloud Platform?

Has anynone been able to upload their project to Google Cloud Platform

Have not tried. Are you having issues? I typically use the Cloud SDK Shell to upload static client-side files, so it "should" work. OR are you referring more to the server-side rendering part?

Managed to upload the build folder to GCP, but the routing is not working. Currently we have an instance that runs the development server in the boilerplate.

Is the contents of the build directory in the root directory OR has it been placed in a subdirectory???
If it is in a subdirectory (e.g. build/) you may have to edit the webpack internals, see:

Webpack config file
Webpack docs for more info

Let us know how you go

Did you get this working @BrandoIsNotASword?

Did you get this working @BrandoIsNotASword ?

Closing due to inactivity, please re-open if needed.


PCBway A noob question

Hi Ben,
I really appreciate the work you've done and the fact that it's open source. I was delighted to find your GitHub with a lead from Gennady's "Skyentific" YouTube channel.
I want to get a quote from PCBway on ten assembled controllers and have a noob question (or two) regarding how to get the quote from their online estimator tool.
I see that there is a build files directory for PCBway , which zip file do I upload to the estimator tool....is it one of files in that directory, is it the MFG.zip in the parent directory, or another?
Is there a another step I'm missing? I've tried uploading MFG.zip but am unclear on the correct values for number of components and number of pads.
I'm sorry to be asking for such basic help, I'm new to this level of hardware R&D, my expertise lies elsewhere and I'm trying to achieve maximum value on a shoestring budget without accidentally blowing the budget by making a procedural error at the quotes stage. Hopefully I'm on the right track and that you won't mind too much me asking.
yours,
John

oops, I may have just described above he steps I went through for trying to arrange a quote from NEXTPCB...but at this stage any insight would be good
regards,
John

@jpol0006  did you have any luck ordering assembled and working boards? I saw the same Skyentific video, and I am also curious as to how to complete an order for assembled boards. Maybe you could provide some more steps as to what you had to select.
@bgkatz are these boards that should use a 2oz copper?

@androiddrew ... I did, they wanted an insane amount for 10 boards assembled, I uploaded the MFG.zip to define the boards and the centroids file per the online upload thingy, used the total parts numbers from the excel spreadsheet... I priced them up with several different pcb manufacturers and the process was slightly different with each... the lowest quote was around $6,000 , the highest was over $13,000... needless to say the project is on hold since the built boards can be ordered via AliExpress for around $60 each, with a small price break on volume purchases

2 oz is good, 1 oz is okay if you're not pushing them too hard.
Many of the components are out of stock at the moment due to the global chip shortage, so I'm not surprised you're quotes were super high.  Normally in qty 10 they cost me about $100 each, and in higher qty about $65 each from pcbway.


New feature: Support of point addition in curve25519 and ed25519

For one of my projects I am missing the addition operation of two points, both in curve curve25519 and edwards25519.
I am wondering if you can add such methods to the classes Ed25519.java and Curve25519.java.

Hi there, I'm sorry but we don't intend to provide such low level APIs.


Twig_Autoloader removed in favor of Composer

Timber/Timber.php checks for the Twig_AutoLoader class. As of Twig 2.0.0, the class has been removed, see Twig/CHANGELOG. This affects all installations of Timber, see Timber/composer.json.

quite serious bug, breaks site after composer update command

Jup same here. Getting fatal error:
Fatal error: You have not run "composer install" to download required dependencies for Timber, you can read more on https://github.com/timber/timber#installation in /srv/www/example.dev/current/vendor/timber/timber/lib/Timber.php on line 76

Confirming that Twig 2.2.0, Timber 1.3.0 and WP 4.7.3 do not work anymore. Same error as @Twansparant.
Any way to fix this while staying on 1.3.0 ?

Thanks so much for this report, I'll take a look at that ASAP. @curtisghanson — if you comment out lines 75-77, does everything work as expected?

Yes. That's how I got around it. I'd submit a pull request but I'm new to this project so I don't want to try and make changes before I know what I'm doing.

@curtisghanson @yantei @exophunk @Twansparant: thanks so much for your flagging and reporting on this issue. I was very confident of the Twig 2.0 and PHP 7 compatibility based on earlier Travis tests passing — but I failed to read the logs which showed that PHP 7 was actually using Twig 1.x despite the composer spec.
Regardless, I've created #1379 that now shows everything working. I'm going to merge this in and release as 1.3.0-beta2 — I'd love any feedback you're able to give about whether it resolves the issues you saw (or creates any new ones).
Thanks again!

@curtisghanson @yantei @exophunk @Twansparant: I've merged #1379 which resolves the issue here. Give it a whirl and let me know if it works, as highlighted above my next step is to update a beta release and see if anything else shakes out before making 1.3.0 official


No case clause for "attribute" with VSCode 1.82.3

I'm not quite sure how to reproduce this, but it just started recently. Everything works fine with VSCode and then at some point I get the following error repeatedly and VSCode stops letting me save the file that I'm working on.
** (CaseClauseError) no case clause matching: "attribute"
    (next_ls 0.13.4) lib/next_ls.ex:337: NextLS.handle_request/2
    (gen_lsp 0.6.0) lib/gen_lsp.ex:248: anonymous fn/5 in GenLSP.loop/3
    (gen_lsp 0.6.0) lib/gen_lsp.ex:308: GenLSP.attempt/3
    (stdlib 5.0.2) proc_lib.erl:241: :proc_lib.init_p_do_apply/3

It seems like one easy fix would be to ignore unknown reference types. I didn't know whether that was a bad idea so I figured that I'd post the issue.

Yes, it should just return nil in that case.


[Add a Play]: Roll the dice

What's the Play about?
Roll the dice is a game that will play you VS computer. Where you can roll the dice as many times as you want then add the points to the main cart if you got 🎲 "1" during roll the dice you will lose points and then computer will roll dice.
If you collect 100 points first before computer, you win 🏆 otherwise you loose 😪
What ReactJS concept will be used to create this play?
useState, useEffect, Hooks,
Are you willing to take it up for implementation?
Yes
Code of Conduct

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

Do you target this play for the #2PlaysAMonth event? @wyarejali


Wireframes for Quick Submit

In progress:
Quick Sub case form shows:

Title
Summary (140 character max)
Media (images & videos)
Location (multiple location and online options)

Quick Sub method form shows:

Title
Summary
Media
Related content (link to existing cases & orgs) *Not sure if this is too complex for quick sub

*should we include keywords?

Thanks @jesicarson this is great. Will create a few dependent issues which need to be implemented before this can actually be completed.
API endpoints:

https://github.com/participedia/api/issues/3
https://github.com/participedia/frontend/issues/53

UI widgets

https://github.com/participedia/frontend/issues/52


Fix readthedocs build process

This PR adds:

documentation build status tag in README,
fix for readthedocs process correctly import lambda_handlers

:tada: This PR is included in version 1.0.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot delete OR option

Once I add something to OR, I cannot remove it when I make a mistake like this (vlastník/provozovatel):

Already implemented


OutOfMemory exception when used in on older phones

And if use project in old device (with Android 5 and 2GB memory), sometimes we can get OutOfMemory. Same problem, if use Image from Core component.
Originally posted by @san3k1990 in https://github.com/PeterStaev/nativescript-image-swipe/issues/50#issuecomment-529382604

About OOM problem. The problem is faster in clearing the memory. The problem occurs if you go to the page with  ImageSwipe several times. Same problem with native component Image. I switched to the plugin nativescript-image (old nativescript-fresco) and the problem is gone. But i can't use nativescript-image in nativescript-image-swipe
Originally posted by @san3k1990 in https://github.com/PeterStaev/nativescript-image-swipe/issues/50#issuecomment-531721943

Have you found a solution to this problem?  I had the same problem.


Refactor statistic aggregation in BackupOp

BackupOp current determines how many resource owners participated in a single backup. However, we're moving to a model where BackupOp (and hence backups the user can see) are 1:1 with resource owners. This provides an opportunity to simplify the code we use for aggregation

Scrubbing backlog. Closing some of the older items that haven't been prioritized - will let them resurface if there is pain.


每行头部圆角、每行间距

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

https://drip-table.jd.com/drip-table-generator/functions/custom-theme


Preserve more separator HTML comment

Fixes https://github.com/lonekorean/wordpress-export-to-markdown/issues/93


Suggestion: add option to ignore CSV/TSV column header mismatch.

The ps command output some headers separated by only one space, so I can't split the output by /  +/, but / +/, then some lines get more fields than the headers. If ramda-cli can ignore those fields, then we can use ramda with ps and more commands.
ps aux | sed -E 's/ +/,/g'

USER,PID,%CPU,%MEM,VSZ,RSS,TTY,STAT,START,TIME,COMMAND
root,1,0.0,0.0,8324,156,?,Ss,15:55,0:00,/init,ro
root,3,0.0,0.0,8332,164,tty1,Ss,15:55,0:00,/init,ro
zhoukun,4,0.1,0.0,21096,6792,tty1,S,15:55,0:03,zsh
zhoukun,127,0.0,0.0,0,0,tty1,Z,15:56,0:00,[tr],<defunct>
root,692,0.0,0.0,8332,164,tty2,Ss,16:16,0:00,/init,ro
zhoukun,693,0.4,0.0,21096,6868,tty2,S,16:16,0:02,zsh
root,1037,0.0,0.0,8332,164,tty3,Ss,16:17,0:00,/init,ro
zhoukun,1042,1.3,0.0,21228,7176,tty3,S,16:17,0:06,zsh
zhoukun,2821,0.0,0.0,0,0,tty3,Z,16:22,0:00,[sed],<defunct>
zhoukun,3223,0.0,0.0,17400,1864,tty3,R,16:25,0:00,ps,aux
zhoukun,3224,0.0,0.0,14668,1040,tty3,S,16:25,0:00,sed,-E,s/,+/,/g

ps aux | sed -E 's/ +/,/g' | ramda -i csv 'id'

Error: Unexpected Error: column header mismatch expected: 11 columns got: 12

I suggest adding an option named unstrict mode to process CSV/TSV data, if a row has more fields than the headers, the extra fields will be ignored, and if fewer fields will consider as empty string fields.

Ramda-cli depends on fast-csv, I took a look into fast-csv, there are already parse options to ignore column mismatch, such as ignoreEmpty, discardUnmappedColumns, strictColumnHandling.

#16


Add a Footer containing link to all social media handles and mailing lists

Add a Footer containing links to all social media handles(we can use the logo of the particular handle) and link to mailing lists or any other important contact information.
Please assign me this if no one is working.

Hi, thanks for the proposal. I am not sure we want to do this. The website in general is pending (not started) an overhaul. So might not worth the effort of review/integration.

oh ok!
Please let me know!

I agree, we're going to get rid of this theme and site layout, so it's not worth much effort. Also, we don't really have any social media anyway - only the mailing list, which is in the README on GitHub as well as on a separate page on the website.
Thanks for the offer of help though @Uttam-Singhh! I think once we get going on the website redesign, there'll be quite a bit to contribute to.

Oh ok sure!
Please let me know if there are any more opportunities for beginners...

We're probably going to base the new site on the NumPy one (just tweak that design a bit and add new content), so if you'd want to try your hand at helping with the NumPy site, that would be great. The good-first-issue and wimlds-sprint labelled issues on https://github.com/numpy/numpy.org/issues are good starter issues.

Hello @rgommers, actually there are no wimlds-sprint issues and only 2 good first issues which are under process so please let me know if I can do something else.

@rgommers I have opened two issues on https://github.com/numpy/numpy.org/issues.
Can you please review and assign me those tasks. I'd would love to work on them.

will do, thanks @Uttam-Singhh

Thank you so much @rgommers!


 And it MUST be run from the cli because it expects to parse arguments using 'term'
It would be great if you could invoke busted inline as part of a lua file. This is useful for writing integration tests for lua code that runs on a framework. You want have access to the APIs exposed by the framework for integration tests. As a result you cant invoke via the busted CLI wrapper
MyFrameworkBustedTestRunner
-- configure the framework as needed for the tests

-- run busted tests
busted.run(busted_config_options)

please let me know if there's a way to do this. The project is composed in a way which makes it seem feasible, but i cant pinpoint a good entry point

:+1:. I'm looking to run tests as part of a larger suite of tasks inside one Lua process, and I can't figure out the correct way to do so. I think it involves duplicating most of the runner code but without the CLI wrapper, but I'm not entirely sure.

Right now there's no documented way for you to do this.
...but, if you just require busted.cor, manually set up the options you would like just like busted.runner does, and publish the appropriate events so the output handler knows what's up, it should work for your use case just fine. If you'd like to make that process generic we'd welcome a pull request!

Thanks for the direction. Much of the loader code depends on penlight LFS based apis, which may not work depending on how the lua env is hosted. Are you suggesting that we would need to implement our own loaders as well as handlers?

I'm not sure I follow, what env causes the loaders to break?

As an example, lua is embedded in something that does not expose a traditional filesystem. Or you may not be able/want to load test specs by inspecting your path.
Similar to how "package.loaders" works we want to load our test files from a different source, not the filesystem (lfs) based

Unfortunately, we have a hard requirement on LFS for this case since we need to be able to discover test files. Lua does not provide this functionality out of the box so we'd have to depend on -something- and it might as well be LFS.
The solution on this strawman system is to create an in memory traditional filesystem of some sort and write the files there, then run busted against that.

I ran into trouble with this too. The problem I ran into with this is that busted.runner gets its configuration directly from arg, and there doesn't seem to be a way to override that. I solved it by resetting arg to an empty table, which is a lame hack, but got the job done.
The busted.runner function accepts an options table, which seems to be the right thing to use if you want to customize it programmatically. However, it still checks arg even if options is provided.
I would suggest that because options implies you are doing your own customization, that it ignore arg if it is present. Getting options from the command-line should be mutually-exclusive with getting them from a function call.
However, if this suggestion is too likely to break things, perhaps there should be a way to explicitly disable checking arg using the options table?
I can submit a patch for either of these solutions if you decide one of them is acceptable.

I'd like to include parts of busted in my program to write integration tests. This program embeds a sandboxed version of Lua. I'd like to provide access to describe, it, and assert, and then run the tests. It shouldn't look for _spec files as those will be used by normal unit tests.
How easy would this be to do? Should this been done? What's the best way to do this and reduce the amount of work needed to keep up to date with busted?

I am trying to add tests with busted to https://github.com/rest-nvim/rest.nvim and I fell upon that limitation as well:
https://github.com/lunarmodules/busted/blob/fa948fd8355c598093cac35b106f2d93178118f2/busted/runner.lua#L11 accepts "options" but these are limited: https://github.com/lunarmodules/busted/blob/master/busted/options.lua .
I would like notably to be able to pass the folder where to run.
Skipping local cliArgs, err = cli:parse(arg) if options != empty or merging options with cli args doesn't seem too hard and would allow for programmatic runners.


Kivy issue when starting new project.

I got this error while trying to start a new project.
Is there any fix ?
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe9 in position 53
## Environment Info

kivy: 1.9.1
pygments: 2.1.3
docutils: 0.12
watchdog: 0.8.3
jedi: 0.9.0
gitpython: 1.0.2
six: 1.10.0
kivy-garden: 0.1.4

Platform: Windows-7-6.1.7601-SP1
Python: 2.7.2

## Traceback

Traceback (most recent call last):
  File "C:\python27\lib\site-packages\kivy\core\window\window_sdl2.py", line 619, in mainloop
    self._mainloop()
  File "C:\python27\lib\site-packages\kivy\core\window\window_sdl2.py", line 362, in _mainloop
    EventLoop.idle()
  File "C:\python27\lib\site-packages\kivy\base.py", line 327, in idle
    Clock.tick()
  File "C:\python27\lib\site-packages\kivy\clock.py", line 515, in tick
    self._process_events()
  File "C:\python27\lib\site-packages\kivy\clock.py", line 647, in _process_events
    event.tick(self._last_tick, remove)
  File "C:\python27\lib\site-packages\kivy\clock.py", line 406, in tick
    ret = callback(self._dt)
  File "C:\Users\zied\Desktop\kivy-designer-master\designer\uix\kivy_console.py", line 291, in run_command
    self._run_command_list()
  File "C:\Users\zied\Desktop\kivy-designer-master\designer\uix\kivy_console.py", line 299, in _run_command_list
    self.stdin.write(self.command_list.pop(0))
  File "C:\Users\zied\Desktop\kivy-designer-master\designer\uix\kivy_console.py", line 855, in write
    self.obj.on_enter()
  File "C:\Users\zied\Desktop\kivy-designer-master\designer\uix\kivy_console.py", line 734, in on_ent

Hi @MeDzIeDaRbI
This traceback is not yet complete. Please, check the bug reporter instructions because the last line of the traceback must have "End of Traceback".
When you face this issue again, use the "Copy to clipboard" button and add the result in a new comment so we can identify what is wrong.
Thank you

Hi,
This bug report is not complete, so I'm closing it. Also, check the master, I've fixed a similar problem, but I'm not sure if it's the problem if you are getting.
Thanks


Update sbt-scalajs, scalajs-library_2.13, ... to 1.13.1

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-library_2.13
org.scala-js:scalajs-test-bridge_2.13

from 1.9.0 to 1.13.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-js" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #100.


RFC: Merge this repo into robot-image

I would like to suggest that we merge this repo into the robot-image repo.
Pros

Easier to develop our image, we don't have to bump in two places
Easier for new volunteers to find our recipes etc.
One place for our OS build
Submodules are for external deps only (i.e poky etc.)

Cons

It's slightly more difficult for other people to use our recipes. We don't know of anybody doing this anyway and are under no obligations to support them if they want to.

@WillB97 @8BitJosh @raccube @Tyler-Ward @RealOrangeOne I'd like to hear your thoughts please!

I agree with merging these repos,  it brings all the image build bits into a single repo making it easier to manage.  I think our image is unique enough that it is unlikely that someone would want to use our recepies anyway.

@raccube has also given approval of this, and I'm interpreting Will's thumbs up as approval too, so I'm going to go ahead and do this now.


Fix selecting a frame on the timeline (with left click), and immediately right clicking it, will be taken as double click (fix #3713)

fix aseprite/aseprite#3713

Ok, I understood. So maybe we need the current and previous click to evaluate a real double click action.

Ready for review.


support hiding canvas actions Island altogether

use case: view-mode without ability to export.

rn the consumer can pass each attribute in canvasActions as false to support this, do you suggest having a boolean variable canvasActions: false for this?

rn the consumer can pass each attribute in canvasActions as false to support this

Technically true, but when you remove every button, the island still renders (~10px square with a shadow).

do you suggest having a boolean variable canvasActions: false for this?

yep


It is not working in Firefox

Hiya! I'm going to need more information than that. In particular:

What OS are you using?
What version of Firefox are you using?
What does the code that you're using the plugin with look like?
Can you confirm that that code is working in Chrome?

sure

Using Mac OS version 12.6
Firefox version 107.0
I am using React so
in tailwind.config.js

     plugins: [
    require('tailwind-scrollbar')({ nocompatible: true }),
    require('autoprefixer')
  ],

and scrollbar.tsx component
interface Props {
  children: ReactNode;
  className?: string;
}
export default function Scrollbar({ children, className }: Props) {
  return (
    <div
      className={`overflow-y-auto scrollbar-thin scrollbar-thumb-greyWhite scrollbar-thumb-rounded-full ${className}`}
    >
      {children}
    </div>
  );
}

yes in Chrome and Safari it is working perfectly

After adding overflow-y-auto it started working but the styling is different from chrome and safari

Apparently, this is a known issue/feature with Firefox on OSX. Nothing that can be done about it. I can look into adding it to the docs, though.


Add custom notification sound

Adds a blip sound file to the Resources and the iOS target.
Changes the notification code to use the sound.

Did you test the Mac version of NetNewsWire to make sure you didn't break anything there?

We also typically keep resource centrally managed in our AppAssets struct. That is probably where we should put your sound definition.

My first instinct was to put it into the asset catalogue as well, but I couldn't see how to without importing UIKit (or AppKit) and that seemed excessive (I suppose they are already involved so it wouldn't be an extra cost?.)
Also I couldn't work out how to marry UNNotificationSound (which is looking for a filename) and NSDataAsset (which provides you with an NSData object.  But maybe I missed something.

We have two different AppAsset structs, one for iOS and one for macOS. There shouldn't be a problem with importing UIKit or AppKit.
Are you thinking that you have to use an Asset Catalog? If so, don't worry about that. Just drop your file in the Shared/Resources folder.

Did you test the Mac version of NetNewsWire to make sure you didn't break anything there?

Just tested now, doesn't appear to cause any issues.  No sound though, obviously.

I think this is ready to go and can be removed from draft status. I'll merge it when you are ready.
We can always roll it back or change the sound if @brentsimmons prefers a different one.

Amazing.  I'm very pleased to had contributed (even in a minuscule way) to an app I've used daily for god knows how long... probably 20 years.
Before you merge, I just wanted to check something - in order to resolve a conflict in project.pbxproj due to main branch having moved-on I manually resolved it as the UUIDs of the added files didn't clash.
I assume that this is fine, but I don't want to break the Xcode project accidentally.
Also, should we credit the guy who made the sound?
https://freesound.org/people/cabled_mess/sounds/350862/
I plan on commenting on that page once it lands, but wondered if we should add him to credits.rtf ?

@brentsimmons See the above comment please.

Happy to merge whenever you are.

For now let's back out the credit changes. They are typically only updated by @brentsimmons and I'm not sure how he wants to handle this. Other very minor contributions like this one sometimes don't get attributed inside the app, but may get attributed in the repository somewhere instead.


rowbuffer changes causes issue in simulator tests

@dillonhuff,
Do you mind taking a look at the branch 'rowbuf'? The simulator is saying that there is an error, but it is not reporting where it is happening. I am not sure how to debug this. This is very much blocking CGRA process for Jeff.

Some more details.
On line 1133 of interpreter.cpp, in order to make my test pass I need to call exeCombinational() one additional time. This seems like incorrect behavior.

I suspect the interpreter is not updating all the signals through the combinational path of memories during exeCombinational(). Specifically " -> mem.raddr -> mem.rdata -> "

Fixing this with PR #460 For now by just simply always calling exeCombinational() twice. I am still keeping this as a bug that needs to be fixed though.


Test-Connection - Improve Logic and Output

PR Summary
Refactor Test-Connection to provide unit-based output with clearer formatting and more useful traceroute timing information.

Uses SendAsync() to send pings and adds event handler to permit Ctrl+C to cancel the cmdlet operation mid-ping (great for giving up on a long timeout).
Refactor output; both regular -Ping and -TraceRoute actions output one object per received ping, formatted to accommodate as much information as possible.
Refactor -Traceroute methodology to use 4 pings instead of 3:

1-3 initial "discovery" pings with low TTL to the final destination to find the router at that hop point (not displayed in output). First ping to come back with TtlExpired is used to target the rest of the pings.

On Unix, TtlExpired is never reported, so we skip this step completely and just target all pings at the final destination.

3 followup pings to that router directly so we get proper latency and status information from the router (all rendered in output).
This fixes an outstanding issue where traceroutes could not provide latency information on intermediate hops.

Slight refactor for MtuSize detection to allow it to work correctly on Unix.
Use new class types for all output in order to better provide easy access to displayed information.
Update ValidateSet attribute on -MaxHops to reflect underlying API limitations (0 is not a valid argument for PingOptions' TTL)
Fix long-standing issue with -Traceroute where -MaxHops is a valid input parameter, but is completely ignored. New behaviour is to write an error if the destination cannot be reached within the set value for -MaxHops, or return $false if also specifying -Quiet.
Fixed an issue where -IPv4 and -IPv6 were completely ignored if you supplied a valid address from the other family. Resolution is to check the AddressFamily, and if it doesn't match the provided switch, pull the host entry and find an address matching the requested family to use.

Added some additional IPv6 tests. After running all the IPv6 tests in CI (including those originally marked -Pending) I think it best we leave them disabled. I've verified the features work in Windows 10 and Ubuntu under WSL, but Azures CI apparently doesn't have great IPv6 support, so it's a very mixed bag on what comes back for those tests. We can reenable them in future if there are improvements or alterations to CI config that might improve their reliability there.

Renamed switches:

-Continues => -Repeat (Alias applied to maintain compatibility)
-MtuSizeDetect => -MtuSize (Alias applied here also)

PR Context
RFC: https://github.com/PowerShell/PowerShell-RFC/pull/172
Also resolves #9235
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@SteveL-MSFT Samples of output:
Default -Ping Output

-TraceRoute Output

@PoshChan please retry Linux

@vexx32, successfully started retry of PowerShell-CI-Linux

@PoshChan please retry windows

@vexx32, successfully started retry of PowerShell-CI-Windows

Have you tested this in any of the docker containers?  We had large differences in behavior in the containers.

@TravisEz13 I have not as yet, no. How would I go about doing that?

Download the build.zip from the linux CI.
extract it.
If you are on windows, make sure you have switch you docker daemon to linux, my right clicking on the icon and choosing the switch option.
run
docker run -it --rm -v /extractedZipLocation:/pwsh mcr.microsoft.com/powershell:ubuntu-18.04 /bin/sh
chmod a+x /pwsh/pwsh
/pwsh/pwsh

Then you should be in the new PowerShell in the container.

Adding Breaking Change as some public types are changed.  Although I think they fit in Bucket 3 which is unlikely to impact anyone.

@TravisEz13 thanks for that!
So, testing in the Ubuntu 18.04 docker container looks very very similar to just testing in Ubuntu under WSL. The only difference I can find is that -IPv6 functionality is simply not there, any attempt to use that switch errors out.
Given the APIs in use, I get the impression that might be simply a lack of IPv6 support in Docker? Not really sure how to properly test that, but everything else works the same way as it does under Ubuntu under WSL, and the same as how it's behaving in CI (though IPv6 support appears to be slightly better in CI than in the docker container, though even that is inconsistent at best).
All IPv4 functionality works the same way as it does in CI under Unix. Pings and MtuSize detection work pretty well, but traceroutes get a slew of TimedOut responses due to the .net core API not giving anything other than Success or TimedOut under Unix at the present time. However, the code is in place and fully-functional under Windows, so if/when the .NET Core team bring the Unix side of the Ping API up to parity in terms of possible results from a ping, the traceroute functionality should just work, or may require only minor adjustments, depending on if the response to TTL expiry is the same Status code under Unix or not.

@vexx32 The one thing is our tests need to pass when we run them in a container.  This is how we test all supported distributions when releasing.  Can you work with @JamesWTruher to make sure you understand how to, change, skip, or otherwise make it so we don't have to triage the failures when when are running in containers for release.

Happy to! The -IPv6 tests are already fairly unstable in the existing CI, which... I don't really know why that is, but they seem to work fine in Win10 and Ubuntu under WSL, so I believe the features are as sound as they can be at present. Those are already disabled under Unix.
@JamesWTruher how can I determine if pwsh is running in a docker container so I can conditionally mark the less stable ones as pending?

Here is our code to setup the environment:
   $vstsCommandString = "vso[task.setvariable variable=__INCONTAINER]1"
      Write-Verbose -Verbose "Setting __INCONTAINER as 1"
      Write-Host "##$vstsCommandString"

From https://docs.docker.com/v17.09/engine/userguide/networking/default_network/ipv6/#how-ipv6-works-on-docker:
By default, the Docker daemon configures the container network for IPv4 only. You can enable IPv4/IPv6 dualstack support by running the Docker daemon with the --ipv6 flag. Docker will set up the bridge docker0 with the IPv6 link-local address fe80::1.
So you may need to run your docker container like this:
docker run --ipv6 -it --rm -v /extractedZipLocation:/pwsh mcr.microsoft.com/powershell:ubuntu-18.04 /bin/sh

@steviecoaster This would have to be done by Azure DevOps in our case so lets continue with the existing plan.

@TravisEz13 @JamesWTruher if I remember my Azure variables correctly, the latest commit should address your concerns with the tests in the container environments.
Let me know if you need me to tweak it any further. 💖

@iSazonov Do you mind reviewing the changes?

@vexx32 Please resolve merge conflicts.

@PoshChan please retry windows

@vexx32, successfully started retry of PowerShell-CI-Windows

@SteveL-MSFT @iSazonov @TravisEz13 I checked https://github.com/dotnet/corefx/issues/28934 recently and saw they pushed it back to 5.0 for now.
Can we make a judgement call on the way forward here? This implementation, while doing more work than it should need to, is at least (in my opinion) more usable and functional on every platform than the existing implementation, and makes use of the most effective parts of the API surface currently available.
In my opinion, having this in PS7 to have a more user-friendly cmdlet would be better than waiting until .NET 5 for the API to be fixed. 🙂
I'm more than happy to do the later rewrite when the .NET Core team (finally) fix their Ping APIs. 😁

@vexx32 I don't think we should hold this waiting on .NET (should open a new issue linked to corefx, I can tag it for vNext).  I'll spend some time reviewing this today.

Went back through ping / test-connection issues and found this one:
https://github.com/PowerShell/PowerShell/issues/7576
I think this PR should address that as well 🤔

@SteveL-MSFT looks like we've been tracking this in #4240; I think we can continue to track this there. 🙂

@PoshChan please retry windows

@vexx32, successfully started retry of PowerShell-CI-Windows

Should we document anything (the check box is set)?

Yeah, we changed quite a bit in terms of the shape of the output in pretty much all the cases here.
Doc issue is https://github.com/MicrosoftDocs/PowerShell-Docs/issues/4987 - I'll link it in the PR description as well. Thanks for the reminder! 💖


VC: syncer fails to get the admin kubeconfig when cluster is creating

syncer fails to get the admin kubeconfig when cluster is creating. then syncer will miss this cluster.
TODO:  check the status of the virtual cluster and requeue the object to the work queue.

will add vc status

/assign @charleszheng44

#229


Library command doesn't give access to contents of the uploaded file within GLM script

Problem description
#library get "wood_pole.glm"

When used within GLM script the file itself doesn't get uploaded - therefore can't use the the objects inside.
Steps to reproduce
(Explain how to reproduce the behavior, e.g.,)

Run "test_pole.glm" as amended above.
test_pole.glm.zip

Expected behavior
Expect to be able to refer to configurations defined in wood_pole.glm in the slacgismo/gridlabd-library repo
System information

GridLAB-D version: HiPAS GridLAB-D 4.2.11-200923 (develop-add-server-control-stop) Darwin
Operating system: Darwin pc95555.lan 19.6.0 Darwin Kernel Version 19.6.0: Thu Jun 18 20:49:00 PDT 2020; root:xnu-6153.141.1~1/RELEASE_X86_64 x86_64

Supporting information
n/a
Attachments
See above

The problem is that #include does not automatically search the library folder and subdirectories and cannot find the downloaded library file in its subfolder within the shared library folder. We need modify #include to use GLPATH when finding its file.


Athlete merge tool / API

There's A TON of duplicate athletes. At some point we'll have to clean it up, if would be great if we had an API or a Django tool where we merge two or more athletes (literally, associating all records with one of them and deleting the others)

I'll look into this.
At present moment, we would have to go into the lifts of the duplicated athlete and then change them to one of the "real" athletes.
I'm learning lots!

For @shivan-s reference: https://docs.djangoproject.com/en/4.1/ref/contrib/postgres/search/

I'm having a lot of issues on my imports around this. I might create a little script tool to do it, and will share once I do.

No worries. I was thinking about this.
Some things to get a head start on. The athlete detail view provides a list of lifts. So if you have a duplicated athlete you could get all the lift ids from that duplicated athletes. Then edit those lifts to move them to the 'real' athlete. Then, delete the duplicated athlete.
Just note that the delete is a cascade, so if you delete the athlete before moving the lifts, the lifts get deleted!!
I'm working on a fuzzy finder so before you create an athelete, it'll scan for similiarly named athletes. Hope this helps.


Methods in BranchEditor.cs are made "public static"

Methods in BranchEditor.cs are made "public static" to allow using that methods in custom post process build tool

@antonargunov Hold on another update for Sojan to send you a new jar. We discovered a bug with the chrome tabs implementation that can cause NPE on Android that he fixed last night. cc @sojanpr


CC0

switch to CC0 at some point. it's apparently better than Unlicense for locations that don't recognize the public domain.

done


Feature/add workflows

This pull request adds a configuration to use GitHub Actions/Workflows.
There were lint errors causing the build to fail previously.

😅 curse defaulting fork merges


View Authorization Groups

From release plan spreadsheet
Estimate from release plan: 5 hours
Expressing interest: none

Create/modify a service to be able to retrieve the groups for the current user
List them on the profile page

as we are working on other aspect of the group management it would be convenient for us to work on that in the coming weeks. Can we get assigned of this task?

Closing as this was already implemented in 7.0. See a logged in user's profile page.


Update EnvironmentCredential and EnvironmentVariables

Add the AZURE_CLIENT_CERTIFICATE_PATH to EnvironmentVariables, and update the  EnvironmentCredential to support AZURE_CLIENT_CERTIFICATE_PATH

I think this was covered as part of https://github.com/Azure/azure-sdk-for-net/pull/10196


artificial-intelligence

make-retina-avatars
create a program from a 4-dimension
can be from a 5-dimension if possible
from all 10 dimensions!

Hello-World

Hello-World

https://patch-diff.githubusercontent.com/raw/Robertino10/artificial-intelligence/pull/8.patch


Loading custom objects?

I have an object .stl file which I have build using a 3D CAD software. Can that be imported into the scene?
Or is there any other library where such objects can be imported and rendered.
Thanks.

I have implemented a class named  TriangleMesh  to load arbitrary .obj files as triangle meshes. So, first convert your .stl file to a .obj format.
However, the rendering is quite slow because we need to loop for every triangle of the mesh, and Python loops are very slow. A bounding volume hierarchy acceleration structure could be implemented, but I think renders will still take a lot of time.
If you want to render arbitrary triangle meshes, Python isn't the language to work with. An implementation in a compiled language like C++ should be needed for speed.


New package: ShortFFTs v1.0.0

Registering package: ShortFFTs
Repository: https://github.com/eschnett/ShortFFTs.jl
Created by: @eschnett
Version: v1.0.0
Commit: 6c41e1ddefc5a5a0582756f708e4134ac3607474
Git reference: HEAD
Description: Efficient short Fast Fourier Transforms

Thank you for submitting your package! However, please make sure to add some documentation before registering. At the very least, that would be a description of the package's purpose and a small usage example in the README


Setup Windows binary script page contains an invalid link.

This page contains the wrong link.
Incorrect link is the following URL set in "Blog" of "If you want to use Python Tools, CNTK, and Visual Studio Code, see this blog."
The wrong address is below url.
http://dacrook.com/cntk-vs-code-awesome/
I think that a malicious site is set up in this link destination, is it the correct link?

It was working some time ago. You can find a cached version here: http://webcache.googleusercontent.com/search?q=cache:xtZcLiXzJ34J:dacrook.com/cntk-vs-code-awesome/+&cd=1&hl=en&ct=clnk&gl=us
I will monitor it for some time and remove it if it doesn't return online soon

This has been fixed. I've talked to the author, which provided another URL for the blog entry. Enjoy it


Calculate Diagnostic Features at Every Diagnostic Cycle

Currently, diagnostic features are calculated at the first and second diagnostic and the difference between the two is reported. This format is amenable to early prediction, but analysis of the battery health across lifetime would be better served by having the features reported across every cycle.
Some diagnostic information is available in the structured data in the diagnostic_summary field, but what's available there is not comprehensive.
I propose making a distinction between 1) "EarlyFeatures", which are a difference on the first two diagnostics (or between the first and some other diagnostic number specified in the featurizer hyperparameters) and 2) "CycleFeatures", which are the diagnostic features reported across all diagnostic cycles.

@ardunn would you help write unit tests for issue please? Thank you!

Sure.
Since this pertains directly to the PR, I'm just going to copy+paste @pasinger 's text there and close this issue, as everything in this issue will be in the PR


Completely delete the build dir

Otherwise it shows a message like this:
(node:22567) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1):
Error: Cannot delete files/folders outside the current working directory.
Can be overriden with the `force` option.

Good catch 👍

Could/should this go in server/build/clean.js as well? I'm trying to set up next in a separate directory (basically treating the frontend as a library...) and running into this.


Please add support for all options on RecognitionConfig

Copied from https://github.com/googleapis/google-api-php-client-services/issues/173

When you are creating the RecognitionConfig object there are no methods to set the following properties which could improve our transcription results.
RecognitionMetadata
SpeakerDiarizationConfig
SpeechContext

@paulsheldrake, are you using the latest version of google/cloud-speech? RecognitionConfig is accepted as an argument to recognize() and longRunningRecognize() and allows use of RecognitionMetadata by calling RecognitionConfig::setMetadata(), and SpeechContext by calling setSpeechContexts().
I do not see any definition in the API for SpeakerDiarizationConfig. Can you provide additional information?

Hello, thanks for getting back to me.   I am using the latest version of the library "google/cloud": "^0.105.0"
For the SpeakerDiarizationConfig question I've realized I was looking at the wrong documentation.  It's an option in the v1p1beta1 version.  Sorry about that.
https://cloud.google.com/speech-to-text/docs/reference/rest/v1p1beta1/RecognitionConfig#SpeakerDiarizationConfig
Do you have an example somewhere of RecognitionConfig::setMetadata().

Sure, try something like this:
use Google\Cloud\Speech\V1\RecognitionAudio;
use Google\Cloud\Speech\V1\RecognitionConfig;
use Google\Cloud\Speech\V1\RecognitionMetadata;
use Google\Cloud\Speech\V1\SpeechClient;

# get contents of a file into a string
$content = file_get_contents($audioFile);

$audio = new RecognitionAudio();
$audio->setContent($content);

$metadata = new RecognitionMetadata;

// Provide to the constructor
$config = new RecognitionConfig([
    'metadata' => $metadata
]);

// Or using the setter
$config->setMetadata($metadata);

$client = new SpeechClient();
$response = $client->recognize($config, $audio);

This is great, thank you!

Despite being documented, the SpeakerDiarizationConfig type is not yet available in the API definitions. We'll ask the API team about this, and I expect that it will become available before too long.

@paulsheldrake, FYI, #2126 was just merged, adding support for SpeakerDiarizationConfig. It'll be included in the next release. :)

Thanks for the heads up!

Hello,
I am trying to add the SpeakerDiarizationConfig to the RecognitionConfig but I am not able to do that and I don't see any example on the documentation page in order to make it work. Could you please help me to achieve this? My approach is the following:
diarization_config = {
"enableSpeakerDiarization": True,
"minSpeakerCount": 2,
"maxSpeakerCount": 3}
config = types.RecognitionConfig(
encoding=enums.RecognitionConfig.AudioEncoding.LINEAR16,
sample_rate_hertz=frame_rate,
language_code="es-ES",
enable_word_time_offsets=True,
diarization_config=diarization_config,
enable_automatic_punctuation=True)
Thanks in advance!


Electric Furnace unable to process anything

We have tried at least 3 different recipes in the electric furnace (as listed in NEI), none seems to work, it accepts power however.
Version Number: 1.7e

Thanks for spotting that - a silly typo from me has caused it - fixed :)


first draft of a map<> explainer

Half of https://github.com/common-workflow-language/common-workflow-language/issues/529

The language is a little breezy, but that's just a matter of personal preference
Should mention the following pattern as a variation of "option two":

requirements:
  InlineJavascriptRequirement: {}

"hyperlink all usage of map<> to this explanation" would require schema salad support, since the "map<>" is generated?

The language is a little breezy, but that's just a matter of personal preference

Yes, it is purposely more in the style of the tutorial. We can mark it non-normative to be clear.

Should mention the following pattern as a variation of "option two":

Good point! I'll add this tomorrow

@tetron Done, ready for review & merge


refactor(llm): Tag configuration to be confirmed and collated

Split settings by type

hugegraph-llm/src/hugegraph_llm/config/config_prompt.yaml needs a liscense header

hugegraph-llm/src/hugegraph_llm/config/config_prompt.yaml needs a liscense header

Roger that! I'll fix it right away!

hugegraph-llm/src/hugegraph_llm/config/config_prompt.yaml needs a liscense header

liscense header

Finished adding the apache liscense header, but we haven't split the data and operation yet. I plan to continue splitting later after lunch🫡


Script that calls OptionButton add_separator() with no arguments crashes Godot on POT generation

Tested versions

Found on v4.2.1.stable.official [b09f793f5]

System information
Godot v4.2.1.stable - Windows 10.0.19045 - GLES3 (Compatibility) - NVIDIA GeForce GTX 1650 (NVIDIA; <IP_ADDRESS>23) - AMD Ryzen 5 3550H with Radeon Vega Mobile Gfx (8 Threads)
Issue description
A script that calls OptionButton add_separator() with no arguments crashes Godot on POT generation with the following error message:
ERROR: FATAL: Index p_index = 0 is out of bounds (size() = 0).
   at: get (./core/templates/cowdata.h:158)

Using add_separator("") works fine, making me think that there is something in the code that expects there to be a string even when it is not a required argument. The problem may thus be more widespread and actually affect all functions that expect a string as an argument but don't outright require it.
Steps to reproduce

Add the following script to your Project Settings > Localization > POT Generation > Files with translation strings:

extends OptionButton

func _ready():
	add_separator()

Generate POT
Crash

Minimal reproduction project (MRP)
POTGenCrash.zip
Generating POT with Crash.gd included will crash Godot.
Generating POT with only NoCrash.gd will not crash Godot.

Not reproducible in 4.3 dev, seems to be fixed by #86091 or #86471.
Thanks for reporting the bug nonetheless!


NetInfo: effectiveType only changes if type changes

Environment
System:
  OS: Windows 10
  CPU: x64 Intel(R) Core(TM) i7-6500U CPU @ 2.50GHz
  Memory: 1.14 GB / 7.89 GB
Binaries:
  Yarn: 1.10.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
  npm: 5.6.0 - C:\Program Files\nodejs\npm.CMD
IDEs:
  Android Studio: Version  <IP_ADDRESS> AI-173.4907809

React Native: v0.57.1
Description
Both NetInfo.getConnectionInfo() and the event listener that I added via NetInfo.addEventListener('connectionChange') don't seem to pick up changes of the network connection if the type stays the same and only the effectiveType changes.
In other words:
When I monitor the Android Emulator with cellular connection and the connection changes from 4g to 3g, the change is not reflected in the effectiveType. If I change to wifi and back to cellular, the effectiveType is adapted accordingly.
Reproducible Demo

Start App on Android Emulator and deactivate wifi
Call NetInfo.getConnectionInfo() - it will correctly return the current effectiveType (e.g. '4g')
Change the network type in the Android Emulator (e.g. to "EDGE")
Call NetInfo.getConnectionInfo() - it will still return the previous effectiveType ('4g')
Activate wifi on the emulator
Deactivate wifi again
Call NetInfo.getConnectionInfo() - it will now return the correct effectiveType ('2g')

Same problem here with the following result to "react-native info":
Environment:
OS: macOS High Sierra 10.13.6
Node: 10.5.0
Yarn: Not Found
npm: 6.1.0
Watchman: 4.9.0
Xcode: Xcode 10.0 Build version 10A255
Android Studio: 3.2 AI-181.55<IP_ADDRESS>56338
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: https://github.com/expo/react-native/archive/sdk-27.0.0.tar.gz => 0.55.2

same issue here, its only detecting changes between wifi <--> cellular , but Cellular: 4g <--> 2g it doesnt trigger 'connectionChange' listner


Update 07_saddle_bracket.py

Codecov Report

Merging #614 (502d3ee) into main (8bc3246) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #614   +/-   ##
=======================================
  Coverage   76.08%   76.08%
=======================================
  Files          32       32
  Lines        2271     2271
=======================================
  Hits         1728     1728
  Misses        543      543


FT-2563 - FT-2563-no-daos-hash

Summary
Older Kong version accepted hash-like tables for DAO definitions.  This is no longer supported, and with this change, a clear error message is produced if a plugin returns a non-array-like table for its DAO definitions.

 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.


Flags with multiple=True trigger TypeError

Adding an option with is_flag=True, is_eager=True, and multiple=True will trigger a TypeError in Parameter.consume_value() due to bad default value.
import click

def version_handler(ctx, param, value):
    if value:
        print("version_handler", ctx, param, value)
        ctx.exit()

@click.command()
@click.pass_context
@click.option(
    "--version",
    "-V",
    callback=version_handler,
    expose_value=False,
    is_eager=True,
    is_flag=True,
    multiple=True,
)
def main(ctx):
    pass

if __name__ == "__main__":
    main()

When run without the flag, a TypeError is raised:
(venv39) jreese@butterfree ~/scratch » python foo.py
Traceback (most recent call last):
  File "/Users/jreese/scratch/foo.py", line 25, in <module>
    main()
  File "/Users/jreese/scratch/venv39/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/jreese/scratch/venv39/lib/python3.9/site-packages/click/core.py", line 1054, in main
    with self.make_context(prog_name, args, **extra) as ctx:
  File "/Users/jreese/scratch/venv39/lib/python3.9/site-packages/click/core.py", line 920, in make_context
    self.parse_args(ctx, args)
  File "/Users/jreese/scratch/venv39/lib/python3.9/site-packages/click/core.py", line 1378, in parse_args
    value, args = param.handle_parse_result(ctx, opts, args)
  File "/Users/jreese/scratch/venv39/lib/python3.9/site-packages/click/core.py", line 2356, in handle_parse_result
    value, source = self.consume_value(ctx, opts)
  File "/Users/jreese/scratch/venv39/lib/python3.9/site-packages/click/core.py", line 2903, in consume_value
    and any(v is _flag_needs_value for v in value)
TypeError: 'bool' object is not iterable

Manually setting a better default value, like default=(), avoids the type error, but this used to work on 7.x without setting the default value.
Environment:

Python version: tested on 3.8.6 and 3.9.3
Click version: worked on 7.1.2; errors on 8.0.4 and 8.1.2

I can't remember what the original discussion about this was, but it seems like this should have been addressed by modifying the default default for multiple=True, flag=True rather than raising an error. The following works fine, the default=(False,) should be implicit.
import click

@click.command
@click.option(
    "-y/-n",
    is_flag=True,
    multiple=True,
    default=(False,)
)
def main(y):
    click.echo(y)

if __name__ == "__main__":
    main([])

Issues #2292 and #2295 are due to the error added by the PR for this issue.


ci: update checked rust-toolchain version

i'm mostly just trying to see a fresh CI run to compare for PR 26

still seeing an error printed during rust install
Run Swatinem/rust-cache@v2
Error: The process '/home/runner/.cargo/bin/cargo' failed with exit code 101
    at ExecState._setResult (/home/runner/work/_actions/Swatinem/rust-cache/v2/dist/restore/index.js:7784:25)
    at ExecState.CheckComplete (/home/runner/work/_actions/Swatinem/rust-cache/v2/dist/restore/index.js:7767:18)
    at ChildProcess.<anonymous> (/home/runner/work/_actions/Swatinem/rust-cache/v2/dist/restore/index.js:7661:27)
    at ChildProcess.emit (node:events:514:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5) {
  commandFailed: {
    command: 'cargo metadata --all-features --format-version 1 --no-deps',
    stderr: '\x1B[1m\x1B[31merror\x1B[0m\x1B[1m:\x1B[0m could not find `Cargo.toml` in `/home/runner/work/setup-rust-toolchain/setup-rust-toolchain` or any parent directory\n'
  }
}

oh i guess that makes sense for this repo

@jonasbb i was so lost wondering where $toolchain came from when i realized that env: block was below the script so i've moved stuff around a bit to make things more clear


Check incoming traces with isRemote and implement logic

Handle spans that have isRemote set to True.

span processor (potel_span_processor.py)
Also check for isRemote when assembling the transaction to be sent to Sentry.
propagator
In the extract method: create new OTel context with a virtual remote span set as the active span containing the data of the incoming trace information.

done in https://github.com/getsentry/sentry-python/issues/3330