Step value for solar pv parks is extremely small

One click on plus increases the amount of solar pv parks from 0.3000 to with 0.3001, and I would like to go to 0.6. 😒

Perhaps a good idea to rethink the slider caps, and steps for scaled scenarios as a whole? 😃

I made it so that the (-) and (+) buttons move the slider by at least 1% of its value each time they're clicked. You can always click the slider value at set the value manually if you have something specific in mind.

Perhaps a good idea to rethink the slider caps, and steps for scaled scenarios as a whole? 😃

:+1:
I'm not a fan of the very small numbers we often see for inputs in scaled scenarios, but I'm not sure what the alternative is... New ideas are welcome!

New ideas are welcome!

Perhaps we can make the step-size relative to the range? So, if a slider ranges from 0 to 2, a step-size of 100 would imply steps of 0.02. This might be more robust to scaled scenario's.


Fixed bolt.guide not working for Lesson 6 using this repo

When working through bolt.guide using this cloned repo as my project directory i noticed that Lesson 6 failed.
When running bolt task show package i was getting the following:
$ bolt task show package

package

USAGE:
bolt task run --targets <node-name> package

MODULE:
/home/nmaludy/tmp/bolt/docs/_includes/lesson1-10/Boltdir/modules/package

When trying to run the task as described in the guide i was getting the following error:
$ bolt task run package action=status name=bash --nodes node1
Deprecation Warning: The --nodes command line option has been deprecated in favor of --targets.
Started on node1...
Failed on node1:
  The task failed with exit code 1
  {
    "status": "failure",
    "error": "Title or name must be provided"
  }
Failed on 1 node: node1
Ran on 1 node in 3.41 sec

Turns out that the package module was included in Boltdir/modules and was messing things up now that Bolt includes the package module internally.
This PR deletes those unnecessary modules so that people following the guide can have a bug free experience!

CLA signed by all contributors.


UI style change

Merging all the UI changes (currently visible at http://step129-2020.appspot.com/)

@googlebot I consent.

@googlebot I consent.


add a command to switch betwen environments

In the future after the UI is developed, I would like to make this show up somewhere for the user to see what account they are using.


Unstable reference from /drive in JupyterLite

Description

Unstable reference from /drive in JupyterLite

Sometimes /drive refers to JupyterLite Storage
Sometimes /drive refers to Local Files (Certain bytes are incorrectly replaced to EFBFBD in this mode)

Is the Settings Storage setting in JupyterLite involved?
If it is related, please write it in the README.
Reproduce

Expected behavior

Some stable behavior
Context

JupyterLite version: 0.1.0-beta.17
jupyterlab-filesystem-access version: v0.5.3
Browser and its version: Chrome 108.0.5359.124 (Official Build) (x86_64)

There is no /drive in JupyterLab.
All file access is server-side.

In a notebook in jupyterlab-filesystem-access where /drive is sometimes empty.
In that case, reading /drive in the notebook in JupyterLite Storage will fix it.

It seems that the location of the notebook that reads /drive affects the behavior.

Notebook is directly in the top directory: /drive refers to JupyterLite Storage
Notebook is in some subdirectory: /drive refers to Local Files

This may be a bug in JupyterLite.
jupyterlite/issues/942

JupyterLite fixed


pull request for e2e test at 01/05/2017 21:45:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Create new OpenEBS version 3.3.0

What this PR does / why we need it:
Create new OpenEBS version 3.3.0
Workaround chart issue installing version 3.3.0.
Which issue(s) this PR fixes:

Fixes NONE
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Adds new [OpenEBS add-on](https://kurl.sh/docs/add-ons/openebs) version 3.3.0.

Does this PR require documentation?

NONE

[sc-56120]


Typo

2048 is pretty cool.

idk why you deleted this but thanks ig

idk why you deleted this but thanks ig

I deleted my fork, didn't know that would close the pull request.

Kinda weird tbh, don't know why it's necessary to close the PR if a patch/diff is still saved for my viewing (/don't think it is necessary)


Context is none even if provided

Change Summary
This PR contains a test that replicates a bug. In summary, when you try to provide context to model_validate it seems like no context is available in the field validator even though it was provided.
Related issue number
No issue has been created to track this, but I'm happy to create an issue as well.
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Hi @jonathan-s,
This is the expected behavior - the validation doesn't run for your model_validate call without the revalidate_instances config setting:
from pydantic import BaseModel, ConfigDict, ValidationInfo, field_validator

def get_context():
    return {'check_title': lambda x: x.istitle()}

class User(BaseModel):
    first_name: str

    @field_validator('first_name', check_fields=False)
    @classmethod
    def validate_choice(cls, v: str, info: ValidationInfo):
        if not info.context:
            return v

        check_title = info.context.get('check_title')
        if check_title and (check_title(v) is False):
            raise ValueError('First name needs to be a title')
        return v

    model_config = ConfigDict(revalidate_instances='always')

user = User(first_name="hello")
print(User.model_validate(user, context=get_context()))
"""
first_name
  Value error, First name needs to be a title [type=value_error, input_value='hello', input_type=str]
    For further information visit https://errors.pydantic.dev/2.8/v/value_error
"""

When you enable that and fix up the conditional, things work as expected :).

Hi @sydney-runkle Thanks for the quick turn around. Then it seems like there is a bug in the documentation. Because that behavior is not mentioned together with the validation context here, I think it could be useful to give it a mention there.

https://docs.pydantic.dev/latest/concepts/validators/#dataclass-validators


Update close-issue.yml

Related Issue
Fixes #226
This version of autocomment on closing an issue won't require any permission grant from workflow settings

@Durgesh-Vaigandla please review this PR and merge it if it is good to go

@Durgesh-Vaigandla can you please add the 'gssoc' and level labels


safe.duckduckgo.com IP address has changed?

Searching with duckduckgo didn't seem to work anymore for me.  I checked the NSLookup for safe.duckduckgo.com and the IP is different than the one in the script.  I changed it in the script and re-ran the script. It is now working.  Thanks for this awesome script!
CNAME support is coming up soon in the next releases of v.5 so it might be good to handle duck duck in the GUI then and only use the script for Google since their are so many domains for it.
https://discourse.pi-hole.net/t/custom-redirects/37272/18

Thank you for the feedback! I will make sure to investigate and update the
script accordingly.
Jayke Peters
<EMAIL_ADDRESS>+1<PHONE_NUMBER>
On Fri, Sep 11, 2020 at 4:19 PM trigdog<EMAIL_ADDRESS>wrote:

Searching with duckduckgo didn't seem to work anymore for me. I checked
the NSLookup for safe.duckduckgo.com and the IP is different than the one
in the script. I changed it in the script and re-ran the script. It is now
working. Thanks for this awesome script!
CNAME support is coming up soon in the next releases of v.5 so it might be
good to handle duck duck in the GUI then and only use the script for Google
since their are so many domains for it.
https://discourse.pi-hole.net/t/custom-redirects/37272/18
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jaykepeters/PSS/issues/13, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJBRUUQ2INMIBVYLJREP4R3SFKH7XANCNFSM4RILKKKQ
.

I recommend using
address=/duckduckgo.com/safe.duckduckgo.com
specifically for domains that need safe.duckduckgo.com
because the IP addresses associated with this domain very by geolocation and how long safe.duckduckgo.com decides to reserve a specific IP address.

I recommend using
address=/duckduckgo.com/safe.duckduckgo.com
specifically for domains that need safe.duckduckgo.com
because the IP addresses associated with this domain very by geolocation and how long safe.duckduckgo.com decides to reserve a specific IP address.

What is the IP address for safe search?  I just ran the script and it's being blocked


Handle screen rotation?

Hello,
Is there a way to reinitialize the control in the event of a screen rotation? When I rotate the screen the selection bubble and shadow become misplaced relative to the correct item. Neither ReloadData() and setNeedsDisplay() is redrawing the control. Selecting the item with a touch correctly resets the control, but I need this to happen automatically. Any ideas?

@ibuprofane Unfortunately, current implementation doesn't properly support dynamic changes of size, though it's expected, marked as a bug.


Integration test for heatmap selector

I need to go back and look at the 4 failing tests which could be due to changes to master and the tests haven't yet been updated. And I'll do another round of refactoring to make the tests more succinct.


MicrosoftWebDriver - Implement version checking and download of latest binary.

Hi @titusfortner,
I have updated the MSWebdriver implementation.  Earlier it would raise an error asking the user to manually download the binary. With this PR it now queries the current Edge version and downloads the corresponding binary from the Microsoft developer website if an existing binary does not exist.
Caveat

Unfortunately, because MicrosoftWebDriver.exe does not provide a --version option, there is no way to verify if an existing binary is the correct version or not.

USAGE:
        MicrosoftWebDriver [OPTIONS]

OPTIONS:
        --host=<HostName>    Host IP to use for the WebDriver server (default: localhost)
        --port=<PortNumber>  Port to use for the WebDriver server (default: 17556)
        --package=<Package>  ApplicationUserModelId (AUMID) for the application to be launched by the WebDriver server
        --verbose            Outputs requests received and responses sent by the WebDriver server
        --silent             Outputs nothing

The implementation simply assumes the existing binary is the expected version and returns the location. If the existing binary is incompatible, the user will see the following from Selenium:
unable to connect to MicrosoftWebDriver localhost:17556

If we proceed with this, we'll have to advise the user to delete the existing binary and retry.

The actual download page for the binary has a redirect on it. This kicks in the Net::HTTPRedirection case in get() on lib/webdrivers/common.rb:80 and uses the location attribute. The location returned in the response is an invalid URL pointing to /en-us/. To workaround this, I have left the base_url as is and added download_page to return the actual download page URL.

Let me know what you think of this implementation. I will push the updated specs soon and squash all commits before the final approval.

Argh... I messed up a rebase. Will submit another PR with changes discussed here.

Argh... I pushed a messed up rebase and GitHub closed it. I am opening a new PR with discussed changes to avoid this mess.


Typescript: Impossible to validate solution

Always:
Server Execution Error:
Request Timeout: The server timed out running the request. This may be due to an issue with the backend server. Please try again.

You could try lots of time with the same error.

There is no information here for me to really act on, but I just closed #656 which seems to be related.

Thanks! It works.


Helpers do not work with kitchen test as expected

I'm not proficient in Ruby, but It appears that the method being used in this fallback handling (vault?) raises an exception that isn't handled in this helper, so instead of ever going to the "databag_falback" case it instead just hard stops and errors. The chef output in my kitchen test shows that it is resolving "chef-vault (4.0.3)".
https://github.com/chef/chef-vault/blob/master/lib/chef-vault/item.rb#L320
================================================================================
Recipe Compile Error in /tmp/kitchen/cache/cookbooks/cookbook/recipes/default.rb
================================================================================

Net::HTTPServerException
------------------------
404 "Not Found"

Cookbook Trace:
---------------
/tmp/kitchen/cache/cookbooks/chef-vault/libraries/helpers.rb:37:in `chef_vault_item'
/tmp/kitchen/cache/cookbooks/cookbook/recipes/configuration.rb:36:in `block in
from_file'
/tmp/kitchen/cache/cookbooks/cookbook/recipes/configuration.rb:32:in `from_fil
e'
/tmp/kitchen/cache/cookbooks/cookbook/recipes/default.rb:13:in `from_file'

Relevant File Content:
----------------------
/tmp/kitchen/cache/cookbooks/chef-vault/libraries/helpers.rb:

30:    # +node['chef-vault']['databag_fallback']+.
31:    # @example
32:    # item = chef_vault_item('secrets', 'bacon')
33:    # log 'Yeah buddy!' if item['_default']['type']
34:    # @param [String] bag Name of the data bag to load from.
35:    # @param [String] id Identifier of the data bag item to load.
36:    def chef_vault_item(bag, id)
37>>     if ChefVault::Item.vault?(bag, id)
38:        ChefVault::Item.load(bag, id)
39:      elsif node['chef-vault']['databag_fallback']
40:        data_bag_item(bag, id)
41:      else
42:        raise "Trying to load a regular data bag item #{id} from #{bag}, and databag_fallback is disabl
ed"
43:      end
44:    end
45:
46:    # Helper method that allows for listing the ids of a vault in a recipe.

System Info:
------------
chef_version=15.10.12
platform=centos
platform_version=7.8.2003
ruby=ruby 2.6.6p146 (2020-03-31 revision 67876) [x86_64-linux]
program_name=/opt/chef/bin/chef-client
executable=/opt/chef/bin/chef-client

Escalating from a comment in another issue to its own issue as it still presents on new chef versions (initially wondered if it was relevant to being on such an old release). https://github.com/chef-cookbooks/chef-vault/issues/64#issuecomment-651356201

I would also be willing to assist in any dev that may or may not be required to fix this, assuming it isn't user error 👍

+1
       ================================================================================
       Recipe Compile Error in /tmp/kitchen/cache/cookbooks/tmg-splunk-client/recipes/default.rb
       ================================================================================

       ChefVault::Exceptions::ItemNotFound
       -----------------------------------
       vault/splunk__default not found

       Cookbook Trace: (most recent call first)
       ----------------------------------------
         /tmp/kitchen/cache/cookbooks/chef-vault/libraries/helpers.rb:37:in `chef_vault_item'
         /tmp/kitchen/cache/cookbooks/chef-splunk/recipes/default.rb:22:in `from_file'
         /tmp/kitchen/cache/cookbooks/tmg-splunk-client/recipes/default.rb:10:in `from_file'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/chef-vault/libraries/helpers.rb:

        30:    # +node['chef-vault']['databag_fallback']+.
        31:    # @example
        32:    # item = chef_vault_item('secrets', 'bacon')
        33:    # log 'Yeah buddy!' if item['_default']['type']
        34:    # @param [String] bag Name of the data bag to load from.
        35:    # @param [String] id Identifier of the data bag item to load.
        36:    def chef_vault_item(bag, id)
        37>>     if ChefVault::Item.vault?(bag, id)
        38:        ChefVault::Item.load(bag, id)
        39:      elsif node['chef-vault']['databag_fallback']
        40:        data_bag_item(bag, id)
        41:      else
        42:        raise "Trying to load a regular data bag item #{id} from #{bag}, and databag_fallback is disabled"
        43:      end
        44:    end
        45:
        46:    # Helper method that allows for listing the ids of a vault in a recipe.

       System Info:
       ------------
       chef_version=16.9.20
       platform=amazon
       platform_version=2
       ruby=ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-linux]
       program_name=/opt/chef/bin/chef-client
       executable=/opt/chef/bin/chef-client

       Running handlers:
       [2021-01-20T16:28:32+00:00] ERROR: Running exception handlers
       Running handlers complete
       [2021-01-20T16:28:32+00:00] ERROR: Exception handlers complete
       Chef Infra Client failed. 0 resources updated in 01 seconds
       [2021-01-20T16:28:32+00:00] FATAL: Stacktrace dumped to /tmp/kitchen/cache/chef-stacktrace.out
       [2021-01-20T16:28:32+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
       [2021-01-20T16:28:32+00:00] FATAL: ChefVault::Exceptions::ItemNotFound: vault/splunk__default not found

+1
       ================================================================================
       Recipe Compile Error in /tmp/kitchen/cache/cookbooks/tmg-splunk-client/recipes/default.rb
       ================================================================================

       ChefVault::Exceptions::ItemNotFound
       -----------------------------------
       vault/splunk__default not found

       Cookbook Trace: (most recent call first)
       ----------------------------------------
         /tmp/kitchen/cache/cookbooks/chef-vault/libraries/helpers.rb:37:in `chef_vault_item'
         /tmp/kitchen/cache/cookbooks/chef-splunk/recipes/default.rb:22:in `from_file'
         /tmp/kitchen/cache/cookbooks/tmg-splunk-client/recipes/default.rb:10:in `from_file'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/chef-vault/libraries/helpers.rb:

        30:    # +node['chef-vault']['databag_fallback']+.
        31:    # @example
        32:    # item = chef_vault_item('secrets', 'bacon')
        33:    # log 'Yeah buddy!' if item['_default']['type']
        34:    # @param [String] bag Name of the data bag to load from.
        35:    # @param [String] id Identifier of the data bag item to load.
        36:    def chef_vault_item(bag, id)
        37>>     if ChefVault::Item.vault?(bag, id)
        38:        ChefVault::Item.load(bag, id)
        39:      elsif node['chef-vault']['databag_fallback']
        40:        data_bag_item(bag, id)
        41:      else
        42:        raise "Trying to load a regular data bag item #{id} from #{bag}, and databag_fallback is disabled"
        43:      end
        44:    end
        45:
        46:    # Helper method that allows for listing the ids of a vault in a recipe.

       System Info:
       ------------
       chef_version=16.9.20
       platform=amazon
       platform_version=2
       ruby=ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-linux]
       program_name=/opt/chef/bin/chef-client
       executable=/opt/chef/bin/chef-client

       Running handlers:
       [2021-01-20T16:28:32+00:00] ERROR: Running exception handlers
       Running handlers complete
       [2021-01-20T16:28:32+00:00] ERROR: Exception handlers complete
       Chef Infra Client failed. 0 resources updated in 01 seconds
       [2021-01-20T16:28:32+00:00] FATAL: Stacktrace dumped to /tmp/kitchen/cache/chef-stacktrace.out
       [2021-01-20T16:28:32+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
       [2021-01-20T16:28:32+00:00] FATAL: ChefVault::Exceptions::ItemNotFound: vault/splunk__default not found

If somebody is googling this.
Fixed by adding the line to the kitchen.yml provisioner section:
provisioner:
  data_bags_path: test/fixtures/data_bags

and placing the data_bag to test/fixtures/data_bags/vault/splunk__default.json

If somebody is googling this.
Fixed by adding the line to the kitchen.yml provisioner section:
provisioner:
  data_bags_path: test/fixtures/data_bags

and placing the data_bag to test/fixtures/data_bags/vault/splunk__default.json


Wip#11

Added 'about' pages to Netlify CMS as suggested in #11

Some extra work is described in #14

@andreasbalevik I think that we should keep on using frontmatter for ToC (toc: true) instead of requiring adding a shortcode. I assume Netlify supports editing frontmatter?

@andreasbalevik I think that we should keep on using frontmatter for ToC (toc: true) instead of requiring adding a shortcode. I assume Netlify supports editing frontmatter?

Ok, added boolean for activating toc in Netlify CMS if needed. Updates value in front-matter.


withGalio() does not forward refs to wrapped components

Describe the bug
Ref for a component passed to the withGalio higher-order component does not get forwarded to the wrapped component but rather references the outermost container component.
To Reproduce
Steps to reproduce the behavior:
import React from "react";
import { TextInput, StyleSheet } from "react-native";
import { withGalio, Block, Button, theme } from "galio-framework";

const Input = React.forwardRef((props, ref) => {
  const { theme, ...rest } = props;
  return (
    <TextInput
      {...{ ref, ...rest }}
      placeholder="name"
      style={{
        width: 250,
        height: 45,
        padding: 10,
        borderColor: theme.COLORS.MUTED,
        borderWidth: 1,
        marginRight: 10
      }}
    />
  );
});

const MyInput = withGalio(Input);

export default function App() {
  const inputRef = React.useRef(null);

  const handlePress = () => {
    if (inputRef.current) {
      inputRef.current.clear();
    }
  };

  return (
    <Block flex middle>
      <Block row style={styles.wrapper}>
        <MyInput ref={inputRef} />
        <Button onPress={handlePress} style={{ width: 100 }}>
          clear
        </Button>
      </Block>
    </Block>
  );
}

const styles = StyleSheet.create({
  wrapper: {
    height: 50
  }
});

It throws an error
inputRef.current.clear is not a function. (In 'inputRef.current.clear()', inputRef.current.clear is undefined)
Expected behavior
The TextInput value should clear onPress of the button.
Screenshots

Smartphone (please complete the following information):

Device: iPhone 11 Pro Max
Type of device: ios simulator
OS: iOS 13.3
React-Native version: 0.59
Expo version: 35.0.0

Additional context

i'll have to look into this, if someone has any idea how to solve this faster please do help out

i will open a PR to fix it...

Closed because we've merged the PR into the dev branch. Thank you!


10x'd test iterations

Making performance tests longer to try and avoid large swings in performance results

@vapor-bot test performance

@vapor-bot test performance

@vapor-bot test performance

@twof maybe we should shoot for tests to take around ~1s? That way running each perf test should take ~8s (since it runs each one 8 times).
If so, these ones are still quite short and could still be inconsistent:

testMinimalRouterCaseInsensitivePerformance
testMinimalRouterCaseSensitivePerformance
testMinimalEarlyFailPerformance

Moving the rest to around 500_000 would get them close to the 1s mark.

Hey @twof, you just merged a pull request, have a coin!
You now have 64 coins.

Shouldn't have the performance function call been updated?


Unable to Download Video from Facebook

masbog$ youtube-dl https://www.facebook.com/willyfoo/videos/vb.96738939244/10152791075154245 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://www.facebook.com/willyfoo/videos/vb.96738939244/10152791075154245', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.10.18
[debug] Python version 2.7.10 - Darwin-15.0.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[facebook]<PHONE_NUMBER>5154245: Downloading webpage
ERROR: An extractor error has occured. (caused by KeyError(0,)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 291, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/facebook.py", line 145, in _real_extract
video_data = params['video_data'][0]
KeyError: 0
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 661, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 297, in extract
raise ExtractorError('An extractor error has occured.', cause=e)
ExtractorError: An extractor error has occured. (caused by KeyError(0,)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Facebook is fixed but not included in the latest release yet. Please wait patiently for the next version.

@yan12125 okey... thank you very much... 👍


Ludo/second order ik

This is the integration of the second order inverse kinematics implemented by @righetti. All the parameters of the new inverse kinematics are added to the config files available in the repo. Since the parameters of the second order IK have not been tuned for the demos, the first order IK is the default formulation used. It is explained in the README.md, how to change the formulation to the second order by changing a boolean in the config files...

@MaximilienNaveau @jviereck Thanks for the comments... I applied all of them, let me know if there is any more comments before merge...

Thanks for the work @majidkhadiv and @righetti !


[BUG] Even if nothing is selected, the plugin still links the new note

It's as the title says, when I have a note open in edit mode, but with nothing selected, then run the plugin, it'll link the newly created note where my cursor was.
From what I understand, it should only do so when I had something selected and wanted to make a new note of it.
I'd like to keep the toggle in the settings on, for cases where I select stuff to be in the new template - maybe make separate options for this? One to decide if there should be a link regardless of the selection, and one if there should be a link when a selection was used.
PS: I've been playing around with your plugin a lot, it really has potential, that's why I'm opening quite a few issues ^^'

I just noticed that the commands don't work when in Preview mode! Since it's somewhat related to the original issue, I'm just putting it here.
Is there any specific reason for this? It's not very convenient to work with unfortunately.

Agreed - this should be more sensible, and configurable! This has been bugging me too. I like the suggestion of different actions when text is selected or not.

I don't know about Preview mode, though - I didn't do anything special around it, so maybe it doesn't work as a normal editor. Something to look at!

New config should be working properly now template-should-replace can be set to always, never or sometimes (i.e. when text is selected)


Switch to chatID-agnostic chat/models API

Cody part: https://github.com/sourcegraph/cody/pull/3588
Test plan

LLM selection works
Chat restore works (proper llm is selected and used)

LGTM too.
The bit about the codyProOnly = false when restoring the chat session I don't get... I realize you're just moving this in this PR, but it seems dodgy. Likewise default = it.model == null ... model = it.model ?: "" What happens when these restored chats are updated? The round trip will change model from null to "" and default to false etc.

@dominiccooney, after you set a chat and send a first human message. It is fixed afterwards. To do the restore all we need is the model (ChatModelProvider::model). The rest of the fields are either redundant (codyProOnly) or needed for UI only (like title and provider - it's used only in the presentation).
I agree this part of code is not perfect - I will rethink it and discuss with Peter.


items disappeared

I tried putting a item into the output slot and after a few attempts, it disappeared.
(1.20.1server and 1.19client)

Added with version 1.2


Preferences window keeps opening at startup

how do i stop karabiner elements preferences window opening at startup? i want KE to run at startup, but i don't want the preferences window to open every time i startup.
macOS 10.12.6
Karabiner Elements 11.1.0

#1068

got it. thanks


[XTypeRecoveryPass] Follow-Up Discovered Calls with [Type|Method] Stubs

Is your feature request related to a problem? Please describe.
XTypeRecoveryPass will often find method invocations that were not immediately clear by the time the initial MethodStubPass runs. This will lead to no methods to hold a CALL edge and potentially miss information one can achieve from the .callee step.
Describe the solution you'd like
To leverage the code in MethodStubPass to generate additional stubs for newly discovered calls that need to be linked in the XTypeHintCallLinker.
Describe alternatives you've considered
MethodStubPass could likely have a few functions moved to an object class whereby a builder is an argument.
Additional context
N/A

Partially fixed by #2232

Closed by #2282


EventHandler: refactor, comment and optimize

To follow up on: https://github.com/playcanvas/engine/pull/5484
Slightly improved EventHandler, by moving from Object to Map for callbacks.
Also added comments to "Active Callback" optimizations to explain it better.
And even found a small optimization which reduced occurrences of unoptimized path when cloning callbacks list. Previously it cloned callbacks list if it there was any active callbacks in place. But now it only does it if previously active callbacks list were modified during its execution.
Behavior is preserved.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Why do we use a mixture of Array.from and .slice() now? Can't we just go for the shorter version, aka keeping .slice()?

Since we are already refactoring...
    /**
     * Reinitialize the event handler.
     *
     * @private
     */
    initEventHandler() {
        this._callbacks = new Map();
        this._callbackActive = new Map();
    }

This method is not private, it has to be called throughout the rest of the engine. This will also fix additional type errors:

You also didn't fix this:
https://github.com/playcanvas/engine/blob/6cc5ebce1984a6237b0e556c58b720cb1ff63820/src/core/events.js#L21-L24
It has to be:
        // ...
        target.hasEvent = ev.hasEvent;
        EventHandler.prototype.initEventHandler.apply(target);
        return target;

@private means that you can refactor the code without a 2nd thought, but this is the 2nd PR with the same mistake, so we shouldn't "over-privatize" what should be public and of course refrain from using private stuff in the rest of the engine, because it's causing refactor bugs that may be hard to find later on.
I guess we can excuse the unit tests for being tests that check internal states, but not normal use-cases from within the engine itself.

Hmm, presumably initEventHandler should be @ignore instead.

Updated PR based on recent comments. Used @ignore (Thanks @willeastcott), and used consistently Array.from - as it is a modern way to clone an array.

and used consistently Array.from - as it is a modern way to clone an array.

I'm not really convinced, if it's only about modern syntax, you could have picked the shorter [...arr] version aswell, which seems to be slightly faster aswell.
I guess the main issue is performance, both [...arr] and Array.from(...) are using the iterator protocol, which makes them rather slow:
https://jsben.ch/IjJQG

Because they have to deal with cases like:
class Vec3 {
    x = 1;
    y = 2;
    z = 3;
    *[ Symbol.iterator ]() {
		yield this.x;
		yield this.y;
		yield this.z;
	}
}
vec3 = new Vec3;
Array.from(vec3); // Result: [1, 2, 3]

Whereas Array.prototype.slice knows exactly what to do - because it's the very own designated method for doing this job.

@willeastcott Yep, PR is looking very good, great job @Maksims :1st_place_medal:


Title: Add styling to scrollbar

Is your feature request related to a problem? Please describe.
The scrollbar is plain vanilla and does not match with the theme of the website.
Describe the solution you'd like.
I'd like to add proper styling to the scrollbar that matches with the color-coding of the website and makes it visually appealing.
Describe alternatives you've considered.
-none-
Add any other context or screenshots about the feature request here.
Hence, I request you to assign this issue to me as GSSoC 2023 Contributor.
Thank You :)

Please assign me this task #129
as GSSOC contributer

Sure @harshita-kaushik2

Make sure to add for the dark mode


Document dependencies

I had to install a few dependencies on Ubuntu in order for the compiled binary to work. Not sure where the best place in the readme is, or else I'd make this a PR.
Dependencies
img depends on newuidmap and execv. To install these on Ubuntu run sudo apt install setpriv uidmap

You want to put it by the other ones? :) there is a spot
On Tue, May 8, 2018 at 9:03 AM Daniel Schep<EMAIL_ADDRESS>wrote:

I had to install a few dependencies on Ubuntu in order for the compiled
binary to work. Not sure where the best place in the readme is, or else I'd
make this a PR.
Dependencies
img depends on newuidmap and execv. To install these on Ubuntu run sudo
apt install setpriv uidmap
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/genuinetools/img/issues/82, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbPsS5tsec9UvCm__MTH8JI7A6muGks5twZefgaJpZM4T2mml
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

Right here: https://github.com/genuinetools/img#installation
On Tue, May 8, 2018 at 9:04 AM Jessie Frazelle<EMAIL_ADDRESS>wrote:

You want to put it by the other ones? :) there is a spot
On Tue, May 8, 2018 at 9:03 AM Daniel Schep<EMAIL_ADDRESS>wrote:

I had to install a few dependencies on Ubuntu in order for the compiled
binary to work. Not sure where the best place in the readme is, or else I'd
make this a PR.
Dependencies
img depends on newuidmap and execv. To install these on Ubuntu run sudo
apt install setpriv uidmap
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/genuinetools/img/issues/82, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbPsS5tsec9UvCm__MTH8JI7A6muGks5twZefgaJpZM4T2mml
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu
http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

:+1:


test: [DI-21337] - Added Cypress tests to verify ACLP UI's handling of API failures

Added Cypress tests to verify ACLP UI's handling of API failure
Description 📝
This pull request focuses on validating the error-handling mechanisms for various API endpoints in the system. The following use cases are being validated to ensure appropriate error messages are displayed when API failures occur:
Changes  🔄
As part of this PR, we are validating the following use cases:
Handling error for the Service Types API with the message: "Failed to fetch service types."
Handling error for the Dashboards API with the message: "Failed to fetch dashboards for Linode, DBaaS, etc."
Handling error for the Regions API with the message: "Failed to fetch regions."
Handling error for the Resources/Clusters API with the message: "Failed to fetch resources" or "Failed to fetch database clusters."
Handling error for the Dashboard by ID API with the message: "Failed to fetch the selected dashboard."
Jira ticket : https://track.akamai.com/jira/browse/DI-21337
Target release date 🗓
12/11/2024
How to test 🧪
yarn cy:run -s "cypress/e2e/core/cloudpulse/cloudpulse-validation.spec.ts"
As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[x] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

I've added a mocked account to my test, so the test cases should now pass

closing this PR due to  additional changes , will open  a new PR

closing this PR due to  additional changes , will open  a new PR


How about adding support for deprecated jQuery methods ?

Recently, I was fixing bugs during migration jQuery to 3.x .
And I found that $(elem).size() has been deprecated and removed in  1.8
What if this plugin supports checking deprecated jQuery methods
@gibson042 @mgol

A pull request that adds a rule for size, suggesting length instead, is welcome.


Prevent using sentinel ranges for DNS upstreams

The proptests discovered this issue which I fixed in c8d87f8 (#6943).
I think we knew this but probably worth discussing the following:

Do we need to document this somewhere?
Should we block this on the portal?

Yeah we should probably prevent this with a validation check in the portal. I'll update the title accordingly.

@bmanifold I know you have a PR that changes the DNS page already. Let's fix it once your PR is merged


BatchLexer performce issue with multiple variables after if

(Original issue 1256 created by roskakori on 2016-07-13T23:48:28.191896+00:00)
Here's is another performance issue in the shell.BatchLexer which has not been addressed by the fixes for #1216 and #1240. It can be reproduced with the following example code:
def test_can_process_variable_after_if():
    lexer = lexers.get_lexer_by_name('bat')
    # text = 'if %suffix% equ .jar set LIBPATH=%LIBPATH%;"%REPO%\\%filename%"'
    text = 'if %xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx% %a%'
    for token in lexer.get_tokens(text):
        print(token)

if __name__ == '__main__':
    test_can_process_variable_after_if()

The more x there are in
if %xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx% %a%

the slower the lexer gets though eventually it will finish. While this is not valid batch code, the example includes an alternative real life code snipplet taken from the Neo4j launcher script:
if %suffix% equ .jar set LIBPATH=%LIBPATH%;"%REPO%\\%filename%"

(Original issue 1256 last updated on 2017-01-22T17:45:25.599030+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by dscorbett on 2016-07-14T01:28:06.851126+00:00)
Fixed in pull request #620.

(Original comment by birkenfeld on 2017-01-22T17:45:25.614643+00:00)
Merged!


Aviod using React's internal modules

Due to internal changes in upcoming React release, it's now unsafe to refer to React's internal modules.
Here's the error I get while using<EMAIL_ADDRESS>with<EMAIL_ADDRESS>Error in ./~/react-art/lib/ReactART.js
Module not found: 'react/lib/ReactInstanceMap' in C:\dev\chart\node_modules\react-art\lib

 @ ./~/react-art/lib/ReactART.js 22:23-60

Error in ./~/react-art/lib/ReactART.js
Module not found: 'react/lib/ReactMultiChild' in C:\dev\chart\node_modules\react-art\lib

 @ ./~/react-art/lib/ReactART.js 23:22-58

Error in ./~/react-art/lib/ReactART.js
Module not found: 'react/lib/ReactUpdates' in C:\dev\chart\node_modules\react-art\lib

 @ ./~/react-art/lib/ReactART.js 24:19-52

--
I'm open to help fixing it, just need some suggestions.
As far as I understand, React ART relies havily on React's internals: https://github.com/reactjs/react-art/blob/db4dcef9532cc9e3e1235bc62e917efcb3fb5b24/src/ReactART.js#L23-L25 Is it possible to use any public APIs instead? Is it OK to just copy-paste required files directly into this project?
Maybe we have something to do here in preparation for new Fiber reconciler? I think I've heard somewhere that custom renderers will have to bring their own reconciler code when it lands…
✌

Update
I noticed that React ART is also exists in React repo where this issue is irrelevant! Is it possible to pubish this package from there?

The best solution here is to change these to import from react-dom. React ART already needs to live inside React DOM. The copy in the React repo is only for tests and this is the source of truth.

Thank you!


Parallel Arrow file format reading

Is your feature request related to a problem or challenge?
DataFusion can now automatically read CSV and parquet files in parallel (see https://github.com/apache/arrow-datafusion/issues/6325 for CSV)
It would be great to do the same for "Arrow" files
Describe the solution you'd like
Basically implement what is described in https://github.com/apache/arrow-datafusion/issues/6325 for Arrow -- and read a single large arrow file in parallel
Describe alternatives you've considered
Some research may be required -- I am not sure if finding record boundaries is feasible
Additional context
I found this while writing tests for https://github.com/apache/arrow-datafusion/issues/8451

I'd like to have a try.

I read related pr about parquet and csv.
Parquet parallel scan is based on rowgroup and csv is based on line. Both of them can be splitted by row and then output RecordBatchs using  a certain method.
I don't think arrow can be handled like that, since arrow file is purely column-based.
But I am wondering whether we can split the scan process into several parts and rebuild the whole Batch, since there maybe more than one array in file.

Merry Christmas!

But I am wondering whether we can split the scan process into several parts and rebuild the whole Batch, since there maybe more than one array in file.

This sounds like a good idea to me in theory -- I am not sure how easy/hard it would be to do with the existing arrow IPC reader
In general, the strategy for paralleizing Paruqet and CSV is to be to split up the file by ranges,  and then have each of the ArrowFileReaders partitions read row groups (or CSV lines) that have their first byte within their assigned rnage
Perhaps we could do the same for arrow files which could use the first byte of the RecordBatches 🤔

Perhaps we could do the same for arrow files which could use the first byte of the RecordBatches 🤔

There maybe several RecordBatches(blocks in arrow-rs) in a Arrow file(I didn't notice it before). We can handle it like rowgroups in parquet.
I will check whether DICTIONARY can be handled correctly since there maybe Delta DICTIONARY.
Thanks.

I will check whether DICTIONARY can be handled correctly since there maybe Delta DICTIONARY.

It seems that delta dictionary batches not supported yet.
And I think a pub function to provide offsets is needed in upstream.  Like
impl<R: Read + Seek> FileReader<R> {
    pub fn blocks(&self) -> Vec<Block> {
        &self.blocks
    }
   //OR
    pub fn blocks(&self) -> Vec<i64> {
        &self.blocks.iter().map(Block::offset).collect()
    }
}

https://github.com/apache/arrow-rs/pull/5249 adds a lower-level reader that should enable this and other use-cases

Delta DICTIONARY.

Delta and replacement dictionaries are only supported by IPC streams, not files

get it! Thanks

The next release is tracked by https://github.com/apache/arrow-rs/issues/5234


Use VSCode terminal for QShell session

Is your feature request related to a problem? Please describe.
A few months ago we discussed that it would be pretty nice to have a 5250 client inside of VSCode because you need an external 5250 terminal all the time at the moment. But it would be a pretty big task to create a 5250 client.
But how about using a QShell / QP2TERM Terminal?
Describe the solution you'd like
VS Code already has a basic functionality to use the local terminal. Maybe this can be also used to create a connection to the IBM i's QShell. By default it could open 1 connection after connection. More connections can be created by hitting +.
Describe alternatives you've considered
The alternative would be to use an external 5250 emulator

Since we use ssh2 for the connection, it may be possible to create a terminal and pipe our connection - this would give access to a pase shell, but it might break the editor experience: https://www.npmjs.com/package/ssh2#start-an-interactive-shell-session
I'd have to look into this further.

https://code.visualstudio.com/api/references/vscode-api#window

I just played around with VSCode's Pseudoterminal (https://code.visualstudio.com/api/references/vscode-api#Pseudoterminal)
I think a Pseudoterminal in combination with IBMi.qshCommand could lead to a good result for the beginning.
What do you think @worksofliam ?

@szsascha That is awesome. I looked for that API before and I couldn't find it. Good idea!
I think we should hard code some commands, for example, cd and maybe one to switch which env to send the commands to?
let env = `qsh`; //would actually be global somewhere

const commandString = ...;
const command = commandString.split(` `);
switch (command[0]) {
  case `cd`:
    await config.set(`homeDirectory`, command.slice(1).join(` `));
    break;
  case `env`:
    env = command[2]; //maybe more validation?
    break;
  default:
    switch (env) {
      case `qsh`:
        connection.qshCommand(commandString); //inherits home dir from config
        break;
      case `pase`:
        connection.paseCommand(commandString); //inherits home dir from config
        break;
      case `ile`:
        //needs additional work to use library list connection
        connection.remoteCommand(commandString); //inherits home dir from config
        break;
    }
    break;
}

What do you think?

@worksofliam
Another alternative could be to create different types of terminals. There could be own types for qsh, pase and ile. Then it's possible to open own sessions.
I don't know if it could be a bit too much work of coding to go the way you've described.
I think the question we're asking now is:
Would we like to run IBM i commands directly in its remote shells or would we like to create an own kind of shell in this extension?
We're able to do both. But it's a more strategic question now.

@szsascha i think each having their own shell could work too!
I do believe the shells belong in the base extension personally. But I'm not opposed to them being in their own!

@worksofliam
I also think it belongs to the base extension. But the question is: Do we have a kind of "common IBM i shell" where we do a lot of coding to issue commands in their right environment and switch between environments or do we create a shell for each environment to prevent this switching?
For example on Mac you're also able to use zsh or bash:

We would have the same thing on IBM i. We could have a single IBM i shell or for each (qsh, ile, bash) an own entry in this list.

@szsascha It does seem like we should separate vscode shells for each environment as you mentioned. Great idea!
But, we need to make sure that certain commands (like cd) also update the connection configuration like in my example.
Please let me know if you'd like me to have a go at spinning this up in a branch.

@worksofliam
Now I got your point, I guess. Sorry for the misunderstanding on my part.
Just to be sure:
We'll get the directory we changed into and update the connection settings. With this we're sure that the correct directory is used in the IFS browser, for example.
Have I understood it now?
I personally would prefer to have the issued commands independent of all other functionalities. The main reason to use the terminal will be to check something in a fast way. The rest of the environment should be untouched.
But this is also your decision.

Please hold this back. I'll maybe start to work on something that will make this issue obsolete soon.

@szsascha You got it. Please update here when you are ready!

@worksofliam & @szsascha , I don't know if you want to look at https://github.com/SchoofsKelvin/vscode-sshfs? I have found this to be very useful and works pretty well. (not just for IBM i but other systems as well)
I like that you can mount the remote folders as local workspace folders and there is a form for the connection etc and an integrated remote terminal

@dennisnel I am not sure how this relates to this issue.

@worksofliam
Working hard on my first PoC but unfortunately nothing to show yet. Unfortunately it will take some time because it is a lot of work.
But I got a question. I'm sure you know the answer :)
As far as I know there are 3 kinds of "shells" on the i:

QShell
QP2TERM
5250

I'm wondering what are the official names of these shells are. And if there are some more shells that I forgot. Maybe SQL also counts as shell?
And what exactly is the difference between QShell and QP2TERM?
In the past I called them as mentioned above and everyone understood me. But the names I use didn't sound very official.
I would be very grateful for an answer :)

https://www.mcpressonline.com/programming-other/general/techtip-qshell-vs-pase

QShell
PASE (this has 3 shells Korn, Bourne, C)
?

@priceaj
Great! Thanks a lot!
And SSH connects always into PASE?

As far as I am aware, yes, but not 100%


⚠ GitLab Server has degraded performance

In b0ed293, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11466 ms

Resolved: GitLab Server performance has improved in 886a8ad.


nsc-events-nextjs_6_341_unit-test-for-event-detail-page

This is the Unit test for Event-detail Page
Issue: #341

I have an error with this part and still not figure out yet.

Not all the tests passed. I did a bit of research to see what the issue here might be and to my knowledge i think to resolve this issue, you need to mock the Next.js router in your test. This can be achieved using jest.mock to create a mock implementation of the useRouter hook. Other classmates might have better solutions to help fix the issue

@kaimanasse15 Did you try @MuhammadNSC's advice?

@kaimanasse15 Did you try @MuhammadNSC's advice?
It doesn't work for me. I think I need someone else to follow this. Thanks, @brinkbrink

It's still failing were you able to try other resources for this?

Closing due to inactivity. Reopen if needed.


Won’t turn on TV

Hi!  I got this app installed and connected to a switch. I can see that it’s running, but my device (Samsung TV model U) won’t turn on.
The MAC address is correct, and I can see in the logs that it’s running but I can also see a log message that says “doesTvSupportVoice TV: false”.  Does that mean it won’t work?
I can confirm that the TV actually does support WOL, I can send a magic packet directly to it and it turns on.
A question for my own diagnostics, what’s the specific method that SmartThings is using to trigger the WOL?  Does it try to send a magic packet over the public internet?  In that case I would need to open up a port no?
Separately, I see that Alexa devices now seem to support WOL directly: https://developer.amazon.com/en-US/docs/alexa/device-apis/alexa-wakeonlancontroller.html#supported-devices. Is that something I should be looking at instead of going through SmartThings?

It doesn't seem like you are using this SmartApp.
It is pretty simple:
https://github.com/rtyle/wake-on-lan/blob/master/smartapps/rtyle/wake-on-lan.src/wake-on-lan.groovy
You can see that it doesn't log anything like what you are reporting.
This is the code that causes the WoL packet to be sent by your SmartThings hub on its LAN.
Your TV must be on the same LAN.
sendHubCommand(new physicalgraph.device.HubAction("wake on lan $targetMacAddress", physicalgraph.device.Protocol.LAN, null))

https://docs.smartthings.com/en/latest/cloud-and-lan-connected-device-types-developers-guide/building-lan-connected-device-types/building-the-device-type.html#wake-on-lan-wol

The last parameter is a map representing the options on the request. For a WOL request, this map will only ever consist of one parameter, secureCode. Some NIC’s support the SecureOn feature which requires the request to not only have a valid mac address, but also supply a valid password. This password must be configured on the NIC. If the NIC does not support SecureOn or does not have a password set, simply leave out the options map.

My TV does not require a secureCode, hence I did not code for one.

Yes, I’m definitely using this app and I do see the debug “WOL” message.
However, I don’t have a separate Smart Hub, just my TV. So likely the TV can’t send a WOL to itself ;-)
I managed to get it working with the WOL Alexa skill which uses Alexa to send the WOL to the TV. It works, just isn’t as ideal as having everything connected in SmartThings.

Yes, you need an agent on your LAN to send the unroutable WoL packet.
A SmartThings hub is required by this SmartApp.
They are pretty cheap:
https://www.amazon.com/Samsung-SmartThings-Generation-GP-U999SJVLGDA-Automation/dp/B07FJGGWJL

Understood now, thanks.  Just to mention, most Alexa devices can now send
WoL in case you think it's worth integrating at that level as well.
Thanks
Perry Krug
<EMAIL_ADDRESS>On Tue, Dec 17, 2019 at 2:02 PM Ross Tyler<EMAIL_ADDRESS>wrote:

Yes, you need an agent on your LAN to send the unroutable WoL packet.
A SmartThings hub is required by this SmartApp.
They are pretty cheap:
https://www.amazon.com/Samsung-SmartThings-Generation-GP-U999SJVLGDA-Automation/dp/B07FJGGWJL
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rtyle/wake-on-lan/issues/1?email_source=notifications&email_token=AACJCYRIZHPEGJGYTVYK63LQZDLXPA5CNFSM4J3UHEF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHCOXDY#issuecomment-566553487,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACJCYUU3Y4WB5WMJR5A2RDQZDLXPANCNFSM4J3UHEFQ
.


Feature Request: Organize Symbol List Only

Because I'm using Abstract, any changes I make to the positions of symbols on the Symbols page leads to headaches when trying to merge changes together, so ideally I'd like to be able to sometimes use this plugin to sort only the order of items in the left Symbols list in Sketch rather than actually changing the artboard positions every time. I was hoping you'd be able to add in a second function, something like "Organize Symbol List Only" that would do this, then I could run this frequently and run the main Organizer a bit less often. Any chance of this happening? 😄

@ryanstockau this is great feedback, I will consider this as an enhancement... thanks.


CIS2.4- Ensure no security lists allow ingress from <IP_ADDRESS>/0 to port 22 and port or 3389

Hello,
We had deployed OCI Landing Zone 2.3.4 version, and when running the compliance script, we're getting the following non-compliance messages:
1- Ensure no security lists allow ingress from <IP_ADDRESS>/0 to port 22.
2- Ensure the default security list of every VCN restricts all traffic except ICMP.
Could you please tell us if there is an indicator to enable in order to remove or restrict these two rules? If not, which files should be edited to replace the address <IP_ADDRESS>/0 with a more restricted address range?
Also, what is the best and soft approach to upgrade from the 2.4.1 version to the latest one?

Hi storj01,
1- Use variable public_src_bastion_cidrs to specify the CIDR blocks allowed to ingress into 22 or 3389. No need to change code. If you are deploying with Resource Manager, the corresponding field is in the attached screenshot.
2 - The CIS Landing Zone default security lists already restricts all traffic except ICMP.
CIS LZ Terraform is backwards compatible between 2.x.x releases. You should be able to upgrade to the latest 2.5.11. Simply refresh the Terraform code while keeping your variable settings.
Thanks,
Andre.


[D1 Retro]: Team Sequence Diagram

Improvement:

[ ] Webserver
[ ] Actor
[ ] UoW / DBContext
[ ] Create / createAsync no such method - (C# is case sensitive)
[ ] Last one to db should be sending SQL statement to DB
[ ] Proper namespace

completed on time - before D2 submission


Update Scala.js to 0.6.8

Current coverage is 97.42%

Merging #68 into master will decrease coverage by -1.38% as of 3e73a9f

@@            master     #68   diff @@
======================================
  Files           16      16
  Stmts          504     504
  Branches        54      54
  Methods          0       0
======================================
- Hit            498     491     -7
  Partial          0       0
- Missed           6      13     +7

Review entire Coverage Diff as of 3e73a9f

Powered by Codecov. Updated on successful CI builds.


Terraform test.pypi.org and associated fastly services

Currently the following domains are served from Fastly services which are unmanaged by terraform:
Warehouse (Staging)

test-files.pythonhosted.org (Equivalent to "PyPI File Hosting" service)
test.pythonhosted.org (Equivalent to "PyPI Docs Hosting" service)
test.pypi.org (Equivalent to "PyPI service")
test.pypi.io (Equivalent to "PyPI service")
warehouse-staging.python.org (Equivalent to "PyPI service")

TestPyPI (Legacy)

testpypi.python.org (Equivalent to "PyPI service")

Goal is to move all of this to three services that match the production setup (warehouse, files, docs), then abstract any necessary differences to configuration, then build both the test and prod environments from the same Terraform module.

Successfully migrated test-files.pythonhosted.org to terraform management with #96

Successfully migrated test.pythonhosted.org to terraform with #98

Successfully migrated test.pypi.org to terraform with #99

Thanks for your work on this @cegerhardson!


fix handling of HelloRetryRequests

Fixes #1821.

Codecov Report

Merging #1822 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1822      +/-   ##
==========================================
+ Coverage   87.03%   87.05%   +0.02%
==========================================
  Files          95       95
  Lines        6538     6548      +10
==========================================
+ Hits         5690     5700      +10
  Misses        603      603
  Partials      245      245

Impacted Files
Coverage Δ

internal/handshake/crypto_setup.go
69.01% <100%> (+0.9%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4bd715...3d799d9. Read the comment docs.


remove duplicate optional link

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[ ] New extensions are written in Swift 5.0.
[ ] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[ ] I have added tests for new extensions, and they passed.
[ ] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[ ] All extensions are declared as public.
[ ] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 580 tests, with 0 failures (0 unexpected) in 11.326 (13.062) seconds

:book:
tvOS: Executed 555 tests, with 0 failures (0 unexpected) in 3.992 (6.718) seconds

:book:
macOS: Executed 418 tests, with 0 failures (0 unexpected) in 0.946 (1.242) seconds

  Generated by :no_entry_sign: Danger

Thank you for contributing to SwifterSwift! I've invited you to join the SwifterSwift GitHub organization - no pressure to accept! If you'd like more information on what that means, check out our contributing guidelines. Feel free to reach out if you have any questions! 😃


Document Dropbox API setup

Add a document that describes how to get Dropbox API credentials for testing
(or deploying) sensitive submission document uploads.

[!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

#1725  👈
#1724
#1712 : 1 other dependent PR (#1719 )
#1711
main

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

Join @sgrimm and the rest of your teammates on  Graphite

Merge activity

Mar 8, 9:47 AM EST: @sgrimm started a stack merge that includes this pull request via Graphite.


why concatenate axis of EncoderStack is -2?

Hi, thanks for this thoughtful paper and sharing code base.
I am confused about concatenate axis using -2, which is sequence length L, at https://github.com/zhouhaoyi/Informer2020/blob/99b59c181f871a1521a798a37287a27ba5b742a7/models/encoder.py#L95
Based on #58 , @cookieminions says and I quotes "can make all Informers in InformerStack predict outputs with the same length" (at https://github.com/zhouhaoyi/Informer2020/issues/58#issuecomment-802907478). Should it be more reasonable to concate on channel axis, which is -1, instead of time axis as in current code?
For my understanding, EncoderStack seems to mimic Inception architecture (https://d2l.ai/chapter_convolutional-modern/googlenet.html#inception-blocks), thus would be more convincing to me if concate on channel axis.
Please correct me if I am wrong. Greatly appreciated in advance.

InformerStack is designed to concatenate all Encoder output on time axis, so that the InformerStack Encoder's output contains long-term and short-term information at the same time. The output length of each Encoder can be different, but their dimension must be the same, otherwise there will be problem at the cross attention in Decoder.

That makes sense. Thanks for reply.


.NET Code Samples web page link

The YouTube APIs code sample repository on GitHub link on the https://developers.google.com/youtube/v3/code_samples/dotnet page links to https://github.com/youtube/api-samples
There should also be a link to the two comprehensive YouTube examples in https://github.com/google/google-api-dotnet-client-samples/

ResumableUpload.CS.Sample
ResumableUpload.VB.Sample

Thanks @mikemeinz, I've created an internal PR to add links to these samples.

Thanks
On Fri, Jul 14, 2017 at 9:20 AM, Chris Bacon<EMAIL_ADDRESS>wrote:

Thanks @mikemeinz https://github.com/mikemeinz, I've created an
internal PR to add links to these samples.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-dotnet-client/issues/1054#issuecomment-315357806,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANOpNMtz6VtvLHTFVlKRMw4sWPS1sPYrks5sN2sogaJpZM4OYMTr
.

The internal PR is submitted. Hopefully it'll propagate to the externally visible site soon...

This change is now live on https://developers.google.com/youtube/v3/code_samples/dotnet
So closing this issue.


Fixed broken women's section redirect

📝 Description
Please include a summary of the change and which issue is fixed.
🔧 Fixes #(19)

Fixed broken women's section redirect

[x] I have performed a self-review of my own code.
[x] I'm a contributor in Hacktoberfest'21.

Hey actually it was not a problem, we will create more pages and link them to the homepage, we now only have women.html blank page hence it was linked to that. You can leave it to women.html and change the link to WOMEN in the footer to women.html

Ah okay, thank's for the info, i'll work on it :)


(PUP-9364) Exclude unnecessary module release metadata

The puppet module install command will download release metadata[1]
for each release of the module and its dependencies so that the PMT can
resolve dependencies locally. The v3 'releases' forge API response is
verbose as the release metadata includes the module's readme, changelog
and license, which are not needed to resolve dependencies.
This commit excludes all currently known fields except version,
metadata, and file_md5 to reduce the amount of transferred data. Ideally
we could specify only the fields we want to be included, but the
include_field parameter doesn't work that way.
For stdlib specifically, this PR drops the amount of data transferred
from 2.5MB to 77KB. This will also benefit modules that depend on
stdlib.
[1] https://forgeapi.puppet.com/#!/release/getReleases

CLA signed by all contributors.

The forge was updated today, and this works as expected. Note all the list of excluded fields is preserved for each "page":
bundle exec puppet module install --debug puppetlabs-apache
Debug: Runtime environment: puppet_version=5.5.9, ruby_version=2.5.3, run_mode=user, default_encoding=UTF-8
Notice: Preparing to install into /Users/josh/.puppetlabs/etc/code/environments/production/modules ...
Debug: Facter: Found no suitable resolves of 1 for ec2_metadata
Debug: Facter: value for ec2_metadata is still nil
Notice: Created target directory /Users/josh/.puppetlabs/etc/code/environments/production/modules
Notice: Downloading from https://forgeapi.puppet.com ...

Info: Preparing to install ...

Notice: Installing -- do not interrupt ...
/Users/josh/.puppetlabs/etc/code/environments/production/modules
└─┬ puppetlabs-apache (v3.5.0)
  ├─┬ puppetlabs-concat (v5.2.0)
  │ └── puppetlabs-translate (v1.2.0)
  └── puppetlabs-stdlib (v5.1.0)

And the total amount of release metadata for all stdlib releases is only 63KB instead of 2.5MB:
$ curl -s https://forgeapi.puppet.com/v3/releases\?module\=puppetlabs-stdlib\&sort_by\=version\&exclude_fields\=readme%2Cchangelog%2Clicense%2Curi%2Cmodule%2Ctags%2Csupported%2Cfile_size%2Cdownloads%2Ccreated_at%2Cupdated_at%2Cdeleted_at | wc -c
   62819

jenkins please test this on windows2012r2-64a

jenkins please test this on windows2012r2-64a

Yeah, unfortunately include doesn't do an "only include" what you request. It's kind of broken that way.

Created FORGE-474 to request the include_only feature.

@nkanderson and I talked about adding a "include only" feature, I think there may already be a ticket filed on that? I'm assuming it's a non-trivial change, and we should move ahead with the exclude for now.


rkt run: missing stage1 message needs improvement

If stage1.aci doesn't exist in the same directory as rkt (and isn't explicitly specified on the command line), the message is incredibly unintuitive:
rkt run https://github.com/coreos/etcd/releases/download/v2.0.0/etcd-v2.0.0-linux-amd64.aci
rkt: searching for app image /usr/bin/stage1.aci
Error finding stage1 image "/usr/bin/stage1.aci": Get http://?ac-discovery=1: http: no Host in request URL

I think for now we should keep things simple and not put the ACI through the same fetch path as for actual application ACIs. The idea of autodiscovering stage1 ACIs is both magical and terrifying, I think we should leave that to explore at a future date :-)

/cc @vcaputo

Agreed, should we even support http/https at all for the stage1 aci?

https://github.com/coreos/rocket/pull/459 keeps the potential for remote fetching if --stage1-image is a url, but disables discovery.

Still getting a similar error today on a fresh Rocket build from the latest source:
~ ❯❯❯ rkt run coreos.com/etcd:v2.0.0                                                                                ⏎
Error finding stage1 image "/usr/bin/stage1.aci": rkt only supports http or https URLs (/usr/bin/stage1.aci)
~ ❯❯❯ rkt version                                                                                ⏎
rkt version 0.3.1+git
appc version 0.3.0+git

Not sure what that error means and what I need to do.

@rasschaert can you copy bin/stage1.aci (which should be created after you ran the build script) to the same directory as rkt? that is where it looks at present, e.g.
  -stage1-image="/home/jon/git/rocket/bin/stage1.aci": image to use as stage1. Local paths and http/https URLs are supported. By default, Rocket will look for a file called "stage1.aci" in the same directory as rkt itself

@jonboulle Thanks! That fixes it.
Turns out the Arch Linux rocket-git package in AUR only puts the rkt binary in /usr/bin/ and throws away all the rest.
Seems a bit... peculiar to have a non-executable file called stage1.aci in $PATH though. Is this something that might still change in a future release?

@rasschaert oh wow, had no idea there was a rocket package already, that's cool!
We do it this way at the moment because it's the most straightforward way to provide a self contained package on GH that works out of the box; we don't actually intend people to put it into /usr/bin generally. The intention is that when people are packaging Rocket for distributions that they will a) potentially create their own stage1, and b) configure/set the stage1 during the build for the packaging.
So for example, when packing Rocket as an RPM (I am totally ignorant about how Arch packages work, sorry), I would expect that the stage1 would be written somewhere like /usr/lib/rkt/stage1.aci, then the actual rkt binary would be built to reference this path as the default.
Does that make sense?

@jonboulle Arch has a reputation for living on the bleeding edge ;)
Thanks for the explanation and sorry about hijacking this issue.

@rasschaert No problem at all. Definitely feel free to open more issues as things like this come up - we very much want Rocket to be packageable/distributable on a variety of systems.
Also relevant (and a bit of a tracking ticket for this stuff): #173


Create a release for package managers

It would be nice to have a release that can be used by npm, yarn,... there is a version 0.3.2 available, but NPM and alike only show 0.3.1.
Looks like the tag has not been created (404 error)

Hi, same request ;)


When scrolling to the auto-close area, it will freeze!

I am confused, why does this happen?

@ppamorim

https://user-images.githubusercontent.com/47734570/144209684-4026022e-734b-40da-a11f-a76ca64f4eda.MP4

Hierarchical structure：
The first layer: A-VC
The second layer:B-VC
Are all popped up through 'presentPanModal'
Bug:
Dismiss B-VC，When it is pulled down to the automatically closed area, release it, and there will be a freeze, and it will automatically close after the freeze.


Issue with Web app | Unsupported operation: Platform._localeName

Good day,
I'm trying out Google Wallet to let users collect generic pass in Flutter's web app. However, it keeps throwing the following error (tested in Web Debug and Build mode):

══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following UnsupportedError was thrown building GoogleWalletButton(dirty):
Unsupported operation: Platform._localeName
The relevant error-causing widget was:
GoogleWalletButton

The error was from this call in _checkAvailable method in the example code: available = await googleWallet.isAvailable();
As a side note, tested on Android simulator and it seems to work.
Any help is much appreciated!

@AnhNgDo
This is more or less intended behaviour, as far as I can tell. Platform._localeName is from dart:io, which does not exist on the web platform, therefore throwing an error. This library currently only supports Android (see https://pub.dev/packages/google_wallet). Maybe web support could be added, not sure whether or not it's possible / planned.

It would be very cool to support a web version, although off the top of my head I don't think it's feasible since there wouldn't be a way to sign the pass JWTs. I'll close this for now, but if someone can prove me wrong please re-open this and we can use it to track the feature request.
Thank you


Compute-Only Release 2024-05-22

Release 2024-05-22
Please merge this Pull Request using 'Create a merge commit' button

Reviewed for changelog.


content is not removable if the last block is code-block

Can not remove content after selecting everything (cmd +a) when the last block is code-block
https://www.dropbox.com/s/mdvn2pqa52yqufl/Screenshot 2018-05-21 15.35.40.png?dl=0

This is a duplicate of #66 I think

@mxstbr I don't think that it is the duplicate of https://github.com/withspectrum/draft-js-markdown-plugin/issues/66. (I left comment there and I think that it should not be an issue any more)
Hiting backspace does not trigger click event, so onClickOutside  https://github.com/withspectrum/draft-js-markdown-plugin/blob/master/src/components/Code/index.js#L78 is not even called

Oh sorry, that's true, good point. Reopening!


Add explicit null returns in flutter/test

Adds null statements at the end of functions which have implicit null returns. These functions all have nullable return types (and not void, dynamic`, and upcoming analysis encourages such functions to have explicit returns).
List which issues are fixed by this PR. You must list at least one issue. https://github.com/flutter/flutter/issues/94759
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

CC @goderbauer

@gspencergoog Can you do a secondary review on this one?


ENH: ignore unsatified operations with partial inputs

Usefull when 2 (or more) operations provide the same output,
and provided inputs fully satisfy only one of operation.
For instance, the graph below with this PR can go ahead and compute

ab_plus_c=21 when given just a=10 & b1=2, and
ab_plus_c=6 when given just a=10 & b2=2:

Before this PR, the above graph would fail on execution time while trying to evaluate needlesly
the un-satisfied operations.

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Codecov Report

Merging #18 into master will increase coverage by 1.8%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master      #18     +/-   ##
=========================================
+ Coverage   77.87%   79.68%   +1.8%
=========================================
  Files           5        5
  Lines         348      379     +31
=========================================
+ Hits          271      302     +31
  Misses         77       77

Impacted Files
Coverage Δ

graphkit/network.py
74.23% <100%> (+4.03%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e70718b...16be1cd. Read the comment docs.

Merged with #23 bc ordered-set is needed to rewrite unsatisfied traversal with topo-sort (without visited nodes).


Duplicate messages, and it seems like they get stuck?

These messages don't go away. Sometimes there is only one message for each server, sometimes more.
I'm not sure but I think this bug gets triggered after getting an error as described in #8

See jose-elias-alvarez/null-ls.nvim#619 for a possible fix.

I am facing the same problem even with https://github.com/jose-elias-alvarez/null-ls.nvim/pull/619

The error message no longer appears (that was fixed by https://github.com/jose-elias-alvarez/null-ls.nvim/pull/619), but the messages are still duplicated. I have noticed this only happens with null-ls

Yeah I think you might be right about this being a duplicate of #10. I'll close this one.


maxconn default is inconsistent between default and global configs

4096 in global, nothing in default

Going to leave this alone for now https://cbonte.github.io/haproxy-dconv/1.8/configuration.html#3.2-maxconn


[MINOR] Following #2070, Fix BindException when running tests on shared machines.

What is the purpose of the pull request
This is a follow-up to #2070, retrying bind does not seem to take effect, because miniDfsCluster is not initialized, calling the stop method we will get a null pointer.
2022-06-23T18:59:25.1514164Z java.lang.NullPointerException
2022-06-23T18:59:25.1515051Z 	at org.apache.hudi.common.testutils.minicluster.HdfsTestService.stop(HdfsTestService.java:106)
2022-06-23T18:59:25.1516107Z 	at org.apache.hudi.common.testutils.minicluster.HdfsTestService.start(HdfsTestService.java:96)
2022-06-23T18:59:25.1517359Z 	at org.apache.hudi.utilities.testutils.UtilitiesTestBase.initTestServices(UtilitiesTestBase.java:118)
2022-06-23T18:59:25.1518418Z 	at org.apache.hudi.utilities.transform.TestSqlFileBasedTransformer.initClass(TestSqlFileBasedTransformer.java:52)

Brief change log

Added a loop when creating HDFS Cluster.

Verify this pull request

Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

Running CI with Azure in a PR encountered this NPE problem, we can fix it.
cc @prashantwason @leesf


Add symbols & tables to version requirements

See https://trello.com/c/0rTzMr4p/542-use-a-table-on-the-version-requirements-page
@manics suggested symbols in tables would be easier to read so I've replaced all the support level terminology with symbols and converted the summaries at the top into tables too.
Also a couple of wording tweaks as I was going through.

Staged at https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/version-requirements.html

Thanks, it's a lot clearer now! 👍  I still think seven different symbols isn't ideal, but I don't have a better solution, and since it's a definite improvement it should be left for a future discussion.
A couple of unrelated points that have become more apparent:
Bitness https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/version-requirements.html#bitness
Mention 64-bit in the table and have 5.3-5.4 as columns instead of rows?
Components https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/version-requirements.html#components

Python 2.6 / OMERO 5.3: I think this would look better if there was a line-break between web and server instead of having them side-by-side.
Python 3.x: Shouldn't this be broken instead of unsupported since it's known not to work?

Changed python 3 to broken (cc @jburel )
Can't do linebreaks in list tables so I've made separate columns for OMERO.web and OMERO.server for version 5.3 instead @manics

for python 3 this is a bit of both. Marking it as broken works and it is probably clearer

it does not. This is more semantics
As i said, I am ok with broken

I ended up with rounded brackets because the square ones looked even worse with the alignment offset caused by the symbol but I could also try using a smaller symbol if you think it looks too odd.

I think it's good enough, no point spending too much time on it. This builds locally for me so good to merge.

👍 I rebuilt staging just to double-check but agree that it's probably not worth messing about with image sizes, it's a big improvement overall.


Adsense support required

This module works great, but while using AdSense module there is no improvement.
Please consider adding AdSense module support if possible.

Hello @Croodo,
Is this about this module? (https://github.com/nuxt-community/google-adsense-module)
Unfortunately there is nothing we can do.
There must be customizations in the module itself.
There should be a possibility, with which the user himself controls when the required scripts are loaded.
The simplest solution would be to set the scripts in the head only in the client.
In the SSR these have nothing to look for and influences the loading behavior.
https://github.com/nuxt-community/google-adsense-module/blob/ac541311b710503562918bc2018d1117f37a7c45/src/index.ts#L62-L67

Close the issue for now, but can be reopened if used.


YAML support for the JSONL files

Is your feature request related to a problem? Please describe.
No, not a problem
Describe the solution you'd like
I see that the configuration file for the plate can be written in plain JSON, that will then be translated to JSONL: https://haswitchplate.github.io/openHASP-docs/0.6.2/integrations/home-assistant/howto/#json-files
My immediate question was: why not YAML too? Since the plugin already translates from JSON to JSONL and it is possible to translate from yaml to JSON I see no reason why this could not be possible.
Describe alternatives you've considered
Nothing really
Additional context
The advantages of this is that YAML is less verbose and easier to write by hand than JSON

Is this something that you are able to work on yourself? Otherwise I am inclined to close this issue, since JSON/JSONL is equally easy to use.


Modded symbols only get eating bonuses once per slot per spin

Is your feature request related to a problem? Please describe.
If a modded symbol destroys another symbol for a bonus, such as gaining a multiple of the destroyed symbol's value for one turn, and then another symbol is added to that same slot and eaten again, the bonus will only apply once. For example; if a modded symbol destroys a pearl and gains 4 coins, then another pearl spawns in that same spot from an oyster and is destroyed, no additional bonus will be given by the second pearl. Vanilla symbols with this effect, such as monkey and dwarf, do not have this issue.
Describe the solution you'd like
Making the symbols function as expected, able to obtain bonuses from the same slot multiple times if new symbols spawn there mid-spin.
Additional context
I'm unsure if the same issue is present for other types of bonuses, like linear value or permanent value, but I would assume so since everything that checks whether the bonus is applied is the same.

(this is horribly wordy but I'm not sure how to improve that)


Refactor min_by and max_by aggregate functions using Simple Function API

Description
The implementations of min_by and max_by aggregate functions are so complex that it became difficult to extend: https://github.com/facebookincubator/velox/pull/8956
Let's re-write these using Simple Function API to reduce complexity and make it easier to extend.
CC: @kagamiori

Hi, I am interested in working on this issue. I just got some hands-on experience in rewriting functions using simple function API. May I get assigned? Thanks!

@Real-Chen-Happy Thank you for helping with this.

It seems like for Presto SQL, min/max_by two-args version is using MinMaxByAggregateBase from velox/functions/lib/aggregates/MinMaxByAggregatesBase.h, which is shared by both Spark SQL and Presto SQL. `
The three-arg versions is only used inside Presto SQL.
Given the complexity, I plan to use two PRs to resolve this issue.

Rewriting two-arg versions, which will touch both Spark SQL and Presto SQL
Rewriting three-arg versions, which will only touch Presto SQL.

CC: @rui-mo @PHILO-HE

template <typename C, typename V, bool isMin>
class MinMaxByAggregate {
 public:
  using InputType = Row<Any, Orderable<T1>>;
  using IntermediateType = Row<Any, Orderable<T1>>;
  using OutputType = Array<Any>;

  using ValueAccumulatorType =
      typename AccumulatorInternalTypeTraits<V>::AccumulatorType;
  using ComparisonAccumulatorType =
      typename AccumulatorInternalTypeTraits<C>::AccumulatorType;

  struct AccumulatorType {
    std::optional<ValueAccumulatorType> value_;
    std::optional<ComparisonAccumulatorType> comparisons_;

    explicit AccumulatorType(HashStringAllocator* allocator);

    void addInput(
        HashStringAllocator* allocator,
        exec::arg_type<Any> value,
        exec::arg_type<Orderable<T1>> comparison) {}

    bool writeIntermediateResult(exec::out_type<IntermediateType>& out);

  };
};

I am currently implementing AccumulatorType for two-args MinMaxByAggregate, and I am thinking how to implement it efficiently and cleanly for complex types specifically. In AccumulatorType, I need to store complex types for keeping intermediate result. In details, exec::arg_type<Any> value and exec::arg_type<Orderable<T1>> comparison, (which is GenericView and GenericView), needs to be stored (for complex type specifically), and then in writeIntermediateResult needs to read the stored data and convert it to exec::out_type<IntermediateType>& out (which will be RowWriter).
Now, I have some ways to approach this:

Approach One: I will write GenericView to the memory areas allocated by HashStringAllocator. However, in writeIntermediateResult it seems non-trivial to convert the result back to RowWriter. We can read the bytes to BaseVector similar to this:

void ValueSet::read(
    BaseVector* vector,
    vector_size_t index,
    const HashStringAllocator::Header* header) const {
  VELOX_CHECK_NOT_NULL(header);

  auto stream = HashStringAllocator::prepareRead(header);
  exec::ContainerRowSerde::deserialize(stream, index, vector);
}

But I am not sure how write BaseVector to GenericWriter.
2. Approach Two: For array types and map complex types, I can use allocator to store individual values. Then, in writeIntermediateResult I can use ArrayWriter and MapWriter to write the values back. However, this seems not efficient and also implementing Row types in this approach will be complex since it is nested.
@mbasmanova do you have any recommendation on how to solve this problems? Thanks!
cc @codyschierbeck @pramodsatya

@Real-Chen-Happy Check out copyValueListToArrayWriter in velox/functions/lib/aggregates/ValueList.h and let us know if you have follow-up questions.

  using InputType = Row<Any, Orderable<T1>>;
  using IntermediateType = Row<Any, Orderable<T1>>;
  using OutputType = Array<Any>;

I wonder why do you use Any here. Since all 3 types must be the same, I'd expect Generic for first arg and Orderable for the second.
  using InputType = Row<Generic<T1>, Orderable<T2>>;
  using IntermediateType = Row<Generic<T1>, Orderable<T2>>;
  using OutputType = Array<Generic<T1>>;

As suggested by @kagamiori , this issue will include the following PRs:

Add support for GenericView to SingleValueAccumulator and ContainerRowSerde, as well as changes to GenericView and GenericWriter. https://github.com/facebookincubator/velox/issues/9629
Add support for Generic type in SimpleAggregateAdapter. https://github.com/facebookincubator/velox/pull/9615
Rewrite two-arg versions of MinMaxBy with additional unit tests. https://github.com/facebookincubator/velox/pull/9498
Rewrite three-arg versions, which will only touch Presto SQL.


Fuel Codes | Auto populate data for next fuel code version not functioning periodically |1|

Description of Issue:
When entering the next version of an existing fuel code, TFRS asks the user if they want to pre-populate the fuel code input fields within the new fuel code web form. When the user clicks yes, the fuel code data does not populate the form on occasion. Note that sometimes the autopopulate feature functions as intended, other times it fails to populate any data from the previous fuel code version.
Possible Cause:
The autosave functionality that has been implemented within the fuel code feature may be causing this issue. We should "turn off" the autosave functionality for the fuel codes to determine if it is in fact the cause. https://trello.com/c/XQm6Dpch/1462-fuel-codes-auto-populate-data-for-next-fuel-code-version-not-functioning-periodically-1

 Fuel Codes | Auto populate data for next fuel code version not functioning periodically |1|


Cannot read properties of undefined (reading 'left') with Zoom Component and React SVG Component in Docusaurus

Issue Type

[x] Bug report
[ ] Feature request
[ ] Question / support request
[ ] Other

Description
react-medium-image-zoom version 5.1.3
I'm using Docusaurus 2.4.0 and when I'm zooming in/out of the page with a Zoom component and a SVG inside , Docusarus will crash eventually with

code is at the end of https://github.com/rpearce/react-medium-image-zoom/blob/main/source/utils.ts

Place detailed information here
fsdf

@LichLord91 Thanks for opening your issue and asking for assistance! Could you please provide an example image for me to test with? The ideal thing to do would be to provide a Codesandbox or example repository, but if all you can provide is an image, that would be helpful.

For starters, I'm thinking altering that logic to be this may be the fix, but I'd like to get a testable scenario from you.
export const getStyleGhost: GetStyleGhost = (imgEl) => {
  if (!imgEl) {
    return {}
  }

  if (testSvg(imgEl)) {
    const parentEl = imgEl.parentElement
    const rect = imgEl.getBoundingClientRect()

    if (parentEl) {
      const parentRect = parentEl.getBoundingClientRect()

      return {
        width: rect.width,
        height: rect.height,
        top: parentRect.top - rect.top,
        left: parentRect.left - rect.left,
      }
    } else {
      return {
        width: rect.width,
        height: rect.height,
        top: rect.top,
        left: rect.left,
      }
    }
  } else {
    return {
      width: imgEl.offsetWidth,
      height: imgEl.offsetHeight,
      top: imgEl.offsetTop,
      left: imgEl.offsetLeft,
    }
  }
}

It seems this code was unwise to begin with, so I may merge it before I hear back from you. Here's the PR: https://github.com/rpearce/react-medium-image-zoom/pull/407

Can you please 🙏 try this fix out? npm i<EMAIL_ADDRESS>

@rpearce  that seemed to do the trick no crashing on the 5.1.4-rc.0!


[UI] Loss of query parameters during paging queries

Bug report:
Search the cluster from UI:
First time query cluster by test is normal: https://example.dragonfly.com/clusters?search=test
Second time query cluster lost the search keyword: https://example.dragonfly.com/clusters?page=2

Expected behavior:
Filter the clusters by query keywords.

How to reproduce it:

Environment:

Dragonfly version: latest
OS: Linux
Kernel (e.g. uname -a):
Others:

Fixed https://github.com/dragonflyoss/console/pull/415.


Upgrade azure-identity msal4j from 1.3 to latest

Tracking work item to investigate upgrading from the Jan 2020 (1.3.0) release of msal4j to a later release, which now has additional bug fixes and better support for throttling instructions
https://github.com/AzureAD/microsoft-authentication-library-for-java/blob/dev/changelog.txt

Moving this back to August, per the 1.0.9 release plan

1.0.9 has been released.


A question of the generated data from the starcoderdata

Thanks for releasing your research codes to everyone.
I noticed that you used the starcoderdata dataset in your research and generated 80K of data including Python, C++, Java, TypeScript, Shell, C#, Rust, PHP, and Swift.
But I did not find any Swift language data in the starcoderdata dataset.
Did you also generate the Swift data from here?

Additionally, in the StarCoder paper, it is mentioned that Swift is not included in their training data. Where does the Swift data come from in this case?


typo of UO_0000036:'year' definition

EuPathDB (http://eupathdb.org/) used UO. We found the definition of "year" contains a typo:
"A time unit which is equal to 12 months which is science is taken to be equal to 365.25 days."
should probably say: " ...which IN science..."?
Thanks!

@leechuck @gkoutos
Any comments on the issue?
Thanks!
Jie

@reality Could you please have a look?


Template type checking not working with Vue 2

Describe the bug
Template type checking doesn't work in Vue 2 project. (But it works fine in Vue 3 project).
To Reproduce

Create new empty folder
Install vue types npm install vue
Add basic tsconfig.json file

{
    "compilerOptions": {
        "noImplicitThis": true,
        "noImplicitAny": true
    }
}

Create MyComponent.vue

<template>

Hello {{ name }}

</template>

Expected behavior
When I hover over {{ name }} in the template, I expect to see var name: string. But what actually appears is var name: any. When I select {{ name }} and press F12, it says "No definition found for name"
Screenshots/Recording

Info (please complete the following information):

OS = Windows 10
IDE = VS Code
TypeScript Version = 3.4.5
Vue Version = 2.6.12 / 3.0.5
VueDX version = 0.4.1

Additional context
N/A

Small information about this issue:
this seem to be an issue only if Vue.extend is used, but in case of default object export it seem to be working.

Small information about this issue:
this seem to be an issue only if Vue.extend is used, but in case of default object export it seem to be working.

Class components and composition-api plugin are not supported yet.

Class components and composition-api plugin are not supported yet.

@znck do we have an issue to track the support for the composition-api plugin in vueDX? I also tried to use that but seems like it gives a type error for the whole component.


[ICD] Add ICD features to BaseDeviceType and to RootNodeDevice Type

Feature description
Capability map in BaseDeviceType should contain SIT and  LIT bits
Platform
all
Platform Version(s)
No response
Anything else?
No response

I don't think this is reflected in our current device XML, but I could be wrong.
@robszewczyk Thank you for getting these issues filed!


chore: upgrade sirv

upgrade the local version we use for testing. it fixes a couple issues that users have hit such as https://github.com/sveltejs/kit/issues/9549
no need to do a changeset. users can upgrade projects on their own
release notes: https://github.com/lukeed/sirv/releases/tag/v2.0.3

Oh yeah, good catch. Though it makes me wonder why it's in devDependencies vs dependencies. If there's ever a security issue in any of these libraries users aren't going to be notified by npm audit. There's another bundling step with rollup in the adapter, so presumably the adapter could bundle the dependency in rather than us doing it before distributing adapter-node.

It should be theoretically possible, but the module resolution might be tricky, since then sirv would be a prod dependency of the Node adapter, and not a dev dependency of the user application. The handler.js file that gets copied into the user app would need to be able to access an indirect dependency when the app is getting bundled.

I don't get why that makes things tricky. adapter-node will invoke Rollup bundling everything except what the user puts in dependencies. Since that's not in dependencies it should get bundled.
I tested it locally and it seemed to work, so I updated this PR to make that change.

The memory leak issue is waiting a new version for adapter-node as well so I agree with Conduitry, it'd be better to get a version out with minimal changes and then think about this.
I was experimenting with getting rid of the build step entirely for the node adapter earlier, branch here. Moving them to deps did work with the linked package but not sure if the published version will behave differently.

Yes please, a swift fix for now is needed 🙏 Memory leak issue affects production applications.

I've reverted the bundling change so this is now a straight upgrade of sirv


Fix config and compile_env stuff for elixir 1.14 warnings

Pull Request Test Coverage Report for Build c0c9094b0141a2b774d46ce1030dd9f2a2353b22-PR-77

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

Totals

Change from base Build 352bc5f38bcc77a3501f1ea07fb65a79c1d4a7da:
0.0%

Covered Lines:
688

Relevant Lines:
688

💛  - Coveralls

Probably just remove config/prod.exs, config/dev.exs and config/test.exs files since this is a library package and probably only config/text.exs would exist ever which would be config/config.exs :2cents:

Yeah, ideally we should remove all config/ files - I think the remaining configuration can be in-lined into the test files, either in setup/2 blocks or module attributes.
I think there will also be some coveralls ignores to apply if we want to upgrade the latest Elixir version in the matrix 1.12->1.14 because of the extra coverage that we can blame @pmonson711 for 😛

Unfortunately Application.compile_env! is introduced in elixir 1.10, so that one will break for 1.7 as well

@the-mikedavis related to Elixir versions. from Ueberauth https://github.com/ueberauth/.github/blob/master/SECURITY.md

We do our best to follow Elixir Compatibility and Deprecations, we are committed to only support the latest version of Elixir.

I am not saying you should do it, but 🤷🏻 I would stick to the idea, last 2 or 3 versions

Yeah, updating minimum Elixir version would probably be ok. I like trying to "golf" the needed Elixir version for build since I have run into scenarios in the past where a library forced my hand for updating an application's Elixir version (and that's a bit annoying because then you need to make a larger change just to update the library version than you were otherwise planning).
It would probably be ok to at least bump the minimum version we test in CI to 1.9 though so that we can use Config consistently


Custom Properties support

Another useful feature could be to export object's Custom Properties metadata. This could be stored in the node.extras glTF2 field.
(Sorry for keep asking stuff, I just don't see this in the readme features table, and maybe is something you didn't think of)

No, please keep asking stuff! I just forgot it :-) We will add it.

Available since:
https://github.com/KhronosGroup/glTF-Blender-Exporter/commit/0835c60fffb153a0c4563c35ab72044bc5830d37

That was fast, thanks! :+1: