Vueditor does not work with latest VueJS (2.4.1)

The example shows:
import "vueditor/dist/css/vueditor.min.css";
but that file does not exist, there's vueditor.min.css in dist/style directory. if i include that, it gives error:
cannot read property 'title' of undefined
If I downgrade to<EMAIL_ADDRESS>then I get a Vue warning:
Do not use built-in or reserved HTML elements as component id: iframe
Do not use built-in or reserved HTML elements as component id: picture

hi there, it is a bug and i fixed it now, please update via npm i vueditor and check the new doc.

Now i'm getting this error:
[Vue warn]: Error in render function: "TypeError: Cannot read property 'title' of undefined"

found in

---> <VeToolbar>
       <Vueditor>

similar bugs here
vue.esm.js?65d7:434[Vue warn]: Error in callback for watcher "content": "TypeError: Cannot set property 'status' of undefined"

found in

---> <VeUndo>
       <Vueditor>
         <FormEditor> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Form/FormEditor.vue
           <FormGroup> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Form/FormGroup.vue
             <FormFrame> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Form/FormFrame.vue
               <CreativeNew> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/CreativeNew.vue
                 <Creative> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Creative.vue
                   <App> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/App.vue
                     <Root>warn @ vue.esm.js?65d7:434handleError @ vue.esm.js?65d7:519run @ vue.esm.js?65d7:2864flushSchedulerQueue @ vue.esm.js?65d7:2615(anonymous function) @ vue.esm.js?65d7:655nextTickHandler @ vue.esm.js?65d7:602
vue.esm.js?65d7:523TypeError: Cannot set property 'status' of undefined(…)handleError @ vue.esm.js?65d7:523run @ vue.esm.js?65d7:2864flushSchedulerQueue @ vue.esm.js?65d7:2615(anonymous function) @ vue.esm.js?65d7:655nextTickHandler @ vue.esm.js?65d7:602
vue.esm.js?65d7:434[Vue warn]: Error in callback for watcher "content": "TypeError: Cannot set property 'status' of undefined"

found in

---> <VeUndo>
       <Vueditor>
         <FormEditor> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Form/FormEditor.vue
           <FormGroup> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Form/FormGroup.vue
             <FormFrame> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Form/FormFrame.vue
               <CreativeNew> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/CreativeNew.vue
                 <Creative> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/components/Creative.vue
                   <App> at /Users/Plutonist/Documents/Projects/unimoe-frontend/src/App.vue
                     <Root>warn @ vue.esm.js?65d7:434handleError @ vue.esm.js?65d7:519run @ vue.esm.js?65d7:2864flushSchedulerQueue @ vue.esm.js?65d7:2615(anonymous function) @ vue.esm.js?65d7:655nextTickHandler @ vue.esm.js?65d7:602
vue.esm.js?65d7:523TypeError: Cannot set property 'status' of undefined
    at UPDATE_BUTTON_STATES (eval at <anonymous> (http://localhost:8080/app.js:1922:1), <anonymous>:5:63624)
    at wrappedMutationHandler (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:597:5)
    at commitIterator (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:321:7)
    at Array.forEach (native)
    at eval (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:320:11)
    at Store._withCommit (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:408:3)
    at Store.commit (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:319:8)
    at boundCommit (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:275:19)
    at t.updateButtonStates (eval at <anonymous> (http://localhost:8080/app.js:1922:1), <anonymous>:5:62908)
    at Array.wrappedActionHandler (eval at <anonymous> (http://localhost:8080/app.js:1910:1), <anonymous>:604:15)

hi, i can't reproduce the issue with version 0.2.8, is there any possibility that you are still using the old version? can you paste your example code here?

To reproduce:
create a new project with vue init test,
then add vueditor and vuex: yarn add vuex --dev; yarn add vueditor --dev
main.js:
let editCfg = {
  toolbar: [
      'undo', 'fontName', 'fontSize', 'foreColor', 'backColor', 'bold', 'italic', 'underline', 'strikeThrough', 'justifyLeft', 'justifyCenter', 'justifyRight', 'justifyFull'
    ],
    fontName: [
      {val: "Arial"}, {val: "Courier New"}, {val: "Times New Roman"}, {val: "Verdana"}
    ],
  lang:'en',
    // the font-size select's options
    fontSize: ['8pt','10pt','11pt','12pt','13pt','14pt','16pt','18pt','20pt','24pt','28pt'],
 }
Vue.use(Vueditor,editCfg);

app.vue:
<template>
  <div id="app">
    <Vueditor></Vueditor>
  </div>
</template>

<script>

export default {
  name: 'app',
}
</script>

@andreixk The lang attribute in config while initializing is no longer a string now. It should be an object. Please check the new doc.

Just remove lang:'en' and it will work, I'll add some type check for the config in later version.

but you don't write what is an object for the lang attribute.
Which object should we fill for the lang attribute


PT#146810903 Adds and wires pf-table deps

https://www.pivotaltracker.com/story/show/147228327
Also fixes a bug where in selected results are not updated when pagination affects list views
No visual changes, just picking the correct deps for this functionality

Been things about this...and just thinking out loud here.  Do we want to inject something that we know is not complete and that it may be awhile before we get it in?

inject something

well we're not really injecting anything here... no more or less offensive than leaving  unused deps hanging out there (first commit fixes this).. except at least we'll be using this at some point in the future


Person leaving early may vote remotely

A person who attends the beginning of a meeting and is issued a code card may leave the meeting and keep the card, still knowing the URL and agenda index.
Solution proposal:
Provide a second factor of auth - a common token which is replaced for each vote and only displayed in the meeting room. By revealing that token only when the vote takes place, each voter must be present or have a rogue accomplice present to transmit the code.

I think that the notion that you must be in the meeting to vote is old fashioned, and that this issue can be disregarded. Imho every member student should be able to vote regardless of their physical position.

I think that the notion that you must be in the meeting to vote is old fashioned, and that this issue can be disregarded. Imho every member student should be able to vote regardless of their physical position.

I think that the notion that you must be in the meeting to vote is old fashioned, and that this issue can be disregarded. Imho every member student should be able to vote regardless of their physical position.

The risk is an uninformed vote I guess since they didn't attend the discussion most likely.

That's another issue, make a motion to edit the bylaws. At this point, the goal of this system is to mimic the characteristics of the voting process as closely as possible.

@krabban91 updates?

I guess that it might be a good idea to add a second verification step such as a code presented by the vote collectors during every vote session.
But I have to add that we have become a lot worse on this point with the years, but if a student is a part of the voting list one MUST tell the meeting president that you are leaving and therein return your vote-slip.


packages not found

When running the Dockerfile....
Step 7 : RUN apt-get -y install iputils-ping firefox openjdk-7-jre icedtea-7-plugin libpango1.0-0 libxft2 libxmu6 libxv1 fonts-takao
 ---> Running in b17ad4a63dab
Reading package lists...
Building dependency tree...
Reading state information...
Package openjdk-7-jre is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'openjdk-7-jre' has no installation candidate
E: Unable to locate package icedtea-7-plugin
E: Unable to locate package fonts-takao

Hi David,
I could not reproduce the issue by re-triggering the build on docker hub. Here is the successful build: https://hub.docker.com/r/ouyi/docker-ubuntu32-firefox/builds/bsemmbsgaxqaonryznjpxsk/
However, I saw the issue before:
https://hub.docker.com/r/ouyi/docker-ubuntu32-firefox/builds/brlkx5h6f5zcojnuvagrrmn/
which seemed to be a temporary issue, because the log states:
  Temporary failure resolving 'archive.ubuntu.com'```

Are you seeing the same message in your build log?

You were right, that was the problem.  I cleaned things up and it starts firefox now.


Revert "Making it multithreaded"

Reverts populationgenomics/tob-wgs#169

Did you mean to revert this one?

Did you mean to revert this one?

Yep - this one has CPG ID's as well no?

Closing in favour of: https://github.com/populationgenomics/tob-wgs/pull/173


[NEW BONUS GUIDE] JoinMarket-webui (Jam)

What

Created JoinMarket-webui (Jam) bonus guide

Why

Jam enables a simple UX for interacting with JoinMarket-clientserver

How

Reviewing and conforming to existing bonus guides

Scope

[ ] significant change to core configuration
[x] independent bonus guide
[ ] simple bug fix

Fixes #1098
Test & maintenance
How can the changes be tested? Is there ongoing maintenance effort? If this is a bonus guide: are you willing to update it from time to time?

Test by following guide and reporting issues.  I will make an effort to keep the guide up-to-date.

Animated GIF (optional)

i've just installed but do not able to connect. see this issue
https://github.com/joinmarket-webui/jam/issues/571
someone say where i can dig deeper to find the solution?

Link needs to be added to https://github.com/raspibolt/raspibolt/blob/master/guide/bonus/bitcoin/index.md.

Trying to run it but without success, due to this issue:
https://github.com/joinmarket-webui/jam/issues/608
Any advice on how to solve it?


Change CLI to not use built-in auth by default

As implemented the task which makes the RSTUF API
Authentication/Authorization disabled by default
(https://github.com/repository-service-tuf/repository-service-tuf-api/issues/282) the default behavior of RSTUF CLI to not use auth by default.
If the user wants to use the CLI to the API with built-in authentication, it requires the parameter --auth.
rstuf --auth admin

Everything looks good to me (the very good test coverage really helps with these changes!). I believe we just need to rebuild the docs, see docs/source/guide/index.rst, line 27 still refers to no-auth option.

Nice catch @KAUTH, just fixed it.


Add role for postgres profile

Describe the feature
The postgres profile has an optional search_path configuration to override the search_path. I would like an optional role configuration to override the user. It would use the set role command before any operations.
This would be useful because objects would be created with the correct ownership and permissions, instead of needing hooks at the end to grant ownership and permissions to created objects.
Describe alternatives you've considered
Grant ownership and permissions via hooks.
Additional context
This feature is only relevant to Postgres.
Who will this benefit?
This will benefit anyone using role-based access control in Postgres.
For example, I have three roles in my data warehouse: dwadmin, dwloader, and dwclient. Each role has multiple users, and a user can have multiple roles granted to them. When operating on the data warehouse, first the user sets their role, and then performs the operation with appropriate permissions.
Currently, dbt runs as a user and has to grant ownership and permissions to the roles at the end of the run. I would like for dbt to run as a given role instead, so ownership and permissions are correct when the objects are created.

This is a cool idea @mpcogito. We don't typically use postgres as an analytical database, so I'm not deeply familiar with role-based access control on pg. Do you think this is as easy as running set role ... every time a connection is opened? I imagine this would be more tractable to support as a profile-level config, not as a model-level config. Does that sound appropriate to you?

@drewbanin That sounds right to me. I think the best time is just after the connection is opened in https://github.com/fishtown-analytics/dbt/blob/e51c942e91a94936f68f2965963d3b46f1257658/plugins/postgres/dbt/adapters/postgres/connections.py#L100
It should be handled as an optional config.
I think this would work for Redshift as well since its permission model is similar to Postgres, but I never did get Redshift permissions working to my satisfaction!
Over the holidays I'll work on a PR for this.

Sounds good - thanks @mpcogito :)


Bump Arrow from 0.17.1 to 1.0.0

Blocked on homebrew update of apache-arrow.

I moved the contents of the original v0.0.11 milestone to v0.0.12.
This PR would complete milestone v0.0.11, in which we make a release based on Arrow v1.0.0.


Making small change to ReutersBridge

Rewrite some code logic, now it's checking type from the API and append it to the description/article content and it make the code it more readable, maintainable than the old one.

Close #9 .


Update Skate Park to include BMX, Roller Blading, and Scooter sports tags

Currently 'Skate Park' only includes 'skateboard' as a sport tag. Adding other sports that use skate parks like inline skating (roller blading), BMX, and Scooter converts it in iD to a 'Sport Pitch' which is less accurate.
While skate parks are most commonly associated to skateboards due to prominent figures like Tony Hawk, they do include these other sports. To add weight to this claim for those who haven't visited a skate park, Wikipedia also lists these sports under 'skate park'.

Hi @sanecito, I agree this behavior isn't great, but the OSM tagging makes this tricky! Skateparks are tagged with the same base tag as soccer fields and tennis courts, leisure=pitch, meaning iD can only differentiate between different pitches via the sport tag. iD's presets cannot parse "one of multiple" tags or multiple values for the same tag, so we can't address the issue as-is.
I'd recommend solving this by switching the OSM tagging to something that better defines what the feature is, like leisure=skatepark, instead of simply what you can do there. This would be in line with a tag like leisure=ice_rink that accepts sport values but doesn't rely on them.

Thanks for the quick, excellent feedback! I'll see what I can do this week to fix this upstream in OSM going off the proposal process documented in the OSM wiki to write and request review of such a leisure=skatepark tag

@sanecito just FYI
It seems there are some of such elements already available in the osm-db, see:
https://taginfo.openstreetmap.org/tags/leisure=skate_park
https://overpass-turbo.eu/s/OEI

Hey @sun-geo thanks for highlighting exact usage counts (looks like about 150 across both skate_park and skatepark). Yes, in the course of researching leisure and skating tags in the OSM Wiki to draw up my proposal draft I found this blurb:

These combinations are not used in high numbers as of Sept 2018, but could be considered for skateparks:
`leisure=skatepark`
`leisure=skate_park`

Having a standardized leisure=skatepark should help fix this duality as well as other issues I'll highlight in the rationale section in the next couple of hours when the page is created.

Tagging proposal is now in RFC, will probably keep in RFC a week before moving to Voting phase
https://lists.openstreetmap.org/pipermail/tagging/2019-December/049619.html

@sanecito Thanks for taking up this proposal and moving it forward!
One note: I'm personally not sure adding a distinct leisure=bike_park is needed since they seem relatively uncommon and their form is nearly identical to skateparks. We could add a "Bike Park" preset in iD using leisure=skatepark + sport=bmx 🤷♂

One note: I'm personally not sure adding a distinct leisure=bike_park is needed since they seem relatively uncommon and their form is nearly identical to skateparks. We could add a "Bike Park" preset in iD using leisure=skatepark + sport=bmx 🤷♂

I would assume leisure=bike_park as a mountain biking park  - https://mtbparks.com/Bike-Parks/
They often double as ski mountains in off season, but many have dedicated stuff build up just for bikes.

One note: I'm personally not sure adding a distinct leisure=bike_park is needed since they seem relatively uncommon and their form is nearly identical to skateparks. We could add a "Bike Park" preset in iD using leisure=skatepark + sport=bmx

Yes, I'm not sure how many instances need to exist in order for it to be justifiable to get an approved tag. While the US has 3,500+ skateparks as of 2014, it looks like going off of this list there are roughly 310 bike parks + the ones referenced in @bhousel's list.
The ones I had in mind were the ones in Houston and Chandler that are skatepark-like in nature, but the ones bhousel would also qualify as bike park. I couldn't find a Wikipedia-like definition for Bike Park like skatepark, but I found this which encompasses both skatepark types and mountain bike trail types although it has more emphasis on MTB elements:

A bike park is a piece of land specifically designed for off-road or mountain biking. Bike parks can be built in small urban areas areas, public lands or integrated with trail systems. A bike park may include skills courses, pump tracks, jumps, trails and features. With many design and budget options available, a bike park is an achievable project for all communities.

I can re-add the BMX only example of leisure=skatepark + sport=bmx to the proposal, but I'd prefer tagging consensus on the mailing list / proposal wiki comments. I feel like recommending leisure=bike_park (and not creating an approved leisure=bike_park given the low count) would have the least amount of contention, but I'm not a tagging list regular, so don't have a good read. I'll have to see how those MTBParks are tagged to see if it sparks any ideas. I imagine they're a combination of trail and biking sport tags though.

Yes, I'm not sure how many instances need to exist in order for it to be justifiable to get an approved tag.

"approved" tag is tag that went through https://wiki.openstreetmap.org/wiki/Proposal_process with a successful vote.

Yes, I'm not sure how many instances need to exist in order for it to be justifiable to get an approved tag.

"approved" tag is tag that went through https://wiki.openstreetmap.org/wiki/Proposal_process with a successful vote. Usage count is a separate metric.

Correct, but as a first time tag submitter, I wasn't sure if there's some soft, undocumented number in the wiki the people who typically vote prefer. For example earlier in the thread by quincylvania:

I'm personally not sure adding a distinct leisure=bike_park is needed since they seem relatively uncommon and their form is nearly identical to skateparks

Basically, I want to figure out the best resolution to this issue to possibly avoid people voting against proposal because they for example don't agree on the side rationale of creating a leisure=bike_park and prefer some other method. I should hopefully have enough examples in the next 12 hours to propose how to deal with bike parks' special case to ask people to vote one way or another in the mailing list thread and adjust the wiki proposal accordingly. Then the proposal phase itself can move to voting.

Hi Folks,
Bike parks are becoming increasingly common in Australia where they are reviving many small towns
https://www.maydenabikepark.com/ - https://www.openstreetmap.org/#map=18/-42.75551/146.62833
https://www.ridebluederby.com.au/ - https://www.openstreetmap.org/#map=17/-41.14896/147.80255
The bike trail network covers quite a large area, often covering both private and reserved land. They are much larger than “skate parks” which are generally small and in urban playground settings
Cheers - Phil
From: Scott Anecito<EMAIL_ADDRESS>Sent: Wednesday, 11 December 2019 9:05 AM
To: openstreetmap/iD
Cc: Subscribed
Subject: Re: [openstreetmap/iD] Update Skate Park to include BMX, Roller Blading, and Scooter sports tags (#7080)
Yes, I'm not sure how many instances need to exist in order for it to be justifiable to get an approved tag.
"approved" tag is tag that went through https://wiki.openstreetmap.org/wiki/Proposal_process with a successful vote. Usage count is a separate metric.
Correct, but as a first time tag submitter, I wasn't sure if there's some soft, undocumented number in the wiki the people who typically vote prefer. For example earlier in the thread by quincylvania:
I'm personally not sure adding a distinct leisure=bike_park is needed since they seem relatively uncommon and their form is nearly identical to skateparks
Basically, I want to figure out the best resolution to this issue to possibly avoid people voting against proposal because they for example don't agree on the side rationale of creating a leisure=bike_park and prefer some other method. I should hopefully have enough examples in the next 12 hours to propose how to deal with bike parks' special case to ask people to vote one way or another in the mailing list thread and adjust the wiki proposal accordingly. Then the proposal phase itself can move to voting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/openstreetmap/iD/issues/7080?email_source=notifications&email_token=ACC6NY7STYKKKKGLUL75743QYAHATA5CNFSM4JTZP7LKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGRDXIY#issuecomment-564280227 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACC6NY6V5JOCFL767STXZETQYAHATANCNFSM4JTZP7LA .  https://github.com/notifications/beacon/ACC6NY2YJY4F7YNJ5MX374LQYAHATA5CNFSM4JTZP7LKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGRDXIY.gif

Bike parks are becoming increasingly common in Australia where they are reviving many small towns. The bike trail network covers quite a large area, often covering both private and reserved land. They are much larger than “skate parks” which are generally small and in urban playground settings

I wouldn't consider the complete trail systems as part of the bike park. There's already good tags for denoting biking trails anyway.

Hmm I had only considered urban bike parks that look essentially like skateparks. A tag like leisure=bike_park makes more sense to me when it covers mountain bike parks as well, since those are quite different from skateparks. Distinct tags for uncommon features is okay if they can't fit cleanly under existing tags.

Distinct tags for uncommon features is okay if they can't fit cleanly under existing tags.

Yes, this is why I still think it would be most convenient if people would be ok with just having their new tag fit under the existing leisure=pitch+sport=* scheme.

Tagging which could thread the needle is leisure=pitch + pitch=skatepark + sport=*. This would define the specific physical form while being backwards-compatible.

@sanecito Any progress on the proposal?

@TheAdventurer64
Thanks for the followup; took bereavement shortly after the prior conversation and then COVID hit and this fell between the cracks. Will target modifying the proposal within the next two weeks to Quincy's suggestion of creating a pitch=* and pitch=skatepark as well as updating the leisure skatepark example to reflect the new potential pitch=skatepark


Question on alwaysIssueNewRefreshToken, how is my saveToken() suppose to know it was from a refresh_token grant type?

When alwaysIssueNewRefreshToken is set to false, the revokeToken() method is never ran, and as such the saveToken() method is called directly without removing the older token (makes sense).
So in my example, saveToken() is called, and in it I have a database query that executes an INSERT to my database. My query fails since in the schema I made, it is expecting both refreshToken and access token, but when alwaysIssueNewRefreshToken is false, I don't get a new refresh_token and only a new access_token, so I have to now run an UPDATE on my db query....Now yes, I can update my saveToken() method to be flexible for this, but the problem is that there is really no way (expect seeing that I have no refresh token) for me to know if saveToken() is being called to save a brand new token or to simply update a new token. I guess I can do
if(!token.refreshToken) run UPDATE query instead
But shouldn't there be some sort of flag or maybe a updateToken() method that runs instead? I guess I could run an upsert query instead, but now saveToken() isn't really "saving a token" now, it's doing an upsert or conditional insert if it doesn't have a refresh_token.
TL;DR -- Maybe incorporate an optional 'updateToken()' method that runs when alwaysIssueNewRefreshToken is false?

In addition, since I only retrieve the access_token from 'saveToken()', I won't be able to fully return the entire token object since I am missing the refresh_token, https://oauth2-server.readthedocs.io/en/latest/model/spec.html?highlight=getRefreshToken#savetoken-token-client-user-callback
The return values expect to contain the refresh_token when I can't provide it because it's not being returned.
What I think needs to happen is instead of only returning access_token in saveToken() during a refresh_token grant, to also include the existing refresh_token that wasn't deleted. This way we can return the refresh_token (which is just the same) and also so we don't lose the reference of the tokens we need to update.
https://github.com/oauthjs/node-oauth2-server/blob/master/lib/grant-types/refresh-token-grant-type.js#L161 here instead of not including the refresh_token if alwaysIssueNewRefreshToken is false, we should include the refresh_token that was not revoked/removed. I can create an MR if needed,  just let me know if this is something you guys would like to see.
I understand that I can add certain checks and change a few things on my side, but then I am not leveragling this API at all and instead finding patches/fixes for things that should be supported.
Let me know if you guys would like me to make an MR for this if you deem it necessary/needed.

In addition, since I only retrieve the access_token from 'saveToken()', I won't be able to fully return the entire token object since I am missing the refresh_token, https://oauth2-server.readthedocs.io/en/latest/model/spec.html?highlight=getRefreshToken#savetoken-token-client-user-callback
The return values expect to contain the refresh_token when I can't provide it because it's not being returned.
What I think needs to happen is instead of only returning access_token in saveToken() during a refresh_token grant, to also include the existing refresh_token that wasn't deleted. This way we can return the refresh_token (which is just the same) and also so we don't lose the reference of the tokens we need to update.
https://github.com/oauthjs/node-oauth2-server/blob/master/lib/grant-types/refresh-token-grant-type.js#L161 here instead of not including the refresh_token if alwaysIssueNewRefreshToken is false, we should include the refresh_token that was not revoked/removed. I can create an MR if needed,  just let me know if this is something you guys would like to see.
I understand that I can add certain checks and change a few things on my side, but then I am not leveragling this API at all and instead finding patches/fixes for things that should be supported.
Let me know if you guys would like me to make an MR for this if you deem it necessary/needed.

but the problem is that there is really no way (expect seeing that I have no refresh token) for me to know if saveToken() is being called to save a brand new token or to simply update a new token. I guess I can do

It is exactly how you know if it's an update or a creation, the lack of a refreshToken... The library is doing it's data validation in the background, so you can safely assume that it's an update if no refreshToken is present.
Having a seperate updateToken function would just split the logic into two functions, besides that, it would work exactly the same way. You can achieve this yourself by creating the logic.
if (refreshToken) this.updateToken(token)
else this.createToken(token)


Copying empty text to clipboard throws javascript error.

I was trying to replace ZeroClipboard with this new clipboard.js library to get rid of the flash dependancy, but I felt this one workflow was unsupported with this library. An example of this is showcased here: https://jsfiddle.net/o6g9gu2w/. When trying to copy empty text, due to user input being empty, I get the following error: Missing required attributes, use either "target" or "text" caused by the following line https://github.com/zenorocha/clipboard.js/blob/d9254459b791785ab3a50c3d13524ba07ea307bb/src/clipboard-action.js#L45. I would like to know if there is a way for me to catch or prevent this error or how this could be fixed. This is the only thing preventing me from switching to this library.

Hey @dweepcan,
Some sort of string needs to be returned, so we can include a space if input is empty.
https://jsfiddle.net/zenorocha/o6g9gu2w/4/
Let me know if you have any other problems.

@zenorocha But this means that whatever was previously in the clipboard is destroyed which might be unexpected/annoying for the user.

Fixed at https://github.com/zenorocha/clipboard.js/commit/7a5a910bcd83aa44718dc30caea503d9da37d4b9. Please use v1.5.10 in order to benefit from this.


[Error]

when I run test.py --submit,  i meet this error,  how to modify code， Wait for you reply!!!
[>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>] 2177/2177, 6.7 task/s, elapsed: 323s, ETA:     0s
Loading testing groundtruth...
[>>>>>>>>>>>>>>>>>>>>>>                            ] 1000/2177, 185.4 task/s, elapsed: 5s, ETA:     6sTraceback (most recent call last):
File "tools/test.py", line 245, in
main()
File "tools/test.py", line 236, in main
metric = dataset.evaluate(outputs, **eval_kwargs)
File "/dahuafs/userdata/46623/PSG/OpenPSG-main/openpsg/datasets/psg.py", line 388, in evaluate
rel_pair_idxes=ann['rels'][:, :2],
IndexError: too many indices for array: array is 1-dimensional, but 2 were indexed

Please check this
https://github.com/Jingkang50/OpenPSG/blob/ae418a15f6c0d558b665ffaca3ec0dd52e0f6f24/openpsg/datasets/psg.py#L79


[Bug] Export custom types as css button not working on mobile

Absolutely nothing happens when you click onto that button.
App: Latest mobile insider (50)
System: Android 11, Samsung Galaxy S21 Ultra

@Obsidian1212
Hello, are you still having issues exporting custom CSS on Mobile? How about .Json files as well?

Yup, still not working. The import button crashes the entire Obsidian app as well... Might need to report to the devs.

There's none on the surface. Not sure about the console though.
Also, here's a screencap.
https://user-images.githubusercontent.com/35908303/234155730-2605f50e-136e-4f20-8378-996fcecaae65.mp4


NestedScrollView problem

Hi. Thanks for this amazing library!
I have a view with the following structure
<NestedScrollView>
    <LinearLayout>
        <ImageView/>
        <RecyclerView/>
    </LinearLayout>
</NestedScrollView>

When I scroll NestedScrollView, the list is also scrolled, but the ImageView is not moving.
How to fix it?

Hey,
Did you find the solution?

Hi!
Formerly in a structure where a ScrollView encompasses a listview, the total size of the list view judging all items was calculated and set in the listview so that only the scroll of scrollview would work, but today has the nestedscrollview option.
Maybe this post can help you.
Take a look at: https://stackoverflow.com/questions/31000081/how-to-use-recyclerview-inside-nestedscrollview

Link is dead.
And I'm using nestedscrollview though its working, its scrollup entire content instead header stick.

Take a look at: https://stackoverflow.com/questions/31000081/how-to-use-recyclerview-inside-nestedscrollview

StickyHeaderLayoutManager
Provides equivalent behavior to a simple LinearLayoutManager, but where section header items are positioned in a "sticky" manner like the section headers in iOS's UITableView. StickyHeaderLayoutManager MUST be used in conjunction with SectioningAdapter.

Well, it seems that it's not the behaviour we want. BUT if StickyHeaderLayoutManager MUST be used in conjunction with SectioningAdapter, maybe you can customize the SetioningAdapter like that:
https://github.com/lopspower/HFRecyclerView

Its not helpful.


fix: honor --contract-name and --start-block

This PR tries to address two issues related to subcommand add:

--contract-name flag ignored: When the --contract-name flag is specified in the command line, the provided value is ignored, and the user is prompted for input

Improved handling of --start-block flag: When the --start-block flag is provided, its value is correctly applied, but an unnecessary asynchronous call was still being made. This resulted in execution hanging due to an unhandled promise. The fix eliminates the redundant asynchronous call

Hey @briceyan, thanks for the PR. Would you mind adding a description of the fix ?
I understand it fixes an issue with the add command not respecting flag parameters and I could reproduce the issue, appreciate the fix.

I initially suspected that the process hanging after completing the code generation was caused by some leftover async operations, but it now seems likely that prompt.ask is the reason for the hang. Unfortunately, I don’t have enough time to investigate this further at the moment—apologies for the incomplete diagnosis.
Below is the output when I ran the command with why-is-node-running:

...

why-is-node-running /Users/brice/Devel/graph-tooling/packages/cli/bin/run add --abi /Users/brice/EverythingFun/v1-core/out/ISet.sol/ISet.json --merge-entities --contract-name Set12 --start-block 1 0x0000000000000000000000000000000000000000
probing program /Users/brice/Devel/graph-tooling/packages/cli/bin/run
kill -SIGUSR1 19346 for logging
(node:19346) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)
✖ Failed to fetch Contract Name: Failed to fetch contract source code: Contract source code not verified
✔ Contract Name · Set12
✔ Running codegen
There are 17 handle(s) keeping the process running.

TTYWRAP
/Users/brice/Library/pnpm/global/5/.pnpm/why-is-node-running@3.2.1/node_modules/why-is-node-running/include.js:6 - console.log('kill -SIGUSR1', process.pid, 'for logging')
TTYWRAP
(unknown stack trace)
TTYWRAP
/Users/brice/Devel/graph-tooling/node_modules/.pnpm/password-prompt@1.1.3/node_modules/password-prompt/index.js:3 - const stdin = process.stdin
FILEHANDLE
(unknown stack trace)
FILEHANDLE
(unknown stack trace)
...

@briceyan Not sure I fully understand, are you running into some other issue even after the fix of this PR ?


Fix GitHub links

All of these links now redirect to a repo under a different GitHub user account.
I don't know whether you want to update the changelog or leave it as it was at the time, but I'm happy to amend the PR to remove these edits if you prefer to leave HISTORY.md untouched.

Sorry, I ironically filed this in the wrong repo :)


Can we document on this page that Databricks sql is only available for Databricks premium workspace

Hi team,
Databricks sql is only available on premium workspace:
https://databricks.com/product/azure-pricing
Can we document this like other pages such as cluster access control:
https://docs.microsoft.com/en-us/azure/databricks/security/access-control/cluster-acl
Thanks a lot for all the help!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c78a0a00-d5ed-e0d5-70b5-57ebf7980d3c
Version Independent ID: 2e880e67-4a6a-b8c6-853a-80d0b0bb1792
Content: Get started with Databricks SQL - Azure Databricks - Databricks SQL
Content Source: databricks/sql/get-started/index.md
Service: azure-databricks
Sub-service: databricks-sql-analytics
GitHub Login: @mssaperla
Microsoft Alias: saperla

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue and apologize for the delayed response. The requested updates have not yet been made. The timeline for resolution varies based on resourcing. We've created an internal work item (DOC-7110) to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


[Question/FR] Way to make a dependency between different StochasticParameters between sequences

I have a specific problem:
I want to translate in one sequence and then to scale in another sequence. Like this:
seq1 = iaa.Sequential([
      iaa.Affine(
          translate_px={
              'x': iap.Uniform(-200, 200),
              'y': iap.Uniform(-200, 200)
          }
      ),
])
seq2 = iaa.Sequential([
    iaa.Affine(
        scale=iap.Uniform(0.2, 0.6),
    ),
])
# Apply seq1
# Apply seq2 separately

But I want amount it is translated to be proportional to how much it will be scaled.
I thought using something like iap.Divide(-200, scale_param) will solve it, but it works differently than I thought.
Is there a way to achieve it?

The scale and translation parameters are sampled independently in Affine, so there is no way to make these dependent on each other (except for subclassing Affine and overwriting _draw_samples()).
The only way I can think of is to do it manually per batch, following roughly this recipe:
import imgaug.parameters as iap

for batch_id, batch in enumerate(batches):
    scales = iap.Uniform(0.2, 0.6).draw_samples((len(batch),), random_state=batch_id)
    seq1 = iaa.Affine(translate_px={
        'x': iap.Multiply(iap.Uniform(-200, 200), iap.DeterministicList(scales), elementwise=True),
        'y': iap.Multiply(iap.Uniform(-200, 200), iap.DeterministicList(scales), elementwise=True),
    })
    seq2 = iaa.Affine(scale=iap.DeterministicList(scales))

    # ... augment batch ...

Not tested, but I think it should work. Needs the latest version from master though, as DeterministicList does not yet exist in the pypi 0.3.0 release.

Oh, nice. I am sorry, for not reporting back. I actually went throught trying to make my own Impl. of StochastickParameter class:
class Repeater(iap.StochasticParameter):
  def __init__(self, param):
    super(Repeater, self).__init__()
    if not isinstance(param, iap.StochasticParameter):
      raise Exception("Expected StochasticParameter object or number or string, got %s." % (type(param),))
    self.param = param
    self.is_on_repear = False
    self.buffer = None

  def repeat(self):
    print("Start reading out", len(self.buffer))
    self.is_on_repear = True

  def reset(self):
    self.is_on_repear = False
    self.buffer = None

  def _draw_samples(self, size, random_state):
    if self.is_on_repear:

      if size[0] > len(self.buffer):
        raise Exception("No more recorded information before repeat")
      self.buffer, values = self.buffer[size[0]:], self.buffer[:size[0]]
      print(values, " returned")
      print(self.buffer, "left")
      return values
    else:
      values = self.param.draw_samples(size, random_state)
      print("New values", values)
      if self.buffer is None:
        self.buffer = np.copy(values)
      else:
        self.buffer = np.append(self.buffer, values)

      return values

Only to realize it can't be used for things I wanted. A later I basically end up using exactly what you suggested! I was planning to report back and save your time.  Sorry, it was 11pm and I forgot.
I need one scale per batch, so I used Deterministic from 0.2.9  and it worked well.
Thank you for your time. Feel free to close this ticket ;-)


Disabling the network halfway through match processing makes it seem like matches are processed quickly

They are not, in fact, processed. They just cycle quickly.

Should be OK for now - long-term I need to fix how errors are displayed to begin with, but that's for another release.


[for review] Fix whitespace trimming of outputs

The markdown representation is not very good when testing whitespace. In the future, I would recommend looking into copying one of the tests that uses a ss_structure representation.

Thanks, @dorey - tests updated.

@dorey I've fixed the problem described.  However, the fix also introduces the following change:
Old

  <output value=" /xml_escaping/a "/>< <output value=" /xml_escaping/b "/><item>

New
<output value=" /xml_escaping/a "/> < <output value=" /xml_escaping/b "/><item>

I think the new behaviour's an improvement, but would appreciate your opinion.

This change also gives more consistent formatting to <text><value> elements that contain an <output>, e.g.
Current
<text id="/delivery/some:label">
  <value><output value=" /delivery/other "/> asdf </value></text>

With this patch
<text id="/delivery/some:label">
  <value><output value=" /delivery/other "/> asdf</value>
</text>

the new behaviour's an improvement, but would appreciate your opinion.

Agreed. The old way changed the content inadvertently. This is the right way of doing it. (Haven't looked at the code yet though)

Take a look at my comment here https://github.com/XLSForm/pyxform/issues/96#issuecomment-310744459. The problem is not with the regular expression. The problem is a conceptual one. You can't introduce spaces where none existed by pretty-printing, then collapse them without also eliminating spaces that are essential. You can play around with half-measures, but the real solution is to use a pretty-printer that doesn't add whitespace where none existed. It should only extend whitespace. Similarly, when collapsing, you can't safely eliminate whitespace where some existed. That was the problem with #96, it's the mirror image of this.

If you take a look at the raw output before pretty printing, the forms look good are perfect with regards to this. It's making them human-readable that's causing the problem. Minidom's formatter just isn't up for the task.

@batkinson thanks for the response.  I don't follow what you're saying about #96 though - this PR fixes the example in the original issue #96 and eliminates the extra leading and trailing whitespace that were introduced by #132.  Can you give an example of what would be handled incorrectly?

this PR fixes the example in the original issue #96 and eliminates the extra leading and trailing whitespace that were introduced by #132.

I assume you understand what I meant about the conceptual issue.
For an example, you already gave one yourself. You need a parser to solve this, regular expressions are not capable of comprehending the recursion. That's normally why pretty printers are implemented as parsers and not scanners. Scanners can't identify whitespace leading and trailing an element.
We're going to play whack-a-mole with this until the root cause is addressed.

I do understand that regular expressions aren't ideal for parsing XML, and that to get correct results you'd use a proper parser.

For an example, you already gave one yourself.

This PR handles the example that I gave.  On first reading I thought you were objecting to the PR being merged, but perhaps that's not the case?

perhaps that's not the case?

Correct, I just wanted to note that this likely isn't solved. I'll take a look at what you did differently. No matter what, it's likely not "right".

Yeah, you've captured the idea well in the issue and pointing to that article sums up the problems with using minidom's formatter well. Thanks.

Closing because

Pretty printing doesn't happen by default anymore
It'd be better to fully address #150
@alxndrsn isn't here to defend himself.


Allow subparsers to have Usage strings

In larger command line tools there can be a few levels of subparsers and a top level USAGE description may not be able to capture all varieties of sub-commands. It would help users of these tools if they can get USAGE strings for lower level sub parsers.

@swift-ci smoke test

Can you add a test?

added assert testSubparser to test USAGE string

@swift-ci smoke test


Fix Haddock

The language pragmas are missing at http://hackage.haskell.org/package/store-<IP_ADDRESS>/docs/Data-Store-TH.html

By the way, it would also be nice to remove four spaces because it makes copying code easier, but I didn't fix it because I wasn't sure it wasn't intentional.

Ah, I'm just not very good at haddock markup.  Thanks for the fix! Dedented here: https://github.com/fpco/store/commit/b42a1e3c874fc2ce5a065b9fe6c06e584a5f2eb0


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/security-guard-approvers
/assign knative/security-guard-approvers
Produced by: knative-extensions/knobots/actions/update-deps

/retest

/approve
/lgtm


License

What license is this code available under? (I grepped the files here and the book itself but couldn't find any mention of a license.)

Good question.  It probably should have a license, but doesn't have one yet.  I'll think about this and update the repo with appropriate licensing text at some point.  More than likely, I'll use a Creative Commons, MIT, or similar.

I am also curious to find out what the license for this git repository (as well as the Book Sample Code). I believe that using a license that is approved by the Open Source Initiative (OSI) and listed on the Software Package Data Exchange (aka SPDX) is a good idea. I probably should mention that currently there is no Creative Commons licenses listed under the OSI, but does has an SPDX entry (SPDX CC0-1.0).
Anyways I have made a short list of licenses similar to the MIT License along with a link to both the OSI Description, SPDX Identifier, and a short informative document over the license ad TLDRLegal.

License Name
OSI Identifier
SPDX Identifier
TLDRLegal Info

Free Public License 1.0.0 ( also called Zero Clause BSD)
FPL-1.0.0
0BSD
0BSD or FPL-1.0.0

The Unlicense
(not recommended) [1]
Unlicense
Unlicense

BSD 1-clause [2]
not listed
not listed
not listed.

Boost Software License 1.0
BSL-1.0
BSL-1.0
BSL-1.0

MIT License
MIT
MIT
MIT/Expat

ZLib License
Zlib
Zlib
Zlib

Apache 2.0 License
Apache-2.0
Apache-2.0
Apache 2.0

[1] OSI does not support public domain dedications. This information is on the [Open Source Initiative - Frequently Answered Questions].
[2] This license is similar to the Boost Software License 1.0. The KDE Project source tree contains a copy of this in the following location (https://websvn.kde.org/trunk/playground/sysadmin/secpolicy/ ). This is simply the 2-Clause BSD License with the following license removed...

Redistributions in binary form must reproduce the above copyright notice,
this list of conditions and the following disclaimer in the documentation
and/or other materials provided with the distribution.

Thanks @kphillisjr, that is really helpful.

The publisher has not responded to my emails about this, and I haven't found any other examples of licenses used by Packt authors.  Still looking.

A great way to see examples of the licenses is to check out the repositories on Packt Publishing's Github Account.

Thanks again!  I wasn't aware of this.  Looks like most use MIT.

Added MIT license.


Cannot create broadcast message from the toolbox

Repro:
Try to create a broadcast message from the dropdown on a broadcast block while it is in the palette. The modal comes up, but submitting the name for the message leaves the checkbox in a strange state because of this error.
Uncaught TypeError: Cannot read property 'setCheckboxState' of null

The "strange state" looks like this, but it i think it is just because of the error in the creation of the variable.

This is the same as #1270. Closing this issue.


libstdc++.so.6: version `GLIBCXX_3.4.29' not found

I encountered the following error while trying to run Wavediff (develop branch) on Jean Zay:
Traceback (most recent call last):
  File "/linkhome/rech/genaim01/uds36vp/.local/bin/wavediff", line 5, in <module>
    from wf_psf.run import mainMethod
  File "/gpfsdswork/projects/rech/ynx/uds36vp/repos/wf-psf/src/wf_psf/run.py", line 14, in <module>
    from wf_psf.utils.configs_handler import get_run_config
  File "/gpfsdswork/projects/rech/ynx/uds36vp/repos/wf-psf/src/wf_psf/utils/configs_handler.py", line 15, in <module>
    from wf_psf.plotting.plots_interface import plot_metrics
  File "/gpfsdswork/projects/rech/ynx/uds36vp/repos/wf-psf/src/wf_psf/plotting/plots_interface.py", line 14, in <module>
    import seaborn as sns
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/seaborn/__init__.py", line 2, in <module>
    from .rcmod import *  # noqa: F401,F403
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/seaborn/rcmod.py", line 5, in <module>
    from . import palettes
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/seaborn/palettes.py", line 9, in <module>
    from .utils import desaturate, get_color_cycle
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/seaborn/utils.py", line 11, in <module>
    import pandas as pd
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/__init__.py", line 48, in <module>
    from pandas.core.api import (
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/api.py", line 47, in <module>
    from pandas.core.groupby import (
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/groupby/__init__.py", line 1, in <module>
    from pandas.core.groupby.generic import (
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/groupby/generic.py", line 76, in <module>
    from pandas.core.frame import DataFrame
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/frame.py", line 171, in <module>
    from pandas.core.generic import NDFrame
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/generic.py", line 169, in <module>
    from pandas.core.window import (
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/window/__init__.py", line 1, in <module>
    from pandas.core.window.ewm import (
  File "/gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/core/window/ewm.py", line 15, in <module>
    import pandas._libs.window.aggregations as window_aggregations
ImportError: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by /gpfslocalsup/pub/anaconda-py3/2022.05/envs/tensorflow-gpu-2.11.0+py3.10.8/lib/python3.10/site-packages/pandas/_libs/window/aggregations.cpython-310-x86_64-linux-gnu.so)

The content of the config file is the following:
---
  training_conf: training_config.yaml

On the training_config.yaml I have set a 2 cycles optimisation with 2 epochs per cycle both for the parametric and non-parametric models.
The Tensorflow module used on JeanZay is: tensorflow-gpu/py3/2.9.1. I am running the latest version of WaveDiff (WF-PSF v2.0.0) on the develop branch.

The bug was solved by uninstalling and re-installing the latest version of the pandas package (2.1.4).
The pandas package is not a dependency of wavediff, however it is called by the seaborn package which is currently a wavediff requirement.

@CentofantiEze you wrote tensorflow version 2.9.1 but in your traceback log it states tensorflow-2.11.0

Oh sorry, I edited the comment.


[err-msg] yaml: unmarshal ...

Rancher versions:
1.6.5+
Symptom:
These errors are seen under numerous circumstances. From using an unsupported docker compose version to malformed yaml
Expected behavior:
This message should indicate exactly what the problem may be E.g. "Docker compose version 3 is not supported. Only docker-compose v1 and v2 are supported. Please check the version identifier at the top of your docker-compose." or "yaml arrays not supported."

Example:
time="2018-03-07T03:21:43Z" level=error msg="Could not parse config for project my-project : yaml: unmarshal errors:\n  line 1: cannot unmarshal !!str `3` into config.RawService"
time="2018-03-07T03:21:43Z" level=fatal msg="Failed to read project: yaml: unmarshal errors:\n  line 1: cannot unmarshal !!str `3` into config.RawService"


Correct and update links in UDF doc[skip ci]

Correct and update links in UDF doc.
This is because the Spark doc link got changed.
And also the configs doc should point to doc site link

build


Does the abbreviation IMDS deserve the definite article "the"?

I found the use of the definition article "the" in front of the IMDS abbreviation to be jarring. It's been my experience that the full name of a thing gets the article, but the abbreviation does not.
One talks about the Transmission Control Protocol, but one also talks about using TCP. One uses the Dynamic Host Control Protocol to get an IP address, but one also used DHCP to retrieve other information.
I question the use of "the" in phrases like "use the IMDS to retrieve metadata related to the load balancer and the instances".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 29fdf9e9-95e6-f92d-c6af-bac8f10e7fa9
Version Independent ID: a51fc9ea-bc12-6a18-a70e-bf3bb3e04c7f
Content: Retrieve load balancer information by using the Azure Instance Metadata Service - Azure Load Balancer
Content Source: articles/load-balancer/instance-metadata-service-load-balancer.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

Hello @jasonzio
Thank you for your feedback! We will review and update as appropriate.

#reassign:thisisanniefang

#please-close I've gone ahead and made the wording fixes - PRs have been merged!


Added user agent to the HTTP request header

I've added a typical Google Chrome user agent to the HTTP header as I noticed that adding it decreases the chance of being recognized as a bot. On average, I gained a dozen more requests before being blocked.

Please make sure to target the develop branch for the PR. Also, note that the PR should pass the tests before we consider merging it.


Does not accept application password

It works with 2FA, but when I tried to use an application-specific password to run unattended, it fails with an authentication exception within pyicloud.

It is a limitation of pyicloud/icloud: https://github.com/picklepete/pyicloud/issues/122


[DISCUSSION] Add SkyWalking go SDK as a plugin

The New Feature
Does the MOSN have a plugin or filter mechanism to add a tracing/APM agent?
SkyWalking has its LUA agent for Nginx, Kong and OpenResty, https://github.com/apache/skywalking-nginx-lua
As it has the go agent https://github.com/SkyAPM/go2sky, so if you have the mechanism, we could try to invite our committer, @arugal to work with your team.
WDYT?

@wangfakang @taoyuanyuan @ peacocktrain

FYI @kezhenxu94, if you have interests, welcome to watch and join.

welcome to join in！
We are also planning to support opentracing，we could work together.
MOSN supports plugin and filter
example：https://github.com/mosn/mosn/blob/master/pkg/filter/stream/mixer/mixer.go

OpenTracing already makes things more complex than they should be. It is just APIs of jaeger only.
I will keep the thing for SkyWalking native only.

yeah，Support skywalking also one of our plans, We are very welcome to add skywalking.
I see that Huawei cloud supports mixer APM adatper， does the skywalking support it too？
btw,mosn can easily support various implementations of tracing by framework
PR welcome   ^_^

hi～@wu-sheng
I'm a developer.
I see that skywalking's go client is well encapsulated, but I have a few questions about whether I should choose skywalking as APM to add to our own mosn.
Which are skywalking's dependencies?
Can skywalking be easily deployed on k8s preferably with a operator management?
Can skywalking automatically collect exportor on k8s ?

Which are skywalking's dependencies?

Is this question for go agent or backend?

Can skywalking be easily deployed on k8s preferably with a operator management?
Can skywalking automatically collect exportor on k8s ?

We have k8s helm3 already, https://github.com/apache/skywalking-kubernetes. So these wouldn't be an issue.

Is this question for go agent or backend?
backend
We have k8s helm3 already, https://github.com/apache/skywalking-kubernetes. So these wouldn't be an issue.
ok, skywalking easy to install in k8s , if exporter push data to skywalking，that's it.

backend

That is complex and a long list. Ff you have interests, read our official LICENSE, https://github.com/apache/skywalking/blob/master/dist-material/release-docs/LICENSE#L204-L408

Hi community.
I read the mosn document and source code, now have a general understanding of mosn.
Implementation plan

Register a Driver named SkyWanling

trace.RegisterDriver("SkyWalking", trace.NewDefaultDriverImpl())

Implement SkyWalkingTracer and support the http1 protocol

tracer configuration:
 "tracing": {
    "enable": true,
    "tracer": "SkyWalking",
    "driver": "SkyWalking",
    "config": {
        "backend_service": "skywalking-oap-server:11800",
        "service_name": "app1"
    }

span data:
[
  {
    "Refs": null,
    "StartTime": "2020-03-12T21:34:09.035262+08:00",
    "EndTime": "2020-03-12T21:34:09.037031+08:00",
    "OperationName": "/GET/v1",
    "Peer": "",
    "Layer": 3,
    "ComponentID": 49,
    "Tags": [
      {
        "key": "http.method",
        "value": "GET"
      },
      {
        "key": "url",
        "value": "<IP_ADDRESS>:2046/v1"
      },
      {
        "key": "status_code",
        "value": "200"
      },
      {
        "key": "bytesReceived",
        "value": "0"
      },
      {
        "key": "bytesSent",
        "value": "400"
      },
      {
        "key": "upstreamHost",
        "value": "<IP_ADDRESS>:8080"
      },
      {
        "key": "downstreamLocalAddress",
        "value": "<IP_ADDRESS>:2046"
      },
      {
        "key": "mosnProcessTime",
        "value": "577.014µs"
      }
    ],
    "Logs": null,
    "IsError": false,
    "SpanType": 0,
    "TraceID": [
<PHONE_NUMBER>035272000,
      0,
<PHONE_NUMBER>533376607
    ],
    "SegmentID": [
      1,
<PHONE_NUMBER>035283000,
<PHONE_NUMBER>148249155
    ],
    "SpanID": 0,
    "ParentSpanID": -1,
    "ParentSegmentID": null
  }
]

@arugal Why we need to implement HTTP1? Is gRPC communication not supported?

@arugal Why we need to implement HTTP1? Is gRPC communication not supported?

This refers to intercepting http1 passing through mosn.

This refers to intercepting http1 passing through mosn.

Thanks for clarification. Please continue. I still prefer to write a native SkyWalking filter rather than just tracer implementation.


Documentation: fix typo

@joshix

Actually, hold off on this. I am all sorts of confused now.

Both --api-servers and --api_servers work on v1.1.8. Seems like flags have moved from _ to - in general. @aaronlevy

lgtm


Plugin Web Service Data Source

I need add plugin Web Service Data Source, but when coppy files to restart container, new configuration jasper-server in tomcat. I need help :)

Hi,
I assume you are following the instructions "Install in TIBCO JasperReports Server" on https://community.jaspersoft.com/wiki/installing-webservice-datasource?
How are you implementing these instructions? by modifying the "entrypoint.sh" or some other way?
Basically from what i see you would need a number of things to add this container image:

Either download jaspersoft_webserviceds_v1.5.zip, commit to git and add in Dockerfile or dynamically download in Dockerfile and unzip
open the file WEB-INF\applicationContext-WebServiceDataSource.xml from the unzip folder from previous step. Go to line 49 of the xml file, inside the bean addQueryLanguage change the property beanName from queryLanguagesPro to queryLanguagesCe
In "entrypoint.sh", after line 28  ./js-ant deploy-webapp-ce, unzip generated war file (so you don't have to stop/start tomcat), this should be "/usr/local/tomcat/webapps/ROOT.war"
After previous point in "entrypoint.sh", Copy all the folder's content from [WS_DS_DIR]\JRS\WEB-INF to "/usr/local/tomcat/webapps/ROOT/WEB-INF" folder in the unzipped war

Then when the final line in "entrypoint.sh" (catalina.sh run) starts tomcat it should start the jasperserver with the Web Service Datasource plugin available.
Btw, as this would have general applicability, we would welcome a Pull Request for this feature :-).

I done add WebServiceDataSource plugin, but i don't permission submit pull request.
https://github.com/chiavegatto/docker-jasperserver

Hi @chiavegatto,
you shouldn't need to have any specific permissions to submit a PR. I've just tried it with a brand new GitHub account, i forked the repo, did a dummy commit, and then created a PR.
What error message are you getting?

I was looking at your changes on your fork and they look really good. My main comment is that it would be nice to have each of the changes in separate branches (or atleast commits) instead of all together in one commit on master.
So i would split probably into 3:

Upgrade to version 6.4.0
Enable cross-domain Tomcat
Add Web Service Datasource plugin

This helps with maintaining a clean history by being able to identify why each change was made.

PR is now merged in. Thank you @chiavegatto for your contribution!


Download artifact from prebuilds job

What does this PR do?:
Only retrieve prebuilds artifact.
Motivation:
No need to retrieve all artifacts and merge them.

Do we not need the merge-multiple to get all the prebuilds for all platforms?

With https://github.com/DataDog/action-prebuildify/pull/46, prebuilds job is no more skipped and takes care of merging all artifacts in a single prebuilds artifact.


Consider switching metadata encodings to JSON instead of XML

This seems like a better option for encoding metadata. This would require corresponding API changes.
e.g., instead of
  "metadata": {
          "type": "xml",
          "global": {
            "Title": "'CMIP5 GIPL Model Output 4 km magt alt'",...

Use
  "metadata": {
          "type": "json",
          "global": {
            "Title": "'CMIP5 GIPL Model Output 4 km magt alt'",

I think this change would make it easier to parse this information (and more, XREF #24) in the API and reduce the number of instances where we have to manually prescribe metadata.


Suggestion: Implement optional custom error messages for validation parameter attributes

The Validate* parameter attributes are a wonderful way of enforcing constraints on arguments (parameter values) that go beyond type checking.
Unfortunately, the generic error messages that result when validation fails are often not helpful enough, and inherently they cannot always be.
Therefore, it would be great if a custom error message could be provided as part of the attribute declaration.
Consider this case:
> function foo { param([ValidateScript({ $_ -match '^\p{L}\d+' })] $bar) }; foo -bar baz
foo : Cannot validate argument on parameter 'bar'. The " $_ -match '^\p{L}\d+' " validation script for the argument with value "baz" did not return a result of True.
# ...

The error message is noisy and unhelpful, and, in the absence of additional information, cannot be more helpful.
A workaround is to have an explicit Throw statement in the script block, but that's clumsy and cumbersome.
It would be nice to do something like this instead:
> function foo { param([ValidateScript({ $_ -match '^\p{L}\d+' }, 'You must specify a letter followed by one or more digits.')] $bar) }; foo -bar baz
foo : Cannot validate argument on parameter 'bar'. You must specify a letter followed by one or more digits.
# ...

Additional considerations:

Support placeholders such as {0} in the message that can access the argument passed as well as the (other) arguments passed to the validation attribute constructor.
(This is already used internally for the generic error messages.)

Provide a mechanism to localize the messages.

Both ValidateScript and ValidatePattern implements an ErrorMessage property. See  #2728.
[ValidateScript("$_ % 2", ErrorMessage = "The item '{0}' did not pass validation of script '{1}'")]
[ValidatePattern("\s+", ErrorMessage="The text '{0}' did not pass validation of regex '{1}'")]

Looks to me like that PR should address this.

Thanks, guys, I wasn't aware of that.
It's a great step in the right direction, but I think it should be implemented for all validation attributes: see #3745


Upload/leaveportal

hi @vickyyadav13051991 , what is a purpose of this PR?

Looks like PR done by mistake after tuning the project for particular organisation.