Build error on Windows w/ CPU & GPU

Today in the windows build:
[Stream 1 Build CPU] D:\jenkins_temp\ccOW8Ead.s: Assembler messages:
[Stream 1 Build CPU] D:\jenkins_temp\ccOW8Ead.s:1433182: Error: invalid register for .seh_savexmm
[Stream 1 Build CPU] make[2]: *** [blas/CMakeFiles/nd4jcpu.dir/build.make:66: blas/CMakeFiles/nd4jcpu.dir/cpu/NativeOps.cpp.obj] Error 1
[Stream 1 Build CPU] make[2]: Leaving directory '/d/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream1/libnd4j/blasbuild/cpu'
[Stream 1 Build CPU] make[1]: *** [CMakeFiles/Makefile2:89: blas/CMakeFiles/nd4jcpu.dir/all] Error 2
[Stream 1 Build CPU] make[1]: Leaving directory '/d/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream1/libnd4j/blasbuild/cpu'
[Stream 1 Build CPU] make: *** [Makefile:153: all] Error 2

[Stream 3 Build CUDA 8.0] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/include\ops/declarable/cpu/parity_ops.h(65) : warning C4365: 'argument' : conversion from 'unsigned __int64' to 'int', signed/unsigned mismatch

[Stream 3 Build CUDA 8.0] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/include\ops/declarable/cpu/parity_ops.h(104) : fatal error C1001: An internal error has occurred in the compiler.

[Stream 3 Build CUDA 8.0] (compiler file 'f:\dd\vctools\compiler\cxxfe\sl\p1\c\toil.c', line 9442)

[Stream 3 Build CUDA 8.0]  To work around this problem, try simplifying or changing the program near the locations listed above.

[Stream 3 Build CUDA 8.0] Please choose the Technical Support command on the Visual C++

[Stream 3 Build CUDA 8.0]  Help menu, or open the Technical Support help file for more information

[Stream 3 Build CUDA 8.0]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/include\ops/declarable/cpu/parity_ops.h(91) : while compiling class template member function 'Nd4jStatus nd4j::ops::matMul<float16>::validateAndExecute(nd4j::graph::Block<T> &)'

[Stream 3 Build CUDA 8.0]         with

[Stream 3 Build CUDA 8.0]         [

[Stream 3 Build CUDA 8.0]             T=float16

[Stream 3 Build CUDA 8.0]         ]

[Stream 3 Build CUDA 8.0]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/include\ops/declarable/declarable_ops.h(349) : see reference to class template instantiation 'nd4j::ops::matMul<float16>' being compiled

[Stream 3 Build CUDA 8.0]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/include\ops/declarable/declarable_ops.h(348) : while compiling class template member function 'nd4j::ops::__registratorHalf<nd4j::ops::matMul<float16>>::__registratorHalf(void)'

[Stream 3 Build CUDA 8.0]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/include\ops/declarable/cpu/parity_ops.h(91) : see reference to function template instantiation 'nd4j::ops::__registratorHalf<nd4j::ops::matMul<float16>>::__registratorHalf(void)' being compiled

[Stream 3 Build CUDA 8.0] -- Removing D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/blasbuild/cuda-8.0/blas/CMakeFiles/nd4jcuda.dir/cuda/./nd4jcuda_generated_NativeOps.cu.obj

[Stream 3 Build CUDA 8.0] C:/msys64/mingw64/bin/cmake.exe -E remove D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/blasbuild/cuda-8.0/blas/CMakeFiles/nd4jcuda.dir/cuda/./nd4jcuda_generated_NativeOps.cu.obj

[Stream 3 Build CUDA 8.0] CMake Error at nd4jcuda_generated_NativeOps.cu.obj.cmake:257 (message):
[Stream 3 Build CUDA 8.0]   Error generating file
[Stream 3 Build CUDA 8.0]   D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream3/libnd4j/blasbuild/cuda-8.0/blas/CMakeFiles/nd4jcuda.dir/cuda/./nd4jcuda_generated_NativeOps.cu.obj
[Stream 3 Build CUDA 8.0]
[Stream 3 Build CUDA 8.0]
[Stream 3 Build CUDA 8.0] NMAKE : fatal error U1077: 'C:\msys64\mingw64\bin\cmake.exe' : return code '0x1'
[Stream 3 Build CUDA 8.0] Stop.
[Stream 3 Build CUDA 8.0] NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\BIN\amd64\nmake.exe"' : return code '0x2'
[Stream 3 Build CUDA 8.0] Stop.
[Stream 3 Build CUDA 8.0] NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\BIN\amd64\nmake.exe"' : return code '0x2'
[Stream 3 Build CUDA 8.0] Stop.

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(23) : while compiling class template member function 'Nd4jStatus nd4j::ops::concat<float>::validateAndExecute(nd4j::graph::Block<T> &)'

[Stream 2 Build CUDA 7.5]         with

[Stream 2 Build CUDA 7.5]         [

[Stream 2 Build CUDA 7.5]             T=float

[Stream 2 Build CUDA 7.5]         ]

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/declarable_ops.h(341) : see reference to class template instantiation 'nd4j::ops::concat<float>' being compiled

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/declarable_ops.h(340) : while compiling class template member function 'nd4j::ops::__registratorFloat<nd4j::ops::concat<float>>::__registratorFloat(void)'

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(23) : see reference to function template instantiation 'nd4j::ops::__registratorFloat<nd4j::ops::concat<float>>::__registratorFloat(void)' being compiled

[Stream 2 Build CUDA 7.5] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(45) : warning C4388: '<' : signed/unsigned mismatch

[Stream 2 Build CUDA 7.5] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(46) : warning C4365: 'argument' : conversion from 'int' to 'unsigned __int64', signed/unsigned mismatch

[Stream 2 Build CUDA 7.5] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(65) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data

[Stream 2 Build CUDA 7.5] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(65) : warning C4365: 'argument' : conversion from 'unsigned __int64' to 'int', signed/unsigned mismatch

[Stream 2 Build CUDA 7.5] D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(104) : fatal error C1001: An internal error has occurred in the compiler.

[Stream 2 Build CUDA 7.5] (compiler file 'f:\dd\vctools\compiler\cxxfe\sl\p1\c\toil.c', line 9442)

[Stream 2 Build CUDA 7.5]  To work around this problem, try simplifying or changing the program near the locations listed above.

[Stream 2 Build CUDA 7.5] Please choose the Technical Support command on the Visual C++

[Stream 2 Build CUDA 7.5]  Help menu, or open the Technical Support help file for more information

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(91) : while compiling class template member function 'Nd4jStatus nd4j::ops::matMul<float16>::validateAndExecute(nd4j::graph::Block<T> &)'

[Stream 2 Build CUDA 7.5]         with

[Stream 2 Build CUDA 7.5]         [

[Stream 2 Build CUDA 7.5]             T=float16

[Stream 2 Build CUDA 7.5]         ]

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/declarable_ops.h(349) : see reference to class template instantiation 'nd4j::ops::matMul<float16>' being compiled

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/declarable_ops.h(348) : while compiling class template member function 'nd4j::ops::__registratorHalf<nd4j::ops::matMul<float16>>::__registratorHalf(void)'

[Stream 2 Build CUDA 7.5]         D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/include\ops/declarable/cpu/parity_ops.h(91) : see reference to function template instantiation 'nd4j::ops::__registratorHalf<nd4j::ops::matMul<float16>>::__registratorHalf(void)' being compiled

[Stream 2 Build CUDA 7.5] -- Removing D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/blasbuild/cuda-7.5/blas/CMakeFiles/nd4jcuda.dir/cuda/./nd4jcuda_generated_NativeOps.cu.obj

[Stream 2 Build CUDA 7.5] C:/msys64/mingw64/bin/cmake.exe -E remove D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/blasbuild/cuda-7.5/blas/CMakeFiles/nd4jcuda.dir/cuda/./nd4jcuda_generated_NativeOps.cu.obj

[Stream 2 Build CUDA 7.5] CMake Error at nd4jcuda_generated_NativeOps.cu.obj.cmake:257 (message):
[Stream 2 Build CUDA 7.5]   Error generating file
[Stream 2 Build CUDA 7.5]   D:/jenkins/workspace/dl4j/all-multiplatform_windows-x86_64/libnd4j/stream2/libnd4j/blasbuild/cuda-7.5/blas/CMakeFiles/nd4jcuda.dir/cuda/./nd4jcuda_generated_NativeOps.cu.obj
[Stream 2 Build CUDA 7.5]
[Stream 2 Build CUDA 7.5]
[Stream 2 Build CUDA 7.5] NMAKE : fatal error U1077: 'C:\msys64\mingw64\bin\cmake.exe' : return code '0x1'
[Stream 2 Build CUDA 7.5] Stop.
[Stream 2 Build CUDA 7.5] NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\BIN\amd64\nmake.exe"' : return code '0x2'
[Stream 2 Build CUDA 7.5] Stop.
[Stream 2 Build CUDA 7.5] NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\BIN\amd64\nmake.exe"' : return code '0x2'
[Stream 2 Build CUDA 7.5] Stop.

log (intervleaves CPU, & 2x GPU builds, grep the beginning of the line to see the issues): https://gist.github.com/huitseeker/0ce117fdfd940e0c3f02513d00a49e49
build: http://master-jenkins.skymind.io:8080/job/dl4j/job/all-multiplatform/2410/consoleFull

Just checked CPU - builds fine on gcc 7.2.
Similar issue was reported by @wmeddie earlier yesterday, and msys gcc update helped.
Still building CUDA though.

Hm, and CUDA is broken for me today :/

Both CPU & GPU fixed, compiles now. Thanks for highlighting this one.

Seems to be related to mysterious AVX instructions getting generated:
https://stackoverflow.com/questions/43152633/invalid-register-for-seh-savexmm-in-cygwin
I'm also getting that error with GCC 6.3.0, so I guess it's something they've fixed in GCC 7.2, but those AVX instructions shouldn't be there in the first place...

Yes, that's the case for CPU.
And for CUDA, surprisingly, issues was in bad handling of default parameters in 1 (one) method. Took some time to find out that...

Kind of strange that it's only newer code from libnd4j that generates AVX like that though. Doesn't occur with any of the libraries in the JavaCPP Presets.

No ideas. For me it's the same kind of question as "why gcc 6/7 on macos tries to use avx-512 instructions?"...


Add more data asset test files

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

coverage: 90.783%. remained the same
when pulling c3b655a0dce843fef002a4b2f5baf72c9edc395f on addMoreTestCasesForAssets
into d7a4f1916829ccdc93ad5e81f47e1893e3898ea0 on main.


readme.md use old syntax

Just jumped into using this for a project and was following the readme example at first.  It still uses hxl.hxl() to read data rather than hxl.data()
Took me a while to figure out.

Thanks -- will fix.


Fixing eslint warnings

Most of the warnings are trivial. I commented the ones I think are more complex.

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 57.391%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3269

Relevant Lines:
5696

💛  - Coveralls

I'm a little more worried about the Sonar security warnings though

Once #343 goes in, I'll have a look at those.


Orderbook follow up TBD

investigate what is happening in the cumulative volume column where there are buys and sells of the whole book
Investigate https://github.com/vegaprotocol/frontend-monorepo/issues/664
https://github.com/vegaprotocol/frontend-monorepo/issues/790
https://github.com/vegaprotocol/frontend-monorepo/issues/818
Consider whether the order books logic should be based whether market is in auction or not rather than just if the prices overlap. Show an error message if something is wrong

In discussion we concluded/discussed:
API bug

https://github.com/vegaprotocol/data-node/issues/851

issues to do with different data from different end points

glitchy api, best bid or best ask can go to 0 for a moment... so consider calculating the best bid/ask/mid based on the values we have from the depth subscription (or this has been fixed with recent fix) ask data node

Optimisations FE

discussed only loading in the data that is in the view port, but may also need to query the top/bottom cumulative volumes to be able to draw bar charts correctly (or just grab a lot of rows and use the extremes from the what has been fetched)

Optimisations API

aggregations on the back end? this is a big win for FE
Frequency of updates
e.g advanced agitation (buys separately from sells pagination)
calculate cumulative volume on the back end

Design actions

change the background of the space between best bid/ask so when the bid/ask are so far apart that you can't see them
consider something to show what is happening when there isn't a line between because the bid/ask are touching
what does the depth query do when it is in an auction (presumably it parks the orders)

CSS

consider absolute positioning instead of grid to improve consistency of row height


Added Hyprland to list of Wayland wms

Description
Added Hyprland to list of wayland wms in $wm variable.
Features
Issues
TODO

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!


Deploy the API documentation

The docs still live in the old GAE/python repo. Now that the subdomain has been switched, the docs are no longer accessible.

API Gateway isn't handling the HTML pages correctly.

resolved this with an API Gateway update. adding a GET method at the root path /
previously, the configuration presumed a path.


Apache Zeppelin-Highcharts displaying blank output

hi @rockie-yang,@knockdata ,@weand ,@honnix
please help us with the below issue-
I am  new to zeppelin and wanted to implement some drill down charts, for that
I configured HighCharts on our Zeppelin Tool to implement a drill down functionality. We placed the below jar files and stated the same in the Dependencies
Zeppelin-highcharts-0.6.4.jar
lift-json_2.11-2.6.3.jar
paranamer-2.4.1.jar
We tried executing the below code-
Para 1
val bank = spark.read.option("header",true).csv("/u01/zepplin/bank/bank2.csv")
bank.show()
Para 2-
import com.knockdata.spark.highcharts._
import com.knockdata.spark.highcharts.model._
highcharts(bank.series("x" -> "age", "y" -> sum(col("balance"))).orderBy(col("age"))).plot()
But this code is not able to plot anything, giving a blank out put.It just shows the tile(Chart Title). To rule out the browser compatibility issue I tried this on all the browsers too. Am I missing something here.Do we have any alternative for High-charts for custom visualization like drill down etc.
Getting output as shown below

Thanks in Advance
Teny

Please let know if anyone can help me to resolve this
Thanks


Verify "Largest Series Product" exercise implementation

There was some confusion in this exercise due to the ambiguous use of the term consecutive in the README. This could be taken to mean contiguous, as in consecutive by position, or as in consecutive numerically. The the README has been fixed (https://github.com/exercism/x-common/pull/200).
Please verify that the exercise is implemented in this track correctly (that it finds series of contiguous numbers, not series of numbers that follow each other consecutively).
If it helps, the canonical inputs/outputs for the exercise can be found here:
https://github.com/exercism/x-common/blob/master/largest-series-product.json
If everything is fine, go ahead and just close this issue. If there's something to be done, then please describe the steps needed in order to close the issue.

All the canonical cases from x-common pass; however, we might want to add a test that checks the error type when passing a negative span.  Currently, it fails with a FunctionClauseError inside of Enum.chunk, but it would probably be more consistent to make that an ArgumentError.

Thanks for checking @rubysolo!


Change the color of the skima for the preview section

@thomas-leguellec what is this ? Maybe we can close it ?


Add support for summation in new Decapode syntax

The following code should work to allow users to specify multi-variate summation.
using Test
using Catlab
using Decapodes
import Decapodes: DecaExpr

@present SchSummationDecapode <: SchNamedDecapode begin
  # Σ are the white nodes in the Decapode drawing
  # Summands are the edges that connect white nodes to variables (the projection maps)
  # because addition is commutative, we don't need to distinguish the order
  (Σ, Summand)::Ob
  summand::Hom(Summand, Var)
  summation::Hom(Summand, Σ)
  sum::Hom(Σ, Var)
end

@acset_type Decapode(SchSummationDecapode,
  index=[:src, :tgt, :res, :incl, :op1, :op2, :type]) <: AbstractDecapode

NavierStokes = quote
    V::Form1{X}
    V̇::Form1{X}
    G::Form1{X}
    T::Form0{X}
    ρ::Form0{X}
    ṗ::Form0{X}
    p::Form0{X}

    V̇ == neg₁(L₁′(V, V)) +
          div₁(kᵥ(Δ₁(V) + third(d₀(δ₁(V)))), avg₀₁(ρ)) +
          d₀(half(i₁′(V, V))) +
          neg₁(div₁(d₀(p),avg₀₁(ρ))) +
          G
    ∂ₜ(V) == V̇
    ṗ == neg₀(⋆₀⁻¹(L₀(V, ⋆₀(p))))# + ⋆₀⁻¹(dual_d₁(⋆₁(kᵨ(d₀(ρ)))))
    ∂ₜ(p) == ṗ
  end
parse_decapode(NavierStokes)

I accidentally pushed my implementation to main, which broke the old syntax. We still need to update the compile function to use the new summation tables rather than Op2.


feat: update onboarding steps

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description

Updates the onboarding steps to match the new design linked in the issue
Related Tickets & Documents

closes #801
Mobile & Desktop Screenshots/Recordings
Step 2:

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Need the images (with the active and normal variants) of the new steps for the left side:

Are they not exportable in Figma? "Export as SVG"

Are they not exportable in Figma? "Export as SVG"

Tried, but they weren't exported with the circle, or maybe I don't know how to properly export them

@getaheaddev perhaps you can share/upload step 2/3 active/inactive svgs first thing tomorrow.

I'm also facing issues with the backend.
1- with /auth/onboarding:

I keep getting server error 500 when trying to send a request to it. I tried with body of "" & and with body of {"id": []} (inspired by how it is called in frontend) and in both I get the same error.
Side note: I don't know how onboarding in api was made, but given that the frontend used to send repo ids to it, that may need changing (since onboarding no longer collects repo ids).

2- with /auth/profile:

When I try to send a request with name, email (sending them because they are required), and timezone, I get error 404 and message "user not found". It works if I omit the timezone.
Suggestion: Would it be possible to make the endpoint not require name & email? As in my use-case I only need to update the timezone of the user.

@Deadreyo is still blocked? I think @0-vortex shipped a fix.

The /auth/onboarding endpoint has been updated to accept the new onboarding fields
https://beta.api.opensauced.pizza/#/Authentication service/postOnboarding

The /auth/onboarding endpoint has been updated to accept the new onboarding fields

Cool. I get a message everyday from people confused by the onboarding.

The /auth/onboarding endpoint has been updated to accept the new onboarding fields
https://beta.api.opensauced.pizza/#/Authentication service/postOnboarding

The issue that I've reported to you still persists:
Error 400: { 	"message": [ 		"interests must be an array", 		"timezone must be a string" 	] }
on sending {"interests":["javascript","react"],"timezone":"Egypt Standard Time"}

Frontend-wise, this PR is ready for review. Backend-wise, onboarding won't work due to api issues but that can be fixed async. We can save time if we want this asap by reviewing it early (excluding the api thing).

Frontend-wise, this PR is ready for review. Backend-wise, onboarding won't work due to api issues but that can be fixed async. We can save time if we want this asap by reviewing it early (excluding the api thing).

Can you update description with screenshots of all steps?

Can you update description with screenshots of all steps?

Updated

The deploy preview is failing to build

@Deadreyo onboarding images are still broken

All the keys here should be lowercase to match the interests
https://github.com/open-sauced/insights/blob/3d6fa77c9fcb18e6a2088aa148f58752422f14fc/components/atoms/LanguagePill/LanguagePill.tsx#L19


Tree: JsonCursor for test, benchmarking, and bootstrapping

Implements ITreeCursor interface for navigating and reading JSONable trees.  Originally intended for benchmarking the overhead of ITreeCursor, can also be used for unit testing and possibly as a temporary stand-in for forest.

Thanks, @CraigMacomber!


Chose Servants

Preparation

[X] I read the Troubleshooting Guide
[X] My question isn't answered in any other issues (even the closed ones)

Your question
Hello, i was using Fate Grand Automata like 1 week and the first day, i get a some pictures from different servants, so i was doing some day i was doing some farm with Ishtar( i was choosing her manually) but i get a random download from a face picture from Isthar to can choose her automatically.
Okey thats my question where i can get more pictures of different servants, to can choose them automatically

https://youtu.be/JOwupZ4W8AQ?t=35m35s


Allow specifying dictionary and plain/phrase in full text search

Fixes #887, #601. Now is possible to:
# Use language in fts query
GET /tsearch?ts_vector=french.fts.amusant

# Use plainto_tsquery and phraseto_tsquery
GET /tsearch?ts_vector=plain.fts.The%20Fat%20Cats
GET /tsearch?ts_vector=phrase.fts.The%20Fat%20Rats

# Combine both
GET /tsearch?ts_vector=phrase.english.fts.The%20Fat%20Cats

# "not" also working
GET /tsearch?ts_vector=not.phrase.english.fts.The%20Fat%20Cats

There is a catch the phraseto_tsquery function is only supported for pg > 9.6, so this should be noted in the docs also that's the reason why I upgraded the circle pg to 9.6.2, otherwise the tests would fail.

@bgamari a while ago I asked if you could ask well-typed to look over postgrest code, but that is before i looked at what you are working on.
Thank you for the work you do on GHC and thanks to well-typed for paying you for that work and releasing it to FOSS.
That work benefits not only postgrest but every haskell developer.
We prioritized this feature for you as a thank you.
PS: and of course @steve-chavez nice work

Ruslan Talpa<EMAIL_ADDRESS>writes:

@bgamari a while ago I asked if you could ask well-typed to look over
postgrest code, but that is before i looked at what you are working
on.
Thank you for the work you do on GHC and thanks to well-typed for
paying you for that work and releasing it to FOSS.

Thanks! However, it should be said that the real credit belongs to
Microsoft Research, who funds this work.

That work benefits not only postgrest but every haskell developer.
We prioritized this feature for you as a thank you.

Thank you!

I knew about Microsoft Research funding ** simonpj**, didn't know it was also your case, well, thank you @microsoft :)
comparing github pages of microstof/apple/google/aws says a lot about who developers should say thank you to :)

Seems the enable/disable of the project in Travis did work, also if we all agree on merging this I would like to rebase and maintain the "migrate circle pg to 9.6" and "remove isnot/notin operators" commits, I think is worth preserving those.

The code seems good to me, and lots of tests. And phraseto_tsquery is only invoked if the client asks specifically for phrase matching, so it should remain generally compatible with pg 9.5 for the other full text searching cases.
Anyone have any other thoughts about this before we merge?
(I'll be sure to do a regular merge commit rather than a squash so it preserves the commits you create in your interactive rebase)

Merge!
@steve-chavez @begriffs do you have a strong feeling against releasing often?
Like 1-2 features ... release 4.x.0 or a few bugfixes ... release 4.3.x?

Releasing more often is fine, especially patches because for minor versions I also advance the docs version to match.

I add a section on the master branch of docs, so the next tagged version there will pick it up. https://postgrest.com/en/latest/api.html#full-text-search


Various voyage calculator fixes

Fix 100% dilemma bug. Fix voyage time bug. Adds primary/secondary sklls to voyage message.

The current duration of an existing voyage seems off with this. My voyage is at the 8hr dilemma right now (awaiting my choice in-game), but the calc says my voyage has been running for 7h 50m.
Seeing the voyage skills in the message is a nice addition. Maybe the current estimate can go in the message too?

I have a couple questions that aren't really related to this, but I've been meaning to ask.

Is it possible to move the filling of bins out of chewable and into voyagestats right before they're used to calculate percentages? Moving that loop out of chewable will speed up MVA calculation by about 20%. I don't think it will speed up the original calculator any, but it shouldn't negatively affect it either (unless I'm misunderstanding how the bins are being used).

I think we can avoid a fetch to ship_schematics in voyagestats because voyagecalculator always fetches it now and passes it down.

Again, these aren't related to the changes you've made here, but I wanted to note them as potential places for optimization and get your insights before I dig into them further.

The duration seems to be generally accurate. It might yet another bug involving dilemmas but is far more preferable to using the reported voyage duration which often just carries on counting when waiting for a dilemma response. I am happy to add the estimate to the message but seems a bit redundant as the estimate appears directly below it.
I am planning a optimization of chewable at  some point. Now it is doing two different jobs there is a lot of stuff in there that isn't always needed. I realised I missed a piece of code so I fixed that which should provide performance improvements.
I am also planning various improvements to the voyagestats component based on various comments in discord and some ideas i've had. I want to show which crew are getting a trait bonus. I will also add an icon representing the crews primary trait which will state something like 'Scroll to 3rd from top in '. I also want to integrate the citation optimiser so that crew that will would improve the voyage if cited/trained are shown. I'm torn between working on this or a new section of the website which will be called Dabo for my next major project.

I really like the additions you made here! Very glad to see so many requests incorporated here. I think we can still improve the way the recommendations are displayed. I have a few ideas for this, if you don't mind. I'll try to post screenshots of some potential alternatives in the next day or so.
I appreciate the attempt to implement a noBins option for MVAM as a way to address my earlier request. As I'm sure you know, the bins are actually required for voyagestats' renderChart method, so simply removing the bins from the MVAM calculation only works if we also stop showing the chart. I like the charts, so I definitely don't want to do this.
What I did instead was move the calculation of bins out of chewable (where the bins may be calculated for results that are never shown) to renderChart (so that the bins are only calculated on a shown recommendation). Of course this assumes that bins aren't needed for anything other than renderChart, so if you have plans for using them elsewhere, please feel free to revert what I did here. For what it's worth, this change does speed up MVAM calculations by about 15-20%; I thought it might also improve original calculations by a small measure, but my tests did not see any appreciable change in original calculation time, one way or the other.
I admit, I had not considered completely removing the extends options to speed up MVAM. But I think the estimates you get for potentially extending are still useful, even if MVAM has no option for optimizing for extends. As a result, I opted to set noExtends to false by default.

Just sent out a voyage with this and noticed a potential area for improvement. Maybe the skill popup rank should be filtered by the seat it's in. For example, right now it told me to "Select 6th crew from top in Medicine" to find Warship EMA (because he's my 6th best MED voyager overall), but he's actually being recommended for a CMD seat, so it might be more accurate for me if it said "Top" (because he is my best MED crew with CMD).

Okay, here are some ideas for different ways to display the recommendation. This first one is closest to what we've got now, except it breaks everything up into a simple table. The second column has the most important info, plus an icon that shows if the crew has a matching trait for that seat. The final column is the skill to use to find that crew in your roster, plus an icon if the suggested crew is frozen.

This second one is basically the same as the first, except it adds avatars for each crew. This obviously expands the height needed to display everything. It's not that much more, but if you're like me and want a mobile view that doesn't need any scrolling, this may be an issue:

Finally, this one attempts to show all information without the need for popups. We're getting really big now, but if you're going to require scrolling on mobile, it might as well show everything. Nevertheless, to keep it compact as possible, the recommendations are split into 2 columns with each skill getting its own row (and the skill icon now displayed to help clarify that). This view also matches the "cards" used to display other info about the voyage.

All thoughts and suggestions are welcome!

Are these coded or did you do it some other way? If it is the former we could implement them all side by side to see how they each work on various platforms.

Are these coded or did you do it some other way? If it is the former we could implement them all side by side to see how they each work on various platforms.

That's a good idea. Options 2 and 3 are coded, though neither has been optimized for mobile and option 3 is pretty sloppily written all around. I don't have time to make proper versions at the moment, but I've attached the two versions here if you want to figure that out. Otherwise I'll try to do this under a new PR later this weekend.
voyagestats-tableavatars.tsx.txt
voyagestats-cards.tsx.txt

Moved all the changes here to a clean branch #377. This one's long overdue, so hopefully a clean branch will build a little easier.


The page doesn't scroll up and the wrong line is highlighted

Hi,
I tried using your plugin with Obsidian v1.3.7 and I noticed this issue:

The line I'm typing on does not scroll up to the top of the page (I've set it to 10%), although the highlight line is on the 10% mark.
But this also means that the line being highlighted is not the line I'm typing on.

@vishae Thanks for your report. Could you please provide me a screenshot / list of all the plugin's settings you have? I could not reproduce the issue by only setting the offset to 10%. I suppose a specific combination of settings causes the problem.


Sparkapplication template

I am interested to contribute something similar to the spark pod template for Sparkapplication.
It can be super useful as a service management standpoint.
For example:
1: If I want to require all my users to install a Prometheus sidecar so I can collect metrics and monitor them without actively checking every submitted Sparkapplication spec nor team user how to set a Sparkapplication for Prometheus sidecar.
2: Some teams may want to enforce securityContext for every running pod in the cluster for security reasons.
3: adding some pod annotations for service management.
It is possible to write your own webhook to mutate or validate in order to achieve the above use cases.
But why not provide this feature within spark operator.
I can think of 2 kinds of templates.
1: default Sparkapplication templates, default if it is not set, or merge if it is a list in the spec.
2: enforce Sparkapplication templates, overwrite user's Sparkapplication.

@liyinan926 What do you think?

That sounds like a good proposal to me. Can you put together a doc on the details? Thanks!

new spark operator flag: --Sparkapplication-default-template, --Sparkapplication-overwrite-template (naming still pending)
Each flag will accept a configmap as input value.
e.g. --Sparkapplication-default-template=/namespace/configmapName
The content of the configMap should be a sparkapplication spec.
We can decide fields that is allowed here:
e.g.
apiVersion: v1
kind: ConfigMap
metadata:
  name: configmapName
  namespace: namespace
data:
  spec: |
    apiVersion: "sparkoperator.k8s.io/v1beta2"
    kind: ScheduledSparkApplication
    metadata:
      name: spark-pi-scheduled
      namespace: default
    spec:
      schedule: "@every 5m"
      concurrencyPolicy: Allow
      successfulRunHistoryLimit: 1
      failedRunHistoryLimit: 3
      template:
        type: Scala
        mode: cluster
        image: gcr.io/spark/spark:v3.0.0
        mainClass: org.apache.spark.examples.SparkPi
        mainApplicationFile: local:///opt/spark/examples/jars/spark-examples_2.12-2.3.0.jar
        driver:
          cores: 1
          memory: 512m
        executor:
          cores: 1
          instances: 1
          memory: 512m
        restartPolicy:
          type: Never

Then the sparkOperator will take this spec, and use as the default value for every submitted sparkapplication.
--Sparkapplication-overwrite-template=/namespace/configmapName
This flag will be mostly the same, but instead it will overwrite from the input spec to user's submitted sparkapplication.
For lists such as metadata sparkconf etc in the sparkapplication spec. It should do append on the list. and default/overwrite the keys in the list.

@jinxingwang thanks for the proposal. Can you elaborate a bit more on the flag sparkapplication-overwrite-template? What is the relationship between it and sparkapplication-default-template?
Also have you considered an alternative approach originally discussed in https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/pull/661?

@liyinan926 Thanks for the feedback.
The relationship between these 2 flags is:

both are optional.
sparkapplication-default-template is less restricted. It is for the cluster operators to recommend sparkApplication setup for the user if not set. for example: if I want to collect user metrics, I can put a default Prometheus sidecar config in the sparkapplication-default-template then if the users don't have a setup for monitoring it will use our recommended way. If users set this themself, they will be using their setup.
in compression, sparkapplication-overwrite-template is more strict, it will overwrite the user's sparkapplcaiton if you put def in the template. for example, if we want to make sure no pod is running as root in our cluster. I will put runAsNonRoot in the securityContext, in this case, doesn't matter what the user puts in the securityContext, it will add runAsNonRoot, so as a system operator, I know no container is running with potential vulnerability.
In case both templates have some same attributes set, it will result to sparkapplication-overwrite-template.

I have looked at #661, It is more like the sparkapplication-overwrite-template I proposed:

1 main feature I wanted it doesn't have is the enforcement, in the PR, the system operator doesn't have a way to enforce(default/overwrite) a good usage of sparkapplicaiton, if the user decided to not usesparkApplicationClass: pyspark2.
Also introducing an additional layer of sparkapplicaiton spec feels a bit overkill.
1 pro, it can provide different default spec, users can choose from.

The argument is if the user cares enough to get the sparkApplicationClass: pyspark2 pyspark2 itself understood enough, they can just set it on their spec.

@liyinan926 Hi.


No valid exports main found for @rollup/pluginutils

Describe the bug
npm run dev show errors in console No valid exports main found for 'vite-test/app/node_modules/@rollup/pluginutils'
Reproduction
I just follow the instructions
$ npm init vite-app <project-name>
$ cd <project-name>
$ npm install
$ npm run dev

System Info

required vite version: 0.20.0
required Operating System: MacOS Catalina 10.15.4
required Node version: v13.6.0

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.

vite v0.20.5
  vite:config env mode: development +0ms
  vite:config env: {} +1ms
  vite:resolve (node_module entry) vue -> dist/vue.runtime.esm-bundler.js +0ms
  vite:optimize optimizing vue (imports sub dependencies) +0ms
[vite] Optimizable dependencies detected:
vue
Pre-bundling them to speed up dev server page load...
(this will be run only when your dependencies have changed)

[vite] Dep optimization failed with error:
No valid exports main found for 'vite-test/app/node_modules/@rollup/pluginutils'
Error: No valid exports main found for 'vite-test/app/node_modules/@rollup/pluginutils'
    at resolveExportsTarget (internal/modules/cjs/loader.js:625:9)
    at applyExports (internal/modules/cjs/loader.js:502:14)
    at resolveExports (internal/modules/cjs/loader.js:551:12)
    at Function.Module._findPath (internal/modules/cjs/loader.js:657:22)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:960:27)
    at Function.Module._load (internal/modules/cjs/loader.js:855:27)
    at Module.require (internal/modules/cjs/loader.js:1033:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (vite-test/app/node_modules/@rollup/plugin-node-resolve/dist/index.js:15:19)
    at Module._compile (internal/modules/cjs/loader.js:1144:30) {
  code: 'MODULE_NOT_FOUND'
}

Can you have a try which remove node_modules and re-install agian?

Can you have a try which remove node_modules and re-install agian?

I did it several times. Still no effect. Maybe the problem with this plugin:
https://github.com/rollup/rollup-pluginutils  - This repository has been archived by the owner.

You likey have corrupted global npm cache.

You likey have corrupted global npm cache.

npm cache clean --force and npm cache verify  didn't help.
After cleaning npm cache I tried starter steps again. Still no effect... What can be wrong?

I had the same problem and got vite to start by installing @rollup/pluginutils version 3.0.10 instead of the most recent one. Moving vue from dependencies to devDependencies also worked but I suspect that's just bypassing the issue.

I had the same problem and got vite to start by installing @rollup/pluginutils version 3.0.10 instead of the most recent one. Moving vue from dependencies to devDependencies also worked but I suspect that's just bypassing the issue.

Thank you! Now it's working
I change package.json to this and it's worked
{
  "name": "myapp",
  "version": "0.0.0",
  "scripts": {
    "dev": "vite",
    "build": "vite build"
  },
  "devDependencies": {
    "@rollup/pluginutils": "^3.1.0",
    "vite": "^0.20.0",
    "@vue/compiler-sfc": "^3.0.0-beta.14",
    "vue": "^3.0.0-beta.14"
  }
}

I was getting this same error on Windows with Yarn. Removing node_modules and re-installing did not work.

Seems to be an issue with the node 13.x versions:
https://github.com/rollup/rollup/issues/3427

@subgan82 this is because Node.js 13.5.0 does not support this feature of the "exports" field. I'd suggest updating Node.js here.

And I can confirm, with node 14.4 I no longer have this problem.

@queckezz thanks for spotting the root cause!


Token Authentication is enabled, but unable to login

I pulled the sagemaker notebook container from Docker Hub using tag tensorflow-p36 and followed the instructions as specified in the README.md. Yet it has been problematic using this jupyter notebook as authentication does not appear to work.
I am presented with the login screen when I open the URL that the start script outputs, and manually pasting the token into the authentication field also results in an 'invalid credentials' message. Disabling token / password authentication doesn't resolve this issue either, as the notebook server still presents me with a login screen.
The command I'm using to run this notebook is the following:
docker run -p 8888:8888 -v $(pwd):/home/ec2-user/SageMaker qtangs/sagemaker-notebook:tensorflow-p36

As I don't need the AWS SDK and CLI for now, I have not included any credentials for these tools in the docker run command.

I tested the same command, it works fine outputting the following URL, note that the token consists of only alphanumeric characters:
[I 08:20:48.332 NotebookApp] The Jupyter Notebook is running at:
[I 08:20:48.333 NotebookApp] http://5f01fc2efe73:8888/?token=b8036342f6f327b3b396553b42bd86819916d312c05c4072
[I 08:20:48.333 NotebookApp]  or http://<IP_ADDRESS>:8888/?token=b8036342f6f327b3b396553b42bd86819916d312c05c4072

If you still can't get this to work, try building a new image with a preset password like this (password is pass123):
#Dockerfile.test
FROM qtangs/sagemaker-notebook:tensorflow-p36

RUN sed -ie "/^#c.NotebookApp.password/c\c.NotebookApp.password = 'sha1:fe3e0bde3016:dd7cbb05c6ed2673448512e3993d1c0f62909ead'" ~/.jupyter/jupyter_notebook_config.py

docker build -t test -f Dockerfile.test .
docker run -p 8888:8888 -v $(pwd):/home/ec2-user/SageMaker test
# once done open this in browser: http://<IP_ADDRESS>:8888/

Note: To get the hashed password for any other password, you can run this, replacing pass123 with a password of your choice:
docker run --rm qtangs/sagemaker-notebook:tensorflow-p36 python -c "from notebook.auth import passwd; print(passwd('pass123'))"

@ruijzndl, I’m closing this issue assuming you’ve got it working.
Feel free to let me know if there’s still any issue.


updater not working

while trying to update novels in library,it is showing error books are skipped.

You need to read all chapters or mark chapter as read in order to make global updater works

@kazemcodes why? Shouldn't the updater work regardless if i have or haven't read a particular novel?

I follow the same logic as tachiyomi, because unnecessary update request for many person who have more than 100 books in library can cause the site to trigger aggressive cloudflare protection as the site thinks it's under attack

Ah ok , so it works only if you actually have read at least one chapter of the novel or all of them?

The key point is marking all chapter as read

The key point is marking all chapter as read

we can do it manually nah?

Yeah, all this restrictions is only applied to global update ، single book update can be done without any restrictions

That makes the global updates useless tho. If you need to put all the chapters read for all the novels you have in the library , what's the points in having a global update?

thank you

I also have the same problem but I can also understand but if you want to do that talk to the developers of Tachiyomi.org cuz it uses Tachiyomi as the core.


[FLINK-12787][python] Allow to specify directory in option -pyfs

What is the purpose of the change
This pull request add support to allow to specify directory in option -pyfs
Brief change log

Add support to allow to specify directory in option -pyfs
Improve PythonDriver to use CommandLineParser for the arguments parsing

Verifying this change
This change added tests and can be verified as follows:

Existing tests in PythonDriverTest
Added tests in PythonDriverOptionsParserFactoryTest and PythonEnvUtilsTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@HuangXingBo Thanks a lot for your review. Updated accordingly.

@sunjincheng121 Thanks a lot for the review. Updated the PR accordingly.

+1 to merged

@flinkbot approve all
Merging...


[BUG] Navbar issue and home page issue

🐛 Bug Report
Describe the bug
A clear and concise description of what the bug is.
Here you can see the navbar is overlapping.

And here you can see the home page contents available more than one time.

Severity
How critical is this issue?

[x] 🔴 Critical
[ ] 🟠 Major
[ ] 🟡 Minor

Possible Fix
Suggest a fix or reason for the bug if known.

Checklist

[x] I have searched the existing issues and this bug has not been reported yet.
[x] I have updated the project to the latest version.
[x] I have provided all the necessary details.
[x] I have starred this repository.

@souvikpramanikgit please provide proper Issue description.

When i write the issue then the issue is there

Please assign this issue i will fix this.


Support Nested Routing and Manifests

Solid App Router supports nested routing. They do so by naming a file the same name as the folder. In so it acts as the parent. I haven't added that yet.
In addition we need to make sure that manifest lookup for preload works as well.

To improve the status quo, this line https://github.com/solidjs/solid-start/blob/main/packages/start/components/Outlet.tsx#L22 could be extended to let temp = list.find(o => o._id !== '/' && o._id && id.startsWith(o._id));. This would make it possible to support this kind of file structure:
/account/registration.jsx
/account/registration/index.jsx
/account/registration/otherpage.jsx

That generates this route structure:
      <Routes>
        <Route path="/account/registration" element=..>
          <Route path="/" element=.. >
            <Route path="/" />
            <Route path="/otherpage" element=.. />
          </Route>
        </Route>
      </Routes>

Compared to the previous route structure that doesn't work at all:
      <Routes>
        <Route path="/account/registration" element=..>
          <Route path="/" element=.. >
            <Route path="/">
              <Route path="/otherpage" element=.. />
            </Route>
          </Route>
        </Route>
      </Routes>


Support custom ports

No longer assume the port is 443/SSL

Thanks for a PR. I'd suggest to use https://pkg.go.dev/net#SplitHostPort to obtain hostname and port

Oh, im not even a GO dev. You can deny my pull request and change it as you see fit, but I can confirm this is a working concept when you give a server url with a custom port.
for example, It will probably fail when you dont give a port in the url, so this pull request is just a sloppy proof of concept. You prolly have to fall back to the default 443 port when no port is given or something like that.


FR: Ignoring property only on Swagger, without using JsonIgnore

Hello.
I have the following scenario in my API:

A data transfer object has a series of properties (i.e. Id, Code, Description...) that I want to return on HTTP GETs, and also want to send in HTTP POST/PUT/PATCH. However, there are also some properties that only make sense to return in GETs but shouldn't be part of any user interactions (CreationDate, UpdateDate...).
I do not want to use JsonIgnore on these, as they must be returned in GETs.

Should this be possible with Swashbuckle, for example, with a custom filter?
This url has an example for non-core Swashbuckle. Is this still the recommended solution?
Thanks.

Yes custom schema would be the way to go, if you are keen on returning the exact same object
Sent from my iPhone

On 10 Nov 2017, at 16.34, José Maia<EMAIL_ADDRESS>wrote:
Hello.
I have the following scenario in my API:
A data transfer object has a series of properties (i.e. Id, Code, Description...) that I want to return on HTTP GETs, and also want to send in HTTP POST/PUT/PATCH. However, there are also some properties that only make sense to return in GETs but shouldn't be part of any user interactions (CreationDate, UpdateDate...).
I do not want to use JsonIgnore on these, as they must be returned in GETs.
Should this be possible with Swashbuckle, for example, with a custom filter?
This url has an example for non-core Swashbuckle. Is this still the recommended solution?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I would say that the return from a post should be the same as getting the newæy created object via Get
Sent from my iPhone

On 10 Nov 2017, at 16.34, José Maia<EMAIL_ADDRESS>wrote:
Hello.
I have the following scenario in my API:
A data transfer object has a series of properties (i.e. Id, Code, Description...) that I want to return on HTTP GETs, and also want to send in HTTP POST/PUT/PATCH. However, there are also some properties that only make sense to return in GETs but shouldn't be part of any user interactions (CreationDate, UpdateDate...).
I do not want to use JsonIgnore on these, as they must be returned in GETs.
Should this be possible with Swashbuckle, for example, with a custom filter?
This url has an example for non-core Swashbuckle. Is this still the recommended solution?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Have you tried restricting the C# access for the properties in question? For example, public get and internal set. Swashbuckle will honor these by setting the readOnly flag accordingly.
Please note though, I can’t recall what the UI does for properties that are marked readOnly. It might hide them which would be perfect but it may just provide a note. Try it out and let me know

I ended up no longer requiring this, as per the consistency argument mentioned upthread. Thanks!


scan server metrics from #4458 based on #4459

This PR is based on the changes proposed by @keith-turner in #4458, using the changes in #4459 as the base.

includes updated documentation in MetricsProducer
includes cache metrics for table metadata cache in sserver (resolves todo)

These changes have not be test yet.
These changes should not be merged until #4459 is complete.

@EdColeman  I added commit d8b4c66e22b9efcd252e122031fc4513345e6713 to #4458 that could be merged into this

@keith-turner I believe this is up to date with your additions, if not we can add them before merge,  This was rebased with the other metric changes so the change-set is more reasonable to review.

Changes in 5dd3c2d311

Renames  String METRICS_SCAN_PREFIX from accumulo.tserver.scans. to accumulo.scan.  Service type (tserver, sserver) is available in the tags
Renames registration to reservation to reflect usage.

@EdColeman I was experimenting with these changes and I was not seeing the stats I expected from the cache.  Finally figured out that stats have to be enabled when building the cache, so pushed a change to do that in 3660ecd.
I was using MetricsIT to experiment with these changes when trying to figure this out, can see those changes here b3fdae9630b9719f7b3fcd7f2e6fae664f2dd1a4 if interested.


Question about different versions

Hi @smarek
We are thinking about using the httpclient-android lib as a replacement for the formerly used apache httpclient library. One think I couldn't completely understand from the documentation on https://hc.apache.org/httpcomponents-client-4.5.x/android-port.html was if httpclient-android-<IP_ADDRESS> targeting v23 of Android is backwards compatible as in supported min-version since our app has v14 as the min version and is targeting v23 at the moment and will target v24 in the future. According to https://github.com/smarek/httpclient-android/blob/master/build.gradle#L22 I guess it does but just wanted to ask to make sure.
Thanks in advance for all your efforts!
Andy

Hi @AndyScherzinger,
yes, this library uses different namespace, which does not collide with those provided or not-provided by Android OS, thus it should not matter, what version of platform you target or what version of platform the user has.
Truth is, that library itself is built with SDK v23, but again, it is both backward and forward compatible. And it does not affect in any way the API target you have in your application.
What should be mentioned is, that using old HttpClient classes, may not work at all times, I've already seen some mentions about it having the same issues as platform provided one (ie. timeouts, I/O blocking, SSL handling problems). However if you use new HttpClient API, such as CloseableHttpClient, HttpClients interface, etc... there should be no issues on neither old or new APIs.
If you're satisfied with the answer, please close this issue. Good luck !

Thank you very much for your quick answer!
Answers my question perfectly :+1:


Update/correct example for role

As a pull request owner and reviewers, I checked that:

[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role

/approve

Far better:
https://logserver.rdoproject.org/43/243/955979d8ad7aaf501e10eefb54333c086402c0e6/github-check/cifmw-doc/22a8226/doc_build/html/Roles/discover_latest_image.html


After scanning wrong QR

Current: share is aborted.
Desired: go back to scan screen (camera) for 2nd chance.

Implemented for text message or single resource share.

Currently not working.

Still not working.
Phone vibrates, error message is not displayed at all, camera keeps filming.

Maybe a Toaster would do the job


Exit with status 1 on directory errors

We added netlify deploy to our CI workflow.
We had a directory error but the exit status was 0.
We wanted the GitHub status check to fail in this case.

Oh good catch.  Thanks!

Released in v2.6.0!

Nice! Great turnaround time.


Remove a special treatment for SSLNetVC in migrateToCurrentThread()

UnixNetVC::migrateToCurrentThread has code related to SSL and also duplicate code. This PR moves the SSL related code into SSLNetVC and simplify the duplicated part.
Another small change is moving out calls for TLSSessionResumptionSupport::bind/unbind from SSLNetVCAttach/Detach. They were integrated on #7278, but I don't think it's a right direction. The calls were intentionally separated from SSLNetVCAttach/Detach because I'm trying to delete SSLNetVCAttach/Detach which is not applicable forQUICNetVC. I'm going to add these functions below on another PR once this is merged.
void
SSLNetVConnection::_bindSSLObject()
{
  SSLNetVCAttach(this->ssl, this);
  TLSSessionResumptionSupport::bind(this->ssl, this);
}

void
SSLNetVConnection::_unbindSSLObject()
{
  SSLNetVCDetach(this->ssl);
  TLSSessionResumptionSupport::unbind(this->ssl);
}

[approve ci autest]
error: no such commit b5d8e27c03738df863564bd843ada79d68edea82


[ISSUE #4298] zombie -> pass update PENDING_SYNC

FIX #4298

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4430 (337533a) into master (f88127f) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #4430      +/-   ##
============================================
- Coverage     68.04%   67.92%   -0.12%
+ Complexity     7500     7492       -8
============================================
  Files          1023     1023
  Lines         29005    29006       +1
  Branches       2592     2589       -3
============================================
- Hits          19736    19702      -34
- Misses         7720     7751      +31
- Partials       1549     1553       +4

Impacted Files
Coverage Δ

...henyu/admin/service/impl/UpstreamCheckService.java
57.71% <0.00%> (-8.96%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
21.27% <0.00%> (-6.39%)
:arrow_down:

...rg/apache/shenyu/plugin/rewrite/RewritePlugin.java
66.66% <0.00%> (-5.56%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.12% <0.00%> (-4.26%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
85.71% <0.00%> (-4.09%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
8.00% <0.00%> (-4.00%)
:arrow_down:

...in/response/strategy/NettyClientMessageWriter.java
71.42% <0.00%> (-3.58%)
:arrow_down:

...a/org/apache/shenyu/plugin/mock/util/MockUtil.java
77.55% <0.00%> (-2.05%)
:arrow_down:

... and 4 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


I need a UIUX designer 😭😭😭 (pretty girl only)


Fix more Kotlin deprecations and warnings

More fixes for toUpperCase(), toLowerCase(), and capitalize(), in addition to fixing some (new?) warnings.

Codecov Report

Merging #480 (f0ff0ca) into master (5f908e5) will not change coverage.
The diff coverage is 81.25%.

@@            Coverage Diff            @@
##             master     #480   +/-   ##
=========================================
  Coverage     58.87%   58.87%
- Complexity      861      862    +1
=========================================
  Files            69       69
  Lines          6451     6451
  Branches       1006     1005    -1
=========================================
  Hits           3798     3798
  Misses         2360     2360
  Partials        293      293

Impacted Files
Coverage Δ

...kotlin/com/microsoft.thrifty.gen/ServiceBuilder.kt
1.80% <0.00%> (ø)

.../com/microsoft/thrifty/schema/FieldNamingPolicy.kt
88.63% <66.66%> (ø)

.../com/microsoft/thrifty/kgen/KotlinCodeGenerator.kt
84.10% <90.00%> (ø)

...in/kotlin/com/microsoft/thrifty/schema/Constant.kt
78.53% <100.00%> (ø)

...n/com/microsoft/thrifty/schema/UserElementMixin.kt
69.23% <100.00%> (ø)

Continue to review full report at Codecov.

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


Handle changes in ARTrackingState

See Changing State with ARTrackingState on the wiki for more details.

Removing this as an Issue


2 Factor Authentication - Captcha not supported by Amazon.in.

I am from India. My Alexa devices are connected to Amazon.in (Site in India). Captcha is not supported by Amazon in India. Instead it allows for an OTP while logging in. OTP does not work while configuring the Alexa Media player. If I re-register my Alexa devices on Amazon.com, the music files in local language are not available on US sites.
I changed settings in Amazon.in to revert to single factor authentication. I was able to configure the media player without the second factor. Is this OK?

There is no support if you don't have 2FA.


Fix pin reference for HTTP request action

Renovate relies on these comments to match hashes to tags, but this action doesn't prepend tags with v like the other actions.

Related: https://github.com/bitwarden/passwordless-server/pull/365


fix(app): ensure single app instance

Description:
Ensure single instance of the app
Motivation and Context:
Fix #229
Types of changes:
Bug Fix
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.03%) to 19.325% when pulling 5bc885efaae4d56ab0d1ff82cd14186b8bdc52bc on mrfelton:fix/single-instance-app into 4055e7ace498adfaa3afda558dd30434fe3db53c on LN-Zap:master.

Coverage decreased (-0.03%) to 19.325% when pulling 5bc885efaae4d56ab0d1ff82cd14186b8bdc52bc on mrfelton:fix/single-instance-app into 4055e7ace498adfaa3afda558dd30434fe3db53c on LN-Zap:master.

@lieteau2 I've tested this on a Windows machine and made some updates to the patch to get it working correctly. Please could you verify. Thanks.

Concept ACK


Webpack 5 target web, es5 error

🐛 Bug Report
When using target ES5 in webpack I'm getting the error
Uncaught TypeError: Cannot read property '__LOADABLE_LOADED_CHUNKS__' of undefined

To Reproduce
Use webpack 5 with target: ["web", "es5"]
Expected behavior
No error appears in the console
Issues without a reproduction link are likely to stall.
System Info
## System:
 - OS: macOS 10.15.7
 - CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
 - Memory: 69.03 MB / 16.00 GB
 - Shell: 5.7.1 - /bin/zsh
## Binaries:
 - Node: 14.9.0 - /usr/local/bin/node
 - Yarn: 1.22.5 - /usr/local/bin/yarn
 - npm: 6.14.7 - /usr/local/bin/npm
## npmPackages:
 - @loadable/babel-plugin: ^5.13.2 => 5.13.2
 - @loadable/component: ^5.14.1 => 5.14.1
 - @loadable/webpack-plugin: ^5.14.0 => 5.14.0

Tracked in #636, but this is not known before behavior

Now that webpack and loadable-components has fixed most issues, I wanted to try this again.
But I'm still running into the same error. Can anyone maybe give any hints what the problem is related to? Maybe some part of our babel is setup weirdly? Maybe webpack output options thats weird or something?
Uncaught TypeError: Cannot read property '__LOADABLE_LOADED_CHUNKS__' of undefined

"@loadable/babel-plugin": "^5.13.2",
"@loadable/component": "^5.14.1",
"@loadable/webpack-plugin": "^5.14.2",
"webpack": "^5.24.3",
"webpack-cli": "^4.5.0"

Compiling with target web works fine, because that doesn't use the browserlist, but if I compile with no webpack target it will default to browserlist which includes IE11
    "browserslist": [
        "last 2 versions",
        "iOS >= 10",
        "IE >= 11"
    ],

target: ["web", "es5"] also breaks

I have the same issue
Uncaught TypeError: Cannot read property '__LOADABLE_LOADED_CHUNKS__' of undefined in IE11.
I use target: ["web", "es5"] and runtimeChunk: true.
If I understand correct, then I receive exception because during access to this.['__LOADABLE_LOADED_CHUNKS__'] this is undefined due to 'use script' in function.
@theKashey can you help?


Runnable Examples

Introduces many runnable examples to the function documentation.

Pull Request Test Coverage Report for Build 525

9 of 16 (56.25%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 92.359%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

R/semi_pheatmap.R
9
16
56.25%

Files with Coverage Reduction
New Missed Lines
%

R/semi_pheatmap.R
1
78.65%

Totals

Change from base Build 524:
0.0%

Covered Lines:
2333

Relevant Lines:
2526

💛  - Coveralls


Boundary condition

Nice work! I think you are applying Dirichlet boundary condition in the FMG function. Is there a way to apply Neumann boundary condition, du/dx = 0 at left/right and du/dy = 0 at top/bottom?

@kaushikb258 Hey there! Thank you. You are correct. The boundary conditions are hard coded for homogeneous Dirichlet conditions on all boundaries.
In the 2d code, changing the boundary condition statements from
  u[ 0,:] = -u[ 1,:]
  u[-1,:] = -u[-2,:]
  u[:, 0] = -u[:, 1]
  u[:,-1] = -u[:,-2]

to
  u[ 0,:] = u[ 1,:]
  u[-1,:] = u[-2,:]
  u[:, 0] = u[:, 1]
  u[:,-1] = u[:,-2]

in the three places in the function Jacrelax() will apply homogeneous Neumann BCs on all boundaries. I have tested it on a corresponding sample problem and it does work. The modified code is attached. (Note that when you have all Neumann BCs the solution is only determined upto a constant)
example_FMG_neumann.zip

Thank you for the prompt response and help! I will run the new code.


HAWQ-1506. Fix multi-append bug of write a encryption zone

This bug is happened on:
1, append a existed file.
2, call multi hdfsWrite()
Test case please see TEST(TestCInterfaceTDE, TestAppendMultiTimes).

@linwen @wengyanqing @amyrazz44 @wcl14 help to review, thanks!

LGTM.

+1

merged


[$250] 2 bottom banners for offline status are displayed

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.41-2
Reproducible in staging?: y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
Action Performed:
Pre-requisite: the user must be logged in and have created a workspace.

Go to Workspace > Bank account.
Initiate the add BA flow until you reach the Personal info double check page.
Disable the internet connection.

Expected Result:
Only 1 bottom banner for offline status should be displayed.
Actual Result:
2 bottom banners for offline status are displayed
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016e92a84369a21d16
Upwork Job ID:<PHONE_NUMBER>607944192
Last Price Increase: 2024-02-14
Automatic offers:

akinwale | Contributor | 0

This is minor design bug not a blocker

cc @shubham1206agra too

Dibs or need to put a proposal ?

@godofoutcasts94 all issues except the TS migration ones require proposals.

@MrMuzyk @akinwale @shubham1206agra what do you think of proposal from @abzokhattab any reason there was the double ScrollView added? anything to look out for if we would remove one?

@godofoutcasts94 all issues except the TS migration ones require proposals.

Thanks for letting me know.

@MrMuzyk @akinwale @shubham1206agra what do you think of proposal from @abzokhattab any reason there was the double ScrollView added? anything to look out for if we would remove one?

@mountiny I am more inclined to go with using the shouldShowOfflineIndicator for the other ScreenWrapper as that wouldn't require changes to the existing styles or layout.

@akinwale @abzokhattab Well is there any reason we need to use two screen wrappers on this page?

Double check spacing around submit button. If it works then I'm fine with such solution

Yeah lets remove the second wrapper and fix any misalignments if that happens

@abzokhattab Let's go with the solution to remove the second wrapper and check the layout and spacing according to https://github.com/Expensify/App/issues/36458#issuecomment-1946333226.
🎀👀🎀 C+ reviewed.

sorry for the ping @techievivek

Payment for @akinwale will be handled here https://github.com/Expensify/App/issues/22901#issuecomment-1977046291

Payment has been issued to @abzokhattab via Upwork
Coming from @mountiny's comment here, it looks like payment for @akinwale will be handled in #22901 (comment)


kubernetes/website release 1.23 freeze

This issue freezes k/website and should remain open until 1.23 is officially released!
The release is scheduled to happen tomorrow 7th of December and No PRs are allowed to merge until the release PR has successfully merged.
If there are any questions, please contact me or anyone from the docs release team
cc: @jimangel @divya-mohan0209 @reylejano
cc: @mehabhalodiya @chrisnegus @ramrodo @nate-double-u

1.23 release PR has been merged and the release is complete. Dropping freeze and closing.


Fix bug that prevents installing package from git on windows

This PR fixes an issues that prevents windows user to install packages from git repositories using poetry, Currently git on windows downloads something into a folder that is write protected. When poetry delets the downloaded stuff for cleanup, remove thig specific folder raises an exception see issue #177.

Would be extremely helpful to have this merged in as it's still an issue on 0.11.3

@sdispater Thanks for fixing it in 0.12.8


Make password reset email deliveries based on case insensitive email …

…addresses
Fixes #177

Coverage decreased (-0.006%) to 53.447% when pulling f45072a36e923ab538ca7b68a059371f668db2aa on aackerman:aackerman/make-password-resets-case-insensitive into 79471256cde60b3143838f0916033783cd2f7ebf on minnestar:master.


Allow #[main] in cxx::bridge

The idea is to be able to use C++ as entrypoint. Usecase: existing codebase that I want to enhance with rust lib. I could not find a way to do this. The closest I got was to create a main.rs file without main() so it would take the main function defined in my c++ codebase, but then, Rust complains that main() must be defined at the root of the crate, -or- mark it with #[main].

If I understand correctly, this is already supported. See https://cxx.rs/binding/cxxvector.html#example or https://cxx.rs/binding/slice.html#example for examples that have entrypoint implemented in C++.

! Thanks


feat: use autogenerated ABI target list

Uses the following sources to autogenerate a relatively recent ABI registry on a daily basis and commit the changes to master:

abi_version_registry.json from Node.js
Node.js release (+ LTS) schedule
lite.json from electron-releases

This should avoid the issues generated whenever a new major Node/Electron version is released.
The GitHub Action is based on something similar I created for Electron Fiddle.

CC: @MarshallOfSound

This looks like a good idea. I don't know how to update the publishing etc. Maybe @lgeiger knows? Could also be combined with resolving #58

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Contact us page Created

Fixes:  #735
Created a Contact us page for Your site and made the site responsive

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@MOHIT-IITP
Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉


UI changes of Add New Account.

-The balance is not needed, we need to have the tag lines here.

-Not a text box but a text field.

-we need to have the tag lines here.


Optional pydantic

Description
Converts pydantic to being optional, as it is required for Airflow 2.6 but not others.
Related Issue(s)
Closes #725
Closes #654
Checklist

[ ] I have made corresponding changes to the documentation (if required)
[ ] I have added tests that prove my fix is effective or that my feature works

@tatiana @jlaneve, Tests passing with Pydantic now marked as optional, as you ok to approve this PR? - thanks

I don’t think the failures here have anything to do with Pydantic now ?
As they relate to cache being removed in the main branch

@pixie79 thanks! I fixed the issues in the main branch, and rebase your branch. Once the tests pass in your PR, I'll merge it.


⚠ Samsung has degraded performance

In b0f6e9a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2457 ms

Resolved: Samsung performance has improved in a7672ab.


Tomcat Authentication Agent using Java OIDC SDK

Create a Tomcat agent(Filter) to use the JAVA OIDC SDK to integrate with a Tomcat based web applications to easily secure those applications.
Depends on: #8956

Duplicate


Update FDO Link in read me

update "Intel FDO" link to just "FDO" and target address to LF Edge https://github.com/fido-device-onboard

If @naphelps is busy, I have a little bit of time to work on this doc issue today.


Conflicting packages

In the last version (1.0.0-rc.4), the following conflicting packages were found:

I found a workaround for now.

Move to the "swanky-cli" directory under the node installation directory.
In my case: "/Users/shin.takahashi/.nodebrew/node/v18.0.0/lib/node_modules/@astar-network/swanky-cli"
Open package.json and edit the version.

"@polkadot/util-crypto": "10.1.13"

Delete the node_modules directory.
Run "npm install".

Thanks for the workaround @realtakahashi !
This is a deeper issue that is likely to keep popping up anytime some of the packages are updated.
The solution we're working on is to pack and distribute the app as a standalone package, dependencies included. That way we'll avoid users having to install it globally and getting mismatched dependencies.

The latest version (v1.0.6) is published on Github as a standalone tarball.
It seems that the npm version doesn't have this issue neither for now.


Preimage cache can get inadvertently messed up

Apologies for the strange title, not sure what to call this. I am not sure if this behavior is a bug or not, if not, it's definitely worth documenting somewhere (and figuring out the general pattern which causes this behavior, to warn users). Funnily enough, this is something where when we have #17, it would naturally be avoided.
Consider the following code, it's natural for a user to grab all edges incident to some vertex along some hom, and then want to further filter that set by an attribute of the edge.  In the example, the second call to incident returns only [1,2], even though the mapping is still [1,1,1]. I pasted the invalidated preimage cache below this block.
using Catlab

wg = @acset WeightedGraph{Symbol} begin
    V=2
    E=3
    src=[1,1,1]
    tgt=[2,2,2]
    weight=[:a,:a,:b]
end

out = incident(wg, 1, :src)
filter!(e -> wg[e, :weight] == :a, out)

incident(wg, 1, :src)

where the issue is:
julia> wg.subparts[:src].m
ACSets.Mappings.VecMap{Int64, Vector{Int64}} with 3 entries:
  1 => 1
  2 => 1
  3 => 1

julia> wg.subparts[:src].pc
ACSets.PreimageCaches.StoredPreimageCache{Int64, Int64, ACSets.IndexUtils.SortedSet{Int64}, ACSets.Mappings.VecMap{ACSets.IndexUtils.SortedSet{Int64}, Vector{ACSets.IndexUtils.SortedSet{Int64}}}}(ACSets.Mappings.VecMap{ACSets.IndexUtils.SortedSet{Int64}, Vector{ACSets.IndexUtils.SortedSet{Int64}}}(1 => ACSets.IndexUtils.SortedSet{Int64}([1, 2])))

I think that, for performance reasons, incident returns a view onto the preimage cache, rather than a copy. This is sensible default because if you're, say, traversing a graph you don't want to be making copies at every move. But this should certainly be documented somewhere if it's not.
We could also considering wrapping the vector in a struct sub-typing AbstractVector but enforcing the immutability. Then it would be impossible to get burned by this.

Actually @epatters I just checked; it is documented. So it's user error on my part, happy to close this being the case.

No worries. If you want to open an issue about the last thing I mentioned, I am happy to consider it.


Add openconnect pa

Motivation for this change
openconnect does not support Palo Alto Network's GlobalProtect protocol. I use a fork of openconnect regularly at work, where all of our traffic goes through a PA firewall. The package is actively maintained by @dlenski and is in use by many active developers. The fork is located at https://github.com/dlenski/openconnect
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions
[x] Ubuntu 16.04

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

I believe that the unchecked boxes don't apply, other than macOS and other Linux distributions. I don't have access to any other machines than Ubuntu, and It worked on that.

@GrahamcOfBorg build openconnect_pa

Success on x86_64-linux (full log)
Attempted: openconnect_pa
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08/lib  /nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08/sbin
patching script interpreter paths in /nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08
checking for references to /build in /nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08...
moving /nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08/sbin/* to /nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/wf6n1nfqh5n5jsk70afr1dcp9z4ksrp3-openconnect_pa-unstable-2018-10-08-dev
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wf6n1nfqh5n5jsk70afr1dcp9z4ksrp3-openconnect_pa-unstable-2018-10-08-dev/lib
patching script interpreter paths in /nix/store/wf6n1nfqh5n5jsk70afr1dcp9z4ksrp3-openconnect_pa-unstable-2018-10-08-dev
checking for references to /build in /nix/store/wf6n1nfqh5n5jsk70afr1dcp9z4ksrp3-openconnect_pa-unstable-2018-10-08-dev...
/nix/store/4cqgljvjsi2msd832kkg9l9q04d1p3g9-openconnect_pa-unstable-2018-10-08

Success on aarch64-linux (full log)
Attempted: openconnect_pa
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08/lib  /nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08/sbin
patching script interpreter paths in /nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08
checking for references to /build in /nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08...
moving /nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08/sbin/* to /nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/4y6did8px6h0k0caziqq4rs9796q1yg4-openconnect_pa-unstable-2018-10-08-dev
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4y6did8px6h0k0caziqq4rs9796q1yg4-openconnect_pa-unstable-2018-10-08-dev/lib
patching script interpreter paths in /nix/store/4y6did8px6h0k0caziqq4rs9796q1yg4-openconnect_pa-unstable-2018-10-08-dev
checking for references to /build in /nix/store/4y6did8px6h0k0caziqq4rs9796q1yg4-openconnect_pa-unstable-2018-10-08-dev...
/nix/store/jmwsh63793kp56y7h4kwkj8pl67ws25m-openconnect_pa-unstable-2018-10-08

thanks for taking the time to review this, @worldofpeace.
as far as it working, i was able to use it to authenticate, attempt to connect to my work's network remotely, get prompted by google duo, and then ssh into a machine there which i would otherwise not have access to

thanks for taking the time to review this, @worldofpeace.
as far as it working, i was able to use it to authenticate, attempt to connect to my work's network remotely, get prompted by google duo, and then ssh into a machine there which i would otherwise not have access to

Thanks for contributing :sparkle:
Good to see that you can verify its function.

@Mic92 erm, those commits were supposed to be squashed :laughing:

Yeah, sorry. I hit the wrong button.


(Docs) ESC Integrations page for GitHub

We should add a GitHub page to https://www.pulumi.com/docs/esc/other-integrations/

Added to epic https://github.com/pulumi/pulumi-service/issues/18145

Same as https://github.com/pulumi/docs/pull/12419


integration-proof-verification failing

integration-proof-verification | end-to-end_1  |   ● proof_verification › UltraVerifier › verifies full proof
  +integration-proof-verification | end-to-end_1  |
  +integration-proof-verification | end-to-end_1  |     expect(received).resolves.toBeTruthy()
  +integration-proof-verification | end-to-end_1  |
  +integration-proof-verification | end-to-end_1  |     Received promise rejected instead of resolved
  +integration-proof-verification | end-to-end_1  |     Rejected to value: [ContractFunctionExecutionError: The contract function "verify" reverted.·
  +integration-proof-verification | end-to-end_1  |     Error: PAIRING_FAILED()··
  +integration-proof-verification | end-to-end_1  |     Contract Call:
  +integration-proof-verification | end-to-end_1  |       address:   0x2279b7a0a67db372996a5fab50d91eaa73d2ebe6
  +integration-proof-verification | end-to-end_1  |       function:  verify(bytes _proof, bytes32[] _publicInputs)
  +integration-proof-verification | end-to-end_1  |       args:            ([encoded data, 4290 characters], ["0x23652d86d8de2b66b77d04345a98c9c9f71985cc758ca845fc2a0ba287bc9514","0x0000000000000000000000000000000000000000000000000000000000000009","0x0d2b42f073eaacb34b171059323b0a7ac60eca9afdf83953624519f12dcf3166","0x0000000000000000000000000000000000000000000000000000000000000008","0x0000000000000000000000000000000000000000000000000000000000000001","0x00e6fae30b8d7cd0877a681585498155b06018cec85bb3999ac9b59960a24fd4","0x00089a9d421a82c4a25f7acbebe69e638d5b064fa8a60e018793dcb0be53752c","0x0007638bb56b6dda2b64b8f76841114ac3a87a1820030e2e16772c4d294879c3","0x1864fcdaa80ff2719154fa7c8a9050662972707168d69eac9db6fd3110829f80","0x0000000000000000000000000000000000000000000000000000000000000080","0x2a5bffed25c1c1295587a6f6722b0e7e585798f42e936540c7d5f3c82a4ee8b9","0x0000000000000000000000000000000000000000000000000000000000000400","0x0a48f5de055f30b334529e48ceea96949af880b4da37fab30a6fd573f51b90c6","0x0000000000000000000000000000000000000000000000000000000000000480","0x23ac5a6f753b6f2aad740069b3849c53315abca4f140d9c21a571b5fd27d8cd9","0x0000000000000000000000000000000000000000000000000000000000000240","0x0000000000000000000000000000000000000000000000000000000000007a69","0x0000000000000000000000000000000000000000000000000000000000000001","0x0000000000000000000000000000000000000000000000000000000000000008","0x00000000000000000000000000000000000000000000000000000000664e13be","0x0000000000000000000000000000000000000000000000000000000000000000","0x0000000000000000000000000000000000000000000000000000000000000000","0x0000000000000000000000000000000000000000000000000000000000000001","0x0000000000000000000000000000000000000000000000000000000000000001","0x000000000000000000000000000000000000000000000000000000000bf113f8","0x00000000000000000000000000000000000000000000000da90a11a58c9d2a06","0x000000000000000000000000000000000000000000000007b0b608008f6b210e","0x00000000000000000000000000000000000000000000000c08ccd01e832d921d","0x0000000000000000000000000000000000000000000000000001bed1b2db17fa","0x0000000000000000000000000000000000000000000000022fd4d85a46371225","0x00000000000000000000000000000000000000000000000168bb309bca38c095","0x00000000000000000000000000000000000000000000000434a64fbf645f9eee","0x00000000000000000000000000000000000000000000000000024bd086c38c3b","0x000000000000000000000000000000000000000000000000f2646454548cf0ae","0x00000000000000000000000000000000000000000000000f43d43b3a4a423930","0x00000000000000000000000000000000000000000000000455409186baedee1b","0x000000000000000000000000000000000000000000000000000103a0b10ea119","0x000000000000000000000000000000000000000000000003a1c98edc72599901","0x0000000000000000000000000000000000000000000000035d958e53e66754f7","0x000000000000000000000000000000000000000000000001a757e793ac81551f","0x0000000000000000000000000000000000000000000000000000a72b24d5f4e0"])·
  +integration-proof-verification | end-to-end_1  |     Docs: https://viem.sh/docs/contract/readContract
  +integration-proof-verification | end-to-end_1  |     Version<EMAIL_ADDRESS>  +integration-proof-verification | end-to-end_1  |
  +integration-proof-verification | end-to-end_1  |       146 |         .toString('hex')}` as const;
  +integration-proof-verification | end-to-end_1  |       147 |
  +integration-proof-verification | end-to-end_1  |     > 148 |       await expect(verifierContract.read.verify([proofStr, publicInputs])).resolves.toBeTruthy();
  +integration-proof-verification | end-to-end_1  |           |             ^
  +integration-proof-verification | end-to-end_1  |       149 |     });
  +integration-proof-verification | end-to-end_1  |       150 |
  +integration-proof-verification | end-to-end_1  |       151 |     it('verifies proof taking public inputs from block', async () => {
  +integration-proof-verification | end-to-end_1  |
  +integration-proof-verification | end-to-end_1  |       at expect (../../node_modules/expect/build/index.js:113:15)
  +integration-proof-verification | end-to-end_1  |       at Object.expect (composed/integration_proof_verification.test.ts:148:13)

See e.g. https://github.com/AztecProtocol/aztec-packages/actions/runs/9199595670/job/25304755035


Python 3 default docker image

Are you planning to user python 3 as default docker image ?

Cabot currently doesn't run on python3 - there's a lot of python2 specific syntax in the codebase

Would you like to migrate to 3 ?


Convert Server dates to Java Date() objects

This includes changing any and all validation or tests associated with the date format.


[Feature][Transform] Add SimpleSQL transform plugin

Purpose of this pull request
We can use  simple SQL to transform input data. For example config:
transform {
  SimpleSQL {
    source_table_name = "fake"
    result_table_name = "fake1"
    query = "select id, concat(name, '_') as name, age+1 as age from fake where id>0"
  }
}

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[x] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

The CI failed, please check and add license

Good Job!
Please add this feature in release-note.

Good Job! Please add this feature in release-note.

ok

Hi, maybe change name SimpleSQL to SQL will be better. Clear, easy to understand.

Hi, maybe change name SimpleSQL to SQL will be better. Clear, easy to understand.

The plugin name SQL is the same name as the old transform of Spark and Flink engine?

Hi, maybe change name SimpleSQL to SQL will be better. Clear, easy to understand.

The plugin name SQL is the same name as the old transform of Spark and Flink engine? Is the old SQL transform plugin removed？

yes


Session not saved sometimes

I'm using set() with a redirect right after and in some conditions, the change is not saved to the database.
session.set(SESSION_KEY_USER, user_id).await;
return Ok(Redirect::to("/"));

Any ideas?

Note that the user is also not set in the session after the redirect.

what version of the library are you currently using and what options do you have set for the session?

postgresl
let session_config = AxumSessionConfig::default()
  .with_table_name("sessions");

axum_database_sessions = { version = "3.0.0", features = [ "postgres", "rustls"] }
axum_sessions_auth = { version = "2.0", features = [ "postgres", "rustls" ] }

I think I have a test case:
https://github.com/bbigras/test_axum_database_sessions
steps:

docker run --name test -e POSTGRES_PASSWORD=password -d -p 5432:5432 postgres
cargo run
visit http://<IP_ADDRESS>:3000/  a couple of times to increment the counter
visit http://<IP_ADDRESS>:3000/redirect once.
session.get("count").await is now None

I hope I didn't do anything stupid.

I am getting
listening on <IP_ADDRESS>:3000
[src\main.rs:40] session.get("count").await = Some(
    0,
)
[src\main.rs:40] session.get("count").await = Some(
    1,
)
[src\main.rs:40] session.get("count").await = Some(
    2,
)
[src\main.rs:40] session.get("count").await = Some(
    3,
)
[src\main.rs:40] session.get("count").await = Some(
    4,
)
[src\main.rs:40] session.get("count").await = Some(
    5,
)
[src\main.rs:40] session.get("count").await = Some(
    6,
)
[src\main.rs:40] session.get("count").await = Some(
    7,
)
[src\main.rs:40] session.get("count").await = Some(
    -10,
)

Please check to make sure your browser is not clearing the cookie's and to make sure your session_id cookie is staying persistent across. Also just to be sure i set the count to be a i32 since usize cant be negative.

I updated my test case to use a String and it's fine. I'll test my real code and get back to you.
Another weird thing is when I use session.destroy() to logout.
If you want to test, git pull my test case and try:

visit http://<IP_ADDRESS>:3000/ a couple of times to "grow" the string
visit http://<IP_ADDRESS>:3000/logout once.
visit http://<IP_ADDRESS>:3000/ again twice and the string doesn't "grow" anymore.

Congrats! you found a bug =3. So yeah the issue with that is destroy is not getting set back to false after resetting the internal hash table.  Thank you for that find. Also we do have a discord channel to if you ever want to talk more in depth and get quicker help on things. Ill release a patch to fix destroy.

I missed a destroy = false....

this has been fixed in 3.0.2


Update scala-library to 2.12.19 (#516)

Co-authored-by: scala-steward-asf[bot]<EMAIL_ADDRESS>Co-authored-by: PJ Fanning<EMAIL_ADDRESS>(cherry picked from commit ab5e2b61cc512188ded61fb52f8070611ab9874e)

LGTM

@mdedetrich is this ok to merge?


Add a trufflehog regex check.

Codecov Report

Merging #283 into master will increase coverage by 0.4%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #283     +/-   ##
=========================================
+ Coverage   83.23%   83.64%   +0.4%
=========================================
  Files          37       37
  Lines        1724     1724
=========================================
+ Hits         1435     1442      +7
+ Misses        289      282      -7

Impacted Files
Coverage Δ

hca/util/__init__.py
87.14% <0%> (-0.29%)
:arrow_down:

hca/cli.py
67.82% <0%> (+6.95%)
: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 b757712...7041669. Read the comment docs.


store parsed schema in cache

so it's only called once instead of everytime when decode/encode is called

Sorry, hectic week. lgtm without being able to test performance.

I'll try to get this merged & released but based on #12 I'll have to consider avsc options as well, the general idea stays the same: store parsed schemas (with options if passed).

out in 1.3.0

Finally doubling back to this, performance has gone from around ~17ms to ~0.02ms per record. This is huge in some of the aggregation work I perform in node! Thanks so much for this. Now I don't have to go manually copy/paste schemas every time I make a microservice lol.

Thanks for validating and reviewing the PR, glad it worked out!


Reproduce crash electrum on unconfirmed negative balances

Description
Test for reproducing electrum service crash when there is an unconfirmed transacction.
Steps:

Run tests
Go to electrum services logs
You going to see an error like this:

The electrum service can't be recovered automatically you need to restart it, electrum is going to crash in requests balances of addresses used into unconfirmed transactions as inputs until the transaction will be confirmed.

The error will be fixed when Electrum disappears.


MAC(version 10.11.3) : odbcinst: SQLInstallDriverEx failed with Invalid install path.

I tried to install mssql driver to MAC for my PH/Laravel application.
When I try following;
#brew install microsoft/msodbcsql/msodbcsql
I get following issue;
==> Installing msodbcsql from microsoft/msodbcsql
==> Using the sandbox
==> Downloading http://download.microsoft.com/download/4/9/5/495639C0-79E4-45A7-B65A-B264071C3D9A/msodbcsql-<IP_ADDRESS>.tar.gz
Already downloaded: /Users/ratha/Library/Caches/Homebrew/msodbcsql-<IP_ADDRESS>.tar.gz
The license terms for this product can be downloaded from
https://aka.ms/odbc131eula and found in
/usr/local/share/doc/msodbcsql/LICENSE.txt . By entering 'YES',
you indicate that you accept the license terms.

Do you accept the license terms? (Enter YES or NO)
YES
==> odbcinst -u -d -n "ODBC Driver 13 for SQL Server"
==> odbcinst -i -d -f ./odbcinst.ini
Last 15 lines from /Users/ratha/Library/Logs/Homebrew/msodbcsql/02.odbcinst.ini:
2017-08-03 14:58:01 +1000

odbcinst -i -d -f ./odbcinst.ini

odbcinst: SQLInstallDriverEx failed with Invalid install path.

If reporting this issue please do so to (not Homebrew/brew or Homebrew/core):
microsoft/msodbcsql

/usr/local/Homebrew/Library/Homebrew/utils/github.rb:226:in `raise_api_error': curl failed!  (GitHub::Error)
curl: (22) The requested URL returned error: 422 Unprocessable Entity
curl: (3) <url> malformed
	from /usr/local/Homebrew/Library/Homebrew/utils/github.rb:184:in `open'
	from /usr/local/Homebrew/Library/Homebrew/utils/github.rb:233:in `issues_matching'
	from /usr/local/Homebrew/Library/Homebrew/utils/github.rb:272:in `issues_for_formula'
	from /usr/local/Homebrew/Library/Homebrew/exceptions.rb:369:in `fetch_issues'
	from /usr/local/Homebrew/Library/Homebrew/exceptions.rb:365:in `issues'
	from /usr/local/Homebrew/Library/Homebrew/exceptions.rb:419:in `dump'
	from /usr/local/Homebrew/Library/Homebrew/brew.rb:133:in `rescue in <main>'
	from /usr/local/Homebrew/Library/Homebrew/brew.rb:26:in `<main>'

SO query
https://stackoverflow.com/questions/45474305/pdoexception-file-not-found-for-mssql-odbc/45474492?noredirect=1#comment77910465_45474492

Can you try the steps in README?
brew tap microsoft/msodbcsql https://github.com/Microsoft/homebrew-mssql-release
brew update
brew install msodbcsql
brew install mssql-tools
brew install autoconf

@vijayaratha This issue is caused changes in homebrew, please see #491

@vijayaratha can you confirm whether your issue is fixed by --no-sandbox?

@Hadis-Fard  Yes That fixed the installation issue of msodbcsql driver in MAC.

Adding --no-sandbox didn't solve the issue for me.
The solution was to create ~/.odbcinst.ini file:
printf "[ODBC Driver 13 for SQL Server]\nDescription=Microsoft ODBC Driver 13 for SQL Server\nDriver=/usr/local/lib/libmsodbcsql.13.dylib\n" >> ~/.odbcinst.ini


Serialize submodels as unions without erroring

This resolves the issue discussed in https://github.com/pydantic/pydantic/issues/5839. But I'm not sure it's the right way to handle it. Looking for feedback before fixing tests/etc.

replaced by #632


Documentation items for 0.3.0

What I am trying to do...
Clean up documentation after #58, which needs to land to unblock other things.
How I would like to learn how to do it...

walk through each page of https://ipyforcegraph--58.org.readthedocs.build/en/58/

land when #58 is squashed
note fixes here

per-page

[x] README.md

[x] update screenshots

[x] CONTRIBUTING.md

[x] clean up title to just "Contributing"

[ ] examples/Shapes.ipynb

[ ] finish up the demo controls

[x] reference/forces.html

[x] fix some dangling "" characters after node`, etc.

How the project might keep the docs accurate...

maybe find some patterns that can be statically checked
maybe add some more CLI/robot tests of notebooks

remove old future notebooks that don't work anymore

Reopening as we didn't get everything: moving to next milestone.


Revert "Revert "Always evaluate the finder in driver.waitFor*()""

Reverts flutter/flutter#11451
This should go back in once we've checked the performance impact of having it reverted.
cc @yjbanov

Looks like it had no effect. TBR= @yjbanov

LGTM \o/