text:
RTL is not supported!

What i'm suffering from is that the tabs still functioning the same as the LTR after switching to RTL, I know what seems to be the issue because I have a custom indicator, but i'm not able to use negative values to translate them to the opposite.
Thank you so much.

Can you send a PR to fix this?

@satya164 I never send one, i'm not sure how to do that?

This bug also affects react-navigation. The TabNavigator is not working:
https://github.com/react-community/react-navigation/issues/860

Inside TabViewPagerScroll.js putting flexDirection: 'row-reverse' on the <ScrollView /> fixes the problem partially. The tab's contents is shown but the order is reverse.

#https://github.com/facebook/react-native/issues/11960

@yaronlevi, look at my last comment here: https://github.com/facebook/react-native/issues/11960#issuecomment-292732636
@moerabaya, I sent a PR #194, until it merged, you can use one of these:

npm i -S https://github.com/dastoori/react-native-tab-view.git#rtl-support
npm i -S https://github.com/dastoori/react-navigation.git#rtl-support

Re-opening this since RTL is not yet supported in TabViewPagerScroll, hence unsupported on iOS.

@satya164, iOS ScrollView has been fixed in ReactNative v0.43.2 (https://github.com/facebook/react-native/issues/11960#issuecomment-292732636)

Nice. I think we may still need to handle the scroll position etc.

I checked this in version 0.0.59, i used the examples in examples folder. These examples does not works properly:

"1. Scrollbar top bar"

"3. Bottom bar with indicator"

Other examples are works fine:

Bottom bar with indicator

This one uses a custom indicator, that's why. Should be easy to fix.

"1. Scrollbar top bar"

Yeah, this one will be more involved

Any updates on this one?

If you need this, pull requests are welcome.

@satya164 - I created a pull request to add support for RTL. I hope it can be accepted and merged.
Thank you.

Adding a link to the PR - not sure if 2 months is a reasonable time or not. I'm sure the team is busy...
https://github.com/react-native-community/react-native-tab-view/pull/556

Hey there @noambonnie. I've taken a look at your fix and I've tested it on both iOS and android. While this fixes the issue on android, it breaks iOS so I've just added a platform specific condition. Take a look at my fork:
https://github.com/horodnicdragos/react-native-tab-view
Not sure if this is the best approach on the long term but having this merged would be amazing. Do you want to update your PR or should I do one?

@horodnicdragos what's broken on iOS? Does your code (ignoring iOS for those calculation) work fine on iOS RTL?
If broken on iOS we'll need to fix it there too. If your fix works on both platforms in RTL (and LTR...) then fine

Yes, for some reason the calculations are not required on iOS. I'll make the PR and hope for the best.

@horodnicdragos please note that your PR failed the linter requirements. Follow the test procedures at the bottom of the project page (under "Contributing"). Otherwise it won't even be considered.

@noambonnie Done!

@satya164 can you check #602 and merge it?

I hope it gets merged soon...

is it going to be merged soon?

it is already merged and released. but the PR is Android only.

@0xori It is working just fine on iOS for me. Did you encounter any RTL issues?

still having the same issue on android , for ios it is fixed using this #675

any updates on this ?

@FadiAboMsalam It's marked as "help wanted". PR welcome.

in TabBar.js
modify this function
_this.resetScroll = function (value) {
const tabLength=this.props.navigationState.routes.length-1;
    const target=I18nManager.isRTL?tabLength-value:value;
    console.log(tabLength+' '+target)
    if (_this.props.scrollEnabled) {
        _this.scrollView && _this.scrollView.scrollTo({
            x: _this.getScrollAmount(_this.props, _this.state,target),
            animated: true
        });
    }
};

this issue doesn't fix, please don't close

@hosseinmd please open a new issue with a runnable repro as mentioned in the issue template.

I resolve this, that is my fault

I am having the same issue on IOS. Did anyone manage to solve it?

+1

+1

this issue is still happening

i have fixed this issue on iOS will make a new PR

authors: 0xori Abdo-zayed FadiAboMsalam MahmoudAliIbrahim YogiHa dastoori gabimoncha horodnicdragos hosseinmd matifriaz moerabaya noambonnie saravanakumargn satya164 syedamirali14 yaronlevi


text:
Bug manager- Execute TC on Rhino automation which passes, Jira comments has been created on the wrong execution ID in the closed bug

Not a bug

source: gharchive/issue

authors: YanivAroch gravity-api

license_type: permissive

license_source: github-api


text:
Support inline fixers for fix and warn plugin

We need to support the following LIT fixers:
https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/test/clang-tidy/checkers/abseil-time-comparison.cpp
// CHECK-MESSAGES: [[@LINE-1]]:7: warning: perform comparison in the time domain [abseil-time-comparison]
// CHECK-FIXES: absl::FromUnixSeconds(x) <= t1;

We need to:

Support this logic in a fix and warn plugin
To do this we need to generate tmp Expected test resource
Compare files and check results

This should be done under inlineFixer = true  option
Keep in mind, that in warn plugin you already have the list with warnings matched to lines, may be it could help you

Bad news are the following:

I do not know how the following code works:

void g() {
  // CHECK-NOTES: [[@LINE+4]]:5: warning: argument name 'y' in comment does not match parameter name 'x'
  // CHECK-NOTES: [[@LINE-3]]:12: note: 'x' declared here
  // CHECK-NOTES: [[@LINE+2]]:14: warning: argument name 'z' in comment does not match parameter name 'y'
  // CHECK-NOTES: [[@LINE-5]]:19: note: 'y' declared here
  f(/*y=*/0, /*z=*/0);
  // CHECK-FIXES: {{^}}  f(/*y=*/0, /*z=*/0);

  f(/*x=*/1, /*y=*/1);

  ffff(0 /*aaaa=*/, /*bbbb*/ 0); // Unsupported formats.
}

The check_clang_tidy.py script provides an easy way to test both diagnostic messages and fix-its. It filters out CHECK lines from the test file, runs clang-tidy and verifies messages and fixes with two separate FileCheck invocations: once with FileCheck’s directive prefix set to CHECK-MESSAGES, validating the diagnostic messages, and once with the directive prefix set to CHECK-FIXES, running against the fixed code (i.e., the code after generated fix-its are applied). In particular, CHECK-FIXES: can be used to check that code was not modified by fix-its, by checking that it is present unchanged in the fixed code. The full set of FileCheck directives is available (e.g., CHECK-MESSAGES-SAME:, CHECK-MESSAGES-NOT:), though typically the basic CHECK forms (CHECK-MESSAGES and CHECK-FIXES) are sufficient for clang-tidy tests. Note that the FileCheck documentation mostly assumes the default prefix (CHECK), and hence describes the directive as CHECK:, CHECK-SAME:, CHECK-NOT:, etc. Replace CHECK by either CHECK-FIXES or CHECK-MESSAGES for clang-tidy tests.

@akuleshov7 btw, why it still pinned? I think we can drop it for now

repo: saveourtool/save-cli

license: MIT


text:
Add generate entities

Hi
this pull request add supports for the orm:generate:entites command. I also have added this to saxulum-doctrine-orm-manager-registry-provider which I can add a pull request for once this is accepted.

@paran01d thank you for your contribution

https://github.com/saxulum/saxulum-doctrine-orm-commands/releases/tag/1.3.0

source: gharchive/pull-request

repo: saxulum/saxulum-doctrine-orm-commands

license: mit

license_source: bigquery


text:
Cannot read property 'getContext' of null

I'm having the same problem as a closed issue, but doesn't seem to be cross domain problem with the PDF. When running my app I get the error
TypeError: Cannot read property 'getContext' of null
Which I assume means it can't get the canvas.
Template file looks like this:
<ng-pdf template-url="partials/viewer.html" canvasid="pdf-canvas" scale="page-fit"></ng-pdf>
viewer.html content looks like this:
<canvas ng-if="pdfUrl" id="pdf-canvas" class="rotate0"></canvas>
And lastly the controller:
$ionicModal.fromTemplateUrl('templates/pdf-viewer.html', {
    scope: $scope,
    animation: 'slide-in-up'
}).then(function (modal) {
    $scope.modal = modal;
});

$scope.viewPDF = function() {
    $scope.pdfUrl = 'http://localhost/commonproperty/bin/temp.pdf';
    $scope.modal.show();
};

I've tried to reference a PDF in the app root but still get the same error.
Any help would be greatly appreciated!

Hi @brettmeyer, this might happen when the canvas is not available in the DOM (the directive is using document.getElementById() to find the canvas.
I assume your canvas is located in a modal view or something similar, which hasn't been loaded yet.

@brettmeyer does the suggestion of @dani-inside solves the issue?

@dennybiasiolli to be honest I can't remember what I did to resolve this, which I did before there was a response, but I think @dani-inside was correct. This is what the code looks like in the working app:
`$scope.showPDF = function(filename) {
$scope.pdfUrl = filename;
$scope.showPDFModal('templates/pdf-viewer.html');
};
$scope.showPDFModal = function(templateUrl) {
$ionicModal.fromTemplateUrl(templateUrl, {
scope: $scope
}).then(function(modal) {
$scope.modal = modal;
$scope.modal.show();
});
};`
And the contents of pdf-viewer.html:
`

PDF

`

@brettmeyer thanks for your response. 😄

I want to add something to the useful explanation by @brettmeyer: I implemented the same thing (PDF preview in a $ionicModal) and I had to make sure to call modal.remove() when closing it. Otherwise, when reopening the PDF would not be rendered correctly. Just thought it may be useful to someone struggling on the same problem as me in the last two hours!

authors: brettmeyer dani-inside dennybiasiolli simonedavico

repo: sayanee/angularjs-pdf


text:
日志记录占用过多存储

连续运行客户端会生成GB级的LOG文件，占用存储空间，是否有关闭日志记录功能的方法呢？

连续运行客户端会生成GB级的LOG文件，占用存储空间，是否有关闭日志记录功能的方法呢？

后续我这边调整下日志文件数量,   现在运行的时候可以使用 '-l` 指定日志等级 , 默认是info. 可选择debug, info, warn, error, 使用warn或者error会减少很多
比如
nt_server -c config_s.json  -l warn

好的，我这边尝试把日志等级更改一下，十分感谢

新版本限制了日志大小

authors: RyanYuanSun sazima

repo: sazima/proxynt


text:
Add random Generator

Closes #22

Merged. There's still work to do adding Generator support to more ops now, but it shouldn't be too hard.

repo: sbrunk/storch


text:
Abstract "console" task over JSON

steps

Start sbt server on terminal1 where a build is located: sbt server
Start sbt client on terminal2 from anywhere like ~/temp/: sbt client localhost:5001
On terminal2 type project echo-client, and then console.

expectation
Scala REPL works on terminal2.

Pushing to 1.1.0

Fixed in https://github.com/sbt/sbt/pull/5620

repo: sbt/sbt


text:
add human readable total time

[X] I've read the CONTRIBUTING guidelines

Not sure this requires change.md file, but let me know if it's needed
Motivation:
for big multimodule projects sbt often report Total time: 123456 s, ...
Which is not human friendly. So this PR adds some human friendly duration. Please see spec for examples

@eed3si9n not sure why, but headerCreate and headerCheck` do nothing
I've added the header manually, hopefully it's okay

looks like travis-ci failure is unrelated Error: Could not retrieve sbt 1.1.6: missing sbt.xMain
@eed3si9n do I need to do anything else here? thanks

thanks!


text:
Backport "Always rewrite empty List() to Nil" to LTS

Backports #21689 to the 3.3.5.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to backport-lts-3.3-22148.
Reference


text:
Lookup by classpath entry

adds the API ctx.findSymbolsByClasspathEntry
def findSymbolsByClasspathEntry(entry: AnyRef): IterableOnce[Symbol]

Classpath is now organised by classpath entry, with structural sharing, and Loaders handles duplicate root entries by favouring earlier classpath entries.

I don't like the AnyRef lookup so I'd prefer something more precise - either typeclass witness that it's some path type, or we have some wrapper type that can only be created by a factory method?

repo: scalacenter/tasty-query


text:
Cross-build paradise for 2.12.

This commit adds 2.12 support for scalameta/paradise macro annotations.
2.11/2.12 specific code is put into 2.11.8/2.12.1 directories, using
the same convention as scalahost.

Thank you very much for your work! I've been going through the pain of accommodating small changes in compiler internals for 3+ years, and it's quite tedious. I really appreciate your help in this area.

Superseded by https://github.com/scalameta/paradise/pull/170

authors: olafurpg xeno-by

repo: scalameta/paradise


text:
ApiReference ignores footer slot

Expected Behavior
ApiReference should respect footer slot.
Actual Behavior
When upgrading to 1.2.5 from 1.0.6, the footer slot does not get rendered anymore.
ApiReferenceBase still has that slot, but exchanging them is not really sufficient in my use case.
Reproduction steps
1. Use #footer in ApiReference

Version
1.2.5
What browsers are you seeing the problem on?
Chrome

Oh no, I’ll check that.

Hi @BenediktHeinrichs! Looks like this got pulled out during the layout refactor we did a while ago, I've added it back to be below the content :)

Love the response time of you guys, really great product & team!

authors: BenediktHeinrichs hanspagel hwkr

repo: scalar/scalar


text:
TypeMatcherHelper Enhancement

Enhanced TypeMatcherHelper to handle primitive types checking.
This should fix the following issue:
https://github.com/scalatest/scalatest/issues/1359
Note: This fix should be cherry-pick into 3.1.x and subsequently pulled into 3.2.x.

I cherry picked these two commits into 3.1.x also.

authors: bvenners cheeseng

repo: scalatest/scalatest


text:
Trying Travis for Js, Native and Dotty (DO-NOT-MERGE)

Trying out travis to build for js, native and dotty.

Closing for re-opening for travis build.

Reopen for travis build.

@cla-bot[bot] check

@cla-bot[bot] check

@cla-bot[bot] check

Closing as no longer applicable, we now use github action for CI build.

authors: artimasites cheeseng


text:
Logo

I would like to change the logo to something like this, using the font Fabulous Steampunk.

The license says "Licensed as: Freeware, commercial use requires donation". Which one of the two would we fall under? :)

I like the old logo better :stuck_out_tongue_closed_eyes:

I know that @wi101 created a nice one, it looked like splash paint font or something. I don't know if she did high-resolution / vector version or just the mock I saw.
We should do a proper logo.

@jdegoes Actually I didn't do a high-resolution.. but what do you think about it?

@ktonga lol, does this work better: https://imgur.com/a/zrVm5jF
I just find the current logo a bit "plain" :) I thought that it could represent the "essential plumbing", that's why I grabbed a steampunky font.

@NeQuissimus  Hi, my name is Richard. I'm a graphic designer and I love making contributions for open source projects. I see that they want a new logo for your application. If you wish, I can do it for you without any problem ... I can present some modern ideas that would improve the appearance and image of your project

@richardbmx Give it a shot :)

perfect I'm going to work on it

@NeQuissimus  Here I present my proposal based on the original logo but with an improved version more personal and with different versions, it can be used as a favicon and in a vertical and horizontal version. with an icon that represents the letter Z with cuts that express and provide an optical illusion, which makes the icon a personalized Z for your project. If you want some change you can ask for it without problems... I hope you like it

@wi101 I really like it, though I think the 'z' should be the same size as the other letters.
@NeQuissimus I do like steampunk. 😄
@richardbmx Thank you for mocking these up. They look very professional. My own personal taste is for something a little more adventurous and spicy. By the way I saw you donated logos to other open source projects. Thank you for your extremely generous contributions to FLOSS!

@NeQuissimus  I present a folder with all the necessary files to manipulate the logo, I deliver the logo in two versions, one vertical and one additional horizontal, that I deliver an icon version to be used where they want, all the archos are 100% editable and in vector format.
If they want to change something, they inform me. regards ;-)
All File
Google drive

I am going to leave it to the group to voice opinions. I just came up with the steampunky theme on a whim since I thought it would be fitting.

@hepin1989 This is very much along the lines of something coming soon... stay tuned! 😉

Really like it, both are speedy, mine is running, the new one is soaring,much better,very nice.

😍

authors: NeQuissimus hepin1989 jdegoes ktonga regiskuckaertz richardbmx wi101

repo: scalaz/scalaz-zio


text:
How to skip the pending invitations control when login in

I have a Rails/Mongoid application using devise and devise_invitable gems with a special scenario where a User has_and_belongs_to_many groups. This in the application means that a User can be invited to one or more groups by other users. I'm using devise_invitable to handle those invites for both new and existing users.
The scenario where I'm having problems is when a user that has already registered and confirmed its account, and that might have already be invited to one group, is invited to another group. In this case the system sends the new invitation as usual but for the existing User instead of creating a new one to be invited.
If the user confirms the invitation everything works ok, but the problem I'm having is that unless the user confirms it, he/she won't be able to login again into his user account because it will get the alert:
You have a pending invitation, accept it to finish creating your account.

So I'm wondering how should I do or what should I override on Devise/Devise_invitable in order to skip that control when trying to login as a user that has already been confirmed but has a pending invitation.
My User model is this one:
class User
  include Mongoid::Document
  include Mongoid::Attributes::Dynamic
  include Mongoid::Timestamps
  include Mongoid::Userstamp::User

  devise :invitable, :encryptable, :database_authenticatable, :registerable, :lockable, :recoverable, :confirmable, :rememberable, :trackable, :validatable, :timeoutable

  has_and_belongs_to_many :groups

Thanks!

You can override block_from_invitation? method, current code is:
        def block_from_invitation?
          invited_to_sign_up?
        end

Great.
Thanks for your quick help!

@scambra
maybe you can share a simple way to do that without forking devise_invitable? Adding it to User model doesn't help.
Thanks

You should add that method to your model, below devise line. For example:
def block_from_invitation?
  false
end

@nfedyashev you must be doing something wrong, because it's working for @marcocanderle

@nfedyashev This is what I did on my User model:
class User
  include Mongoid::Document
  include Mongoid::Attributes::Dynamic
  include Mongoid::Timestamps
  include Mongoid::Userstamp::User

  devise :invitable, :encryptable, :database_authenticatable, :registerable, :lockable, :recoverable, :confirmable, :rememberable, :trackable, :validatable, :timeoutable

  has_and_belongs_to_many :groups

  def block_from_invitation?
    # If the user has not been confirmed yet, we let the usual controls work
    if confirmed_at.blank?
       return invited_to_sign_up?
    else # if the user was confirmed, we let them in (will decide to accept or decline invitation from the dashboard)
       return false
    end
  end

@scambra @marcocanderle thanks!
The reason why it didn't work for me was the old 1.4.0 version that I was using.

For people googling into this issue, you might need invitation_accepted_at.blank? instead of confirmed_at .

authors: IamNaN marcocanderle nfedyashev scambra


text:
Shikari/html template improve

Warning: This update touches on a potentially controversial design topic!
After spending a bunch of hours reviewing reports in the light theme, I realized I needed something that worked better in dark mode. The purple tones were toxic and overwhelming in a dark theme, so I opted for a softer, less aggressive palette.
Updates list:

Introduced a new color scheme for the HTML report template.
Added full support for dark mode.
Enhanced readability, layout, and overall style.

New color pallet:

Old color pallet:

@japroc do you have the logo in #000? Can you update PR, please?

BTW Gradient here could work better or it's too fancy for a report?  @japroc

repo: scan-io-git/scan-io


text:
test(message): added unit tests

Describe pull-request
Adding unit test files for Message component
Solving issue
Fixes: CDEP-2963
How to test

Check do test cases meet description defined in here.
Try to run a branch on your local and try to change some prop/test in index files, test should fail.

update names for each screenshots after updating the name of the test to the correct variant version.

Nice catch! Code is updated in last commit, please check it when you have time.

repo: scania-digital-design-system/tegel


text:
Post-shutdown errors

Some apps have a built-in shutdown -- for instance, ShoesSpec runs do a full shutdown afterward. This eventually causes wasm errors when we use deleted objects. It would be good to stop dispatching all events, not just heartbeat, after shutdown.

This is the same basic kind of error -- related to ruby.wasm and emscripten -- as in #1.


text:
🛑 Appinn Images Server(AIS) is down

In b38bb6e, Appinn Images Server(AIS) (https://img3.appinn.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Appinn Images Server(AIS) is back up in 9b39777.

repo: scavin/appinnuptime


text:
Add stricter validation for field names

This outputs a error message for an unexpected input for the field name. We validate currently two things.

field Key === field name (this may be change in future)
field name only supports a-z, A-Z, 0-9 and _ (see #426)

fix #426
FYI @Toflar

PS: I'm using internal assert of PHP to avoid overheads in validation for production. For CI using the widely used https://github.com/shivammathur/setup-php It is required to configure zend.assertions=1 via::
            - name: Install and configure PHP
              uses: shivammathur/setup-php@v2
              with:
                  php-version: ${{ matrix.php-version }}
                  tools: 'composer:v2'
                  ini-values: memory_limit=-1, zend.assertions=1

repo: schranz-search/schranz-search


text:
how to detect changes after simulation step

I'm trying to implement server-side physics calculations for my game with cannon.js and I need to stream that calculation's result to a player. Basically it is objects positions and rotation. But I don't want to stream updates on all objects. If something is lies on the ground and do not moving, then I should not send update for this object. So what options do I have to detect which objects's rotation\position was changed after each simulation step? The first and obvious solution that came to mind is to just have array of observable entities properties and then compare them with objects in world.bodies after each simulation step. But this seems to be a bit costly especially given the fact that I run physic simulation 3 times per second.

Hi,
a good solution would be to add some network node type on top of the viewnode.
Then you can add setters like:
set position(val){         this.needsSync = true;         this.__position = val;     }
That way you can loop over the nodes and send only the updated ones.
I hope that helps :)
Cheers
Stefan

Won't change position:

Static bodies
Sleeping bodies
Dynamic and Kinematic bodies with velocity.length() == 0

The rest will update their position.
Same goes for angularVelocity + quaternion.
This may be related: https://udn.epicgames.com/Three/NetworkingOverview.html#Physics

found a way to mark bodies that moved since last simulation step:
body.moved = !body.position
      .vsub(body.previousPosition)
      .almostZero(0.2)

authors: SET001 digitaledge schteppe


text:
Import symbols into google namespace to maintain backwards compatibility

Original issue 83 created by schuhschuh on 2014-05-02T10:09:18.000Z:
As the C++ namespace of the gflags functions and types has changed from "google" to "gflags", it requires the update of existing project that make use of it, in particular other Google projects such as glog. Setting the CMake variable GFLAGS_NAMESPACE is possible for single users, but not for the creation of binary packages of gflags for common Linux distributions.
Therefore, to maintain backwards compatibility, I suggest to add a gflags_google.h file which contains the following code:
namespace google {
using gflags::FlagRegisterer;
// ....
}
with a using statement for each function and type that existed before version 2.1 in the google namespace.

Merged into master branch and ready for release.

repo: schuhschuh/gflags


text:
Add dask support and rocketfft to speedup FFT

Hi @schutyb ,
Sorry for the delay! I now had some time to hopefully collaborate with some code.
I was comparing both our phasor calculation functions and I think we could have a common version so that, for example, I could add this package as one of the dependencies of napari-flim-phasor-plotter instead of the less preferable current way, which is referencing and copying an older version of the function.
The suggested changes bring support to dask arrays and use rocket-fft to speed up FFT calculation. This may be good, but also consider that your package will have to add these extra dependencies.
If you agree, I would ask you to check if such changes would not introduce errors to your package.
Let me know what you think of this, if you would need me to change something, or if you'd rather not add these dependencies.
Best,
Marcelo

Thanks Marcelo, I merged, but I have the following error
numba.core.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Unknown attribute 'fft' of type Module(<module 'numpy.fft' from '/usr/lib/python3/dist-packages/numpy/fft/init.py'>)
Do you have any idea why? I searched on google but numba doc its a little confusing.
After I solve it I will taste it with some large spectral file in order to compare time.

authors: schutyb zoccoler

repo: schutyb/rep-hsipy


text:
New volume rendering and iso contour control

updated volume rendering code
added iso surface rendering control
removed limited zoom in

go ahead , you can commit on visus_javascript directly

authors: scrgiorgio spetruzza


text:
sample_domain array generated by the dataset should contain integer domain labels rather than floats

I guess the change happened somewhere in the process of moving from older version of the API, unintentionally.

Excuse me but do you still have the piece of code for which the array contains float domain labels?

Excuse me but do you still have a piece of code where the array contains float domain labels?

Mr. Gnassounou just gave me an exemple, so nevermind

authors: BuenoRuben kachayev

repo: scikit-adaptation/skada


text:
Pydocstyle advanced fixes

This PR is a continuation of fixing pydocstyle linting errors. The work is still incomplete, but further fixes will require significant review.
Please complete the following checklist:

[x] I have read the contribution guidelines.

[ ] I have documented all public-facing changes in the changelog.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied.

It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[x] This pull request does not include code, documentation, or other content derived from external source(s).

Note: This document may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.

@mataton Thank you! Merged.

authors: mataton qiyunzhu

repo: scikit-bio/scikit-bio


text:
Implement ROOTStore

Pandas features HDFStore, a class for easily inspecting and modifying HDF5 files.
Implementing a similar ROOTStore could alleviate some of the pain points that remain when working with root_pandas.
Examples where a ROOTStore could speed up working with ROOT files:

Which trees are in this file?
How many rows/columns do they contain?
Saving several trees at once
Adding a tree to an existing file
Deleting a tree

@chrisburr, I would simply close this task at this point.

Nobody will work on this at this stage. Closing.

authors: eduardo-rodrigues ibab


text:
classname_of gives Exception when used on TBranch

Please consider the following source code
import uproot
import numpy as np

with uproot.recreate("test.root") as f:
    f["Tree"] = {"x": np.array([1,2,3,4,5])}

f = uproot.open("test.root")
print(f.classname_of("Tree/x"))

I would expect it to print "TBranch". However, it actually raises an exception: AttributeError: 'Model_TTree_v20' object has no attribute 'key'.
Uproot version: 4.1.9

I hadn't thought of this as a potential use, but since "/" hides the distinction between paths into nested directories and paths into nested branches, it makes sense that one would try passing such a path into classname_of, streamer_of, class_of, title_of, etc.
The difficulty is that the path-handling is a specialized implementation in uproot.ReadOnlyDirectory.key
https://github.com/scikit-hep/uproot4/blob/3fe33aff6d6ed88203eed90f4edb5f2bc720f3e6/src/uproot/reading.py#L2006-L2021
which would have to get generalized out to be able to use them in the other functions that take paths.
Or maybe it just means that uproot.TBranch.HasBranches objects (i.e. TTree and all TBranch classes) should have a key method? (What about TBranches that contain TBranches? Would that work? I'm talking to myself...) Such a method can't return an uproot.reading.ReadOnlyKey because TTrees and TBranches don't have them the way that uproot.ReadOnlyDirectory does, but maybe a mocked "FakeKey" could duck-type to supply the appropriate information...

This feature request is still open, but I'm not going to attempt to get it in before the 5.0.0 release (next Monday). I'm just removing that label.

authors: jpivarski jrueb


text:
i got  504 error when i followed your config steps

nginx:
server {
listen 80;
error_log  /usr/local/nginx/logs/error.log;
charset utf-8;
root /Users/liufa/Documents/www/laravel_blog/public;
server_name  www.laravel_blogfly.com;
# useful when debug
#504 Gateway timeout error http://stackoverflow.com/questions/18229757/nginx-php-fpm-xdebug-netbeans-can-start-only-one-debug-session

# for firefox, otherwise firefox may dowload some page; default file type is: octet-stream
include /usr/local/nginx/mime.types; default_type text/html;
location / {
    # you can place index.html on document root dir
    index  index.php index.html;
    try_files $uri $uri/ @other;
}

location = / {
    proxy_http_version 1.1;
    proxy_set_header Connection "keep-alive";
    proxy_pass http://127.0.0.1:9501;
    proxy_set_header    X-Real-IP        $remote_addr;
    proxy_set_header    X-Forwarded-For  $proxy_add_x_forwarded_for;
    proxy_set_header    Host             $http_host;
}
location @other {
    proxy_http_version 1.1;
    proxy_set_header Connection "keep-alive";
    proxy_pass http://127.0.0.1:9501;
    proxy_set_header    X-Real-IP        $remote_addr;
    proxy_set_header    X-Forwarded-For  $proxy_add_x_forwarded_for;
    proxy_set_header    Host             $http_host;
 }

}

?
504 Gateway timeout error
http://stackoverflow.com/questions/18229757/nginx-php-fpm-xdebug-netbeans-can-start-only-one-debug-session

I have seen  the page you gave me , it is different from my problem.  There is nothing to do with xdebug . I do not know why

is  your laravelfly server ( that is a swoole server )  working well?
first visit   ip:9501  ( default 'listen_port' is  9501 )

默认端口9999 直接 curl127.0.0.1 无法访问 swoole正常运行

我也遇到一样的问题  用的是swool2.0+ 请问是不是不支持呢？

authors: jonny77 kevingarnetttwo saiwfy scil

repo: scil/LaravelFly


text:
topogen: do not allocate quic port

RPCs over quic do a negotiation first.
There is no default quic port required.
topogen should set it to 0 in order to catch bugs where a specific quic port is assumed.

Implemented in #3917

authors: matzf oncilla

repo: scionproto/scion


text:
dll load error scipy\special_init_.py  Line 641 from ._ufuncs import *

My issue is about ...
While running the Winpython Default checker this DLL Load Error Occurred
Reproducing code example:
C:\Program Files\WPy64-3740\python-3.7.4.amd64\lib\site-packages\scipy\special\__init__.py in <module>
    639 from .sf_error import SpecialFunctionWarning, SpecialFunctionError
    640
--> 641 from ._ufuncs import *
    642
    643 from .basic import *

ImportError: DLL load failed: The specified module could not be found.

Sample code to reproduce the problem:

Error message:
ImportError                               Traceback (most recent call last)

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  ...

Scipy/Numpy/Python version information:
1.3.0 1.16.4 sys.version_info(major=3, minor=7, micro=4, releaselevel='final', serial=0)

Hey @mldavis99; hope you don't mind but I edited your markdown a bit; a lot of important bits had been left between <!-- -->, which won't show up in the compiled text.

@mldavis99 are you using the dev setup for scipy? I usually get a similar message when I pull from upstream master into my local repo.  Fix for me is ‘git clean -xdf’ the. Rerun the inplace build from root of dev directory for scipy.

On Aug 25, 2019, at 3:49 AM, Josh Wilson notifications@github.com wrote:
Hey @mldavis99; hope you don't mind but I edited your markdown a bit; a lot of important bits had been left between , which won't show up in the compiled text.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Looks like this is a WinPython distribution, which we don't distribute, so this is likely better taken up with them. Also, based on the error it's likely that there are conflicting SciPy packages installed. Closing this since I don't think it's an issue on our side, but feel free to continue the discussion.

repo: scipy/scipy


text:
Input data validation for RectSphereBivariateSpline

I am trying to interpolate some 2D data on a sphere using scipy.interpolate.RectSphereBivariateSpline. Despite following the documentation, I get a data validation error. I've uploaded a sample netcdf file with data to trigger the error. As far as I can see, I've satisfied the requirements on the input latitude and longitude variables.
Reproducing code example:
from netCDF4 import Dataset
from scipy import interpolate

with Dataset('spl_debug.nc', 'r') as fid:
    u = fid.variables['colatitude'][:]
    v = fid.variables['longitude'][:]
    r = fid.variables['input_var'][:]

spl = interpolate.RectSphereBivariateSpline(u, v, r, s=0, pole_continuity=(True,True))

Error message:
ValueError                                Traceback (most recent call last)
/gpfsm/dhome/sbasu1/Code/GEOS/mass_flux/spl_bug.py in <module>
      7     r = fid.variables['input_var'][:]
      8
----> 9 spl = interpolate.RectSphereBivariateSpline(u, v, r, s=0, pole_continuity=(True,True))

/discover/nobackup/projects/tm5var4d/packages/python/3.8.2/lib/python3.8/site-packages/scipy-1.4.1-py3.8-linux-x86_64.egg/scipy/interpolate/fitpack2.py in __init__(self, u, v, r, s, pole_continuity, pole_values, pole_exact, pole_flat)
   1721         if ier not in [0, -1, -2]:
   1722             msg = _spfit_messages.get(ier, 'ier=%s' % (ier))
-> 1723             raise ValueError(msg)
   1724
   1725         self.fp = fp

ValueError:
ERROR: on entry, the input data are controlled on validity
       the following restrictions must be satisfied.
          -1<=iopt(1)<=1, 0<=iopt(2)<=1, 0<=iopt(3)<=1,
          -1<=ider(1)<=1, 0<=ider(2)<=1, ider(2)=0 if iopt(2)=0.
          -1<=ider(3)<=1, 0<=ider(4)<=1, ider(4)=0 if iopt(3)=0.
          mu >= mumin (see above), mv >= 4, nuest >=8, nvest >= 8,
          kwrk>=5+mu+mv+nuest+nvest,
          lwrk >= 12+nuest*(mv+nvest+3)+nvest*24+4*mu+8*mv+max(nuest,mv+nvest)
          0< u(i-1)<u(i)< pi,i=2,..,mu,
          -pi<=v(1)< pi, v(1)<v(i-1)<v(i)<v(1)+2*pi, i=3,...,mv
          if iopt(1)=-1: 8<=nu<=min(nuest,mu+6+iopt(2)+iopt(3))
                         0<tu(5)<tu(6)<...<tu(nu-4)< pi
                         8<=nv<=min(nvest,mv+7)
                         v(1)<tv(5)<tv(6)<...<tv(nv-4)<v(1)+2*pi
                         the schoenberg-whitney conditions, i.e. there must be
                         subset of grid co-ordinates uu(p) and vv(q) such that
                            tu(p) < uu(p) < tu(p+4) ,p=1,...,nu-4
                            (iopt(2)=1 and iopt(3)=1 also count for a uu-value
                            tv(q) < vv(q) < tv(q+4) ,q=1,...,nv-4
                            (vv(q) is either a value v(j) or v(j)+2*pi)
          if iopt(1)>=0: s>=0
          if s=0: nuest>=mu+6+iopt(2)+iopt(3), nvest>=mv+7
       if one of these conditions is found to be violated,control is
       immediately repassed to the calling program. in that case there is no
       approximation returned.

Scipy/Numpy/Python version information:
import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.4.1 1.18.4 sys.version_info(major=3, minor=8, micro=2, releaselevel='final', serial=0)

I suppose we may want to check with a more recent version of SciPy, if there is indeed a genuine concern of a bug here.

If someone already has a more recent version of scipy installed, can they run the above code snippet to verify the presence/absence of the bug? Would be more efficient than me changing my scipy stack, only to find that the bug is still there.

I can confirm I can reproduce this on latest.

Indeed, adding +2 to nuest = mu+6+2 in the f2py wrappers fixes it, cf gh-16872.
The minimal reproducer is
    u = np.arange(1, 10) *np.pi / 10
    v = np.arange(1, 10) *np.pi / 10
    r = np.zeros((9, 9))
    RectSphereBivariateSpline(u, v, r, s=0, pole_continuity=(True, True))

and I checked locally that gh-16872 fixes the error with the OP dataset too. The latter is a 850k netCDF file with 361x507 size masked arrays --- a bit too much to add to the test suite unless really needed.

authors: AgilentGCMS ev-br tupui tylerjereddy


text:
BUG: high istft reconstruction error

Describe your issue.
While doing testing I discovered that the reconstruction of a signal via scipy.signal.stft->istft is very high compared to other implementations (e.g. librosa). The below example shows the max absolute error in the reconstruction of a sample signal as -78dB compared to -347dB for librosa and my reference code.
I do get a NOLA warning in the call, but I'm confused as to why - when I call scipy.signal.check_NOLA with my window manually, the check passes.
Full disclosure stride.py is my library :grin: - I provided it as a reference as the implementation is transparent compared to the librosa implementation which has several function calls.

SA1.wav.zip
Reproducing Code Example
# %% Setup
import matplotlib.pyplot as plt
import numpy as np
import stride as st
import soundfile as sf

x, fs = sf.read('SA1.wav') # see attached zip file
blocksize = 0.020
hopsize = 0.010
nfft = 2048

L = int(fs*blocksize)
K = int(fs*hopsize)
w = np.hanning(L+1)[1:]    # DFT tapering window

#%% Reconstruct signal and calculate error
# Note I ignored a chunk of samples off the end because librosa has a high error here for some reason.

# stride.py (https://github.com/tgarc/stride)
strider = st.STFTStrider(w, K, nfft)
X = strider.stft(x)
y = strider.istft(X)
y = y[:-5000]
xp = x[:len(y)]
mask = xp != 0
print("Error:", 10*np.log10(np.mean(np.abs(xp-y)**2)))
yst = y
est = np.abs((xp[mask]-y[mask]))

# librosa
import librosa as lr
Z = lr.stft(x, hop_length=K, window=np.pad(w, (0,nfft-L)), n_fft=nfft, center=False)
y = lr.istft(Z, hop_length=K, window=np.pad(w, (0,nfft-L)), n_fft=nfft, center=False)
y = y[:-5000]
xp = x[:len(y)]
mask = xp != 0
print("Error:", 10*np.log10(np.mean(np.abs(xp-y)**2)))
ylr = y
elr = np.abs((xp[mask]-y[mask]))

# scipy reconstruction
from scipy import signal
Z = signal.stft(x, nperseg=L, nfft=2048, noverlap=L-K, padded=False, boundary=None)[-1]
y = signal.istft(Z, window=w, nperseg=L, nfft=2048, noverlap=L-K, boundary=False)[-1]
y = y[:-5000]
xp = x[:len(y)]
mask = xp != 0
print("Error:", 10*np.log10(np.mean((xp-y)**2)))
ysp = y
esp = np.abs((xp[mask]-y[mask]))
# sf.write('SA1hat.wav', y, fs, 'PCM_16')

#%% Plot the error
fig, axes = plt.subplots(nrows=3, sharex=True)
for ax, title, err in zip(axes, 'stride.py librosa scipy'.split(), (est, elr, esp)):
    ax.set_title('$\it{'+ title + '}$')
    ax.plot(err[:-5000])
    ax.set_xlim([0, len(err)-5000])
    ax.text(0.99, 0.95, 'Max Error: %.2fdB' % (10*np.log10(np.mean(err[:-5000]**2))),
    color='red',
     horizontalalignment='right',
     verticalalignment='top',
     transform = ax.transAxes)
fig.add_subplot(111, frameon=False)
plt.tick_params(labelcolor='none', which='both', top=False, bottom=False, left=False, right=False)
plt.ylabel(r'Magnitude Error := $|x-\hat{x}|$')

Error message
C:\anaconda3\lib\site-packages\scipy\signal\spectral.py:1439: UserWarning: NOLA condition failed, STFT may not be invertible
  warnings.warn("NOLA condition failed, STFT may not be invertible")

SciPy/NumPy/Python version information
1.6.2 1.20.1 sys.version_info(major=3, minor=8, micro=8, releaselevel='final', serial=0)

I studied your example a little bit and found two issues:

The window w was given as parameter for scipy.istft() but not for scipy.stft(). That results in Hann windows of different length.
Setting the parameter padding=None gives problems at the end, since the last slices are missing. This is also the reason for the NOLA warning.

Fixing that delivers the expected accuracy. I modified your example a little bit, to demonstrate the boundary effects:
import numpy as np
from scipy import signal
import soundfile
import librosa as lr

x, fs = soundfile.read('SA1.wav')
t_x = np.arange(len(x)) / fs
block_size = 0.020
hop_size = 0.010
nfft = 2048

L = int(fs * block_size)
K = int(fs * hop_size)
w = np.hanning(L+1)[1:]    # DFT tapering window

Z_lr = lr.stft(x, hop_length=K, window=np.pad(w, (0, nfft-L)),
               n_fft=nfft, center=False)
y_lr = lr.istft(Z_lr, hop_length=K, window=np.pad(w, (0, nfft-L)),
                n_fft=nfft, center=False)

Z_no_b = signal.stft(x, window=w, nperseg=L, nfft=2048, noverlap=L-K,
                     padded=False, boundary=None)[-1]
y_no_b = signal.istft(Z_no_b, window=w, nperseg=L, nfft=2048, noverlap=L-K,
                      boundary=False)[-1]

Z_b = signal.stft(x, window=w, nperseg=L, nfft=2048, noverlap=L-K,
                  padded=True, boundary='zeros')[-1]
y_b = signal.istft(Z_b, window=w, nperseg=L, nfft=2048, noverlap=L-K,
                   boundary=True)[-1]

i1 = len(x) - (L - K)
nn = ('librosa', 'SciPy no boundary', "SciPy with boundary='zeros'")
for n_, y_, Z_ in zip(nn, (y_lr, y_no_b, y_b), (Z_lr, Z_no_b, Z_b)):
    print(f"{n_}:")
    print(f" * {len(x)=}, {len(y_)=}, diff: {len(y_)-len(x)}, {Z_.shape=}")
    i0 = min(len(x), len(y_))
    e_full = np.max(np.abs(y_[:i0] - x[:i0]))
    print(f" * Max deviation for complete signal: {e_full}")
    e1 = np.abs(y_[:i1] - x[:i1])
    print(f" * Max deviation without end: {np.max(e1)}")

# Output:
#
# librosa:
#  * len(x)=54580, len(y_)=54528, diff: -52, Z_.shape=(1025, 329)
#  * Max deviation for complete signal: 0.00140380859375
#  * Max deviation without end: 0.00140380859375
# SciPy no boundary:
#  * len(x)=54580, len(y_)=54560, diff: -20, Z_.shape=(1025, 340)
#  * Max deviation for complete signal: 3.0517578125e-05
#  * Max deviation without end: 8.326672684688674e-17
# SciPy with boundary='zeros':
#  * len(x)=54580, len(y_)=54720, diff: 140, Z_.shape=(1025, 343)
#  * Max deviation for complete signal: 8.326672684688674e-17
#  * Max deviation without end: 8.326672684688674e-17

I have no idea where the large deviation in librosa come from. I did not look at stride.py.

Thanks for looking at this, obviously it was the missing window parameter that made for the bad results. Everything looks good now.

Thanks for looking at this, obviously it was the missing window parameter that made for the bad results. Everything looks good now.

Thanks for looking at this, obviously it was the missing window parameter that made for the bad results. Everything looks good now.

I guess this issue has been resolved for a while now.

authors: DietBru tgarc tjgarcia-mchp vollbier


text:
ImportError: No module named ndimage

I am trying to import ndimage from spicy like
from scipy import ndimage
it gives me an error stating that

Import Error: No module named ndimage

Please let me know how to resolve this issue

Your install is incomplete or somehow messed up. You can see where scipy is installed with::
>>> import scipy
>>> scipy.__file__

Complete reinstallation may fix it. Please provide details on how you installed if you need more help

`import scipy

scipy.file
'/usr/local/lib/python2.7/site-packages/scipy/init.pyc'`

I see that scipy is installed in the correct location.
I installed scipy using ports
sudo port install py27-scipy

authors: ashirsat rgommers


text:
ENH: add Dag Sverre's cythonize command

./tools/cythonize checks for modifications in pyx files relative to the
corresponding c file, and updates the c file if it detects a mismatch.

Looks good. An option to regenerate all Cython files at once would also be quite useful, since this often has to be done before releasing.
Is Tempita the standard in Cython-world now? If so we may want to convert the Mako templates at some point.

Is merged now. It already made me find something new, a rewrite of cluster/vq in Cython that wasn't finished.
Will look at adding the regenerate-all-files option next time I need it. Thanks Matthew.

authors: matthew-brett rgommers


text:
interp1d bounds_error is redundant

The bounds_error argument of interp1d seems to be completely redundant. The extrapolation behaviour of the function could be controlled with just the fill_value argument using the following logic:

If fill_value is a tuple, ndarray, or 'extrapolate', the behaviour is as it is now except no error would be raised.

If fill_value = None (or possibly False?), error is raised when trying to extrapolate (this would be new behaviour)

Even though the current behaviour of the function is documented well, it is counter-intuitive that two separate arguments need to be set when the user wants to use a constant fill value.
Perhaps another way to refactor the current behaviour would be to replace both of the arguments with one extrapolate argument which can be a bool or an ndarray-like. If The argument is set to True, the values are extrapolated; if False, an error is raised; if ndarray, then those values are used as constant fill values.

The interface of interp1d is very constrained by backwards compatibility. Essentially any tweak is bound to break something downstream.
New code should indeed use extrapolate= argument, but for interp1d specifically, I'd rather keep the status quo.

No follow-up for over a month, closing.


text:
ENH: Update Scipy Gitpod

Reference issue

Related to #13677
What does this implement/fix?

This PR tackles the issues mentioned in the above issue and makes improvements to the GitPod integration
Editor

[x] Make sure the Python extension is installed -> since we are using our custom image
[x] Remove deprecated settings in VSCode (PythonPath)

Docker images

[x]  Add the gh cli (https://github.com/cli/cli) to the gitpod container
[x] Build Gitpod image nightly instead of on pushes to master (in sync with SciPy builds)
[x] Add VSCode extension RestructuredText and sphinx-autobuild -> should remove the rst warning when initialising a workspace
[x] Added use of ccache (thanks to @thomasjpfan for the inspiration) to speed up build in Gitpod

GitHub actions

[x] Cache image layers to reduce build times
[x] Base image now only gets built and pushed when there are changes to environment.yml -> Deleted environment.yml to root so that it can be overwritten by @rgommers update later on
[x] Add workflow to build and push the gitpod Docker image (.github/workflows/docker.yml) -> this is the built nightly

Gitpod

[x] Use scipy/scipy-gitpod:latest which is built nightly
[x] ensuring that the most recent prebuild is picked -> building SciPy in Docker container then rebuilding in init so no need to rely on prebuild as a workaround now
[x]  re-run of make html-scipyorg now works as expected

Additional information

[ ]  see if we can name scipy/scipy as upstream rather than origin, to match our contributor guide -> no solution atm though removing the dependency on the build enables folks to work directly from their forks :warning: They seemed to have changed things again on the Gitpod side so this does not seem doable atm; workaround would be to have folks fork and work from those forks
Sending separate PR for documentation on how to use GitPod 📖

🙈 I am sorry I accidentally renamed the branch and this closed the previous PR
Link to my workspace https://gitpod.io#snapshot/d669f48a-02ec-43bf-bbc7-9460257c248d

@trallard this failed in CI with:
 #7 [3/3] RUN git clone https://github.com/scipy/scipy.git  --depth 1 --single-branch . &&     conda activate scipy-dev &&     python setup.py build_ext --inplace &&     ccache -s
#7 sha256:8c862134c5b4de03ff7977352957df41f0b92710f4946a66c8b629a267699775
#7 ERROR: executor failed running [/bin/bash --login -o pipefail -c git clone https://github.com/scipy/scipy.git  --depth 1 --single-branch . &&     conda activate ${CONDA_ENV} &&     python setup.py build_ext --inplace &&     ccache -s]: unable to find user gitpod: no matching entries in passwd file
------
 > [3/3] RUN git clone https://github.com/scipy/scipy.git  --depth 1 --single-branch . &&     conda activate scipy-dev &&     python setup.py build_ext --inplace &&     ccache -s:
------
.gitpod.Dockerfile:29
--------------------
  28 |
  29 | >>> RUN git clone https://github.com/scipy/scipy.git  --depth 1 --single-branch . && \
  30 | >>>     conda activate ${CONDA_ENV} && \
  31 | >>>     python setup.py build_ext --inplace && \
  32 | >>>     ccache -s
  33 |
--------------------
error: failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/bash --login -o pipefail -c git clone https://github.com/scipy/scipy.git  --depth 1 --single-branch . &&     conda activate ${CONDA_ENV} &&     python setup.py build_ext --inplace &&     ccache -s]: unable to find user gitpod: no matching entries in passwd file
Error: buildx call failed with: error: failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/bash --login -o pipefail -c git clone https://github.com/scipy/scipy.git  --depth 1 --single-branch . &&     conda activate ${CONDA_ENV} &&     python setup.py build_ext --inplace &&     ccache -s]: unable to find user gitpod: no matching entries in passwd file

Instead of "user gitpod", I guess this needs some token?

it was working just fine when I was building the images locally - might need to add gitpod manually to /etc/passwd in the container will look into it

Note that the doc build is currently insanely slow. We need to fix that, and then it'll work better on starting up Gitpod as well. For now we can just hit Ctrl-C if the slow build is annoying.

I did notice this recently - it is rather annoying

I did notice this recently - it is rather annoying

Down to 15-20 minutes now, will see later how much further it can be improved.

authors: rgommers trallard


text:
DOC: added mention of the limitations of Thomas' algorithm

Reference issue
Closes gh-14198.
What does this implement/fix?
Added description of the practical limitations of Thomas' algorithm, and mentioned solve_banded as an alternative when solveh_banded fails but a is invertible.

addressing an order of operation issue with string formatting. I am having trouble compiling scipy after I made changes (not related to my code changes),, so I am sorry I didn't catch this issue before commit.


text:
DOC: Change nb and nb to conventional m and n

Also point out that zero-based indexing means that the last coefficient is m-1, not m.
or could be changed to TeX?
$$
H(\omega) = \frac
{b_0 (j\omega)^{(m-1)} + b_1 (j\omega)^{(m-2)} + \cdots + b_{(m-1)}}
{a_0 (j\omega)^{(n-1)} + a_1 (j\omega)^{(n-2)} + \cdots + a_{(n-1)}}
$$

Ok I rebased this and I found problems and am making more changes to it, so I'm going to take notes here as I go along so it's easier for others to check it.  Summary:

zero-based indexing
b is the TF numerator / feedforward coefficients
a is the TF denominator / feedback coefficients
M is the number of zeros, or the order of the numerator
N is the number of poles, or the order of the denominator
there are M+1 b coefficients, the last of which is a[M]
there are N+1 a coefficients, the last of which is b[N]
in SciPy's implementation, difference equations have minus signs before the a coefficients, and therefore transfer function has plus signs before them

So first formula is in
lfilter:
   a[0]*y[n] = b[0]*x[n] + b[1]*x[n-1] + ... + b[nb]*x[n-nb]
                           - a[1]*y[n-1] - ... - a[na]*y[n-na]

This is copied from Matlab's filter, except for 0-indexing, and the implementation is the same.  Instead of n_b and n_a, I'm changing to N and M:
   a[0]*y[n] = b[0]*x[n] + b[1]*x[n-1] + ... + b[M]*x[n-M]
                         - a[1]*y[n-1] - ... - a[N]*y[n-N]

http://www.cs.man.ac.uk/~barry/mydocs/COMP30291/Notes/DMP09-5.doc is the same except they swap a and b.
eq 8.2 in http://www.eas.uccs.edu/wickert/ece2610/lecture_notes/ece2610_chap8.pdf is equivalent except the a terms are negated.
same as section 4.3.2 in google books except they swap M and N.
Then it has the transfer function:
                         -1               -nb
             b[0] + b[1]z  + ... + b[nb] z
     Y(z) = ---------------------------------- X(z)
                         -1               -na
             a[0] + a[1]z  + ... + a[na] z

Also the same as Matlab except 0-indexing.  I'm changing nb and na to M and N:
                         -1              -M
             b[0] + b[1]z  + ... + b[M] z
     Y(z) = -------------------------------- X(z)
                         -1              -N
             a[0] + a[1]z  + ... + a[N] z

same notation as eq 8.8 in http://www.eas.uccs.edu/wickert/ece2610/lecture_notes/ece2610_chap8.pdf except the a terms are negated.
same as section 4.3.2 in google books except they swap M and N.

residue and invres:
Originally used M = len(b) and N = len(a):
          b(s)     b[0] s**(M-1) + b[1] s**(M-2) + ... + b[M-1]
  H(s) = ------ = ----------------------------------------------
          a(s)     a[0] s**(N-1) + a[1] s**(N-2) + ... + a[N-1]

Changed to M, N = order of polynomials:
          b(s)     b[0] s**(M) + b[1] s**(M-1) + ... + b[M]
  H(s) = ------ = ------------------------------------------
          a(s)     a[0] s**(N) + a[1] s**(N-1) + ... + a[N]

Simplifies formula and is now consistent with other docstrings and also Matlab's residue (other than 1-indexing)
invres had x instead of s, made them consistent with each other.
residuez and invresz:
Originally:
            b(z)     b[0] + b[1] z**(-1) + ... + b[M-1] z**(-M+1)
    H(z) = ------ = ----------------------------------------------
            a(z)     a[0] + a[1] z**(-1) + ... + a[N-1] z**(-N+1)

Changed to:
            b(z)     b[0] + b[1] z**(-1) + ... + b[M] z**(-M)
    H(z) = ------ = ------------------------------------------
            a(z)     a[0] + a[1] z**(-1) + ... + a[N] z**(-N)

Changed to order of num,den instead of number of coefficients

Also was going to add some examples for the residue functions, but now I see there are no unit tests for them...  So I think that should be another PR.

Should I change the word "order" to "degree"?  http://mathworld.wolfram.com/PolynomialOrder.html http://en.wikipedia.org/wiki/Order_of_a_polynomial

not sure - for polynomials "degree" is better, but "order" seems to be used specifically for filter design. I don't work with these filters enough to have a clear expectation.

I changed them to "degree".
I'm confused by lfilter's use of n for both sample number and model order, and use of m for sample number, too.

I updated the state variable version of the difference equations, which was originally just copied from matlab.

changed to sample number n for consistency with other difference equation
I moved state variables last, same as http://cnx.org/contents/ef583915-43cb-4189-be10-f0d776088304@1/Filter_Selection_and_Construct
a0 is allowed to be not 1, but the difference equation (and matlab's diagram) assume it is 1, so I added it back in.
changed state variable names to d to keep from confusion with z⁻¹ .  There's no consistent name for these elsewhere: d, s, w, x, etc.
Matlab's isn't really correct when N>M, either, so I just made the assumption they are equal, which is common, for simplicity.
total number of coefficients was wrong, if N is the degree of the numerator, then there are N+1 coefficients, numbered from b0 to bN.

I'm leaving this on the 0.16.0 milestone, so I'll make sure to have a look at it before that release - this has been open for too long already.

Can this be reviewed please?  :)  It's almost 2 years old now

Can this be reviewed please?  :)  It's almost 2 years old now

From my first read through of the discussion and changes, this all looks good with one minor language quibble of mine: I think "degree of the denominator" sounds a little weird, I would prefer something like "Degree of the denominator polynomial."
Caveat: I haven't looked for other docstrings that may also need adjustment, which may have cropped up given the time since the last commit...

I think "degree of the denominator" sounds a little weird

looks like this is the normal way to say it:
https://www.google.com/search?q="degree+of+the+denominator"

Squashed and merged in 4ffc02a and a71ab83, thanks @endolith!

authors: e-q endolith rgommers


text:
add Bpoly.antiderivatives / integrals

Add the functionality for integrating piecewise polynomials directly in the Bernstein basis without converting them to the power basis.

Fixed the typo --- thanks Alex! (the typo and the comment by @argriffing have disappeared when I force-pushed the last commit)

I believe I addressed all review comments, all branches seem to be tested.

Thanks, lgtm

@ev-br: Ok, I missed the new axis argument. It probably should be passed to construct_fast also here in the antiderivative routine.

authors: ev-br pv


text:
FIX: iterable checks longest

Fixes https://github.com/scipy/scipy_doctest/issues/161 by iterating the longest array

Nice catch! Could you add a test please.
ATM it's probably easiest to add a case to https://github.com/scipy/scipy_doctest/blob/main/scipy_doctest/tests/module_cases.py
(we'll need to refactor tests into explicitly testing DTChecker, but that's separate)

I don't know how to create such test. I want to test the failing behavior, so shouldn't I place tests at failure_cases.py ?
196
197     >>> [1, 2, 3]
Expected:
    [1, 2, 3, 4]
Got:
    [1, 2, 3]

Yes indeed, failure_cases.py or failure_cases_2.py. You probably want something like
--- a/scipy_doctest/tests/failure_cases.py
+++ b/scipy_doctest/tests/failure_cases.py
@@ -15,3 +15,10 @@ def func10():
     >>> import numpy as np
     >>> np.arraY([1, 2, 3])
     """
+
+
+def func_lengths():
+    """
+    >>> [1, 2, 3]
+    [1, 2, 3, 4]
+    """
diff --git a/scipy_doctest/tests/test_testmod.py b/scipy_doctest/tests/test_testmod.py
index 0c0a839..2de341b 100644
--- a/scipy_doctest/tests/test_testmod.py
+++ b/scipy_doctest/tests/test_testmod.py
@@ -99,6 +99,13 @@ def test_user_context():
                 config=config)

+def test_wrong_lengths():
+    config = DTConfig()
+    res, _ = _testmod(failure_cases, strategy=[failure_cases.func_lengths],
+                      config=config)
+    assert res.failed == 1      # or use raise_on_error=True and pytest.raises
+
+

Minor: as far as commit message acronyms go, FIX: is not a thing. You probably want BUG:

authors: ev-br luxedo


text:
Add warning as suggested by @jakobvogel #199

I add the warning as suggested by @jakobvogel to the respective line of code discussed in #199.
Please let me know if you want any changes in the comment.

@akwick Many thanks!! 😊

authors: akwick jakobvogel


text:
TestFailOnceActivityInterceptor - available but disabled by default.

I have been running into many problems with https://github.com/sclasen/swfsm/pull/57; however, it does not appear to be because of this change directly, but rather the extra time this change introduces to tests.  Because there is still utility in this interceptor (and other logging introduced during debugging), I am going to pull this in as is, but disabled by default.

@dpiddy already reviewed the enabled version, so merging this, but feedback is welcome.

repo: sclasen/swfsm


text:
Add listing of source packages to common scripts

Factor repoquery argument construction to common/functions.sh
Generate text file listing SRPMs for all packages currently present in
the tested repository.

LGTM, let's try it in practice.

authors: hhorak khardix

repo: sclorg/sclo-ci-tests


text:
X509 certificate signed by unknown authority

Using the LAN email
ESMTP

I have this problem too ,can you have resoves ,thanks

in windows ,it is ok ,but in linux ,it doesnt work .

Hi,
I remember I had this problem myself some time ago and did a small test that fixed it: https://github.com/scorredoira/basicSmtp/blob/master/smtp.go.
I will check it again to integrate it.
thanks

Hi ,
I modify some codes in smtp.SendMail() :
Befor ：
if ok, _ := c.Extension("STARTTLS"); ok {
config := &tls.Config{ServerName: c.serverName}
if testHookStartTLS != nil {
testHookStartTLS(config)
}
if err = c.StartTLS(config); err != nil {
return err
}
}
After：
if ok, _ := c.Extension("STARTTLS"); ok {
config := &tls.Config{ServerName: c.serverName,# InsecureSkipVerify:true}
if testHookStartTLS != nil {
testHookStartTLS(config)
}
if err = c.StartTLS(config); err != nil {
return err
}
}
Then ,it works .

authors: CodyGuo scorredoira zhang1shuo

repo: scorredoira/email


text:
No YAML output for failing tests

Hi @scottcorgan,
I have this TAP output:
# it gets the correct least significant bit [TestCase: 0]
not ok 7 should be equal
  ---
    operator: equal
    expected: 1111
    actual:   0
  ...

This should result in the YAML output being displayed by tap-dot (see the TAP 13 specification).
This expected behaviour can be seen in the screenshots in README.

However when I then pipe the output to tap-dot, I get the following output:

I'm having this same issue.  The command I'm running is ./node_modules/.bin/tape -r babel-register test/**/*.js | ./node_modules/.bin/tap-dot.

Having same issue

I've done some investigation into this, and it appears that in tap.on('assert', function (res) {, res.error is this:
{
    operator: undefined,
    expected: undefined,
    ctual: undefined,
    at: undefined
}

I've reported this here https://github.com/scottcorgan/tap-out/issues/23
It is an issue with tap-out, not an issue with tap-dot specifically.

@searsaw, @luispablo, I have fixed this in #12. @scottcorgan needs to merge it but then it will be fixed :smile:

authors: Jameskmonger luispablo searsaw

repo: scottcorgan/tap-dot


text:
Added basic support for SDK style projects by mimicking previous behaviour of package.config

Upon build, a .targets file copies CoreTools and Spkl from "C:\Users<USERNAME>.nuget\packages" to a packages folder at the solution level.
Due to the new limitations on nuget restore and PackageReference, the project needs to be built before or the Spkl artifacts will be copied across.

This alleviates issue #274 - "spkl: PackageReference in csproj"

Replaced with #410

repo: scottdurow/SparkleXrm


text:
Gracefully handle unreferenced gradients with --keep-unreferenced-defs

Closes: #156
Signed-off-by: Niels Thykier niels@thykier.net

Codecov Report

Merging #173 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
+ Coverage   91.47%   91.48%   +<.01%
==========================================
  Files           5        5
  Lines        2136     2137       +1
==========================================
+ Hits         1954     1955       +1
  Misses        182      182

Impacted Files
Coverage Δ

scour/scour.py
93.13% <100%> (ø)
: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 7115e82...26f7eb6. Read the comment docs.

Looks fine!
Before I merge a more fundamental question: Is it fine to de-duplicate gradients if --keep-unreferenced-defs is used to start with?
I tend towards "yes" as I assume --keep-unreferenced-defs is mostly used for non-gradients and even then a single copy of the unreferenced gradient should be enough unless the user has some very uncommon use case for duplicated and unreferenced gradients, but it seems like a good time to discuss this.

I think I am the wrong person to ask whether de-duplication makes sense here. :) I have no use-case for either behaviour (I just noticed that I now knew scour well enough to provide a potential patch for #156).
The only thing I can think of is: Do people rely on --keep-unreferenced-defs so they can reference it from parts of the SVG that scour does not understand/check for references?  (Issue #164 is the closest example I can think of, but I know too little SVG to say whether that case is applicable to this problem).

Well, obviously people could create CSS that overrides properties of gradients as well, but considering we do not account for any style attribute on gradients they are unfortunately "doomed" either way...
Scour currently only supports inline CSS, so using it in a different way is likely to cause undefined/broken behavior.
I think it's safe to remove duplicate gradients for now unconditionally (I can only try to make an educated guess myself...) and is probably what the majority of users would expect.

authors: Ede123 codecov-io nthykier

repo: scour-project/scour


text:
If server doesn't specify an encoding, requests always tries to guess it

The logging statement that uses response.text means that if a character encoding isn't specified in the response header, requests always guesses the encoding. This can be slow for large responses.

And again...

repo: scraperwiki/data-services-helpers


text:
error: venv/lib/python3.9/site-packages/dateparser/date_parser.py:35: PytzUsageWarning: The localize method is no longer necessary, as this time zone supports the fold attribute (PEP 495). For more details on migrating to a PEP 495-compliant implementation, see https://pytz-deprecation-shim.readthedocs.io/en/latest/migration.html

I'm getting
 venv/lib/python3.9/site-packages/dateparser/date_parser.py:35: PytzUsageWarning: The localize method is no longer necessary, as this time zone supports the fold attribute (PEP 495). For more details on migrating to a PEP 495-compliant implementation, see https://pytz-deprecation-shim.readthedocs.io/en/latest/migration.html
running 1.1.1, and I saw some similar discussions, but since I'm on the latest version it doesn't' look like it's fixed.   The code keeps going, so I guess it's a warning, but it's breaking other things for me.

Another duplicate of https://github.com/scrapinghub/dateparser/issues/1013

This can be closed now with the merge of #1062

authors: gutsytechster ian-barnes the1gofer

repo: scrapinghub/dateparser


text:
Improvements in locale:translate_search fixes

This PR fixes #930
Adds support for last decade, next decade, etc in search_dates
PL suggest if any change is required.
Thanks

Thanks, @noviluni, for the quick review, I have made the changes you suggested.


text:
ec pairing: panic inside final_exp

What command(s) is the bug in?
No response
Describe the bug
cmd
C="ecpairing_two_point_oog_d0_g0_v0"
FLAG=" --features=scroll"
export CIRCUIT=ecc
RUST_BACKTRACE=1 RUST_LOG=trace cargo run --release $FLAG -- --circuits basic --suite nightly --inspect "${C}"

output:
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `0`,
 right: `1`', /home/ubuntu/.cargo/registry/src/github.com-1ecc6299db9ec823/subtle-2.4.1/src/lib.rs:612:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/dfe3fe710181738a2cb3060c23ec5efb3c68ca09/library/std/src/panicking.rs:575:5
   1: core::panicking::panic_fmt
             at /rustc/dfe3fe710181738a2cb3060c23ec5efb3c68ca09/library/core/src/panicking.rs:64:14
   2: core::panicking::assert_failed_inner
   3: core::panicking::assert_failed
   4: halo2_ecc::fields::FieldChip::divide_unsafe
   5: halo2_ecc::bn254::final_exp::<impl halo2_ecc::fields::fp12::Fp12Chip<F,halo2_ecc::fields::fp::FpConfig<F,halo2curves::bn256::fq::Fq>,halo2curves::bn256::fq12::Fq12,9_i64>>::cyclotomic_decompress
   6: halo2_ecc::bn254::final_exp::<impl halo2_ecc::fields::fp12::Fp12Chip<F,halo2_ecc::fields::fp::FpConfig<F,halo2curves::bn256::fq::Fq>,halo2curves::bn256::fq12::Fq12,9_i64>>::cyclotomic_pow
   7: halo2_ecc::bn254::final_exp::<impl halo2_ecc::fields::fp12::Fp12Chip<F,halo2_ecc::fields::fp::FpConfig<F,halo2curves::bn256::fq::Fq>,halo2curves::bn256::fq12::Fq12,9_i64>>::final_exp

Concrete steps to reproduce the bug. If it's able reproduce via testool, please share test_id from jenkins report
No response

5 panics in total
[2023-09-02T04:35:40Z INFO  testool::statetest::results] 💀PANIC ecpairing_two_point_oog_d0_g0_v0 assertion failed: `(left == right)`
[2023-09-02T04:38:58Z INFO  testool::statetest::results] 💀PANIC ecpairing_two_point_oog_d0_g3_v0 assertion failed: `(left == right)`
[2023-09-02T06:11:03Z INFO  testool::statetest::results] 💀PANIC ecpairing_two_point_match_2_d0_g0_v0 assertion failed: `(left == right)`
[2023-09-02T06:14:27Z INFO  testool::statetest::results] 💀PANIC ecpairing_two_point_match_2_d0_g3_v0 assertion failed: `(left == right)`
[2023-09-02T06:17:48Z INFO  testool::statetest::results] 💀PANIC ecpairing_two_point_match_1_d0_g0_v0 assertion failed: `(left == right)`

Closed by the updated made in #889

authors: lispc roynalnaruto

repo: scroll-tech/zkevm-circuits


text:
Model and example in Swift for iOS development

I have modeled the sqlite to iOS Swift and made an example iOS app written in SwiftUI. Hope it will be helpful for the other iOS developers in the future.

I will look closer into this, thank you for your contribution. There is also a separate database set here https://github.com/scrollmapper/bible_databases_deuterocanonical
But that is less popular for obvious reasons.

authors: scrollmapper zjkuang


text:
Add pp/pl/tl into docs and init

closes #23 #29

@flying-sheep can we merge this?

Sure, looks fine!

repo: scverse/cookiecutter-scverse


text:
上滑加载更多的时候,滑动显示出了加载更多的布局,会直接触发父布局(viewpager)de 横向滑动的事件

加载更多的时候,滑动显示出了加载更多的布局,如果网络较慢,没有一下子加载出来数据,此时接着再滑动加载更多的数据就会直接触发父布局(viewpager)de 横向滑动的事件,重新SmartRefreshLayout的requestDisallowInterceptTouchEvent方法,请求父布局不要拦截事件,但是此方法已被源码屏蔽,无效,请问改如何解决?

mark，遇到同样问题

解决了吗

没有解决
------------------ 原始邮件 ------------------
发件人: "xlu"<notifications@github.com>;
发送时间: 2020年11月27日(星期五) 下午4:19
收件人: "scwang90/SmartRefreshLayout"<SmartRefreshLayout@noreply.github.com>;
抄送: "杨文华"<136013903@qq.com>; "Author"<author@noreply.github.com>;
主题: Re: [scwang90/SmartRefreshLayout] 上滑加载更多的时候,滑动显示出了加载更多的布局,会直接触发父布局(viewpager)de 横向滑动的事件 (#1258)
解决了吗
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

请问有方案了嘛


text:
嵌套StickyDecoration标题无法固定

依靠这几个貌似子组件中找不到view的id

app:srlFixedFooterViewId="@+id/header_fixed"
app:srlFixedHeaderViewId="@+id/footer_fixed"
app:srlHeaderTranslationViewId="@+id/header_translation"
app:srlFooterTranslationViewId="@+id/footer_translation"

不兼容 StickyDecoration，可以试试其他库


text:
[BUG] Input settings for NAM v0.7.0 on MacOS

Thanks for taking the time to develop this open software !!
I would like to report a bug found after installation and testing it...
Bug description
At least when using a Focusrite audio interface, the input used is always input 1 whatever settings for *Input 1(L) and Input 2(R).

To Reproduce
Steps to reproduce the behavior:

Go to 'Preferences'
Click on 'Input 1(L)'
Scroll down to '2' (i.e select input 2 of interface)
See error

Expected behavior
A clear and concise description of what you expected to happen.
Desktop (please complete the following information):

OS macOS 11.7.4
Plugin version 0.7.0
Standalone (other not tested yet)

** Thanks !!**

I have the same experience on a Behringer U-Phoria UMC404HD.
No matter what the Settings in NAM display it is the signal from the mic I have in channel 1 on the interface that is actually heard by NAM. All other guitar sim stand alone software works correctly.
iMac 2017 4.2 GHz Quad-Core Intel Core i7
MacOS 13.2.1

Appears to be a duplicate of #117. We'll continue tracking there.
Thanks!

authors: jonatanmv robiRagan sdatkinson

repo: sdatkinson/NeuralAmpModelerPlugin


text:
Moved username regex into defaults.yml to allow custom validation rules

The username field still requires 3 or more characters, but it's configurable now :smile:
Fixes #405

We also have regexes for mentions in 1 or 2 places, we'll need to fix those too.

@sibartlett I didn't find any other usage of that regex. Any hints where I should look?

authors: maximilian-krauss sibartlett

repo: sdelements/lets-chat


text:
poetry search doesn't exist  [CommandNotFound]   Command "search" is not defined.

When I run poetry search requests it returns this error:
[CommandNotFound]   Command "search" is not defined.
This command appears in the documentation and the README file, could you please tell me if this command will be part of poetry?

Yes. It was actually missing.
It will be available in the next release: 0.8.0

repo: sdispater/poetry


text:
PR 3: upgrade Gradle to 6.3, upgrade many dependencies, use Gradle Kotlin DSL instead of Groovy DSL

Includes commits from PRs 1 & 2.
Upgraded Gradle to 6.3.
Upgraded many dependencies.
Use Gradle Kotlin DSL instead of Groovy DSL.

[ ] a conversation was held in the appropriate SDKMAN Slack channel.
[ ] a Github Issue was opened for this feature / bug.
[ ] test coverage was added (Cucumber or Spock as appropriate).

This PR spans 43 files and few of these commits have anything to do with upgrading Gradle to 6.3. I would like to see only the files affected by upgrading Gradle in this PR.

The other commits should be in PRs 1 & 2.  The PRs are sequential; each should contain the commits of my previous PRs.  We should finish with PR 1 first, then PR 2, etc.  If extra commits are included in this PR other than commits from prior sequential PRs, and other than those specific to this PR, then I will need to compare the branches in github versus those in my local clone, as my local clone is correct.

There are conflicts now between my PRs & master, so I'll fetch, merge & rebase to fix them.
Once the conflicts have been resolved:

start with PR 1: https://github.com/sdkman/sdkman-cli/pull/741
then do PR 2: https://github.com/sdkman/sdkman-cli/pull/742
then do this PR
do all PRs sequentially

If you have any issues with any PR along the way, don't go on to the next one.  Let me fix up the problem PR, then, after you've merged that PR, go on to the next one.

Each PR should be in it's own feature branch, and should be taken directly off master. There should be only commits related to that feature in the branch, and hence in the PR.

I've rebased off current master, so you should be cleared to look at PR 1 #741.

I made some changes and added them as subsequent commits.

Went back to the binary distribution of Gradle as the -all incurs a huge download penalty.
I reformatted the build.gradle to have default formatting.
Went back to the shorthand notation of build dependencies. The group, name, version is a legacy notation and way too verbose.

repo: sdkman/sdkman-cli


text:
Remove stale / deprecated / HCL-only resources

This draft PR marks stale, outdated and deprecated resources, hoping to get a second opinion before removing important resources.
Resources marked with ❌ should probably be removed.
Resources marked with ❓ should be considered for removal.
I would love to get some input from one of the main collaborators, before moving on to deletion.
At first I considered whether they should be removed, or moved to a different location (or remain permanently marked as "v1"), but the big majority of the marked elements seem completely abandoned, or at least, without any attention from their authors. Since this is the "Awesome GitHub Actions" list and not the "All GitHub Actions" list - I considered abandoned/unmaintained repos "not awesome".
See the marked README.
cc: #158

Changes

Removed all HCL-only resources
Removed Internet of Things category, since it only had one deprecated resource
Removed mentions such as "supports the new YAML syntax"
Placed the two "GitHub Actions for Yarn" next to each other
Changed link to "Snyk Test Action" to the official one (https://github.com/clarkio/snyk-cli-action/issues/3#issuecomment-582050799)
Changed link for "Automatically add reviewers to pull request based on the configuration file" to go directly to its action repository

@peaceiris can I get your thoughts on this one? Thanks

I have checked all the changes of this Pull Request.
Yes! All marked link is for the GitHub Actions beta v1 (HCL syntax) not the v2 YAML.
I agree with this suggestion.
@DannyBen Great work!

I agree with this suggestion.

Great. So I will submit an updated file with all the marked items removed.

LGTM!

Ok - I undrafted it, it is ready for you now.
I added details in the description - all these were already done in the first commit, I just thought I should state them clearly for posterity.

@peaceiris thanks for taking a look at this.  I am more than happy to merge this based on your and @DannyBen feedback here.
Going forward, when looking at new additions to this list, what exactly should I, or others, be looking for, specifically?

@DannyBen huge thank you for taking this on, and getting it all updated!  Really appreciate it!

My pleasure.

Going forward, when looking at new additions to this list, what exactly should I, or others, be looking for, specifically?

Although I am not sure this question was directed at me - as it is more of a policy question - I would like to provide my thoughts here, since as a GitHub Actions user, I found quite a few questionable resources in this awesome list.

Resources that show the HCL syntax or show the old visual workflow editor should not be accepted.

The exception to this is established actions with a lot of stars (>50) that also mention the YAML syntax (and only have the HCL for backwards compatibility).

Actions that are developed by users (as opposed to official actions), should be more carefully examined.

If they have less than 10 stars, I would search to see if there is a better or official similar action.

In addition, I would consider:

Updating the collection to have a clear distinction between what is an action, and what is not.
Adding an "Official Vendor Actions" section - which will hold actions from companies that provide actions to their own products (snyk, aws for example).

authors: DannyBen gep13 peaceiris

repo: sdras/awesome-actions


text:
Easy way to determine what end of closed hallway robot is at

It would be nice if a robot is positioned at one end of a closed hallway that it would report which end.
You can figure this out nontrivially from the pose of course, but it would be nice if there was an easier way.

This should be doable since the 2 nodes added to the graph are exactly ordered as [start, end] -- are you just using the regular Python API?

Yes, I am using the regular python API.
I found that I could test for the hallway end using this:
        hallway_end: str | None = None
        if isinstance(robot.location, Hallway):
            # Add information about what end of hallway robot is at if still
            # in a room.
            hallway = robot.location
            pose = robot.get_pose()
            for room in [hallway.room_start, hallway.room_end]:
                if intersects_xy(room.buffered_polygon, pose.x, pose.y):
                    hallway_end = room.name
                    break

You could also do this without a polygon check, knowing that hallway.graph_nodes[0] and [1] correspond to the start and end room, respectively.
So something like:
hallway = robot.location
pose = robot.get_pose()
if pose.is_approx(hallway.graph_nodes[0]):
    hallway_end = hallway.room_start.name
elif pose.is_approx(hallway.graph_nodes[1]):
    hallway_end = hallway.room_end.name
else:
    print("uh oh")

Or if you want to be somewhat more robust, you could also check the distance between the robot pose and each graph node, e.g.,
dist_start = pose.get_linear_distance(self.graph_nodes[0])
dist_end = pose.get_linear_distance(self.graph_nodes[1])
hallway_end = room_start.name if (dist_start < dist_end) else room_end.name

authors: analog-cbarber sea-bass

repo: sea-bass/pyrobosim


text:
Add Operation Area

Lädt langsam. Keine Karte zum einzeichnen.

eingeloggt?

authors: JoshuaKrueger niczem

repo: sea-watch/sea-watch-app


text:
The Move Buttons Are Unable To Move To The Next Moderation Page

HI ! When i try to moderate this issue keeps happening again and again in different moderation panel with the extension. I am unable to switch between different moderation tabs it gets hanged or stuck. This keeps on happening every time at the end I have to refresh the page in order to be able to moderate again.It gets stuck and you are unable to do anything in the moderation tab.
Here is the detailed video describing the issue. Please have a look:
https://drive.google.com/file/d/1S3pMkn4QY3yEFfrWB90g2m0kNvrddhdo/preview

Got it!
Tools that enhances mod all page needs to be reconstructed. In the next release, the tools will be in BETA stage

Great ! Good job

I will close this issue because I have found the exact cause of this problem which is related to Brainly and it's already been reported to them.
For example, https://brainly.in/question/11362775 this question has a response that contains Latex contents init and this fails browser to parse the content. Therefore, moderation panel can't continue to moderate.
I can't fix this issue directly but I can probably find a workaround to close the failed moderation panel at least.

authors: ArcelBrainly seahindeniz


text:
refactor: use net.IP after dealing IP string in validation

Signed-off-by: Allen Sun shlallen1990@gmail.com

Describe what this PR does / why we need it

This pull request modify all IP string in the sealer's main procedure with the type of net.IP.

In my opinion, IP string is only used for end users to input. While after the validation layer of sealer, they all should be the type of net.IP. Communicating with net.IP is quite convenient for inner functions. We don't have to transfer them at all. And when reading code, hackers could clearly catch the meaning of net.IP type instance very soon.
What is more, when we use net.IP to pass though all code, it would bring much convenience for us to support IPv6. And this is one part of work of @VinceCui
Does this pull request fix one issue?
none

Describe how you did it
none
Describe how to verify it
none
Special notes for reviews
@kakaZhou719 This updated lots of code, please help to test this for the pull request with /test apply.

Codecov Report

Merging #1566 (2f7e0dc) into main (e0dc04c) will increase coverage by 0.04%.
The diff coverage is 6.89%.

:exclamation: Current head 2f7e0dc differs from pull request most recent head 65838d5. Consider uploading reports for the commit 65838d5 to get more accurate results

@@            Coverage Diff             @@
##             main    #1566      +/-   ##
==========================================
+ Coverage   10.84%   10.88%   +0.04%
==========================================
  Files          88       88
  Lines        8033     8058      +25
==========================================
+ Hits          871      877       +6
- Misses       7055     7070      +15
- Partials      107      111       +4

Impacted Files
Coverage Δ

apply/run.go
0.00% <0.00%> (ø)

pkg/cert/cert_cmd.go
0.00% <0.00%> (ø)

pkg/ipvs/ipvs.go
0.00% <0.00%> (ø)

pkg/plugin/etcd_backup_plugin.go
4.54% <0.00%> (ø)

pkg/plugin/labels.go
12.85% <0.00%> (ø)

pkg/plugin/plugins.go
0.00% <0.00%> (ø)

pkg/plugin/shell_plugin.go
23.07% <0.00%> (ø)

pkg/plugin/utils.go
0.00% <0.00%> (ø)

pkg/runtime/init.go
0.00% <0.00%> (ø)

pkg/runtime/kubeadm.go
0.00% <0.00%> (ø)

... and 28 more

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 e0dc04c...65838d5. Read the comment docs.

@bxy4543 Thank you for your review.

/test apply

repo: sealerio/sealer


text:
Add docstrings for intellisense for all non-generated public methods and classes

https://github.com/seamapi/python/pull/135
https://github.com/seamapi/python/pull/134
https://github.com/seamapi/python/pull/115
https://github.com/seamapi/python/pull/113

https://github.com/seamapi/python/pull/135

Current plan is to add some docstrings, see how they work with editor support, and then refine and copy edit them with @DebbieAtSeam's support.

All publicly exposed methods and classes are now documented with docstrings.
Here are a couple examples of how that looks like in the editor:

The only class without docstrings is SeamWebhookVerificationError. It's an alias of WebhookVerificationError imported from svix, which itself doesn't appear to have docstrings.

Nice. Next steps

@DebbieAtSeam should review this and copy edit it to a uniform style if needed.
Should we align on Sphinx style or "Google style". Looks like Sphinx format is not yet supported in VS code:
https://stackoverflow.com/questions/76855616/how-can-i-get-hover-info-in-vs-code-for-google-sphinx-style-python-class-attribu
https://github.com/microsoft/pylance-release/issues/1576#issuecomment-1668718385

If we go with Sphinx it might be make for better doc gen, but if we go with Google Style RST or markdown it may work "out of the box" in VSCode, yet we lose control over the output format if we put this on docs.seam.co.

authors: andrii-balitskyi razor-x

repo: seamapi/python


text:
seamail: show red dot indicator for threads with unread messages on index

Can use flutter icon brightness_1 (https://docs.flutter.io/flutter/material/Icons/brightness_1-constant.html)

Totally.

Done.

authors: Hixie rlkammerer

repo: seamonkeysocial/cruisemonkey


text:
Bug: Non-file Submissions Appearing in Files Page

Summary:
Submissions that do not contain any file attachments are mistakenly being displayed in the Files page. This issue clutters the Files page with entries that are not relevant to file management. Ideally, only submissions with attached files should be listed on this page to maintain clarity and relevance.
Steps to Reproduce:

Navigate to the form submission page.
Submit a form without attaching any files.
Go to the Files page to view submissions.
Notice that the submission without any file attachments is also listed among the entries.

Expected Behavior:
Only submissions that include file attachments should be displayed on the Files page. Entries without any attached files should be excluded to keep the page focused on file management.
Actual Behavior:
All submissions, including those without file attachments, are being displayed on the Files page. This includes entries that have no relevance to file management, leading to unnecessary clutter on the page.
Possible Solution:
Implement a filter in the query or data retrieval logic that only fetches submissions with attached files for display on the Files page. This could involve checking for the presence of file-related metadata in the submission data and excluding entries that lack this information.
Additional Context:
This issue impacts the user experience by presenting irrelevant information on the Files page and complicates file management tasks. Addressing this bug will enhance the clarity and usability of the Files page, ensuring that users can focus on managing actual file submissions.

#17

repo: seangray-dev/formail-app


text:
Enable Shell Piping

Enable the ability to use the gsd command as the right hand side of a Unix pipe on the command line.
$ grep ^web servers.txt | gsd

Also, enable directing standard input into gsd utility.
$ gsd < servers.txt

repo: seanmalloy/get-server-data


text:
feat({Request,Response}): derive Debug and PartialEq; feat(benches): set number of bytes in all benches

Please let me know if you have any feedback on the changes!

Looks good, thanks!

authors: seanmonstar utkarshkukreti

repo: seanmonstar/httparse


text:
[Python API] Logging level control does not work

According to the Python doc on logging, it should be possible to adjust the logging level:
import logging

logging.basicConfig(level=logging.WARNING)
logging.getLogger('nmslib').setLevel(logging.WARNING)

import nmslib

nmslib.init(method='hnsw', space='cosinesimil')

Seems like it does not work.
Outputs:
017-12-01 07:10:52 spacefactory.h:44 (Register) [INFO] Registering at the factory, space: dummy distance type: INT
2017-12-01 07:10:52 spacefactory.h:44 (Register) [INFO] Registering at the factory, space: dummy distance type: FLOAT
2017-12-01 07:10:52 spacefactory.h:44 (Register) [INFO] Registering at the factory, space: dummy distance type: DOUBLE
2017-12-01 07:10:52 spacefactory.h:44 (Register) [INFO] Registering at the factory, space: bit_hamming distance type: INT
2017-12-01 07:10:52 spacefactory.h:44 (Register) [INFO] Registering at the factory, space: leven distance type: INT
...
2017-12-01 07:10:52 methodfactory.h:49 (Register) [INFO] Registering at the factory, method: nonmetr_list_clust distance type: INT
2017-12-01 07:10:52 methodfactory.h:49 (Register) [INFO] Registering at the factory, method: simple_invindx distance type: FLOAT
2017-12-01 07:10:52 methodfactory.h:49 (Register) [INFO] Registering at the factory, method: wand_invindx distance type: FLOAT
2017-12-01 07:10:52 methodfactory.h:49 (Register) [INFO] Registering at the factory, method: blkmax_invindx distance type: FLOAT

@yonromai sorry, it's only in the dev, I will propagate this into master shortly.

Great to hear, thanks for the prompt answer!

Any workaround for 1.6.3? I can't upgrade to 1.7 because of https://github.com/nmslib/nmslib/issues/352

authors: goldan searchivarius yonromai

repo: searchivarius/nmslib


text:
Instantiator Error Messages

The errors output here do not always reflect the true problem.
https://github.com/searchspring/snap/blob/v0.43.1/packages/snap-preact/src/Instantiators/RecommendationInstantiator.tsx#L260-L263
^ this block will log even if the profile does have a display template when the recommendation API returns a 404.
The logging should more accurately reflect the error situation.
Ex:

pr is waiting on err beacon refactor to be merged - https://github.com/searchspring/snap/pull/803

repo: searchspring/snap


text:
Cannot retrieve survey responses

We have a survey (https://www.seasketch.org/nwmed/surveys/318/0) in which the responses cannot be retried. The error is "Failed to render responses grid". See below. A fix is high priority as this survey will be used in very short order.

Fixed in cdfac7e

authors: underbluewaters wmcclin

repo: seasketch/next


text:
Using zookeeper as the configuration center, the startup report cannot read the configuration parameters.

2019-10-09 09:16:10.050 INFO [main-SendThread(127.0.0.1:2181)]org.apache.zookeeper.ClientCnxn.logStartConnect:1025 -Opening socket connection to server 127.0.0.1/127.0.0.1:2181. Will not attempt to authenticate using SASL (unknown error)
2019-10-09 09:16:10.052 INFO [main-SendThread(127.0.0.1:2181)]org.apache.zookeeper.ClientCnxn.primeConnection:879 -Socket connection established to 127.0.0.1/127.0.0.1:2181, initiating session
2019-10-09 09:16:10.096 INFO [main-SendThread(127.0.0.1:2181)]org.apache.zookeeper.ClientCnxn.onConnected:1299 -Session establishment complete on server 127.0.0.1/127.0.0.1:2181, sessionid = 0x16dae0c482c0003, negotiated timeout = 6000
2019-10-09 09:16:10.104 INFO [main-EventThread]org.I0Itec.zkclient.ZkClient.processStateChanged:713 -zookeeper state changed (SyncConnected)
2019-10-09 09:16:10.128 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig metrics.enabled is error or timeout,return defaultValue false
2019-10-09 09:16:10.132 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.mode is error or timeout,return defaultValue file
2019-10-09 09:16:10.158 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.boss-thread-prefix is error or timeout,return defaultValue null
2019-10-09 09:16:10.160 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.worker-thread-prefix is error or timeout,return defaultValue null
2019-10-09 09:16:10.161 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.share-boss-worker is error or timeout,return defaultValue false
2019-10-09 09:16:10.163 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.type is error or timeout,return defaultValue TCP
2019-10-09 09:16:10.165 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.worker-thread-size is error or timeout,return defaultValue null
2019-10-09 09:16:10.170 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.server is error or timeout,return defaultValue NIO
2019-10-09 09:16:10.176 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.heartbeat is error or timeout,return defaultValue false
2019-10-09 09:16:10.234 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.boss-thread-size is error or timeout,return defaultValue 1
2019-10-09 09:16:10.236 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.boss-thread-prefix is error or timeout,return defaultValue NettyBoss
2019-10-09 09:16:10.240 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.boss-thread-size is error or timeout,return defaultValue 1
2019-10-09 09:16:10.279 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig transport.thread-factory.worker-thread-prefix is error or timeout,return defaultValue NettyServerNIOWorker
2019-10-09 09:16:10.290 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.file.dir is error or timeout,return defaultValue sessionStore
2019-10-09 09:16:10.295 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.file.session.reload.read_size is error or timeout,return defaultValue 100
2019-10-09 09:16:10.303 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.file.file-write-buffer-cache-size is error or timeout,return defaultValue 16384
2019-10-09 09:16:10.305 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.file.flush-disk-mode is error or timeout,return defaultValue null
2019-10-09 09:16:10.309 INFO [main]io.seata.common.loader.EnhancedServiceLoader.loadFile:237 -load TransactionStoreManager[FILE] extension by class[io.seata.server.store.file.FileTransactionStoreManager]
2019-10-09 09:16:10.309 INFO [main]io.seata.common.loader.EnhancedServiceLoader.loadFile:237 -load SessionManager[FILE] extension by class[io.seata.server.session.file.FileBasedSessionManager]
2019-10-09 09:16:10.315 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.file.max-global-session-size is error or timeout,return defaultValue 512
2019-10-09 09:16:10.319 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig store.file.max-branch-session-size is error or timeout,return defaultValue 16384
2019-10-09 09:16:10.346 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig recovery.committing-retry-period is error or timeout,return defaultValue 1000
2019-10-09 09:16:10.348 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig recovery.asyn-committing-retry-period is error or timeout,return defaultValue 1000
2019-10-09 09:16:10.349 ERROR[main]io.seata.config.zk.ZookeeperConfiguration.getConfig:103 -getConfig recovery.rollbacking-retry-period is error or timeout,return defaultValue 1000
本地zk中配置了/config/下所需配置参数。seata版本为0.8.1，zk服务器版本为3.4.6

@luojiachengaccp You need to push the configuration to ZooKeeper before starting. Refer to the nacos-config.sh(py) push script in the nacos configuration center. zk does not provide script initialization configuration.So if you don't initialize the configuration, all configuration item values will be replaced with default values.

@luojiachengaccp You need to push the configuration to ZooKeeper before starting. Refer to the nacos-config.sh(py) push script in the nacos configuration center. zk does not provide script initialization configuration.So if you don't initialize the configuration, all configuration item values will be replaced with default values.

我已将所需的配置信息推送至zk中，但还是会报如issue的错误，经调试查验是zk读取配置的时候的一个序列号问题，seata服务器在读取zk的时候节点能够获取到就是读取不到值，不知下一版本是否有针对该问题的修复？

@luojiachengaccp You need to push the configuration to ZooKeeper before starting. Refer to the nacos-config.sh(py) push script in the nacos configuration center. zk does not provide script initialization configuration.So if you don't initialize the configuration, all configuration item values will be replaced with default values.

我已将所需的配置信息推送至zk中，但还是会报如issue的错误，经调试查验是zk读取配置的时候的一个序列化问题，seata服务器在读取zk的时候节点能够获取到就是读取不到值，不知下一版本是否有针对该问题的修复？

已改用nacos作为配置中心~

authors: luojiachengaccp slievrly

repo: seata/seata