openldap_module undefined local variable

There may be a typo in the openldap_module provider:
Error: Failed to apply catalog: undefined local variable or method `expection' for Puppet::Type::Openldap_module::ProviderOlc:Class
Should this be 'Exception' instead of 'expection'? (on line 24 of openldap_module/olc.rb)

@djtaylor thanks

@djtaylor thanks


FWRZNETxxx Modelling search interface - table "Owner" filter does not display fully

when there are very few entries, the height of the filter pop-up is too small (scrolling necessary)
also - just for the Owner filter, the input field is not displayed - for other fields this is not the case

@tpurschke Additional: This Table/Component doesn't include the PageSizeComponent. Should i add this while i'm working on the issue?

@tpurschke Additional: This Table doesn't include the PageSizeComponent + Pager Component. Should i add this while i'm working on the issue?

yes, sure


Make SSH_AGENT_PID mapping to Windows PID

Port of https://github.com/dahlbyk/posh-git/pull/699.  This does drop -ErrorAction Ignore as unnecessary.  It further adds [0] to full name parent directory for bin / usr lookup of ssh location as the lookup of 'ps' and and will get multiple (2) results.  This ensures it only uses the first in that situation.

ping @dahlbyk


Document that it's not active in debug mode

I was pulling my hair out trying to get this working until I saw "if app.debug " in the source code and switched off debugging. I think this should be added to the README.md,

I agree. I will amend the documentation tomorrow. Thank you for letting me know!

no problems, thanks for making this!

I re-added the debug documentation that got stripped out a few commits back. I should have caught this during review.
It probably makes more sense to have the documentation purely in README.md at this point. I will look into porting over.

Is there a good reason to turn it off during debug? I observe that this confused me, and separately another dev on my team too.

(and docs probably wouldn't have helped the existence of the confusion much)

The original idea was that during development you weren't forced to look at the compressed javascript, css, etc. With browsers automatically formatting those files even if they are compressed, this probably isn't a concern. Since both of you got tripped up by it, it is probably a good idea to remove it. If someone wants to temporarily disable the compression all they have to do it comment out one line in their app. What do you think?

Yes, that would be good.
I don't understand though, why would a developer ever find themselves looking at compressed javascript? All of the tools I use (curl, browsers, etc) show the resources after decompression..

It was purely out of flexibility. I agree is a dated belief with modern browsers. I will pull that code out while I'm porting to README.md.

I went ahead and made the change. If you are using trunk you will no longer need to have app.debug = false. I'll put out an official release once I port over the documentation.

I removed the app.debug = false requirement, updated the Sphinx documentation and moved most of it into README.md for easier viewing in Github. Are there any other improvements you all would like to see before I spin a new release?

None that spring to mind for me.

Thanks for maintaining this useful little library :)

My pleasure. I'm just happy people find it useful.


Shortcut

I dati necessari sono stati scritti negli appunti perché erano eccessivi per l'invio. Incollarli.

Google translate: The necessary data was written to the clipboard because it was too much to send. Glue them.

Seems that this user forgot to paste the needed data.


login form functionality

This is registration form

Registration will now be with Email only

90% complete - 05/08


feat(file-helpers): add revokePreviewUrls and getFileExtensionsFromTypes function

Short Summary
Screenshots

these methods are not candidates as general helpers


"error trying to connect: invalid dnsname" on some websites

On some websites, e.g. http://juraganfilm.io, reqwest fails with the following error:

error trying to connect: invalid dnsname

Firefox and curl work fine.
44 websites out of the top million from Feb 3 Tranco list are affected.
Tested using this code. Test tool output for all occurrences: reqwest-invalid-dnsname.tar.gz

When I try the simple reqwest example with that domain, it worked fine. Seems like a temporary DNS failure...


feat(cdk/Overlay): Allow using string values in cdkConnectedOverlayOffsetX/Y

Feature Description
Right now CdkConnectedOverlay.cdkConnectedOverlayOffsetX/Y only accepts numbers.
Please allow it to accept string values, like 2rem, for example.
Use Case
In one of my apps, it is the only part that doesn't scale when html.font-size is modified (or font size is changed in browser's settings).

The reason it only accepts numbers is because we're using the value in pixel-based calculations.

Is it too much to ask for something like this?
let vPx = 0;
if (typeof v === 'string') {
  if (!v.endsWith('rem')) {
    vPx = parseFloat(v);
  } else {
    const rem = parseFloat(getComputedStyle(document.documentElement).fontSize);
    vPx = rem * parseFloat(v);
  }
}

I would be hesitant to support something like that, because some components authors may start expecting to also respond to dynamic changes, e.g. a media query that sets the font-size on the root node to a different value.

That's fair, I understand that.
It would be easier to implement on my end :)
But still, I hope you move away from pixels some day 😎


Image zoom issue in edit mode for linked images

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
7.7.6
Describe the bug
I encounter an issue with images that are added via links in my notes. In edit mode, these images automatically enlarge and stick to the right side of the note, making typing and editing very difficult. This happens consistently whenever I add an image link and switch to edit mode. This issue only occurs when the image zoom settings are enabled in the theme. The image becomes so large that a horizontal scrollbar appears at the bottom.
Steps to reproduce:

Enable the image zoom settings in the Minimal theme.
Add an image link to a note in Obsidian. → ![](https://stephango.com/assets/kepano-wilderness.jpg)
Switch to edit mode.
Observe that the image enlarges and adheres to the right side, obstructing the text.

Expected behavior: Images should not automatically enlarge in edit mode and should remain at their normal size to avoid interfering with the note's text.
Screenshots:

Debug info
SYSTEM INFO:
Obsidian version: v1.6.5
Installer version: v1.6.3
Operating system: Windows 10 Pro 10.0.19045
Login status: not logged in
Insider build toggle: off
Live preview: on
Base theme: light
Community theme: Minimal v7.7.6
Snippets enabled: 0
Restricted mode: off
Plugins installed: 2
Plugins enabled: 2
1: Style Settings v1.0.8
2: Minimal Theme Settings v8.0.2

thank you


Upgrade ws library

Breaking changes in 3.0.0 were:
    Removed the upgradeReq property (#1099).
    Removed unnecessary events (#1100).
    Removed the flags argument from the 'message', 'ping', and 'pong'
    events (#1101).
    The permessage-deflate extension is now disabled by default on the server
    (#1107).

Only flag argument change affects botkit, but flags argument was unused.

@benbrown I made pull request to master branch because lib/Slackbot_worker.js uses websockets.


Image.resizeMode is undefined in RN 0.56

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
  React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.6
      CPU: x64 Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
      Memory: 41.49 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 8.9.4 - ~/.nvm/versions/node/v8.9.4/bin/node
      Yarn: 1.6.0 - /usr/local/bin/yarn
      npm: 6.1.0 - ~/.nvm/versions/node/v8.9.4/bin/npm
      Watchman: 4.7.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
      Android SDK:
        Build Tools: 23.0.1, 23.0.2, 25.0.0, 25.0.2, 25.0.3, 26.0.1, 27.0.0, 27.0.3, 28.0.0
        API Levels: 23, 24, 25, 26, 27
    IDEs:
      Android Studio: 3.0 AI-171.4443003
      Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.4.1 => 16.4.1
      react-native: 0.56.0 => 0.56.0
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-git-upgrade: 0.2.7

Description
This is possible regression:  I have the following code working under RN 0.55:
  render() {
    const { state: { params: { imageUrl } } } = this.props.navigation;

    const {
      SCREEN_HEIGHT,
      SCREEN_WIDTH,
    } = this.state;

    return (
      <View style={{ flex: 1 }}>
        <ImagePanZoom
          cropWidth={SCREEN_WIDTH}
          cropHeight={SCREEN_HEIGHT}
          imageWidth={SCREEN_WIDTH}
          imageHeight={SCREEN_WIDTH}
          style={{ paddingBottom: 60 }}
        >
          <Image
            resizeMode={Image.resizeMode.center}
            source={{ uri: imageUrl }}
            style={{ flex: 1, width: SCREEN_WIDTH, marginTop: -50 }}
          />
        </ImagePanZoom>
      </View>
    );
  }

Under RN 0.56 it throws
Unhandled JS Exception: Error: Error: Error: Error: Error: Error: Error: TypeError: TypeError: Cannot read property 'center' of undefined

This error is located at:
    in ImageViewer (at SceneView.js:10)
    in SceneView (at StackViewLayout.js:488)
    in RCTView (at View.js:43)
    in AnimatedComponent (at StackViewCard.js:12)
    in Card (at createPointerEventsContainer.js:28)
    in Container (at StackViewLayout.js:539)
    in RCTView (at View.js:43)
    in RCTView (at View.js:43)

Changing code to
          <Image
            resizeMode="center"
            source={{ uri: imageUrl }}
            style={{ flex: 1, width: SCREEN_WIDTH, marginTop: -50 }}
          />

resolves it.  So in a nutshell, Image.resizeMode appears to be undefined where before it allowed abstraction of the enum values away from my code.

ImageResizeMode is not exposed in 0.56.0 anymore because of the removal of the @providesModule (commit).
As you already mentioned you can simply do a search & replace to change it its string value. If you still want to use "from the lib" you can import like so import ImageResizeMode from 'react-native/Libraries/Image/ImageResizeMode' and use it like ImageResizeMode.cover.


[ENHANCEMENT] add commands for adding sockets and socket extender slots to item in main hand

Describe the solution you'd like
I would like to be able to add empty sockets and empty socket extender slots via a command provided by MythicDrops.
Describe alternatives you've considered

Adding the lore manually via /md lore modify add

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

v7.0.0
GitHub release

Your semantic-release bot :package::rocket:


feat: add support for raw folder permissions

Added support for raw folder permissions (JSON).
If we ever add support for teams, we can implement something more complex.

@NissesSenap yeah, I agree with your suggestion, let's add it in one of the next PRs.


Folder structure

Set up the folder structure so the code is organised in scripts, packages and sub-packages.
Description

[x] Create a scripts folder scripts  for omDownloadInputs.py, omCreateDomainInfo.py,  omPrior.py
[x] Create a package src/open_methane for omGeoJSON.py, omInputs.py, omOutputs.py, omPriorVerify.py, and omUtils.py
[x] The modules to calculate the layers go into src/openmethane_prior/layers:  omAgLulucfWasteEmis.py, omElectricityEmis.py, omFugitiveEmis.py, omGFASEmis.py, omIndustrialStationaryTransportEmis.py, omTermiteEmis.py, omWetlandEmis.py
[x] Generate distribution archives of the package and install the package
[x] Update the import commands in the modules

Checklist
Please confirm that this pull request has done the following:

[x] Tests added (should be covered with existing tests)
[ ] Documentation added (will add when we decided on packaging and ditribution)

Notes
For packaging the code I have used build and setuptools.  poetry would be more consistent with our project development. At the same time it introduces a new tool that every user needs to install.

@crdanielbusch Can you merge the changes from main into this branch. That should resolve the conflicts.

Thanks for the feedback! I'm merging this


Modify avenue_wresnet.yaml file

Hi, I have an error when trying to change the config file after downloaded the wider_resnet38.pth.tar file. Please let me know how to fix this. I am trying to use the test.py file on the avenue pretrained model.

I fixed this by change the path of DATASET. Instead of Avenue, I create a folder name avenue with the structure like author, and copy path of this folder to DATASET in config file.  But it still another issue :))


Feature Request: Report Issues should make it easier to report non-dup issues with version information

SQL Operations Studio Version: sqlops 0.24.1 (af2414efcb485ab5e435d1db693d3f7569c48453, 2017-12-19T02:19:09.998Z)
OS Version: Windows_NT x64 10.0.16299

Click the Report Issues button and you go to the Issues tab in GitHub with a new issue open and the version info filled in. This is good but the first thing you really should do is verify the issue hasn't already been reported. For that you have to click on Issues. Once you've identified that the issue is new you want to create a new issue. If you simply click to create a new issue then the version information about the app is gone. You can go to the Help\About and get the version info but none of that is copyable text.
The current workarounds I see:

While on the new issue page, open the Issues in a new tab so you don't lose the initial issue information.
Close the GitHub tab after verifying the issue isn't a dup and then use Report Issues to open the window again.

This feature comes from upstream vscode so we'd prefer to see this fixed there, and we'll pick-up in a source refresh.


Update to Zenoh 1.0

Zenoh 1.0 has introduced lots of API changes compared to 0.11.0.
In addition to adapting to the new API, the code has been improved with
some error handling.

@lukasmittag and @max-grzanna would you mind taking a look

@lukasmittag would you mind taking another look?


Add baseline instrumentation + bug squashing

It is possible that it now actually works with sockets plugged.

Will need to add callstack propagation


French translation

Provide language specific main screen. I don't think any setup/settings screens need to be translated but main screen should offer translation.
(i can help with French/French Canada translation)

There are already translations for Portuguese, Dutch, and Spanish.   If you would like to contribute, there are a few files that need to be translated.  The translation will update the entire application, not just the main screen.
The first file is the string.xml file, this contains all the text displayed in the UI.  It's in xml format and once translated will be place into the appropriate translation folder:
https://github.com/thanksmister/android-mqtt-alarm-panel/blob/master/app/src/main/res/values/strings.xml
The second part that needs translation is the Google Play Store listing which is in very basic HTML 1.0.  The basic HTML tags must be preserved. There is no file but the text is as follows:
A MQTT Alarm Control Panel for Home Assistant and home automation platforms.

MQTT Alarm Control Panel

Reuse and old Android devices as a home alarm control panel that integrates with Home Assistant or any home automation platform that supports MQTT messaging.

MQTT Alarm Control Panel is an alarm control panel for use with home automation platforms that support MQTT messaging.  The application was originally built for Home Assistant'sMQTT Manual Control Panel. However, the application can be used with any home automation platform that supports the MQTT messaging protocol such as Jeedom, openHab, or Node-Red.

The alarm control panel acts as an interface for your home alarm system and allows for two way communication using MQTT.  You can set the alarm state to away or home, or disarm the alarm using a code.  Your home automation system controls the sensors or automation that triggers the siren or notifies users of possible intrusion.

Features
* Stream video, detect motion, detect faces, and read QR Codes.
* Capture and emailing images when the alarm is disabled.
MQTT commands to remotely control the application (speak text, play audio, display notifications, alerts).
* Device sensor data reporting over MQTT (temperature, light, pressure, battery, etc.).
* Day/Night mode themes based on set start and end times.
* Fingerprint unlock support to disable the alarm. (on supported devices).
* Optional screensaver mode using a digital clock or Imgur images.
* Seven day Weather forecast (requires DarkSky api key).
* Home Automation Platform webpage support for viewing home automation dashboards.

The alarm panel is designed to be used as a dedicated device which runs continuously in full-screen mode. The alarm panel is open source and available on GitHub. There is also a version that runs on a Raspberry Pi using Android Things

For the complete setup instructions and additional information,  please visit the Project Home Page.

Home Assistant
Home Assistant's MQTT Manual Alarm Control Panel
Project's Home Page
Raspberry Pi Version
Support<EMAIL_ADDRESS>
If you are willing to do the translation to French, I will give you contributor credit and assign the ticket to you 👍 .

Ok, I will work on the FR translation next week.

Sorry, I'm late with the FR translation. I got sick and had a big IT issue at work. Half translation is done. I hope to finish it mid next week.

Thanks, take your time it’s no rush

And here is the french translation for the Google Play Store
MQTT alarm panel - Google Play Store description - FR.zip
Hope this help.

Both of you guys are great, thanks for the translation.  @electricJP I had forgot that @jncanches also volunteered to do the translations.  It has been a while and I didn't think you had time to do them.  I will be giving both of you credit for the work.  Thanks!!!

This is Fr

Hello,
I did a quick review of jncanches translation and fixed a few typos.
strings.zip

Hello,
I did a quick review of jncanches translation and fixed a few typos.
strings.zip

Wonderful @electricJP ! Thank you very much :)
Excited about having Control panel in french (more user friendly for my family )
@thanksmister Thank you, your app is very, very good and great !

Wonderful, thanks!

Ok, rolled out a new version with the translations.

@thanksmister Sorry, I noticed some little mistakes in the google play store description.
Here is an updated html.
MQTT alarm panel - Google Play Store description - FR.zip

Updated!

Moreover, seeing app capture on Google Play Store makes me think that main screen may display "ALARME DESACTIVEE" or "ALARME MODE ABSENT" and not "SYSTEME DESACTIVEE" (which contains a grammatical error) and "SYSTEME MODE ABSENT".
I updated the strings.xml if @electricJP  and you @thanksmister are ok with that
strings.zip

Damn, I have not attach the right Google Play store description in my previous comment. Really sorry :(
Here is the really updated one
MQTT alarm panel - Google Play Store description - FR.zip

@jncanches  You can't change the strings at this point, that ship has sailed.  I truncated the wording for the alarm modes because they were too long.  If they are incorrect, then just provide the exact strings text because I have already updated an fixed the string resources to be escaped properly.  I can not replace all strings.

@thanksmister ok I will look at it and see if it can be shorter. Which word key espacially must be brief ?
Have you the re-corrected wording for Google Play Store ?

I am updating Google Play now.   I think it can just be "Arm Away" or "Arm Home" or if needed just "Away" or "Home" for the modes, along with "Disarmed".     I don't know if the documents for the alarm component in HA are in french, but the commands are shown https://www.home-assistant.io/components/alarm_control_panel.manual_mqtt/.

The play store is once again updated.

Right now I have these values:
```

SYSTEME
DESACTIVEE
MODE ABSENT
ACTIVER MODE PRESENT
ACTIVER MODE ABSENT
MODE PRESENT
MODE ABSENT

I think you can use :
ALARME
DESACTIVEE
MODE ABSENT
ACTIVER PRESENT
ACTIVER ABSENT
MODE PRESENT
MODE ABSENT
ALARME
DESACTIVEE

Thanks, I will try to add this to the next release.


[DOC] Examples below Docstrings

Sklearn provides a list of examples below the docstring of each estimator.
I think this is incredibly useful and I would love to have something similar in aeon. I am not sure, how this is generated?
See for example:

https://scikit-learn.org/stable/modules/generated/sklearn.linear_model.LogisticRegression.html#sklearn.linear_model.LogisticRegression

This would be amazing to have I agree, it seems quite complex after taking a quick look, however. No easy switch or plugin for this one unfortunately 🙂.
It seems like one of those things which would be worth the effort eventually, though.


[Enhancement] Make EXP from commands able to be shared with ProSkillAPIParties

Issue:
I have set-up the entirety of my server to function with commands, and it seems that ProSkillAPI Parties does not share EXP with commands. It would be wonderful if it could!
Solution:
Please allow commands to be considered in ProSkillAPI Parties when it comes to EXP sharing.
Alternatives:
I have not been able to consider an alternative for this, so we are wholeheartedly banking on the support from the dev team.

User Jammwood also suggested that it could be part of the command, so we would have more options rather than 100% on or 100% off, like add to the command /class exp 100 -share for example!

This will be more easily done after the command rework

The same thing happens to me, I operate with commands and there is no way to divide the experience. so..
+1


python37Packages.pyrsistent: 0.15.2 -> 0.15.4

Motivation for this change
Some tests are broken on 0.15.2 if used with pytest 5+ as discussed
in #64145. The issue is fixed in tobgu/pyrsistent#175 and released in
0.15.4.
As this package is a required dependency to build poetry this fix takes use one step closer to be able to build poetry on unstable, however it is not enough as poetry now fails to build with the following error:

ERROR: Invalid requirement: ''

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip" NOTE poetry and python27Packages.poetry still fails to build
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @desiderius

https://github.com/NixOS/nixpkgs/issues/68361 for visibility.

@GrahamcOfBorg build poetry python27Packages.pyrsistent python37Packages.pyrsistent

python2Packages.poetry build being fixed #70225

technically not from this PR, but build backported and fixed in https://github.com/NixOS/nixpkgs/commit/20e214bd23133f1cc9b1100aba69928807136928

The fix has been integrated: https://github.com/NixOS/nixpkgs/commit/9d9aa6099f6c284aa5df7d45d83c56e57552251a

there's a large overlap between the two commits, essentially one uses pytest_4 and other pytest_5.

after the rebase, this PR was essentially reduced down to the substituteInPlace command


Unable to run integration tests

Version of Warden
0.12.0
Operating System and Installation Method
Mac OS Big Sur via Brew
Describe the Bug
I tried to run the integration test in warden but getting the following error:
vendor/bin/phpunit -c $(pwd)/dev/tests/integration/phpunit.xml ./app/code/MagePsycho/SeoSuite/Test/Integration

Exception: SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo for tmp-mysql failed: Name or service not known

I already I configured the .env as
#WARDEN_DB=1
WARDEN_TEST_DB=1

And dev/tests/integration/etc/install-config-mysql.php as
return [
    'db-host' => 'tmp-mysql',
    'db-user' => 'root',
    'db-password' => 'magento',
    'db-name' => 'magento_integration_tests',
    'db-prefix' => '',

I then run
warden env stop
warden env start
warden env up -d

After the command ran successfully, I saw this message:
 ⠿ Container mage246-elasticsearch-1     Running
 ⠿ Container mage246-redis-1             Running
 ⠿ Container mage246-db-1                Running
 ⠿ Container mage246-elasticsearch-hq-1  Running
 ⠿ Container mage246-php-fpm-1           Running
 ⠿ Container mage246-php-debug-1         Running
 ⠿ Container mage246-nginx-1             Running

Where is tmp-mysql ? And WTH is elasticsearch-hq ?
Have anyone successfully run integration tests on warden?
To Reproduce
Run the command:
vendor/bin/phpunit -c $(pwd)/dev/tests/integration/phpunit.xml ./app/code/MagePsycho/SeoSuite/Test/Integration

Exception: SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo for tmp-mysql failed: Name or service not known

Expected Behavior
It should run the Integration test under MagePsycho_SeoSuite module.
Additional context
No response

@MagePsycho elasticsearch-hq is a frontend for ElasticSearch to view details about the ES server. You can disable this by adding WARDEN_ELASTICHQ=0 to your .env file.
I haven't tried integration tests yet, so not sure why the mage246-tmp-mysql-1 container isn't starting

Could you verify this issue continues to exist?

I'm not experiencing this issue in Warden 0.14.0 / beta.  I'm going to go ahead and close this.  If it continues to be an issue, please open a thread in Help & Support as it may be environmental.


[FEATURE] VolumeSnapshot Management

Is your feature request related to a problem? Please describe.

add a new page named Snapshot to show resouces in /v1/snapshot.storage.k8s.io.volumesnapshots

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Test Plan;

create a vm
create a vm snapshot by Take Snapshot

should not show volume snapshot in Volume Snapshots page

should not show volume snapshot in volume detail page (currently ,we can see these snapshots, need enhancement @DaiYuzeng)

The snapshot list will be filtered on Volume page
@futuretea @lanfon72

Verified this feature has been implemented.
Test Information

Environment: qemu/KVM 2 nodes
Harvester Version: master-9c3f8a63-head
ui-source Option: Auto

Verify Steps:

Install Harvester with any nodes
Create an Image for VM creation
Create vm vm1 and start it
*Take Snapshot on vm1 named vm1s1
Navigate to Volumes, click disks of vm1 then move to Snapshots tab, volume of snapshot vm1s1 should not displayed
Navigate to Advanced/Volume Snapshots, volumes of snapshot vm1s1 should not displayed
Navigate to Advanced/VM Snapshots, snapshot vm1s1 should displayed


Add support for httpHeaderConfig

Add de HttpHeaderConfig support for ALB ListenerRule

Thanks.
Note: this change use "str" as the type instead of "basestring" and there were also other Condition configs needed. I'm going to fix these up with a subsequent commit.


Colin's centered menu with logo (zurb building block)

How can this bug be reproduced?
Unable to get the menu items to position to the centre of the screen
http://zurb.com/building-blocks/centered-top-bar-with-logo
What happened instead?
I can only get the 2 menus to line up with the edge of my grid, unlike the demo which starts from the logo in the centre
Please List the Following:
OS X El Cap

Node version (node -v) [Node v4+ is required] : 6.2
Foundation version (bower list) : FoundationPress 6.2.2

I have just used the code "as is" on the zurb site but instead of the menu items starting at the centre and growing out, mine are starting from the edge of the screen and growing in, I have added
class="top-bar column row" to contain to the grid but its still working outside in.
Also changing the variable $logo-padding: $logo-width - 32px;  is having no effect at all

I haven't tried it with FoundationPress, and this Building Block was made with an older version of Foundation (either 6.0.x or 6.1.x). Did you try recreating it with just Foundation in 6.2.2?

Not had a chance to test this yet, but I will do. Thanks


GH-25025: [C++] Build core compute kernels unconditionally

This includes the core compute machinery in libarrow by default - in addition to all cast kernels and several other kernels that are either dependencies of cast (take) or utilized in libarrow/libparquet (unique, filter). The remaining kernels won't be built/registered unless ARROW_COMPUTE=ON (note that this would slightly change the option's meaning, as currently, nothing in arrow/compute is built unless it's set).
Initially this was more substantial as the original goal was to build the extra kernels as a shared library (suggested in the orginal issue). After some discussion in the issue thread, I opted not to do that - primarily because I can't personally see the utility of a separate lib here, even ignoring the complexity it introduces. However, there may be a good reason that simply hasn't occured to me.

Closes: #25025

One thing I haven't decided is how to deal with the compute unit tests since most of them make heavy use of the extra kernels, so a good chunk of them will fail without them. Easiest option would be force ARROW_COMPUTE=ON if ARROW_BUILD_TESTS=ON (not the worst idea in the world, i guess - as this is a packaging-focused feature). Alternatively, we could just not build the tests in question - although that would include most of the tests in compute/exec.
(Also, I'll look into the unity build failures - not quite sure what's going wrong there...)

cc @felipecrv

This isn't enough to close the issue, right? Do you want to associate this one with a sub-issue of #25025 so you can merge it before working on the shared library setup?

Well to be fair, that might still be the right move as it'd be easy to make that a follow-up PR if we decide to go down that road (assuming we get the library boundaries right in this one).

The goal is to eventually reduce the size of the "core" library right?  Do we have any idea how slim this set of baseline kernels is compared to the full set?

Do we have any idea how slim this set of baseline kernels is compared to the full set?

Here's what would be present in the default build:
array_filter
array_take
cast
dictionary_encode
drop_null
filter
indices_nonzero
take
unique
value_counts

There are currently 240 kernels in the full set, so it's a pretty deep cut.

Do you want to add a CI job (or does one already exist?) that builds without ARROW_COMPUTE to ensure basic functionality (e.g. parquet reading/writing and csv reading/writing) still works?

That would be a good idea, yes. AFAIK none of the existing jobs build without ARROW_COMPUTE. Even if they did, the CSV writer/STL tests wouldn't be included and libparquet wouldn't be built at all.

This failure looks related: https://github.com/apache/arrow/actions/runs/4247620924/jobs/7385865305
Perhaps just a changing of include orders has angered the unity build gremlins in some way.  Given the two implementations are identical maybe we could put them in util_internal.h?

I still need to add the CI job, but in preparation, I set things up so that certain tests won't be built without the complete kernel registry - so we wouldn't need any special ctest flags to avoid  expected failures.
The unity build redefinition errors should be fixed now. Most of the problematic code in scalar_round.cc was actually completely unused, so I just removed it.

@felipecrv / @lidavidm I haven't been following the discussion on #25025 very closely.  However, this change seems good.  I assume we want to proceed with it?

Yes, either way, I think this is a necessary first step before we can unbundle the kernels + I hope it is easier to review this way


(maint) Cleanup string externalization in lib/puppet/property/ensure.rb

This commit fixes some over-marking, marks a couple additional errors,
and replaces format string keys with more meaningful words for
translators.
This will need to be rebased once https://github.com/puppetlabs/puppet/pull/5969 is merged.

Sufficiently fixed in the above referenced PR, closing.


expected conv2d_1

I'm getting
ValueError: Error when checking input: expected conv2d_1_input to have shape (150, 150, 3) but got array with shape (224, 224, 3)
after clicking 'Predict' using my own model trained with Keras.
Attached is the model.h5 if anyone is interested in trying it out.
Tensorflow: 1.12.0
Keras: 2.2.4
Python: 3.5.2
model.zip

Seems that you need to resize the input image to (150, 150, 3), so that it will fit the input layer of your own trained model.

Thanks.
But unfortunately now I'm getting:
ValueError: decode_predictions expects a batch of predictions (i.e. a 2D array of shape (samples, 1000)). Found array with shape: (1, 1)

You need to modify the code based on your model structure.
You could print it using model.summary()
For other reference, go to the official document https://keras.io/


Integration tests inconsistently throw in the Finally block

Integration tests inconsistently throw in the Finally block on Restore-WebConfiguration -Name $tempName with this error:
IOException: The process cannot access the file 'C:\windows\system32\inetsrv\config\applicationHost.config' because it is being used by another process.

Would it help adding a Start-Sleep before Restore-WebConfiguration and wait a few seconds for it to release the lock? Or should we just re-try a few times with a few seconds interval until an error is thrown?

Closing this in favor of issue #385


#7944 add dockerfile and installation steps

Changes as follows

Revised INSTALLING.md to incorporate using docker for install workflows.
I had to re-order some of the steps but hopefully instructions should be clear enough.
I believe you can close your eyes and just copy all the code blocks and it should still work

Added dockerfile folder: This holds relevant dockerfile so users can build locally if desired

Added scripts/docker: This folder holds relevant shell scripts to build/install docker and necessary dependencies. Also includes a shell script to bring up and automatically bring user into the docker container to begin development

Caveats:
When a docker container is brought up, a workspace folder is created under the ${HOME} directory so that files and folders can be shared between the user and the docker container. This is to prevent users from needing to endlessly rebuild tt-metal repo if a container is continuously brought up and down

@adifrancescoTT
Feel free to go over the README and see if there's anything confusing to you

I'll be honest, I find this usage of docker-compose to be... odd. It's usually used to run a service permanently.
Sure, that's how we use Docker internally, to set up a container that you ssh into to do work, but we don't have to use it that way.
I have kept a similar Dockerfile locally, and I just launch the build and test from within it, without "moving into" the docker environment. So, I have a launch script that runs the build, like the below. Note: This isn't for sharing directly, since it has paths/dependencies on my paths, etc.
#!/bin/bash -x

set -e

export CONTAINER=2404:latest

[ -d /work/.pipcache ] || mkdir /work/.pipcache

function run_docker {
	sudo docker run \
		--rm -t -i \
		-v /work:/work \
		-v /home:/home \
		-v /dev/hugepages-1G:/dev/hugepages-1G \
		-w /work/tt-metal \
		-e TT_METAL_HOME=/work/tt-metal \
		-e TT_METAL_ENV=dev \
		-e ARCH_NAME=grayskull \
		-e PYTHONPATH=/work/tt-metal \
		-e XDG_CACHE_HOME=/work/.pipcache \
		-e DEVICE_CXX=/usr/bin/clang++ \
		-e SILENT=0 \
		--net host \
		--device /dev/tenstorrent/0 \
		--privileged \
		${CONTAINER} \
		"$@"
}

run_docker script -c "make build -k -j$(nproc)" build.out.$(date +%Y%m%d%H%M%S)
run_docker script -c "make tests -k -j$(nproc)" tests.out.$(date +%Y%m%d%H%M%S)
#run_docker script -c ". ./build/python_env/bin/activate ; ./tests/scripts/run_tests.sh --tt-arch grayskull --pipeline-type post_commit" pytest.out.$(date +%Y%m%d%H%M%S)

FWIW, the Dockerfile I've been using is similar but maybe a bit less verbose. This is the Ubuntu 24.04 version:
FROM ubuntu:24.04

# Local setup to use a proxy server for all package installs for speedup
#COPY 00aptproxy /etc/apt/apt.conf.d
RUN apt-get update
RUN apt-get install -y --no-install-recommends apt-utils dialog
RUN DEBIAN_FRONTEND=noninteractive apt-get install -y libboost-all-dev sudo zip unzip
RUN DEBIAN_FRONTEND=noninteractive apt-get install -y tmux vim git
RUN apt-get install -y pciutils kmod python3-pip
RUN apt-get install -y python3 python3-dev python3-venv
RUN apt-get install -y curl jq
RUN apt-get install -y software-properties-common build-essential python3-venv libgoogle-glog-dev libyaml-cpp-dev libboost-all-dev libsndfile1 libhwloc-dev
RUN apt-get install -y clang git git-lfs cmake pandoc libtbb-dev libcapstone-dev pkg-config
RUN apt-get install -y python-is-python3
RUN apt-get install -y clang

# Ubuntu 24.04 has a new enough rustc/cargo, so just go with that instead of rustup for now
RUN apt-get install -y rustc cargo

# Doxygen and gtest install
RUN curl -L "https://www.doxygen.nl/files/doxygen-1.9.6.linux.bin.tar.gz" -o doxygen-1.9.6.linux.bin.tar.gz
RUN tar -xvf doxygen-1.9.6.linux.bin.tar.gz
RUN cd doxygen-1.9.6 && sudo make install

RUN curl -L "https://github.com/google/googletest/archive/refs/tags/v1.13.0.tar.gz" -o v1.13.0.tar.gz
RUN tar -xvf v1.13.0.tar.gz
RUN cd googletest-1.13.0 && cmake -DCMAKE_INSTALL_PREFIX=/usr -DBUILD_SHARED_LIBS=ON . && make && sudo make install

# 24.04 comes with a pre-populated ubuntu account, which conflicts with my own default UID,
# so delete it and create one with my own username. This is obviously non-portable.
RUN deluser ubuntu
RUN useradd -m -s /bin/bash -u 1000 olof

USER olof

@olofj
thanks for your input! It was intended to add a docker compose file as a way of just getting a permanent environment setup for users/devs more as a convenience/example rather than forcing users down a particular dev path.
I don't have any strong opinions either way on this so I'm happy to remove docker compose and use your method if you think using it this way is too strange.

Split this PR into smaller PRs
https://github.com/tenstorrent/tt-metal/pull/8566
https://github.com/tenstorrent/tt-metal/pull/8565
@TT-billteng @tt-rkim


🌱 Bump CAPI  to v1.7.0-rc.0

as title says

/test-centos-e2e-integration-main
/test-ubuntu-integration-main


Create a disabled state for the .btn-with-arrow button

Just added it to the default reference sheet

@souperneon It looks like the background/text color values for the secondary buttons are wrong, and I'm assuming from the images that nothing is changing.

Correct. Ignore everything except the button with arrow I guess :P


Use Formula#oldnames to silence deprecation notices

This doesn't really fully safely use Formula#oldnames, just
effectively uses what Formula#oldname does now.
https://github.com/Homebrew/brew/blob/4bbc0af6935816c75e0057364a84fd7fa8de071b/Library/Homebrew/formula.rb#L502-L515
Fixes #1223

Test failure looks like a temporary failure to retrieve the latest https://formulae.brew.sh/api/cask.jws.json.

Thanks for workaround here @colindean and for merging @jacobbednarz!


Update deployment docs

Reorganizes the docs to include Cloud Run, App Engine, and Docker configurations.

I've made a note for deploying to Hugging Face.


Add note to README.md

Adding a note to generate only using Linux since the generation locally doesn't work with Windows

Codecov Report

Merging #1802 (60e198e) into main (9f4933b) will not change coverage.
Report is 11 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1802   +/-   ##
=======================================
  Coverage   95.00%   95.00%
=======================================
  Files           2        2
  Lines          20       20
=======================================
  Hits           19       19
  Misses          1        1

Flag
Coverage Δ

dotnet
95.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Seems like this is a long-standing bug with nodejs & Windows. We generally aim to be cross-platform in our OSS products, so I'd rather not just decide we're not going to support Windows.
There are some options to handle this - could you try playing around with the maxRetries option documented in https://nodejs.org/api/fs.html#fspromisesrmpath-options and seeing if that makes a difference?

I tried setting MaxRetries to 3 but it made no difference 😞


Update URL in repository details

Hi,
I noticed that the URL set in the repository details is incorrect. Currently, it is set to kubernetesfury.com, but it should be docs.kubernetesfury.com. The current URL points to an index.html page that is rendered twice, causing display issues (see this issue).
We should consider updating the URL to fix it.

Nice catch. Fixed.
Thanks!


Result::coalesce

For use with Vec::binary_search_by (I guess it's technically part of slice), it would be nice to have an Result::coalesce that takes an Result<T, T> and turns it into an T.

Decided against in https://github.com/rust-lang/rust/pull/54219; please direct any new information that way.


How to display branch nodes / parallel blocks

I am confused about how to transform the following structure which has branches coming out from a node.
This is what I tried
hl.transforms.Fold("""Conv > LeakyRelu > Conv > Concat > LeakyRelu """, "Fire","FireBlock")

I find this in demos:
And the expressions Conv | MaxPool will match any Conv and MaxPool layers that are in parallel branches (i.e. have the same parent node).
So I think """Conv > LeakyRelu > (Conv | Conv) > Concat > LeakyRelu """ may work.

Thanks. It worked.


copy.deepcopy cannot copy torchtyping.utils.frozendict objects

copy.deepcopy is used in some training loops to save a copy of the model parameters that has the best validation loss. Currently, the following code gives an error:
import copy
from torchtyping import TensorType
copy.deepcopy(TensorType["batch", "embedding"])

TypeError: __setitem__() takes 2 positional arguments but 3 were given

Instead, copying the dictionary should just return the dictionary unaltered. Since it cannot be changed, the copy does nothing.

LGTM, thanks!


Small fixes for 3-rewrite

fixed lint, add support for options.color and upgrade to<EMAIL_ADDRESS>

great work on that fork btw, thank you

Hey, thanks for the PR!
@jbourassa will review shortly 😄

Bonjour @revolunet,
Thanks for your contribution.

fixed lint
I'm all up for linting, but it only works if there's a standard and tool we can agree to use. What linting config did you use? Is there one provided by the videojs project?

hello, the linting use videojs-standard which is in your package.json ?

the linting use videojs-standard which is in your package.json ?

Awwwh. Right. Sorry. This is a boilerplate from videojs that I started with, as you can tell I'm not all that familiar with it!

Thank you!


About CNNVD-202205-4094

This cve is invalid, if you're running the dev server in production you have bigger security issues. The dev server is never intended to be run in production. The cve is also misattributed, it is about Python's http.server.
That said, upgrade Werkzeug
Originally posted by @davidism in https://github.com/pallets/werkzeug/issues/2420#issuecomment-1137369788

Is that means, upgrade werkzeug is neccssary? Or it's realy invalid

Both

Yeah, thanks. But i'm still using py2.7 , it's diffcult for me to update dependencies all.

You have bigger problems then. Work on upgrading to Python 3.8 or later as soon as possible.

That can't be decided by me, only the whole team, it's a terrible thing and will take a long time for us to update.
Thanks for your idea and time~


Search field for game/module id

Added module search page, but will have to isolate the search component for possible future use


Add a New Project: 3D Distance Calculator

Project Overview:
I am looking to add a new project to the "30 Days of Code" website. This project will be a 3D Distance Calculator that allows users to compute the distance between two points in a three-dimensional space. The calculator should take the coordinates of two points as input and output the distance between them.
Project Requirements:
Input: The calculator should accept two sets of coordinates as input, each consisting of three values (x, y, z).
Output: The output should be the calculated distance between the two points.
Formula: Use the Euclidean distance formula:

User Interface:
A clean and simple UI for users to input coordinates.
A "Calculate" button to trigger the distance computation.
Display the result in a user-friendly manner.
Responsive Web Design Basics

Let's enhance the learning experience on "30 Days of Code" with this exciting new project! If you have any questions or need further clarification, feel free to ask. Happy coding!

Assignment Request:
Please assign this task to me under GSSoC'24.
Labels: GSSoC Level3

@MrPC7 All the best 👍🏻and make sure it is mobile responsive also.


fix: lesson page blowing up when a download existence check fails

Music year: 2022
Fixes the lesson page blowing up when a the download existence check throws.
If the download existence check fails, we should not block the page from rendering. There appear to be two ways the getDownloadResourcesExistence will report a missing resource. Either by returning an object in the form { resources: ["worksheet-pdf", { exists: false }] } or by throwing. Catching and reporting the error matches the behaviour of the teachers downloads page and allows the lesson page to render
How to test

Go to the migrated legacy lesson https://deploy-preview-2253--oak-web-application.netlify.thenational.academy/pupils/programmes/english-secondary-ks3/units/the-oral-tradition-7424/lessons/the-origins-of-storytelling-cmrk0r — this lesson does not have a worksheet
You should see the overview page
Click "Intro"
You should see the intro page with no worksheet

:tada: This PR is included in version 1.353.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


stats wanted

As per Skype call:
Numbers of annotations made total/ number of annotations per paper per year
Also, It would be useful if we could get an overview of the phenotype annotation
i) How many phenotype annotations have we made so far (and using how many papers)
We may have further questions about phenotype data later.
Original comment by: ValWood

ancient, and there are fairly recent query results in dropbox


Add sample notebooks for multi-model endpoints functionality

Issue #, if available:
Description of changes: Add sample notebooks for multi-model endpoints functionality.

advanced_functionality/multi_model_bring_your_own/multi_model_endpoint_bring_your_own.ipynb shows how to deploy multiple models to a realtime hosted endpoint with your own custom algorithm.
advanced_functionality/multi_model_xgboost_home_value/xgboost_multi_model_endpoint_home_value.ipynb shows how to deploy multiple models to a realtime hosted endpoint using a multi-model enabled XGBoost container.
advanced_functionality/multi_model_sklearn_home_value/sklearn_multi_model_endpoint_home_value.ipynb shows how to deploy multiple models to a realtime hosted endpoint using a multi-model enabled SKLearn container.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The notebook example here- sklearn_multi_model_endpoint_home_value.ipynb does not work. The outputted docker image won't deploy to an endpoint and gives the following error:
Building wheels for collected packages: None
  Building wheel for None (setup.py): started
  Building wheel for None (setup.py): finished with status 'done'
  Created wheel for None: filename=None-1.0.0-py2.py3-none-any.whl size=3777 sha256=517ba93ca18c25ecd4336352ebf8930c8decb81d858abf77739218f269d00b5b
  Stored in directory: /tmp/pip-ephem-wheel-cache-t39q9rdl/wheels/3e/0f/51/2f1df833dd0412c1bc2f5ee56baac195b5be563353d111dca6
Successfully built None
Installing collected packages: None
Successfully installed None-1.0.0
Traceback (most recent call last):
  File "/miniconda3/lib/python3.7/site-packages/sagemaker_containers/_modules.py", line 269, in import_module
    module = importlib.import_module(name)
  File "/miniconda3/lib/python3.7/importlib/__init__.py", line 118, in import_module
    if name.startswith('.'):
AttributeError: 'NoneType' object has no attribute 'startswith'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/miniconda3/bin/serve", line 8, in <module>
    sys.exit(main())
  File "/miniconda3/lib/python3.7/site-packages/sagemaker_sklearn_container/serving.py", line 107, in main
    modules.import_module(serving_env.module_dir, serving_env.module_name)
  File "/miniconda3/lib/python3.7/site-packages/sagemaker_containers/_modules.py", line 274, in import_module
    six.reraise(_errors.ImportModuleError, _errors.ImportModuleError(e), sys.exc_info()[2])
  File "/miniconda3/lib/python3.7/site-packages/six.py", line 702, in reraise
    raise value.with_traceback(tb)
  File "/miniconda3/lib/python3.7/site-packages/sagemaker_containers/_modules.py", line 269, in import_module
    module = importlib.import_module(name)
  File "/miniconda3/lib/python3.7/importlib/__init__.py", line 118, in import_module
    if name.startswith('.'):
sagemaker_containers._errors.ImportModuleError: 'NoneType' object has no attribute 'startswith'


error

Challenge Increment a Number with JavaScript has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

var myVar = 87;

// Only change code below this line
var i = myVar ;
myVar = i + 1;
myVar = i++ ;

this is my code,and i have passed this course,just like this :
var myVar = 87;
// Only change code below this line
myVar ++;
var myVar;


Add warning that diagnostic settings extension resources are not deleted with the parent resource

Currently diagnostic setting extension resources are not (at least immediately) deleted when the parent resource is deleted. This can result in unexpected behavior, where resources that are deleted and recreated with the same name resume diagnostic logging to the previously configured destinations.
If this behavior is by design, add a warning that diagnostic settings should be deleted before deleting parent resources.

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

ID: 89743709-c3fa-9ce1-55fb-e9b7f6d2dba9
Version Independent ID: a4c9f0b7-5dde-d60c-1110-3ca53d904d61
Content: Diagnostic settings in Azure Monitor - Azure Monitor
Content Source: articles/azure-monitor/essentials/diagnostic-settings.md
Service: azure-monitor
Sub-service: essentials
GitHub Login: @rboucher
Microsoft Alias: robb

@maskati
Thanks for your feedback! We will investigate and update as appropriate.

@maskati, Thank you for this question. Based on the question, I assume you are writing about Diagnostic settings in Azure VM through extension.
These extensions depend on the parent resource (VM) and do not exist without it. They get removed as part of VM deletion process. It is possible that you were still seeing them temporarily post removal pending portal/cache refresh.
Please let us know if you have any questions.

Emailed the product group to clarify behavior.

@maskati What type of resource where you using when you ran across this behavior? Was it an Azure VM and while using the Diagnostic Extension or something else?

My experience is that this is a general issue impacting diagnostic settings applied to many different types of resources. Below is a simple example of how the issue manifests when a diagnostic setting is applied to a log analytic workspace (audit log queries to itself). The diagnostic setting is created, the log analytic workspace is deleted, the log analytic workspace is recreated with the same name, the old diagnostic setting reappears since it seems to have been orphaned by the previous log analytics workspace instance. You can also confirm they are different workspaces by checking the customerid.

@maskati, thank you for the detailed explanation and sample. Please allow us some time to get back on this.

@maskati Thank you for the feedback. I added a warning to the article, which is located a few lines above the video.
#please-close


Link to blogposts in documentation

Your blog posts in  http://bitsbytesgates.blogspot.com/search/label/PyVSC would form a nice chapter with background info, or some parts could be a tutorial, but for now I would just add that link to the introduction.

BTW,  http://bitsbytesgates.blogspot.com/2020/03/modeling-random-stimulus-and-functional.html is not tagged "PyVSC".

Good suggestion, @cmarqu! I've linked the blog posts for now.


How to set seed within the sampling process?

I am wondering if anyone can help point to the places where we can set seed?
Thanks!

Hi @jxie-im ,
I can't test it now but does the following work?
seed = 24
os.environ['TF_DETERMINISTIC_OPS'] = 'true'
os.environ['PYTHONHASHSEED'] = f'{seed}'
np.random.seed(seed)
random.seed(seed)
tf.random.set_seed(seed)


Find in Files feature does not work

VSCode Version: 1.41.1
OS Version: Windows X

Steps to Reproduce:

Press Ctrl+Shift+F
Type in some text which you are sure exists in one of the unopened files.
VS Code 'Find in Files' feature will not locate the unopened file.

Does this issue occur when all extensions are disabled?: Yes

Please see the tips here and let me know if you still have issues: https://github.com/microsoft/vscode/wiki/Search-Issues#missing-search-results

Thanks for the link. It does not address the 'Find in Files' feature not searching the whole workspace. It only searches open files.

Did you check your exclude patterns and gitignore files? Can you try a search with the "Use exclude settings and ignore files" button turned off as a test?

I just try a search with the "Use exclude settings and ignore files" button turned off as a test. It did not search my unopened application folders. But it searched in the previously excluded folders even though those folders were unopened as well.

It did not search my unopened application folders. But it searched in the previously excluded folders even though those folders were unopened as well.

I don't understand, it searched some folders in your workspace but not all? Is there something special about the others that are not being searched?

Only differences I can tell is that the excluded folders are not source files.
The included files list contains name of the project folder
The excluded files list contains: .history, dist

Are you sure that .history and dist are not excluded by your .gitignore file or exclude patterns? They sound like folders that would be purposely excluded.  https://github.com/microsoft/vscode/wiki/Search-Issues#missing-search-results

They are in the excluded patterns list in the search activity view of the project explorer. They are being excluded from the search as intended. The issue though is that search term occurrences within unopened source files are not being reported. I have been researching up on this issue. The feature appears more and more to have been designed to search only open files. Might that really be the case?

I see. Open files will always be searched even if they are excluded (there is an issue for this) but otherwise I am not sure what your issue is. I guess I am still not clear on exactly what files are not searched and what might be special about them.

Nothing special about them really - just unopened files in the ./src folder. They are not in the exclude patterns list. For reasons unknown, they are not being searched. Are you able to run the 'Find in Files' feature in your VSCode IDE, and get matches occurring in both opened and unopened files in your search results?

Yes. You might get more info by looking at the search logs to see the exact ripgrep command run:

Run the command "Set log level" and change it to "Trace"
Run the search again
Show the search-related messages in the "Log (Window)" output panel

Most definitely, @roblourens . The following is a copy of log messages I received. Just a little background: there are several occurrences of the 'COLLECTION_KEY' search term throughout the sandbox/ws/playground directory. Only the 6 occurrences within the one open file were reported.
[2020-02-09 13:27:34.640] [renderer1] [trace] telemetry/workbenchActionExecuted {"id":"workbench.output.action.clearOutput","from":"panel"} [2020-02-09 13:27:39.696] [renderer1] [trace] SearchService#search {"_reason":"searchView","folderQueries":[{"folder":{"$mid":1,"fsPath":"c:\\sandbox\\ws\\playground","_sep":1,"external":"file:///c%3A/sandbox/ws/playground","path":"/C:/sandbox/ws/playground","scheme":"file"},"excludePattern":{"**/.git":true,"**/.svn":true,"**/.hg":true,"**/CVS":true,"**/.DS_Store":true,"**/node_modules":true,"**/bower_components":true,"**/*.code-search":true,"**/playground-legacy/**":true},"fileEncoding":"utf8","disregardIgnoreFiles":false,"disregardGlobalIgnoreFiles":true,"ignoreSymlinks":false},{"folder":{"$mid":1,"fsPath":"c:\\sandbox\\ws\\playground-legacy","_sep":1,"external":"file:///c%3A/sandbox/ws/playground-legacy","path":"/C:/sandbox/ws/playground-legacy","scheme":"file"},"excludePattern":{"**/.git":true,"**/.svn":true,"**/.hg":true,"**/CVS":true,"**/.DS_Store":true,"**/node_modules":true,"**/bower_components":true,"**/*.code-search":true,"**/playground-legacy/**":true},"fileEncoding":"utf8","disregardIgnoreFiles":false,"disregardGlobalIgnoreFiles":true,"ignoreSymlinks":false}],"usingSearchPaths":false,"excludePattern":{"**/*.history/**":true,"**/*.history":true,"**/dist/**":true,"**/dist":true},"includePattern":{"**/playground/**":true,"**/playground":true},"maxResults":10000,"type":2,"contentPattern":{"pattern":"COLLECTION_KEY","isRegExp":true,"isCaseSensitive":true,"isWordMatch":false,"wordSeparators":"~!@#$%^&()-=+[{]}\|;:'",.<>/?"},"previewOptions":{"matchLines":1,"charsPerLine":10000},"usePCRE2":false}
[2020-02-09 13:27:39.712] [renderer1] [debug] SearchService#search provideTextSearchResults COLLECTION_KEY, {"folder":"file:///c%3A/sandbox/ws/playground","excludes":["**/.history/","/.history","/dist/","/dist","/.git","/.svn","/.hg","/CVS","/.DS_Store","/node_modules","/bower_components","**/.code-search","/playground-legacy/"],"includes":["/playground/","/playground"],"useIgnoreFiles":true,"useGlobalIgnoreFiles":false,"followSymlinks":true,"encoding":"utf8","maxFileSize":17179869184,"maxResults":10000,"previewOptions":{"matchLines":1,"charsPerLine":10000},"usePCRE2":false}
[2020-02-09 13:27:39.712] [renderer1] [trace] telemetry/searchResultsFirstRender {"duration":15}
[2020-02-09 13:27:39.713] [renderer1] [debug] SearchService#search rg --hidden --case-sensitive -g '/playground/' -g '/playground' -g '!/*.history/' -g '!/*.history' -g '!/dist/' -g '!/dist' -g '!/.git' -g '!/.svn' -g '!/.hg' -g '!/CVS' -g '!/.DS_Store' -g '!/node_modules' -g '!/bower_components' -g '!/*.code-search' -g '!/playground-legacy/' --max-filesize '17179869184' --no-ignore-parent --follow --crlf --auto-hybrid-regex --regexp 'COLLECTION_KEY' --no-config --no-ignore-global --json -- '.'

cwd: c:\sandbox\ws\playground
[2020-02-09 13:27:39.734] [renderer1] [debug] SearchService#search provideTextSearchResults COLLECTION_KEY, {"folder":"file:///c%3A/sandbox/ws/playground-legacy","excludes":["/*.history/","/*.history","/dist/","/dist","/.git","/.svn","/.hg","/CVS","/.DS_Store","/node_modules","/bower_components","/.code-search","/playground-legacy/"],"includes":["/playground/","/playground"],"useIgnoreFiles":true,"useGlobalIgnoreFiles":false,"followSymlinks":true,"encoding":"utf8","maxFileSize":17179869184,"maxResults":10000,"previewOptions":{"matchLines":1,"charsPerLine":10000},"usePCRE2":false}
[2020-02-09 13:27:39.734] [renderer1] [debug] SearchService#search rg --hidden --case-sensitive -g '/playground/' -g '/playground' -g '!**/.history/' -g '!/.history' -g '!/dist/' -g '!/dist' -g '!/.git' -g '!/.svn' -g '!/.hg' -g '!/CVS' -g '!/.DS_Store' -g '!/node_modules' -g '!/bower_components' -g '!**/.code-search' -g '!/playground-legacy/' --max-filesize '17179869184' --no-ignore-parent --follow --crlf --auto-hybrid-regex --regexp 'COLLECTION_KEY' --no-config --no-ignore-global --json -- '.'
cwd: c:\sandbox\ws\playground-legacy
[2020-02-09 13:27:39.926] [renderer1] [debug] SearchService#search Got data from stdout
[2020-02-09 13:27:39.926] [renderer1] [debug] SearchService#search No result from parser
[2020-02-09 13:27:39.926] [renderer1] [debug] SearchService#search
[2020-02-09 13:27:39.927] [renderer1] [debug] SearchService#search Got data from stdout
[2020-02-09 13:27:39.927] [renderer1] [debug] SearchService#search No result from parser
[2020-02-09 13:27:39.927] [renderer1] [debug] SearchService#search
[2020-02-09 13:27:39.931] [renderer1] [trace] SearchService#search: 234ms
[2020-02-09 13:27:39.932] [renderer1] [trace] telemetry/textSearchComplete {"reason":"searchView","workspaceFolderCount":2,"endToEndTime":234,"scheme":"file","usePCRE2":false}
[2020-02-09 13:27:39.939] [renderer1] [trace] telemetry/searchResultsShown {"count":6,"fileCount":1,"options":{"isRegExp":true,"isCaseSensitive":true,"isWordMatch":false,"wordSeparators":"~!@#$%^&*()-=+[{]}\\|;:'\",.<>/?"},"duration":243,"scheme":"file","searchOnTypeEnabled":true} [2020-02-09 13:27:39.939] [renderer1] [trace] telemetry/searchResultsFinished {"duration":243}

This shows the excludes that are applied, are the missing files in any of these? -g '!**/.hg' -g '!**/CVS' -g '!**/.DS_Store' -g '!**/node_modules' -g '!**/bower_components' -g '!**/*.code-search' -g '!**/playground-legacy/**'. Also, any patterns in a .gitignore or .ignore file?

I just noticed that you are using Windows X. It's possible that the bundle isn't including the correct rg build, how did you install vscode?

Also, you could try running C:\Users\USER\AppData\Local\Programs\Microsoft VS Code\resources\app\node_modules.asar.unpacked\vscode-ripgrep\bin\rg.exe (or wherever you have installed vscode) to see whether it runs at all on your system, or produces any errors.

My apologies for my late response, @roblourens. I have been busy with other aspects of the project for a while. Thanks for your efforts. When I get a chance, I will take the time to attempt your latest suggestions. Hopefully, there is a solution embedded somewhere in there.


Plex integration

Feature request:
Plex-checker which checks the playing status of Plex and decides whether or not to capture the screen.
Plex added Webhooks recently: https://support.plex.tv/hc/en-us/articles/115002267687

Greetings,
regarding integrations we are little bit splitted as you could read here: https://github.com/hyperion-project/hyperion.ng/issues/350
So this should be better a "plugin" on client side which reads the webhooks and controls Hyperion.
As we still hang around at basics it is unlikely that someone will get in touch with it soon.
We are lacking of developers (as you see at the commit/pr history :))

As brindoch told already It is better when a plugin in the media center (kodi, plex ,...) controls hyperion via json commands over raw tcp or websockets.
I also want direct kodi support out of hyperion, because it is hard to support different kodi versions and I fear we would have the same problem with plex.
Please ask in plex developers if they can made the integration in plex.


(what I believe to be) emoji doesn't render on Bot model

https://docs.top.gg/api/bot/#bot-structure:
I believe this is supposed to render as ⚠?
Browser: Chromium 93.0.4577.63 (Official Build) Arch Linux (64-bit)

It's meant to be rendered as a warning symbol. I am working on a PR already for it ^-^

Also #45

This was fixed by https://github.com/top-gg/docs/pull/65 can now be closed


Experience report with JSON Merge Patch and JSON Patch

Our guidelines have a section about how to use the PATCH operation and it basically has the following list of preferences:

PUT
JSON Merge Patch
JSON Patch
POST

I wanted to express my first experiences with this after implementing PATCH in a recent API. The specialty was (and is) that our resource is particularly large. We're talking about a data structure that has up to 20k nested JSON objects on 2 nested levels. Splitting it up didn't make sense for us since the whole thing, from a business perspective, is the actual unit of work (to be precise it's  a truck load of goods). We constructed an example of a typical size and measured/estimated an size of ~5MB.
We have multiple different consumers of this API operating on the resource and they need to update different, individual parts. E.g. adding a single property to one of the 20k elements. I tried to follow the list:
PUT
Too big for individual, smaller updates. For 20k individual, parallel updates (worst case) we would generate 100GB traffic.
JSON Merge Patch
Section 2 states:

If the patch is anything other than an object, the result will always be to replace the entire target with the entire patch.  Also, it is not possible to patch part of a target that is not an object, such as to replace just some of the values in an array.

That significantly reduces the usefulness of the media type.
JSON Patch
Although I was not a big fan initially, because its imperative, RPC-style notion of operations in my mind didn't fit well with my understanding of REST. But after implementing it I have to say it's rather nice. Library support in my case was excellent. The content of the patch is still readable, although not as clear as JSON Merge Patch. Also a big benefit over PUT is that the server can implement optimistic locking without involving the client. PUT would require preconditions (ETag and If-Match). JSON Patch can simply be retried by the server if a conflict is detected.
I don't think that we have a strong need to adjust/change the guidelines right now. The current wording is fine. I just wanted to share some experience. If we feel like the guidelines could be extended, just let me know.

Just a question from the implementation point of view: Did you apply the patch to a full representation of the object which then was stored back into the database (i.e. load object, apply patch, store object), or did you apply the changes just to the parts which actually were affected?

Did you apply the patch to a full representation of the object which then was stored back into the database (i.e. load object, apply patch, store object)

Yes, including retries because we use optimistic locking.

@whiskeysierra thx. for this experience sharing.
I was not aware that JSON Patch does not support patching of array elements -- this is a mayor restriction.
Yes, for the time being, I also see no need for changing the guideline.
However, adding a hint would be great:  in your scenario of updating single objects in large collection JSON Patch is the right choice due to high payload volume and JSON Merge Patch restriction.
Perhaps we can also http://erosb.github.io/post/json-patch-vs-merge-patch/ as further read reference.


Missing test: Bad error-message for missing closing parenthesis

Getting an "unexpected token 'EOF'" for the following dart code:
foo(x {  // <= missing closing ")"
  return x;
}

Verified fixed in 79a8654e5fb30dfce902636961d6a5ffb69e6f85


style(transformer/arrow-functions): semicolon after return statements

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7169
#7168  👈
#7167
#7166
#7165
#7164
#7163
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


Nadam optimizer arguments description missed schedule_decay

Both documentation https://keras.io/optimizers/ and the source code https://github.com/keras-team/keras/blob/master/keras/optimizers.py#L605 missed the description of an argument schedule_decay. Currently, it is:
Arguments
    lr: float >= 0. Learning rate.
    beta_1/beta_2: floats, 0 < beta < 1. Generally close to 1.
    epsilon: float >= 0. Fuzz factor. If `None`, defaults to `K.epsilon()`.

While actual list of parameters is: lr=0.002, beta_1=0.9, beta_2=0.999, epsilon=None, schedule_decay=0.004
I propose to add a line like schedule_decay: float >= 0, determines "what it actually does". Could do it myself also.

Thanks for the report. It strange that our continuous integration system didn't pick up this anomaly. This is an issue. This line: https://github.com/keras-team/keras/blob/e59570ae26670f788d6c649191031e4a8824f955/tests/test_documentation.py#L115 should have triggered an error.
Fix for this test welcome.

@gabrieldemarmiesse The test passes because it only checks that methods have a proper docstring (see inspect.ismethod in here). However, __init__ has the function type. I don't think we want to apply handle_function to the init as the docstring in at the class level. Would adding a match between class docstring and init arguments make sense ?

Yes it would make sense and should be enforced. Since we use it for the docs, the docstring should be correct. This issue shows it. PR welcome to enforce this.

Fixed in #12192


Swagger ModelValidation erroneously fails when required x-ms-secret properties are omitted from responses in examples

From the x-ms-secret documentation:

When a property is modeled as both "required" and "x-ms-secret": true, which means that this property must not exist in the response but has to be present in request.

The Swagger ModelValidation pipeline fails for this scenario, erroneously flagging properties that match this scenario that are missing from example responses as errors, e.g.:
https://github.com/Azure/azure-rest-api-specs/pull/18645/checks?check_run_id=6586657182

@keryul , can you check if this issue was fixed?

Hi @keryul, thanks for responding. We are unable to make the suggested changes because our service's swagger has already been published as a stable version and they would constitute breaking changes. But more generally, could you please point me to where in the documentation it forbids applying x-ms-secret to $ref properties? I don't see any such restriction called out, and I can imagine many scenarios (such as ours) where we would need to mark an entire complex object as a secret, not just individual string properties.


Update i18n support to work with ember-i18n.

Re: https://github.com/edgycircle/ember-pikaday/issues/78

Can you revert changes to package.json? We're absolutely interested in releasing another version (...soon!) but we might get some more commits in.

Done

Thanks!


2.7.1 Breaks PDF attachments

For some strange reason, 2.7.1 and head breaks PDF attachments.

Reverting back to 2.7.0 fixes the issue.

Could you attach or link to a sample email to reproduce?
This is likely an issue with quoted-printable transfer encoding, which is not safe for PDFs or binaries.

Sure, attached is a sample pdf with an image and code to reproduce the error.
yo.pdf
mail = Mail.new
mail.attachments["yo.pdf"] = File.read("yo.pdf")
File.open("yo-out.pdf", "wb") {|f| f.write mail.attachments.first.decoded}
system "open yo-out.pdf"


Update README.ru.md

Fix error "This expression has a type of 'void' so its value can't be used"

Hi, thanks for that.
Could you let me know if other languages have the same problem? If so, could you change it to all languages?


[Table] Support fixed footer

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
I'm always frustrated when add styles or create classes just for stickyFooter.
<TableFooter>
   <TableRow>
      <TableCell style={{ backgroundColor: '#fafafa', position: 'sticky', bottom: 0 }} />
      ...
    </TableRow>
</TableFooter>

It would be nice MUI support stickyFooter that fixed column footers natively.
<Table stickyHeader stickyFooter />

Examples 🌈

@xiaoyu-tamu See the demo at https://next.material-ui.com/components/tables/#fixed-header, the footer is already sticky. The difference between the footer and the header is that the header horizontal scroll positions need to be sync with the cells, it's not the case for the footer.

@oliviertassinari
Right, the fixed-header example has fixed footer ( i would say it's a fixed toolbar/paginations ).
What I refer to is make tfoot > td fixed inside <table />, it should sync horizontal scroll with cells.
I think it's useful when ppl want to duplicate column names in bottom or display aggregate(sum,avg,..etc) value below each cell for long content table.
for example:


DllNotFoundException in ASP.NET Core 3.1 Docker container

Hello
I try to use TextToSpeech service, but i get exception when I publish to docker.
Exception:
'System.DllNotFoundException: 'Unable to load shared library 'libMicrosoft.CognitiveServices.Speech.core.so' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: liblibMicrosoft.CognitiveServices.Speech.core.so: cannot open shared object file: No such file or directory''
I want to use this service like an API ->  <IP_ADDRESS>:8000/GetAudio
Docker file:
`FROM mcr.microsoft.com/dotnet/core/aspnet:3.1-buster-slim AS base
WORKDIR /app
EXPOSE 80
FROM mcr.microsoft.com/dotnet/core/sdk:3.1-buster AS build
WORKDIR /src
COPY ["SweetHome.Api/SweetHome.Api.csproj", "SweetHome.Api/"]
RUN dotnet restore "SweetHome.Api/SweetHome.Api.csproj"
COPY . .
WORKDIR "/src/SweetHome.Api"
RUN dotnet build "SweetHome.Api.csproj" -c Release -o /app/build
FROM build AS publish
RUN dotnet publish "SweetHome.Api.csproj" -c Release -o /app/publish
FROM base AS final
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "SweetHome.Api.dll"]`

@ubid Please check corresponding issue https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/481 and let us know if that helps

@ubid Please check corresponding issue https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/481 and let us know if that helps

hi!
Thanks for your help! Now its works!

hi!
Thanks for your help! Now its works!


Made buff exclusion search case insensitive

Background: a few months back, I added the ability to exclude certain buffs from the buff duration multiplier. Last night, I noticed there was a debuff multiplier on the army paralyzed debuff, so I decided to add some exclusions. Turns out, I made the search case sensitive and it took me a while to figure out why I couldn't find the buff. This just fixes that problem.
@ArcaneTrixter, I know you're planning on making a patch soon, so I thought you might want to roll this with that release, but if you'd prefer me to add version/patch notes, I can do that.

Please in the future add an update to the readme describing your change and giving yourself credit.  For this change I took care of this for you.


s2i: unexpected error entry in s2i build logs

Bug 1364965
Bugzilla link https://bugzilla.redhat.com/show_bug.cgi?id=1364965
Detailed explanation when gathering build output, the intent is to
ignore expect "errors" like EOF, but an EOF error showed up in the logs;
we could not decipher actual type besides io.EOF that arrived, but we augmented
to log to print the type; next time it arrives, we can add that type to the check list
@bparees PTAL

lgtm [merge]


issue#24-validate

Validate via the frontend that users cannot leave any form fields blank, their password
must be at least 10 characters long, and their rating must be a one-digit integer. An
error message should appear if a user does not conform to any of these requirements.

alrreayd finished in issue#19-login/registration


Reference Sandbox link is broken

Following link is broken https://devnetsandbox.cisco.com/RM/Diagram/Index/a4ab71bc-f7a0-4d63-bedb-05a051818569?diagramType=Topology
Was not able to find any other available SDWAN Always On Sandbox

Link is working now, looks like cisco was doing some updates to that specific resource


Problem ActiveField Init Config.

What steps will reproduce the problem?
Layout config ActiveField
What's expected?
Create config Layout ActiveField
Additional info
ActiveField.php (work)
/**
* {@inheritdoc}
*/
public function init(): void
{
    $layoutConfig = $this->createLayoutConfig();
    $config = ArrayHelper::merge($layoutConfig, ArrayHelper::getObjectVars($this));
    AbstractContainer::configure($this, $config);
}

ActiveField.php (no works)
/**
* {@inheritdoc}
*/
public function init(): void
{
    AbstractContainer::configure($this, $this->createLayoutConfig());
}

possible solution:
/**
 * Constructor for activefield
 */
public function __construct(array $config = [])
{
  if (!empty($config)) {
       	AbstractContainer::configure($this, $this->createLayoutConfig());
  }
}

Q
A

Yii vesion
3.0

PHP version
7.1.23

Operating system
Centos 7

@terabytesoftw it seems that your PR broke tests.
Could you show the code you were using when you found the problem? "Layout config ActiveField" isn't clear enough.


ApiAuthorizationDbContext does not support IdentityRole

From @neoffer on Wednesday, September 18, 2019 7:49:21 AM
ApiAuthorizationDbContext does not support IdentityRole
Using .NET Core 3.0 rc1 with IdentityServer4 and JWT. I couldn't find solution to add IdentityRole or my own ApplicationRole entity to the base class of ApplicationDbContext.
ApiAuthorizationDbContext works but I need to use roles.
Copied from original issue: dotnet/core#3395

@javiercn @HaoK does scaffolding not support this easily?

ApiAuthorizationDbContext is a convenience class to get you started. If you need to customize the DbContext you can do so as follows
public class ApplicationDbContext : IdentityDbContext<ApplicationUser,ApplicationRole>, IPersistedGrantDbContext where TUser : IdentityUser
    {
        private readonly IOptions<OperationalStoreOptions> _operationalStoreOptions;

        public ApiAuthorizationDbContext(
            DbContextOptions options,
            IOptions<OperationalStoreOptions> operationalStoreOptions)
            : base(options)
        {
            _operationalStoreOptions = operationalStoreOptions;
        }

        public DbSet<PersistedGrant> PersistedGrants { get; set; }

        public DbSet<DeviceFlowCodes> DeviceFlowCodes { get; set; }

        Task<int> IPersistedGrantDbContext.SaveChangesAsync() => base.SaveChangesAsync();

        protected override void OnModelCreating(ModelBuilder builder)
        {
            base.OnModelCreating(builder);
            builder.ConfigurePersistedGrantContext(_operationalStoreOptions.Value);
        }
    }


MG-2214 - Add LTREE Path to Groups

What type of PR is this?
This is a feature because it adds LTREE to groups.
What does this do?
This adds ltree to groups
Which issue(s) does this PR fix/relate to?

Related Issue #2214
Resolves #1980

Have you included tests for your changes?
Yes, I have included tests for my changes.
Did you document any new/modified feature?
Yes, I have updated the documentation for the new feature.
Notes
Signed-off-by: WashingtonKK<EMAIL_ADDRESS>

This will be addressed in #2409.


Can't Launch Batch Custom AMI with ParallelCluster

Environment:

AWS ParallelCluster 2.1.0
OS: alinux
Scheduler: awsbatch
Master instance type: t2.micro
Compute instance type: optimal

Bug description and how to reproduce:
I'm trying to create a custom AMI in which I have some software pre-installed for doing weather modeling. I was able to successfully create and launch an RHEL-based AMI using the pcluster CLI.  The problem with that image was that I don't think I had the container agent set up correctly because I could never get my job to run...it just stayed in the "RUNNABLE" phase.  So I figured I'd try to use the Amazon ECS-optimized Linux AMI as my base image and build my computing platform on top of that.  For a test, before installing anything I just tried to launch a batch instance using an AMI from this list: https://docs.aws.amazon.com/AmazonECS/latest/developerguide/ecs-optimized_AMI.html
I continue to get a variation of this error:
Cluster creation failed.  Failed events:

AWS::CloudFormation::Stack AWSBatchStack Resource creation cancelled
AWS::EC2::Instance MasterServer Received FAILURE signal with UniqueId i-0f70fb2c183c37506

Additional context:
Here's my configuration file:
[aws]
aws_region_name = us-east-2
[cluster awswrf]
scheduler = awsbatch
compute_instance_type = optimal
key_name = ########
vpc_settings = public
ebs_settings = awswrf
master_instance_type = t2.micro
#master_root_volume_size = 40
min_vcpus = 0
max_vcpus = 40
desired_vcpus = 4
cluster_type = ondemand
#custom_ami = ami-009973ece6fe45688
custom_ami = ami-0c3da6571b6cfbe9a
[ebs awswrf]
shared_dir = data
ebs_snapshot_id = snap-00fa1f5bc9a7a9490
volume_type = gp2
volume_size = 500
volume_iops = 1500
encrypted = false
#ebs_volume_id = vol-0385e5d9d5f7b280d
[vpc public]
master_subnet_id = subnet-268aac4e
vpc_id = vpc-c2c6f8aa
[global]
update_check = true
sanity_check = true
cluster_template = awswrf
[aliases]
ssh = ssh {CFN_USER}@{MASTER_IP} {ARGS}

Hi Luca
Checking back in on this thread. I haven’t been able to get my custom AMI to work. I would start with the v2.1.0 AMI template, add all of my software, save a new AMI, and then pcluster consistently fails to “create” a compute cluster from that new AMI.  It would be great to try to close this loop on this to figure out if it is possible to do.
I ended up just tarring up all of my software, putting it in an s3 bucket and then copying it over to a new instance spawned from the v2.1.0 AMI template.  There is some upfront work that this requires, but once it’s set up I’ve had reasonable success doing what I want to do.  I’m using a post-install script to automate some of this, although there are a few steps that require manual intervention (like installing the PGI compiler).
My concept of creating a custom AMI with all of my software already loaded is where I’d like to be, rather than the s3 bucket approach because it would avoid having to do some manual configuration before the new instance is ready for live operations. I would like to create a platform-as-a-service custom AMI that I can share with my colleagues. It would be great if I can reduce any barriers to using this AMI, such as avoiding the manual configuration steps.
Any thoughts on things I should be looking for in the pcluster error logs to troubleshoot why my custom AMI wasn’t working?  I recall seeing error messages about the chef command, but I think I see these in the logs even for clusters to that do spin up correctly.
Best,
Zac

On Jan 17, 2019, at 3:52 PM, Luca Carrogu<EMAIL_ADDRESS>wrote:
From your plan it sounds like using a traditional scheduler is the way to go at the moment.
The "build custom AMI" part you already did is good, just a minor comment is that it is better to keep the version of the pcluster package aligned with the instance list you start from. So you should take the AMI from the v2.1.0 list here https://github.com/aws/aws-parallelcluster/blob/v2.1.0/amis.txt instead of the 2.0.2 you used.
Using a traditional scheduler (sge, torque or slurm is up to your choice), the software you installed into the AMI will be available in the Compute Nodes as well, where your computation will take place.
Then the following is just an example on how to run an "helloworld" MPI job with sge:
create a file helloworld.sh with the following content
#!/bin/bash
#$ -cwd
#$ -j y
#$ -pe mpi 32
#$ -S /bin/bash
module load mpi/openmpi-x86_64
mpirun -np 32 hostname
submit the job
$ qsub helloworld.sh
At this point, ParallelCluster will try to spin up the number of instance needed to fit your job requirements. Please make sure that the max_queue_size configuration and instance limits are well sized for your job requirements.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/aws/aws-parallelcluster/issues/829#issuecomment-455345558, or mute the thread https://github.com/notifications/unsubscribe-auth/Asj0CjHboUp9aLhiKWhi8WHauRUVA9rrks5vEPCkgaJpZM4aBlX2.

@zadelman
When you launch the cluster, launch with:
pcluster create new_cluster --norollback

When it fails creation, grab the ip address from the console and ssh in, check the /var/log/cfn-init.log, /var/log/cloud-init-output.log, and /var/log/cloud-init.log.
Post here what errors you find and I can help figure out what's going on. Have you tried using the --createami flag? See https://aws-parallelcluster.readthedocs.io/en/latest/commands.html#createami

Hi Sean
There appears to be an issue with chef, but I’m not sure what the problem is.  I’m attaching the logs here from the cluster that I created using the pcluster create command in which I tried to use an AMI that I created from the console.
I’m now trying to create a new ami now using the pcluster createami command.  Working through some issues on my local machine that was causing this approach to fail (maybe having to do with the wrong version of pip?).
Please let me know if you see anything in these logs that may give me a clue about why my custom AMI won’t launch through pcluster.
Best,

Zac Adelman
Lake Michigan Air Directors Consortium
office:<PHONE_NUMBER>
mobile:<PHONE_NUMBER>
www.ladco.org

On Feb 6, 2019, at 2:46 PM, Sean Smith<EMAIL_ADDRESS>wrote:
@zadelman https://github.com/zadelman
When you launch the cluster, launch with:
pcluster create new_cluster --norollback
When it fails creation, grab the ip address from the console and ssh in, check the /var/log/cfn-init.log, /var/log/cloud-init-output.log, and /var/log/cloud-init.log.
Post here what errors you find and I can help figure out what's going on. Have you tried using the --createami flag? See https://aws-parallelcluster.readthedocs.io/en/latest/commands.html#createami https://aws-parallelcluster.readthedocs.io/en/latest/commands.html#createami
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/aws/aws-parallelcluster/issues/829#issuecomment-461180951, or mute the thread https://github.com/notifications/unsubscribe-auth/Asj0CvX8T4ddyLWOywb_X4BWAhRapUSFks5vKz8rgaJpZM4aBlX2.

Hi @zadelman
we could not see any attached logs, I think because you replied to the ticket via mail.
Can you please post them through the GitHub portal?
tnx

Here are the logs, including a chef-stacktrace that may include some useful information.
cfn-init.log
cloud-init.log
cloud-init-output.log
chef-stacktrace.out.log

Hi @zadelman
I have the suspect that you
1 - created a cluster parallelcluster-ladcowrf
2 - customized the running master server instance and created a new AMI from this
3 - used that AMI to create parallelcluster-ladcowrf2
If this is what happened, it's not the correct way to create your custom AMI. The AMI you want to customize needs to be launched independently from ParallelCluster (e.g. from the EC2 console).
The procedure is described here Modify an AWS ParallelCluster AMI

Hello everyone,
My two cents:
I also run MPI-based solvers and installing my software can take up to several hours so a customized AMI was also of interest to me. I have been tinkering with it for the last two days and this is what has worked and what hasn't:
i) You must follow the instructions linked by @lukeseawalker thoroughly. I started with an original AMI from a different region and got the same error as @zadelman from chef.
ii) I tried CentOS (personal preference) and although the previous error was fixed, the cloud-init cheft script still failed with a different error message (logs are attached below).
iii) I switched to ALinux and the cloud-init cheft script finally seemed to go through (the logs showed a successfull completion) but the deployment still failed with the following error message: "-AWS::Autoscaling::AutoscalingGroup ComputeFleet Received 1 FAILURE signal(s) out of 1.  Unable to satisfy 100% MinSuccessfullInstancesPercent requirement." The information that I was able to find refers to CloudFormation and suggests modifying the JSON file, particularly the MetaData stuff. However, I'm not sure if fixing this issue would require some extra inputs in the ParallelCluster script (I tried maintain_initial_size with true and false but it didn't make any difference).
My sample script for a small cluster reads:
[aws]
aws_region_name = us-west-2
aws_access_key_id = XXXX
aws_secret_access_key = XXXX
[cluster odycluster]
vpc_settings = odyvpc
placement_group = DYNAMIC
placement = compute
key_name = XXXX
master_instance_type = t2.micro
compute_instance_type = m5a.large
initial_queue_size = 2
max_queue_size = 2
maintain_initial_size = false
cluster_type = ondemand
extra_json = { "cfncluster" : { "cfn_scheduler_slots" : "cores" } }
scheduler = sge
base_os = alinux
custom_ami = ami-0de4698be5f2698c8
[vpc odyvpc]
master_subnet_id = subnet-XXXX
vpc_id = vpc-XXXX
vpc_security_group_id = sg-XXXX
[global]
update_check = true
sanity_check = true
cluster_template = odycluster
[aliases]
ssh = ssh {CFN_USER}@{MASTER_IP} {ARGS}
Thanks,
boot.log
cfn-init.log
cfn-init-cmd.log
cfn-wire.log
cloud-init.log
yum.log

Hi @afernandezody,
from the logs I see that you are using ParallelCluster 2.1.0, but the AMI you are using has been created from the version 2.0.2.
To create your custom AMI you need to start from the released AMI matching your ParallelCluster version, so you must pick up the AMI from this list https://github.com/aws/aws-parallelcluster/blob/v2.1.0/amis.txt
In your case for Centos7, start from

us-west-2: ami-03abc3d448a987e73

for alinux start from

us-west-2: ami-0bdc62a7fa9026972

Hi @lukeseawalker,
Thank you for your quick reply. I went back and made sure that the original AMI version agrees with that of ParallelCluster. When this is not the situation, chef fails. However, the deployment is still falling through even when the versions agree with the message "-AWS::Autoscaling::AutoscalingGroup ComputeFleet Received 1 FAILURE signal(s) out of 1. Unable to satisfy 100% MinSuccessfullInstancesPercent requirement." The logs (attached below) indicate that chef has completed successfully, and that leads me to believe that something else is missing.
cfn-init.log
cfn-init-cmd.log
cfn-wire.log
cloud-init.log
cloud-init-output.log
supervisord.log

The issue seems to be with the Compute nodes. What would really help here to identify the root cause are the logs pulled from the compute instance. Could you try to create the cluster with the --norollback option, log into the compute node (from the master) and pull the logs from there?
Thank you!

Hi @demartinofra,
I can't provide that info because I cannot ssh from the master to the compute nodes. Apparently the cluster creation has not finished and when sshing, I get the known
[centos@ip-172-31-11-40 ~]$ ssh <IP_ADDRESS>
The authenticity of host '<IP_ADDRESS> (<IP_ADDRESS>)' can't be established.
ECDSA key fingerprint is SHA256:3bMzUh4bMLRHm2Cr05GI2glu0Qnrwp9++hKk/P+Hipw.
ECDSA key fingerprint is MD5:af:20:62:8f:d0:56:e4:95:ec:43:bd:ea:2b:42:dc:22.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Permission denied (publickey,gssapi-keyex,gssapi-with-mic)._
The command qhost doesn't show any compute node either.
Thank you.
P.S. I upgraded everything (pcluster + AMIs) to 2.2.1 and that didn't make any difference.

I think it is just failing to ssh because you either need to forward the agent and add the key to the agent when sshing into the master or copy the ssh key to the master node and use that explicitly when sshing into the compute (with the -i option).

Hi @demartinofra,
Sorry for the delay. I had some trouble sshing from the master node but was able to ssh directly to the compute node through the public IPv4 (which I'm not sure if the configuration aims to do). Anyhow, the logs are attached. I'm unfamiliar with chef so there is only so much I can interpret from the logs. It read like the problem is either with the mounting process or some metadata. I tried the same process but erased everything from the shared subdirectory (just in case this was interfering with the mounting thing) and the only modifications vs the original AMI reside in the /opt subdirectory, but it didn't work either and got the same error message.
Thanks.
boot.log
cfn-init.log
cfn-init-cmd.log
cfn-wire.log
cloud-init.log

Hello again,
I was able to isolate and trace back the issue to the specification of the 'vpc_security_group_id=sg-XXXX' (same one used to create the custom AMI) in the script. This seems to somehow conflict with chef. I'm not sure about the details but will update my post as I figure out what works and what doesn't.

Hi @afernandezody ,
thank you for your analysis and for the logs.
The error in the cfn-init.log is:
---- Begin output of mount -t nfs -o hard,intr,noatime,vers=3,_netdev ip-172-31-12-97.us-west-2.compute.internal:/home /home ----
STDOUT:
STDERR: mount.nfs: Connection timed out

It means the compute node is not able to mount the nfs exported by the master.
This could depend by your vpc_security_group_id=sg-XXXX setting since the compute nodes must be able to access to the port 2049 of the master node.
Please let us know if it helps.

Hi Enrico,
I haven't looked into this issue for the last month plus, but I'll check what you're suggesting next week. Thanks.

Hello Enrico,
I was able to circle back to this matter but I'm still running into unanticipated problems. I double checked that every item & version of the customized AMI matches with ParallelCluster (the exception being the vpc_security_group_id that I still need to leave blank or the deployment fails). The deployment finishes and everything seems in place (e.g. software, $PATH). However, SGE is now not functioning properly when I submit MPI jobs (e.g. qsub -pe mpi 4 test_Par.sh) as it starts a single MPI rank. I've been using the same procedure for the last ten weeks and was working fine (when I did the software installation after cluster deployment). The problem with the latter procedure is obviously that it's more time consuming and expensive, which is what started the whole thread. Any hint on what I'm missing or could be wrong? Thanks.


Stable release v0.18.0

Feat: Generate EnvoyConfigs thorugh saas-operator (instead og managing them externally to operator) #240
Fix: Randomization in parallel ginkgo test runs (but bug still present, coincidence on supposed random namespace names resulting on test pipeline failing on GitHub Action) #242
Feat: Refactor EnvoyConfig generation API so we can benefit from strategic merge patches at kustomize level #243
Feat: Add sphinx batch size field to System CR #244
Cleanup: Remove not needed redis ENV VARs from System Sphinx statefulset #245
Fix: Add System-config secret hash trigger #246

/kind release
/priority important-soon
/assign

/retitle Stable release v0.18.0

/approve