Isolate no cluster tests

What type of PR is this:
/kind tests
What does this PR do / why we need it:
This PR moves integration tests for commands not needing a cluster to a specific directory
so they can be executed separately, in an environment without any cluster.
Which issue(s) this PR fixes:
Fixes partly #6237
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

LGTM, but I was wondering if the setupCluster argument we have in helper#CommonBeforeEach now does not seem kind of redundant with the nocluster label.
Thinking about this because I noticed that tests that are labelled with nocluster actually need to also call helper.CommonBeforeEach(false), but there is nothing guaranteeing that this is the case.
At least, I think we can maybe programmatically determine the presence (or absence) of the nocluster label and based on that information determine the value of setupCluster. I can see that Ginkgo has an interesting CurrentSpecReport().Labels() function that could return such information during the spec run phase.
What do you think?

I'm not very comfortable with this kind of hidden side effect.
I find meaningful to have the nocluster label confirmed by helper.CommonBeforeEach(false): every time I added the label, I checked that this declaration was confirmed by something in the code.


wasmparser: Improve FuncType

This reduces the number of separate allocations per FuncType instance by 1.
Also this adds some public API and hides the implementation details.
Other crates of the workspace have been adjusted.
The Debug implementation has been adjusted to output the same output as before.
The same optimization could be applied to Wasmtime's FuncType, too.

Thanks for this! Do you have some profiling information which has been guiding this or some performance numbers for before/after? This is one of the less-efficient parts of wasmparser right now and is indeed ripe for improvement.

I executed the compare-with-main.sh script but it found no performance differences.
However, in theory the new implementation has less pressure on the heap allocator (if we actually make use of that in the parser) and parameter and result types are more data local and therefore should technically be more cache efficient, i.e. less pointer chasing.
~/p/w/c/wasmparser> ./compare-with-main.sh
Already on 'main'
Your branch is up to date with 'origin/main'.
   Compiling wasmparser v0.82.0 (/home/robrob/projects/wasm-tools/crates/wasmparser)
   Compiling wasmparser-dump v0.1.0 (/home/robrob/projects/wasm-tools/crates/dump)
    Finished bench [optimized] target(s) in 7.67s
     Running unittests (/home/robrob/projects/wasm-tools/target/release/deps/benchmark-5530eeb73a768754)
WARNING: HTML report generation will become a non-default optional feature in Criterion.rs 0.4.0.
This feature is being moved to cargo-criterion (https://github.com/bheisler/cargo-criterion) and will be optional in a future version of Criterion.rs. To silence this warning, either switch to cargo-criterion or enable the 'html_reports' feature in your Cargo.toml.

it works benchmark      time:   [7.5139 ms 7.5460 ms 7.5824 ms]
                        change: [-0.4231% +0.1704% +0.7660%] (p = 0.59 > 0.05)
                        No change in performance detected.
Found 10 outliers among 100 measurements (10.00%)
  4 (4.00%) high mild
  6 (6.00%) high severe

validate benchmark      time:   [21.502 ms 21.636 ms 21.798 ms]
                        change: [-1.0421% -0.1693% +0.6935%] (p = 0.71 > 0.05)
                        No change in performance detected.
Found 10 outliers among 100 measurements (10.00%)
  4 (4.00%) high mild
  6 (6.00%) high severe

Already on 'main'
Your branch is up to date with 'origin/main'.
    Finished bench [optimized] target(s) in 0.05s
     Running unittests (/home/robrob/projects/wasm-tools/target/release/deps/benchmark-5530eeb73a768754)
WARNING: HTML report generation will become a non-default optional feature in Criterion.rs 0.4.0.
This feature is being moved to cargo-criterion (https://github.com/bheisler/cargo-criterion) and will be optional in a future version of Criterion.rs. To silence this warning, either switch to cargo-criterion or enable the 'html_reports' feature in your Cargo.toml.

it works benchmark      time:   [7.5560 ms 7.5860 ms 7.6192 ms]
                        change: [-0.0888% +0.5304% +1.1422%] (p = 0.10 > 0.05)
                        No change in performance detected.
Found 9 outliers among 100 measurements (9.00%)
  8 (8.00%) high mild
  1 (1.00%) high severe

validate benchmark      time:   [21.822 ms 22.008 ms 22.280 ms]
                        change: [+0.5872% +1.7173% +3.1834%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 9 outliers among 100 measurements (9.00%)
  7 (7.00%) high mild
  2 (2.00%) high severe

Oh so is this not motivated by a real-world module? I understand how this is theoretically better but I would prefer that there would be something concrete showing it before merging.

Oh so is this not motivated by a real-world module? I understand how this is theoretically better but I would prefer that there would be something concrete showing it before merging.

I guess I cannot deliver on such a module then. Feel free to either merge or close this PR then.


Swiping 'back' in Safari on iOS disables interactivity

Even on a freshly installed Rails <IP_ADDRESS> with a generated User scaffold, swiping 'back' once (doesn't matter on which page) renders anything with a turbo-method useless. The only code that I changed is the Destroy button. However, I have first spotted this behavior in one of my more complex apps.
Expected behavior: "Destroy this user" should still work even after
Steps to reproduce:

In a default Rails setup with a User scaffold update the destroy button's code with a turbo_method.
Open website in Safari on iOS. Add a user. Destroy works at this point. Click 'Back to users'. Swipe 'back' and destroy no longer works.
Only thing that restores interactivity is a refresh.
In Firefox on iOS this issue is (strangely enough) not present. However, on Firefox Focus on iOS turbo does not seem to work at all.

<%= button_to "Destroy this user", @user, method: :delete, form: { data: { turbo_method: :delete, turbo_confirm: 'Are you sure?' } } %>

I believe my issue might be related to #302, though I'm not sure.

The only workaround I know of is the one posted in the above-mentioned Turbolinks issue. Keep in mind this bug only affects  Safari on iOS. Safari on Mac and WKWebkit (for you Turbo Native devs) are perfectly fine. I'd be curious to know if @domchristie has any new workarounds since posting the one 5+ years ago:
;(function () {
  var ua = window.navigator.userAgent
  var isIpad = ua.indexOf('(iPad') > -1
  var isIphone = ua.indexOf('(iPhone') > -1 || ua.indexOf('(iPod') > -1
  var isChrome = ua.indexOf('Chrome/') > -1 || ua.indexOf('CriOS/') > -1
  var isSafari = !isChrome && ua.indexOf('Safari/') > -1
  var isIosSafari = (isIpad || isIphone) && isSafari

  if (isIosSafari) {
    var popped
    window.addEventListener('popstate', function () { popped = true })

    document.addEventListener('turbolinks:before-render', function (event) {
      if (popped && event.data.newBody.querySelector('[data-confirm]')) {
        window.location.reload()
      }
    })
  }
})()

I tried that code and I couldn't get it to work (tried adding it in application.js and application.html.erb). The only tubing I changed was turbolinks to turbo.
My code looks for Safari but excludes macOS, but not iPadOS, since it's also affected.

You will also need to update [data-confirm] to [data-turbo-confirm]:
;(function () {
  var ua = window.navigator.userAgent
  var isIpad = ua.indexOf('(iPad') > -1
  var isIphone = ua.indexOf('(iPhone') > -1 || ua.indexOf('(iPod') > -1
  var isChrome = ua.indexOf('Chrome/') > -1 || ua.indexOf('CriOS/') > -1
  var isSafari = !isChrome && ua.indexOf('Safari/') > -1
  var isIosSafari = (isIpad || isIphone) && isSafari

  if (isIosSafari) {
    var popped
    window.addEventListener('popstate', function () { popped = true })

    document.addEventListener('turbo:before-render', function (event) {
      if (popped && event.data.newBody.querySelector('[data-turbo-confirm]')) {
        window.location.reload()
      }
    })
  }
})()

(I have not tried this as I'm still using Turbolinks, but this should work in principle.)

@domchristie thanks for providing the updated code, but unfortunately the issue persists. Browser detection does seem to work (tested with alert()), but no reload occurs.

@petru this will only trigger a reload if the incoming visit's HTML includes an element with the data-turbo-confirm attribute, so it won't work with arbitrary alert calls

I added alert() just after if (isIosSafari) { to confirm the script is in the right place, then removed it. I do have elements with data-turbo-confirm on the page, but no reload occurs and they're not working.

Still no luck, I'm afraid.

Has anyone submitted an issue on https://bugs.webkit.org?

I don't think it's a Webkit bug. All other browsers on iOS are forced to use Webkit, but Safari's the only one affected (tested on Brave and Firefox on iOS).

I wonder if there's a better way to detect this? User agent testing does feel a little bit unreliable.


feat: Support for multiple versions of cudevshr for vGPUmonitor

What type of PR is this?
/kind feature
What this PR does / why we need it:
Support for multiple versions of cudevshr for vGPUmonitor. Add the time elapsed since the last kernel launch for vgpu monitor.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/assign

/lgtm


Update release

What type of PR is this?
/kind cleanup
/kind feature
What this PR does / why we need it:
similar we are doing for zeitgeist, applying the same here, have the gorelease to build the binaries and sign it and push to github releases and the image we follow the regular promotion process

build binaries with goreleaser and not with cloudbuild
add goreleaser and release jobs
update ko to 0.13.0

/assign @saschagrunert @puerco @xmudrii @ameukam
cc @kubernetes-sigs/release-engineering
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/lgtm


fix check -238 segment number limit earlier

Proposed changes
Previously we check segment number when build rowset because we don't want throw an error in the intermediate state, hoping segment compaction can reduce segment number later.
This PR adds a earlier segment number check, while doing segment compaction.


Possible bug in toHaveText matcher

Hello, I've been seeing some unexpected behavior from the toHaveText matcher, and wanted to let you know in case it's a bug. I am using Jasmine-Node to test a Backbone.js app. The test in question is this:
it('doesn\'t include backlogged tasks', function() {
  data.taskCollection.models[0].set('backlog', true);
  view.render();
  expect(view.$('div.dash-widget[data-name=tasks] div.huge')).toHaveText(data.taskCollection.length - 1);
});This test should fail because I have noticed the app does indeed fail to meet the expectation. Indeed, when I change the expectation, it does fail:
expect(view.$('div.dash-widget[data-name=tasks] div.huge').text().indexOf(data.taskCollection.length - 1).toBeGreaterThan(-1);This generates the following output:  1) Dashboard Top Widget View elements task widget doesn't include backlogged tasks
Message:
Expected -1 to be greater than -1.
Stacktrace:
Error: Expected -1 to be greater than -1.
at null. (/private/var/www/spec/views/partialViews/dashboardTopWidgetsViewSpec.js:79:116)I believe the matcher uses indexOf to test for the presence of a string literal. This seems very straightforward, and I know the behavior of my app because I have tested it both manually and with Selenium. Any ideas?
Thank you for this tool, it's very helpful!

The matcher uses test when you pass a regular expression, and indexOf in all other cases. You can check out the code here, it's pretty straightforward: https://github.com/unindented/jasmine-jquery-matchers/blob/0c1c11094beb623da43911d9165110f568fafd56/src/matchers.js#L127-L138
Can you create a small test case that I can easily reproduce? Something like the tests that are already there would be awesome: https://github.com/unindented/jasmine-jquery-matchers/blob/0c1c11094beb623da43911d9165110f568fafd56/test/specs/matchers.js#L94-L114
Thank you for your help!

I’ll see what I can come up with. I won’t have time today, unfortunately, because one of my servers got hacked and I have to get  my sites back online. I should be able to put something together by the end of the weekend.

On Oct 15, 2015, at 2:04 AM, Daniel Perez Alvarez<EMAIL_ADDRESS>wrote:
The matcher uses test when you pass a regular expression, and indexOf in all other cases. You can check out the code here, it's pretty straightforward: https://github.com/unindented/jasmine-jquery-matchers/blob/0c1c11094beb623da43911d9165110f568fafd56/src/matchers.js#L127-L138 https://github.com/unindented/jasmine-jquery-matchers/blob/0c1c11094beb623da43911d9165110f568fafd56/src/matchers.js#L127-L138
Can you create a small test case that I can easily reproduce? Something like the tests that are already there would be awesome: https://github.com/unindented/jasmine-jquery-matchers/blob/0c1c11094beb623da43911d9165110f568fafd56/test/specs/matchers.js#L94-L114 https://github.com/unindented/jasmine-jquery-matchers/blob/0c1c11094beb623da43911d9165110f568fafd56/test/specs/matchers.js#L94-L114
Thank you for your help!
—
Reply to this email directly or view it on GitHub https://github.com/unindented/jasmine-jquery-matchers/issues/1#issuecomment-148324064.

Sorry to hear that. No rush.


Update README with multiline examples

Change-type: patch
Signed-off-by: Zahari Petkov<EMAIL_ADDRESS>

@balena-ci I self-certify!


Files For Configs To Be Written To

Checklist

[x] I read the README
[x] I read the FAQ
[x] I searched the issues
[x] My issue is about the script, and not OpenVPN itself

To keep things neat, create a directory in the home directory where all openvpn configs are written to and stored.

PR welcome :)


Select-String output doesn't have context

With PsColor loaded in profile:
> Set-Content

cmdlet Set-Content at command pipeline position 1
Supply values for the following parameters:
Value[0]: abc
Value[1]: def
Value[2]: ghi
Value[3]:
Path[0]: test
Path[1]:

> sls test -pattern 'def' -Context 2

test:2:def

without psColor loaded the context is displayed correctly:
> sls test -pattern 'def' -Context 2

  test:1:abc
> test:2:def
  test:3:ghi

Good catch! I'll fix it as soon as I get some spare time.


created autoscale policy's policy arn doesn't return as expected

def setup_autoscale_group():
    mocked_networking = setup_networking_deprecated()
    conn = boto.connect_autoscale()
    config = LaunchConfiguration(
        name="tester", image_id="ami-abcd1234", instance_type="m1.small"
    )
    conn.create_launch_configuration(config)

    group = AutoScalingGroup(
        name="tester_group",
        max_size=2,
        min_size=2,
        launch_config=config,
        vpc_zone_identifier=mocked_networking["subnet1"],
    )
    conn.create_auto_scaling_group(group)
    return group

@mock_autoscaling_deprecated
def test_create_policy():
    setup_autoscale_group()
    conn = boto.connect_autoscale()
    policy = ScalingPolicy(
        name="ScaleUp",
        adjustment_type="ExactCapacity",
        as_name="tester_group",
        scaling_adjustment=3,
        cooldown=60,
    )
    conn.create_scaling_policy(policy)

    policy = conn.get_all_policies()[0]
    print policy.arn

The policy arn returned here is not expected, It should return policy arn for "ScaleUp", and autoscale group name should be tester-group.
But arn is like this: arn:aws:autoscaling:us-east-1:803981987763:scalingPolicy:c322
761b-3172-4d56-9a21-0ed9d6161d67:autoScalingGroupName/my-test-asg:policyName/MyScaleDownPolicy
I think this is a bug that when create policy, it won't assign correct policy arn to policy.

Hi @PeterCui1991, I can't reproduce this I'm afraid. I'm getting the ScaleUp-policy, as expected: ScalingPolicy(ScaleUp group:tester_group adjustment:ExactCapacity)
Are you creating another policy first, that is also being returned in the get_all_policies() call?

Going to close this due to a lack of response, but feel free to let me know if this is still an issue.


Produces not showing up on Swagger UI?

I'd assume using doc.produces({...}) would have something show up in the UI but nothing is showing up.

I can second this, in 0.4.0 models don't get translated into definitions and the produces decorator doesn't work correctly (probably as a consequence of it). I've reverted to 0.3.0 which does what you'd expect.

@daenney A possible workaround is:
@doc.route(produces=Car, ...)

same issue with 0.4.0. @joar's work-around worked, though (thanks).

same issue with 0.4.0.

@doc.produces() is worked after merge #58, close this issue.


pinnable_mapped_file persistence

This PR add two parameters for pinnable_mapped_file

on_dirty_mode: configure how to handle the mapped file when it's dirty
-- new delete_on_dirty is useful for tests.
persistent: determine whether to persist into disk when using heap and lock mode

As you mentioned this should be merged into a different branch than eosio


how to add a new entry in Edit CMake Cache UI?

Brief Issue Summary
When use CMake in my project, I want to install both Debug/Release. Then, I need to set CMAKE_DEBUG_POSTFIX=d. However, I can not set CMAKE_DEBUG_POSTFIX=d in Edit CMake Cache UI. Hope CMake can add this feature.
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

For context, I suggested this user to raise a feature request.


Source Control Pane Hide/Show Issues

Issue Type: Bug
For some reason I can't get the "Source Control Providers" option to display (Source Control: Git).  Right-clicking the SC title bar and checking that option (SCP) doesn't seem to do anything.
Furthermore, unchecking an option appears to remove the option below it (if checked) instead of itself.  Perhaps there is an wrong-index issue here.
VS Code version: Code 1.42.1 (c47d83b293181d9be64f27ff093689e8e7aed054, 2020-02-11T14:45:59.656Z)
OS version: Windows_NT x64 10.0.18362

I don't understand what you mean, can you please provide a screen recording?

I have seen this issue too, it appears to be caused the the customise-ui extension. Disabling this extension and reloading the monkey patch code causes the SCM side bar to function properly again.
I've created a new issue on the extension repo, here: https://github.com/iocave/customize-ui/issues/35


flatten doesn't work without expand

Enabling flatten does nothing when expand is disabled
Tested with the following settings
{
  copy: {
    css: {
      expand: false,
      flatten: true,
      src: [
        'bower_components/normalize-css/normalize.css'
      ],
      dest: 'source/css/vendor/'
    }
  }
}

Glad to see this issue is getting some attention, having the same problem myself...

This happens to me as well

Nowhere is it explained what 'expand' does. Every example has expand: true. What does expand mean?


On_LandMineExploded not triggering

function On_LandMineExploded(Landmine)
    Server.Broadcast("A landmine has exploded!")
end

The hook appears to never be called when a landmine explodes.

see #147


project.json to .csproj for Less/Sass/FontAwesome

Fixes #2827
@Rick-Anderson @spboyer
Review link: https://review.docs.microsoft.com/en-us/aspnet/core/client-side/less-sass-fa?branch=less

Nice update!


memory leak battle

changes:

rebase onto the tip of 0.17.0-prerelease
fixed memleaks:

method args and return value on MRI/2.2
method return value in jruby
get rid of object space module as it's not supported in rbx or jruby

Argh, a typo after rebase.

@niamster - great stuff.
I find the mailbox code quite confusing with the timers, checking, locks, receive()+check() API ... and that break returning a message (which was a break from the timer loop).

@e2 not so tricky IMHO. What I hate here is that we have to violate nice ruby syntax to avoid memory leak =|

@niamster no worries about syntactic beauty. Make it work then make it beautiful :wink:

@niamster the followup #585 to this was functionally beautiful. It has really helped with the fractures I've been seeing. Celluloid is starting to behave like itself again.


struct_pack serialize

I have a question. The data serialized with struct_pack goes to stack memory or heap memory, and it is not listed in the document.

It's determined by user. You can serialized data to any container/ofstream/memory address
For example
auto buffer = struct_pack::serialize<std::vector<char>>(object); // save to heap
auto buffer2 = struct_pack::serialize<std::string>(object); //save to stack/heap (SSO optimize)
std::array<char,1000> buffer3;
auto length=struct_pack::get_needed_size(object);
assert(length<1000);
struct_pack::serialize_to(buffer.data(),length, object); // save to stack
std::ofstream ofs("1.sav", std::ios::out| std::ios::bin); //save to disk
struct_pack::serialize_to(ofs, object);

You can even use user-defined stream. See: https://alibaba.github.io/yalantinglibs/en/struct_pack/struct_pack_intro.html#custom-output-stream


Add referrerPolicy support for page.goto

As we can see Page.goto method does not support referrerPolicy parameter.
DevTools protocol documentation says, that we can specify referrerPolicy for navigation with specified referer: https://chromedevtools.github.io/devtools-protocol/tot/Page/#method-navigate
https://chromedevtools.github.io/devtools-protocol/tot/Page/#type-ReferrerPolicy
Any plans to add referrerPolicy support?

No plans, happy to review a PR.

Closing as completed by #9561


React 19

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

Project coverage is 99.39%. Comparing base (3fde0c8) to head (ff8cf60).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #542   +/-   ##
=======================================
  Coverage   99.39%   99.39%
=======================================
  Files          33       33
  Lines         331      331
  Branches       87       87
=======================================
  Hits          329      329
  Misses          2        2

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


Language server crashes because of 'Could not find file'

Recently I have to use some vue-like single component file, but its extension name is '.wpy' but not '.vue',
vetur works perfect for '.wpy' in the past and I don't know when it starts to crash but I just cannot use it for '.wpy' and here's is the log.

C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\node_modules\typescript\lib\typescript.js:84996
throw new Error("Could not find file: '" + fileName + "'.");
^
Error: Could not find file: 'c:\Users\Tony Ke\Desktop\liru\src\app.wpy'.
at getValidSourceFile (C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\node_modules\typescript\lib\typescript.js:84996:23)
at Object.getSyntacticDiagnostics (C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\node_modules\typescript\lib\typescript.js:85204:52)
at Object.doValidation (C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\modes\javascriptMode.js:131:49)
at C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\htmlServerMain.js:142:43
at Array.forEach (native)
at validateTextDocument (C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\htmlServerMain.js:140:59)
at Timeout._onTimeout (C:\Users\Tony Ke.vscode\extensions\octref.vetur-0.6.4\client\server\htmlServerMain.js:134:9)
at tryOnTimeout (timers.js:232:11)
at Timer.listOnTimeout (timers.js:202:5)
[Info  - 16:50:01] Connection to server got closed. Server will restart.

vetur: 0.6.4
vscode: 1.11.2

You can try adding wpy files to your jsconfig / tsconfig like this:

But there wpy files are not supported right now.

Your solution doesn't work...
I know vetur does not officially support any wpy files but actually the format of wpy file is just like vue single component file[template+style+script].So I  associate *.wpy file with vue by modifying setting.json.
Vetur works perfect in the past but I don't remember the version. what happens to the update...

Try downloading 0.5.6 here https://github.com/octref/vetur/issues/136#issue-223411869 and don't upgrade. I'll think about adding wpy support.


[Agent] Jira Story Facilitator

systemRole
You are a specialized Technical product manager focused on creating stories for a sprint board in Jira. Your primary function is to transform spoken or written feature ideas into comprehensive, well-documented with technical best practices stories using the following fields:

A Short Title
Summary (Required) use the statement structure of "as a [persona], I [want to], [so that]"
Description
Acceptance criteria
Questions

You as an assistant must follow the rules under:

Clarity: Ensure that the 'Summary' and 'Description' fields are clear, concise, and unambiguous.
Interactivity: Ask for all necessary details to populate these fields accurately.
Completeness: Make sure all required and applicable fields are filled out for each story.
Quality Assurance: Include any additional information in the 'Description' or other fields that will assist in QA testing.

Your objective is to facilitate a seamless transition from feature idea to actionable Jira story, fully utilizing the specified fields to make it as easy as possible for developers to implement and you may suggest technical best practices.
identifier
jira-product-manager
avatar
📋
title
Jira Story Facilitator
description
Specialized in transforming feature ideas into comprehensive Jira stories
tags
technical product management, story creation, Jira
locale
en-GB

@emad-pg hi, it seems that you have sumbitted this agent, do you want to just updated it?

@arvinxx Hi! Yes! I have improved the prompt.


Search in selcted shelfs

Describe the feature you'd like
as you can search in books we would love to see a feature to search in a shelf or tick/untick shelves on the left corner of a global search.
Describe the benefits this would bring to existing BookStack users
this way you could filter the search and reduce the number of unwanted search results on bigger libraries.
Can the goal of this request already be achieved via other means?
no
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
1 to 5 years
Additional context
No response

Thanks for the request @stothew,
Would just the addition of a search bar to the shelf, to search within, as per the books view, cover your needs?
Providing a select/checklist for shelves within the search interface would require a lot of custom UI and handling to implement properly, which I don't think would  be worth to be honest since there's been little other demand, but I could see some value in aligning book and shelf behaviour.

Thank you for the reply.
The addition of a search bar to the shelf, to search within, would totally fulfill my needs!
And yes, for me, this would also add more consistency to the UI and enhance the freedom for own Data structures.

I also would like such a feature! I think it's a msut :-)

Any indication on how to implement this? I could help. I would like to be able to search in a book, but a shelf is also a good target.

for books, this already exists... If you select a specific/single book you get a search bar on the top left side menu to search only within this book.
Anyway, I would still love to search on shelves. So feel free to help :)

Searching on shelves is a must if you have books with a similar title to other books in other shelves. I have to fight with that issue myself.
As pointed out in multiple other Issues, that have been closed in favor of this stale one, prioritizing such feature (maybe even in the means of adding it to a "help wanted" board) would be a massive functionality and UX improvement. I totally get it, why suggested compromises like adding the shelf name to the bread crumbs of the book would not be possible, because of the theoretical possibility of the same book being in multiple shelves. I do not do this, as do probably many others, but don't break what is already a core feature and the reason why someone, somewhere in this world uses BookStack over other solutions.
In regards to @stothew :

for books, this already exists... If you select a specific/single book you get a search bar on the top left side menu to search only within this book.

While it's true, that books have a dedicated additional search bar to search within a book, i don't think this is best for UI / UX anyways. The main search at the top center of the current BookStack frontend is a lot more prevalent to a user. If this feature gets implemented, it should also be considered to redesign the main search as a single source of truth. For example by having a small dropdown within the search bar where the user can specify between "all", "book" and "shelf", depending what part of the UI the user is currently at. this could also be enhanced in the future for in-page searches by highlighting the matching text within the currently viewed page.

however, to make it possible in the first place, a first step in the right direction would be to extend the search engine filters currently used in BookStack with a new option, to filter by a shelf slug (as the name of a shelf can be the same as another)

We are encountering the same issue and are considering a practical and effective solution.
How about implementing a context-specific search box? When you're at the root level, the search would cover all shelves.
if you're within a specific shelf, the search would be limited to books in that particular shelf.
Additionally, the Advanced Search could be enhanced with an option to select multiple shelves?


Improve and align Rejseplanen with other transport components (Breaking)

Breaking Change:

All attributes changed to snake_case, and Later departure changed to next_departures.
Returns next_departures as a list of dictionaries containing the same info as the next departure, instead of a string.

Documentation
Pull request in home-assistant.io: home-assistant/home-assistant.io#9939
Description:
Additional Changes:

Additional transport types added.
Restricts the search to just the relevant types to return more results.

Configuration remains unchanged, and the current documentation does not mention the output.
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[X] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[X] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[X] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[X] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Let's cc @tomatpasser since he is the only author so far.

Thanks @amelchio.
I think most looks great. Good idea to use the search filter functionality from the API.
I don't understand how the next_depatures is going to make sense on the card now. This is what I see when I run your branch and click the sensor:

That does not seem very user-friendly. Before it gave information on the next coming departure ("Bus 350S towards  Herlev St. in 14 from Elmegade (Nørrebrogade) ". Now there is no straight-forward way of viewing that information.
Also, why not capitalize the attributes? Is this a new home-assistant thing? Doing a quick check, i found no common format for two different sensor types. Captalizing looks better when expanding a sensor card.

@tomatpasser Generally, the more info cards aren't useful for anything other than the most basic  sensors.
What the next_departures field now returns, is a dict of the upcoming departures, which can then be used to feed other things, like this lovelace card:, or in automations and templates.

I get that, but it still makes no sense to print [object Object] in the UI. I'm guessing I should complain about this somewhere else.
Great job with the lovelace card.

@elupus Can you verify that your requested changes have been implemented correctly?

Yes the changes I requested are good. I'm not sure about the attributes being objects though. I'd need to get some verification on that being okey.

Yes the changes I requested are good. I'm not sure about the attributes being objects though. I'd need to get some verification on that being okey.

The reason the times are output as object in the frontend, is that the javascript code for the default more info modal, just naively converts every value to a string. In JS, dicts are just generic objects, and running toString() on that yields [Object object].
While the more info modal certainly could use a lot of improvement, I don't think that's within the scope of this PR. :)

@elupus ?

@martinhjelmare since it's a breaking change. Do we need to note it somewhere?

Yes. :+1:
@DarkFox please update the breaking change paragraph with what the user needs to do to cope with the breaking change.

@MartinHjelmare I've updated the paragraph, please let me know if it needs any further improvements.

Great!


Create-React-App

Does anyone know how to get this working with Create-React-App ?
I've been struggling all day and I am still nowhere forward :(
It works if I build the CRA itself BUT I have to keep doing this every time I change something. Create-REact-App comes with it's own Dev server that hot reloads and I wanted to be able to plug into it but when I do a console.log(eel) it always prints undefined.

How about something like this to reload that app: https://github.com/AoiKuiyuyou/AoikLiveReload
I have not tried it... but I plan to.

@AnthoniG this is something I want to get working, but I don't have time until next month. If you get it working in the meantime I please post here

I played around with this for just a few minutes. I did it using VueJS instead, but the concept should be the same. It is not yet an elegant solution, but it works without making any modifications to Eel.
Files
Here is my file structure.
.project
├── app.py
├── start.sh
└── web
    ├── build
    ├── config
    ├── index.html
    ├── node_modules
    ├── package.json
    ├── README.md
    ├── src
    ├── static
    └── yarn.lock

Here is start.sh:
#!/bin/bash

cd web
yarn run dev

And, here is app.py
import eel
import threading
from subprocess import call
from time import sleep

def start_web():
    call(['./start.sh'])

def start_eel():
    sleep(4)
    eel.init('web')
    eel.start('', options={
        'port': 8080,
        'host': 'localhost',
    })

if __name__ == '__main__':
    t1 = threading.Thread(target=start_web)
    t2 = threading.Thread(target=start_eel)
    t1.start()
    t2.start()

Plan
As you can see, the idea is simple: start both the eel webserver, and whatever other webserver you want. Then, tell the eel browser to open your other webbrowser instead of a static file.
Pitfalls
I hacked this together in just a few minutes as a proof of concept. Clearly we can improve upon this, and looks like something that we could and should be able to bake into eel itself.
As you can see, I needed to throw a sleep in there just to make sure the webpack server was up and running first. This needs to be fixed.
Maybe also there is a better solution to using Threading
This may not be the best solution, but it does work. I was able to startup the servers, and make changes to my Vue app and have it live reload inside of the eel browser.
Kudos to @ChrisKnott for an awesomely simple and elegant implementation.

I have been playing around with this, and I think I have an idea on how to set this up with an API sort of like this:
eel.init('web')
eel.start(
    {
        'port': 8888,
        'host': 'localhost',
        ‘path’: ‘’
    },
    services=[
        {
            ‘dir’: ‘some/working/path’,
            ‘exec’: ‘my_executable_command.sh’
        }
    ]
)

The thought would be to block the run_lambda function if services is present. In that case, Eel would go out and startup whatever services it could, and once they were up and running, continue on with run_lambda.
The second change would be to make the first parameter in start to be either a string or an object. If it is an object, then it would be used for setting the location of the browser so that the options keyword argument could still be used for gevent.
@ChrisKnott If you agree with this approach, I’d be happy to put together a PR for you. Am I on the right track?

I created a PR #33. This allows users to set any arbitrary location inside eel.start.
The effect of this would be to serve (for example) an application running from npm run dev.
HOWEVER, this does not cover the second half of launching services. I am hoping to get some feedback on @ChrisKnott about my strategy if it is a worthwhile endeavor or not.
This could (as shown above) be something entirely handled by app.py and not be bundled inside of eel. I just happen to think it would be a nicer API if eel could manage spinning up the other services as well.
Of course, in the react or vue instance, this would be in development only. When packaged up, this would be bundled into static files that COULD be served by eel.

Here is an example of how this is running using Threading outside of eel.
https://github.com/sockpuppetapp/sockpuppet

@ahopkins Loving the idea and going to play around with your git
Also going to have a play around with React and see what that thinks of this. Like I mentioned I got it to partially work, but this looks to be a cleaner solution.
BTW: How did you get it to not complain about address in use ?
Also, what do you do for a build? Have seperate app.py or conditional checks to see if in production etc ?

@AnthoniG

BTW: How did you get it to not complain about address in use ?

The options parameter is used to tell gevent which port to hook up to. So, I tried to separate the location of the browser from the binding of gevent. The way I have it running actually is to let my webpack service run on 8080, and I changed gevent to 8888. As long as they are running on different ports, all is fine.
page = {
    'port': 8080,
}
eel.start(page, options={
    'port': 8888,
})

Right now, I am just launching webpack in a separate terminal session waiting to head back from @ChrisKnott. I think the best route would be to create a services parameter that can spawn new processes in an identical manner as used already, and then initiate the process, also as currently done for launching the browser.

Also, what do you do for a build?

I have not tested it out yet. But, conceptually, there would be at least a different operation to run. Once there is a build of the Javascript app, that could be handled statically by gevent and served without needing to get another process in there. I have not tried it yet, but it would be no different conceptually from what is on the README.

Have seperate app.py or conditional checks to see if in production etc ?

Perhaps. Or, rather than building the conditionals into it, just replace the script with a build version. I suppose it depends how much abstraction gets introduced.

@ahopkins Thanks for that. I won't be able to fully test it until this weekend. My idea then is to use your code as a base and re-factor it for my Create-React-App version of it.
Before this, I was thinking of going the way of using React in the browser and compile in-memory like JSBin / JSFiddle et al. do

Awesome. Can you post your code here? Curious to see how your solution may differ

@ahopkins Yep can do, although it won't change much. Will just use Create-React-App itself rather than VueJS.
The other method of having it compile in-memory I never got around to testing, but was going to be doing it all over the weekend.
BTW: What is that eel.set_client you have in your code? It's setting a Web Socket address.
I've been looking into Python Web Sockets however they are much easier to get up and running with Javascript it seems. I intend to have a NodeJS Server app with a Web Socket server listening. Then my Python / JS app will communicate through that. The JS will pick up the socket info and then pass that info over to Python via eel. Kind of like a Hybrid web / desktop application :)

Here's my code. Like I said it's based on your VueJS code :smiley_cat:
https://github.com/AnthoniG/eel_react
Need to play around with it and work how to do a build / deploy scenario but other than that it works. Guess just need main Eel distro to be altered with your pull request.
Also included a requirements.txt BUT it's listing Eel as the one from Pypi. However I did a pip install git+ from @ahopkins REPO :smile:

I've hit a problem on the Python call to Javascript side of things.
No matter what I do, I can not get Python to call a Javascript function.
It keeps throwing an AttributeError and crashes (I wrapped it in a try..except to prevent that, but it still happens)
I will amend my repo with the code I have and hopefully some one can figure it out.
@ahopkins Your code works fine (I only put the JavaScript function in index.html, did not try it further down the pipeline because I don't understand Vue that much)

I was planning to try and get eel to be exposed inside webpack tonight. I’ll post any code changes I have. Push what you have to your repo and I will be happy to take a look.

@ahopkins Code updated.
You will see in App.tsx (my main app entry) that I  have several comments in there that explain how I was trying to setup the functions.
Files changed :-
eel_react.py
App.tsx
Files Added :-
src/classes/eel.tsx
(this is because Typescript requires a definition file and this will allow me to call eel functions from anywhere in my code by simply importing this file.)
[Will work on a proper .d.ts file later, once I've got everything working lol]

@AnthoniG I figured out the problem. Actually, there were a couple.

The way that eel is "exposing" methods in JS is by crawling the source files looking for eel.expose. In order to get it to look for methods in a tsx file, I needed to add it to __init__.py in the eel package:
allowed_extensions = '.js .html .txt .htm .xhtml .tsx'.split()

There was a sort of scoping problem because the function being exposed was inside a component. Once I moved it outside into eel.tsx, it worked fine.

I think this sort of underscores a potential flaw in the setup of Eel. No offense to @ChrisKnott, but this method I think is not scalable for an app built on (for example) React, and may potentially be flawed. The startup time is delayed quite a bit because of the size of node_modules directory. Granted, once built, this would not be a problem.
I would rather see eel.expose in the JS be a call to the Websocket that pushes the name of the method. This would make it a little more transparent what eel.expose is doing, and would allow for dynamic changes to the exposed functions, rather than everything being declared up front at initialization.

@ahopkins Do I need to re-pull Eel from your git then in order to make this work ?

You would need to stash a local copy of the eel repo, and make the change. I have not committed it.
git clone https://github.com/ahopkins/Eel.git
Then, install that.
pip install -e /path/to/Eel
Go into Eel/eel/__init__.py and add .tsx into allowed_extensions as shown above. That will get it working for now until there is a solution. Your alternative would be to define all of your eel methods in a file with a .js extension.

@ahopkins I am not familiar with React at all (my background is in games dev and C++ mainly!) do you mind if I add you as a contributor to the project, then I can stand back and let you integrate the React things?
I honestly wouldn't even know how to go about testing it.

Awesome. I would love very to help  out. Not sure I'd call myself a React guru, although my main background is as a full stack web developer. I am happy to help and provide my input.

@ahopkins Not sure if anyone is still working on this issue. If so, maybe it would be nice to open it as a separate branch so everyone can check the latest progress and potentially contribute.
I do agree that eel should have a way to build with popular framework such as React, Angular or Vue. Especially for the complied files of those front-end frameworks, I wondering if there is an elegant way for eel to detect those "exposures".  :/

Yes, I am working on it. I have not committed code because it still needs some more tests and (quite honestly) the last couple weeks were simply too busy for me. Hopefully I will have a chance to push soon.


GetKnownService/GetKnownCharacteristic

Hi, I am really liking your plugin so far, but there are a few things that is stopping me from using it. We have specialized firmware that requires authentication values to be written to the peripheral after connection otherwise it will terminate the connection - there is a timeout period for this.
We have very specific, known services and characteristics that is responsible for this. Hence it is important to us to be able to connect to a known service and known characteristic for this known service. Currently we have to check each characteristic/service as it is discovered to find the one were are looking at. Because of the limited time we have to send the authentication value, and the fact that having to scan for all services and all characteristics is too expensive, we reach the timeout before we can send the required values.
We are therefore looking for something like
IGattCharacteristic characteristic = device.GetKnownService(UUID of service)
.GetKnownCharacteristic(UUID of characteristic);
I know your time is precious, and this is not an issue, but a feature request, sorry I did not know where else to put it.
Thanks
Marius

It of course goes without saying that in the implementation, this should directly connect to the known services/characteristics and not just filter/linq to it :)

Done for next release


getting started typo

https://github.com/baku89/glisp/blame/master/docs/get-started.md#L11
I first thought 左上 is correct, but anyways the origin is at the center and +x is right, +y is down. I don't know what is a good way to describe this.
for example, doc in p5.js says

(0,0) can be found at the top left with the positive direction to the right horizontally and down vertically.
-- https://p5js.org/learn/coordinate-system-and-shapes.html

so I will rephrase that (origin at center) in the english doc

https://github.com/baku89/glisp/blame/master/docs/get-started.md#L45
右上→左上

https://github.com/baku89/glisp/blame/master/docs/styles.md#L24
塗り→線のプロパティ

https://github.com/baku89/glisp/blame/master/docs/draw-tree.md#L20
ベタが来した→ベタ書きした

Sorry for checking the issue so late...! I had just merged the fixes :)


Create clean/ca/fremont_pd.py

I believe this is ready for a PR (https://github.com/naumansharifwork/clean-scraper/tree/ca-77), i believe the last update was that this depends on #99 , @newsroomdev let us know the status on that so we can close the loop


Added adapter::get_config()

I need a way to read the interface state and peer statistics, so I decided to give it a go.
I was able to test this on a ipv4 network and it seems to work just fine. Unfortunately, I don't have an access to an ipv6 network, so I wasn't able to test that. In particular, I wasn't able to verify if we need to convert ipv6 flowinfo and scope_id to little endian like we need to convert the port.
I made some dubious choices around the API design (struct/field names, having get_config() accept a Vec<SetPeer>, etc.). In particular, I tried to add the new method without changing any of the existing methods (incl. making &self mutable), which made it necessary to include Vec<SetPeer> in the arguments. Feel free to suggest alternatives.
@TroyNeubauer

Apologies for the radio silence. I agree with your comments (see replies), I guess some unit tests for the StructReader wouldn't hurt either. I should have time to get this done next week

I think I've covered all the changes you suggested, let me know what you think

Hey, I added some more changes, can you take a look?

I'll publish this tomorrow after a bit more testing

@TroyNeubauer would you be able to publish it sometime this week?

Yes. Will do some testing now that #7 is merged, and will publish later today

Published as 0.3.0: https://crates.io/crates/wireguard-nt
@pradt2


WIP: Swift 5.1 support

Description

Resolve compilation errors and warnings relating to changes in the Swift 5.1 release:

import FoundationNetworking on Linux to access HTTPCookie and similar types, due to changes to Foundation structure
fix compilation warnings relating to vars that were never mutated

** Work in progress:** CI needs to be fixed to run against 5.1
Motivation and Context

How Has This Been Tested?

Checklist:

[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

For some reason, trying to install and use the 5.1 toolchain inside the swift:5.0.1 Docker image is failing (https://travis-ci.org/IBM-Swift/Kitura/jobs/547756128) with:
error: toolchain is invalid: could not find the swiftc at expected path /home/travis/build/IBM-Swift/Kitura/swift-5.1-DEVELOPMENT-SNAPSHOT-2019-06-16-a-ubuntu18.04/usr/bin/swiftc
... however I cannot recreate this failure outside of Travis (even running the exact same commands that Travis is running, ie. to run the build via Package-Builder inside a swift:5.0.1 container).  It just works.
The swiftc command is a symlink to swift in the same directory, and Package-Builder unpacks the swift toolchain to the project's directory.  When running under Docker, this directory is a mounted volume.  I wonder whether there's some property of the filesystem that Travis provides that, when mounted inside Docker as a volume, causes this symlink not to work?
In ba897be I tried taking the Docker layer out (running the same 5.1 toolchain directly in Travis' 16.04 image).  This works in that we successfully build Kitura using that toolchain, but it then seems to hang while building the the TestTypeSafeMiddleware test (https://travis-ci.org/IBM-Swift/Kitura/jobs/547790233).   However, I do seem to be able to recreate this locally (on 16.04).  Doesn't hang on 18.04.

Update: It isn't a hang, just takes several minutes to compile.  And I do see the same behaviour on 18.04, it's not 16.04-specific, I must have misled myself somehow.
Seems the output from swift build is a little misleading.  Although it usually pauses when it says [mm/nn] Compiling KituraTests TestTypeSafeMiddleware.swift, after a process of elimination, it's actually TestCodableRouter.swift that is taking a long time to compile.
That source file contains a large number of tests, so I'm in the process of narrowing down which test(s) specifically are causing the problem.  Once done, we can guard the offending ones with a version check and raise a bug.

Finally got to the bottom of the failures of 5.1 toolchains under Docker running in Travis.
It relates to https://github.com/apple/swift-corelibs-foundation/pull/1837 which introduces the use of the statx syscall to obtain file attributes.  This is only available in 'recent' Linux kernels (4.11, circa April 2017). and Foundation checks that it is available.
However, even if it is available, we may not have permission to call it.  It requires Docker 18.04 and libseccomp 2.3.3 or higher.  Both are available on 16.04 and 18.04.
Travis' 16.04 (xenial) environment provides libseccomp2 2.3.1-2.1ubuntu2~16.04.1, which is too old.  As a result, even though Docker is recent enough, the statx call is not permitted, and so Foundation throws an error, which is swallowed by SPM - and only surfaced eventually as a "swiftc not found" message.
The fix for this is to update Foundation so that if the statx call is available, but results in an errno of 1 (EPERM - not permitted), it falls back to the earlier stat approach.
A possible workaround is to upgrade libseccomp2 before running under Docker in Travis.


fix direct memory leak in partition transition

In our usage, we found there is a direct memory leak issue,  the partition buffer hasn't been reclaimed when datasources when helix drop partitions in client. We use awaiting consumer callback  shutdown and forced direct memory release to avoid this issue.

@groelofs @chavdar any comments?


Warning when user trying to enter a stopped container

It's obvious that we can't enter a stopped container and docker inspect will return pid as 0. It would be nice to have a friendly warning instead of display the error like below.
cat: /proc/0/environ: No such file or directory
nsenter: neither filename nor target pid supplied for ns/ipc

Absolutely! I'm swamped right now, so I can't really work on this, but if you (or someone else) wants to provide a patch, I will be very grateful and happy to merge it.


LBaaS enhance models for aws refresh

Purpose or Intent
Enhancing models based on real refresh for AWS, few missing links were discovered.
Steps for Testing/QA
No QA testing needed

One nitpick about indexes.  Otherwise, this should be ready.


Nuget package does not work with IIS or IIS Express

When upgrading from version <IP_ADDRESS>-pre to <IP_ADDRESS>-pre I found that IIS hosted applications to fail with the following exception:

Could not load file or assembly 'osrm.net.DLL' or one of its dependencies. The specified module could not be found.

Investigation showed that tbb.dll and tbbmalloc.dll were no longer being loaded, however adding them to c:\windows\system32\inetsvr would allow the application to find them and resolve the error.  The issue is related to IIS only copying managed assemblies to its temporary directory.  Normal applications have no issue loading the assembly since they check their working directory for any additional dependencies.
To replicate the issue do the following:

Create a basic IIS hosted application. I specifically used an OWIN based WebAPI application hosted in IIS.
Include the OSRM.Net <IP_ADDRESS>-pre nuget package.
Open an endpoint into the application.  The endpoint doesn't even need to be valid, it just needs to call into the application so IIS begins loading assemblies.

Ben, I am not sure if this is something that can be addressed by nuget itself (or even if it's doable, it'll be worth doing). It's either you turn off the shadow copying in IIS or make it part of your deployment strategy of your app.


Add 1.1 new crop

Coffee Bean

Thanks! This will be in the v2 update of the Crop Planner, which I'll be releasing this coming Monday.


Raiders of the Lost RDP: The Haag Crusade

Enhanced RDP Security Monitoring Suite
This PR introduces a comprehensive set of detections and supporting components for monitoring Windows Remote Desktop Protocol (RDP) activities, enhancing our ability to detect potential lateral movement and unauthorized remote access.
New Detections
1. Windows RDP File Execution

Monitors execution of .rdp files from high-risk directories
Focuses on temp folders, download directories, and Outlook temporary locations
Helps identify potential spear-phishing campaigns using malicious RDP files

2. Windows RDPClient Connection Sequence Events

Tracks RDP ClientActiveX connection attempts (Event ID 1024)
Provides visibility into initial connection sequences
Helps identify unusual remote access patterns and potential lateral movement

Infrastructure Additions
New Data Source

Added support for Microsoft-Windows-TerminalServices-RDPClient/Operational logs
Specifically tracking Event ID 1024 for connection sequence monitoring
Enhances visibility into RDP client-side activities

New Macro

Introduced wineventlog_rdp macro for standardized RDP event querying
Improves consistency across RDP-related detections
Simplifies future RDP detection development

Screenshots

Added detection screenshots for validation and documentation
Demonstrates expected data presentation and field mapping

Windows RDP File Execution

Windows RDPClient Connection Sequence Events

Testing

Validated against Attack Range dataset
Confirmed detection of RDP connection sequences
Verified file execution monitoring capabilities

cool use case buddy !


Does this plug-in support IE?

Does this plug-in support IE?

Sorry haven't tested. It should support most modern browsers.


Unable to connect to znc over ssl after a period of time

I recently switched over from a Centos server running ZNC 1.4 installed from yum. I am now using Ubuntu 14.04.3 x64, and made and installed ZNC 1.6.2 from the stable release listed on wiki.znc.in, and ran it using the same config directory I copied from my old server. I also remade the znc.pem, updating the certificate and chain, and adding the DH params to the end.
Directly after starting the znc process, I was able to connect with ssl to both the web interface and with an IRC client. However I noticed at after a few hours I was unable to connect to the server over ssl anymore, although any clients that were currently connected would remain connected.
I have tried a few different versions from tarball: 1.4, 1.6, 1.6.2 (recommended version), and nightly (1.7.x-nightly-20151215-e210ade2). All of these have seemed to have the same problem for me, so it is possible it might not have to do with znc alone (although I have not had problems with ssl on anything else on this new server), but I was hoping that someone might have some insight into what might be causing this.
More details
With -D on the nightly, it prints this when I fail to connect over ssl (trying from localhost here, but the same happens for remote):
[2015-12-21 01:39:18.469994] _LISTENER == ConnectionFrom(<IP_ADDRESS>, 41569) [Allowed]
[2015-12-21 01:39:18.470046] There are [0] clients from [<IP_ADDRESS>]
[2015-12-21 01:39:18.470150] src/Csocket.cpp:1690 WARNING: MakeConnection .... SSLv23_server_method failed!

Ports: I am listening for non-ssl irc and web on 6002, IRC over ssl on 5000, and web over ssl on 6001. Here is what sudo netstat -plutn looks like after I can no longer connect:
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:6001            <IP_ADDRESS>:*               LISTEN      19290/znc
tcp        0      0 <IP_ADDRESS>:6002            <IP_ADDRESS>:*               LISTEN      19290/znc
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      2696/sshd
tcp        0      0 <IP_ADDRESS>:25              <IP_ADDRESS>:*               LISTEN      6346/master
tcp        0      0 <IP_ADDRESS>:5000            <IP_ADDRESS>:*               LISTEN      19290/znc
...

Trying to connect with openssl s_client to either IRC or web:
$ openssl s_client -host localhost -port 6001
CONNECTED(00000003)
140448375039648:error:140790E5:SSL routines:SSL23_WRITE:ssl handshake failure:s23_lib.c:177:
---
no peer certificate available
---
No client certificate CA names sent
---
SSL handshake has read 0 bytes and written 295 bytes
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
---

My znc.pem consists of in this order: The private key, the certificate, the intermediates, the CA (I know this isn't needed, but I don't think it is the problem), and finally the DH params. The certificate is valid and signed by RapidSSL. (I was also having this problem both when I had an invalid cert, and when I was missing the DH params)
Again, it works fine at first, but after a period it stops accepting new connections.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29295726-unable-to-connect-to-znc-over-ssl-after-a-period-of-time?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

If you just ssh into server, does it fix the connection?
I recall that some OSes automount and autounmount home directory of user when user is not logged in...


Add support of timestamp as buildNumber

Hi, we use timestamp as buildNumber (version : 4.1-20170403114142).
This patch avoid NumberFormatException with Integer.parseInt("20170403114142"). I use long instead of int for buildNumber.

seems to be a duplicate of #57 which fixes #56


Mailtracker issues

first, thank you very much for the new mailtracking feature, I just wanted to start working on such a feature :)
But some issues came up for me:

had to change Comment.email_message_id to CharField and remake migrations, because mysql cannot use TextField as key and throws an error. TextField does not make sense here by the way?!
preserve=false setting does not work for me, emails are not deleted
the TODO_MAIL_BACKENDS first confused me, until I realised that they are not necessary for the mailtracking feature, its just that you can now have seperate email configs for each list, correct?
when a new task is created, why is he mailbody posted as comment? Is this intended? I think its good to save it as task description
at the the moment, anyone can use the feature when the email address is known. This should be limited to the user email addresses, even if emailaddresses can be spoofed. This way the messages could be just connected to the account and not to blank emailadresses

I will start working on that last issue and connect mailaddresses to accounts..

Heyo,

I had to change Comment.email_message_id to CharField and remake migrations, because mysql cannot use TextField as key and throws an error. TextField does not make sense here by the way?!

Ouch this is worrying. I wasn't aware of this limitation, and I'm not sure about how it should be handled :/ TextField does makes sense because message-ids have no size limitations, and actually, postgresql doesn't make much of a difference between these.
https://stackoverflow.com/questions/33211540/is-there-anyway-to-create-unique-textfield-in-django-with-mysql-db-backend
What fix would you suggest ?

the TODO_MAIL_BACKENDS first confused me, until I realised that they are not necessary for the mailtracking feature, its just that you can now have seperate email configs for each list, correct?

That's it

preserve=false setting does not work for me, emails are not deleted

I guess it needs a separate issue and further diagnosis. It's probably due to an unfortunate regression during development, or IMAP madness

when a new task is created, why is he mailbody posted as comment? Is this intended? I think its good to save it as task description

I know that's weird, but that absolutely intended !

it keeps the original message read-only
avoids an ugly task_message_id thing
enables commenting the mail task without interfering with the original message

at the the moment, anyone can use the feature when the email address is known. This should be limited to the user email addresses, even if emailaddresses can be spoofed. This way the messages could be just connected to the account and not to blank emailadresses

If you add end up implementing this, can you add the corresponding setting ?
It was implemented this way for multiple reasons:

it was designed as a customer support sort of thing, where all email answers need to be tracked read-only, and be obviously displayed as email answers
attaching mail to django users felt cringe because of the low binding between the two, and brought little apparent benefit
hiding the source email felt like hiding the domain name when browsing

Thanks for testing !

@ezzra see #53

@multun Iam just diving into the tracker.py code, and Iam very confused why you have installed this nested system of related_messages and answer_thread and best_task and so on, why didn't you just take the In-Reply-To which is build with the task_id in utils.py ? It looks like you are planning something with nested comments and discussion thread ?!

@ezzra that's sort of it ! this code is about properly handling mail threads. It's not used to create a mail tree inside django-todo, but it just takes in account the tree-like structure of mail threads when considering where incoming emails should go to.
There are two separate things going on here:

related_messages and best_task are about finding the most relevant task to attach the message to. Using only In-Reply-To isn't acceptable because it only references the last email in the chain !
What if someone responds to a thread, no CC-ing the tracker anymore, and someone answers the non-CCed message, adding back the tracker ? If you parse references, you're going to be able to attach that message back, not using In-Reply-To
answer_thread, when it exists, it used to attach notification answers

this complex nested system

A full mail threading algorithm looks like that. We aren't anywhere close, fortunately. tracker.py is just a glorified max(len(task.message_ids.intersection(references)) for task in tasks)

* Using only In-Reply-To isn't acceptable because it only references the last email in the chain !
  What if someone responds to a thread, no CC-ing the tracker anymore, and someone answers the non-CCed message, adding back the tracker ?

I still don't see the problem or any benefit here. Maybe I don't understand your workflow. In the end, regardless if you take answer_thread or best_task, its just a Task object that will be connected to the new comment. First I do not understand the confusing wording here, but when the track_id is all we need, and its within the header, why not just use it?
I expect users just to answer to the comment notification, which will send an email to the server with something like In-Reply-To: <notif-19.164355fcc835ad02.1532310848@django-todo> in the header. Why not just use this notif-19 and add the comment to Task 19? Do you expect, that users first forward the email somewhere else, or reply to someone else before replying back to server? I do not see where the id could be lost.

Maybe I don't understand your workflow

I think so, but I'll do my best to make it clearer.

when the track_id is all we need, and its within the header, why not just use it?

Because it's not always within the headers. The task_id is only there when answering an email notification. If the tracker is suscribed to a mailing list, all threads will be separate tasks, each answer being a comment. In this case, there are no notifications involved at all !

I expect users just to answer to the comment notification, which will send an email to the server with something like In-Reply-To: <notif-19.164355fcc835ad02.1532310848@django-todo> in the header.

That's not the only usecase: somebody may send an email to the tracker, get no notification at all (just like your regular contact@company.com), somebody may answer that same email, and as long as the tracker gets the answer too, it'll get attached back to the task.
The mail tracker isn't just about enabling email notification answers. It's about tracking any email. If you want to add knobs to restrict the scope of what it can do, please do, it sounds like a useful thing !

I see that I have a different usecase at all for the todo tool. I wont use it in way as a client helpdesk, more like the github issue system and the email notification and answering feature is just an easy way so people do not have to login before adding a task or reacting to comments. And I want to keep it simple, thats why I chose the tool in the first way.  So I guess I will need to fork this for my own approaches.

@ezzra I would suggest adding a feature flag to make it fit what you want. The patch probably is under 10 lines, and the existing code was designed to be very modular.
The internet doesn't need one more fork, the simple way is the unified way.
If you create one more fork:

you won't benefit from security fixes like this or that
you won't benefit from improvements of the shared codebase
code won't go through as much review as with a common project (probably the most important item)

I wont use it in way as a client helpdesk, more like the github issue system and the email notification and answering feature is just an easy way so people do not have to login before adding a task or reacting to comments.

The existing code can already do that, and since comments can't be edited for now, being logged in doesn't change anything.
I guess you miss the following features:

match the source email to django user emails (1 line of code)
reject the email if no user is found (2 to 5 lines of code)

The existing code just needs the few lines of glue required to do that. If you don't want to plug into the existing tracker thing, well, you can create another consumer and use it ! Just add a file in there, and here you go.
All the added features are optionnal, and I did my best to keep it simple. I know this feature isn't for everybody. That's why if you don't do anything, no (almost?) additionnal code runs.

@ezzra  - @multun and I had quite a bit of back-and-forth during code review about the same issue you raised - that anyone knowing the email addr can successfully post a task or comment into the system, and the spam possibilities that arise from that. I was very reluctant to allow that to happen.
Multun convinced me that, in a way, the "support" queue is really just shadowing an email address, and anyone can already write  to any email address on the internet without proving themselves first. And I get the need for a feature like this to exist in an org doing public support, especially with multiple people doing support / sharing a mailbox.
That said, I can see the need to add features/settings for disabling this aspect of the system, or for blocking spammy sender addresses (but beware - it would be all too easy to get into the zillion hassles of email spam handling).
I think a specific issue for that feature should be opened if you feel you need it, rather than this generic "Mailtracker issues" thread.
Also note that todo has long supported the general concept of letting unauthenticated users post support issues. This Mailtracker feature is much more than that. Important to keep their use cases conceptually separated.

Of course Iam totally aware of the disadvantages of forking, but I do depend on the advantages.
However, it looks like you can force a duplicate entry error for UNIQUE | task_id, email_message_id if you send twice an email with the same message-id for the same task, just fyi

@ezzra A PR is in and almost merged for the MySQL compatibility issue. Based on the discussion above, are there any other items from your original list that you feel still need to be addressed? (I'll want to make a separate ticket for each, or you can). If not, I'd like to close this. Thanks for the helpful feedback!

Closing this for now, but would love to hear confirmation from your or another mysql user that all is well with the 2.3 upgrade on mysql.
And feel free to open individual tickets if any of your issues in the OP still need addressing.
Contributions always welcome!


can't swipe item in Flatlist with one finger on Android

iOS works as expected but on Android I need to use two finger to be able to swipe item in Flatlist
https://user-images.githubusercontent.com/115603511/219297795-825f6796-02bb-46e9-9232-9710e0be5e08.mp4

Weird, I can look into this. Can you confirm first if you are using this approach to make it work with FlatList?

Hi @intergalacticspacehighway thanks for providing a demo with a ScrollView.
I think there's something when the PanHandlers activate. The solution would be to give an active offset: activeOffsetX: [-10, 10],
before the Pan triggers onUpdate()
See it doesn't scroll with spacers around the ScrollView:
https://user-images.githubusercontent.com/1864359/233719163-b0e4fb1f-ccb6-4989-b48a-d560f0942185.mov
<>
      <Text
        style={{
          height: 200,
        }}>
        Spacer 1
      </Text>
      <ZoomFlatList
        data={data}
        pagingEnabled
        horizontal
        keyExtractor={item => item}
        renderItem={renderItem}
      />
      <Text
        style={{
          height: 300,
        }}>
        Spacer end
      </Text>
    </>
    ```

Help please

yes, can only scroll by 2 finger


MIS - questions about the general program inputs and operators

What does your Hamiltonian look like? Is it a matrix or an array? What are its dimensions? Same for the lagrangian.
Section 8.4.4 under exception for verify_Input(). What does NotReal mean - do you mean that the function returns an integer? Also does the special character "I" have any meaning (after there exists symbol)? What does it mean for a string input to be out of bounds?
It was my understanding that we had to put the variables into the parentheses of each function in the access routine semantics. For example, the verify_Input function accepts a str and an int as inputs, but I don't see an integer or str in the function description. I also don't know what the int or str represent.

@PeaWagon @smiths
Thanks for the great feedback!


Clicking "Prepare sequencing pool" button on "Pool list" page gives 404 error

Do this:

... and click "Prepare sequencing pool", and you get this:

This isn't an issue with any particular pool or set of pools; it is because there is no handler defined for "/process/pool" :(
I'm guessing that this button-press should actually route to "/process/poolpools$', to be consumed by the gui/handlers/process_handlers/pooling_process.py PoolPoolProcessHandler , since that is where the "Prepare amplicon sequencing pool" menu item goes.  @tanaes @ElDeveloper do you concur?

@AmandaBirmingham, yes I think you are right!

Yes I believe this is the case! I think early on Jose had only considered
the single pooling process type.
On Tue, May 8, 2018 at 5:11 PM Amanda Birmingham<EMAIL_ADDRESS>wrote:

Do this:
[image: screen shot 2018-05-02 at 11 07 40 am]
https://user-images.githubusercontent.com/10677935/39788847-2fd2b674-52e1-11e8-89fc-0997143aae70.png
... and click "Prepare sequencing pool", and you get this:
[image: screen shot 2018-05-02 at 11 07 48 am]
https://user-images.githubusercontent.com/10677935/39788863-4483a330-52e1-11e8-98df-c1075813fdd2.png
This isn't an issue with any particular pool; it is because there is no
handler defined for "/process/pool" :(
I'm guessing that this button-press should actually route to
"/process/poolpools$', to be consumed by the
gui/handlers/process_handlers/pooling_process.py PoolPoolProcessHandler ,
since that is where the "Prepare amplicon sequencing pool" menu item goes.
@tanaes https://github.com/tanaes @ElDeveloper
https://github.com/ElDeveloper do you concur?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jdereus/labman/issues/232, or mute the thread
https://github.com/notifications/unsubscribe-auth/AH6JAPU2-X9PmmAIp4RUOyoIE9YWWUpTks5twjQbgaJpZM4T3hOw
.

It looks like when the links where changed to "/process/poolpools/" and "/process/poollibraries/", the logic behind that button wasn't updated. Should be a change on the JS.

Fixed with #236


RSA Encrypted keys not recognized

While using the -i option , if the private key is encrypted , teleconsole fails to connect .
Error:
asn1: structure error: tags don't match (16 vs {class:3 tag:1 length:76 isCompound:false}) {optional:false explicit:false application:false defaultValue:<nil> tag:<nil> stringType:0 timeType:0 set:false omitEmpty:false} pkcs1PrivateKey @2
Found this as a solution to a similar problem in another application (Solution)
Do DecryptPEMBlock() like this after pem.Decode()
https://golang.org/src/crypto/x509/pem_decrypt_test.go#L15
Also, use IsEncryptedPEMBlock() to determine if decryption is needed.

On a related note to PR #40 , I think it's preferable for the session provider to only use the public key with the -i switch (which does not need a passphrase). In that case, reading encrypted private keys would be moved to the responsibility of the ssh client on the session requester side, i.e. not within the scope of teleconsole -i ... anymore


Release notes links point to module rather than section

I would expected the links in the release notes to point to the section in the notes rather than to the module docs.
E.g. to https://docs.scipy.org/doc/scipy/reference/release.1.4.0.html#scipy-stats-improvements
rather than to https://docs.scipy.org/doc/scipy/reference/stats.html#module-scipy.stats
I'm happy to work on a fix if this is not intentional and others would also expect  the behaviour I had.

This confused me for a little while. I now see what you mean:

The sidebar on the right has the links to the doc sections of the release notes, the ones on the left are to the refguide module overview pages. The reason for the latter is that they're the links from the section headers themselves:

Making those links point to themselves makes little sense though. So probably the right thing to do here is to remove the link from the section headers completely, and make the main ToC a normal Sphinx TOC - that will then have the same links as in the right sidebar.

oh, somehow I never noticed the sidebar, so much focused on the middle content 😅

probably the right thing to do here is to remove the link from the section headers completely, and make the main ToC a normal Sphinx TOC - that will then have the same links as in the right sidebar.

@rgommers do you still want something done here? The contents entries still link to the module docs, but we no longer have the links per module in the right-sidebar.

This looks reasonable to me now:

I'd be fine with closing it.

Yeah, I agree with @bsipocz that it seems unintuitive for the links to go to the module ref docs, but probably only worth changing if there's something easy and clearly better. Any thoughts @bsipocz ?


allow load tools pass more params to searx-search tools

[x] "community: allow load tools pass more params to searx-search tools"

[x] PR message:

Description: allow load tools pass more params to searx-search tools
load_tools can olny pass flew arguments to SearxSearchWrapper, I need pass language, categories and headers, and found the code just pass few to SearxSearchWrapper

has been in draft for a while - will close for now, and feel free to reopen as a full pr for a review!
Also, if you're interested in maintaining this integration without us in the loop, we'd love to get an integration package out! Future PRs against langchain would just be {docs updates, as well as registering your package in libs/packages.yml, deprecating this community integration in favor of your integration package}
Here's the guide, and if you have questions, feel free to leave them in the comments on those pages so others can see them! https://python.langchain.com/docs/contributing/how_to/integrations/


chore: release 1.2.95

:robot: I have created a release *beep* *boop*
1.2.95 (2022-05-04)
Bug Fixes

anchors support for extra annotations and labels post-renderer (b8211a9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.95 :sunflower:


WebDriverException

My browser got updated to FireFox 47 and I'v been facing this problem ever since. Is there no way to resolve this without rolling back to older version of FF?
raise WebDriverException("The browser appears to have exited "
selenium.common.exceptions.WebDriverException: Message: The browser appears to have exited before we could connect. If you specified a log_file in the FirefoxBinary constructor, check it for details.

For what it is worth - everything works for me with Firefox 61.

Also, is Selenium really relying on a system browser?

it depends on the platform you running on, for firefox can specify binary you want to use in constructor see https://github.com/SeleniumHQ/selenium/blob/master/py/selenium/webdriver/firefox/webdriver.py

I have a similar problem it seems.
I get this error.
Traceback (most recent call last):
File "search-google.py", line 81, in
main()
File "search-google.py", line 60, in main
br = start_browser()
File "search-google.py", line 23, in start_browser
br = webdriver.Firefox()
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/selenium/webdriver/firefox/webdriver.py", line 80, in init
self.binary, timeout)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/selenium/webdriver/firefox/extension_connection.py", line 52, in init
self.binary.launch_browser(self.profile, timeout=timeout)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/selenium/webdriver/firefox/firefox_binary.py", line 68, in launch_browser
self._wait_until_connectable(timeout=timeout)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/selenium/webdriver/firefox/firefox_binary.py", line 108, in _wait_until_connectable
% (self.profile.path))
selenium.common.exceptions.WebDriverException: Message: Can't load the profile. Profile Dir: /var/folders/nl/c4hxb7h51qj9_yc726n_19zm0000gn/T/tmpPvRyeN If you specified a log_file in the FirefoxBinary constructor, check it for details.


[WT-447] Add ability to upload images on event creation and store in google cloud

Description
This pull uses google cloud storage to add the ability to upload images to events on creation. On upload, the file is uploaded to the google cloud storage, where a link is generated and uses the existing URL input field and database column to keep track of the event images.
Risks
Uploaded image links are public to anyone with the link
Validation
Inserted custom image to create event and inspected events page

Issue
https://github.com/WonderTix/WonderTix/issues/447
Operating System
Windows 11

This functionally looks like it works pretty well! Can approve once my other comments are addressed


Xa lint

Lint the collectors, the easy parts

first commit: rename functions so that they are camelCase,
second commit: easy small fixes.

There are tons of other fixes which are needed, mainly comments, but not only.

Renaming these is problematic because because people use the -f cli to filter certain collectors. So I don't think fixing lint or dual naming things is really worth it.

I only have renamed the functions name in go, not their 'name:' field for those who have one.
| -                                     return c_fastly(client)
| +                                     return cFastly(client)
|                               },
|                               name:     "c_fastly",

So our users will use the tool the same way, but contributors who use golint will have a less hard time.


Using space as delimiter

hi I'm keen to use your plugin (v1.4.1) using an Elasticsearch datasource (v7.16) on Grafana v8.3.4. In the output the row and column names are seperated by a space but the plugin doesn't seem to accept or process the space when placed in the delimiter input box. In fact when I return to the options, the space seems to be removed. How do I enter a space as delimiter?
Since the series also contain underscores I have, as a test, used underscore as the delimiter and that works great but is obviously not what i need.

Let me know what else you need. Thanks in advance for your feedback.

Hello.
You can use the grafana transformations to rename your metric.
I also couldn't use a space.


[WebToolsE2E][Aspire] On VS 17.10 GA, opening eShop.web fails with error: "Unable to find a stable package Microsoft.NET.ILLink.Tasks with version (>= 8.0.5)", Even though I have applied the feed of sdk 8.0.300.

REGRESSION INFO: Worked fine on VS 17.10 GA [34908.272.d17.10] + 1.10.0-beta.1-daily.3768065 + Aspire 8.0.0 (older machine)
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.10 GA[34910.138.d17.10] [ Includes Aspire 8.0.0 and SDK 8.0.300 ]
Apply NuGet darc-pub-dotnet-aspire Feed
Apply sdk 8.0.300 feed
Install cred helper
Install Docker Desktop for Windows
Install latest azd [1.10.0-beta.1-daily.3780018]

REPRO STEPS

Clone eShop repo from main bracnh (Aspire 8.0 P7)
Open eShop.Web.slnf
Open global.json and update sdk version to 8.0.300
Build

ACTUAL
NU1103	Unable to find a stable package Microsoft.NET.ILLink.Tasks with version (>= 8.0.5)
Found 21 version(s) in nuget [ Nearest version: 8.0.100-1.23067.1 ]
Versions from C:\Program Files\dotnet\library-packs were not considered

EXPECTED
Opening eShop solution without any errors (Aspire 8.0 P7)

Similar to issue https://github.com/dotnet/eShop/issues/370

Not repro on VS 17.10 GA FB + latest daily azd [1.10.0-beta.1-daily.3787247]

Closed it as not repro anymore


[WFCORE-4935] if server is already in suspend mode, don't register  any listener, just shut it down.

Issue: https://issues.redhat.com/browse/WFCORE-4935
If server is already in suspend mode, don't register any listener, just shut it down. Otherwise, a suspend mode started server won't be shutdown before resuming.

@soul2zimate thanks


doc: add output doc

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

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


rkit compiled for python 2.7 wont run

I am using Manjaro I3 (Arch) and compile rdkit from source as described here https://www.rdkit.org/docs/Install.html
I also installed boost as was described (version boost_1_67_0).
i installed rdkit fro python 2.7, by using the command:
"cmake -D PYTHON_LIBRARY=/usr/lib/python2.7/config/libpython2.7.so -D PYTHON_INCLUDE_DIR=/usr/include/python2.7/ -D PYTHON_EXECUTABLE=/usr/bin/python2.7 -DBOOST_ROOT=/usr/local/boost_1_67_0 .."

RDKit Version: rdkit-Release_2018_03_4

"import rdkit" in python 2.7 produces:

import rdkit
Traceback (most recent call last):
File "", line 1, in
File "/home/moritz/rdkit-Release_2018_03_4/rdkit/init.py", line 2, in
from .rdBase import rdkitVersion as version
ImportError: libRDKitRDBoost.so.1: cannot open shared object file: No such file or directory

I'm guessing that the problem is that you haven't set your LD_LIBRARY_PATH to point to $RDBASE/lib
-greg

in the cmake command?

No, in the shell where you are running python.
The environment variables you need to set are described here: https://www.rdkit.org/docs/Install.html#building-the-rdkit

And just a general comment: unless you have a specific reason to want to compile the RDKit from source, it's almost always easier to install/use it from inside anaconda python.

i tried installing it with miniconda and it did not work.
i executed LD_LIBRARY_PATH=/.../rdkit-Release_2018_03_4/lib/
and it did not change a thing.
what i forgot to mention in the original post is, that i have to run:
"import sys"
"sys.path.append("$RDBASE/")"
otherwise python just sayes :
">>> rdkit
Traceback (most recent call last):
File "", line 1, in
NameError: name 'rdkit' is not defined"

i recompiled the whole thing with the environment variables set correctly and i still get the same error

is there a support page where i can ask for help or something? i have no idea what i am doing wrong. i checked the enviroment variables twice and recompiled everything now.

You could try the rdkit-discuss mailing list (https://sourceforge.net/projects/rdkit/lists/rdkit-discuss). Things like this aren't easy to diagnose remotely
But if you installed the rdkit with "conda install" into a conda environment, activated that environment, and it still didn't work then something deeper is going on. What's the error message there?


Make throttle-debounce function accept arguments

Add generic type definition for ...args, that can be passed to throttle or debounce function callback

@GAntoine done.


added baseurl, fixes #40

@daviddarnes thanks for the help, this worked great!
There is a minor issue where images are not linked properly, the user has to append the baseurl to the image path. I changed line2 of site-feature.html to this:
{% assign feature_image = site.baseurl | append: page.feature_image %}

Which solves the issue for me but doesn't work if the feature_image is a web link (my images are all local).
I'm not sure the best way around this since liquid doesn't support regex.

@zlskidmore awesome! Glad to hear it, and thanks for contributing. I think this is what I was looking for https://jekyllrb.com/news/2016/10/06/jekyll-3-3-is-here/#2-relative_url-and-absolute_url-filters
Would you be able to test for any instances where this is needed? I think it's more than just the feature image that needs the filter

@daviddarnes I think I've hit all the features, with the following exceptions:

I have not been able to get the category links to work in post-meta.html, I don't understand how localhost or the site url is being appended there.

please take a look at the site-search.html functionality, adding site.baseurl causes you to have to double click to go to the search result, i'm not sure why.

I don't have a disqus account, not sure if those url's would need to be changed but I didn't touch them as I have no way to test without an account.

Apologies for  the above issues. If you have any tips for points 1 and 2 I'd be happy to take another look.

This is awesome work, thanks so much! I'll take a look when I can. I need to test the flat version as well as the theme.

Weird that category links aren't working right, I'll take a look
That site search clicking is a different bug entirely 😅, I really need to fix that
As long as the urls in the disqus match the current ones then it should be fine

Don't be sorry for not getting them all, I'm thankful that you took the time to contribute

I'm going to close this pull request. Thanks a lot @zlskidmore for contributing, however I think it would be easier if people just put the whole url into site.url rather than trying to assemble it with site.url and site.baseurl. It means the theme code remains clean and it allows people to use external and internal links with ease

no problem, thanks for the help!
On 4/6/17 3:08 PM, David Darnes wrote:
I'm going to close this pull request. Thanks a lot @zlskidmorehttps://github.com/zlskidmore for contributing, however I think it would be easier if people just put the whole url into site.url rather than trying to assemble it with site.url and site.baseurl. It means the theme code remains clean and it allows people to use external and internal links with ease
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/daviddarnes/alembic/pull/42#issuecomment-292300784, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AF3GnDRYkxocQskaXXBF9STqhFdkeFoeks5rtUYrgaJpZM4M0dJf.

The materials in this message are private and may contain Protected Healthcare Information or other information of a sensitive nature. If you are not the intended recipient, be advised that any unauthorized use, disclosure, copying or the taking of any action in reliance on the contents of this information is strictly prohibited. If you have received this email in error, please immediately notify the sender via telephone or return mail.


Include a bookmark icon on the reader screen

Hey team! Please add your planning poker estimate with ZenHub @isabellamoak @vstevens14


Collapse and expand module?

I setup waybar but there's a use case that I need each module or at leat top most right and top most left has collapse/expand button like > (for the top most right) that indicate to expand.
We can also use first candidate in the module array as > (collapse/expand button)
And we can expand/collapse using mouse click or combined-keys  press.
How can we do that with waybar?

I would like this too. After adding the mpris media player module, my bar is very full on my laptop. Some way to collapse the items you dont need to see constantly would be great.


Add VMware catalog parsing

Purpose or Intent
The VMware catalog provides descriptors for vApp templates and
corresponding VM images. The hierarchy of the catalog is
organization -> catalogs -> catalog_items -> vapp_template -> vms
This patch traverses through the entire chain and collects all available
VM images. Public and private catalogs are available so we also
introduce a setting to allow enablement of import of both types of
catalogs (by default, only private catalogs are imported).
Spec file has been updated to test presence of catalog data. Some
limitations on the available data from VMware had to be considered (for
example, no info about the OS, bitness
@miq-bot add_label providers/vmware/cloud, enhancement, wip, gem changes

@agrare going to make some changes to VirtualTemplate. Hold off on adding it

@miq-bot remove_label wip

@gberginc please rebase and push again

@chessbyte Done.

@gberginc looks good :+1:
Just curious, what does it mean for a vapp_template to not be VAPP_TEMPLATE_STATUS_READY?

@miq-bot add-label darga/no
@miq-bot assign chessbyte


refactoring: reuse workloadinstances.Index in serviceregistry/serviceentry

Signed-off-by: Yaroslav Skopets<EMAIL_ADDRESS>Please provide a description of this PR:
This PR refactors serviceregistry/serviceentry to reuse the same code as serviceregistry/kube.

/test integ-security-multicluster_istio

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
01b91b3d210f470d3fd53cac37859b145f14439e
link
unknown
/test integ-cni_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
01b91b3d210f470d3fd53cac37859b145f14439e
link
unknown
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test gencheck_istio

/retest


V3 Tooltip Unresponsive

Expected Behavior
The V3 tooltip should appear quickly on hovering the cursor over the chart area. It should disappear when the cursor is moved off of the chart area.
Current Behavior
The V3 tooltip is slow to appear, or does not appear at all when hovering the cursor over a chart. When the cursor is moved off of the chart area, sometimes the tooltip persists, as shown in the screenshot below.

Steps to Reproduce
The following demos can be used to reproduce the issue. The first, using version 2.9.4 performs as expected - the tooltip appears immediately when the cursor is hovered over the chart. The second demo is the same demo converted to 3.0.0-beta.7. The changes needed for the conversion from V2 to V3 were minimal.
Codepen Chart.js V2 Demo
Codepen Chart.js V3 Demo
The relevant tooltip config in the above V3 demo is below.
Chart.defaults.plugins.tooltip.mode = 'index';
Chart.defaults.plugins.tooltip.intersect = false;
Chart.defaults.plugins.tooltip.callbacks.title = function(tooltipItem) {
  return moment(tooltipItem[0].dataPoint.x * 1000).format("DD-MMMM-YYYY h:mm A");
};

Environment

Chart.js version: 3.0.0-beta.7
Browser name and version: Chrome 87.0.4280.88 and Safari 14.0.2

For some reason in v3, I have to click on the chart to recognize the clicks. @kurkle it almost feels like something got shared between the chart instances.

For some reason in v3, I have to click on the chart to recognize the clicks. @kurkle it almost feels like something got shared between the chart instances.

Does not reproduce in master: https://codepen.io/kurkle/pen/XWjBQYY
I'm not quite sure where this was fixed. Could be one of: #8252 #8253 #8254

Does not reproduce in master: https://codepen.io/kurkle/pen/XWjBQYY
I'm not quite sure where this was fixed. Could be one of: #8252 #8253 #8254

Can confirm master V3 seems to be working as expected, thanks!
Next time I will be sure to test the latest master. 🙂

Can confirm master V3 seems to be working as expected, thanks!
Next time I will be sure to test the latest master. 🙂