[BUG] Visualize failing with newer package versions (perhaps due to GeometryBasics)

The following code is leading to an error:
using NamedGraphs.NamedGraphGenerators: named_comb_tree
using ITensorGLMakie

let
  g = named_comb_tree([3,3,3])
  @visualize g
end

This is with the following package versions:
GraphMakie v0.5.6
GeometryBasics v0.4.11
UnicodePlots v3.6.4
Entire package status report for reference:
[79e6a3ab] Adapt v4.0.4
[7d9fca2a] Arpack v0.5.4
[dac988ce] AutoVectors v1.0.1
[6e4b80f9] BenchmarkTools v1.5.0
⌅ [8e7c35d0] BlockArrays v0.16.43
[336ed68f] CSV v0.10.14
⌃ [13f3f980] CairoMakie v0.10.12
[d360d2e6] ChainRulesCore v1.24.0
[cdddcdb0] ChainRulesTestUtils v1.13.0
[17a596ad] ChebyshevApprox v0.4.1
[3da002f7] ColorTypes v0.11.5
[5ae59095] Colors v0.12.11
[861a8166] Combinatorics v1.0.2
[34da2185] Compat v4.15.0
[a93c6f00] DataFrames v1.6.1
[b5a273c3] DataGraphs v0.2.4
[864edb3b] DataStructures v0.18.20
[31a5f54b] Debugger v0.7.9
[85a47980] Dictionaries v0.4.2
[0c46a032] DifferentialEquations v7.13.0
[31c24e10] Distributions v0.25.109
[e30172f5] Documenter v1.4.1
[b1794770] EinExprs v0.6.6
[da5c29d0] EllipsisNotation v1.8.0
[7a1cc6ca] FFTW v1.8.0
[5789e2e9] FileIO v1.16.3
[1a297f60] FillArrays v1.11.0
[3521c873] GenericTensorNetworks v2.0.1
[5c1252a2] GeometryBasics v0.4.11
[c27321d9] Glob v1.3.1
⌃ [1ecd5474] GraphMakie v0.5.6
[86223c79] Graphs v1.11.1
[06909019] GraphsFlows v0.1.1
[19dc6840] HCubature v1.6.0
[f67ccb44] HDF5 v0.17.2
[f0d1745a] HalfIntegers v1.6.0
⌃ [3f718f31] ITensorGLMakie v0.1.1
[2be41995] ITensorGaussianMPS v0.1.10
[f1f99f3b] ITensorMPOConstruction v0.1.0 ~/.julia/dev/ITensorMPOConstruction
[0d1a4710] ITensorMPS v0.2.4
[72ca75eb] ITensorMakie v0.1.3
[2919e153] ITensorNetworks v0.11.14 ~/.julia/dev/ITensorNetworks
[23d98011] ITensorTCI v0.0.1 ~/.julia/dev/ITensorTCI
[25707e16] ITensorTDVP v0.4.4
[73163f41] ITensorUnicodePlots v0.1.5
[cd2553d2] ITensorVisualizationBase v0.1.11
[9136182c] ITensors v0.6.11 ~/.julia/dev/ITensors
[4e3cecfd] ImageShow v0.3.8
[842dd82b] InlineStrings v1.4.0
[c8e1da08] IterTools v1.10.0
[033835bb] JLD2 v0.4.48
[98e50ef6] JuliaFormatter v1.0.56
⌃ [0b1a1467] KrylovKit v0.6.1
[2b0e0bc5] LanguageServer v4.5.1
[f4a74d36] LayeredLayouts v0.2.10
[16fef848] LiveServer v1.3.1
[eb30cadb] MLDatasets v0.7.14
[1914dd2f] MacroTools v0.5.13
⌅ [ee78f7c6] Makie v0.19.12
[c03570c3] Memoize v0.4.4
[d8a4904e] MutableArithmetics v1.4.4
[23ae76d9] NDTensors v0.3.27 ~/.julia/dev/ITensors/NDTensors
[678767b0] NamedGraphs v0.6.0 ~/.julia/dev/NamedGraphs
[46757867] NetworkLayout v0.4.6
[6f22d1fd] OMEinsumContractionOrders v0.8.3
[338f10d5] Observers v0.2.5
[6fd5a793] Octavian v0.3.27
[77e91f04] OptimKit v0.3.1
⌃ [1dea7af3] OrdinaryDiffEq v6.70.1
[14b8a8f1] PkgTemplates v0.7.50
[1fd47b50] QuadGK v2.9.4
[634c7f73] QuanticsGrids v0.3.2
[b11687fd] QuanticsTCI v0.6.1
[189a3867] Reexport v1.2.2
[295af30f] Revise v3.5.14
[1bc83da4] SafeTestsets v0.1.0
[03a91e81] SplitApplyCombine v1.2.3
[90137ffa] StaticArrays v1.9.5
[5e0ebb24] Strided v2.0.4
[4db3bf67] StridedViews v0.2.2
[fd094767] Suppressor v0.2.7
[286e6d88] SymRCM v0.2.2
[b261b2ec] TensorCrossInterpolation v0.9.7
[c2297e78] TensorInference v0.4.2
[6aa20fa7] TensorOperations v4.1.1
[a759f4b9] TimerOutputs v0.5.24
[b8865327] UnicodePlots v3.6.4
[0625e100] ValSplit v0.1.1
[44d3d7a6] Weave v0.10.12
[e88e6eb3] Zygote v0.6.70

I see what's going on. In order to visualize NamedGraph you need to load ITensorNetworks.jl as well.
There is a generic definition of visualize for AbstractGraph: https://github.com/ITensor/ITensorVisualizationBase.jl/blob/v0.1.11/src/visualize.jl#L106-L109 but it assumes the vertices are integers, so when it gets non-integer vertices it throws the error you are seeing deep down in the code.
This definition in ITensorNetworks.jl: https://github.com/ITensor/ITensorNetworks.jl/blob/v0.11.14/src/visualize.jl#L5-L19 adds support for non-integer vertices.
Ideally we would have a VisualizationCore.jl or VisualizationInterface.jl package split off from ITensor that defines the visualize/@visualize interface, and then we could add package extensions NamedGraphsVisualizationCore.jl, DataGraphsVisualizationCore.jl, ITensorsVisualizationCore.jl, ITensorMPSVisualizationCore.jl, ITensorNetworkVisualizationCore.jl, etc. Unfortunately for the time being without that the definitions for visualizing certain types are spread out a bit randomly.

Yes, I can confirm that adding using ITensorNetworks does fix the problem. I see, so there are sort of layers of features that are split across some packages. Not a problem for now, since this isn't so high-priority, but good to know what the issue is here and what a solution could be.


Scrolling bug in commands list

How to reproduce:

To make this bug more annoying, I tested in on mobile(the css is a clown fiesta) but the sticky div has the expected behavior

The imgur is no longer available, are you able to reproduce and reupload to nuuls? Tks


Prevent Villager Breeding

It would be great to have a switch that prevents villager breeding.

closing as its similar to #1845


Remove da.unify_chunks()

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes issue #184
Removes call to da.unify_Chunks() that does not appear to change anything in our case

[x] Tests for the changes have been added (for bug fixes / features)

Thanks @ellesmith88


Newsboat won't display non-ascii characters

Describe the bug
Newsboat doesn't display characters that are non-ascii. I'm trying out newsboat due to the Debian package being two years out-of-date.
Steps To Reproduce
I installed newsboat from the unstable branch using nix-env -iA nixpkgs.newsboat
Expected behavior
Nix's newsboat should look the same as Debian's.
Screenshots

Additional context
I'm using single-user mode because of #6966. I found #60845 but it doesn't seem relevant.
When I switch back to Debian's newsboat everything appears correctly. Is Nix's newsboat unable to access Debian's system fonts?
$ locale
LANG=en_US.UTF-8
LANGUAGE=
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE=C
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=
$ localectl status
   System Locale: LANG=en_US.UTF-8
       VC Keymap: n/a
      X11 Layout: us
       X11 Model: pc105

Notify maintainers

@dotlambda @nicknovitski
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.17.0-1-amd64, Debian GNU/Linux, noversion, nobuild`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.10.3`
 - channels(user): `"nixpkgs"`
 - nixpkgs: `/home/user/.nix-defexpr/channels/nixpkgs`

This seems to be related with Nix's 559 issue, since, by doing its workaround, the problem looks like it has gone away.
Workaround as provided in Stack Exchange:

nix-env -iA nixpkgs.glibcLocales
export LOCALE_ARCHIVE="$(nix-env --installed --no-name --out-path --query glibc-locales)/lib/locale/locale-archive"

Thanks @onemoresuza. the issue prompted me to move to elfeed but your link led me to https://nixos.wiki/wiki/Locales (your step 2) which worked without needing to install anything else.


FY2020Q1 quarterly refresh

Updated all packages versions
Updated exercise instructions
Updated slide decks

@jasonjoh Just added the example files to the demos with updated README instructions. I think this is what you were requesting. If not, can you let me know and I'll fix?


docs: adds sangeeth to organizers list

Congratulations on your first PR at VueJS-IN 🚀 @sangeeth96


SPIKE: Check if Startup Sample Project is aligned with LZ2 built-in Security Groups

Describe the issue
The Startup Sample Project was created prior to adopting GoC SEA Landing Zone (LZ2) and thus has Security Groups in it which may be redundant and not fully align with the LZ2 built-in security groups:

sample-load-balancer-security-group
sample-ecs-tasks-security-group
sample-endpoints-security-group

Which Sprint Priority is this issue related to?
Goal no. 2
Additional context

LZ2 leverages Shared VPC across the team accounts with predefined subnets, NACL's, and Security Groups
RCN code has already been refactored to align with LZ2 networking (VPC and Subnet)
recent app changes may have solved some of this

Definition of done
Startup Sample App uses LZ2 built-in Security Groups:

Web_sg
App_sg
Data_sg

The LZ sgs are directly attached to their associated subnets
The sample app uses the default alb that is created for the env as part of the account setup, this is in the web subnet/sg
ecs is created in the app subnet and has a custom sg that allows traffic from the web sg on the specific port used. This is stronger than the default sgs attached to the web and app subnets


some questions about the experimental results and threshold Selection.

Hello , thanks for sharing your excellent work!
I have some questions about the experimental results and threshold Selection.
On the SWAT dataset, I conducted experiments using the two threshold selection methods you provided. When I use the maximum error of the validation set as the threshold, F1 is 0.5. When I use the second method to search the optimal threshold on the test set, F1 is 0.80, which can achieve the effect of the paper report. Based on this, I have two questions:

Which threshold selection method is corresponding to the results reported in your paper? In your experiment, do the results of the two threshold selection methods differ greatly?

As for the second threshold selection method, I understand that it is to select the threshold that makes F1 the highest under the assumption that the test set anamoly label is known. But I have a question. The label of the test set is invisible in the real scene, so is this reasonable? I see that other works in recent years also adopts the optimal threshold method, so do we focus on the optimal F1 that can be achieved in theory?

I look forward to receiving your reply. Thank you very much!

I have the same Issue with the results. I can't come close to the results of the paper with the validation Threshold which is why I'm also curious about question one.

这是来自QQ邮箱的自动回复邮件。你好，您的邮件我已收到，祝您心情愉快。

Thanks for your interest in our work.

The reported results are based on the validation set threshold, such that it might be varied with different random seeds. Based on some seeds, the results of using validation-based and best F1 are very close, but for some seeds, there might be some variation.

Yes, there are some works using best-F1 as evaluation metrics. As F1 scores require threshold selection, I think a better way for evaluation could also consider using some threshold-agnostic metrics, e.g., AUROC, together with F1 related metrics.


Fix for the 10 sec delay when opening links with notif enabled on Win10

Fix for issue 58

Oh actually don't merge it yet, this brings back the tray icon not disappearing !

I will investigate this in the next day or two. Thanks for your trouble.


Fix redis store TTL

Hello Bryan,
On redis store example when you set custom TTL with set function.
It doesn't work because set function check for options.ttl key and it doesn't check if the options parameter is numeric.

Hi @homaily,
Thanks for the pull request.  However, I'm in the process of removing support for the numeric ttl so that the caching code is simpler and can be more easily applied to different cache backends.


Request does not wait for ReadableStream.pipeTo() to finish

We have a basic TransformStream defined like so:

Pretty please provide this directly in the sdk

export class TransformStream<R = any, T = any> {
  readonly writable: WritableStream<R>
  readonly readable: ReadableStream<T>
  private transform: (chunk: R) => T
  private readableController?: ReadableStreamController<T>

  constructor(props: { transform: (chunk: R) => T }) {
    this.transform = props.transform
    this.readable = new ReadableStream({
      start: (constroller) => {
        this.readableController = constroller
      }
    })
    this.writable = new WritableStream({
      write: (chunk: R) => this.readableController?.enqueue(this.transform(chunk)),
      close: () => this.readableController?.close()
    })
  }
}

We're then using this transform stream like so:
addEventListener('fetch', (event) => event.respondWith(handleRequest(event)))

async function handleRequest(event: FetchEvent) {
  const res = await fetch(...)
  const ts = new TransformStream(...)
  res.pipeTo(ts.writable)
  return new Response(ts.readable)
}

This usage of pipeTo is not waiting for the entire stream to be piped through the transform stream.
If I implement our own version of pipeTo it works correctly, but is slow:
addEventListener('fetch', (event) => event.respondWith(handleRequest(event)))

async function handleRequest(event: FetchEvent) {
  const res = await fetch(...)
  const ts = new TransformStream(...)
  _pipeTo(res, ts.writable)
  return new Response(ts.readable)
}

async function _pipeTo<T>(
  fromStream: ReadableStream<T>,
  toStream: WritableStream<T>
): Promise<void> {
  const reader = fromStream.getReader()
  // eslint-disable-next-line no-constant-condition
  while (true) {
    const res = await reader.read()
    if (res.done) {
      return
    }
    await toStream.getWriter().write(res.value)
  }
}

We now have a native implementation of TransformStream, so to some degree this is moot :) I'm not 100% sure why using the native pipeTo with your TransformStream polyfill didn't work, but given that nobody should be using a TransformStream polyfill anymore (because as you note it's much slower than the native one) I'll close this.

@tschneidereit Awesome! Will 23 be published to NPM today as well?

I don't think we'll have it out today, but it should be released this week. If you want to test beforehand, you could download the binary for your platform from the release here, and place it in node_modules/@fastly/js-compute/bin/[your platform].


Chip widget showing square corners after update

After updating Flutter to (v1.13.3-pre.60) on master channel, all of the Chip() widgets have square corners. Applying a RoundedRectangleBorder() to the shape property doesn't have any effect.
`[✓] Flutter (Channel master, v1.13.3-pre.60, on Mac OS X 10.15.2 19C57, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.41.0)
[✓] Connected device (3 available)
• No issues found!
Brians-MacBook-Pro-2:rydrapp briancurley$ flutter doctor -v
[✓] Flutter (Channel master, v1.13.3-pre.60, on Mac OS X 10.15.2 19C57, locale en-US)
• Flutter version 1.13.3-pre.60 at /Users/briancurley/Documents/Rydr/flutter
• Framework revision ceef460a64 (7 hours ago), 2019-12-18 00:23:02 -0800
• Engine revision 500f9cedf6
• Dart version 2.8.0 (build 2.8.0-dev.0.0 2d332ee1d3)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/briancurley/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3, Build version 11C29
• CocoaPods version 1.8.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 41.0.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.41.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.7.1
[✓] Connected device (3 available)
• iPhone 11 Pro Max • 1BD8D1EA-8724-4D64-86BC-738AD9791F6B • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)
• Chrome            • chrome                               • web-javascript • Google Chrome 79.0.3945.79
• Web Server        • web-server                           • web-javascript • Flutter Tools
• No issues found!`

It looks like this is a duplicate of https://github.com/flutter/flutter/issues/47350
CC @goderbauer

I rolled back the bad change.


ss

ss

Hi @maioneyza,
Just letting you know we are consolidating all open Sails issues into one repo.
~Cheers


Icon component

If anyone else was looking for a more reasonable way to use bootstrap4's icons, I tossed together this rough idea. explanation inside on how to populate the mapping object needed. Hope it saves someone else some of the frustration these icons come with..
usage:
https://gist.github.com/dstaley41/495dca87a04d168519a57c93a040c86e

Oh nice!  Thank you, this is a good addition to v5

Hey @dstaley41 , I've added an Icon component to simplify this further:
https://sveltestrap.js.org/?path=/story/components--icon
Thanks for the idea, let me know if you see any issues with it.


feat(jenkins.io): change uplink CNAME target from prodpublick8s to publick8s

To be merged after https://github.com/jenkins-infra/kubernetes-management/pull/4025/
Ref: https://github.com/jenkins-infra/helpdesk/issues/3351

Plan: 0 to add, 1 to change, 0 to destroy.


[bug] fail to run examples/file-associations

Describe the bug
Run the following inside examples/file-associations/src-tauri
error[E0599]: no method named asset_protocol_scope found for struct AppHandle in the current scope
--> examples/file-associations/src-tauri/src/main.rs:23:34
|
23 |   let asset_protocol_scope = app.asset_protocol_scope();
|                                  ^^^^^^^^^^^^^^^^^^^^ method not found in AppHandle
For more information about this error, try rustc --explain E0599.
warning: tauri-file-associations-demo (bin "tauri-file-associations-demo") generated 1 warning
error: could not compile tauri-file-associations-demo (bin "tauri-file-associations-demo") due to 1 previous error; 1 warning emitted
Reproduction
https://github.com/tauri-apps/tauri/tree/dev/examples/file-associations
Run the following inside examples/file-associations/src-tauri
cd tauri/examples/file-associations/src-tauri
cargo build
Expected behavior
Build suceeds
Full tauri info output
my other project

pnpm tauri info

><EMAIL_ADDRESS>tauri /Users/aya/Code/tauri/ddu
> tauri "info"

[✔] Environment
    - OS: Mac OS 12.7.6 x86_64 (X64)
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.81.0 (eeb90cda1 2024-09-04)
    ✔ cargo: 1.81.0 (2dbb1af80 2024-08-20)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-apple-darwin (default)
    - node: 20.11.0
    - pnpm: 9.0.1
    - yarn: 1.22.21
    - npm: 10.2.4
    - deno: deno 1.44.0

[-] Packages
    - tauri 🦀: 2.1.1
    - tauri-build 🦀: 2.0.3
    - wry 🦀: 0.47.0
    - tao 🦀: 0.30.8
    - @tauri-apps/api : 2.1.1
    - @tauri-apps/cli : 2.1.0

[-] Plugins
    - tauri-plugin-autostart 🦀: 2.0.1
    - @tauri-apps/plugin-autostart : 2.0.0
    - tauri-plugin-dialog 🦀: 2.0.3
    - @tauri-apps/plugin-dialog : 2.0.1
    - tauri-plugin-shell 🦀: 2.0.2
    - @tauri-apps/plugin-shell : 2.0.1
    - tauri-plugin-fs 🦀: 2.0.3
    - @tauri-apps/plugin-fs : 2.0.2

[-] App
    - build-type: bundle
    - CSP: connect-src ipc: http://ipc.localhost; font-src https://fonts.gstatic.com; img-src 'self' asset: http://asset.localhost blob: data:; style-src 'unsafe-inline' 'self' https://fonts.googleapis.com; default-src 'self' customprotocol: asset:
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: Vue.js
    - bundler: Vite

Stack trace
No response
Additional context
No response

The example's readme is wrong, i think cargo build --features protocol-asset should work


torch version erro

AttributeError: module 'torch.nn.functional' has no attribute 'interpolate'

pytorch version ==0.4.0

@allendred
Maybe you should update pytorch version to 0.4.1


Unable to check remaining uses without triggering an increment

For longer term rate limits, it's useful to show the end user a count of how many uses they have left within a rate limit window. Currently, there doesn't appear to be any way to access the key consistently without also triggering an increment on the number of uses. The key appears to be appended with a variable number value that gets in the way of doing a standard redis GET query.
Is there some way to consistently do a GET call on the key without implementing some further custom logic? If not, this would be nice to have.

+1 on this request - having a limiter.checkLimit(key); would be really helpful - otherwise the only way for users to know how many requests they have left is to actually use a credit

Also wondering if there is a timeline for this - I know you mentioned it was backlog so just curious if it's coming anytime soon!

Damn, I didn't see your responses, sorry about that
@AtotheY @jamesward1
I'm working on some improvements and will ship this as well: tomorrow or next week.

Hi @chronark just wondering if this was still in the backlog. I am trying to look into how the increment events are being counted on redis but I can't seem to figure out how I would implement such a checkLimit() feature. 🙏

Ah not really, unfortunately @chronark. What I'd need is a way to check for a given identifier what is the remaining limit for the current window (fixed in my case). The same way remaining is returned by the .limit() method.
This is useful so that the user can see their remaining limit without having to actually make another request. I am using this rate limiting for an expensive OpenAI API call and I'm only allowing X requests per day.
The node-rate-limiter has a getRemainingTokens() method that does exactly what I need, but since I am using a serverless function, that package is not an option 😢

I see
the tricky thing is that it's different for each algorithm, which one are you using?

I am using the fixed window. I assume it is difficult because neither the identifier string entry nor the event hash in redis actually tells you how many tokens were consumed at what time?

ah ok that one is fairly trivial
const intervalDuration = 1000 // 1s
const prefix = "@upstash/ratelimit"
const key = [prefix, identifier, Math.floor(Date.now() / intervalDuration)].join(":")

const remaining = await redis.get<number>(key)

Thanks, that is good to hear. I did try this but have a mismatch between what .limit() returns and what this calculation returns. Do you think it has something to do with the fact that I am testing this in the middle of an ongoing window and by tomorrow these two values will sync?

Ah sorry, yes you are correct.
It returns the used amount in the current window

Very cool, I think this would be nice to be documented. Shall I make a PR?

that'd be great, thanks

Revisiting this because I'm not 100% sure if this remaining token calculation for the fixed window really works. I checked the .limit() method and as far as I understand, it increments the redis key first before checking whether to set success to true: https://github.com/upstash/ratelimit/blob/c509628a6c1490121d810822ce7c880ddb02c44c/packages/sdk/src/single.ts#L158-L160
This means that when I call .limit() and check the returned remaining number it can also return a negative remaining value. For some reason, this doesn't happen every time, though.

yeah that looks like an oversight
it can return negative, but the check if it may pass also accounts for that


[Bug]: When all recognition rules are selected, the disabled rules will be recognized together

ODC version
odc422
OB version
not have
What happened?

What did you expect to happen?
When selecting all, there will be no disabled recognition rules
How can we reproduce it (as minimally and precisely as possible)?
Identify Rule Selection All
Anything else we need to know?
No response
Cloud
No response

pass


Force login using SSO

Is your feature request related to a problem?
I use the OIDC provider to provide authentication services for the memo, but every time I log in, I will still be redirected to the login page of the memos itself.
Describe the solution you'd like

Provide a switch in the settings to specify an SSO provider as the default login provider, and directly initiate OIDC authentication instead of popping up the built-in login interface when login is required.
Provide a url parameter sso for /auth. When its value is false, it is allowed to use the default interface to log in (used for troubleshooting when the third-party provider fails to log in due to an exception).

Additional context
Currently, I complete the jump by inserting a piece of js in the login interface, identifying the login interface with a frequency of 1 second, and forcibly triggering SSO login. Although this method fulfills my needs to a certain extent, the implementation method is very disgusting, and before the jump, the user can still see the built-in login interface for about 1-3 seconds.

I'd also +1 support for this as a built-in feature. As a workaround similar to the original poster, here is the script that I use in additional scripts to force an immediate redirect from initial load of any page to my SSO login. This works by making a call to the Memos API user endpoint to see if there's a user logged in, and if not then redirect.
(async function() {
  // Remember to replace this authentik style placeholder URL with your own.
  const LOGIN_URL = 'https://AUTHENTIK_BASE_URL/application/o/authorize/?client_id=MEMOS_CLIENT_ID&redirect_uri=https://MEMOS_BASE_URL/auth/callback&state=auth.signin.Authentik-1&response_type=code&scope=openid profile email'

  // Prevent a redirect loop by not checking login state from
  // login callback path.
  if (window.location.pathname !== "/auth/callback") {
    // Call Memos API to determine if the user is logged in.
    // If logged in, HTTP status is 200, if logged out 401.
    const status = (await fetch("/api/v1/user/me")).status;
    if (status === 401) {
      window.location = LOGIN_URL;
    }
  }
})();

I use Authentik as my SSO provider, so LOGIN_URL here will vary based on what your provider is. To get the exact URL, I recommend opening up an incognito window, opening developer tools, selecting the network tab and enabling "persist logs" from the settings menu, and then clicking the "Login with " link from Memos. The first URL to your auth provider in the network tab will be the one you want to use.

Disable user signup + Disable password login might help.

@K-J-VV I expect my workaround will work with any OIDC provider, although I can't speak directly to anything besides Authentik, since that's what I use. The protocol is a standard though, so they all operate about the same. The main difference should just be the login URL that you redirect to since it will differ between providers.

I'd also +1 support for this as a built-in feature. As a workaround similar to the original poster, here is the script that I use in additional scripts to force an immediate redirect from initial load of any page to my SSO login. This works by making a call to the Memos API user endpoint to see if there's a user logged in, and if not then redirect.
(async function() {
  // Remember to replace this authentik style placeholder URL with your own.
  const LOGIN_URL = 'https://AUTHENTIK_BASE_URL/application/o/authorize/?client_id=MEMOS_CLIENT_ID&redirect_uri=https://MEMOS_BASE_URL/auth/callback&state=auth.signin.Authentik-1&response_type=code&scope=openid profile email'

  // Prevent a redirect loop by not checking login state from
  // login callback path.
  if (window.location.pathname !== "/auth/callback") {
    // Call Memos API to determine if the user is logged in.
    // If logged in, HTTP status is 200, if logged out 401.
    const status = (await fetch("/api/v1/user/me")).status;
    if (status === 401) {
      window.location = LOGIN_URL;
    }
  }
})();

I use Authentik as my SSO provider, so LOGIN_URL here will vary based on what your provider is. To get the exact URL, I recommend opening up an incognito window, opening developer tools, selecting the network tab and enabling "persist logs" from the settings menu, and then clicking the "Login with " link from Memos. The first URL to your auth provider in the network tab will be the one you want to use.

I'm not sure if this thread is just too old at this point, or the API has changed, but this code does not work. Mind you, I do have things running through Cloudflare proxy, so not sure if that is complicating things. If anyone could offer any suggestions, that would be greatly appreciated.

It looks Memos API has changed since I wrote that script, (the test I was using fetch("/api/v1/user/me") no longer works since the internal API seems to have moved to use grpc.
It's not as pretty, but this works for me with the new version of the API:
(async function() {
  // Remember to replace this authentik style placeholder URL with your own.
  const LOGIN_URL = 'https://AUTHENTIK_BASE_URL/application/o/authorize/?client_id=MEMOS_CLIENT_ID&redirect_uri=https://MEMOS_BASE_URL/auth/callback&state=auth.signin.Authentik-1&response_type=code&scope=openid profile email'

  // Prevent a redirect loop by not checking login state from
  // login callback path.
  if (window.location.pathname !== "/auth/callback") {
    // Call Memos API to determine if the user is logged in.
    // If logged in, this returns data, if logged out it returns no data.
    const response = (await (await fetch("/memos.api.v1.UserService/GetUserSetting", {
      method: "POST",
      // This magic string is copied from a real request after logging in.
      body: "\u0000\u0000\u0000\u0000\u0000",
      headers: {"Content-Type": "application/grpc-web+proto"}
    })).text());
    if (response.length === 0) {
      window.location = LOGIN_URL;
    }
  }
})();

It looks Memos API has changed since I wrote that script, (the test I was using fetch("/api/v1/user/me") no longer works since the internal API seems to have moved to use grpc.
It's not as pretty, but this works for me with the new version of the API:
(async function() {
  // Remember to replace this authentik style placeholder URL with your own.
  const LOGIN_URL = 'https://AUTHENTIK_BASE_URL/application/o/authorize/?client_id=MEMOS_CLIENT_ID&redirect_uri=https://MEMOS_BASE_URL/auth/callback&state=auth.signin.Authentik-1&response_type=code&scope=openid profile email'

  // Prevent a redirect loop by not checking login state from
  // login callback path.
  if (window.location.pathname !== "/auth/callback") {
    // Call Memos API to determine if the user is logged in.
    // If logged in, this returns data, if logged out it returns no data.
    const response = (await (await fetch("/memos.api.v1.UserService/GetUserSetting", {
      method: "POST",
      // This magic string is copied from a real request after logging in.
      body: "\u0000\u0000\u0000\u0000\u0000",
      headers: {"Content-Type": "application/grpc-web+proto"}
    })).text());
    if (response.length === 0) {
      window.location = LOGIN_URL;
    }
  }
})();

Works perfectly - thank you so much!

It's not as pretty, but this works for me with the new version of the API:

Amazing work - and works like a charm. I couldn't elevate my Authentik account to Admin within Memos. I still have the credentials to login as original Admin...but to do that, I need to be able to login without SSO. Any workaround to disable the script?


Feat/check executable

Adds a check to see if a guide is said to be executable (for the Playground) in FrontMatter but defines no App\Playground\request function

Closing as we are not going this way


Is there a way to prevent jittering during video pose estimation using DWPose?

Is there a way to prevent jittering during video pose estimation using DWPose?
Frequent jittering occurs even when standing still.

Yes, I meet this problem, too. How ho make it smooth, are there any post-processing algrithms?

Hi @MoonEese @TZYSJTU ,
Please refer to this repo: https://github.com/cure-lab/SmoothNet. This is a plug-and-play smooth model in the post-process stage. You can simply input the 2d pose sequences, and output the smooth sequences.
BTW, if you do not need to use a network, you can try simple filters: https://docs.scipy.org/doc/scipy/reference/generated/scipy.signal.savgol_filter.html.
There are two parameters: window_length (you can try 7), polyorder (you can try 2).

@TZYSJTU
I have used simple average filtering, and it was little bit simple and effective for me
@ailingzengzzz
Thanks for great work and help!

@TZYSJTU I have used simple average filtering, and it was little bit simple and effective for me
@ailingzengzzz Thanks for great work and help!
Does simple average filtering not result in blurred video？

@learnuser1
I used each coordinate from n, n+1, n+2, n+3, n+4 frame for averages, and start/end 2frames were original coordinates.

@MoonEese Thank you for your answer.

@learnuser1 I used each coordinate from n, n+1, n+2, n+3, n+4 frame for averages, and start/end 2frames were original coordinates.

My result has a bad ghosting. Could you provide specific implementation?🙏
https://github.com/IDEA-Research/DWPose/assets/142297792/a2af9c0e-1d32-46b5-9114-75244a69b73c

@learnuser1 I used each coordinate from n, n+1, n+2, n+3, n+4 frame for averages, and start/end 2frames were original coordinates.

@My result has a bad ghosting. Could you provide the specific implementation?🙏

@ry555555
I can't view your result video, but I believe the term 'bad ghosting' arose because you averaged the frames of the DWPose result video itself. The method I mentioned earlier involves averaging the coordinate values extracted from DWPose, rather than directly averaging the result video frames. This means that the coordinate of, say, keypoint 1 in frame 1 is replaced by the average of keypoint 1's coordinates in frames 1, 2, 3, 4, and 5. While this approach reduces jitter, it may cause issues with actions such as waving hands back and forth in greeting.

@MoonEese Got it, Thx!


Method matcher does not work as expected

When trying to find methods, the method patterns are not returning the expected results.
For example, if I want to try and find the classpathFromResources method, I tried entering * for the fully qualified receiver type and .. as the argument types and I got 0 results when scanning the openrewrite organization.
I can manually find many instances of this method across many repositories - but the recipe can't. We should figure out why.
Related internal thread: https://moderneinc.slack.com/archives/C01S4M53THT/p1690917553347359

Jonathan mentioned that we should be using *..* for the first parameter. Sounds like this is perhaps a user-error rather than a bug? Closing - cc @traceyyoshima


Error : Hot reload

Hi,
I have this bug when i reload page with 'hot'  (npm run hot)
Thanks

The project has been update to use Vite and hot reloading works now.


[AutoPR containerinstance/resource-manager] Container Instance: Adding September Update

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3801

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3055


When docker isnt present

Would be nice if one could tell the subclass of GenericContainer that one is assume-ing that docker is present. So that if someone that doesn't have docker, get ignored tests instead of failing tests.
Due to the fact
DockerClient is instantiated during  of GenericContainer. One has no ability to use the jUnit way of conditionally ignoring tests with assumThat and AssumptionExceptionViolation.
Instead one gets:
java.lang.ExceptionInInitializerError
	at sun.misc.Unsafe.ensureClassInitialized(Native Method)
	at sun.reflect.UnsafeFieldAccessorFactory.newFieldAccessor(UnsafeFieldAccessorFactory.java:43)
	at sun.reflect.ReflectionFactory.newFieldAccessor(ReflectionFactory.java:142)
	at java.lang.reflect.Field.acquireFieldAccessor(Field.java:1088)
	at java.lang.reflect.Field.getFieldAccessor(Field.java:1069)
	at java.lang.reflect.Field.get(Field.java:393)
	at org.junit.runners.model.FrameworkField.get(FrameworkField.java:73)
	at org.junit.runners.model.TestClass.getAnnotatedFieldValues(TestClass.java:230)
	at org.junit.runners.ParentRunner.classRules(ParentRunner.java:255)
	at org.junit.runners.ParentRunner.withClassRules(ParentRunner.java:244)
	at org.junit.runners.ParentRunner.classBlock(ParentRunner.java:194)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:362)

So create the DockerClient later should make it possible to use assumptions if one would like to.
Also noticed that there is a getName after setting name that causes issues with the creation of the object. One would say it's doing to much in its construction, there is a "start" phase.

Some discussion before I implement and PR?

IMO letting the test fail is the correct behavior.

If the test was actually to run docker I would agree.
But having issues with docker doesn't actually tell you anything about your actual test.
As an example. I'm using JDBCContainer to spin up a mysql db and then performing some operations.
I verify that said operations have resulted in a specific state.
Should I assume that the operations are failing which is the actual test?
jUnit Assume would mark them as ignored and as such would require everyone working with the project to have docker installed. The CI-env will take care of the docker stuff.

Thinking the same way: what if MySQL is not available for any reason ? Do you expect your tests to fail or to be ignored ?
I agree with @kiview that failure is the correct behavior.

Might have been unclear, since I don't want a breaking change I just want the ability to do something like:
@ClassRule
MysqlContainer mysqlContainer = new MysqlContainer().assumeDocker();

The whole idea behind assumptions is that tests should not fail just because you made an assumption. They will not pass, they will not fail.
And adding this to ones project just makes the assumption that people aren't on Windows and that everybody has docker installed.
Letting me as the user control this assumption would be a great thing IMO.

If the ability to assumeDocker or ability to create assumptions easy on your own is not wanted this can be closed.
I however strongly feel that failing a DAO test because of container failure or missing docker is wrong.
Since SUT(System/subject under test) is the DAO and not the testcontainers/docker.
Had it been a manual test, it would has been marked as Not run aka Ignored. Ignored is an unknown state in regards to the SUT. Fail tests where the SUT hasn't failed. Test results should report state of SUT and not the test and/or environment.

Let's wait for the opinion of @rnorth.

Thanks all, and I'm sorry I haven't replied on this thread until now.
It's an interesting suggestion; I've used Assume once or twice but not come across use in the wild much, so you're the first person to ask for this. I can see how you'd find this useful, though. I can also see how Testcontainers has unique information about the Docker environment that is useful for such a Assumption.
It won't be something that everyone wants to use, so I'd like to find a way to do it that's not too far-reaching. How about this:
Could we perhaps proceed to do this separately from the container classes? E.g. Create something to allow either assumeTrue(DockerClientFactory.isDockerAvailable()), or DockerAvailableAssumption.assumeDockerAvailable()?
Another reason for keeping it off the container classes is that I'd like to separate out any JUnit coupling from the container classes, so adding an assume method right now would seem to be a step in the wrong direction.
Thoughts on the above approach?
Richard

@rnorth
1, Due to the creation of DockerClient during the construction of the class, it sort of falls on TestContainer to support this.
2, I've already implemented this it's with an added fluent .assumeDocker() so it's opt-in
3, The container classes already implements TestRule from jUnit, so it's already coupled.
Since assume is just more or less a bubbling exception i just added a new "phase" in FailureDetectingExternalResource which is assumptions which is called before start. I was thinking of just using a list of Functions so that assumptions could be added willy-nilly. Just to prevent the start from happening (avoid starting the container realize that the jdbc-driver is missing retry for 120s and so on and so on)

@rnorth @kiview @martin-g
SimpleMariaDBTest.java
Depends on what is considered "in the wild".
Should that assume be removed to get that failing test since it doesn't have a correct environment?

@npetzall Yes, I think this assumption should be removed. I see no reason this test to fail on Windows.
But I get your point!
I just don't agree that a test that uses TestContainers library should not fail when Docker is not available or fails to start for any reason. TestContainers library is about Docker! If Docker is not there then the whole test has no meaning.
To me it sounds like javac printing warnings while compiling the test in absence of JUnit in the classpath. It is an error!

@martin-g I don't care anymore.
But I agree with you "If Docker is not there then the whole test has no meaning" ergo Ignored, since Pass/Fail both have meaning.

@rnorth
You could do composition over inheritance and remove the implements TestRule and the starting(Description description). Create a new module junit and just have a ContainerRule which takes a supplier as it's constructor argument.

That sounds like a good idea and it's something we were already discussing, since we want to decouple TestContainers and JUnit (and there are some more parts in the codebase that would profit from using composition instead of inheritance). And once we've decoupled, I'd personally be fine to add more JUnit specific stuff to the JUnit part of the API.
AFAIK it's planned for version 2.0, but we might need to have a discussion regarding the final design first.

@kiview So really this can be implemented now, since it will later be refactored and moved to a junit module?

If schemaspy otherwise has to fallback to using the Jitpack build of it's own fork of TestContainers, it might be the more sensible approach to have some trade-off solution at this point in time. Depending on the current progress of the refactoring, I'd be fine with it, but I think @rnorth or @bsideup should decide, since they are more involved in the core.

The fork also solves #344, #345 so only doing this will not super-seed the fork.
However there is nothing stopping you from creating the junit module today I guess.
Which means that you currently can deprecate the direct usage of Container keep backwards compatability. And just incline user to switch to the junit module before 2.0.

And the reason might be, that it's currently working for all of us, so this seems like a bit of yak shaving? :wink:

Really nice attitude.
But then again why is a fork a problem. Isn't the whole point that if we have different opinions in regards to the code, one can make a fork and customize it to ones liking?

@npetzall all we're asking is not to add another JUnit-related method to GenericContainer.
As I said, we can see the value of what you're asking for, but please try and understand that for us this is something we have to maintain in the long term. If we add another JUnit-related feature to a core class then that just increases the likelihood of other users using it (not just yourself), and suffering when we move it (as we plan to do).
We're not ready to refactor JUnit out yet, so this is just the position we're in.
My suggested approach is this to have this in relevant tests for JUnit 4:
    @Rule
    public AssumptionRule assumptionRule = AssumptionRule.assumeDockerPresent();

I have a working implementation of this sitting right in front of me, and I think we can easily include it. If you don't want to use it, I can't stop you, but I would be grateful if you could adopt the approach that we're comfortable with supporting.

@rnorth I can't use it due to issue 344.
Also could you do the following test for me then.
Order of rules is undetermined according to junit javadoc for @Rule and @ClassRule Rule [ClassRule] (http://junit.org/junit4/javadoc/4.12/org/junit/ClassRule.html)
So order is somewhat unimportant.
  @ClassRule
  public static MysqlContainer mysqlContainer = new MysqlContainer<>();

  @ClassRule
  public static AssumptionRule assumptionRule = AssumptionRule.assumeDockerPresent();

Turn of docker so that It isn't present and run the test.
You might get some ignored test, but that's only because you get an error java.lang.ExceptionInInitializerError.
This is because construction of GenericContainer does to much. It fails when creating the MysqlContainer. The AssumptionRule might not even be evaluated.
As for the refactoring out. You can add @Deprecated to the junit stuff in core right now and create the junit module. There is no reason why they can not co-exist.

You should use RuleChain when you need to order your rules
On Jun 3, 2017 10:58 PM, "Nils Petzäll"<EMAIL_ADDRESS>wrote:

@rnorth https://github.com/rnorth I can't use it due to issue 344.
Also could you do the following test for me then.
Order of rules is undetermined according to junit javadoc for @Rule and
@ClassRule Rule http://junit.org/junit4/javadoc/4.12/org/junit/Rule.html
[ClassRule] (http://junit.org/junit4/javadoc/4.12/org/junit/ClassRule.html
)
So order is somewhat unimportant.
@ClassRule
public static MysqlContainer mysqlContainer = new MysqlContainer<>();
@ClassRule
public static AssumptionRule assumptionRule = AssumptionRule.assumeDockerPresent();
Turn of docker so that It isn't present and run the test.
You might get some ignored test, but that's only because you get an error
java.lang.ExceptionInInitializerError.
This is because construction of GenericContainer does to much. It fails
when creating the MysqlContainer. The AssumptionRule might not even be
evaluated.
As for the refactoring out. You can add @Deprecated to the junit stuff in
core right now and create the junit module. There is no reason why they can
not co-exist.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-java/issues/343#issuecomment-305997935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOKQl2IqwYtnGDbcy8u_gscLFvc7Pysks5sAbrlgaJpZM4NjOZ8
.

@martin-g that's correct it says so in the linked Javadoc. But it doesn't matter in this case since the test fails due to inability to instantiate the field that has the @ClassRule/@Rule annotation

@rnorth
I see what you did there, lazy init of DockerClient I like it, and the removal of getDockerName.

There may be another use case to consider as well: someone wants to (maybe just temporary) deactivate testcontainers in his build
see --> https://github.com/testcontainers/testcontainers-java/issues/2833#issue-630565097
and --> https://github.com/testcontainers/testcontainers-java/issues/2833#issuecomment-685992783

We (Eclipse Jetty) build and test on a huge variety of OS's, architectures, and hardware combinations.
Not all environments that we build/test on have docker, which is fine.
It's also not possible to have all of our possible configurations tested at the same time during a single test run.
Can we at least get a unique exception like org.testcontainers.dockerclient.DockerUnavailableException so that we can make the decision on pass/ignore/fail on our vast junit5 test suite easier?
Currently, we get an IllegalStateException, which is useless to work with in this situation.
java.lang.IllegalStateException: Could not find a valid Docker environment. Please see logs and check configuration
	at org.testcontainers.dockerclient.DockerClientProviderStrategy.lambda$getFirstValidStrategy$7(DockerClientProviderStrategy.java:215)
	at java.base/java.util.Optional.orElseThrow(Optional.java:408)
	at org.testcontainers.dockerclient.DockerClientProviderStrategy.getFirstValidStrategy(DockerClientProviderStrategy.java:207)
	at org.testcontainers.DockerClientFactory.getOrInitializeStrategy(DockerClientFactory.java:136)
	at org.testcontainers.DockerClientFactory.client(DockerClientFactory.java:178)
	at org.testcontainers.LazyDockerClient.getDockerClient(LazyDockerClient.java:14)
	at org.testcontainers.LazyDockerClient.authConfig(LazyDockerClient.java:12)
	at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:310)
	at org.eclipse.jetty.tests.distribution.HazelcastSessionDistributionTests.testHazelcastRemoteOnlyClient(HazelcastSessionDistributionTests.java:65)

@joakime that's an interesting idea. Also, have you considered failing fast by checking DockerClientFactory#isDockerAvailable?

@bsideup I can report that the following junit5 behavior works on the various architectures and OSs that have proven difficult. (think ARM64 and new/experimental/alternative OS's)
@Test
public void testSomethingThatNeedsDocker() {
    Assumptions.assumeTrue(DockerClientFactory.instance().isDockerAvailable(), "Docker is not available on this environment");
    // do something that needs docker here
}

It's a shame that the docker resolution occurs twice though.
Once for our direct usage of DockerClientFactory, and again for the deep usage of testcontainers.
However some of our usage of testcontainers is other libs that use testcontainers, so we don't have access to modify behavior within those libs (if the new DockerUnavailableException (extended from IllegalStateException?) existed then we can at least know why the testing lib had an issue in a programmatic way.


OTA on ESP32-IDF from AWS S3 bucket (IDFGH-4316)

Hi,
How do i do OTA on ESP32-IDF from AWS S3 bucket?
I tried simple_ota_example and modified the firmware URL in configuration as below
esp_http_client_config_t config = {
// .url = CONFIG_EXAMPLE_FIRMWARE_UPGRADE_URL,
.url = "https://my-url-link-ap-south-1.amazonaws.com/firmware.bin" ,
.cert_pem = (char *)server_cert_pem_start,
.event_handler = _http_event_handler,
};
but i get error as below:
E (5044) esp-tls: mbedtls_ssl_handshake returned -0x2700
I (5044) esp-tls: Failed to verify peer certificate!
I (5054) esp-tls: verification info:   ! The certificate is not correctly signed by the trusted CA
E (5064) esp-tls: Failed to open new connection
E (5064) TRANS_SSL: Failed to open a new connection
E (5074) HTTP_CLIENT: Connection failed, sock < 0
E (5074) esp_https_ota: Failed to open HTTP connection: ESP_ERR_HTTP_CONNECT
E (5084) esp_https_ota: Failed to establish HTTP connection
How to go about in solving the issue
Thanks

@ManjuMandya
Please see pointer below that should help:
https://github.com/espressif/esp-idf/issues/3021#issuecomment-460917118
Also for how-to questions please prefer using esp32.com forum

@mahavirj  Thanks for the link. I am able to connect to AWS after creating certificate but iam getting access denied response for AWS. It looks like i have to send access token, which iam not sure how to do it from ESP32-IDF.
Let me know if you have any idea.

@ManjuMandya
You may use pre-signed URL method as mentioned in my earlier linked comment, see here for more details.

@ManjuMandya Closing as there are no followup questions here. However, if you are still facing any issues from ESP-IDF side please feel free to re-open.

Thank you Mahavir. Our cloud team had not given read access, now we are able to download firmware from S3 bucket.


Android: Touch Exploration Mode state & Detect state change

Description
Android allows you to detect whether or not any accessibility service is running that uses touch exploration mode. This used to be more important, due to older versions of Talkback having this as an optional user setting, but since it is still supported in the API, theres no harm in adding a method to check it if desired.
This will generally be a better thing to check than isScreenReaderEnabled if you are only worried about whether or not you need to handle focus-navigation related differences, and not something specific to screen readers, such as audio feedback.
React Native version:
v0.63
Expected Behavior
The AccessibilityInfo API should have a method to detect if touch exploration mode is enabled, and a callback that can be triggered when its state changes.
Android Details
The AccessibilityManager class has methods to detect both whether touch exploration is enabled as well as a method to run a callback when its state changes.
https://developer.android.com/reference/android/view/accessibility/AccessibilityManager#isTouchExplorationEnabled()
https://developer.android.com/reference/android/view/accessibility/AccessibilityManager#addTouchExplorationStateChangeListener(android.view.accessibility.AccessibilityManager.TouchExplorationStateChangeListener, android.os.Handler)

Closed since this is no longer necessary now that we have the ability to detect enabled services.


Problem with "Dense layer" when used with Convnet

All my code is working probably, until it reach the dense layer
This error pop up
OverflowError: Range exceeds valid bounds

I am using Keras on top of Theano and anaconda with python 2.70 on windows 10_64X
same problem in  #2681 and #3781
My code :
# import dataset
from keras.datasets import mnist
# import the type of the modal
from keras.models import Sequential
# import layers
from keras.layers.core import Dense, Activation, Dropout,Flatten
#import convolution layers
from keras.layers.convolutional import Convolution2D, MaxPooling2D
from keras.utils import np_utils
#to plot matplotlib
import matplotlib
import matplotlib .pyplot as plt
#%%
#define prameters
#batch size to train
batch_size=128
#number of output classes
nb_classes =10
#number of epocs to train
nb_epoch=12

#input image dimension
img_rows,img_cols=28,28
#number of Convolution filters to use
nb_filters=32
#size of pooling area for max pooling
nb_pool=2
#Convolution kernel size
nb_conv=3
#%%
#the data shuffled and split between tran and test

#y_train & y_test labels from 0 to 9
#x_train & x_test features : pixles
(X_train, y_train), (X_test, y_test) = mnist.load_data()

#reshape the data
X_train=X_train.reshape(X_train.shape[0],1,img_rows,img_cols)# reshape for one channel
X_test=X_test.reshape(X_test.shape[0],1,img_rows,img_cols)

X_train=X_train.astype('float32')
X_test=X_test.astype('float32')

#normalization
X_train /=255
X_test/=255

print('X_train shape:',X_train.shape)
print(X_train.shape[0],'train samples')
print(X_test.shape[0],'test samples')

#convert class vectors to binary class matrices
Y_train=np_utils.to_categorical(y_train,nb_classes)# convert labels to binary format
Y_test=np_utils.to_categorical(y_test,nb_classes)

i=4600
plt.imshow(X_train[i,0],interpolation='nearest')
print("label:",Y_train[i,:])

#%%
#define model

model=Sequential()

#add layers
#first conv
model.add(Convolution2D(nb_filters,nb_conv,nb_conv,border_mode='valid',input_shape=(1,img_rows,img_cols)))
convout1=Activation('relu')
model.add(convout1)
#second conv
model.add(Convolution2D(nb_filters,nb_conv,nb_conv))
convout2=Activation('relu')
model.add(convout2)

model.add(MaxPooling2D(pool_size=(nb_pool,nb_pool)))
model.add(Dropout(0.25))#regulizatin layer
#full connected layer
model.add(Flatten())
model.add(Dense(128))
model.add(Activation('relu'))
model.add(Dropout(0.5))
model.add(Dense(nb_classes))
model.add(Activation('softmax'))
model.compile(loss='categorical_crossentropy',  optimizer='adadelta')#optmizer#

#%%

model.fit(X_train,Y_train,batch_size=batch_size,nb_epoch=nb_epoch,show_accuracy=True,verbose=1,validation_data=(X_test,Y_test))
model.fit(X_train,Y_train,batch_size=batch_size,nb_epoch=nb_epoch,show_accuracy=True,verbose=1,validation_split=0.2)

have you changed the .keras json file to reflect the Theano/'th' dimension ordering?

yes
I make this change in (( .keras json))
{
    "image_dim_ordering": "tf",
    "epsilon": 1e-07,
    "floatx": "float32",
    "backend": "tensorflow"
}

I am sorry this is copy error
yes, I am running theano. And I fix the problem you mentioned earlier but I still getting this
OverflowError: Range exceeds valid bounds

Can not repeat your error,
it can run well in my computer..
('X_train shape:', (60000, 1, 28, 28))
(60000, 'train samples')
(10000, 'test samples')
('label:', array([ 0.,  0.,  0.,  0.,  0.,  1.,  0.,  0.,  0.,  0.]))
/usr/local/lib/python2.7/dist-packages/Keras-1.0.8-py2.7.egg/keras/models.py:603: UserWarning: The "show_accuracy" argument is deprecated, instead you should pass the "accuracy" metric to the model at compile time:
model.compile(optimizer, loss, metrics=["accuracy"])
warnings.warn('The "show_accuracy" argument is deprecated, '
Train on 60000 samples, validate on 10000 samples
Epoch 1/12
60000/60000 [==============================] - 35s - loss: 0.3669 - val_loss: 0.0913
Epoch 2/12
60000/60000 [==============================] - 34s - loss: 0.1343 - val_loss: 0.0583
Epoch 3/12
20224/60000 [=========>....................] - ETA: 21s - loss: 0.1049

I am getting the same error , were you able to resolve this?

check the file "keras.json" in the following path C:~.keras
add the following code before the cnn model

from keras import backend as K
K.set_image_dim_ordering('th')

  Try this, this may work

@Jellyyoung  thanks a lot !!! my problem have been solved!

I am still getting the same problem:
OverflowError: Range exceeds valid bounds
I tried both what @Jellyyoung and what @varoudis suggested..


[Task] Add unit test cases for useStylesInternal hook

Task Description:
Currently, we run unit tests using jest which can be found here.
However, the test cases are not yet comprehensive enough and work can be done to cover more grounds. Specifically for this issue, we hope to add test cases for the useStylesInternal hook.
Deliverable(s):
New unit test cases that cover the testing of the useStylesInternal hook.
Additional Context:

Hooks API can be found here
Existing unit test for hooks can be found here which may serve as a good source for reference.

@tjtanjin I didn't find the useStylesInternal hook in the project

@tjtanjin I didn't find the useStylesInternal hook in the project

Hey @julianez, thank you for pointing this out. I'll replace this with the useBotIdInternal hook instead, since the styles hook doesn't actually have any functions at the moment. If you're keen to work on useBotIdInternal hook (or any other hooks) feel free to let me know 🙂

sure I can work on useBotIdInternal, please assign it to me :)


typo '-' => '->'

Coverage remained the same at 83.499% when pulling 8ba24b786bf99b92bb65727d0c252f430624cb92 on swuecho:patch-1 into 9d6770b949e93a65daa6e3454c46d8e5e2432b42 on mongodb:master.

Nice catch!  I've amended the commit message subject to match company standards and merged it as d5cddaf9dc5ed7203aca787729f0f18c5c58f860.  You'll be credited in the next release metadata.
Thank you very much!


WFLY-16819 Upgrade SmallRye Fault Tolerance to 6.0.0

Resolves
https://issues.redhat.com/browse/WFLY-16819

@kabir Haha, though, honestly, these are ideally best left to the component owners.

@rhusar Well, since these are best left to the component owner, can you fix the merge conflict? Otherwise I'd have done it via the github ui. ;-D

@bstansberry Haha, that's our daily routine anyway by now :-)

Thanks @rhusar


feat: automatic onion services

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Non-functional change (docs, style, minor refactor)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my code changes.
[x] All new and existing tests passed.

Do you have any idea why the commit linter fails?
Since changes cannot be uploaded without complying with this. It's not something that should fail

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

50 of 56 (89.29%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 86.336%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/app/app.ts
2
8
25.0%

Files with Coverage Reduction
New Missed Lines
%

src/adapters/web-socket-server-adapter.ts
2
78.57%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
884

Relevant Lines:
1008

💛  - Coveralls


Remove the -webkit-outer-spin-button CSS rule

Unfortunately simply using appearance: textfield;, or even -webkit-appearance: textfield;, doesn't actually hide the "spinner" in number-input elements in e.g. Google Chrome.
Hence we need to use a work-around with the -webkit-inner-spin-button rule, however in our CSS code we also have -webkit-outer-spin-button currently. According to both the MDN compatibility data and also manual testing in Google Chrome Beta 99, the -webkit-outer-spin-button rule is no longer necessary and we can thus clean-up the CSS a tiny bit.

/botio-linux preview

Thanks!


Request: limit the folders to find useless image attachments

Hi~ This plugin is very useful, but it would be nice if it could limit the scope of the search folder! This should be able to control the filtering results more precisely, especially for users with folders that have more nested structures.
Honestly, not all my notes need to be connected to each other, so global search will make the list very long, it would be easier to limit the folders to find useless image attachments that were added in the body of the MD note and then deleted. In this case, only the folder of the current note needs to be searched.
just a small request. I don't know if anyone has similar needs to me, If this demand is too niche, please do not care.

There is already an option to ignore specific directories, but I guess you want an option to only include specific directories?

Yes, it is, because I have too many folders that I want to ignore, and the structure of my folders keeps changing... So you know, it would be nice to search for unlink files in a specific directories.


Inject well-known/acme-challenge path at the top of rules

#587
The order is important here.

i am trying to issue cert using the HTTP-01 challenge, and I have created the cert but validation failed acme: Error 403 - urn:acme:error:unauthorized - Invalid response from http://example.com/.well-known/acme-challenge/ajaKN1Y6M3629Xr5EKHSJ4SgM1Ti89Gv2Owf_JI-5u8 (404 not found follows)
when looking at the configmap generated for haproxy, the frontend section is as such:
frontend http-0_0_0_0-80
  bind *:80
  mode http
  option httplog
  option forwardfor
  acl is_proxy_https hdr(X-Forwarded-Proto) https
  acl host_acl_example.com hdr(host) -i example.com
  acl host_acl_example.com hdr(host) -i example.com:80
  use_backend adminer.default:4180-jjppyc if host_acl_example.com
  acl host_acl_ww2.example.com hdr(host) -i ww2.example.com
  acl host_acl_ww2.example.com hdr(host) -i ww2.example.com:80
  use_backend arthur-oauth-protected.core:80-uzzpm5 if host_acl_ww2.example.com
  acl url_acl___.well-known_acme-challenge_ path_beg /.well-known/acme-challenge/
  use_backend voyager-common-ingress.default:56791-tekat2 if  url_acl___.well-known_acme-challenge_

so it seems that /.well-known/acme-challenge/ does not get matched early enough in the haproxy config.


docs clarification of mapper, reducer, combiner functions format

Discussion here: #1123
Not sure, that assertion in generator functions call (mapper, reducer and combiner) and every line check  is good.
So I have added hint to docs.
Maybe you have ideas how to implement assertion without overhead, I will add it later.

I don't think it's true that the reducer should yield a tuple.
(I don't know why this PR was left hanging unnoticed. Sorry!)

Closing due to inactivity. Feel free to reopen when you have time.


Throttle scroll and resize events

Hello,
I think events such as scroll and resize which can fire rapidly should use a (configurable) timeout. I created a gist here https://gist.github.com/gribelu/f9cb34d9a7610dd5fc11
Cheers

Hi, we launched AniJS Studio on KickStarter, that allow to designers and programmers add animations and interactions for Web pages in an easy and intuitive way. Please, take a look at the project and if you find it interesting or simply want to help us, you can make a small donation. I would be grateful, Greetings Dariel


heise.de/video links to cdnapisec.kaltura.com: URLError: <urlopen error Tunnel connection failed: 503 Connect failed>

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

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

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

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

youtube-dl -v https://www.heise.de/video/artikel/Kurz-informiert-vom-27-02-2018-IT-Verschluesselung-Netzneutralitaet-Selbstfahrende-Autos-Russisches-3981632.html?wt_mc=rss.video.beitrag.atom
[debug] System config: [u'--recode-video=mp4', u'-f', u'(bestvideo[width>=1920]/bestvideo)+bestaudio/best']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.heise.de/video/artikel/Kurz-informiert-vom-27-02-2018-IT-Verschluesselung-Netzneutralitaet-Selbstfahrende-Autos-Russisches-3981632.html?wt_mc=rss.video.beitrag.atom']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.03.10
[debug] Python version 2.7.10 (CPython) - Darwin-17.4.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.4.2, ffprobe 3.4.2, rtmpdump 2.4
[debug] Proxy map: {'http': 'http://<IP_ADDRESS>:8118', 'https': 'http://<IP_ADDRESS>:8118'}
[Heise] 3981632: Downloading webpage
[Kaltura] 1_kau1wws8: Downloading video info JSON
[Kaltura] 1_kau1wws8: Downloading m3u8 information
[debug] Invoking downloader on u'https://cdnapisec.kaltura.com/p/2238431/sp/223843100/playManifest/entryId/1_kau1wws8/format/url/protocol/https/flavorId/0_fnlkixyl?referrer=aHR0cHM6Ly93d3cuaGVpc2UuZGU='
ERROR: unable to download video data: <urlopen error Tunnel connection failed: 503 Connect failed>
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1896, in process_info
    success = dl(filename, info_dict)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1835, in dl
    return fd.download(name, info)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 364, in download
    return self.real_download(filename, info_dict)
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 340, in real_download
    establish_connection()
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 109, in establish_connection
    ctx.data = self.ydl.urlopen(request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2199, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 437, in open
    response = meth(req, response)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 550, in http_response
    'http', request, response, code, msg, hdrs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 469, in error
    result = self._call_chain(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 656, in http_error_302
    return self.parent.open(new, timeout=req.timeout)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 431, in open
    response = self._open(req, data)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 449, in _open
    '_open', req)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1101, in https_open
    req, **kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 1197, in do_open
    raise URLError(err)
URLError: <urlopen error Tunnel connection failed: 503 Connect failed>

> ping cdnapisec.kaltura.com
PING e4669.g.akamaiedge.net (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: icmp_seq=0 ttl=56 time=20.216 ms
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=56 time=23.900 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=56 time=21.079 ms
^C
--- e4669.g.akamaiedge.net ping statistics ---
3 packets transmitted, 3 packets received, 0.0% packet loss
round-trip min/avg/max/stddev = 20.216/21.732/23.900/1.573 ms

> curl -v https://cdnapisec.kaltura.com/p/2238431/sp/223843100/playManifest/entryId/1_kau1wws8/format/url/protocol/https/flavorId/0_fnlkixyl?referrer=aHR0cHM6Ly93d3cuaGVpc2UuZGU=
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to cdnapisec.kaltura.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/cert.pem
  CApath: none
* TLSv1.2 (OUT), TLS handshake, Client hello (1):
} [512 bytes data]
* TLSv1.2 (IN), TLS handshake, Server hello (2):
{ [108 bytes data]
* TLSv1.2 (IN), TLS handshake, Certificate (11):
{ [2890 bytes data]
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
{ [333 bytes data]
* TLSv1.2 (IN), TLS handshake, Server finished (14):
{ [4 bytes data]
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
} [70 bytes data]
* TLSv1.2 (OUT), TLS change cipher, Client hello (1):
} [1 bytes data]
* TLSv1.2 (OUT), TLS handshake, Finished (20):
} [16 bytes data]
* TLSv1.2 (IN), TLS change cipher, Client hello (1):
{ [1 bytes data]
* TLSv1.2 (IN), TLS handshake, Finished (20):
{ [16 bytes data]
* SSL connection using TLSv1.2 / ECDHE-RSA-AES256-GCM-SHA384
* ALPN, server accepted to use http/1.1
* Server certificate:
*  subject: C=US; ST=New York; L=New York; O=Kaltura Inc; OU=Operations; CN=*.kaltura.com
*  start date: Jan 25 00:00:00 2018 GMT
*  expire date: Apr 26 12:00:00 2019 GMT
*  subjectAltName: host "cdnapisec.kaltura.com" matched cert's "*.kaltura.com"
*  issuer: C=US; O=DigiCert Inc; CN=DigiCert SHA2 Secure Server CA
*  SSL certificate verify ok.
> GET /p/2238431/sp/223843100/playManifest/entryId/1_kau1wws8/format/url/protocol/https/flavorId/0_fnlkixyl?referrer=aHR0cHM6Ly93d3cuaGVpc2UuZGU= HTTP/1.1
> Host: cdnapisec.kaltura.com
> User-Agent: Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; Trident/5.0)
> Accept: */*
>
< HTTP/1.1 302 Moved Temporarily
< Server: Apache
< X-Me: ny-front-api38
< X-Kaltura: cached-dispatcher,playManifest-fb8eed9aa1159a9f1afd0142852e8e09,0.00060486793518066
< Location: https://cfvod.kaltura.com/scf/pd/p/2238431/sp/223843100/serveFlavor/entryId/1_kau1wws8/v/2/ev/5/flavorId/0_fnlkixyl/name/a.mp4?Policy=eyJTdGF0ZW1lbnQiOlt7IlJlc291cmNlIjoiaHR0cHM6Ly9jZnZvZC5rYWx0dXJhLmNvbS9zY2YvcGQvcC8yMjM4NDMxL3NwLzIyMzg0MzEwMC9zZXJ2ZUZsYXZvci9lbnRyeUlkLzFfa2F1MXd3czgvdi8yL2V2LzUvZmxhdm9ySWQvMF9mbmxraXh5bC9uYW1lLyoiLCJDb25kaXRpb24iOnsiRGF0ZUxlc3NUaGFuIjp7IkFXUzpFcG9jaFRpbWUiOjE1MjA3OTcyNjV9fX1dfQ__&Signature=OOXKroE3Vr1PIxHoK0vBkDQgqVHuuDkJzc6fE-WI5htx~hQcJhIGRQ7HJPnUbJVb8TdPNFFXnh0rO4oiJUsimKgnbZcuWBElTz~f3V~TQPAgtjmnSNGSIreI9UcK1mjv88lfiEy6W0ODSshEAHKi-~kMHIeaISKPjCwjvjwVvsdJiQhjcdtEwIwE0bkmeoMRUcq5A6049iClTTDdKOj616156PhV6mYZ0vqLdSvarVDloSDM0MsMpmTJBwp2OupLzZthm1qov7FFd~JCBwFsA7P1b5Dxz7y4g62ikrvNe4bZ5ZLLoZUsosb2cuyiPQGb4Z09ci~Bhzq8XAm-PVU~tg__&Key-Pair-Id=APKAJT6QIWSKVYK3V34A
< Access-Control-Allow-Origin: *
< Access-Control-Expose-Headers: Server,range,Content-Length,Content-Range
< Content-Encoding: gzip
< Content-Length: 21
< Content-Type: text/html
< Expires: Sat, 10 Mar 2018 19:41:05 GMT
< Cache-Control: max-age=0, no-cache, no-store
< Pragma: no-cache
< Date: Sat, 10 Mar 2018 19:41:05 GMT
< Connection: keep-alive
<
{ [21 bytes data]
######################################################################## 100.0%* Connection #0 to host cdnapisec.kaltura.com left intact

> cat ../.curlrc
progress-bar
show-error
#remote-time
ipv4
user-agent = "Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; Trident/5.0)"
remote-name-all

Most likely a proxy issue.


Add --ignore-overloaded-functions flag to ignore overload decorators

Hey, I just made a Pull Request!
Description
Add --ignore-overloaded-functions flag to ignore overload decorators
Motivation and Context
Fixes #97
Have you tested this? If so, how?
I have included unittests and ran interrogate against a codebase with overloads.
Checklist for PR author(s)

[x] Changes are covered by unit tests (no major decrease in code coverage %).
[x] All tests pass.
[x] Docstring coverage is 100% via tox -e docs or interrogate -c pyproject.toml (I mean, we should set a good example :smile:).
[x] Updates to documentation:

[x] Document any relevant additions/changes in README.rst.
[x] Manually update both the README.rst and docs/index.rst for any new/changed CLI flags.
[x] Any changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.  Find the appropriate next version in the project's __init__.py file.

Release note

Add ``--ignore-overloaded-functions`` flag to ignore overload decorators (`#97 <https://github.com/econchick/interrogate/issues/97>`_).

dang! thanks for the contribution @ErwinJunge ! I'm wrapping up a bunch of work stuff, but I'll review this over the next few days. Thanks!

Apparently I failed at running black :facepalm:
I've fixed the codestyle and forcepushed a new commit.

dang! thanks for the contribution @ErwinJunge ! I'm wrapping up a bunch of work stuff, but I'll review this over the next few days. Thanks!

For me this is work stuff :) I'm finishing up my week now and will pick this back up monday if there's any feedback.

@econchick friendly ping on the review :)

Hi @econchick , I'm sorry to hear about the covid and I hope you had a full recovery!
You're welcome for the PR. The test suite could indeed use some work, there were a lot of unrelated testing changes required to add my new thing. However, on the whole I'm mostly just happy there is a test suite! :smile:
Thank you for the review and the improvement suggestion. I've implemented it in the update I've just pushed.

uff - @ErwinJunge , looks like that small change broke some tests 😬 If you don't have time to fix them in the next couple of days, I can try to dig in. (This indeed makes me want to rewrite the test suite 🤦🏻)

Tests updated in latest push.
Re: pycon US, thanks for the offer but I don't think that'll be likely :) I live in the Netherlands, and it's currently a bit hard to travel out of Europe :sweat_smile:

@econchick those test failures look like the docstring coverage results are different on windows than they are on linux :confused: I don't have a windows box available to run those tests locally and can't imagine what would make the results different on windows so I don't think I'll be able to fix these.

@econchick do you have some input on the windows tests? Is there perhaps something I can do without having access to a windows box?

@econchick I took a closer look today and what's failing are the test expectation files in tests/functional/fixtures/windows/ I didn't update those (because I got all green on my machine :sweat_smile: ). However, I don't think I can update these, since I don't have access to a windows box to recreate them on. I tried copy-pasting the non-windows files and checking what changed using git diff, but that indicates the entire file is different (including stuff like the first line that definitely didn't change). There's some windows-specific magic going on that I can't replicate without access to a windows box. Can you regenerate those files and update the PR for me?
What I did for the linux files is this:

Add the below into a relevant test

    with open(expected_fixture, "w") as f:
        f.write(expected_out)

Sanity check the git diff. There might be added lines that shouldn't be there, remove those. Make sure there are only changed lines that make sense according to the python changes in the PR.
Once the diff is clean: commit :)

@econchick friendly ping on the above :) Please let me know if there's anything I can do to move this along.

:wave: @econchick please don't forget about this one :pleading_face:

:wave: @econchick please don't forget about this one :pleading_face:

Hi All,
I found this pull request as I the same problem with interrogate. My question would be if there is any chance that this pull request is merged at some point? I think this could be very useful feature.
Thanks and Best


Fix the container build which broke in #9d32811

On merging #1088, the subsequent container build step failed. On investigation, there were a few issues - wget is not actually installed on the base image, the package/repo references were the wrong way round, the package was a .tgz not a .deb, etc.
This PR fixes the build by:

replacing wget with curl
downloading the .deb package instead of the .tgz
adding the all-important './' to the apt-get install cmd

I've tested the image can now be built locally with docker build . (I should have tested the build on the branch locally before merging rather than leaving it to CI, my bad)
This application is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Thanks both! I've also tested that the new Dockerfile builds in the CI environment (see here & here) so I think we're good to merge this :+1:


fix for #320 #333 Node.js 0.5+ removes require.paths array

Although I did not add any new tests to prove this fix is complete, the current test suite is passing on the latest Node.js 0.5.5
Addresses issues #320 and #333

What's the status of this pull request?

gerardo:
I've been using my own fork for the time being. It's been in my development environment, as well as in production since this pull request was made, without a problem. So, for now:
https://github.com/kixxauth/less.js

Nice @kixxauth, thanks!

Tested this and it works.
Thanks!

I'm waiting for this to update my node to 0.6.0. Now I'm still in 0.4.12.
Is there anyway I can use npm to install https://github.com/kixxauth/less.js instead of the master one?
Thanks

you can use npm to install from a url - use its help to find out how.

Can't merge this anymore, as the commit wasn't isolated in its own branch. I will cherry-pick.


fix: show empty pickups in history

Codecov Report

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

@@            Coverage Diff             @@
##           master     #502      +/-   ##
==========================================
+ Coverage   98.48%   98.48%   +<.01%
==========================================
  Files         222      222
  Lines        3300     3304       +4
==========================================
+ Hits         3250     3254       +4
  Misses         50       50

Impacted Files
Coverage Δ

...ient/app/components/group/_history/history.spec.js
100% <100%> (ø)
:arrow_up:

...pp/components/group/_history/history.controller.js
94.82% <100%> (+0.18%)
: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 e886b86...6df65ed. Read the comment docs.


Update: Throttle and highlight clicks on metric/dimension selections

Description
Users find it hard to tell when they've added a metric or dimension in the new column config, so they end up adding the same metric/dimension multiple times.
Proposed Changes

Highlight a clicked metric/dimension so that users have some feedback that something has happened
Throttle clicks on metrics/dimensions to prevent unintentional additions/deletions

Screenshots
It doesn't show in the gif, but selected metrics/dimensions still have a light @denali-brand-100 background color.

License
I confirm that this contribution is made under an MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

probably should change this in metric config as well.. lost the cursor: pointer there and no animation

@daniellevinson Metric config updated and the datetime issue have been fixed. We don't need to throttle without request preview enabled.


fixing link

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingAds/campaign-management-service/campaign.md
:white_check_mark:Succeeded

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

@crojasaragonez thank you! The update should be live Monday.


Content Size  not changing after reload

First off, thanks for this awesome library. This is how the default layout should work!
Now, I'm experiencing the following issue:
When reloading data and displaying less cells than before, the content size does not change and I'm left with a way too high collection view.
The only solution I found does not work with this layout.
I tried to do things like:
collectionView.collectionViewLayout.invalidate()
collectionView.collectionViewLayout.prepare()
collectionView.reloadData()

...but without any luck.
Any ideas?

This was a mistake in my implementation, sorry!


Corrects the Device Tree Source (DTS) files of Arm MPS3-AN547

This PR corrects the base addresses and region names of some memories present in the MPS3-AN547 hardware.
Fixes #71920

@kristofer-jonsson-arm and/or @galak, would you mind giving this PR a quick review please?


Fix documentation formatting

panvimdoc's definition lists didn't seem to work out quite as well as I had hoped ):
skill issue; I was just using them wrong.
Also, I should move the documentation to the main README.md, as is conventional for other Neovim plugins.

Not really a bug anymore, but it does still need more love…

Also add usage examples to demonstrate the configuration


Remove PrivateAssets=All for Microsoft.Data.Sqlite.

Microsoft.Data.Sqlite.dll was not being copied to the output directory of generated projects referencing GxClasses package.
Issue:102677

Do not distribute all assemblies for Microsoft.Data.Sqlite. It should be conditioned based on the datastore type.


AsyncIterable support in from() in 6.x minor version?

I saw https://github.com/ReactiveX/rxjs/issues/1624 is already implemented in an experimental branch. Since it's a feature, could we release it in 6.x?

This is supported in 7.x now.