Can't run microsoft/aspnet interactively

From @mrichman on November 1, 2017 20:52
Can't run microsoft/aspnet interactively using docker run -it microsoft/aspnet:latest cmd.exe
Service 'w3svc' has been stopped
ERROR ( message:Cannot find requested collection element. )

APPCMD failed with error code 4312
Applied configuration changes to section "system.applicationHost/applicationPools" for "MACHINE/WEBROOT/APPHOST" at configuration commit path "MACHINE/WEBROOT/APPHOST"

 Service 'w3svc' started

Output of docker info:
Containers: 3
 Running: 0
 Paused: 0
 Stopped: 3
Images: 8
Server Version: 17.09.0-ce
Storage Driver: windowsfilter
 Windows:
Logging Driver: json-file
Plugins:
 Volume: local
 Network: ics l2bridge l2tunnel nat null overlay transparent
 Log: awslogs etwlogs fluentd json-file logentries splunk syslog
Swarm: inactive
Default Isolation: hyperv
Kernel Version: 10.0 15063 (15063.0.amd64fre.rs2_release.170317-1834)
Operating System: Windows 10 Enterprise
OSType: windows
Architecture: x86_64
CPUs: 4
Total Memory: 19.84GiB
Name: LPT1729
ID: XXWO:OECR:OYZV:QCO5:6SKW:QPGN:7ZPR:MMUA:B4JX:YRKD:6LWN:B4YY
Docker Root Dir: C:\ProgramData\Docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: -1
 Goroutines: 28
 System Time: 2017-11-01T16:51:24.8496393-04:00
 EventsListeners: 1
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Copied from original issue: aspnet/aspnet-docker#323

Any update on this?  I have the same problem.
Added Docker Support to an existing application in Visual Studio 2017, and when I try to launch a debug version of the docker container, I get this same problem.
Running Windows 10 Pro inside VMWare Fusions from Macbook Pro
OS: Windows 10 Pro
Version: 1703
Build: 15063.674
Docker Info:
Containers: 1
Running: 1
Paused: 0
Stopped: 0
Images: 6
Server Version: 17.09.0-ce
Storage Driver: windowsfilter
Windows:
Logging Driver: json-file
Plugins:
Volume: local
Network: ics l2bridge l2tunnel nat null overlay transparent
Log: awslogs etwlogs fluentd json-file logentries splunk syslog
Swarm: inactive
Default Isolation: hyperv
Kernel Version: 10.0 15063 (15063.0.amd64fre.rs2_release.170317-1834)
Operating System: Windows 10 Pro
OSType: windows
Architecture: x86_64
CPUs: 2
Total Memory: 6.835GiB
Name: AUNDERWOODVMWIN
ID: OLCF:LZ5Y:TK67:GOKS:5PSF:M6WY:DV7E:YHPQ:RJ2X:Q7NP:GT7C:5AFE
Docker Root Dir: C:\ProgramData\Docker
Debug Mode (client): false
Debug Mode (server): true
File Descriptors: -1
Goroutines: 37
System Time: 2017-11-03T15:27:59.7896702-04:00
EventsListeners: 0
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false

@shirhatti Not sure if this directly answers your question, but you could try creating a Dockerfile and specify the entrypoint to be ["powershell"]:
FROM microsoft/aspnet

ENTRYPOINT ["powershell"]

Otherwise if you don't overwrite the entrypoint, the entrypoint is defined as ENTRYPOINT ["C:\\ServiceMonitor.exe", "w3svc"]. When you add cmd.exe as the run parameter, it is attempting to add that as an argument after ServiceMonitor.exe w3svc.
With that Dockerfile, you could then do docker build -t aspnet . and then run it interactively by docker run -it aspnet.

I had a similar issue and docker run -it --rm --entrypoint powershell aspnet worked

This worked for me. Not sure why. Mind explaining @caokennedy

I had a similar issue and docker run -it --rm --entrypoint powershell aspnet worked

This worked for me. Not sure why. Mind explaining @caokennedy

--rm  deletes the container once it's finished running, erase everything related to it and save the disk space. I usually do this when running containers interactively so they're destroyed after I exit them.
--entrypoint powershell sets the entry point to powershell, so when the container runs interactively it's in a PowerShell shell
I did this following the advise of @bsedg, who suggested changing the entry point to powershell (he suggested doing it in the Dockerfile but I just did it when running the image)


Set isManagedResource to false on any cluster updates

Test Plan
isManagedResource is persisted as false when updating cluster with backend change
https://www.loom.com/share/c89443b35aae4c36bb0aef70831b810b
isManagedResource is persisted as false when creating cluster with backend change
https://www.loom.com/share/bfa3315843bb40f3894f94a3e00f8e81
isManagedResource is persisted as false when updating cluster capacity with backend change
https://www.loom.com/share/bd240f05641f4198ac17f5d4be638a94?sid=3e69f69e-9698-49ba-a02e-605c3b712503
Cluster update still works without backend change
https://www.loom.com/share/a6158ed5feb44c8c8e077744ee33209f?sid=95f4700f-14a2-48cf-9c70-045b710a13da
Cluster creation still works without backend change
https://www.loom.com/share/18ca52987d754dff889d105187fd3d67
Cluster capacity update still works without backend change
https://www.loom.com/share/74d9ef456e2f403ca5622757d0b70401?sid=0d631b28-8a9f-4e35-8f22-861d454bf53e

For the cases:

isManagedResource is persisted as false when updating cluster with backend change
isManagedResource is persisted as false when updating cluster capacity with backend change

Should it not just pass the previous value when updating?
I know that this can only be updated if it is false, but is it a bit easier to reason about if it only sets it on "create" and otherwise persists whatever value previously existed?
I am not 100% certain on the best logic, but just thinking about how to reason through the system.
The actual set to "false" for the 2 cases update / update cluster would come from the form contents rather than hardcoded into the update function?
Is there an additional reason to hardcode "false" in update? Thanks!


Move error summary at the end of the output

The line:
ERROR  Failed to compile with 10 errors

Is above all the errors. It would be more usefull if this summary would be at the bottom of the output.

Sure, good idea!
I'll gladly accept a PR for this :wink:


update Go version aliases for Go 1.8

Go 1.7.5 https://groups.google.com/forum/#!topic/golang-announce/ilVnFZO59nw
Go 1.8 https://groups.google.com/forum/#!topic/golang-announce/7q2AdUHAuuI
Note: I was unable to test locally due to the json gem not installing native extensions on my system. UPDATE: force pushed with tests updated too.

Thanks for taking time to open this PR. We merged #961 instead, so I am closing this.
Again, thank you for your contribution!


Configuration Handler Bugfix - Fixed scope violation

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[x] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Changes

Changed handler for TEPP configurations. Was using Start-DbaRunspace & Stop-DbaRunspace. Both are internal and would hang the console when trying to change the settings

who approves? you or zippy or?


Deprecate use of omdb api

The creator of OMDBAPI has deprecated the free version of the API, now requiring an API key to use. This also means that this application doesn't work anymore unless a purchased key is appended to the end of the URL.
https://www.patreon.com/posts/api-is-going-10743518

Please send PR .


Disable ExpiredTargetSdkVersion lint check

WHAT
Disable ExpiredTargetSdkVersion lint check.
Related issue: https://issuetracker.google.com/issues/196420849
WHY
Our builds are now failing with:
Error: Google Play requires that apps target API level 31 or higher.
 [ExpiredTargetSdkVersion]
        targetSdk 30

Due to this check.
However, Wear OS apps required target API level are still 30, according to: https://support.google.com/googleplay/android-developer/answer/11926878
HOW
Add baseline lint files for each module, with only ExpiredTargetSdkVersion listed there.
Other alternatives that were tried but did NOT work:

Current disable lint option is not being taken into consideration: https://github.com/google/horologist/blob/8d1b285f5819a9cdc17185217fef4f1bdd1d041b/build.gradle#L209
Disable lint option in each module's build.gradle
Comment suppress

        //noinspection ExpiredTargetSdkVersion
        targetSdk 30

Checklist :clipboard:

[N/A] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[N/A] Update metalava's signature text files

Closing as found better approach.


layout: fix "Delete" and "Insert"

Hope this fixes issue #13, seems good to me.

Yep, looks good! Let's try.


update free courses id - Typescript

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

looks like your text editor has changed all the list formatting


#issue86

Codecov Report

Merging #87 into master will decrease coverage by 0.19%.
The diff coverage is 87.5%.

@@             Coverage Diff             @@
##             master      #87     +/-   ##
===========================================
- Coverage     83.75%   83.55%   -0.2%
  Complexity     1855     1855
===========================================
  Files            93       93
  Lines          4228     4238     +10
  Branches        451      461     +10
===========================================
  Hits           3541     3541
- Misses          451      458      +7
- Partials        236      239      +3

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/jsonvalues/ScalaImmutableVector.java
90.81% <ø> (ø)
56 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/ImmutableMap.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/MutableSeq.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/MutableMap.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/MyMap.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/ImmutableSeq.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/OpPatchAdd.java
89.47% <ø> (ø)
6 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/JavaList.java
83.62% <ø> (ø)
41 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/JavaMap.java
92.98% <ø> (ø)
41 <0> (ø)
:arrow_down:

src/main/java/jsonvalues/OpPatchReplace.java
83.33% <ø> (ø)
4 <0> (ø)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


'limits' file not found

Hello, I also can not compile the Rust crate opencv = "0.76.3"

OS Ubuntu 22.04 LTS
I installed opencv using $ sudo apt install libopencv-dev clang libclang-dev
Also, I checked that opencv works using $ python3

     import cv2
     cv2.__version__

Outputs: '4.7.0'

rustc v1.67.0

When I compile, I get this

===    /usr/include/opencv4/opencv2/core/cvdef.h:185:10: fatal error: 'limits' file not found
thread 'main' panicked at '=== Errors during header parsing', /home/myuser/.cargo/registry/src/github.com-1ecc6299db9ec823/opencv-binding-generator-0.55.0/src/generator.rs:512:17
stack backtrace:
0:     0x564ad6630c0a - std::backtrace_rs::backtrace::libunwind::trace::h34aec3ef6cd8ad7e
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/../../backtrace/src/backtrace/libunwind.rs:93:5
1:     0x564ad6630c0a - std::backtrace_rs::backtrace::trace_unsynchronized::h8035d38698d0f7a8
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
2:     0x564ad6630c0a - std::sys_common::backtrace::_print_fmt::hff968f6695a1ba22
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/sys_common/backtrace.rs:65:5
3:     0x564ad6630c0a - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h7eea0efe77acf1ec
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/sys_common/backtrace.rs:44:22
4:     0x564ad6653fce - core::fmt::write::hc553f17407eb0b48
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/core/src/fmt/mod.rs:1208:17
5:     0x564ad662dad5 - std::io::Write::write_fmt::h62e5f01a089f48c0
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/io/mod.rs:1682:15
6:     0x564ad66309d5 - std::sys_common::backtrace::_print::h52d116aff3db4cd1
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/sys_common/backtrace.rs:47:5
7:     0x564ad66309d5 - std::sys_common::backtrace::print::h9e7d2f98fb7af075
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/sys_common/backtrace.rs:34:9
8:     0x564ad663205f - std::panicking::default_hook::{{closure}}::hf74999dab2d0a95c
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:267:22
9:     0x564ad6631d9b - std::panicking::default_hook::hc11ca7d10c44c42f
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:286:9
10:     0x564ad663276c - std::panicking::rust_panic_with_hook::hdb4da1ae79c845a5
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:688:13
11:     0x564ad66324c2 - std::panicking::begin_panic_handler::{{closure}}::h02b5b35b126d5cf2
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:577:13
12:     0x564ad66310bc - std::sys_common::backtrace::__rust_end_short_backtrace::h6c6853376cf416d1
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/sys_common/backtrace.rs:137:18
13:     0x564ad6632212 - rust_begin_unwind
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:575:5
14:     0x564ad649e603 - core::panicking::panic_fmt::hfd9e949092070b66
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/core/src/panicking.rs:64:14
15:     0x564ad64e1b8a - opencv_binding_generator::generator::Generator::handle_diags::h4494e53de6b85989
16:     0x564ad64a5ce2 - opencv_binding_generator::generator::Generator::process_opencv_module::h2ed6f6e32793d3dc
17:     0x564ad64a9fca - binding_generator::main::h61103a86d77429b6
18:     0x564ad64ab3d3 - std::sys_common::backtrace::__rust_begin_short_backtrace::hebec183eaf01f78e
19:     0x564ad64a0c69 - std::rt::lang_start::{{closure}}::hf507f37f302d6e4f
20:     0x564ad662a84c - core::ops::function::impls::<impl core::ops::function::FnOnce for &F>::call_once::h9ab31282e87f134a
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/core/src/ops/function.rs:606:13
21:     0x564ad662a84c - std::panicking::try::do_call::h42ddf5b01d0b4bc7
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:483:40
22:     0x564ad662a84c - std::panicking::try::hfb70320d7386c61a
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:447:19
23:     0x564ad662a84c - std::panic::catch_unwind::h978c9edbad2bb4d4
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panic.rs:137:14
24:     0x564ad662a84c - std::rt::lang_start_internal::{{closure}}::h04ede5bd2f26b553
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/rt.rs:148:48
25:     0x564ad662a84c - std::panicking::try::do_call::ha6b9da35a0885c93
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:483:40
26:     0x564ad662a84c - std::panicking::try::h3325520cab3a642e
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panicking.rs:447:19
27:     0x564ad662a84c - std::panic::catch_unwind::h160beec6f047175b
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/panic.rs:137:14
28:     0x564ad662a84c - std::rt::lang_start_internal::h79190e3a877a769d
at /rustc/fc594f15669680fa70d255faec3ca3fb507c3405/library/std/src/rt.rs:148:20
29:     0x564ad64aa7b5 - main
30:     0x7fe1e5312d90 -
31:     0x7fe1e5312e40 - __libc_start_main
32:     0x564ad649ec75 - _start
33:                0x0 -

Can you please attach the full build log as produced by cargo build -vv?

Please find attached the output of $  RUST_BACKTRACE=full cargo build -vv
log.txt

Can you please show me the output of the find / -name limits.h on your machine? My fresh Ubuntu 22.04 container shows:
/usr/lib/gcc/x86_64-linux-gnu/11/include/limits.h
/usr/lib/llvm-14/lib/clang/14.0.0/include/limits.h
/usr/include/c++/11/tr1/limits.h
/usr/include/limits.h
/usr/include/linux/limits.h

Also I see the difference in clang arguments between your log and my container:
your log
"-isystem/usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++",

container:
"-isystem/usr/bin/../lib/gcc/x86_64-linux-gnu/11/../../../../include/c++/11",
"-isystem/usr/bin/../lib/gcc/x86_64-linux-gnu/11/../../../../include/x86_64-linux-gnu/c++/11",
"-isystem/usr/bin/../lib/gcc/x86_64-linux-gnu/11/../../../../include/c++/11/backward",

Do you maybe know why this difference?

$ sudo find / -name limits.h
/usr/src/linux-headers-5.4.0-137/include/linux/limits.h
/usr/src/linux-headers-5.4.0-137/include/uapi/linux/limits.h
/usr/src/linux-oem-headers-4.15.0-1027/arch/arm/include/asm/limits.h
/usr/src/linux-oem-headers-4.15.0-1027/include/uapi/linux/limits.h
/usr/src/linux-headers-5.15.0-58/include/vdso/limits.h
/usr/src/linux-headers-5.15.0-58/include/linux/limits.h
/usr/src/linux-headers-5.15.0-58/include/uapi/linux/limits.h
/usr/share/virtualbox/src/vboxhost/vboxnetadp/include/iprt/nocrt/limits.h
/usr/share/virtualbox/src/vboxhost/vboxnetflt/include/iprt/nocrt/limits.h
/usr/share/virtualbox/src/vboxhost/vboxdrv/include/iprt/nocrt/limits.h
/usr/lib/gcc/x86_64-linux-gnu/7/include/limits.h
/usr/lib/gcc/x86_64-linux-gnu/11/include/limits.h
/usr/lib/gcc/x86_64-linux-gnu/12/include/limits.h
/usr/lib/gcc/x86_64-linux-gnu/9/include/limits.h
/usr/lib/llvm-14/lib/clang/14.0.0/include/limits.h
/usr/include/linux/limits.h
/usr/include/c++/11/tr1/limits.h
/usr/include/limits.h
find: ‘/run/user/1001/doc’: Permission denied
find: ‘/run/user/1001/gvfs’: Permission denied
There are a lot of "limits" files here. I do not know why the compiler doesn't find the correct one. Just a guess that maybe some path is wrong.

Can you please also show the result of the following command find / -type f -name limits?

$ sudo find / -type f -name limits | grep -v "proc"
/usr/include/c++/11/limits

Can you please try installing libstdc++-12-dev (apt install libstdc++-12-dev) package and trying the build again?

Thank you, man. I have installed this libstdc++-12-dev package, and everything compiles now.

Same issue solution worked for me.

@twistedfall, I am on Windows and I get the same error.


Accordion View

This would be a cool view, was thinking about tackling it possibly at same time as tab view refactor.

The accordion is a graphical control element comprising a vertically stacked list of items, such as labels or thumbnails. Each item can be "expanded" or "collapsed" to reveal the content associated with that item. There can be zero expanded items, exactly one, or more than one item expanded at a time, depending on the configuration.

Wikipedia

Another example, this one from wpf

I feel like your first VS Code example with one-liners can be accomplished with your treeview-in-a-table #2685 and using special formatting for the items at the lowest level.  But I take it the idea is borders around each item, like this (where the asterisks indicate a different color than the contents)?
┌────────────────────┐
│*Mail************ ˅ │
├────────────────────┤
│*Contacts******** ˄ │
├─John Smith─────────┤
│ johnsmith@e-       │
│ mail.com 555-5632  │
├─Bob Martin─────────┤
│ bobmartin@e-       │
│ mail.com 555-6128  │
├─Ricky Dylan────────┤
│ rickydylan@e-      │
│ mail.com 555-2647  │
├────────────────────┤
│*Tasks*********** ˅ │
└────────────────────┘

I feel like your first VS Code example with one-liners can be accomplished with your treeview-in-a-table #2685 and using special formatting for the items at the lowest level. But I take it the idea is borders around each item, like this (where the asterisks around the categories indicate a different color than the contents)?

but I think the key differences between what I am proposing and tree view is that the section expanded from the title is a full view.
For example the vs code top section (Source Control) expands to reveal a panel with a text box (commit message), a button (commit) and a tree view (changes).
So it would be more like TileView.
I starting initial exploration on my accordion branch.  Heres a draft PR of my initial thoughts:  https://github.com/gui-cs/Terminal.Gui/pull/2712
But it's not laying out correctly.  I think adding/removing views is not refreshing/working correctly.
I'm also leaning towards making the 'header' just text instead of a full View.

Oh, for some reason my brain blocked out source control as separate from the accordion.
VS Code's behavior is interesting, how individual accordion views can be set to grab all available vertical space, shoving the entries below to the bottom (which seems to be the default), or only as much as the view needs but not being user-resizeable (e.g., "Open Editors" in the Explorer accordion).
Were you planning on tackling drag-to-reorder as well?

Oh, for some reason my brain blocked out Source Control as separate from the accordion.

Its easily overlooked ;)

individual accordion views can be set to grab all available vertical space, shoving the entries below to the bottom (which seems to be the default), or only as much as the view needs

Nice, I hadn't noticed that.  Perhaps theres an optional MaxHeight for sections.  I can look into making that a core feature.

Were you planning on tackling drag-to-reorder as well?

No, not initially anyway.  I think a good first pass feature set is the ability to expand and collapse and drag resize when 2+ are expanded.  Re-ordering could always come later.

Super basic but here's the idea.
The complicated bit is going to be around sizing e.g. the table view here is set to fill.  So the Accordion needs to let it fill but only within its section.

Looking great so far!

I think this is addressed via ExpanderButton which I added to the AdornmentEditor in UI Catalog.
https://i.imgur.com/4KWoMIq.gif
This works with any View, as long the View has a Border.Thickness.Top >= 1. If the View's Title is blank, and Border.LineStyle = LineStyle.None it works exactly as described above.

More progress (in #3798):


Sicelor without matched Illumina data

Hi,
now that the error rate felt down with nanopore technology, is there a way to run sicelor without having matched Illumina data, please ?
Thank you for this very cool tool !

Please visit Sicelore-2.1 github repository for this issue:
https://github.com/ucagenomix/sicelore-2.1


feat(query): add wildcard '*' query

Using * as the query term presents a shortcut for the user to match all items in the library.

Codecov Report

Merging #28 (133d383) into master (1c2fa07) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   98.38%   98.40%   +0.01%
==========================================
  Files          22       22
  Lines         993     1001       +8
==========================================
+ Hits          977      985       +8
  Misses         16       16

Flag
Coverage Δ

integration
57.44% <25.00%> (-0.27%)
:arrow_down:

unit
91.80% <100.00%> (+0.06%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.


Tip: Indenting blocks of code on atom in MAC OS

I looked this tip up and am finding it very useful for this week's assignment. If you want to indent entire blocks of code in atom on mac os, all you need to do is highlight the code and press the tab button.
If you you want to decrease the indent (move all the code to the left) highlight the block of code and press shift + tab.

Cool! Thanks for the tip, I'll have to try it!

Another cool package for Atom to get is called “beautify”
https://atom.io/packages/atom-beautify https://atom.io/packages/atom-beautify
You can invoke it from the command line in atom, and it will automatically indent your code correctly, and make it look “pretty”. It takes a heavy-hand approach though, so it is certainly not great for every instance.

Thanks @anon-music-and-art - I will give that a try.

Currently using both of these suggestions and they're super helpful. thanks!


Forgot Password

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
Currently, forgot password is not implemented properly.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Forgot password can be implemented using verification links
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Describe how this feature will be useful to our readers.
A clear and concise description of how this feature will be useful for our  other readers.
Additional context
Add any other context or screenshots about the feature request here.

Yup, @ashutoshkrris assigning to you.


feat: remove redundant code and add styling

Fixes Issue
Closes #1410
Changes proposed
@rupali-codes
There was redundant code in the Tailwind styling, so even if you don't need the border, you can definitely merge this.

duplicate code

If you want a border, let me know so that I can make the change with GitHubStarButton. If you don't want a border, then I will remove the extra code.
Please let me know your decision.

Hi @Anmol-Baranwal! :) I think a border would look good. Let's see what @rupali-codes thinks.

Yeah, I agree. I will make the changes once she responds.

Fixes Issue
Closes #1410
Changes proposed
@rupali-codes
There was redundant code in the Tailwind styling, so even if you don't need the border, you can definitely merge this.

duplicate code

If you want a border, let me know so that I can make the change with GitHubStarButton. If you don't want a border, then I will remove the extra code.
Please let me know your decision.

no, we don't need border

Okay, I will make the changes accordingly.
Also, I would like to know why a border is not preferable, apart from the distractions of a CTA, which I totally understand.

@rupali-codes
Done


Add integration tests from aqua-playground to CI

This will avoid a lot of errors before merging PRs

There are correct integration tests in the client rn.


DataFrame.drop_duplicates raises an exception

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Ubuntu 19.10

Modin version (modin.__version__):

0.7.1+2.g4f36f23

Python version:

Python 3.7.5

Code we can use to reproduce:

#import pandas as pd
import ray
ray.init(huge_pages=False, plasma_directory="/localdisk/gashiman/plasma", memory=1024*1024*1024*200, object_store_memory=1024*1024*1024*200)
import modin.pandas as pd

df = pd.DataFrame([["one", 1, 10],
                   ["two", 4, 20],
                   ["three", 7, 30]],
                  index=[1, 2, 3],
                  columns=['name', 'max_speed', 'health'])
print(df)

df1 = df.drop_duplicates("name")
print(df1)

Describe the problem

On Pandas I get this output
    name  max_speed  health
1    one          1      10
2    two          4      20
3  three          7      30
    name  max_speed  health
1    one          1      10
2    two          4      20
3  three          7      30

while on Modin I get an exception:
2020-02-27 10:22:13,644 INFO resource_spec.py:216 -- Starting Ray with 200.0 GiB memory available for workers and up to 200.0 GiB for objects. You can adjust these settings with ray.init(memory=<bytes>, object_store_memory=<bytes>).
2020-02-27 10:22:13,899 WARNING services.py:1365 -- WARNING: object_store_memory is not verified when plasma_directory is set.
UserWarning: Distributing <class 'list'> object. This may take some time.
    name  max_speed  health
1    one          1      10
2    two          4      20
3  three          7      30
Traceback (most recent call last):
  File "drop_duplicates_test.py", line 13, in <module>
    df1 = df.drop_duplicates("name")
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 208, in drop_duplicates
    subset=subset, keep=keep, inplace=inplace
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/base.py", line 1120, in drop_duplicates
    duplicates = self.duplicated(keep=keep, subset=kwargs.get("subset"))
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 242, in duplicated
    if len(df.columns) > 1:
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/series.py", line 227, in __getattr__
    raise e
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/series.py", line 223, in __getattr__
    return object.__getattribute__(self, key)
AttributeError: 'Series' object has no attribute 'columns'

Source code / logs

Thanks @gshimansky this bug exists when subset is not a list. Our incorrect assumption was that the values passed in would be a list. This is happening in duplicated.
As a temporary workaround, you should be able to pass subset=["name"] and it should work normally.
For the fix, in the case of a single label we should just convert a single label to a list of labels with one value.

Well, with ["name"] I get a different exception:
if __name__ == "__main__":
    import modin.pandas as pd
    df = pd.DataFrame([["one", 1, 10],
                       ["two", 4, 20],
                       ["three", 7, 30]],
                      index=[1, 2, 3],
                      columns=['name', 'max_speed', 'health'])
    print(df)

    df1 = df.drop_duplicates(subset=["name"])
    print(df1)

UserWarning: Distributing <class 'list'> object. This may take some time.
    name  max_speed  health
1    one          1      10
2    two          4      20
3  three          7      30
UserWarning: User-defined function verification is still under development in Modin. The function provided is not verified.
Traceback (most recent call last):
  File "drop_duplicates_test.py", line 14, in <module>
    df1 = df.drop_duplicates(subset=["name"])
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 208, in drop_duplicates
    subset=subset, keep=keep, inplace=inplace
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/base.py", line 1120, in drop_duplicates
    duplicates = self.duplicated(keep=keep, subset=kwargs.get("subset"))
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 246, in duplicated
    duplicates = hashed.apply(lambda s: s.duplicated(keep=keep)).squeeze(axis=1)
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 362, in apply
    if axis == 0 and result.name == self.index[0]:
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/base.py", line 3403, in __nonzero__
    self.__class__.__name__
ValueError: The truth value of a Series is ambiguous. Use a.empty, a.bool(), a.item(), a.any() or a.all().

BTW this workaround fails because of a lucky name collision between my reproducer and Modin code. If I rename column to anything other than "name", it works :)
if __name__ == "__main__":
    import modin.pandas as pd
    df = pd.DataFrame([["one", 1, 10],
                       ["two", 4, 20],
                       ["three", 7, 30]],
                      index=[1, 2, 3],
                      columns=['name1', 'max_speed', 'health'])
    print(df)

    df1 = df.drop_duplicates(subset=["name1"])
    print(df1)

The problem is with this code which doesn't allow me to have a column with name "name" in my dataframe:
            if hasattr(result, "name"):
                if axis == 0 and result.name == self.index[0]:
                    result.name = None
                elif axis == 1 and result.name == self.columns[0]:
                    result.name = None

BTW, Pandas allows not only name of column or a list of columns. It also allows a tuple of columns which currently doesn't work as well. I created a test with all combinations that I could think of:
#import pandas as pd
import ray
ray.init(huge_pages=False, plasma_directory="/localdisk/gashiman/plasma", memory=1024*1024*1024*200, object_store_memory=1024*1024*1024*200)

if __name__ == "__main__":
    import modin.pandas as pd
    #import pandas as pd
    df = pd.DataFrame([["one", 1, 10],
                       ["two", 4, 20],
                       ["two", 5, 20],
                       ["three", 7, 30]],
                      index=[1, 2, 3, 4],
                      columns=['name1', 'max_speed', 'health'])
    print(df)

    def test_func(df, arg):
        print("ARG=", arg)
        df1 = df.drop_duplicates(arg)
        print(df1)

    test_func(df, ["name1"])
    test_func(df, ("name1"))
    test_func(df, "name1")
    test_func(df, ["name1", "health"])
    test_func(df, ("name1", "health"))
    test_func(df, ["name1", "max_speed"])
    test_func(df, ("name1", "max_speed"))

and it fails for me on tuple test:
2020-03-02 12:57:15,579 INFO resource_spec.py:216 -- Starting Ray with 200.0 GiB memory available for workers and up to 200.0 GiB for objects. You can adjust these settings with ray.init(memory=<bytes>, object_store_memory=<bytes>).
2020-03-02 12:57:15,877 WARNING services.py:1365 -- WARNING: object_store_memory is not verified when plasma_directory is set.
FutureWarning: pandas.core.index is deprecated and will be removed in a future version.  The public classes are available in the top-level namespace.
UserWarning: Distributing <class 'list'> object. This may take some time.
   name1  max_speed  health
1    one          1      10
2    two          4      20
3    two          5      20
4  three          7      30
ARG= ['name1']
UserWarning: User-defined function verification is still under development in Modin. The function provided is not verified.
   name1  max_speed  health
1    one          1      10
2    two          4      20
4  three          7      30
ARG= name1
Traceback (most recent call last):
  File "drop_duplicates_test.py", line 22, in <module>
    test_func(df, ("name1"))
  File "drop_duplicates_test.py", line 18, in test_func
    df1 = df.drop_duplicates(arg)
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 196, in drop_duplicates
    subset=subset, keep=keep, inplace=inplace
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/base.py", line 1098, in drop_duplicates
    duplicates = self.duplicated(keep=keep, subset=kwargs.get("subset"))
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/dataframe.py", line 230, in duplicated
    if len(df.columns) > 1:
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/series.py", line 229, in __getattr__
    raise e
  File "/nfs/site/proj/scripting_tools/gashiman/modin/modin/pandas/series.py", line 225, in __getattr__
    return object.__getattribute__(self, key)
AttributeError: 'Series' object has no attribute 'columns'


Short Dump Downloading Variant

Hi,
I get short dump when downloading a test variant in test mode. (Note quite sure if it is because of the fact that I changed some of the check's priority from Warning to Error).
This is the first time that I'm trying to download a check variant.

SAP_BASIS	750	0006	SAPK-75006INSAPBASIS	SAP Basis Component

what is the value of iv_class ? the upload download is quite a hack, and some classes are special, so try excluding it if possible

Thanks Lars, the class is CL_CI_TEST_CRITICAL_STATEMENTS.

okay, cool, then we can change ZCL_AOC_CHECK_100 so it works with the upload download, but requires some debugging

If I get a chance a bit later I'll debug and get back to you if I find something.....

I'm fixing it

try again,
https://github.com/larshp/upDOWNci/commit/cd6b063d4c415dd94bc3c3ec12fd71f9de02501a
https://github.com/larshp/abapOpenChecks/commit/5217eacbaa95f003e442ce71c59dc03155a171b0

Thanks Lars, the issue is resolved.


Ferrodri/agora 1599 fix balance and voting power are getting a lot of time to

Hi Stepan,
Could you please take a look at this refactor? While solving another issue I saw balance was taking a lot of time to load and due to the requests not being run concurrently sometimes it took up to 10 seconds. I am attaching two screenshots of before and after.
I think this is a good optimization due to the nature of server actions and handling all the concurrency in the server. Let me know what you think!

Lastly, here is the preview link: https://agora-next-gyuueimo5-voteagora.vercel.app/

PR-Codex overview
This PR focuses on refactoring the useConnectedDelegate hook and adding a new function fetchConnectedDelegate in the actions.ts file.
Detailed summary

Added fetchConnectedDelegate function to fetch delegate, advanced delegators, and balance of a connected delegate.
Refactored useConnectedDelegate hook to use the new fetchConnectedDelegate function.
Removed unused imports from useConnectedDelegate.ts file.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

The acceptable loading time is 2s
Generally, we should strive for 1s though


be_valid_types doesn't seem to be able to catch exception

I'm migrating a unit test for a ruby type from an older rspec version.
If I'm not mistaken, the following should be equivalent:
old style
expect { described_type.new(name: 'test', force: 'nope') }.to raise_error(Puppet::Error)

new style
expect {be_valid_type.with_set_attributes({:force => 'nope'})}.to raise_error(Puppet::Error)

However, while the former works as expected (test succeeds), the latter fails with:
     Failure/Error: expect { be_valid_type.with_set_attributes({:force => 'nope'})}.to raise_error
       expected Exception but nothing was raised

Hi @faxm0dem,
That matcher does raise exceptions on invalid parameter values and is tested here (the fake type being tested can be viewed here). Can you provide a more complete example to reproduce the scenario you're seeing?

you can check travis for the above failure for more details:
https://travis-ci.org/ccin2p3/puppet-cpan/jobs/652374296?utm_medium=notification&utm_source=github_status

I guess I forgot the should statement (sorry, rspec is kind of black magic to me)


[kotlin] Add 1.7

https://github.com/JetBrains/kotlin/releases/tag/v1.7.0

Given that I don't see any older branches getting a release after a new branch exists ( https://kotlinlang.org/docs/releases.html#release-details ) isn't it safe to assume that once a new branch is released the older branch is end of life?

@BiNZGi nope you are wrong :

Sorry i tagged wrong person ,
@Evernow

Sorry i tagged wrong person , @Evernow

My bad, you're correct.


Q: Excessive “consul: Health changed to #XXX” messages in fabio's log

Hi,
it feels awkward to ask here but it seems to be the right place…
I’ve got Fabio up and running without any issues except, that I’m getting very frequent log entries like this:
Dec 12 18:39:41 c-b4fa fabio[25680]: 2016/12/12 18:39:41 [INFO] consul: Health changed to #54416876
Dec 12 18:39:41 c-b4fa fabio[25680]: 2016/12/12 18:39:41 [INFO] consul: Health changed to #54416877
Dec 12 18:39:43 c-b4fa fabio[25680]: 2016/12/12 18:39:43 [INFO] consul: Health changed to #54416882
Dec 12 18:39:43 c-b4fa fabio[25680]: 2016/12/12 18:39:43 [INFO] consul: Health changed to #54416883
Dec 12 18:39:44 c-b4fa fabio[25680]: 2016/12/12 18:39:44 [INFO] consul: Health changed to #54416884
Dec 12 18:39:48 c-b4fa fabio[25680]: 2016/12/12 18:39:48 [INFO] consul: Health changed to #54416894

They usually come at least once per second.  Is that normal or is something weird about our Consul/Vault/Nomad/Fabio setup?
TIA, Fabio’s great!

Check if one or more of your services have flapping or lagging health checks.
Frank Schröder

On 13 Dec 2016, at 04:44, Hynek Schlawack<EMAIL_ADDRESS>wrote:
Hi,
it feels awkward to ask here but it seems to be the right place…
I’ve got Fabio up and running without any issues except, that I’m getting very frequent log entries like this:
Dec 12 18:39:41 c-b4fa fabio[25680]: 2016/12/12 18:39:41 [INFO] consul: Health changed to #54416876
Dec 12 18:39:41 c-b4fa fabio[25680]: 2016/12/12 18:39:41 [INFO] consul: Health changed to #54416877
Dec 12 18:39:43 c-b4fa fabio[25680]: 2016/12/12 18:39:43 [INFO] consul: Health changed to #54416882
Dec 12 18:39:43 c-b4fa fabio[25680]: 2016/12/12 18:39:43 [INFO] consul: Health changed to #54416883
Dec 12 18:39:44 c-b4fa fabio[25680]: 2016/12/12 18:39:44 [INFO] consul: Health changed to #54416884
Dec 12 18:39:48 c-b4fa fabio[25680]: 2016/12/12 18:39:48 [INFO] consul: Health changed to #54416894
They usually come at least once per second. Is that normal or is something weird about our Consul/Vault/Nomad/Fabio setup?
TIA, Fabio’s great!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@hynek As for the awkwardness see my comment on #203. Please feel free to ask at any time.

@hynek I'm going to close this issue for now. Feel free to comment and I'll re-open it if necessary.

yes thank you, I’m still investigating :-/
nothing is flapping but i'm not sure what a lagging health check would be?  in any case this is probably not a fabio issue…
happy holidays anyways. :)

Hi, I've seen this exact same behaviour on my cluster. Just to throw some ideas, I've been debugging and found that is nomad the one causing this.
The check that is performed on nomad is on /v1/agent/servers which outputs the servers of the cluster.
The problem is that this array is not ordered and changes randomly on each request, so when there's more than one the check becomes the one causing the blocking query to end.
I'll open an issue on nomad to see what they think about this.

@jorgemarey I'll ping them as well. Thx.

This PR should fix the issue: https://github.com/hashicorp/nomad/pull/3214


[xc7] race condition in tests

when running make all_xc7 -j40 the tests fail with:
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] Removing direcory 'ff_ce_sr_17_fdpe_proof'.
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] Copy 'ff_ce_sr_17_fdpe/artix7-xc7a50t-basys3-roi-virt-xc7a50t-basys3-test/top_bit.v' to 'ff_ce_sr_17_fdpe_proof/src/top_bit.v'.
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] Copy 'ff_ce_sr_17_fdpe.v' to 'ff_ce_sr_17_fdpe_proof/src/ff_ce_sr_17_fdpe.v'.
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] Copy 'ff_type.v' to 'ff_ce_sr_17_fdpe_proof/src/ff_type.v'.
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] engine_0: abc pdr
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] nomem: starting process "cd ff_ce_sr_17_fdpe_proof/src; /symbiflow-arch-defs/build/env/conda/bin/yosys -ql ../model/design_nomem.log ../model/design_nomem.ys"
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] nomem: ERROR: No matching port in gate module was found for \led!
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] nomem: finished (returncode=1)
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] nomem: job failed. ERROR.
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] summary: Elapsed clock time [H:MM:SS (secs)]: 0:00:00 (0)
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] summary: Elapsed process time [H:MM:SS (secs)]: 0:00:00 (0)
SBY 10:55:55 [ff_ce_sr_17_fdpe_proof] DONE (ERROR, rc=16)

This is because fasm2bels produces empty top_bit.v (actually empty top.bit.fasm and then empty top_bit.v):
module top();
endmodule

The fasm2bels' input file (top.bit) is correct (at least when I checked it after the fail).
I rerun the tests a few times and they were always failing on one of the ff_ce_sr*
This looks like a race condition

@mithro @litghost PTAL

I have replicated this issue when using the Makefile CMake driver + a VM.  I've not replicated on my own machine or on a VM using the Ninja CMake driver.  I cannot explain why Makefile + VM is different than the other conditions.

can we switch to ninja in kokoro?

can we switch to ninja in kokoro?

Yep, https://github.com/SymbiFlow/symbiflow-arch-defs/pull/991

I believe this has been resolved


Silent refresh with implicit flow do not reload all the claims

Describe the bug
As the title said when the auth_token expire and the client oidc and call a silent refresh it does not reload all the claims, but only the claims related to the auth_token
To Reproduce
Steps to reproduce the behavior:
1, In AuthConfig set a low timeoutFactor (like 0.05)
2. Log in the angular app and load the userProfile
3. Check the claims
4. wait for a silent refresh
5. Re-check the claims
Expected behavior
All the claims must be reloaded
Desktop (please complete the following information):

Browser Chrome
Version 8.0.4

Quick question: would you consider this possibly related to- or even a duplicate of #514?

Hi,
I forgot to mention in the post:
This problem occur in a project that use implict-flow
I see the same problem in a project with Code flow.
I doubt it is related to #514 because the problem (bug?) is different. In my case it does not reload the claims at all, if I force the reload the claims are correct
A possible workaround is this:
this.oauthService.events.subscribe(event => {
            if (['silently_refreshed', 'token_refreshed'].indexOf(event.type) > -1) {
                this.oauthService.loadUserProfile();
            }
        });

But i am not sure it's the best way to fix the problem.

The real problem is that on automatic refresh the storage (and so claims) are overwritten with id token claims.
When only very few claims live in Indentity token and most of them are loaded through userprofile endpoint that cause a problem.

I would reload the user profile after each token refresh in that case. Otherwise, you have a mix of old and new claims. Hence, @lfongaroScp's solution looks good for me.


Fix Image becoming unclickable after using double tap zoom

Fixes https://github.com/saket/telephoto/issues/18

Can you rebase on trunk so that this can be merged? It should be a fast forward if you squash your commits into one.


BitcoinAzuki Collection

Bitcoin Azuki Colletion.
10000 byte perfect inscriptions free to mint.
https://bitcoinazuki.org/

@MillHaus33 Let us know if there's anything you need from our side.


Renaming

moved the readme updates to a separate PR: https://github.com/opendifferentialprivacy/smartnoise-core/pull/306


doc: add docs around moving to step-by-step migrations

Resolves https://github.com/simolus3/drift/issues/2601

@simolus3 - hopefully this gives you a good starting point. Happy to make changes - will leave it to you to merge!
Thank you - and thank you for your efforts on this project!


Avoid requesting an Avatar image bigger than what the template asks for

Hopefully fixes #16287
It makes NO sense to request an avatar of 1160x1160 when it is rendered MUCH MUCH smaller.
See https://try.gitea.io/strk and right-click -> open image in new tab, then check the URL, is formed with ?size=1160 now. I think this PR makes it ask for a size=256 instead (default 128, sizeFactor 2)

@strk https://github.com/strk/gitea/pull/1  makes this configurable.

Codecov Report

Merging #17422 (5ae0ac9) into main (f58e687) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #17422      +/-   ##
==========================================
+ Coverage   45.28%   45.30%   +0.01%
==========================================
  Files         820      820
  Lines       90932    90932
==========================================
+ Hits        41179    41195      +16
+ Misses      43194    43184      -10
+ Partials     6559     6553       -6

Impacted Files
Coverage Δ

models/avatars/avatar.go
31.76% <ø> (ø)

modules/queue/workerpool.go
49.23% <0.00%> (-1.91%)
:arrow_down:

modules/queue/queue_channel.go
95.00% <0.00%> (-1.67%)
:arrow_down:

modules/queue/queue_bytefifo.go
59.28% <0.00%> (-0.60%)
:arrow_down:

services/pull/pull.go
41.78% <0.00%> (-0.41%)
:arrow_down:

models/issue_comment.go
53.04% <0.00%> (+0.56%)
:arrow_up:

models/repo_list.go
84.01% <0.00%> (+0.81%)
:arrow_up:

models/notification.go
65.73% <0.00%> (+0.86%)
:arrow_up:

modules/notification/ui/ui.go
62.31% <0.00%> (+1.44%)
:arrow_up:

modules/process/manager.go
82.94% <0.00%> (+1.55%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

Thanks @zeripath - I've merged your PR

Can we make default more close to current behavior, for example 3?

Yes, default should be 3 for reasons outlined in https://github.com/go-gitea/gitea/pull/17951.

I put a pr against @silverwind's to add configuration - I'd be happy to approve that or if you want to make the suggestions against this pr to make the default 3 I think that would be acceptable to.

https://github.com/go-gitea/gitea/pull/17951 is ready again.

Oh, I see the different, then we use #17951  😊

Thanks guys !


Before a Postgres table is created physically, allow columns drag&drop to change their order

Is your feature request related to a problem? Please describe.
While designing a Postgres table in UI, and before it is created, I would like to be able to change column orders. It is safe to allow that because the table does not exist yet, so the user should be free to change the order of columns.
Describe the solution you'd like
I would like to be able to drag&drop columns in the UI before the table is created physically in database.
Describe alternatives you've considered
The are two alternatives, both tedious and time-wasting.

Once I finish designing the table in the UI, copy the CREATE SQL, reorder the columns manually in the query, and execute the query.
Re-create the table in the UI from scratch.

Additional context
Order of columns is important because it will make the table take more or less space depending on the alignment padding.
Every data type has specific alignment requirements:
https://www.postgresql.org/docs/current/catalog-pg-type.html
You can minimize space lost to padding between columns by ordering them favorably. The following (extreme) example would waste a lot of physical disk space:
CREATE TABLE t (
    e int2    -- 6 bytes of padding after int2
  , a int8
  , f int2    -- 6 bytes of padding after int2
  , b int8
  , g int2    -- 6 bytes of padding after int2
  , c int8
  , h int2    -- 6 bytes of padding after int2
  , d int8)

To save 24 bytes per row, use instead:
CREATE TABLE t (
    a int8
  , b int8
  , c int8
  , d int8
  , e int2
  , f int2
  , g int2
  , h int2)   -- 4 int2 occupy 8 byte (MAXALIGN), no padding at the end

This is elaborated in great detail in many articles. For more, see this article at StackOverflow: https://stackoverflow.com/a/7431468/335304

Thank you for your suggestion.

I'm closing this as duplicate of https://github.com/dbeaver/dbeaver/issues/16411


Use govulncheck to check security vulnerabilities

It might be a good idea to have a make target that runs govulncheck and/or have a CI workflow to run it on the project periodically.
https://github.com/golang/vuln
Note that this command is available only from Go 1.18.

I will take care of this.

Maybe we can run this only on PRs with dependency changes, ie https://github.com/apache/camel-k/blob/main/.github/workflows/coverage.yml#L22-L23 (instead we just checkout go.mod and go.sum). WDYT?

It could be a good idea for the PR part, but I fear we will run in the case when it's always in warning if we find ourself in a case where it is a vulnerability from one of the golang libs that come with the golang version. It would be a clear signal that we need to upgrade golang but it would mean having check errors for PRs before the upgrade is done.

Considering all the importance security is getting lately, I'd try to do that on the PR. If we will in a situation where we cannot stay after it, we can easily turn the action into a nightly check. Or we can do the other way around. Feel free to do the way you prefer. Once the action is in place is relatively easily to adjust the process.


returns html

i added the max file size for php to 40m in /etc/php/apache2/

now it just returns domain/u/

Could you please provide context, such as excerpts from the files you have changed and screenshots of error messages? Could you please also provide the output of phpinfo()?

this is the info https://furry-pics.club/info.php ill let this up till this is not needed anymore, my config looks like this https://hastebin.com/kahihocaro.xml these are my sharex settings: https://gyazo.com/e3931efd0664af73667980d3997b73a5 https://gyazo.com/0b38a068604e19606378912b3aa908ce
and this is the error: https://gyazo.com/dafecf79f70e2b693e463221904dab05

Doesn't look that you have changed them in settings.
https://gyazo.com/1284ecdada4c96abb367074d15167463
https://gyazo.com/5fd7671abcb383b4f515d782514ebae4

Where do I edit that tho I just find php.inis in /php/7.2/ and then apache2 folder?

In php info shows all paths to ini files:
https://gyazo.com/85be99fd9c083b228abc8b548bcfeca2

oh thanks im really new im going to edit it and gonna check if it works

https://gyazo.com/c93d8ef26cbd239d81ff99a555720595 it dosent seem to update

nvm apache2 restart fixed it i guess

it still does the same tho


Example "larger.hs" doesn't compile in stack lts-12.21

I haven't investigated this a lot yet, but in my app, it looks like trying to use the env var-driven config is a recipe for build errors. I'm following along with the larger.hs example. Removing the typeclass instance for DefConfig results in a missing instance, having DefConfig but no FromEnv is fine (until you want to parse config from env vars), and having DefConfig/FromEnv without a definition for fromEnv doesn't change the behavior.
I have not tried multiple LTS resolvers yet. If there's a particular resolver I should use, I'm open to suggestions. I also have not double-checked the official library's documentation. Even if there's relevant info in upstream documentation, we should work on making sure the examples compile for reasonably recent stack setups.
Here's a sample of what happens when compiling the example larger.hs straight from this repository in my project (As close as I can easily get to a minimal, complete, verifiable example):
user@host ~/project $ stack ghc larger.hs
[1 of 1] Compiling Main             ( larger.hs, larger.o )

larger.hs:18:13: error:
    • Overlapping instances for DefConfig LargeAppConf
        arising from a use of ‘gFromEnvCustom’
      Matching instances:
        instance Default α => DefConfig α -- Defined in ‘Magicbane’
        instance DefConfig LargeAppConf -- Defined at larger.hs:12:10
    • In the expression:
        gFromEnvCustom
          Option {dropPrefixCount = 0, customPrefix = "LARGEAPP"}
      In an equation for ‘fromEnv’:
          fromEnv
            = gFromEnvCustom
                Option {dropPrefixCount = 0, customPrefix = "LARGEAPP"}
      In the instance declaration for ‘FromEnv LargeAppConf’
   |
18 |   fromEnv = gFromEnvCustom Option { dropPrefixCount = 0, customPrefix = "LARGEAPP" }
   |

I'm familiar with basic Haskell, and very new to Magicbane (and all of its dependencies), but let me know if there's anything else I can do to help with this.

You're on magicbane 0.3.0, which is old. Unfortunately, magicbane is not currently in Stackage. You have to use extra-deps:
extra-deps:
  - magicbane-0.4.0
  - wai-middleware-metrics-0.2.4
  - monad-metrics-<IP_ADDRESS>
  - rio-orphans-<IP_ADDRESS>
  - ekg-wai-<IP_ADDRESS>
  - ekg-json-<IP_ADDRESS>

(these other dependencies are required with the latest nightly, maybe you need less/more — try just magicbane and see what stack tells you.)

For now, just add the proper version of magicbane as an extra dep did the trick. I'll have to keep track of that list in case I bump into something later. Out of curiosity, why was magicbane removed from stackage? Is there something I should be working on/looking into in order to get magicbane back into stackage?

Looks like there's some problem with the ekg dependency:
magicbane < 0 # via ekg-wai
ekg-wai < 0 # via ekg-json
ekg-json < 0 # via aeson-<IP_ADDRESS>

Hmm, OK. That sounds like a lot of follow-up. As nice as it'd be to get magicbane back into stackage, I'm going to try to focus on the app I'm working on for now. Thanks for your help.

Yeah, I hope someone will fix ekg. Or I can drop it, it's not like anyone actually measures anything with metrics, right? :D


Add resources configuration

What this PR does / why we need it: Adds the option to configure compute resources in the ark container.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # https://github.com/helm/charts/issues/7342
Special notes for your reviewer: @domcar @unguiculus Hi, I hope you find this PR useful and constructive. I think having the option to configure compute resources is a necessity for community charts. I understand that ark can be bursty in its resource utilization, so I thought it best to not include commented out defaults and leave that option completely open to chart consumers.

/ok-to-test

/lgtm

/assign @seanknox

/approve


Improvements to the {{excerpt}} helper

It's unquestionable, given the vast number of issues, PRs, forum posts, support requests and other mentions of our {{excerpt}} helper, that it's leaving a lot to be desired.
Yet, as demonstrated by the wide range of different ideas on how it should be improved, it's hard to find consensus on what 'better' actually looks like. Having looked through the issues, discussions, PRs and what themes are currently using, there are two broad categories into which the concerns fall into: the first being improving the excerpt that Ghost generates from the content, and the second is adding features for custom excerpts.
Custom excerpts are a niche requirement, and we want to focus our efforts on making apps a possibility so that it is possible to add an excerpt field, or a subtitle field, or a standfirst field or whatever custom field suits your use-case, rather than adding these things to core. Therefore this issue exists purely to address the former - improving the generated excerpts.
There are two key things which makes the current {{excerpt}} helper's output quite undesirable:

It cuts off mid-sentence (ugly)
It strips all formatting (confusing)

The formatting problem has meant many themes and Ghost users are using {{content}} instead of the {{excerpt}} helper. This is not ideal as it outputs images and other media that don't really make sense for an excerpt as well as resulting in the need for some sort of appending feature to make it possible to display read more links. It also still doesn't solve the cut off problem.
To solve the cut off problem, we want to introduce {{excerpt paragraphs="X"}}, which will return the first X text paragraphs from the content.
To solve the formatting problem, it makes sense to change the helper so that it leaves valuable links and text formatting in place, rather than stripping all HTML.
The combination of these two things working in tandem should lead to a better excerpt. The media, script and all other non-formatting tags should be stripped from the content first, and then from the remaining text content we can return as many paragraphs as the {{excerpt}} helper requires.
Moving forward, I think the {{excerpt}} helper's default should also be changed from words="50" to 'paragraphs="1"`.
When processing HTML in this way, it's important to do it in such a way that bad HTML doesn't trip up the code. At the moment we make heavy use of the very smart downsize library for truncating HTML, however the excerpt helper does it's own brute force stripping of HTML. Therefore this is likely to need a bit of a rethink.
The following is a list of elements that will be permitted in excerpts:
a, abbr, b, bdi, bdo, blockquote, br, cite, code, data, dd, del, dfn, dl, dt, em, i, ins, kbd, li, mark, ol, p, pre, q, rp, rt, rtc, ruby, s, samp, small, span, strong, sub, sup, time, u, ul, var, wbr
This list was generated from https://developer.mozilla.org/en/docs/Web/HTML/Element, and includes all block and inline text formatting elements. Once all other elements are removed, the first X paragraphs should then be returned, not including any empty paragraphs.
In the long term, the excerpt tag whitelist will become extensible via a filter, so that extensions to the editor can also declare additional elements that should appear in excerpts (I'm thinking of things like MathML here).
In the short term, the next step here is to review the downsize library and determine whether these features can be added or whether this needs a bit of a re-think.

not including any empty paragraphs

That means paragraphs that were stripped empty, ie

 -(remove unallowed tags)->

 wouldn't count towards the paragraph quota

Exactly that :+1:

it's cool to see this task become more formalized. I'm not going to try to be involved in any implementation, but i'm familiar with downsize and worked on the earlier attempt at this and I'd be happy to talk to anyone who picks this up.
Some food for thought that I think could be useful:

what type of 'bad html' could make it to the excerpt helper post markdown encoding? is there any direction w/r/t to this aside from making the system robust?
does anything other than a
 count as a paragraph? i.e. does the markdown parser output
 or  blocks that are not
 wrapped.
there's a good chance that downsize could be extended to support what's described here, or would make a good starting point, but if this task only involves counting
 tags, it might just make the task heavier.
good luck!

what type of 'bad html' could make it to the excerpt helper post markdown encoding? is there any direction w/r/t to this aside from making the system robust?

I mentioned 'bad html' purely as an indicator that replacing downsize with a regex would be undesirable, I'd like to keep the existing functionality we have in terms of handling that at least a little bit.

does anything other than a  count as a paragraph? i.e. does the markdown parser output  or  blocks that are not  wrapped.

This is a really good question, and one that needs a little bit of experimentation.
 definitely counts as a paragraph - the way showdown does this is put a
 inside it which is somewhat odd. I'd suggest a simple rule-of-thumb would be any of the permitted excerpt tags (which includes
 and ) should be considered a paragraph if they exist at the paragraph level.

there's a good chance that downsize could be extended to support what's described here, or would make a good starting point, but if this task only involves counting  tags, it might just make the task heavier.

I think it definitely makes a really great starting point, and I really hope we can continue to extend it to support more advanced cases.

@ErisDS Is there any alternative, recommended way to control or curate what is displayed on an index page? Wether you call it an excerpt, summary or a standfirst.
For example, is there a way to add a standfirst field that is populated separately and is displayed instead of an excerpt?
FWIW from my experience, in many larger organizations (publications), there is a need for tighter curation and control over appearance.  Having a post break at an automated point to provide a preview would not fly and typically the desire is either to place the "break" manually, or if to take it step further to write the copy separately. For larger organization, this might even be a separate editor that writes the preview paragraph.
In my case, I'm migrating to ghost for my personal blog and am planning on writing both extensive technical posts that appeal only to developers and also opinions about the tech industry which appeal to a larger audience. I would prefer to break the more technical posts before any of the tech speak or code starts and just provide a small intro, to allow any non-interested reader to skip over it and to prevent it from alienating non-techies.
Even though I'm just checking out Ghost for my personal blog, I have worked in tech / development for large content publishers for many years and this seems to be pretty consistent.
(Perhaps the decision is that ghost is tailored to the more individual blogger, amateur publications and small orgs? But from everything I'm seeing about Ghost thats not a formal decision).

Hi @galori, appreciate your feedback. As I have attempted to explain in the issue here - the problem with the concept custom excerpts is that everyone wants or means something different when they think about how it might work. That makes it difficult to implement a 'one size fits all' solution.  We have a wishlist where feature requests can be posted and voted on - both a custom field and a more-tag style cutoff have been proposed and neither has much traction.
It seems you've already opted to patch Ghost with the more tag style option. Other options is to add an HTML block with a class to the top of your post, and use JS or CSS to show/hide that part of your content as and when you like, or to tag your technical posts with a particular tag, and show a much shorter excerpt for posts with that tag.

For those who want manual control over excerpts, the wishlist item can be found here:

http://ideas.ghost.org/forums/285309-wishlist/suggestions/7237838-custom-excerpt-for-articles

@ErisDS Shouldnt it be a list of tags to strip, instead of a list of tags to allow? Are there more tags other than img and hr that you'd like to see removed?
Even then, I still think it is weird for the theme to dictate what an excerpt should be made of. Like "x" number of paragraphs, or "y" number of words. I feel that writers should be able to decide what goes in it and what doesn't. Be it by specifying a special tag in the post after which the content is stripped, or perhaps by adopting a rule like "everything before the first heading goes in the excerpt".
I agree that most of the time excerpts are only the first paragraph of the post, but specifying "1" paragraphs at the theme level is not flexible for the end user.

This issue exists purely as a collection of the information I gathered on how our existing excerpt feature should be improved. Custom excerpts are a completely separate feature & you can vote for them on the wishlist.
As for whether or not we should whitelist or blacklist tags, the issue specifies a clear whitelist which includes only tags that would produce sensible markup inside a small block area, which is what an excerpt is. There are many, many tags that do not make it into the list, including everything to do with tables, forms and structure.

In PR 6706 I am addressing as much of the above as I can.

@ErisDS i have just added the html stripping feature that you requested in PR 6706 and it works on my personal blog. Travis is building it now.
Also, I think that we should definitely keep downsize. It works well, and it keeps the html safe. Plus, it already has paragraph rounding. it could be forked and edited to include sentence rounding.
I think the biggest concern with writing something else from scratch is load time. Most ghost themes heavily use {{excerpt}} or {{content}}, and putting bad loops or something like that would seriously increase load time. I'm going to fork downsize (guscaplan/downsize) and try to implement paragraphs and sentence rounding. I should have something by monday.

hello! all of the things that are mentioned here (and more!!!) are in my latest pull request. This PR adds html splitting, append options, and sentence rounding!!!! right now sentence rounding only works if you specify the length in characters, not words. e.g. {{excerpt characters="140" sentenceRounding="true"}} will work but {{excerpt words="50" sentenceRounding="true"}} will just skip sentence rounding.

@GusCaplan  I think the biggest feature to be desired here is truncation by nbr of paragraphs. I tried starting with the original downsize, but I found the changes to be non-trivial and the code wasn't very clean in any case, so in my attempt I changed it dramatically. If you want, you can take a look at what I did and perhaps you'll find a better way to do the same thing.

Hi there,
I got an idea but it would require some extra work and probably a lot of tests:

Add a new optional custom Markdown syntax to generate a  HTML tag
Excerpt helper would pick up this tag (why not using cheerio) where ever it is and whatever its content length is
Use also cheerio to remove unwanted tags (


Specify which offset _interlockedbittestandset uses

Update the documentation to explain the offset is from the least-significant bit, and that it's zero-indexed, so 0 is the least-significant bit, and 31 is the most significant bit.
The documentation for related APIs should also probably be updated, but I wanted to get feedback on this change first.

Thanks for the update.


GitHub Actions automatic build fails

GitHub Actions automatic build fails during pre-build check.

#73 fixes this.

Hi @radar ,
Can you review this PR, please?

Fixed by #73


documentation not found

https://wavespectra.readthedocs.io/en/docs/ returns this page does not exist yet.

added the documentation in https://wavespectra-metocean.readthedocs.io/en/latest/


Nested JUnit testsuite causes duplication

Hi folks!
We've run into a problem where nested testsuites – i.e. <testsuite>s within <testsuite>s cause duplication in the Buildkite interface.
So this JUnit.xml
<?xml version="1.0" encoding="UTF-8"?>
<testsuites>
  <testsuite name="Full Suite" tests="1" assertions="6" errors="0" warnings="1" failures="0" skipped="5" time="0.03">
    <testsuite name="Unit Tests" tests="1" assertions="6" errors="0" warnings="0" failures="0" skipped="0" time="0.03">
      <testsuite name="DuplicatedTest" file="/app/tests/DuplicatedTest.php" tests="1" assertions="6" errors="0" warnings="0" failures="0" skipped="0" time="0.03">
        <testcase name="testcase1" class="DuplicatedTest" classname="DuplicatedTest" file="/app/trunk/application/tests/validator/DuplicatedTest.php" line="115" assertions="2" time="0.01"/>
        <testcase name="testcase2" class="DuplicatedTest" classname="DuplicatedTest" file="/app/trunk/application/tests/validator/DuplicatedTest.php" line="115" assertions="2" time="0.01"/>
        <testcase name="testcase3" class="DuplicatedTest" classname="DuplicatedTest" file="/app/trunk/application/tests/validator/DuplicatedTest.php" line="115" assertions="2" time="0.01"/>
      </testsuite>
    </testsuite>
  </testsuite>
</testsuites>

will result in this duplication:

whereas when the outer testsuites are removed, there's no duplication:
<?xml version="1.0" encoding="UTF-8"?>
<testsuites>
  <testsuite name="DuplicatedTest" file="/app/tests/DuplicatedTest.php" tests="1" assertions="6" errors="0" warnings="0" failures="0" skipped="0" time="0.03">
    <testcase name="testcase1" class="DuplicatedTest" classname="DuplicatedTest" file="/app/trunk/application/tests/validator/DuplicatedTest.php" line="115" assertions="2" time="0.01"/>
    <testcase name="testcase2" class="DuplicatedTest" classname="DuplicatedTest" file="/app/trunk/application/tests/validator/DuplicatedTest.php" line="115" assertions="2" time="0.01"/>
    <testcase name="testcase3" class="DuplicatedTest" classname="DuplicatedTest" file="/app/trunk/application/tests/validator/DuplicatedTest.php" line="115" assertions="2" time="0.01"/>
  </testsuite>
</testsuites>

NB: This is the plugin we're using to perform in upload in the pipeline, but happy to move this issue to another repo if it makes more sense.

Hey @tristankenney! Thanks for all your detailed info! 🙏🏻
This is interesting! I'm not sure it's a problem with the plugin yet 🤔. Can you also try using curl directly (https://buildkite.com/docs/test-analytics/importing-junit-xml#how-to-import-junit-xml-in-buildkite-not-using-a-plugin). So we can narrow this thing down :)  Thanks!

Hi @pzeballos! That was the one piece of key info I forgot — it occurs with curl as well.

Thanks @tristankenney! I'll raise it with our TA team, as it doesn't seem to be an issue with the plugin 🙂

Hey @tristankenney! The TA team shipped a fix for this, can you confirm if it's working as expected? Thanks!

hey @tristankenney! This issue should be resolved, let us know if that is not the case :) Thanks for bringing this up!


When using vinstances the cell names have "-"

Is it possible to sanitize and replace "-" for "m" from cells names added as vinstances?
example:
c.create_vinst(splitter())
gives this name:
splitter_MFalse_S28p0_X0p0002999883330545572_Y9p54709817424515e-06
could it be
splitter_MFalse_S28p0_X0p0002999883330545572_Y9p54709817424515em06
instead?
Thanks

I mean, could, but why? - is fully compliant gds cellname.

Just to be consistent since that's the way it's done for user-Provided arguments in cells.
Also tHis is breaking sOme internal tools, however we can also Go modify the iNtErnal tools. ThX!

Thanks!


Stall detector cause DOM error on SmartTV(issue #2920)

Avoid pause request when play request still in process.

The whole point of the stall detector is to trigger when we are supposed to be playing but are not.  There are cases where the readyState, playbackRate, and paused all say we should be playing but the playhead doesn't move.  So we need to poke the media pipeline to fix it and get the browser to actually play.  This basically just disables the stall detector.
You can disable the stall detector with setting the streaming.stallEnabled config to false.  I'll also update the original bug and we can discuss a better fix.

Actually, I got the condition backwards.  I still think this isn't the correct fix since this shouldn't be called for some of the conditions anyway and the error you list shouldn't cause any problems.  That function shouldn't be called when paused is true; we want to be called when readyState is any value since some browsers don't change it; and we can still get a stall at time 0.

Hi @TheModMaker,
I wrote an update on the issue itself, this issue could happen AFAIK only on the first play request, maybe the correct solution is to enable the stall detector after playing or any other idea you have to know when playback play request didn't reject by the browser.
The native implementation after play promise rejected change paused to false so we have to call pause after promise rejected, so this rejection could cause issues.

Hi @TheModMaker,
I thought about something else, expose the default config before it changed by device type, does it acceptable?

Hi @TheModMaker,
I thought about something else, expose the default config before it changed by device type, does it acceptable?


Avatar component

Added classes for Avatar component.
Minor fixes in variables.css.

Looks good! Suggestion: Add a Netlify deploy preview link or gifs and images, it makes reviewing process easy.

I haven't made a HTML docs page yet, I'll add the demo image in the upcoming PRs.
Thanks for your suggestions.

Thanks for reviewing @AnjaliDeshwani @Pratik1005 @Tarandeep-s1ngh !


Upgrade getLedgerEntries examples to latest pre-stable releases of the SDKs

A shortlist of the changes:

no need to use the git version for Python
use stellar-sdk over the now-deprecated soroban-client
use strkeys over hex IDs
use misc. other existing abstractions over hard-coding XDR stuff

cc @overcat for sanity-checking the Python stuff

Preview is available here:http://soroban-docs-pr635.previews.kube001.services.stellar-ops.com/

This article seems to be outdated. In the increment example, the COUNTER is stored in the instance (env.storage().instance().set(&COUNTER, &count)), so the code provided here cannot correctly retrieve the COUNTER value

Preview is available here:http://soroban-docs-pr635.previews.kube001.services.stellar-ops.com/

Thanks @stellarsaur and @ElliotFriend!
@overcat I think updating the examples themselves deserves a separate pass throughout the docs to line up with the latest schemas, Soroban SDKs, etc.


Extended Stats

Provides statistics of all (LB/virtual server) child objects of a given (LB/virtual server)

check

Needs #416 merged first.


Uncaught TypeError: Cannot set property '0' of undefined

I'm getting this error right out of the box, testing it with the examples, using latest version from bower, any idea why?
THanks

Hmm not sure - just put this together http://jsfiddle.net/bvmp3gqe/ using fluid example

Could it be related that my wrapper div has a fixed height?

Yep. That would cause it. It needs a flex height in order to size properly.
Sent from my Delorean

On Jun 5, 2015, at 10:55 AM, Marti Planellas<EMAIL_ADDRESS>wrote:
Could it be related that my wrapper div has a fixed height?
—
Reply to this email directly or view it on GitHub.

Hi @DrewDahlman , I'm trying to do a horizontal grid, so I need a fixed height, is there any way of doing that? How would I need to modify the library to achieve it?
Thanks!

Hi @DrewDahlman , your mansonjs is cool. However, when I create 2 grids in the same page ( #grid1 & #grid2), the Error "Cannot set property '0' of undefined" always appears despite I config the grids seperately:

$("#grid1").mason(...)
$("#grid2").mason(...)
Could you please give me some ideas about this?

Thanks!

Applying on the container position:relative fixes it for me.
The problem is caused by the calculation of a block's top offset. If positioning is not relative it will be the offset to the top of the page, instead of the container:
https://github.com/DrewDahlman/Mason/blob/758d4ed3a45e0b8a6ab8a55f923dd465386bc722/src/mason.coffee#L301


InfluxDB 09: Feature Request: JSON time values as epoch numbers instead of date strings

I am not sure this is possible now. But it would be good to have an option to return epochs instead of full JSON Date strings.
Reduces the response size and parse speed for browsers.

In InfluxDB 0.8 it was epochs not sure why this was changed. It is not a big issue for me, but would be a nice to have.

For response size, GZIP should help with that. What are you seeing in terms of the network traffic?
On parse speeds, do you have any numbers of parsing date strings vs. epochs?
This can probably be added at some point without too much work. Just curious what the level of pain is on this feature.

I do not have any benchmarks or stats but can imagine that the deserialization will take longer and take more memory. Also worried about memory in general, grafana slows down during browser GC and so need to conserve memory :)

Date string parsing is 46% slower than epoch numbers, on my browser - http://jsperf.com/date-parsing-string-vs-epoch-number

This seems pretty easy to do and I know we have a few cycles while waiting for me to finish some query engine refactoring for the new clustering design. @dgnorton, can you have a look?
I'm thinking this should be done through a single query parameter named epoch. For instance epoch=ms, which would tell us to return all times as millisecond scale epochs.
Should support ms, us - micro, ns - nano, and s - second.

That sounds great.


Error: [PDF2XML_CONVERSION_FAILURE] PDF to XML conversion failed on pdf

>>>>>>>> GROBID_HOME=C:\grobid-master\grobid-home [main] INFO org.grobid.core.main.LibraryLoader - Loading external native CRF library [main] INFO org.grobid.core.main.LibraryLoader - Loading Wapiti native library... [main] INFO org.grobid.core.main.LibraryLoader - Library crfpp loaded [main] INFO org.grobid.core.jni.WapitiModel - Loading model: C:\grobid-master\grobid-home\models\header\model.wapiti (size: 36094028) org.grobid.core.exceptions.GrobidException: [PDF2XML_CONVERSION_FAILURE] PDF to XML conversion failed on pdf file 1.pdf  	at org.grobid.core.document.DocumentSource.processPdf2XmlThreadMode(DocumentSource.java:184) 	at org.grobid.core.document.DocumentSource.pdf2xml(DocumentSource.java:133) 	at org.grobid.core.document.DocumentSource.fromPdf(DocumentSource.java:62) 	at org.grobid.core.document.DocumentSource.fromPdf(DocumentSource.java:49) 	at org.grobid.core.engines.HeaderParser.processing2(HeaderParser.java:84) 	at org.grobid.core.engines.Engine.processHeader(Engine.java:434) 	at org.grobid.core.engines.Engine.processHeader(Engine.java:410) 	at WeRe.Grobid.performFun(Grobid.java:25) 	at WeRe.MainClass.main(MainClass.java:12) [Wapiti] Loading model: "C:\grobid-master\grobid-home\models\header\model.wapiti" Model path: C:\grobid-master\grobid-home\models\header\model.wapiti
The above error shows up when I select a particular pdf. The same pdf gets processed for the header document over the web application. Can you inform as to what the error could be?

Hello which version of GROBID are you using?
Normally our fork version of pdf2xml will fix most of the PDF parsing failures. To check it, you can send me this particular pdf by email for instance.
The fact that the header-only works fine is because for headers, only the first two pages of the PDF are parsed.

Just checked for the full document with the web application. It works fine. I am using 0.42 Snapshot here.

[pdf2xml[1.pdf]] ERROR org.grobid.core.process.ProcessRunner - IOException while launching the command [bash, -c, ulimit -Sv 6242304 && C:\grobid-master\grobid-home\pdf2xml\win-64/pdftoxml -blocks -noImageInline -fullFontName  -l 2  '1.pdf' C:\grobid-master\grobid-home\tmp\c3aKfwnJye.lxml] : Cannot run program "bash": CreateProcess error=2, The system cannot find the file specified org.grobid.core.exceptions.GrobidException: [TIMEOUT] PDF to XML conversion timed out 	at org.grobid.core.document.DocumentSource.processPdf2XmlThreadMode(DocumentSource.java:157) 	at org.grobid.core.document.DocumentSource.pdf2xml(DocumentSource.java:118) 	at org.grobid.core.document.DocumentSource.fromPdf(DocumentSource.java:57) 	at org.grobid.core.document.DocumentSource.fromPdf(DocumentSource.java:47) 	at org.grobid.core.engines.HeaderParser.processing2(HeaderParser.java:76) 	at org.grobid.core.engines.Engine.processHeader(Engine.java:428) 	at org.grobid.core.engines.Engine.processHeader(Engine.java:405) 	at WeRe.Grobid.performFun(Grobid.java:25) 	at WeRe.MainClass.main(MainClass.java:12)
Switched to 0.41 and this is the error I just got. Any ideas as to what could be causing it? Thanks in advance.

pdf parsing library has been updated in version 0.4.2-SNAPSHOT, version in 0.4.1 is less robust.

So, with the 0.4.2-SNAPSHOT version and this as the error:
>>>>>>>> GROBID_HOME=C:\grobid-master\grobid-home [main] INFO org.grobid.core.main.LibraryLoader - Loading external native CRF library [main] INFO org.grobid.core.main.LibraryLoader - Loading Wapiti native library... [main] INFO org.grobid.core.main.LibraryLoader - Library crfpp loaded [main] INFO org.grobid.core.jni.WapitiModel - Loading model: C:\grobid-master\grobid-home\models\header\model.wapiti (size: 36094028) org.grobid.core.exceptions.GrobidException: [PDF2XML_CONVERSION_FAILURE] PDF to XML conversion failed on pdf file 1.pdf at org.grobid.core.document.DocumentSource.processPdf2XmlThreadMode(DocumentSource.java:184) at org.grobid.core.document.DocumentSource.pdf2xml(DocumentSource.java:133) at org.grobid.core.document.DocumentSource.fromPdf(DocumentSource.java:62) at org.grobid.core.document.DocumentSource.fromPdf(DocumentSource.java:49) at org.grobid.core.engines.HeaderParser.processing2(HeaderParser.java:84) at org.grobid.core.engines.Engine.processHeader(Engine.java:434) at org.grobid.core.engines.Engine.processHeader(Engine.java:410) at WeRe.Grobid.performFun(Grobid.java:25) at WeRe.MainClass.main(MainClass.java:12) [Wapiti] Loading model: "C:\grobid-master\grobid-home\models\header\model.wapiti" Model path: C:\grobid-master\grobid-home\models\header\model.wapiti
What could be the issue?
Am I missing any particular library or anything else perhaps?

I can't really say without testing the pdf myself.

Link to the pdf I tried with
I have tried with different pdfs giving the same error. If it adds to anything, I added some of the libraries independently as external JARS and haven't used Maven dependency. Thanks in advance towards your reply.

The error points to this statement btw:
			String tei = engine.processHeader(pdfPath, false, resHeader);
Hoping that leads to the cause of the error*

This PDF is working fine on Linux with the new pdf2xml fork. So you need to wait for a recompiled version of this new pdf2xml on Windows. I am not able to recompile it on Windows because I have no Windows machine but hopefully it will be done when releasing version 0.4.2 of GROBID by another contributor.
Here is the resulting TEI:
https://grobid.s3.amazonaws.com/NascimentoLSG-JCDL2011.tei.xml
it looks actually very nice ;)

It does indeed. I will give it a try in Linux once. Thanks.

see #161 and #166
need to recompile pdf3xml fork for Windows 64

Not to sound silly but in the crash #166, it is stated that it works well with 0.4.1 which provides an error in my case as stated above. Is it due to pdf3xml fork for Win 64?

The fact that #166 doesn't work with 0.4.2-SNAPSHOT is because pdf2xml is not recompiled for Windows - a different symptom but the same cause ;)
All these issues will disappear by updating pdf2xml.

Oh, okay. Thanks for the quick responses, @kermitt2 I will check it out in Linux. Great stuff @GROBID btw!

thanks @rathancage you're welcome!

grobid actually works out of the box in "Linux Subsystem for Windows", so you can develop in Windows, deploy and run grobid as a service in Linux and call grobid-APIs from Windows again, all on one machine, without messing around with VMs.

Dear @rathancage, @UntoterOstgote,
the new pdf2xml version for windows has been pushed yesterday, commit 75536cd4ea965fbcf7d26e12b6f43fc01a085c64.
You should be able to run grobid on Windows without problems. Please bear in mind that the reference architecture for GROBID is Linux (moreover compiling things on windows is a real pain).

Hi,
We talked today and saw that some files work, some don't on windows (both work on Unix).
Here are example files for testing:

it does not work: https://github.com/DESIR-CodeSprint/trackB/blob/38630cb788bc316e8aa374a38cb2c8ed65cdcb98/backend/src/test/resources/hube2018towards.pdf
it works: https://github.com/DESIR-CodeSprint/trackB/blob/2b07d70d2bf2d0922f62b88fc66e731accaeaea0/backend/src/test/resources/EAD-ODD_A_solution_for_project-oriented_EAD_schemes.pdf
Cheers,
Yoann


Infinite loop when using Meteor Collection Hooks

Hi there,
Has anyone used meteor-collection-hooks along with angular-meteor?
I'm having an issue where .before.update used with $meteor.object.save() results in an infinite loop. Maybe there's a better way to do this within this framework that I'm missing?

I discovered it was due to using meteor-collection-hooks and meteor-collection2 together


General spacing and aesthetics of the resulting form

Mostly, there are some checkboxes that seem like they should be on new lines, as sub bullets under existing sections, that aren't. I'll take some quick passes at these soon, once I figure out which of the DOCXs are the canonical ones.

Improved a bit. Could be better, but don't want to idle on a relatively minor point here for too long.


[SPARK-45511][SS][FOLLOWUP] fix comment in StateDataSourceReadSuite

What changes were proposed in this pull request?
Fix comment in StateDataSourceReadSuite because it is not precise and doesn't match the actual data.
Why are the changes needed?
To avoid confusion in the test.

@HeartSaVioR

Merged to master.


Add developer and enable cd for sysdig-secure-plugin

Description
Plugin: https://github.com/jenkinsci/sysdig-secure-plugin
Adding @airadier as a developer and enabling CD to automate releases.
Recent commits in https://github.com/jenkinsci/sysdig-secure-plugin/commits/master were done by @airadier and releases were being manually performed by @nestorsalceda
Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Closing and reopening to re-trigger the build.

@nestorsalceda to approve

@timja approved
Thank you!


Benchmarking script changes

Description
What is the current behavior?
Since we added filetype dimension to benchmarking there are few changes required to reflect the profiling data better
related: #972
What is the new behavior?

Added a dataset path, type, name in the profiling data.
Created a new table to store updated schmea
Modified dag_id to contain filetype

Does this introduce a breaking change?
Nope

Results are populated in table - astronomer-dag-authoring.benchmark.load_files_to_databaseV2
SELECT * FROM astronomer-dag-authoring.benchmark.load_files_to_databaseV2 LIMIT 1000


Service Bus trigger displaying incorrect Split On setting

Describe the Bug with repro steps

Add a service bus trigger to a workflow
Disable the Split On setting
Click the Save button
Refresh the page
Open the trigger settings, Split On will appear as enabled with no array value
Open the code view, the Split On setting will be absent

What type of Logic App Is this happening in?
Standard (Portal)
Are you using Preview Designer or GA Designer
Preview
Screenshots or Videos
No response
Browser
Chrome

@rllyy97 - Is this a servicebus azure trigger or inApp trigger?

@preetriti1 - I saw it on the In-App trigger


Fix SAMPLES query for gles

The OpenGL ES 3.0 Specification states that:

An INVALID_ENUM error is generated if internalformat is not color-, depth- or
stencil-renderable; if target is not RENDERBUFFER; or if pname is not SAMPLES or
NUM_SAMPLE_COUNTS.

(Note that this is different from what the OpenGL or ARB Extension Specification say).

Thanks


add MilvusContainer wrapper,  update testcontainers version

update testcontainers version to 1.19.8
add wrapper for Milvus

@dimafeng Hi there! Thank you for this library! If you have some time, please take a look at the pull request.


add characteristic.username

i think it would be good to have a characteristic.username that can be set from a platform module and is then used with accessory_Factor.Accessory instead of the one create by createUsername.
this would allow to keep the id tied to a specific device even if the user visible name changes.

So this question is very relevant to #58 - it would be great if we had something like username for every accessory that remained stable so that we can associate HomeKit "IDs" with it.
Currently I was planning on just using he "display name" (or "user visible name" as you suggest) instead of username, for the following reasons:

HomeKit seems to require that Accessory "display name" is unique inside a home
I'm having difficulty imagining developers of HomeBridge plugins being able to create and maintain a consistent "other string" that remains unique across user visible name change.

for the FHEM platform shim that i'm currently working on i have access to the device internals and most of the devices do have some sort of physical serial number or id. this is true for homematic, MAX, HUE and SONOS devices and some more, for FS20 and intertechno the device rf address can be used in combination with the device type. for other devices i will create a unchanging address on device creation in FHEM that will remain constant across renames and restarts.
the current HUE shim should be able to do the same.
#58 will be very welcome for the FHEM shim. if i export all devices in my installation the current way of doing this runs out of port numbers :)

ps: you can find the current version of the FHEM shim here: https://github.com/justme-1968/homebridge/blob/master/platforms/FHEM.js
i will do some more cleanups and then it would be good to merge it to your repository.

Excellent points, and very good to know. I'm currently going the direction of requiring a globally-unique UUID for every Accessory - which can be deterministically generated based on an arbitrary string if needed (like serial number as you mention).

that sounds good.

Closing this as we now only have one single username associated with the Bridge; Accessories don't need to generate their own anymore.


March 29th, 2021 Community Meeting

Please add a topic in this thread and add a link to the Github issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on Github before coming into the meeting
(optional) Paste the image of an animal :smile_cat:

🚨 DEMO 🚨  https://github.com/imjasonh/build-task

@dmage would like to present his concept for ImageJobs

Meeting notes:

status checks on the CLI repo;  per @qu1queee Code Engine folks are ready to start contributing;  between @alicerum and @otaviof the thought is enough is there for additional contributions to start, though is one more active PR
if we are to keep with the 6 week cadence, we should look into cutting 0.4.0 this week;  @gabemontero and @qu1queee will link up in slack
Discussed @ImJasonH  's prototype, which allows via a Tekton CustomTask to leverage Tekton Triggers to initiate a Shipwright Build, via whatever event Triggers can consume (i.e. git PR, among many others); also facilitates use of Shipwright from a Tekton Pipeline
Discussed @dmage 's early thoughts on sorting out how to monitor image changes (i.e. new sha's from a image ref) via the controller pattern; exposes ways to connect pluggable image registry webhooks (with fall back to pull monitoring); concepts still need more curation but show potential;  @dmage will come back to use when ready with a Google Doc we can review/contribute to
Next week's meeting falls on holiday for Europe; either cancelling or rescheduling needs to occur

If I've missed things, others of course correct / update as needed.


fix: Add form validation for max priority fee

What it solves
Resolves #3464
How this PR fixes it
Extends the form validation to compare ethMaxPrioFee against ethGasPrice.
How to test it

Open the safe
Create a pre EIP-1559 transaction
Change Max fee per gas and observe that the form can be saved
Create a post EIP-1559 transaction
Change Max priority fee per gas to be higher than Max fee per gas
Observe that there is a validation error

Screenshots

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 33.69%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Transactions/helpers/EditTxParametersForm/index.tsx
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3233

Relevant Lines:
8572

💛  - Coveralls

Does this allow it to be 0?

So, the validation is fine, but I'm not sure about the "must be higher than X", because the person will not understand why that number until he changes the "max fee per gas". I also understand that an error like "the value must be higher than max fee per gas" is just too long for the input.
I would not mind passing the ticket the way it is, having a validation is better than not having any, but I'd like to ask for a suggestion on what the error message can be

Does this allow it to be 0?

Yes, since the minValue validator is set to be 0 inclusive (second parameter).

So, the validation is fine, but I'm not sure about the "must be higher than X", because the person will not understand why that number until he changes the "max fee per gas".

Agreed, it would be easier to understand if we referenced Max fee per gas for the validation. I tried "Must be higher than Max fee per gas" but thats already too long. I don't see a solution to this short of changing the layout i.e.

For the scope of this ticket I would leave the validation message as it is.

Agree, let's leave it this way, but at some point lets bring this up for a new design at some point (or another error message)