[Help] The results about the sdvrp environment

Hi, When I test the sdvrp env, I found the results is not normal, when i setting the Env config below, Basiclly i think the vehicle should revisited the same target location for many times, while the results and printed actions shows that each target location only be visited once.
Plus, when i setting the min_demand to 0 and train the model , it sometimes happened error for the start_loc demand will be the negative and raise the error.
We are  are appreciated with your work and waiting for your reply，it will be better if there will be more tutorial and example about how to load the trained model for that we are unfamiliar with your structure.
sdvrpenv = SDVRPEnv(num_loc=20,
min_loc=0,
max_loc=1,
min_demand=1,
max_demand=10,
vehicle_capacity=1.0)

Thanks for your interest!
First of all, we made some small fixes so that checkpointing works well for notebooks too (we made checkpoint loading mostly based on Hydra, but we would like it to as accessible for anyone to use!). Make sure to check the updated notebook here, where we added more comments as well as checkpoint saving and loading, logging and more testing! :)
Now about your questions:

Basically i think the vehicle should revisited the same target location for many times, while the results and printed actions shows that each target location only be visited once.
We could not reproduce the result, could you provide more context? For example, in the notebook above, if we print the duplicate actions we obtain:

# Find duplicate actions for each batch and print them
for actions in out["actions"]:
    unique_elements, counts = torch.unique(actions, return_counts=True)
    duplicates = unique_elements[counts > 1] # exclude visited once only
    visited_more_than_once = duplicates[duplicates != 0] # exclude depot
    print(f"Number of duplicate actions: {visited_more_than_once}")

Number of duplicate actions: tensor([3, 5], device='cuda:0')
Number of duplicate actions: tensor([10, 16, 17], device='cuda:0')
Number of duplicate actions: tensor([ 2, 17], device='cuda:0')

you may also see it from the plot

Plus, when i setting the min_demand to 0 and train the model , it sometimes happened error for the start_loc demand will be the negative and raise the error.

You are correct. If min_demand <= 0 some problems may arise, because it does not make sense from the problem's standpoint (e.g. if demand  of a node is 0, it is always suboptimal to visit it since there is no point in going there). We may add some check like assert  min_demand > 0. Thanks for pointing this out!

Closing since answered - feel free to re-open should you have any further issues ;)


Implement estimated reading time in Support section

So I was thinking that adding an estimated reading time (i.e. 2 min read) might encourage viewers to actually read the guides. I saw some blogs on how to do it on Jekyll:
Jekyll reading time without plugins
reading time
simple reading speed estimate in jekyll
No rush on this!

Done and implemented beautifully!


Question concerning  and <\s> tokens

Note that [ and ] stands for < and > in this comment since arrows are used for formatting.
Based on the tutorial, the data pipeline is set up ending in the following line of code:
((source, source_lengths), (target, target_lengths)) = batched_iterator.get_next()
where source is a padded batch and target is a padded batch (no [s] or [/s] tokens appended)
However, placeholders are described at the top of the tutorial as if they already have these tokens:
    encoder_inputs [max_encoder_time, batch_size]: source input words.
    decoder_inputs [max_decoder_time, batch_size]: target input words.
    decoder_outputs [max_decoder_time, batch_size]: target output words, these are decoder_inputs shifted to the left by one time step with an end-of-sentence tag appended on the right.

or see image: https://github.com/tensorflow/nmt/blob/master/nmt/g3doc/img/seq2seq.jpg
How should the start/end tokens be appended? It seems like decoder_inputs should be shifted to the right and receive a [s] tag at the beginning before being embedded, and decoder_outputs should receive a [/s] tag at the end, but this is never mentioned in the explanation.

@nave01314 The inputs and outputs preparations are implemented in iterator_utils.py as linked in the tutorial.
Specifically, The decoder inputs and outputs are shifted here.

Thank you! It might be useful to leave a mention in the Data Preparation section of the explanation to clarify that Tensorflow does not intrinsically append tokens as part of the encoding/decoding process.


Enhancement: human readable sdns:// stamps

Any chance of changing the sdns:// stamps to something more human-readable?
sdns://resolvername/providername=xxxx?protocol=dnscrypt&addr=xxxx&port=xxxx&publickey=xxxx&dnssec=yes&log=no&block=no
resolvername is e.g. "scaleway-fr"
providername is e.g. 2.dnscrypt-cert.fr.dnscrypt.org
protocol is dnscrypt-v1, dnscrypt-v2, or doh
addr is the ip address
port is the port number
publickey is the server public key
dnssec/log/block are the props
and so on.
for example, compare these two
sdns://AQcAAAAAAAAADjIxMi40Ny4yMjguMTM2IOgBuE6mBr-wusDOQ0RbsV66ZLAvo8SqMa4QY2oHkDJNHzIuZG5zY3J5cHQtY2VydC5mci5kbnNjcnlwdC5vcmc
vs
sdns://scaleway-fr/providername=2.dnscrypt-cert.fr.dnscrypt.org?protocol=dnscrypt-v2&addr=<IP_ADDRESS>&port=443&publickey=e801b84ea606bfb0bac0ce43445bb15eba64b02fa3c4aa31ae10636a0790324d&dnssec=yes&log=no&block=no
This is just an example. I like to be able to read strings I can understand rather than a bunch of meaningless characters. Any input welcome on this.

No.

In public-resolvers.md, we should note port of server, because I need port number to whitelist the firewall.


Fix netutils tests

openshift-sdn's (minimal) tests fail now because pkg/netutils/common.go now requires kubernetes's pkg/util/errors. So this just copies that into Godeps. I didn't bother trying to do it "properly". Maybe we should just rm Godeps/Godeps.json and Godeps/Readme?
@pravisankar

Other option is to move GetHostIPNetworks() to pkg/ovssubnet/common.go (used only in this file).
I'm okay with either copying kerrors file or moving the method.
Currently, hack/sync-to-origin.sh checks Godeps/Godeps.json to confirm the current repo belongs to openshift-sdn. We need to patch the script if we plan to get rid of Godeps.json.

Other option is to move GetHostIPNetworks() to pkg/ovssubnet/common.go

I'm still assuming we're eventually moving all this to origin, so functions should stay where they make the most sense


[Event Request] Table 179 "Reversal Entry" - procedure Caption()

Describe the request
I would need a new event in function Caption of table 179 Reversal Entry.
The event should be placed at the beginning of the function (for example onBeforeCaption) and as parameters it should have the Rec (Reversal Entry), the IsHandled variable and a return parameter that contains the new Caption.
Exited from the event, if IsHandled = true it must exit the function by placing the new caption in the exit
for example :
onBeforeCaption(Rec, NewCaption, IsHandled)
if IsHandled then
exit(NewCaption)
thank you
Additional context
For one of our apps we need to modify the caption that is displayed on page 183 Reverse Transaction Entries
Internal work item: AB#524158

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


Update emojis.json

New Keyword were added in emojis.json file so it's strength is improve .

Update emojis.json file :+1:


UI doesn't scale to the size of the output image

How to reproduce:

Run fastsdcpu on Linux (haven't tested on Windows)
Change width & height of generated image to 768
Generate image and observe that only a portion of the image is displayed in the UI.

Actual output result:

@veikk0 It is fixed in the master branch

Thanks, it works now.


contextMenuService not showing the menus in angular 11

I have updated my application from angular 8 to 11.
ngx-contextmenu is not showing the menu through,contextMenuService .
        this.contextMenuService.show.next({
I have updated the packages version to recent also but still context menu is not showing up.
Any help would be really appreciated!!
Angular: 11
CDK/Material<EMAIL_ADDRESS>Browser(s): Chrome and firefox
Operating System ( Ubuntu):

This and #236 seem to be related to https://stackoverflow.com/questions/60432992/expression-has-changed-after-it-was-checked-using-viewchild-reference
I haven’t been able to reproduce outside of our app, but it seems to be related to detect changes not detecting changes as well as possibly a similar error message mentioned in the SO link.

This and #236 seem to be related to https://stackoverflow.com/questions/60432992/expression-has-changed-after-it-was-checked-using-viewchild-reference

No, doing ViewChild with static=true seems to fix the error message, but not the menu coming up.

We were using too old a version of angular material.  Upgrade fixed it.

what version did you used for angular 11?


chore: minor improvement to the kibana_alerting_rule docs

Minor improvement to the new kibana_alerting_rule documentation specifying how to get the valid rule_types.

❤ seeing contributions like this 🎉


Update to JuMP v0.23

Closes #222

Sorry for the changes :) But I think they are an improvement

Sorry for the changes :) But I think they are an improvement

No worries, this isn't too bad. I am just very excited that you secured funding to work on revamping the nonlinear side of things!

@odow I noticed that JuMP v0.23 retains functions like nl_expr_string and nl_constraint_string. Were these intentionally not renamed with nonlinear_?

@odow I noticed that JuMP v0.23 retains functions like nl_expr_string and nl_constraint_string. Were these intentionally not renamed with nonlinear_?

Ah damn. Looks like my find-and-replace-foo skills need work. I must have searched the documentation because these don't s. Can you open an issue with an example of how you're over-riding it? I'm not sure if it should be public. It takes a private argument:
https://github.com/jump-dev/JuMP.jl/blob/e7ffb7c01c94aecf6d6ad45f5ced6540499aa7e1/src/print.jl#L369-L390

@odow I noticed that JuMP v0.23 retains functions like nl_expr_string and nl_constraint_string. Were these intentionally not renamed with nonlinear_?

Ah damn. Looks like my find-and-replace-foo skills need work. I must have searched the documentation because these don't s. Can you open an issue with an example of how you're over-riding it? I'm not sure if it should be public. It takes a private argument: https://github.com/jump-dev/JuMP.jl/blob/e7ffb7c01c94aecf6d6ad45f5ced6540499aa7e1/src/print.jl#L369-L390

I just use them in my tests as a quick and dirty way to check that I am creating nonlinear expressions correctly in the JuMP model. I do not extend them.

But yes, I can open an issue.

Opened https://github.com/jump-dev/JuMP.jl/issues/2892 to address this


Broken Debian/Ubuntu repositories

Thank you for providing nvidia-container Debian packages.
However by following the instructions on the official site I am not able to fetch the list of packages.
By navigating to: https://nvidia.github.io/libnvidia-container/stable/ubuntu18.04/  I get # Unsupported distribution! # Check https://nvidia.github.io/libnvidia-container

There is no listing available from the webssrver. That is the standard 404 message. If you really need to see what is available you can look directly in the gh-pages branch of the repo:
https://github.com/NVIDIA/libnvidia-container/tree/gh-pages/stable/ubuntu18.04


Segmentation fault

When I used the gbm_dtrees example for vcu118, I compiled the hw section and installed the driver
insmod coyote_drv.ko
A segment error has occurred

Image first needs to be loaded, then the driver.


Apple rejection guideline 2.5.1 (#1)

Guideline 2.5.1 - Performance - Software Requirements Your app uses or references the following non-public APIs
Removed App-Prefs:root=Privacy&path=LOCATION and switched to UIApplicationOpenSettingsURLString so that the app doesn't get rejected during Apple Store Review.

When this PR will get merged?

Currently, I'm pointing to this master and  building the app.


Extend preprocessing options

The set of available preprocessing options defined in the config .yaml files can be expanded to include all available options in SpikeInterface. This should be simple, except for bad channel detection / removal which is sensitive to the pre-processing steps it is performed after, and will require more care. Finally, this is a good time to check the problems caused by most recent spikeinterface version API changes.
This issue subsumes #33, #18

These issues have been fixed in #50. However, some issues arose meaning not all preprocessing steps were added, will make a new issue for this.


Secondary Temporary Directory

Hi,
First of all, thanks for your hard work! It's awesome.
I found out that almost all third party plotter missing the function of secondary temp directory. I know most people rarely use it but it works great on my particular setup (I have two machine running with 2nd temp, and it smooths out the stress on the NVME). So I'm wondering if it is possible to add that option to the GUI?
Thanks again.

This is the next feature I am working on adding. I have actually already added it, but have not merged the branch yet because I need to do some testing. I am also adding changing KSizes, Saving jobs, basic directory input option in this update as well. You can check out the branch here - https://github.com/MrPig91/PSChiaPlotter/tree/Add2ndTempDrive

This has now been added. A lot was added to the latest update so please report any issues with it. The feature requested is in version 1.0.32 and above.


How many files were available during pre-training on WebVid+CC3M?

Thank you for sharing this code.
I have reproduced pre-training by following GitHub script.
I got below results which are much lower than those of the paper.
(zero-shot results on MSRVTT with WebVid+CC3M in the paper: 28.4, 50.2, 59.5)
txt_r1  txt_r5  txt_r10  txt_r_mean  img_r1  img_r5  img_r10  img_r_mean  r_mean
msrvtt_1k_test/        22.3    40.3     49.3        37.3    25.6    46.0     55.8       42.47   39.88
msrvtt_1k_test_emb/    17.8    34.6     43.9        32.1    20.3    40.8     51.5       37.53   34.82
I thought it was caused by lack of data which was not available to download.
How many files were available during pre-training on WebVid+CC3M?

We have data statistics in the appendix that matches exactly the number of files used in the paper. If you have similar #files, You can probably check #frames used etc.

I checked that my CC3M dataset has only 2.18M compared to 2.95M of yours.
I should check the download process.
Thank you!


UCE writeback sequence should wait for the write response

Currently, during a writeback, UCE sends a read commands followed by a write command to writeback the dirty data. But after sending these commands it only waits for the read response before jumping back to the ready state. This could cause the late write response from the cache to happen at the same time as the response for the next instruction and break the top-level arbiter between cache and other devices.

We shouldn't wait on writebacks (performance hit) but we should fix the response arbitration.
The sequence here is:

WB read

WB write

IO read

WB read response

WB write response / IO read response

We should write a basic test for this, also

What's the point of the wb write response? Where is it used?

credit counting for fences

Seems like a warning sign about this module that there is some concurrency
that has not been thought through?
On Mon, May 4, 2020 at 6:48 PM Farzam Gilani<EMAIL_ADDRESS>wrote:

Currently, during a writeback, UCE sends a read commands followed by a
write command to writeback the dirty data. But after sending these commands
it only waits for the read response before jumping back to the ready state.
This could cause the late write response from the cache to happen at the
same time as the response for the next instruction and break the top-level
arbiter between cache and other devices.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/475, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AFLRYDC67BCXYW2ODTRP5V55ANCNFSM4MZGLC7Q
.

Yeah, the problem is that responses from different destinations can return unexpectedly. The arbitration was written as one-hot because it was assumed there was only 1 outstanding request from each cache, but this does not hold when CLINT or I/O is involved

Sounds like in the comments there should be some explicit reasoning about
what concurrent events can happen. Arbiters dropping things under rare
concurrent events is common bug pattern that takes a long time to track
down due to the infrequency of such events.  Generally the assumption for
any module should be that if it has multiple interfaces, they are
concurrent; and when possible non-concurrent interfaces should be collapsed
into a single interface (particularly if the non-concurrent interface comes
from the same place).
Will update the style guide with this...
On Mon, May 4, 2020 at 10:53 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

Yeah, the problem is that responses from different destinations can return
unexpectedly. The arbitration was written as one-hot because it was assumed
there was only 1 outstanding request from each cache, but this does not
hold when CLINT or I/O is involved
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/475#issuecomment-623868742,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AAWOKYAW5MMY3LBUCTRP6SWFANCNFSM4MZGLC7Q
.

The bug isn't in the UCE.  The bug is in the arbitration here:
https://github.com/black-parrot/black-parrot/blob/master/bp_top/src/v/bp_softcore.v#L428
Note the (incorrect) comment that says the responses are guaranteed to be one-hot

Oh I see. Perhaps we should have a BSG_ASSERT_ONE_HOT(clk,signal) and a
BSG_ASSERT_ONE_HOT_OR_ZERO(clk,signal) in BaseJump STL to check one
hotness? There are some SVA primitives, but presumably we want to have
module-wide control over them for tool independence.
M
On Wed, May 6, 2020 at 3:44 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

The bug isn't in the UCE. The bug is in the arbitration here:
https://github.com/black-parrot/black-parrot/blob/master/bp_top/src/v/bp_softcore.v#L428
Note the (incorrect) comment that says the responses are guaranteed to be
one-hot
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/black-parrot/issues/475#issuecomment-624929397,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFG5AE7WXFT4ZHJGMSDFATRQHR6LANCNFSM4MZGLC7Q
.


Use Library without UI

Hi!
I would like to use the TWAIN library and be able to scan programmatically.  My goal is to encapsulate the scanning logic behind HTTP APIs to allow a Web Application to scan documents.  When I try to acquire the image, it throws an error saying "Cannot start UI".  Is what I'm trying to do possible?
Thanks!!

You will need to provide more information.

it throws an error

If your scanner allows the scanner to operate without a user interface, then it can be used in this manner, however the error you are stating is not one that DTWAIN generates.  DTWAIN does not generate the error "Cannot start UI".  What component generates this error?  Is it your scanner that generates the error?

DTWAIN is simply a DLL that can be called from any environment that can call exported DLL functions.  What computer language are you using?

this can be closed.  Sorry, I was using the llibrary incorrectly.  Its working now.  Thanks!!


3d prints do not affect light level of blocks

Even if the 3d print stretches either the entire x and z dimensions or entire z/x and y it does not decrease light level leaving a weird fullbright like effect in structures made fully of these prints

If I'm reading this correctly you're saying that prints act as transparent blocks, even if they are full opaque textures?
If so, I'm not sure there's a way to easily fix this, as a print can partially or fully consist of translucent/transparent textures. It would be non-trivial to determine which faces do not allow light through.


Question: smart auto-update

Issue: Chromium has a disk and bandwidth cost: (~170MB Mac, ~282MB Linux, ~280MB Win) and apparently one can only update the whole chromium package because updates do not come by little patches on top of the existing installation.
Proposal: It would be cool if the current installation was able to update just the least amount of bytes by fetching only the required diff instead of having to download the whole package each time.
Use-Case: This would be particularly useful in the case of Carlo applications when shipping them to production.
Request: Can chromium auto-update feature using "filesystem updates with file-level binary diffs" be made programmatically available in puppeteer-core ?

his would be particularly useful in the case of Carlo applications when shipping them to production.

@ab35 I though Carlo primarily targets stable chrome which already has auto-update. Am I missing anything?

Can chromium auto-update feature using "filesystem updates with file-level binary diffs" be made programmatically available in puppeteer-core ?

I had an impression that auto-update doesn't work for Chromium, or at least there's no infrastructure to support this. I might be wrong though.

@aslushnikov, targeting stable chrome is a great feature of Carlo, as it allows to quickly jump into development without thinking about downloading anything nor waiting for it: it works out of the box...
But for production applications built with Carlo, it sounds more reasonnable to use the built-in channel capability of using chromium instead, because not every user has Chrome installed on their computer...

I had an impression that auto-update doesn't work for Chromium, or at least there's no infrastructure to support this.

I do not really know... in fact I do not really know either how is Chrome updating itself, if it patches only files or if it downloads a whole new chrome version at each release. For millions of users, it should be quite a huge demand on the network !

@ab35 I see there's a downstream issue in Carlo; let's shift all discussion there.
Closing this in favor of https://github.com/GoogleChromeLabs/carlo/issues/92


Fixes for dashboard offers

Fixes an issue with timezone aware datetime objects (django.utils.timezone takes care of this)
When not all steps are completed in the wizard (eg directly save an edited offer) you would get a 'NoneType' object has no attribute 'save' exception. This is because there is no serialized version of these objects (yet) as these forms are not initialized when pressing 'save' in the first wizard step.
Added tests for pull request #1921

Lovely work, thanks.


Calculate mAp on custom data?

Can we calculate mAp for perclass with this code?

merging with https://github.com/zzh8829/yolov3-tf2/issues/125


Example usage

Thanks for putting together this plugin--I'm curious what the recommended approach for integrating this with our DAGs is. A simple example would be very much appreciated.

It's great to hear that @maxcountryman! A simple example of how to use this plugin to handle Airflow wide exceptions would be to add sentry_plugin/ folder into $AIRFLOW_HOME/plugins directory, and then setting your SENTRY_DSN in the environment variables. Then, Sentry will start catching exceptions that happen throughout Airflow.
An example of an error that would be handled is:
from airflow import DAG
from airflow.operators.bash_operator import BashOperator
from airflow.operators.python_operator import PythonOperator
from datetime import datetime, timedelta
from airflow import AirflowException

def start(**kwargs):
    return "Hello World"

def fail(**context):
    raise AirflowException("Oh no a failure!") # Fail here

default_args = {
    'owner': 'Osmar',
    'depends_on_past': False,
    'start_date': datetime(2019, 7, 1),
    'email'<EMAIL_ADDRESS>    'email_on_failure': False,
    'email_on_retry': False,
    'retries': 1,
    'retry_delay': timedelta(minutes=1),
}

dag = DAG('failed_dag', default_args=default_args, schedule_interval=timedelta(1))

t1 = PythonOperator(task_id="start", provide_context=True, python_callable=start, dag=dag)

t2 = \
    PythonOperator(task_id='fail',
                   provide_context=True,
                   python_callable=fail,
                   dag=dag)

t3 = BashOperator(task_id="middle", bash_command="true", dag=dag)
t4 = BashOperator(task_id="concurrent", bash_command="true", dag=dag)
t5 = BashOperator(task_id="other_start", bash_command="true", dag=dag)
t6 = BashOperator(task_id="irrelevant", bash_command="true", dag=dag)

t1 >> [t3, t4] >> t2
t5 >> t2
t1 >> t6

The end result would be,

and in Sentry

This example doesn't work anymore


afterStateHydrated not being called on $set

Package
filament/filament
Package Version
v3
Laravel Version
v11
Livewire Version
v3
PHP Version
PHP 8.3
Problem description
I'm using https://filamentphp.com/plugins/leandrocfe-brazilian-form-fields to fill out address information based on the zip code. When a zip code is fetched, it updates fields with the zip information.
But for the state_id, I need to hydrate the info (it comes as a CODE, example: NY), and find the state in the DB (State::where('code', $code)->id).
The thing is, when state_id is updated with the CODE using $set, or any other field (if I select other field to be updated), afterStateHydrated is not being called, as well as afterStateUpdated. Both fields already have ->live() attribute.
Expected behavior
afterStateHydrated should be called when $set for the field is called.
Steps to reproduce

Create a field that sets other field value
Try to listen to those changes using afterStateHydrated

Reproduction repository (issue will be closed if this is not valid)
https://github.com/leandrocfe/filament-ptbr-form-fields
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

This is not the intended behaviour so it's not really a bug, "hydration" is the process of filling the form with predefined/default values and ensuring anything without a value still exists in the state. $set() is a method used for state updates, not hydration. I think in v4 there is an option to call afterStateUpdated() from a $set()


User Feedback  for "Deriving a users Slack ID from a Github ID" PoC

This sorry follows on from "Proof of concept: Deriving a users Slack ID from a Github ID" #3535
We have proved the concept pf being able to derive one user ID from another.
We now want to be able to provide something to users for feedback purposes and to test what we learned from user research.
User Need
As a Developer
I want to be able to translate a GitHub username to a corresponding Slack user
so that I can quickly communicate with colleagues on MoJ Slack.
Value
As a Developer it is easier to identify another user's slack ID from a Github ID and as a result have to waste less time tracking another user down.
Functional Requirements:

Integrate PoC with some sort of UI to enable a users to interact with the data

Non-Functional Requirements:

Process requires authentication method as involved user details
What other security consideration do we have before we can make this available to users for feedback purposes.

Acceptance Criteria:

We have a means through which a user can interact with the PoC.
We can ask for feedback on whether this solution meets user need.

Notes:

Feedback sheet from sessions with the users:
https://docs.google.com/document/d/10jR4IoMc90DsLhCF_i34hMnkgT06lB8dlFtD44izuQI/edit
Got some really good suggestions from both Ops Eng and the original interviewees.
Next steps are to create some more cards to determine where we're next going with this.


Cloud reader not loading on iOS Chrome correctly

iOS8.1 Mobile safari loaded correctly.
iOS Chrome(39.0.2171.50) not loaded correctly(continue spinning).

Duplicate: https://github.com/readium/readium-js-viewer/issues/352


When initializing a WebImage with isAnimating, if the initial value is false, the image never animates

Given the following code:
struct MyView: View {
    let url: URL

    @State private var isAnimating = false

    var body: some View {
        ZStack {
            WebImage(url: url, isAnimating: $isAnimating)
            Button {
                 isAnimating = true
            } label: {
               Text(.localizable.gifButton)
            }
        }
    }
}

When I tap the button, the image will never start animating. It looks like the issue is in init(url:scale:options:context:isAnimating:transaction:content:) for WebImage here:
// provide animated image class if the initialized `isAnimating` is true, user can still custom the image class if they want
if isAnimating.wrappedValue {
    if context[.animatedImageClass] == nil {
        context[.animatedImageClass] = SDAnimatedImage.self
    }
}

So if the initial value of isAnimating is false, the context never gets an .animatedImageClass and therefore the image never animates.
This is unexpected -- the init accepts a Binding<Bool>, so if the value changes to true, I'd expect the image to start animating.
Here's my workaround, which I'd prefer not to do:
WebImage(url: url, context: [.animatedImageClass: SDAnimatedImage.self], isAnimating: $isAnimating)

When I do this, I can start animating the image by setting the value of my isAnimating to true.

You're correct. Currently it relies on the extra information passed to SDWebImage level. So the binding may cause issue

The AniamtedImage one seems work fine. Because the custom option is provided during updateUIView callback, so it allows for state changes

@dreampiggy thanks for the explanation! Sorry I missed the comments -- that makes sense to me now. At any rate, thanks for the quick fix in #333!

Released in v3.1.2

@dreampiggy thanks so much!


docs(legend): 图例完善默认不选中以及点击选中交互的文档 & 增加 demo

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] commit message follows commit guidelines

Description of change

Pull Request Test Coverage Report for Build 694470602

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 93.522%

Files with Coverage Reduction
New Missed Lines
%

src/chart/controller/gesture.ts
7
82.14%

Totals

Change from base Build 685937074:
-0.07%

Covered Lines:
10904

Relevant Lines:
11307

💛  - Coveralls

这个的答疑量很高！以后就可以甩文档了。


introducing mini change to normalise OWLAPI serialisation

minor change is an extra space in the term 'lung abscess (disease).
I have done steps:

Checkout master
make new branch
Make minor edit
Save
Commit to branch
Make minor edit
Save
this was the diff:

to verify that this diff is indeed NOT due to non deterministic OWLAPI behaviour but just a version thing. This PR normalises Mondo to protege55 OWLAPI serialisation.
@cmungall, you can either merge this now or play it safe and wait for tomorrow for @matentzn  to review the diff one last time (we did it before).  He will deal with it before you wake up.
when this is merged, I will rebase the remaining open pull requests

-property_value: excluded_subClassOf MONDO:0015118 {source="Orphanet:454836"} ! rare pulmonary disease
+relationship: excluded_subClassOf MONDO:0015118 {source="Orphanet:454836"} ! rare pulmonary disease

This is correct, so can be ignored. If AP between two entities, OBO uses relationship, not property value.

@cmungall consider that the normalisation may cause conflicts with other PRs that are currently open and were serialised in the old way. I can help rebasing, up to you.


Use zap as custom logger

Closes #10

Actually I don't like this way of implementing it, will look for examples


Preview: Unable to load IDT codes for unsplit monomers we have in the library

Steps to Reproduce

Go to Macromolecules mode - Flex mode
Load from IDT file: Unable to load IDT codes for unsplit monomers we have in the library.zip
or paste from clipboard the following monomers:

/5Super-dG/
/55Br-dU/
/5Super-dT/
/55HydMe-dC/
/55NitInd/
/5AmdA/
/iAmMC6T/

Actual behavior

Nucleotides recognized as unknown monomers
Expected behavior
Nucleotides recognized as monomers from the library

Versions

Ketcher Version 2.22.0-rc.2 Build at 2024-06-28; 05:30:15
Indigo Toolkit Version 1.22.0-dev.2.1-gd5637f75-wasm32-wasm-clang-12.0.0
Windows 10 Pro
Chrome Version 126.0.6478.127 (Official Build) (64-bit)

Issue found while testing - https://github.com/epam/ketcher/issues/4382

Verified

Versions:
Ketcher Version 2.23.0-rc.2 Build at 2024-07-10; 19:39:22
Indigo Version 1.22.0-rc.2.0-ga21fb5d8f-wasm32-wasm-clang-19.0.0
Chrome Версия 126.0.6478.127 (Официальная сборка), (arm64)
MacOs Sonoma 14.5


Create konradlorenz

domain file name to www.konradlorenz.edu.co

@alex0520 Please rename lib/domains/co/edu/konradlorenz to lib/domains/co/edu/konradlorenz.txt. Thank you!

@alex0520 You might miss my previous message regarding this request. Could you please update it as described above to reflect the repository rules? Thank you!

@philipto I updated the file, as you suggested, sorry about the delay.

@alex0520 Thank you! Pull request merged. I am sorry that its processing took a bit more time than it was expected.


Added a showPager attribute

Short description of what this resolves:
This should allow the ion-slides to hide its pager
Changes proposed in this pull request:

The new ion-slides need the ability to hide the pager

Ionic Version: 1.x
Fixes: #
This should allow the showPager to be hidden if needed.

Nevermind, found this issue: #4982


After recent build wrong sessionCookieValue signing in linkedin successfully

I entered the correct li_at cookie value for the sessionCookieValue and it didn't login linkedin and give me this error https://screencast.com/t/eC3alM9K9Na
But when i entered 'abcxyz' for sessionCookieValue  when i see logs it successfully signed in linkedin but this is the wrong cookie.

Thanks for creating this issue. It seems the login check is outdated. I've pushed a change to fix that.
Install the update:
npm install<EMAIL_ADDRESS>
And try again :)
If you are still getting timeout errors after that, you can try this undocumented option:
const scraper = new LinkedInProfileScraper({
   sessionCookieValue: '', // Fill in with your cookie value
   timeout: 30000 // Let the scraper wait max 30 seconds for the pages to load
})

I'll update the documentation somewhere in the next few days to document all the available options.
Let me know if this update fixed your problem

Thanks. npm install<EMAIL_ADDRESS>worked for me.


pyenv: init at 2.3.21

Description of changes
pyenv is a collection of bash scripts that help with managing Python versions.
I have only tested some basic functionality on aarch64-darwin. Will resolve https://github.com/NixOS/nixpkgs/issues/238898.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

It seems that we need to patch around the python build to actually use it on NixOS.
However, those using NixOS probably don't need pyenv, so it's fine now.

Related: https://github.com/NixOS/nixpkgs/pull/94769

Thank you for pointing me to that attempt, I had not seen it before. Its implementation doesn't look too different from mine except that the (major) version is different.
It prompted me to do a more thorough end-to-end test, and I'm able to get it to work on my macOS system. The default PYENV_ROOT seems to be ~/.pyenv, and indeed Python is installed there and can be used.


Handle Bruteforce integration directly in Python

Integrate bruteforcer results by yelding them so that they can be used in scripts directly.

Bump


Progress on HTML reporter always 0% when delay

When using the html reporter, and the root test suite is delayed, the progress is always 0% when completed.

using

  <meta charset="utf-8">
  <title>Progress on delay</title>
  <link href="https://cdn.rawgit.com/mochajs/mocha/2.2.5/mocha.css" rel="stylesheet" />

and the following tests.js
setTimeout(() => {
    describe('Window', () => {
      it('it has to exist', () => {
        expect(window).to.exist;
      });
    });
  run();
}, 3000);

As a sanity check, if you remove the delay, does the process work?

@MarcusSorealheis, yes, the progress works if its ran without delay.

I think this is because runner.total is set on init, so the front-end reporter will always get a total amount of 0.


Release for ScalaTest 3.1.x

Hello there,
could you please release a version which supports ScalaTest 3.1.x? Thanks a lot in advance.
Cheers :)

the latest version, 5.1.0, supports 3.1.x, right?

ah really, i didn't see that in github releases

5.1.x supports scalatest 3.1.x


wekan snap 6.09.0 does not update since 2022

I noticed, that my wekan is not updating since long time (2022) what is the proble? Im using the snap version. I dont want to loose my boards. What is todo? Im running on ubuntu 20.04. Thanks for help.

@docknack
Yes, there will be automatic upgrade, that will be tested well, that you would not loose your boards.
More info at https://github.com/wekan/wekan/issues/5475#issuecomment-2271611017

Moved to https://github.com/wekan/wekan/issues/5562


Hactoberfest Contributions

Any valid submissions in the following category would be appreciated:

Red, Blue & Purple Team Resources(Custom Scripts/Tools/Ideas)
Data Privacy (Ideas/Implementations/Guides)

Please Assign me this issue. (Hactoberfest 2021)

Please assign me this issue, Need this for hacktoberfest2021!

could you please assign me this issue, i need this for hacktoberfest 2021


Windows Terminal 0.7 immediately crashes on startup

Environment
Microsoft Windows [Version 10.0.18363.476]
Windows Terminal 0.7
Steps to reproduce
Start Windows Terminal.
Expected behavior
Was hoping the application would open..
Actual behavior
The terminal icon appears in the start bar, then a moment later it disappears.  The application never starts.
I've been using Windows Terminal for the last few versions and it's been fine.  I upgraded to 0.7 today and now it no longer opens.  I tried uninstalling and reinstalling it.  I also tried explicitly removing profile.json, because i had previously changed it.
I've also tried rebooting.

Thanks for the report! Would you mind filing a report with the feedback hub so we can get a stack trace and see what's going on?
/feedback

That was quick!  I've submitted something through feedback hub now.  I didn't realise it could be used for things like this.

@johnc44 Could you share the aka.ms link from the "Share my feedback" button here? I can't look it up without that unfortunately 🤣

Oops, sorry - https://aka.ms/AA6ptgu

I have almost the same issue/crash - the window shows as a non-transparent single color rectangle and then disappears. I tried resetting and reinstalling but nothing helped. I tried on 2 very different devices and it was the same. I already submitted a feedback in the Feedback Hub but unfortunately I didn't include a stack trace.

Same problem

Same problem,Microsoft Windows [Version 10.0.18363.476]

Just noticed that I do get a single colour rectangle as per @Viruslav - I just didn't notice it yesterday.

Same problem here.

Same here. Updated today and app crashes after start. Only single grey color and then window immediately closed.
I updated through MS Store.

Something to try...
Delete your terminal settings file (make a backup first!).
My settings file is below but there is a hash in the directory name so yours will probably be different:
<User>\AppData\Local\Packages\Microsoft.WindowsTerminal_8wekyb3d8bbwe\LocalState\profiles.json
My guess is that will get you up on your feet and you could play around with introducing parts of the old file to see what is causing the crash.

I use custom scaling set at 160% on both PCs.

It would be useful to know if original poster has custom scaling.  If so this issue can be marked as duplicate of #3749.  It's entirely possible there is more than one reason for startup....

I have the same
tried deleting the appdata folder and i have no custom scaling set.

@owennewo  Yes, I have custom scaling set to 135%, mirrored displays.  (
I've already tried deleting profile.json which didn't help.

this is my first install - i'm seeing the same symptoms. 175% scaling. Yeah, deleted profile, same thing.

I am having the same issue as this after an app update.
Feedback Hub link with Error Logs:
https://aka.ms/AA6q4df

Hey guys, several usages are fine with normal 125% scale.
But the Terminal crash when i use custom scale 138%.

I disable advanced scale, Now It works fine.

Hey guys, several usages are fine with normal 125% scale.
But the Terminal crash when i use custom scale 138%.

Thanks for your tips.

Disabling custom scaling worked for me as well! Thanks so far.

 I'd rather not use this terminal. I'm sorry but this is not a solution. Everything else works just fine.

It’s working when I change the custom scale factor into default. Thanks a lot.

This looks like a /dupe of #3749. Thanks!


[Bug]: Doesn't work when running on Laravel Sail

Short bug description
Unable to connect to DB when running with Laravel Sail
Extended bug description
Application launched on Laravel Sail - https://laravel.com/docs/10.x/sail

If I use Tinker from Sail everything works fine

Your project is not the problem

[X] I tried using a fresh laravel install - I'm fairly certain it has nothing to do with my project.

How to reproduce?

Create a project with Laravel Sail

curl -s "https://laravel.build/example-app?with=pgsql,redis" | bash
cd example-app
sail up

Run the Laravel Tinker Plugin

<?php
use App\Models\User;
echo "123\n\n";
User::all();

Expected behavior
Successful connection to the database. Display all users
Screenshots
No response
IDE & Version
PhpStorm 2023.2.1
PHP Version
8.2.9
OS and version
Ubuntu 22.04.3
Laravel Tinker plugin version
2.6.1
Interpreter type
Docker compose
Laravel Version
10.20.0
Tinker version
2.8.1
Psysh version
0.11.20
Additional Context
No response

Hey! 99.9% sure you are using a local interpreter to reach your docker database. That, indeed, does not work.
Check #250 :)


Preferred way to represent "not yet known" value

Hi,
What is the canonical way to represent "not yet known" value? For example, result of an async expression?
The simplest way (achievable now) is to simply use Store<T option>.
Stores and their derivations are implementations of IObservable<T>, which do not guarantee that values are always present (that is, the observable is not "behavior subject", as described here. This would most likely work with current functions (e.g. Bind.el), however, there is no way to represent "loading" state (or is there?). A possibility would be to add a new function overload with loader element to be displayed when value is not available yet.
What are your thoughts on this?

Hi,
I use Bind.promise for this. Here's one example of usage:
https://sutil.dev/#examples-await-blocks
That's interesting to read about BehaviorSubject. Store does have an initial value, but you're right that as soon as you start projecting and filtering, you end up with IObservable. (Store does guarantee to send the initial value out to any new subscribers though)
Here are the Bind.promise functions:

    static member promises (items : IObservable<JS.Promise<'T>>, view : 'T  -> SutilElement, waiting: SutilElement, error : Exception -> SutilElement)=
        Bind.el( items, fun p -> Bind.promise(p, view, waiting, error) )

    static member promise (p : JS.Promise<'T>, view : 'T  -> SutilElement, waiting: SutilElement, error : Exception -> SutilElement)=
        Bind.el(  p.ToObservable(), fun state ->
            match state with
            | PromiseState.Waiting -> waiting
            | PromiseState.Error x -> error x
            | PromiseState.Result r ->  view r
        )

    static member promise (p : JS.Promise<'T>, view : 'T  -> SutilElement) =
        let w = el "div" [ CoreElements.class' "promise-waiting"; text "waiting..."]
        let e (x : Exception) = el "div" [ CoreElements.class' "promise-error"; text x.Message ]
        Bind.promise(p, view, w, e )

Thanks!
I see you use a similar approach to the 'T option one (with the ability to handle errors). I tweak my store initialization so I can directly modify the model in the store (plus wrapping it in Some). Error handing is usually done upstream (or I use Result):
module Store =
   let makeAsyncOption (x: Async<'T>) : IStore<'T option> =
        let store = Store.make None
        async {
            let! result = x
            result |> Some |> Store.set store
        } |> Async.StartImmediate
        store

Suggested variant would be to use a custom store-like class implementing IObservable<'T>, that would not have an initial value. Set would work the same (and would initialize the store), Modify would trigger an error when called in uninitialized state, Value would become TryGetValue. The upside is that the consumers would have no notion of its "lazy" flavor, be it promise, async or something completely else. I wonder what the repercussions would be. It probably needs to be put to test ;)

I'm open to this, but I need to understand why it would be desirable to having this extra code to maintain when we can already implement Store<'T> with 'T = Option<'U>
I think it's because I haven't understood your use case properly, and why using Option<> isn't expressive enough.

It is more about what is the behavior of Bind.el and similar methods / functions for IObservable<> which do not provide a value right away. These do not have to be necessarily derived from Store (for any reason) and having no initial value is not forbidden by the interface contract. Currently, any binding to empty IObservable becomes a SideEffect, which is in practice similar to Html.none (?).
One could take advantage of this behavior and use it for "value not yet available". The extensions which would allow more user-friendly API (creating stores without initial value, having Bind.el with alternative "loader" element, etc.) do not even have to be inside the core library.
On the other hand, this means any current and future usages of IObservable must never rely on a value present (for example, checking the subscriber has been called by the time the call of Subscribe is finished).

I see. It's not so much about the way Store behaves, but consumers of IObservable. It's true that in Sutil, I know that when I'm handed an IObservable, and subscribe to it, that I will immediately receive the initial value (because of the way Store works, which I modelled on Svelte's stores/cells).
Would it be useful then to have a overloads of Bind.el (et al) like the following?
         Bind.el( data : IObservable<'T>, view: 'T -> SutilElement )   // Current signature (or at least, very close to it

         // Using Option
         Bind.el( data : IObservable<'T>, view: 'T option -> SutilElement ) //  view None will be shown until a value arrives

         // Without Option, using init/view pair
         Bind.el( data : IObservable<'T>, init: unit -> SutilElement, view: 'T -> SutilElement ) //  init() will be shown value arrives

Option feels more canonical, but I don't like that we have to wrap every value in Some just to handle an initial case.
Regardless of that small point, am I addressing the issue you're raising?
Cheers

Yes, it's exactly what I had in mind. Sorry it took me several posts to make myself clear :)
Unless you have some prior objections to the suggested behavior, I'll test it in one of my projects to see how it performs in real scenarios. Then you should be able to write something like:
let store : IObservable<_> = getDataAsync() |> DelayedStore.makeAsync

// ...

Bind.el(store, (fun () -> Html.text "Loading...."), (fun data -> Html.text $"Data received: {data}"))


error "embark new TokenFactory"

Hi @pkrasam
Let's try somethings so better understand what's going on.

What version do you get with embark version ?
did you do npm -g install embark ? was there any errors?
what happens if you try npm -g install scrypt? (and then re-run embark new)

@pkrasam looking into it more, it's likely an issue with node-gyp or the gcc version in your system. see https://github.com/barrysteyn/node-scrypt/issues/108

@pkrasam has your issu been solved?


Varnishstat plugin

It will be great to have a varnishstat plugin to collect varnish metrics

the exec module and varnishstat -j do the job !

perfect! can we close this then?

Closing because it appears the exec plugin can be used, please re-open if otherwise

@damstux Would it be possible for you to document your configuration as an example for future googlers, like me? 😄

fwiw, we've also got #1173 coming soon


TypeError: 'generator' object is not subscriptable

Traceback (most recent call last):
File "udc_predict.py", line 58, in
print("{}: {:g}".format(r, prob[0, 0]))
TypeError: 'generator' object is not subscriptable

I have same error running it with different versions of python and tensorflow.
Has anybody fixed it?

I try this and it worked :dancing_men:
print("{}: {:g}".format(r, next(prob)[0]))


misleading name cpr::Header lead to silently overwriting headers

cpr::Header is a misleading name. The correct name should be cpr::Headers.
By using the name cpr::Header, it implies that the following code is correct. But alas, the following code is not correct.
cpr::Response response = cpr::Post(cpr::Url{"https://httpbin.org/post"},
   cpr::Header{{"Authorization", myAuthorization}},
   cpr::Header{{"Content-Length", "0"}}
);

The above code snippet will not send the Authorization header, because it is overwritten by the Content-Length header on the next line.
Here's the correct code, which behaves as expected under the current version.
cpr::Response response = cpr::Post(cpr::Url{"https://httpbin.org/post"},
   cpr::Header
   {
        {"Authorization", myAuthorization},
        {"Content-Length", "0"}
   }
);

This confusion could've been prevented by either:

Pluralization cpr::Header -> cpr::Headers
Any example code showing a use case where two or more custom headers are used, such as that pictured above.

It took too long to figure out that this was why our requests were failing. Hopefully others do not suffer the same fate.

Read the documentation, it's actually quite good.
If you had, you might have seen where they specify that Header and other classes are "Map-like" in how they handle key-pairs, etc., as well as seen an actual implementation of how to use them.
The set of double-braces should be a clue, they're braces within braces, implying a container of containers.


If the url was nil,the SVGA will crash.

I found the reason for crash in the MD5 Method finally.If the url was nil,it will crash certainly

You should not deliver nil URL.


Clean up GeoPoint port to C++

... based on things learned porting Timestamps.
There's no good way to include conversions equivalent to toGeoPoint in
FIRTimestamp, so pull out separate conversion functions that match other
similar things.
Also clean up leftover references to FIRGeoPoint internally.

I'm not convinced this is the best way to do this so feedback welcome.
The pattern for this kind of conversion elsewhere is Target MakeTarget(Source). This works in C++ because the argument type gets burned into the mangled name. Unfortunately this part of the code is still patterned after Objective-C so it looks a little out of place.
An alternative would be to create Objective-C looking C functions, something like FSTCreateGeoPointFromFIRGeoPoint. I'm not really wild about that.
A final option would be to stick with the -toFoo +fromFoo but just find a way to make that work reliably for FIRTimestamp. Unfortunately, we're aiming to eventually share that code with Functions so just embedding/manipulating our C++ type there doesn't work.
Making a category in a separate file forces end users to need to link with the -ObjC flag, which CocoaPods users get for free, but not other users. People get this wrong often enough that I don't want to rely on it. There are games we can play with dummy functions to force linkage but I'm skeptical these approaches work in the presence of app thinning and bitcode.
Anyway, this way leaves nice concise names that feel slightly out of place which seems like the least worst way to do it, but I value your input.


Remove old Cthulhu localization code.

This is to remove all dependency on ZeroMQ. Cthulhu will be upgraded to the new remote control protocol. See mbari-org/cthulhu#26

@hohonuuli This is done, correct?


doc: refine test doc

Signed-off-by: letty<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

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

@@           Coverage Diff           @@
##           master     #856   +/-   ##
=======================================
  Coverage   14.89%   14.89%
=======================================
  Files         114      114
  Lines        6826     6826
=======================================
  Hits         1017     1017
  Misses       5723     5723
  Partials       86       86

Continue to review full report at Codecov.

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

LGTM


WIP: Add cookie-banner component

Trello ticket: https://trello.com/c/oEZ51BvR/838-3-reimplement-cookie-message-component

will close this but keep the branch for now as it has modernised code (lifted fro mages ago)


Fix type checking for ordered lists of schemas.

We weren't recursing into checking whether lists of inline schema types
were compatible at a read/write level, and instead were just using
straight equality.
This CL also tidies up a previous fix (same issue but for direct
inline references) to keep everything consistent.

I will merge this in Shane as I need for another PR.


[GSSOC'23] To add code for Measuring Internet Speed

Hii @avinashkranjan,  I've successfully added a feature that allows users to measure their Internet speed.
#1481 My Issue Number
Description
I've added a  feature for performing an internet speed test using the Speedtest library with GUI. It uses the Tkinter library to create the GUI window. The GUI provides a user-friendly and visually appealing way to measure download speed, upload speed, ping latency, and identify the best server for the speed test. Users can initiate the speed test by clicking a button, and the results are dynamically displayed in the interface.
Output

Before Running the Speed Test

After Running the Speed Test

Fixes #1481

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My documentation follows Template for README.md
[x] I have added the project meta data in the PR template.

Project Metadata
Title: Add Code for Measuring Internet Speed
Folder: Internet-Speed-Test
Script: main.py
Contributor: Abhishek-Rajput-81
Description: Adding Internet Speed Test GUI with Dynamic Results Display.

@avinashkranjan please review this

@avinashkranjan @avinashkranjan

@avinashkranjan I have added a README file
please check now

@avinashkranjan @avinashkranjan

@avinashkranjan
@avinashkranjan

@avinashkranjan

@avinashkranjan @avinashkranjan


fix(version): update fabric8-runtime-console to 0.12.1

There were the following issues with your Pull Request

Commit: 5fe88d0de98c091b0b20a62952b29a1422aecd97
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


WeibullFitter object doesn't have an event_table attribute

plotting a weibull fit results in an error if show_censors = True due to line 350 of plotting.py.
WeibullFitter object doesn't have an event_table, it has an event_observed. If this is the same, we could copy event_observed into event_table?
The WeibullFitter class probably needs something like the KaplanMeierFitter class:
v = _preprocess_inputs(durations, event_observed, timeline, entry, weights)
self.durations, self.event_observed, self.timeline, self.entry, self.event_table = v
Example to reproduce:
from lifelines import WeibullFitter
wf = WeibullFitter()
wf.fit(durations = [3,4,5,6], event_observed = [1,1,1,0])
wf.plot(show_censors=True, ci_alpha=0.05, ax=ax)
AttributeError: 'WeibullFitter' object has no attribute 'event_table'

Thanks for the reproducible example and issue. I don't think this is a bug, but a problem with the KaplanMeierFitter (and other classes') API. Here's my reasoning:

The KaplanMeierFitter needs the event_table to do it's product estimation.
Early on, this table was so useful that I exposed it in the fitted class.
However, most other fitters (like Weibull) don't need this event table to do the calculation, hence didn't have it as part of their API and probably don't need it either. Users interested can use lifelines.utils.survival_table_from_events
Really, I should remove the table from the KaplanMeierFitter API, and force everyone to use survival_table_from_events instead.


Why is eval used? Vulnerable to XSS attacks

When I use vue-pdf-app in my Vue Chrome extension, nothing can run because eval is used in the source code.
I get errors like:
Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".
And errors like:
Use of eval in "node_modules/vue3-pdf-app/dist/vue3-pdf-app.umd.js" is strongly discouraged as it poses security risks and may cause issues with minification.
My question is why this is used in the source code because I don't want my application to be vulnerable to XSS attacks.

I second this bug. I really like this library but I can't use it because of the issue described here.
It looks like the original issue in pdfjs has been resolved so I wonder if the code can port that fix?


Use ThrowHelper.ThrowIfNull to validate args in generated code

Motivation
Using a throw helper reduces JIT code size and improves inlining for a possible performance gain.
Modifications
For all generated client code where arguments are being validated as non-null, use ThrowHelper.ThrowIfNull rather than throwing ArgumentNullException directly. This includes prewritten client code as well as generated client code. MethodHelpers are updated to use this pattern.
Include the CallerArgumentExpressionAttribute in downlevel frameworks so we may consistently avoid passing the parameter name to ThrowIfNull manually. Simplify all call sites that are currently passing the name manually.
Add a ThrowKeyNotFoundException throw helper for use in the TypeSerializerRegistry.
Use is not null instead of != null in MethodHelpers.IfNotNull to ensure that equality operator overloads are never used.

/merge


improve patch print message

LGTM! Thank you @klieret

question- is this the last thing the system prints? I think it should be, that way it can't be missed.

question- is this the last thing the system prints? I think it should be, that way it can't be missed.
yup


COCO object detection image size and inference speed

First of all, thank you for publishing all this hard work!
After some detectron2 struggles (it has to be build from the EVA fork in /EVA/det/, not from the facebookresearch repo!) I managed to run the COCO evaluation example to reproduce your object detection and instance segmentation results. Amazing stuff!!
My first question is regarding the image size. The detectron2 default image size for COCO is 1024. In your paper you say that you fine-tuned on COCO and LVIS using 12802 inputs, but in the code example published here you use a default image size of 1536 (cascade_mask_rcnn_vitdet_eva_1536.py). Why did you use 1536 here?
I did some inference experiments and it seems 1280 is indeed an optimal image size for inference:

image_size
inference speed (3090)
bbox mAP (only on 100 val2017 images)

1536
~2.35 s/iter
67.1

1280
~1.05 s/iter
67.4

1024
~1.23 s/iter
66.5

612
~0.95 s/iter
59.8

Second question. I used AMP to autocast the torch model to FP16 which already reduced the 12802 inference time from 1.05 s/iter to 0.75 s/iter. I am interested in further optimizing the model for faster inference speeds. Do you think it is possible to optimize your model using TensorRT or ONNX?
Thanks again!

Hi @CamielK, thanks for your interest in EVA and good to see that you have successfully reproduced our results!
Regarding to your questions:

Q1: Why did you use 1536 here?

A1: For single-scale inference / evaluation, we find using 1536px during inference can achieve a little better result than using 1280px (the training resolution). We also find in some circumstances, using even larger input resolution (e.g., 1664px) can further boost the performance.

Q2: Do you think it is possible to optimize your model using TensorRT or ONNX?

We don't have experience & plan to optimize the speed-accuracy trade-off of EVA. But since EVA det & seg is built upon the widely used Detectron2. Perhaps you may seek for help from the community?

I believe the issue at hand was addressed, as such I'm closing this. Feel free to ask if you have further questions.

Hi,
I am unable to repeat the experiments on Instance Segmentation that you mention.
Some links are stale (e.g. (cascade_mask_rcnn_vitdet_eva_1536.py)
You mention "(it has to be build from the EVA fork in /EVA/det/, not from the facebookresearch repo!" ? How?
Thanks.

Hi,
I am unable to repeat the experiments on Instance Segmentation that you mention. Some links are stale (e.g. (cascade_mask_rcnn_vitdet_eva_1536.py)
You mention "(it has to be build from the EVA fork in /EVA/det/, not from the facebookresearch repo!" ? How?
Thanks.

All the config related to EVA-01 det are available at: https://github.com/baaivision/EVA/tree/master/EVA-01/det/projects/ViTDet/configs.


Generate new ObjectId

Is there chance that you add possibility to generate new Object Id?
As far I checked right now ObjectId it is only a wrapper for Id which you have to provide.
<EMAIL_ADDRESS>Expected result:
{ _id: ObjectId() }
Example in action:
collection.updateOne(
        { _id: ObjectId(SOME_ID) },
        {
          $push: {
            modules: {
              _id: ObjectId(),
              name,
            },
          },
        });

Can we also have the _id.getTimestamp()?
Deno code can be something like:
const timestamp = _id.$oid.toString().substring(0,8)
new Date(parseInt(timestamp, 16 ) * 1000)

Could be invoked like:
ObjectId(_id).getTimestamp()

Thank you

Currently, an id can be generated by new Bson.ObjectId()
import { Bson } from<EMAIL_ADDRESS>const id = new Bson.ObjectId();


Fix: Change some names in the Readme to Trireme-Lib

This PR addresses the recent change of name from trireme to trireme-lib:

Links for the badges corrected.
Links and descriptions for the implementation of trireme changed.
Name changed to trireme-lib wherever relevant

Codecov Report

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

@@           Coverage Diff           @@
##           master     #404   +/-   ##
=======================================
  Coverage   56.59%   56.59%
=======================================
  Files          45       45
  Lines        6066     6066
=======================================
  Hits         3433     3433
  Misses       2327     2327
  Partials      306      306

Continue to review full report at Codecov.

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

@dstiliadis can you review pretty please ? :)


[Task]: Zone space distribution management

create data structure on sections to hold space distributed values in columns. a section shall be divided into 12 columns.
create edit layer on sections to edit space distributed amongst zones.

handles to drag

should be visible on hover in between two zones
dragging is not pixel based but column based.

add limits to extend a zone(not beyond the section limit of 12 columns or less than 2 columns per zone).
add responsive behaviour to zones to not fold below 2 columns.

sagas to update sections or zones(based on datastructure decided for holding space of zones).

Miscellaneous:

make sure to test user-select cases in safari
make sure the handle is not continuing to be held even after mouse down like in canvas resizer

Internal discussion ref

This has been completed.


Add /r/children/:inscription_id/inscriptions endpoint

It also adds delegate to the inscription details.

Nice! Saves some time. Should we not just add a new field in the current children endpoint responses? @casey Did we decide we did not want to add fields to recursive endpoint updates?
pub struct Children {
  pub ids: Vec<InscriptionId>,
+ pub inscriptions: Vec<InscriptionRecursive>,
  pub more: bool,
  pub page: usize,
}

It also adds delegate to the inscription details.

If we are not adding fields then should we update this? It would update the current /r/inscription endpoint right?

We need to figure out a way to evaluate the performance impact of PRs like this. Probably the way to do it is to simulate a production workload with tons of requests, and see how it holds up. The worst-case scenario would be we add an endpoint like this, it gets widely used, and we can't ultimately support it, because it overwhelms the ordinals.com servers.

We need to figure out a way to evaluate the performance impact of PRs like this. Probably the way to do it is to simulate a production workload with tons of requests, and see how it holds up. The worst-case scenario would be we add an endpoint like this, it gets widely used, and we can't ultimately support it, because it overwhelms the ordinals.com servers.

Then we buy more powerful servers haha

We need to figure out a way to evaluate the performance impact of PRs like this. Probably the way to do it is to simulate a production workload with tons of requests, and see how it holds up. The worst-case scenario would be we add an endpoint like this, it gets widely used, and we can't ultimately support it, because it overwhelms the ordinals.com servers.

We can simulate a production workload and test with 10,000, 25,000, 50,000, and 100,000+ requests. Is this what you had in mind? Or do you just mean like 1,000 requests?

The worst-case scenario would be we add an endpoint like this, it gets widely used, and we can't ultimately support it, because it overwhelms the ordinals.com servers.

Couldn't you simply reduce the page size until the servers can cope? Don't commit to the page size staying at 100 so inscriptions don't start depending on any specific page size.

Adding some more context here on how it works from @gmart7t2. There's some interesting implications to draw from it about how ord works for those who aren't familiar.
For our purposes, honestly all we really need is part B to get the blockheight. It would be super nice to get the delegate ID, but I see that is the same as getting the content (assuming you put some kind of limit on the size of the content that would be returned). If we mine for IDs via an inscription service, then we don't need anything from A (albeit that's not a great UX, it's definitely workable).
I'm not sure what others will want. But the crux is the blockheight and the content/delegate can be simulated by mining IDs.
An interesting bit of info for those who aren't familiar with ord, is that we are starting with the existing list of inscriptions here. We can't override the function that gets all the child inscriptions to include the delegate ID or blockheight because that would require to change how ord is indexed. Ultimately, without re-indexing, it seems the most performant thing we can do to get what we need is just simplify the PR to only include call B.
@casey if we do that should we still do some performance testing? Are you confident that just part B would be a sufficiently low burden?

I was wrong that "D" would be heaviest because it's fetching the raw tx, because "A" also fetches the raw tx, and so the 2nd (redundant?) fetch is quick due to caching.
And the timings for the Pizza Ninja children are for the first page of 100, not for the whole set

Should we not just add a new field in the current children endpoint responses?

No. The current endpoint is fast, only using the ord database. This new endpoint runs getrawtransaction twice per child.

Did we decide we did not want to add fields to recursive endpoint updates?

No. We decided the opposite. See https://docs.ordinals.com/inscriptions/recursion.html :
"additional object fields may be added or reordered, so inscriptions must handle additional, unexpected fields"

Closing this in favor of #3771


[Pluralsight] ERROR: An extractor error has occurred. (caused by KeyError(u'course',))

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.07.23. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.07.23

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--playlist-start', u'125', u'--verbose', u'--username', u'PRIVATE', u'--password', u'PRIVATE', u'https://app.pluralsight.com/library/courses/pattern-library', u'-o', u'%(playlist)s/%(chapter_number)s. %(chapter)s/%(playlist_index)s. %(title)s.%(ext)s', u'--sleep-interval', u'60']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.23
[debug] Python version 2.7.9 - Linux-4.9.35-v7+-armv7l-with-debian-8.0
[debug] exe versions: avconv 11.9-6, avprobe 11.9-6, rtmpdump 2.4
[debug] Proxy map: {}
[pluralsight:course] pattern-library: Downloading course JSON
ERROR: An extractor error has occurred. (caused by KeyError(u'course',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pluralsight.py", line 344, in _real_extract
    })['payload']['course']
KeyError: u'course'
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 776, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 446, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'course',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

i am also getting this error
ERROR: unable to download video data: HTTP Error 404: Not Found
Traceback (most recent call last):
File "c:\python27\lib\site-packages\youtube_dl\YoutubeDL.py", line 1863, in process_info
success = dl(filename, info_dict)
File "c:\python27\lib\site-packages\youtube_dl\YoutubeDL.py", line 1805, in dl
return fd.download(name, info)
File "c:\python27\lib\site-packages\youtube_dl\downloader\common.py", line 361, in download
return self.real_download(filename, info_dict)
File "c:\python27\lib\site-packages\youtube_dl\downloader\http.py", line 61, in real_download
data = self.ydl.urlopen(request)
File "c:\python27\lib\site-packages\youtube_dl\YoutubeDL.py", line 2166, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "c:\python27\lib\urllib2.py", line 435, in open
response = meth(req, response)
File "c:\python27\lib\urllib2.py", line 548, in http_response
'http', request, response, code, msg, hdrs)
File "c:\python27\lib\urllib2.py", line 473, in error
return self._call_chain(*args)
File "c:\python27\lib\urllib2.py", line 407, in _call_chain
result = func(*args)
File "c:\python27\lib\urllib2.py", line 556, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
HTTPError: HTTP Error 404: Not Found

No such course.


Add Shell script to ADS supported languages

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution or feature you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Not enough information provided in feature description, please open a feature request with detailed information


It's for client or server side

I want to know about the client's hardware information use my web app. Is there any option to know about hardware by using this NPM?

@ssuriyayuvan If you mean, if this package can be used client side (in browser)? No unfortunately not. This module is intended to be uses only on server side (node.js). Browsers would refuse system calls like I am doing in systeminformation package due to security reasons.

Yeah thanks!
On Fri, 12 Jul 2019, 5:13 pm Sebastian Hildebrandt, <
<EMAIL_ADDRESS>wrote:

@ssuriyayuvan https://github.com/ssuriyayuvan If you mean, if this
package can be used client side (in browser)? No unfortunately not. This
module is intended to be uses only on server side (node.js). Browsers would
refuse system calls like I am doing in systeminformation package due to
security reasons.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sebhildebrandt/systeminformation/issues/261?email_source=notifications&email_token=AIKHV4GBIDGQ37XMJMWFYRTP7BU7VA5CNFSM4ICH6542YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZZQXLQ#issuecomment-510856110,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIKHV4GQ2HSHP6UD6MT3SZ3P7BU7VANCNFSM4ICH654Q
.


Support private Docker repositories

Private Docker repositories often need a password for authentication (https://docs.cloudfoundry.org/devguide/deploy-apps/push-docker.html#private-repo).
This pull request adds the required password to the environment.

Was missing docker-username on push.
Added it.

Thanks!


[MM-28140] e2e/system_console/customization: add T1030

Summary
Add T1030: Custom branding removed - Blank site name and description, and brand image and text - false
Ticket Link
https://mattermost.atlassian.net/browse/MM-28140

Hi @mgdelacroix,
Can you please change the title of the test to indicate Default site name?  This is because a blank site name can no longer be saved so the current title is misleading.  (I've updated the existing test in TM4J to indicate the updated title)
Thanks!

@lindy65 done! :)

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.


Add support for {*_FILTER} keybinding replacements (refs #240)

This pull request adds support for 4 new keybinding replacement variables, each of which resolves to task filter expressions that underly the currently visible list of tasks displayed by vit:

{EXTRA_FILTER}: any extra filter expression applied by the user
{REPORT_FILTER}: the report.___.filter of the currently visible report
{CONTEXT_FILTER}: filter expression of the currently active context
{VIEW_FILTER}: this is just a concatenation of the other three which preserves internal precedence by adding parentheses (if necessary)

Testing
The easiest way to test the correctness of the replacements is by adding the following keybinding (change the actual key to your liking): G = :all {VIEW_FILTER}
Now open any report in any context in vit and apply any extra filters. By invoking the binding you can first inspect the correctness of the filter expression, when you confirm it you will end up at the all report showing the exact same tasks as the view that you just came from (tasks shown and tasks completed in the status bar should be the same as well). Repeatedly invoking the binding will just bring you back to the same report with the same filter applied over and over again.

Code looks good on initial review, nice work! Just that one small comment to address.
I don't really have time to do any extensive testing, can anyone else jump in here to make sure we're not breaking anything?

One other thought: Most of the current keybinding replacements start with TASK_, which I like because the name goes general to specific. For these, I'm thinking we might want to flip them around, FILTER_VIEW, etc.
Thoughts?

Very good point about naming. Funnily enough I read the existing TASK_... replacements as having a semantic OBJECT_ATTRIBUTE or PARENT_CHILD format, so I went for ..._FILTER because in each case there is some task/vit object (the report, context or view) each of which has the same type of attribute (a filter expression).
I'm happy with whatever naming you think is better, but note that there is already a ACTION_FILTER_REPORT magic word, so I feel like having a FILTER_REPORT replacement could be confusing. Even if actions are marked with the ACTION_ prefix I had the feeling that FILTER_VIEW suggests reading 'filter' as a verb, i.e. instructing vit to perform an action rather than return an attribute.
If you think a common prefix for all replacements would be good, how about adding CURRENT_ or ACTIVE_? I.e. there would be CURRENT_REPORT_FILTER, CURRENT_CONTEXT_FILTER etc.

I'm pretty sure it's ACTION_REPORT_FILTER and not ACTION_FILTER_REPORT.
I wouldn't be too concerned if these were internal names, but they are user-facing, and it seems like a good idea to pause and do our best to get them right.
Going with your understanding of object/attribute naming, I'll counter with these names:
REPORT_FILTER
REPORT_FILTER_EXTRA
CONTEXT_FILTER
VIEW_FILTER
So the 'objects' are report/context/view, and the attribute is filter.
Thoughts?

Ah yes you're right about the action name, and also true about getting the names right. The 'extra filters' are a weird one since it's not really clear what task/vit entity they belong to, I'm not sure they actually belong to the report but if the action that applies them is already called _REPORT_FILTER then it's only consistent to keep that name.
Do you think VIEW_FILTER is appropriate? In the code the object that actually displays all tasks in the end is the 'TaskList', but I don't think it has actually been referred to as anything in the documentation, so I thought that 'view' might be more intuitive.
If you're ok with the names I will probably create a new pull request (from a separate branch where the helper functions are neatly stowed away in util.py).

Yeah the 'view' is a weird one because it's in a way the most intuitive one of all (every task list you ever look at is based on applying it), but it doesn't have its own name, it's just...the list.
Tell you what, the keybinding replacement is actually just an optional stepping stone on the way to bulk editing. Having figured out the internal logic of the filters I think it might be more fruitful if I just have a go at the next stop of implementing bulk editing, without worrying about introducing new user-facing magic words (which would probably only see rather infrequent use anyway, especially once bulk editing proper is implemented).

Yep, love it. Go for it.


⚠ Bofrost.be has degraded performance

In 82e5455, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1592 ms

Resolved: Bofrost.be performance has improved in be7eb9b.


make it possible to use without package.json

even when pnpm-version param is passed, it is currently still required to have a package.json file

looks like setup node required a lock file when cache is used.


Note creation for P2ID

As our Wallet PoC will only support P2ID transaction, so we need to have and endpoint where we can pass the faucetId, amount and destination address and the return would be the note object that will be created which can be passed to the transaction input.

Closed as done.


ratio of gsnap to rapsearch servers needs to increase

based on recent observations with the reduced gsnap chunk size

This is super low risk, and I tested locally.    Feel free to merge into master at any point.    The impact on cost is considerable as it reduces the time the entire rapsearch tier sits idle while waiting for gsnap to finish.


Support for v1.2 dockstore.yml for services

v1.1 of the dockstore.yml only supported services. The v1.2 version supports workflows and services, along with things like multiple entries per dockstore.yml, and entry names. Users should be able to add services using the v1.2 format.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Sprint: Seabright Sprint 29 Dugong
┆Issue Number: DOCK-1213

This should include tests that show how a v1.1 service would be converted into a v1.2 service.

Tried reviewing this on dev. It was really difficult getting a service updated on Dockstore. I only got it to happen once, randomly, for a branch. Regardless, the one update that did happen was still labeled as version 1.1, but a was marked invalid with the message that it's missing a primary descriptor. https://dev.dockstore.net/services/github.com/NatalieEO/xenahub:master?tab=files

@NatalieEO , @Ldcabansay used your 1.1 service as a model to bring a service into Docktore with a .dockstore.yml, and it failed because your service's .dockstore.yml has subclass: docker-compose, whereas it should have type: docker-compose. When she changed it, it worked. Do you think that's the issue you are running into, or is it something else.
As background, it's subclass is in the 1.2 version; but type is in the 1.1 version.

Was able to add a service using v1.2 on production
API - 1.9.0 dockstore
UI - 2.6.0


[bug] Unknown current metrics but HPA is actually working

[root@kube-master-1 ~]$ kubectl -n gaoce describe hpa
Name:                       tea
Namespace:                  gaoce
Labels:                     <none>
Annotations:                kubectl.kubernetes.io/last-applied-configuration:
                              {"apiVersion":"autoscaling/v2beta1","kind":"HorizontalPodAutoscaler","metadata":{"annotations":{},"name":"tea","namespace":"gaoce"},"spec"...
CreationTimestamp:          Tue, 21 May 2019 14:03:42 +0800
Reference:                  Deployment/tea
Metrics:                    ( current / target )
  resource cpu on pods:     <unknown> / 1005m
  resource memory on pods:  <unknown> / 300Mi
Min replicas:               1
Max replicas:               2
Deployment pods:            1 current / 1 desired
Conditions:
  Type            Status  Reason              Message
  ----            ------  ------              -------
  AbleToScale     True    ReadyForNewScale    recommended size matches current size
  ScalingActive   True    ValidMetricFound    the HPA was able to successfully calculate a replica count from memory resource
  ScalingLimited  False   DesiredWithinRange  the desired count is within the acceptable range
Events:
  Type    Reason             Age    From                       Message
  ----    ------             ----   ----                       -------
  Normal  SuccessfulRescale  9m50s  horizontal-pod-autoscaler  New size: 2; reason: memory resource above target
  Normal  SuccessfulRescale  2m14s  horizontal-pod-autoscaler  New size: 1; reason: All metrics below target

I am running autoscaling/v2beta1 HPA in Kubernetes 1.12. The status.CurrentMetrics is empty, but it seems that the HPA can work according to memory usage.
I run a deployment with one pod which uses 2Mi memory. Then I set:
    - type: Resource
      resource:
        name: memory
        targetAverageValue: 1Mi

Then it scaled up to 2. After a while I set it to
    - type: Resource
      resource:
        name: memory
        targetAverageValue: 300Mi

Then it scaled down to 1.
I am wondering why the HPA works but the status is empty. Is it caused by the metrics server or the HPA controller?
I'd appreciate it if you could help me.

 It still working but is showing <unknown>.
Im using:
     - command:
       - /metrics-server
       - --kubelet-insecure-tls
       - --kubelet-preferred-address-types=InternalIP,Hostname,ExternalIP
       image: k8s.gcr.io/metrics-server-amd64:v0.3.3

and
	apiVersion: autoscaling/v2beta2
	kind: HorizontalPodAutoscaler
	metadata:
	  name: ingressos-api-jobs-hpa
	  namespace: production
	spec:
	  scaleTargetRef:
	    apiVersion: apps/v1
	    kind: Deployment
	    name: ingressos-api-jobs-deployment
	  minReplicas: 1
	  maxReplicas: 4
	  metrics:
	  - type: Resource
	    resource:
	      name: cpu
	      target:
	        type: AverageValue
	        averageValue: 1000Mi

/remove-lifecycle rotten

Same issue here

Hey, can you verify that Metrics Server is working using "kubectl top pod" and looking into metrics server logs?

@serathius You can see the log here https://github.com/kubernetes-incubator/metrics-server/issues/270#issuecomment-494261098

Sorry for not noticing it. I don't see any problems in metrics-server logs. Could you verify that Metrics API returns correct results for pods under HPA
kubectl get --raw /apis/metrics.k8s.io/v1beta1/namespaces/gaoce/pods?labelSelector=app%3Dtea

If metrics are correct the problem is not with Metrics Server but with HPA. Please create issue in https://github.com/kubernetes/kubernetes

Sure. But it cannot be always reproduced. I will have a look when I meet it again.


The identifier 'StyleTxt' could not be found.

I found that if I use "StyleExpr" option on page field I get error The identifier 'StyleTxt' could not be found.

StyleTxt is global

I get StyleTxt on aftergetrecord

There are no errors on compiler. I get error after publishing.

Duplicate of #538


Doesn't work with Kicad 7

Hi guys and thank you for your hard work.
I tried recently to use this tool with Kicad 7 but I am getting this error :

Can you help me to fix it please ?
Thanks !

Interesting. I thought I'd got this working for KiCad 7 a couple of weeks ago, but it looks like I didn't deal with hierarchical sheets properly. (I thought I had...) I'll take a look later today. It should be a simple fix, I think.

Ok, thanks !!

It works now ! Thanks you !!!

Hi guys and thank you for your hard work. I tried recently to use this tool with Kicad 7 but I am getting this error :

Can you help me to fix it please ? Thanks !


MultiparameterAssignmentErrors

This issue is exactly the same as #27.
1.9.3p0 :002 > p = Profile.new("birthday(3i)" => "2000", "birthday(2i)" => "2", "birthday(1i)" => "31")
ActiveRecord::MultiparameterAssignmentErrors: 1 error(s) on assignment of multiparameter attributes
	from /Users/david/.rvm/gems/ruby-1.9.3-p0/gems/validates_timeliness-3.0.8/lib/validates_timeliness/extensions/multiparameter_handler.rb:63:in `execute_callstack_for_multiparameter_attributes_with_timeliness'
	from /Users/david/.rvm/gems/ruby-1.9.3-p0/gems/activerecord-3.2.1/lib/active_record/attribute_assignment.rb:122:in `assign_multiparameter_attributes'
	from /Users/david/.rvm/gems/ruby-1.9.3-p0/gems/activerecord-3.2.1/lib/active_record/attribute_assignment.rb:98:in `assign_attributes'
	from /Users/david/.rvm/gems/ruby-1.9.3-p0/gems/activerecord-3.2.1/lib/active_record/base.rb:495:in `initialize'
	from (irb):2:in `new'

I'm using version 3.0.8 of the plugin on Rails 3.2.1

Can you try using the plugin from master. I have some fixes for the multiparameter extension but have not released a gem update yet.

So I gather you have the html option text as "day" etc. but the value of the option is still nil?
<option value="">day</option> or <option value="day">day</option>
Even if the option value was "day", AR does a to_i on all the values and should end up as 0. This would still not produce a valid date or time value. Can you submit a failing spec for this?

I think the prompt values are a misdirection. Try turning off the prompt and you will probably get the same bad behaviour.

I don't understand. If I disable the prompt, then all values in select boxes will be valid values (1-31 for day, 1-12 for month, and some valid years). The bad behavior in this case does not occur. It's pointless (or maybe I'm missing something).
So, why should I disable prompt? I need it to force the user to select an initial date.

Ah sorry, I really meant try :include_blank => true to verify the prompt words
are not the problem. I will investigate it more.
On 6/03/12 4:14 AM, idavemm wrote:

I don't understand. If I disable the prompt, then all values in select boxes will be valid values (1-31 for day, 1-12 for month, and some valid years). The bad behavior in this case does not occur. It's pointless (or maybe I'm missing something).
So, why should I disable prompt? I need it to force the user to select an initial date.

Reply to this email directly or view it on GitHub:
https://github.com/adzap/validates_timeliness/issues/67#issuecomment-4327023

Ah yes, :include_blank also has this issue. I have tested it.
Thanks for investigating it!


Missing time import with preview.37

RP: recoveryservices, dataprotection
Bug: missing time import for the splitted custom marshallers and unmarshallers file

Fixed in upcoming preview.38


feat: port forward

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

Describe the solution you'd like

[x] be able to port-forward
[x] be able to close port-forward
[x] see what port-forwards are currently active

#122 solves this


Implement not and neq

Fixing #37 and #38.
@kyprifog Would you like to check, if this solves your use-case?

Also related to #11

Codecov Report

Merging #40 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #40   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          620       633   +13
  Branches        82        83    +1
=========================================
+ Hits           620       633   +13

Impacted Files
Coverage Δ

dask_sql/physical/rex/core/call.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

I was able to test
SELECT * from table where NOT col = 'value'

and this seems to work properly.   I'm still trying to figure out how to test
SELECT * from table where col IS NOT NULL

where i'm getting an error related to isnan.   I have to decide if this even makes sense, this aspect of dask/pandas is a bit more confusing than spark.

^ @tomaugspurger , want to comment on the state of pandas nullable floats? I would say keep using pandas style isnan, and don't worry about NaN

Open PR at https://github.com/pandas-dev/pandas/pull/34307.
On Fri, Sep 18, 2020 at 3:41 PM Martin Durant<EMAIL_ADDRESS>wrote:

^ @TomAugspurger https://github.com/TomAugspurger , want to comment on
the state of pandas nullable floats? I would say keep using pandas style
isnan, and don't worry about NaN
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nils-braun/dask-sql/pull/40#issuecomment-695075936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAOIXDC2B7DATJ3D3AO6TSGPAWNANCNFSM4RRVPXBA
.

@kyprifog Thanks for testing! Could you maybe share some test data and the error message for the NULL case? Because for me, it seems to work:
import dask
import dask_sql
import numpy as np

context = dask_sql.Context()
df = dask.datasets.timeseries()

# Fill in some NaN here and there
df["x"] = df["x"].apply(lambda x: np.NaN if x > 0 else x, meta=("x", "float64"))

context.register_dask_table(df, "timeseries")
print(context.sql("SELECT * FROM timeseries WHERE x IS NOT NULL", debug=True,).compute())

@martindurant I tried to read through the linked discussion (thanks for that!), but I am now a bit unsure what the current best usage is. Are you proposing to move to df.isna instead of np.isnan? Sorry if I understood this incorrectly...

@nils-braun I think the problem I was facing is most closely exemplified if you change your query above:
SELECT * FROM timeseries WHERE name IS NOT NULL

IE looking for None or its equivalent in a column with dtype O (object), which results in
E           TypeError("ufunc 'isnan' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''")

Object is the default datatype inferred by dask for any string columns with missing values so this seems like a pretty common occurence.   Digging deeper it looks like that since object is a less specific in these cases the type for missing values is float.nan (even though the other values have type str), so this may be the connection that @martindurant is driving at.   This isn't strictly a sql-dask problem, but looking at pandas documentation it seems like there is still talk of a concept of nullable type for object dtype:
https://pandas.pydata.org/pandas-docs/stable/user_guide/missing_data.html#missing-data
IE it should behave roughly like:
df.dropna(subset=["name"])

which works as expected.

Thank you @kyprifog for the excellent description. Now I finally understand :-)
This means, if moving to df.isna (which I did in the PR), should already solve the problem. At least your example use case works now:
df["name"] = df["name"].apply(
    lambda x: None if x.startswith("A") else x, meta=("x", "object")
)

sql = "SELECT name IS NOT NULL FROM timeseries"

I have also added tests for this.

Tested on my end and works, and fix makes sense :+1:


Manually reset Kourend Library known locations

Sometimes when the library resets, or when you log off or change worlds, or any number of things can cause the info in the plugin to no longer be accurate, I find myself just going blind until the plugin finally catches up.
It would be very beneficial to have a clear button to reset known locations, as if you were showing up for the first time in a while.

In the sidebar, click on the Book icon. You'll find what you want there.


Add test for very large heartbeat interval

E.g., set heartbeat timeout to 1000 sec.
Ensures that logs will still be forwarded to followers/learners.
Because log forwarding should be driven by new data that are written to the Leader, not by heartbeat.

first found in https://github.com/datafuselabs/databend/issues/3474

Originally posted by @drmingdrmer in https://github.com/datafuselabs/openraft/issues/54#issuecomment-1004663451

/assignme