[FLINK-6302] Documentation build error on ruby 2.4

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Would be great if someone else could confirm that this works. Ideally, we get a +1 from a Linux user, a MacOS X user, and a Cygwin user...

I hit the same problem on my Mac. Let me try it out later today.

+1 (non-binding)
Tested on OSX Sierra. The documentation builds successfully.

I can confirm that you can build the docs on W10 with the Linux Subsystem.

I am not familiar with the Ruby / Bundler / etc space, but it looks like Gemfile is the original source of truth and Gemfile.lock is in some way derived from that.
Is it correct to only update Gemfile.lock and not Gemfile?
@uce or @alpinegizmo can you maybe shed some light here?

The GemFile should be updated as well. It would still work as long as the GemFile.lock file is present. If it deleted however the bundler would load the dependencies from GemFile and create a new GemFile.lock.

I prefer a conservative approach for as long as we still have to support building the docs on Ruby 1.9, so I'd keep the number of updated gems as small as possible.
As for adding support for Ruby 2.4, both docs/Gemfile and docs/ruby2/Gemfile should be updated, along with their corresponding (and derived) Gemfile.lock files. Moreover, the octokit downgrade is not needed.
I'm not sure what motivates some of the proposed gem upgrades. I was able to get the docs to build on ruby 2.4 by making changing both Gemfiles as shown below. This leaves the classifier-reborn, ffi, libv8, posix-spawn, rb-inotify, redcarpet, and sass gems as they are rather than upgrading them. Do we need newer versions of any of those?
-gem 'json'
+gem 'json', '2.0.4'
+gem 'yajl-ruby', '1.2.2'
+gem 'jekyll-coffeescript', '1.0.2'
With these upgrades, the build script still works for me on Ruby 1.9.3 and 2.3.3, as well as 2.4.0.

@mtunique What do you think about @alpinegizmo 's suggestion?

@StephanEwen I think it is the best way to solve this issue. I will follow the suggestion.

This looks good, thank you. However, I now understand what was going on with the downgrade of the octokit dependency. I think we should first merge https://github.com/apache/flink/pull/4043 and then rebase this PR. @zentol @StephanEwen

@alpinegizmo @zentol Now that #4043 is merged, can we rebase this one?

Yes, let's get this done.
On Jun 6, 2017 19:03, "Stephan Ewen"<EMAIL_ADDRESS>wrote:

@alpinegizmo https://github.com/alpinegizmo @zentol
https://github.com/zentol Now that #4043
https://github.com/apache/flink/pull/4043 is merged, can we rebase this
one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/flink/pull/3720#issuecomment-306551741, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACqWFH4ZhNVBH2qCD6bsOrQaukYd3Dfks5sBYZKgaJpZM4M8pJW
.

merging.


select current line keyboard shortcut gone

VSCode Version: 1.32.0 (1.32.0)
OS Version: 10.13.6 (High Sierra)

I constantly use cmd-I to select the current line and it suddenly stopped working today after the latest release. Please put it back!  I don't see an option to add it in the Keyboard Shortcuts either.  It's even on the Keyboard shortcuts PDF (https://code.visualstudio.com/shortcuts/keyboard-shortcuts-macos.pdf).

It's Cmd+L now. I was pretty upset about this as well. A notice would've been nice.
https://code.visualstudio.com/updates/v1_32#_default-keybinding-change-for-expandlineselection

@ewolfe I'm using Windows.  The Ctrl+L can't trigger expandLineSelection.  If I change keybinding to Ctrl+I then it works.  Do you know why?

Neither Cmd+I or Cmd+L works for me

@doggy8088 I don't know. I haven't used Windows in quite some time. You might have something else already mapped to Ctrl L - either at a higher system level or within VSCode itself. I would recommend searching for Ctrl L in your keyboard shortcut preferences (in VS Code).

I restarted my laptop and VSCode. It works now. Don't know why. 😝

@doggy8088 Can you please check what you have bound to Ctrl+L?

@alexandrudima I do have a existing Ctrl+L bound to a command, but the when are limited to markdown language only.  So this could possibly a bug in VSCode somewhere.  No matter how, my VSCode is ok now. I don't know why.

Same issue.  Please put it back.  You shouldn't just replace/remove a widely used keybinding...

I don't think cmd + L can perform the same action as cmd + I

When I use "ctrl + l" I get this message:

However in my key bindings the only shortcut for "ctrl + l" is expand line selection

Quite an annoying bug!

@HughGrovesArup YES, this message is what I was facing few days ago. (Ctrl+L) was pressed. Waiting for second key of chord....

I also had the same issue as @HughGrovesArup and @doggy8088 with the Default keybinding.
When I set the keybinding to CTRL+I, it works (as a User binding).
Then, setting it back to CTRL+L, it continues to work (as a User binding).

It is not really helpful to keep this issue open, as I plan no further changes in this area.
TL;DR Starting with 1.32, expandLineSelection is now bound to Ctrl+L and no longer to Ctrl+I. Out of the box, there is no conflict and it works under Ctrl+L.
There is an extension out there (perhaps a keymap extension, I don't know exactly which one) that binds show output to a chord which starts with Ctrl+L. If you have that extension, please let me know which one it is, so I can reach out to the extension author.
To unblock yourself, you can edit the keybindings and search for expandLineSelection and bind it back to Ctrl+I.


Option to exclude files from publishing (or to include specific files)

In Flowershow we used to have contentExclude and contentInclude config options allowing to either exclude specific file paths from publishing or to publish only the ones directly included. Also, single files can be excluded by setting isDraft: true in frontmatter. We want the same behaviour in DataHub Cloud.
Acceptance

[x] contentExclude config option - an array of file paths or directory paths (just regular path strings; no regex or glob patterns for now)
[x] contentInclude config option - an array of file paths or directory paths (just regular path strings; no regex or glob patterns for now)
[x] both can be used simultaneously, e.g. you can set contentInclude: ["/blog"] and contentExclude: ["/blog/archived"]
[x] isDraft frontmatter option

Notes

[ ] File path should include extension .md
[ ] Should return 404 if accessed via url

@olayway agree?

Yes, that's right.

@willy1989cv Now I think about it, let's just add support for excluding files/dirs. We already have an option to publish from a subfolder and it should be sufficient for now to explicitly "include" only a subset of files.

FIXED


How to use DataModule and StructResult?

❓ Questions and Help
I see newest has these two  features, and it seems very good.  Bug I can't find docs about how to use them. Can you give some examples? thanks.

the docs is coming soon... :rabbit:
cc: @nateraw @williamFalcon

Ok, v1 of the docs are live :)
The quick guide:
https://pytorch-lightning.readthedocs.io/en/latest/new-project.html#step-1-build-lightningmodule
https://pytorch-lightning.readthedocs.io/en/latest/new-project.html#data
The detailed docs for datamodule:
https://pytorch-lightning.readthedocs.io/en/latest/datamodules.html


21 | Merge Two Sorted Lists | C++

Title - Merge Two Sorted Lists
What will change - C++ solution will be added for the problem
Type of Issue -

[x] Adding New Code

Programming Language

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

@Yashica7 @SSKale1 Could you kindly assign me this problem?

Please assign it to me.

Can you please assign this problem to me?

issue already present #542

could you please assign me this problem?


Refactor: Http.Status names to Camel Case

Codecov Report

Merging #1129 (4f33982) into main (94be6b9) will decrease coverage by 0.04%.
The diff coverage is 96.80%.

@@            Coverage Diff             @@
##             main    #1129      +/-   ##
==========================================
- Coverage   57.76%   57.71%   -0.05%
==========================================
  Files          73       73
  Lines        2415     2417       +2
  Branches       77       80       +3
==========================================
  Hits         1395     1395
- Misses       1020     1022       +2

Impacted Files
Coverage Δ

zio-http/src/main/scala/zhttp/http/HttpError.scala
50.00% <ø> (ø)

...io-http/src/main/scala/zhttp/service/Handler.scala
75.00% <ø> (ø)

zio-http/src/main/scala/zhttp/http/Http.scala
65.38% <33.33%> (ø)

zio-http/src/main/scala/zhttp/http/Status.scala
93.49% <98.24%> (-1.55%)
:arrow_down:

zio-http/src/main/scala/zhttp/http/Response.scala
85.50% <100.00%> (ø)

...tp/src/main/scala/zhttp/http/middleware/Auth.scala
72.72% <100.00%> (ø)

...tp/src/main/scala/zhttp/http/middleware/Cors.scala
80.00% <100.00%> (ø)

...tp/src/main/scala/zhttp/http/middleware/Csrf.scala
80.00% <100.00%> (ø)

...ttp/src/main/scala/zhttp/http/middleware/Web.scala
78.78% <100.00%> (ø)

.../scala/zhttp/service/server/WebSocketUpgrade.scala
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 94be6b9...4f33982. Read the comment docs.


图表的Y轴单位，可以设置html么，目前只能接收文本字符串

如题

if you expect get style by html,try rich

you also can set yLabel opacity to 0,then use echarts.getModel.getComponent('yAxis') and echarts.getViewOfComponent($model) get these label's position, draw them by html

#16916


[BUG] Server crash on launch

Version of Advanced Rocketry

1.12.2-1.6.0-160

Version of LibVulpes

1.12.2-0.4.1-60
I have tried the latest version of LibVulpes+AdvRocketry as well.

Have you verified this is an issue in the latest unstable build
I have verified this is an issue in the latest unstable build.
Version of Minecraft
1.12.2
Does this occur without other mods installed
I've pinpointed that it is AdvancedRocketry causing the server crash (and not LibVulpes).
For a non-custom modpack like All The Mods 3, it launches just fine.
Crash report or log or visualVM (if applicable)

https://pastebin.com/Yssd5x4k

Description of the problem
I've only managed to reproduce this issue by starting the server with AdvancedRocketry so far. The client is fine and unaffected, so unlike a previous issue I've seen it should not necessarily be OpenTerrainGenerator at fault. (#1030)
I'm personally completely mystified why this would be fine on client but crashes the server, hence I tried replicating the issue again with the same version All The Mods 3 uses.
With that in mind, it's probably a mod conflict. (I'm so sorry I don't have further pointers.)

Are you running Advanced Rocketry on a Sponge server? If so, that would likely be why.

Nope, running it on the latest Forge version server.
Op zo 6 okt. 2019 13:23 schreef Silfryi Kalsandryn<EMAIL_ADDRESS>
:

Are you running Advanced Rocketry on a Sponge server? If so, that would
likely be why.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1713?email_source=notifications&email_token=ANM5RPE5PBIGRZOO3NF7D2TQNHDE3A5CNFSM4I53K3B2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAOHXJA#issuecomment-538737572,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANM5RPABIQLA3EYR7RKXPPDQNHDE3ANCNFSM4I53K3BQ
.

Try to delete JustEnoughIDs?

Hey, sorry for the late response.
I've made a new modpack without JustEnoughIDs and most of the same mods and it seems to have fixed the issue. My only issue is AdvRocketry saving biomes for world 8, 29 and 70.
You're free to close this issue as I figure out what I'm doing!


Creating VPN using shadow socks

Hi,
I stumbled upon the wikis but could not find how to connect to shadow socks. There are options to enable internal shadow socks but could not find one to create vpn using shadow socks.
I would like to create a vpn using shadow socks from pproxy.
Is it possible using custom.conf.
Since plain socks5 proxy to create vpn is not supported, I thought of creating shadow socks proxy using plain socks proxy and then creating docker vpn using gluetun.
Thank you

Shadowsocks or socks5 is not a vpn, it's a proxy. Note shadowsocks is NOT a socks5 proxy, it is implemented similarly but has an encryption layer on top given a pre-shared password/key. So it won't work with socks5 client at all, since it's still an entirely different protocol.
There is a built-in shadowsocks proxy server in gluetun you can enable, see https://github.com/qdm12/gluetun-wiki/blob/main/setup/connect-a-lan-device-to-gluetun.md#shadowsocks-proxy
This should clear any confusion and instructions are well detailed on how to use shadowsocks.


how to send and receive ipv6 packet throw thread protocol with nrf52840

I'm working on a project with nrf52840 and i'm trying to send and receive ipv6 packet throw thread protocol so i tried to use to send with otIp6Send and receive with otIp6SetReceiveCallback but code doesn't see the function that i pass to otIp6SetReceiveCallback so i don't can't receive from transmitter.
i also tried to handle testing example(packet_error_rate) in nRF5SDKforThreadv 0.11 sdk which sends udp packets  so i can remove udp layer  and work with ip6 functions of openthread but i stuck in the same problem.
questions :
are there any clear steps to send and receive ip6 packets over thread and what needs to be initialized ?
any clear example that sends and receive ip6 packets over thread ?

Can you clarify what you mean by: "code doesn't see the function that i pass to otIp6SetReceiveCallback"?

i'm working on thread_simple_coap example from nordic with some changes
in server side : we added case for button 2 to set new receiving callback function when receiving  an ipv6 packet. and at this function we print received packet.
in client side : we added case for button 2 to send ipv6 with otIp6Send function after we getting addresses from provisioning_response_handler function and construct header as mentioned before
main at server side (receiver)
/**
 * Copyright (c) 2017 - 2017, Nordic Semiconductor ASA
 *
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without modification,
 * are permitted provided that the following conditions are met:
 *
 * 1. Redistributions of source code must retain the above copyright notice, this
 *    list of conditions and the following disclaimer.
 *
 * 2. Redistributions in binary form, except as embedded into a Nordic
 *    Semiconductor ASA integrated circuit in a product or a software update for
 *    such product, must reproduce the above copyright notice, this list of
 *    conditions and the following disclaimer in the documentation and/or other
 *    materials provided with the distribution.
 *
 * 3. Neither the name of Nordic Semiconductor ASA nor the names of its
 *    contributors may be used to endorse or promote products derived from this
 *    software without specific prior written permission.
 *
 * 4. This software, with or without modification, must only be used with a
 *    Nordic Semiconductor ASA integrated circuit.
 *
 * 5. Any software provided in binary form under this license must not be reverse
 *    engineered, decompiled, modified and/or disassembled.
 *
 * THIS SOFTWARE IS PROVIDED BY NORDIC SEMICONDUCTOR ASA "AS IS" AND ANY EXPRESS
 * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
 * OF MERCHANTABILITY, NONINFRINGEMENT, AND FITNESS FOR A PARTICULAR PURPOSE ARE
 * DISCLAIMED. IN NO EVENT SHALL NORDIC SEMICONDUCTOR ASA OR CONTRIBUTORS BE
 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
 * GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
 * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 */
/** @file
 *
 * @defgroup simple_coap_server_example_main main.c
 * @{
 * @ingroup simple_coap_server_example_example
 * @brief Simple CoAP Server Example Application main file.
 *
 * @details This example demonstrates a CoAP server application that provides resources to control BSP_LED_3
 *          via CoAP messages. It can be controlled by a board with related Simple CoAP Server application.
 */

#include <stdint.h>

#include "app_timer.h"
#include "bsp_thread.h"
#include "nrf_log_ctrl.h"
#include "nrf_log.h"
#include "nrf_log_default_backends.h"

#include "thread_coap_utils.h"
#include "thread_utils.h"

#include <openthread/openthread.h>

void ot_receive_handler(otMessage *aMessage, void *aContext)
{

      NRF_LOG_INFO("packet received");
}

void change_rec(void)
{

        otIp6SetReceiveFilterEnabled(thread_ot_instance_get(), true);
	otIp6SetReceiveCallback(thread_ot_instance_get(),ot_receive_handler, NULL);
}

/***************************************************************************************************
 * @section Buttons
 **************************************************************************************************/

static void bsp_event_handler(bsp_event_t event)
{
    switch(event)
    {

         case BSP_EVENT_KEY_2:
            change_rec();
            break;

        case BSP_EVENT_KEY_3:
            thread_coap_utils_provisioning_enable(true);
            break;

        default:
            return;
    }
}

/***************************************************************************************************
 * @section Callbacks
 **************************************************************************************************/

static void thread_state_changed_callback(uint32_t flags, void * p_context)
{
    if (flags & OT_CHANGED_THREAD_ROLE)
    {
        switch (otThreadGetDeviceRole(p_context))
        {
            case OT_DEVICE_ROLE_CHILD:
            case OT_DEVICE_ROLE_ROUTER:
            case OT_DEVICE_ROLE_LEADER:
                break;

            case OT_DEVICE_ROLE_DISABLED:
            case OT_DEVICE_ROLE_DETACHED:
            default:
                thread_coap_utils_provisioning_enable(false);
                break;
        }
    }
    NRF_LOG_INFO("State changed! Flags: 0x%08x Current role: %d\r\n",
                 flags,
                 otThreadGetDeviceRole(p_context));

}

/***************************************************************************************************
 * @section Initialization
 **************************************************************************************************/

/**@brief Function for initializing the Application Timer Module.
 */
static void timer_init(void)
{
    uint32_t error_code = app_timer_init();
    APP_ERROR_CHECK(error_code);
}

/**@brief Function for initializing the nrf log module.
 */
static void log_init(void)
{
    ret_code_t err_code = NRF_LOG_INIT(NULL);
    APP_ERROR_CHECK(err_code);

    NRF_LOG_DEFAULT_BACKENDS_INIT();
}

/**@brief Function for initializing the Thread Board Support Package.
 */
static void thread_bsp_init(void)
{
    uint32_t error_code = bsp_init(BSP_INIT_LED | BSP_INIT_BUTTONS, bsp_event_handler);
    APP_ERROR_CHECK(error_code);

    error_code = bsp_thread_init(thread_ot_instance_get());
    APP_ERROR_CHECK(error_code);
}

/**@brief Function for initializing the Thread Stack.
 */
static void thread_instance_init(void)
{
    thread_configuration_t thread_configuration =
    {
        .role                  = RX_ON_WHEN_IDLE,
        .autocommissioning     = true,
        .poll_period           = 2500,
        .default_child_timeout = 10,
    };

    thread_init(&thread_configuration);
    thread_cli_init();
    thread_state_changed_callback_set(thread_state_changed_callback);
}

/**@brief Function for initializing the Constrained Application Protocol Module.
 */
static void thread_coap_init(void)
{
    thread_coap_configuration_t thread_coap_configuration =
    {
        .coap_server_enabled               = true,
        .coap_client_enabled               = false,
        .coap_cloud_enabled                = false,
        .configurable_led_blinking_enabled = false,
    };

    thread_coap_utils_init(&thread_coap_configuration);
}

/***************************************************************************************************
 * @section Main
 **************************************************************************************************/

int main(int argc, char * argv[])
{

    log_init();
    timer_init();

    // Initialize Thread CoAP utils.
    thread_coap_utils_led_timer_init();
    thread_coap_utils_provisioning_timer_init();
    thread_coap_utils_led_timer_start();

    // Initialize the Thread stack.
    thread_instance_init();
    thread_coap_init();
    thread_bsp_init();

    while (true)
    {
        thread_process();
    if (NRF_LOG_PROCESS() == false)
        {
            thread_sleep();

        }
    }
}

/**
 *@}
 **/

I believe this is due to your IPv6 packet not being formatted properly at the sender, as mentioned in #3600.

I believe this is due to your IPv6 packet not being formatted properly at the sender, as mentioned in #3600.

this the address i'm using now
header[2]=(0xADDEFD);
header[3]=(0xEFBE);
header[4]=(0xFF000000);
header[5]=(0x6400FE);
and that's the addresses i'm getting from ipaddr command
fdde:ad00:beef:0:0:ff:fe00:fc00
fdde:ad00:beef:0:0:ff:fe00:6400 (i'm using that address)
fe80:0:0:0:e036:1ed7:3316:76dd
fdde:ad00:beef:0:660b:9537:5373:7a2e

this's the packet i'm trying to send
start of header as : version,traffic class,flow label,payload length,next header,hop limit
source address : fdde:ad00:beef:0:0:ff:fe00:6400
destination address : FF030000 : : 1
data : aData [20]={1,0,2,3,1,0,2,3,1,0,2,3,1,0,2,3,1,0,2,3};

Duplicate of #3600.


Reduce Method Call sampling rate

What does this PR do?
Reduce the Method Call sampling rate from 5% to 0.1%
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@jonathanmos this should target release branch if we are going to do a hotfix


channel parallelism

Codecov Report

Merging #90 (8e666e0) into main (99d1354) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff          @@
##            main     #90   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files         14      14
  Lines        358     367    +9
=====================================
- Misses       358     367    +9

Impacted Files
Coverage Δ

src/gram_matrix.jl
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Brian/rebalancing refactor propose

Realized I just plopped the StandardProposeLibrary in the tokens folder. Which maybe it belongs there but in it's own folder? Where do y'all think it should go?


Use 'override' whenever possible instead of 'virtual'

Description
Pure refactoring change. Mostly automated with clang-tidy, but also some manual work to fix-up some comments and some places where clang-tidy failed.
Context: http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines.html#c128-virtual-functions-should-specify-exactly-one-of-virtual-override-or-final
Tasks

[ ] Tested on Linux
[x] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
CI. I've also ran some of the examples and everything was fine.

@Bromeon: wanna take a look at this if you have some time? Should be fairly straightforward

Looks good! Just some unrelated line changes where I wasn't sure if they are intended.

Thanks. They should be fixed now.

Maybe in the future, we could enforce override in a CI lint?

Absolutely, I will create an issue for it.

As discussed here, -Wsuggest-override can/should be added to the set of warning flags in this PR.

I have performed the change and rebased, let's see if CI likes it.


The ImmediatelyFast mod causes the interface opacity leave tooltips, and items unaffected on hotbar

This is an Immediately Fast optimization, not a Legacy4J issue


New Publish Action on Secure Runner

❤  Thank you for this, I wonder we give it a try , but with some really old version and I thin the over-ride will fail at least we should get 401 or something I guess?


[ROCKETMQ-31] Fix "No such file or directory" warning for $HOME/rmq_b…

The PR is to resolve the issue ROCKETMQ-31

Dear @shroman , thanks for your suggestion :)
In my concept, broker thread is time sensitive. When broker thread in a MQ broker is in I/O operation and GC occurs, an I/O competition appears between broker thread and GC thread. It may bring long time latency to broker thread.
Linux distributions based on the 2.6 kernel and later offer /dev/shm as shared memory in the form of a RAM disk (link). Pointing GC log to /dev/shm can avoid I/O competition between broker thread and GC thread.
In conclusion, I tend to use -Xloggc:/dev/shm/mq_gc_%p.log :)

yes, I see, you want to reduce disk I/O by placing the file into /dev/shm.
Well, GC logging has a minimal overhead. And to get a high-performance broker, I would tune my JVM to get smaller GC pauses, which has much larger impact on the overall performance.
Also, is it ok not to persist your GC logs? Long-term logging is essential for analysis of the brokers' performance.

@zhouxinyu @iskl's concerns are valid indeed.
My major concern is the logs can be lost when the system crashes.  I think it's a good idea to discuss it in another JIRA issue.
I will create a JIRA issue, if it's ok with you.

@shroman it's ok surely.
Please @lizhanhui @stevenschew help review this pr.

it's good idea, review ok

@zhouxinyu @stevenschew Thanks, guys, https://issues.apache.org/jira/browse/ROCKETMQ-36 is created to consider the improvement.


Use released bellperson

There is no longer the need to patch bellperson. Instead the cuda-supraseal feature is enabled on the Filecoin crates, which makes them use supraseal-c2.
This commit also simplifies the dependencies a bit. The filecoin-proofs-api crate isn't really needed.
It also sets publish = false on the c2-test crate so that it doesn't accidentally get published.
The c2-test can now simply be run by
cargo test --release -- --nocapture

As promised here's the version that doesn't need a patched bellperson anymore. Please note that curently the test fails, as the seed no longer can be all zeros. It means that the commit-phase1-output needs to be recreated with a non-zero seed.

I've force-pushed a bit. I've added a new commit-phase1-output so now the test passes.

I've exercised editorial power to adjust the commit question. Thanks!


How do I give myself an admin rank?

There is nothing in the documentation and I cannot find a database with ranks... Please help.

yeah okay


What is the point of the $user and $group parameters?

The $user and $group parameters appear to provide a way to define who the agent should run as, but, based on our testing, Dynatrace creates its own "dt_user" during installation, thus the only thing these parameters actually seem to get used for is who will own the downloaded installation file (which should be temporary anyhow).
I propose that we get rid of these parameters and all the unnecessary confusion they cause.

fixed on latest version


thanks

looks good :)
will kick the tires...

works on my side.
Can we have a chat sometime on telegram ?

ah i jumped the gun:
gomason-publish....

failed to run "gpg":

I can get tat setup. Would like to extend the signing to be some sot of provider pattern so we can sign with ssh or other....


Tharper/bump marathon with flake fix

High-level description
Bump Marathon to 1.10.6, fixing a critical issue that would cause Marathon to sometimes fail to start.
Corresponding DC/OS tickets (required)

MARATHON-8741 LaunchQueueActor receives command to load instances before leader election occurs

Related tickets (optional)
Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): 6f2a6aec4..4303d3859

@mesosphere-mergebot label ready for review

@mesosphere-mergebot label Ship It


Fixed exact-match to numeric literal

I came across this problem in testing #91. Without the fix in serialization.ts, the new test fails.
SPARQL test results are:
test-rdf:sparql10: 219 / 438 tests succeeded
test-rdf:sparql11: 97 / 271 tests succeeded
I'll merge this change into #91 too if OK.

This is an interesting issue that probably requires more research. The reason why I use the rangeBoundary parameter is that I sometimes want to check for equality only on the string representation of the numerical value:
See https://github.com/beautifulinteractions/node-quadstore/blob/d31766221ea9163e85296c6ad80367e04ed12825/lib/rdf/serialization.ts#L80-L83 :
https://github.com/beautifulinteractions/node-quadstore/blob/d31766221ea9163e85296c6ad80367e04ed12825/lib/rdf/fpstring.ts#L27-L31
What this allows us to do is to compare numerical values across different numerical datatypes, which is very important in our primary use case. Dropping that parameter prevents us from being able to do so. This said, there clearly is an issue somewhere that needs to be addressed; I will look at this as soon as possible and get back to you.

@gsvarovsky I might have found something but I won't have time to test it properly until Thursday of Friday. If you have the spare cycles, could you test the following change?
src/lib/rdf/serialization.ts, line 164:
      case 'Literal':
        if (rangeBoundary) {
          const value = importLiteralTerm(term, rangeBoundary);
          return { gte: value, lte: value };
        }
        return importLiteralTerm(term, rangeBoundary);
      default:

Closing this PR as I've manually brought in the fix and the test into my current working branch, which is about to become the new master.


conversion from potx to pdf doesn't work

The documentation on your /content MS Graph driveItem API endpoint call states that it can convert from POTX to PDF, but this doesn't work and returns this error:
{
"error": {
"code": "notSupported",
"message": "potx",
"innererror": {
"code": "InputFormatNotSupported"
}
}
}
Please make it support POTX as well as the other formats as we need that.

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

ID: a53d4b0e-4bf4-01a7-229c-5073dd56f45d
Version Independent ID: 8dc763d8-6252-d6af-5be5-99b7a9277fbd
Content: Convert to other formats - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-get-content-format.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: MSGraphDocsVteam

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Exposing a KownType

One of our Controller Actions returns a type which specifies a few [KnownType]s. However, only the the base type is exposed.
Is there a way that we can expose all [KnownType]s for a given type and also exclude the base type?
Example
[DataContract, KnownType(typeof(DerivedModelA), KnownType(typeof(DerivedModelB))
public class BaseResponseModel
{
     //...
}

[DataContract]
public class DerivedResponseModelA
{
     //...
}

[DataContract]
public class DerivedResponseModelB
{
     //...
}

Closing as this is a dup of #534. In short, this requires Swagger support for the oneOf construct which is not currently supported by the Swagger 2.0 specification.


Fix a possible crash in rand_drbg_get_entropy

Ah, I forgot to set the 1.1.1 label, is your approval also valid for 1.1.1 ?

Shure, reapproved. All bugfixes should go into 1.1.1 by default IMO.

Self-promotion for my new cherry-checker tool: Your pull request fixes 3064b55134, which was backported to dbf0a49625.

prnum
fixes
br
commit
subject

#7382
#7381
->
dbf0a49625
DRBG: fix reseeding via RAND_add()/RAND_seed() with large input

#7382
#7381
<-
3064b55134
DRBG: fix reseeding via RAND_add()/RAND_seed() with large input

Approved to 1.1.1 too.

Merged to master: 21311777ad8b8c2f2fb8eb7a4f9618ceae1e043d / 1.1.1: f3f7f1a826617af80a92f58d494ec2c8348251a5
Thanks!


circuit-breaker client: continue collecting updates while request is in flight

Dispatch request to controller in a fork, continue collecting updates meanwhile (only one consecutive in flight request allowed to keep consistency)
Roll label registrations into the batch update collection logic
Fix a bug that was sending circuit breaker updates much more frequently than configured interval
Add more metrics!

Closed because it was cherry picked into master from v32 branch.


Progress spinner shakes when specifying a diameter.

Bug:
When specifying a diameter (16-32px) on the progress spinner the animation will eventually start to shake by a couple pixels as it animates. Using smaller diameters make it more noticeable. Using other components with animations (material inputs, etc.) makes it worse.
What is the expected behavior?
Progress spinner has a smooth animation that doesn't shake as it animates.
What is the current behavior?
Over time, the progress spinner will begin to shake up and down as it animates.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-xg5xgq

Load the stackblitz demo
Click on the input (trigger the label animation)
Click away from the input (return it to normal)
Watch for the progress spinner to shake.
The progress spinner animation will never recover.

What is the use-case or motivation for changing an existing behavior?
The issue appears worse the more material components are on the page. Currently, progress spinner is unusable in our app because the shaking is distracting and unpleasant.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular: 6.0.3
Material: 6.2.0
Chrome: Version 67.0.3396.79 (Official Build) (64-bit)

I am unable to reproduce this issue. Everything seems fine on my end. Perhaps, something is fishy on the way you are implementing it.

@HeathJared I am unable to reproduce the issue that you outlined.  Can you confirm if this is still occuring?  Unfortunately we have slightly different version of chrome 67.0.3396.99 so I am not sure if maybe this is the difference.

Yes, I just double checked today using the provided Stackblitz link. It's easier to see if you zoom the page in.
I have reproduced it in Chrome Version 67.0.3396.99 (Official Build) (64-bit) and in the latest versions of electron.

same issue here

@HeathJared I am still unable to reproduce this.
Are you seeing this in other browsers?  Additionally, what OSes are you seeing this issue on?  Additionally, if you know what GPUs are being used in your reproductions that could prove helpful as well.

I haven't tried any other browser at the moment.
All of the tests were on Windows 10.
I've seen the issue on the following GPUs:

GTX 1080
GTX 1060
GTX 960M
GTX 700 series
Intel HD Graphics 4000
Intel HD Graphics 530

@crisbeto Any chance you would be able to take a look in a  Windows 10 environment?

I was able to reproduce it with @cisasteelersfan, but trying out a few quick solutions didn't help (setting the transform-origin in all dimensions, using 3d transforms). Here's an example:

Seeing the same "jittery" behaviour, Chrome 68.0.3440.106 on Mac OS 10.13.6. Macbook Pro 2016-model. We're using diameter 20 on the spinners.
However Safari manages to render it flawless, Firefox seems to have minor issues (way more smooth than Chrome, but still seems a bit off).
GPU info:
Radeon Pro 455 2048 MB
Intel HD Graphics 530 1536 MB

This issue persists to this date.

Still happening for me, it goes away   when I zoom into the page

I've looked into this a bit, and it seems that when the spinners are spinning out of sync with one another, the shakiness goes away.
It's easiest to test with just two - StackBlitz

Open up dev tools
Inspect one of the circle elements inside of the svg
Un-Check animation-name property
Re-Check animation-name property

They should now be out of sync with one another, and no longer shaking.  Haven't found a fix or workaround for this yet.

We've replaced the material spinner with a one found on codepen:
https://codepen.io/jczimm/details/vEBpoL
MIT licenced (same as all public codepens). Had to do some minor tweaks to it but works just fine for our purposes (watch out as it may not work on IE)

if you dont want to use diameter you can work around it using zoom style property
<mat-spinner style="zoom:0.24"></mat-spinner>
https://stackblitz.com/edit/angular-material2-issue-h47gmt

@tal-abziz sadly, zoom will not work in firefox.
https://www.caniuse.com/#feat=mdn-css_properties_zoom

If you're looking for a simple fix:
https://stackblitz.com/edit/angular-material2-fallback
Note that this forces spinner into fallback mode. You'll loose the advanced stroke-dashoffset animation, but once Angular Material team fixes it, it's an ease change. This is the gist of it:
@Component({
  selector: 'spinner',
  template: `<mat-spinner [diameter]=diameter></mat-spinner>`,
  // when Material team fixes the spinner jitter, remove this provider override
  providers: [{ provide: Platform, useValue: { isBrowser: true, EDGE: true }}]
})
export class Spinner {
  @Input() diameter: number = 100;
}

I suspect the problem may be related to https://github.com/angular/components/pull/9833 and the issue: https://github.com/angular/components/issues/8984

The following seems to work in Chrome, Firefox, Safari and IE:
.mat-spinner circle {
  animation-name: progress-spinner-stroke-rotate !important;
  animation-timing-function: linear !important;
}

@keyframes progress-spinner-stroke-rotate {
  $start: 200%; /* approx 2/3 of circle circumference */
  $end: 20%;
  0%   { stroke-dashoffset: $start; }
  50%  { stroke-dashoffset: $end; }
  100% { stroke-dashoffset: $start; transform: rotate(360deg); }
}

Test here: https://stackblitz.com/edit/angular-material2-spinner-fix
Removing transform: rotate(360deg); causes stroke shaking to reappear. Maybe this is due to outer mat-spinner rotation being countered by inner circle animation keyframes without rotation.
The fix/workaround to avoid stroke shaking is to avoid keyframes without rotation. A full understanding probably requires knowledge of how animating stroke-dashoffset is affected by two rotating elements - maybe someone knows?
Also, the original animation requires special keyframes for every dimension, which makes a general fix hard. Instead I've opted for using percentages (of circle width) to approximate circle circumference, making the above keyframes fit any diameter.
The result is a somewhat different animation but imho still nice and a bit closer to Chrome's own browser tab spinner.

The following seems to work in Chrome, Firefox, Safari and IE:
.mat-progress-spinner-indeterminate-animation {
  /* keep anim slow to avoid wobble from rounding errors */
  animation-duration: 9s !important;
}

.mat-progress-spinner-indeterminate-animation circle {
  animation-name: progress-spinner-stroke-rotate !important;
  animation-timing-function: linear !important;
  animation-duration: 4s !important;
}

@keyframes progress-spinner-stroke-rotate {
  $start: 200%; /* approx 2/3 of circle circumference */
  $end: 20%;
  0%   { stroke-dashoffset: $start; }
  50%  { stroke-dashoffset: $end; }
  /* removing transform: rotate(720deg) causes wobble */
  100% { stroke-dashoffset: $start; transform: rotate(720deg); }
}

Test here: https://stackblitz.com/edit/angular-material2-rotate-fix
There are multiple problems addressed:

The main fix for stroke shaking is to avoid keyframes without rotation. Changing 720deg to 0deg causes stroke shaking to reappear. Maybe this is due to outer mat-spinner rotation being countered by inner circle animation keyframes without rotation - I'm not sure.
Fix for small wobble in some browsers from outer rotation, likely due to rounding errors. Since main rotation has been moved to inner element, the outer animation now merely masks the pattern, and thus it can be slowed considerably (10s) to make wobble negligible.
While the original animation required special keyframes for every dimension, this solution uses percentages (of circle element width) to calculate circle circumference, making the keyframes fit any diameter.

The result is a somewhat different animation but imho still nice - and a bit closer to Chrome's own browser tab spinner.

Having mat-spinner inside mat-card can assist.

Hello guys, I know it's not always convenient, but you can just hide spinner and show it using opacity property, instead of display property or ngIf, it seems like if you don't change the flow for this component, then it works well
https://stackblitz.com/edit/angular-material2-rotate-fix-uv6ggd

I have commented animation of mat-spinner styles, it is okay.


Add zhangyiluo's file

The next field helps you provide a description of the changes you made. Feel free to add a description of what you’ve accomplished so far. As a reminder, you have: created a branch, created a file and made a commit, and opened a pull request

good delete


Template not working

Fix this template and make it current

This should be resolved now with commit 0eb57ff.

Closing since there was no response. Please reopen if issue reoccurs.

Closing since there was no response. Please reopen if issue reoccurs.


Twitter Agent has an account and an API subscription

Investigate current boundaries of a Bot account on X.
Decide on a subscription level https://developer.x.com/en/products/x-api
Some concerns to address:

Twitter has a lot of restrictions per IP and if we run it on our server - nothing will be available for free, thus we likely need a subscription.
free interactions with the smart contract after some twits are also not trivial, it has to be covered and ordered by us
if the smart contract's treasury is high, probably all bots will just spam us with various prompts just to try their luck for free. We will hit even pro limits quite fast.
With those twitter api prices, it's tricky to expect a lot of forks where users have to pay their own keys

I assume Twitter integration will consume most of the time in this place. We have to purchase at least Basic plan ASAP. Free Tier is very restricted

Twitter account created. Awaiting Basic subscription.

Basic Subscription is set up. Passing this on to you @ewiner to see if you have any thoughts on the concerns listed in the description.

Thanks for setting that up.
Just want to verify something I think you're saying: if we encourage people to sign up for their own Free tier accounts and put their API key in, we'll get banned b/c they'll all have the same outbound IP (is that true?).   And the 100 reads/month limit greatly limits the extent to which you can interact with a bot running on a Free API account.
If we use the same API Key for everyone, we'll hit limits pretty soon on the Basic plan, and may have to do top-ups or go to the $5k/month plan.

if the smart contract's treasury is high, probably all bots will just spam us with various prompts just to try their luck for free. We will hit even pro limits quite fast.

If I understand their docs correctly, Pro plan allows reading a million tweets per month – that seems like a good problem to have.
So ultimately, it seems like using one paid API key for everyone is the better move here.  How about... if you built a Twitter agent in our framework, it only works if you put some $NEAR in the agent's account?  Of course that's trivial and cheap to work around (e.g. create 10,000 NEAR accounts each with one yocto), but it would increase the cost to burning our API credits above 0 and encourage NEAR usage.


zoom level token

It would be useful to have a zoom level token like the bbox token. This is needed with some data sources, e.g.
SELECT * FROM natural_earth_places WHERE display_rank <= !zoom!

Yeah this would be helpful and not a big level of effort. Thanks for opening the issue.

We've just hit the same requirement when assessing Tegola. Good to see this is on the radar!

@emacgillavry I'm going to try and address this in our upcoming v0.4.0 release. We have a v0.4.0 branch you can track if you don't mind building from source.

@pnorman @emacgillavry The v0.4.0 branch now supports the !ZOOM! token in SQL statements. Here's an example of the SQL:
SELECT gid, ST_AsBinary(geom) FROM ne_10m_land_scale_rank WHERE scalerank=!ZOOM! AND geom && !BBOX!

If you don't mind building from source can you give the implementation a test per your situation?

If you don't mind building from source can you give the implementation a test per your situation?

I don't have any Tegola-based vector sources at the moment, this was just something I had identified as a blocker for implementing one

@pnorman got it.

Wow, thanks for the update and rapid implementation. We'll certainly give
this a go this week.
On 12 March 2017 at 07:49, Alexander Rolek<EMAIL_ADDRESS>wrote:

@pnorman https://github.com/pnorman got it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terranodo/tegola/issues/88#issuecomment-285926275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABe9ALBtk2Zz94TnW5xZIiYiyviHKhQXks5rk5WUgaJpZM4KsdWs
.

I've built branch v0.4.0 but I'm getting the following error:
Error Getting MVTLayer: Got the following error (ERROR: column "zoom" does not exist (SQLSTATE 42
703)) running this sql (                select
                        gid,
                        wkb_geom AS geom
                from
                water.water
                where rank = !ZOOM!
                and geom && ST_MakeEnvelope(567124.5312829204,6.819253040484466e+06,567162.7497970577,6.8192148219703
29e+06,3857)
        )

It may be I'm not compiling things correctly, I don't know ... my first time compiling a go project, it was successful ...

Maybe this isn't the place for go compilation support, but then again maybe I'm just missing something small and obvious. I tried the instructions for building from source in the docs, and get:
~/go/tegola/cmd/tegola$ go build -o tegola *.go
main.go:16:2: cannot find package "github.com/BurntSushi/toml" in any of:
        /usr/local/go/src/github.com/BurntSushi/toml (from $GOROOT)
        /home/hans/go/src/github.com/BurntSushi/toml (from $GOPATH)
main.go:18:2: cannot find package "github.com/terranodo/tegola/mvt" in any of:
        /usr/local/go/src/github.com/terranodo/tegola/mvt (from $GOROOT)
        /home/hans/go/src/github.com/terranodo/tegola/mvt (from $GOPATH)
main.go:19:2: cannot find package "github.com/terranodo/tegola/mvt/provider" in any of:
        /usr/local/go/src/github.com/terranodo/tegola/mvt/provider (from $GOROOT)
        /home/hans/go/src/github.com/terranodo/tegola/mvt/provider (from $GOPATH)
main.go:20:2: cannot find package "github.com/terranodo/tegola/provider/postgis" in any of:
        /usr/local/go/src/github.com/terranodo/tegola/provider/postgis (from $GOROOT)
        /home/hans/go/src/github.com/terranodo/tegola/provider/postgis (from $GOPATH)
flags.go:6:2: cannot find package "github.com/terranodo/tegola/server" in any of:
        /usr/local/go/src/github.com/terranodo/tegola/server (from $GOROOT)
        /home/hans/go/src/github.com/terranodo/tegola/server (from $GOPATH)

Whereas if I first go get ./... in the repo root, the build command does succeed. But could it be I'm then pulling in dependencies from a different branch than v0.4.0?

@hpfast could you post your config file (without database credentials) so I can take a look at it? Also what version of Go are you using to compile and did you checkout the project into your $GOPATH (/home/hans/go/src/ according to the output logs)?

I'm using the !ZOOM! token on the water layer.
[webserver]
port = ":9090"

# register data providers
[[providers]]
name = "kaart"          # provider name is referenced from map layers
type = "postgis"                        # the type of data provider. currently only supports postgis
host = "localhost"                      # postgis database host
port = xxxx                            # postgis database port
database = "xxxx"      # postgis database name
user = "xxxxx"                  # postgis database user
password = "xxxxx"              # postgis database password
srid = 3857                     # The default srid for this provider. If not provided it will be WebMercator (3857)

        [[providers.layers]]
        name = "buildings"                                      # will be encoded as the layer name in the tile
        #tablename = "buildings"           # sql or table_name are required
        sql = """
                select
                        id,
                        st_asbinary(geom) AS geom,
                        height,
                        building
                from
                buildings
                where geom && !BBOX!
        """
        geometry_fieldname = "geom"                     # geom field. default is geom
        id_fieldname = "id"                                     # geom id field. default is gid
        srid = 3857                         # the srid of table's geo data.

        [[providers.layers]]
        name = "water"                                          # will be encoded as the layer name in the tile
        #tablename = "water"                     # sql or table_name are required
        sql = """
                select
                        gid,
                        st_asbinary(geom) AS geom
                from
                water
                where geom && !BBOX!
                AND rank >= !ZOOM!
        """
        geometry_fieldname = "geom"                     # geom field. default is geom
        id_fieldname = "gid"                                    # geom id field. default is gid
        srid = 3857                         # the srid of table's geo data. Don't need to specify this as it will inherit this from the provider.

        [[providers.layers]]
        name = "border"                                         # will be encoded as the layer name in the tile
        sql = """
                select
                        gid,
                        ST_AsBinary(geom) AS geom
                from
                border
                where geom && !BBOX!
        """
        geometry_fieldname = "geom"                     # geom field. default is geom
        id_fieldname = "gid"                                    # geom id field. default is gid
        srid = 3857

# maps are made up of layers
[[maps]]
name = "nederland"                                              # used in the URL to reference this map (/maps/:map_name)#
        [[maps.layers]]
        provider_layer = "kaart.border"
        min_zoom = 6
        max_zoom = 24
        [[maps.layers]]
        provider_layer = "kaart.water"  # must match a data provider layer
        min_zoom = 8                                            # minimum zoom level to include this layer
        max_zoom = 24                                           # maximum zoom level to include this layer#
        [[maps.layers]]
        provider_layer = "kaart.buildings"      # must match a data provider layer
        min_zoom = 12                                           # minimum zoom level to include this layer
        max_zoom = 24                                           # maximum zoom level to include this layer

yes, these are my steps:
go version       #--> go version go1.7.4 linux/amd64
export GOPATH=/home/hans/go
export GOROOT=/usr/local/go
export PATH=$GOPATH/bin:$GOROOT/bin:$PATH
cd ~/go
git clone https://github.com/terranodo/tegola && cd tegola
git checkout -b v0.4.0 origin/v0.4.0
cd cmd/tegola
go build -o tegola *.go #'cannot find package' errors

I think I see the issue in your build steps. Go expects the source code to be in in the src/ folder and named per the repository location. After cd ~/go run the following commands:
mkdir -p $GOPATH/src/github.com/terranodo
cd $GOPATH/src/github.com/terranodo

The next several commands look great.

Thank you, sir, for the quick Go introduction! I was wondering about the src and pkg directories. It now builds and the !ZOOM! token works.
I definitely think it's a worthwhile feature. Our use-case at the moment is on a map with all water bodies in the Netherlands; by adding a zoom rank column based on width class, area and some other properties the map stays performant even as we zoom out. There's 200k+ features in the whole country but with this feature we can reduce that to 6000 at zoom level 9. Quite nice!
What I also like is that there's minimal style-related instructions put into the server config file this way. Otherwise we were thinking we'd have to make separate layers for different zoom levels.

@hpfast anytime! Providing your setup steps made it easy to identify the issue.
I'm glad this feature is paying off for you. You're use case is perfect and it sounds like results are fantastic. Do you have a link to the project so I can check it out? Also I'm considering creating a wiki page with links to projects using Tegola. Once your project is live we can include a link to it, if you're interested.

@ARolek the project we're using this for should be online by next week, I'll let you know.


Simplify packaging of privacy framework

Commit 5be643487a8bc7db977a2a73c9d70a43a42241eb contains the new get_sources.sh
This new script is meant for users developing their own business logic that don't want to use git merge for updating to latest code.
The script will copy sawtooth cxx sdk, rest api and all src files except for business logic and config.cpp to a desired folder.

https://github.com/hyperledger-labs/private-transaction-families/blob/master/get_source.sh


JavaScript critical error at line 25, column 48002 in https://unpkg.com/v-tooltip\n\nSCRIPT1003: Expected ':'

The tool tip failed on IE. it worked fine with Chrome and Edge. How do I get around the issue? I am using VueJS version 2.5.16

@dn978 I can confirm this here's a temp fix - Lock versions
V-tooltip uses  vue-resize as a dependency.
For now, downgrading the dependency versions fixed it for me, however its ideal to make it work with the latest release
"v-tooltip": "2.0.0-rc.33",
"vue-resize": "0.5.0",

Duplicate of #615

I ended up going back to version prior to 2.10 and it worked without error. Thanks,


JBPM-5711 - elementname stored in default charset other than UTF-8 in…

… bpmn2

jenkins please retest this

@krisv @manstis Hi guys, could we get someone to please review this? Thanks!

@manstis for reads i see we use IOService readAllBytes which i assume already does that and ioService.readAllString where this code already pases the encoding. If I'm missing something please show me line number so I understand. Thanks.

@tsurdilo Great.. remember I don't know the code as intimately as you; so I might ask stupid questions based on what I can see in the PR! Thanks for the pointers.


Invalid Prop

error
[Vue warn]: Invalid prop: custom validator check failed for prop "config".
at
my code
  <template>
    <VuePlayer url="https://www.youtube.com/watch?v=dQw4w9WgXcQ" playing />
</template>

<script>
import { VuePlayer } from "@voltra/vue-player"
export default {
  name: 'Test',
  components: {
    VuePlayer
  }
}
</script>

I'll take a look at this

As per the testing I've been doing, it seems the current version of Vue 3 has a bug that only ignores validation if null is the value. When not providing the props, its value is undefined instead (which makes sense). What bugs me is that the default value doesn't seem to be used.
Looks like Vue 3 kinda forces you to use the composition API when writing components as defaults (or at the very least validation) don't work as they should.
Passing :config="null" does get rid of the error, but nothing is being rendered which I'm currently debugging

In fact, passing null skips validation and doesn't use the defaults... that's very odd


Remove log that was creating noise

I am unsure what else was wanted in ticket: https://jira.coreos.com/browse/SREP-1766
So some insight on from @jharrington22 and @rogbas would be great

@ALimaRedHat the second part of SREP-1766 was to add conditions for when we open a ticket, when its updated and when its resolved.

/retest

Closing this in favor of a new PR for SREP-1766


nixos/i3lock: i3lock program with u2fSupport option

Description of changes
Created programs.i3lock.
With option u2fSupport, to create wrapper around i3lock with setuid set  and pam u2fAuth enabled.
Which enables yubikey logins when screen locked using i3lock
setuid is workaround a known bug in i3lock https://github.com/i3/i3lock/issues/198
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Shouldn't this be programs.i3lock and have a u2fSupport option? I feel like this is a very small module for a very trivial thing that can be documented but I have no strong feeling.

Shouldn't this be programs.i3lock and have a u2fSupport option? I feel like this is a very small module for a very trivial thing that can be documented but I have no strong feeling.

Then there would be no need for you to use it as a program instead of package, i3lock works just fine as a package when you don't want the u2fSupport.
I do agree that it would be probably better for documentation but it would also make it 2 step process to enable first enable i3lock program which would have no other effect then adding i3lock into your pkgs and then enabling suboption u2fSupport which would wrap it and enable the  u2f.
I feel like that could be confusing instead of being more documenting but that's just how I feel about it, but that's just me so if you think the other way would be better I can definitely change it.
PS:
I also wished to make this work with option for choosing your i3lock package for example: i3lock-color etc..
But every package has different binary name :( not sure if there is any workaround but it would be also nice to have.

Shouldn't this be programs.i3lock and have a u2fSupport option? I feel like this is a very small module for a very trivial thing that can be documented but I have no strong feeling.

There would be no need for you to use it as a program instead of package, which could be confusing, i3lock works just fine as a package when you don't want the u2fSupport. I do agree that it would probably be better for documentation but it would also make it 2 step process to enable first enable i3lock program which would have no other effect then adding i3lock into your pkgs and then enabling suboption u2fSupport which would wrap it and enable the u2f.
I feel like that could be confusing instead of being more documenting but that's just how I feel about it, but that's just me so if you think the other way would be better I can definitely change it.

Technically, programs like this already exist: gpg, etc. Of course, they are often correlated with setuid binaries stuff.
But I would not be surprised some of them can work without programs with some serious limitations.
So i3lock do not feel that different to me. Also, I think as long as this is well documented in search.nixos.org and release notes, then, it's fine.
But, I have no strong feeling on that to be honest, I would love to have other i3lock users weigh in here.

PS: I also wished to make this work with option for choosing your i3lock package for example: i3lock-color etc.. But every package has different binary name :( not sure if there is any workaround but it would be also nice to have.

Adding a symlink to i3lock for the closure of each of this package?

Thanks for the response!
Will change it as you suggested add symlinks so that you can chose your own i3lock pkg.
Which I will add as an option to the program.i3lock.

@RaitoBezarius What do you think about that?
Tested all the edited packages and everything seems to be working nicely.
Also thanks to your recommendations it looks much cleaner as well.

Result of nixpkgs-review pr 205578 run on x86_64-linux 1

2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

6 packages built:

betterlockscreen
i3lock-blur
i3lock-color
i3lock-fancy
i3lock-fancy-rapid
multilockscreen

Should be all done now except for the literalExample which seems to be deprecated, so I left it as it is.
Thanks again for all the help.

Result of nixpkgs-review pr 205578 run on x86_64-linux 1

2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

6 packages built:

betterlockscreen
i3lock-blur
i3lock-color
i3lock-fancy
i3lock-fancy-rapid
multilockscreen

Hi, how can this be used with betterlockscreen?

Hi, how can this be used with betterlockscreen?

From quickly checking on my phone it probably will not work.
But the fix should be quiet simply just creating link named i3lock just like the second commit of this PR did.
Note: I am not familiar with betterlockscreen I am making the assumption that it is just some sort of i3lock wrapper.

I solved it. Betterlockscreen has a config file where you can specify the path to the i3lock-color binary. Since this binary is symlinked to i3lock, all that's necessary is to set i3lockcolor_bin="/run/wrappers/bin/i3lock" in said config.

I solved it. Betterlockscreen is a wrapper around i3lock-color. It has a config file where you can specify the path to the i3lock-color binary. Since this binary is symlinked to i3lock, all that's necessary is to set i3lockcolor_bin="/run/wrappers/bin/i3lock" in said config.

Noice good to know. My fix wouldn't work now that I checked how the betterlockscreen works. Glad you found solution. There isn't probably better way to do this right now :( at least not one that I can think of, since it's using wrapper with set PATH variable.


Use user configured kubectl path in node and pod menu items

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>fixes #2948
Note:

This PR adds to the extension API so that extensions can have a read-only view of user settings.

Previously when kubectl command is sent to the terminal, it will use versions for kubectl that matches with kubernetes server version. Now this will change it so that it will use bundled kubectl instead. I think this is unwanted change?

Looks like it will fallback to bundledKubectlPath which should match the , see getKubectlPath

Previously when kubectl command is sent to the terminal, it will use versions for kubectl that matches with kubernetes server version. Now this will change it so that it will use bundled kubectl instead. I think this is unwanted change?

Looks like it will fallback to bundledKubectlPath which should match the server version, see getKubectlPath

Lens sets kubectl binary folder into PATH for terminal and when running kubectl command it will use correct version. Now we are switching to use bundled kubectl binary for these commands.
With Lens 5.1.3
➜  ~ which kubectl
/Users/lauri/Library/Application Support/Lens/binaries/kubectl/1.19.7/kubectl
➜  ~ kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.7", GitCommit:"1dd5338295409edcfff11505e7bb246f0d325d15", GitTreeState:"clean", BuildDate:"2021-01-13T13:23:52Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.10+k3s1", GitCommit:"72e8196c9b9eb277f56951db6d2d9e5f9f70feac", GitTreeState:"clean", BuildDate:"2021-04-16T18:36:38Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}
➜  ~ exec kubectl exec -i -t -n lens-metrics prometheus-0 -c prometheus "--" sh -c "clear; (bash || ash || sh)"

With this PR
➜  ~ kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.7", GitCommit:"1dd5338295409edcfff11505e7bb246f0d325d15", GitTreeState:"clean", BuildDate:"2021-01-13T13:23:52Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.10+k3s1", GitCommit:"72e8196c9b9eb277f56951db6d2d9e5f9f70feac", GitTreeState:"clean", BuildDate:"2021-04-16T18:36:38Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}
➜  ~ exec "/Users/lauri/Projects/lens/binaries/client/darwin/x64/kubectl" exec -i -t -n lens-metrics prometheus-0 -c prometheus -- sh -c "clear; (bash || ash || sh)"
➜  ~ /Users/lauri/Projects/lens/binaries/client/darwin/x64/kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.15", GitCommit:"73dd5c840662bb066a146d0871216333181f4b64", GitTreeState:"clean", BuildDate:"2021-01-13T13:22:41Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.10+k3s1", GitCommit:"72e8196c9b9eb277f56951db6d2d9e5f9f70feac", GitTreeState:"clean", BuildDate:"2021-04-16T18:36:38Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}

@nevalla Then there seems to be a different issue at play here too because if you look at the original issue, the user reported that their configured path was not being used.

Previously when kubectl command is sent to the terminal, it will use versions for kubectl that matches with kubernetes server version. Now this will change it so that it will use bundled kubectl instead. I think this is unwanted change?

Ah I think I understand your concern now. I want to use the setting if specified but not use the fallback if it isn't.

Lens sets kubectl binary folder into PATH for terminal and when running kubectl command it will use correct version. Now we are switching to use bundled kubectl binary for these commands.

I am not switching to the bundled version though. I am switching to the specified version in settings that the user said to use instead.


Added less and sass compilers

Vaadin seems a stable product that uses sass as css pre-processing. Gladly they pulled out the sass java compiler from vaadin sources and made it public. It's also an active project.
The reason I choose less4j was that it's a pure java compiler (lesscss uses rhino for javascript)

@danjee Thanks for your explanation.

I have a dream to pack the Less, Sass functionalities in StaticResourceHandler via LessResourceHandler respective SassResourceHandler :smile:
Also we can make public (now it's private) the method Application.addStaticResourceRoute and the code will become beautiful:
addStaticResourceRoute(new LessResourceHandler(...));
addStaticResourceRoute(new SaasResourceHandler(...));

What you say? Is it possible?

That sounds like a good plan.
These handlers should also cache the generated CSS in PROD & TEST mode - no need to continuously recompile.

@decebals I will make the modifications to use the handlers in that way.
@gitblit I had an idea to keep a map with last modified time as key and the generated css as value but I think I will have a problem when there are multiple less/scss files and only a subset gets modified. I don't think I can check all of them for time modification. Do you have an idea on a different approach ?

These are supposed to be classpath resources, yes?  So in the production app the LESS/SASS is within the jar/war artifacts and they can not change so they only need to be compiled once.  In DEV mode we can recompile on every request.
This is similar to the approach we take with the TemplateEngine implementations.  In PROD & TEST mode the templates are compiled once and cached by the engines.  In DEV mode they are not cached at all and are therefore recompiled on every request.
I should think a ConcurrentHashMap<String, String> would be sufficient for caching, keyed by SASS/LESS resource name.  Am I overlooking something?

Thanks!

@danjee Thanks for contributing this.  I plan to take advantage of it, looks to be quite useful!


官方文档select组件,切换紧凑主题查看下拉框，最后一条留白太大

Reproduction link
https://github.com/NG-ZORRO/ng-zorro-antd
Steps to reproduce
官方文档select组件,切换紧凑主题查看下拉框，最后一条留白太大

What is expected?
下拉框样式正常
What is actually happening?
不正常

Environment
Info

ng-zorro-antd
17.0.1

Browser
chrome

Hi @EnochGao
Thank you for submitting this issue.
I will start working on it as soon as I can.


Fixes metadata for external help file.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Remove-CsHybridApplicationEndpoint.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Easylogging config option %time doesn't work

C++ compiler version: N/A
Hazelcast Cpp client version: 3.12
Hazelcast server version: 3.12
Number of the clients: N/A
Cluster size, i.e. the number of Hazelcast cluster members: N/A
OS version (Windows/Linux/OSX): Linux
Please attach relevant logs and files for client and server side.
Expected behaviour
Using %time as a logging configuration option in a logging configuration file should cause time to be printed in logs.
Actual behaviour
It causes %time to be printed in logs e.g.
15/04/2020 %time : (0x7f13eb3c9700): INFO: (20190418:3f465ee6) LifecycleService::LifecycleEvent STARTING

The %date, %datetime, %thread options all work. Only %time seems to be broken.
Steps to reproduce the behaviour
Use this as the logging configuration file passed to setConfigurationFileName()
* ALL: // We need to start level with star '*' and end with colon ':'
    FORMAT                              =       %date %time : (%thread): %level: %log  // Quotes are not necessary but to make it more readable we wrap quotes around.
    FILENAME                            =       var/cluster-cloud.log
    ENABLED                                     =       true
    TO_FILE                             =       true
    TO_STANDARD_OUTPUT          =       false
    MILLISECONDS_WIDTH          =       6
    PERFORMANCE_TRACKING        =       false
    ROLL_OUT_SIZE               =  2097152 // Throw log files away after 2MB

* DEBUG:
   ENABLED              = false

Hello @andysCaplin We utilized the easylogging library (https://github.com/amrayn/easyloggingpp/tree/v8.91) underneath and it processes the config file. It looks like it can not handle %time format.
Well, actually we will retire easylogging in the upcoming release and use another logging library that utilizes C++11 capabilities (We could not do this until today since we had to use C++98 compatible, but we are moving to C++11 in this upcoming release, hence there will a lot changes including the logging library).
I am setting the milestone to 4.0 for this issue until the new logger is in place. By the way, I also understand that this is not a blocker since you were able to use the alternative format %datetime which is a lot more powerfull.
Thanks for reporting this issue.

It is minor, we wanted to use %time because using %datetime means we produce a log with a timestamp that's inconsistent with our other logs but it's not a big deal.

fixed by #638 We now have integration point to any logging framework where you the user can have full  control of the logger. You just can configure using the logging configuration. Hence, the easylogging is removed from the library now.


Getting errors while make all

fuckedup@ubuntu:~/TeamSpeakHasher# make all g++ -std=c++11 -Wall -Iinclude   -c -o TunedParameters.o TunedParameters.cpp TunedParameters.cpp: In static member function ‘static TunedParameters TunedParameters::parse(const string&)’: TunedParameters.cpp:102:15: error: ‘invalid_argument’ is not a member of ‘std’          throw std::invalid_argument("Tuned parameter section is invalid.");                ^ TunedParameters.cpp:112:11: error: ‘invalid_argument’ is not a member of ‘std’      throw std::invalid_argument("Tuned parameter section is invalid.");            ^ make: *** [TunedParameters.o] Error 1

Make sure you are using gcc version 4.9.x or later. If that does not help, try to add the following line at the top of TunedParameters.cpp:
#include <stdexcept>

Make sure you have OpenCL installed for your graphics card (AMD, Nvidia, or Intel). Then, ensure that the OpenCL library is accessible at a known library location. For example, if the library is at /usr/lib/x86_64-linux-gnu/libOpenCL.so.1, you may want to set the following symlink:
sudo ln -s /usr/lib/x86_64-linux-gnu/libOpenCL.so.1 /usr/lib/libOpenCL.so
Note that the issue you are facing now has nothing to do with TeamSpeakHasher. It is just a matter of setting up OpenCL. I recommend you search the web for more information on how to do this.

No, OpenCL works on servers just as well. Again: You need to setup OpenCL on your system. Did you do this? If yes, what exactly did you install? What GPU are you using?

Do servers even have a GPU?

Thats what i mean.
I made this steps, but make all at the end of the steps dont work.
https://www.wikihow.com/Install-Mesa-(OpenGL)-on-Linux-Mint
Do you guys maybe have some alternative step by step guide or installer script?

Its all for Windows! I want to run it on my Dedicated Linux Root. :-D

https://www.wikihow.com/Install-Mesa-(OpenGL)-on-Linux-Mint
The URL you posted is about OpenGL, not OpenCL. What GPU does your server have?

It does not have any.

So you don't have a proper GPU that supports OpenCL. Quote from the FAQ:

Can I use my CPU to increase the security level?
Currently, only GPUs are supported. This is mainly due to the current code not being vectorized properly. However, OpenCL also supports CPUs. You can try to change DEV_TYPE in TSHasherContext.h to CL_DEVICE_TYPE_CPU.

Note, moreover, that even though you could modify TeamSpeakHasher to run OpenCL on a CPU, this hardly makes any sense. It will be at least 1000 times (!!) slower than running it on a modern GPU.

Shit, ok thanks! :)

Presumably, it will be a bit faster, but most likely not more than a single-digit integer factor.
If you absolutely want to increase your security level and have no access to a decent GPU, I would recommend renting a GPU-server from a cloud hoster for a couple of hours (during which you can compute what it would take your CPU years (!!) to compute).


chore(starknet_sierra_compile): move env var from run time to compile time

Stack:

#2455
#2454 ⬅
#2453
#2450
#2449

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


S3 Lock Down

Creating a restriction policy for an s3 bucket.  Useful when trying to lock down S3 buckets for compliance reasons.

The missing-policy-statement requires a statement_ids array for comparison of missing statements.
I think that s3 buckets by default have empty policies and by default only give access to the bucket owner and root users. To lock down a bucket with no policy, I think that it would be sufficient to remove the global grants with the delete-global-grants action.

i think we want to go a slightly different route to just have the  remove statements handle the 'matched' statements as an option, i've got a branch next with that implemented.

merged the matched statement capability in #1463, use the cross-account filter for filtering and populating the matched statements.


[WIP] add more coverage for hard fork test

Explain your changes:
*
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #15416

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-nightly-me

!ci-nightly-me


prompt to long

Describe the bug
prompt is too long: 200153 tokens > 200000 maximum error needs to fixed can't update the work.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-agumc6
Steps to reproduce
Asked Botl.new to Perform a comprehensive system verification including unit tests, integration tests, and end-to-end tests. Check for code coverage, performance bottlenecks, security vulnerabilities, and accessibility compliance. Provide actionable recommendations for improvements, prioritized by severity and impact.
prompt is too long: 200153 tokens > 200000 maximum. Two words, one word, nothing works. I even bought more tokens, and refreshed the browser and cleared the history.
Expected behavior
return to normal chat.
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 149904306,
  "usedJSHeapSize": 108511978,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = REBEL33Soul
Chat ID = 3d5c681ebb63

Additional context
Perform a comprehensive system verification including unit tests, integration tests, and end-to-end tests. Check for code coverage, performance bottlenecks, security vulnerabilities, and accessibility compliance. Provide actionable recommendations for improvements, prioritized by severity and impact.
prompt is too long: 200153 tokens > 200000 maximum. Two words, one word, nothing works. I even bought more tokens, and refreshed the browser and cleared the history.

https://github.com/stackblitz/bolt.new/issues/1322

@REBEL33Soul As @yasinkocak says, we are tracking this issue and the workarounds in #1322 so please head over there for more info!


Respect dictionary encoding

As soon as node-spellchecker tries to be as much system-native as possible, I think it makes sense to support non UTF-8 encoded dictionaries.
It seems that many distributions package their hunspell-* packages directly from libreoffice repo (so does Chrome with its own additions). Unfortunately, the encodings in the repo vary a lot. So to fully support system dictionaries we have to transcode strings. Probably, this way.
Related to #51, #54

This should be resolved by https://github.com/atom/node-spellchecker/pull/95.

Since I haven't heard a response, I'm going to assume this is working. Feel free to reopen if it isn't.


Add support for Bravyi-Kitaev transformation in UCCSD ansatz

The UCCSD ansatz uses the HF state as the initial state. This PR prepares that initial state depending on the transformation (JW or BK) used.
(Note: I confirmed that this works with the H2 VQE workflow.)

Should be good to go now. I would like either @jromerofontalvo or @jgonthier to take a look before merging though.

Sorry for the delay! I'm going to have some dinner and then review this code.


Generate constructor helper

Closes #155
To do:

[x] update tutorial example in readme.

This PR tests will be fixed with new version of plugin #164

Should I merge the other PR and create a new RC?

Yes

Done. I assume you want to add the following setting to the docs project.
nativeBindgenPath := {
  Some((ThisBuild / baseDirectory).value / "bindgen/target/scala-native-bindgen")
}


Define parsing of negative values to unsigned

Define parsing of negative values to unsigned types for stringTo.
In particular, this allows "-1" to be parsed as the maximum possible unsigned integer value for the given unsigned type. It also allows wraparound of large positive values too. That may be expected for unsigned values, which have well define wraparound, as per the C++ standard.
Related, is the definition of std::stoul, which is used for parsing the smaller unsigned values:

If the minus sign was part of the input sequence, the numeric value calculated from the sequence of digits is negated as if by unary minus in the result type, which applies unsigned integer wraparound rules.

Fixes the bug found after merging PR #920.

Could probably use a second set of eyes on this one to check if this is even a good idea.

Could probably use a second set of eyes on this one to check if this is even a good idea.

Hrm. :/ It depends. If you are intending for "negative one means infinite", and that's the only reason to support negative unsigned values, I wouldn't do it. Instead explicitly document the use, but parse it as a regular int and detect the use of -1 explicitly but convert it to a "large" number that's likely to "take forever" or "never finish" before the end of the program.

Given the finding in #935, I think I'm going to close this and explore another option.

So I see that this is closed but will comment anyway: I originally intended -1 to be a sentinel but later opted to just stick with 0 since that seemed simpler. Plus, why would someone want a 0-length delay?
Anyway, that's the direction I went with later sprite definitions. Will need to go over them to be sure they're set to '0' and not -1 but yeah, '0' should be the way to go forward with this.

Huh, I just checked. You did have one data asset that used the value "0". Though that wouldn't have worked with the previous code.
grep -rn 'delay="0"'
structures/warehouse.sprite:11:		<frame sheetid="main" delay="0" x="0" y="0" width="128" height="128" anchorx="0" anchory="64" />

Anyway, with the updates I applied, that value would now work as expected. Of course, now that the default when not specified is "0", that particular value has now been removed. I figure use the defaults when possible to simplify things.


Allow to use dioxus-cli within a cargo workspace

I'd like to work on this on my own if this feature is welcomed.
Specific Demand
I'm using dioxus in a cargo workspace and want to run the dx command within the workspace root instead of cd to the dioxus project before run dx. Also, although I did not do it yet, a workspace could include multiple dioxus projects in the monorepo context.
Implement Suggestion
Add -p, --package [<SPEC>]        Package with the target to run option to several dx's commands.

This would be a great addition to the CLI! This would be useful for the main Dioxus repo which uses workspace.

Cargo build, serve, bundle, and fmt now all apply based on the package arg.
Only check is not yet supported on a per-package basis.


Remove packages used only in development phases from requirements.txt

This pull request contains changes to reduce packages used only in development from the requirements.txt file which lists the packages in product images.
Ref https://github.com/docker-science/cookiecutter-docker-science/issues/94

LGTM

Thanks 🙏


feat: add @index directive

Description of changes
This PR adds a new @index directive to the GraphQL Transformer v2. It also ports the @key E2E tests to v2. It also includes the commit from #7879 which is a bug fix needed to get the test suite passing.
Issue #, if available
Description of how you validated changes
Manual, unit, and E2E testing.
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #7887 (7c4e934) into master (346bc69) will increase coverage by 0.30%.
The diff coverage is 99.07%.

@@            Coverage Diff             @@
##           master    #7887      +/-   ##
==========================================
+ Coverage   53.38%   53.68%   +0.30%
==========================================
  Files         557      558       +1
  Lines       28159    28344     +185
  Branches     5455     5501      +46
==========================================
+ Hits        15033    15217     +184
  Misses      12140    12140
- Partials      986      987       +1

Impacted Files
Coverage Δ

...es/amplify-graphql-index-transformer/src/schema.ts
99.48% <98.59%> (-0.52%)
:arrow_down:

...amplify-graphql-index-transformer/src/resolvers.ts
99.41% <98.80%> (+0.37%)
:arrow_up:

...index-transformer/src/graphql-index-transformer.ts
100.00% <100.00%> (ø)

...ges/amplify-graphql-index-transformer/src/index.ts
100.00% <100.00%> (ø)

...rc/graphql-transformer/transform-graphql-schema.ts
15.87% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 346bc69...7c4e934. Read the comment docs.


Consider integrating fork synchronization task via workflow bot

Inside the Zeebe modeler we've created a sync task which helps automatically making changes from the base camunda-modeler to it's zeebe fork. For now it has to be manually triggered by executing npm run sync locally. Furthermore the resulted commit has to be pushed remotely, left and not cleaned conflicts has to be fixed on the machine by the user itself, also a Pull request with the changes has to be opened manually.
Moreover there is no guideline on when doing the sync. Should we do it once a week? Once a month? On every new camunda-modeler release?
As a future task we should consider to do this all automatically via a GitHub bot. It should include different tasks:

Trigger sync task automatically in a certain period on zeebe-modeler, e.g. on every new camunda-modeler tag
Create a pull request on zeebe-modeler side with made changes
Verifies made changes are stable in the zeebe-modeler (CI)
Give an overview of merged changes
Give an overview of cleaned changes
Give an overview of unresolved conflicts which have to be merged manually after synchronization

Maybe the workflow could be orientated by the way Dependabot keep dependencies in sync. Also the merge-me bot could be an example on how building a proper Probot workflow bot.

We'll merge both editors, mid-term.


How do I set the trigger request using regex?

Hello,
I understand Chatterbot uses question and answer type of training , where the response from the bot is based on the request it sends to.
Although that's interesting, how can I make the response generic..
For example, the bot can handle 'Do you know Java'?
But how do I make '* Java *' to match any string with Java in it.. This is something tools like Rivescript is able to.. this is one thing I look to before deciding on which framework I start with for chatbot Dev..

@senthilrameshjv IMO using regular expression in the requests, your will see poor performance getting accurate response.
Instead did you tried different comparison methods available in chattbot https://chatterbot.readthedocs.io/en/stable/comparisons.html

Thank you @vkosuri , let me check on that.


CUPS support

Can we send data to cups server and print from there instead of usb or serial? I tried Network but seems not working. I also tried PR#348 but failed-perhaps it is incompatible for today's python-escpos.Any solutions?

Hi @SydneyOwl
The Network connector is not suitable for printing through a CUPS server. You should use the CupsPrinter connector at #348 , but since that PR hasn't been merged yet, you need to follow these instructions to update your python-escpos installation to that commit. You should then be able to start sending commands to your ESC/POS printer's CUPS queue.
Please, let us know if that works for you.
Anyway, don't doubt to ask for help if needed.

Hello @belono ,
Since my machine works in offline mode so pip is unavailable, I tried pip3 uninstall escpos and pull your code from here which should be same and manually copied src/escpos to /usr/local/lib/python3.7/dist-packages/. Also I have to remove version-import part from __init__.py since it raises ImportError if escpos is not installed from PyPI as it said in sourcecode. However, it finally reached a TypeError:
Traceback (most recent call last):
  File "/home/raspi-weather/rsp/flask_upload/tese.py", line 1, in <module>
    import escpos.printer as printer
  File "/usr/local/lib/python3.7/dist-packages/escpos/printer.py", line 18, in <module>
    from .escpos import Escpos
  File "/usr/local/lib/python3.7/dist-packages/escpos/escpos.py", line 48, in <module>
    from .magicencode import MagicEncode
  File "/usr/local/lib/python3.7/dist-packages/escpos/magicencode.py", line 23, in <module>
    from .codepages import CodePages
  File "/usr/local/lib/python3.7/dist-packages/escpos/codepages.py", line 1, in <module>
    from .capabilities import CAPABILITIES
  File "/usr/local/lib/python3.7/dist-packages/escpos/capabilities.py", line 47, in <module>
    PROFILES = CAPABILITIES['profiles']
TypeError: string indices must be integers

I'll try connecting my device to internet and using pip later. Thank you for your reply!

I use pip install -U git+https://github.com/belono/python-escpos.git@f07d5e06109e37aa45a5a139f49f56d6399b98d3 instead of pip install -U git+git://github.com/python-escpos/python-escpos/pull/348/commits/f07d5e06109e37aa45a5a139f49f56d6399b98d3. Pip failed to clone pr via git+git://, so I used the former one.
I made some changes to your code (constructor of CupsPrinter(Escpos)) to enable connecting to remote CUPS server:
def __init__(self, host=None,port=None,printer_name=None, *args, **kwargs):
            Escpos.__init__(self, *args, **kwargs)
            if host!=None:
                cups.setServer(host)
            if port!=None:
                cups.setPort(port)
            self.conn = cups.Connection()
            self.tmpfile = None
            self.printer_name = printer_name
            self.job_name = ''
            self.pending_job = False
            self.open()

And according to your instruction I ran
import escpos.printer as printer

cups_conn = printer.CupsPrinter(host="<IP_ADDRESS>",port=631,printer_name="XP-58IIH")  # Default CUPS printer
cups_conn.textln("Hello CUPS!")  # or any python-escpos command
cups_conn.close()

It works!
Here are detailed info of my devices. Hope they're useful to you.

CUPS server: raspi zero w, Linux raspberrypi 5.15.32+ #1538 Thu Mar 31 19:37:58 BST 2022 armv6l GNU/Linux
Printer: XP-58IIH
print request sender: an simple armv7l device based on armbian

If I understood, my fork's version didn't work for you due to the lack of .setServer() and .setPort().
What happened before you added them? Any exception thrown?

No exception thrown. This is just an enhancement for those who separated the print command sender and the print command processer.
For example, Raspi is the cups server and is only responsible for receiving printing instructions and performing printing operations. In other words, It will not print without receiving instructions. Therefore, the terminal (instruction sender) needs to specify the address and port of cups service to connect to raspi, then a series of operations can be performed by sending instructions to cups server. Or the default connection of  CupsPrinter(Escpos) will be localhost instead of raspi.

Oh! So the raspi is a remote CUPS server on a lan, right?.
I will add host and port to the parameters of the CupsPrinter constructor to configure the .setServer() and .setPort() setters, but let them be optional via args, kwargs for convenience to maintain consistency with the Win32Raw class constructor.
Your feedback is being very helpful, thanks.

Glad that I could be of help😁


Added the theme to the repo, as you cannot run without the folder

@HuyHung1408 I didn't see an explanation/reason for not having the themes in your repo and I had to go to the release to get the theme versus cloning the repo.

Thank you for adding for me @TimothyMalahy . I didn't add it yet because I'm using the web version, which cannot upload a folder. Much thanks

No problem! I added the darkmode feature. Feel free to use it as inspiration or refactor it. I am sure you want to name things the way you want. 🙂


Repair discount. Feature

I absolutely love the plugin as is but I thought what if a server doesn't want to give free repairs but like %60
Example permission
ultrarepair.discount.
ultratepair.discount.60
I'll be writing a review soon... Already guaranteed 5 star

Great suggestion. Unfortunately, I'm a bit occupied at the moment, but I will try to get around to doing this when I can. Thank you!


Can't login when groups are nil

Bug Report
Description
We are getting  User::OAuthError (undefined method `compact' for nil:NilClass for @DanCoughlin  when he logs in. I can't be sure, but it appears that the id_token returned from azure has no groups key. I'm not sure under what circumstances this could happen. but apparently it can. In the strategy, maybe we should always return an array in the strategy?
https://github.com/psu-stewardship/scholarsphere/blob/5b5ca81ef4b2d20d4a6c794a31732a3207aaa5d0/app/controllers/users/omniauth_callbacks_controller.rb#L10
To Reproduce
Steps to reproduce the behavior:

Go to '.Home page'
Click on '.Login' (and login as @DanCoughlin )
See error

as a side, when rescuing this error, we should perhaps return a 5xx style error, this  was returning 200

as a side, when rescuing this error, we should perhaps return a 5xx style error, this  was returning 200


[feature request/question] Form wise errors

On form validation I want to raise errors such as: "at least a field must be filled" which do not belong to one field, but to the form as a whole.
Would it be possible to raise have them on the form? (I'm currently using a hidden field and validating these kind of errors on it).

Is any work being done for this? A perfect use-case for this is in a login-form where you want to tell the user that the combination of the username and password field is invalid.

I think this is actually not needed, because you can always override the validate method and check for field errors after calling the superclass validate method there.
For a complex example, see: http://flask.pocoo.org/snippets/64/

Warehouse (PyPI) has this: https://github.com/pypa/warehouse/blob/efc2a3fc18783d4375f878de548318d00f206924/warehouse/forms.py#L109-L116
They use the string "__all__" for the key, @azmeuk suggested None. Pinging @di because he edited the code I linked.


test(dtslint): add skip

Description:
This PR adds dtslint tests for skip.
Related issue (if exists): #4093

Coverage decreased (-0.2%) to 96.809% when pulling 396c34118066a2a4c6b462b1437bc6bde991bdb6 on timdeschryver:pr/dtslint-skip into 03ee88cce186ae67ab871383f63e4d746e11ea41 on ReactiveX:master.


Updating the Warning

updating the warning under "For the primary node type"

@pkcsf : Thanks for your contribution! The author, @ChackDan, has been notified to review your proposed change.

#sign-off

#sign-off

@pkcsf: I'm sorry - only the author of this article, @ChackDan, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off