Multiple token issuance

Hi there! Hope you guys are doing well.
I am writing to you about multiple token issuance. I have been playing with that. I wrote a little module which calls issueByPartition multiple times. Obviously this is necessary because I am not going to call 100 times issueByPartition, I want to call 1 transaction call to issue for 100 investors.
    function issueByPartitionMultiple(bytes32[] calldata partitions,
        address[] calldata tokenHolders,
        uint256[] calldata values,
        bytes calldata data)
    external
    withControllerPermission //checks controller permission on contract, modifier works on other calls.
    {
        require(partitions.length == tokenHolders.length, "Check array lengths");
        require(partitions.length == values.length, "Check array lengths");
        for (uint i=0; i < partitions.length; i++) {
            IERC1400(address(securityToken))
            .issueByPartition(partitions[i], tokenHolders[i], values[i], data);
        }
    }

How have you guys solved this issue?

Hi @victorwiebe , thanks for your comment.
Indeed, transaction batching is an important topic.
We've added a "BatchTokenIssuer" to the repo, you can find here now.
The maximum number of token issuances that can be done in one single transaction is 46 with this implementation.

That is interesting as well thank you @gauthierpetetin
Can you please clarify why 46 is the maximum number? Is this due to gas restrictions? How do you estimate that

Yes it's due to gas restrictions.
You can play with it by updating the number in the unit tests (l.22):
const MAX_NUMBER_OF_ISSUANCES_IN_A_BATCH = 46;


Unable to authorize using OAuth

I tried to run this script to receive messages using examples/greet-and-react but getting the following error:
Error: Failed to serialize user into session application.js:630     at pass (e:\Convee\Telegram\node_modules\passport\lib\authenticator.js:281:19)     at Authenticator.serializeUser (e:\Convee\Telegram\node_modules\passport\lib\authenticator.js:299:5)     at SessionManager.logIn (e:\Convee\Telegram\node_modules\passport\lib\sessionmanager.js:14:8)     at IncomingMessage.req.login.req.logIn (e:\Convee\Telegram\node_modules\passport\lib\http\request.js:50:33)     at SlackStrategy.strategy.success (e:\Convee\Telegram\node_modules\passport\lib\middleware\authenticate.js:248:13)     at verified (e:\Convee\Telegram\node_modules\passport-oauth2\lib\strategy.js:177:20)     at Object.passport.use.SlackStrategy [as verify] (e:\Convee\Telegram\app.js:44:3)     at SlackStrategy._verify (e:\Convee\Telegram\node_modules\@aoberoi\passport-slack\dist\strategy.js:78:24)     at e:\Convee\Telegram\node_modules\passport-oauth2\lib\strategy.js:184:24     at skipIt (e:\Convee\Telegram\node_modules\passport-oauth2\lib\strategy.js:348:12)
Slack API's are so complex to work with. I just want to receive the messages which is sent from my app/bot on any channel but unable to find any solution. And in this library also, you need to maintain many keys,secrets, token and then authentication etc. why not to make it simple as telegarm, facebook,kik,skype bots?  A big Downvote Slack!!!!

this error would occur if you've enabled using a session, but did not provide the serializeUser and deserializeUser callbacks. the example as written uses the options { session: false }, so it seems like you must have changed that. if that's not the case, please describe any other changes you've made so we can help you debug this.
thanks for your feedback about the complexity. the Slack APIs can range from very very simple (like incoming webhooks) to substantially complex. we know that developers have varied needs from the platform and try to make APIs that fit the complexity on all points along that range. if we're not hitting that mark, we're always listening in order to make improvements.
specifically, you say you just want to receive messages that your own app/bot sent. i'm confused by this. if you're own app/bot sent those messages, then you already know about them, right? if you need metadata like the ts of those messages after they are sent, then the response from the Web API chat.postMessage should contain that information. you might not need to use the Events API. if your needs are different, let me know and i can try to guide you towards a solution that fits your needs better.


Object.assign failing to notify changes

Description
I am using Object.assign to copy from one element property to another.  It doesn't seem to invoke any observers on the target object.
According to this write up for Firefox (an am trying this in chrome), Object.assign should invoke setters. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign
Live Demo
https://jsbin.com/hohuzu/edit?html,console,output
Steps to Reproduce
Create an element with two object properties.  use databinding to invoke a function on the element to format a the value of a property of the object for display on screen.
Copy data from elsewhere (eg an ajax response) into this object for display using Object.assign(this.targetObject, this.sourceObject)
Expected Results
value of property copied over to be displayed on screen
Actual Results
Object.assign successfully copies the object, but the formating function for the screen is not called.
Browsers Affected

[ x] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari 9
[ ] Safari 8
[ ] IE 11

Versions

"1.7.0"

webcomponents: v0.7.22

The workaround
this.targetObject = Object.assign({}, this.sourceObject)

Works for me - but of course has different functionality as unique properties of this.targetObject are not preserved.  I've not tried
this.targetObject = Object.assign(this.targetObject,this.sourceObject)

but I would presume that works too.
The documents I've read on this don't say targetObject setters only get set in this form, but perhaps that is the case.  I'll close as it seems not to be a Polymer issue


Git commands

VSCode Version: Latest
OS Version: Win 10 x64

Hi guys will there be a way to customize the Git commands in VSCode ?
For example in my work environment we're working with Gerrit and each time we update the code base we need to git fetch && git rebase in order to update the code base because Gerrit doesn't allow me to merge branches the same goes for seperate branches is there a way to enter custom commands for Git in VSCode or to fetch && rebase to a given branch from origin/master - the closest I've come is the 'Pull (Rebase)' in the Git section on the SIde Panel of VSCode - or you know you the terminal :D .
Ok thanks guys.

You can easily write an extension which does this for you.

👍


Further changes to the ListView UI

Another slight change, tell me what you think about it! :-)
I saw in issue #45 that you already made changes to the ListView UI but I think it could be improved even further.
I did the following:
-> Made it compacter (changed title font style and size). It's in a way inspired by Google Drive ListView.
-> Made subtitle a little bit darker and thus more readable
-> Added a round background to the file/folder icon with different colors for file and folder (Maybe that helps a bit with #46?)
-> Darkened the secondary color a tone down
Screenshots for comparison: (Left is new, right is old)

Looks cool! I like the more compactness without making it too thing.
Main concern: the circled file/folder icons look too similar to the action
button. Although I do also want to make folders vs files more obvious...
Here's one of my ideas I've been toying around with:
I've been looking at Drive as an example too and what they do is separate
them with a list view header (eg. "Folders"). I may just do that instead of
color changes since colors would also be delicate with themes.
Another:
Add an arrow to the right of each list view item for folders to indicate it
has children. Not so much a fan of this one though...

Thanks for the response!
I decided to go for the circle icons because I saw them in various apps and thought it looked pretty good. I understand your concerns though that's why I thought about using headers at first too.
For now, I'll revert the icons and keep the rest. Is that ok with you, @jffrymrtn?

@Minty123 Sounds good!
Which apps did you see uses them btw? I'll take a look too -- reviewing a bunch of options right now. Might still go back to circles! Though I'm worried users might be confused between the FAB and the circles. Not ruling it out though

I got the main idea from Amaze File Manager. I also took a look at the Lollipop contacts app and Jair Player (see third screenshot)

@Minty123 Ahhh, interesting thanks, I'll take a look! First glance looks like they use them as buttons (for ActionMode) which would make sense

@jffrymrtn Icons are back to normal now.

Also, quick note, mostly informed by my reading "information dashboard design" (ie. not totally pulled from my ass). (Note: not a designer, but worked in design-focused dev shop where lots of convos rub off)
The thought was that by adding color to the icons, it makes the UI yell "FOLDERS AND FILES" louder than it yells the name of the file itself. When you're skimming a long list of files, the name is where you want attention drawn, not the icon for the filetype, which is kinda secondary. So yeah, my $0.02 is that big circled color icons are distracting considering how the diff between files/folders just needs a small visual hint that can be observed easily :)


Keyboard Accessibility: Shift+Tab cannot escape searchable multiselects

Normally when you Shift+Tab on a select field, the focus moves to the previous field along the tabindex. Right now, Shift+Tab can't seem to escape a searchable multiselect field, even after a value is selected.  The issue goes away if you mark the multiselect component as NOT searchable.
Repro: Check the first fiddle, open the second multiselect field and attempt to Shift+Tab.
Searchable (not working)
https://jsfiddle.net/7ma9xuvp/1/
Unsearchable (working)
https://jsfiddle.net/7ma9xuvp/2/

@jettoki is this different from #411?


bug fix

removed '/' from filename. python treats '/' as a filepath in windows.

But this works in ubuntu. Find a way to incorporate both of them.


Render extra controls with some entries

Nice tool, thanks for providing it.
I'd like to have the ability to attach other controls to some of the items in the TreeView - one or more buttons that allow you to operate on the entries. I guess I can probably do this with some of the customisation options you've suggested but it's not that clear to me and some examples would really help. It would also be great to add some CodePen / JSFiddle examples which I always find super helpful to get up-and-running with a new module like this.
Just a few suggestions :)

That sounds like a good idea! Yet I don't have time to do that right now.
For the time being, please refer to the story https://github.com/iannbing/react-simple-tree-menu/blob/master/stories/index.stories.js and the storybook demo https://iannbing.github.io/react-simple-tree-menu/ to see the use cases.
In case for people who are not familiar with storybook, you can clone this repo and run yarn && yarn storybook to see it in action.


Detecting when the backend is unreachable

When calling the authenticate method and the backend is down the authenticate method fails since the preflight OPTIONS request is not successful.
The details of this error does not seem to pass back through to authenticate e.g. in the controller this code will Result in the reason being empty.
'this.get('session').authenticate(authenticator, credentials).catch((reason) => {'
Is it possible to return a bit more information in this case?  It seems like this is a common enough event (especially in the case of apps which have some offline capability)

@angelosarto you may want to open this in the main simplabs/ember-simple-auth repo...

This is probably out of the scope of this package.


Mediaelements.js breaks Google SEO results

I'm the developer of a website that includes some Mediaelement.js players. Yesterday, the owner of the website pointed out to me that the description for Google results is totally wrong. I strongly believe it's a Mediaelement's fault. If one searches the website on Google, the description says:
Initializing... stage: 320x240 file: HIDDEN_WEBSITE_URL/news/ardito.mp3 autoplay: false preload: none isvideo: true smoothing: false timerrate: 250 ...
The same problem seems to appear for ALL the websites that use Mediaelement.js! mediaelementjs.com too!

Still any real fix for this issue ?

@alessandrofrancesconi Closing old issues. If you'd like to reopen this, let us know. Thanks!


doc: Use --build-dir in custom installs

If atom was built in a custom directory, a custom install will require the use of --build-dir.

Can you make the build directories in both examples match?

Done!

Thanks very much for contributing!


Issue #20: create install target for OpenIGTLinkIO

create the INSTALL target, install target files, include files, Targets-file and Config file.
modify installed files to be machine independent.

Closed #20 in favor of this issue

So, a bit of a philosophical discussion: how can the installed files be machine independent? Do you mean, on machines that are using the same development environment, but with just different paths?
The reason I ask this, is because I've never seen the IS_INSTALL mechanic used. Is there no best practice CMake way of doing this?

Not best practice to my knowledge.
I can change it to use the "VTK-way", assuming VTK to use best practice.

regarding machine dependency. I should really say build environment setup dependency.
The dependency on the compiler and target machine used cannot be removed. But the dependency on the specific folder setup can be removed.

I addressed the issues put forth here.

I have some changes, could I send them to you as a patch?

Sure, I’ll happily have a look and give my opinion.
Do you need my email?
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 20:54
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
I have some changes, could I send them to you as a patch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277345868, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0kOLIN5yK-D_yFNAYXzu0yXlTvt6ks5rY4XpgaJpZM4Ly4K3.

0001-Fixing-target-filename-capitalization.zip

I think I created it properly? I've never created a git patch before, just SVN.

Hold off. The generated CMake Config.cmake for the install doesn't seem to be correct. Investigating.

This part is throwing me off:
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_LIBRARY_DIR}" DIRECTORY)
What is OpenIGTLinkIO_LIBRARY_DIR supposed to be? The directory above the package dir? For people who use find_library(DIR SUFFIX /lib) ?

Well, typically the install will be into CMAKE_INSTALL_PREFIX and that will be “recovered into OpenIGTLinkIO_INSTALL_PREFIX” if you look at the typical targets file, the same pattern is typically found there as well, or in the typical vtk-install file.
Obviously expecting every installed file to be relative to CMAKE_INSTALL_PREFIX
So the CMAKE Config file will be ${CMAKE_INSTALL_PREFIX}/lib/igtlio/cmake/igtlio/OpenIGTLinkIOConfig.cmake
so OpenIGTLinkIO_CMAKE_PACKAGE_DIR should be ${CMAKE_INSTALL_PREFIX}/lib/igtlio/cmake/igtlio
so OpenIGTLinkIO_INSTALL_PREFIX should be ${CMAKE_INSTALL_PREFIX}/lib/igtlio/cmake
so OpenIGTLinkIO_CMAKE_PACKAGE_DIR should be ${CMAKE_INSTALL_PREFIX}/lib/igtlio
so OpenIGTLinkIO_LIBRARY_DIR should be ${CMAKE_INSTALL_PREFIX}/lib
so OpenIGTLinkIO_INSTALL_PREFIX should be ${CMAKE_INSTALL_PREFIX}
That is the thought process
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:12
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
This part is throwing me off:
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_LIBRARY_DIR}" DIRECTORY)
What is OpenIGTLinkIO_LIBRARY_DIR supposed to be? The directory above the package dir? For people who use find_library(DIR SUFFIX /lib) ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277350047, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0g0aro1EmJzWleSdy1q7964DiLR1ks5rY4oEgaJpZM4Ly4K3.

Ok, I think it's sorted. Let me know if I did it wrong.
0001-Fixing-target-filename-capitalization.zip

So the patch actually builds and installs for me, so could you send me the exact error message?
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:12
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
This part is throwing me off:
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_LIBRARY_DIR}" DIRECTORY)
What is OpenIGTLinkIO_LIBRARY_DIR supposed to be? The directory above the package dir? For people who use find_library(DIR SUFFIX /lib) ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277350047, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0g0aro1EmJzWleSdy1q7964DiLR1ks5rY4oEgaJpZM4Ly4K3.

Oh, but it breaks if it's not put into CMAKE_INSTALL_PREFIX

Shouldn't everything be relative to where the install directory is found?
I rewrote it to generate the following:
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
set(OpenIGTLinkIO_INCLUDE_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}/include/igtlio")
set(OpenIGTLinkIO_BINARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}/bin")
set(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}/@OpenIGTLinkIO_LIBRARY_INSTALL@")

set(OpenIGTLinkIO_INCLUDE_DIRS "${OpenIGTLinkIO_INCLUDE_DIR}" "${OpenIGTLinkIO_INCLUDE_DIR}/DeviceWidgets")
set(OpenIGTLinkIO_LIBRARY_DIRS "${OpenIGTLinkIO_BINARY_DIR} ${OpenIGTLinkIO_LIBRARY_DIR}")

include("${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/lib/cmake/igtlio/OpenIGTLinkIOTargets.cmake")

(I didn't realize it was supposed to be /lib/igtlio/cmake/igtlio, qt has it directly as lib/cmake/...)

If what is not put into CMAKE_INSTALL_PREFIX … all installed files should “always” be placed into a subfolder of CMAKE_INSTALL_PREFIX (which can be “/”).
As I understand it, this is the intended way for it to work.
(the install macro actally places all files into folders relative to CMAKE_INSTALL_PREFIX)
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:24
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Oh, but it breaks if it's not put into CMAKE_INSTALL_PREFIX
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277352852, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0jDzQb5kWG9_2NtuFJdK-QHMZR0Fks5rY4zXgaJpZM4Ly4K3.

Oh, but that's wrong too... it should be
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)

set(OpenIGTLinkIO_INCLUDE_DIR "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/include/igtlio")
set(OpenIGTLinkIO_BINARY_DIR "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/bin")
set(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/@OpenIGTLinkIO_LIBRARY_INSTALL@")

set(OpenIGTLinkIO_INCLUDE_DIRS "${OpenIGTLinkIO_INCLUDE_DIR}" "${OpenIGTLinkIO_INCLUDE_DIR}/DeviceWidgets")
set(OpenIGTLinkIO_LIBRARY_DIRS "${OpenIGTLinkIO_BINARY_DIR} ${OpenIGTLinkIO_LIBRARY_DIR}")

include("${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/lib/cmake/igtlio/OpenIGTLinkIOTargets.cmake")

Everything should be placed into CMAKE_INSTALL_PREFIX and relative to CMAKE_INSTALL_PREFIX, which is done, if relative paths are used for the install signature.
And then in the Config and Targets files should reference accordingly (they should be placed into a specific folder in there)
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:26
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Shouldn't everything be relative to where the install directory is found?
I rewrote it to generate the following:
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
set(OpenIGTLinkIO_INCLUDE_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}/include/igtlio")
set(OpenIGTLinkIO_BINARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}/bin")
set(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_INSTALL_PREFIX}/@OpenIGTLinkIO_LIBRARY_INSTALL@")
set(OpenIGTLinkIO_INCLUDE_DIRS "${OpenIGTLinkIO_INCLUDE_DIR}" "${OpenIGTLinkIO_INCLUDE_DIR}/DeviceWidgets")
set(OpenIGTLinkIO_LIBRARY_DIRS "${OpenIGTLinkIO_BINARY_DIR} ${OpenIGTLinkIO_LIBRARY_DIR}")
include("${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/lib/cmake/igtlio/OpenIGTLinkIOTargets.cmake")
(I didn't realize it was supposed to be /lib/igtlio/cmake/igtlio, qt has it directly as lib/cmake/...)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277353396, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0giAHCeJacyw7Gwm_MtXem6Z4QEkks5rY41rgaJpZM4Ly4K3.

Maybe your confusing is, that the config file is not placed into the top folder and so the OpenIGTLinkIOConfig.cmake file is not placed into the top-level directory either.
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:28
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Oh, but that's wrong too... it should be
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
set(OpenIGTLinkIO_INCLUDE_DIR "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/include/igtlio")
set(OpenIGTLinkIO_BINARY_DIR "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/bin")
set(OpenIGTLinkIO_LIBRARY_DIR "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/@OpenIGTLinkIO_LIBRARY_INSTALL@")
set(OpenIGTLinkIO_INCLUDE_DIRS "${OpenIGTLinkIO_INCLUDE_DIR}" "${OpenIGTLinkIO_INCLUDE_DIR}/DeviceWidgets")
set(OpenIGTLinkIO_LIBRARY_DIRS "${OpenIGTLinkIO_BINARY_DIR} ${OpenIGTLinkIO_LIBRARY_DIR}")
include("${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/lib/cmake/igtlio/OpenIGTLinkIOTargets.cmake")
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277353779, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0t1pDjl0WhbQDOj6n2JWDnmEJaefks5rY43PgaJpZM4Ly4K3.

Oh, so relocatable packages can't be placed anywhere?

They can be placed anywhere (as long as the file + paths names do not conflict with the filesystem specifications (esp. FAT has filename-length-limitations)
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:31
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Oh, so relocatable packages can't be placed anywhere?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277354383, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0hML36y3AoyWObfgUSaouHdH9u8Tks5rY45zgaJpZM4Ly4K3.

Lets change the first OpenIGTLinkIO_INSTALL_PREFIX to _TEMP_DIRECTORY
This is equivalent and might clear things up….
so OpenIGTLinkIO_CMAKE_PACKAGE_DIR should be ${CMAKE_INSTALL_PREFIX}/lib/igtlio/cmake/igtlio
so _TEMP_DIRECTORY should be ${CMAKE_INSTALL_PREFIX}/lib/igtlio/cmake
so OpenIGTLinkIO_CMAKE_PACKAGE_DIR should be ${CMAKE_INSTALL_PREFIX}/lib/igtlio
so OpenIGTLinkIO_LIBRARY_DIR should be ${CMAKE_INSTALL_PREFIX}/lib
so OpenIGTLinkIO_INSTALL_PREFIX should be ${CMAKE_INSTALL_PREFIX}
This part is throwing me off:
get_filename_component(OpenIGTLinkIO_CMAKE_PACKAGE_DIR "${CMAKE_CURRENT_LIST_FILE}" DIRECTORY)
get_filename_component(_TEMP_DIRECTORY "${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}" DIRECTORY)
get_filename_component(OpenIGTLinkIO_LIBRARY_DIR "${_TEMP_DIRECTORY }" DIRECTORY)
get_filename_component(OpenIGTLinkIO_INSTALL_PREFIX "${OpenIGTLinkIO_LIBRARY_DIR}" DIRECTORY)

So, undoing my changes, and setting OpenIGTLinkIO_DIR (in PlusBuild configure) to
C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install
produces the following error:
CMake Error at C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/OpenIGTLinkIOConfig.cmake:25 (include):
  include could not find load file:

    C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/OpenIGTLinkIOTargets.cmake
Call Stack (most recent call first):
  SuperBuild/External_OpenIGTLinkIO.cmake:3 (FIND_PACKAGE)
  CMakeLists.txt:388 (INCLUDE)

The main issue is, that the Config File should be placed into ${CMAKE_INSTALL_PREFIX}/lib/cmake/igtlio
But I put an extra igtlio as a path into the paths… complicating things. I’ll go ahead and fix that
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:23
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Ok, I think it's sorted. Let me know if I did it wrong.
0001-Fixing-target-filename-capitalization.ziphttps://github.com/IGSIO/OpenIGTLinkIO/files/751484/0001-Fixing-target-filename-capitalization.zip
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277352616, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0kGVOiXCwMHGDJHWmWceSWxlPl3Eks5rY4yPgaJpZM4Ly4K3.

Well, typically the install will be into CMAKE_INSTALL_PREFIX and that will be “recovered into OpenIGTLinkIO_INSTALL_PREFIX” if you look at the typical targets file, the same pattern is typically found there as well, or in the typical vtk-install file.

Ah, I see. Unfortunately this is probably a reasonable assumption on Linux/Mac, but doesn't hold up on Windows.

On Windows, you can „produce individual Folders for different packages“
So you would use “C:/Packages/OpenIGTLinkIO” as CMAKE_INSTALL_PREFIX, if you want clean folders with easy upgradeability or just “C:/Packages” if you do not mind the clutter. In the second case you can use CMAKE_PREFIX_PATH as “C:/Packages” to be able to autofind the OpenIGTLink_DIR variable.
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:41
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Well, typically the install will be into CMAKE_INSTALL_PREFIX and that will be “recovered into OpenIGTLinkIO_INSTALL_PREFIX” if you look at the typical targets file, the same pattern is typically found there as well, or in the typical vtk-install file.
Ah, I see. Unfortunately this is probably a reasonable assumption on Linux/Mac, but doesn't hold up on Windows.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277356754, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0hK8zDfaDKaITdhrREWEre2DiQ8Kks5rY5D5gaJpZM4Ly4K3.

This again assumes a single folder where all installs go. This is not a safe assumption. Is there no way to make the install completely self-referencing regardless of where it's been installed?
It may mean that the user has to set the OpenIGTLinkIO_DIR to the install location, but that's a reasonable expectation.

There seems to be a missunderstanding.
This install is a completely self-referencing “installation”… this is the point of the install.
I just pushed a fix to my pull request, that tries fixing all your changes not related to the location of the Config file.
I will also install my install folder, so you can see how I do it… you should unpack it and
This is a install without SHARED LIBRARIES (that is why there is no bin folder).
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:54
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
This again assumes a single folder where all installs go. This is not a safe assumption. Is there no way to make the install completely self-referencing regardless of where it's been installed?
It may mean that the user has to set the OpenIGTLinkIO_DIR to the install location, but that's a reasonable expectation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277359647, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0vxeFJF9vbyzkEvFKnzju278g3Diks5rY5PzgaJpZM4Ly4K3.

Just as a reference: my PlusLib Project has the following Entry for
OpenIGTLinkIO_DIR
C:/Packages/OpenIGTLinkIO/lib/cmake/igtlio
I just zipped the folder C:/Packages/OpenIGTLinkIO for you.
Actually it is C:/Programs/OpenIGTLinkIO, but that makes NOT difference whatsoever. My Students have completely different folders including different drives and different folders. It is independent. (Well I started building OpenIGTLinkIO as a STATIC Library, so I would not need to distribute OpenIGTLinkIO any more it is just built into Plus, but anyway, that change is recent.
Typically I INSTALL my Packages into C:/Packages/ and rename the Package to C:/Packages/- so I can archive it and use multiple Versions of the same Package at the same time (e.g. VTK-6.3.0 and VTK-7.1.0).
The renaming does not imply any change in any files, except I need to update using CMakeChache files to reference the folder including the Version.
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 21:54
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
This again assumes a single folder where all installs go. This is not a safe assumption. Is there no way to make the install completely self-referencing regardless of where it's been installed?
It may mean that the user has to set the OpenIGTLinkIO_DIR to the install location, but that's a reasonable expectation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277359647, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0vxeFJF9vbyzkEvFKnzju278g3Diks5rY5PzgaJpZM4Ly4K3.

(I didn't realize it was supposed to be /lib/igtlio/cmake/igtlio, qt has it directly as lib/cmake/...)
Which is correct and I changed that in my pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277353396, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0giAHCeJacyw7Gwm_MtXem6Z4QEkks5rY41rgaJpZM4Ly4K3.

Ok, I built and installed your branch (with a minor fix to OpenIGTLinkIO_LIBRARY_DIR population)
C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install
then tried to configure PlusBuild and I get the following error;
CMake Error at C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/OpenIGTLinkIOConfig.cmake:26 (include):
  include could not find load file:

    C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/OpenIGTLinkIOTargets.cmake
Call Stack (most recent call first):
  SuperBuild/External_OpenIGTLinkIO.cmake:3 (FIND_PACKAGE)
  CMakeLists.txt:388 (INCLUDE)

If I change
set(OpenIGTLinkIO_LIBRARY_TARGETS_FILE "\${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/OpenIGTLinkIOTargets.cmake")
to
set(OpenIGTLinkIO_LIBRARY_TARGETS_FILE "\${OpenIGTLinkIO_CMAKE_PACKAGE_DIR}/${OpenIGTLinkIO_PACKAGE_INSTALL}/OpenIGTLinkIOTargets.cmake")
that portion works.
However, the value of
message(${OpenIGTLinkIO_INCLUDE_DIRS})
message(${OpenIGTLinkIO_LIBRARY_DIRS})

is incorrect, the output is
C:/d/PlusB-vs2015/include/igtlioC:/d/PlusB-vs2015/include/igtlio/DeviceWidgets
C:/d/PlusB-vs2015/bin C:/d/PlusB-vs2015/lib

when it should be
C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/include/igtlioC:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/include/igtlio/DeviceWidgets
C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/bin C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/lib

Ooooooooooohhhhhhhhhh!!!! I'm referencing the wrong file.
Why does /OpenIGTLinkIOConfig.cmake exist?

You created it and install does not „clean”, so this file is from an earalier build… look at the last modified date…
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 22:42
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Ooooooooooohhhhhhhhhh!!!! I'm referencing the wrong file.
Why does /OpenIGTLinkIOConfig.cmake exist?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277370012, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0gJq8GSOaEuNbHSe67iKP3GXsqYrks5rY59BgaJpZM4Ly4K3.

No, it is created by configure of OpenIGTLinkIO. If I delete it and try to build (without configuring), I get
1>  CMake Error at cmake_install.cmake:59 (file):
1>    file INSTALL cannot find
1>    "C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/OpenIGTLinkIOConfig.cmake".

That is a file, which is produced by configure, so you need to hit configure on OpenIGTLinkIO once so install/OpenIGTLinkIOConfig.cmake is created
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 22:46
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
No, it is created by configure of OpenIGTLinkIO. If I delete it and try to build (without configuring), I get
1>  CMake Error at cmake_install.cmake:59 (file):
1>    file INSTALL cannot find
1>    "C:/d/PlusB-vs2015/Deps/OpenIGTLinkIO-bin/install/OpenIGTLinkIOConfig.cmake".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277370685, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0hbZh4nqwumHCG75weiHK4o3PLJbks5rY5_8gaJpZM4Ly4K3.

Ah! It's a temporary file. Can we put it somewhere different? Like, <build>/CMakeFiles?

Aaahhh wonderful wonderful! This is great. Sorry for all the confusion, I've never developed a relocatable build. I will send my last patch with some minor fixes.

Sure, it is in „/install“ as this file is for install, but I will happily put it into /CMakeFiles/install …
Von: Adam Rankin<EMAIL_ADDRESS>Gesendet: Freitag, 3. Februar 2017 22:54
An: IGSIO/OpenIGTLinkIO<EMAIL_ADDRESS>Cc: Kügler, David<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [IGSIO/OpenIGTLinkIO] Issue #20: create install target for OpenIGTLinkIO (#21)
Ah! It's a temporary file. Can we put it somewhere different? Like, /CMakeFiles?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IGSIO/OpenIGTLinkIO/pull/21#issuecomment-277372520, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMQL0sa2hwh2TzOKxmWndgy9qQ2dCdUVks5rY6HigaJpZM4Ly4K3.

Thanks so much for your patience.
0001-Fixing-indentation-in-generated-config-code.zip


Family Reunion Quest not starting

The family reunion quest with Abe doesn't start

Details
The quest for family reunion the Constant doesn't start after speaking to Abe when the Grav Drive is installed. It repeats it as an activity to speak to him but you never get the letters from him and start the quest.
Game Version
Latest Version
New Game
Yes
Reproduction Steps
Install the grav drive and speak to Abe on the constant

This bug also happens if you let them stay at Paradiso.
For reference, the quests in question are:

FFConstantR02Misc "Misc Pointer" [QUST:00211126]
FFConstantR02 "Family Reunion" [QUST:00211125]

The FFConstantR02 should start if players agrees to help Abe in FFConstantR02Misc at stage 1000 but for some reason this never happens, no idea why.

It appears that the game completely refuses to start FFConstantR02, I tried to start it with console commands (both via sending the FFConstantR02QuestStartKeyword [KYWD:0021108B] (like the script for starting this quest does) via CreateEvent and via StartQuest - it does not start.

I made an ESM with all the aliases set to Optional; FamilyLoc, FamilyMarker, Family did not get filled
Family seems to require FamilyMarker which in turn requires FamilyLoc which is supposed to be a location with LocTypeSettlement keyword that contains Location Cell Static Reference of type FFConstantR02_CitizenMarker_LocRef [LCRT:0021108C]
alright, I think I've figured it out! the game for some reason thinks that all locations that would be valid for this quest are "cleared", since this alias cannot be filled with cleared locations it does not get filled
I'd wager the same thing is causing #471
adding "Allow Cleared" to the "FamilyLoc" alias makes the FFConstantR02 quest work but this
this however is not a good solution, the proper solution would to figure out why those locations are marked as cleared

What makes you think ignoring cleared locations is the wrong approach? I think it might be an error that the alias is even checking for that.

Family Reunion has been exclusively sending me to the Red Mile (ironically in the same system as the Constant's original location. I haven't completed the Red Mile quest on this run. If I'm reading the comments correctly, this is the same bug?

Yes, this would be the same bug.

This bug is still present in 1.10.32.

Still bugged as of today

This is still bugged.

isLazy's fix for this has been working just fine. It's a minimally invasive fix that wouldn't cause any issues even if Bethesda changes how settlements can be marked cleared. IMHO including it is a no-brainer. See https://www.nexusmods.com/starfield/mods/8361
It's just conjecture that settlements being marked as cleared is a mistake anyway. We don't know that, and changing it could cause issues (i.e. in future quests/DLC) if it is intended. Go with the simple, proven fix that won't cause issues elsewhere.

SFCP_157.zip

isLazy's fix for this has been working just fine. It's a minimally invasive fix that wouldn't cause any issues even if Bethesda changes how settlements can be marked cleared. IMHO including it is a no-brainer. See https://www.nexusmods.com/starfield/mods/8361

All he's doing is setting the Allow Cleared flag, like @ja231c speculated was the problem.


[Search] az search service create: Add --hosting-mode argument to support S3HD SKU

Related command

search
Description

Add support for the S3HD sku to az cli
https://docs.microsoft.com/en-us/azure/search/search-sku-tier#tier-descriptions
https://docs.microsoft.com/en-us/rest/api/searchmanagement/2021-04-01-preview/services/create-or-update
Testing Guide

az search service create -n {name} -g {rg} --sku standard3 --replica-count 1 --partition-count 1 --hosting-mode highDensity
Create an S3HD service with 1 replica and 1 partition

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

Search


Can not use Dockerfile

docker build -t docker.io/openshift/origin-cluster-image-registry-operator:latest .
Sending build context to Docker daemon 180.2 MB
Step 1/9 : FROM openshift/origin-release:golang-1.10
Trying to pull repository registry.access.redhat.com/openshift/origin-release ...
Trying to pull repository docker.io/openshift/origin-release ...
sha256:76e8479eb8c137cff06a5d35b27e5b5b5d4ce70f7a30de4ef01c5b3159a4d1ce: Pulling from docker.io/openshift/origin-release
256b176beaff: Already exists
2bd622ac2b02: Pull complete
Digest: sha256:76e8479eb8c137cff06a5d35b27e5b5b5d4ce70f7a30de4ef01c5b3159a4d1ce
Status: Downloaded newer image for docker.io/openshift/origin-release:golang-1.10
 ---> b017c842702b
Step 2/9 : COPY . /go/src/github.com/openshift/cluster-image-registry-operator
 ---> d442a944130b
Removing intermediate container 830fd06ddc48
Step 3/9 : RUN cd /go/src/github.com/openshift/cluster-image-registry-operator &&     go build ./cmd/cluster-image-registry-operator
 ---> Running in 5f596199bb25
 ---> beb23f9b901d
Removing intermediate container 5f596199bb25
Step 4/9 : FROM centos:7
Trying to pull repository registry.access.redhat.com/centos ...
Trying to pull repository docker.io/library/centos ...
sha256:6f6d986d425aeabdc3a02cb61c02abb2e78e57357e92417d6d58332856024faf: Pulling from docker.io/library/centos
256b176beaff: Already exists
Digest: sha256:6f6d986d425aeabdc3a02cb61c02abb2e78e57357e92417d6d58332856024faf
Status: Downloaded newer image for docker.io/centos:7
 ---> 5182e96772bf
Step 5/9 : RUN useradd cluster-image-registry-operator
 ---> Running in 0af16ee1240f
 ---> af4ca97226ce
Removing intermediate container 0af16ee1240f
Step 6/9 : USER cluster-image-registry-operator
 ---> Running in 6c2ead775a45
 ---> 860d888ecc10
Removing intermediate container 6c2ead775a45
Step 7/9 : COPY --from=0 /go/src/github.com/openshift/cluster-image-registry-operator /usr/bin
Unknown flag: from
make: *** [Makefile:7: build-image] Error 1

$ docker version
Client:
 Version:         1.13.1
 API version:     1.26
 Package version: docker-1.13.1-60.git9cb56fd.fc28.x86_64
 Go version:      go1.10.3
 Git commit:      bdb8293-unsupported
 Built:           Sun Jul  8 08:29:45 2018
 OS/Arch:         linux/amd64

Server:
 Version:         1.13.1
 API version:     1.26 (minimum version 1.12)
 Package version: docker-1.13.1-60.git9cb56fd.fc28.x86_64
 Go version:      go1.10.3
 Git commit:      bdb8293-unsupported
 Built:           Sun Jul  8 08:29:45 2018
 OS/Arch:         linux/amd64
 Experimental:    true

Can we remove this modern option ?

I'll answer to myself: no, this can not be removed.
This Dockerfile is completely unsuitable for Fedora 28 (the latest release).

I'm going to revert https://github.com/openshift/cluster-image-registry-operator/pull/3 since it leads to regression. In this situation, I see only one way: generate an image with the same name using operator-sdk build.

for posterity:
the solution is to use docker-ce, or switch the makefile to use buildah.
Personally i'd prefer the former because i think using buildah may be slower to build the image, but you can try it and see.


dashboard entrypoints

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What version of the Traefik's Helm Chart are you using?
3.5.0
What version of Traefik are you using?
2.6.6
What did you do?
N/A
What did you see instead?
N/A
What is your environment & configuration?
N/A
Additional Information
This PR https://github.com/traefik/traefik-helm-chart/pull/100 is Exposing dashboard by default but only on traefik entrypoint.

I put a similar comment in that PR as well.

I understand what you guys want to do, and I know that we can either use port-forward or create an IngressRoute to expose the dashboard, but how is `traefik/templates/dashboard-hook-ingressroute.yaml` useful at all?

I spent hours trying to figure out how to use the `dashboard-hook-ingressroute.yaml` to expose the dashboard.
My intuition is that if there is an IngressRoute that is kinda doing the exposing thing, I must be able to mutate it a little in order for me to access it from the Service.

The answer is I can't. `dashboard-hook-ingressroute.yaml` is now become useless.

I would suggest either removing `dashboard-hook-ingressroute.yaml` from the templates because it is causing confusion for people like me or making `entryPoints` changeable so we can actually use that chart.

Hello @yidongw
Thanks for your interest in Traefik !
You're right, you cannot modify object managed by this Helm Chart.
As you have guessed, this IngressRoute is used to provide an easy-to-go access to dashboard, without security issues.
This Helm Chart is designed to help when installing traefik, not managing IngressRoute. The dashboard is not an exception to this design.


ENH: Use Windows build script to generate Python wheels

Replaces inline Command Prompt build setup procedures in favor of ITKPythonPackage PowerShell script.
Centralizing the Windows Python build procedures in an ITKPythonPackage script will improve ease of maintenance and allow the process to be more easily replicated on a local system for testing.
Testing in ITKSplitComponents with accompanying ITKPythonPackage changes has succeeded: https://github.com/tbirdso/ITKSplitComponents/actions/runs/3685964482/jobs/6237566090
Depends on changes in ITKPythonPackage: https://github.com/InsightSoftwareConsortium/ITKPythonPackage/pull/242

Updated itk-python-package-tag to reference merge commit. Will validate one more time in ITKSplitComponents before merge.


Add "icon" visibility true/false settings

I like your plugin, but I did not want to show an icon (checked / unchecked).
And some corrections.

Thank you!


See this paper.

This has also been shown to work well with Extreme Learning Machines, which are very much out of fashion - closing.


Error: Aborted: 966 commits are not yet pushed to

Is everything correct? ([y]es / [n]o / [e]dit) y
Submitting…
Aborted: 966 commits are not yet pushed to pahakalle/cask_repair_update-pyfa
(use `-f` to force submit a pull request anyway)
Aborted: 966 commits are not yet pushed to pahakalle/cask_repair_update-pyfa
(use `-f` to force submit a pull request anyway)

There was an error submitting the pull request. Please open a bug report on the repo for this script (https://github.com/vitorgalvao/tiny-scripts).

For some reason issue fixed itself 😂


MudTextField automatically converts decimals/doubles to integers

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudTextField
What happened?
When I enter a value like 2.2 and trigger the ValueChanged event, it converts the value to 22. I put the InputType on InputType.Number and set T to "@(double?)".
Expected behavior
I want the textfield to give me the proper decimal value
Reproduction link
https://try.mudblazor.com/snippet/mucIFYktxcbpjqKN
Reproduction steps

Enter a decimal
Click outside the field

Relevant log output
No response
Version (bug)
6.13.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The same happens on MudNumericField. You can test it in your own docs (first code snippet): https://mudblazor.com/components/numericfield#api

Nvm, it had to do with the browser's culture.


Remove screen with dependencies

tmux replaces screen. Therefor I kicked screen out. grml-scripts-core was only used for the screen-setup.

Great, thanks for the contribution!


403 on connecting from Grafana

We have been using Grafana and GMP to visualize data.
It was working, and suddenly, we have been getting 403 from grafana when we connect to prom-frontend.
We looked at the logs and couldn't figure out the reason for the same.

What could be wrong?

We are seeing requesting GCM failed
err="Get \"https://monitoring.googleapis.com/v1/projects/xyz/location/global/prometheus/api/v1/query_range?end=1665420750&query=up&start=1665399150&step=15\": context canceled"
This is a simple up request which is failing.

Hi @naveensrinivasan,
Can you confirm that you are able to see up for your project in the GCP Metrics Explorer?

Hi @naveensrinivasan,
Can you confirm that you are able to see up for your project in the GCP Metrics Explorer?

@pintohutch  Thanks, I am able to

we figure it out. It was because we had enabled workload identity on these clusters that was causing this.

@naveensrinivasan but I have enabled workload identity for prometheus frontend ui service account which has proper role to monitoring. then why its not working?
Enabling workload identity on cluster level isn't the same things?

ok solved it in another way.. Added "roles/monitoring.viewer" role to the k8s service account. Thanks for the suggestions.


[Docs Integrations & Extensions] - Missing list item description

Current State
Link: Meshery Docs
Currently the majority of the list items under "Integrations & Extensions" --> sub-section "Extensions" have a description. However, there are items without one.
Screenshot

Desired State
Add missing descriptions for items under "Extensions".
CC: @iArchitSharma

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@sandramsc Can you assign this to me


Readme: simplify install; highlighting added

After every tag I made here I was like "Oh... I forgot to add highlighting.... Again........"
But now I see, that there are some compatibility problems with new Nette. Please, can I ask you to do a rebase after I repair that? Or add eg [ci skip] to commit message? I really don't like this "Failed" message

Sorry, I just realized that it's complete waste of time to update this PR like I wanted......
So just thank you

Thanks :)


[WIP] [CI] Add CUDA 10.2

Add CUDA 10.2
Remove CUDA 10.0: mostly to conserve CI resource. Also: CUDA 10.0 is incompatible with G4 instance type on Windows, and we'd like to use G4 if at all possible (much better bang for the buck than P2)

@rongou @sriramch

@hcho3 our releases go out primarily on 10.0. i see the reason behind removing 10.0 (due to the need to use g4). is it possible to still retain this?
should 9.x be removed instead, if resource savings is another goal?

@hcho3 we can probably remove 10.1 if we still want to keep cuda 9.

@RAMitchell Found some issues with CUDA 10.2?

Maybe we should wait a bit, given that CUDA 10.2 is still pretty new?

Closing for now. Will re-visit later.

I had segfaults on 10.2, not sure if its local to my machine. Will do some more investigation.


Improved ribbon menu width

Your theme is really impressive! it has made me want to use obsidian again.
I think the ribbon menu could be improved a bit by adjusting its width:

I have used the following code
.is-mobile .menu { max-height: calc(100% - 100px); width: calc(100% - 40px); max-width: calc(90% - 20px); min-width: unset; position: absolute; bottom: 0; border-radius: 9px 9px 0 0; padding: 10px 20px; left: 10px !important; top: unset !important; overflow-y: auto; }
I think it needs just one adjustment as you can see in the screenshot (it doesn't quite match the width of the text).
Otherwise it's perfect for me 💯 Thank you !
Bwt: how can I easily change the color or transparency of the text (inline title and content). I have tried with snippets but it doesn't work and within the RGB it makes me completely lost 😂

Hi, glad you like the theme! The messed up prompt width is actually leftover code that I forgot to delete when I was planning to redesign the mobile interface a bit more than I ended up doing. A quick fix would be this snippet, but I'll also fix it in the next update:
.is-phone .prompt {
width: 100vh;
}

And for changing the opacity of the text, you could use this snippet:
body {
    --text-opacity: 0.5; /* <-- put desired opacity here */
}
.theme-light {
    --text-normal: rgba(0, 0, 0, var(--text-opacity)) !important;
}
.theme-dark {
    --text-normal: rgba(255, 255, 255, var(--text-opacity)) !important;
}

/*
@settings
name: Text Opacity
id: text-opacity-settings
settings:
    -
        id: text-opacity
        title: Text Opacity
        type: variable-number-slider
        default: 1
        min: 0
        max: 1
        step: 0.1
*/

This snippet also includes Style Settings support, so you can change the opacity that way, too.

Fixed this in the latest update. Enjoy!


feat(translation): Add Traditional Chinese translation

Question
Response

Version?
v1.4.1

Devices tested?
iPhone 7

Bug fix?
no

New feature?
no

Includes tests?
no

All Tests pass?
yes

Related ticket?
#439

Description

Added Traditional Chinese translation
Replaced a string at with i18n translate function in events.screen.js

I understand Traditional Chinese was not mentioned in #439, but I still spent some time to translate strings to Traditional Chinese.
Please provide suggestions if you are Traditional Chinese speaker.
Many thanks! 😄😄😄

Coverage increased (+0.04%) to 39.937% when pulling a33e78b437b14a052abbe5a363eca4e5d695d32c on patw0929:feature/lang-zh-tw into dd5665248114167cdc9c52657ca3e494d008f2b5 on gitpoint:master.

Coverage increased (+0.04%) to 39.937% when pulling a405562ee2cd29fbe93621b3bd769a8735975867 on patw0929:feature/lang-zh-tw into dd5665248114167cdc9c52657ca3e494d008f2b5 on gitpoint:master.

👍 LGTM. For traditional Chinese, people in Taiwan and Hong Kong are usually more familiar than Mainland China.

Coverage increased (+0.04%) to 39.937% when pulling e87fcbcf2d60f54203cf003579249bb75f6761b7 on patw0929:feature/lang-zh-tw into dd5665248114167cdc9c52657ca3e494d008f2b5 on gitpoint:master.

Beautiful 😍


Spurious mode warning when mode in template with more than one mode

Later on, this issue could not be reproduced with the same test case - am closing.


Simulation information tab to small

Perhaps fix the layout (property name<br/>property) or increase the right panel width.

Fixed with 83ae25ec902179d7b0f28335f45aee69b273af07.


source cleanup

need to make sure that this relation is there while this is not strictly
speaking related to keepalived

Pull-request updated, HEAD is now 06b3da1124de7256a6ba9d894ead7b09451298a9

Pull-request updated, HEAD is now a92bad537d9997d5a9904100c327ee0cf203a6fc


How to group server handle methods into single interface?

Problem statement
I wanted to know if it's possible to group the interface generated into a single interface.
i.e. instead of
handler = myHandler{}
api.GetV1EntityHandler = handler
api.PostV1EntityPathHandler = handler

Have a single interface for the Tag "Entity"
api.EntityHandler =  myHandler{}

Right now I have 50 different routes, all of them have terrible names, if there are other organization tips I'd also be happy to hear.

@aclowkey you may define tags: this splits the handlers into small sub-packages, which have their own interface


fix: Resolve the error of missing map file.

Description: When there is no sourcemap file, an error occurs because the file path with .map added automatically cannot be found.

BREAKING CHANGE: no

Related issue (if exists):

@kdy1 Please help review this.

Can you add a test for this?

@kdy1 done


Update scala-library, scala-reflect to 2.12.17

Updates

org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.12.11 to 2.12.17.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.12.11).
You might want to review and update them manually.
.travis.yml

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #242.


Using uint8 as token type to reduce memory usage

@aisk thanks for the PR! do you have any numbers to show the amount of memory it reduces though?
asking because our CI does a comparison benchmarking between the PR branch & master branch at the end of each build, like this. and this PR's builds don't show an obvious difference as far as I can see:
benchmark                              old allocs     new allocs     delta
BenchmarkBasicMath/add-2               41             41             +0.00%
BenchmarkBasicMath/subtract-2          41             41             +0.00%
BenchmarkBasicMath/multiply-2          41             41             +0.00%
BenchmarkBasicMath/divide-2            41             41             +0.00%
BenchmarkConcurrency/concurrency-2     68552          68500          -0.08%
BenchmarkContextSwitch/fib-2           72309          72309          +0.00%
BenchmarkContextSwitch/quicksort-2     47935          47935          +0.00%

benchmark                              old bytes     new bytes     delta
BenchmarkBasicMath/add-2               1736          1736          +0.00%
BenchmarkBasicMath/subtract-2          1736          1736          +0.00%
BenchmarkBasicMath/multiply-2          1736          1736          +0.00%
BenchmarkBasicMath/divide-2            1736          1736          +0.00%
BenchmarkConcurrency/concurrency-2     3412003       3402804       -0.27%
BenchmarkContextSwitch/fib-2           2997256       2997264       +0.00%
BenchmarkContextSwitch/quicksort-2     2184213       2184209       -0.00%

I'm not sure if our default benchmarking cases are the right ones for this change though. so perhaps you can show some statistics as well? 😄

I did not run any bench mark and think thie optimise is obvious, but to my surprise, I and test it and found the memory reduce is so minimum. I think maybe go have some string intern stuff?

@aisk I think one reason could be the tokenizer is only called once when running a program. so we may need to see it take effect in a large codebase. while I do appreciate your attempt to optimize Goby's performance, I think at this stage such optimization is not necessary, especially when it could increase the complexity of the code 🙂
(also sorry for the late reply, I've been quite busy working on multiple projects recently 🙏)


Add jdbc consumer

Merged as https://github.com/pivotal/java-functions/commit/0f61eb1883b5bf4a413aae4ce93fc22dbcc5283d after agreed clean up and code style refactoring.


Saving bitmap with layers

Hi @davemorrissey,
I have seen #19 and other issues related to the topic, but I still can't figure out something.
I want to save a map (with the PinMapView), which has a marker on it.
You mentioned in these issues, that I need a https://developer.android.com/reference/android/graphics/BitmapRegionDecoder.
It's fine, but where do I get an input stream to instantiate that decoder?
It should be the bitmap, which is private in your library.
Or am I completely wrong?
Can you give me a hint please?
Thanks!

Solved, nvm.

@Benyuss Do you mind sharing your solution for this? Even I am trying to extract Bitmap with layers, but unable to do so. Can you share the code you tried to extract the bitmap?


add orphanRemoval attribute to @OneToMany and @OneToOne to remove associated document

hi
with version 2.2.1, bug fixed, its works fine with fetch = LAZY.
i not see any implemenation of orphanRemoval in this version

The version 2.3.0 of RelMongo which includes this feature was released
see https://github.com/kaiso/relmongo/releases/tag/v2.3.0
please close this issue

hi
i will try it soon and give you a feedback
thanks


updaitng sql-database-project extension VSIX path and version details in

@ssreerama Please provide a link to the build containing the VSIX

Sure @Charles-Gagnon , Waiting for the RC build to finish! thanks :)


vi.mock makes vi.spyOn  to fail

Describe the bug
My code looks like this:
//vi.mock(("anything???!!!!"))
describe('tests', () => {

    it("this works but fails when i uncomment vi.mock, why", () => {
        vi.spyOn(Module, 'moduleFunction')
        expect(moduleFunction).not.toHaveBeenCalled()
    })

    it("this works as well", () => {
        const spy = vi.spyOn(Module, 'moduleFunction')
        expect(spy).not.toHaveBeenCalled()
    })
});

In short, in jest everything it is working, in vitest first case it is failing when i add any call to vi.mock.
It would be nice to have any documentation about this issue or explanation in migration guide.
Reproduction
Edit: https://stackblitz.com/edit/vitest-dev-vitest-t5pdkp?file=test%2Fsuite.test.ts,src%2Fbasic.ts&initialPath=vitest/
//vi.mock(("anything???!!!!"))
describe('tests', () => {

    it("this works but fails when i uncomment vi.mock, why", () => {
        vi.spyOn(Module, 'moduleFunction')
        expect(moduleFunction).not.toHaveBeenCalled()
    })

    it("this works as well", () => {
        const spy = vi.spyOn(Module, 'moduleFunction')
        expect(spy).not.toHaveBeenCalled()
    })
});

System Info
System:
    OS: macOS 14.1.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 293.77 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.2 - /usr/local/bin/node
    npm: 9.8.1 - /usr/local/bin/npm
    pnpm: 8.10.5 - /opt/homebrew/bin/pnpm
  Browsers:
    Chrome: 119.0.6045.199
    Safari: 17.1
  npmPackages:
    @vitejs/plugin-react: ^4.2.0 => 4.2.0
    vite: ^5.0.0 => 5.0.2
    vitest: ^0.34.6 => 0.34.6

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

In short

Are you describing an issue or telling a joke?

In short

Are you describing an issue or telling a joke?

Yes, that's exactly what I meant. Was it funny ?
https://stackblitz.com/edit/vitest-dev-vitest-t5pdkp?file=test%2Fsuite.test.ts,src%2Fbasic.ts&initialPath=__vitest__/

bug confirmed


Not able to set input

I am using M1 MAC Air to connect monitor LG 27UN880 with DP USB-C.
I am able to use most of commands such as to get/chg volume, contrast but failed to set input.
The monitor will flash and back to current view after I execute set input 17.
Switch input is working if I use the physical button on the monitor.
Could you please advise if you have any insight into my problem?

Hi there,
well, m1ddc certainly sends the proper command - what the display does with it is entirely up to the display firmware. Not all displays support input changing properly, but LG displays generally (not always) do. It might be that number 17 does not correspond to the desired input?

@waydabber  Thanks a lot for your quick response. I found I can use ControlMyMonitor on Windows to change the property of Volume and Contrast like what I can do with m1ddc on MAC Air M1. But failed to change the Input value even though the permission for this property in ControlMyMonitor is Read+Write which is like volume. The possible values of Input Select in ControlMyMonitor is showing 0,17,18,15,0. I have tested these values and got the same problem. The display will flash and back to current view. I am going to search if any new firmware for my display and post updates here.

Great, thank you! :)

(btw, I'd try input numbers from 0-32 regardless of the reported supported values, as those are not always reliable).

@waydabber Thanks a lot for your quick response. I found I can use ControlMyMonitor on Windows to change the property of Volume and Contrast like what I can do with m1ddc on MAC Air M1. But failed to change the Input value even though the permission for this property in ControlMyMonitor is Read+Write which is like volume. The possible values of Input Select in ControlMyMonitor is showing 0,17,18,15,0. I have tested these values and got the same problem. The display will flash and back to current view. I am going to search if any new firmware for my display and post updates here.

I also met the same problem, have you solved it?

Same issue here.
I've enumerated 0-32 and it won't switch to the other input. On some values I get the same flash to black @vwumumu was getting then back to current input.
Monitor: LG 27GL850-B connected via DP
macOS: 13.0.1 (22A400)
Going to also look into a firmware update for my monitor.

The possible values of Input Select in ControlMyMonitor is showing 0,17,18,15,0. I have tested these values and got the same problem.

For me, it didn't show the real number options. I've got a Samsung Monitor where these numbers are assigned to the ports:

HDMI1: 5
HDMI2: 6
DisplayPort: 15

I use winddcutil.exe setvcp 1 96 5 to switch input to HTML1. (Download: https://github.com/scottaxcell/winddcutil.) Note: 96 is the decimal representation of the vcp feature number 60 hexadecimal.


Bug: Cannot find name 'vShow'

After upgrade to 0.33.0 I get an error when using the v-show directive:

This didn't occur prior to this update.

Getting the same behavior

Will release new version soon, thanks for the report!


Adi c cpp

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #2595
Added documentation on Fenwick tree
Name - Aditya Raj
Batch - C_CPP batch 5
This PR is with reference to issue #2595

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Closing cause of the Plagiarism as discussed above.


Bombay Network Connection

The docs say:
"The network rules passed by the Terra Station Mobile are 0 is testnet, 1 is mainnet.
Always set testnet for 0 and mainnet for 1."
What do we do if we want to use bombay?

@Ruborcalor The station mobile is still dropping network information only with mainnet=1 and testnet=0. We started working on modifying this. I'll let you know when the work is done.

Gotcha so this is a bug with terra station mobile and not wallet-provider?

@Ruborcalor This problem has been resolved by updating Terra Station Mobile.
https://github.com/terra-money/wallet-provider#important-notice-for-columnus-5
Please refer to this link and use wallet-provider. The bombay network will be recognized normally.

Cool thanks!


Added minimum OS version requirtement

Adds IBM i os 7.3 TR 5 as the minimum required version to run Code for IBM i without issues.
See https://github.com/codefori/vscode-ibmi/issues/2017#issuecomment-2369143236

@sebjulliand

to run Code for IBM i without issues.

That was an optimistic comment! 😉


xarray-cartopy broken?

running the cartopy example in the docs leads to bunch of errors that are hard to decode, I tried in my two different conda environments issue seems to be the same.
Example:
import xarray as xr
import matplotlib.pyplot as plt
import cartopy.crs as ccrs

air = (xr.tutorial
       .load_dataset('air_temperature')
       .air
       .isel(time=0))
ax = plt.axes(projection=ccrs.Orthographic(-80, 35))
ax.set_global()
air.plot.contourf(ax=ax, transform=ccrs.PlateCarree())
ax.coastlines()
plt.savefig('cartopy_example.png')

Error in callback <function post_execute at 0x7f984d50a050> (for post_execute):
TypeError                                 Traceback (most recent call last)
miniconda2/lib/python2.7/site-packages/matplotlib/pyplot.pyc in post_execute()
145             def post_execute():
146                 if matplotlib.is_interactive():
--> 147                     draw_all()
148
..........
miniconda2/lib/python2.7/weakref.pyc in setdefault(self, key, default)
446
447     def setdefault(self, key, default=None):
--> 448         return self.data.setdefault(ref(key, self._remove),default)
449
450     def update(self, dict=None, **kwargs):
TypeError: unhashable type: 'MultiLineString'

That is unrelated to xarray. It is a cartopy/shapely bug due to version mismatch. If you update both that should work.

ooh yaa that fixed it, which version couples to who was not obvious, in the end what worked was installing cartopy from conda-forge repository.
Thanks

which version couples to who was not obvious

Sorry, this is an old issue that I had in the back of my head but I do not remember the versions :grimacing: but a general update should get you going.

in the end what worked was installing cartopy from conda-forge repository.

:+1:


#5876 broken pipe handling

    # how to reproduce bug
    # first set up redis timeout to 5 seconds in /etc/redis/redis.conf as described in http://redis.io/topics/clients
    $redisConfig = [
        'hostname' => 'localhost',
        'port' => 6379,
        'connectionTimeout' => 5,
    ];
    $redis = new \yii\redis\Connection($redisConfig);
    $redis->executeCommand('PING');

    sleep(20);

    try {
        // this is will cause yii\db\Exception in Connection::parseResponse since Redis dropped connection
        $redis->executeCommand('PING');
    }
    catch (\Exception $e) {
        // now Connection::close will try to send QUIT command but got yii\base\ErrorException
        // in Connection::executeCommand on fwrite
        // So we can't send any command and can't close connection
        $redis->close();
    }

pull request needs to be merged into redis extension. Added a note to the issue: https://github.com/yiisoft/yii2-redis/issues/57


Made the adding new keyboard language support easier

https://issues.apache.org/jira/browse/CLOUDSTACK-8665
This branch has implemented following improvements in console proxy keyboard language support

ajaxviewer.js and ajaxkeys.js are main files involved in key code translations. These files now can be copied in systemvm/js folder and they will be copied to CPVM with stop/start performed on it.
Started passing parameters to CPVM needed to resolve the ambiguous cases of keycode translations.
Generalise the framework such that one needs to modify only ajaxkeys.js (file which has keycode mappings) without need of much knowledge in js.

FS for this feature is available at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Support+for+non-US+keyboards+in+Console+Proxy
After these changes how to add keyboard support for new language or fix existing broken keys WIP document  is available at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Adding+support+for+non-US+Keyboard+for+Console+Proxy

@sedukull , Can you give a LGTM now?

@sedukull @anshul1886 please rebase against latest branch and help re-review

@anshul1886 please rebase against latest master and share status of your PR
This looks like an interesting PR we should have

This PR currently have merge conflicts.  Can you rebase please?  Thx...

@swill Its showing lot of conflicts  currently and will take some time to resolve those. Will update PR once  I will get some time.

Thanks @anshul1886.  Keep in mind that the freeze date for 4.9 is Monday next week.

@swill Rebased against the current master and also squashed the commits into one.

@milamberspace Added the localisation key in message.properties and now using it in consoleKeyboardOptions

@anshul1886 can you review the Jenkin errors and close and reopen the PR if they do not seem relevant to kick off the Jenkins run again.  Thx...

@swill They are failing on a test which don't even exist in source code. That test is in PR #1360 which is not yet merged so looks like some cleanup issue.

CI RESULTS
Tests Run: 85
  Skipped: 0
   Failed: 1
   Errors: 0
 Duration: 4h 18m 27s

Summary of the problem(s):
FAIL: Test redundant router internals
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/smoke/test_routers_network_ops.py", line 842, in test_01_isolate_network_FW_PF_default_routes_egress_true
    "Attempt to retrieve google.com index page should be successful!"
AssertionError: Attempt to retrieve google.com index page should be successful!
----------------------------------------------------------------------
Additional details in: /tmp/MarvinLogs/test_network_A6H565/results.txt

Associated Uploads
/tmp/MarvinLogs/DeployDataCenter__May_18_2016_08_23_35_7BOYAF:

dc_entries.obj
failed_plus_exceptions.txt
runinfo.txt

/tmp/MarvinLogs/test_network_A6H565:

failed_plus_exceptions.txt
results.txt
runinfo.txt

/tmp/MarvinLogs/test_vpc_routers_GFBFLL:

failed_plus_exceptions.txt
results.txt
runinfo.txt

Uploads will be available until 2016-07-18 02:00:00 +0200 CEST
Comment created by upr comment.

This shows that it does not break anything, but I don't think we have any verification that this feature works as expected yet...

LGTM.
Tested manually with a real test deployment of CS49, with my French azerty keyboard: works fine.

I think with @milamberspace's review, we are only missing one code review to get this one in.  Thanks everyone...

Really like the functionality so I think we must get this in. I have some remarks about the code that I stand by even though they are not entirely to blame on @anshul1886 as he calls methods fthrowing Exception, rom SshHelper mainly. It is a bad smell due to the older code and will not facilitate trouble shooting for the end user. So I would like to see the exception-handlers be enriched with some better diagnosis and maybe messages. nothing else to nag about.

This one needs some work. We have merge conflicts and we need another code review.

@swill Resolved the merge conflicts. They were because of import statements.

Can I get one more review on this one?  I think we are basically set otherwise.  Thx...

ACS CI BVT Run
Sumarry:
Build Number 134
Hypervisor xenserver
NetworkType Advanced
Passed=68
Failed=5
Skipped=3
Link to logs Folder (search by build_no): https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
Failed tests:

test_vpc_vpn.py

ContextSuite context=TestRVPCSite2SiteVpn>:setup Failing since 10 runs

ContextSuite context=TestVpcRemoteAccessVpn>:setup Failing since 10 runs

ContextSuite context=TestVpcSite2SiteVpn>:setup Failing since 10 runs

test_volumes.py

test_06_download_detached_volume Failed

test_vm_life_cycle.py

test_10_attachAndDetach_iso Failed

Skipped tests:
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_deploy_vgpu_enabled_vm
Passed test suits:
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_routers.py
test_reset_vm_on_reboot.py
test_snapshots.py
test_deploy_vms_with_varied_deploymentplanners.py
test_login.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_disk_offerings.py

@anshul1886 it appears that there was a push to get this PR into 4.9, but @bvbharat found5 test failures and 3 skips.  What is the status of resolving these issues?

@swill @jburwell Updated in according to newer changes in UI. Also moved consoleKeyboardOptions to js file after jsp change.

@jburwell Failed tests are not related to change in this PR.

Added the missing license on one js file.

tag:needslove

@sureshanaparti, Can you have a look at this?

@anshul1886 Please address the open questions and rebase. Post the latest test results.

ACS CI BVT Run
Sumarry:
Build Number 382
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=7
Link to logs Folder (search by build_no): https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
Failed tests:
Skipped tests:
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm
Passed test suits:
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_routers_network_ops.py
test_disk_offerings.py

Tested manually. LGTM.

@karuturi, We can merge this now.

tag:mergeready

tag:mergeready

@anshul1886 you asked to show you how it can be done.
I opened a PR https://github.com/anshul1886/cloudstack-1/pull/1 for the branch you are using in this PR. Can you take a look?
OVM (not implemented)  and XenServer has a different flow of execution (I would like to check that later)
So, what do you think of a code like the one I showed you?
@DaanHoogland could you also take a look at the proposal?

@rafaelweingartner good improvement. @anshul1886 can you merge Rafael's stuff? He addresses the code doublure and the generic exception.

@DaanHoogland @rafaelweingartner Will have a look at the proposed changes and get back to you.

@anshul1886 @karuturi Also help on this one would be appreciated.

@rhtyd is this relevant with primate in view? or can we close?

@anshul1886 please rebase and re-open if this is still relevant


I just encountered a pretty unique error about Paypal Express transactions.

I will provide information as soon as I finish checkout part.
PS: Best package ever! :+1:

There is this code in the docoumenation:
        // (1) - Set gateway
        Shop::setGateway('paypalExpress');

        // (2) - Call checkout / OPTIONAL
        // You can call this to keep a standard flow
        // Although this step for this gateway is not needed.
        Shop::checkout();

        // (3) - Create order
        $order = Shop::placeOrder();

        // (4) - Review order and redirect to payment
        if ($order->isPending) {

          // PayPal URL to redirect to proceed with payment
          $approvalUrl = Shop::gateway()->getApprovalUrl();

          // Redirect to url
          return redirect($approvalUrl);
        }

        // (5) - Callback
        // You don't have to do anything.
        // Laravel Shop will handle the callback and redirect the customer to the configured route.

There is a problem by this code. The 5th point is wrong. When there is an error about the transaction, there won't be any errors displayed to the user.
 I saw that the only wrong thing was the currency in the settings folder

Well the whole point of this was to show that the application doesn't throw exceptions after catching them.

How is your item configuration looking? What type of currency are you
using?
On Dec 2, 2015 11:39 AM, "Ozan Kurt"<EMAIL_ADDRESS>wrote:

There is this code in the docoumenation:
    // (1) - Set gateway        Shop::setGateway('paypalExpress');        // (2) - Call checkout / OPTIONAL        // You can call this to keep a standard flow        // Although this step for this gateway is not needed.        Shop::checkout();        // (3) - Create order        $order = Shop::placeOrder();        // (4) - Review order and redirect to payment        if ($order->isPending) {          // PayPal URL to redirect to proceed with payment          $approvalUrl = Shop::gateway()->getApprovalUrl();          // Redirect to url          return redirect($approvalUrl);        }        // (5) - Callback        // You don't have to do anything.        // Laravel Shop will handle the callback and redirect the customer to the configured route.

There is a problem by this code. The 5th point is wrong. When there is an
error about the transaction, there won't be any errors displayed to the
user.
 I saw that the only wrong thing was the
currency in the settings folder
[image:
https://slack-imgs.com/?url=http%3A%2F%2Fi.imgur.com%2FkpVZlTU.png&width=853&height=234]
https://camo.githubusercontent.com/5ab76009feb26a89304363aaebf189ec26dd2440/68747470733a2f2f736c61636b2d696d67732e636f6d2f3f75726c3d68747470253341253246253246692e696d6775722e636f6d2532466b70565a6c54552e706e672677696474683d383533266865696768743d323334
Well the whole point of this was to show that the application doesn't
throw exceptions after catching them.
—
Reply to this email directly or view it on GitHub
https://github.com/amsgames/laravel-shop/issues/22#issuecomment-161376083
.

I used to use a non existent currency (TL), then i fixed it. The point is that the developer doesn't see any errors even if you try to throw an exception.

Have you seen if these errors are in the log file?
On Dec 2, 2015 2:44 PM, "Ozan Kurt"<EMAIL_ADDRESS>wrote:

I used to use a non existent currency (TL), then i fixed it. The point is
that the developer doesn't see any errors even if you try to throw an
exception.
—
Reply to this email directly or view it on GitHub
https://github.com/amsgames/laravel-shop/issues/22#issuecomment-161428897
.

Sadly, no.
Ozan Kurt

On 2 Dec 2015, at 23:56, Alejandro Mostajo<EMAIL_ADDRESS>wrote:
Have you seen if these errors are in the log file?
On Dec 2, 2015 2:44 PM, "Ozan Kurt"<EMAIL_ADDRESS>wrote:

I used to use a non existent currency (TL), then i fixed it. The point is
that the developer doesn't see any errors even if you try to throw an
exception.
—
Reply to this email directly or view it on GitHub
https://github.com/amsgames/laravel-shop/issues/22#issuecomment-161428897
.

—
Reply to this email directly or view it on GitHub.


can i use doctrine admin bundle with laravel?

No, i don't think so.

Closing because project is no longer maintained. Thanks.


Auto-adds several infowindow CSS classes

This PR removes the need to add certain CSS classes. Instead, it uses the required .js- classes from the template to add them.

cc: @xavijam, can you take a look, please?

For me it makes sense, so :+1:


Downloaded SVG is not scalable

Download as SVG is a PNG embedded in an SVG file. It would be amazingly helpful if the included graphics were segregated into discrete vector images.
Thanks!

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

ID: 1b68d40f-4b98-396b-f4c3-b8650fc4633f
Version Independent ID: 26b362d7-b46a-fa03-c2aa-48b581b67bf0
Content: Advanced Analytics Architecture - Azure Solution Ideas
Content Source: docs/solution-ideas/articles/advanced-analytics-on-big-data.yml
Service: architecture-center
Sub-service: solution-idea
GitHub Login: @doodlemania2
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 363239

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 363239

You can close bug #2605 as duplicate

You can close bug #2605 as duplicate

these Icons are scalable SVG.  I am able to scale using Inkscape.

these Icons are scalable SVG.  I am able to scale using Inkscape.


Add documentation

Add installation instructions, developer guide and usage

Hey I just wanted to reach out and see if there were any more plans to create a docs folder similar to the one on the Elasticsearch python client? https://github.com/elastic/elasticsearch-py/tree/v7.15.0/docs
I know I used those types of docs several times when I was learning to use the Elasticsearch client. Would it be okay if I started adding that type of documentation here?

I think it'd be great if this client exported documentation into https://github.com/opensearch-project/documentation-website as well as make it available in gh-pages through README. Please contribute!

@dblock If wanted I can contribute a GH-pages site generated with Sphinx.
Do you want me to?

@PhilipMay  I would want you to! haha I was going to ask if we wanted to do that actually as I feel like that is kinda the standard

@dblock If wanted I can contribute a GH-pages site generated with Sphinx. Like this: https://telekom.github.io/HPOflow/code-doc.html Do you want me to?

Please! Don't ask for permission to contribute.

Please! Don't ask for permission to contribute.

The main reason I asked was to avoid duplication of work.

Started a PR to generate documentation with Sphinx. See #112

Is there a way to access the gh-pages doc site?

Thanks for bringing this up!
Is is here: https://opensearch-project.github.io/opensearch-py/
A PR to add this to the README.md is still in my list.


Bridge: Bot discarded an Ace

Discarded A of spades instead of 2 of spades on a club lead. A of spades would have been the setting trick.

% PBN ---------------------------
[Event "Trickster Cards: Practice Bridge"]
[Date "2023.06.29"]
[Dealer "E"]
[Vulnerable "None"]
[Deal "E:KQ95.T82.JT97.QJ J84.KJ6.A854.964 T6.A97.63.AK8532 A732.Q543.KQ2.T7"]
[Declarer "W"]
[Contract "3C"]
[Auction "E"]
Pass Pass 1C Pass
1S Pass 2C Pass
3C Pass Pass Pass
[Play "N"]
DK D7 D4 D3
H3 H2 HJ HA
C7 CJ C4 C2
CT CQ C6 C3
HQ H8 H6 H7
H4 HT HK H9
D2 D9 DA D6
DQ DT D5 C5
SA S5 C9 CA
S2 SQ - S6

Digging into this, there's a case calling GetSureWinners() when playing 2nd to the trick where we're missing a check for the led suit. This causes the bot plays the Ace thinking it will win the trick (which it won't).


Want to contribute to your repo.

As mentioned in readme you can no longer contribute to this repo however you can contribute to the new repo Algorithms


Responsive Percentage

Options responsivePercentage: if it is true I use percentage in the width of the boards and it is not necessary gutter and widthBoard

in my pc it works. :grinning:


App Crashes when Backgrounded

Hey Guys,
Running into an issue with this crashing our app on some phones when the app is backgrounded.
calls getWindow(), which returns null, since it's backgrounded, causing a null pointer references, crashing the app.
FRAME	MODULE	CRASH DETAILS
0	com.wix.reactnativekeyboardinput
ViewUtils.java  Line 15
ViewUtils.getWindow
1	com.wix.reactnativekeyboardinput
ReactSoftKeyboardMonitor.java  Line 136
ReactSoftKeyboardMonitor.removeAllLayoutListeners
2	com.wix.reactnativekeyboardinput
ReactSoftKeyboardMonitor.java  Line 98
ReactSoftKeyboardMonitor.onHostDestroy
3	com.facebook.react.bridge
ReactContext.java  Line 236
ReactContext.onHostDestroy
4	com.facebook.react
ReactInstanceManager.java  Line 651
ReactInstanceManager.moveToBeforeCreateLifecycleState
5	com.facebook.react
ReactInstanceManager.java  Line 575
ReactInstanceManager.onHostDestroy
6	com.facebook.react
ReactInstanceManager.java  Line 588
ReactInstanceManager.onHostDestroy
7	com.facebook.react
ReactActivityDelegate.java  Line 142
ReactActivityDelegate.onDestroy
8	com.facebook.react
ReactActivity.java  Line 72
ReactActivity.onDestroy
9	android.app
Activity.java  Line 6471
Activity.performDestroy
10	android.app
Instrumentation.java  Line 1149
Instrumentation.callActivityOnDestroy
11	android.app
ActivityThread.java  Line 4063
ActivityThread.performDestroyActivity

Thanks @jsm !
@d4vidi is it a know issue?

Nope. I bet it's new, introduced due to recent navigation support implementation.

@jsm thanks for this submission! The issue btw is with the activity (or the whole app) going down. I'll work it out asap.

Should be fixed in 3.0.18.

@d4vidi thanks a bunch!


ARROW-XXXX: [R] Test R-in-conda as a nightly build

JIRA is currently unresponsive and thus I was unable to open an issue.
@kszucs I always forget where to put things so that they are run as a nightly. Can you point me to it?

@xhochy dev/tasks/tests.yml

@ursabot crossbow test docker-r-conda

The badge shows failing for in-progress...

docker-r-conda passed successfully :)

RAT is failing and misses a ticket number, will do tomorrow.


'MySQL' is not recognized as an internal or external command,

Steps to Reproduce
running flutter or flutter doctor gives below error
Logs
'MySQL' is not recognized as an internal or external command,
operable program or batch file.
'MySQL' is not recognized as an internal or external command,
operable program or batch file.
'where' is not recognized as an internal or external command,
operable program or batch file.
Error: Unable to find git in your PATH.

flutter doctor -v  also gives same log as above. i'm stuck at here.

That's really weird. Flutter doesn't use MySql in any way.

Yes I also checked code. And found no use of MySQL . That is why I had
opened issue.
On 25-May-2018 8:41 PM, "Günter Zöchbauer"<EMAIL_ADDRESS>wrote:

That's really weird. Flutter doesn't use MySql in any way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/17907#issuecomment-392088878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM7ji30NCjht1TchoTOpzLMkF-8-UuRHks5t2B8hgaJpZM4UOBEG
.

hii @zoechi ,
any idea on this? i'm stuck at starting flutter.

No ideas. I can't imagine this being caused by Flutter.

hii @zoechi ,
removing C:\Program Files (x86)\MySQL\MySQL Fabric 1.5 & MySQL Utilities 1.5\; and C:\Program Files (x86)\MySQL\MySQL Fabric 1.5 & MySQL Utilities 1.5\Doctrine extensions for PHP\; from system variable Path solved issue.
but still doesn't understand why this error was raising.

Sounds like these MySQL directories contain a flutter command.

Having & in the path in a linux based system sounds quite dangerous. Your OS is clearly not linux but there may be assumptions inside flutter tools that get confused by that.
If you're willing to do an experiment, add something in the fashion of C:\Program Files (x86)\MySQL\MySQL Fabric 1.5 & ABC123 and see if you get an error for ABC123 not found instead.
Can you post the whole output from flutter -v into a gist (http://gist.github.com/)  and add the link here? Better if you can reproduce it with the ABC123 example.

hii @fmatosqg ,
adding C:\Program Files (x86)\MySQL\MySQL Fabric 1.5 & ABC123; does gives me below error.
'ABC123' is not recognized as an internal or external command,
operable program or batch file.
'where' is not recognized as an internal or external command,
operable program or batch file.
Error: Unable to find git in your PATH.

you where right original issue was having & inside the path.
also issue only exist for flutter command. other commands like git and npm working perfectly fine.
here is gist as you asked: https://gist.github.com/bhavincb/af059afc27c814808b2e92d3a6aa910e

Please check this : I managed to face the error and fix it with a different lib
https://stackoverflow.com/questions/50776550/flutter-mysql-is-not-recognized-as-an-internal-or-external-command

@Eng-MohamedGadala I appreciate your upvote

I assume this is solved.
Please add a comment if you think there is something that needs to be done by the Flutter team and I'll reopen.

@dawningtruth
Flutter doesn't contain any mention of mysql anywhere it it's codebase.
The issue has to be on your system setup.
Please add the output of flutter doctor -v and everything where mysql appears in your environment setting.
You should get the environment by executing
env

in a terminal window
You can also try
env | grep -i mysql

to get only the output where mysql appears.

Unfortunately, the Zip provided on flutter.io gave me a problem. To avoid this error, I had to use the GitHub repository following the following procedure.
Go to C:\ Drive and create a folder called flutter.
Go to that folder and start git bash
Type git clone -b beta https://github.com/flutter/flutter.git and enter
After its complete, add the path C:\flutter\flutter\bin in environment variables
Finally, start CMD and type flutter doctor and the error is disappeared
source: https://stackoverflow.com/questions/50776550/flutter-mysql-is-not-recognized-as-an-internal-or-external-command

Same Error:  zip file downloaded from flutter site.
'MySQL' is not recognized as an internal or external command,
operable program or batch file.
'MySQL' is not recognized as an internal or external command,
operable program or batch file.
Error: Unable to find git in your PATH.


use the official docker buildx

Oh I saw this:
https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands

We should use: https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-commands-for-github-actions#environment-files
So replace:
run: echo '::set-env name=BUILD_TAG::${{ matrix.dockerfile }}-${{ matrix.openzwave }}'

With
echo "BUILD_TAG=${{ matrix.dockerfile }}-${{ matrix.openzwave }}" >> $GITHUB_ENV


Deploys project on GKE 1.15.7

This change fixes a problem whereby deployment of the marketplace
app would fail on GKE clusters with Kubernetes versions >= 1.15.
The main problem was that we were using the Google Marketplace Tools
Version 0.7.0, and this uses a version of kubectl (1.12) that does
not know how to authenticate with Kubernetes version 1.15 or newer.
The changes included are as follows:

Upgraded the Google Marketplace Tools container image used by the Deployer from 0.7.0 to 0.10.0. The newer version of these tools provide an adaptive Kubectl client version (tools read the Kubernetes server version, then select a matching kubectl binary).
Upgraded the Google Marketplace Tools submodule to 0.10.0.
Upgraded the version of Helm used by the deployer from 2.6.1 to 2.16.1 to eliminate this Helm bug: https://github.com/helm/helm/issues/2998
Deleted x-google-marketplace section for tester.image in schema.yaml to be consistent for Google Marketplace Tools v0.10.0
Added a build.sh flag (-p or --persist) to persist the application deployment after testing.
Fixed the deployment's deploy-info annotations to use keys that are valid JSON keys (i.e. with quotes).
Fixed up a bunch of inaccuracies in README.md and DEVELOPMENT.md.

Fixes Issue #25 "Project fails to deploy on GKE 1.15.7-gke.23"

@sgnn7 - I made the changes that you suggested.
I also added a parallel CI stage so that our CI tests our Marketplace deploy on:

A gke-1.12.10 cluster (Kubernetes 1.12.10)
A gke-1.15.9 cluster (Kubernetes 1.15.9)
(I don't think I have access to the CI Conjur, so the secrets.yml section for gke-1.15.9 uses a string for the GKE cluster name, rather than a Conjur variable.)


Create player movement

Decide on and create player movement
Options include:

relative to player (v. change/impulse)
relative to screen (v. change/impulse)
anything else you can think of

Create the following controls:
Movement

WASD = movement (relative to camera)
space = jump (relative to ground)
mouse down = launch ball
mouse up = activate ball
mouse move = turn

General turn algorithm:

Every frame (f) :
a.  check mouse position (m)