Add license property 'include-copyright-binary'.

Add an additional license property: "Must this license be part of a binary distribution?"
To make this clear, rename 'include-copyright' to 'include-copyright-source', so that we
can call the new property 'include-copyright-binary'.
Note that this commit does not yet set this property for the existing licenses-- that requires actually reading them. :-)
Background: In the course of my work it is very useful to know whether the binary form of someone else's product actually must be accompanied by a license, or whether it may be redistributed without.

Closing as stale, but feel free to continue the discussion or submit a PR if there's still interest in seeing this.


Enable -show-implicit-tests-count command line argument

Fix #65

:tada: This PR is included in version 5.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add 'use_https' parameter for FiOS Quantum Gateway and upgrade Pypi

Description:
Added HTTPS support for Quantum Gateway routers which may have been pushed a firmware update forcing HTTPS connections to query the device list.
Related issue (if applicable): fixes #21692
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8826
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Can confirm the updated code is working for me


Error installing MACS2 with Cython installed

Dear all,
I am having a problem with installing MACS2 (v 2.1.0, downloaded on 25-Feb-2015).
When I run
python setup.py install --prefix=/my/fav/folder

I get the following error. Cython, v 0.22 is installed. I know a similar issue was reposrted in issue #36, but no solution to the problem is provided there. Can anybody help? Thanks!
running install
/my/fav/folder/setuptools/lib/python2.7/site-packages/setuptools-12.2-py2.7.egg/pkg_resources/__init__.py:2512: PEP440Warning: 'distribute (0.6.24dev-r0)' is being parsed as a legacy, non PEP 440, version. You may find odd behavior and sort order. In particular it will be sorted as less than 0.0. It is recommend to migrate to PEP 440 compatible versions.
running bdist_egg
running egg_info
writing requirements to MACS2.egg-info/requires.txt
writing MACS2.egg-info/PKG-INFO
writing top-level names to MACS2.egg-info/top_level.txt
writing dependency_links to MACS2.egg-info/dependency_links.txt
writing requirements to MACS2.egg-info/requires.txt
writing MACS2.egg-info/PKG-INFO
writing top-level names to MACS2.egg-info/top_level.txt
writing dependency_links to MACS2.egg-info/dependency_links.txt
reading manifest file 'MACS2.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: no files found matching 'MANIFEST'
writing manifest file 'MACS2.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
running build_ext
building 'MACS2.cProb' extension
gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -I/usr/lib/pymodules/python2.7/numpy/core/include -I/usr/include/python2.7 -c MACS2/cProb.c -o build/temp.linux-x86_64-2.7/MACS2/cProb.o -w -Ofast
gcc: error: MACS2/cProb.c: No such file or directory
gcc: fatal error: no input files
compilation terminated.
error: command 'gcc' failed with exit status 4

Response from Tao that helped me on CentOS:

A simple workaround although I don’t have CentOS to test: open MACS2 ‘setup.py’ file and replace all ‘-Ofast’ with ‘-O3’ or ‘-O2’. If it still doesn’t work, remove ‘-Ofast’ entirely.

Thanks for the tip, but it didn’t work, with either –O3, -O2 or nothing. I don’t have a problem installing MACS (not version 2).
Nicola
From: mdozmorov<EMAIL_ADDRESS>Sent: 26 February 2015 15:20
To: taoliu/MACS
Cc: nicolazilio
Subject: Re: [MACS] Error installing MACS2 with Cython installed (#69)
Response from Tao that helped me on CentOS:
A simple workaround although I don’t have CentOS to test: open MACS2 ‘setup.py’ file and replace all ‘-Ofast’ with ‘-O3’ or ‘-O2’. If it still doesn’t work, remove ‘-Ofast’ entirely.
—
Reply to this email directly or view it on GitHub https://github.com/taoliu/MACS/issues/69#issuecomment-76185357 .  https://github.com/notifications/beacon/ACq66AGmpwKBkSoIYnrhyNLoklIWK0Y_ks5nvyMkgaJpZM4DmMgx.gif

Hi Nicola,
Perhaps you downloaded incomplete source codes. Could you remove your current MACS2 source codes and old installation from your computer then download and try again?
-T

Hey Tao,
I actually solved the issue in a much simpler way, I installed Anaconda to ensure all Python dependencies are satisfied and then did pip install. Worked without an issue.
Nicola
From: Tao Liu (τν<EMAIL_ADDRESS>Sent: 20 April 2015 20:43
To: taoliu/MACS
Cc: nicolazilio
Subject: Re: [MACS] Error installing MACS2 with Cython installed (#69)
Hi Nicola,
Perhaps you downloaded incomplete source codes. Could you remove your current MACS2 source codes and old installation from your computer then download and try again?
-T
—
Reply to this email directly or view it on GitHub https://github.com/taoliu/MACS/issues/69#issuecomment-94535696 .  https://github.com/notifications/beacon/ACq66FGrssikMuNQbt7BHC8yNQojVpsvks5oBUBOgaJpZM4DmMgx.gif

@nicolazilio Good to know. Now I just closed this ticket.


grafana 9.1.1 -> 9.5.3

Description

Please include a brief summary of the changes.

Version change: 9.1.1 -> 9.5.3
Context / Why are we making this change?

Please include details on why this change is necessary, and any applicable tickets, design docs, or documentation links.

Testing and QA Plan

How has this work been tested or QA'd?

Impact

What are the implications of these changes? Are there any cross-cutting concerns to keep in mind?

Still need to test before merging.

Tested in dev:


Fix jsonable_encoder for dataclasses with pydantic-compatible fields

This PR fixes an issue (#3608) I encountered when returning dataclasses responses with pydantic-compatible (but incompatible with json encoder of Python standard library) fields, e.g. datetime objects or pathlib file paths.
This was achieved by adding a call to jsonable_encoder(dict) after converting the dataclass to a dict in jsonable_encoder.
For testing, I have added a datetime field to the  Item class in tests/test_serialize_response_dataclass.py.
Without this PR, the last two tests in this file fail (test_no_response_model_objectlist, test_no_response_model_object).
Note that the tests also pass with a pre 0.67 version of FastApi (before official dataclass response support).

I am having this same issue, and was going to propose the same fix.
I think test cases should be added to test_jsonable_encoder.py since the jsonable_encoder is the affected component, and it can be used for other purposes than just response serialization.

Amazing, thanks @himbeles! And thanks for adding several tests! :rocket: :cake:


Add proper application and web logs using pino

Adds pino and uses it for logging both application and web log messages. This also adds pino-pretty as a dev-dependency for pretty-printing the messages to a more human friendly form, and uninstalls morgan.
Pino logs JSON messages and is tailored to be fast and simple. SQLPad doesn't really need the extra performance it brings, but I really enjoy its simplicity, and the fact that transports are handled separately. (I've battled the complexity quirky behaviors of winston in other projects, and pino has been really enjoyable to implement).
This adds additional configuration to manage this logging, and some additional documentation in the README to expand on how to use it and what to expect. I'll probably be investing in adding to the SQLPad project page with more in-depth instruction and examples on how to run and administer SQLPad.
Addition to the README for reference:
Logging
SQLPad logs json messages using pino, a simple and fast logging library. Log messages are sent to stdout, leaving how to handle the messages up to you. The pino ecosystem supports a variety of transports http://getpino.io/#/docs/transports that should cover most logging setups.
Two categories of messages are logged by SQLPad: "app" messages containing info messages, warnings, and errors relating to application code, and "web" messages relating to the actual http requests handled by SQLPad. The level at which these logs are logged can be configured separately, or even turned off, with the appLogLevel and webLogLevel settings. This level represents the minimum level to be logged. The value used should be one of fatal, error, warn, info, debug, trace or silent. Setting the level to silent will effectively disable the logger.
In production, log messages will be a single json object per line:
> node-dev server.js --dbPath ../db --port 3010 --debug --baseUrl /sqlpad --config './config.dev.ini'

{"level":30,"time":1581974034149,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Loading users","v":1}
{"level":30,"time":1581974034150,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Loading connections","v":1}
{"level":30,"time":1581974034150,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Loading connectionAccesses","v":1}
{"level":30,"time":1581974034150,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Loading queries","v":1}
{"level":30,"time":1581974034150,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Loading queryHistory","v":1}
{"level":30,"time":1581974034150,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Loading cache","v":1}
{"level":30,"time":1581974035055,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Creating access on every connection to every user...","v":1}
{"level":30,"time":1581974035066,"pid":7226,"hostname":"hostname.local","name":"sqlpad-app","msg":"Welcome to SQLPad! Visit http://localhost:3010/sqlpad to get started","v":1}
{"level":30,"time":1581974050933,"pid":7226,"hostname":"hostname.local","name":"sqlpad-web","req":{"id":2,"method":"GET","url":"/sqlpad/signin"},"res":{"statusCode":200},"responseTime":3,"msg":"request completed","v":1}
{"level":30,"time":1581974050970,"pid":7226,"hostname":"hostname.local","name":"sqlpad-web","req":{"id":3,"method":"GET","url":"/sqlpad/javascripts/vendor/tauCharts/tauCharts.min.css"},"res":{"statusCode":304},"responseTime":2,"msg":"request completed","v":1}

During development, the logs can be piped to pino-pretty, which is used when running npm start by default, producing a more human friendly format:
> node-dev server.js --dbPath ../db --port 3010 --debug --baseUrl /sqlpad --config './config.dev.ini' | pino-pretty

[1581974767500] INFO  (sqlpad-app/7387 on hostname.local): Loading users
[1581974767501] INFO  (sqlpad-app/7387 on hostname.local): Loading connections
[1581974767501] INFO  (sqlpad-app/7387 on hostname.local): Loading connectionAccesses
[1581974767501] INFO  (sqlpad-app/7387 on hostname.local): Loading queries
[1581974767501] INFO  (sqlpad-app/7387 on hostname.local): Loading queryHistory
[1581974767501] INFO  (sqlpad-app/7387 on hostname.local): Loading cache
[1581974768401] INFO  (sqlpad-app/7387 on hostname.local): Creating access on every connection to every user...
[1581974768413] INFO  (sqlpad-app/7387 on hostname.local): Welcome to SQLPad! Visit http://localhost:3010/sqlpad to get started
[1581974774313] INFO  (sqlpad-web/7387 on hostname.local): request completed
    req: {
      "id": 1,
      "method": "GET",
      "url": "/sqlpad/signin"
    }
    res: {
      "statusCode": 304
    }
    responseTime: 8
[1581974774342] INFO  (sqlpad-web/7387 on hostname.local): request completed
    req: {
      "id": 2,
      "method": "GET",
      "url": "/sqlpad/javascripts/vendor/tauCharts/tauCharts.min.css"
    }
    res: {
      "statusCode": 304
    }
    responseTime: 3

I'm going to go ahead and merge this. I'm open to following this up with adjustments as necessary. As is though I really like the logging experience with pino.


Improve OO structure and documentation

The functionality of some classes does not seem to match their name.
The functionality of some classes does not seem to match their name.

ChangeSet appears to model a change set, while its main/only responsibility actually seems downloading and parsing that dataset. It would seem more logical that a change set manager (or change set downloader) object creates changesets. If changesets themselves only need to be represented by the triples they contain (as seems to be the case), then perhaps a class for ChangeSet is not needed.

In JavaScript, don't write things like x.getTriples; just directly expose x.triples.

ChangesetCounter is a good example (name matches functionality). The documentation is a bit off though: instead of Method: ChangesetCounter, say A ChangesetCounter represents…. Then you can explain how the constructor can be called.

In ChangesetManager, don't monkey patch the this._afterListDownload etc. methods. Instead, directly use this.downloadChangesets, and patch these in unit tests as needed.

Ensure that a PollingAgent can be started and stopped. (The constructor should never start a complex job.)

In all classes, write a short statement about what the object does (example). This makes it easier to see what an object does, and to verify whether it needs to be an object indeed.

I started working on these.
Regarding the Changeset class: I created a class of its own, to encapsulate the functionality of decompressing and parsing a changeset, so that if the changeset format changes in any way, only the Changeset class itself will require changes. Also, I felt like this functionality escapes the scope of the ChangesetManager class. It stores not only the triples, but also the operation to apply with this triples (removal, or addition); as well as URL. Do you feel this doesn't warrant its own class?
We could also add a ChangesetDownloader class that keeps this functionality, and the changesets are simple dictionaries.
What do you think, Ruben?

I'd go for ChangesetDownloader, because this best reflects its functionality.

Okay, sounds reasonable. I'll work to add that. The changesets will now just be dictionaries then.

Added ChangesetDownloader and removed Changeset classes. Their code is almost exactly the same.

Great, thanks. It's not that I want to be a purist, but giving things the right name makes it easier to go back on the code afterwards.

I understand and agree :)


Refactor code

Coverage decreased (-0.005%) to 90.263% when pulling 3ef7eefae2e580f05541d7bfc834e55d91e422c8 on jelneo:refactor into 5b4e3ffa0b6d7f9fdf39dacf0fd980e68ee5ae24 on CS2103AUG2017-T17-B1:master.


FunctionParameterListSyntax result builder init does not insert space between labels

Description
When using FunctionParameterListSyntax.init() with a result builder, trailing trivia should automagically be inserted after the non-nil first name of a parameter if a second name is specified. Otherwise, something like this:
FunctionParameterListSyntax {
  FunctionParameterSyntax(
    firstName: x, // assume == "x"
    secondName: y, // assume == "y"
    type: z // assume == "Z"
  )
}

Produces xy: Z when x y: Z is presumably intended.
Steps to Reproduce
No response

Tracked in Apple’s issue tracker as rdar://112664577

This behaves correctly. The syntax node initializers don’t add any trivia, they just construct the syntax tree. You need to run .formatted() on the tree to add the formatting whitespace.

I don’t agree that result builder initializers should automatically format your code for two reasons:

The FunctionParameterSyntax initializer that you are invoking definitely shouldn’t format your code because it’s a pure syntax node initializer defined in the SwiftSyntax module, which doesn’t know anything about formatting. That’s a fact that we cannot touch. Now, the result builder initializers should be natural extensions of these initializers and it would, IMO be very inconsistent if FunctionParameterSyntax(…)gave you xy:z but then you wrap it in the result builder FunctionParameterListSytnax { … } and suddenly a space starts showing up between x and y, which isn’t even part of the list.
We don’t know which format you want to you to format the tree. Do you want to indent with two spaces or four spaces? Do you want to define your own BasicFormat that adds newlines between function parameters? While there’s probably only one reasonable format here, these questions become very important once you use e.g. the CodeBlock initializer that takes a result builder. The only answer to this question would be that you’d need to pass a format to every result builder initializer and that’s would just clutter everything.

So, I agree in general it shouldn't be trying to format the output. But I think it should be producing syntactically correct output, at least when you're using the convenience paths and not the "here's every token in this node"-style initializers. If I use the latter, then yes, it's on me to do it properly. But if I'm using a result builder with a convenience initializer, and it results in bad code unless I manually take extra steps to correct it, then what was the value in using the result builder?
To be clear, I am not advocating for full formatting of output, just the bare minimum required to make the result of a result builder syntactically valid.

So, I agree in general it shouldn't be trying to format the output. But I think it should be producing syntactically correct output, at least when you're using the convenience paths and not the "here's every token in this node"-style initializers. If I use the latter, then yes, it's on me to do it properly. But if I'm using a result builder with a convenience initializer, and it results in bad code unless I manually take extra steps to correct it, then what was the value in using the result builder?

My main concern here is that users of swift-syntax shouldn’t need to worry about which initializers are convenience initializers and which once are the dedicated initializers. The distinction between the two is a fairly arbitrary implementation detail. In the long run, the two being inconsistent wrt formatting will cause more frustration than it solves. Right now we have a simple rule: You need to format the output yourself.

To be clear, I am not advocating for full formatting of output, just the bare minimum required to make the result of a result builder syntactically valid.

That’s all that BasicFormat (and thus .formatted()) does, really: Add sufficiently much trivia to separate tokens so that it parses again. Note that to separate code block items, we also need to add newlines, so it’s not just about adding spaces. And the final piece is that we also do the basic indentation handling (no line wrapping or anything complicated) so that it’s somewhat readable.

I defer to you here, ultimately. I've made my case, but if you still feel the current API is sufficient, we can close this issue.

Most of all, I feel like the current API is consistent with the other APIs and I usually strive for consistency as a main goal because it produces the least amount of surprise.


Screen reader support is lacking.

Please consider making this library better for people with disabilities. I have opened a PR (#47) that will make the very simple changes required to make this widget WAI-ARIA 1.1 compliant.

👍


Add a system check for inconsistent subpage_types and parent_page_types

Trivial example:

Page A has subpage_types set to [Page B]
Page B has parent_page_types set to [Page C]

-> Can't add sub pages to Page A
A very minor gotcha, but an inconsistency warning would still be a nice enhancement.

A couple of problems I see with this:

What are the criteria for determining that a particular combination of subpage_types / parent_page_types is "inconsistent"? Saying "it's inconsistent if it results in being unable to add sub-pages" doesn't work, because there can be legitimate reasons for disallowing subpages.
If we do come up with an answer for the above, but the criteria are too broad - i.e. it warns about a setup that is actually what the developer intended - how would a developer silence that warning? If we're going to start warning about things that aren't technically invalid but just look dubious, I think we need to approach that very carefully - otherwise people will get 'warning fatigue' and overlook more serious issues.


Change icon color on SearchView

Kind (Required)

Improvement

Overview (Required)

Change right icon color on SearchView. change to colorPrimary.

Links

🙋

Assigned🔎


1.1.1 tagging

@doutriaux1 once v1.1.1 has been tagged, we'll need to:

[ ] Validate version info is correctly written (version indicated v1.1.1)
[ ] Update wiki so correct paths are noted (for test cases, demos etc)
[ ] Obtain zenodo DOI
[ ] Finalise README.md with updated DOI
[ ] Move tag point to new commit
[ ] After tag point set, reset all checkout tagpoints to point back to master or nightly development tags (metrics, UV-CDAT)
[ ] Generate conda release and update https://anaconda.org/PCMDI/pcmdi_metrics/files

@doutriaux1 @gleckler1 is it tag-a-roony time yet?

We are not going to make it before you take off,  but count on this happening on Aug 3.
Happy travels
P
From: "Paul J. Durack" <notifications<EMAIL_ADDRESS>Reply-To: PCMDI/pcmdi_metrics <reply<EMAIL_ADDRESS>Date: Thursday, July 21, 2016 at 1:27 PM
To: PCMDI/pcmdi_metrics <pcmdi_metrics<EMAIL_ADDRESS>Cc: gleckler1 <pgleckler<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [PCMDI/pcmdi_metrics] 1.1.1 tagging (#375)
@doutriaux1https://github.com/doutriaux1 @gleckler1https://github.com/gleckler1 is it tag-a-roony time yet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PCMDI/pcmdi_metrics/issues/375#issuecomment-234373463, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEV6rcRdniyqEp2fu-F6hsjIIlM5oQCyks5qX9YhgaJpZM4JHXdb.

@gleckler1 if you have another week, may as well get more ambitious then.. What else can be wedged in in a week?

Just had a pow wow… change of plans.   Tagging now.   Will be ready for beautification with zenodo at all shortly…
What time are you leaving home?
From: "Paul J. Durack" <notifications<EMAIL_ADDRESS>Reply-To: PCMDI/pcmdi_metrics <reply<EMAIL_ADDRESS>Date: Thursday, July 21, 2016 at 1:55 PM
To: PCMDI/pcmdi_metrics <pcmdi_metrics<EMAIL_ADDRESS>Cc: gleckler1 <pgleckler<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [PCMDI/pcmdi_metrics] 1.1.1 tagging (#375)
@gleckler1https://github.com/gleckler1 if you have another week, may as well get more ambitious then.. What else can be wedged in in a week?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PCMDI/pcmdi_metrics/issues/375#issuecomment-234381284, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEV6rR5SycfePAz-5WvBcim4o26WTcu2ks5qX9zJgaJpZM4JHXdb.

@doutriaux1 @gleckler1 #374 and #343 are still open?

@doutriaux1 [![DOI](https://zenodo.org/badge/23150/PCMDI/pcmdi_metrics.svg)](https://zenodo.org/badge/latestdoi/23150/PCMDI/pcmdi_metrics) for Zenodo DOI/Badge


(markdownlint-cli) Support installing with background service enabled

markdownlint-cli package installs a global npm package. When using background service this will be installed global for the user under which the agent service runs and therefore won't be available for the user installing the tool.
After switching to background service, uninstalling the package also no longer will uninstall previous versions of the node module installed with a different user.

Global installation path can be changed using npm config set prefix (and current prefix can be read and temporary stored using npm prefix -g).
Open is the question how to get the correct folder while running under background service. See also Gitter chat).

PR for an extension which supports installation / uninstallation of NPM packages when using background service: https://github.com/chocolatey-community/chocolatey-coreteampackages/pull/1560


feat(price-feeder): computed price api routes

Description
I did some performance measurements before and after this change and the timing difference is negligible. That said, I think we could get some performance improvements by computing these per-provider prices first and passing them to the convert/filter/compute functions however it will require a decent amount of re-factoring that would make this PR much larger.

Added additional API routes for the price-feeder to debug the per-provider computed prices
Remove unused error handling from ComputeVWAP
Updated the lock on the Oracle.prices variable to lock when writing

closes: https://github.com/umee-network/umee/issues/1263#event-7378865002

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[x] added ! to the type prefix if API or client breaking change
[x] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[x] provided a link to the relevant issue or specification
[x] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

Merging #1445 (5e45dbc) into main (421da96) will decrease coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head 5e45dbc differs from pull request most recent head de65ba3. Consider uploading reports for the commit de65ba3 to get more accurate results

@@            Coverage Diff             @@
##             main    #1445      +/-   ##
==========================================
- Coverage   52.77%   52.76%   -0.01%
==========================================
  Files          71       71
  Lines        6759     6762       +3
==========================================
+ Hits         3567     3568       +1
- Misses       2922     2924       +2
  Partials      270      270

Impacted Files
Coverage Δ

app/upgrades/v3/min_commission.go

app/upgradev3/migrations.go
78.26% <0.00%> (ø)

ante/ante.go
60.00% <0.00%> (+1.37%)
:arrow_up:


Added version to datadir to store the version of the how-to-npm package.

If you have an old version of how-to-npm installed that doesn't use scoped packages it is not possible to finish 04 Install a module because that new module has not been unpacked in the local storage. This PR checks if how-to-npm has been installed prior to this case and reapplies the unpacking.

Thanks for the patch! Let's move to https://github.com/workshopper/how-to-npm/pull/113.


Revert back to original string from leet string.

1-1&7LO

↓
Hello

The converted string cannot be reverted completely because there are duplicate strings in the converted string with other converted strings.
If it is not complete, similar functionality may be feasible, but we will close this issue.


Ensure Tile from tile() has integer x, y, z

Resolves #115

Coverage remained the same at 99.6% when pulling b71e5017d5a955657c7e79bbd377306752d53d9b on missing-int into 9e7b52f6a2019f904b2ceba3f5f9ca6ac8f02b80 on master.


Fix peers installation

Added prepare script to install peer-dependencies after the yarn install command

Closes https://github.com/jungsoft/eslint-config/issues/3


[BUG]

just installed this, when i try to hide the date and time it hides the network connection and sound buttons as well

I am not sure if it in conflict of translucentTB and ElevenClock

If your carefully read the readme file on the cover, what Win11ClockToggler does is hiding not only the clock but everything that is in that area. That includes the system icons:

What happens in your case is that ElevenClock hides the original Windows Clock and puts its own. When my app runs it hides the UI element that contains the clock, but it's not aware of the one added by ElevenClock. So the net result is that only the system icons are hidden.
Summing up: it won't work with Elevenclock, just with the native Windows clock.
I didn't know of the existence of ElevenClock. I'll check its code to see if it gives me a clue on how to hide just the native clock.

I've check the code of ElevenClock and what they do is to position a new window above the native clock, hiding it below. It doesn't find the real position of the clock: just asume a width and height and put its own clock above it. In fact, in my System, it hides part of the system icons. So it doesn't hide or change the native clock: just creates a new one and tries to hide the original positioning itself where it is supposed to be.
So, as I correctly supposed, if you're using that app, Win11ClockToggler correctly hides the native clock (and system icons), but the extra ElevenClock window remains in place.
Just don't use ElevenClock if you want to hide the native clock.


Write a unittest for get_full_manifest

Description
Write unittest/s for get_full_manifest function, this will act as an example for writing more unittest in parser directory
You will likely need to

constructing a Manifest object using fixture/functions generated in #9665 and mock the loader.load() to return it directly.
create some fixture/function to generate a RuntimeConfig.
do some mocking for tracking.

Non-functional requirements
Please organize tests related to parser/manifest.py and util functions related to make it easier for the next person to find it.
Potential existing arts: https://github.com/dbt-labs/dbt-core/blob/ebc22fa26c9cb26b6deef86cc0a7ceb1ee3fb642/tests/unit/test_manifest.py#L108

Determine whether we want to refactor this function into a individual file after adding the unittest.


Blog Go Web-apps repackaging and copyedits

Followup to #1581

Preview: https://deploy-preview-1646--opentelemetry.netlify.app/blog/2022/go-web-app-instrumentation/

[x] Create page bundle with local & compressed images
[ ] Get approval from @NavehMevorach @svrnm @cartermp
[ ] Remove draft status

This is ready for review @NavehMevorach @svrnm @cartermp.
I won't have time to make a full pass for copyediting, but this article is fine to publish as is.

@svrnm - is this good to go?


Bugfix for flowing text always adding new page

Allow text added on a page to flow onto the next page if it already exists, rather than always adding a new page below.
Bugfix
https://github.com/foliojs/pdfkit/issues/1201
What kind of change does this PR introduce?
Text added to any page that is followed by another can flow onto the next existing page. This fixes a bug that would always add a new page regardless of whether the page is the last one or not.
Checklist:

[x] Unit Tests N/A
[x] Documentation
[x] Update CHANGELOG.md
[x] Ready to be merged

Manual tests have been run in the case of using buffered and unbuffered page output.

This will require re-testing with the amend of this.document.addPage() to this.document.continueOnNewPage();

Sorry for the delay.
I'm trying to cut a new release.
Could you add a sample example and a unit test?
@liborm85 @devongovett can you review if this behavior is desired?

Sorry for the delay.
I'm trying to cut a new release.
Could you add a sample example and a unit test?
@liborm85 @devongovett can you review if this behavior is desired?

Just to clarify, the original issue this is meant to correct was from a use-case where I was producing a two-column layout which was intended to flow onto the next page. See the issue  https://github.com/foliojs/pdfkit/issues/1201 for a minimal code sample.

Thanks.
I will wait for other dev opinions on the behavior. BTW: the two column layout example seems not the best one since AFAIK, a complete column implementation is a lot more involved: when a text in the left column overflows should be to continue on right column in the same page. And when the right column overflows should continue on the left column in next page.
A better API would allow to customize what to do when text overflows, add page or not and from what position restarts
Anyway having unit tests and small examples in demo folder improves the likehood of being merged. See recent attachment and form PRs.

Thanks.
I will wait for other dev opinions on the behavior. BTW: the two column layout example seems not the best one since AFAIK, a complete column implementation is a lot more involved: when a text in the left column overflows should be to continue on right column in the same page. And when the right column overflows should continue on the left column in next page.
A better API would allow to customize what to do when text overflows, add page or not and from what position restarts
Anyway having unit tests and small examples in demo folder improves the likehood of being merged. See recent attachment and form PRs.

I agree, having an api to control flow would be better. My case is for columns that span pages vertically, rather than the content continuing from left to right in the next column, hence not using the column structure. I don't think my fix should interfere with that default behaviour, although more tests are needed.

I just added visual tests and slightly improved the code #1255

Closed in favor of #1256


Update Date(Time) Procedures

Update duration function (Section 8.1)
Update precision method (Section 8.2)
Add list of time units (Section 11.3)

Thanks!


remove failed/early-stopped trials from self.running_data in parallel-tpe

Short summary about the issue/question: remove failed/early-stopped trials from self.running_data in parallel-tpe
https://github.com/microsoft/nni/blob/master/src/sdk/pynni/nni/hyperopt_tuner/hyperopt_tuner.py#L318

https://github.com/microsoft/nni/pull/1456, fixed, close


Added ch (Switzerland) phone mask

Added the ch (Switzerland) phone mask because we need it in our current project. Would be awesome if you would accept this request. Thanks for your work!

Changes Unknown when pulling d3e4d76bd467eaf5f43297ca019050113fd301a7 on philippspinnler:master into ** on assisrafael:master**.

Let me fix the selenium tests. Just a sec ;)

Changes Unknown when pulling d3e4d76bd467eaf5f43297ca019050113fd301a7 on philippspinnler:master into ** on assisrafael:master**.


First version :1st_place_medal:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@91187a0). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 2295013 differs from pull request most recent head bbabea6. Consider uploading reports for the commit bbabea6 to get more accurate results

@@         Coverage Diff         @@
##             main   #2   +/-   ##
===================================
  Coverage        ?    0
===================================
  Files           ?    0
  Lines           ?    0
  Branches        ?    0
===================================
  Hits            ?    0
  Misses          ?    0
  Partials        ?    0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


MINOR: Add license header to files

Scanned and added License header to the file.
Not sure for some file like requirements.txt etc, hence skipped them

No clue really. Might've been a flake or something

@lidavidm let's take r package in a different PR. I shall try to get the setup working locally. Updated title accordingly

Thanks!


Warning [validator.w3.org]:  The type attribute is unnecessary for JavaScript resources.

What steps will reproduce the problem?
https://validator.w3.org/nu/?doc=https%3A%2F%2Fsko4.com%2Fevents%2Fvero--igor-glushko-at-port
What is the expected result?
no warnins
What do you get instead?
Warning [validator.w3.org]:  The type attribute is unnecessary for JavaScript resources.
Additional info

Q
A

Yii version
Yii version 2.0.14-dev

PHP version
PHP 5.6.30-0+deb8u1

Operating system
2.6.32-openvz-042stab125.3-amd64 GNU/Linux

You tell about use <script> instead of <script type="text/javascript">?

I could confirm this appears when


Avoid spurious updates in "git-go-patch extract"

Resolves https://github.com/microsoft/go/issues/751

Rendered test data docs: https://github.com/microsoft/go-infra/tree/dev/dagood/skip-effective-unchanged/patch/testdata

For example, take this patch conflict resolution commit: https://github.com/microsoft/go/commit/566a95f0ab053e4cf1648b20b3b95f5fcf581158. When I used git go-patch extract, it changed patch file 0001, too, not just the 0003 and 0005 that needed fixes. I had to manually revert the spurious 0001 change. The 0001 diff looked like this:
diff --git a/patches/0001-cmd-dist-add-JSON-output-support-for-some-tests.patch b/patches/0001-cmd-dist-add-JSON-output-support-for-some-tests.patch
index f007b02af4..20548be539 100644
--- a/patches/0001-cmd-dist-add-JSON-output-support-for-some-tests.patch
+++ b/patches/0001-cmd-dist-add-JSON-output-support-for-some-tests.patch
@@ -17,10 +17,10 @@ in JSON format, and the ordinary logs are still important.
  1 file changed, 10 insertions(+), 1 deletion(-)

 diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go
-index 759377ffa54568..595e618cf29763 100644
+index 1663ff8c94868e..1ce8546bde668d 100644
 --- a/src/cmd/dist/test.go
 +++ b/src/cmd/dist/test.go
-@@ -31,6 +31,7 @@ func cmdtest() {
+@@ -29,6 +29,7 @@ func cmdtest() {

        var noRebuild bool
        flag.BoolVar(&t.listMode, "list", false, "list available tests")
@@ -28,7 +28,7 @@ index 759377ffa54568..595e618cf29763 100644
        flag.BoolVar(&t.rebuild, "rebuild", false, "rebuild everything first")
        flag.BoolVar(&noRebuild, "no-rebuild", false, "overrides -rebuild (historical dreg)")
        flag.BoolVar(&t.keepGoing, "k", false, "keep going even when error occurred")
-@@ -57,6 +58,7 @@ type tester struct {
+@@ -55,6 +56,7 @@ type tester struct {
        msan        bool
        asan        bool
        listMode    bool
...

The changes are to the index hash (related to the last commit that updated the file that was patched) and the line numbers of the chunks that are modified. None of these changes are necessary to successfully apply the patch file, and none of them have any effect on the result: a git diff between the result of applying the "old" patch and "new" patch returns zero diff.
This PR essentially adds logic to run the git diff on the results and keep the old patch if there is no diff. This feature means that after I fix up patch conflicts and run git go-patch extract, I get https://github.com/microsoft/go/commit/566a95f0ab053e4cf1648b20b3b95f5fcf581158 without doing any more manual work.
This also means that if there aren't any patch conflicts, git go-patch apply -> git go-patch extract will round-trip properly without modifying any patch files. This also means you can git go-patch rebase, edit an individual commit, and only that specific patch file will be changed after git go-patch extract.
This feature is all-or-nothing per patch file. Chunks within each modified patch file might still have spurious changes. Editing out unnecessary changes to chunk line numbers seems like it could work (am/apply is smart and will find the correct place to make the change), but would require us to parse patch files much deeper.

Spurious changes don't always show up. The reason for the spurious changes is updating the submodule pointer to a later commit without regenerating the patch files. (We routinely do this in microsoft/go--the ability to do this is what makes the project reasonably maintainable.)
git am is smart enough to be able to apply old patch files to new submodule commits, but git format-patch has no knowledge of the existing patch files and the old commit they might be based on, so it chooses fresh index and chunk line numbers based on the current submodule commit.

Unfortunately, this process is slow, because it involves a lot of Git commands run in sequence. On my machines, the "reducing spurious changes" stopwatch reads 2.172s on Linux and 9.879s on Windows. Before this feature, extracting patches took 50-200ms. I think the time is worth it. Other teams might have (many) more patches than we do, so I included a built-in timer and a -verbatim flag to let people disable the feature if necessary. The -verbatim flag is also useful to bypass the feature in case it makes something go terribly wrong.
I haven't truly investigated speeding it up. I'm skeptical that more can be done in parallel, because we're limited by am and running patches in sequence on a single cloned Git repo. We could potentially find a Git library that has less overhead than running many git processes. We could also use the existing submodule instead of making a fresh clone, which increases the risk of destroying work-in-progress but reduces copying. am may also have more options to make it faster directly.

Thanks. 🙂


[feature request] live log improvements

To improve how efficient we can use the live log, i would like to suggest these options:\

add an icon for the traffic direction as seen from the interface
=> i.e. instead of having WAN or LAN like now, have => WAN / LAN and <= WAN / LAN to show if the traffic is coming into the interface or leaving it
(because now you have to: stop the live log, find the rule, click the info button, and look over the information to see the direction)

an option to resolve hostnames automatically (related: issue #2287), instead of having to click the info icon, then the resolve icon
=> this would be an option because not everyone needs it, and extra dns requests mean extra resources.
maybe the resolving, if enabled, can be limited to occur only when on the live log page and not for the entire logging?

for suggestion 1 i have a quickfix in the file core/fw_log.volt :
`                                    case 'interface':
if (interface_descriptions[record[column_name]] != undefined) {
if (record['dir'] == 'in') {
log_td.html('' + interface_descriptions[record[column_name]] + '');
}
else {
log_td.html('' + interface_descriptions[record[column_name]] + '');
}
`

I can add an extra field for inbound/outbound, which you can then also use in your filters (e.g. lanout)

@thutex https://github.com/opnsense/core/commit/e53953001489f8e7cce491787fdad60eafd87fad adds direction, I suggest we keep your other feature request in https://github.com/opnsense/core/issues/2287 and close this one

@AdSchellevis thank you, this is exactly what i was looking for.
i agree that we can close this one and continue the hostname resolution request in #2287


Change input encoding and add HPO description to output

The loading of the input file resulted in the error and resolved by setting UTF-8 encoding:
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 57: ordinal not in range(128)

The output did not include the HPO names, which were added at the end
263     271     HP:0012072      0.88584137      Aciduria
294     317     HP:0025356      0.99174535      Psychomotor retardation
319     336     HP:0001999      0.8883195       Abnormal facial shape

Hope that helps

Thanks for the fix.


Cannot read property 'appendChild' of undefined at lstr.js:26

Bug Report or Feature Request (mark with an x)
- [X ] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [X] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

node --version 8.9.3
npm --version  6.4.0
ng --version 6.0.8
Repro steps

ng create somecomponent
cd somecomponent
npm run start
The log given by the failure

Error in google web tools console:
core.js:3123 Angular is running in the development mode. Call enableProdMode() to enable the production mode.
lstr.js:26 Uncaught TypeError: Cannot read property 'appendChild' of undefined
at lstr.js:26
(anonymous) @ lstr.js:26
Desired functionality

No errors when creating a new project.
localhost-1535624327980.log
Mention any other details that might be useful

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

GitHub repo created.
https://github.com/peter-snr/repro-app
I also got this warning from GitHub.
_

We found potential security vulnerabilities in your dependencies.
Some of the dependencies defined in these manifest files have known security vulnerabilities and should be updated:
package-lock.json 2 vulnerabilities found

_

@filipesilva I tried incognito mode with Chrome and it is fine, I also checked with Firefox and also fine.  I guess you are correct, it is most likely related to a plugin.  Thank you for taking a look.


Crashes everytime

Checklist

[  ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.02.22
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '--skip-download', '--no-warnings', '--default-search', 'ytsearch', '-x', '-g', ' https://www.youtube.com/watch?v=qNf9nzvnd1k']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2021.02.22
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg 2021-01-09-git-2e2891383e-full_build-www.gyan.dev
[debug] Proxy map: {}
Trailing option(s) found in the command: may be ignored.
[tls @ 000001aaa26c3a80] Error in the pull function.
[mov,mp4,m4a,3gp,3g2,mj2 @ 000001aaa26bfa80] Packet corrupt (stream = 0, dts = 5751808).
https://r3---sn-uxaxoxu-cg0r.googlevideo.com/videoplayback?expire=1614093387&ei=68c0YO2lIe2ix_APle-hiAs&ip=2a02%3A1810%3A4e0a%3Aa300%3A413a%3A900e%3Abd44%3A6ed&id=o-ABXV0BnhA3APDaLWbAmrQj-YEAqikKca3to_42WRmLZH&itag=140&source=youtube&requiressl=yes&mh=Ob&mm=31%2C29&mn=sn-uxaxoxu-cg0r%2Csn-5hne6nzs&ms=au%2Crdu&mv=m&mvi=3&pl=33&initcwndbps=1701250&vprv=1&mime=audio%2Fmp4&ns=fPaLxzdK_xDR-W1TL65-H_sF&gir=yes&clen=2330999&dur=143.986&lmt=1604957343412413&mt=1614071316&fvip=3&keepalive=yes&fexp=23983797&c=WEB&txp=5531432&n=Ew6LOmrq0y_20eB6&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIgc7yzwMwC8l5IMBM4s2tQ93E0gIbgVaenk2yic5t1LFACIQDsOCMiUYvSN0Ihy6O4Xck7T6yQj9zswa13AEJn79O3ag%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRgIhANhhd_eBE0Z6MAq-PuY6ivofpBG2kMvEjqaLjcNQ1FwpAiEAvNl26BVzFcwkoItkP925gPLkCncZ7r8DTpT5KCaZ_Sw%3D: corrupt input packet in stream 0
[aac @ 000001aaa2d5db80] TYPE_FIL: Input buffer exhausted before END element found
Error while decoding stream #0:0: Invalid data found when processing input
[mov,mp4,m4a,3gp,3g2,mj2 @ 000001aaa26bfa80] stream 0, offset 0x204027: partial file
https://r3---sn-uxaxoxu-cg0r.googlevideo.com/videoplayback?expire=1614093387&ei=68c0YO2lIe2ix_APle-hiAs&ip=2a02%3A1810%3A4e0a%3Aa300%3A413a%3A900e%3Abd44%3A6ed&id=o-ABXV0BnhA3APDaLWbAmrQj-YEAqikKca3to_42WRmLZH&itag=140&source=youtube&requiressl=yes&mh=Ob&mm=31%2C29&mn=sn-uxaxoxu-cg0r%2Csn-5hne6nzs&ms=au%2Crdu&mv=m&mvi=3&pl=33&initcwndbps=1701250&vprv=1&mime=audio%2Fmp4&ns=fPaLxzdK_xDR-W1TL65-H_sF&gir=yes&clen=2330999&dur=143.986&lmt=1604957343412413&mt=1614071316&fvip=3&keepalive=yes&fexp=23983797&c=WEB&txp=5531432&n=Ew6LOmrq0y_20eB6&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIgc7yzwMwC8l5IMBM4s2tQ93E0gIbgVaenk2yic5t1LFACIQDsOCMiUYvSN0Ihy6O4Xck7T6yQj9zswa13AEJn79O3ag%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRgIhANhhd_eBE0Z6MAq-PuY6ivofpBG2kMvEjqaLjcNQ1FwpAiEAvNl26BVzFcwkoItkP925gPLkCncZ7r8DTpT5KCaZ_Sw%3D: Invalid data found when processing input

Description

I use ffmpeg, youtube-dl and lua to make my discord bot play yt videos. Everything works well but when playing a youtube video, it stops when it almost reaches the end. It never completes no matter the duration.

You are not downloading with youtube-dl in the first place thus see your downloading code.

I see, I will change the code. Thanks.


fix: [CDS-76667]: Add defaulting scenerio for managing services and environment using gitEx

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659ab0a45fbe69952c78c141--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659ba4defd300724b090dd48--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659ba51702b53d1eb598001b--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659ba5348ca97b256191f110--harness-developer.netlify.app


Scrollable Tabs not work when user scale display settings to 125%

Problem
I am having some problems with Scrollable Tabs.
Some users change their display settings, scale from 100% to 125%, and when user clicks the next button on Scrollable Tabs, the tabs will go back to original position.

My users' environment

Tech
Version

Windows
10

Material-UI
1.1.0

Browser
Google Chrome 66.0.3359.181

Screen Resolution
1920x1080

Thanks!

This reproduces on the Material UI demo page, in chrome, with display zoom set to 125%

Go to chrome://settings -> Appearance -> Page Zoom -> Set to 125%
Navigate to https://material-ui.com/demos/tabs/
Scroll down to the "Automatic scroll buttons demo"
Click the scroll button on the right.
The tabs will scroll right, then back to the original position.
You can work around this by first selecting the right-most visible tab, then clicking the scroll button.

I tested these steps on Mac and Linux.

Thanks for reproduction instructions! @novascreen has found a nice workaround https://github.com/mui-org/material-ui/issues/10826#issuecomment-406889161.

Sorry but  continue to happen after updated to 1.4.1...
Its specified that it's included in this version (CHANGELOG.md)

@michael-ecb ?


[Windows 10] [C LIBRARY] : PNG read always returns unexpected end of file

Hello,
I try to use wuffs to open png files within a c++ project, though I encounter some difficulties on Windows. I can't open a png, for every single image that I tried, wuffs always returns :
wuffs_aux::DecodeImage: unexpected end of file
I tried on Linux (latest ubuntu version) and it works well, but I'm having this issue on Windows (though the images are not corrupted, OpenImageIO and libpng are opening them).
The code I use to test is there and very simple (I copied what was in the examples):
#include "iostream"
#include "chrono"

#define WUFFS_IMPLEMENTATION
#define WUFFS_CONFIG__MODULE__PNG
#include "wuffs-unsupported-snapshots.c"

uint32_t g_width = 0;
uint32_t g_height = 0;
wuffs_aux::MemOwner g_pixbuf_mem_owner(nullptr, &free);
wuffs_base__pixel_buffer g_pixbuf = {0};

bool load_image(const char* filename)
{
  FILE* file = stdin;
  const char* adj_filename = "<stdin>";
  if (filename) {
    FILE* f = fopen(filename, "r");
    if (f == NULL) {
      printf("%s: could not open file\n", filename);
      return false;
    }
    file = f;
    adj_filename = filename;
  }

  g_width = 0;
  g_height = 0;
  g_pixbuf_mem_owner.reset();
  g_pixbuf = wuffs_base__null_pixel_buffer();

  wuffs_aux::DecodeImageCallbacks callbacks;
  wuffs_aux::sync_io::FileInput input(file);
  wuffs_aux::DecodeImageResult res = wuffs_aux::DecodeImage(callbacks, input);
  if (filename) {
    fclose(file);
  }

  // wuffs_aux::DecodeImageCallbacks's default implementation should give us an
  // interleaved (not multi-planar) pixel buffer, so that all of the pixel data
  // is in a single 2-dimensional table (plane 0). Later on, we re-interpret
  // that table as XCB image data, which isn't something we could do if we had
  // e.g. multi-planar YCbCr.
  if (!res.pixbuf.pixcfg.pixel_format().is_interleaved()) {
    printf("%s: non-interleaved pixbuf\n", adj_filename);
    return false;
  }
  wuffs_base__table_u8 tab = res.pixbuf.plane(0);
  if (tab.width != tab.stride) {
    // The xcb_image_create_native call, later on, assumes that (tab.height *
    // tab.stride) bytes are readable, which isn't quite the same as what
    // wuffs_base__table__flattened_length(tab.width, tab.height, tab.stride)
    // returns unless the table is tight (its width equals its stride).
    printf("%s: could not allocate tight pixbuf\n", adj_filename);
    return false;
  }

  g_width = res.pixbuf.pixcfg.width();
  g_height = res.pixbuf.pixcfg.height();
  g_pixbuf_mem_owner = std::move(res.pixbuf_mem_owner);
  g_pixbuf = res.pixbuf;

  if (res.error_message.empty()) {
    printf("%s : ok\n", adj_filename);
  } else {
    printf("%s: %s\n", adj_filename, res.error_message.c_str());
  }

  return res.pixbuf.pixcfg.is_valid();
}

inline auto get_time()
{
    return std::chrono::high_resolution_clock::now();
}

int main(int argc, char** argv)
{
    auto start = get_time();

    bool loaded = load_image(argv[1]);

    if(loaded) std::cout << loaded << "\n";

    auto end = get_time();

    std::chrono::duration<double> elapsed = (end - start);

    printf("Wuffs : %fs\n", elapsed.count());

    return 0;
}

Let me know if this is not appropriate/not the place to ask for help on this subject.
Cheers,

Perhaps you need fopen(filename, "rb"), i.e. specify binary-mode, to avoid eol shenanigans? Windows-specific weirdness.

Oh yeah, I didn't paid attention to this... Works fine, thanks !


Should be 'https' in TLS example.

Contributing guidelines
Please read our contribution workflow before submitting a pull request.

@fbarbeira Could you fix your commit title to clustering.md: fix typo?


How to add a panic recovery filter or middleware

/kind support
I want to add a panic recovery filter or middleware, but can't found a right way.

It can work correctly in middleware, so i close this issue.


AutoSize is not working correct (when top row is bold)

Here's my code:
$IMPORT = Import-Csv "C:\Input.csv"
$IMPORT | Export-Excel -Path "C:\Output.xlsx" -FreezeTopRow -BoldTopRow -AutoSize -AutoFilter -WorkSheetname "My Excel" -CellStyleSB {
# HorizontalAlignment "Center" of columns A-H
param($WorkSheet)
$WorkSheet.Cells["A:H"].Style.HorizontalAlignment="Center"
}

It seems that -AutoSize uses the standard text and NOT the bold one, so I always have to correct the auto size manually in Excel. Is this a bug or do I have to use another command/code?
Thanks in advance!

I think it is because -AutoFilter is rendered when the sheet is opened and cannot be calculated when the sheet is created.


enable web_long_running_tests shard

Description
Enable web_long_running_tests shard. The led runs are successful:

https://ci.chromium.org/p/flutter/builders/try/Linux SDK Drone/91634?
https://ci.chromium.org/p/flutter/builders/try/Linux SDK Drone/91635?
https://ci.chromium.org/p/flutter/builders/try/Linux SDK Drone/91642?

Related Issues
Fixes https://github.com/flutter/flutter/issues/69028

https://ci.chromium.org/p/flutter/builders/try/Linux SDK Drone/91634

Did you directly triggered the shards? I wonder what happens when we start from flutter.py

https://ci.chromium.org/p/flutter/builders/try/Linux SDK Drone/91634

Did you directly triggered the shards? I wonder what happens when we start from flutter.py

We will see it in: https://ci.chromium.org/p/flutter/builders/try/Linux web_long_running_tests/1 :)


[REQUEST] VPN Killswitch

Is your feature request related to a problem? Please describe.
A VPN killswitch feature which will completely disable downloading or uploading when a bound network adapter is unavailable. Qbittorrent has a this feature.

Describe the solution you'd like
A similar dropdown for selecting a network adapter to force all traffic through.
Describe alternatives you've considered
None
Additional context
None

+1

+1


Release docker to docker hub or public github docker registry

Problem description
When we run guardian in production, we dont want to rebuild all the docker image and push to our private registry
Requirements

Each release of guardian, the docker image should be built by CI on tag and publish to docker hub/github registry

Definition of done
Acceptance criteria
What are the criteria by which we determine if the issue has been resolved?

We need to create a Docker account to store all the docker images. We are working on it.

Thanks @prernaadev01, We can using github docker registry if you have any problem with docker hub however, i believe that you will have no issue at

@prernaadev01 just as an FYI - I am in the process of writing a full stack Terraform setup for this - as part of this I am pulling down the hashgraph/guardian repo, building all of the Docker images and pushing them to Docker Hub (that the user needs to provide credentials for).
If Envision can publish these docker images it will simplify things but there needs to be strict adherence to versions that match the Git release tags so we can specify a version number

Is it ok if we publish to GCR? @voycey @samuraitruong

Yep as long as they are public we are fine with wherever!

From: Serg Metelin @.>
Sent: Tuesday, May 3, 2022 7:12:12 AM
To: hashgraph/guardian @.>
Cc: Dan Voyce @.>; Mention @.>
Subject: Re: [hashgraph/guardian] Release docker to docker hub or public github docker registry (Issue #741)
Is it ok if we publish to GCR? @voyceyhttps://github.com/voycey @samuraitruonghttps://github.com/samuraitruong
—
Reply to this email directly, view it on GitHubhttps://github.com/hashgraph/guardian/issues/741#issuecomment-1115373243, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIEBCSY6A5COWQXDFH55WTVIBAKZANCNFSM5TXEFJCA.
You are receiving this because you were mentioned.Message ID: @.***>

https://github.com/hashgraph/guardian/pull/868

Is the image available in the public registry?
On 3 May, @sergmetelin mentioned "GCR", which i interpret as "Github Container Registry" https://ghcr.io
On 12 May, @prernaadev01 says "GCP", which i interpret as "Google Cloud Platform" , which has a container registry (https://cloud.google.com/container-registry), but not public as far as i know?

Is there a container image available in a public registry?
On 3 May, @sergmetelin mentioned "GCR", which i interpret as "Github Container Registry" https://ghcr.io On 12 May, @prernaadev01 says "GCP", which i interpret as "Google Cloud Platform" , which has a container registry (https://cloud.google.com/container-registry), but not public as far as i know?

https://console.cloud.google.com/gcr/images/hedera-registry/GLOBAL/ << its all here :)


Unicode: Added hardcoded aliases

Description
Updated API to 1.42.0 since no newer change is needed
Updated code to support newer API
Added a few hardcoded aliases
Screencast

Closes #3715

Hi @mmazzarolo
Do you have time to give it a quick spin 😊


Security issue with github.com/gogo/protobuf version < 1.3.2

github.com/gogo/protobuf versions lower that 1.3.2 there is security weakness.
An issue was discovered in GoGo Protobuf before 1.3.2. plugin/unmarshal/unmarshal.go lacks certain index validation, aka the "skippy peanut butter" issue.

Same problem relevant to github.com/cayleygraph/quad as well.

Should be fixed now. Updated to 1.3.2.


fix(dependencies): 💫 update crates

Description
Update dependencies. notify-debouncer-mini required one small code change,
removing a no longer needed argument in new_debouncer call.
Fixes # 108
Type of change

[X] Dependency update

How Has This Been Tested?

[X] cargo test run with all tests passing

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Current dependencies on/for this PR:

main

PR #109   👈

This comment was auto-generated by Graphite.


Does pushDataToPage handle display of returned json ? What should be ID name and structure

pushDataToPage uses id attribute. It does not use name attribute. from teh json received from server, values are pushed to the tag based on id. for example if josn is {"myName":"James Bond", "state":"CA"....}
and in your html somewhere  <span id="myName"...> then the innerHTML of this tag is replaced with James Bond. And, f there is California......  then california option is selecred.
In general, a tag is matched based on id, and the value is pushed based on the type of tag, including check-box and radio etc..


Fix access permissions in EnsureMemoryExecutable.

Some configuration in Ubuntu 22.04 seems to require "PROT_EXEC | PROT_READ" instead of "PROT_EXEC". I don't see a reason not to explicitly grant read access.
Bug: https://github.com/google/orbit/issues/4304
Test: Local run and unit tests worked fine.

Thanks!!


Wrong numbering sequence

[The page https://learn.microsoft.com/en-us/azure/communications-gateway/deploy has wrong numbering sequence. The step "Register your deployment's domain name in Active Directory" should be Step 8, instead of Step 6. ]

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

ID: d1b6ad84-0340-8569-d927-5482c3469993
Version Independent ID: 4ed80232-5387-e555-3fdb-e9638f71bcc6
Content: Deploy Azure Communications Gateway
Content Source: articles/communications-gateway/deploy.md
Service: communications-gateway
GitHub Login: @rcdun
Microsoft Alias: rdunstan

@faizyakob
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @faizyakob,
Thanks for reporting this. I've made an update, which should be visible within the next few hours.
Thanks,
Rachel

#please-close


fix: cli summary shows all run-specific scores

closes https://github.com/empirical-run/empirical/issues/167
@saikatmitra91
two things are done first

all the scorer names are taken out , instead of assuming that first run will have all the jobs as done previously in your code .
 const scorerNames = [  ...new Set(    runs.flatMap(      (run) => run?.stats?.scores?.map((score) => score.name) || [],    ),  ), ];

then if their is no score then is should show "-" instead of "0% "
if (scoreStats) {          const metric = percentStr(scoreStats.average * 100);          return enableColors            ? setMetricColor(metric, scoreStats.average)            : metric;        } else {          return "-";        }

@arjunattam comment is removed


Unable to build Ollama on Cluster

Hi,
Thanks for this great work. I am trying to build Ollama on my cluster and I don't have administrative access. My cluster has the following configuration:
LSB Version:	:core-4.1-amd64:core-4.1-noarch
Distributor ID:	CentOS
Description:	CentOS Linux release 7.9.2009 (Core)
Release:	7.9.2009
Codename:	Core

I follow these steps:
a) Clone the ollama repo using: git clone https://github.com/ollama/ollama
b) Following https://github.com/ollama/ollama/blob/main/docs/development.md, I do go generate ./... but I get the error:

I tried cloning the repo with the recursive submodules but it didn't help me much. Also, I tried older commits but it wasn't helpful either.

How is the repo cloned? It can be a problem if the ollama repo is itself a submodule which looks to be the case here.
You can skip this with by setting OLLAMA_SKIP_PATCHING to a non-empty value but this may leave the repo in a undefined state so it should be used as a last resort.

Hi @mxyng I cloned it normally using the command: git clone https://github.com/ollama/ollama I also tried with the --recursive flag to clone all the submodules but it didn't help either.

What version of git are you using?

Hi @mxyng I am using the ollama version 0.1.6. Although, I found a workaround utilizing the compiled binary provided in https://github.com/ollama/ollama/blob/main/docs/linux.md#download-the-ollama-binary It works fine as of now :)

Can you describe your build environment, specifically distro and its version, so we can try to reproduce?
I'm curious. Is there a specific reason you're interested in 0.1.6? There's been significant improvements since 0.1.6 so I'd suggest update as soon as possible.

Hi, there is no specific reason for me to use 0.1.6.
Sure, my build environment is a slurm cluster having this config:

My GPU is:

I am using the commit version:
git show

I'm fairly sure this is due to an old git version in your cluster.  I don't know what the minimum version is that's required to be able to init a submodule from a child directory in the repo, but if you can upgrade git, I believe the problem should go away.

Hi @dhiltgen Sure, I will retry. Thanks!


ServerPilot just updated to PHP 7.0 RC1

@reinink hey no need to work on this now :D

That was easy!


Add deprecated complement (~) operator to RegExp

Previously all regexp parsing required determinization and minimization up-front, which can be costly (exponential time).
Lucene 10 removes the determinization and minimization from RegExp and allows the user to choose:

determinize() the result and get the DFA query execution of previous releases.
don't determinize() and possibly get a new NFA query that determinizes-as-it-goes.

Complement of arbitrary automata is incompatible with this choice, as it requires determinization for correctness. It was previously a non-default operator that could be enabled with a special flag: RegExp.COMPLEMENT, or would be included with RegExp.ALL, which turns on all special syntax flags. Lucene 10 removed the operator, as it can't be supported while still giving the user the NFA/DFA choice, and requires exponential time during parsing.
To ease transition: add RegExp.DEPRECATED_COMPLEMENT syntax flag and Kind.DEPRECATED_COMPLEMENT node:

syntax flag can be enabled with RegExp(s, RegExp.DEPRECATED_COMPLEMENT);
syntax flag is NOT included by RegExp.ALL: e.g. you must do RegExp(s, RegExp.ALL | RegExp.DEPRECATED_COMPLEMENT) to get ALL flags and also the deprecated complement (~) operator. This enforces a java deprecation reference in the calling code to enable the flag.
deprecated complement (~) runs with an internal limit: Operations.DEFAULT_DETERMINIZE_WORK_LIMIT. It is not configurable. If it is exceeded, you get TooComplexToDeterminize exception.
there is intentionally only a single dead-simple test so that this hack doesn't cause us pain with CI/builds. We don't want random automata testing to only rarely encounter an exponential algorithm!

After lucene 10 is branched, this deprecated support can be removed by reverting this commit.

I think I overestimated the obscurity of this operator. In my opinion, with this strategy it won't annoy lucene at all and give users a lot more time to deal with migration. Just seems like an easy win. Maybe other users/projects are affected too, but just not as proactive about discovering this before the release.

… with this strategy it won't annoy lucene at all and give users a lot more time to deal with migration. Just seems like an easy win.

++ agreed. If you want to add migration advise in this PR then I’ll close the other #13732. Otherwise, I’ll update that PR to reflect what you’ve done here.
Thanks @rmuir

Lets update your migration PR with upgrade notes. Not many people want to mess with automaton regex parser, so I see some value in keeping the commit revertable? In general, trying to minimize surface area here to dodge any future issues with 10.x back porting too.

Thanks @rmuir!


Added PasswordAuthenticator Logic to CassandraConnector

Fixed #116. Note that the username and password specified in the make file don't actually affect anything, because the current docker image that we are using does not specify an authenticator in the cassandra.yml file (I don't believe our image even has that file at all): https://docs.datastax.com/en/cassandra/2.1/cassandra/security/security_config_native_authenticate_t.html

Changes made 👍


[Feature] Torque Calculator

Description
This torque calculator helps you find the torque arising in a rotating object. The torque (tendency of an object to rotate) depends on three different factors:
τ = r F sin(θ)
where:
r -> the distance from the axis of rotation to the point where a force is applied (Lever arm).
F ->  Force acting on the object.
θ -> Angle between the force vector and lever arm. Typically, it is equal to 90°.
τ -> Torque, whose units are newton-meters (Nm).
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Please assign this issue to me

@Rakesh9100 this issue can be assigned

@Rakesh9100 could you please assign this?

We already have Torque Calculator, check and lmk further @kauxp

okay then I'll change the calculator.

Do you want to work @kauxp

Yes, I want to work on it.

Proceed @kauxp

Provide your PR @kauxp

Done, please review the PR.


transforms fix

Description
#336
Fix on transforms

@jiata fyi this PR is to master

@jiata I am leaving some minor comments now. Have a look if this PR is merging more into staging that wanted (e.g. changes to README).

@jiata I am leaving some minor comments now. Have a look if this PR is merging more into staging that wanted (e.g. changes to README).

that was because i accidentally set base to master earlier

How about the below based on the commit on staging on Sep 28, 2019 (2e3deb4):
class DetectionDataset(Dataset):
    def __init__(
        self,
        root: Union[str, Path],
        batch_size: int = 2,
        transforms: Union[Trans, Tuple[Trans, Trans]] = (
                get_transform(train=True),
                get_transform(train=False)
        ),
        train_pct: float = 0.5,
        anno_dir: str = "annotations",
        im_dir: str = "images",
    ):
        ...
        ...
        if transforms and len(transforms) == 1:
            self.transforms = (transforms, ) * 2
        self.transforms = transforms
        ...
        ...

    def split_train_test(
        self, train_pct: float = 0.8
    ) -> Tuple[Dataset, Dataset]:
        ...
        ...
        self.is_test = np.zeros((len(self),), dtype=np.bool)
        self.is_test[test_idx] = True
        ...
        ...

    def _get_transforms(self, idx):
        """ Return the corresponding transforms for training and testing data. """
        return self.transforms[self.is_test[idx]]

    def __getitem__(self, idx):
        ...
        ...
        if self.transforms:
            im, target = self._get_transforms(idx)(im, target)
        ...
        ...

seems like a bit of a complicated way to determine which transforms are for test vs train when we can just use to params to store them.
Is there an advantage to have the 'is_test' over the subset method?

seems like a bit of a complicated way to determine which transforms are for test vs train when we can just use to params to store them.
Is there an advantage to have the 'is_test' over the subset method?

seems if
test = copy.deepcopy(Subset(self, indices[: test_num + 1]))

then the whole original dataset (self) will have two copies.  If I guess correctly, may it break the intention of using self to manipulate training and test datasets simultaneously in DetectionDataset without duplicating the data itself and data preprocessing?

seems like a bit of a complicated way to determine which transforms are for test vs train when we can just use to params to store them.
Is there an advantage to have the 'is_test' over the subset method?

seems if
test = copy.deepcopy(Subset(self, indices[: test_num + 1]))

then the whole original dataset (self) will have two copies. If I guess correctly, may it break the intention of using self to manipulate training and test datasets simultaneously in DetectionDataset without duplicating the data itself and data preprocessing?

Thats actually a really good point, thanks for bringing it up.

so i looked at the torchvision sample code in their docs and their code also replicates two versions of the dataset.
They do something like this:
dataset = Dataset(..., get_transforms(train=True))
dataset_test = Dataset(..., get_transforms(train=False))

dataset = Subset(...)
dataset_test = Subset(...)
...

In this case, they also have two copies of the dataset object (since Subset keeps the reference).
I'll merge the PR in for now.


convert to use streaming

Might alleviate the memory problems. Not sure if it would still be possible verifying that the JSON is valid (to prevent XSS through JSONP), but it might be ok to say "not our problem".

/cc #65


refactor(cmd): Extract command functions for Init and Start

I extracted command functions to test the Init and Start functions (closes #2442)
To be able to test these command functions they should be rewritten to accept functions from other packages as dependencies. But it would lead to significant changes, so I propose to create dedicated issues for that.
What do you think about it?

Thank you! Yes, do you mind creating the issues? Sounds good to me

Created two issues #2774 #2775

@rise2semi i'm not sure the tradeoff here between extracting the command and making the code slightly less legible is worth it for the quality of the unit test scope. What do you think about us abandoning this strategy, and working on more of an "integration" test that runs separately, spins up a node and then executes commands to evaluate their success/fail rate? Sure its not "unit" testing, but my sense is this current approach wont really test what happens in the Run of the executed commands, vs the command runner itself? We could work through it together if you like

Sure, I agree with your proposal.
I was trying to address the issue #2442.
But, I want to note, that I would like to see the command function itself as a simple wrapper rather than a big function with a lot of logic (what we have right now with cmd/Start).

i'm going to close this and move discussion about the refactoring of start and init, plus our strategies around making them tested, into the respective issues that were made


[docker/feature] create a light-weight init that can forward signals and reap zombies

Description
It would be great if Docker could handle the zombie processes within the container. Currently users have to add custom init applications in order to solve this issue (you can find more context here: https://blog.phusion.nl/2015/01/20/docker-and-the-pid-1-zombie-reaping-problem/).
A very cool project that solves that is: https://github.com/krallin/tini.
/cc @crosbymichael @justincormack @shykes

@rogaha This is already in master.
Set --init at the daemon level or at the container level.

Awesome, thanks for the heads up @cpuguy83! I wasn't aware about it!

Do you have a link to the documentation or an example?

https://github.com/docker/docker/blob/master/api/types/swarm/network.go#L74
Not really much to demo, just add --init to either dockerd or docker run and you will get a light-weight init in the container at PID1 that handles zombie reaping and signal forwarding.
It's using https://github.com/crosbymichael/grimes

Awesome, thanks @cpuguy83. I'll try it out!

@thaJeztah do we have docs for that feature? I'm assuming it will land on 1.13.

@rogaha nope, no docs yet; unfortunately docs review was skipped (and I was on vacation); I labeled the PR with "docs-revisit", and will do a docs PR to fix that https://github.com/docker/docker/pull/26061

Understood. Thanks for the heads up @thaJeztah. I'm glad to see it was already there! :)

props to @crosbymichael!


DOCS: Warn that Python 3.11 is not supported

It currently goes into a tailspin in 3.11 due to pytorch... to save people time adding a note to the readme may help.

Thanks! The latest version is supposed to prevent the attempt.
https://github.com/brycedrennan/imaginAIry/blob/master/setup.py#L110
Wonder what happened. Welcome any additional info you have about the environment and version you were installing.

Howdy Bryce,
Yeah I am certainly not a Python expert so treat everything I say with many grains of salt...
I am on Arch, Python version 3.11.3
The problem is that pip install in this case takes an enormous amount of time somehow on my install just to end up failing. Is there any chance that pip in 3.11 just chooses one of the earlier version that "did" support that version of python. Ideally ... what should happen on 3.11 is that it does not dependency download at all ... just says sorry ... use a diff version.

Oh I see. I think you're exactly right. I should remove the python_requires attribute and just have it throw an error as described.  This type of problem is discussed here:
https://discuss.python.org/t/requires-python-upper-limits/12663/73


[BUG] Part pricing not showing

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

Describe the bug*
When purchasing parts with sales orders and receiving them. Or adding pricing info to parts. The pricing information does not show in Parts > Pricing
More information of steps taken here: https://github.com/inventree/InvenTree/discussions/4232#discussioncomment-4742187
Steps to Reproduce

Perform one of the following:
a. Purchase a part via sales order with pricing information.
b. Manually add pricing information to a part.
Check the Parts > Pricing tab.

Expected behavior

Pricing information available.
'Price range data is not available for this part' is not shown.

Deployment Method

[ ] Docker
[X] Bare metal

Version Information
Version Information:
InvenTree-Version: 0.9.0
Django Version: 3.2.16
Database: sqlite3
Debug-Mode: False
Deployed using Docker: False
Active plugins: []
Relevant log output
No response

Hi there @method1020! Thank you for filing the report.
Would you have time to try reproducing this on the demo instance?
While you are running the current stable series I still was not able to reproduce this issue on master when creating a new part and going through the motions. Just want to make sure it is not related to instance-specific stuff to make debugging faster.

This is a bug in my version I think.
Steps to recreate:

Create a part.
Add supplier for part
Create purchase order for part, including line item for part, with pricing on line item.
Receive part. -- price now shows as a purchased price in the DEMO install, but not in my install.

@method1020 thanks for reporting :) The pricing overview data are calculated by the background worker process - you'll need to confirm that the background worker is running:
Under the user dropdown menu (top right corner of the screen) select "system information":

Can you please share a screenshot of what you see here?

Instance Name: InvenTree
Database: sqlite3
Plugin Support: Plugin support enabled
Server Status: Healthy

I'll try and figure out how to update everything to v0.9.2

@method1020 how did you go here? Any updates?

No. I ran out of time to mess with it so I exported the data, installed v0.9.2, and imported. It seems to be working as expected now.
I did go from bare metal-scripted install to docker-production install.
Thank you for the support.


Support for SELinux label

I don't know what we need. The first step is to identify what we need to do.
https://github.com/opencontainers/runtime-spec/blob/a3c33d663ebc56c4d35dbceaa447c7bf37f6fab3/config.md?plain=1#L213-L214

I also found https://opensource.com/article/18/2/understanding-selinux-labels-container-runtimes, which is from the author who contributed to selinux go bindings for OCI : https://github.com/opencontainers/selinux

@yihuaf Hi! Maybe are you interested?

I can try to do some investigation around this and see if I can come up with a design. Work starts to take up more time, so I may work slower than before.

@yihuaf That's right. If you will be busy and it gets difficult, please let me know via discord DM or something. It's no problem at your own pace. THanks again, you're a big help 😍

This is carried on in https://github.com/containers/youki/issues/2718


Added the auth page as per the requirements

💬FIXES (#190)
have added the auth page using the typescript at a route /auth . you can change the route as per your requirements .
Summary: Added the  exacr same auth page as per the requirements and also added the functionality of dark and light mode as required. i have implemented at a route /auth . you can change the routes as per your requirement
(Fixes #190)]
📂 Area of Change

[] 📱 Android
[] 🖥 Backend
[x] 🖼 Frontend
[] 📝 Docs

🧬 Type of change

[ ] 🐛 Bug fix
[ ] ♻ Refactor
[x] ✨ New feature
[ ] 📦 Dependency upgrade
[ ] 📝 Documentation update
[ ] 🔧 Other

✨ What's New

[]

🔒 Why

✅ Changes

[]

🛠 Checklist before requesting a review

[x ] 🔍 Style guide has been followed
[ x] 🧑💻 Performed a self-review of my code
[x ] 🔒 No backdated packages have been added

💡 Additional informations

Please check the mobile Device UI -

Missing App Bar
Non-Responsive on Mobile Devices

can you tell  me what is the app bar ...i am making it responsive

Appbar means the navigation menu on top of the page Home and Signup links will be there.

@Anurag-singh-thakur thank you for your contribution


fix(encoders): add error handling to en/decodeURI

Hello,
Tool: Encode/Decode - URL
Fixes: #30
Please find PR for your consideration.
This pull request adds error handling in the useEffect hook that is responsible for encoding and decoding URLs. The current implementation of the useEffect hook does not handle errors that may occur during the encoding or decoding process, leading to a URIError: URI malformed error when inputs are provided that the encodeURI or decodeURI was unable to proess properly.
To fix this issue, I have added a try-catch block to the useEffect hook to handle errors that may occur during the encoding or decoding process, If an error occurs, a message is displayed in the output.

Thanks

Thanks a lot for this fix, appreciate it!


Part of detail from question

How to get details from the conversation.. like.
User: hi
Bot:hi
User: how can u help
Bot:help u on different mobile details
User:can you provide me details for Samsung and apple
How do bot can get the two words Samsung and apple from user conversation..
Help is appreciated

You can do something like this:
+ can you provide me details for ((samsung|apple)) and ((samsung|apple))
- Let me find you some details for <cap1> and <cap2>.

You could even extract it to a concept:
+ can you provide me details for (~phone_brand) and (~phone_brand)
- Let me find you some details for <cap1> and <cap2>.

More info on concepts here: https://github.com/superscriptjs/superscript/wiki/Knowledge


Fixed broken Languages link and Updated spelling for consistency in app-insights-overview.md

Fixed broken 'Many languages' link to point to the 'Supported Languages' header
Updated autoinstrumentation to auto-instrumentation for consistency with other pages

@microsoft-github-policy-service agree

@AaronMaxwell
Thanks for reviewing the proposed changes.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Support for @custom tag

While the standard comment tags (@notice, @dev, ..) work great, Solidity also provides support for @custom:<something>.
Although everyone's use-case will be slightly different from custom tag, it would be super helpful to get a how-to on how to add a custom tag support to docusaurus.sqrl, or even provide sample ones for a common custom tags such as for the events emitted from a function.

That's a good idea, I'm still not sure how to implement this though. Maybe the easiest way is to add custom tags to the JSON blob object generated by Dodoc and then let templates display these tags the way they want.


Power attack doesnt work with unarmed attacks

Power attack active and monk attacks are not unaffected.

Each game is unique. Imagine if I have to setup a hero/save for each reported issue? Please do better, share your save, screenshots, mod and game version, steps to repro.

For example, if I had to choose which one to tackle first. This or https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/4920


Import Order for Multiple Functions

I followed the example (shown in the Readme.md and examples/hello.py) to import Python functions into a module.
As a next step, I want to import multiple Python functions into the moduleby instantiating with
instance = instance = Instance(store, module, [func_hello1, func_hello2, func_hello3])

From my code example (see below), I expect that the output would be ordered like:
Hello from Python1!
Hello from Python2!
Hello from Python3!

Instead I get a mixed output:
Hello from Python2!
Hello from Python3!
Hello from Python1!

Is the linking of multiple functions done in another way or what is causing the different order?
Below is the Python code I use to compile and run the module.
from wasmtime import Store, Module, Instance, Func, FuncType

store = Store()
module = Module(store.engine, """
  (module
    (func $hello1 (import "env" "hello1"))
    (func $hello2 (import "env" "hello2"))
    (func $hello3 (import "env" "hello3"))
    (func (export "run") (
        call $hello1
        call $hello2
        call $hello3
    ))
  )
""")

def hello1():
    print("Hello from Python1!")

def hello2():
    print("Hello from Python2!")

def hello3():
    print("Hello from Python3!")

func_hello1 = Func(store, FuncType([], []), hello1)
func_hello2 = Func(store, FuncType([], []), hello2)
func_hello3 = Func(store, FuncType([], []), hello3)

instance = Instance(store, module, [func_hello1, func_hello2, func_hello3])

run = instance.exports(store)["run"]
run(store)

Thanks for the report! Everything works as you'd expect but the problem here is a bit subtle. The s-expression syntax you're using is what's tripping up this example. Your input module, when printed via wasm-tools print, will print this for example:
(module
  (type (;0;) (func))
  (import "env" "hello1" (func $hello1 (;0;) (type 0)))
  (import "env" "hello2" (func $hello2 (;1;) (type 0)))
  (import "env" "hello3" (func $hello3 (;2;) (type 0)))
  (func (;3;) (type 0)
    call $hello2
    call $hello3
    call $hello1
  )
  (export "run" (func 3))
)

You'll notice here the call instructions aren't in the same order that you listed. The reason for that is the parentheses that you're using. The body of a function doesn't need extra parentheses so you'll want to use this instead:
  (module
    (func $hello1 (import "env" "hello1"))
    (func $hello2 (import "env" "hello2"))
    (func $hello3 (import "env" "hello3"))
    (func (export "run")
        call $hello1
        call $hello2
        call $hello3
    )
  )

Thanks @alexcrichton for the help. This solved my issue.


Something wrong when I converted pcd files to depth images, using the python script

@dougsm
Thank you for your code and paper first!
when I used the file generate_cornell_depth.py with the commandline
"python -m utils.dataset_processing.generate_cornell_depth ", the result  depth picture is strange,all result depth
images look the same,just like this

Hey @Zhangeaky
I've seen this before. I think the problem isn't that the files are incorrect, but rather the image viewer (e.g. the default viewer in ubuntu) doesn't know how to interpret the floating point data.
To test, you can try and visualise it using matplotlib instead:
import matplotlib.pyplot as plt
import imageio

img = imageio.imread("<path_to_img.tiff>")
plt.imshow(img)
plt.show()

Hopefully that will show that the correct data has been saved.  Otherwise, shoot me a reply and I'll take another look.

Thanks! I just solved it by your solution


TeamViewer

Find the linux version: https://www.teamviewer.com/en/
An old tutorial: http://www.tonisoto.com/2013/07/launching-teamviewer-remotely-throught-ssh/

@finnlennartsson Installed successfully and can be launched by command teamviewer within a remote desktop.

Multiple concurrent users in one target machine: https://community.teamviewer.com/English/discussion/25871/multiple-concurrent-users-in-one-target-machine


py-ews - Added check that md4 is enabled to verify.py

Status
Ready
Related Content Pull Request
Related PR: 11572.
Description
As we did in py3ews, I added a check to the verify.py of the py-ews image to make sure that the MD4 is enabled.
That way we should catch problems in future upgrades.

Closing the PR.
Since md4 isn't in the list of hashlib.algorithms_available this shouldn't be an issue.


Remove agent persistent storage

We don't need the persistent storage in the agents since we are using the postgresql plugin, we can delete the persistent storage claims and free up some space.

remove persistent storage claims in OpenShift
remove persistent storage from templates
update OpenShift manage script and remove step to delete wallets on persistent storage

Reviewed and merged configuration template changes.
@esune, you'll also want to remove the unused parameters from teh *.param files.

Reviewed OpenShift environments and confirmed the PVCs have been released.

*.param files were updated in https://github.com/bcgov/dFlow/pull/227


Bugfix/rhornung67/fix sycl

This PR resolves issues related to RAJAPerf for running GitLab CI SYCL job on corona.

@adrienbernede This has been tested in RAJA and RAJAPerf and fixes issues in both with running SYCL CI job on corona in GitLab.
After this is merged, this will need to be pulled in to RAJA develop.
Then, a RAJAPerf PR with RAJA updated to develop + a change to RAJAPerf build_and_test.sh script will be needed to close the loop.

ENABLE_SYCL is not needed and there are no changes to the RAJAPerf source or build system. The issue was the incorrect conditional in the RAJAPerf build_and_test.sh script. There is a RAJAPerf PR that has all GitLab CI working with this branch. When this is merged, other PRs will pull this into RAJA and RAJAPerf.


compile error with generic function: cannot find type parameter in scope

Aim
fn hello<N>() {
    for i in 0..N {}
}

Expected Behavior
The code should compile successfully since the generic type parameter N is correctly defined in the function signature.
Bug
When trying to compile a function with a generic type parameter, the compiler reports that it cannot find the type parameter N within the scope, even though it should be correctly defined.
error: cannot find `N` in this scope
   ┌─ array.nr:97:17
   │
97 │     for i in 0..N {}
   │                 - not found in this scope
   │

To Reproduce

Installation Method
Compiled from source
Nargo Version
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

I think the issue is in how we identify/infer numeric type variables currently. They're only added to scope if after checking the function signature we can actually confirm they're numeric by finding them in an array length somewhere.

@f01dab1e is this still relevant?

Duplicate of https://github.com/noir-lang/noir/issues/4633


[1.12.2] crash on 5.2.61

Issue Description:
crashing on startup with ender io 5.2.61
the crash screen claims its ender io powertools module.
this issue doesnt happen with ender io 5.2.59 which is what i upgraded from due to the gas conduits addon requiring 5.2.60
update: tried 5.2.60 and the same crash happens, downgrading back to 5.2.59 causes no crash
What happens:
gets to about 5/7 in initialization then crashes

Affected Versions (Do not use "latest"):

EnderIO: 5.2.61
EnderCore: 0.5.76
Minecraft: 1.12.2
Forge: 2847
SpongeForge? no
Optifine? yes
Single Player and/or Server? crashes on both client and server with the same issue

Your most recent log file where the issue was present:
crash report: https://pastebin.com/qzz5t4ge

Nope, that's something new.
Some mod called "sledgehammer" is trying to modify our code and crashes.
Still not our problem.

so i need to report it to them, ok.


Update README.md

Added codecov badge

Codecov Report
Patch and project coverage have no change.

Comparison is base (8e7e533) 97.27% compared to head (4780d9f) 97.27%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   97.27%   97.27%
=======================================
  Files           7        7
  Lines         220      220
=======================================
  Hits          214      214
  Misses          6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Adding customCancelButton and customConfirmButton types

Thank you guys for an amazing lib!!
PS: I am re-creating this PR because the I did not have the right email set in my commit, so the commit signature was unverified. Sorry for that!
Overview
The library is missing the CancelButton and ConfirmButton components for the iOS only properties customCancelButtonIOS and customConfirmButtonIOS. I also found a good idea to export their styles, just in case someone (myself included) wants to do some slight changes.
All I mentioned works fine, but whenever you try to import any of these two components or their styles you get some red on your code editor. That's because the declaration file is missing these elements. So all I did was I exported from  ìndex.d.ts the CancelButton and ConfirmButton components, told the declaration file the customConfirmButtonIOS and customCancelButtonIOS properties were a React.FunctionComponenet<props> with some extra props. I also exported the style types of both components.
Test Plan
There is not much to do actually, all I did was passing the buttonTextColorIOS property to the custom button I was using. For example:
import ModalDateTimePicker, { CancelButton };

<ModalDateTimePicker
    // ... <normally used props>
    customCancelButtonIOS={(props) => <CancelButton {...props} buttonTextColorIOS="red"  />}
/>

If you want to change the button styles, you might want to do something like that:
import ModalDateTimePicker, { CancelButton, CancelButtonStylePropTypes, cancelButtonStyles };

const customStyles: CancelButtonStylePropTypes = {
    ...cancelButtonStyles,
    text: {
        ...cancelButtonStyles.text,
        color: 'red',
    },
};

<ModalDateTimePicker
    // ... <normally used props>
    customCancelButtonIOS={(props) => <CancelButton {...props} style={customStyles} />}
/>

Changes are looking good to me now! Thanks so much for your PR and the discussion 🙌 I'm planning to merge it as a major change. Could you double-check that it works correctly for you one last time before I release it? Thanks!

Yeah, sure. I created this repository to test some cases.
Here are the tested cases:

Change ConfirmButton and CancelButton's styles (using the originally exported styles)
a. Changing buttonTextColorIOS
b. Changing onPress default behavior
Using a different component

Feel free to clone it and test as many cases as you feel necessary.
To do the tests I pulled the package from my on PR as you can check here.
P.S.: I didn't use the latest version of React Native because I reused most of a project I was working on.


How to set the default to 3d mode

I tried to follow before closed issue, but I didn't fix this problem yet, please help me.
Thanks.

src/demo/ui/UiOpenMenu.js@finalizeSuccessLoadedVolume:
change store.dispatch({ type: StoreActionType.SET_MODE_VIEW, modeView: ModeView.VIEW_2D }); to store.dispatch({ type: StoreActionType.SET_MODE_VIEW, modeView: ModeView.VIEW_3D });

@dream811 thank you for opening this issue.
That would be good option to have it configurable, we have created a ticket for this feature.