create APIMethod for obtaining the view port

This is needed by GeoExt to append the map panel items there so that they don't interfere with the zoombox for example
This has been discussed with ahocevar, but it would be good to get someone else's opinion (e.g. @elemoine)
TIA.
See also https://github.com/geoext/geoext/pull/24

looks good to me


Comments in JSON is fidgety

Lots of problems handling comment markers in json value fields.

Possibly related to #3  .

Closed with version 1.5


Use centimeters instead of inches

Inches are used only in the US. Since this document is not US-specific, I think it makes sense to use centimeters instead.

Thanks @svick. @BillWagner what do you think? We don't really have guidelines in terms of which measurement units to choose.


Create table should use strict existence check to see if table exists

Create table should compare the schema to ensure that the table that exists has the same schema.

This was closed by #74.


groupadd: group 'sfm' already exists

I'm getting multiple errors like this in the log:
groupadd: group 'sfm' already exists
...and the container seems to get stuck in restarting mode. Tested on a clean install of SFM 1.5.1.

Downgrading to 1.5.0 works.

Hey @peterk - Are you using internal or external volumes?  (The VOLUME CONFIGURATION section of your .env file.)

Also, what do you see in the logs for the heritrix container?  Sometimes startup problems with the heritrix container can cause problems for the web harvester container.
If the problem appears to be with the heritrix container, try removing it (docker-compose rm -vf webharvester heritrix) and then bringing it back up (docker-compose up -d).
I haven't been able to replicate the problem, which is typical for issues I've been having with Heritrix.  I'm going to update the known issues section of the docs.

External volumes. Writing to the external volume works fine from other harvesters.

Logs for heritrix shows this:
2017-03-08 10:07:50.968 WARNING thread-57 org.archive.modules.extractor.ExtractorJS.considerString() problem unescaping some javascript org.apache.commons.lang.exception.NestableRuntimeException: Unable to parse unicode value: tils 	at org.apache.commons.lang.StringEscapeUtils.unescapeJava(StringEscapeUtils.java:337) 	at org.apache.commons.lang.StringEscapeUtils.unescapeJava(StringEscapeUtils.java:287) 	at org.apache.commons.lang.StringEscapeUtils.unescapeJavaScript(StringEscapeUtils.java:406) 	at org.archive.modules.extractor.ExtractorJS.considerString(ExtractorJS.java:170) 	at org.archive.modules.extractor.ExtractorJS.considerStrings(ExtractorJS.java:155) 	at org.archive.modules.extractor.ExtractorJS.considerStrings(ExtractorJS.java:134) 	at org.archive.modules.extractor.ExtractorJS.innerExtract(ExtractorJS.java:121) 	at org.archive.modules.extractor.ContentExtractor.extract(ContentExtractor.java:37) 	at org.archive.modules.extractor.Extractor.innerProcess(Extractor.java:102) 	at org.archive.modules.Processor.innerProcessResult(Processor.java:175) 	at org.archive.modules.Processor.process(Processor.java:142) 	at org.archive.modules.ProcessorChain.process(ProcessorChain.java:131) 	at org.archive.crawler.framework.ToeThread.run(ToeThread.java:148) Caused by: java.lang.NumberFormatException: For input string: "tils" 	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65) 	at java.lang.Integer.parseInt(Integer.java:492) 	at org.apache.commons.lang.StringEscapeUtils.unescapeJava(StringEscapeUtils.java:331) 	... 12 more

And therein lies the problem with Heritrix and the motivation for the new warning in the docs about collecting web resources.
There are no changes in Heritrix between 1.5.1 and 1.5.0.  I suspect the reason it worked in one and not the other is because the Heritrix container was killed and recreated.  When I have web harvesting problems, this is the process I usually follow.

I'm seeing this after a reboot of SFM server:
heritrix_1                | Oracle Corporation OpenJDK Runtime Environment 1.7.0_95-b00
heritrix_1                | Using ad-hoc HTTPS certificate with fingerprint...
heritrix_1                | SHA1:AC:10:4E:0B:50:4D:C2:8A:3F:D9:4A:25:34:D4:76:09:4A:7C:A6:D2
heritrix_1                | Verify in browser before accepting exception.
heritrix_1                | SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
heritrix_1                | SLF4J: Defaulting to no-operation (NOP) logger implementation
heritrix_1                | SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
heritrix_1                | engine listening at port 8443
heritrix_1                | operator login set per command-line
heritrix_1                | NOTE: We recommend a longer, stronger password, especially if your web
heritrix_1                | interface will be internet-accessible.
heritrix_1                | Heritrix version: 3.3.0-LBS-2016-02
heritrix_1                | 2017-03-10 11:48:08.677 INFO thread-12 org.archive.crawler.framework.Engine.addJobDirectory() added crawl job: sfm
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists
heritrix_1                | groupadd: group 'sfm' already exists

Moving ticket to https://github.com/gwu-libraries/sfm-ui/issues/739. (We use sfm-ui for tickets on all of the SFM repos.)

Great, thank you! (one day, when I have a lot of spare time, which is probably never I'd like to write a better web harvester)...


Cannot read property 'interval' of undefined

[ x ] I confirm that this is an issue rather than a question.

Bug report
Steps to reproduce

After building the website it appears, that it breaks with pagination.
Getting error:
wait Rendering static HTML...
[Vue warn]: Error in render: "TypeError: Cannot read property 'interval' of undefined"

found in

---> <BaseListLayout>
       <Anonymous>
         <GlobalLayout>
           <Anonymous>
             <Root>
error Error rendering /anime/1/: false
undefined

For the first article.
More here: https://uwaylu.github.io/blog/2020/02/24/hello-blog/
What is expected?
yarn build without issues
What is actually happening?
Project cannot be built.
Other relevant information
One workaround can be found here https://www.gitmemory.com/issue/ulivz/vuepress-plugin-blog/1/504744139
But unfortunately not solving the issue.

Output of npx vuepress info in my VuePress project:

  Binaries:
    Node: 11.13.0 - ~/.nvm/versions/node/v11.13.0/bin/node
    Yarn: 1.22.4 - ~/.yarn/bin/yarn
    npm: 6.14.6 - ~/.nvm/versions/node/v11.13.0/bin/npm
  Browsers:
    Chrome: Not Found
    Edge: Not Found
    Firefox: 81.0
    Safari: Not Found
  npmPackages:
    @vuepress/core:  1.6.0
    @vuepress/theme-default:  1.6.0
    vuepress: ^1.6.0 => 1.6.0
  npmGlobalPackages:
    vuepress: Not Found

Thanks for the report.

One workaround can be found here https://www.gitmemory.com/issue/ulivz/vuepress-plugin-blog/1/504744139
But unfortunately not solving the issue.

The issue you found has already been solved.

More here: https://uwaylu.github.io/blog/2020/02/24/hello-blog/

It would be easier if you provide a repo to reproduce. Since you're facing the problem during building, but a site is what you got when you built with broken.

Maybe GA will have some useful information regards to solving the bug.
Please, let me know if this helps.
https://github.com/core-coin/cip/runs/1261249311

Fixed by https://github.com/vuepress/vuepress-plugin-blog/commit/4abe10c8faf519e9aa682aae00ea80e1b4ec4185.


Search relays gets reset when new search is executed

Hi!
If I go to https://nostrudel.ninja/#/search and click the "x Relays" button in the top right corner, add a relay to the search and then execute the search, the relay I've added is automatically deselected and only the 3 default select relays are selected.

Fixed with<PHONE_NUMBER>af7bfd61031d3e9dbffb5ec455cdad in the alpha version

Released


Can't build with ARM64 version of Visual Studio

Describe the bug
When using the latest preview of Visual Studio 2022 for ARM64, compiling WinUI from commandline causes the following error:
(MarkupCompilePass2 target) ->
  C:\Users\mn\.nuget\packages\microsoft.windowsappsdk\1.1.2\buildTransitive\..\tools\net472\..\x64\genxbf.dll : X
amlCompiler error WMC0621: Cannot resolve 'GenXbf.dll' under path 'C:\Users\mn\.nuget\packages\microsoft.windowsa
ppsdk\1.1.2\buildTransitive\..\tools\net472\..\x64\genxbf.dll'.  Please ensure that the Nuget was properly restored and
 that the path is available on disk.

There is an ARM64 version of genxbf.dll in \tools\arm64\ and if I take that and copy it into the x64 folder, the build does work. Looks like it is mostly an issue of not loading the correct DLL in the xaml compiler.
Steps to reproduce the bug
Install VS2022 Preview onto a PC with Windows 11 ARM64.
Open the VS Developer Prompt and compile a WinUI 3 project with msbuild.
Expected behavior
Build compiles fine on ARM64
Screenshots
No response
NuGet package version
1.1.2
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 21H2 (22000)
IDE
Visual Studio 2022-preview
Additional context
Same thing happens from within VS:

This looks like a duplicate of https://github.com/microsoft/microsoft-ui-xaml/issues/7140, and we're tracking it internally with http://task.ms/39763998.

Duplicate of https://github.com/microsoft/microsoft-ui-xaml/issues/7140


refactor DeleteChatModalAction

In commit 2f05f86, I changed the selector path to the corresponding unique path to avoid making the code too complicated.


Can't update Threadly with SPM

Using Threadly through RandomKit. SPM freezes while doing swift package update.
Swift 4, SPM PackageDescription v4.

Can I see how your Package.swift looks like?

Hey,
it was an issue with the Swift Package Manager, my Package.swift made the graph unresolvable. Already confirmed by the SPM team: https://bugs.swift.org/browse/SR-5956


Sir 311 filter by assigned

Codecov Report

Merging #67 (72d02ad) into main (6529f74) will increase coverage by 1.67%.
The diff coverage is 63.71%.

@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
+ Coverage   49.90%   51.57%   +1.67%
==========================================
  Files          14       15       +1
  Lines         509      539      +30
==========================================
+ Hits          254      278      +24
- Misses        200      210      +10
+ Partials       55       51       -4

Impacted Files
Coverage Δ

internal/sirius/get_applied_filters.go
0.00% <0.00%> (ø)

internal/sirius/get_current_user_details.go
71.42% <ø> (ø)

internal/sirius/get_task_details.go
63.46% <0.00%> (ø)

internal/sirius/get_assignees_for_filter.go
54.54% <54.54%> (ø)

internal/sirius/get_task_list.go
67.08% <63.33%> (ø)

internal/server/workflow.go
52.04% <92.30%> (+2.04%)
:arrow_up:

internal/sirius/get_task_types.go
67.74% <100.00%> (ø)

internal/sirius/get_teams_for_selection.go
59.52% <100.00%> (ø)

Continue to review full report at Codecov.

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


Implement posterization normalizer

Pull Request
Brief description.
Added a Normalizer to posterize the input function, i.e. separate it into "steps" with discrete values.

What Issues Does This Fix?
Closes #204

Licensing

[X] I am the original author of this code, and I am willing to release it
under GPLv3.
[ ] I am not the original author of this code, but it is in public domain or
released under GPLv3 or a
compatible license.

Goal of the PR

[X] Add a normalizer to perform posterization with a configurable number of steps.

Affects of the PR

Types of changes

[ ] Bug Fix
[ ] Build system
[ ] Documentation
[X] New Feature
[ ] Performance
[ ] Refactoring
[ ] Repository
[ ] Revert
[ ] Style
[ ] Tests
[ ] Translation

Compatiblity

[ ] Breaking
change
[X] Non-Breaking change.

Contribution Guidelines.

[X] I have read
the CONTRIBUTING.md
document in the root of the git repository.
[X] My code follows the code style for this
project.

Documentation

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Testing

[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

This should throw an exception of some sort when a non-positive value is provided for steps.
Further, this will fail if 1 is provided as steps.


I can't read a FLV video (codec FLV1) with MediaElement v 3.2.4

Hello
With MediaElement v 3.2.4, i can read a FLV video (codec H.264) but i can't read a FLV video (codec FLV1).
If i use mediaelementplayer.js and not mediaelement-and-player.js, i can read a FLV video (codec FLV1).
Thanks

The problem is that mediaelementplayer.js is not trying to use native FLV support: is using Flash only. So in your case you will need to do something like this:
$('video').mediaelementplayer({
      renderers: ['flash_video'],
      pluginPath: '/path/to/swf files/',
      // other configuration you might need
})

Thank you for this solution.
It works !


SideBar box Sort order - new item

please add "my assignments" to the sort order.
It is available when a user joins a classroom in school i.e. "shares progress"

Thanks for the report, can you tell me where the box goes with the extension disabled?
This screenshot I found on Google images shows it just below the XP box, but I think it is probably fairly old

I have added the handling for box this in 0a0497e. Currently I have placed it in the unsorted order just before the Progress Quiz box. Let me know if this is incorrect and I will move it to the proper place.

FYI with disabled extension it shows at the top... above leagues

Fixed in v2.0.18


Any scheduled plan to update the codes for Python 3?

Dear developers and authors,
First I really have to thank all of you for writing such a nice book for introducing the fundamental codes to use machine learning for predictive analysis. I've benefited a lot from following your recipes!
However, it seems that most codes were written either in Python 2.7 (e.g. the existence of the print "xxx" syntax) or using older machine learning packages. Things have changed quite a lot in recent years, and as I proceed to later chapters in the book, a lot of the codes are either not supported any more, or there were some major updates in the fundamental syntax in Python 3 (esp. data types often became mis-matched in longer codes consisting of many functions). For example, in Chapter 9, many syntaxes involved in tree, GridSearchCV, best_estimator_ etc. seem to have updates in recent versions, and thus now it took me some time to google how to update the sample codes in the book in order to generate the same plot or analysis.
Therefore, may I know if the authors have any plan or intention to update the codes for a future version of the book? Thank you!
Best,
Jason.

Hey @jpiggod ,
I wanted to inform you that, we won't be able to make any code changes at this stage.
And with due respect to your query, regarding the new update of the code for a future version of book.
I am not in the best position to respond to this.
But, we highly appreciate the time you took in highlighting this.
Thanks a lot for your feedback.
Take  Care!
Thanks!


enhance(oidc): provide support for custom issuer

This change will allow platform admins to publish the OpenID configuration (and in turn the JWKS) to custom URLs.
Of course, in order to have accurate information, these custom URLs will need to source their data from the standard Vela routes. But this allows selective exposure of OpenID data to external services such as GCP and AWS.

sweet, i think we also need it here?

also should we add url validation for oidc-issuer?
like we do for server-addr
addr, err := url.Parse(c.String("server-addr"))
if err != nil {
	return err
}


fix(swift5): upgrade deployment target to fix cocoapods deployment

Merged in https://github.com/apivideo/api.video-api-client-generator/pull/382 instead


Update FairOSS listing on oss.fund

I made some changes to the FairOSS listing.  Thanks again for thinking of us!

Thanks for the update. I'll apply your changes and fix a bit the categories/tags to match existing


[FLINK-19464][runtime] Rename CheckpointStorage interface to Checkpoi…

…ntStorageAccess
What is the purpose of the change
This is a trivial renaming of CheckpointStorage to CheckpointStorageAccess so we can use the former name as a public interface. This PR is just a result of IntelliJ rename. I decided to make it its own PR since the diff is trivial but rather large. This is prereq work for FLIP-142
Brief change log

Rename interface

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.

+1 (belated).
Just a reminder that it includes user-detective changes (such as the changes in StateBackend interface) thus please remember to add explicit ReleaseNote on JIRA @sjwiesman . Thanks.


AWS ECS: Firefox can't establish a connection to the server" Error in Email Reports

Bug description
I am running Superset within an ECS Cluster, which I'm trying to set up email reporting within Superset, I'm currently using the Docker 3.1.0 non-dev container.
#Installing Webdriver for Reporting
RUN apt-get update && \
    apt-get install --no-install-recommends -y firefox-esr

ENV GECKODRIVER_VERSION=0.34.0
RUN wget -q https://github.com/mozilla/geckodriver/releases/download/v${GECKODRIVER_VERSION}/geckodriver-v${GECKODRIVER_VERSION}-linux64.tar.gz && \
    tar -x geckodriver -zf geckodriver-v${GECKODRIVER_VERSION}-linux64.tar.gz -O > /usr/bin/geckodriver && \
    chmod 755 /usr/bin/geckodriver && \
    rm geckodriver-v${GECKODRIVER_VERSION}-linux64.tar.gz

My SMTP Configuration is the following;
WEBDRIVER_BASEURL = 'http://<IP_ADDRESS>:8088'
WEBDRIVER_BASEURL_USER_FRIENDLY = WEBDRIVER_BASEURL

#Slack Config
SLACK_API_TOKEN =  get_env_variable("SLACK_TOKEN")

#SMTP Config
SMTP_HOST = get_env_variable("SMTP_HOST")
SMTP_PORT = get_env_variable("SMTP_PORT")
SMTP_STARTTLS = True
SMTP_SSL_SERVER_AUTH = True
SMTP_SSL = True
SMTP_USER = get_env_variable("SMTP_USERNAME")
SMTP_PASSWORD = get_env_variable("SMTP_PASSWORD")
SMTP_MAIL_FROM  = get_env_variable("SMTP_MAILFROM")
EMAIL_REPORTS_SUBJECT_PREFIX = "[Superset]"

How to reproduce the bug
When I try and setup a report I'm getting the following
Failed taking a screenshot Message: Reached error page: about:neterror?e=connectionFailure&u=http%3A//<IP_ADDRESS>%3A8088/login/&c=UTF-8&d=Firefox%20can%E2%80%99t%20establish%20a%20connection%20to%20the%20server%20at%<IP_ADDRESS>%3A8088.
 Failed generating csv <urlopen error [Errno 111] Connection refused> Traceback (most recent call last): File "/usr/local/lib/python3.9/urllib/request.py", line 1346, in do_open h.request(req.get_method(), req.selector, req.data, headers, File "/usr/local/lib/python3.9/http/client.py", line 1285, in request self._send_request(method, url, body, headers, encode_chunked) File "/usr/local/lib/python3.9/http/client.py", line 1331, in _send_request self.endheaders(body, encode_chunked=encode_chunked) File "/usr/local/lib/python3.9/http/client.py", line 1280, in endheaders self._send_output(message_body, encode_chunked=encode_chunked) File "/usr/local/lib/python3.9/http/client.py", line 1040, in _send_output self.send(msg) File "/usr/local/lib/python3.9/http/client.py", line 980, in send self.connect() File "/usr/local/lib/python3.9/http/client.py", line 946, in connect self.sock = self._create_connection( File "/usr/local/lib/python3.9/socket.py", line 844, in create_connection raise err File "/usr/local/lib/python3.9/socket.py", line 832, in create_connection sock.connect(sa) ConnectionRefusedError: [Errno 111] Connection refused During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/app/superset/commands/report/execute.py", line 258, in _get_csv_data csv_data = get_chart_csv_data(chart_url=url, auth_cookies=auth_cookies) File "/app/superset/utils/csv.py", line 92, in get_chart_csv_data response = opener.open(chart_url) File "/usr/local/lib/python3.9/urllib/request.py", line 517, in open response = self._open(req, data) File "/usr/local/lib/python3.9/urllib/request.py", line 534, in _open result = self._call_chain(self.handle_open, protocol, protocol + File "/usr/local/lib/python3.9/urllib/request.py", line 494, in _call_chain result = func(*args) File "/usr/local/lib/python3.9/urllib/request.py", line 1375, in http_open return self.do_open(http.client.HTTPConnection, req) File "/usr/local/lib/python3.9/urllib/request.py", line 1349, in do_open raise URLError(err) urllib.error.URLError: <urlopen error [Errno 111] Connection refused> The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/app/superset/tasks/scheduler.py", line 94, in execute AsyncExecuteReportScheduleCommand( File "/app/superset/commands/report/execute.py", line 729, in run raise ex File "/app/superset/commands/report/execute.py", line 725, in run ReportScheduleStateMachine( File "/app/superset/commands/report/execute.py", line 684, in run state_cls( File "/app/superset/commands/report/execute.py", line 586, in next raise first_ex File "/app/superset/commands/report/execute.py", line 555, in next self.send() File "/app/superset/commands/report/execute.py", line 452, in send notification_content = self._get_notification_content() File "/app/superset/commands/report/execute.py", line 364, in _get_notification_content csv_data = self._get_csv_data() File "/app/superset/commands/report/execute.py", line 262, in _get_csv_data raise ReportScheduleCsvFailedError( superset.commands.report.exceptions.ReportScheduleCsvFailedError: Failed generating csv <urlopen error [Errno 111] Connection refused>
Screenshots/recordings
No response
Superset version
3.1.0
Python version
3.9
Node version
Not applicable
Browser
Firefox
Additional context
I've tried using
<IP_ADDRESS>, localhost, <IP_ADDRESS> and the Public Hostname for the Superset running within ECS all have the same issue as above
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Hello, it appears this is a question of how to configure Reports on AWS ECS, rather than a bug in Superset's codebase.  I'm going to convert it to a discussion to keep the bug backlog manageable, then I'll respond there.


Multi-Omics Factor Analysis of all data modalities

https://github.com/scverse/muon?tab=readme-ov-file#multimodal-omics-analysis


Added Food website

Hello Sir,
I have added my Food website
Please take a review of my PR & please merge it under OSEN

@Shu12388y Please take a review of my PR & please merge it under OSEN

move the author below the screenshot

@Shu12388y I have made the changes. Please merge my PR.

@Shu12388y Can you please add SIMPLE/MEDIUM/HARD tag with the PR , otherwise the PR will not be counted in OSEN


Fix frontend memory leak

Description

This PR fixes the current frontend memory leak by addressing an issue with the initialization of the ui store. For increased ease of testing, I created an object outside of the store to capture the default state (so it could be imported into the test spec) and then spread the properties of that object into the store's state object. Apparently, that is very bad. I'd like to figure out exactly why, so I'll try and look at Pinia's internals tomorrow. It might be very obvious to someone else.
Testing Instructions

This should be fairly straightforward to prove it works but there are many steps. You also might need the change I made locally to utilities/load_testing/Dockerfile before running these steps, if you're not able to pull the load testing Docker image.

With the main branch checked out, build the Openverse frontend with: ov just frontend/run build.
Run the frontend with ov node --inspect frontend/.output/server/index.mjs
In a chromium-based browser, visit chrome://inspect. Choose the app from the "target" list to open a debugger (by clicking the link "inspect" below the app name, probably frontend/.output/server/index.mjs.
Press the record icon in the top left of the "profile" tab to take an initial heap snapshot
Back in your terminal cd utilities/load_testing and run (replacing the IP with the correct host for your environment, probably host.docker.internal on MacOS) FRONTEND_URL=http://<IP_ADDRESS>:3000/ just k6-frontend en.
Press the record icon in the top left to take a new heap snapshot.
Observe that the 2nd heap is much, much larger, in the hundred of MBs.
Checkout this PR and repeat. The 2nd snapshot should only be a little bit larger than the first now :smile: (we must have some smaller but much less consequential leaks still)

Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (ov just catalog/generate-docs for catalog
PRs) or the media properties generator (ov just catalog/generate-docs media-props
for the catalog or ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I've been playing with this locally a bunch. I don't know why this is the case, but removing the object spread fixes the leak.
diff --git a/frontend/src/stores/ui.ts b/frontend/src/stores/ui.ts
index 4144d799d..949b75e8a 100644
--- a/frontend/src/stores/ui.ts
+++ b/frontend/src/stores/ui.ts
@@ -69,19 +69,21 @@ export interface UiState {

 export const breakpoints = Object.keys(ALL_SCREEN_SIZES)

+export const defaultUiState: UiState = Object.freeze({
+  instructionsSnackbarState: "not_shown",
+  innerFilterVisible: false,
+  isFilterDismissed: false,
+  isDesktopLayout: false,
+  breakpoint: "sm",
+  dismissedBanners: [],
+  shouldBlurSensitive: true,
+  revealedSensitiveResults: [],
+  headerHeight: 80,
+  colorMode: "system",
+})
+
 export const useUiStore = defineStore("ui", {
-  state: (): UiState => ({
-    instructionsSnackbarState: "not_shown",
-    innerFilterVisible: false,
-    isFilterDismissed: false,
-    isDesktopLayout: false,
-    breakpoint: "sm",
-    dismissedBanners: [],
-    shouldBlurSensitive: true,
-    revealedSensitiveResults: [],
-    headerHeight: 80,
-    colorMode: "system",
-  }),
+  state: (): UiState => defaultUiState,

   getters: {
     cookieState(state): OpenverseCookieState["ui"] {

I spent a long time trying to find a reason for this, digging through heap snapshots, allocation timelines... I didn't come up with anything. I looked through Pinia's code a lot, in particular the parts that use state(), and didn't see anything suspicious.
The only thing that jumped out to me when looking at the snapshots on the broken code, is that there are heaps of objects with large shallow size referencing the i18n strings, and that are allocated over and over again and never freed. These objects have an allocation size of 8160. There are also many i18n objects as well, with allocation size 3424. After that, there are many Pinia objects at allocation size 3160, and those appear to just be pinia store references.
I don't see any of that with the fix applied. There are still some seemingly redundant i18n objects allocated, but I don't see those Pinia objects.
I can't figure out more than this from the snapshots and timelines. We can try looking at it together sometime, that helped when @obulat and I were individually stuck with debugging the first big frontend memory leak.


PLE0237 false positive for slots literal concatenation

I have a class, whose slots are formed by concatenating other literal tuples via RUF005 style. Below B is reported to violate PLE0237, which is a false positive, I also tried a few variants of B:

C: If B is a subclass of any other class, PLE0237 seems to work fine
D: If B slots has no other literal items, PLE0237 seems to work fine
E: If B slots uses tuple concatenation, which violates RUF005, but PLE0237 seems to work fine

class A:
    pass

class B:
    names = ("x",)
    __slots__ = (*names, "a")

    def __init__(self) -> None:
        self.x = 1  # PLE0237: Attribute `x` is not defined in class's `__slots__`

class C(A):
    names = ("x",)
    __slots__ = (*names, "a")

    def __init__(self) -> None:
        self.x = 1

class D:
    names = ("x",)
    __slots__ = (*names,)

    def __init__(self) -> None:
        self.x = 1

class E:
    names = ("x",)
    __slots__ = names + ("a",)  # RUF005: Consider `(*names, "a")` instead of concatenation

    def __init__(self) -> None:
        self.x = 1

Thank you for opening this issue! I think we should ignore raising PLE0237 if __slots__ contain an unpacked argument.


Test Sonar tweaks

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

jenkins please retest this

The Linux check is successful.


[Bug] When using Fluent API and setting the key on the column, that column is displayed twice in the table

Bug report
When using Fluent API and setting the key on the column, that column is displayed twice in the table.
What I did
    protected function setupListOperation()
    {
        CRUD::column('name')->key('name_1');

What I expected to happen
I expect to see one column named 'name' and key 'name_1'
What happened
Two columns are added, one with key 'name', and the other with key 'name_1'.
What I've already tried to fix it
Switching to array syntax to add columns fixes the issue. Maybe that should be documented in the Fluid API docs?
Is it a bug in the latest version of Backpack?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.7 (cli) (built: Jun 23 2022 07:15:05) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.7, Copyright (c) Zend Technologies
with Xdebug v3.1.5, Copyright (c) 2002-2022, by Derick Rethans
LARAVEL VERSION:
v9.14.1@2ca86f96118635a79ee3389013ccab110ae14bea
BACKPACK VERSION:
5.0.19@a71715d51a0bd2827d535deb407761ee9252d04

4

Hey @viperet it's indeed an unintended behaviour.
I am pretty sure I know what's happening I will implement a solution to fix it, but my real question is:

What's your usecase that you need to change column key ?

Thanks for the report 🙏

@pxpm I don't have a real use case for changing key, found this bug when working on server-side export operation and did tests to make sure it works with all edge-case configurations.

Thanks @viperet .
Anyway it's a bug that must be fixed.
And ... You will be happy to know (I hope) that the next version of Backpack (5.2 probably out next week) will have an ExportOperation for server-side export 👍
Cheers

Thanks for reporting this @viperet 🙏
I've just merged Pedro's PR, it'll be available with a composer update starting Monday, when we tag it along with other bug fixes.
Thanks again, keep 'em coming!

I think the easiest solution is to only allow to set key in the constructor as a second optional parameter:
CRUD::column('name', 'key');

@viperet the PR is merged but not tagged yet, I think that is the issue here I tried now with:
CRUD::column('owner')->key('owner');
CRUD::column('owner')->key('owner_2');
CRUD::column('owner')->key('owner_3');

and it worked I guess:

Yup - @viperet  to actually get it before we tag it on Monday, you can do composer require backpack/crud:"v5-dev as 5.1.5" - that will pull the v5 branch directly. You should then undo this, after Monday.

@tabacitu before that for my test I just manually applier PR to sources in /vendor/backpack/crud, after a message from @pxpm thought "oh, maybe there are other changes that I don't have that make a difference", so I installed latest v5 branch from git - the same result.
Today I installed using your command (had to change it to composer require backpack/crud:"v5.x-dev as 5.1.5").
Still can't get the same results as in @pxpm screenshot.
Will probably wait till the official release and then will write some unit tests.

Hello @viperet
I revisited this today and could reproduce your issue, my solution was not final and was not working properly.
I've submitted https://github.com/Laravel-Backpack/CRUD/pull/4548 with a fix that I think will solve this problem for good.
Would you mind give it a try and let us know your results ?
Thanks and sorry for the bad experience here 😿


Plugin only work for english client!

Hello.
Im not sure how do adress this issue, but in short words:
Every time if i open this plugin window. its empty. Has zero data in it.
Doesnt matter if i search for an item or not.
After switch my Game to English it works. Is there a way how i can use it in an another game language? I dont want to be forced to use it in english..
Best regards and thanks for help.

Hello!
I think this is really an internationalization issue.
I'm sad to know that you came across this problem.
I'll try my best to solve this problem, and let you know when it's done.
Sorry for the inconvenience and thanks for trying to use the plugin.
Regards.

Hello,
Sorry for taking so long to solve this issue.
The new version of MLH will solve the problem you're having. Now it should work regardless of the language the game is in.
As soon as the update is live on Dalamud you'll be able to use the plugin.
Thanks for you patience.
Regards.

Hello,
It seems not completely solve.
There is a list, but plugin don't find information about loot. Probably because the source is only in English and can't find item's name in other language.
Thanks for your work.

Ah, that is indeed correct.
The wiki is only in english. Since the plugin grabs the info from the wiki and the name of the item is not in english, it cannot retrieve the information.
I'd have to add a separate source for this case, where the primary source cannot provide the information.

Hello,
I have submitted a new version of MLH for Dalamud which I hope solves this issue where that information in not retrieved.
Now it should be able to get the information for the items you're searching even if your client is not in english.
I ask that when the update is available you report back if the issue was solved.
Regards.

Hello.
Sorry, it take a long time for me to reply, work issues!
It looks like it work out for me.

The Information if there are something will shown (mostly in english but better then nothing)
So i can work niw. but maybe Selene have to say something about it from her side.
best regards.

Hello,
Update resolve the issue. Informations stay in English but it's not a big problem for me.
Thanks for your work.
Regards

Great to know that it's now working.
The information unfortunately is in english since the plugin gets it from the Wiki.
I'll make a plan to in the future be able to have an internal source, or a way that the information can be internationalized.
Best regards and thanks for using the plugin!


Page Component - Question on using Allowed Template

The current version Email Page component has a metadata property that allows us to specify "Allowed Templates".  I know that the WCM Page component also has a field that allows you to specify the /conf folder to associate the allowed Editable templates.
Is there a plan to add that option to the Email Page component? or should we continue to use "Allowed Templates" fields to associates editable templates.

For now, we will make use of "Allowed Templates" within the Email Core Components


koord-scheduler: support reservation affinity

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/hold

/hold cancel
TBR. PTAL @hormes @FillZpp


fix: raise incorrect post event data

Description of changes
Using an AmplifyPostPushEventData instance when raised a PostInit event, and an AmplifyPostPushEventData instance when raised a InitEvent event. These use the wrong event data for each other.
Fixed to use the correct event data.

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hey @fossamagna - Thanks for the contribution! I see the function names were changed to match, but what is the context here? Was it breaking anything and was this intentional? Are there any tests we can add to ensure this works? Going to run the CLI test suite now.

@danielleadams Thank you for your review.
I think that this change is not breaking, because AmplifyPostPushEventData and AmplifyPostPushEventData  is not same class, but both class is extended same parent class and both class no have property.
I will add tests to ensure this work.

Codecov Report

Merging #10954 (ac62b06) into dev (3c39eba) will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev   #10954      +/-   ##
==========================================
+ Coverage   47.82%   48.12%   +0.29%
==========================================
  Files         669      669
  Lines       32678    32678
  Branches     6608     6608
==========================================
+ Hits        15627    15725      +98
+ Misses      15419    15329      -90
+ Partials     1632     1624       -8

Impacted Files
Coverage Δ

packages/amplify-cli/src/execution-manager.ts
54.85% <100.00%> (+38.83%)
:arrow_up:

packages/amplify-cli/src/domain/amplify-event.ts
85.71% <0.00%> (+3.57%)
:arrow_up:

packages/amplify-cli/src/plugin-manager.ts
28.98% <0.00%> (+11.59%)
:arrow_up:

...li/src/domain/amplify-usageData/getUsageDataUrl.ts
100.00% <0.00%> (+12.50%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@danielleadams I added test for execution-manager.ts. It looks like there were nothing failures in the CI, Is there anything blocking to merging this PR?

Not sure if this issue is related, but I'll leave it here: https://github.com/aws-amplify/amplify-cli/issues/11021


possible issue in delayOnNextBySelector?

I'm trying to create an Observable[Int] that delays every element by its amount, e.g., [100, 200, 300] => first element is emitted after 100 ms, after that the second after 200 ms, etc
I have tried
def delayBySelf(in: Observable[Int]) =
  in.delayOnNextBySelector(x => Observable.evalDelayed(x.millis, ()))

but this doesn't seem to work - all the elements are emitted instantly.

Nevermind, I did a mistake :)


HiroSystems.Clarinet version 1.5.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @blockstack-devops,
The package installs normally, but requires Microsoft.VCRedist.2015+.x64 to run:

Pending:

https://github.com/microsoft/winget-cli/issues/163

[Policy] Dependencies

/AzurePipelines run


Need content for 22

https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/adc0ed27-ac22-4817-b4e8-be36b8f5d458/

Need to device to check the initial POC @ASN-BitHyve


Add doc and arglists metadata to defnc macro

This PR adds support for an optional doc string parameter to the defnc macro, as well as updating the var metadata with an :arglists key using the props-bindings arg. This is super useful for documenting components and quickly seeing available props in e.g. CIDER's info bar.

I understand that this project is not currently being maintained. We're still happy users at Nosco and not planning to move to Helix or another library soon. We've considered forking, but for everyone else using hx, it would be great to get small improvements like this upstream.

Much appreciated! I know that there are people who use hx out there, so I'm sure they'll appreciate this too :)

Awesome, thanks for merging! Do you plan on cutting a new release? I've upgraded our project to include hx as a git dependency.


Add new Branch Protection "Require Last Push Approval"

New Branch Protection features added in October are not yet supported https://github.blog/changelog/2022-10-20-new-branch-protections-last-pusher-and-locked-branch/
https://docs.github.com/en/rest/branches/branch-protection#get-branch-protection
require_last_push_approval

Thank you, @fproulx-boostsecurity !
This would be a great PR for any new contributor to this repo or a new Go developer.
All contributions are greatly appreciated!
Feel free to volunteer for any issue and the issue can be assigned to you so that others don't attempt to duplicate the work.
Please check out our CONTRIBUTING.md guide to get started. (In particular, please remember to go generate ./... and don't use force-push to your PRs.)
Thank you!

Hey there, is this issue still available? I'm currently learning how to contribute to open source, and this was marked as a good place to start.

Hey there, is this issue still available? I'm currently learning how to contribute to open source, and this was marked as a good place to start.

Cool! Yes, this issue is available.
Thank you, @XaurDesu , this issue is yours.

Where should this change be located, exactly? I am currently reviewing the code structure of this project, yet I am not really sure where should we add this change. commit cb24cabfacd4bff06727d5b3acea48ba56e5f690 is concerned with this functionality too and references github/github-accessors.go, github/github-accessors_test.go and github/repos.go, yet i'm not sure if either of those is the target for this change, sorry if this is an amateurish question, then again, i'm pretty new.

@XaurDesu Dang you beat me getting to this ticket. 😄
I was also kinda poking around as well. And i think https://github.com/google/go-github/commit/cb24cabfacd4bff06727d5b3acea48ba56e5f690 added require_last_push_approval but there still no implementations for lock_branch and allow_fork_synching.
So...i would think you need to update the Protection struct with those 2 new fields.
and following the patterns with the other fields in that struct.  Add 2 new get functions GetLockBranch and GetAllowForkSynching .  But that just kinda a guess, from a guy trying to learn as well. 😅
cc: @gmlewis

Thanks, @dtdao and @XaurDesu - yes, that sounds like a promising plan. Let's please take a look at this in a PR and see how well it works out.

I've got an uncommited version of the changes proposed by @dtdao in my local machine, mostly basing it in the way cb24cabfacd4bff06727d5b3acea48ba56e5f690 was strtuctured, with all changes being inside repos.go, modifying Protection and adding the type structs AllowForkSyncing and LockBranch tests don't indicate any failures but i'm not sure of the format, should i commit to my fork those changes?
cc. @gmlewis

tests don't indicate any failures but i'm not sure of the format, should i commit to my fork those changes?

I'm not sure I understand the question, but feel free to create a PR and add "Fixes: #2574" to the description and we can take a look at it, and see where to go from there. See our CONTRIBUTING.md guide for general helpful advice about contributing to this project.

Just linked a pull request for it, again, it is fairly barebones but has the functions discussed in this issue.


[Feature] Add support for Cassandra Vector Store

Description
This adds a VectorStore implementation for working with Apache Cassandra® databases, including the cloud database DataStax Astra DB.
This vector store supports both "default" and MMR query methods.
This is a high-performance, highly-scalable, zero-downtime NoSQL database and as such is a good match for robust applications using vector-search techniques at scale.
Type of Change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[X] Added new unit/integration tests: unit tests (there do not seem to be integration tests throughout the codebase)
[x] Added new notebook (that tests end-to-end)
[X] I have successfully run a few end-to-end Python programs that make use of the functionalities ("manual integration tests", if you will)

Other observations
The implementation is based on the cassIO library, which abstracts the database access in a ML- and genAI-oriented way.
Around line 103 of cassandra.py I use the construct getattr(result.node, DEFAULT_TEXT_KEY, "") to extract the "text" contained in a node. I resort to this since, looking at DEFAULT_TEXT_KEY, it seems unsafe to just assume it will always be called "text". Can that be considered idiomatic? Should that be changed?
Also, incidentally: when implementing MMR, I noticed that in the SimpleVectorStoreData implementation the possibility to find member mmr_threshold directly in the passed VectorStoreQuery object (as suggested by the definition of the latter structure) is overlooked, and the parameter is searched directly in the method kwargs. To err on the safe side, in my implementation of the MMR search I check for mmr_threshold in both places, with the one in query taking precedence.

I have resolved the minor problems you mentioned (including the testing failure above, an oversight on my part when employing mocks, sorry) but since we are still in progress on the "grouping nodes the best way for deletion and retrieval" I would refrain a little more from pushing. Will keep you informed, and thank you for the quick review!

@hemidactylus is this still in progress?

Hi @logan-markewich , thank you for asking! Yes, very much in progress. Presumably by this week we will publish a new version of cassIO which will resolve the one pending issue that is keeping this PR. Stay tuned :)

Hello, @logan-markewich , I am happy to re-submit this PR for your approval!
We have been releasing an new version of the cassIO library, as anticipated, and updated the code in this PR with the correct behaviour of the delete method (now it works at document level), plus a few other improvements:

a more agnostic terminology (less confusion with Llamaindex' node-vs-document concept)
handling of concurrent insertion of rows to the store
support for metadata filtering in querying
updates to the example notebook to reflect the above (and demonstrating metadata filtering).

Thank you for your patience, and let me know if there are any pending issues still!

Hi @logan-markewich  A friendly nudge on this. I would love to try this with Astra from the mainline.


Numeric range slider

It would be nice to have support for a range slider that has two handles to select a range between a min and a max value.

What are expected input values for that control?

If I understand correctly, it would be the min value, the max value, the value where the first handle will appear and the value where the second handle will appear. The handles could default to the given min and max values if there are no values given for them.

For example, X-Y panel will be provided for input {x: 123, y: 456}. So what kind of input values should require a range slider? {min: 0, max: 100}? Are there any other input values for that control?

It would be the same input that a regular slider has. It would just have two handles instead of one.

Tweakpane should provide output in the same format as input value because this is tweaking.
const params = {
  range: {min: 0, max: 100},
};

pane.addInput(params, 'range');

So probably it doesn't have the same input as a regular slider.

From what I can see at:
https://www.jqueryscript.net/demo/Flexible-Touch-Friendly-jQuery-Range-Slider-Plugin-Ion-RangeSlider/
https://www.cssscript.com/demo/animated-customizable-range-slider-pure-javascript-rslider-js/
https://www.jqueryscript.net/demo/Highly-Customizable-Range-Slider-Plugin-For-Bootstrap-Bootstrap-Slider/
https://mdbootstrap.com/plugins/jquery/multi-range/
there are two ways to approach this:
One is that each handle tweaks from min to max.
The other is that the left handle tweaks from min to value of the right handle and the right handle tweaks from the value of the left handle to the max value.
Personally the second option makes more sense to me.
Where the single handle slider returns a value the double handle value returns an object (or something else) with two values.

I'm not sure that it should be included in the core or be implemented with the plugin system #78...

I was just thinking something like this would be very useful. For example, I have 2 values: alpha-min and alpha-max, which takes up 2 sliders and runs the risk of crossing values low > high. I could write checks, of course, but it would be super helpful to combine 2 sliders into a single range slider. Something like this
const params = {
    alpha: {low: -0.5, high: 0.5}
}

const pane = new Tweakpane();
pane.addRange(PARAMS, 'alpha');

// Or alternatively, declare the upper and lower bounds of the range:
pane.addRange(PARAMS, 'alpha', {min: -1, max: 1});

I was just thinking something like this would be very useful. For example, I have 2 values: alpha-min and alpha-max, which takes up 2 sliders and runs the risk of crossing values low > high. I could write checks, of course, but it would be super helpful to combine 2 sliders into a single range slider. Something like this
const params = {
    alpha: {low: -0.5, high: 0.5}
}

const pane = new Tweakpane();
pane.addRange(PARAMS, 'alpha');

// Or alternatively, declare the upper and lower bounds of the range:
pane.addRange(PARAMS, 'alpha', {min: -1, max: 1});

Thank you for details. I already know the usefulness of that component, but my concern is... is this common structure enough to add to the core?
For example, many famous libraries have Point2D-like class (PixiJS, three.js, ...) so I decided to add its support to the core. But I have barely seen the range-like class or the structure in these libraries.
If it is not common enough, I think that it should be implemented with the plugin system (#78, but not implemented yet).

Thank you for details. I already know the usefulness of that component, but my concern is... is this common structure enough to add to the core?
For example, many famous libraries have Point2D-like class (PixiJS, three.js, ...) so I decided to add its support to the core. But I have barely seen the range-like class or the structure in these libraries.
If it is not common enough, I think that it should be implemented with the plugin system (#78, but not implemented yet).

Oh, I see what you mean. Yeah, Three.js doesn't exactly have a one-dimensional range data structure. The closest is a Box3 Bounding Box, but that's three-dimensional.

Oh, I see what you mean. Yeah, Three.js doesn't exactly have a one-dimensional range data structure. The closest is a Box3 Bounding Box, but that's three-dimensional.

@cocopon I actually think that this component is much more useful (to me at least) than the Point2D component. But this aside, providing the functionality is much more important (to me) that where the functionality lives.
Once it is implemented in the core ... you can decide if it belongs there or in a plugin (and hopefully it will be easy to move).

@cocopon I actually think that this component is much more useful (to me at least) than the Point2D component. But this aside, providing the functionality is much more important (to me) that where the functionality lives.
Once it is implemented in the core ... you can decide if it belongs there or in a plugin (and hopefully it will be easy to move).

@gaitat

this component is much more useful than the Point2D component

It's your personal opinion... If I accepted all suggested components, this library would become huge, be less maintainable, and be hard to understand the usage.
So I think that one of the important things for a feature request is a reason that is objectively convincing. (e.g. famous library has a support for this kind of data structure)

Once it is implemented in the core ... you can decide if it belongs there or in a plugin

I disagree. This is a broken change and force users to fix their code. As a package maintainer, I have to be careful about backward compatibilities.

@gaitat

this component is much more useful than the Point2D component

It's your personal opinion... If I accepted all suggested components, this library would become huge, be less maintainable, and be hard to understand the usage.
So I think that one of the important things for a feature request is a reason that is objectively convincing. (e.g. famous library has a support for this kind of data structure)

Once it is implemented in the core ... you can decide if it belongs there or in a plugin

I disagree. This is a broken change and force users to fix their code. As a package maintainer, I have to be careful about backward compatibilities.

Tweakpane 2.0.0 is now! It has a plugin system so you can register a custom binding view to the pane.
https://cocopon.github.io/tweakpane/plugin.html
FYI: Here is my first plugin currently working on...
https://user-images.githubusercontent.com/602961/109491757-59e50e00-7acd-11eb-9c6b-d8b56300fcfc.mp4
https://twitter.com/cocopon/status/1366350034531323904?s=20

Here is the very first version of the plugin:
https://github.com/cocopon/tweakpane-plugin-interval
Demo:
https://codesandbox.io/s/tweakpane-plugin-interval-6buv9


MRG, ENH: Allow saving non-head cf in mon.save

As discussed with @agramfort, there is no need to have this restriction. Simultaneously makes things more DRY and removes two # XXX comments from our code, which is nice.

thx @larsoner !

This won't work...
def write_dig(fname, pts, coord_frame=None):
    return _dig_write_dig(fname, pts, coord_frame=None)

Also, after correcting the error mentioned below, the following code generate and assert exception
        montage = mne.channels.make_dig_montage(ch_pos=ch_pos,
                                                nasion=brain_fiducials.loc["Nz"].values,
                                                lpa=brain_fiducials.loc["LPA"].values,
                                                rpa=brain_fiducials.loc["RPA"].values,
                                                hsp=hsp,
                                                coord_frame="mri")

        montage_fname = fs_subject_path / subject / "{}-montage.fif".format(montage_str)

        mne.io.meas_info.write_dig(montage_fname, montage.dig, coord_frame=FIFF.FIFFV_COORD_MRI)
        dig = mne.channels.read_dig_fif(montage_fname).dig[:3]
        for d, want in zip(dig, (FIFF.FIFFV_POINT_LPA,
                                 FIFF.FIFFV_POINT_NASION,
                                 FIFF.FIFFV_POINT_RPA)):
            assert d['kind'] == FIFF.FIFFV_POINT_CARDINAL
            assert d['coord_frame'] == FIFF.FIFFV_COORD_MRI

WRITING coord_frame in mne.io.write.write_dig_points 5

---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-14-f785d8f255d5> in <module>
     71                                  FIFF.FIFFV_POINT_RPA)):
     72             assert d['kind'] == FIFF.FIFFV_POINT_CARDINAL
---> 73             assert d['coord_frame'] == FIFF.FIFFV_COORD_MRI
     74
     75

AssertionError:

The printing of "WRITING coord_frame in mne.io.write.write_dig_points 5" indicates that the up to mne.io.write.write_dig_points the coordinate frame seems to be taken into account. Maybe the issue is in mne.channels.read_dig_fif ?

Apologies. Please disregard the last message, I had not rebased my git when I tested. The previous one is valid though and this slight bug should be corrected for the full functionality of write_dig to work.

@christian-oreilly the idea of this PR is not use mne.io.meas_info.write_dig (this isn't really meant for public consumption despite its name), this should work fine if you just do montage.save(...) -- your assertions can pass. Can you see if using this more visible method works?

Sure... I can check... but this code is still broken no?
def write_dig(fname, pts, coord_frame=None):
    return _dig_write_dig(fname, pts, coord_frame=None)

At least, if it is not considered broken, the documentation of write_dig should not say that coord_frame is a dummy attributes that do nothing.

Sure... I can check... but this code is still broken no?

Yes we should fix this

FYI, montage.save(...) worked fine


Add rock-paper-scissor game

Is your feature request related to a problem? Please describe
It's not related to a problem, I'd just like to add a new game to the project
Describe the solution you'd like
I would like to contribute the rock-paper-scissors game to the project

I would like to work on this game. It would possible, you could assign me this issue and tag it hacktoberfest!?

Hi, I would like to contribute to the rock-paper-scissor game as I have experience in making games with JavaScript.


make squid cache independent of pkg-mirror

caching only works when we always use the same proxy, which will not happen.
try this: https://wiki.squid-cache.org/Features/StoreID

closed via https://github.com/Finkregh/archstrap/pull/10


Fixed API version for K8s Ingress

First things first, thank you for your amazing work on Ackee!
As I tried to use it for myself, I ran into the following error:
Error: UPGRADE FAILED: resource mapping not found for name: "test-ackee-heka-2" namespace: "" from "": no matches for kind "Ingress" in version "extensions/v1beta1"
ensure CRDs are installed first

It came from the API version of K8s Ingress which as been deprecated.
Here is the fix, tested on my project.
Have a good day
Phaoll

What is it with this? @suda will this ever be merged.
This breaks the entire helm chart.


Unrendered goals text in "Waypoint: Comment your JavaScript Code"

Challenge http://freecodecamp.com/challenges/waypoint-comment-your-javascript-code has an issue. Goal text in "Waypoint: Comment your JavaScript Code" is unrendered.

apparently this is intended.


Add "Already have an account Login" in signUp page and visa versa

Add feature if user have already account and add if user not have account

Please assign me to fix issue

someone has already make a pr regarding this
check #30


chez-modules: Fix path to csv-site.

Motivation for this change
It seems that the installation path for chez modules no longer includes
the Chez version. This fixes the build for chez-mit and chez-scmutils.
ZHF: #97479
Note: I have tried to load the libraries in a nix-shell -p chez chez-scmutils, but it seems like some environment variables need to be set manually to get that working. I don't know whether that was the case beforehand. I suppose @jitwit or @thoughtpolice would know.
Things done

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

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

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

Ah yes, the environment variables ought to be set up automatically. I'm more familiar with guix, a nix native-search-paths equivalent is done through setupHook?

It's funny that you mentioned it, I switched from Guix to Nix literally yesterday. I'm not sure how setupHooks work yet, but if nobody beats me to it, I'll submit a follow-up PR that makes the packages work in an interactive nix-shell (and probably simplifies the build procedures).

Two guix refugees meeting on a scheme PR, who would have thought
Builds happily, linux x86_64

Also, welcome to nixpkgs!

I've opened a follow-up PR that redoes this with a setupHook: #98128. I'm planning to backport both at once...


Add RawTherapee/Design Tools

RawTherapee

RawTherapee - RawTherapee is designed for developing raw files from a broad range of digital cameras, as well as HDR DNG files and non-raw image formats (JPEG, TIFF and PNG). The target audience ranges from enthusiast newcomers who wish to broaden their understanding of how digital imaging works to semi-professional photographers. Knowledge in color science is not compulsory, but it is recommended that you are eager to learn and ready to read our documentation (RawPedia) as well as look up basic concepts which lie outside the scope of RawPedia, such as color balance, elsewhere.
License: GPL 3.0
Why do you think this helps users privacy?: Open source, used alongside darktable as Adobe Lightroom free replacements.
Under what section should this service be listed?: Design Tools
Additional comments / info: Github page here.

Maybe this would better fit under Photo Editing and Management?

Yes, it would!


Allow passing a template config file to gitlab-runner register

Describe the bug
Some options cannot be passed to gitlab-runner service, for example, the autoscaling periods, replacing off-peak-time configuration for the machine executor.
Notify maintainers

@petabyteboy
@misuzu
@zowoq
@bgamari

Metadata
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
  - services.gitlab-runner.services

What do you think about adding an option to pass a template toml configuration file?
https://docs.gitlab.com/runner/register/#runners-configuration-template-file

Some options cannot be passed to gitlab-runner service, for example, the autoscaling periods, replacing off-peak-time configuration for the machine executor.

Everything that you can do using gitlab-runner register can be configured via registrationConfigFile or registrationFlags.

What do you think about adding an option to pass a template toml configuration file?
https://docs.gitlab.com/runner/register/#runners-configuration-template-file

Using registrationConfigFile option:
services.gitlab-runner.services.<name>.registrationConfigFile = pkgs.writeText "gitlab-runner-registration" ''
    TEMPLATE_CONFIG_FILE=${pkgs.writeText "template.toml" ''
      <template here>
    ''}
  '';
};

Using registrationFlags option:
services.gitlab-runner.services.<name>.registrationFlags = [
  "--template-config" (pkgs.writeText "template.toml" ''
    <template here>
  '')
];

Nice! I didn't think about this way in!
It's enough for me to make progress.
I imagine though that it would gain to be documented a bit more clearly (maybe there's a section about gitlab-runner in the nix manual?). I'll leave it to someone else to close this if it's not something that we can/should do.


reversing deltaPhi order, scaling GW inputs for val and tst partitions

This PR makes 3 changes:

It reverses the order of the deltaPhi calculation so that the phase of the water signal is subtracted from the phase of the air signal and does not convert the phase to the day of the year with the maximum temperature. This yields a positive phase difference when the air signal precedes the water signal.
It scales the calculated GW inputs for the validation and testing partitions. Previously scaled values were only calculated for the training partition, but scaled values are needed for the new training function that calculates the validation loss for each epoch (from #141)
It adds an option to the config file to specify the calculation method for the GW loss (fft or linalg)

Sounds good. I'll take a look at this sometime this week!

Thanks! Subtracting the phase from (3/2)*pi/2 converts the phase to the day of the year with the max temp. We could do this, but then we'd need to subtract the air temp day from the water temp day to get a positive lag, rather than water phase from air phase as we're currently doing. There's a few more details on this in the ExaSheds_meeting_notes.docx file (on sharepoint in 40.1 Data-driven temperature / Meeting Notes)


Move the Book of Medivh to Lordaeron if Dalaran is not picked at the start of the game

May also need to look at Scourge's intro text to see if it references Dalaran specifically.


Minor improvements

use new pip strategy for dev-install
fix documentation string

Codecov Report

Merging #221 (36f539a) into master (27bec63) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
- Coverage   87.37%   87.20%   -0.17%
==========================================
  Files           5        5
  Lines         998      985      -13
==========================================
- Hits          872      859      -13
  Misses        126      126

Impacted Files
Coverage Δ

oscovida/oscovida.py
85.64% <ø> (-0.15%)
:arrow_down:

oscovida/metadata.py
95.77% <0.00%> (-0.18%)
:arrow_down:

oscovida/plotting_helpers.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Firefox 80 failing after a couple minutes in parallel

I've been testing out Selenoid to see if it would be a good solution for us and I was super impressed when Chrome was so fast and worked incredibly well.  However I am not able to run tests effectively in Firefox.  I run 5 tests in parallel (which for a 6 core machine should be no problem and there were no problems with Chrome) and after just a few minutes I get random failures but they all have 1 of 2 errors.
The main error is: org.openqa.selenium.NoSuchSessionException: Tried to run command without establishing a connection
This error will usually happen after 1 or 2 tests pass in a class.
Less rarely I see this error: NoSuchWindowException: Browsing context has been discarded
My guess is Firefox is probably crashing, but I'm unsure why.
Here is my browsers.json
{     "chrome": {         "default": "85.0",         "versions": {             "85.0": {                 "image": "selenoid/chrome:85.0",                 "port": "4444",                 "path": "/",                 "tmpfs": {"/tmp": "size=512m", "/var": "size=128m"},                 "shmSize" :<PHONE_NUMBER>             }         }     },     "firefox": {         "default": "80.0",         "versions": {             "80.0": {                 "image": "selenoid/firefox:80.0",                 "port": "4444",                 "path": "/wd/hub",                 "tmpfs": {"/tmp": "size=512m", "/var": "size=128m"},                 "shmSize" :<PHONE_NUMBER>             }         }     } }
I start selenoid with this command:
HTTP_PROXY=proxy.company.com:8080 HTTPS_PROXY=proxy.company.com:8080 ./cm selenoid start --args "-limit 10" --shm-size 2000
I am unsure of what else to try.  Is anyone else having this issue?
Let me know what else I can provide.  This is what I get from running "docker logs selenoid":
[2020/09/04 15:46:35 [-] [INIT] [Loading configuration files...] 2020/09/04 15:46:35 [-] [INIT] [Loaded configuration from /etc/selenoid/browsers.json] 2020/09/04 15:46:35 [-] [INIT] [Video Dir: /opt/selenoid/video] 2020/09/04 15:46:35 [-] [INIT] [Logs Dir: /opt/selenoid/logs] 2020/09/04 15:46:35 [-] [INIT] [Using Docker API version: 1.40] 2020/09/04 15:46:35 [-] [INIT] [Timezone: UTC] 2020/09/04 15:46:35 [-] [INIT] [Listening on :4444] 2020/09/04 15:50:16 [-] [NEW_REQUEST] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [-] [NEW_REQUEST_ACCEPTED] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [-] [NEW_REQUEST] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [-] [NEW_REQUEST_ACCEPTED] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [1] [LOCATING_SERVICE] [firefox] [] 2020/09/04 15:50:16 [0] [LOCATING_SERVICE] [firefox] [] 2020/09/04 15:50:16 [-] [DEFAULT_VERSION] [Using default version: 80.0] 2020/09/04 15:50:16 [-] [DEFAULT_VERSION] [Using default version: 80.0] 2020/09/04 15:50:16 [1] [USING_DOCKER] [firefox] [80.0] 2020/09/04 15:50:16 [0] [USING_DOCKER] [firefox] [80.0] 2020/09/04 15:50:16 [0] [CREATING_CONTAINER] [selenoid/firefox:80.0] 2020/09/04 15:50:16 [1] [CREATING_CONTAINER] [selenoid/firefox:80.0] 2020/09/04 15:50:16 [-] [NEW_REQUEST] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [-] [NEW_REQUEST_ACCEPTED] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [2] [LOCATING_SERVICE] [firefox] [] 2020/09/04 15:50:16 [-] [DEFAULT_VERSION] [Using default version: 80.0] 2020/09/04 15:50:16 [2] [USING_DOCKER] [firefox] [80.0] 2020/09/04 15:50:16 [2] [CREATING_CONTAINER] [selenoid/firefox:80.0] 2020/09/04 15:50:16 [-] [NEW_REQUEST] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [-] [NEW_REQUEST_ACCEPTED] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [3] [LOCATING_SERVICE] [firefox] [] 2020/09/04 15:50:16 [-] [DEFAULT_VERSION] [Using default version: 80.0] 2020/09/04 15:50:16 [3] [USING_DOCKER] [firefox] [80.0] 2020/09/04 15:50:16 [3] [CREATING_CONTAINER] [selenoid/firefox:80.0] 2020/09/04 15:50:16 [0] [STARTING_CONTAINER] [selenoid/firefox:80.0] [c5dc9643c8a7d52fc17105f98d177b7c48af0d021612c36917111d4c11a13f9a] 2020/09/04 15:50:16 [1] [STARTING_CONTAINER] [selenoid/firefox:80.0] [5e354dabcb5a60e53664e14ebc6c752fefc33ed9429c780f07690b3b417b082b] 2020/09/04 15:50:16 [2] [STARTING_CONTAINER] [selenoid/firefox:80.0] [f61efae793d25677f57487bc53b0334602d0f8c83775f755d67dff5337200f8d] 2020/09/04 15:50:16 [3] [STARTING_CONTAINER] [selenoid/firefox:80.0] [a86ae6b1d0e4797fafa50d167d43f62a4f771c745f03dd23b05513cb7146c7f2] 2020/09/04 15:50:16 [-] [NEW_REQUEST] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [-] [NEW_REQUEST_ACCEPTED] [unknown] [<IP_ADDRESS>] 2020/09/04 15:50:16 [4] [LOCATING_SERVICE] [firefox] [] 2020/09/04 15:50:16 [-] [DEFAULT_VERSION] [Using default version: 80.0] 2020/09/04 15:50:16 [4] [USING_DOCKER] [firefox] [80.0] 2020/09/04 15:50:16 [4] [CREATING_CONTAINER] [selenoid/firefox:80.0] 2020/09/04 15:50:16 [4] [STARTING_CONTAINER] [selenoid/firefox:80.0] [96e3579c37d4b92c4d3c192c3a09291963e89b4781020887d1c83fcdc9eacfcf] 2020/09/04 15:50:17 [1] [CONTAINER_STARTED] [selenoid/firefox:80.0] [5e354dabcb5a60e53664e14ebc6c752fefc33ed9429c780f07690b3b417b082b] [0.81s] 2020/09/04 15:50:17 [0] [CONTAINER_STARTED] [selenoid/firefox:80.0] [c5dc9643c8a7d52fc17105f98d177b7c48af0d021612c36917111d4c11a13f9a] [0.84s] 2020/09/04 15:50:17 [2] [CONTAINER_STARTED] [selenoid/firefox:80.0] [f61efae793d25677f57487bc53b0334602d0f8c83775f755d67dff5337200f8d] [0.87s] 2020/09/04 15:50:17 [4] [CONTAINER_STARTED] [selenoid/firefox:80.0] [96e3579c37d4b92c4d3c192c3a09291963e89b4781020887d1c83fcdc9eacfcf] [0.75s] 2020/09/04 15:50:17 [3] [CONTAINER_STARTED] [selenoid/firefox:80.0] [a86ae6b1d0e4797fafa50d167d43f62a4f771c745f03dd23b05513cb7146c7f2] [0.98s] 2020/09/04 15:50:20 [4] [SERVICE_STARTED] [selenoid/firefox:80.0] [96e3579c37d4b92c4d3c192c3a09291963e89b4781020887d1c83fcdc9eacfcf] [2.83s] 2020/09/04 15:50:20 [4] [PROXY_TO] [96e3579c37d4b92c4d3c192c3a09291963e89b4781020887d1c83fcdc9eacfcf] [http://<IP_ADDRESS>:4444/wd/hub] 2020/09/04 15:50:20 [4] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/wd/hub] [1] 2020/09/04 15:50:20 [3] [SERVICE_STARTED] [selenoid/firefox:80.0] [a86ae6b1d0e4797fafa50d167d43f62a4f771c745f03dd23b05513cb7146c7f2] [2.81s] 2020/09/04 15:50:20 [3] [PROXY_TO] [a86ae6b1d0e4797fafa50d167d43f62a4f771c745f03dd23b05513cb7146c7f2] [http://<IP_ADDRESS>:4444/wd/hub] 2020/09/04 15:50:20 [3] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/wd/hub] [1] 2020/09/04 15:50:20 [2] [SERVICE_STARTED] [selenoid/firefox:80.0] [f61efae793d25677f57487bc53b0334602d0f8c83775f755d67dff5337200f8d] [2.97s] 2020/09/04 15:50:20 [2] [PROXY_TO] [f61efae793d25677f57487bc53b0334602d0f8c83775f755d67dff5337200f8d] [http://<IP_ADDRESS>:4444/wd/hub] 2020/09/04 15:50:20 [2] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/wd/hub] [1] 2020/09/04 15:50:20 [0] [SERVICE_STARTED] [selenoid/firefox:80.0] [c5dc9643c8a7d52fc17105f98d177b7c48af0d021612c36917111d4c11a13f9a] [3.01s] 2020/09/04 15:50:20 [0] [PROXY_TO] [c5dc9643c8a7d52fc17105f98d177b7c48af0d021612c36917111d4c11a13f9a] [http://<IP_ADDRESS>:4444/wd/hub] 2020/09/04 15:50:20 [0] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/wd/hub] [1] 2020/09/04 15:50:20 [1] [SERVICE_STARTED] [selenoid/firefox:80.0] [5e354dabcb5a60e53664e14ebc6c752fefc33ed9429c780f07690b3b417b082b] [3.07s] 2020/09/04 15:50:20 [1] [PROXY_TO] [5e354dabcb5a60e53664e14ebc6c752fefc33ed9429c780f07690b3b417b082b] [http://<IP_ADDRESS>:4444/wd/hub] 2020/09/04 15:50:20 [1] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/wd/hub] [1] 2020/09/04 15:50:26 [3] [SESSION_CREATED] [e2d933ec-34ab-419a-9a41-cbefaa0ba9b4] [1] [9.89s] 2020/09/04 15:50:26 [2] [SESSION_CREATED] [3838cabf-6541-40f2-bd42-dba12aa2d5dc] [1] [9.90s] 2020/09/04 15:50:26 [0] [SESSION_CREATED] [b347acd1-cf49-4898-8602-6e2232c911d8] [1] [9.94s] 2020/09/04 15:50:26 [4] [SESSION_CREATED] [e3669da3-4b97-492e-8fb9-f7d327fa24f4] [1] [9.79s] 2020/09/04 15:50:26 [1] [SESSION_CREATED] [bb5b0298-09e3-4a91-b4f6-f389ef0e0b94] [1] [10.03s] 2020/09/04 15:54:29 [7832] [SESSION_DELETED] [b347acd1-cf49-4898-8602-6e2232c911d8] 2020/09/04 15:54:29 [7835] [SESSION_DELETED] [e3669da3-4b97-492e-8fb9-f7d327fa24f4] 2020/09/04 15:54:29 [7837] [SESSION_DELETED] [3838cabf-6541-40f2-bd42-dba12aa2d5dc] 2020/09/04 15:54:29 [7836] [SESSION_DELETED] [e2d933ec-34ab-419a-9a41-cbefaa0ba9b4] 2020/09/04 15:54:29 [7838] [SESSION_DELETED] [bb5b0298-09e3-4a91-b4f6-f389ef0e0b94] 2020/09/04 15:54:30 [3] [REMOVING_CONTAINER] [a86ae6b1d0e4797fafa50d167d43f62a4f771c745f03dd23b05513cb7146c7f2] 2020/09/04 15:54:30 [2] [REMOVING_CONTAINER] [f61efae793d25677f57487bc53b0334602d0f8c83775f755d67dff5337200f8d] 2020/09/04 15:54:32 [3] [CONTAINER_REMOVED] [a86ae6b1d0e4797fafa50d167d43f62a4f771c745f03dd23b05513cb7146c7f2] 2020/09/04 15:54:32 [2] [CONTAINER_REMOVED] [f61efae793d25677f57487bc53b0334602d0f8c83775f755d67dff5337200f8d] 2020/09/04 15:54:34 [0] [REMOVING_CONTAINER] [c5dc9643c8a7d52fc17105f98d177b7c48af0d021612c36917111d4c11a13f9a] 2020/09/04 15:54:34 [4] [REMOVING_CONTAINER] [96e3579c37d4b92c4d3c192c3a09291963e89b4781020887d1c83fcdc9eacfcf] 2020/09/04 15:54:35 [0] [CONTAINER_REMOVED] [c5dc9643c8a7d52fc17105f98d177b7c48af0d021612c36917111d4c11a13f9a] 2020/09/04 15:54:35 [4] [CONTAINER_REMOVED] [96e3579c37d4b92c4d3c192c3a09291963e89b4781020887d1c83fcdc9eacfcf] 2020/09/04 15:54:35 [1] [REMOVING_CONTAINER] [5e354dabcb5a60e53664e14ebc6c752fefc33ed9429c780f07690b3b417b082b] 2020/09/04 15:54:35 [1] [CONTAINER_REMOVED] [5e354dabcb5a60e53664e14ebc6c752fefc33ed9429c780f07690b3b417b082b]

@jacattau I would take a look at memory consumption first.

@vania-pooh maybe I am missing it but I am not seeing what command to run to check this.  Could you point me in the right direction?  If i do see that i don't have sufficient memory how would I go about adding more?  I set the shmSize to 2gb already.

@jacattau I mean using commands like free and top.

I wasn't getting free to work but I did get top to work even though it wasn't showing me much in the way of memory usage.  I used docker stats and that got me some usage in real time.
I upped the memory from 2gb to 4gb but that only seemed to help a little.  From the point my tests started failing here was the usage statistics:
CONTAINER ID        NAME                   CPU %               MEM USAGE / LIMIT     MEM %               NET I/O             BLOCK I/O           PIDS
5ef51bfc985e        selenoid               8.55%               7.445MiB / 3.848GiB   0.19%               361MB / 377MB       0B / 0B             19
0d09bf806e4c        focused_wescoff        0.29%               29.5MiB / 3.848GiB    0.75%               26.2MB / 5.51MB     0B / 0B             49
c87fb79d137d        gifted_chandrasekhar   75.79%              727.8MiB / 3.848GiB   18.47%              22.6MB / 4.6MB      0B / 0B             202
506747aff18c        modest_cohen           142.32%             725.6MiB / 3.848GiB   18.42%              19.1MB / 4.46MB     0B / 0B             201
b8a84da8cba6        xenodochial_liskov     68.34%              852.8MiB / 3.848GiB   21.64%              19.2MB / 3.95MB     0B / 0B             202
dc616d60aaed        gracious_engelbart     67.33%              718.3MiB / 3.848GiB   18.23%              17.3MB / 3.85MB     0B / 0B             200
So it looks like even though upping the memory to 4gb shouldn't have made a difference from 2gb.  However, it did work better with 4gb but I am still seeing this issue.  Right now it's about 24 failures out of 600 tests.  I am running on a 6 core machine so it should 100% be able to handle 5 parallel sessions right?
I do not think I have video recording running unless I am misunderstanding the documentation.  Do I need to do anything to disable video?  From what I understand it's disabled unless I set the capability to true.

@jacattau could you attach a log of failed session here (it will be automatically saved when enableLog capability is set)? Video recording is turned off by default.

Hi @vania-pooh I will try to get that to you, but I think there might actually be an issue with Firefox 80.0 itself.  I noticed they quickly released version 80.0.1 with this release notes: https://www.mozilla.org/en-US/firefox/80.0.1/releasenotes/
I started running tests on Firefox 79 (I didn't see a selenoid image for 80.0.1) and so far it seems to be working but I haven't finished a single regression suite yet. I will hammer 79 pretty hard and see if I can reproduce.

@jacattau they are doing hotfix releases almost for every stable release. But if your theory confirms - let's rebuild and image with 80.0.1 then.

@vania-pooh i do think there is an issue with 80.  I've been having success with 79 outside of the issue in #713  although i am seeing this on Chrome as well.  At this point #713 might be the only thing keeping me from adopting selenoid right now.


Legacy FAQ links stopped working

Where to find the issue
https://www.coronawarn.app/en/faq/
Describe the issue
FAQ Links using the legacy format https://www.coronawarn.app/en/faq/#FAQ_article_anchor no longer open the article https://www.coronawarn.app/en/faq/results/#FAQ_article_anchor
Steps to reproduce
Try to open https://www.coronawarn.app/en/faq/#where_can_i_get_tested in a webbrowser
Note that the article is not opened. Instead the page https://www.coronawarn.app/en/faq/ is presented.
Suggested change
Check whether the updated deployment is responsible for this issue.

Since the FAQ links used in CWA make use of legacy format, this means that the app links to the FAQ do not currently work.
See
https://github.com/corona-warn-app/cwa-app-android/blob/main/Corona-Warn-App/src/main/res/values/links.xml for an example Android file specifying external links.

@MikeMcC399 Fixed.


Use x86_64-darwin as platform in Gemfile.lock

Maintaining specific Mac OS X versions in the Gemfile.lock is cumbersome, so let's just stick to a generic entry x86_64-darwin.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

LGTM


[ttfautohint] run with default options if --autohint is specified without argument

The --autohint option uses nargs=? which allows to specify the option without argument (or with 1 argument), defaulting to None.
However, in save_otfs, the ttfautohint subprocess is called only if the autohint keyword argument is not None.
I would expect that calling fontmake with -a would run ttfautohint using the latter's default options, but at the moment it is simply ignored.
Either we make -a option require an argument and raise if one is not provided (which is the argparse default behavior, without nargs=?) or we use const to use a constant empty string when the option is provided without argument following it, which is what I do here:
https://docs.python.org/3/library/argparse.html#const

also, I see @jamesgk is calling ttfautohint with no extra arguments in Roboto's ttfautohint branch:
https://github.com/google/roboto/commit/f4e8d7a1306087e3ea1d6e90b15afefff65fac1f
this PR basically does the same, without needing to spell out the ttfautohint arguments as an empty string like --autohint " "


[Feature Request] Tunneling

It would be great if it could provide the web service via a tunnel. Then you could avoid opening a port. You could use localtunnel, which offer a node API. If needed, you could then share the web server briefly and use the URL in the mail.

I had no idea this existed and it's dope as hell, so I added it. Thanks!


v0.4 and dev pip installation broken

    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/private/var/folders/vx/6hg82skd71d_f8_mk97gp72h0000gn/T/pip-build-FOo3AV/Flask-Scss/setup.py", line 38, in <module>
        data_files=make_data_files('doc/.build/html', 'doc') + [
      File "/private/var/folders/vx/6hg82skd71d_f8_mk97gp72h0000gn/T/pip-build-FOo3AV/Flask-Scss/setup.py", line 14, in make_data_files
        for elem in os.listdir(root_path):
    OSError: [Errno 2] No such file or directory: 'doc/.build/html'

In setup.py, you do an os.listdir on this directory, but it doesn't exist in the repository; it must be .gitignored on your local machine.
Please test pip installs from github in a virtualenv before pushing to pip in the future!

It is fixed in a new 0.5 version.
I'm sorry for this screw-up


fix test work flow

Adding the composer action and fix the work flow

@gekkone @shahradelahi
you just need to create new environment  named (test)
https://github.com/Moustafa-Elgammal/core/settings/environments/
and add the test bot token
test bot can be created using @BotFather
and a subscripted user id   can be retrieved by @raw_data_bot for your user


Background monitoring on Android

Hi!
I’ve tried to enable background monitoring on Android, as suggested on #40 adding that to the example app.
However, as soon as app goes to background, it stops receiving zone changes.
Do I need to add intent or something special to Android app for background beacon detection to work?
Thank you!

Hi @snadal
What Android phone do you use? I use it at my daily office projects, some devices need to add some lines to enable background monitoring directly on Java/Kotlin code within Android project.

Hi Alann,
I am trying with a Pixel 2 and a very old Samsung Galaxy S4, among others.
Could you please share the lines on your Java / Kotlin code?
Thank you!

Hello,
Would you mind to share the lines you used to enable background monitoring on Android?
Thanks!

I'm having the same issue and don't get monitoring events on either Android or iOS when the app has been terminated. I see that the https://pub.dev/packages/beacons package has support for background monitoring but it looks like it's been abandoned and no longer works properly...

Hi, sorry for delaying response..
In Android, I'm implementing BootstrapNotifier on my custom Application class. I set the region without UUID, major or minor. So when it detect beacon it will trigger the app to start again. Don't forget to make your MainActivity's android:launchMode to singleInstance.
In iOS, from Estimote community :

Even if the app is not running, location events (related to the beacons in this case) are handled the same way as any other app launching events. Every time a phone enters or exits a region while the app is terminated, it will be automatically launched.
application:didFinishLaunchingWithOptions: method (of AppDelegate class) is called with UIApplicationLaunchOptionsLocationKey key existing in launchOptions parameter.

Please read the full article here. You can put your iOS code at application:didFinishLaunchingWithOptions: to waking your app.
For both iOS and Android, you can silently send a push notification that trigger app to start scanning, if it is terminated.
Sometimes I use all of them, sometimes only one or two of it. Find what best suite for your case.
I hope this can helps.
Regards.

@alann-maulana Thanks for your response. Do you have any plans to provide some more concrete code examples for both platforms? I'm unfortunately pretty new to native app development, and while I have a decent grasp of Flutter, my knowledge doesn't extend much beyond that. I do appreciate the pointers in the right direction.

@alann-maulana  I am also stuck in beacon-scanning for Android and IOS while app is in background.
Can you please make one example like this https://pub.dev/packages/beacons package.

Hi, sorry for delaying response..
In Android, I'm implementing BootstrapNotifier on my custom Application class. I set the region without UUID, major or minor. So when it detect beacon it will trigger the app to start again. Don't forget to make your MainActivity's android:launchMode to singleInstance.
In iOS, from Estimote community :

Even if the app is not running, location events (related to the beacons in this case) are handled the same way as any other app launching events. Every time a phone enters or exits a region while the app is terminated, it will be automatically launched.
application:didFinishLaunchingWithOptions: method (of AppDelegate class) is called with UIApplicationLaunchOptionsLocationKey key existing in launchOptions parameter.

Please read the full article here. You can put your iOS code at application:didFinishLaunchingWithOptions: to waking your app.
For both iOS and Android, you can silently send a push notification that trigger app to start scanning, if it is terminated.
Sometimes I use all of them, sometimes only one or two of it. Find what best suite for your case.
I hope this can helps.
Regards.

Hi,
I have no issue in ios when app terminated and its work great.
But, in android ,app can not scan beacon when terminated.
any solution?

maybe by migrating it to v2? #67

Hi, sorry for delaying response..
In Android, I'm implementing BootstrapNotifier on my custom Application class. I set the region without UUID, major or minor. So when it detect beacon it will trigger the app to start again. Don't forget to make your MainActivity's android:launchMode to singleInstance.
In iOS, from Estimote community :

Even if the app is not running, location events (related to the beacons in this case) are handled the same way as any other app launching events. Every time a phone enters or exits a region while the app is terminated, it will be automatically launched.
application:didFinishLaunchingWithOptions: method (of AppDelegate class) is called with UIApplicationLaunchOptionsLocationKey key existing in launchOptions parameter.

Please read the full article here. You can put your iOS code at application:didFinishLaunchingWithOptions: to waking your app.
For both iOS and Android, you can silently send a push notification that trigger app to start scanning, if it is terminated.
Sometimes I use all of them, sometimes only one or two of it. Find what best suite for your case.
I hope this can helps.
Regards.

Hi,
I have no issue in ios when app terminated and its work great.
But, in android ,app can not scan beacon when terminated.
any solution?

+1
@alann-maulana I'll glad to help you to migrate but my android native knowledge is pretty basic. :(

Hello @alann-maulana, could you take some doubt on how is implemented the bootstrap notifier?
I'm not sure how to use my custom application class. Should I use the FlutterApplication that comes set on android:name? or  should I create another class and then set this new class on android:Name? I appreciate your numerous help with the community. Your package and explanations have saved me several times. Thank you

Hi,  after a few attempts I managed to implement the Custom Application class with bootstrap notifier. I'll try to make an explanation to help those who are having difficulty in this part. As soon as there's a time left, I'll post an example on github to make it easier to understand.
1. In the AndroidManifest file, you must modify the "android:name" parameter.
By default it is set to FlutterApplication. That would be something like "io.flutter.app.FlutterApplication".
You should remove this value and replace it with the name of your CustomClass, for example ".CustomApplication".  So press option + enter to create this new class ( On android studio ), after that you should make some modifications to the class created to implement Bootstrap notifier. After implementing bootstrap notifier, your new class will look like this:
`
public class CustomClass extends FlutterApplication implements BootstrapNotifier, BeaconConsumer, PluginRegistry.PluginRegistrantCallback {
private RegionBootstrap regionBootstrap;
BeaconManager beaconManager;

@Override
public void registerWith(PluginRegistry registry) {
    // GeneratedPluginRegistrant.registerWith(registry);
    FlutterBeaconPlugin.registerWith(registry.registrarFor("io.flutter.plugins.flutterbeacon.FlutterBeaconPlugin"));
}

@Override
public void onCreate() {
    super.onCreate();
    FlutterMain.startInitialization(this);

    beaconManager = org.altbeacon.beacon.BeaconManager.getInstanceForApplication(this);

    beaconManager.bind((BeaconConsumer) this);

    Region region = new Region("region", null, null, null);
    regionBootstrap = new RegionBootstrap(this, region);

    try {
        beaconManager.startMonitoringBeaconsInRegion(new Region("region", null, null, null));
    } catch (RemoteException e) {
        e.printStackTrace();
    }
}

@Override
public void didEnterRegion(Region region) { }

@Override
public void didExitRegion(Region region) { }

@Override
public void didDetermineStateForRegion(int i, Region region) { }

@Override
public void onBeaconServiceConnect() { }

private Activity mCurrentActivity = null;

public Activity getCurrentActivity() {
    return mCurrentActivity;
}

public void setCurrentActivity(Activity mCurrentActivity) {
    this.mCurrentActivity = mCurrentActivity;
}

}
`

@Marcoslima016 I am facing the same issue, any update on whether you have posted that example somewhere?

@Marcoslima016 I am facing the same issue, any update on whether you have posted that example somewhere?

hi @evanblasband, soon I put a complete example of an Android application working in the background. I'm busy, but I'm going to have some time to do it soon.

hi @evanblasband, soon I put a complete example of an Android application working in the background. I'm busy, but I'm going to have some time to do it soon.

Hi @Marcoslima016, I encountered the same issue as you. Could you please put a complete example as you mentioned?

Yes Please

hi @evanblasband, soon I put a complete example of an Android application working in the background. I'm busy, but I'm going to have some time to do it soon.

Hi @Marcoslima016, I encountered the same issue as you. Could you please put a complete example as you mentioned?

hi @evanblasband, soon I put a complete example of an Android application working in the background. I'm busy, but I'm going to have some time to do it soon.

Hi @Marcoslima016,
I encountered the same issue as you. Could you please put a complete example as you mentioned?

Hi @Marcoslima016,
I encountered the same issue as you. Could you please put a complete example as you mentioned?


Release Nagios 1.2.0

What does this PR do?
Release Nagios 1.2.0, bringing support for instance level tags to the check
Motivation
Keep releases up to date.
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
[ ] If PR adds a configuration option, it has been added to the configuration file.

Additional Notes
Anything else we should know when reviewing?

Codecov Report

Merging #2797 into master will increase coverage by 5.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2797      +/-   ##
==========================================
+ Coverage   84.75%   89.83%   +5.08%
==========================================
  Files         659        7     -652
  Lines       37793      433   -37360
  Branches     4539       53    -4486
==========================================
- Hits        32031      389   -31642
+ Misses       4427       30    -4397
+ Partials     1335       14    -1321

Indeed was tested, included a screenshot of working event on original PR and introduced a test case for this. Thanks!


Publishing from other kernels

Is there a chance to publish from other kernel notebooks like:

R : https://github.com/IRkernel/IRkernel
Scala: https://github.com/alexarchambault/jupyter-scala

Right now I have a wp icon in other kernel notebooks but when I press submit button nothing happens.
Thanks in advance.

Hi,
Are you on windows?

Linux Ubuntu

is it working from the command line?
I should remove the gui way as ipython/jupyter changes very fast and
sometimes the changes break all!!.
2016-02-19 14:54 GMT+01:00 Sergey Bushmanov<EMAIL_ADDRESS>
Linux Ubuntu
—
Reply to this email directly or view it on GitHub
https://github.com/Pybonacci/ipy2wp/issues/11#issuecomment-186222158.

From command line:
python ipy2wp.py --xmlrpc-url *** --user *** --password *** --nb 'Breeze.ipynb' --title 'Breeze' --categories python --tags python python: can't open file 'ipy2wp.py': [Errno 2] No such file or directory
nothing posted
python /home/sergey/ipy2wp/ipy2wp/ipy2wp.py --xmlrpc-url *** --user *** --password *** --nb 'Breeze.ipynb' --title 'Feature engineering: Entity name extraction' --categories python --tags python
/home/sergey/anaconda3/lib/python3.5/site-packages/IPython/nbconvert.py:13: ShimWarning: The `IPython.nbconvert` package has been deprecated. You should import from nbconvert instead.
  "You should import from nbconvert instead.", ShimWarning)

draft is posted

Still no joy?

Hi, from the command line you should be able to publish from whatever kernel as the tool is kernel agnostic.
The GUI tool should be deprecated so don't use it more. Maybe I will come back when there is a stable frontend API (jupyter 5???).
BTW, i continue the development here.

Thanks for  prompt reply!
No images whatsoever: should it be inline images sitting in my ./img directiry or print outputs of plots (the latter I suspect lost somewhere due to image 64 encoding?).
Same for cli and R notebook switched to Python kernel for the sake of sending the nb to wordpress hoster.