added x86 support for Linux

Commit looks good to me, but it would build with go1.4+ (which is still relative new at this time)
 #include "textflag.h"

I think I'll merge it, with some note in the README or commit message.
So users having issues can either update their Go, or revert this single commit as a workaround.

Hold on, it breaks ARM build as well.
And backup a little bit, I can build a 386 binary without this commit.
So, is it trying to fix some runtime error, or ?

ah, yes I forgot to include that you had to be on go 1.4, nice catch. In what ways is it breaking the ARM build? I don't have an ARM device to test with, so I can't check. Backing up a bit, the reason I did this PR, as when I tried to build for Linux i386 with what is currently in master I got an error like this:
austin:edison ACherry$ GOARCH=386 GOOS=linux go build
# github.com/paypal/gatt/linux/internal/socket
../../paypal/gatt/linux/internal/socket/socket.go:91: undefined: syscall.SYS_BIND
../../paypal/gatt/linux/internal/socket/socket.go:99: undefined: syscall.SYS_SETSOCKOPT

Looking at http://golang.org/pkg/syscall/, I noticed that SYS_BIND was 49 and SYS_SETSOCKOPT was 54, both which exist on 64bit Linux. Looking at the i386 table in the Linux source:
https://github.com/torvalds/linux/blob/master/arch/x86/syscalls/syscall_32.tbl
49 and 54, do not map to those calls. Instead it uses another layer of indirection through socketcall (syscall 102 in i386) as noted in the Go syscall source here:
https://github.com/golang/go/blob/master/src/syscall/syscall_linux_386.go#L177
The changes in this PR, use the same strategy as the sys/unix package does. That being said, if I went this trail and got way off track, I would be happy to be shown the way back. :smile: Also, if there is anything in ARM build that I need to fix, let me know and would be happy to do so.

Cool, that's a decent effort and analysis.
But I still can't reproduce the 386 build fail with either go1.4 or go1.4.2.
Does the "HOST" have to be 386 to reproduce it? Which go version are you using precisely?
My build machine:

$ uname -a
Linux localhost 3.14.25 #36 SMP Mon Nov 24 09:06:07 PST 2014 x86_64 Intel(R) Core(TM) i7-4960HQ CPU @ 2.60GHz GenuineIntel GNU/Linux

Current tip commit of master branch
HEAD is now at df506cf Merge pull request #14 from wolfeidau/improved_error_handling

$ go version
go version go1.4.2 linux/amd64
$ GOARCH=386 GOOS=linux go build sample.go && file sample
sample: ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV), statically linked, not stripped

p.s. ARM build can be fixed by renaming the _amd64.go to something else,(_common.go maybe?) as you already had a build flag in it.

// +build !386

Okay, I got it. It has to be the "tip" of go to reproduce this. :-)
I see @davecheney was actively working on it, is it possible that you found a regression, and a opportunity to contribute upstream Go? I'm not sure, I'll try pick up more context tomorrow.
Thanks

Somehow, I got the same 386 build error with go1.4/go1.4.2 now...
So, please ignore my previous comment.
Thanks for the work!
BTW, I'm bring this change to the "next" branch, would you be able to test it on 386?

@roylee17 I would be happy to. I also run OS X as my primary machine, so I can test that as well, if you need.

@acmacalister that will be great!


Not Playing Saved Audio file on iPhone

Hello,
I am implementing version 3.2.0 and streaming from a url is ok but playing a saved audio file is not working on iPhone but working on android. I tried both cordova and web audio. Below is error I got as I will appreciate your help.
Audio error => track file:///var/mobile/Containers/Data/Application/4286D16E-3EE4-41B4-93FC-9FE4749552C1/Documents/M950.mp3 [object Object]
2017-10-31 09:15:07.588967+0100 RCCG LAP[1443:642590] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Invalid number value (NaN) in JSON write'
Kindly note that the file is on the device
2017-10-31 09:15:06.552694+0100 RCCG LAP[1443:642590] Loading track file:///var/mobile/Containers/Data/Application/4286D16E-3EE4-41B4-93FC-9FE4749552C1/Documents/M950.mp3
2017-10-31 09:15:06.555248+0100 RCCG LAP[1443:643619] Will attempt to use file resource 'file:///var/mobile/Containers/Data/Application/4286D16E-3EE4-41B4-93FC-9FE4749552C1/Documents/M950.mp3'
2017-10-31 09:15:06.555487+0100 RCCG LAP[1443:643619] Unknown resource 'file:///var/mobile/Containers/Data/Application/4286D16E-3EE4-41B4-93FC-9FE4749552C1/Documents/M950.mp3'
2017-10-31 09:15:06.572677+0100 RCCG LAP[1443:642590] Audio error => track file:///var/mobile/Containers/Data/Application/4286D16E-3EE4-41B4-93FC-9FE4749552C1/Documents/M950.mp3 [object Object]
2017-10-31 09:15:07.588967+0100 RCCG LAP[1443:642590] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Invalid number value (NaN) in JSON write'
*** First throw call stack:
(0x18426fd38 0x183784528 0x18426fc80 0x184cef4c0 0x184cee044 0x184cf0688 0x18420942c 0x184cefda8 0x184ceddf4 0x184ceee44 0x104a78884 0x104a83444 0x104a782c8 0x104a6a494 0x104a82200 0x104953dd8 0x104953c3c 0x10495581c 0x193553250 0x1934e7e00 0x1934e9fe0 0x1932d5b10 0x1934b2820 0x1932960ec 0x193298ae0 0x18b49cd10 0x18b49cfd0 0x184218358 0x1842182d8 0x184217b60 0x184215738 0x1841362d8 0x185fc7f84 0x18d6e3880 0x104922cfc 0x183c5a56c)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)

did you get any solution for this?

Not at all
On Tue, 21 Nov 2017 at 3:10 PM, Mubasshir Pawle<EMAIL_ADDRESS>wrote:

did you get any solution for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arielfaur/ionic-audio/issues/164#issuecomment-346055927,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAr5jNnZ0FoY5Qif5CYT1UNpHhGcXzJKks5s4uflgaJpZM4QNKRP
.

what did you do then?

For IOS I removed option of saving audio file to be played later but for
iOS you can save file locally and play latter
On Wed, 22 Nov 2017 at 8:28 AM, Mubasshir Pawle<EMAIL_ADDRESS>wrote:

what did you do then?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arielfaur/ionic-audio/issues/164#issuecomment-346277887,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAr5jM2MUXUakADnI3wPqqTpnIWGKRzXks5s49s1gaJpZM4QNKRP
.

I bit confused with your reply.

@adedapo
Could you explain what you mean by:

for iOS you can save file locally and play latter

I am also looking for an app which can download and play audio locally but getting an error

Hello
Samething I wish to implement. On Android I was able to download the audio
and play locally. On iOS, I was also able to download but couldn't play
locally reason I opened the issue.
I am looking at removing the option to download and play locally from the
iOS version.
Hope it helps
On Wed, Nov 22, 2017 at 9:33 AM, Himanshu Phirke<EMAIL_ADDRESS>wrote:

@adedapo https://github.com/adedapo
Could you explain what you mean by:
for iOS you can save file locally and play latter
I am also looking for an app which can download and play audio locally but
getting an error
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/arielfaur/ionic-audio/issues/164#issuecomment-346279091,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAr5jDR1h0FaN0CrbSIUBnsROL1GKpE2ks5s49xzgaJpZM4QNKRP
.


Fix mutated html report file not recovered in subsequent run

Closes #798
Okay, I have found that there is a hash validation scheme.
But it is only comparing the hash of the coverage data residing in the memory, with the hash written in status.json
I think that the hash scheme should be changed, so that the hash of the actual file should be written in the json, and used in comparison.
This is an initial draft to fix the bug I have encountered.

Codecov Report

Merging #799 into master will decrease coverage by 10.45%.
The diff coverage is 80%.

@@             Coverage Diff             @@
##           master     #799       +/-   ##
===========================================
- Coverage   90.67%   80.22%   -10.46%
===========================================
  Files          81       81
  Lines       11110    11119        +9
  Branches     1148     1149        +1
===========================================
- Hits        10074     8920     -1154
- Misses        925     2028     +1103
- Partials      111      171       +60

Impacted Files
Coverage Δ

coverage/html.py
94.49% <80%> (-4.55%)
:arrow_down:

coverage/backunittest.py
35.71% <0%> (-64.29%)
:arrow_down:

tests/plugin2.py
0% <0%> (-62.97%)
:arrow_down:

tests/test_plugins.py
44.78% <0%> (-54.46%)
:arrow_down:

tests/osinfo.py
18.51% <0%> (-48.15%)
:arrow_down:

tests/test_context.py
57.92% <0%> (-42.08%)
:arrow_down:

coverage/phystokens.py
62.5% <0%> (-37.5%)
:arrow_down:

tests/test_api.py
65.19% <0%> (-34.81%)
:arrow_down:

tests/test_oddball.py
67.2% <0%> (-32.8%)
:arrow_down:

coverage/backward.py
68% <0%> (-31%)
:arrow_down:

... and 35 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 c0dd3e9...94c1d8f. Read the comment docs.

Thanks for the work on this, but it isn't worth the cost. In my experiments, this adds significantly to the cost of HTML generation.
I tested with an HTML directory with 2783 files. It took 59s to generate the report from scratch into an empty directory.  Changing one source file and generating again took 25s.  Reading and hashing all the .html files took between 6s and 33s (it's hard to get repeatable numbers, but I tried hard to use uncached disk reads).  That's too much time to add to 25s.
As I understand it, you are modifying all the HTML files. You should delete the directory before running the HTML report.

Okay, Thanks for the review. But I have some questions

What about adding this as an option for html_report rather than default, if speed is a concern.
If not, this may have to be documented as #798

And one other question I have is that checking whether the file existing or not would not have any concern for speed, than checking the file mutation.
And this PR also addresses the issue.


feat: paste window open hook

Because the paste window needs a common filetype like bash to get the correct syntax highlighting and completion, It's hard to do something like setting keymaps or adjusting window options. I suggest exposing a hook function that receives winid or bufnr as arguments for doing these things.

Hey @ofseed thanks for reporting.
Regarding adjusting window options, there are overrides exposed via:

Options
Type
Description

paste_window.winopts_overrides
fun(paste_winopts: KsbWinOpts): table<string,any>?
Paste float window overrides, see nvim_open_win() for configuration

paste_window.footer_winopts_overrides
fun(footer_winopts: KsbWinOpts, paste_winopts: KsbWinOpts): table<string,any>?
Paste footer window overrides, see nvim_open_win() for configuration

There is an example of this in Advanced Configuration Examples named ksb_example_paste_win_winopts.
But, I agree, there is no general hook if you want to do something based on the bufnr or winid. What are you trying to trigger when the paste window opens?

I know this option, and I fixed a bug about this : ), see: https://github.com/mikesmithgh/kitty-scrollback.nvim/pull/182
There's a more complicated situation in my use case. I'm using 'statuscolumn' to arrange my sign columns, there's no way to disable 'statuscolumn' except a hook to do this.

I know this option, and I fixed a bug about this : ), see: #182

😅 oops, didn't realize.

There's a more complicated situation in my use case. I'm using 'statuscolumn' to arrange my sign columns, there's no way to disable 'statuscolumn' except a hook to do this.

Ah, I see yeah you are right.

:tada: This issue has been resolved in version 4.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Hey @ofseed I added the callback after_paste_window_ready that has a first parameter containing the window and buffer ids of the scrollback buffer, paste window, and paste window footer. The paste window data will be in form similar to:
local paste_window_data = {
  scrollback_buffer = { bufid = p.bufid, winid = p.winid },
  paste_window = { bufid = p.paste_bufid, winid = p.paste_winid },
  paste_window_footer = { bufid = p.footer_bufid, winid = p.footer_winid },
}

I played around a bit and created this example:
require('kitty-scrollback').setup({
  {
    callbacks = {
      after_paste_window_ready = function(pastewin_data, kitty_data, opts)
        local stc = '%#ErrorMsg#%{(v:lnum%2)?" ".v:lnum:""}' .. '%#WarningMsg#%{!(v:lnum%2)?" ".v:lnum:""} '
        vim.api.nvim_set_option_value('statuscolumn', stc, {
          win = pastewin_data.paste_window.winid,
        })

        vim.keymap.set({ '' }, '<Tab>', '<Plug>(KsbToggleFooter)', {
          buffer = pastewin_data.paste_window.bufid,
        })

        if pastewin_data.paste_window_footer.winid then
          vim.api.nvim_set_option_value('statuscolumn', 'help:', {
            win = pastewin_data.paste_window_footer.winid,
          })
        end
      end,
    },
  },
})

Please let me know if you run into any issues

It runs without problems, thank you!


Vault is not unsealing on latest integ

Even thou it seems like all the unsealing is geared. By the time it gets to opt.yml to do  TASK: [vault | authenticate with vault opt]  the vault is still sealed

Logs from the failing component, if applicable. Might be any of the following:

   failed: [mantl-dev1-control-03] => {"changed": true, "cmd": ["vault", "auth", "tls-skip-verify", "bcbc13f5-6e18-8124-f8ed-904fcf581937"], "delta": "0:00:00.062328", "end": "2017-04-30 21:08:20.371792", "rc": 1, "start": "2017-04-30 21:08:20.309464", "warnings": []}
stderr: Error validating token: Error making API request.

URL: GET https://<IP_ADDRESS>:8200/v1/auth/token/lookup-self
Code: 503. Errors:

* Vault is sealed
failed: [mantl-dev1-control-02] => {"changed": true, "cmd": ["vault", "auth", "tls-skip-verify", "bcbc13f5-6e18-8124-f8ed-904fcf581937"], "delta": "0:00:00.060132", "end": "2017-04-30 21:08:20.373100", "rc": 1, "start": "2017-04-30 21:08:20.312968", "warnings": []}
stderr: Error validating token: Error making API request.

URL: GET https://<IP_ADDRESS>:8200/v1/auth/token/lookup-self
Code: 503. Errors:

* Vault is sealed
failed: [mantl-dev1-control-01] => {"changed": true, "cmd": ["vault", "auth", "tls-skip-verify", "bcbc13f5-6e18-8124-f8ed-904fcf581937"], "delta": "0:00:00.059668", "end": "2017-04-30 21:08:20.377044", "rc": 1, "start": "2017-04-30 21:08:20.317376", "warnings": []}
stderr: Error validating token: Error making API request.

URL: GET https://<IP_ADDRESS>:8200/v1/auth/token/lookup-self
Code: 503. Errors:

* Vault is sealed

FATAL: all hosts have already failed -- aborting

sample.yml: none
Ansible version (ansible --version): ansible 1.9.6
Python version (python --version): Python 2.7.12
Git commit hash or branch: 95b5d70e9ee64c0cf8c06c9acccbdb1e833821fd
Cloud Environment: aws
Terraform version (terraform version): Terraform v0.9.3

This shouldn't be affecting master any longer as master has been reset to 1.3 and the changes have all been moved to integ. Please confirm.

Will check integ, replace master with integ then :). Will revert if it is still an issue on integ

issue is there on integ branch:
TASK: [vault | authenticate with vault] ***************************************
failed: [mantl-dev1-control-03] => {"changed": true, "cmd": ["vault", "auth", "-tls-skip-verify", "bcbc13f5-6e18-8124-f8ed-904fcf581937"], "delta": "0:00:00.051562", "end": "2017-05-01 13:43:02.926980", "rc": 1, "start": "2017-05-01 13:43:02.875418", "warnings": []}
stderr: Error validating token: Error making API request.
URL: GET https://<IP_ADDRESS>:8200/v1/auth/token/lookup-self
Code: 503. Errors:

Vault is sealed

ok shall I merge this in?

There is no fix for it as far as I know?

sorry, deleted my comment

will check this momentarily, but make sure that the previous several vault
commands actually succeeded.  They can fail silently and then you don't get
your unseal keys.  Also, check the git history on the lines around the
failing task to see if a fix has been reverted.  It was definitely working
for me last week.
On Mon, May 1, 2017 at 8:46 AM, Travis Rowland<EMAIL_ADDRESS>wrote:

sorry, deleted my comment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mantl/mantl/issues/1977#issuecomment-298335918, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKErsrYafo3s26XEjtSFTv76AuIrOB3vks5r1eJEgaJpZM4NMsYh
.

--
sr. devops engineer
wavefront

It is definitely since the major merge that it does not unseal anymore

8 files has changes from master->integ in roles/vault
There is a bunch of ca-cert things that have been removed but have put them back does not make a difference must be something else

on int or master?  I'll try it now.
On Mon, May 1, 2017 at 9:07 AM, Joee<EMAIL_ADDRESS>wrote:

It is definitely since the major merge that it does not unseal anymore
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mantl/mantl/issues/1977#issuecomment-298339256, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKErssyVMpCfCE5k7a0RkZcCAZo6JnTzks5r1ecvgaJpZM4NMsYh
.

--
sr. devops engineer
wavefront

@kbroughton integ branch

There are two separate issues here as far as I can see.

In order the tls-auth.yml to work we have to fix the vault_command_options default var. It was fixed on master by @kbroughton but the changes didn't propagate to integ branch.

At some point, OTP support was introduced https://github.com/mantl/mantl/pull/1165 which needs a little refinement in order to work properly. OTP is not present on master branch. Also it reverts the vault_command_options configuration https://github.com/mantl/mantl/pull/1165/files#diff-6142088e026c4d7163f2f8865f3bf5e0R4

Vault related configuration has been pushed to next branch. If the fix works for you we can close the issue.

@KaGeN101 are you able to test?


Deployement Docs

I have Already finished a small project and i want to deploy it for test, and i had some problems when deploying the dockerfile to heroku , and my porblem is the nothing work and in the screen they tell application error .
could you make a small docs about the deployement !
and Thanks for this amazing framework

Hi!
I haven't used Heroku. Are you placing a docker container there?
Is docker image being assembled normally? What type of compilation do you use when building docker image? Have you added assets to the project (to the appsetting.json file)?
In general. When creating a project, a dockerfile is created. Minerva contains 2 types of dockerfile templates for 2 types of compilation, respectively: JIT and AOT. You can regenerate dockerfile using the docker command (and specify the necessary type of compilation). Also, if you add assets to the project, that is, files that should be in the final assembly, which are necessary for work, you must regenerate the dockerfile.
If you added assets to appsetting.json, then you must regenerate the dockerfile so that there are lines in the docker file with the addition of these files to the docker image.
If you are successfully building docker image, then try running docker container locally. Make sure it works and it's not about Heroku (I haven't used it, so I don't know if I need to configure the project for it in some specific way).

I will add more information about dockerfile and deployment to the documentation.

Yes I add all the assets to the appSetting.json , Yes May be i didn't regenerate the docker file  that causing the issue , I have read the heroku docs about how to deploy a Docker Container and may be the docker regenrating that make the problem ! it will be great that you add more info about the deployement ,
Thank you for your time

I have added deployment information to the documentation.


WebView2 doesn't expose NewWindowRequested event

Describe the bug
The NewWindowRequested event from WebView (1) and the underlying WebView2 API is not exposed to the XAML based WebView2. ☹
Expected behavior
NewWindowRequested event is on WebView2 XAML control.
Version Info
NuGet package version: WinUI 3 Preview2

Not sure if this is a feature request or a bug. @adambarlow

@StephenLPeters it's exposed on the underlying WebView2 C++ API, it's just missing in the projection/wrapper in WinUI 3. So it's both? Maybe we call it an unimplemented requirement from the spec? 😂

The WinRT wrapper for the WebView2 is still in the works. We hope to have something to share soon. But yes, it's neither bug nor feature request as it is in plan for delivery.

The work to expose the CoreWebView2 WinRT APIs with its full capabilities is starting now (internal task). And, we have another more specific task to ensure NewWindowRequested works (internal task).

With the recently released WinUI Preview3, this is now exposed as WebView2.CoreWebView2.NewWindowRequested.


fix: read log from DLT logs

Description
This PR fixes an issue reading tables created by DLT, specifically, when the metadata action does not contain schema information.
This is a draft, since I want to first clarify if this is intended. In the protocol the schemaString property is marked as required.
Related Issue(s)

closes #1302

Documentation

Any progress on this fix?

A duplicate of the issue here: https://github.com/delta-io/delta-rs/issues/1557
and a discussion thread from another encounter with this issue here: https://github.com/delta-io/delta-rs/discussions/1604
👍 for getting eyes on this to review and merge in

I am assigning this to myself. I will take @roeap and probably re-do it on top of the current HEAD of main after my other pull request #1629 merges.


Adding a performance improvement guide

Change Type
[ X] Addition
[ ] Correction
[ ] Deprecation
[ ] Cleanup (formatting, typos, etc.)
I would like to have a guide in the documentation about how to improve netbox performance once the amount of objects is scaling up.
How can performance can be audited? Where should I start digging when my API calls are slows? etc...

Are you volunteering to write this guide? Can you provide a rough outline?

It should be noted that since Netbox is built on the Django framework, a user can search out "Django performance improvements" to get relevant tips for Netbox.

Yes but where do you start? Also, keep in mind that users are not necessarily developers that know how to develop with Django.

I'd start with the official Django documentation.
Performance tuning is a complex beast and is going to require (likely a lot of) self-started research to tackle. Perhaps as the Netbox userbase (and scale) increases and there's an idea of what the most common performance issues people are running in to are, then some Netbox-specific tuning ideas could be put forth.
Maybe it's already there - but considering the bulk of the application is Django-code, I'd say start there.

I don't think it's feasible to produce a static document on performance troubleshooting. The developers take reasonable efforts to optimize performance as it is, and while I'm sure there's room for improvement, if we knew what additional steps we could take, we would have taken them already. As @tyler-8 suggests, the Django documentation offers some excellent advice on general performance; for anything more specific it's probably best to share your specific issue on the mailing list for guidance.


depends failed

Because every version of flutter from sdk depends on meta 1.1.6 and every version of retrofit depends on meta ^1.1.7, flutter from sdk is incompatible with retrofit.
So, because demo depends on both flutter any from sdk and retrofit any, version solving failed.
pub get failed (1)
exit code 1

Feel free to reopen it if this issue is reproduced.


zerocode.io  the website could not open

zerocode.io  the website could not open

@LoveDu  its UP now, please check


Should not return www-authenticate request method is OPTIONS

Task Description
This used to work, but for some reason it's broken now (not sure if it's a newer Chrome version or we just broke it somehow).
In general, when a browser is trying to access an api endpoint, it would first send an OPTIONS request to check the CORS settings of the server and then send the actual request. The first request will not contain the X-Requested-With header (#457) so that's what's causing the native HttpBasic browser popup to appear.
Steps to reproduce:

Set a small session timeout (i.e. 2-3 seconds)
Login
Wait a few seconds
Refresh page

# curl -I 'http://localhost:48080/api/ping/token' -H 'Accept: application/json, text/plain, */*' -H 'Referer: http://localhost:48080/' -H 'DNT: 1' -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.92 Safari/537.36' -H 'Authorization: Bearer INSERT_A_REAL_TOKEN' -X OPTIONS --compressed

HTTP/2 401
server: nginx
date: Tue, 18 Sep 2018 21:29:14 GMT
content-type: application/json
www-authenticate: Basic realm="Strongbox Repository Manager"
x-content-type-options: nosniff
x-xss-protection: 1; mode=block
cache-control: no-cache, no-store, max-age=0, must-revalidate
pragma: no-cache
expires: 0
x-frame-options: DENY

Acceptance Test

[ ] Building the code with mvn clean install -Dintegration.tests still works.
[ ] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Task Relationships
This task relates to #457.
Help

Our chat
Points of contact:

@steve-todorov
@sbespalov
@fuss86

I'll be taking a look at this issue

Okay! Good luck! :)

I've been able to reproduce the error and found out that the tokens of the authentication is inside the BasicAuthorizationSupplier.java
But I'm having troubles understanding the desired solution, could you explain to me what is the expected result?

@steve-todorov , @fuss86 , @ptirador : Any thoughts?

#1265

Hey since most of the questions I asked and reports of my progress were done through the strongbox community chat. I think it would better for the community that the findings on this issue are shown in public.
The issue occurred inside the Http401AuthenticationEntryPoint.java class. Since the updates on some browsers, pre-flight OPTIONS calls are sent for all GET/POST requests unless they are classified as simple requests when sending a CORS request.
This pre-flight OPTIONS request doesn't contain the "X-Requested-With" header and because of this what happened is that the response always returned with the "www-authenticate" header which prompted the authentication pop-up.
In the PR #1265  I added this validation inside the Http401AuthenticationEntryPoint :
!request.getMethod().equalsIgnoreCase(IS_REQUEST_OPTIONS)
With this, we make sure that the "www-authenticate" response isn't triggered every time the API receives the pre-flight OPTIONS call.
Here are some blog post for reference about pre-flight requests:
link 1
link 2

Thanks for following up with an explanation! :)


Thanks for making this library.


Fixed mass history

#19
The reason is the Next()/Prev() pass the real []rune out, which is a pointer, so when the RuneBuf update the []rune directly, it also will change the history item.

@michalpristas I think this can fixed the problem

yes, many thanks.
tested ok


Update from 0.6.4 -> 0.7.0 completed with error message

Update was triggered from the Apps page and seems to be completed successfully - everything is working and all the pods are green, but the App is still in 'Installing' state and App: longhorn-system page has following message on top in red:
'Failed to install app longhorn-system. Error: UPGRADE FAILED: StorageClass.storage.k8s.io "longhorn" is invalid: provisioner: Forbidden: updates to provisioner are forbidden.'
Kubernetes Version: v1.16.3
Rancher Version: 2.3.2

This error is caused by the Longhorn StorageClass upgrade failure, see here for details.

It helped, all is good now!


After selecting the normal filter in the filters list

Describe the bug
After selecting the normal filter in the filter section, the selected image fails to select. WHen any other filter is selected it works well but only the normal filter fails to work.
1 a: After selecting Normal FIlter LIke Below

b:

2 a: Then selecting any other filter works perfectly like this

b: After selecting it shows perfectly

Am using swift 4.2 with the respective YPImagePicker cocoapods version

@pdee2131 wow that is a very weird bug, I unfortunately couldn't reproduce it on my side, could you try to investigate what's happening in the code a little further?

@pdee2131 are you still having this issue?

There is 2022 year outside🎉 Old outdated issues are closing. Please search if there are already existing issues with the same problem before opening new ☮


Added other features, such as getByClassname, closeToTop, scrolledToTop, scrolledToBottom

A more updated version of my other pull request, which fixes some issues that it could cause.

So you want me to close the other PR without merge?

Yes that first PR I sent should be canceled
On Tue, Dec 11, 2018 at 12:55 PM p0o<EMAIL_ADDRESS>wrote:

So you want me to close the other PR without merge?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/p0o/react-is-scrolling/pull/8#issuecomment-446298264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOeJgQ2TBGOAMfKanuckrV4kVOdEAusaks5u3_GsgaJpZM4ZNPUi
.


Feature/Product Request: Adapt your visualizer to become a strategy tool

The world lacks a diagramming tool for source control. Your visualizer would be a perfect solution to that. Especially if I can just punch in git commands to visualize. So for example, if I need to create a branching strategy for my next 4 roadmap releases, I could go through the entire process in your tool and have it chart out what that would look like. It would also be great to be able to save that as an ongoing strategy for team collab purposes.

@dasjestyr  we do support "permalinks" so to speak, so you can link people to a specific instance of a tree (or have it playback a sequence of commands)
would that help?

I'll have to give it a try. I'm really looking for a documentation tool for visualizing vcs strategy so if that would it into a workflow then I suppose it would help. Although relying on am external site for documentation feels awkward. Maybe a combo of the permalink and image/file export?
-----Original Message-----
From: "Peter Cottle"<EMAIL_ADDRESS>Sent: 8/5/2015 1:34 PM
To: "pcottle/learnGitBranching"<EMAIL_ADDRESS>Cc: "Jeremy Stafford"<EMAIL_ADDRESS>Subject: Re: [learnGitBranching] Feature/Product Request: Adapt yourvisualizer to become a strategy tool (#304)
@dasjestyr we do support "permalinks" so to speak, so you can link people to a specific instance of a tree (or have it playback a sequence of commands)
would that help?
—
Reply to this email directly or view it on GitHub.

Yeah that could work! Here is an example of linking to a permalink with commands:
http://pcottle.github.io/learnGitBranching/?NODEMO&command=git commit; go -b side; git commit
and then loading a tree:
http://pcottle.github.io/learnGitBranching/?NODEMO&command=importTreeNow {"branches":{"master":{"target":"C1","id":"master","remoteTrackingBranchID":null},"side":{"target":"C2","id":"side","remoteTrackingBranchID":null},"another":{"target":"C3","id":"another","remoteTrackingBranchID":null}},%22commits%22:{%22C0%22:{%22parents%22:[],%22id%22:%22C0%22,%22rootCommit%22:true},%22C1%22:{%22parents%22:[%22C0%22],%22id%22:%22C1%22},%22C2%22:{%22parents%22:[%22C1%22],%22id%22:%22C2%22},%22C3%22:{%22parents%22:[%22C2%22],%22id%22:%22C3%22}},%22tags%22:{},%22HEAD%22:{%22target%22:%22another%22,%22id%22:%22HEAD%22}}
With some image exporting you could probably get this to work! I believe Raphael.js has support for that
Closing out since I dont think theres much actionable here :)


Per-language settings not being picked up under macOS Sierra

I’m using atom-editor-settings 1.1.2 with Atom 1.14.1, and have recently upgraded to macOS 10.12.3.
Since upgrading my OS, I’ve noticed that atom-editor-settings doesn’t seem to be applying per-language settings by default when opening a file. Thus, tab length, soft tabs, etc are configured as per Atom’s default.
However, if I then open the grammar config file from the command palette and close it again, the settings are now correctly applied to both the open file, plus any subsequently-opened files using the same grammar.
This will continue until all files that use that grammar are closed, or a new file that should use that grammar is created, whereupon Atom’s defaults will be used instead.
Steps to reproduce
Note: I’m using Ruby as an example, but I can reproduce this with every language I’ve tried where I’ve also specified a grammar configuration.

set Atom’s default editor settings to use soft tabs, with four spaces per tab
create a grammar configuration for Ruby, using hard tabs, set to two spaces per tab
create a number of empty Ruby files in a blank directory, and open it with Atom.
open one of the files. You should find that hitting tab will insert soft tab at 4 spaces
open and close the grammar configuration: hitting tab in the Ruby file should now insert a hard tab at 2 spaces
open one of the other Ruby files. Hitting tab should continue to insert a hard tab at2 spaces
close both Ruby files and re-open one. This should reset back to inserting a soft tab at 4 spaces.
open + close the grammar configuration (back to a hard tab at 2 spaces)
create a new Ruby file. This should go back to a hard tab at 2 spaces until you open/close the grammar config again
close all ruby files, and re-open one. You should be back with soft tabs at 4 spaces once more

…
My apologies for the long issue, but I wanted to give as much information as I could. Please do let me know if I can provide you with any further information that might help.
J

No need to apologise for a long issue, the more information the better 👍
I still use El Capitan on my primary Mac, but will test this out on another that has Sierra.


FIX: Import FileDialog from PyVista

This PR updates the import logic of FileDialog with the changes of pyvista 0.25

Thanks @GuillaumeFavelier


Mobile browser Brave does not display the camera image.

Hi!
Thank's for your integration! Very cool!
But, Mobile browser Brave does not display the camera image.
In other bruiser (chrome, duckduck.go) it's ok.
Any ideas?

Maybe the makers of Brave need to fix their browser? :)


Checkpointing with collections raising exception in Keras

Hi,
A TF newbie here.   Am trying to use a  pre-trained Keras Resnet-50 model on very large biological images(3000x4000 pix) of larvae.  Since the image size is huge have to resort to gradient checkpointing.
have done the requisite monkey patching
import memory_saving_gradients K.__dict__['gradients']=memory_saving_gradients.gradients_collection
I have manually defined a collection of all the checkpoint nodes like so
[tf.add_to_collection("checkpoints",base_model.get_layer(i).get_output_at(0)) for i in ["add_4","add_8","add_12","add_16"]]
However I get the following exception
 File "/home/satish/anaconda3/envs/tensorflow/lib/python3.6/site-packages/keras/optimizers.py", line 244, in get_updates     grads = self.get_gradients(loss, params)   File "/home/satish/anaconda3/envs/tensorflow/lib/python3.6/site-packages/keras/optimizers.py", line 78, in get_gradients     grads = K.gradients(loss, params)   File "/home/satish/anaconda3/envs/tensorflow/lib/python3.6/site-packages/memory_saving_gradients.py", line 31, in gradients_collection     return gradients(ys, xs, grad_ys, checkpoints='collection', **kwargs)   File "/home/satish/anaconda3/envs/tensorflow/lib/python3.6/site-packages/memory_saving_gradients.py", line 185, in gradients     raise Exception('no checkpoints nodes found or given as input! ') Exception: no checkpoints nodes found or given as input!
Not sure why the collection is deemed empty.
Would greatly appreciate any feedback.
Thanks
Satish

@skottapa Were you able to figure out the issue with here and fix it? I plan on using gradient checkpointing with a large model in the near future.

Unfortunately no.  I just shifted from 1080 to K80, and was able to fit my model in, though it was a lot slower.  Need to get back to it, time permitting
Get Outlook for Androidhttps://aka.ms/ghei36

From: Shyam Saladi<EMAIL_ADDRESS>Sent: Tuesday, October 16, 2018 2:51:37 AM
To: openai/gradient-checkpointing
Cc: skottapa; Mention
Subject: Re: [openai/gradient-checkpointing] Checkpointing with collections raising exception in Keras (#28)
@skottapahttps://github.com/skottapa Were you able to figure out the issue with here and fix it? I plan on using gradient checkpointing with a large model in the near future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/openai/gradient-checkpointing/issues/28#issuecomment-430018021, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIfvPh4hd22hZbByJ9Y282STq-QSZMdIks5ulPxhgaJpZM4Vv0Da.


Add comments to commits

When implementing checking pushes (not pull requests), I neglected to have GopherCI write comments on those commits.
I don't know why I skipped doing that, I believe this to be a mistake, and GopherCI should post comments whether it was discovered in a PR or a push.

That explains why it's been surprisingly quiet. :p

Yeah, sorry, and I'll mark it as a bug, it's a straight forward fix, but just focusing on web component right now.

OK, planning this now. I'll likely take the opportunity to refactor the comments into a Reporter interface of some description. There will be multiple reporters, such as:

PR inline comments: needs to dedupe and write only maxIssueComments, this is current behaviour.
Commit comments: needs to only write maxIssueComments, this allows us to comment on a commit (this issue).
Status API: Needs to set success or failure.
PR review comments: needs to dedupe and write only maxIssueComments, using the pull request review comment methods. See #112.
PR comment summary: Posts a comment on the PR itself with a summary explaining which linters have which number of errors. This probably won't be implemented, but it's just an option.
Send email: Sends the person submitting the request an email with a summary and link to the analysis.

I won't implement all of these, and some tools may be configured to not report some errors in the future, see #3.

OK, here's the blocker.
A commit comment occurs at a particular ref, file and diff position: https://developer.github.com/v3/repos/comments/#create-a-commit-comment
If you push a commit and there's an issue, we can easily comment on the exact location.
But if you push two commits, and the issue was in the first commit, GopherCI doesn't have knowledge of that commit ref, it knows ref before and after (head). Further, the diff that we generate is between before and after, so if there's an issue on after/head, the diff position is still likely to be wrong.
There's a few options the immediately come to mind:

Skip commenting on a push, and instead send the user our own email. I don't hate this, but it feels like a kludge, the user might not want an email and enjoy to GitHub workflow (I'm looking at you @shurcooL).
Run an analysis for every commit. Due to the number of potential commits, and even with rate limiting, this would be too exhaustive.
Pass the Analyser a list of all the commits instead of just before and after, have it determine which commit and position an issue occurs on.

Option 3 is basically the only viable option, so I'll be going with that if I don't have any other thoughts that aren't hacks.

This is going to be too difficult to do.

GopherCI wants to run the tools just once for all commits.
We need to know which issue occurred in which commit and the position in that commit.

The problem comes with this example:
--- /dev/null
+++ b/main.go
@@ -0,0 +1,7 @@
+package main
+
+import "fmt"
+
+func main() {
+       fmt.Println("Hi")
+}

This is our initial correct program. There are no problems. We push this code to GitHub.
--- a/main.go
+++ b/main.go
@@ -3,5 +3,5 @@ package main
 import "fmt"

 func main() {
-       fmt.Println("Hi")
+       fmt.Println("Hi %s", "1") // possible formatting directive in Println call
 }

In our next commit, we've introduced a problem: main.go:6: possible formatting directive in Println call
We still haven't pushed yet.
--- a/main.go
+++ b/main.go
@@ -2,6 +2,7 @@ package main

 import "fmt"

+func init() {}
 func main() {
        fmt.Println("Hi %s", "1") // possible formatting directive in Println call
 }

We've now added some lines above the problem, committed and pushed.
GopherCI sees the problem as main.go:7: possible formatting directive in Println call. But individually neither of the two last patches see line 7 as being modified, only the aggregate patch sees them as being modified:
--- a/main.go
+++ b/main.go
@@ -2,6 +2,7 @@ package main

 import "fmt"

+func init() {}
 func main() {
-       fmt.Println("Hi")
+       fmt.Println("Hi %s", "1") // possible formatting directive in Println call
 }

The aggregate patch is what GopherCI sees right now, and we can record the issues with the aggregate patch position, and because of the way we fetch patches in the analysis summary page, we can even show these issues in line.
But GitHub requires us to know the commit ref and position relative to that commit of where the problem was introduced.
Unless we could somehow merge these patches in reverse order (checking HEAD, then HEAD~1 HEAD, then HEAD~2 HEAD) until we found a modified line at that position, could this work, but may still have false positives, what if a different change occurred on that line in an older commit that wasn't our actual issue.
Another alternative could be that GopherCI does run the tools for every commit (or more practically, the last n commits). In the above example, GopherCI would see the problems for both commits (as it does appear in each commit), so it'd need to deduplicate them somehow, but another commit just before we pushed could remove the issue entirely - this would result in us reporting issues in older commits that aren't issues anymore. This would be correct but an annoyance, yes the problem occurred at that commit (again if we can deduplicate correctly), but it's not a problem when I pushed it, so it's probably nothing to worry about (perhaps it was a style issue).
Having said all of that, right now, GopherCI can show the diff of all commits and exactly which line the issue is on and display it to the user - it just can't comment inline.
This leads me back to two options:

Leave the behaviour as is, providing an option for GopherCI to email the user if there's a problem.
Comment on the latest commit, but not on a position and just say there are issues with the push (although there's nothing to suggest to the user these commits were pushed at the same time in GitHub, GopherCI knows though), and link the user to GopherCI.

Example pushing multiple commits, with many commits introducing issues (9 in total).

If we comment each issue on the commit itself (not defining path or position in https://developer.github.com/v3/repos/comments/#create-a-commit-comment), it looks like so:

This can easily be improved with (potentially adding filename and line number as a link to the correct location/branch etc):

Or more simply:


BUG: ShutdownSignalBarrier causes Java runtime shutdown hooks to not run

Description:
When the ShutdownSignalBarrier class is loaded, its static initialiser overrides the default JVM Signal handler. This means on shutdown of the JVM, when the SIGTERM signal is sent, the ShutdownSignalBarrier handler class is called. However, as the ShutdownSignalBarrier's registered signal handler does not then call the previous handler (which is what is returned from the Signal.handle(..) method), the JDK does not then run its own shutdown hooks, which can break logic of applications that have other resources they would like to gracefully shut down. i.e. applications that call:
Runtime.getRuntime().addShutdownHook(
        new Thread(() -> System.out.println("server shutting down"), "shutdown-thread"));

This is because the signal handler that is registered by default, (in the java.lang.Terminator class), is what is responsible for calling System.exit(code), which eventually runs the default JVM shutdown hooks.
In the ShutdownSignalBarrier class, a possible fix could then look like:
public static final String[] SIGNAL_NAMES = { "INT", "TERM" };
private static final ArrayList<CountDownLatch> LATCHES = new ArrayList<>();

static
{
    for (final String signalName : SIGNAL_NAMES)
    {
        final var previous = Signal.handle(new Signal(signalName), (signal) -> {});
        Signal.handle(new Signal(signalName), (signal) -> {
            LATCHES.forEach(CountDownLatch::countDown);
            previous.handle(signal);
        });
    }
}

which would allow for the ShutdownSignalBarrier class to still take priority and run before any other handlers, but after completing it can then run the previously registered handler, allowing the JDK to run its shutdown hooks correctly.
If this is an acceptable fix, I am happy to raise a PR and contribute to the library.

PR merged.

In fact, there are 2 places will call the ShutdownHooks:

Shutdown.exit(), this method is called either by invoking Runtime.exit() manually, or by SIGINT or SIGTERM handler registered in Terminator.setup().
Shutdown.shutdown(), as it mentioned in its doc:  Invoked by the JNI DestroyJavaVM procedure when the last non-daemon thread has finished, it will be invoked automatically when there is no non-daemon threads in the JVM.

After using ShutdownSignalBarrier, the SignalHandlers registered by Terminator.setup() is replaced, so the Shutdown.exit() method will not be called when the JVM is killed by signal INT or TERM, so the ShutdownHooks will not be called through Shutdown.exit().
However, you can shutdown all your daemon threads when ShutdownSignalBarrier.await() is returned from blocking, this way, the Shutdown.shutdown() method will be called, so that all the ShutdownHooks can be called.
Here is the code:

import org.agrona.concurrent.Agent;
import org.agrona.concurrent.AgentRunner;
import org.agrona.concurrent.BusySpinIdleStrategy;
import org.agrona.concurrent.ShutdownSignalBarrier;

public class ExitDemo0 {
    public static void main(String[] args) throws InterruptedException {
//        use the ShutdownSignalBarrier to get notified when SIGTERM is received.
        ShutdownSignalBarrier barrier = new ShutdownSignalBarrier();

        Runtime.getRuntime().addShutdownHook(new Thread(() -> {
            System.out.println("shutdown hook invoked....");
        }));

        //create an AgentRunner to be closed when SIGTERM is received.
        var runner = new AgentRunner(new BusySpinIdleStrategy(), err -> err.printStackTrace(), null, new Agent() {
            @Override
            public int doWork() throws Exception {
                return 0;
            }

            @Override
            public String roleName() {
                return "agentxxx";
            }

            @Override
            public void onClose() {
                System.out.println("onClose");
            }
        });
        //start the agent runner, the thread here is a non-daemon thread.
        AgentRunner.startOnThread(runner);
        //wait for terminal signal
        barrier.await();
        System.out.println("Shutdown Signal received");

        //Here if we don't close the AgentRunner, the above ShutdownHook if not be called.
        //If we close the AgentRunner, the above ShutdownHook will be called.
//        CloseHelper.close(runner);
        System.out.println("finished");
    }
}

@medusar you should close all non-daemon threads regardless of using a signal hander or not. The code below is how we would recommend you would implement your ExitDemo0.
import org.agrona.concurrent.Agent;
import org.agrona.concurrent.AgentRunner;
import org.agrona.concurrent.BusySpinIdleStrategy;
import org.agrona.concurrent.ShutdownSignalBarrier;

public class ExitDemo0
{
    public static void main(final String[] args)
    {
        final ShutdownSignalBarrier barrier = new ShutdownSignalBarrier();

        Runtime.getRuntime().addShutdownHook(new Thread(() -> System.out.println("shutdown hook invoked...")));

        final Agent agent = new Agent()
        {
            public int doWork()
            {
                return 0;
            }

            public String roleName()
            {
                return "agentxxx";
            }

            public void onClose()
            {
                System.out.println("onClose");
            }
        };

        try (AgentRunner runner = new AgentRunner(
            BusySpinIdleStrategy.INSTANCE, Throwable::printStackTrace, null, agent))
        {
            AgentRunner.startOnThread(runner);

            barrier.await();
            System.out.println("Shutdown Signal received");
        }

        System.out.println("finished");
    }
}


How to add multiple path conditions？Or

Hello
I am looking for way to input multiple path conditions to gjson.
or（||）path condition
Input Path
friends.#(last=="Murphy" || first=="Dale")#
Can you please explain how to provide multiple matching conditions?
Thanking You
Niren

This will help to solve my problem as well: https://github.com/tidwall/gjson/issues/194

One way to almost do it would be to create a new object with the results from both queries:
[friends.#(last=="Murphy")#,friends.#(first=="Dale")#].@flatten

The problem here is that it has duplicated items and so would need a custom unique modifier to remove them, something like #230.


Web UI Client

Hi,
I collected the image data via iPad and upload these data to my private server which is based on your git code.
Based on the guide, i can configure the webui server to handle the image data from iPad.
Guide: https://3dlg-hcvc.github.io/multiscan/read-the-docs/web-ui/index.html#ui-overview
However, when I visit to my local page as http://localhost:3030/scans/manage to see the local server data. it shows "Loading" text and  it did not show anything.

I tried to debug web browser to see the error with dataTable.Editor as following.
manage-ajax.js 324 line Uncaught TypeError, $.fn.dataTable.Editor is not constructor
manage:4

Do I need to purchase licensed dataTable.Editor ?
Otherwise, can I avoid this error to see /scans/manage webpage?
Thank you for your help in advance.
Best Regards,

Hi @bagelcafe, it is great to hear that you scanned and uploaded data from iPad.
The web-server is the backend for the web UI, to browse the uploaded data and visualize the reconstructed textured meshes, we use web-client, the frontend for the web UI.
We updated our code and documentation, so it will not need the vendor package (licensed dataTable.Editor) anymore. Please try out the updated web UI, thank you!
All the best.

Hi @SamMaoYS ,
Thank you for your support and I updated web ui client and server codes via git system.
After updating code, I can see local custom datasets from web UI server as following screen.

From now on, I wanna to annotate one of our custom datasets to click annotate button and segmentation menu selected,
However, Segment Annotator webpages is trying to connect to sfu server as following screen shot.

Basically, index.vue file indicated sfu.ca server.
src="https://aspis.cmpt.sfu.ca/stk-multiscan/multiscan/grouped?annotate=true">
How do I install and configure custom Segment Annotator on our server?
I appreciate your efforts and time for supporting this in advance.
Best Regards,
Jin.


WebSocket upgrade not working

When trying to upgrade HTTP connection to WebSocket, the following exception is thrown:
[SEVERE] conduit: Failed to send response, draining request. Reason: Bad state: Header already sent
To Reproduce

ResourceController handler in conduit

    final httpRequest = request?.raw;
    if (httpRequest == null) {
      return Response(400, {}, null); //400 - bad request
    }
    _socket = await WebSocketTransformer.upgrade(httpRequest);
    int counter = 0;
    Stream.periodic(const Duration(seconds: 1)).take(5).listen((event) {
      counter += 1;
      _socket?.add('message ${counter}');
    });

/// the doc says:
/// It's important that a request that is upgraded to a websocket is removed from the channel
/// by returning null from the controller.
    return null;

Client flutter web application invokes
final socket = WebSocketChannel.connect(Uri.parse('ws://localhost:8888/something'));

Expected behavior
Upgrade to websocket and send 5 messages to the client

I've already merged your change, but now with the CI fixed up, would you mind writing a test? I can make an attempt, but it would be nice if I had some help.

Thank you! I'll write some unit tests


DCOS-13072: Proper access denied when user has no permissions in services

This commit ensures that the user has mesosAPI and marathonAPI permissions in order to grant access to the /services page. If not, it redirects to the /access-denied page.
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run integration tests

@wavesoft this should not be handled here. Please move to plugins.

@orlandohohmeier - how do you feel about this?

@orlandohohmeier is this something we want still?

Should be reimplemented without exposing the hook in the open version.


Flowify

Chucking Flow into the project would be sweet!

Isn't this one already closed with 6ec169308d0cc24fbcfc26191d04b6e4373f49a1 ?

It is. Thanks!


Allow to customize pip repos during node image build

Make building calico/node more friendly to CI.
Since calico/node image build process hardly depends on two pip
repositories (from git):

libcalico
libnetwork-plugin
we need to have an ability to test pull-requests/patches to the
above repos against calico-containers before merge. This patch
doesn't break the default behaviour (installing from github repos),
but allows us to customize pip repositories path.

Changes:

Add ARG parameters to calico/node Dockerfile with the defaults:
ARG LIBCALICO_REPO="https://github.com/projectcalico/libcalico.git"
ARG LIBCALICO_VER="master"
ARG LIBNETWORK_REPO="https://github.com/projectcalico/libnetwork-plugin.git"
ARG LIBNETWORK_VER="master"
COPY share dir node_share inside container to /tmp/node_share
Since docker build doesn't support mounting we have to use copy
This share folder can be used to "deliver" custom pip repos
inside container
run pip install from path defined by variables
Dockerfile for node. We add NODE_CONTAINER_BUILD_ARGS which
can be used to bypass ARGs variables in format (see docker build
with --build-arg):
--build-arg LIBCALICO_VER=v0.22.0 --build-arg ...

How to use:

Simple case. Build calico/node using libcalico pip repo from
tag v0.15.0:
make node_image
NODE_CONTAINER_BUILD_ARGS="--build-arg LIBCALICO_VER=v0.22.0"

Advanced case. Build calico/node with not yet merged pull
request:
a. Prepare custom pip repo in calico_node/node_share/libcalico
by checking out libcalico code with merging pull request
b. Run build with new repo path:
make node_image
NODE_CONTAINER_BUILD_ARGS="--build-arg LIBCALICO_VER=mytag
--build-arg LIBCALICO_REPO=file:///tmp/node_share/libcalico"

I'll take a look at the space issue.

I would like to take a look at this before it's merged

Can you please rebase this?  We've tried to improve Semaphore space use a bit.

@matthewdupre @tomdee thanks, rebased - ready for review

@tomdee hi, any news on this change?

@SergK I've taken a look, I really like the idea of being able to override the sources.  Just one concern about stray node_share directories.  Maybe @tomdee or @matthewdupre can suggest a better way of doing it, which would work for release builds too.

OK, LGTM.


Add legislative maps

From Jake Walsh's email:

The second idea is adding the state legislative maps to ECCO. The maps would be put under the Locations menu and a user could select either the state House of Representatives or state Senate map. The selected map would then be overlayed over either the county or census tract maps. I already have the shapefiles for both district maps and an Excel sheet that has all of the legislators' names and contact information. I just want to see if it's possible to import these maps and overlay them in ECCO.

Since as Jake mentioned we don't have much data on a district level, there's no reason for it to be a geometry type alongside the county and tract maps. Essentially, the district regions are the data and should be superimposed over a selected measure for a county or tract.
Data required for implementation:

shapefiles of the district maps
spreadsheet with additional information for each district, e.g. the representative's name and contact info (displayed in a popup for the region)

2021_Approved_House_Plan_w_Final_Adjustments.zip
2021_Approved_Senate_Plan_w_Final_Adjustments.zip
Colorado State Representatives(Sheet1).csv
The two zip files contain folders with all of the shapefiles for both district maps. I also added an excel file that has all of the legislators' names, their districts, and their contact information. Some of the information on the excel sheet might have to be updated after the state elections but it's good enough for now.

@JWalsh75 Excellent, thank you! I was able to convert the house and senate shapefiles to GeoJSON, i.e. the format we use in the web app, without any issues; I visualized them as well and they appear to match up to the legislative boundaries I found on another resource. Both the house and senate maps' geometries were annotated with their district number, so that should be sufficient to join them to your CSV.
Regarding the updated information, do you think https://leg.colorado.gov/legislators would be a good source? It seems to contain the same data as your CSV; we could pull from it in our regular data updating pipeline. Let me know if you have another source that you'd prefer we use.

To get ahead of this, I started #81 which sets up the frontend to be able to show "area" locations in addition to the "point" locations that are already there.
At the moment, it's just hard-coding the data you provided into the frontend. @falquaddoomi feel free to work off of that PR to add this stuff to the backend. And perhaps you might want to move the "point" locations to the backend at the same time too #77.


caching_sha2_password requires either TCP with TLS or a unix socket: TRILOGY_UNSUPPORTED

I ran into something I don't quite understand the other day when making a gem compatible with trilogy: https://github.com/cainlevy/scenic-mysql_adapter/pull/2 / https://github.com/cainlevy/scenic-mysql_adapter/actions/runs/10516090228/
That gem CI uses the default mysql baked into GitHub Actions ubuntu-latest images:
    steps:
      - uses: actions/checkout@v3
      - name: Setup Ruby
        uses: ruby/setup-ruby@v1
        with:
          ruby-version: ${{ matrix.ruby }}
          bundler-cache: true
      - name: Start and create DB
        run: |
          sudo service mysql start
          sleep 10
          mysql -u root -proot -e "CREATE DATABASE scenic_mysql_adapter_test;"
      - name: Run tests
        env:
          DATABASE_URL: ${{ matrix.client<EMAIL_ADDRESS>        run: |
          bundle exec rake test

Nothing else changed in the test suite, just the same code using Active Record, one with mysql2 the other with trilogy.
The mysql2 jobs went fine, but the trilogy ones failed to connect with:
ActiveRecord::ConnectionNotEstablished: trilogy_auth_recv: caching_sha2_password requires either TCP with TLS or a unix socket: TRILOGY_UNSUPPORTED
    vendor/bundle/ruby/3.3.0/gems/activerecord-7.2.1/lib/active_record/connection_adapters/trilogy_adapter.rb:34:in `rescue in new_client'
    vendor/bundle/ruby/3.3.0/gems/activerecord-7.2.1/lib/active_record/connection_adapters/trilogy_adapter.rb:30:in `new_client'
    vendor/bundle/ruby/3.3.0/gems/activerecord-7.2.1/lib/active_record/connection_adapters/trilogy_adapter.rb:[17](https://github.com/cainlevy/scenic-mysql_adapter/actions/runs/10516090228/job/29137551437#step:5:18)4:in `connect'
    vendor/bundle/ruby/3.3.0/gems/activerecord-7.2.1/lib/active_record/connection_adapters/trilogy_adapter.rb:182:in `reconnect'
    vendor/bundle/ruby/3.3.0/gems/activerecord-7.2.1/lib/active_record/connection_adapters/abstract_adapter.rb:662:in `block in reconnect!'

So I don't know what mysql2 is doing, if it somehow accept to do caching_sha2 without TLS, or if somehow it fallbacks to another method, but I think we should do the same thing if we want to ease the transition from mysql2 to trilogy.
cc @adrianna-chang-shopify @eileencodes @jhawthorn @matthewd

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Maybe this? https://github.com/mysql/mysql-server/blob/596f0d238489a9cf9f43ce1ff905984f58d227b6/sql-common/client_authentication.cc#L695-L791

Good catch, looks like it.

Actually I wonder if this is mainly a difference in mysql2 defaulting to ssl enabled and trilogy defaulting to disabled. If I disable it I get a similar error:
Mysql2::Client.new(host: '<IP_ADDRESS>', username: 'caching_sha2', password: 'abc', ssl_mode: :disabled)
#=> Authentication plugin 'caching_sha2_password' reported error: Authentication requires secure connection. (Mysql2::Error)

AFIK this is by design by the trilogy authors, see https://github.com/trilogy-libraries/trilogy/issues/26#issuecomment-1995521936 and https://github.com/trilogy-libraries/trilogy/pull/165

Note that we have chosen on purpose to only implement the path where TLS or a unix socket is used.

Yeah, I discussed this with GitHub folks. It wasn't implemented because it's a lot of work and tricky, and not the most used part. So it was deemed better to ship the caching_sha2+TLS part and leave the raw TCP part of the protocol unimplemented for now.
I still think we should implement it at some point because if mysql-server ship with a caching_sha2 user but no SSL that may be an issue.

It would be nice to also be closer to a drop-in placement for the mysql2 gem in many cases. I for example stopped evaluating it some months ago because of this. Would probably have worked fine in prod, but would have added some complications to our dev and test setup. Still possible, but wasn't worth it.

We'd gladly accept a PR, but per-the previous PR, it is quite complicated.


Improvements to the IIIF API translation

@jronallo ping?

Actually, hold off on this.. I'm still not sure the "make it fit within" directive is right.

:ok_hand: , it should be all good now.

@jronallo :question:

Thanks @cbeer !


BUG: Switching to Wikitext breaks images with space and underscore in their filename

Steps to reproduce:

Create a new wiki page
Add an image containing a space and an underscore in its filename (example: my test_image.jpg)
Switch to Wikitext, notice the space got replaced with an underscore
Switch back to normal editing mode
Image is not shown anymore

Tested with version <IP_ADDRESS>8.
The space character is always being replaced by an underscore in the file name which works as long as both are not mixed in the same file name.
This replacement also leads to the wrong file being chosen when a second file, only differing in these two characters, is uploaded. So if a b.jpg is used and later a_b.jpg is uploaded this file will always be used, even when before it was a b.jpg.

Hello @ServiusHack we added this issue in our private bugtracker, number 44631. As soon as there is information - we will inform you.


Fix UniqBeforePluck cop

Resolves #91
Notes:

I'm not sure if we need a separate cop to change uniq for distinct or if it's fine to convert Model.pluck(:id).uniq to Model.distinct.pluck(:id) within this cop.
I'm not sure if we want to rename the cop to be DistinctBeforePluck or not, since that would be a breaking change.

@santib Sorry for the late reply.

I'm not sure if we need a separate cop to change uniq for distinct or if it's fine to convert Model.pluck(:id).uniq to Model.distinct.pluck(:id) within this cop.

First of all, I think that the current implementation is good to prevent the error.

I'm not sure if we want to rename the cop to be DistinctBeforePluck or not, since that would be a breaking change.

Yeah, certainly breaking change is not desirable. It's a bit weird, but let's keep the cop name as Rails/UniqBeforePluck. Renaming is an issue for RuboCop Rails 3.0.
Can you rebase with the latest master branch and add the changelog entry?

@koic updated. I put it in the "bug fixes" section since it could be considered a bug for newer versions of rails, but let me know if you want to change it or something else

@santib Thank you for the updating. Can you add VersionChanged: '2.6' to config/default.yml?
https://github.com/rubocop-hq/rubocop-rails/blob/v2.5.2/config/default.yml#L182

@koic done :)

Nicely done, Thank you!


ubuntu 18.04 support

Hi there, fails on bionic, please check.
/root/.chef/local-mode-cache/cache/cookbooks/poise-languages/files/halite_gem/poise_languages/system/mixin.rb:

 80:          end
 81:
 82:          # Find the first value on system_package_candidates that is in system_packages.
 83:          system_package_candidates(options['version'].to_s).each do |name|
 84:            return name if system_packages.include?(name)
 85:          end
 86:          # No valid candidate. Sad trombone.
 87>>         raise PoiseLanguages::Error.new("Unable to find a candidate package for version #{options['version'].to_s.inspect}. Please set package_name provider option for #{new_resource}.")
 88:        end

@coderanger would you please publish a release that incorporates this fix?

@jeffbyrnes Asking for a release is almost never a good plan. I will get to it when I get to it.


Support zone id when parsing / formatting SocketAddrV6

SocketAddrV6 contains a field scope_id (AKA zone id in RFCs).
In RFC4007, a textual representation for an IPv6 address with a zone id is specified, where the zone id is appended to the ipv6 address with % as a separator.
I think it would be very useful and quite easy to add parsing of a numeric zone id to read_socket_addr_v6.
For improved usability, adding support for interface names would be nice, but those would need to be converted to interface indices using OS specific interfaces (like libc::if_nametoindex on linux).
When formatting a SocketAddrV6, the zone id could be printed if it's not 0. (But that could be seen as an incompatible change.)

A related problem is around parsing a Ipv6Addr address with a zone index. This panics in Rust now but should be parseable.
cc https://github.com/rust-lang/rust/issues/32904

Yes, this is not just related, it's exactly the same issue. Thanks for pointing it out, I didn't know that ticket.
However, I think this should be handled in SocketAddrV6, not Ipv6Addr, for these reasons:

SocketAddrV6 already contains the field scope_id, so the fix would only affect parsing / formatting code, not the data structures.
Most everyday usages of IPv6 addresses don't need a scope id, so making Ipv6Addr more complicated may do more harm than good.
There are many cases where a scope id can't easily be communicated in a meaningful way, e.g. in URLs. This is because scope ids are host-specific, and only minimally specified. RFC6874 lists some of the shortcomings. Chrome and Mozilla chose not to implement them in URLs. So applications may or may not want to support scope ids when parsing IPv6 addresses.
A SocketAddrV6 is a more technical thing, less likely to be presented to end users and more likely to be used in situations where a scope id is neccessary. (Well - I can't give a citation for this claim. It's just my experience.)


How to wrap   in custom component and keep it's styling.

Hi, i have a component which has a lot of Content and i thought it will be better if I refactor
<Footer> component as it had lots of functions and buttons to navigate around and i need to reuse it.
So I wrappered it in another component (FooterWrapper) and imported it in the parent component (ParentComponent) it was first in.
In the FooterWrapper component i return <Footer> .... </Footer>; and in the parent component i call <FooterWrapper> inside it's <Content> block.
It mainly looks like this.
export class FooterWrapper extends Component {
      render() {
         return <Footer>
                          // Some code
                </Footer>
       }
}

Then in the parent i have

export default class ParentComponent extends Component {
     render() {
           return (
                 <Container>
                        <Content>
                               // So much stuff
                               <FooterWrapper />
                         </Content>
                  </Container>
              );
         }
}

Problem is, Footer gets to be shown but without Footers behaviour, as in it's not on the bottom of the page but it's on the end of everyother component.
But i need to reuse the footer and still keep it in the bottom.. I don't understand flex very much too and react-native newbie.
Please help or give reference. :)
Thanks.

@maotora Try wrapping FooterWrapper in View
Also looking into your code, put your FooterWrapper outside Content

@maotora NativeBase as of now does not support custom components.
For now you can do something like below example:
<Container>
  <Content>
    // So much stuff
  </Content>
  <Footer>
    <FooterWrapper />
  </Footer>
</Container>

@SupriyaKalghatgi Hey your solution worked perfectly!
I was about to get into <Footer>'s styling so i can copy it to my code.
But this is awesome!
Thank you very much! :)

Thanks, helpful. I would like to create a component for my header, but the above approach does not work since header needs an enclosing tag:

Send Inspection

Wrapping the above in a View tag causes the Header not to render properly (styling messed up and right button not displayed). Any ideas on how I can fix this?

I'm also having this kind of problem. How to wrap a footer so I don't have to repeat on every screen?

@mrpatiwi Answered above:
https://github.com/GeekyAnts/NativeBase/issues/297#issuecomment-261163999

Not working:
const TABS = [{
      name: 'Home',
      icon: 'ios-home',
      scene: 'home',
    }, {
      name: 'Maps',
      icon: 'ios-map',
      scene: 'events',
    }];

class TabBar extends Component {
  onPress = (tab) => {
    // ...
  }

  render() {
    return (
      <View>
        {TABS.map((tab, index) => (
          <FooterTab key={tab.name}>
            <Button transparent onPress={() => this.onPress(tab, index)}>
              {tab.name}
              <Icon name={tab.icon} />
            </Button>
          </FooterTab>
        ))}
      </View>
    );
  }
}

class Home extends Component {
  render() {
    return (
      <Container>
        <Header>
          <Title>Home</Title>
        </Header>

        <Content></Content>

        <Footer>
          <TabBar />
        </Footer>
      </Container>
    );
  }
}

This works, but is not very reacty:
const TabBar = () => {
  const onPress = (tab) => {
    // ...
  };

  const tabs = [{
    name: 'Home',
    icon: 'ios-home',
    scene: 'home',
  }, {
    name: 'Maps',
    icon: 'ios-map',
    scene: 'events',
  }];

  return tabs.map((tab, index) => (
    <FooterTab key={tab.name}>
      <Button transparent onPress={() => onPress(tab, index)}>
        {tab.name}
        <Icon name={tab.icon} />
      </Button>
    </FooterTab>
  ));
};

        <Footer>
          {TabBar()}
        </Footer>

I found my error, I was creating multiple FooterTab.
Final and working code:
class TabBar extends Component {
  onPress = (tab) => {
    // ...
  }

  render() {
    const { ...props } = this.props;
    const tabs = [{
      name: 'Home',
      icon: 'ios-home',
      scene: 'home',
    }, {
      name: 'Maps',
      icon: 'ios-map',
      scene: 'events',
    }];

    return (
      <FooterTab {...props}>
        {tabs.map((tab, index) => (
          <Button key={tab.name} transparent onPress={() => this.onPress(tab, index)}>
            {tab.name}
            <Icon name={tab.icon} />
          </Button>
        ))}
      </FooterTab>
    );
  }
}

       <Footer>
          <TabBar />
        </Footer>

@jesouhaite08 It is mentioned earlier that, NativeBase as of now does not support custom components. This has been on our list, to be fixed with the rewrite of NativeBase.
For now you can wrap the custom Header/Footer components in View and this requires some extra styles.
<Container>
  <Header>
    <MyComponent />
  </Header>
  <Content padder>
    <Text>Custom Header and Footer</Text>
  </Content>
  <Footer>
    <MyComponent />
  </Footer>
</Container>

MyComponent.js
<View style={styles.headerAndFooter}>
  <Button transparent>
    <Icon name="ios-arrow-back" />
   </Button>
   <Title>Title</Title>
   <Button transparent>
     <Icon name="ios-menu" />
   </Button>
 </View>

Hey guys - I am not able to get any more than 3 tabs for iOS - using this approach:
  <FooterTab>
    <Button active vertical onPress={() => handleTabClick(data.state)}>
        <Text>{data.text}</Text>
        <Icon name={data.icon} />
      </Button>
  </FooterTab>

Tried most every approach for flex box to make it behave, but nope.


Avoid warning message when bundle install

This pull changes avoid the following messages.
Your Gemfile lists the gem pry (>= 0) more than once.
You should probably keep only one of them.
While it's not a problem now, it could cause errors if you change the version of just one of them later.

Pings:
@cheerfulstoic
@subvertallchris

Great catch, thanks!

Thank you for merging :smile:


Feature Request Link our own private dns zones with "virtual_network_resource_ids_to_link"

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to a problem?
Hi guys,
I would like to link my own private dns  zones on a other  virtual network resource, when i use the functionality  "virtual_network_resource_ids_to_link" the entreprise scale module link these own dns private but i also want to the privates dns zones that i've added myself.

Thanks a lot in advance,
Describe the solution you'd like
If it's possible to link my own private dns zones to an other virtual network resources id whit the option "virtual_network_resource_ids_to_link"
Additional context

Just tested it and it links both types of dns private zones, the ones created by the module and the ones you tell the module to create from the variable dns.config.private_dns_zones


docs: add glossary to specifications

⚠ Moved: https://github.com/stardust-ui/react/pull/4

Added initial glossary specification. We can split this later into multiple files.

⚠ Moved: stardust-ui/react#4


How many time did you spend on testing?

Hello, I used your pretrained model to test on the dataset on a gpu(Tesla K80) , but cost a long period without a result. Can you give me the actual testing time of the model?

Hi,
the method seems to be unable to find valid hypotheses, which indicates that the scene coordinate prediction failed.
Most likely there is something wrong with the LUA/CUDA/C++ communication on your machine. I suspect the LUA methods do not run through (unfortunately there is no proper error handling in the code which means that LUA errors will just be swallowed by C++). To debug this, you could add console outputs (e.g. print('done')) to the end of each LUA method in train_obj.lua and train_score.lua.
Btw. I assume you are using the original DSAC code? However, this is the DSAC++ repository.
Kind regards.
Eric


Make bulk/testport -b URL not only support official repositories, but any repository

Prerequisites

[X] Have you checked for an existing issue describing your idea?

What is your proposal?
Remove the terms "official only" from this option and if any of the symbolic names, latest, quarterly, release_X keep repo name FreeBSD and enforce SRV RR, signatures, but when a custom URL is provided, set FreeBSD to no, add a new repo, e.g., custom where priority is higher (10?) and hardcode mirror_type, pubkey, signature_type to NONE. A more sophisticated way is to accept a file path where the admin can supply custom .conf which contains arbitrary repos copied to LOCALBASE into the jail. In this case it needs to be clarified how to provide public keys, if required. (This will provide nice layering of repos)
What is the existing behavior, if any?
When providing the custom, selfhosted repo with -b <URL> the fetch fails because my packages aren't signed which I don't need nor is this an official repo.
What is the motivation / use case for the change?
I build a base set of ports with custom options because the default ones do not work for our environment. Those options aren't only for direct, but also for automatic ones. On top there are several other use cases (sets) which use those base ports partially as dependency which I don't want to build over and over again. The option for bulk/testport -b <URL> expects only official repos which quite limits this nice feature. I want to use it with any selfhosted repo which will satisfy the fetch requirements.
Did you consider any alternatives?
Yes, patching common.sh, but this is an ugly hack. More specificly, in https://github.com/freebsd/poudriere/blob/5f69efba7802c412193cecf2d00a34fd77bb56fd/src/share/poudriere/common.sh#L3805-L3809 I had to switch of signature check and DNS SRV RRs to use my custom repo.
Hack:
@@ -3804,7 +3797,10 @@
    fi
    cat >> "${MASTERMNT}/etc/pkg/poudriere.conf" <<-EOF
    FreeBSD: {
-           url: ${packagesite};
+           url: ${packagesite},
+            mirror_type: "NONE",
+            pubkey: "NONE",
+            signature_type: "NONE";
    }
    EOF

Is this really a ports feature request?
Yes, it is a poudriere issue.
Additional context
This was found in https://github.com/freebsd/poudriere/issues/883.
Real world example
port set ldadw_base:
poudriere bulk -j 123-release-amd64 -z ldadw_base -O ldadw-custom -f /usr/local/etc/poudriere.d/123-release-amd64-default-ldadw_base-ports

with the following in /usr/local/etc/poudriere.d/:
123-release-amd64-default-ldadw_base-make.conf
123-release-amd64-default-ldadw_base-options/
123-release-amd64-default-ldadw_base-ports

123-release-amd64-default-ldadw_base-ports:
devel/git
devel/subversion-lts
devel/tig
editors/vim
editors/vim-colorschemes-legacy
ftp/curl
lang/python3
lang/python37
devel/py-pip
misc/mc
net/openldap26-client
security/krb5
shells/bash
shells/bash-completion
sysutils/htop
sysutils/msktutil
sysutils/tree
textproc/colordiff
textproc/rubygem-asciidoctor
net-mgmt/zabbix5-agent
security/siemens-cacerts
security/nss-siemens-cacerts-java
dns/register-hostnames
sysutils/system-update-periodic

Available internally at https://deblndw011x.ad001.siemens.net/FreeBSD/packages/${ABI}/ldadw-base
Now for a specific purpose bastille jail I have the following:
poudriere bulk -j 123-release-amd64 -z ldadocgen -O ldadw-custom -f /usr/local/etc/poudriere.d/123-release-amd64-default-ldadocgen-ports -b 'https://deblndw011x.ad001.siemens.net/FreeBSD/packages/${ABI}/ldadw-base'

with the following in /usr/local/etc/poudriere.d/:
123-release-amd64-default-ldadocgen-make.conf
123-release-amd64-default-ldadocgen-options/
123-release-amd64-default-ldadocgen-ports

123-release-amd64-default-ldadocgen-ports:
devel/apache-ant
java/openjdk8
math/gnuplot-lite
misc/getopt
net/py-ldap
www/apache24
www/mod_auth_gssapi
www/mod_wsgi4
www/py-flask
www/py-requests
net/py-urllib3
www/tomcat85
www/tomcat-native
x11-toolkits/libXmu
x11-fonts/fontconfig
print/ghostscript9-agpl-base

123-release-amd64-default-ldadocgen-options/ contains merged options from ldadw_base and ldadocgen to satisfy -b's requirements. Many of the ports in ldadocgen will use a lot from ldadw_base, e.g., apache24, tomcat85, tomcat-native will use apr which in base has been pulled by subversion already along with many other build dependencies , e.g., ninja, cmake, GNU autotools, etc.

@bdrewery As promised, here is the issue to improve -b.

@bdrewery gave it some thoughts today since I don't want to live with a patch forever. Here is my idea how we could move this forward: As far as I understand only a single repo is supported since packagesite is used throughout, would be much more work to rewrite to multiple -b options or supplying an external file. Compromise: If a URL is passed, don't threat is as official, disable FreeBSD repo and add a repo custom using that supplied URL.
WDYT? I'd be willing to work on a PR if this is acceptable for you...

I found this issue on my way to open my own.
I came up with the same type of fix:
        FreeBSD: {
                url: ${packagesite};
                mirror_type: NONE;
        }

I use -b https://myownbinaryrepo
Seems like an easy fix would be to detect that -b is set to an URL and disable SRV in that case

I found this issue on my way to open my own.
I came up with the same type of fix:
        FreeBSD: {
                url: ${packagesite};
                mirror_type: NONE;
        }

I use -b https://myownbinaryrepo
Seems like an easy fix would be to detect that -b is set to an URL and disable SRV in that case

That was my observation as well, but does not solve my wish situation where still only one repo is supported.


Edits of project settings are not persisted.

Description
Changes to project settings via Options -> Edit -> Save are not persisted.
To Reproduce
Steps to reproduce the behavior:

Go to Options
On a project entry, click on 'Edit'
Make a change to title
Click on 'Save'
=> change is not persisted

Expected behavior
Changes get persisted.

:tada: This issue has been resolved in version 5.1.5 :tada:
The release is available on:

GitHub release
v5.1.5

Your semantic-release bot :package::rocket:


Base URL "doubling" when using Cossack::Client

Issue Summary
Am trying to use Hi8 to automatically record and playback HTTP interactions for a Web service client, but am using Cossack rather than just HTTP::Client.
The first time the spec is executed, it passes and HI8 records the interaction as expected. However, subsequent test runs fail with Real HTTP connections are disabled. Unregistered request: ....
Further investigation reveals that the recorded uri in the cassette has the base URL "doubled". E.g., "https://api.fixer.io/latest" becomes "https://api.fixer.iohttps://api.fixer.io/latest".
The same behaviour was not observed using whether using HTTP::Client.get() or HTTP::Client.new().
Sample code
    HI8.use_cassette("using_cossack_client") do
      client = Cossack::Client.new("https://api.fixer.io")
      resp = client.get("/latest?base=USD")
      resp.status.should eq(200)
    end

The full gist can be found at here
Expected behaviour
Spec should pass.
Actual behaviour
Real HTTP connections are disabled. Unregistered request: GET https://api.fixer.iohttps://api.fixer.io/latest?base=USD with headers {"User-Agent" => "Cossack v0.1.4", "Host" => "api.fixer.io"}

Environment

Hi8 commit: 1e9981ab94846ffa2e6f6635f2649ac51d42d3b6
Crystal 0.24.1
Cossack 0.1.4
macOS 10.13.3

So I've traced through this and I seem to have isolated where the problem occurs.
In HI8::HTTPHelpers#uri_creator (episode.cr:15) it concatenates host + resources.
For some reason, when using Cossack the @request.resource.to_s (episode.cr:47) also embeds the host. Hence, the "doubling" when the interaction is recorded.
Not sure if this is a Cossack issue or whether it should be treated as a special case by HI8.

Your fix works for me.  I'll close this out.


Build the tests with release=8 whenever it is possible

Fixes https://github.com/eclipse-openj9/openj9/issues/19888
The tests have to be built with release=8 in order to avoid calling the new overridden methods with covariant return types in java.nio.ByteBuffer/CharBuffer/LongBuffer, like position(int), rewind, flip...
The only source file that has to be built with source=1.8 and target=1.8 is the gnu/testlet/java/lang/Class/classInfo/getDeclaredMethod.java, because it requires sun.reflect.annotation.AnnotationType that is not visible using release=8 from jdk9+
The specifying of release="8", source="1.8" and target="1.8" allows ant 1.10.x to use either release if supported or the source/target if release is not supported.

@fridrich Are you able to sign the Eclipse CLA?

Thanks @fridrich for providing the fix. Could you try to sign ECA? https://github.com/adoptium/aqa-tests/blob/master/Contributing.md#eclipse-contributor-agreement
It should be a straightforward process. Unfortunately, we cannot accept the change without the ECA signed.

@annaibm Please help to test this PR. Thanks

FYI the comment which I'll copy here.
fridrich commented Sep 23, 2024
So, that pull request basically fixes the build of the mauve.jar, so that it works with the java 8. It is hanging in eclipsefdn/eca. I am not sure whether we (SUSE) have ever signed something like this. I also don't care to be credited, so anybody who has the right paperwork on file can submit that patch. I transfer automatically the copyright to anybody brave enough to get it integrated.

At the end, I signed it in my personal capacity. So it should be good I think.

Quick update: We encountered other issues while upgrading to JDK 17 for compilation for all dependent jars (though mauve.jar is fine). This PR can't be merged until those issues are resolved.

@smlambert we are ready to merge this PR.


imu issue

Why 3D cartographer must have an IMU，but 2D cartographer can be not?

In 3D, the IMU integration is part of the optimization. In 2D, it is used only to provide an initial estimate for scan matching and can be replaced with correlative scan matching.


cast sig - param of uint256 and uint return the same selector

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (04ce078 2023-04-17T00:04:57.209269000Z)
What command(s) is the bug in?
cast sig
Operating System
macOS (Apple Silicon)
Describe the bug
When using cast sig with the uint param type it returns the same signature as the uint256 type.
Example:
cast sig "test(uint)" returns 0x29e99f07
cast sig "test(uint256)" returns 0x29e99f07
but bytes4(keccak256("test(uint)")) returns 0x1acb6f77
Not sure if this was done on purpose as uint is an alias for uint256 but there is a difference when generating the function selectors.

This is intended since it's just an alias for "uint256". (newer versions of?) Solidity does the same, treats "uint" and "uint256" in a function sig as the same and throws an error if both functions are present in the same contract.

Whilst I agree if you're using the types directly it differs if you use strings or encodeWithSignature to generate the selector.
Say you do a low level call:
address.call(abi.encodeWithSignature("test(uint)", amount))
The signature this generates for uint and uint256 is different. So if the contract you're calling has the signature test(uint256) it won't match the signature we generated above.

I don't see where the problem is if Solidity generates "uint256" in both cases. You shouldn't be using "uint" in the first place, and if you really have to, you can use the manual keccak256

I don't follow this comment? "I don't see where the problem is if Solidity generates "uint256" in both cases."
The problem is with cast in the end the utility shouldn't return the same sig for both uint and uint256 because if you create the selector yourself bytes4(keccak256("test(uint)")) vs bytes4(keccak256("test(uint256)")) the selectors are different.

Solidity generates the same ABI for both uint and uint256, namely both are encoded as uint256. "cast sig" generates a selector from a solidity function signature by parsing it as such. It is intended that it has the same behaviour as Solidity.
What it doesn't do is run "keccak256(bytes4())" as if it were a raw string. If you want that behaviour, you can use this command instead to hash the raw bytes:
cast sha3 "sig()" | cut -c-6

Closing issue, above makes sense. Thanks @DaniPopes


Redis: Support publish and subscribe methods

While running the examples from the blog post How to instrument a polyglot microservices application with Elastic APM, I noticed Redis doesn't show up on the service map as being connected to the Python service in this example.
It looks like that's because according to our documentation we don't have the publish and subscribe methods instrumented.
If these methods were instrumented we would be able to see Redis on service maps for applications that are using it for pub/sub.

Thanks for the request!


Adding a QuadEM PVGroup and using this in the BaffleSlit PVGroup

This PR aims to add a QuadEM PVGroup, to simulate the PVs from an NSLS-II standard quad electrometer, and to use this as a subgroup inside the BaffleSlit IOC server.
As the quad EMs have areadetector based IOCs this unfortunately meant I had to implement a bunch of PVs. To do this I have added an 'areadetector' sub-package containing plugin_base, stats_plugin and quad_em modules. This will allow for the re-use of much of the code for these in future areadetector IOCs. Some notes on connections between the various PVs in the QuadEM PVGroup are given below.
NOTES:
1. Unless otherwise listed below the PVs generated in this PR are 'Dummy' PVs
that are not modified by any inputs, or modify any other PVs, except there own
values when they are updated.
2. In QuadEM when self.acquire is set to 1 the sequence of events is:
    i. record the initial time and set self.num_averaged to 0.
    ii.  calculate the current to be set on each channel using self._generate_current
    and write these to the self.current(x).mean_value attributes (x in [1,2,3,4])
    iii. if self.averaging_time has elapsed continue otherwise wait until it has.
    iv. set self.num_averaged to self.num_average and self.acquire to 0
3. In QuadEM when self.averaging_time or self.integrating_time are updated self.num_average is
updated using the following relationship:
    - self.num_average = floor(self.averaging_time/self.integration_time)

@Jiemin-Li and @wen-hu please review this at your convenience. Comments/suggestions from others are strongly encouraged as well !-).

Sorry for the bunch of new commits. I noticed an issue whereby there was a 'Caproto_servers' directory and and a 'caproto_servers' directory. The new commits resolve that and merge them.

@Jiemin-Li thanks for the comments, I respond below.

Two quick comments:
1. The area_detector.plugin_base and area_detector.stats_plugin come from the Ophyd package, right? If so, should we add 'ophyd.' to the front of these two imports?

Yes and no, there are class objects within ophyd with the same names but they are not the same as the ones in this PR. The items in ophyd are for creating 'ophyd objects' that talk to a bunch of PVs. I created new objects in this package that create the PVs that those ophyd objects will talk to and used the same names. So long story short, the references here are for the objects from this package which have the same name as the objects in ophyd. This package will be run in a different python session than the ophyd objects so it is not necessary (and not advised) to reference ophyd in this package !-).

2. In the function of _reset_num_average(self), there is no argument after 'return', is it following the python syntax?

A quick google search showed this statement on python return usage:
You can use explicit return statements with or without a return value. If you build a return statement without specifying a return value, then you'll be implicitly returning None
So in this case it is fine, although not strictly required as I do not use:
some_object=_reset_num_average(self)
On the off chance that we do use it in that way later I would prefer to keep the 'return' at the end, is this ok with you @Jiemin-Li ?

@wen-hu please take a look at this when you get a chance.


Dial to multiple destinations dropping calling party after pickup

I am trying to dial two endpoints:
dial<EMAIL_ADDRESS><EMAIL_ADDRESS>:from => 'tel:+13035551212'
Both end points ring, I answer the tel one and the sip one stops ringing. But, as soon as I do this the calling party leg drops but the tel leg stays up.
https://gist.github.com/843eafe3f387e29c4c1a

@jsgoecke writes: "hey guys, on this issue Jose confirmed Adhearsion is hanging up the call"

[2/17/12 2:58:49 PM] Ben Langfeld: So A calls ahn, does a dial to B & C, B picks up, C stops ringing, A gets hung up by ahn, right?
[2/17/12 2:58:49 PM] Ben Langfeld: I can't find the cause right now
[2/17/12 2:59:10 PM] Ben Langfeld: If more details about the observed behaviour can be compiled, I can investigate in the morning

alice -> Adhearsion
Adhearsion accepts alice
Adhearsion uses the 'dial' method in the DSL to call bob/mobile and bob/web
bob/mobile and bob/web both ring
bob/mobile answers
bob/web and alice hangup
bob/mobile stays connected

Based on your updated description, I have a hunch that the call is trying to bridge before being answered (accepted, but not answered).  Investigating...

Found it. CallController#dial blocks on a CountDownLatch. That latch is counted down each time a dialed call ends. As such, that latch needs to be the same size as the number of dialed calls. It's currently hard-coded as 1, so the first outbound call being hung up unblocks #dial and then we hang up the original call.
We need to change that so #dial blocks until all outbound calls complete.
The other issue is that hanging up the original call does not hang up the outbound calls. I've filed a separate issue for that here: #102


chore: remove bindCallback workaround

Description:
This PR removes the workaround type that was added for bindCallback in #6050. TypeScript is correct that the signatures aren't compatible because callback arguments are contravariant.
Related PR: #6050

BTW, this relies upon a 4.2 feature, so TS 4.2 is our minimum supported version. For 4.1 to be our minimum, we'd have to re-instate the workaround, as 4.2 deems our previous signatures to be incompatible.

BTW, this relies upon a 4.2 feature, so TS 4.2 will need to be our minimum supported version.

It seems like the output .d.ts files would be okay for 4.1 though, no? Either way, this seems like what we want to land in 7. So... :+1:


Integration with HA

Hello,
It is possible to integrate in HomeAssistant?
As I know it is support MQTT as well.
Thank you in advance.
Regards,

Yes you can either use the MQTT and add the topics yourself.
I doubt there is any auto-discovery for the KM50/KM100/KM200 topics built into the buderus2mqtt bridge yet.


Test Cases

Starting to create test cases for the game.
Current ones done:

Test that the generated deck card count is 52

Ones to do:

Test blackjack score with different combination of cards.

Hitting a hurdle with the blackjack score test. I have a draw phase which sets the score to 21 if they initially draw an Ace and a ten value card. Issue is that I pass through the deck which isn't easy to test because I want to pass specific cards that I make for testing purposes.
This is why I have been using the Hit method instead because I can pass through specific cards that I generate. That method does not have the logic to scan over the entire hand to check for the presence of an Ace and Ten value card. Either I remove the concept of a "Draw Phase" and just call the hit method with that logic in there or I repeat the logic in the hit method.

NVM. I made a big mistake into thinking that Ace was 10 or 1 this entire time. Changing the default to 11 and altering the corrector minus if it is over 21 in Hit method makes test pass.
https://github.com/rwx-yxu/blackjack/issues/1#issuecomment-1367678500

There is an opportunity to reduce the logic code in the DrawPhase method by calling Hit from it rather than using incorrect game logic to check if the cards being drawn are aces and ten value cards.

New tests added:

Card names match
Card values are set to defaults
Deck size is reduced by one of a card is drawn
The card received is of the card type struct.