Min Max options don't work for Variable Type lists

When configuring a variable type list, like so:
import author from "../components/author";
import seo from "../components/seo";

const campaigns = {
  name: "campaigns",
  folder: "campaigns",
  label: "Pages",
  i18n: true,
  create: true,
  delete: false,
  fields: [
    author,
    seo,
    { label: "Title", name: "title", widget: "string", i18n: true },
    {
      label: "Publish Date",
      name: "date",
      widget: "datetime",
      i18n: "duplicate",
    },
    {
      label: "Featured Image",
      name: "thumbnail",
      widget: "image",
      i18n: "duplicate",
    },
    {
      label: "Landing Page Type",
      name: "lander",
      widget: "list",
      min: 1,
      max: 1,
      types: [
        {
          label: "Long Form",
          name: "long",
          widget: "object",
          i18n: true,
          summary: "{{fields.type}}",
          fields: [
            { label: "Layout", name: "layout", widget: "hidden", i18n: "duplicate", default: "long-form" },
            { label: "Title", name: "title", widget: "string", i18n: true },
            {
              label: "Publish Date",
              name: "date",
              widget: "datetime",
              i18n: "duplicate",
            },
            {
              label: "Featured Image",
              name: "thumbnail",
              widget: "image",
              i18n: "duplicate",
            },
            { label: "Body", name: "body", widget: "markdown", i18n: true },
          ],
        },
        {
          label: "Listicle",
          name: "listicle",
          widget: "object",
          i18n: true,
          summary: "{{fields.type}}",
          fields: [
            { label: "Layout", name: "layout", widget: "hidden", i18n: "duplicate", default: "top-list" },
            { label: "Title", name: "title", widget: "string", i18n: true },
            {
              label: "Publish Date",
              name: "date",
              widget: "datetime",
              i18n: "duplicate",
            },
            {
              label: "Featured Image",
              name: "thumbnail",
              widget: "image",
              i18n: "duplicate",
            },
            { label: "Body", name: "body", widget: "markdown", i18n: true },
          ],
        },
      ]
    },
  ]
};

export default campaigns;

The min max options don't do anything. I can still select as many of the Listicle or Long Form types as I want.
Expected behavior would be that after adding 1 total, of any of the two available types, I would receive an indication that I cannot add more.
This was introduced in PR #4394
I'm using manual initialization, there are no errors in the console or anything that would point to soemthing not working.
It just doesn't do anything at all.
PS: The CMS is manually initialized,  but I'm using the latest CDN version to do that, like so:
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>{{ site.name | title }} Content Manager</title>

    <script>
      // This global flag enables manual initialization.
      window.CMS_MANUAL_INIT = true;
      window.CMS_SITE_URL = "{{ site.url }}";
      window.CMS_DISPLAY_URL = "{{ site.url }}";
      window.CMS_LANGUAGES = "{{ site.cms.supported_languages }}";
      window.CMS_PUBLISH_MODE = "{{ site.cms.publish_mode }}";
      window.CMS_LOCALE = "{{ site.cms.default_language }}";
      window.CMS_BRANCH = "{{ site.cms.branch }}";

      {% if helpers.environment === "development" %}
      window.CMS_BACKEND_NAME = "proxy";
      window.CMS_LOCAL_BACKEND = true;
      window.CMS_PROXY_URL = "{{ helpers.netlify_proxy_url }}";
      {% else %}
      window.CMS_BACKEND_NAME = "git-gateway";
      {% endif %}
    </script>

    <!-- Include the Netlify Identity platform to authenticate CMS access -->
    <script src="https://identity.netlify.com/v1/netlify-identity-widget.js"></script>
  </head>
  <body>
    <!-- Include the script that builds the page and powers Netlify CMS -->
    <script<EMAIL_ADDRESS>    <script src="/assets/js/cms.js"></script>
  </body>
</html>

I will close this, as it turns out it works as expected. This was user error :)
The thing is, that wasn't clear to me at the beginning, Netlify CMS will throw error messages when validation fails, but only when attempting to save.
I simply assumed it would be happening in realtime.


How to match \u{4}?

I have some strings looks like this:
Ok("@\u{4}name\u{b}varchar(25)\u{2}NO\u{0}\u{0}")
Ok("@\u{2}id\u{19}int(10) unsigned zerofill\u{2}NO\u{0}\u{0}")

And I tried to remove that ' \u{2 } ' stuff.
The document says \u is a reserved pattern as follow
\u007F      hex character code (exactly four digits)
\u{7F}      any hex character code corresponding to a Unicode code point
\U0000007F  hex character code (exactly eight digits)
\U{7F}      any hex character code corresponding to a Unicode code point

So I try to escape that like this let re = Regex::new(r"@?\\u\{[0-9a-z]*?\}").unwrap();, but sadly that does not work.
Is there a way to match these things?

but sadly that does not work

When creating issues in projects, please never say "it doesn't work" and leave it at that. Because now I have to do the extra work if figuring out what you mean by "it doesn't work." Please show me how it doesn't work.
You can do that by providing a minimal Rust program. Tell me your expected or desired output and your actual output of that program.

but sadly that does not work

When creating issues on projects, please never say "it doesn't work" and leave it at that. Because now I have to do the extra work if figuring out what you mean by "it doesn't work." Please show me how it doesn't work.
You can do that by providing a minimal Rust program. Tell me your expected or desired output and your actual output of that program.

Sorry, my bad, here is the program.
use regex::Regex;

fn main() {
    let re = Regex::new(r"@?\\u\{[0-9a-z]*?\}").unwrap();
    let s = "@\u{2}id\u{19}int(10) unsigned zerofill\u{2}NO\u{0}\u{0}";
    // Expect s is not equal to the string after the replacing function
    // trying to remove "\u{2}" and similar stuff
    assert_eq!(s, re.replace_all(s, " "));
}

Thank you. This is exactly why you should always do your best to provide a compilable Rust program so that others can reproduce your exact problem, because your problem is not your regex. Your problem is what you're searching. You've written your haystack like this:
let s = "@\u{2}id\u{19}int(10) unsigned zerofill\u{2}NO\u{0}\u{0}";

But in Rust, \u{2} is itself a special escape sequence corresponding to the Unicode codepoint U+0002. So you either have to escape it:
let s = "@\\u{2}id\\u{19}int(10) unsigned zerofill\\u{2}NO\\u{0}\\u{0}";

Or use a raw string:
let s = r"@\u{2}id\u{19}int(10) unsigned zerofill\u{2}NO\u{0}\u{0}";

Playground link: https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=43f8049430375a3068913efae7bde303
Here's more information on Rust string literals: https://doc.rust-lang.org/reference/tokens.html#literals


Mldb 1841 distinct on

working on the doc

Very good test coverage.
reviewed briefly the c++ code
+1


roles-path for ansible-galaxy dependency cannot find paths relative to scenario directory

Issue Type

Bug report

Molecule and Ansible details
ansible --version
ansible 2.6.5

molecule --version
molecule, version 2.18.1

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Desired Behavior
I'd like to share the requirements.yml file across scenarios similar to what's documented for sharing other resources. I would think specifying a relative roles-file would work.
dependency:
  name: galaxy
  options:
    roles-path: ../resources/requirements.yml

Actual Behaviour (Bug report only)
Dependency is skipped because the roles file is not found. It looks like the path is relative to the working directory rather than the scenario directory. e.g. this works:
dependency:
  name: galaxy
  options:
    roles-path: molecule/resources/requirements.yml

Yes this can be a pain, specially because when you are configuring ansible options, they are relative to the Scenario, so it becomes very complicated to have paths where one goes from root folder up to the scenarios, and others relative to the scenario. Consistency is the key.

Scratch my last, it does work if you specify the path relative to the base role directory. I should trying reading the issue description ;) Still would be nice if this worked like other paths in the molecule.yml file.

I mean not your entire last @twbecker: Doing the role file like that doesn't work.
I would have thought that the molecule/<scenario>/requirements.yml would have been a default location that it would check first, too, without specifying one (similar to how prepare.yml works, etc.)
At least after this issue I can get things working by doing this:
dependency:
  name: galaxy
  options:
    role-file: molecule/default/requirements.yml

Just wish I didn't have to when it's in the scenario folder...


handle serialization/deserialization for prob fx dataframes

[ ] Closes #495 .
[ ] I am familiar with the contributing guidelines.
[ ] Tests added.
[ ] Updates entries to docs/source/api.rst for API changes.
[ ] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[ ] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[ ] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

So far I have a test that reproduces the failure.
@alorenzo175 I started trying to make serialize_timeseries work with DataFrames and it's turning out to be less straightforward than I thought it would be. The schema has 'column': 'value' - do we want to change this to 'columns': [{df.columns}] and then deserialize_timeseries has some conditional on this? Or do we want to let the calling functions serialize the DataFrame column by column, and the reverse for deserialize?
I can keep hacking at this but feel free to take over if you have a good idea of what to do.

The schema has 'column': 'value' - do we want to change this to 'columns': [{df.columns}] and then deserialize_timeseries has some conditional on this?

I think that is ok. maybe worth adding another key to schema to specify output should be dataframe or series then process column as appropriate. Definitely keep some data serialized with the current schema in the tests to make sure deserialize works with old and new

@alorenzo175 can you check this approach before I add the tests? Example below

index = pd.date_range(start='20200101', periods=2, freq='12h', tz='UTC')
s = pd.Series([1., 2], index=index)
df = pd.DataFrame({0.: [0., 1], 0.5: [0.5, 1.5]}, index=index)

s_str = utils.serialize_timeseries(s)
s_str

'{"schema":{"version": 0, "orient": "records", "timezone": "UTC", "column": "value", "index": "timestamp", "dtype": "float64", "objtype": "Series"},"data":[{"timestamp":"2020-01-01T00:00:00Z","value":1.0},{"timestamp":"2020-01-01T12:00:00Z","value":2.0}]}'

df_str = utils.serialize_timeseries(df)
df_str

'{"schema":{"version": 0, "orient": "records", "timezone": "UTC", "column": [0.0, 0.5], "index": "timestamp", "dtype": ["float64", "float64"], "objtype": "DataFrame"},"data":[{"timestamp":"2020-01-01T00:00:00Z","0.0":0.0,"0.5":0.5},{"timestamp":"2020-01-01T12:00:00Z","0.0":1.0,"0.5":1.5}]}'

utils.deserialize_timeseries(s_str)
timestamp
2020-01-01 00:00:00+00:00    1.0
2020-01-01 12:00:00+00:00    2.0
Name: value, dtype: float64

utils.deserialize_timeseries(df_str)

  | 0.0 | 0.5
-- | -- | --

0.0 | 0.5
1.0 | 1.5

A few things:

probably different enough to warrang version 1
does the column selection work? I would think there might be issues since the column has to be a string key in the records, but maybe it went in as a float, so a roundtrip should have float column labels.

I think the column keys should be string no matter and avoid precision issues, but that might require changes elsewhere

does the column selection work? I would think there might be issues since the column has to be a string key in the records, but maybe it went in as a float, so a roundtrip should have float column labels.

Creating a dataframe with string columns and then serializing leads to this json: "column": ["0.", "0.5"]
Whereas creating a dataframe with float columns and then serializing leads to this json: "column": [0.0, 0.5]
pd.read_json creates string column names, perhaps due to the presence of 'timestamp' or maybe that's just how it always works. In any case, the str_col_map = {str(col): col for col in schema['column']} does the job of ensuring the roundtrip works.
And column selection works in either case so long as you use the right type.

I think the column keys should be string no matter and avoid precision issues, but that might require changes elsewhere

api.get_probabilistic_forecast_values returns a DataFrame with float columns because it creates a dict with keys cv.constant_value as it loops over the forecasts. Adding str is easy there, though I guess I'm not sure if it's really the right thing to do or not.
calculator._transform_prob_forecast_value_and_prob performs indexing using the column as specified, and contains a couple lines with float(col) so a string would be covered here.
We might need to slightly change a couple of tests.
I think a couple of tests would need to be updated too.

Thanks. Go ahead and merge if you’d like or I can in an hour  or so.
On Tue, Jun 30, 2020 at 6:46 AM Tony Lorenzo<EMAIL_ADDRESS>wrote:

@alorenzo175 approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SolarArbiter/solarforecastarbiter-core/pull/496#pullrequestreview-440030783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBOER7GCUBVYJMEOUZBJIDRZHUD7ANCNFSM4OJXWMVA
.


Remove duplicate content

Remove duplicate content in "Considerations for managing accounts" section and tidy up formatting of remaining content accordingly

Hi @AndyHerb, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@squillace & @cynthn - VM pull request

@AndyHerb - Due to some refactoring, I can't merge this PR. However, I did make the update in the branch we are using for the refactoring - sorry we couldn't merge it directly.


[analyzer] Handle wrong clang version

Closes #2730

If clang was a symlink for example to ccache we tried to get the version
information of this binary. CodeChecker threw an exception in this case.
This commit will handle this use case properly.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- analyzer/codechecker_analyzer/analyzers/clangsa/clang_options.py  1

See the complete overview on Codacy

@gyorb This is also handled by this commit. CodeChecker --help command just show me the help message without exception.

I don't like this solution.
clang being a symlink to ccache is a common setup use case that should be handled.
Since  calling /usr/lib/ccache/clang works from the command line, it CodeChecker should be able to invoke it also. Swallowing the version detection error silently is also not so nice.
/usr/lib/ccache/clang --version
clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final)
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
iIt should w

I'm another user who couldn't figure out why CodeChecker crashed.  It started working with I took ccache out of my PATH.

Implemented by #3204.


Visibility of view

Not an issue but on loading of screen invisible method displays view and then makes it invisible .

when you use .start or .setNow ?

.setNow

Since setNow method  execute after draw ,i m setting visibility gone before that method and making it visible whenever needed

ok :/ I will try to change that


Issue 391 vector sum skipnil

Adds support for skipnil flag as discussed in issue #391 . There is some common code between add_skipnil and v2v_binary but I decided to keep these two separate to avoid performance penalty for all binary operation to support one scenario of vector_sum with skipnil. Suggestions welcome.

Ha ha. Didn't realize @baarkerlounger had created a PR for the same request. Perhaps @zverok and @v0dro can merge one. I am OK if the other PR #398 is merged.
In case this one is to be merged, please let me know, I would like to add documentation for add_skipnil. In the process of fixing this I realized that vector addition v0 + v1 suffers from this issue. So having a public method add_skipnil can give users an option to add two vectors. v0.add_skipnil v1.

Weird failure on Ruby 2.0 travis.
     Failure/Error: @cat_hash = categories.map { |cat, group| [cat, group.map(&:last)] }.to_h
     NoMethodError:
       undefined method `to_h' for [[:a, [0, 2]], [1, [1, 3]], [:c, [4]]]:Array

Anyone knows what that is? Looks like PR #387 is also seeing the same error.

@parthm it's the removal of backports completely in @zverok's PR. I fixed it here #398 by just requiring the specific backport needed.

Since this looks to be a better solution I'll close my PR.


Builds:Revert go version to 1.8.7

Fix #977
go v1.9.2 has this issue and flannel is involved. That cause some VMs cannot start flannel successfully.
https://github.com/golang/go/issues/22724
It has fixed in go v1.10.1
So i suggest revert go to 1.8.7
And i have test flannel built by 1.8.7 can operate on these VMs well.

Let's go forward rather than back - replaced by #1016


Simplify applying the Example

Simplify applying the Example by doing it via sed and using the correct file name.

Ping @vaikas-google


Add uncertainty to predicted responses

Figures showing yearly change (scripts/analysis-linear-model.R) should include measurements of uncertainty (maybe CIs) to accompany lines.

Example of adding standard errors to predictions in da9a347 (see scripts/analysis-linear-model-avgt.R).

Can be accomplished by passing se.fit = TRUE to predict().


Enhancement: Better keyboard focus

Nice extension! I've started playing around with it, and something light and straightforward is exactly what I was looking for. With that said, I have a few suggestions to improve its usability:

Clicking on the + button to add a task should set keyboard focus on it so I can immediately start typing.
Clicking on an existing item once should be enough to trigger editing mode, I don't see why I need to click twice.
Pressing Ctrl+Enter on editing a note could finish the edition and display the rendered formatting without having to click somewhere else.
After that Ctrl+Enter, maybe let people press Enter to resume editing that note (good for quick corrections).

Your suggestions are (mostly) feature complete in the  1.6 branch. There are just a couple remaining issues:

When a new task is made, it's in display mode for a split-second before switching to edit mode. This looks janky.
When focusing on a task, all of the text gets highlighted now. I'm okay with this behavior for Ctrl+Enter, but it's kind of weird when clicking on the task. I haven't found a native way to emulate the behavior of moving the caret to the cursor's position, and non-native ways I've thought up seem excessively intensive (computationally speaking) for such a simple UI action.

Anyway, I want to resolve those two issues before publishing these changes on the main branch.

Alright, the first issue is fixed.
As of right now I have no intention of changing the text-highlighting behavior from what I described previously. If caretPositionFromPoint ever becomes a browser standard and works as expected, then I may use that. Otherwise, there's too much work (for the CPU) for me to justify changing the behavior without feeling gross about it.

Thank you, much appreciated!


ngx-datatable: Update styling

Demo: http://swimlane.github.io/ngx-datatable/
Github: https://github.com/swimlane/ngx-datatable
Design: https://www.experience-hub.ch/document/2803#/organisms-module/tables
To do:

[ ] Selectable rows (with checkboxes)
[ ] Cell edition
[ ] Hover and active states
[ ] Responsive behavior
[ ] Active/selected cells and rows
[ ] Loading indicator with virtual paging

@gfellerph we need to discuss this again, before I'll do any change to this component.

Moved out of scope for v5.0. Will be implemented for a later feature (minor) release.

ngx-datatable does not seem to be actively maintained. We're looking for supporting another datatable library. Closing this ticket because of this.


Something In This Category Crashes Xcode

Not an issue with this category itself probably, but it seems that an invalid character somewhere in the comments here crashes Xcode. Just thought I should give you a heads up.
Upon further investigation, it seems to be this

Thanks! Going to correct it right now!

Ok! fixed and new cocoa pod released on version 2.0.1. Thanks!


Repo is not publishing latest image

The image on dockerhub is 8 months old.

yeah, we haven't set up any CI/CD yet. for now, it's a manual process.

Hi Tim, we have just pushed new images to https://hub.docker.com/r/raintank/graphite-mt/tags
the last one at this time is 9-cc00aa18ae-rebase-v2


MissingMethodException for uint24FromBytes(java.util.ArrayList$SubList, java.lang.Integer)

I'm using version 0.08 from date 2017-03-16. After couple of iterations of scanning and setting values I hit  Clean Up button. Now when I hit Scan General button I see following in Live logging:
da12f524-995b-43b4-9518-2984f6eef03b  22:13:44: error groovy.lang.MissingMethodException: No signature of method: static physicalgraph.zwave.ZwaveUtils.uint24FromBytes() is applicable for argument types: (java.util.ArrayList$SubList, java.lang.Integer) values: [[0, 56, 64, 1], 0]
Possible solutions: uint24FromBytes(java.util.ArrayList, int) @ line 1550
da12f524-995b-43b4-9518-2984f6eef03b  22:13:43: info Version Report: Application Version: 1.01, Z-Wave Protocol Version: 3.92, Z-Wave Library Type: 03 (Enhanced Slave)
da12f524-995b-43b4-9518-2984f6eef03b  22:13:41: info Powerlevel Report: Power: NormalPower, Timeout: 0
da12f524-995b-43b4-9518-2984f6eef03b  22:13:40: info Manufacturer-Specific Report: Manufacturer ID: 0115, Manufacturer Name: Z-Wave.Me, Product Type ID: 0100, Product ID: 0101
da12f524-995b-43b4-9518-2984f6eef03b  22:13:38: info scanGeneral(): Scanning for common device attributes.

The device is Z-Wave.me Secure Wall Controller. This is what I see I hit Print General button (I have reversed order of log entries):
General Information:
Device Name: Vypínač
Raw Description: zw:Ss type:1801 mfr:0115 prod:0100 model:0101 ver:1.01 zwv:3.92 lib:03 cc:5E,8F,73,98,86,72,70,85,2D,8E,80,84,5A,59,5B ccOut:20,5B,26,27,2B,60 secOut:62 role:04 ff:8B00 ui:8B01
Supported Command Classes: [SCENE_CONTROLLER_CONF (0x2D), ASSOCIATION_GRP_INFO (0x59), DEVICE_RESET_LOCALLY (0x5A), CENTRAL_SCENE (0x5B), ZWAVEPLUS_INFO (0x5E), CONFIGURATION (0x70), MANUFACTURER_SPECIFIC (0x72), POWERLEVEL (0x73), BATTERY (0x80), WAKE_UP (0x84), ASSOCIATION (0x85), VERSION (0x86), MULTI_CHANNEL_ASSOCIATION (0x8E), MULTI_CMD (0x8F), SECURITY (0x98)]
Security: Device is paired securely.
=> Command classes supported with security encapsulation: UNKNOWN (0xNULL)
=> Command classes supported for CONTROL with security encapsulation: UNKNOWN (0xNULL)
Manufacturer ID: 0115
Manufacturer Name: null
Product Type ID: 0100
Product ID: 0101
Firmware Metadata: Firmware ID: null, Checksum: null
Application (Firmware) Version: 1.01
Z-Wave Protocol Version: 3.92
Z-Wave Library Type: 03 (Enhanced Slave)
Powerlevel: NormalPower
Protection Mode: [ Local: null (null), RF: null (null) ]
Switch_All Mode: null (null)

Discovery Stats:
Number of association groups discovered: 5 [Print Assoc Groups]
Number of endpoints discovered: 0 [Print Endpoints]
Number of parameters discovered: 31 [Print Parameters]
Number of unique command types received: 10 [Print Commands]
Number of MeterReport types discovered: 0 [Print Sensor]
Number of NotificationReport types discovered: 0 [Print Sensor]
Number of SensorMultilevelReport types discovered: 0 [Print Sensor]

Interesting, this looks like an issue within the physicalgraph.zwave.commands.securityv1.SecurityMessageEncapsulation class itself. Line 1550 calls the encapsulatedCommand method of this class.
Best to post on the SmartThings community forum and tag Duncan.

In fact, this is already on Duncan's issue list, see here: https://community.smartthings.com/t/wakeupintervalget-doesnt-work-uint24frombytes-missing-from-response/10577/4
Basically, it looks like there is a bug in hysicalgraph.zwave.commands.securityv1.SecurityMessageEncapsulation which means it can't decrypt WakeUpIntervalReports.
So this is an issue with the SmartThings core zwave library, there's nothing I can do to fix it in my device handler.

I posted here also: https://community.smartthings.com/t/z-wave-library-changes-feb-2017/79021/11

Thanks! I haven't realized that this could be an issue with the platform.


fix(pay): dont focus on value input

Description:
This PR fixes a bug where Zap crashes when a user tries to send to an on-chain address

Motivation and Context:
When entering an on-chain address into the Pay form the Pay form goes blank. This is because it is trying to focus on amountInput which does not exist.
How Has This Been Tested?
You can reproduce the issue on master by simply pasting an on-chain address in the pay form and watch the app crash.
If you checkout to this PR and paste an on-chain address the app acts normally.
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[] I have added tests to cover my changes where needed.
[] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.06%) to 16.173% when pulling 161fec013326c4069c630e70ebe4ba282dc0f736 on fix/pay-onchain into 07a070b8cf4e619d19ab42af57c864b3a0267875 on master.

Coverage decreased (-0.06%) to 16.173% when pulling 161fec013326c4069c630e70ebe4ba282dc0f736 on fix/pay-onchain into 07a070b8cf4e619d19ab42af57c864b3a0267875 on master.


Cleanup Jenkins 2 GH Actions Migration

What does this pull request do?
During the Jenkins 2 GH Actions Migration https://github.com/elastic/apm-agent-python/pull/1731 we decided to keep the Jenkins pipeline intact and only remove Jenkins related things after we proved that everything is working as expected in GH actions.
TL;DR
Removes Jenkins related configs but keeps benchmark testing, which will be handled in a follow-up.

Related issues

https://github.com/elastic/apm-agent-python/pull/1731

LGTM
NIT, setting the ONLY_DOCS env variable can be also removed
            script {
              dir("${BASE_DIR}"){
                // Skip all the stages except docs for PR's with asciidoc and md changes only
                env.ONLY_DOCS = isGitRegionMatch(patterns: [ '.*\\.(asciidoc|md)' ], shouldMatchAll: true)
              }
            }

https://github.com/elastic/apm-agent-python/pull/1769/commits/0606a13dd02aa9cad07aa2ad2833db7cce956685


[SSoC 2.0] Create a Readme for Projects

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

Readme for the Projects folder to enlist all the projects

Describe the solution you'd like

By adding a README.md file in the projects folder

@TusharKesarwani @zugzwang03 sir, please assign this task to me


countries: No schema

The list of countries contained in countries.json is schema-less, and a schema should be defined for them.

Done in #10


unable to load

Added to HACS, then modified json to add version number. I can then attempt to ad integration (aka it is in the list), attempting to add it errors on work flow, below is the error log.

Version
core-2021.8.7

Installation Type
Home Assistant OS

Development
false

Supervisor
true

Docker
true

user
root

Virtual Environment
false

Python Version
3.9.6

Operating System Family
Linux

Operating System Version
5.10.17-v8

CPU Architecture
aarch64

Timezone
America/New_York

Logger: homeassistant.config_entries
Source: config_entries.py:698
First occurred: 11:23:11 AM (3 occurrences)
Last logged: 11:29:18 AM
Error occurred loading configuration flow for integration vlc_telnet: No module named 'homeassistant.components.vlc_telnet.config_flow'
Error occurred loading configuration flow for integration ecowitt: cannot import name 'VOLT' from 'homeassistant.const' (/usr/src/homeassistant/homeassistant/const.py)

resolved by changing VOLT to other variable


[bug] MQTT no longer respect QOS or retain setttings

Version
Checklist:

[ ] I am not using HomeAssistant. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog problem was not mentioned there.

Build/Run method

[X] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - yarn install - yarn run build )

zwavejs2mqtt version: 4.2.1
zwave-js version: 7.4.0
Describe the bug
The mqtt publish function does not respect either the QOS setting or the retain flag in the config after upgrading to 4.2.1 from 4.0.1. All MQTT messages, when published will be QOS: 0 and Retain: false even though in the SILLY logs it says the message was sent with QOS: 0 and Retain: true.
To Reproduce
Steps to reproduce the behavior:

Go to Control Panel set MQTT QOS setting to 2 and Retain to True,
Save Settings
Change a state or re-discover a device. The resulting MQTT message in the broker will show QOS 0 and retain false

Expected behavior
QOS and retain settings should be respected
Additional context
I believe this is related to After pull #1166. But since it was such a major re-write, and I'm not proficient at TS, I cannot figure out the exact problem, but I believe it is in how the options are handled when they are published

I spent sometime trying to dig into this, everything seems right up to MqttClient.js : 172 . However I am not following how to go past that.
In the logs I can clearly see this message:
2021-05-15 14:09:45.453 INFO ZWAVE: Node 8: value updated: 38-0-duration [Duration: 0 seconds] => [Duration: 0 seconds] 2021-05-15 14:09:45.454 DEBUG MQTT: Publishing to ZwaveJs700/Dining_Room/DR_Ring_Dimmer/38/0/currentValue: { time:<PHONE_NUMBER>454, value: 97 } with options { qos: 2, retain: true }
But the MQTT message is QOS 0 retain false.

sigh, this seems to have resolved it self, or I'm losing my mind... either way I'll close this issue.

Let me know if you expect this again


Wrong behavior of a Parallax after changing the size of the browserwindow

Hi,
I recentely discovered an unnsual behavior of the Parallax component.
If you zoom out or use a 4k screen everytime you resize the page for more than a bit the parallax image does not get resized as well. Only when you start scrolling and trigger a scrolling event I suppose the image gets aligned corretely
Expencted Behavior
I expected the top image to fill its own space after the reisze, as it does as soon as you scroll out.
Steps to Reproduce

zoom out to of the codepen to like 30% on a 1080p screen,
reload and resize the page abruptly (opening developer tools)

Codepen
Somehow the issue can only be seen in firefox on codepen, but exists in every browser in my html.
The issue is also apparent in your parallax demo in chrome, firefox and sometimes in edge.

When I reduce the browser window width the height of the parallax-container remains the same. However white space appears above the image, pushing the image down and out of the parallax.
With a smaller image and larger window this can lead to a large white space appearing above the image, and a small slither of the image being displayed


In this issue I suggest to add either your email or your phone number on the same text input of the SIGN IN card  because there exist people that does not use email just they use their email just they use their phone number

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
I have a sign-in page that contains email and password so I suggest to add also a phone number on the same text input of the email and this is the code in Js that I suggest:
// Assuming you have a single input field for email/phone
const inputField = document.getElementById('email_or_phone');
if (value.length === 0) {
// If the input field is empty, show an error message
alert('Please enter your email or phone number');
return;
}
if (isValidEmail(value)) {
// If it's a valid email, sign in the user using email
signInWithEmail(value);
} else if (isValidPhoneNumber(value)) {
// If it's a valid phone number, sign in the user using phone number
signInWithPhoneNumber(value);
} else {
// If it's not a valid email or phone number, show an error message
alert('Please enter a valid email or phone number');
}
});
I suggest to more update this code so it's going to more help the users
Motivation and context
It is used to give an option for people they don't use email just use their phone number to sign in with it and it provides more options for the users

I'm not sure to catch what's suggested here. Do you suggest improving the Sign-In example in our documentation?

Sounds spammy or unrelated to us. Closing and locking.


[Basic] Improve the class component defaultProps example

What cheatsheet is this about? (if applicable)
Basic cheatsheet
--
In the current example, we have a type definition for Props but do not make use of it for defaultProps. This can lead to a mismatch between the actual fields in defaultProps and what the Props type defines.
Adding an explicit Partial<Props> type to defaultProps seems like good example code.

good eye! thanks very much!

@sw-yx Actually, I just checked, and the caveat that is stated under "Typescript 2.9 and earlier" still applies - defaultProps should not be typed as Partial<Props>, because that causes all component props to be considered optional in JSX (tested with typescript: 3.4.1 and react: 16.8.6.
Another approach would be to use:
static defaultProps: Pick<Props, 'name'> = { name: 'world' }`

What do you think?

A good way of working with defaultProps is to actually reverse the props definition so that it uses the defaultProps type and adds additional types:
type Props = typeof MyComponent.defaultProps & {
  age: number
}

class MyComponent extends Component<Props> {
  static defaultProps = {
    name: 'world'
  }
}

o shit lets do that instead! much nicer.

I ran into a case where the typeof MyComponent.defaultProps pattern did not provide good ergonomics:
// in a separate file
interface ModelValues = {
  title: string
  description?: string
}

// in a custom formik form
type Props = {
  onSubmit: (values: ModelValues, actions: FormikActions<ModelValues>
  initialValues: ModelValues
}

class ModelForm extends React.Component<Props> {
  static defaultProps: Pick<Props, 'initialValues'> = {
    initialValues: {
      title: ''
      description: ''
    }
  }
  render() { ...}
}

The reason to want to have a type definition directly on static defaultProps is that it protects me from omitting some value from the defaults, especially when dealing with more complicated props. Also, when there are type errors, they will directly show up in this file, instead of in the call sites.

i see you @RiaanWest. wanna PR something?

This will break LibraryManagedAttributes inference for that component. It will cause all of its props to be optional when creating JSX elements of that component.
export type IconSize = 'small' | 'regular' | 'medium' | 'large';

interface IconPropsInterface {
  icon: string;
  size?: IconSize;
}

export class Icon extends Component<IconPropsInterface> {
  static defaultProps: IconPropsInterface = {
    icon: '',
    size: 'regularrr',   <-- Throws: Type '"regularrr"' is not assignable to type '"small" | "regular" | "medium" | "large" | undefined'.ts(2322)
  }
}

const myIcon = <Icon /> // this will not throw an error even though `icon` is required,

Thanks for pointing this out @ferdaber - great point. I wasn't aware of that!


[Proposal] Improve performance with Torch.compile

Proposal
Seeing as we use a lot of einops, using torch.compile should result in a good performance boost. Note we need to tell einops it is safe to do this and also have a check that the user is on torch 2.x .

I'm testing this now with some improved DLA & residual decomposition code

Unfortunately it doesn't work with hooks yet


Ops added after calling start_queue_runners() are broken

(Using the OSX CPU-only  v.0.6 wheel).
I think this is a minimal example:
image = tf.constant(0)
images = tf.train.shuffle_batch([image], batch_size=1, capacity=100, min_after_dequeue=1)

a = tf.constant(1)
with tf.Session():
    b = a+1
    tf.train.start_queue_runners()
    c = a+1

    print(a.eval())
    print(b.eval())
    print(c.eval())

It usually crashes on the c.eval() giving the following error:
Traceback (most recent call last):
  File "test.py", line 11, in <module>
    print(c.eval())
  File ".../python2.7/site-packages/tensorflow/python/framework/ops.py", line 460, in eval
    return _eval_using_default_session(self, feed_dict, self.graph, session)
  File ".../python2.7/site-packages/tensorflow/python/framework/ops.py", line 2910, in _eval_using_default_session
    return session.run(tensors, feed_dict)
  File ".../python2.7/site-packages/tensorflow/python/client/session.py", line 368, in run
    results = self._do_run(target_list, unique_fetch_targets, feed_dict_string)
  File ".../python2.7/site-packages/tensorflow/python/client/session.py", line 446, in _do_run
    six.reraise(e_type, e_value, e_traceback)
  File ".../python2.7/site-packages/tensorflow/python/client/session.py", line 428, in _do_run
    target_list)
tensorflow.python.pywrap_tensorflow.StatusNotOK: Not found: FetchOutputs node add_1:0: not found

Even just a clearer error message in this situation would be a big help.

The issue is that start_queue_runners() starts (one or more) threads that are running portions of the graph.  Since graph modification is not thread-safe, you will see problems if the graph is modified after start_queue_runners().  @mrry do we have a way of making this an error?

We could use tf.Graph.finalize(), which is intended for such purposes. To the original point: the tf.Graph methods for adding ops are definitely not thread-safe, and it would require a big redesign to achieve that (with only a small upside).

An error would be nice, it could save other beginners a headache.
Thanks for the clarifications.

Sounds like start_queue_runners should call finalize (possibly with a flag to turn it off if you really know what you're doing)?

@josh11b: Should I reassign?

I think this is fixed since acac487ac4ebaa6edb3e3f866d41cbd12546a107.


Dimensional Transceivers GUI: Channel list selection off while Minecraft at half screen

Issue Description:
While playing with game taking up half the screen, the ability to chose an option from the channel list in the dimensional transceivers GUI is doesn't square up with the visual listings of the channels.  Nothing happens when you click on the channels, you have to click off to the right side of the channel name a lot until you can find where the clickable area is or type the channel in the add channel box for it to bring you down to where you have it.
What happens:
Not able to click directly on the channel in the dimensional transceiver GUI, it's off to the right.
What you expected to happen:
Same as in fullscreen, able to click directly on the desired channel.
Steps to reproduce:

Resize Minecraft to the half screen size (left/right side of the screen)
Open dimensional transceiver GUI to energy/item/fluid tabs (where the channels are listed)
Attempt to pick a channel that is already added.
...

Affected Versions (Do not use "latest"):

EnderIO: <IP_ADDRESS>_beta
EnderCore: <IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge:

Your most recent log file where the issue was present:
Not necessary

You did not specify the forge version. Does it still happen with the latest version of EnderIO/Core on current forge?

Forge is <IP_ADDRESS>2
Yes, the problem persists in the latest releases
EnderIO: 3.1.156
EnderCore: <IP_ADDRESS>-beta
I forgot to add: using the small GUI setting I was not able to find the bounding box at all.


VEGA-897 Move failing test to ignored pact to unblock sirius

Codecov Report

Merging #57 (4b5864c) into main (7711050) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   88.92%   88.92%
=======================================
  Files          30       30
  Lines         813      813
=======================================
  Hits          723      723
  Misses         47       47
  Partials       43       43

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 7711050...4b5864c. Read the comment docs.


Samsung pay -> google pay doesn't work (wear os 3.2)

Samsung galaxy watch 4 and wear os 3.2. Samsung pay to google pay stopped working... tried with 2 different watches and same thing.

See https://github.com/BigBoot/GW4Remap/issues/10#issuecomment-1191855225


disk: Feature http://lv2plug.in/ns/ext/state#makePath is not supported

I get this message when running in jalv.gtk.
As for the cause, @x42 wrote on IRC:
make_path is a feature passed to  state_restore, state_save.  not instantiate

According to http://lv2plug.in/ns/ext/state/#makePath a host can do both. Ardour passes it via instantiate, jalv only to the state methods.

@magnetophon this is actually a bug in jalv. eteroj#disk requires this feature [1], so a host not supporting it (here jalv) should not try to load it in the first place.
I need a file path to stream events to disk from start (e.g. in instantiate). I could stream to a tempfile and then upon state#save copy the file, but LV2 plugins should not save anything outside their allotted disk space.
Don't see the best way to fix this, yet
[1] https://github.com/OpenMusicKontrollers/eteroj.lv2/blob/master/eteroj.ttl#L144

Sorry for responding so late, this one fell trough the cracks.
@x42 Do you have any ideas for a fix?

Sorry for responding so late, this one fell trough the cracks.
@x42 Do you have any ideas for a fix?

jalv? that's @drobilla territory.

jalv? that's @drobilla territory.

@magnetophon the eteroj:disk plugin has been deprecated in version 0.6.0, so there won't be any fix needed.

@magnetophon the eteroj:disk plugin has been deprecated in version 0.6.0, so there won't be any fix needed.

A very long time ago (nearly a decade) Jalv didn't refuse to instantiate plugins that required features it does not support, but that shouldn't be the case now.  If not, please file an issue.
Re: passing makePath in those places, both are valid but they have different use cases.

A very long time ago (nearly a decade) Jalv didn't refuse to instantiate plugins that required features it does not support, but that shouldn't be the case now.  If not, please file an issue.
Re: passing makePath in those places, both are valid but they have different use cases.


chore(release): fix npm release script

Short description of what this resolves:
There is a "typo" in package.json concerning the release script.
npm changelog is not an npm command but a script and thus has to be invoked as npm run changelog instead of npm changelog

@masimakopoulos,
Can you rebase and update? Sorry, I just noticed this now. I'll merge once you do. I want you to get credit so it's easier if you fix the merge conflict.
Thanks,
Dan

@danbucholtz is this ok?

@masimakopoulos, No 😄 but I don't know how to fix it either. I'll chat with Adam about it and see how we can merge it so you get credit! Thanks for doing this!
Thanks,
Dan

@masimakopoulos,
This has diverged a bit so I am just going to close it. Thank you so much for helping us out.
Thanks!
Dan


Add AWS Neuron metric origin

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge

/merge


[WIP] Remove ConfigProvider, many unit tests are broken

Fixes #329
Changes proposed in this PR:

I love the simplification. Why haven't we done this earlier?

I love the simplification too!  I'm guessing this change wasn't done earlier because no one was looking at it or it just wasn't high on the priority list.

Is there any discussion on how this will affect other components?

Aquarius uses ConfigProvider, and get_contracts_addresses which relies on ConfigProvider. So yes, removing ConfigProvider will affect Aquarius, but the effect is small.

I am working on removing ocean.py from Aquarius, but is it a blocker?

I believe your work to remove ocean.py from Aquarius is going to also remove ConfigProvider so it has a low risk of directly conflicting with what I'm doing. So no, I wouldn't call it a blocker.

Does the Provider (repo) use the ConfigProvider (class)?

No, Provider doesn't use the ConfigProvider, nor does it use any functions that will be affected by removing ConfigProvider.

This PR includes non-backward-compatible API changes. Since ocean.py is still <1.0.0, I believe it is okay to merge this to main (as indicated by the PR target branch) and release it with an incremented minor version number.
If anyone disagrees, please let me know.  @alexcos20 @trentmc @calina-c @AnaLoznianu

This is fine by me.


chore: bdd-spring-1580714887 to 0.0.1

chore: Promote bdd-spring-1580714887 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Define :commands.

This makes it much nicer to set up custom keybindings.  I've also taken the liberty to fix some corner cases with the infoview commands (e.g. toggle didn't reopen it if you closed it with :q).

I'll remove the __maybe_autoopen calls. If I understand the invariant correctly, every lean(3) buffer should have a get_current_infoview, right?

Good question -- autocmd BufEnter ... will trigger it, so before it was unnecessary since all the mappings to these calls were buffer-local. But now it looks like you want them to be able to be called from anywhere? So they may in fact be necessary at this point...


Added methods for getting user's subscribed, contributed to, and moderated, subreddits

Implements:

GET subreddits/mine/contributor
GET subreddits/mine/subscriber
GET subreddits/mine/moderator

Coverage decreased (-0.99%) to 48.62% when pulling 756bc5b09c487a2422b9be27419bd86dda7b613d on mleef:master into 667b3f6c97f0e26a79806ab06ebc5f7c8cf5e618 on karan:master.

Coverage decreased (-0.99%) to 48.62% when pulling c1b53cdb3192e2a98f72b82daad5fe57d8d56c07 on mleef:master into 667b3f6c97f0e26a79806ab06ebc5f7c8cf5e618 on karan:master.

Coverage decreased (-0.99%) to 48.62% when pulling c1b53cdb3192e2a98f72b82daad5fe57d8d56c07 on mleef:master into 667b3f6c97f0e26a79806ab06ebc5f7c8cf5e618 on karan:master.

Coverage decreased (-0.99%) to 48.62% when pulling c1b53cdb3192e2a98f72b82daad5fe57d8d56c07 on mleef:master into 667b3f6c97f0e26a79806ab06ebc5f7c8cf5e618 on karan:master.

@mleef can we try adding some unit tests for this puppy. Anyway, thanks for the contribution !

@sfat Will do, thanks for the feedback.

Updated methods by adding limit parameter and unit tests for each, not sure why the build is failing, all of the tests are working locally...

@mleef think it was some hiccup from travis. I ran the build again and it seems to be okay. Merging. thanks!

Coverage increased (+0.1%) to 49.72% when pulling bccc3d43506b30889b9142236163eca01a6b2c71 on mleef:master into 667b3f6c97f0e26a79806ab06ebc5f7c8cf5e618 on karan:master.


Eliminate redundancy in configuration and prerequisite sections in the README.md

There should be only one section for configuration and pre-requisite.

refer to this be3b402


Added postcss-modules-local-by-default dependency to package.json

I was getting webpack issues: Error: Cannot find module 'postcss-modules-local-by-default', so I added that dependency to the root package.json file.

It was unused. Fixed now. Thanks


My students: Add homeroom column and filter

Who is this PR for?
K8 educators with schoolwide access, came up for folks in Bedford
What problem does this PR fix?
No obvious way to filter by homeroom on the "My students" page
What does this PR do?
Adds a column and supports sorting, filtering, and searching.
Factors FilterStudentsBar out from my_students into components, since it was already being re-used by the counselor meeting page.  Updates tests for new cases, and for regressions on older cases.
Screenshot (if adding a client-side feature)

Checklists
Which features or pages does this PR touch?

[x] My students
[x] Counselor meetings

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

[x] Included specs for changes
[x] Improved specs for existing code in need of better test coverage
[x] Manual testing made more sense here

selfie


decouple SelectFeatures and store update

Right now, features are updated in the maplibre map object, and then in the zustand store, within SelectFeatures. These should be decoupled such that the map object is updated while painting occurs, and setting the store happens onMouseUp or something similar to get rid of the need to debounce.

closed by #22


Update osgEarth

OSG has been updated (see #3814)  to 3.6.2 which breaks compilation of osgEarth.
This PR fix it.

Thanks for the fix!


issue-208 Add support for Custom POD Annotations

Change log description
Updates the Helm chart to allow custom annotations to be expressed on the final Schema registry PODs
Purpose of the change
Fixes #208
What the code does
Provides a podAnnotations value on the Helm chart allowing custom POD annotations to be expressed.  The value of the podAnnotations is copied to the final deployment template
How to verify it
Using the following values file called "values.txt"
podAnnotations:
  first: one
  second: two

helm install schema-registry charts/schema-registry --values values.txt
The following shows the final output:
spec:
  progressDeadlineSeconds: 600
  replicas: 2
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      app.kubernetes.io/name: schema-registry
  strategy:
    rollingUpdate:
      maxSurge: 25%
      maxUnavailable: 25%
    type: RollingUpdate
  template:
    metadata:
      annotations: <<<<<<<<<<<<<<<<<
        first: one
        second: two
      creationTimestamp: null
      labels:
        app.kubernetes.io/name: schema-registry

With no custom POD annotations specified:
  template:
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/name: schema-registry
    spec:

README updated and all commits signed-off


Import Map to Local File

:wave: Hello!
I'd like to be able to set up an import map to a 'local' file (local to the playground, that is - meaning a file that the playground includes)
Consider:
{
  "files": {
    "index.html": {
      "content": "<script type=\"module\" src=\"script.js\"></script>"
    },
    "script.js": {
      "content": "import '@x/y/z';"
    },
    "xyz.js": {
      "content": "console.log('hidy-ho');",
      "hidden": true
    }
  },
  "importMap": {
    "imports": {
      "@x/y/z": "xyz.js"
    }
  }
}

In this example config, I'm attempting to alias the bare module specifier @x/y/z.
I expect to see 'hidy-ho' logged to the console
I actually see this:
Import map key "@x/y/z" is invalid because address "xyz.js" is not a valid URL. playground-project.js:139:17

Same results for ./xyz.js as the import target.

I solved this (at least, for my case) with this diff:
diff --git a/node_modules/playground-elements/playground-project.js b/node_modules/playground-elements/playground-project.js
index e1f386c..489fc59 100644
--- a/node_modules/playground-elements/playground-project.js
+++ b/node_modules/playground-elements/playground-project.js
@@ -637,7 +637,7 @@ const validateImportMap = (importMap) => {
                 ` address "${resolutionResult}" must end in a forward-slash.`);
         }
         try {
-            new URL(resolutionResult);
+            new URL(resolutionResult, import.meta.url);
         }
         catch {
             errors.push(`Import map key "${specifierKey}" is invalid because` +

Hello,
I was wondering if there are any updates on the possibility of referencing a local folder project? It would be incredibly beneficial, particularly for those of us working on libraries with components.
Imagine this scenario: you're developing a library with various components, and you're eager to showcase them in the sample demo pages within your documentation. In such cases, having the ability to reference a local folder project becomes invaluable. The current way of loading the library from unpkg presents a challenge, as it hinders the seamless development of new features while simultaneously building example pages for them.
Looking forward to hearing your thoughts on this!

Hi @adrian-bobev. No promises but this requirement also came up independently in another project I'm working on, so there's a chance I'll be working on it soon. Maybe sometime in January.

hi @aomarks any updates on local file imports?


Converting .mat files to .fif files

Hello all, @agramfort
I am facing trouble using mne-python as all the EEG readings are in .mat files, and mne-python works with the .fif files.
Can anyone suggest how to convert this data into the fif format?
The pymatreader library allows the .mat files to be loaded and returns a python dictionary, but that isn’t the fif format.
Below is the dictionary data of one of the readings.
Any help or lead is highly appreciated.
Thanks,
Poorva
MNE-Python version: latest
operating system: Windows
{‘header’: b’MATLAB 5.0 MAT-file, Platform: GLNXA64, Created on: Tue Jul 11 15:16:19 2017’,
‘version’: ‘1.0’,
‘globals’: [],
‘data21_a’: array([[[ -4.02996501, -5.21812053, -2.12189425, …, -5.24047166,
-23.1109507 , -0.07900585],
[ -4.07999741, -4.1793486 , -5.30395057, …, -8.2638478 ,
-18.6511773 , -4.79279383],
[ -3.14417591, -5.76256529, 1.26975923, …, -8.32801818,
-11.31398043, -8.08436272],
…,
[ -0.89876645, 4.29588313, 0.57285615, …, -0.99713776,
1.13475826, -14.90647201],
[ -2.46851891, 5.69858911, -0.50480844, …, -2.37357903,
0.98512574, -9.60950669],
[ -0.69313008, 7.1196399 , -0.15364449, …, -3.54282948,
-1.49179595, -7.06937039]]]),
‘fs’: 125,
‘subs21_a’: [‘S1’,
‘S4’,
‘S6’,
‘S9’,
‘S10’,
‘S11’,
‘S14’,
‘S19’,
‘S28’,
‘S43’,
‘S56’,
‘S58’]}

This is a usage question, see my response here: https://mne.discourse.group/t/converting-mat-files-to-fif-files/3420/2?u=cbrnr


tests(performance) - eliminate small deltas from automated PR annotations

Problem:
Our perf tests show all results, and we suppress details if the absolute percentage difference is less than 5. This generates a spammy level of lines. We already show all results in the perf graph anyway (though maybe even that is overkill)
Fix:
Only show lines where abs delta is > 5%
NB: rage fix. Also, Gresham's Law stimulus for a / a better solution

Does this completely eliminate all of the graphical chart generation?


[modify] tryb rescues anything

Coverage decreased (-0.05%) to 83.805% when pulling b87c652d9c8ba9802272dbccd0416922c7ceda21 on sunny4381:features/tryb-rescues-anything into ff5c461a103a10209ddc41c4c71d9d4a4593b0a9 on shirasagi:master.


repeat-until with check before first iteration?

Hi!
If I understand #145 correctly, the repeat-until pattern always executes the step once. Is there an elegant way to change that behaviour such that the condition is checked first?
And I am not sure if I missed sth. in the documentation, but a small hint about the current behaviour would be useful.

Your understanding is correct. IIRC the main thing that made us drop while/repeat semantics is that we didn't find a nice way to express it in the DSL. Looking back at it now, I guess repeat/while is the best option, but while is a scala keyword and therefor would needed to be quoted. How about this:
trav.repeat(_.out)(_.until(_.hasLabel("FOO"))   // only checks after the first iteration - as before
trav.repeat(_.out)(_.`while`(_.hasLabel("FOO"))  // also checks at the first iteration
trav.repeat(_.out)(_.whilst(_.hasLabel("FOO"))    // same as above, simply given it a different/alternatiuve name to avoid quoting

Thank you for the info! Avoiding quotes sounds good.

Thanks for bringing this up, it makes sense to add and doesn't look too hard. I'll take a look in the next few days.

implemented in https://github.com/ShiftLeftSecurity/overflowdb/pull/227

@mal-tee looks good to you?

I'm in no position to judge the implementation, but the usage in the tests looks good to me! Thanks!


[Cryptography]: Implement Stark Curve/Signer

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

Immutable X Blockchain require to register an existing eth key with a stark key in order to use their blockchain API

Describe the solution you'd like

C++ Caigo/Stark Curve
C++ Caigo/Stark Signer
Modern C++17/20

Checklist

[ ] Caigo/Stark Curve

[ ] C++ Logic
[ ] C Interface
[ ] C++ unit tests
[ ] Swift unit tests
[ ] Kotlin unit tests

[ ] Caigo/Stark Signer

[ ] C++ Logic
[ ] C Interface
[ ] C++ unit tests
[ ] Swift unit tests
[ ] Kotlin unit tests

Resources
Code Logic in Golang
Code Logic in Rust

Not needed anymore, can re-open later if we implement through trezor-crypto


Add more checks ValidateBasic message method

ValidateBasic of sdk.Msg allows performing static checks for an sdk message.
These checks are performed before the message is broadcast to the network, therefore it is more optimized to have these checks in ValidateBasic rather that in the message handler to save gas for txs bound to fail.
This PR adds some more checks in message validate basic like contract address check or coins validity checks

Super awesome! I just approved the CI for this and thanks so much for this PR.


Unexpected pattern match behaviour

Using the test data:
 - - - 2022-12-23T05:25:05.584Z ERROR vsan-mgmt[20337] [VsanHttpRequestWrapper::urlopen opID=noOpId] Exception while sending request : <urlopen error timed out>
and the pattern:
 - - - %{TIMESTAMP_ISO8601} %{NOTSPACE:event_severity} %{PROG:event_process}\[%{INT:event_pid}\] \[%{NOTSPACE:event_thread_name}( sub=%{NOTSPACE:event_sub_thread})? (?i)opid=%{NOTSPACE:event_operation_id}\]
Matches up until the final sementic (event_operation_id).  This field is not populated and any further fields are right-shifted.  The exact same test data on multiple other grok debuggers does not give the same result.

Hi, thanks for letting me know. I'll have to dig into this. It's probably an issue with the dependency https://github.com/cjslack/grok-js-web, which was forked from https://github.com/Beh01der/node-grok
I would avoid using flag modifiers for now. Alternatively, for your specific example you can do something like this to achieve the desired result:
op(id|ID)=%{NOTSPACE:event_operation_id}\]

Full example here


Fixes for [builtin-yosys] on Mac M2, latest OSX

Sorry, I cannot accept a PR whose description is just "fixes" without specifying what it is that is being fixed.


sending and receiving ODID data simultaniously

I have been trying to put together the Id_scanner and Id_open codes to both receive and transmit data simultaneously mostly via Wifi-NAN. They seem to have different wifi settings for either rx or tx. Is it a way to make them work simultaneously on the same ESP32 board or is it doomed to fail?

I don't know. I have never looked at doing this. I wrote the scanner because I don't have an Android device that will receive NAN and don't have an app that will receive French style beacon frames.
I don't like the way that the id_open library sets up the ESP32 Wi-Fi and have been meaning to see if there is a better way to do it.

If it gets set up in a way that works with the promiscuity setting for the scanner, then it might work. Because I´m getting readings from other nodes now but get an error message every time squitter.transmit executes.

There seem to be a way to send on 802.11 with esp_wifi_80211_tx() and that is while promiscuity is running.


Create a single promise in streamPromise.

I'm creating the pull request to expose a bug when the destination does not close (stdout & stderr).

Note that this passes all tests at a91130c, but it clearly does not handle errors from source or transform streams at all.

I think this issue was resolved by #10 since we ignore events on the stdout/stderr streams and wait for other streams in the chain to finish (this is also how it worked before #8 and I changed it back now).
If you agree it's ok now, update the PR to only include the additional test and I'll merge it.

@MartinKolarik I pushed a fix that still creates a single promise per pipeline (streams array) instead of one promise per stream.
If you like the change, great. If not, just cherry-pick b7125a1 instead. That commit is only the new test.

Ok, I think I'll cherry-pick just the test for now and check this together with #2 later.


Improvement suggestion on Time Picker

I noticed that on the time picker if you enter a value in the 'minutes' field that begins with a '0' that it omits the '0' and puts in the number you enter after. But when you tab/click off it changes back to prepending the 0 to the value you entered (eg: 07). I think it might be more intuitive to leave the '0' in place when entering a value.
Additionally it throws an error if you enter a value higher than 59. Perhaps any value entered higher than 59 should just be changed to 59?

Closing due to inactivity. Please re-open if you can provide a plunkr and confirm this is still an issue with Angular 1.3 and UI Bootstrap 0.13.


Add --no-progress global option to hide all progress animations

Summary
Fixes #5082.
Adds a new Printer::NoProgress that is identical to Printer::Default but doesn't draw any progress bar.
Test Plan
It seems to me that as of now it's not possible to use insta-cmd to get any progress bar in the comparable output of command.
Best way to test this would be to run any command that usually shows progress indicators like uv pip install with and without --no-progress options.

Maybe --no-progress is a better name, because this hides all progress output, not just bars? We previously discussed an option that would hide the concurrent progress bars but still show a single progress indicator.

Thanks again!


Issue #SC-1011 chore: Create ingestion spec for Error events and update samza jobs

@sunilpes @anandp504 Please review

Closing this PR as the PR was raised against an incorrect branch.


Create a simple landing/home page

A connect wallet button and show them their ENS/address once they're connected
A simple header that says '.devdao domains'
A brief description of the project
Links to D_D's twitter and the repo of this project at the bottom

I would like to work on this.


Detection of wrong credentials for PasswordFlow

When the users passes credentials, that are not accepted by the identity provider, I would like to detect this and present an error message, pointing the user into the right direction. I cannot seem to get this to work, here is a code snippet:
 this.oauthService
            .fetchTokenUsingPasswordFlowAndLoadUserProfile(this.user.username, this.user.password)
            .then(() => {
                console.debug('successfully logged in');
                let claims = this.oauthService.getIdentityClaims();
                this.loginFailed = false;
            })
            .catch((err) => {
                console.error('error logging in', err);
                this.loginFailed = true;
                this.errorMessage = this.labels.APP_UNAVAILABLE;
            })

The err object is a ProgressEvent and I cannot find anything useful to differentiate the error that happened.

Is this currently supported?

Can you please check this again. In the case of an error you should get sth like this:

Any luck with this?

Sorry for going dark on this. @manfredsteyer I think the reason that this did not work as expected is, that the server send the 401 response without setting the CORS Header. So instead of receiving the actual error (401, Bad Authorization), I received the error about missing CORS headers.


How to set Timezone?

What happened?
First, thank you for this awesome Software!
I know it’s a little thing, but:
I would like to set the correct Timezone, so in the logs it is right.
My Timezone is Europe/Berlin
I tried it with -e TZ=Europe/Berlin in the setup command, but don’t worked.
AdguardHome-Sync Version
0.5.2
AdguardHome Version
0.107.43
Configuration
Docker cli
Sync between two AdGuard Instances.

Relevant log output
-

Anything else?
No response

Hi @Gamerou
Timezone data was not present in the image, but will be in the next release.

released with released with https://github.com/bakito/adguardhome-sync/releases/tag/v0.5.3


remove fixed dependency for juptyer-client version

depending on resolution of
https://github.com/jupyter/jupyter_client/issues/523

@luiztauffer has this been resolved?

@luiztauffer maybe this was causing the import error for @Saksham20?

i can't be it, I fixed the jupyter-client==5.3.4 in setup.py

The fixing of jupyter-client package is not optimal though, because they already crossed > 6.0
So let's keep this issue here to remind me to test new releases once they get it solved


docker file restrictions needs change

In order to get the provisioning tools we need to run the following command but it has authorization problem
docker pull kubenow/provisioners:master
One needs to change mode of a particular docker file as under
sudo chmod 777 /var/run/docker.sock

@laeeq80 I think that this is a docker error that has not to do with KubeNow. You need to add your user to the Docker group in order to be able to see /var/run/docker.sock. Please feel free to reopen if I am wrong.


Null owner

Why is the owner shown as "null" in the study list view

but set to a concrete value when looking at its details?

Cause that User/Person has no name registered in the system ...

.. and perhaps related to https://github.com/biocore/qiita/issues/304, as the study owners are different than the users of the system ...


Setup workflow for deployments

Update: I am currently working on the default workflows that we need for development. When we have access to the VM, I will add a deployment workflow as well.

Working on to have a GitHub io page looks like this


Suggestion: change --tls-cert and --tls-key flags to take a file path

Usability improvement suggestion: change --tls-cert and --tls-key flags (or, for backward compatibility, introduce --tls-cert-file and --tls-key-file) to take a file path to files storing key and certificate; instead of requiring multi-line strings on the command line.
The reason for that is that passing a one-line string containing \n characters to --tls-cert or --tls-key will be incorrectly written to disk on the concourse web instance, leading to the ATC and grafana not starting up with an error since the tls_cert and tls_key files cannot be parsed. (This can happen, for instance, if someone does something similar to cat config.json | jq .concourse_key to re-use the previous key stored in the config)
Alternatively, a conversion from \n to line breaks could be done before writing the tls_key and tls_cert files on the web nodes.

I think this is a good idea. I'll leave it to the team to decide if/when to schedule it though.
In theory it wouldn't be too hard to make the flag validation fail if \n appears in the cert or key.
In the mean time, I think your jq loading example would work if you use the raw-output flag. So jq -r .concourse_key config.json.


Floating IP pool not available

Hi,
My openstack does not have Floating IP pools.
[root@openstack-dev ~(keystone_admin)]# openstack floating ip pool list
Floating ip pool operations are only available for Compute v2 network.

[root@openstack-dev ~(keystone_admin)]# nova floating-ip-pool-list
WARNING: Command floating-ip-pool-list is deprecated and will be removed after Nova 15.0.0 is released. Use python-neutronclient or python-openstackclient instead                                                                      .
+--------+
| name   |
+--------+
| public |
+--------+

Any workaround for this?
thank you

neutron net-external-list
or you can leave it blank and it will only use the private ips

Hi @Masber,
Can i ask, has the workaround fixed your issue?


ERROR: ('invalid_grant: Bad Request', '{\n  "error": "invalid_grant",\n  "error_description": "Bad Request"\n}')

hadn't checked on my crontab run backup, but recently I tried to run manually and noticed,
/opt/bin/gyb --email<EMAIL_ADDRESS>--action backup --local-folder /home/<>/Backup/gyb-<>
ERROR: ('invalid_grant: Bad Request', '{\n  "error": "invalid_grant",\n  "error_description": "Bad Request"\n}')
$ gyb --short-version
1.39
Not sure what I need to do to resolve this issue, but for the time being I can't backup my email anymore.
thanks!

gyb-1.38-windows-x86_64.zip last worked in June, today throw error
after update to gyb-1.39-windows-x86_64.zip result the same

You can just delete the .cfg file named after the email address and then reauth.
Jay

https://groups.google.com/g/got-your-back/c/D-k547AuyBo/m/c_FzpQPjCAAJ
It worked


Can I retrieve metadata along with the retrieved documents from a Vectorstore by modifying the parameters in asRetriever()?

Hi 👋
For my use case I would need to return not only the text of the most similar documents in the vector database, but also one of the metadata fields of each returned document.
Currently I have a chain created, passing the vector database as vectordb.asRetriever(5, customFilter) so that it returns the 5 most similar documents and I also apply a filter.
After reading the documentation and source code of the asRetriever() method,
asRetriever(
    kOrFields?: number | Partial<VectorStoreRetrieverInput<this>>,
    filter?: this["FilterType"],
    callbacks?: Callbacks,
    tags?: string[],
    metadata?: Record<string, unknown>,
    verbose?: boolean
  ): VectorStoreRetriever<this> {
    if (typeof kOrFields === "number") {
      return new VectorStoreRetriever({
        vectorStore: this,
        k: kOrFields,
        filter,
        tags: [...(tags ?? []), this._vectorstoreType()],
        metadata,
        verbose,
        callbacks,
      });
    } else {
      const params = {
        vectorStore: this,
        k: kOrFields?.k,
        filter: kOrFields?.filter,
        tags: [...(kOrFields?.tags ?? []), this._vectorstoreType()],
        metadata: kOrFields?.metadata,
        verbose: kOrFields?.verbose,
        callbacks: kOrFields?.callbacks,
        searchType: kOrFields?.searchType,
      };
      if (kOrFields?.searchType === "mmr") {
        return new VectorStoreRetriever({
          ...params,
          searchKwargs: kOrFields.searchKwargs,
        });
      }
      return new VectorStoreRetriever({ ...params });
    }

it is not clear to me what the function of the "metadata" parameter is and that is why I would like to know if in this parameter I can indicate the metadata that I want to be returned along with the text of the most similar documents to be passed together as context to the LLM.
Thanks in advance 😄
P.S. The definition of the Retriever() method can be found in the following link:
https://github.com/langchain-ai/langchainjs/blob/54beee324444f1915f99a34a6b7c66539d74202e/langchain-core/src/vectorstores.ts#L340C3-L376

@dosu-bot
What I would like to achieve would be to pass as context to the LLM in a RAG not only the text of the documents returned by the vector database, but also certain metadata fields that each returned document contains.
How could I pass this context with text + metadata to the LLM in the RAG?

@dosu-bot
What's the difference in "creating a chain" by using a RunnableSequence or by using a, for example, RetrievalQAChain?
Is one way or another more recommended or better practice?


Package feedback for migration #842

As part of the new GOV.UK EKS, the Docker file has been created for the feedback app in the integration environment
This is part of the All apps migration for launch.
Additional changes include:

Puma substituted for Unicorn as web server (only in the Dockerfile)
GitHub Actions workflow has been added to enable the pushing of image to ECR upon merge to main

Unicorn substituted for as web server

...what change does this?

@KludgeKML - The config/puma.rb has been added to the code to overwrite Unicorn - I have amended the original comment to "Unicorn substituted for Puma as web server"

Ah, I was confused by the way round - substituted for implies the first thing is the new one. So we're moving to Puma, is that right? In which case:
Puma substituted for Unicorn as web server

That's right
On Tue, 1 Mar 2022 at 11:13, Keith Lawrence @.***>
wrote:

Ah, I was confused by the way round - substituted for implies the first
thing is the new one. So we're moving to Puma, is that right? In which case:
Puma substituted for Unicorn as web server
—
Reply to this email directly, view it on GitHub
https://github.com/alphagov/feedback/pull/1374#issuecomment-1055317314,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXHT2CP7XOODDCQHYVKL6LDU5X3VHANCNFSM5PR2VQEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Ah, I was confused by the way round - substituted for implies the first thing is the new one. So we're moving to Puma, is that right? In which case:
Puma substituted for Unicorn as web server
The statement above is correct and only applies to Dockerfile
Many thanks

Hi All
Could this PR be reviewed please?
Regards


kadane's Algorithm in C

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
Max subarray sum implemented using Kadane's Algorithm in C.
Checklist

[X] I've read the contribution guidelines.
[X] I've checked the issue list before deciding what to submit.
[X] I've edited the README.md and link to my code.

Related Issues or Pull Requests
ISSUE: #178
Plz review my Pr @Kajol-Kumari  @harsh-9in

Done the changes @Kajol-Kumari

Plz merge my PR @Kajol-Kumari @SKAUL05 @iamrajiv

@all-contributors please add @raksha009 for code


CI-92 assessment orgs by standard

the next build passed, it was pointing at the wrong ES for the integration tests


Dependency conflict with morphometrics

It would be cool if napari-stress was compatible with morphometrics. Unfortunately, we have a dependency conflict at the moment.
If morphometrics is installed correctly (numpy 1.21.5), pip complains:
pyshtools 4.10 requires numpy>=1.22.3, but you have numpy 1.21.5 which is incompatible.

If napari-stress is installed correctly (numpy 1.22.4), pip complains:
morphometrics 0.0.1 requires numpy==1.21.5, but you have numpy 1.22.4 which is incompatible.

After a quick chat with @kevinyamauchi in https://github.com/kevinyamauchi/morphometrics/issues/25 , I presume it would be good if we could pin numpy < 1.22 in napari-stress. But it needs to be tested if our functionality depending on pyshtools still works then.

Hey, @haesleinhuepf ! I agree it would be great to maintain compatibility. It looks like numba  has resolved the numpy issues that required < 1.22 (links below) in 0.55.2. As of a couple of weeks ago, 0.55.2 was available on pypi and it looks like it will be on Vonda-forge soon (already available on numba channel. In light of this, I will try removing the numpy constraint and instead add a numba constraint in morphometrics, which should alleviate this dependency conflict. I’ll ping you in the PR once I’ve made the test.
https://github.com/numba/numba/issues/7754

FYI, I've updated the numpy dependency in morphometrics: https://github.com/kevinyamauchi/morphometrics/pull/26

I had a quick look at the pyshtools library out of curiosity and I don't really understand what is setting their numpy version requirement. To me it actually looks like they set it to whatever version of numpy is currently in the environment upon installation (see this line). I am curious if this is going to cause an issue where the required numpy version will depend on when and in what order pyshtools was added to the environment.

I'd like to explore anyway if we can relax this dependency version constraint... There have been other projects (stardist?) which are quick picky about their numpy version...

I agree it makes sense to try to be as relaxed as possible with the numpy dependency seeing as many (most?) scientific python packages depends on it.

To me it actually looks like they set it to whatever version of numpy is currently in the environment upon installation (see this line).

I had the same suspicion. It could be a different random-ish version depending on who releases it from where.


Fix error message about invalid surrogate pairs

I tested following strings with invalid surrogate pair and unpaired surrogate in files:

"a\uD800\uD800x"
"a\uD800x"

The error messge was: "... invalid string: surrogate U+DC00..U+DFFF must be followed by U+DC00..U+DFFF; ..."
I think it must be: "... invalid string: surrogate U+D800..U+DBFF must be followed by U+DC00..U+DFFF; ..."

Pull request checklist
Read the Contribution Guidelines for detailed information.

[x]  Changes are described in the pull request, or an existing issue is referenced.
[x]  The test suite compiles and runs without error.
[x]  Code coverage is 100%. Test cases can be added by editing the test suite.
[x]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for these kind of bugs). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Good catch!!!

🔖 Release item
This issue/PR will be part of the next release of the library. This template helps preparing the release notes.
Type

[ ] ✨ New Feature
[ ] 🐛 Bug Fix
[ ] ⚡ Improvement
[ ] 🔨 Further Change
[ ] 🔥 Deprecated function

Description

Fix error message about invalid surrogate pairs. #2076


Fixed bug #230: whatsapp count

I tried to fix bug #230 by first finding out the proper CSS selector (here: class name) for the span with the unread count texts and then add the counts in its content.
Hope that this method is more stable as it relies on a variable name for the color of unread count badge.
Could not figure out what the previously contained height comparison is used here, so I left it out, but even if it is necessary, the first part of my code (to find out the CSS class name) could still be useful, as the class names are mini- and uglyfied.

Just FYI, there's an alternative method to fix this in https://github.com/getferdi/recipes/pull/894.

I am well aware of the other fix (well - I wasn't when I started mine... ;-) but after comparing both, although the other fix changes less code, I think that mine is a bit more stable in case of future changes of the DOM structure.

Hello! :D Just tried this fix and it didn't worked for me. I image that neither for @vraravam will work

Hello! :D Just tried this fix and it didn't worked for me. I image that neither for @vraravam will work

Hey again! Just tried this fix without dev mode and it working properly for me 😄 seems like in DEV mode the Ferdium.setBadge is not setting properly the badge count.
So I vote to merge this PR if @vraravam can also confirm it to be working on his machine!

This works for me (EN and DE languages).
Just two bugs:

Pinned and Muted

When you have a pinned and muted chat, it doesn't detect the muted state, because the pinned state is not the previous sibling then, but the sibling before:

Using the if clause fixes it for me:
        if (
          !unreadElem.parentNode.previousSibling ||
          unreadElem.parentNode.parentNode.querySelector(
            '[data-icon=muted]'
          ) === null

Manually marked unread

If a chat has been manually marked as unread, the result becomes NaN, because the badges doesn't contain a number:

Proposed fix:
        let countValue = parseInt(unreadElem.textContent);
        if (isNaN(countValue)) countValue = 1;

@oliverrahner thanks for the hint with the combined muted/pinned state.
I cannot reproduce the NaN situation, as I use Ferdium.safeParseInt(unreadElem.textContent) I could never trigger a NaN result, even with an empty badge.
But: it does not count towards the unread messages in that situation - did you mean that?

This PR still doesn't work for me. But, if it does work for a portion of the users, then at least that number of users won't keep raising duplicate bugs!

@vraravam I see from your screenshot that you tested with a group chat, is that correct? I just tested with direct conversations, so maybe that's the problem with the fix...

@vraravam I see from your screenshot that you tested with a group chat, is that correct? I just tested with direct conversations, so maybe that's the problem with the fix...

yes - most of my chats are group chats. The expectation is that the solution should work for both

@vraravam I tested my fix now also successfully with a group chat. (It was also my expectation that the fix works with both ;-) Will continue testing a bit, but that will take some time.

@oliverrahner thanks for the hint with the combined muted/pinned state.
... as I use Ferdium.safeParseInt(unreadElem.textContent) I could never trigger a NaN result
But: it does not count towards the unread messages in that situation - did you mean that?

Oops, I tried the fix out in native Chrome, and thus hat to replace Ferdium.safeParseInt() by parseInt(), my bad...
But then, yes, if I mark a chat unread by intention I would at least like to be reminded of this in the unread count.

Closed in favor of: https://github.com/ferdium/ferdium-recipes/pull/235#issuecomment-1320027760 . A new PR should be open with this fix.
Thank you!


GET /v3/conversations returns 405 http code

GET /v3/conversations returns 405 HTTP code but should return a list of conversations the bot has participated in (according to this https://docs.microsoft.com/en-us/azure/bot-service/rest-api/bot-framework-rest-connector-api-reference?view=azure-bot-service-4.0#get-conversations)
Steps to reproduce the behavior:

Get access token
Perform GET /v3/conversations
See the response

Request:
GET /amer/v3/conversations HTTP/1.1
Authorization: Bearer {access_token}
Host: smba.trafficmanager.net
Connection: close
User-Agent: Paw/3.1.8 (Macintosh; OS X/10.15.0) GCDHTTPRequest

Response:
HTTP/1.1 405 Method Not Allowed
Allow: POST
Content-Length: 72
Content-Type: application/json; charset=utf-8
Server: Microsoft-HTTPAPI/2.0
Date: Fri, 20 Sep 2019 12:31:51 GMT
Connection: close

{"message":"The requested resource does not support http method 'GET'."}

Additional context
This issue looks like #1264
I tried two endpoints: https://smba.trafficmanager.net/apis and https://smba.trafficmanager.net/amer.
Also, tried different networks and routings.
Clients: Paw and Postman.
Feel free to ask any additional details.

@apetrovs What channel are you trying to use?

Closing due to inactivity.

@tdurnford Hey! Sorry for the late reply. What do you mean by "channel"?
I use https://smba.trafficmanager.net/amer as the service URL because I receive it in incoming events. And https://smba.trafficmanager.net/apis was mentioned in #1264, so I decided to try it also.

By channel I mean the BotFramework supported channels - Teams, Skype, Facebook Messenger, Line, Direct Line, etc. Basically how are you interacting with your bot?

@tdurnford thanks for the explanation. My bot is for Teams and I use the Java framework and REST API.

It appears that most channels do not support the get conversation endpoint which is why the request is responding with a 405 error. From what I can tell, Direct Line is the only channel that supports the request. This is not well documented unfortunately. Take a look at https://github.com/microsoft/BotFramework-Services/issues/88 for more details.

@JonathanFingold I am using GET Conversations in Teams, and it returns "Method not allowed". You mentioned "Please comment on that thread, and we'll try to get you unblocked". What do I need to do to get this unblocked for my bots?

@tdurnford thanks for the information! But it would be great if that appears in documentation.

@JonathanFingold I am using GET Conversations in Teams, and it returns "Method not allowed". You mentioned "Please comment on that thread, and we'll try to get you unblocked". What do I need to do to get this unblocked for my bots?

@tdurnford Can you also help @dgancho ? If it's more convenient, feel free to reopen the other issue I closed as a duplicate of this one.

@JonathanFingold It is the same issue. The get conversations get request does not support the Teams Channel. I don't believe there are any plans to change this behavior.

Can you please take a look at this related issue - https://github.com/MicrosoftDocs/bot-docs/issues/1934 ?

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


gettext: Add a binding for setlocale()

This adds a Gettext.setlocale() binding for the setlocale() function, which is
sometimes needed in programs that use Gettext.
Also adds a Gettext.LocaleCategory enum which contains all the LC_... values
from locale.h, for use as the first parameter of this function.
[endlessm/eos-shell#5411]

@mariospr Let me know if this fixes your problem. If so, then we can submit it upstream as well.

It works wonderfully well. Thanks! I'll leave this unmerged for now, though, while the whole solution for #5411 is not yet ready, so that everything can be merged together.


Reconcile CRHP with BCRHP sites (automatic or manual)

Reconcile what is in the CRHP with what is in the BCRHP. CRHP status is manually recorded in HRIA, so could be discrepancies. Need to find a way to reconcile going forward.

Resolved


Cannot use Qiskit simulation when running example1

I tried to run torchquantum-master\artifact\example1\mnist_example.py, but I ran into some trouble when doing qiskit simulation, too.
Because the flie "examples.core.datasets" is missing, I copied it from the file from https://zenodo.org/record/5787244#.YbunmBPMJhE.(\torchquantum-master\examples\core\datasets).
To avoid BrokenPipeError, I reset the "num_workers" in line 152 to 0.
After these messages:
Test with Qiskit Simulator
[2022-10-20 14:33:05.579] Before transpile: {'depth': 36, 'size': 77, 'width': 8, 'n_single_gates': 52, 'n_two_gates': 21, 'n_three_more_gates': 0, 'n_gates_dict': {'ry': 19, 'rz': 18, 'rx': 13, 'cx': 20, 'crx': 1, 'h': 1, 'sx': 1, 'measure': 4}}
[2022-10-20 14:33:06.257] After transpile: {'depth': 31, 'size': 61, 'width': 8, 'n_single_gates': 37, 'n_two_gates': 20, 'n_three_more_gates': 0, 'n_gates_dict': {'ry': 11, 'rz': 7, 'rx': 6, 'cx': 20, 'u3': 11, 'u1': 2, 'measure': 4}}

an error occurred, saying "need at least one array to stack". the details are in the file "errorlog1"
I also tried to add ", parallel=False" and modify the file qiskit/assembler/assemble_circuits.py as the Issue#9 does, but another error occurred, the details are in the file "errorlog2"
The version information is as follow,
>>> import qiskit
>>> qiskit.version.QiskitVersion()
{'qiskit-terra': '0.19.2', 'qiskit-aer': '0.10.3', 'qiskit-ignis': '0.7.0', 'qiskit-ibmq-provider': '0.18.3', 'qiskit-aqua': '0.9.5', 'qiskit': '0.34.2', 'qiskit-nature': None, 'qiskit-finance': None, 'qiskit-optimization': None, 'qiskit-machine-learning': None}

and I'm running the code under python 3.9.
By the way, I tried the code in https://zenodo.org/record/5787244#.YbunmBPMJhE.. The same problem occurred.
I wonder how to deal with it. Any help would be greatly appreciated.
errorlog1.txt
errorlog2.txt

Hi yzr-mint,
Thank you for your feedback, I think that may because of the version mismatch between the torchquantum-master\artifact\example1\mnist_example.py and the current QiskitProcessor. Could you try to run the new code in examples folder?  https://github.com/mit-han-lab/torchquantum/blob/master/examples/simple_mnist/mnist_example_no_binding.py
Please let me know if that works. Thanks!

It works! Thanks!


Support for Orleans 7

Hi Guys,
Love your project which makes the unit testing with Orleans very easy.
I know that Orleans 4.0 is not released yet but we are trying to migrate our current Orleans version to the latest preview ( currently preview 2). We used this Project to build our Unit Tests for our Application.
I am currently in the upgrading process and noticed a bunch of errors regarding reference type problems but I assume there are many more
My Question is :  Are there any plans to support the upcoming Version ?

I haven't had an opportunity to work on an Orleans 4.0 upgrade myself. However, I think there's a lot of opportunity to simplify the mock-based testing implementation given the changes. I don't think I'll have time to invest in this until shortly after the official release. I would also anticipate we'll want to create a branch specific for 4.0 compatibility, given the big differences with 3.x.

Thank you for your reply.
Would it be okay if I would contribute a branch where I would try to migrate the current Project to the 4.0 preview 2 ? I would not try to change the current State or Architecture but rather trying to fix the current Issues regarding the upgrade ?

@Badabunga Of course! I just created a new branch named orleans-7. Feel free to open any pull requests against it. (Of note- it looks like they're going with Orleans 7.0 instead of 4.0 to align with the upcoming .NET release!)

Awesome progress! I started playing around with Orleans 7 over the past week, and I bumped into a number of new, tricky scenarios. There are some Orleans internal classes that are proving problematic.

I think the main problem are the "old" type of grains that inherit from Grain or from Grain<>, the new POCO style Grains can be easily created by just injecting the GrainContext.

Any chance we might see a prerelease NuGet package before long?

+1 to the prerelease test nuget question

Release to NuGet as 4.0.0-beta.1. Thanks, @Badabunga!


Adds Twill Turkish translations

Description
Adds Twill Turkish translations

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @trk thank you so much for this! Can you align with the contributor that submitted #1134? If you guys can agree on which of you 2 has the best translations, that'd be great.

I see #1134 pull-request yet,
I checked translation file on #1134 pull-request, all translations looks done, but for auth:oauth-link-title translation term :providervalue missing for Turkish translation. Other translations look like acceptable.
One more missing thing is available_user_locales tr locale key on https://github.com/trk/twill/blob/9462506c9c57e86cd8a393a867fef35752b5a0ae/config/twill.php#L206

Closed after merging https://github.com/area17/twill/pull/1134
Thanks again @trk!


Option to hide brightness slider

Before opening the issue, have you...?

[X] Searched for existing issues

Is your feature request related to a problem? Please describe
I only want to use MonitorControl to change the volume of my monitor's speaker.
Sometimes I by mistake change the brightness instead. I'd love to have an option to remove the brightness sliders from the menu.
Describe the solution you'd like
There's already an option "Show volume slider in menu" and "Show contrast slider in menu".
I think it would be consistent to add another option "Show brightness slider in menu".
Describe alternatives you've considered
I don't see a real alternative here.
Anything else?
By allowing to hide the brightness sliders there's a chance a user will disable all of the sliders which makes the menu nearly useless beside the options to quit the app or open preferences dialog.
It may be worth to think about changing the "Menu contents" selection a little bit so the user is not able to disable all the options. I'd could imagine a dropdown instead of checkboxes or some simple validation could do the job.

Hi, the unreleased 4.0.0 version already has this feature. There is an interim build (unsigned) at the bottom of the opening post here. You can add your feedback in the discussion.
You need to enable Advanced options and  unclick "Show brightness slider".

(This is a duplicate of https://github.com/MonitorControl/MonitorControl/issues/324 so I'll close it)
Thank you!


consider using django-graphos instead of chartit

chartit was quite complicated to configure
This reddit post suggests graphos is better
https://www.reddit.com/r/django/comments/2kzrom/thoughts_on_djangochartit/
graphos
django-graphos http://agiliq.com/demo/graphos/tutorial/

Ended up using highcharts api directly in templates instead of graphos or chartit.


All download file links are broken with no public access

All file links throughout the lab you're instructed to access and download are showing no access.
Example:
Fabrikam's 2023 Income Statement.

@SkillableBB this has now been resolved. Please test and confirm. If the issue still occurs, reopen this issue. Thanks!


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_6f4nw7tmpd/clone/recipe/meta.yaml:27: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_6f4nw7tmpd/clone/recipe/meta.yaml:54: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_6f4nw7tmpd/clone/recipe/meta.yaml:54: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_6f4nw7tmpd/clone/recipe/meta.yaml:63: missing_description: The recipe is missing a description
Errors were found


HOL 3, •	Exercise 1, need to add new step after step 30: Dialog Box

1)Allow SSL for use 2)Accept SSL certificate installation.

@Senajs Which language?

@stevenfollis AFAIK .NET

I just performed the steps in the documentation, and they worked perfectly.


Discord

Would be cool if there was a discord channel where you can the latest developments and creations as well as asking for help.

Sure, it's probably going to be very quiet at first though but here's the invite link
https://discord.gg/utUtB5r
I need to put that somewhere on the wiki or in the readme too


Investigate correlation trends

[x] Description
Understand ways to apply correlation formula for two dimensional data. Understand the visual trends for different functions we will be correlated with.
[x] Use cases
N/A
[x] Proposed design
N/A
[x] Alternative design approaches
N/A
[x] Workaround
N/A
[x] Additional information
N/A

Hey @silvs110,
Can you add the information you've collected to this issue.


Pipe through logs from the spawned openapi generator process

Log the command and arguments passed to openapi-generator-cli
Add a handler to stdout for the spawned child process, and log data using the nx devkit logger.
Add silent option to suppress output
docs: add joeflateau as a contributor for doc

👋 Hey @youngchen7,
🚀 Thank you! This is an awesome contribution!
🙏 Before I take it for a spin, and most certainly merge it, would you mind also adding relevant test cases to the unit tests for the executor?

Yup, I'll look into adding unit tests either today or tomorrow!

I've added the unit tests. It's a little odd to mock / expect from a logger, but I didn't see another good way to validate that the output was being forwarded.
Originally I wanted to add a unit test that would validate the fix for the child buffer size causing process hangs, but that's a spawn implementation specific issue and not possible to test, I think.

closing in favor of #72!


Simplify and remove most of these

https://github.com/dantrim/parquet-writer/blob/fa1b24a97d3f1e0df15428619dc90092db0ae9fd/src/cpp/parquet_writer_types.h#L59

addressed in #30