Can't generate tables from Columns of not Arrays

I'm not sure if this is intended. I can create a Columns object even if the arguments are not strictly arrays, but then I cannot convert it to table:
julia> c = Columns(1:10, rand(10))
10-element Columns{Tuple{Int64,Float64}}:
 (1, 0.187685)
 (2, 0.364163)
 (3, 0.650937)
 (4, 0.646284)
 (5, 0.995686)
 (6, 0.700621)
 (7, 0.217061)
 (8, 0.935344)
 (9, 0.859523)
 (10, 0.414292)

julia> table(c)
ERROR: MethodError: Cannot `convert` an object of type Array{Int64,1} to an object of type UnitRange{Int64}
This may have arisen from a call to the constructor UnitRange{Int64}(...),
since type constructors fall back to convert methods.
Stacktrace:
 [1] IndexedTables.Columns{Tuple{Int64,Float64},Tuple{UnitRange{Int64},Array{Float64,1}}}(::Tuple{Array{Int64,1},Array{Float64,1}}) at /Users/pietro/.julia/v0.6/IndexedTables/src/columns.jl:25
 [2] similar(::IndexedTables.Columns{Tuple{Int64,Float64},Tuple{UnitRange{Int64},Array{Float64,1}}}) at /Users/pietro/.julia/v0.6/IndexedTables/src/columns.jl:189
 [3] copy(::IndexedTables.Columns{Tuple{Int64,Float64},Tuple{UnitRange{Int64},Array{Float64,1}}}) at ./abstractarray.jl:698
 [4] #table#91(::Array{Int64,1}, ::Void, ::Array{IndexedTables.Perm,1}, ::Bool, ::Bool, ::Array{Nullable{Float64},1}, ::Function, ::Val{:serial}, ::Tuple{UnitRange{Int64},Array{Float64,1}}) at /Users/pietro/.julia/v0.6/IndexedTables/src/table.jl:214
 [5] (::IndexedTables.#kw##table)(::Array{Any,1}, ::IndexedTables.#table, ::Val{:serial}, ::Tuple{UnitRange{Int64},Array{Float64,1}}) at ./<missing>:0
 [6] #table#95(::Void, ::Array{Any,1}, ::Function, ::Tuple{UnitRange{Int64},Array{Float64,1}}) at /Users/pietro/.julia/v0.6/IndexedTables/src/table.jl:247
 [7] #table#96(::Array{Any,1}, ::Function, ::IndexedTables.Columns{Tuple{Int64,Float64},Tuple{UnitRange{Int64},Array{Float64,1}}}) at /Users/pietro/.julia/v0.6/IndexedTables/src/table.jl:250
 [8] table(::IndexedTables.Columns{Tuple{Int64,Float64},Tuple{UnitRange{Int64},Array{Float64,1}}}) at /Users/pietro/.julia/v0.6/IndexedTables/src/table.jl:250

The issue I believe is due to the fact that similar changes type of its argument sometimes. For example similar(1:10) is an Array. Same happens when one or the columns is a SubArray.

We could add another type parameter for the columns_buffer field... Or, we could just not parameterize the type and use Any for this field... The purpose of this buffer is to allow for mutation operations (esp, push! and append!) to be stored in the buffer, then a flush! function would "merge" these buffered changes to the main table

My intuition (as in #106 ) is that there are two cases:

All columns are Arrays, everything works
Not all columns are Arrays. There should probably not be a buffer at all in this case as you can't push! to a view or a range.

With the changes in #106 the table constructor when called with copy = true converts to Arrays all the columns (table(1:10) is equivalent to table(collect(1:10)).  table(1:10, copy = false) still errors, not sure if we want to change that. view(t, 1:2) in #106 is the only way to get a NextTable of type 2.

Consider the case where you have PooledArrays. They can be push!ed to.
Would be nice to have table(1:10, copy = false) working.

I see your point. I've been inaccurate in my previous post, actually PooledArrays also work: the sufficient and necessary condition for the current implementation is that:
typeof(v) == typeof(similar(v))
Which is true for Arrays and PooledArrays and false for Ranges and SubArrays. If there are examples where that condition is not valid but still it makes sense to push! or append!, then I agree we need a different solution.
What do you think table(1:10, copy = false) should do? Create a table of type 2 without buffer?

At this I point I think the best way forward is to make the columns_buffer field Any :)
With that, table(1:10, copy=false) would create a table using the UnitRange as a column, then with your fix in #106 things should just work.

Deal :) I'll update the PR. So instead of putting nothing in the columns_buffer I should simply put whatever similar gives.

Yeah. If someone tries to push! on a table of subarrays, then that will crash. But that's OK, it's supposed to anyway.


Support for adding a value to the user agent string when using AppConfigurationClient

When using AppConfigurationClient, I'd like to add an additional value to the user agent string sent with the app configuration requests. That would enable us from the app configuration side to distinguish my feature's usage when looking at logging/telemetry.
Note: When making AppConfigurationClient calls, I found that I can use customHeaders to replace the user agent (but that will lose the part of the user agent string that the SDK would have sent)

Thanks for reporting @et13
@bterlson, I believe the guidelines around user agent string allows end users to prefix/suffix custom strings to the value being set as user agent string. Can you confirm?

@richardpark-msft The pipeline options should already allow this as far as I know


Simplify Automated Dependency Updates

Tools like renovate or dependabot have a hard time generating automated dependency updates for flapdoodle artifacts due to the fact that the spring boot version is contained in the artifactId (e.g. de.flapdoodle.embed.mongo.spring30x and de.flapdoodle.embed.mongo.spring31x).
Would it be possible to release flapdoodle under a single artifactId and use the version to differ between spring releases?
E.g. something like:
<dependency>
    <groupId>de.flapdoodle.embed</groupId>
    <artifactId>de.flapdoodle.embed.mongo.springboot</artifactId>
    <version>4.10.1</version> <!-- for spring boot 3.1.x -->
</dependency>

<dependency>
    <groupId>de.flapdoodle.embed</groupId>
    <artifactId>de.flapdoodle.embed.mongo.springboot</artifactId>
    <version>3.10.1</version> <!-- for spring boot 3.0.x -->
</dependency>

@DarkAtra As the code does not differ for spring 3.0.x and 3.1.x you could just change the spring version. It would be better if the spring version would NOT be contained in the artifact.. but i am not sure, if everyone would be happy with this change.
If you have any idea how to solve this, i would welcome a PR for that:)

hey @michaelmosmann, well the only thing i could think of is what i suggested above, e.g. relying on the artifact version to distinguish between spring boot releases and keeping the artifactId constant. However, i totally get that this might make others unhappy and is rather drastic.
Could you give me a few details about the release process? It seems like it's not automated with Github Actions and i also didn't find anything in the Jenkinsfile. Is it a manual process atm?

@DarkAtra yes.. its a manual process atm..

Another idea would be to leave the old versions as is, and only switch to the versioning described above for new releases (one for Spring Boot 3.2 is probably in sight). I think I could prepare a PR for this when I find some time.

@DarkAtra so would you expect a release with every new spring release?

@DarkAtra so would you expect a release with every new spring release?

only if changes are necessary to keep the library functional

@DarkAtra so the artifact could be de.flapdoodle.embed.mongo.spring3x instead with just rolling version numbers?

@DarkAtra so the artifact could be de.flapdoodle.embed.mongo.spring3x instead with just rolling version numbers?

at least if there are no changes expected in sb 3.x. i'd personally stick with de.flapdoodle.embed.mongo.spring just in case a minor SB release introduces breaking changes but your idea would probably also work

@DarkAtra i will think about that.. an other way would be mark spring as provided so the spring version does not come with the dependency..

i've switched to the new artifactId. feel free to close the issue, i will re-open if necessary - e.g. if dependabot and renovate still don't raise automated dependency updates


Coracle sites not loading (coracle.social & app.coracle.social)

@staab FYI, please see GIF of experience below within CDN link. This is the 2nd day has not been loading for me.
Tried various browsers.
https://cdn.satellite.earth/250a67c37f8ad7c21f50318defddf9b0a5b71a9cd80e30fa6944c1b617363059.gif

Odd, are you using a VPN or anything?

Not using VPN. Tried laptop over Wifi, using incognito and standard from various browsers. Also tried mobile data only on phone, over Safari, but still getting the same issue.

Could you send me a screenshot of your devtools?

Sure. This is the console view from Google Chrome.

Forgot to send Network view. Here goes:

Thanks for the extra information! Looks like this only happens when you're not logged in. Pushing a fix for this now.

Awesome! Thanks.


fix(versions): update Activiti/activiti-cloud-example-chart-modeling versions into master

UpdateBot pushed helm dependency: activiti-cloud-modeling to: 7.1.91

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-modeling 7.1.91

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-x-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-modeling 7.1.92


Can not Sync Python runbooks from Azure Devops Git Repo

Hello! I hope this has an obvious solution. I am implementing source control between our Visual studio/Azuredevops Git repository and it has been working great. I just implemented a python environment and as soon as i added some .py files, the sync job fails fails. It will successfully synchronize the .ps1 files with no issue, but .py throws the following error.
Exception: Cannot validate argument on parameter 'Type'. The argument "Python2" does not belong to the set "Graph,PowerShell,PowerShellWorkflow" specified by the ValidateSet attribute. Supply an argument that is in the set and then try the command again.
This even occurs with python example files that existed in my azure automation repository that were exported... and then imported into the devops repo.
Any  direction you can provide would be appreciated!
Thank you for your time!

@Csmithson-qh Thank you for your feedback. Could you link the URL of the documentation you were following? That way, we can pass your feedback to the appropriate content author.

sure thing! its https://docs.microsoft.com/en-us/azure/automation/source-control-integration
I apologize I thought hey were linked, this was at the bottom of the page.

adding @georgewallace

@csmithson-qh Thanks for reaching out.. I just confirmed that syncing a python runbook works just fine.. Have you tried creating a new source control connection?

no dice.
Just created a new connection, the powershell scripts sync, but not the python files.
Is there any sort of chance that there is something about the python files that have to change to make them a valid runbook? Its saying it cant validate the "type" "Python2"...
"Failed to import runbook 'FirstPython.py'.
Exception: Cannot validate argument on parameter 'Type'. The argument "Python2" does not belong to the set "Graph,PowerShell,PowerShellWorkflow" specified by the ValidateSet attribute. Supply an argument that is in the set and then try the command again."

@Csmithson-qh What region is your Automation Account in?

eastus2

Update Azure modules

That was it.
I had no idea i had to update those for this fucntionality, i had assuemd this was baked into the platform as a service portion of this.
Thank you VERY much for your solution!


middleware: add Render for JSON, XML, binary and text

Codecov Report

Merging #34 (24647c4) into main (923d848) will decrease coverage by 3.13%.
The diff coverage is 4.76%.

@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
- Coverage   88.24%   85.10%   -3.14%
==========================================
  Files          16       17       +1
  Lines        1021     1061      +40
==========================================
+ Hits          901      903       +2
- Misses         84      122      +38
  Partials       36       36


Rendering issue when using a touch screen

Using Firefox 50, the search button is not rendered correctly when using a touch screen.

On further research, it looks as though the issue happens with touch screens, when the .ol-touch event is triggered.

So, adding the following styles does fix the size of the component when a touch screen is used and detected:
.ol-touch .ol3-geocoder-search {
    height: 39px;
    width: 39px;
}
.ol-touch .ol3-geocoder-search-expanded {
    width: 222px;
}
.ol-touch .ol3-geocoder-input-search {
    top: 5px;
    left: 39px;
}

Now, there is a separate issue in both Chrome and Firefox with the default value of the "top" property:
calc(.5em+65px)
Both Firefox and Chrome say "Invalid property value".

Another style is needed to fix the results popup in touch screens:
.ol-touch UL.ol3-geocoder-result {
    left: 39px;
}

Hey @jmgomezpoveda sorry this long delay but with #109 things will adjust soon.

@jmgomezpoveda can you confirm that this is solved with v2.4.0?

All my positioning and styling is broken with 2.4.0. I will need to rework the customisation.
In a quick test, the styles still seem to be broken when using a touch screen (though of course my fix no longer works).

Sorry for that. v2.4.0 has a big change in CSS selectors.

Rendering with 2.4.0 with the traditional control has some glitches, which so far I have been unable to fix in my project.
As per the jsfiddle:

I can't confirm this in my machine, latest version (v3.0.0).
Is this still relevant? @jmgomezpoveda

Apologies, I am not working on the website these days, so I won't be able to test it anytime soon.

Will reopen if this persist.

This still persists. You can open the fiddle on a touch device to replicate it.

Is there anyone willing to provide a fix?

Thanks @Nadeflore


Add ability to NOT show compliance results when running chef-client

Describe the Enhancement:

I'd like the ability for the new compliance phase to have a way to NOT display the compliance phase output
Describe the Need:

https://github.com/chef/chef/issues/11916 forced always outputting the compliance phase output and I'd like the ability to NOT output the compliance phase output. I'd like this to replicate the behavior of the old 'audit' cookbook which when chef-client ran compliance profiles it said it was running the tests but did NOT show the actual results from the tests.
I understand the desire to be able to see the results when manually running 'chef-client' and that behavior should continue, but I'd like the ability to turn this compliance reporting off (with the results still sent up to Automate)
Current Alternative

None other than teaching people to ignore the compliance phase output
Can We Help You Implement This?:

The current behavior seems at odds with this line in the docs "Once turned on, the Compliance Phase always outputs its results in the CLI on manual runs. The output for automated runs is handled by reporters." which seems to indicate that the cli would only be forced on for manual runs but looking at my /var/log/messages it's very clearly still outputting to cli for runs executed by systemd


Cleanup code and some missing functionality

Add :

new acceptable value 'Skip_empty' to feval tag of tasks
move tasks templates to application folder under tasks/templates
always start instrument drivers in prepare so that it is always there in perform (avoid a conditional statement).

Fix :

typos and missing docstrings

Closes #41 #40

[Current coverage][cc-pull] is 99.89%

Merging [#42][cc-pull] into [master][cc-base-branch] will increase coverage by -0.09%

File ecpy/tasks/plugin.py was modified. more

Misses +2
Partials 0
Hits -2

@@           master        #42   diff @@
========================================
  Files          96         96
  Lines        6637       6627    -10
  Methods         0          0
  Messages        0          0
  Branches        0          0
========================================
- Hits         6637       6620    -17
- Misses          0          7     +7
  Partials        0          0

Powered by Codecov. Last updated by [fb94251...dc01278][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Ecpy/ecpy/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Ecpy/ecpy/compare/fb94251a26d363819d07df163758a652b92a8580...dc0127844be5e80a5c17cc561afaeab026b63933
[cc-pull]: https://codecov.io/gh/Ecpy/ecpy/pull/42?src=pr

Closes #43


Login form not saves password in Safari Keychain

When I use login form at first, Safari should ask user to save password, but it doesn't. Propably it doesn't see something like form submit.

I believe this is because of the way Polymer is submitting the form. I'm not sure there is a very good fix rather that a decent amount of restructuring to the frontend.
There is a workaround, however. On OSX, using Safari, you can enter your password and, before submitting, click the key that appears on the right hand side of the password field. This drops down a menu, click Save This Password.
I think that is probably the best that can be done at this point.

Thank you for workaround.
Btw, Keychain works with many other SPAs, where logins are not handled by regular form and action request, so there should be easy way.
Also it is good to test it in other password managers, there could be similar issue with same fix.
Jan Pobořil
http://honza.poboril.cz/

2016 v 5:41, Ryan Kraus<EMAIL_ADDRESS>

I believe this is because of the way Polymer is submitting the form. I'm not sure there is a very good fix rather that a decent amount of restructuring to the frontend.
There is a workaround, however. On OSX, using Safari, you can enter your password and, before submitting, click the key that appears on the right hand side of the password field. This drops down a menu, click Save This Password.
I think that is probably the best that can be done at this point.
—
Reply to this email directly or view it on GitHub.

This is solved with both Safari and Chrome + LastPass.


question: multiple exit classes?

I'm already using Sandra's rfc logon exit.
abapGit's ZCL_ABAPGIT_EXIT is not set up to handle multiple implementation classes.
The way I understand it, I have to modify that class, correct?

yes, you have to combine the methods into zcl_abapgit_user_exit. from there you could of course call other classes.
putting zcl_abapgit_user_exit into a repo with some exit coding isn't the best idea since pulling would overwrite whatever other exit methods you already use.
maybe one day we enhance zcl_abapgit_exit to call all implementations of the zif_abapgit_exit interface and not just zcl_abapgit_user_exit

Ok, thanks.


Pass tags auths to pop up

Check my comments. Tag is not available in the pop-up when you want to create a Tag, we should use 'tagsAuthorizations' instead of 'tag'. Let me know.

Thanks, did the modifications !


breadcrumb_trail options has no effect

Passing any match options to breadcrumb_trail inside views has no effect.
<% breadcrumb_trail match: :exact do |crumb| %>
  <%= link_to_unless crumb.current?, crumb.name, crumb.url, class: 'text-gray-700 underline' %>
  <% unless crumb.current? %><span>::</span><% end %>
<% end %>

https://github.com/piotrmurach/loaf/blob/6a8d586d3a49b4e524be425dee86d5def8eb79f1/spec/unit/view_extensions/breadcrumb_trail_spec.rb#L237
Changing this line to
view.breadcrumb_trail(&block)
or to
view.breadcrumb_trail(match: : inclusive, &block) # any other valid match value
does not make test fail

Hi! Thanks for this report. This sounds like a bug. Would you have time to submit a fix with a test?

I cant catch this bug yet, I arrived to https://github.com/piotrmurach/loaf/blob/6a8d586d3a49b4e524be425dee86d5def8eb79f1/lib/loaf/view_extensions.rb#L90
This regex works not as expected. It doesn not show that "/" is active within "/posts"


Critical Vulnerability Fix

This is a fix for the critical vulnerability identified by Tim McClean in this article

@KSanthanam JSONWebToken in Swift is not vulnerable to this problem since you are explicitly stating which algorithms with the keys you want to use.
From the example, in the following example, we are explicitly supporting HS256 with the key secret and `secret2.
try JWT.decode("eyJh...5w", algorithms: [.HS256("secret"), .HS256("secret2")])

It's not possible for a user to downgrade to the none algorithm since it's not supplied. It's not possible to switch to HMAC because it isn't supplied (and also the library doesn't support it). It's not even possible to use any other algorithm than HS256 where the key is secret and secret2.
The developer is in full control of the allowed algorithms.


schema: Add source and links fields

(So far we've mostly been abusing comments fields for this)

Yeah you're right. I've not done it there because in the most cases there's just one link or two, but yeah. I'll add it.