Try external runner for Windows + Cuda

This is based on on the work done in #137 and forms an attempt to use external runners following the discussion here. Permission to use these runners has not been granted, yet, though. This comment suggests that the outlook for it is positive. The related PR in conda-forge/.cirun is being prepared can be found here.
An analogous (albeit much more complicated) PR using these runners to build pytorch can be found here.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.
ℹ The recipe is not parsable by parser conda-recipe-manager. Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12294989804. Examine the logs at this URL for more detail.

Do you have any 🎄 -feedback on this approach @hmaarrfk @traversaro @xhochy ? I'll be largely offline for the next 4 weeks but I might take a peek at this every now and then.

The only thing I would add is the ability to do mega builds.
I had tried, but it significantly decreases build times and helps enact CFEP03 (since we can start the builds with one command locally)

Do you have a good resource about megabuilds? My understanding is that using the external runner would remove the need for cfep03 for this package, would it not?

The best resource is the PyTorch feedstock


Keyboard issues

I've noticed when DCIntrospect is enabled, the keyboard will not animate in correctly. Also sometimes settings becomeFirstResponder doesn't work or does work and the keyboard is not visible.
Not a huge deal, just noticing.

Apparently when DCIntrospect is enabled the keyboard won't show at all for text fields inside UIWebView.
To reproduce:

create this simple project http://www.iphonesdkarticles.com/2008/08/uiwebview-tutorial.html
Add [[DCIntrospect sharedIntrospector] start];
result Keyboard won't show for the google text field.

Seeing this too - even if we're not enabling DCIntrospect.
It would be really nice if there was a way to include DCIntrospect in code but have it not run at all. We wrapped the code with
#if USE_DCINTROSPECT == 1
    [[DCIntrospect sharedIntrospector] start];
#endif

And DCIntrospect was still firing with the constant at 0.

Actually I'll file this as a new bug.


Permissions for managing navigation

This page doesn't indicate what permission levels are necessary to manage global and local permissions. What levels are required? Thank you. - tom braman

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

ID: 2f072d4a-23e6-d73f-397e-4ffc000570a9
Version Independent ID: 1ee08317-82cd-9f23-a1e9-d2380fb7ff78
Content: Understanding permission levels in SharePoint - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/understanding-permission-levels.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Hello @tombraman how are you?
Do you mean the permissions required to manage permissions?
Please see here https://docs.microsoft.com/en-us/sharepoint/customize-sharepoint-site-permissions#customize-site-permissions

To do the following steps, you need a permission level that includes permissions to Create Groups and Manage Permissions. The Full Control level has both.

Does this answers your question? Let me know if I understood you correctly.
Thank you

@tombraman did you see my comment here? thank you

Apologies for not responding sooner -- I was on vacation. My initial question was unclear, too -- double apologies. I meant to write that the page doesn't indicate what permission is necessary to manage global and local navigation. I have a customer who wants to grant that permission, but not via the Full Control permission level, which grants more permissions than the customer wants. Thank you for your persistence! - tom

@tombraman Understood Tom
I think you have here the answer
https://support.microsoft.com/en-us/office/customize-the-navigation-on-your-sharepoint-site-3cd61ae7-a9ed-4e1e-bf6d-4655f0bf25ca?ui=es-es&rs=es-es&ad=es
You get that link from here https://docs.microsoft.com/en-us/sharepoint/plan-navigation-modern-experience

No, I'm not looking for information on how to customize navigation. Rather, I need to know what specific SharePoint permission (not permission level) allows users to customize or prevents users from customizing navigation. Thank you.

No, I'm not looking for information on how to customize navigation. Rather, I need to know what specific SharePoint permission (not permission level) allows users to customize or prevents users from customizing navigation. Thank you.

The site I referenced says:

You need to be a site owner to make changes to the navigation of a site.

Is that the answer you are looking for? Thank you

No. I think you may want to bubble this question up to somebody who understands SharePoint permissions. A Site Owner is simply a group to which permissions are applied. Full Control is a permission level often given to Site Owner groups. The Full Control permission level is comprised of various SharePoint permissions. I am trying to determine which of those permissions controls a user's ability to manage navigation. A person with expertise in SharePoint permissions would likely know the answer. Thank you.

I understand SharePoint permissions and what you are looking for, however, what is documented is the role. which is site owner.
Will try to find out for you if the specific permission exists for site navigation.
Thank you!

@tombraman Verified that the "Manage List" right is the minimum required permission (which grants other rights along with it, such as Open Items).
From Microsoft.SharePoint.ApplicationPages.TopNavigation:
protected override SPBasePermissions RightsRequired =>
    SPBasePermissions.EmptyMask | SPBasePermissions.ManageLists;

@tombraman Hello, does @tseward information answers your question? thank you

yes it does, thank you!


ENH: Add rand-to-best mutation strategy to differential_evolution.

The existing 'randtobest1exp' and 'randtobest1bin' strategies actually perform current-to-best mutation.
This PR renames them to 'currenttobest1exp' and 'currenttobest1bin' and add the correct rand-to-best mutation methods.

I think the strategy was named wrongly. The question is whether this will cause problems downstream with changed behaviour.

This will also need to be added to the release notes. @rgommers, is there an updated way to supply modifications to the release notes?

@bjude I think this is almost ready to go. It just needs to have updated release notes added.
Firstly to mention the two added strategies, secondly to mention the changed behaviour of the two randtobest strategies.
You can find the release notes in doc/release

@rgommers, is there an updated way to supply modifications to the release notes?

Yes, see http://scipy.github.io/devdocs/dev/index.html#release-notes

Note that adding directly to doc/release is also not a problem, since it's already done in this PR. We switched to wiki to avoid merge conflicts; if there are none here then no problem to merge.

Once the restarted travisCI tests pass I'm going to merge.

Tests are green, so in it goes. Thank you @bjude for your contribution.


feat: ...

What
GUI to change OLD BLS (0x00  => 0x) and set new withdrawal credential
Why

Wagyu keygen only works for ethereum mainnet.
Over a year, I tried everything but was not able to change withdrawal credentials.

Refs
Raised with beacon chain https://github.com/gobitfly/eth2-beaconchain-explorer/issues/2761 but no luck.

Hey @mtr980 , if you're using Dappnode, the GUI is being developed by the team. Meanwhile, you can follow this tutorial  to change your withdrawal credential by using ethdo. For more details, please check: https://docs.gnosischain.com/faq/node/changingwc


Merge with the israeli-bank-scrapers-desktop repo

TODOs:

[x] Merge files from israeli-bank-scrapers-desktop to have the code from both repos in the same place.
[x] Change all the code to using es imports instead of require
[ ] Unify README files
[ ] Remove irrelevant code of original repositories once the ui and the node code work together
[ ] Handle all // TODO in code- fix or open an issue.
[ ] Show error when vuex-persist failed, and re-fetch the config

Related issues:

[x] Make the configManage use the crypto module and remove hard coded encryption key
[ ] Make the node code expose an api as defined here
[ ] Update the electron UI flow to work with two main flows - configure, run scraping as defined here
[ ] Make the electron UI work the node "library" code
[ ] Remove the Contributors part or hide it in a Menu or something.
[ ] Align filename #57

Post Merge Actions:

[ ] Decide how to release (by Tag or by pushing to master)
[ ] Copy relevant issues from israeli-bank-scrapers-desktop.
[ ] New repo/app name?
[ ] Check if there unused packages
[ ] Add MacOS to tests
[ ] Create Splash screen (until the renderer process loaded)
[ ] Set show browser in configManager? Define it for all importers?
[ ] Add ability to disable importer and exporter (and keep them configured)

Note that the folder nuxt contains the Github Pages, do you want to keep it?
I think the regular users need a beautiful webpage in Hebrew, with a big download button and help content.
But if you want to rethink that, maybe to use another technology...

About the file structure, I already started to reorg the project in https://github.com/baruchiro/israeli-bank-scrapers-desktop/pull/247, but it is not important, since I think the most of my code will move to BudgetTrackingApp, or just gone, and the rest of the code is the components folder.

Regarding the githup pages, you mean you wanted to use something else instead?
I am fine with removing it.

About the file structure, I already started to reorg the project in baruchiro/israeli-bank-scrapers-desktop#247, but it is not important, since I think the most of my code will move to BudgetTrackingApp, or just gone, and the rest of the code is the components folder.

Ok, I updated the todo to be to remove the code that is no longer relevant when the ui starts using the budget tracking code.

About the Vue vuex-persist, I choose this way to save the state from the Vuex state manager to configManager, each time the state changes.
I think using the configManager itself in any place in the app is not a good way because in this way the config is not reactive.
The limitation of the vuex-persist way is that it act as a "side effect". I mean, if there is a problem when saving the state with configManager, it will affect the flow. In this case, you can use the app and you don't know that your changes are not saving in the file system.

But why do we need it to persist? We have the persisted state in the filesystem managed by the config manager, and we have the state in memory in the store that is the reflection of what is in the filesystem. Why do we need another layer of persistent state?
The app works with the state that is in the store so any change in the config in the store will cause the app to react. If we have a side effect that the store saves to the config manager on every change, we should have all we need. The additional layer of persisting the view store sounds error prone and I don't see the added value.
WDYT?

I think that is ok as long as if there is an error in persisting the config we show the error. This is basically optimistic UI which is quite a common pattern - the ui works as if the backend update succeeded even before it knows if it did, and if there was an error it shows it when it happens.

If in case of an error we refetch the data from the configManager (and show an error) we will only have a short time of inconsistent state

I added Show error when vuex-persist failed, and re-fetch the config task to this issue, because as I said, saving the config by vuex-persist is not in the main flow- it working in the background, so we need to register a callback or something.
(I wrote it under TODOs, but you can drag it to the other sections, it's very cool 🎉)

@brafdlog I created a new branch called cleanIndex, but it turns to be a very large change, so I leaving it, but I'm sharing with you what I learned.
I want to remove all the code from the root index.ts, to get a clear list about what the backend exporting. I have three (or more) points:

Move all the code from index.ts to separate files in a logical order. The export functionality should be in its file, the same with the import functionality, and the method that takes from import to export in another file.
In this way, we can validate (I hope there is lint for that) the Vue access only to the @/originalBudgetTrackingApp, and the backend itself access only to the flow module, and this module uses its "internal" methods to import and export.
With this structure you can understand half of the code only by its module.
Inject sub-config to internal functions. The function should get only the relevant config, for example, the export function should get only the outputVendors part from the Config.
Separate types from the code. I have no much knowledge about best practices with TypeScript types declaration, but I think if we have a module with a lot of types declarations and more than small function, we should declare the types in a separated file.

I'm sure you share my thoughts, but I just wanted to document what I did, because it came out too big so I decided to stop for now.

I general I agree. For these types of things I suggest opening and issue and linking it to here so it will be easier to follow.
Next week I will have more time and one of the things I plan is to organize the open issues in a clearer way so we have a clearer view about where things stand and what are the open issues.

My suggestion is to create a new "board" for "pre-merge" issues.

@baruchiro I opened a project for the pre merge issues: https://github.com/brafdlog/budget-tracking/projects/4 and removed them from the description here.
Regarding the Todo's in the code, I suggest you open tickets for the ones you think we should handle and remove the ones we don't. We can sync on this if needed.

@brafdlog I created a new branch called cleanIndex, but it turns to be a very large change, so I leaving it, but I'm sharing with you what I learned.
I want to remove all the code from the root index.ts, to get a clear list about what the backend exporting. I have three (or more) points:

Move all the code from index.ts to separate files in a logical order. The export functionality should be in its file, the same with the import functionality, and the method that takes from import to export in another file.
In this way, we can validate (I hope there is lint for that) the Vue access only to the @/originalBudgetTrackingApp, and the backend itself access only to the flow module, and this module uses its "internal" methods to import and export.
With this structure you can understand half of the code only by its module.
Inject sub-config to internal functions. The function should get only the relevant config, for example, the export function should get only the outputVendors part from the Config.
Separate types from the code. I have no much knowledge about best practices with TypeScript types declaration, but I think if we have a module with a lot of types declarations and more than small function, we should declare the types in a separated file.

I'm sure you share my thoughts, but I just wanted to document what I did, because it came out too big so I decided to stop for now.

Opened issues for 1 and 2:
#89
#90
Regarding 3 its a matter of taste, we can discuss.

@baruchiro

Make the node code expose an api as defined here

The current library code already exposes the config manager and the scrapeAndUpdateOutputVendors so I think we are good on that.

Update the electron UI flow to work with two main flows - configure, run scraping as defined here

We did implement those main flows except for the configuration of the category logic for which we have an issue - #31
So I am marking those two as done


Update rebuses.js

Added a rebus
Associated Issue: #121
Summary of Changes

Added Peace of Mind rebus

Pull Request Test Coverage Report for Build 655

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.143%

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


dict's setdefault is understandable and simple

Maybe you want to create your Darknet's Conv2D and some other functions, preserving the original arguments, you can use setdefault to make it easier.

Hi, it seems that this repo is inactive for a while... (more than a year :worried:)
Would you consider to pass your changes to this fork https://github.com/Borda/keras-yolo3 ?


Gettimestamp is not working

Hi,
I am not sure if this is an issue. But I re-installed everything and everything seems to work. except, during the logs and sitemap, I get:
Rule 'Roomba Last Update Timestamp': The name 'getTimestamp' cannot be resolved to an item or type; line 48, column 5, length 12

Nick, i have mine mostly working but I'm also having an issue with the Date(). I'm getting error message in VSCode:
"Date cannot be resolved.(org.eclipse.xtext.diagnostics.Diagnostic.Linking) [10,66]"
My OH log shows the the following error:
2019-02-09 14:01:14.710 [ERROR] [ntime.internal.engine.RuleEngineImpl] - Rule 'Roomba Last Update Timestamp': null
All the associated items that use the "Timestamp" function are null as well.
I have the lambda at the top of my rules file with the recommended imports.
Here is my rules file:
import org.eclipse.xtext.xbase.lib.Functions    //apparently needed for OH2 lambda's
import org.eclipse.xtext.xbase.lib.Procedures   //apparently needed for OH2 lambda's

val Functions$Function2<GenericItem, String, String> getTimestamp = [  //function (lambda) to get a timestamp. Returns formatted string and optionally updates an item
    item,
    date_format |

    var date_time_format = date_format
    if(date_format == "" || date_format === null) date_time_format = "%1$ta %1$tT" //default format Day Hour:Minute:Seconds
    var String Timestamp = String::format( date_time_format, new Date() )
    if(item != NULL && item !== null) {
        var t = new DateTimeType()
        if(item instanceof DateTimeItem) {
            item.postUpdate(t) ////postUpdate(item, t)
            logInfo("Last Update", item.name + " DateTimeItem updated at: " + Timestamp )
            }
        else if(item instanceof StringItem) {
            item.postUpdate(Timestamp) //postUpdate(item, Timestamp)
            logInfo("Last Update", item.name + " StringItem updated at: " + Timestamp )
            }
        else
            logWarn("Last Update", item.name + " is not DateTime or String - not updating")
    }
    Timestamp
    ]

/* Roomba Rules */
rule "Roomba start and stop"
when
    Item roomba_control received command
then
    logInfo("Roomba", "Roomba ON/OFF received command: " + receivedCommand)
    if (receivedCommand == ON)
        sendCommand(roomba_command, "start")
    if (receivedCommand == OFF) {
        sendCommand(roomba_command, "stop")
        Thread::sleep(1000)
        sendCommand(roomba_command, "dock")
    }
end

rule "Roomba Auto Boost Control"
when
    Item roomba_carpetBoost changed
then
    logInfo("Roomba", "Roomba Boost changed to: Auto " + roomba_carpetBoost.state + " Manual: " + roomba_vacHigh.state)
    if (roomba_carpetBoost.state == ON && roomba_vacHigh.state == ON)
        sendCommand(roomba_vacHigh, OFF)
end

rule "Roomba Manual Boost Control"
when
    Item roomba_vacHigh changed
then
    logInfo("Roomba", "Roomba Boost changed to: Auto " + roomba_carpetBoost.state + " Manual: " + roomba_vacHigh.state)
    if (roomba_carpetBoost.state == ON && roomba_vacHigh.state == ON)
        sendCommand(roomba_carpetBoost, OFF)
end

rule "Roomba Auto Passes Control"
when
    Item roomba_noAutoPasses changed or
    Item roomba_twoPass changed
then
    logInfo("Roomba", "Roomba Passes changed to: Auto " + roomba_noAutoPasses.state + " Manual: " + roomba_twoPass.state)
    if (roomba_noAutoPasses.state == ON && roomba_twoPass.state == ON)
        sendCommand(roomba_twoPass, OFF)
end

rule "Roomba Last Update Timestamp"
when
    Item roomba_rssi received update
then
    getTimestamp.apply(roomba_lastheardfrom, "%1$ta %1$tR")
end

rule "Roomba Bin Full"
when
    Item roomba_full changed from OFF to ON
then
    val Timestamp = getTimestamp.apply(roomba_lastheardfrom, "%1$ta %1$tR")
    sendPushoverMessage(pushoverBuilder("BIN FULL reported by Roomba at: " + Timestamp))
end

rule "Roomba Error"
when
    Item roomba_error changed from OFF to ON
then
    val Timestamp = getTimestamp.apply(roomba_lastheardfrom, "%1$ta %1$tR")
    sendPushoverMessage(pushoverBuilder("ERROR reported by Roomba at: " + Timestamp))
    sendMail("myemailaddress", "Roomba", "ERROR reported by Roomba at: " + Timestamp + "See attachment for details", "http:XXX.XXX.X.XX:8080/static/map.png")
end

rule "Roomba percent completed"
when
    Item roomba_sqft received update
then
    var sqft_completed = roomba_sqft.state as Number

    var max_sqft = 470  //insert max square footage here
    var min_sqft = 0

    var Number completed_percent = 0

    if (sqft_completed < min_sqft) {completed_percent = 0}
    else if (sqft_completed > max_sqft) {completed_percent = 100}
    else {
        completed_percent = (((sqft_completed - min_sqft) * 100) / (max_sqft-min_sqft)).intValue
        }
    logInfo("Roomba", "Roomba percent complete "+roomba_sqft.state+" of "+max_sqft.toString+" calculated as " + completed_percent.toString + "%")
    postUpdate(roomba_percent_complete,completed_percent)
end

rule "Roomba update command"
when
    Item roomba_phase received update
then
    logInfo("Roomba", "Roomba phase received update: " + roomba_phase.state)
    switch(roomba_phase.state) {
        case "run"          : postUpdate(roomba_command,"start")
        case "hmUsrDock"    : postUpdate(roomba_command,"pause")
        case "hmMidMsn"     : postUpdate(roomba_command,"pause")
        case "hmPostMsn"    : {
                              postUpdate(roomba_command,"dock")
                              getTimestamp.apply(roomba_lastmissioncompleted, "%1$ta %1$tR")
                              }
        case "charge"       : postUpdate(roomba_command,"dock")
        case "stop"         : postUpdate(roomba_command,"stop")
        case "pause"        : postUpdate(roomba_command,"pause")
        case "stuck"        : postUpdate(roomba_command,"stop")
    }
end

rule "Roomba Notifications"
when
    Item roomba_status changed
then
    logInfo("Roomba", "Roomba status is: " + roomba_status.state)
    val Timestamp = getTimestamp.apply(roomba_lastheardfrom, "%1$ta %1$tR")
    switch(roomba_status.state) {
        case "Running"                  : sendPushoverMessage(pushoverBuilder("Roomba is RUNNING at: " + Timestamp))
        case "Docking - End Mission"    : {
                                          createTimer(now.plusSeconds(2)) [|
                                              sendPushoverMessage(pushoverBuilder("Roomba has FINISHED cleaning at: " + Timestamp))
                                              sendMail("myemailaddress", "Roomba", "Roomba has FINISHED cleaning at: " + Timestamp + "See attachment for details", "http://XXX.XXX.X.XX:8080/static/map.png")
                                              ]
                                          }
        case "Stuck"                    : {
                                          sendPushoverMessage(pushoverBuilder("HELP! Roomba is STUCK at: " + Timestamp))
                                          sendMail("myemailaddress", "Roomba", "HELP! Roomba is STUCK at: " + Timestamp + "See attachment for location", "http://XXX.XXX.X.XX:8080/static/map.png")
                                          }
    }
end

rule "Roomba Schedule Display"
when
    Item roomba_cycle changed or
    Item roomba_cleanSchedule_h changed or
    Item roomba_cleanSchedule_m changed
then
    logInfo("Roomba", "Roomba Schedule: Day " + roomba_cycle.state + " Hour: " + roomba_cleanSchedule_h.state + " Minute: " + roomba_cleanSchedule_m.state)
    var String schedule = ""
    var String days = (roomba_cycle.state as StringType).toString
    var String hours = (roomba_cleanSchedule_h.state as StringType).toString
    var String minutes = (roomba_cleanSchedule_m.state as StringType).toString
    val ArrayList daysOfWeek = newArrayList("Sun","Mon","Tues","Wed","Thur","Fri","Sat")
    val ArrayList<String> daysList = new ArrayList(days.replace("[","").replace("]","").replace("'","").split(","))
    val ArrayList<String> hoursList = new ArrayList(hours.replace("[","").replace("]","").split(","))
    val ArrayList<String> minutesList = new ArrayList(minutes.replace("[","").replace("]","").split(","))
    daysList.forEach[ item, i |
        if(item.trim() == "start") {
            schedule += daysOfWeek.get(i) + ": " + hoursList.get(i) + ":" + minutesList.get(i) + ", "
        }
    ]
    postUpdate(roomba_cleanSchedule, schedule.trim())
end

Everything else seems to be funtioning normal with exception of another VSCode error regarding "ArrayList cannot be resolved" which I don't think is related.
Any ideas?


Should have "Live" toggle activated by default

When we open the embedded browser with ctrl+alt+o
The "Live" toggle should be activated by default.
Or at least have the option to activated it by default in the settings of the package ;)

This is simply a browser. If you are looking for live reload options...check out the pp(preview-plus) package
https://atom.io/packages/pp

I need this feature by default too. Is there a command available to do this so that we don't have to manually click live always. Please let me know if anyone has any solution.


Refer to current state from ui-sref (e.g. for setting url parameters).

The documentation explicitly states "You can also use relative state paths within ui-sref, just like the relative paths passed to $state.go()." However, even though we are able to refer to the current state with $state.go('.', {foo: 'bar'}) it doesn't seem to work when using ui-sref directive. I naively tried ui-sref=".({foo:'bar'})" which failed.
Of course it is possible to wire up ng-click and $state.go, but that leaves us with href-less anchors, which is not very useful from the user perspective.
I've ventured a related question on stackoverflow in case somebody had figured it out already. But it appears not to be the case.
Does this functionality exist already (maybe I just missed/misused it)? If not, should it be implemented?

+1 on this. Pretty pressing for handling user-friendly async pagination.

+1

+1

+1

+1

+1 using ui-sref="{foo: 'bar'}" renders the href tag on the first state and does not continue to update on state changes.

Okay, so sounds like maybe the $watch() is wrong. If somebody can post me up a quick Plunkr I'll take a look at it.

Pretty sloppy, but it should give you an idea. The expected outcome is for it to stay on either the color or size route when you click on a different item. http://plnkr.co/edit/fARlus6UmokOsdRxbaoS

I just updated the plunk with AngularJS 1.3.15 and ui-router 0.3.14 and the href tag still does not get updated on state change. http://plnkr.co/edit/8jZCbEVuoO4srwU8fIY5?p=preview

Okay, I see the issue. Should be able to patch it later today.

@nateabele any luck?

Confirm I am able to switch between items in my current state by changing one param value using

angular 1.3.14, angular-ui-router 0.2.13

This only works the first time.

On later changes, the itemId is correct, but the suffix part of the pageload URL is used.
Angular 1.3.15, angular-ui-router 0.2.15.

This seems to work for me:
ui-sref='.({ id: theID })'

@ruleb I later found what you said: I was mistaken, it only worked the first time, but not on later changes. Ended up with a workaround involving marking some states as needing reload using the data object (data: { reload: true }), and then a $on('$stateChangeStart' handler to see that and perform the reload.
Seems to be working as needed (~2 months use) but feels like a fragile workaround.

I would like to submit a possible solution this issue.
The proposed method is to treat the dot placeholder as current state name i.e.:
ui-sref=".({locale: langCode})"

Change details:
In the ui-sref directive code:
  var update = function(val) {
    if (val) def.params = angular.copy(val);

    // If specified state is the "." placeholder, substitute it for the current state name:
    def.href = $state.href(ref.state === '.' ? $state.current.name : ref.state, def.params, def.options);

    if (active) active.$$addStateInfo(ref.state, def.params);
    if (def.href !== null) attrs.$set(type.attr, def.href);
  };

And then watch for the state change to update the href when current state changes:
  if (ref.state === '.') {
    scope.$on('$stateChangeSuccess', function (event, toState, toParams, fromState) {
      if (fromState.name !== toState.name) {
        update();
      }
    });
  }

+1 for handling "."

+1 for an elegant notation such as "." to refer to the current state.
Has there been any further progress on this?

+1 for this

+1  for handling "."

+1 for the dot proposal!

+1

+1

+1

Anyone who would like to submit a PR to implement this is welcome to do so.

Submitted my solution

@shprota works for me, thanks!

It would be great if you could merge @shprota's PR.

I've done some analysis on this. We can't use "." to mean "the current state" because it already has specific meaning, and it is context sensitive.   A relative link is relative to its context (the state that the ui-sref was created in).  This means that ui-sref="." (defined in the state called home)  always links to home, even if home.foo is the current active state.
"." cannot refer to the currently active state, and we already have syntax for parameter-only ui-srefs, but it is broken in some cases.  I am fixing the existing parameter-only ui-sref syntax so it updates when the state changes.  i.e., ui-sref="{ id: 345 }"

Here's an example which has two states: home and home.foo.  There are both styles of ui-sref links, and their equivalent ui-state links.  Note the difference in behavior (when the current state is home.foo) between links with a "." links without one.
http://plnkr.co/edit/JzuxiVezaAaW697JYE8o?p=preview


Feedback for Arrays and Hashes Blog

Yo Chris!
I definitely like your writing style and I like your clean site layout! I wish I coulda seen what it looked like before hand though just to see how much you've improved, but anyways, nice work!
I like the examples you used and your clear breakdown of how arrays and hashes work, ouch to the sports teams you mentioned though, don't know if that 49ers example will fly once you're out in SF though haha. You have a great way of presenting your explanation, too, even though you wrote a good amount, your use of spacing is definitely something to take note of in my opinion to make it seem less text heavy.
In terms of feedback, I don't know if I any real actionable thing I can think of, besides maybe just providing an example of when you may want to use one rather than the other!
Keep it up man. Hope to pair with you sometime, let me know if you're interested!

Thanks dude,
I meant to @ you on Twitter your site design inspired me to redo mine and
make it better and simpler. Thanks for the feedback on the blog I'm glad
you enjoyed it, and I'm definitely down to pair next week if you want!
Hit me up!
On Saturday, April 25, 2015, Harvey Ngo<EMAIL_ADDRESS>wrote:

Yo Chris!
I definitely like your writing style and I like your clean site layout! I
wish I coulda seen what it looked like before hand though just to see how
much you've improved, but anyways, nice work!
I like the examples you used and your clear breakdown of how arrays and
hashes work, ouch to the sports teams you mentioned though, don't know if
that 49ers example will fly once you're out in SF though haha. You have a
great way of presenting your explanation, too, even though you wrote a good
amount, your use of spacing is definitely something to take note of in my
opinion to make it seem less text heavy.
In terms of feedback, I don't know if I any real actionable thing I can
think of, besides maybe just providing an example of when you may want to
use one rather than the other!
Keep it up man. Hope to pair with you sometime, let me know if you're
interested!
—
Reply to this email directly or view it on GitHub
https://github.com/Yago580/Yago580.github.io/issues/2.

Yoo whattt.
That means a lot to me too man, I'm honored you felt inspired by mine cause the feeling's mutual!
So far I got a pairing session planned with Aaron next Wednesday I think, but I'll hit you up if there's another thing we can work on, else let's make sure to pair the week after!


macOS: missing libs

When trying to use this on macOS MS Excel, it appears the following libraries are not provided:

MISSING: Microsoft Scripting Runtime#Microsoft Scripting Runtime
MISSING: Microsoft VBScript Regular Expressions 5.5#Microsoft VBScript Regular Expressions 5.5

Any chance we can figure out how to make this work without these or another workaround?

Wow, a Mac user - cool! 🤩
Sorry this isn't working smoothly. I've only ever developed/used this on Windows, so I had no way to know this would happen.
I don't think I'll be able to help much on this, since I don't have access to a Mac, much less a Mac with Office. The project might need to be forked to a Mac-dedicated version and maintained by someone else.
That said, the first step is to look through the references that are available and try to find replacements for the two that are missing.

The Microsoft Scripting Runtime is what provides access to the filesystem, and reading/writing files.
Microsoft VBScript Regular Expressions 5.5 provides regex functionality.

If you can find these replacements, you can try disabling these Windows-specific ones and enabling the Mac-specific ones. I rather doubt that will be all that's needed for a fix, though -- the interfaces are likely different.
I'm happy to advise on any efforts working toward a Mac version...just let me know.


If button drop-down is on bottom of page, not attached to button anymore & hides button

Steps:

Go to Create Deployment
Click on Storage Tab
Click on Add Volume drop-down button
Add a bunch of  Ephermeral Volumes until the Add Volume button is on bottom

Resuts:  the list is no longer attached to the button and the tab at the top is pointing at the field above it.

Convert this to v-select if possible.

Verified fixed


Bug Report Rabbits foot replaced with Bucket of Milk

What happened?
The rabbits foot is just flat out replaced with a bucket of milk. In both texture and translated name.
How to replicate/reproduce:
Get a rabbits foot.
What version did you find the issue on?
1.21.3
If you have a screenshot or video of the issue, please link it here.

The way everything is set up by itself is really weird. Why are they rabbits foot? chocolate milk is a firework star like all other foods. To be honest. Using obtainable items for anything is an odd choice I'd personally recommend the command block/structure block items as they cannot be placed in survival, nor obtained. Also overriding base items with data/resource pack textures is ill adviced and recommended to use custom data models.

This is actually intentional behavior. Due to the way I had to code this to add custom crafting ingredients to the game, a couple vanilla items had to be essentially replaced. You can exploit this a fair bit if you know which items have been replaced, but in the case of Rabbit's Feet, it still works the same way as a normal version. Killing Rabbits will still drop the regular textured item.
Fixing all Rabbit's Foot items from containers is quite annoying since I don't know if any other datapacks have messed with it, and making a function to update them automatically might break other packs. I'll see if I can fix this in the future but for now it's low priority.
(I also don't want to use admin items like Command Blocks, even if they can't be placed in Survival Mode, I prefer to not give players those kinds of items, despite them not being able to use them)

it still works the same way as a normal version. Killing Rabbits will still drop the regular textured item.

It doesn't. Well, at least, it doesn't for the rabbits foot given to you by cats that sleep on you. If this is a separate issue reopen it with a renamed title.

Oh. Wow, I actually missed that with the cat gifting. Thank you for pointing that out.
You know what's funny? There was a similar issue with Phantoms not dropping the correct item, but NOBODY has told me presumably because nobody plays with phantoms enabled.
Anywho, it's fixed. Thanks! Any Rabbit's Foot items still inside chests won't be updated but killing rabbits and getting cat gifts should be fine now.

It is kinda funny, On my server we do play with phantoms. So we would've noticed if that was a bug.


Memory leak in Loop

I've found a memory leak inside the icicle loop when yielding from a co routine.
This can be replicated with the below code:
<?php

require_once("./vendor/autoload.php");

use Icicle\Loop;
use Icicle\Coroutine\Coroutine;

function gen()
{
    while(true){
        yield array_fill(0, 10000, 4);
        print memory_get_usage(false).PHP_EOL;
    }
}

Loop\immediate(
    function() {
        new Coroutine(gen());
    }
);

Loop\Run();

This is possibly related to https://bugs.php.net/bug.php?id=69887 and https://bugs.php.net/bug.php?id=62210 which is reported to be fixed in php7.1.

This occurs due to an optimization made to the method used to executed queued callbacks in the loop. The coroutine engine uses queued callbacks to asynchronously continue coroutine execution and avoid blowing up the call stack. The values used in these callbacks is not removed until all callbacks in the current loop iteration are executed. Callbacks added during the same loop iteration will be run during the same iteration. This means each time the coroutine yielded an array, a reference would be kept until the next loop iteration. Since so much data was being produced, memory would run out before another loop iteration could occur and the arrays garbage collected.
See https://github.com/icicleio/icicle/blob/master/src/Loop/Structures/CallableQueue.php#L96-L115
array_slice() is not executed until all callbacks are executed, causing the arguments (in this case, the array generated by array_fill()) to build up in the queue.
I'll have to take a different approach to avoid hanging onto references longer than needed.


tree

Introduces two new options transforms: Plot.treeNode, for laying out tree nodes, and Plot.treeLink, for laying out tree links. The new convenience mark Plot.tree combines a link, dot, and text with these options transforms to generate a tree diagram.
The Plot.treeNode options transform supports special evaluators for tree nodes. If a channel is defined as a string starting with “node:” then it is interpreted as one of the built-in node evaluators:

node:path - the normalized, slash-delimited path to this node
node:name - the last part of this node’s path
node:depth - this node’s distance from its furthest ascendant
node:height - this node’s distance from its furthest descendant
node:internal - true if this is an internal node; false if this is a leaf node

Alternatively you can specify a channel as an {node} object where node is a function. This function is then passed the d3-hierarchy node instance.
The Plot.treeLink options transform similarly supports special evaluators for tree links. If the “node:” prefix or {node} syntax is used, it refers to child node of the link; if the “parent:” prefix syntax is used, it refers to the parent node of the link. If the {link} syntax is used, the function is passed two arguments (child, parent) and can compute the desired corresponding value.
TODO

[x] Use root.count or root.sum and expose a node:value somehow? Not now.
[x] Support different sort orders (as a treeSort option?)
[x] Support different separations (as a treeSeparation option?)
[x] Support different orientations of the tree (as a treeAnchor option?)
[x] Support faceting
[x] Support imputed nodes
[x] Documentation

Related #136.

Okay, I figured out how to get faceting and imputed nodes to work: now the tree transforms generate one element per node or per link, respectively, and for imputed nodes the associated data is null (but you can still access node-related metadata using a node accessor). Here’s a little example of the flare dataset with small nodes (<5000 bytes) on the left and large nodes (≥5000 bytes) on the right. Fun!


remove boost dependency from Dash-specific code

This is only a partial replacement, as some of the boost items aren't simple swaps. This PR replaces lexical_cast and the boost dependency in CSuperblock shared_ptr. The latter should be straightforward, doesn't use anything specific about boost::shared_ptr that can't be done with std::shared_ptr

Not sure if it's a good idea to touch message signing parts while we are in feature-freeze mode and since this fixes no bug I'd postpone this till 12.4.

I absolutely agree w/you there, my intent was to just get it out for later. I can remember to make a note for PRs that I don't intend to be in the next release, esp. for cases like this where it's in freeze mode.

Rebased this onto latest develop, so will need a re-ack at this point.

@nmarley what is the point of removing it from dash-specific code if boost is still needed to compile, due to not changing boost depends from btc?

@PaulieD So that we can drop it when it's time. It's a big ugly dependency that's largely been superceded with the new C++11/14/17 standards.
Are you aware that Bitcoin is also migrating off Boost? : https://github.com/bitcoin/bitcoin/projects/3


Add getter for current count to counter

Add a getter for the counter.
Should we also change setCounterValue(value: number): to just be a setter for the property?

Not sure I follow re:  %property notation, do you mean adding them to the blockCombine portion?

Yes blockComine. A property is much nicer in TS anyway.

From: Joey Wunderlich<EMAIL_ADDRESS>Sent: Monday, April 15, 2019 7:07:37 PM
To: Microsoft/pxt-arcade
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt-arcade] Add getter for current count to counter (#892)
Not sure I follow re: %property notation, do you mean adding them to the blockCombine portion?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-arcade%2Fpull%2F892%23issuecomment-483483198&data=02|01|jhalleux%40microsoft.com|962864b86d1842be69ac08d6c2104c95|72f988bf86f141af91ab2d7cd011db47|1|0|636909772589074783&sdata=3c%2FNAZIV3FbodUgkMO5hvAF%2BlX32cMxsDMk3nl4OrvM%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4Kb7mnSM9KRFWV1Xm9hP56AHWour5ks5vhTBpgaJpZM4cxMmG&data=02|01|jhalleux%40microsoft.com|962864b86d1842be69ac08d6c2104c95|72f988bf86f141af91ab2d7cd011db47|1|0|636909772589074783&sdata=MlQ8JmswVjsE3b7UuJnN4tyIKpDrub8zYjzJf%2BHleZ0%3D&reserved=0.

As written this will be a breaking change for the api, I can see about upgrade rules if we want em

This is a secondary library and we do not have any project using it? Let’s break it.

From: Joey Wunderlich<EMAIL_ADDRESS>Sent: Monday, April 15, 2019 7:19:01 PM
To: Microsoft/pxt-arcade
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt-arcade] Add getter for current count to counter (#892)
As written this will be a breaking change for the api, I can see about upgrade rules if we want em
[Screen Shot 2019-04-15 at 7 16 59 PM]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F5615930%2F56177345-2705ff80-5fb3-11e9-8095-f303b317ba8f.png&data=02|01|jhalleux%40microsoft.com|6b0762338ba44f0f275808d6c211e472|72f988bf86f141af91ab2d7cd011db47|1|0|636909779436844940&sdata=%2FSS01BECY0DWZNYk0xTovV8qi%2FoWA2Nj%2FagWVfAS0HE%3D&reserved=0
[Screen Shot 2019-04-15 at 7 16 52 PM]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F5615930%2F56177346-2705ff80-5fb3-11e9-8a3a-1984eebfd770.png&data=02|01|jhalleux%40microsoft.com|6b0762338ba44f0f275808d6c211e472|72f988bf86f141af91ab2d7cd011db47|1|0|636909779436854949&sdata=FFs3wvF903CBlc2YElkdMgPKkJ4KrobmbbgsLSrnInk%3D&reserved=0
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-arcade%2Fpull%2F892%23issuecomment-483485296&data=02|01|jhalleux%40microsoft.com|6b0762338ba44f0f275808d6c211e472|72f988bf86f141af91ab2d7cd011db47|1|0|636909779436854949&sdata=JnEVNw6Y22nQ9tqzdnoThRPN360%2FBeUJrMD95Ay1o8U%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KZKQjHtnZcDRV9HvSzPsz9HcG3V7ks5vhTMVgaJpZM4cxMmG&data=02|01|jhalleux%40microsoft.com|6b0762338ba44f0f275808d6c211e472|72f988bf86f141af91ab2d7cd011db47|1|0|636909779436864953&sdata=Ry24%2BDMTN%2FKDjIU7m1jKx3cJQ2ArzgVD28NbDcxSiDc%3D&reserved=0.

Sounds good, I'll drop the 'increment' function too then as it's already handled by change by

Okay, made a pass through digits to make sure things were consistent:
current:

After this change:

What is the different between counter and digits?

From: Joey Wunderlich<EMAIL_ADDRESS>Sent: Monday, April 15, 2019 8:02:12 PM
To: Microsoft/pxt-arcade
Cc: Peli de Halleux; Review requested
Subject: Re: [Microsoft/pxt-arcade] Add getter for current count to counter (#892)
Okay, made a pass through digits to make sure things were consistent:
current:
[Screen Shot 2019-04-15 at 8 01 24 PM]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F5615930%2F56179079-4f90f800-5fb9-11e9-8b30-f55e7c2975c9.png&data=02|01|jhalleux%40microsoft.com|a3bb6860a7e64684a8f408d6c217ecaa|72f988bf86f141af91ab2d7cd011db47|1|0|636909805338943862&sdata=FxGJuJRxHsmH9nXZHZDc55Mve%2FNGkO7MgeeQUwxWWlk%3D&reserved=0
After this change:
[Screen Shot 2019-04-15 at 7 58 18 PM]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F5615930%2F56178988-1062a700-5fb9-11e9-94e5-04c2e05a831c.png&data=02|01|jhalleux%40microsoft.com|a3bb6860a7e64684a8f408d6c217ecaa|72f988bf86f141af91ab2d7cd011db47|1|0|636909805338953875&sdata=N09uluCveiErwU4e48blvUEAGb%2Fbw9pXZcltZdrVqGQ%3D&reserved=0
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-arcade%2Fpull%2F892%23issuecomment-483492840&data=02|01|jhalleux%40microsoft.com|a3bb6860a7e64684a8f408d6c217ecaa|72f988bf86f141af91ab2d7cd011db47|1|0|636909805338963883&sdata=fs10pQpNaIEadFkYXtNx4%2BhSyfc66A6wioOwy%2Blf0mQ%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KeS4DyBjPbS-VH92WObfGZrSpQlbks5vhT00gaJpZM4cxMmG&data=02|01|jhalleux%40microsoft.com|a3bb6860a7e64684a8f408d6c217ecaa|72f988bf86f141af91ab2d7cd011db47|1|0|636909805338963883&sdata=voCaXlsprZD%2FVNdRXoqb5YA6tTeT6fmR7i7nVSTlV6Y%3D&reserved=0.

A counter is a series of digits, where digits are just the single numbers that show up on the screen

why is "set display value" a string? it seems like it is a number in the other blocks.

That maps to setDigitAlpha. Typically you would use the blockCombined set value block, but this one has a few extra characters which can be displayed
setDigitAlpha(alphaChar: string) {
    const matchChars = "0123456789ABCDEFHJLoPUY-°";
    let maxChars = Math.min(this._radix, matchChars.length)
    if (alphaChar.length > 0) {
        for (let i = 0; i < maxChars; i++) {
            if (matchChars[i] == alphaChar[0] && i != this.value) {
                this.value = i;
                sevenseg.drawDigit(this.digit, this.value, this.thickness, this.scale, this.color);
                break;
            }
        }
    }
}

I left it in for because it's used in Galen's clock example on https://arcade.makecode.com/examples/seven-segment , which is pretty cool. Does seem like a bit of an advanced feature, or maybe one that could be associated from an enum of 'special characters' now that I think of it

What does it do?

Just extra characters that can be displayed:

https://makecode.com/_6HyCoe1g2DV8

IMO, it should not be a string, it is misleading since there is very limited set of characters.

@jwunderl - Thanks for doing these mods.


Binding attribute properties for files attempt to load the file from the runtime folder in Azure, not the deployed folder

This is the output binding attribute I used:
[Kafka(
"%brokerlist%",
"",
SslCaLocation = "cacert.pem",
AuthenticationMode = BrokerAuthenticationMode.Plain,
Protocol = BrokerProtocol.SaslSsl,
Username = "$ConnectionString",
Password = "%SendEventHubConnString%")]
IAsyncCollector<KafkaEventData> kafkaOut
When this was deployed to Azure, the function fails with this exception message:
ssl.ca.location failed: .\crypto\x509\by_file.c:254: error:0B084002:x509 certificate routines:X509_load_cert_crl_file:system lib:
I have tracked it down to the library is attempting to load the cacert.pem file from this location: D:\Program Files (x86)\SiteExtensions\Functions\3.0.14191\32bit\cacert.pem
Obviously the file does not exist there, it is in D:\home\site\wwwroot\cacert.pem
Giving the full path to the file works, and the function is able to send messages correctly.
I would expect the library, if given a file name, to attempt to load it from the folder where the function code is deployed to.

Hi @ThomasManson
Thank you for creating the issue.  The current recommended solution is:

Download CA certificate (i.e. from https://curl.haxx.se/ca/cacert.pem).
Rename the certificate file to anything other than cacert.pem to avoid any conflict with existing EventHubs Kafka certificate that is part of the extension.
Include the file in the project, setting "copy to output directory"
Set the SslCaLocation trigger attribute property. In the example we set to confluent_cloud_cacert.pem

In addition, I'm working on this https://github.com/Azure/azure-functions-kafka-extension/pull/151 Once released, we don't need to have CA cert anymore.
https://github.com/Azure/azure-functions-kafka-extension#connecting-to-confluent-cloud-in-azure

I tried this. Downloaded the CA certificate again, called it cloud_cacert.pem. Added to the solution and ensured it was deployed.
Changed the attribute to use "cloud_cacertpem" for the SslCaLocation. Deployed to Azure functions.
Checked that the file has been deployed. It is in both the D:\home\site\wwwroot and D:\home\site\wwwroot\bin folders.
Got the same error message when trying to send to a Kafka enabled Service Bus queue:
ssl.ca.location failed: .\crypto\x509\by_file.c:254: error:0B084002:x509 certificate routines:X509_load_cert_crl_file:system lib:
If I change the SslCaLocation to be D:\home\site\wwwroot\cloud_cacert.pem, it works correctly.

@TsuyoshiUshio just assigned this to you. If your work on #151 will resolve the need for this, then that could be a solution. Else we need to look in to this as a customer is currently blocked and asking for our plan.

In an output binding (to write to Kafka) these values are specified as properties on an attribute, and are therefore specified at compile time. It's not great that we have a hardcoded path in code at compile time.
This could be fixed in the library.
There is already code in the library to determine the root location for the files, because we load some other files from that location. The library should use that same code to find the full path to the cacert.pem file, and load it from there.
If this is not addressed by #151 then we should fix this.


Limit public endpoint access to specific CIDRs

Describe the feature
Allow limiting access to the public EKS/K8S endpoints to a list of CIDRs.
Use Case
This would allow users to protect the endpoint from being accessed by anyone, without having to set it to private and then establish a link of some kind to the VPC (e.g. VPN).
Proposed Solution
EKS clusters with EndpointAccess set to PUBLIC_AND_PRIVATE (or PUBLIC) can optionally specify a list of CIDR blocks that are allowed to access the endpoint. Currently the ClusterProvider only exposes the privateCluster option as a boolean, with the default (true) setting the cluster to PUBLIC_AND_PRIVATE. This option could be replaced with one that accepts an EndpointAccess object, as it already includes a method for restricting public access.
https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_eks.EndpointAccess.html
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request

CDK version used
2.43.0
EKS Blueprints Version
1.3.0
Node.js Version
v18.9.0
Environment details (OS name and version, etc.)
macOS M1

@aceat64 you can do it using ClusterProvider and setting endpointAccess: eks.EndpointAccess.PUBLIC_AND_PRIVATE.onlyFrom(...cidr: string[])
See example bellow.
export default class YourClusterConstruct extends Construct {
  constructor(scope: Construct, id: string, props?: cdk.StackProps) {
    super(scope, id);

    const account = props?.env?.account!;
    const region = props?.env?.region!;

    const clusterProvider = new blueprints.GenericClusterProvider({
      version: KubernetesVersion.V1_21,
      endpointAccess: eks.EndpointAccess.PUBLIC_AND_PRIVATE.onlyFrom(['<IP_ADDRESS>/32', '<IP_ADDRESS>/32']),

    });

    const blueprint = blueprints.EksBlueprint.builder()
      .clusterProvider(clusterProvider)
      .account(account)
      .region(region)
      .addOns()
      .teams()
      .build(scope, id+'-eks');
  }
}


[Android] Google Play's Photo and Video Permissions policy.

https://support.google.com/googleplay/android-developer/answer/14115180
Timeline information
See the following timeline for the Photo and Video Permissions policy. Note that this is subject to change; updates will be posted in this article.
October 2023: We announced the new Photos and Video Permissions policy.
August 31, 2024:
Apps with one-time or infrequent use of photos are requested to remove READ_MEDIA_IMAGES and READ_MEDIA_VIDEO permissions from their app manifest and migrate to a system photo picker if necessary.
Apps with one-time or infrequent use that need time to remove permissions or migrate to a picker can request an extension until January 2025.
Apps with core use-case or broad access use can begin providing use-case rationale using the declaration form within Play Console.
In the given situation, it is necessary to remove the permissions. However, removing these permissions causes the library to not function correctly, so assistance is needed.

Check new version: 2.0
Docs: https://baronha.github.io/react-native-multiple-image-picker/


Adding docker for DLC generation

The previous docker file was removed from the flutter version of the app. So, adding it back, to be used for DLC generation for QTI backend.

Can you move the file flutter/android/docker/dataset_models/Dockerfile to datasets/Dockerfile?

I think it should be datasets/docker/Dockerfile, for consistency with other dockerfiles.

Can you move the file flutter/android/docker/dataset_models/Dockerfile to datasets/Dockerfile?

I think it should be datasets/docker/Dockerfile, for consistency with other dockerfiles.

the dockerfile is required to be used for model generation as well. So, thought of keeping it in flutter/android.

updated the paths

the dockerfile is required to be used for model generation as well. So, thought of keeping it in flutter/android.

I did not test it, but would be great if we can use the already existed flutter/android/docker/Dockerfile instead of the newly uploaded datasets/docker/Dockerfile.


Feature: Make internal constructors protected so that they can be inherited by others.

Because the constructors of our wrapper classes are internal we can't extend them in other libraries as is the case described in https://github.com/KristofferStrube/Blazor.FileSystemAccess/issues/27.
To solve this and ensure that others can potentially extend this package we will make all constructors that are currently internal protected instead.

I found that having all constructors be protected meant that we would have to use the Create methods in all places where we would create a new instance which would be an extensive change so we simply did it only for WritableStream, at least for now.


Enable tx partition depth 2

Fix for #1195 and current issue 3 of tiling, https://github.com/xiph/rav1e/pull/1126.

Since this PR has commit which disables directional intra prediction, there will be loss of coding gain as before. With 2n commit applied, i.e. enabling intra tx depth partition depth = 2, there will be small amount of coding gain.

Coverage decreased (-0.6%) to 82.254% when pulling 78788193d0bb57238e5dccdc4ddcdbb8363e26bc on ycho:enable_tx_partition_depth_2 into 5ee66c1c978d2dc25dd16e58f77b74d468dd1b44 on xiph:master.

By 1st commit,
AWCY loss of coding gain

PSNR
PSNR Cb
PSNR Cr
PSNR HVS
SSIM
MS SSIM
CIEDE 2000

0.5464
0.5360
0.6911
0.5730
0.6738
0.6032
0.4765

For -s 3,
AWCY disabling directional intra prediction

PSNR
PSNR Cb
PSNR Cr
PSNR HVS
SSIM
MS SSIM
CIEDE 2000

0.8206
0.7155
0.3183
0.8639
0.9814
0.9463
0.7791

then with 2nd commit, i.e. enabling depth = 2 tx partition,
AWCY

@ycho might be good to open an issue about the directional predictors.

@lu-zero , sure I will create separate issue for the fragile directional intra prediction. Thanks-


list kinds not working in kind: Kustomization (only?)

This one works

The annotation "part-of" looks like the reason


Allow check and diff together

This PR closes #68
Changed

allow --check and --diff to be used together [#68]

Note: The coverage has reduced as I've removed lines from snakefmt. Annoying that codecov can't figure out that that doesn't technically mean coverage has decreased, especially considering the diff coverage is 100%.

@mbhall88 happy to merge this in and then could you try this branch on your workflows especially to check #67 .
If happy we can then PR to master and make new release, I will need to add to CHANGELOG though

@mbhall88 happy to merge this in and then could you try this branch on your workflows especially to check #67 .
If happy we can then PR to master and make new release, I will need to add to CHANGELOG though

Sure. Feel free to push your changelog additions directly to dev once this PR is merged :+1: then I'll pull dev locally tomorrow and start testing it


Fix unique key issue in WizzardStepper

Fixes a web console complaint about missing unique keys in the WizzardSteppers. I deem using .filter() a nicer solution than a ternary operator that checks if the hidden property of the current label object is false and rendering an empty fragment. This is probably also what made the console unhappy.

Rebased.

Can't reproduce the web console warning anymore, so we can probably close this. @ziegenberg, what say you?

Yeah, it was fixed in another way. So, closing it is.


Please add a config to weight eggs in chests

Hello Barracuda!
Can you please add a config to weight the eggs in the chests?
When we use other plugins that modify chests and even spawn structures that contain generic chests  (like the ruins mod among others) the eggs get totally out of hand. Like 10+ eggs per hour while exploring!
An option to adjust this per config would be very welcome! =)

Yeah. This would be extremely handy. Or maybe add some sort of generated structure "dragon nest" with one or two adult wild dragons and some dragon eggs.


Fixed typo in decodeUriComponent example

Hello - while working on unit tests for my expression language emulator I realized that the worked example for decodeUriComponent is missing an S in the http

@nonodename : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Just noticed same issue is in https://docs.microsoft.com/en-us/azure/logic-apps/workflow-definition-language-functions-reference#encodeuricomponent as well

#sign-off

@nonodename, all instances where this typo happens should be fixed after today. Thanks!


Album art, pause, track title & progress

So this might be old news, but I've also seen that it was fixed so I'm not sure.  I just installed tonight for the first time.  After getting all the files finally unblocked, I was able to start the toolbar and add Spotify as the source.  Went into the audio band options and pasted in the client numbers from the dashboard.  I put the local url in the "redirect url" in the Spotify dashboard.  When I did, I got this in a new browser window:
SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data
Additionally, I'm not getting album art, track title, progress bar, or pause.  The previous, next, and play work though.  I've attached a screen shot and a log file.  Any help would be appreciated.  I've been looking for something like this for years.

AudioBand.log

Normally, the browser window should open with a Spotify log-in page. There's an error somewhere. Are you adding http://localhost to the "redirect url" field? If so, could you try http://localhost:8000? For me, everything is working as it should:

Thanks for getting back to me.  I went into the dashboard, removed the old link, pasted in the one you wrote.  I rebooted and this message came up when I logged into windows:
INVALID_CLIENT: Invalid redirect URI
Additionally, the buttons are no longer functioning.  I'll add the local host back (leaving both in) and will reboot to see what happens.

Another update.  Completely removed the files and deleted the entry in the dashboard.  Reinstalled and set up a new app in the dashboard.  After rebooting, got the browser prompt to log in.  Did so, and the screen refreshed to the JSON error again.

I've seen a JSON parse error before but not inside a browser window. I'll investigate.

The json parse error comes from spotify sending html in their 5xx errors instead of json. Not much that can be done other than adding a delay and re-sending the request.


Provide version number

When updates change core functionality (e.g. 6.1 -> 6.2 changed the name of the slider start parameter) it's helpful to be able to know which version of jpsych an experiment is running. But it seems that there is nowhere in the code (even in the comments) that shows the version number. It would be helpful to add this parameter that could be tested for.

I've been meaning to do this. Thanks for the nudge!


Prediction with refinement error

When trying refinement option with celeba small masks model:
[2023-09-03 09:44:32,915][main][CRITICAL] - Prediction failed due to not enough values to unpack (expected 2, got 1):
Traceback (most recent call last):
File "/home/lama/bin/predict.py", line 80, in main
cur_res = refine_predict(batch, model, **predict_config.refiner)
File "/home/lama/saicinpainting/evaluation/refinement.py", line 308, in refine_predict
image_inpainted = _infer(image, mask, forward_front, forward_rears, image_inpainted, orig_shape, devices, ids, n_iters, lr)
File "/home/lama/saicinpainting/evaluation/refinement.py", line 128, in _infer
z1,z2 = forward_front(masked_image)
ValueError: not enough values to unpack (expected 2, got 1)

When trying refinement option with celeba small masks model:
[2023-09-03 09:44:32,915][main][CRITICAL] - Prediction failed due to not enough values to unpack (expected 2, got 1): Traceback (most recent call last): File "/home/lama/bin/predict.py", line 80, in main cur_res = refine_predict(batch, model, **predict_config.refiner) File "/home/lama/saicinpainting/evaluation/refinement.py", line 308, in refine_predict image_inpainted = _infer(image, mask, forward_front, forward_rears, image_inpainted, orig_shape, devices, ids, n_iters, lr) File "/home/lama/saicinpainting/evaluation/refinement.py", line 128, in _infer z1,z2 = forward_front(masked_image) ValueError: not enough values to unpack (expected 2, got 1)

Have you solved this porblem?

Thank you for your work, can you please provide the pre-trained model weights for the lama-refine

@MacM2Pro @leayz-888 Hello, looking into the problem, should be an easy fix.
@mldemox the weights for the lama-refine models are same as the original models. The refinement step optimizes the image during runtime, it doesn't use any separate weights.


Constant name "ldap_password" doesn't conform to UPPER_CASE naming style

https://github.com/lbonanomi/scripts/blob/d4369094fb277f5c08521500b3186b83b4c530b9/active_directory_last_password_reset.py#L22

Issues for naming style are now ignored.


Update vendored assets from RAML 1.0 feature branch of api-console

@matt-wratt @gingermusketeer would be great if you could have a look at this though its not ready to merge as I'm getting some wierd js errors locally from attempting to include files dynamically which don't exist.. think I've done something wrong or the lastest version that i've updated to (via the bin/update script) isn't compatible somehow.. the error i'm seeing in the rails logs:
[faa4ac40-7df8-4c26-83bc-692837a710a0] [No Consumer]
ActionController::RoutingError (No route matches [GET] "/assets/raml_console/map.js.map"):
  actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/debug_exceptions.rb:21:in `call'
  newrelic_rpm (<IP_ADDRESS>1) lib/new_relic/agent/instrumentation/middleware_tracing.rb:79:in `call'
  actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
  newrelic_rpm (<IP_ADDRESS>1) lib/new_relic/agent/instrumentation/middleware_tracing.rb:79:in `call'

so its trying to load: map.js.map  which the only reference to which is deep within api-console-vendor.js line: 361565

	'use strict';
	var Type = __webpack_require__(188);
	module.exports = new Type('tag:yaml.org,2002:map', {
	    kind: 'mapping',
	    construct: function (data) { return null !== data ? data : {}; }
	});
	//# sourceMappingURL=map.js.map

Any help much appreciated!

@danielocallaghan My suspicion is that the map needs to include a hash. What do you get when you precompile the assets?

@gingermusketeer hmm precompiling the assets... thats a novel idea indeed!
will try that and see how I go cheers.

hey @gingermusketeer so for 'precompiling the assets' how would I actually go about trying to do that? (thought I knew but think i am off track...)

from what I can tell it looks like the dist/ folder should contain precompiled assets but looks like api-console-vendor.js has a bunch of references to things like:
		/// <reference path="../typings/main.d.ts" />
		var invariant = __webpack_require__(13);


Docker

Before merging this pr, you need to create a docker hub account, and an access token.
The DOCKERHUB_USERNAME secret should be your docker username
The DOCKERHUB_TOKEN secret should be your docker access token
I assumed in this pr, that your username would be MikuAuahDark, and your repo name NPPS4. If this is not going to be true, you will need to update the docker workflow with the correct information
This pr will build arm64 and amd64 images

@MikuAuahDark
I've modified the workflow to push images to github, as well as the example to pull from github. This requires no additional configuration before merging

Looks nice, good job :)


Making work with this program...

I know this is not a real issue, more a plead for assistance:)
Trying to use https://github.com/ssokurenko/jquery-mobile-password-generator in a classroom project and need a copy to clipboard function.
Have tried for days to get clipboard.js working with above but failed over and over.
Anyone have time to look at and see if clipboard.js will work with above?
I and students would appreciate more than you know:)
Cheers!

Hey @2braincells2go, we provide a lot of examples under the demo folder so you can try and adapt them to your needs: https://github.com/zenorocha/clipboard.js/tree/master/demo
Good luck with your project!


fix cabgen exclusions

Docs Build status updates of commit 45f2b93:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/mapping/cabgenConfig.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Allow outputting logs of updated / deleted keys with Azure App Configuration Push task when Dry Run is false

Currently the Azure App Configuration Push task outputs a list of which keys would have been updated / deleted if Dry Run is set to true. However, when Dry Run is set to false it only outputs how many records have been updated. Would it be possible to get an option to output a list of the keys that were updated / deleted when Dry Run is set to false?

@zhenlan My thoughts are that it would be useful to see a list of the keys that were added, updated and deleted as part of the push to ensure nothing unexpected happened. This output doesn't have to include the values of each key, it could just include the keys themselves similar to the output when dry run is true. This would avoid the issue of exposing sensitives values.
This can currently almost be achieved by running the task twice. The first time doing a "dry run" to display the summary of what will be done and the second time doing an actual push. However, there is the possibility that the app configuration changes between the two and thus the "dry run" doesn't represent the result of the actual push. This possibility increases if there is additional time between the two so it would be good to be able see the result of the actual push in more detail to see more easily if the push did something unexpected.

@brental, thanks for sharing your thoughts.
@MaryanneNjeri, any thoughts on how users can enable verbose logs?

@zhenaln am thinking we can support an optional input with the default as false, but when users opt-in, keys that have been updated are logged.

@MaryanneNjeri does the Pipeline task have any built-in properties for log verbosity?

@zhenlan the pipeline task does have the option to configure logs to be more verbose by setting system.debug to true, but since we don't log anything when dry run is set to false, updated/deleted keys will not be outputted

@brental would it be a viable solution for you if we log keys when you set system. Debug to true?

@zhenlan Sorry for the lack of reply. Using System.Debug could work, but are there any other common approaches used in tasks to enable more verbose logs? My concern with System.Debug is that it will also increase logs for all tasks in the pipeline which might not be desirable for every run. Unless there is a way to set System.Debug for a single task?

@zhenlan I am just following up on this to see if it is still an option. I feel like the opt in parameter that @MaryanneNjeri mentioned is probably a better option then using System.Debug (see my comment above).

@brental sorry for my late response. What you said makes sense to me. Since this is only about logging the updated keys (like dry run), the verbose log may not be a good name for it (too generic). We will figure out something else.


Order of configured global options should not be enforced

Context
I'm re-using a title that has been used in #895 and a dedicated feature to address this issue as discussed in #649 . Also, there is a SO post related. However, the available solutions do not work for me.
The problem
The order of the specified global options is important. I'm unsure if it is because how the library is used or there is a problem within the library, but will describe what I'm trying to do.
Given a top command, one sub-command and three options in the following order:

topCommand option1 option2 sub-command option3

option1 and option2  are ignored and only option 3 is considered.
However, all options are considered if this is changed to:

topCommand sub-command option1 option2 option3

How the code looks like at the moment (which is in Kotlin):
@Command(mixinStandardHelpOptions = true,...)
abstract class Base {

    @Option(names = ["--logging-level"],
        completionCandidates = LoggingLevelConverter.LoggingLevels::class,
        converter = [LoggingLevelConverter::class])
    var loggingLevel: Level = Level.INFO

    @Option(names = ["-v", "--verbose", "--log-to-console"])
    var verbose: Boolean = false

    open fun initLoggin() {

    }

    override fun call() {

    }
}

abstract class WrapperBase: Base {

      // These will be programmatically added here
      val subCommands

    override fun call() {

    }
}

class SubCommand:  WrapperBase {
     // More options specific to registration
}

// This is the top command
class TopCommand: WrapperBase {

     @Mixin
     val cmdLineOptions

     // This is where the subcommands are initialised and added to subCommands
     val mySubCommand = SubCommand()
}

@palade Thank you for raising this!
I want to try to reproduce the issue, so we can fix it.
The code you provided gives some hints at what you are trying to do, but I had some trouble getting it to compile. I made some changes, and it compiles now, but it is still incomplete and does not run yet.
My version is here. Can you help me make the minimal changes to get this to run so it demonstrates the issue?

package picocli.examples.kotlin.globaloptions1105

import picocli.CommandLine
import picocli.CommandLine.*
import java.util.concurrent.Callable
import java.util.logging.Level
import kotlin.system.exitProcess

/**
 * This class is not an example (yet)
 * but is work in progress to resolve this issue:
 * https://github.com/remkop/picocli/issues/1105
 */
@Command(mixinStandardHelpOptions = true)
abstract class Base(val alias: String, val description: String): Callable<Any> {

    lateinit var args: Array<String>

    @Option(names = ["-v", "--verbose", "--log-to-console"],
            description = ["If set, prints logging to the console as well as to a file."]
    )
    var verbose: Boolean = false

    @Option(names = ["--logging-level"],
            completionCandidates = LoggingLevelConverter.LoggingLevels::class,
            description = ["Enable logging at this level and higher. Possible values: \${COMPLETION-CANDIDATES}"],
            converter = [LoggingLevelConverter::class]
    )
    var loggingLevel: Level = Level.INFO

    open fun initLogging() {
        println("$this is initializing logging with verbose=$verbose and loggingLevel=$loggingLevel")
    }

    override fun call() {
        println("Hello from $this; verbose=$verbose and loggingLevel=$loggingLevel")
    }
}

abstract class WrapperBase(alias: String, description: String): Base(alias, description) {

    // These will be programmatically added here
    val cmd by lazy {
        CommandLine(this).apply {
            commandSpec.name(alias)
            commandSpec.usageMessage().description(description)
            subCommands.forEach {
                val subCommand = CommandLine(it)
                it.args = args
                subCommand.commandSpec.usageMessage().description(it.description)
                commandSpec.addSubcommand(it.alias, subCommand)
            }
        }
    }

    override fun call() {
        println("Hello from $this; verbose=$verbose and loggingLevel=$loggingLevel")
    }

    val subCommands: Set<Base> by lazy {
        additionalSubCommands()
    }

    protected open fun additionalSubCommands(): Set<Base> = emptySet()
}

class SubCommand: WrapperBase("sub-command", "does some sub-command work")

class TopCommand: WrapperBase("top-command", "does top-command work") {

    private val subCommand by lazy { SubCommand() }

    override fun additionalSubCommands() = setOf(subCommand)

    override fun call() {
        super.call()
    }
}

fun WrapperBase.start(args: Array<String>) {

    this.args = args

    val exceptionHandler = object : DefaultExceptionHandler<List<Any>>() {

        override fun handleParseException(ex: ParameterException?, args: Array<out String>?): List<Any> {
            super.handleParseException(ex, args)
            return listOf()
        }
        override fun handleExecutionException(ex: ExecutionException, parseResult: ParseResult?): List<Any> {

            val throwable = ex.cause ?: ex
            if<EMAIL_ADDRESS>||<EMAIL_ADDRESS>{ it.verbose }) {
                throwable.printStackTrace()
            }
            return listOf()
        }
    }

    val results = cmd.parseWithHandlers(RunLast().useOut(System.out),
            exceptionHandler.useErr(System.err), *args)

    // If an error code has been returned, use this and exit
    results?.firstOrNull()?.let {
        if (it is Int) {
            exitProcess(it)
        } else {
            exitProcess(0)
        }
    }

    exitProcess(1)
}

fun main(args: Array<String>) {
    val cli = TopCommand()

    // This does change the logging-level
    // The logging level is correctly set
    //cli.start(arrayOf("sub-command", "--logging-level=FINEST", "--log-to-console"))

    // This does NOT change the logging-level
    // The logging-level should be FINEST after this but it shows as INFO
    cli.start(arrayOf("--logging-level=FINEST", "--log-to-console", "sub-command"))
}

class LoggingLevelConverter: ITypeConverter<Level> {
    object LoggingLevels: Iterable<String> {
        override fun iterator(): Iterator<String> {
            return listOf(Level.ALL.name,
                    Level.CONFIG.name,
                    Level.FINE.name,
                    Level.FINER.name,
                    Level.FINEST.name,
                    Level.INFO.name,
                    Level.OFF.name,
                    Level.SEVERE.name,
                    Level.WARNING.name).iterator()
        }
    }

    override fun convert(value: String?): Level {
        return Level.parse(value)
    }
}

If the main is replaced with one below:
fun main(args: Array<String>) {
    val cli = TopCommand()

    // This does change the logging-level
    // The logging level is correctly set
    //cli.start(arrayOf("sub-command", "--logging-level=FINEST", "--log-to-console"))

    // This does NOT change the logging-level
    // The logging-level should be FINEST after this but it shows as INFO
    //cli.start(arrayOf("--logging-level=FINEST", "--log-to-console", "sub-command"))

    cli.start(arrayOf("--help"))

    val stream = ByteArrayOutputStream()
    val ps = PrintStream(stream)
    CommandLine.usage(cli, ps)

    println(stream.toString())
}

The usage message shows like:
Usage: top-command [-hvV] [--logging-level=<loggingLevel>] [COMMAND]

This can be confusing because it shows that the global options and COMMAND can be placed in any order

@palade Great, thank you! I will take a look. It may take me some time, I am not very fluent in Kotlin (so this is a nice learning opportunity for me :-) ...

@remkop Cheers for all the help. Integrated your suggested changes and global options can now be specified in any order

You are very welcome!
Enjoy using picocli!


Fix no sep

Coverage increased (+0.2%) to 93.407% when pulling e032846b717250f7969588eac72d448a47cd7042 on fix_no_sep into 0454c7786816d884d4c12f647ab3d3b605e4756d on master.


Feature Requests

--When flipping through tabs (ex. Delegate information), the back button switches only between tabs. It does not take me to the delegate selection screen. Can get annoying if I've moved around a bit.
-- At a certain point in the window size, objects on the task bar begin to overlap disruptively. Maybe add a minimum window size ?
-- Can only filter Delegates by name, cannot filter delegates by approval rating, or Pay rate.
-- It would be nice to be able to sort Categories on the Delegate Screen

--Blocks Produced added as delegate category
-- There is no place to view people who you have added as contacts (if there is it is not easy to find)

Moved: https://github.com/BitShares/web_wallet/issues/555


Removed unsupported apply_server_side_encryption

Not sure if you're open to pull requests on this but I picked up a bit of a bug in the code.
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket#enable-default-server-side-encryption
https://github.com/hashicorp/terraform-provider-aws/issues/23106
This is read-only and so the terraform apply fails as it's unable to use this feature.
Removing this code so that any other noobs like myself going through the project/YouTube series don't get tripped up on this.

Heyy THANK YOU so much for the catching this!!  And you're right, this property has been changed around so that, in order to add it we have to use a separate resource, the aws_s3_bucket_server_side_encryption_configuration resource.
  Would you be interested in adding the new resource to this PR @conormccullough1 ?  It'd be the equivalent of adding the following to the s3.tf file:
# S3

## S3 Bucket for Vault Data
resource "aws_s3_bucket" "vault_data" {
  bucket_prefix = "${var.main_project_tag}-"

  tags = merge({ "Project" = var.main_project_tag })
}

# THIS would be the NEW resource
resource "aws_s3_bucket_server_side_encryption_configuration" "vault_data_sse" {
  bucket = aws_s3_bucket.vault_data.bucket

  rule {
    apply_server_side_encryption_by_default {
      sse_algorithm     = "AES256"
    }
  }
}

## S3 Bucket Public Access Block
resource "aws_s3_bucket_public_access_block" "vault_data" {
  bucket = aws_s3_bucket.vault_data.id
  block_public_acls = true
  block_public_policy = true
  ignore_public_acls = true
  restrict_public_buckets = true
}

I just need someone to give it a run through and test.  It'd be nice to keep this all in your PR, but if not, I appreciate you bringing this to my attention either way!!!

Hey, just trying to test this now, but for some reason after a terraform destroy / re-apply, although Terraform creates the S3 bucket, it's now failing to place the encrypted credentials in there. Tried both with the new code block and without, so I guess it's something else that's been created along the way and is blocking it.

Interesting (also, thanks for trying it).  What's the full error message it's giving?

Hi @jcolemorrison - committed the change + left an explanation of the previous behaviour. Cheers for the feedback.

Hey awesome @conormccullough1 and thanks for the detailed explanation and code!  Everything looks good so I'll go ahead and merge it in.
As far as the first hiccup of it not working, I have a feeling there's a race condition going on somewhere in which some resources are being created faster than others that depend upon them.  It might be the case where a depends_on argument is needed somewhere.  My bet is currently the Auto Scaling Groups creating the instances in the private subnet and attempting to install packages before the NAT gateway is set up..but that's a convo and PR for a different time.  For now, thanks a ton for your contribution!


Trouble starting up

Hi all,
On my Raspberry Pi I run the new Raspbian Jessie (with PIXEL). I run it headless through ssh, so I just run multi-user, not graphical.
Now I just downloaded the pigpio package using the recipe at this page. So I did:
cd
wget abyz.co.uk/rpi/pigpio/pigpio.tar
tar xf pigpio.tar
cd PIGPIO
make -j4
sudo make install

So far, so good...
So now I started pigpiod:
sudo pigpiod
no errors, and I can see that it's running:
pi@Columbus:~/PIGPIO $ ps alx | grep gpio
5     0 17784     1  20   0   9904  1684 -      SLsl ?          0:01 pigpiod
0  1000 17793   884  20   0   4276  1824 pipe_w S+   pts/0      0:00 grep --color=auto gpio
pi@Columbus:~/PIGPIO $

Now for the big test:
pi@Columbus:~/PIGPIO $ python
Python 2.7.9 (default, Sep 17 2016, 20:26:04)
[GCC 4.9.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import pigpio
>>> pigpio.start()
**Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'module' object has no attribute 'start'**
>>> exit()
pi@Columbus:~/PIGPIO $

Also this test fails:
pi@Columbus:~/PIGPIO $ sudo ./x_pigpio # check C I/F
2016-12-28 14:14:21 initInitialise: Can't lock /var/run/pigpio.pid
pigpio initialisation failed.
pi@Columbus:~/PIGPIO $

Unfortunately, as you can see it gives me errors.
Any clues?
Gerard

The pigpio module start() function was deprecated several years ago.
See issue #84.

Seems that I was looking at an old page on the Raspberry Pi site.
Sorry for your trouble!
Gerard


Handle :attribute/_reverse in transactor. Fixes #187.

See commit comments.  I've intentionally not accepted reverse notation with nested vectors and maps; it's difficult to handle with our current parse-then-apply model.  I think it's a reasonable trade-off, and the error message is clear.

Is it possible to additionally handle the simple cases — perhaps just [:db/add a b c] — in the parser?
My reasoning is that we can reject obviously invalid input, like [:db/add 123 :foo/_bar 9.95], much earlier in the process, just by looking at the bare EDN — without reference to the schema, without parsing the rest of the input, and before taking locks, hopping into queues, etc.

I did the entire thing in the parsing layer, and decided against it.  The hard thing is that we split the tempid space into internal (allocated by Mentat) and external (strings allocated by the user), and there's no easy way to represent internal tempids in the edn::Value space.  That adds a lot of book-keeping, roughly equivalent to the dynamic approach but bulkier, which I didn't like.  And the parsing layer produces worse errors than the dynamic transacting layer (paradoxically).
Anyway, I cannibalized by original parsing approach just for the simple direct notation, as you say.  Sadly, it's hard to get the error messages to be informative... but it's definitely more efficient.  See what you think about the last commit.


Paragraph containing "Guest" is returning from Japanese to English

After #1426, paragraph containing "Guest" is returning from Japanese to English
https://docs.microsoft.com/ja-jp/microsoftteams/set-up-guests

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 042d7ce6-ee22-690f-8188-ff8a4a585847
Version Independent ID: 3e5bc99c-e08e-44e0-cda3-687ed3633175
Content: Microsoft Teams へのゲスト アクセスをオンまたはオフにする
Content Source: Teams/set-up-guests.md
Service: msteams
GitHub Login: @somakbhattacharyya
Microsoft Alias: sbhatta

@OfficeGlobal -Thank you for submitting this feedback.
I will get this over to for investigation. Thank you for reporting and making the docs better. Much appreciated.

@samanthaw  Working on a fix (Bug 3233070)

@OfficeGlobal Any updates on this issue? Thanks!

@officedocsbot assign @OfficeGlobal

@hyoshioka0128 Article has been HumanTranslated and published live. The issue is fixed

@OfficeGlobal Thanks.
Close #1743


Pressing enter while navigating filter-list just makes a new line

It's impossible to select an element in the filter list with enter.
The keymap seems to be good:       [:filter-list.input "enter" :filter-list.input.select!].
I'm having this issue since I updated to 0.7.

It took me a while to figure this out myself so I'm adding this:
You can use TAB to select an item from the autocomplete list, ENTER will make a new line by default.


lib/sciond: expose metrics

Number of requests by type
Result (including "empty") by request type
Connection/reconnection stats

The SCIOND client API doesn't have reconnecting logic; it establishes one connection for each RPC.
Also, the client API library never looks within the reply payloads at the moment, so it's a bit strange to look there just to add a metric for empty paths. We should definitely track these events, but it makes more sense to track them at the RPC call site; then we also know which module is failing to find paths.


assert failed when use pread

https://s3.amazonaws.com/clickhouse-test-reports/69862/331872a7e496cdd9d6420f989eb73d5aa5a1afee/stateless_tests__debug__s3_storage_/stderr.log
assertion will failed, when use pread

https://github.com/ClickHouse/ClickHouse/pull/69862


Fedi bridge to Bluesky not taking up new domain name

I attempted to follow the documentation to set my bridge Mastodon account on Bluesky to my own domain, but the bridge does not appear to up updating my handle.

I created a DNS record for my desired handle:

I checked it in the debug page:

I sent a DM to the bridgy account from Mastodon:

Now, I can reached my bridged account on Bluesky by including the domain in the URL, but the account handle itself is still the long Bridgy name:

Should the Bridgy account bot respond when I send it the command? It responded when I ran the did command.

Sorry! We got badly backed up last night, #1520. We've almost dug ourselves out of the backlog, you should get a response when we're caught up.

Same issue again now but with<EMAIL_ADDRESS>and a number of other ones in the same domain. All the verifications mentioned above are green. I send a message, but no update. The bridge seems to be working though.

Sorry for the trouble @yurukov! I'm seeing a timeout from your DNS server on GovAlertEu.m.govalert.eu: https://bsky-debug.app/handle?handle=GovAlertEu.m.govalert.eu

That's strange. It works fine here and it actually worked to transfer the account. The issue turned out to be that the first word of the DM should literally be "username" followed by "govalerteu.m.govalert.eu". Resolved it here https://github.com/snarfed/bridgy-fed/issues/1616


podman pod top: incomplete output

I know this topic has been discussed to exhaustion. I still believe there is a problem that needs addressing: pod top is sometimes producing incomplete output on  low-cpu systems.
#!/bin/bash

wait_for_output() {
    while :;do
        x=$(podman logs $1)
        if expr "$x" : ".*PPID" >/dev/null; then return; fi
        sleep 0.1
    done
}

set -x
podman pod create --infra=false --name p

podman run -d --pod p --name c1 alpine top -d 2
podman run -d --pod p --name c2 alpine top -d 2

wait_for_output c1
wait_for_output c2

# Most of the time, this (correctly) shows 1 header and 2 process lines.
# About one time in five, on a 1-CPU system, this produces only one process
podman pod top p

podman rm -f c1 c2
podman pod rm p

Consistently seen and tested up to master @ c600014643fec02b5d48f092b90cc8a511c74eff
Downloadable test script (same as above): pod-top.sh

@baude We never parallelized top, right?

@edsantiago do you believe this is still an issue? Worth Fixing? Or can I close?

@rhatdan I can't reproduce any more - loop script ran for an hour without triggering. So I'm closing. Thanks.


Chrome chrashes and doesn't load other tabs

Describe the bug
When I visit https://stackblitz.com/edit/github-8qeupk?file=package.json,
the terminal says it's Fetching dependencies but gets stuck. Maybe infinite loop?
It doesn't seem I'm allowed to visit the same blitz in a new tab either
Link to the blitz that caused the error
https://stackblitz.com/edit/github-8qeupk?file=package.json
Steps to reproduce
visit: https://stackblitz.com/edit/github-8qeupk?file=package.json
observer frozen browser
Expected behavior
webapp's devserver boots
Screenshots
No response
Platform

OS: Ubuntu
Browser: Chrome
Version: 101.0.4951.41

Additional context
No response

Hey there! If I open your project it works fine for me and I can't seem to reproduce it. Any errors in the console?

I'm not using the same setup atm, but on Chrome canary I got an error.
ReferenceError: Object is not defined
    at Function.build (/home/projects/github-8qeupk/node_modules/console-ui/node_modules/chalk/index.js:129:2)
    at Function.get (/home/projects/github-8qeupk/node_modules/console-ui/node_modules/chalk/index.js:61:17)
    at writeError (/home/projects/github-8qeupk/node_modules/console-ui/lib/write-error.js:101:24)
    ```

Why wouldn't Object be defined for chalk?

It's not chalk, and that error is basically a duplicate of #673 which is some weird issue with the vm module and the esm package. I have looked briefly into it and saw that for some reason the global was modified which results in that strange error you mentioned. We have to take a closer look and see how we can resolve this issue.


fix: need default value for getSizeAsMb(EXECUTOR_MEMORY.key)

Which issue does this PR close?
Closes #1045 .
Rationale for this change
Provide a default value 1g to getSizeAsMb(EXECUTOR_MEMORY.key), which is the default value of spark.executor.memory in Apache Spark 3.5.3.
What changes are included in this PR?

Define private val EXECUTOR_MEMORY_DEFAULT in the class
Call getSizeAsMb(EXECUTOR_MEMORY.key) with EXECUTOR_MEMORY_DEFAULT

How are these changes tested?
Tested with the datafusion-benchmarks as described in the issue #1045  .

Thanks @neyama @andygrove


Use ~ or ^ instead of specific version

Currently, a specific version is used (4.17.11) but it should use ~ or ^ so that when users update they don't have to have multiple versions installed.

A specific version is still being used. Can a ~ or ^ be used so it's not locked to a specific version?

it's "lodash": "^4.17.15"
https://github.com/wix/eslint-plugin-lodash/blob/master/package.json#L32

Oh sorry, I just looked at the referenced PR 🤦♂

no problem :)


Python logging

replace print statements with python
config file logging.yaml in configs dir (or override)

Coverage decreased (-0.3%) to 97.875% when pulling 9cee9feff98e413cb98b866ea3fd216b6774210d on python-logging into eb8fdde7c24b78d973cf0ad2f9101a31e17b6b2f on master.


Palette Selector in demo app [SBK-379]

Let users pick between different color themes with a simple click on the demo app, to see how the components can be rendered differently.
Eg. have a dark theme

Moving away from using React


Valid padding in CapNets

Hello sir,
I am following the Capsule Network paper and your implementation.
I have a quick question about the valid padding in the conv2 you used to get output for the Primary Caps. So as I understand, after the 1st conv layer, the size of output is (batchsize,20,20,256). So if the conv2 has 256, 9x9 kernel, stride 2 then the formula output should be (20-9+2*p)/2+1 = 6. However, mathematically, the formula above can not be solved so I would like to ask how did exactly padding (valid) works in this situation to have the output is (batchsize,6,6,256).
Thanks !

Hi Phong,
After the first layer, indeed there are 256 20x20 maps, because 28-9+1=20. After the second layer, there are 256 6x6 maps, because 20-9+1=12, and stride 2 means we divide 12 by 2, which gives 6.
I hope this helps,
Aurélien

Thanks Aurélien,
I kind a understand it now. before i am just following the formula given by CS231 course so I dont understand it.
Thanks for your reply then :D


Add APN settings configuration for 4G

To make it easier to configure 4g, add APN settings configuration

Some more documentation is available here. I don't think we can write to this path from within the container, but I venture to guess that we can communicate with NetworkManger over DBUS instead.

https://www.balena.io/docs/reference/OS/network/2.x/#cellular-modem-setup

To set up a cellular connection with your device, just drop the below example configuration into a file in the /resin-boot/system-connections/ directory in the .img or on SD card (if you have it flashed with the OS already) and name it something like cellular. Replace the apn= and number=

[connection]
id=cellular
type=gsm
autoconnect=true

[gsm]
apn=giffgaff.com
number=*99#
password=password
username=giffgaff

[serial]
baud=115200

[ipv4]
method=auto

[ipv6]
addr-gen-mode=stable-privacy
method=auto

Also see https://www.balena.io/docs/reference/OS/network/2.x/#changing-the-network-at-runtime

Duplicate of #554