text:
Signature Quality

The signature is of course made up of linear segments.  Is there a parameter that controls the size of these segments? Can they be made smaller so the signature looks more natural?

I have the same problem. To sign is very slow on most devices. I already tested galaxy tab 4 10" and Dell Venue 7. It worked well on my iPad Air 2, but just for signature it's to expensive :-)
You have some ideas how to speed up?
Regards
H

I have the same problem. To sign is very slow on most devices. I already tested galaxy tab 4 10" and Dell Venue 7. It worked well on my iPad Air 2, but just for signature it's to expensive :-)
You have some ideas how to speed up?
Regards
H

Wow.  Caught me by surprise here.  Have not been contacted on this old
issue in some time.  We are running very well most of the time on Amazon
Kindles.
Samsung Galaxy's worked for a while and then also got slow.  We joked that
they ran out of ink.  The pattern we've noticed is described below and I'd
love to better understand it.
Once I downloaded Chrome onto the Samsung instead of using the default
browser, I did not have a speed problem anymore.  And the default browser
worked early on but not after a Samsung update.
So there seems to maybe be a  subtle issue on certain browsers.  We even
noticed that if we refreshed the page then sometimes it would work.  It
almost seems like a subtle timing problem with maybe when the canvas get
initialized compared to some other init process.
Your email is cdn123 and signature is H.  Can I know anymore about you?
On Wed, Apr 8, 2015 at 11:29 AM, cdn123 notifications@github.com wrote:

I have the same problem. To sign is very slow on most devices. I already
tested galaxy tab 4 10" and Dell Venue 7. It worked well on my iPad Air 2,
but just for signature it's to expensive :-)
You have some ideas how to speed up?
Regards
H
—
Reply to this email directly or view it on GitHub
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-90966992
.

--
Ed (eCharbonnet)
Edward C. Charbonnet
Forgiveness does not change the past, but it does enlarge the future. Paul
Boese.
“The difference between genius and stupidity is that genius has limits.” –
Albert Einstein.

Thank you for your fast reply. I try to use your signature to sign a service report. Acutal it's a beta project.
Yes it feels like, that they have timing problems. The resolution seems to be very ok, but it's too slow. After finishing the signature you have to wait for 1 sec to see the whole signature. You have any idea to fix this?
You are using a kindle fire and this worked well? You have experience with windows tablets?

I’ll jump in as a user and past really tiny contributor.
We use signature-pad on Windows Tablets – both RT and Pro, Windows phones, Windows laptops, Android phones and Tablets including Kindle fire – but we always use the Chrome browser, we told our customers years ago we don’t support the ‘default’ browser that Android manufactures install..
We have not seen any slowness like you mention on any of our test devices and have yet to have a customer complain about performance.
We only support:

     IE (current versions), Chrome, FireFox on Windows

     IE on Windows RT

     Chrome on Android

     Safari and Chrome on iStuff. (Note Chrome on iStuff is not ‘real’ Chrome, it’s Safari with ‘chrome’ that makes it called ‘Chrome’)

Though we only occasionally test in Opera and SeaMonkey and had no problems the last time (several months ago.)
But as I said, we explicitly do not support any ‘default’ Android browsers – we had too many other problems unrelated to signature-pad to even attempt to test signatures, so I can’t comment or give any opinions on those.
I do acknowledge that my customers might consider 1 second to redraw to not be a problem so it is possible some of our customers have the slowdown and haven’t reported it to us. Our users are all signing work reports, and sometimes they sign multiple times at different stages – it is a Maintenance Management system. So, while it is possible, it would seem to me if they aren’t seeing instant feedback while they sign, that they would have complained! My signature I draw the outer whole first, then fill in the middle – so I could not possibly draw my signature without seeing what I am drawing instantly.
I hope that helps you narrow down the problem.
peter
From: cdn123 [mailto:notifications@github.com]
Sent: Wednesday, April 8, 2015 11:13 AM
To: thomasjbradley/signature-pad
Subject: Re: [signature-pad] Signature Quality (#76)
Thank you for your fast reply. I try to use your signature to sign a service report. Acutal it's a beta project.
Yes it feels like, that they have timing problems. The resolution seems to be very ok, but it's too slow. After finishing the signature you have to wait for 1 sec to see the whole signature. You have any idea to fix this?
You are using a kindle fire and this worked well? You have experience with windows tablets?
—
Reply to this email directly or view it on GitHub https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-90977755 .  https://github.com/notifications/beacon/AC7G08YAR2ZvFDxSsGikYxyBSvWvxOsbks5n9VkggaJpZM4AvFBg.gif

I totally agree that as long as we stay away from the android default
browser we've had no difficulties.  We have no experience using this on
Windows Tablets.
On Wed, Apr 8, 2015 at 12:57 PM, peter horwood aka Madman Pierre <
notifications@github.com> wrote:

I’ll jump in as a user and past really tiny contributor.
We use signature-pad on Windows Tablets – both RT and Pro, Windows phones,
Windows laptops, Android phones and Tablets including Kindle fire – but we
always use the Chrome browser, we told our customers years ago we don’t
support the ‘default’ browser that Android manufactures install..
We have not seen any slowness like you mention on any of our test devices
and have yet to have a customer complain about performance.
We only support:

IE (current versions), Chrome, FireFox on Windows

IE on Windows RT

Chrome on Android

Safari and Chrome on iStuff. (Note Chrome on iStuff is not ‘real’
Chrome, it’s Safari with ‘chrome’ that makes it called ‘Chrome’)

Though we only occasionally test in Opera and SeaMonkey and had no
problems the last time (several months ago.)
But as I said, we explicitly do not support any ‘default’ Android
browsers – we had too many other problems unrelated to signature-pad to
even attempt to test signatures, so I can’t comment or give any opinions on
those.
I do acknowledge that my customers might consider 1 second to redraw to
not be a problem so it is possible some of our customers have the slowdown
and haven’t reported it to us. Our users are all signing work reports, and
sometimes they sign multiple times at different stages – it is a
Maintenance Management system. So, while it is possible, it would seem to
me if they aren’t seeing instant feedback while they sign, that they would
have complained! My signature I draw the outer whole first, then fill in
the middle – so I could not possibly draw my signature without seeing what
I am drawing instantly.
I hope that helps you narrow down the problem.
peter
From: cdn123 [mailto:notifications@github.com]
Sent: Wednesday, April 8, 2015 11:13 AM
To: thomasjbradley/signature-pad
Subject: Re: [signature-pad] Signature Quality (#76)
Thank you for your fast reply. I try to use your signature to sign a
service report. Acutal it's a beta project.
Yes it feels like, that they have timing problems. The resolution seems to
be very ok, but it's too slow. After finishing the signature you have to
wait for 1 sec to see the whole signature. You have any idea to fix this?
You are using a kindle fire and this worked well? You have experience with
windows tablets?
—
Reply to this email directly or view it on GitHub <
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-90977755>
. <
https://github.com/notifications/beacon/AC7G08YAR2ZvFDxSsGikYxyBSvWvxOsbks5n9VkggaJpZM4AvFBg.gif>
—
Reply to this email directly or view it on GitHub
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-90988567
.

--
Ed (eCharbonnet)
Edward C. Charbonnet
Forgiveness does not change the past, but it does enlarge the future. Paul
Boese.
“The difference between genius and stupidity is that genius has limits.” –
Albert Einstein.

This is really strange. Did you test with android 4.4? I tried to use chrome, dolphin, opera, firefox. No improvements :-(
Maybe it's about the newest android version?

We are at this time entirely focused on the Kindle from Amazon and its
version of Android.  Not sure what that is.  But it works very smoothly.
On Thu, Apr 9, 2015 at 3:13 AM, cdn123 notifications@github.com wrote:

This is really strange. Did you test with android 4.4? I tried to use
chrome, dolphin, opera, firefox. No improvements :-(
Maybe it's about the newest android version?
—
Reply to this email directly or view it on GitHub
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-91146221
.

--
Ed (eCharbonnet)
Edward C. Charbonnet
Forgiveness does not change the past, but it does enlarge the future. Paul
Boese.
“The difference between genius and stupidity is that genius has limits.” –
Albert Einstein.

I just tried it on my Android 4.4.4 machine, using chrome 34.0.1847.114. Moto G (fairly slow phone for modern smart phone – I usually buy fairly slow ones for testing) Says total space is 5.5 Gig. Lots of free memory (about 80%).
The signature is instantaneous, following my finger as I sign.
From: Edward C Charbonnet [mailto:notifications@github.com]
Sent: Thursday, April 9, 2015 10:05 AM
To: thomasjbradley/signature-pad
Cc: peter horwood aka Madman Pierre
Subject: Re: [signature-pad] Signature Quality (#76)
We are at this time entirely focused on the Kindle from Amazon and its
version of Android. Not sure what that is. But it works very smoothly.
On Thu, Apr 9, 2015 at 3:13 AM, cdn123 <notifications@github.com mailto:notifications@github.com > wrote:

This is really strange. Did you test with android 4.4? I tried to use
chrome, dolphin, opera, firefox. No improvements :-(
Maybe it's about the newest android version?
—
Reply to this email directly or view it on GitHub
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-91146221
.

--
Ed (eCharbonnet)
Edward C. Charbonnet
Forgiveness does not change the past, but it does enlarge the future. Paul
Boese.
“The difference between genius and stupidity is that genius has limits.” –
Albert Einstein.
—
Reply to this email directly or view it on GitHub https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-91275150 .  https://github.com/notifications/beacon/AC7G09Hn1ZPemY2YNvI4L3kaLPWJlxMwks5n9pqjgaJpZM4AvFBg.gif

OK. I'm using Dell Venue 7 with Chrome v35.
I've made 2 videos. First with my dell, second with my iPad. Can you confirm the slowness on reaction?
http://www.yourfilelink.com/get.php?fid=1053788
http://www.yourfilelink.com/get.php?fid=1053789

Can you agree with the speed with you experience?

I've had no luck trying to see these videos.
When I go to the forum and click on your links they just take me to a place
that wants me to download a FileLink application.
On Mon, Apr 13, 2015 at 2:51 AM, cdn123 notifications@github.com wrote:

Can you agree with the speed with you experience?
—
Reply to this email directly or view it on GitHub
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-92255922
.

--
Ed (eCharbonnet)
Edward C. Charbonnet
Forgiveness does not change the past, but it does enlarge the future. Paul
Boese.
“The difference between genius and stupidity is that genius has limits.” –
Albert Einstein.

Here we go again with youtube links:
https://youtu.be/QyNMwuiDxjE
https://youtu.be/yWLHlJX7eos

Those looked quite good to me.
You were writing fast and the line seemed to be keeping up with you.
On Mon, Apr 13, 2015 at 5:45 AM, cdn123 notifications@github.com wrote:

Here we go again with youtube links:
https://youtu.be/QyNMwuiDxjE
https://youtu.be/yWLHlJX7eos
—
Reply to this email directly or view it on GitHub
https://github.com/thomasjbradley/signature-pad/issues/76#issuecomment-92304943
.

--
Ed (eCharbonnet)
Edward C. Charbonnet
Forgiveness does not change the past, but it does enlarge the future. Paul
Boese.
“The difference between genius and stupidity is that genius has limits.” –
Albert Einstein.

authors: cdn123 eccharbonnet madmanpierre


text:
🛑 Syncthing-KA-TM is down

In cbbcc09, Syncthing-KA-TM ($URL_SYNCTHING_MERZ_NIMBUS_TM) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Syncthing-KA-TM is back up in 7ba64ff.

source: gharchive/issue

repo: thomasmerz/upptime

license: MIT

license_type: permissive

license_source: github-api


text:
🛑 Syncthing-NBG is down

In 3db6a27, Syncthing-NBG ($URL_SYNCTHING_HETZNER_CLOUDSERVER_DE) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Syncthing-NBG is back up in dfd55f8 after 10 minutes.


text:
IP Whitelist

I have added the posibility to whitelist one or more IP/CIDR addresses.
I use it to allow free access from inside my enterprise network but requiring login when doing it from outside.
It's not a perfect implementation but works well. Also I have created a realip helper that allows to extract the real user ip when working behind a proxy like cloudflare or an enterprise proxy.

Hi,
Thanks for your work on this so far - at the moment this isn't an immediate merge for me as I don't believe white listing IPs is in keeping with the BeyondCorp model of authentication. My main issue is that you'd be unable to identify which user sent which request, so you could not offer fine grained access control on a per-service basis.
What's your use case for bypassing the authentication?

We use this bypass in our preproduction environment.
In our case when we are in the office we don't need authentication to be faster, but we want to authenticate foregein access.
Also we have some private network access, in our case we allow access from 10.0.0.0/8 ip network because some internal api calls that must satisfy the same processing.

OK, another option would be to create another traefik frontend (on another port) with an IP whitelist in there to only allow access from your desired subnets?
I will leave this open for now so anyone who supports this can weigh in, but I'm not sure I love this concept at the moment.

Trusting these headers to determine source IP is a risky and not great security as any user could spoof these headers and you would assume they are coming from internal address.
You can only trust headers like this if they are injected by the proxy and you have total trust that the proxy will strip out any spoofed headers.
Any user that isn't running behind cloudflare would be exposed by just doing curl -H CF-Connecting-IP: 10.0.0.1 or something similar... and traefik / this libary would never know the difference.

I'm closing this based on the earlier discussion, I'm glad you were able to implement a working solution for your use case but as noted, there's some risk associated with this method.
Thanks for taking the time to build the PR

source: gharchive/pull-request

authors: drubin elliotmoso thomseddon

repo: thomseddon/traefik-forward-auth


text:
Produce a complete ONIX record

At present we output ONIX files based on the specifications provided by each platform, which are often incomplete. We should produce a complete record that implements our data model in full, and output it under onix_3.0::thoth

Make notes and report back on any Thoth fields which are not (fully) supported by ONIX

data model comparison.odt

repo: thoth-pub/thoth


text:
Add an adapter for Campaign Monitor

Hey 👋
We've needed to migrate from SendGrid to Campaign Monitor, so we've written an adapter for it, and largely based it off the SendGrid adapter.
Currently, it supports classic email and smart email, but Smart email is not documented or tested well yet. I wanted to check first to see if this is something you want to add before spending too much time on it, happy to update it before merging too.
https://www.campaignmonitor.com/api/transactional/#send-classic-email
https://www.campaignmonitor.com/api/transactional/#send-smart-email

@jackmarchant Thanks for this! At this point our preference is for new adapters to be their own separate libraries. See for example the others under https://github.com/thoughtbot/bamboo#available-adapters. Can you implement this as its own library? Then we can add a link to your new one to the list as well. 😄

Yes, no problem at all! I just thought I’d submit a PR here in case. I’ll submit a PR when it’s ready
Thank you!

https://github.com/thoughtbot/bamboo/pull/480
@maymillerricci

authors: jackmarchant maymillerricci

repo: thoughtbot/bamboo


text:
drag_to support

What's the status of drag_to with capybara webkit?  I just tried something like:
page.find("#something").drag_to(page.find("#another))

Only to get the error:
NotImplementedError

Should this be working by default?  Do I need to do something to get this to work?
Capybara 1.1.2
Capybara Webkit 0.7.2

Are you sure that you're using capybara-webkit and not rack-test as the
driver for that scenario? Can you paste the full backtrace?

oops.  sorry, you're right, forgot my :js => true for that particular context.

authors: bradrobertson jferris

repo: thoughtbot/capybara-webkit

license: mit

license_source: bigquery


text:
Problems with webkit_server through Docker

I'm seeing the following error for tests that have :js
  1) creating move edits the move
     Failure/Error: visit new_move_path(as: user)
     Capybara::Webkit::ConnectionError:
       /box/gems/capybara-webkit-1.8.0/bin/webkit_server failed to start.
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit/connection.rb:87:in `parse_port'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit/connection.rb:93:in `discover_port'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit/connection.rb:74:in `start_server'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit/connection.rb:31:in `initialize'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit/driver.rb:14:in `new'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit/driver.rb:14:in `initialize'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit.rb:15:in `new'
     # /box/gems/capybara-webkit-1.8.0/lib/capybara/webkit.rb:15:in `block in <top (required)>'
     # /box/gems/capybara-2.6.2/lib/capybara/session.rb:84:in `driver'
     # /box/gems/capybara-2.6.2/lib/capybara/session.rb:70:in `initialize'
     # /box/gems/capybara-2.6.2/lib/capybara.rb:276:in `new'
     # /box/gems/capybara-2.6.2/lib/capybara.rb:276:in `current_session'
     # /box/gems/capybara-2.6.2/lib/capybara/dsl.rb:46:in `page'
     # /box/gems/capybara-2.6.2/lib/capybara/dsl.rb:51:in `block (2 levels) in <module:DSL>'
     # ./spec/features/credits/moves_spec.rb:24:in `block (2 levels) in <top (required)>'
     # ./spec/support/headless.rb:4:in `block (3 levels) in <top (required)>'
     # /box/gems/headless-2.2.3/lib/headless.rb:134:in `run'
     # ./spec/support/headless.rb:3:in `block (2 levels) in <top (required)>'
     # /box/gems/activesupport-4.2.5.1/lib/active_support/dependencies.rb:268:in `load'
     # /box/gems/activesupport-4.2.5.1/lib/active_support/dependencies.rb:268:in `block in load'
     # /box/gems/activesupport-4.2.5.1/lib/active_support/dependencies.rb:240:in `load_dependency'
     # /box/gems/activesupport-4.2.5.1/lib/active_support/dependencies.rb:268:in `load'
     # /box/gems/spring-commands-rspec-1.0.4/lib/spring/commands/rspec.rb:18:in `call'
     # -e:1:in `<main>'

The command I run is docker-compose run web bin/rspec spec/features.
Dockerfile:
FROM ruby:2.3.0

RUN apt-get update -qq && apt-get install -y build-essential

# for postgres
RUN apt-get install -y libpq-dev

# for nokogiri
RUN apt-get install -y libxml2-dev libxslt1-dev

# for capybara-webkit
RUN apt-get install -y libqt4-webkit libqt4-dev xvfb

# for a JS runtime
RUN apt-get install -y nodejs

RUN mkdir /app
WORKDIR /app
ENV BUNDLE_PATH /box
COPY . /app/

docker-compose.yml:
db:
  image: postgres:9.4
  ports:
    - "5432"
web:
  build: .
  command: bin/rails s -b 0.0.0.0 -p 3000
  volumes:
    - .:/app
  volumes_from:
    - box
  stdin_open: true
  ports:
    - "3000:3000"
  links:
    - db
box:
  image: busybox
  volumes:
    - /box

If I jump into the container with docker-compose run web bash and run web_server manually I see this
root@3f00491727e9:/app# /box/gems/capybara-webkit-1.8.0/bin/webkit_server
/box/gems/capybara-webkit-1.8.0/bin/webkit_server: error while loading shared libraries: libQt5WebKitWidgets.so.5: cannot open shared object file: No such file or directory

I have tried using headless as described here but that didn't work.
Any idea what I'm doing wrong?

I have now read through the installation instructions in the wiki but it didn't help. My Dockerfile now looks like
FROM ruby:2.3.0

RUN apt-get update -qq && apt-get install -y build-essential

# for postgres
RUN apt-get install -y libpq-dev

# for nokogiri
RUN apt-get install -y libxml2-dev libxslt1-dev

# for capybara-webkit
RUN apt-get install -y libqt5webkit5-dev gstreamer1.0-plugins-base gstreamer1.0-tools gstreamer1.0-x xvfb libqt4-webkit libqt4-dev

# for a JS runtime
RUN apt-get install -y nodejs

RUN mkdir /app
WORKDIR /app
ENV BUNDLE_PATH /box
COPY . /app/

Is it possible that you're providing vendored/compiled gems from the host system or a different Docker container? If the necessary libraries aren't present, capybara-webkit can't compile, so it looks like it was compiled when the libraries were available but then run in an environment where they weren't. How are you running bundle install in the Docker environment?

Yes I am installing gems into another container that runs busybox. This is to cache gems and make bundle install faster. So sounds like that might be the issue.
Any ideas how I would fix that?

I haven't worked with that particular Docker workflow, but you'll need to compile capybara-webkit on a host that has the linked libraries at the same locations as the host you're going to run the tests on.
This doesn't look like a bug in capybara-webkit, so I'm going to close this issue. Feel free to continue commenting if you have further questions or ideas.

I've spent lots of time trying to solve this issue.
Here's what I've found:

Instead of running rspec you should run xvfb-run -a rspec

It seems capybara needs a X Server, so that's for it. Make sure you have xvfb installed.
In case your docker base image is CentOs, you may need run one more step.

dbus-uuidgen > /var/lib/dbus/machine-id

You can do this either adding a RUN at your Dockerfile or running it before a test.

authors: davidpdrsn igorvpcleao jferris


text:
Do not validate user when resetting password

User records can be invalid for many reasons that are not central to
Clearance's concerns. An application may decide at some point to begin
requiring all user records to have a zip code. They reasonably assume this
will effect only new registrations or existing users who attempt to edit
their profile. Unfortunately, it would have also prevented users from
resetting their password because we were running validations as part of
that operation. With zip code not present on the password reset form, the
user would have no way to complete the password reset.
This change still calls valid? but considers only errors that are on the
password field when deciding whether to proceed. This allows for
additional validations on password, such as length. Ultimately, the save
is done without validations assuming password itself is valid.
The nested before blocks in the specs caused a great deal of frustration when
adding tests for this functionality. The specs were re-written in a style
consistent with how we write specs today, taking care to ensure tests still
fail appropriately.
Closes #406

couple of comments otherwise LGTM

I'm going to merge in the spec improvements but leave the functionality unchanged. @mxie is working on a new password reset mechanism and I think given the problems in implementing this here it's reasonable for this issue to be addressed only there.

authors: MattMSumner derekprior

repo: thoughtbot/clearance


text:
Your Ruby version is 3.1.3, but your Gemfile specified 3.0.5

When running suspenders app_name on this commit, I get the following error:
Your Ruby version is 3.1.3, but your Gemfile specified 3.0.5

This might relate to #1073, but requires further investigation, since we just updated our supported Ruby version.

@stevepolitodesign Is this issue a duplicate of:
https://github.com/thoughtbot/suspenders/issues/1073

@thiagoa thank you for referencing that issue. I think this might be related, although in my case, the script fails to continue.

@stevepolitodesign It's because Suspenders is now aborting:
https://github.com/thoughtbot/suspenders/pull/1087
When the issue was created, it didn't abort when a subprocess failed. However, the issue seems to be the same still. Thoughts on updating it?

Adding more context, the idea of that issue is to make the Ruby version more flexible. Suspenders forces us to run with the exact Ruby version declared here:
https://github.com/thoughtbot/suspenders/blob/main/.ruby-version
Which isn't great. That issue has a few thoughts on how we could fix the problem.

@thiagoa ah, thank you for the additional context. I will close this in favor of the existing issues 👍
Also, in case you're wondering why we settled on version 3.0.5 and not the latest version of Ruby, it's because

We were unable to support the latest version of Ruby because of a failing build. A future commit could explore how to solve for this, but for now, we decided to just stick with version 3.0.5. We settled on 3.0.5 and not 3.0.0 in an effort to suppress parser warnings.

authors: stevepolitodesign thiagoa

repo: thoughtbot/suspenders


text:
Add navbar with dropdown on webstack template

Elements in Navbar:

[x] Home (logo linking to home page)
[x] WHY (about page)
[x] LEARN : dropdown : PRODUCT; READ MORE (blog); KNOWLEDGE; COMMUNITY (TF forum)
[x] PEOPLE : dropdown : TEAM; PARTNERS
[x] MORE : dropdown : NEWSROOM; BLOG; FORUM; CONTACT; COMPANY

We are missing dark/light switch mode.

authors: Vilnite samaradel


text:
Error in mnemonic words

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/base/factory.py", line 206, in get
    instance = self.find(name)
               │         └ 'main'
               └ <jumpscale.core.base.factory.jumpscalecoreidentityIdentity object at 0x7f9bef44a780>
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/base/factory.py", line 439, in find
    instance = super().find(name)
                            └ 'main'
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/base/factory.py", line 129, in find
    instance = getattr(self, name, None)
                       │     └ 'main'
                       └ <jumpscale.core.base.factory.jumpscalecoreidentityIdentity object at 0x7f9bef44a780>
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/base/factory.py", line 492, in getter
    instance = self._get_object_from_config(name, factory.store.get(name))
               │                            │     │                 └ 'main'
               │                            │     └ <jumpscale.core.base.factory.jumpscalecoreidentityIdentity object at 0x7f9bef44a780>
               │                            └ 'main'
               └ <jumpscale.core.base.factory.jumpscalecoreidentityIdentity object at 0x7f9bef44a780>
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/base/factory.py", line 404, in _get_object_from_config
    instance = self._create_instance(name, **data)
               │                     │       └ {'_tid': 195, 'words': 'panda foil grit addict snake dwarf quality swallow copper grunt swallow repair arm forward myself earl n...
               │                     └ 'main'
               └ <jumpscale.core.base.factory.jumpscalecoreidentityIdentity object at 0x7f9bef44a780>
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/base/factory.py", line 161, in _create_instance
    instance = self.type(*args, **kwargs)
               │          │       └ {'_tid': 195, 'words': 'panda foil grit addict snake dwarf quality swallow copper grunt swallow repair arm forward myself earl n...
               │          └ ()
               └ <jumpscale.core.base.factory.jumpscalecoreidentityIdentity object at 0x7f9bef44a780>
  File "/sandbox/code/github/threefoldtech/js-sdk/jumpscale/core/identity/__init__.py", line 46, in __init__
    self.verify_configuration()
    └ Identity(_Base__instance_name='main', _Base__parent=None, ___tid=195, __admins=['play.3bot', 'waleedhammam.3bot', 'ranatarek.3bo...
  File "/sandbox/code/github/threefoldtech/js-sdk/jumpscale/core/identity/__init__.py", line 69, in verify_configuration
    if self.nacl.get_verify_key_hex() != user.pubkey:
  File "/sandbox/code/github/threefoldtech/js-sdk/jumpscale/core/identity/__init__.py", line 51, in nacl
    seed = mnemonic.mnemonic_to_key(self.words.strip())
           │                        └ Identity(_Base__instance_name='main', _Base__parent=None, ___tid=195, __admins=['play.3bot', 'waleedhammam.3bot', 'ranatarek.3bo...
           └ <module 'jumpscale.data.encryption.mnemonic' from '/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/d...
  File "/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/data/encryption/mnemonic.py", line 60, in mnemonic_to_key
    raise exceptions.NotFound('Unable to find "%s" in word list.' % word)
          │                                                         └ 'earl'
          └ <module 'jumpscale.core.exceptions' from '/root/.cache/pypoetry/virtualenvs/js-sdk-sEfpU_Ir-py3.6/src/js-ng/jumpscale/core/excep...
jumpscale.core.exceptions.exceptions.NotFound: Unable to find "earl" in word list.

special case for @ahelsayd identity 😁

wrong input from user

repo: threefoldtech/js-sdk


text:
Fix sending locked tokens

Solves #1737

Codecov Report

Merging #1741 (6ca7329) into development (11be30c) will increase coverage by 0.00%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##           development    #1741   +/-   ##
============================================
  Coverage        32.36%   32.36%
============================================
  Files              134      134
  Lines            12843    12843
============================================
+ Hits              4156     4157    +1
+ Misses            8687     8686    -1

Impacted Files
Coverage Δ

jumpscale/clients/stellar/stellar.py
21.87% <40.00%> (+0.20%)
:arrow_up:

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 11be30c...6ca7329. Read the comment docs.

authors: codecov-io robvanmieghem


text:
update default wikis to info_threefold

https://github.com/threefoldtech/jumpscaleX_threebot/blob/ecd6452aceffee4a37ad55347484dd3e9684a207/ThreeBotPackages/threefold/wikis/package.py#L8
TF_WIKIS_LINKS = {
    "info_grid": f"https://github.com/threefoldfoundation/info_grid/tree/{BRANCH}/docs",
    "info_foundation": f"https://github.com/threefoldfoundation/info_foundation/tree/{BRANCH}/docs",
    "info_tokens": f"https://github.com/threefoldfoundation/info_tokens/tree/{BRANCH}/docs",
}

all of these to be replace with github.com/threefoldtech/info_threefold

3aae01fa44a1991659035cc8c49964197ffb772d
76c2061967f58b97f05fc523116d9c670273a5a9


text:
add nodes legend

Description
Add a legend for nodes table
Changes

Related Issues

#2038

Checklist

[ ] Tests included
[x] Build pass
[x] Documentation
[x] Code format and docstrings
[x] Screenshots/Video attached (needed for UI changes)

Pr not following template, please update it.

Pr not following template, please update it.

done

I think there should be spaces between the chips along the row.

@ehab-hassan are you agree?

I think there should be spaces between the chips along the row.

@AlaaElattar we tested it before was not good especially on large screens too much space also we need to make it like the legend.

authors: AlaaElattar ehab-hassan samaradel

repo: threefoldtech/tfgrid-sdk-ts


text:
add optional metrics collector

measure page_requests_count, page_requests_count, page_fetch_time, and value_fetch_time

#193  👈
master

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

Join @gabriele-0201 and the rest of your teammates on  Graphite

repo: thrumdev/nomt


text:
Auto-indentation madness.

I'm experiencing some extraordinary behavior with the new auto-indentation feature.

Note that after typing class Bar: I'm not pressing any other keys.  That just happens by itself on the last line of the file

Thank you for reporting this, will be fixed promptly

Fixed in b6ac74ee
v0.2.2 is now on the marketplace
Thanks for reporting :+1:

Thank you!  This fixed the method parameter indentation weirdness and the infinite spacing problem.


text:
Feature/3 Implement Footer

Desktop

Mobile

Reduce duplicate code and check the font in your project. It's look diff with design


text:
Migrate to production

Siirtyminen tämän projektin kautta tulisi tehdä localhostissa ja
sen jälkeen integrointi verkkoon tapahtuu puhtaalla koodilla

wp-config.php:n kautta onnistuu
olisi kiva myös koulussa meneillään olevalle projektille, jos sama migraatio työkalu auttaisi esim. testidatan pystyttämisessä.

Voisitko laittaa exportin (database dump) versiohallintaan ja screenshotin tähän issuelle (lista kaikista asennetuista plugareista yms).
Tämän jälkeen yritän asentaa itse ja kokeilla

Näytit tänään ohjeita ja niitä pitäisi hieman tarkentaa, sillä ajatuksella että "kuka vaan joka saa nämä ohjeet, pystyy niiden avulla 100% onnistumaan migraatiossa)

repo: thxmuffe/kakisite


text:
Remove outdated fallbacks

It seems that there are a couple of fallback that we can now remove:

scroll-margin. This property is now widely available: https://caniuse.com/mdn-css_properties_scroll-margin. Having scroll-snap-margin as a fallback

https://github.com/thysultan/stylis/blob/f7bbabecb4801659bf6c3223b52d5dc7460ca6f3/test/Prefixer.js#L268
doesn't seem to make a lot of sense. It seems that scroll-snap-margin was an experimentally CSS property used before 2017: https://github.com/w3c/csswg-drafts/issues/1954. That feature is barely used https://chromestatus.com/metrics/css/popularity.

others?

Related to #334

cc @brijeshb42 adding this in your radar. It fits well under the "improving the style engine of MUI" north start objective of your team (so far 1 person but hopefully we can grow this).

repo: thysultan/stylis


text:
Generate auto-document when wrapping all response into a custom object with error code, message, and data.

Question
Hi, I'm working on a backend project, and so far all fastapi code works well. In the fastapi official document, we use HTTP status code to identify errors. However, in my project, all responses are wrapped by a custom model and always return 200 status code. For example, if I cannot find a resource in my database, I will return the following object and the HTTP status code is 200:
{
  "code":1,
  "message":"Cannot find the resource",
  "data":{}
}

If I want to return success, then the response will be
{
  "code":0,
  "message":"OK",
  "data":{"name":"foo"}
}

It is easy to implement this custom response, but I cannot find a way to generate corresponding documents automatically compared with the REST API in the official document. My current solution is to define a wrapper for every original response model, for example, assume that my normal response model is
class Person(BaseModel):
    name:str
    age:str

My wrapper model is
class PersonResponse(BaseModel):
    code:int
    message:str
    data: Person

Since I have about 100 response models, I have to repeat myself 100 times. Do you guys have some better methods? Or is there a way to dynamically generate these models and put them as the response_model param?

I suggest you look at the documentation for generic models, they're exactly what you're asking for: https://pydantic-docs.helpmanual.io/usage/models/#generic-models

I suggest you look at the documentation for generic models, they're exactly what you're asking for: https://pydantic-docs.helpmanual.io/usage/models/#generic-models

I suggest you look at the documentation for generic models, they're exactly what you're asking for: https://pydantic-docs.helpmanual.io/usage/models/#generic-models

I suggest you look at the documentation for generic models, they're exactly what you're asking for: https://pydantic-docs.helpmanual.io/usage/models/#generic-models

@shengwubin i hava the same problems, will you show me the solution code ?

@shengwubin i hava the same problems, will you show me the solution code ?

@XiaobinZhao Define somethings like this:
DataT = TypeVar("DataT")

class BaseResponse(GenericModel, Generic[DataT]):
    code: int
    message: str
    data: Optional[DataT] = None

class SuccessResponse(BaseResponse[DataT], Generic[DataT]):
    code: int = 0
    message: str = "OK"

class ErrorResponse(BaseResponse[DataT], Generic[DataT]):
    code: int = 1
    message: str = "Error"

Then wrap data when returning:
return SuccessResponse[Person](data=data)

# or

return ErrorResponse[dict](message="Cannot find the resource", data={})

Not so good, but just worked.

Thanks, @mookrs  it worked perfectly thanks @XiaobinZhao you can check how it is implemented on response_schema https://github.com/jonra1993/fastapi-alembic-sqlmodel-async
https://github.com/jonra1993/fastapi-alembic-sqlmodel-async/blob/main/fastapi-alembic-sqlmodel-async/app/schemas/response_schema.py

Thanks for the help here everyone! 👏  🙇
Thanks for reporting back and closing the issue 👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.

authors: Mause XiaobinZhao jonra1993 mookrs shengwubin tiangolo

repo: tiangolo/fastapi


text:
ModuleNotFoundError: No module named 'src'

Hello everyone. I am new to Python and focused about imports. I researched for a long time but could not solve the problem. My project structure is like below:
── src
    ├── app
    │   ├── api
    │   │   ├── main.py
    │   │   └── model
    │   │       ├── JobMatchResult.py

// JobMatchResult.py

from pydantic import BaseModel, Schema

class JobMatchResult(BaseModel):

and in main.py file i am importing JobMatchResult like:
from src.app.api.model.JobMatchResult import JobMatchResult
once i try to run server with command below:
uvicorn main:app --reload
i am getting error ModuleNotFoundError: No module named 'src'
Can anyone help me please?

this is not a FastAPI question, a good read would be https://chrisyeh96.github.io/2017/08/08/definitive-guide-python-imports.html# it's extremely detailed how imports work, my guess is that in your main.py you should do just from model import JobMatchResult

@euri10 Thank for reply. I have read before link you. Also i read a few long articles about import statements in Python. I solved the problem 5 mins ago by writing command int terminal like:
 uvicorn src.app.api.main:app

this is not a FastAPI question, a good read would be https://chrisyeh96.github.io/2017/08/08/definitive-guide-python-imports.html# it's extremely detailed how imports work, my guess is that in your main.py you should do just from model import JobMatchResult

Thanks for the help here @euri10 ! :clap: :bow:
Thanks for reporting back and closing the issue @tsdmrfth  :+1:

authors: euri10 tiangolo tsdmrfth


text:
🌐 Add Tamil translation for docs/ta/docs/features.md

#5434
This PR adds translation of features.md to tamil.

Great! Let me know when the PR is ready for review and I'll add the label awaiting review so that others can come and review it and then I can merge it. :rocket:

📝 Docs preview for commit f15f6bc0ea47fe57de3796b69eb9dc22a710a40e at: https://649a2f0297baca1ed99aa116--fastapi.netlify.app

📝 Docs preview for commit 559aafe2f3b4d1654d55376b5b38ce967d09effb at: https://649a31ce1636a4208c534ca6--fastapi.netlify.app

Great @dineshsrd! Yes, please help review the PR.
You can see the instructions here: https://fastapi.tiangolo.com/contributing/#translations that includes a link with instructions to add a review to a PR.

@dineshsrd, it would be great if you could add your review to the PR. You would have to go to the "Files" tab and there click the button to add your review.

@dineshsrd If you happen to know two other Tamil-speaking developers that you can also review this PR, even better. Here's how you can review PRs btw.

@tiangolo Review done and comments added.

@dineshsrd thank you for the review. i will update the docs with suggestions by end of this week.

I am closing this pull request as I am not able to prioritize time for this, sorry!

authors: arunppsg codespearhead dineshsrd tiangolo


text:
Add max of 16 chars to be created in the same account

Should limit to 16 players to a single account. In other words, POST /account/player should not allow player creation if it the account has already 16 players.

Manda pro pai

authors: ClaudioCarvalhoo felipelaptrin


text:
🛑 TICKETS.TJ is down

In 3daec30, TICKETS.TJ (https://tickets.tj) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TICKETS.TJ is back up in d3b159b after 1 hour, 2 minutes.

repo: tickets/upptime


text:
🛑 TICKETS.ORG.IN is down

In a0ae3f2, TICKETS.ORG.IN (https://tickets.org.in) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TICKETS.ORG.IN is back up in e53616e after 2 hours, 32 minutes.


text:
🛑 TICKETS.SV is down

In 176267f, TICKETS.SV (https://tickets.sv) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TICKETS.SV is back up in 67a4bc6 after 46 minutes.


text:
Memory Usage when sending a large message

Hi, it seems that if I send a large string to a different socket server, after sending the message, the ram is never released. The string is 5MB.
Works fine, but.. even after the message is send, and closed.. ram is not released.
ws.send (text: self.jpegPhotoData!.base64EncodedString(options: .lineLength64Characters))
Any ideas?

@jshafer817, I noticed this library creates some nasty retain cycles by virtue of the functional interface design of the open, close, error, and message methods on event. I did not see an obvious way to provide a weak reference to the library for those callbacks and decided the best way to stop the bleeding was to wrap the Websocket object in another object that is allocated only once. I hope the memory issues get fixed in future builds, but expect it'll require a significant change in how we interface with the library.

"wrap the Websocket object in another object that is allocated only once".. Do you have an example you are willing to share?

This is my wrapper object (https://gist.github.com/gacrichards/e652ac7281218dffbc5b72ce1b08c898). The basic idea here is that I take the functional bindings of a Websocket() instance and map them to a weakly held delegate owned by my wrapper object. I only ever want to create one instance of my wrapper object since it is in a strong reference cycle with it's websocket object - releasing my wrapper will cause a leak. In many of my iOS apps I have a dependency factory which is a singleton and holds on to core objects (as few as possible) that need to be injected throughout the app. I place my wrapper object in the dependency factory and that gives me pretty tight control on its life cycle. Here is an example of what that might look like (https://gist.github.com/gacrichards/fb714b34ea5304729b8fae26fec23196). The only state I care about for my application is the request that I use to open the websocket since some websocket connections need to be authorized with an access token. I use my wrapper object add authorization or revoke authorization from that request, i.e. to mutate any state. Fortunately the socket doesn't appear to hold a strong reference to the request so I'm free to change that as I need. The obvious drawback to this approach is that I'll always have a websocket and wrapper object in memory, but that is better, IMHO, than leaking the socket and any other objects that it comes into contact with. If I've gone off in a weird and unfruitful direction I'd love to be corrected, but this passes memory profiling tests for my app.

@jshafer817 , thanks, do you mean you find that

sending ordinary small messages it is released

but only "large" messages are not released

?
It seems strange there would be any difference, you know?  What do you think?

authors: gacrichards jshafer817 smhk

repo: tidwall/SwiftWebSocket


text:
奔溃

请求完数据出现奔溃现象

(void)_productWithTargetView:(UIView *)targetView isCard:(BOOL)isCard这个方法

请看下，这个问题很严重

图片加载不了

就是奔溃到我复制的那个bughttps://user-images.githubusercontent.com/18162711/91036340-bdcbdd00-e639-11ea-8f72-808b15f22f35.png

https://user-images.githubusercontent.com/18162711/91036340-bdcbdd00-e639-11ea-8f72-808b15f22f35.png

提供具体堆栈

不是必现的，偶尔出现，这个堆栈信息没有，全局断点就到这里，我看不到信息

又出现了

结合qmui使用

这些堆栈够了吧，现在出现频率还挺高，希望大神解决下

暂时没有时间，处理不了

这么尴尬，那等你有时间

v2.4.6已修复，请确认。

@ZClee128 新版本做了修复，需要你验证下。

我这边更新了最新版本，目前好像没出现了，应该是解决了，如果再出问题，我再激活这个bug咯

repo: tigerAndBull/TABAnimated


text:
Add waf default core ruleset as configmap

Description

For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

Replaced by #3649.

authors: LorcanMcVeigh gantony

repo: tigera/operator


text:
Error in Query Params

Description
Unexpected error in query string
Expected behavior
This is my route in Vue Component
 <a class="btn btn-default btn-xs" :href="route('posts.show', [{id: item.id}, {event: 1}, {venue: 2}, {page: 5}])">
And this is the Result:
posts/32?0[event]=1&1[venue]=2&2[page]=5
There are wrong characters like an array of params [0] event [1]venue
Environment

Laravel version: 8.0
Ziggy version: 0.9.4

Can you please share your route definition for that route and the relevant portion of the output when you run console.log(Ziggy.routes)?

This is the route generated with ziggyjs:
"posts.show":{"uri":"admin\/posts\/{post}","methods":["GET","HEAD"],"domain":null}
and this is the console log of that path:
posts.show:
domain: null
methods: Array(2)
0: "GET"
1: "HEAD"
length: 2
__proto__: Array(0)
uri: "admin/posts/{post}"

can I share anything else that may be useful?

I think the problem is when i call a route with id.
I try to change route with optional parameters but doesn't work
Route::get('/posts/{id?}', [UserController::class, 'show'])->name('posts.show');

Try passing all the params as just one object:
route('posts.show', { id: item.id, event: 1, venue: 2, page: 5 });

There was an issue with Ziggy v0.9.x at some point that had to do with passing params called id, so if the above doesn't work, I would suggest either updating to Ziggy v1 or separating out the query parameters like this:
route('posts.show', item.id).withQuery({ event: 1, venue: 2, page: 5 });

Resolved with
route('posts.show', item.id).withQuery({ event: 1, venue: 2, page: 5 });
Thank You!

authors: bakerkretzmar filippodicostanzo

repo: tighten/ziggy


text:
Does not work with Symfony Console 5

I just a had fresh install with the following latest Laravel tools:

PHP 7.4
Laravel Installer 3.0.1
Composer version 1.9.1
Laravel Valet 2.7.0

These packages all use Symfony 5's console, (v5.0.1 according to composer.lock).
tlint does not install with this version:
tightenco/tlint 3.0.0 requires symfony/console ^4.3 -> satisfiable by symfony/console[v4.3.0, v4.3.1, v4.3.2, v4.3.3, v4.3.4, v4.3.5, v4.3.6, v4.3.7, v4.3.8, v4.3.9, v4.4.0, v4.4.1].
Simply changing  "symfony/console": "^4.3", to "symfony/console": "^4.3|^5.0" might solve the problem, but I could not run your tests (could not get your Docker setup replicated in a reasonable time).

You should be able to PR against it and have travis run the tests (I suspect your assumption is correct, and that will resolve it). Otherwise I will check it out as soon as possible, thanks for the heads up!

Here you go: https://github.com/tightenco/tlint/pull/122

Awesome, I'll get this in Friday along with https://github.com/tightenco/tlint/pull/120, thanks so much for taking the time to PR / raise the issue! You are certainly on the edge 😄

Closed by https://github.com/tightenco/tlint/pull/122

authors: loganhenson minthemiddle

repo: tightenco/tlint


text:
Define Step Function

Reason
Create a step function to execute the requests required for subscription estimation
Description
The step function requires the following request parameters

query
requestId

The step function should then execute the following tasks

Wrap the query in a count statement: SELECT COUNT(*) as "total" FROM ( query );
Execute the count query using the Athena StartQueryExecution Step
Wrap the query in a sample statement: SELECT * FROM ( query ) LIMIT 3;
Execute the sample statement using the Athena StartQueryExecution Step
Using the Call third-party API step, with a body containing the requestId and the locations of the results (look for field OutputLocation)

For auth of the Call third-party API step use the existing registered core-account-sf endpoint
Use callback path of POST https://account.mytiki.com/api/latest/ocean?event=subscription-estimate

Success Criteria

[ ] Defined input and output json to the step function
[ ] Step function executes count query
[ ] Step function executes sample query
[ ] Step function calls back to account service with the results

Additional Information
No response

@MiroBenicio is waiting on AWS permissions

authors: mike-audi ricardobrg

repo: tiki/core-account-subscription-estimate


text:
When the business has a hot spot, one node is mistakenly detected as a slow node

Bug Report

[root@copy-of-vm-ee-centos76-v1 bin]# ./tikv-server -V
TiKV
Release Version:   5.2.0
Edition:           Community
Git Commit Hash:   d1967b87e7c47c241931fe920965febb4c37ef7f
Git Commit Branch: heads/refs/tags/v5.2.0
UTC Build Time:    2021-08-14 02:02:01
Rust Version:      rustc 1.56.0-nightly (2faabf579 2021-07-27)
Enable Features:   jemalloc mem-profiling portable sse protobuf-codec test-engines-rocksdb cloud-aws cloud-gcp
Profile:           dist_release
What operating system and CPU are you using?
tiup ：
1PD
1TiDB
3TiKV
all node：500G NVME、4 cores、8G mem
Steps to reproduce
1、run sysbench insert prepare
threads=128
tableNum = 512
tableSize = 10000000000
2、A few minutes later，one node is mistakenly detected as a slow node
What did you expect?
When the business has a hot spot, node should not be detected as a slow node，and leaders of the node is evicted
What did happened?
When the business has a hot spot, one node is mistakenly detected as a slow node

/type bug

/severity major

/assign 5kbpers

repo: tikv/tikv


text:
tikv-ctl: Add delete-range subcommand to tikv-ctl

Signed-off-by: MyonKeminta MyonKeminta@users.noreply.github.com
What have you changed? (mandatory)
This PR added a subcommand named delete-range which scans and deletes all keys in a range.
It deletes in all of default_cf, write_cf, lock_cf.
It doesn't invoke delete_files_in_range, but scan and delete one by one.
Since it's a dangerous operation, I added some verbose warnings and asks for confirmation.
What are the type of the changes? (mandatory)

New feature (non-breaking change which adds functionality)

How has this PR been tested? (mandatory)
By manual test
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

This PR is based on master but not 2.0.2. Please don't misuse.

Do we really need such a dangerous tool?

@zhangjinpeng1987 I have no opinion about it since this tool is wrote in a hurry for a certain need. Maybe not.

This functionality is dangerous and not useful. Maybe it can provided by other tools. Close it.


text:
util: Change the tikv_util::collections alias to HashMap/HashSet in the standard library

What have you changed? (mandatory)
As a successor of #4568 , this pr changes the tikv_util::collections alias to HashMap/HashSet in the standard library.
The HashMap implementation in the latest Rust std points to hashbrown which we are using now. So it's useless to add hashbrown to our dependencies and use them explicitly.
For simplicity and considering we may change to another hash algorithm in the future, I just change the type alias in tikv_util::collections instead of changing all tikv_utils::collections occurrences in the repo to std::collections.
What are the type of the changes? (mandatory)
Engineering.
How has this PR been tested? (mandatory)
make dev is run
Does this PR affect documentation (docs) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
The actual hashmap implementation and hash algorithm is not changed. We already use hashbrown. So I think a benchmark unnecessary.
Add a few positive/negative examples (optional)

I left my opinion in inline comments above and update the pr. What do you think? @BusyJay

That's the whole point of merging all the collections into one place: change the collections without searching and replacing. I prefer the old way unless we can make sure there will be no changes in the future.

As we still have to use fx-hash, we cannot just replace it with std hashmap. The tikv_util is still necessary. So I close it.

authors: BusyJay sticnarf


text:
Artisanal Production Builds 2

applying changes in https://github.com/tildeio/glimmer/compare/artisinal-code-stripping to find where the test failures are coming from

closing as I've located the issue

repo: tildeio/glimmer


text:
The rendered content via template_include action is not displayed after get_footer() function is called.

I use a rockettheme template called Gemini and also i use a plugin for rent cars and when somebody went to use the forms and put the submit button it must show a general resume form and it didn't show it.
It is a bug in the timber theme, we debugged it and everything is ok until wp_footer is called, there is a timber theme function which not render previously loaded content. The theme used is gantry5 framework so it is not easy to debug something else there. why the rendered content via template_include action is not displayed after get_footer() function is called.

can you answer please?

bump

Hey there
Sorry, there’s a lot of work going on in different areas of Timber. It would definitely help if you could provide more detailed instructions for how to reproduce the error. That’s also why there is an issue template with information that we’d like to know.
It would be good to see how that plugin for renting cars is hooking into the theme. Is it in the official plugin directory? Can you give a link to that? Or is it not specially related to that plugin? Do you have some example code that illustrates the problem?

It is defined as:
add_action('template_include', array('CarRental', 'carrental_summary'));
where is our template file included.

Hey @kataskeviwebsite
It’s really hard to reproduce the error with the information that you’ve given. If you can’t provide us with the steps to reproduce the error, I’m afraid we can’t help you.
Usually, the template_include hook is a filter and not and action. If it’s added as an action and doesn’t return a value, it may break your theme.

there is a timber theme function which not render previously loaded content

So what function is this? Is in the Gantry 5 theme? I’m afraid we can’t provide support for issues that are directly related to theme.
Does the error also occur if you use the Starter Theme?

authors: blizam gchtr kataskeviwebsite

repo: timber/timber


text:
Fix new typos

Found new misspellings.
https://github.com/crate-ci/typos

coverage: 88.193%. remained the same
when pulling ed1219857f3f8b7472621c35c38334534e1aea45 on szepeviktor:typos
into 3f6e73feadf5d547dff4992f645805da7fbc4d3a on timber:2.x.

authors: coveralls szepeviktor


text:
what are valid parameters for file names in load_regression

may you share
what are valid parameters for file names in load_regression
def load_regression(name, split=None, extract_path=None, return_metadata=True)
to use for
from aeon.datasets import load_regression

Hi, seems like this was created for the wrong package. The function appears to be for aeon.

repo: time-series-machine-learning/tsml-py


text:
Improve logging of background batch producer

@brunodecarvalho or @bdotdub please :eyes: thanks!

lgtm - this is mostly adding the success case, correct?

lgtm

great, thanks!

this is mostly adding the success case, correct?

yes, and the logging, and the minor readability improvement to sendBatch.

yes, and the logging,

I meant "and the testing"

authors: aviflax bdotdub

repo: timehop/go-kinesis


text:
import data in R w/ jsonlite dosent work..

Error in parse_con(txt, bigint_as_char) :
lexical error: invalid char in json text.
ppy", "#goodtime", "#funny", ...],        "likes": 284,
(right here) ------^
i used the given example in the READ.ME but all my scraped json files dosent work w/ jsonlite

have you checked the json files with other tools? Perhaps R/jsonlite has a problem. Perhaps also some encoding problems?

yes, the problem is the output json isn't valid (checked w/ https://jsonlint.com/; https://jsonformatter.curiousconcept.com/)..

@phermann1988 could you pls update und check if there is still an error.
if yes: please post the json file with error

repo: timgrossmann/instagram-profilecrawl


text:
Errors since last update

Hi, yesterday I was working on my gamemode and the weapons were working perfectly, but today I found that they were not, something must have changed at facepunch because the sandbox mode weapons are working.
https://ezshare.it/s8AvNmUWtf.mp4
Regards

Having the same issue.

How strange, taking a look atm!

Fixed! Turned out to be caused by the new component update.

authors: JMorell Ludendorff timmybo5


text:
chore: move to GitHub Actions and use semantic-release

Depends on: https://github.com/raix/semantic-release-vsce/pull/61

@timonwong This is a big change. What do you think?

@felipecrs Wow just go ahead, it's fascinating :)

I depend on the PR mentioned in the description before being able to submit this PR.. =/

@timonwong could you please take a look in the scripts in package.json? I refactored it again.

Interesting, tests are failing on Windows.

Interestingly, tests are failing on Windows. The extension works fine here, though.

Strange, should we just skip the tests for windows temporarily 😂  ?

@timonwong, I need you to:

Set up the semantic-pull-request app on this repo: https://github.com/apps/semantic-pull-requests
In the repo settings -> secrets, add the VSCE_TOKEN with your token to VS Marketplace
Configure the branch rules for the master branch:

Create a protection rule for branch master
Add the ci/test for Linux and macOS as required checks, also add the semantic-pull-request once set
Enforce the rule for administrators (we don't want to be able to commit broken code to master branch)

Disable Merge, and Rebase and Merge as merge strategies for this repository. Since we need to enforce commit message conventions, we should use only the squash and merge button.

Please tell me if you don't agree with any of these.

@felipecrs  Done

The status check for semantic-pull-request will be enabled once it's done

@timonwong Amazing!
But I found an issue in the @felipecrs/semantic-release-vsce plugin, I'll solve tonight so we can finally merge this.
Also, https://github.com/microsoft/vscode-vsce/issues/480 was implemented, so we don't need --yarn in vsce anymore. 🎉

@timonwong thank you very much for the help, let's see how it goes in next commits. :)
Just make sure to follow Conventional Commits.
This helps a bit: https://commitlint.io/

@timonwong Actually, you have to remove the "Include administrators" option. This is why the build is failing.

Actually, you have to remove the "Include administrators" option. This is why the build is failing.

@felipecrs Ok it's now disabled :)

Also, you can add the "release" job in the list of required. It validates the release.config.js, so we don't merge broken code.

Great! Thanks a lot for introducing those awesome tools into our project  🎉

:tada: This PR is included in version 0.11.0 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:

authors: felipecrs timonwong

repo: timonwong/vscode-shellcheck


text:
effect of skip  is not clear

It's not clear in the documentation what is the effect of adding # isort:skip at the end of a line. From my observations, what it does is considering an import statement as a line of code, and moving it down after the import section, as if it was the first line of code. The skip name is maybe self explaining from the tool developers point of view, maybe because the import statement filter will skip this line, but from a user perspective, I bet most people will not guess what will be the effect of it. This could be better explained in the docs. Something in the line of:
"Adding # isort:skip as a comment at the end of a line will prevent an import statement to be recognized as such. In consequence, this line will be treated as code and be pushed down to below the import section of the file."

Thanks for reporting!
The improved documentation for this feature is now live, here: https://timothycrosley.github.io/isort/docs/configuration/action_comments/#isort-skip

Good! Thank you for the nice work with isort!

authors: antoine-gallix timothycrosley

repo: timothycrosley/isort


text:
setters - falsey values

When using setters with falsey values (e.g. 0) the getter is called instead.  This is fixed by a null comparison to use the setter for anything but null and undefined.

Ah yes, that slipped by me.
Can you change the pull request to the dev branch though? I'm working on stuff for 1.1.0 and I want this to make it in.

Sure!  I'll just create a new pull request I suppose?  I don't see a way to change it retroactively.

authors: braddunbar timrwood

repo: timrwood/moment


text:
error on documentation regarding constructor

The parser ignores non-alphanumeric characters, so both moment("12-25-1995", "MM-DD-YYYY")and moment("12995", "MM-DD-YYYY")will return the same thing.
may be it should say:
The parser ignores non-alphanumeric characters, so both moment("12-25-1995", "MM-DD-YYYY")and moment("12-25-1995", "MMDDYYYY")will return the same thing.

It should be 12\25\1995. Looks like an unescaped backslash in the following line.
https://github.com/timrwood/moment/blob/master/sitesrc/docs.jade#L235
Thanks for catching this.

This has been fixed.

authors: anmaso timrwood


text:
Lightbox with Mosaic

When I try to add the popular lightbox plugin, then jquery mosaic won't work.
This works, even with lazyload added:

But this won't work:

I tried to add the image as background of a div, but that also won't work.
Is there any way to make it work, since I really like jquery mosaic more than masonry.js

Hi @FullDome! I might need to do some tests, but unless the lightbox plugin is messing too much with the dom, it should work.
jQuery Mosaic needs to know either the width and the height, or the aspect ratio of each element, and I think the problem you're having might be related to that.
Indeed, jQuery Mosaic works fine also with  and  elements instead of , but you should be giving him a hint on their aspect ratio to let him do his thing. I think your example is not working because you're not giving the  element any implicit size, try this:

or this:

Of course you would need to know these sizes (or the aspect ratio) in advance, that would be the downside. But if you're building your mosaic with PHP or similar, it shouldn't be a problem.
Cheers!

Thank you for your help!
The image size or ratio is being handled by the lightbox. The smaller image is the one being handled with jquerymosaic.
Even if I add the size or resolution for the big image, jquerymosaic won't work.
I noticed that when I add a link, it doesn't work. So it probably doesn't concern the lightbox, but a href:

Hi, I think you're having conflicts by using two plugins that both might be trying to set the size of an element. It is true that jQuery Mosaic will read the width and height of each element if it has been set (either manually, or, say, by another plugin like Lightbox), but you'll need to be sure that jQuery Mosaic is called after Lightbox has done its thing.
I suggest the possible following scenarios for you to follow:

Find out whether Lightbox has some kind of a callback function that is called after the lightbox has been set up and try calling jQuery Mosaic in that function to ensure Lightbox has set the sizes before calling jQuery Mosaic.
Having two plugins messing with image sizes at the same time seems too overkill to me. I would try to use the Lightbox only for the purpose of showing a zoomed version of the image in a popup, not to set image sizes in my DOM.

Let me know if that works for you!

The padding is added on my page, but also yours. I thought it came from the lightbox plugin, but it appears when I add a link to the images. Also on your example page. See image attached, where I added a link and copied the image twice:

It's not a padding, it looks more like the plugin hasn't been executed, or it took the wrong sizes. I'm afraid I would need to take a look at your code, just having some description of the behavior and screenshoots won't do the trick.
Send me your code so I can reproduce the problem here and I might be taking a quick look at it whenever I can. The tin cat is very busy. Please check all my suggestions first.
Meow.

Dear tincat,
I used your suggestions. But adding a link to an image in my code or your code makes the same thing.
The screenshot is your code from your website. I just added a link to an image in the mosaic div.
If you add an url or image link to a photo inside the jquerymosaic, the plugin won't excecute.

The problem is that there are many thins that could cause this. For example, It seems to me the plugin is working well and the  has been resized to the adequate size, but you're expecting it to also resize the image you placed inside the , this is not what the plugin does. Take into account that the plugin resizes the container element, it doesn't touches anything inside.
There's another example in the source (https://jquery-mosaic.tin.cat/example_advanced.php) to exemplify what you're trying to do. You'll see in this example that each image in the mosaic is linkable (which is what you're trying to achieve), but in order for the image to be resized, it's being applied as a background image, with a background-size that allows for it to grow to the container's size. Since jQuery Mosaic resized the container and not the contents, this is one possible solution to achieve it.
Take a deep dive into this example and let me know if that solves your doubt.
Cheers!

Applying an image as background image made it possible to use a link!

Thank you very much!

Glad you solved it! Cheers!

authors: FullDome tin-cat

repo: tin-cat/jquery-mosaic


text:
Unable to create a new barebones or tina cloud starter projects

Describe the issue that you're seeing. Any Loom videos or screenshots usually help a lot!
Current behavior:
I am unable to create a new tina cloud or barebones starter project. I am using the instructions from here (Dashboard quickstart):
https://tina.io/docs/setup-overview/
I successfully created two projects with the same instructions back in Nov. 2022 and on Feb. 2023. I have not been able to create these starter projects as of April 24th, 2023.
There are two main issues that constantly come up when creating any of the above starter projects.

GitHub Authentication keeps on loading for over 10 mins. It never finishes.
The project is visible in Vercel but it always fails due to a missing a client id.

Screenshot

Reproduction
https://app.tina.io/quickstart
Steps to reproduce
Follow the tina cloud or barebones starter steps.
System Info
GitHub authentication never finishes loading.
The Vercel build fails with missing client id.
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug.

I am using Tina as a new user with admin role.

@ggalicia2021 I just went threw the flow with no issues. If you go to the "environment variables" section of the settings are the environment variables set?
Should have the following

Im also having trouble with creating a new project and importing an existing project on GitHub, bc the authenticate with gut hub btn just has a loading page and never authenticates... I miss forestry

@bradstockUser if you want help with this issue please open a thread in discord: https://discord.com/invite/zumN63Ybpf in the #help channel

repo: tinacms/tinacms


text:
efficient way to collect mac address

Tinkerbell need to know the mac address of the machine previsiously
is there any way to collect the mac address  of the machine efficiently?
expecially the mac of PCI-e  Nic Card.
with great appreciate.

Duplicated of #178

authors: nshalman sheepcat

repo: tinkerbell/boots


text:
style for a tag

update link style
fix packet slack url

@gauravgahlot @mrmrcoleman I'm not allowed to assign any reviewers. So would you review this PR when you have time? thanks


text:
sup isn't recognized as a valid subtitle format

What TMM version are you using?
2.9.3.1
release, pre-release, nightly, or directly from GitHub/branch?
Release
What is the actual behaviour?
Subtitles in the .sup format are not recognized as subtitles by TMM.
What is the expected behaviour?
They should be.
Steps to reproduce:
Have 2 files.
movie.mkv and movie.eng.sup
Update the library with TMM

will be added to our default list - thx.
Meanwhile, just add it for yourself - Settings>general>file types...

this fix will be in 2.9.4

authors: LeLobster mlaggner myron0815

repo: tinyMediaManager/tinyMediaManager


text:
remove tuple from type annotation in Tensor.init

See https://github.com/tinygrad/tinygrad/pull/1210#issuecomment-1627846455.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/tensor.py                         1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0

repo: tinygrad/tinygrad


text:
voice changer model added

WIP
so-vits-svc added for use in voice changing functionality
referencing: bounty a22 - voice changer model (so-vits-svc, rvc). make me sound like drake

.DS_Store does not belong to git - you added it to .gitignore, but you still kept the file

My apologies I thought it was left out. I will correct the commit and resubmitJason BalthisSoftware Engineer / Fullstack Developerweb       https://portfolio.barelyjason.onlinegithub    https://github.com/jbalthislinkedin  https://linkedin.com/in/jasonbalthiscell       @.@. Jul 25, 2023, at 4:33 AM, Pavol Rusnak @.***> wrote:
.DS_Store does not belong to git - you added it to .gitignore, but you still kept the file
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Yes I thought it was left out. Thanks for letting me know. I will remove this pull request and resubmit with just the correct file

I definitely will make use of vits as much as possible and thanks for the advice from you all. On Jul 25, 2023, at 12:19 PM, wozeparrot @.***> wrote:
Can try to reuse as much as possible from the existing vits model, because so-vits is based on vits there should be a bit of an overlap
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

authors: jbalthis prusnak