[BACKLOG-26741] 8.2 Carte status page unable to acess Scaleable Vecto…

…r Graphic (.svg) files

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetJobStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/java/org/pentaho/di/www/WebServer.java

Newly Broken Tests:
org.pentaho.di.www.GetJobStatusServletTest.testGetJobStatus:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetJobStatusServletTest.testGetJobStatus(GetJobStatusServletTest.java:150)

org.pentaho.di.www.GetJobStatusServletTest.testGetJobStatusServletEscapesHtmlWhenTransFound:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetJobStatusServletTest.testGetJobStatusServletEscapesHtmlWhenTransFound(GetJobStatusServletTest.java:114)

org.pentaho.di.www.GetJobStatusServletTest.testGetJobStatusServletEscapesHtmlWhenTransNotFound:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetJobStatusServletTest.testGetJobStatusServletEscapesHtmlWhenTransNotFound(GetJobStatusServletTest.java:84)

org.pentaho.di.www.GetStatusServletTest.testGetStatusServletEscapesHtmlWhenTransFound:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetStatusServletTest.testGetStatusServletEscapesHtmlWhenTransFound(GetStatusServletTest.java:94)

org.pentaho.di.www.GetStatusServletTest.testGetStatusServletEscapesHtmlWhenTransNotFound:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetStatusServletTest.testGetStatusServletEscapesHtmlWhenTransNotFound(GetStatusServletTest.java:71)

org.pentaho.di.www.GetTransStatusServletTest.testGetTransStatus:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetTransStatusServletTest.testGetTransStatus(GetTransStatusServletTest.java:146)

org.pentaho.di.www.GetTransStatusServletTest.testGetTransStatusServletEscapesHtmlWhenTransFound:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetTransStatusServletTest.testGetTransStatusServletEscapesHtmlWhenTransFound(GetTransStatusServletTest.java:111)

org.pentaho.di.www.GetTransStatusServletTest.testGetTransStatusServletEscapesHtmlWhenTransNotFound:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException
	at org.pentaho.di.www.GetTransStatusServletTest.testGetTransStatusServletEscapesHtmlWhenTransNotFound(GetTransStatusServletTest.java:83)

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: -0.2% (click to expand)
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.4% :small_red_triangle_down:
Line Change: -3.8% :small_red_triangle_down:

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.www.GetJobStatusServlet

Branch Change: -53.6% :small_red_triangle_down:
Complexity Change: -28.3% :small_red_triangle_down:
Instruction Change: -56% :small_red_triangle_down:
Line Change: -57% :small_red_triangle_down:
Method Change: -12.5% :small_red_triangle_down:

org.pentaho.di.www.GetStatusServlet

Branch Change: -15.3% :small_red_triangle_down:
Complexity Change: -12.2% :small_red_triangle_down:
Instruction Change: -67.8% :small_red_triangle_down:
Line Change: -71.8% :small_red_triangle_down:
Method Change: -38.4% :small_red_triangle_down:

org.pentaho.di.www.GetTransStatusServlet

Branch Change: -30% :small_red_triangle_down:
Complexity Change: -15.1% :small_red_triangle_down:
Instruction Change: -38.5% :small_red_triangle_down:
Line Change: -44.1% :small_red_triangle_down:
Method Change: -12.5% :small_red_triangle_down:

org.pentaho.di.www.SlaveServerJobStatus

Complexity Change: -3.3% :small_red_triangle_down:
Instruction Change: -1.2% :small_red_triangle_down:
Line Change: -2.4% :small_red_triangle_down:
Method Change: -4% :small_red_triangle_down:

org.pentaho.di.www.StatusServletUtils

Branch Change: -10% :small_red_triangle_down:
Complexity Change: -14.3% :small_red_triangle_down:
Instruction Change: -42.6% :small_red_triangle_down:
Line Change: -48.5% :small_red_triangle_down:
Method Change: -50% :small_red_triangle_down:

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UIEERepositoryUser

Branch Change: -8.3% :small_red_triangle_down:
Complexity Change: -6.7% :small_red_triangle_down:
Instruction Change: -0.9% :small_red_triangle_down:
Line Change: -3.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java

:heavy_exclamation_mark: 'else' child has incorrect indentation level 8, expected level should be 10. at Line: 344

line 198 in GetJobStatusServlet.java is past the normal 80 character limit. seems like it's the same for others @ddiroma

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetJobStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/java/org/pentaho/di/www/WebServer.java

Newly Broken Tests:
org.pentaho.di.trans.step.mqtt.MQTTStreamSourceTest.testMultipleTopics:
:red_circle: junit.framework.AssertionFailedError
junit.framework.AssertionFailedError
	at org.pentaho.di.trans.step.mqtt.MQTTStreamSourceTest.getQuickly(MQTTStreamSourceTest.java:253)
	at org.pentaho.di.trans.step.mqtt.MQTTStreamSourceTest.testMultipleTopics(MQTTStreamSourceTest.java:141)

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.core.injection.bean.BeanInjector

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.step.BaseStep

Branch Change: -0.5% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.7% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:
Method Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.step.StepStatus

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.6% :small_red_triangle_down:
Instruction Change: -1.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.7% :small_red_triangle_down:
Complexity Change: -3% :small_red_triangle_down:
Instruction Change: -0.7% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -2.1% :small_red_triangle_down:
Complexity Change: -2.1% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.2% :small_red_triangle_down:
Complexity Change: -2.6% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

Note: The list of offending coverage changes extends beyond the presented results and was truncated to prevent reaching the comment size limit.

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java

:heavy_exclamation_mark: 'else' child has incorrect indentation level 8, expected level should be 10. at Line: 345

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetJobStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/java/org/pentaho/di/www/WebServer.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Branch Change: -2.6% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:

org.pentaho.di.www.GetJobStatusServlet

Branch Change: -1.6% :small_red_triangle_down:
Complexity Change: -4.8% :small_red_triangle_down:

org.pentaho.di.www.GetStatusServlet

Complexity Change: -1.6% :small_red_triangle_down:
Instruction Change: -0.8% :small_red_triangle_down:
Line Change: -2.1% :small_red_triangle_down:

org.pentaho.di.www.GetTransStatusServlet

Complexity Change: -2.3% :small_red_triangle_down:

org.pentaho.di.www.StatusServletUtils

Branch Change: -10% :small_red_triangle_down:
Complexity Change: -14.3% :small_red_triangle_down:
Instruction Change: -42.6% :small_red_triangle_down:
Line Change: -48.5% :small_red_triangle_down:
Method Change: -50% :small_red_triangle_down:

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UISecurityRole

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java

:heavy_exclamation_mark: 'else' child has incorrect indentation level 8, expected level should be 10. at Line: 345

@NJtwentyone @pentaho/r2d2 please don't merge PRs with check style violations.
This is a really bad practice that won't bring us any good.

@pentaho-lmartins This checkstyle violation was bogus. I noted that in the PR. The line in question had correct indentation...it was a wingman glitch.


Remove /organisations endpoint

The /organisations endpoint was used by the frontend application to render the select box of organisations. The OrganisationRegistry augmented the organisation data for the select box to display Ministerial departments on top of the list. It was added in https://github.com/alphagov/rummager/pull/103 (May 2013).
Frontend stopped using the endpoint since moving to "Unified Search", and the call to rummager was removed in PR https://github.com/alphagov/frontend/pull/601. Kibana confirms that the endpoint has not been used in the last 30 days:

This PR removes the endpoint and associated code. This will allow us to further refactor the Document class and better separate indexing and searching in rummager.
Trello: https://trello.com/c/oCWEBqER
Thanks to @jamiecobbett for explaining the history of this feature to me.

Insta-closing this to wait for an update to gds-api-adapters.


[Bug]: Invalid virtaulenv created bypy_venv

What happened?
Based on my testing, in 0.5.0 python 3.11 works in general, but the virtual environment created by py_venv still had the same issues as reported in #159, https://github.com/aspect-build/rules_py/issues/159#issuecomment-1931446119.
In 0.6.0 (thanks Alex for creating a release for me to test), the python interpreter is able to start, but neither packages from pip nor in the bazel workspace can be loaded. The first_party.pth file looks like this
../../../..
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../
../../../../

The same behavior is also reported here https://github.com/aspect-build/rules_py/pull/226#issuecomment-1833958996.
I don't know if this will be fixed by the rust venv work. I decided to create this issues since the recurring issue is now spread across multiple threads. It might also be helpful to have this feature tested in CI.
Version
Development (host) and target OS/architectures: Arch Linux x86
Output of bazel --version: 6.5.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:

rules_python: 0.31.0
aspect_rules_py: 0.5.0 / 0.6.0

Language(s) and/or frameworks involved:

Python 3.11

How to reproduce
No response
Any other information?
No response

0.6.0 is the commit before the refactor. I can't repo at HEAD
$ bazel run //:venv
INFO: Analyzed target //:venv (3 packages loaded, 2207 targets configured).
...
INFO: Running command line: bazel-bin/venv

$ /Users/matt/workspace/rules_py/.venv/bin/python
Python 3.11.7 (main, Jan  7 2024, 23:23:46) [Clang 16.0.3 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import site
>>> site.getsitepackages()
['/Users/matt/workspace/rules_py/.venv/lib/python3.11/site-packages']

$ .venv/bin/python
Python 3.11.7 (main, Jan  7 2024, 23:23:46) [Clang 16.0.3 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import site
>>> site.getsitepackages()
['/Users/matt/workspace/rules_py/.venv/lib/python3.11/site-packages']

Generated .pth file contains full paths when generating outside the sandbox for action,
$ cat .venv/lib/python3.11/site-packages/venv.venv.pth
/private/var/tmp/_bazel_matt/442be3b1483cc77971c0d7bfbd105b2d/execroot/aspect_rules_py/bazel-out/darwin_arm64-fastbuild/bin/venv.runfiles/.
/private/var/tmp/_bazel_matt/442be3b1483cc77971c0d7bfbd105b2d/execroot/aspect_rules_py/bazel-out/darwin_arm64-fastbuild/bin/venv.runfiles/aspect_rules_py

Oh that looks much better. Do you guys have an ETA for the refactor?

It's landed at HEAD, so you could dep on that instead.
We are currently working through releasing the binaries needed for the toolchains, they are currently just committed to the repo here, so if you don't mind that can give it a try now, no rust toolchain should be needed.

I just tested v0.7.1 and it appears to be working. It would be nice to document glibc requirement somewhere, as it won't run on centos7 (which is reasonable at this point).


Cargo shuttle should just use FTL destinations

It's added now. You can probably just re-use the centcomm map as the cargo map.

Also now that I think about it we should probably just make it an unpaused map and have people FTL there. Cargo shuttle console I think would probably be removed entirely.


How to find msgGlobalData msgID from snmpV3 inform

We are using pysnmp to transform snmpV3 informs from devices and are successfully processing the messages.
Problem:
If alarms do not arrive in the correct order for example linkUp(Clear alarm) then linkDown(Set alarm), since snmpV3 does not support alarm timestamp created by the device we cannot determine the order therefore if the device was showing a fault and then subsequently has been cleared.
Scenario:
If the device fails to sends the snmpV3 inform linkDown to the snmp-collector because of network issues or it is being restarted, the device will retry in 30 seconds. if the snmp-collector comes back up within the 30 seconds and the device is cleared within this time, the device will send a linkUp to the snmp-collector which it will receive successfully. The device will then send the linkDown to the snmp-collector after the linkUp has been processes.
Possible solution:
The unique msgGlobalData msgID of the snmpV3 inform remains the same for each repeated attempt of sending the alarm inform and therefore we can use this to order the sequence of when the alarm was created from a device.
Question
How do we obtain the msgGlobalData msgID from the snmpV3 inform message from a device using pysnmp module or is there another way to correctly order the snmpV3 inform messages?

The statement of "snmpV3 does not support alarm timestamp created by the device" is not true. SNMP INFORM message body does contain the timestamp in PDU.
Thus, in the scenario you described the manager side can tell from that PDU at what time the INFORM message was constructed and decide how to respond in the best way. The solution or follow-up question you posted were not applicable then.


Add parens on lru_cache

Summary
Very small fix on the usage of @lru_cache, where not putting parentheses gives a TypeError: Expected maxsize to be an integer or None. This leads to errors when trying to run tests locally.
Below is a copy of the error logs:
    from core.models.model_list import PositionDynamics
src/core/models/model_list.py:3: in <module>
    from core.models.model import ActuatorModel, EnvironmentModel, SensorModel, MODEL_TYPES
src/core/models/model.py:3: in <module>
    from core.state.statetime import StateTime
src/core/state/statetime.py:4: in <module>
    from core.models.derived_models import DERIVED_MODEL_LIST
src/core/models/derived_models.py:3: in <module>
    from utils.astropy_util import get_body_position
src/utils/astropy_util.py:14: in <module>
    def get_body_position(time: int, body: BodyEnum) -> Tuple[float, float, float]:
/opt/anaconda3/lib/python3.7/functools.py:490: in lru_cache
    raise TypeError('Expected maxsize to be an integer or None')
E   TypeError: Expected maxsize to be an integer or None

Just realized, the default size of the cache, if not specified, is 128. Maybe instead we can set it to something else.

Just realized, the default size of the cache, if not specified, is 128. Maybe instead we can set it to something else.

Good point— I believe it's unbounded in size when maxsize=None, but it seems that 128 is indeed the default value. Not sure what criteria we would evaluate to decide whether or not to change this.

Also, taking a look at the docs for lru_cache, it seems that the non-parentheses version is used when no arguments are passed.

Oh! Well, we only need to really reuse info that's "repeated" and the rest is not necessary. The amount of "repeated calls" has to do with how much we truncate the data points by. So for example, it's been hardcoded now:
        r_mo = np.array(get_body_position(10 * t // 10, BodyEnum.Moon))
        # Sun to origin
        r_so = np.array(get_body_position(10 * t // 10, BodyEnum.Sun))

to be truncate to the nearest 10 (i.e. 10, 14, and 18 map to the same). The cache probably be some really small number then, right? Once we get past that then those values won't be needed anymore.
I guess, no real reason to change anything right now I think. I can close this for now.


installation problem

Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for mdanter/ecc dev-master#89640a0873e212a723212e7f374f973b7d1fe6f6 -> satisfiable by mdanter/ecc[dev-master].
- mdanter/ecc dev-master requires phpunit/phpunit ~4.1 -> no matching package found.

This appears to have been happening for some time - though should be fixed now. Please advise if the problem persists still.
(Btw - and I'm telling everyone - it might be worth looking into https://github.com/Bit-Wasp/bitcoin-php. bitcoin-lib-php will not see new features or improvements)

Closing due to inactivity


What is the difference between redlock and simple-redlock

which one is recommended to use?

Hi @Diluka, Looks like I forgot to update README...
Basically, I recommend using @anchan828/nest-redlock. The main logic is to use node-redlock because you can expect extensive documentation, number of users, and bug fixes.
The simple-redlock was first made for convenience in my project's specification. This will be removed as it will only cause confusion.

Removed simple-redlock from repo. If you use it, please use @anchan828/nest-redlock.😀


resize error

Hi ZFTurbo,
Thanks very much for your great work！ but I encountered an error when resize the image. Many thanks for your help in advance.
the error is as below:

I have solved this problem. This error is caused by the 1 channel input(16,32,32).  The default input expects a 3D image  in color mode(None, None,None,3). So we need to change the  code on  ../volumentations/augmentations.functionals.py  in line 138 as below:

type =1 for color(3channel) mode input shape: (None, None, None, 3) ,
type=0 for grayscale(1 channel) mode input shape: (None, None, None)


Cannot remove taskId from the queue shard conductor.test.QUEUE.HTTP.a

Hi @apanicker-nflx @kishorebanala ,
We are facing a lot of these warnings in conductor.
cannot add 2b5a87f2-c97f-4294-b883-57869f5fd65d from the queue shard conductor.test.QUEUE.HTTP.a
cannot remove b3ce517b-eeef-474f-94e9-3468e6b0927b from the queue shard conductor.test.QUEUE._deciderQueue.a
cannot add b3ce517b-eeef-474f-94e9-3468e6b0927b from the queue shard conductor.test.QUEUE._deciderQueue.a

We are running Redis cluster setup. Also, enabled workflow execution locking a few days back but that did not help much.
We also see the deciderQueue getting piled up and we run Sweeper. But running Sweeper also does not help much because sweeper also does pop from redis and I see that in pop itself these exception comes.
Can you please help us with this?
Does it cause any harm in running workflows?
We also see PENDING_WORKFLOWS set also gets piled up and we terminate all old workflows based on the workflow creation time.

@smukil Can you comment on these errors? These errors seem to be exceptions thrown by dyno queues.

This is also the reason for Sweeper not working correctly.
116101-2020-03-31 23:54:06 DEBUG WorkflowSweeper:82 - Sweeper's current deciderqueue size: 2806.
116102:2020-03-31 23:54:06 DEBUG WorkflowSweeper:84 - Sweeper retrieved 0 workflows from the decider queue.```
It never receive the workflow from dynoQueue because it is not able to pop.


Script Idea: Spotify Song data scrapper

Aim

Let's user scrape Songs data from Spotify.

Details

This script will help anyone scrape songs data such as the artist name, song name, album name and so on.
Can come in handy when a person wishes to build a song recommender using real songs from Spotify.

P.S -  I am a GSSOC'21 Participant.

@AM1CODES We already have an assigned issue for this #607


🌐 Add Spanish translation for docs/es/docs/fastapi-people.md

This is the Spanish translation for the people page.
Add Spanish translation for docs/es/docs/fastapi-people.md

Version >= 0.108.0

Looks good to me (I'm a native Spanish speaker)

Hi, thanks for the help @pablocm83 🤓
I noticed that most of the internal links don't work because those files haven't been translated yet. There's a new section in Contributing that includes some tips for translation (https://fastapi.tiangolo.com/contributing/#translation-specific-tips-and-guidelines). I hope this helps. Thanks!

📝 Docs preview for commit 3eb04260c2dfe6a621dbe163f9697efb157002fa at: https://aad0db50.fastapitiangolo.pages.dev

@alejsdev internal links ready.
thanks

📝 Docs preview for commit b4c8ba1c79559a33e1a8025d0e5f7f88f15441f3 at: https://c9476a29.fastapitiangolo.pages.dev


Feature Request: will you consider adding snopt?

the optimizer here (https://github.com/ghliu/snopt) seems a good speed up. Will you incorporate it into your system?

Hey, thanks for the link. I won't include it directly into torchode, but it should not be too difficult to port the code to run on top of torchode instead of torchdiffeq.


Segfault from pyo3 trying to call PyBuffer::drop after Python interpreter already finalized

Bug Description
I got a segfault in my Python library from pyo3 trying to call PyBuffer::drop after the Python interpreter was already finalized. (See https://github.com/kylebarron/arro3/issues/230).
Trying to acquire the GIL made this assert_ne! panic.
The similar C++-based Python library pyarrow performs a check for whether the interpreter is initialized before trying to drop the value.
I already worked around this in https://github.com/kylebarron/arro3/pull/231 by checking if the interpreter is initialized before calling drop.
But perhaps this is something that should be fixed upstream in pyo3.
Steps to Reproduce
git clone https://github.com/kylebarron/arro3
cd arro3
git checkout 692e40bd557b9a5402bfc37df28c2b9d11afed53
# Install uv if necessary
# https://docs.astral.sh/uv/
uv run maturin develop -m arro3-core/Cargo.toml
uv run maturin develop -m arro3-compute/Cargo.toml
uv run maturin develop -m arro3-io/Cargo.toml
# Enter a Python environment
uv run python

Then run
import numpy as np
import pyarrow as pa
from arro3.core import Array

numpy_arr = np.array([0, 1, 2, 3], dtype=np.float64)
arro3_arr = Array(numpy_arr)
pyarrow_arr = pa.array(arro3_arr)

Then exit the Python interpreter, e.g. with control+D. You should see a segfault backtrace. This appears to be consistently reproducible for me.
Backtrace
thread '<unnamed>' panicked at /Users/kyle/.cargo/registry/src/index.crates.io-6f17d22bba15001f/pyo3-0.22.3/src/gil.rs:198:21:
assertion `left != right` failed: The Python interpreter is not initialized and the `auto-initialize` feature is not enabled.

Consider calling `pyo3::prepare_freethreaded_python()` before attempting to use Python APIs.
  left: 0
 right: 0
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at core/src/panicking.rs:221:5:
panic in a function that cannot unwind
stack backtrace:
   0:        0x11889c274 - _PyInit__core
   1:        0x1188b4e28 - _PyInit__core
   2:        0x11889a8ec - _PyInit__core
   3:        0x11889d3f0 - _PyInit__core
   4:        0x11889cfa0 - _PyInit__core
   5:        0x11889de6c - _PyInit__core
   6:        0x11889d870 - _PyInit__core
   7:        0x11889c738 - _PyInit__core
   8:        0x11889d560 - _PyInit__core
   9:        0x1188de260 - _PyInit__core
  10:        0x1188de2d8 - _PyInit__core
  11:        0x1188de450 - _PyInit__core
  12:        0x1187a5620 - _PyInit__core
  13:        0x128018cf0 - __ZNSt3__120__shared_ptr_emplaceIN5arrow12_GLOBAL__N_117ImportedArrayDataENS_9allocatorIS3_EEE16__on_zero_sharedEv
  14:        0x12801ac3c - __ZN5arrow12_GLOBAL__N_114ImportedBufferD1Ev
  15:        0x128019d14 - __ZN5arrow9ArrayDataD2Ev
  16:        0x1281a4a00 - __ZN5arrow12NumericArrayINS_10DoubleTypeEED1Ev
  17:        0x105fc5b34 - __ZL36__pyx_tp_dealloc_7pyarrow_3lib_ArrayP7_object
  18:        0x105226690 - _dictkeys_decref
  19:        0x105228e6c - _dict_dealloc
  20:        0x10523938c - _module_clear
  21:        0x1053364b8 - _gc_collect_main
  22:        0x1053358f8 - __PyGC_CollectNoFail
  23:        0x10530f914 - _finalize_modules
  24:        0x10530ec9c - _Py_FinalizeEx
  25:        0x1053346c0 - _Py_RunMain
  26:        0x1053350c4 - _pymain_main
  27:        0x105335164 - _Py_BytesMain
thread caused non-unwinding panic. aborting.

 Did you use a virtualenv?
Installed Python via pyenv. Virtualenv provided by uv.
Additional Info
No response

It feels to me like we should consider changing PyBuffer::drop to be more like Py<T>::drop where we don't try to automatically acquire the GIL (this can in theory lead to deadlocks) and instead defer the release until we know we have the GIL? 🤔

In my own case I assume that the only instance where the interpreter isn't initialized is when the Python session is ending, and then I assume if I "leak" the memory, it'll still get cleaned up when the process closes.

True; it's probably correct to add a check that the interpreter is initialized somewhere in that machinery. I think it's complicated a bit by the possibility of finalizing and restarting the interpreter (or a subinterpreter), but we don't really support that anyway (yet, probably not for a while).


Issues with golang 1.13 and windows 10 x64

Seems like I can't 'get' it with golang 1.13 on Win10x64. Installed 1.12rc1 and testing it now, but seems more promising. Which exact version was used? I am new to golang and would like some more instructions in the readme for the first steps.  Thanks in Advance!
Update: 1.12rc1 didn't work either. Probably Windows related stuff, so I stopped bothering. I switched to WSL with Debian for now. Hope I can get it running now. (I now I should have used linux in the first place haha)
Update 2 Everything works now with Windows Subsystem for Linux and Debian.

Thanks for the issue! We tested with golang 1.12. I will close the issue for now since it seems that you got it working :)


Add golden tests for disabled CupertinoTextField

Add golden tests for disabled CupertinoTextField. Related: #32974

Tests for PR #32974 should be part of that PR.
It's possible to verify that a paint method is clipping correct without writing a golden image test. See for example: https://github.com/flutter/flutter/blob/master/packages/flutter/test/material/ink_paint_test.dart#L49


Code in images?

[Enter feedback here]
This page instructs the user to insert JSON into a box.  The JSON presented is not in text, but is an image.  The user cannot copy the JSON, but must OCR it or retype it.  Code which isn't copyable defeats the point of presenting code.

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

ID: 002a1ee9-dc9f-8dc1-6f3c-ef23297b83d4
Version Independent ID: 82565c7e-b264-94f5-8708-d4757482e133
Content: Use view formatting to customize SharePoint
Content Source: docs/declarative-customization/view-formatting.md
Service: sharepoint-online
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Can you clarify which area you can't copy? I don't see any where where it says to copy the JSON (i may be missing it). But the JSON that is in the photos, is the default value when you format the column, there would be no need to copy it as it already exists.

I apologize - this is probably a non-issue.  Yes, it's those boxes' content I was trying to copy, for whatever reason.  I'll go ahead and close as, on reflection, it's not a good issue to have raised.

Closing as clarified 🚀


Can't install rack

Hello :)
I think the problem is on the rack build itself

$ brew install rack

==> Using the sandbox
==> Cloning https://github.com/rackspace/rack.git
Updating /Users/azr/Library/Caches/Homebrew/rack--git
==> Checking out tag 1.2
==> script/build rack
Last 15 lines from /Users/azr/Library/Logs/Homebrew/rack/01.build:
2016-10-06 16:58:07 +0200

script/build
rack

Building rack!
# github.com/rackspace/rack
/usr/local/Cellar/go/1.7.1/libexec/pkg/tool/darwin_amd64/link: -X flag requires argument of the form importpath.name=value

READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/Homebrew/homebrew-core/issues

ran brew update twice, brew doctor is quiet.
Cheers !
Edit: go getting github.com/rackspace/rack and running the build command did the same, made an issue : https://github.com/rackspace/rack/issues/446

This issue was moved to Homebrew/homebrew-core#5630


Added a C# event for handling pointer events to the input system

This is so we can have global pointer event listeners that are not scripts. This is required for the upcoming PR addressing #3957 .
Fixes #4535 .

This feels potentially confusing. All our other events are handled by Unity, but for one specific type we have a duplicate C# event. Questions could come up about why we don't have C# events for everything.

This feels potentially confusing. All our other events are handled by Unity, but for one specific type we have a duplicate C# event. Questions could come up about why we don't have C# events for everything.

Any suggestions on alternative ways of addressing the issue this PR deals with or on avoiding the potential confusion?

There's also the other question of how events are consumed.
using the event data we have currently I suppose we can raise the event, but there's still a question of who gets the events first?

The C# objects or the scene objects?
Are the C# objects technically global listeners? (As they could get and consume input before scene elements)
Lots more questions here as well.

using the event data we have currently I suppose we can raise the event, but there's still a question of who gets the events first?
The C# objects or the scene objects?
Are the C# objects technically global listeners? (As they could get and consume input before scene elements)

I can answer you those two: first the scene objects and yes, C# objects are global listeners. We can document that better, I don't see it as an obstacle for this making it in.

Lots more questions here as well.

I can only answer the questions I know about.

One easy work around is having a global listener object that just reacts to input, but that's not ideal either.

It is indeed far from ideal.

I really don't like this solution but I can't think of a better alternative. The event system has backed us into a weird corner.

This feels potentially confusing.

@keveleigh I think as long as we drown users in documentation we'll be okay. It's only confusing when you don't get the motivation.

@luis-valverde-ms Just had a thought. How about creating a global listener object that's hidden in the scene heirarchy, a la the AsyncCoroutineRunner? That object could subscribe via unity's event system, then relay events via C# delegates, which you can access via a singleton... and this suggestion is so awful and gross that I regret even typing it up. Never mind, please don't do this.

It looks like this PR hasn't had progress in a couple days, do we have a clear set of next steps? @luis-valverde-ms it looks like you have two different approaches to add ability for non-game object to listen to pointer events. This PR and then #4605. Should we close this PR and then complete #4605 instead?

Closed at @julenka 's suggestion as it seems we will go for the option in #4605


build(deps): remove unneeded dependencies

PR summary
This PR removes the "json" and "cloudant-client" dependencies since they are no longer needed by the code.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Memory Explosion

Hi I run the train.py under following env:

Ubuntu 16.04, docker env: pytorch 1.8.1 with GPU support
Machine Memory: 32G, swap area size: 12GB; GPU: GF RTX 2080 Ti 12 GB
Num of epochs:50, batch_size:1, load_size: 256, input_size:224, with other parameters default/unchanged
Trainning dataset consists of 150 non-defect images with 1300*2300 resolution
Test dataset consists of 100 defect and non-defect images, with ground truth images where a pure black for non-defect images and write musk for the defect areas for defect images

Problem: After epoch 11, the machine memory (not GPU memory) takes up to 40GB,  and at epoch 15, the usage of machine memory would be over 50GB, then the system would kill the training process.
Any ideas why this happen, and how to fix?

As I understood, PatchCore doesn't need more than 1 epochs.
because data from 2nd ~ inf epochs, it is duplicated from first epochs' data.
It is named as training, but actually it is samping....

I'm also facing the same issue. What is the potential solution?


Added error logs for if it fails to create or update keycloak user

Added error logs for if it fails to create or update keycloak user
Fixes https://github.com/fabric8-services/fabric8-wit/issues/1739

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-1

Codecov Report

Merging #1738 into master will decrease coverage by 0.13%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1738      +/-   ##
==========================================
- Coverage   56.95%   56.81%   -0.14%
==========================================
  Files         128      128
  Lines       14948    14962      +14
==========================================
- Hits         8513     8501      -12
- Misses       5756     5782      +26
  Partials      679      679

Impacted Files
Coverage Δ

controller/login.go
11.11% <0%> (-2.05%)
:arrow_down:

remoteworkitem/jira.go
76.19% <0%> (-23.81%)
:arrow_down:

remoteworkitem/scheduler.go
46.93% <0%> (-14.29%)
:arrow_down:

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 fc5e7c1...39a3a93. Read the comment docs.

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-2

@kishansagathiya  , Error logs might make folks feel that the tokens weren't generated right, could you please add a WARNING and not an ERROR.

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-3

[test]

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-4

Let's just proxy "generate" to Auth when https://github.com/fabric8-services/fabric8-auth/pull/168 is merged.

[test]

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-5

[test]

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-6

@kishansagathiya snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1738-8


admin: security council multisig

The security council will be a multisig: it needs different threshold for signatures depending on the function to call on the protocol handler.
Benchmark existing solution:

braavos -> not possible at the moment
argent -> not possible at the moment
snapshot
custom implementation

to fix your trouble check this solution click
maybe this will solve your problem.


getting a IllegalBlockSizeException

Hi, when I write a long string (>600 bytes) , the exceptiong throws.
java.lang.IllegalArgumentException: Unsupported value: javax.crypto.IllegalBlockSizeException: input must be under 256 bytes
any idea? thank you!

Same problem here!

Thanks for reporting
I will check

Same problem here.

I will update implementation.

generate random KEY and encrypt it with RSA.
encrypt payload with AES, using KEY

Likely, it would be breaking change 🎉

Added basic implementation #8


Create About page with the licenses of all third-party icons used on the website

Description
We are currently using some icons that is under the CC 4.0 License. The about page will contain a disclaimer that we are using these icons and a link to the CC 4.0 international license.
URL: https://creativecommons.org/licenses/by/4.0/deed.en

I would prefer it to be under "legal" or "privacy" option.


codecov is broken

Codecov hasn't been updated by our build script since February 1 when we transitioned from v3 to v4 of the GitHub action (https://github.com/rs-station/reciprocalspaceship/commit/4cd04712b936ef67bcac3896aeca6e6cb59a4a4d). I suspect this is because codecov disallowed tokenless uploads in v4.

fixed by #273


Add Tempo Changer support

If a layer is named "Tempo Changer", then the song's tempo is set to the value of any note that's played on said layer.

The tempo changer is unofficial and will fully change within the near future. Additionally, this isn't really fit for NBS.js, and should be the responsibility of the application utilizing the library.


Hot Maze onboarding: empty QR

Draw the 3 corners of an (empty) QR-code.
Paint "DROP FILE HERE" and upload button in its center.

After drop and upload, the freshly generated QR-code should take the same area as the empty stub.

Done  (except the final real QR looks a little smaller than the initial empty one).


@conda-forge-admin add bot automerge

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pymultinest-feedstock#11.


[SPARK-15907] [SQL] Issue Exceptions when Not Enough Input Columns for Dynamic Partitioning

What changes were proposed in this pull request?
CREATE TABLE table_with_partition(c1 string)
PARTITIONED by (p1 string,p2 string)

INSERT OVERWRITE TABLE table_with_partition
partition (p1='a',p2)
SELECT 'blarr3'

In the above example, we do not have enough input columns for dynamic partitioning. The first input column is already taken as data columns. This PR is to issue an exception in this scenario.
For your reference, below is the behavior of Hive:
hive> insert overwrite table t9 partition(p1='1',p2) select col from t1;
FAILED: SemanticException [Error 10044]: Line 1:23 Cannot insert into target table because column number/types are different 'p2': Table insclause-0 has 2 columns, but query has 1 columns.

How was this patch tested?
Added a test case and fixed an existing test case

Test build #60373 has started for PR 13628 at commit 4c49112.

There is a better fix in https://github.com/apache/spark/pull/12313. Let me close it.

Test build #60373 has finished for PR 13628 at commit 4c49112.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60373/
Test PASSed.

Merged build finished. Test PASSed.


Fixes Universal Transformer not converging bug

fixes issue discussed in https://github.com/tensorflow/tensor2tensor/issues/1191

Thanks to both @rllin-fathom and @MostafaDehghani -- I can take both the PRs :)
Will merge in both of them shortly.
Many thanks to both of you for fixing this the right way!


Fix flag bypass for editing description

While editing an MR or issue description if any flag different from -m and -F is used the editor is not opened. Add the --linebreak to this list of "accepted" flags to be used for editing the description.
Also, this MR bring other lateral fixes and improvements related to code organization, make sure to check each commit separately.

Of course, I forgot to update the test.

@prarit the code behavior didn't change. I just updated the test code for including the description editing through file (-F) and fixed a variable declaration.


Can not edit existing SCIM attributes mapping

Describe the issue:
Can not edit existing SCIM attributes mapping.
https://user-images.githubusercontent.com/39077751/164958462-63f93592-c88e-4de7-acc8-092a149d6840.mov
How to reproduce:

Login to console
Navigate to the Manage-> attribute-> SCIM attribute
Select enterprise schema
Search for Lite User from existing list
Click edit
Select some other claim to map and click update

Expected behavior:
Should be successfully update the existing attribute mapping.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.12.0-alpha20
OS: Mac
Database: MSSQL
Userstore: JDBC

Optional Fields
Related issues:

Suggested labels:

Fixed "Something went wrong" issue with https://github.com/wso2/identity-apps/pull/2972


About test results

Hello!,Thank you for your great job!
I conducted the test according to the steps, but the test result failed to reach MOTA81.4 and IDF180.4, is the parameter not set correctly?Or is there some post-processing? Could you please help me to answer if I am wrong somewhere, thank you very much~

Hello,Thank you for your attention!
Please make sure the '--default-parameters' is True, and run 'tracker/tools/interpolation.py' when you get results from the tracker.


Run linting&formatting on commit, delete unused template code

add lint & format step on commit

@vercel redeploy


⚠ PassThePopcorn has degraded performance

In a5c3126, PassThePopcorn ($PTP) experienced degraded performance:

HTTP code: 502
Response time: 5724 ms

Resolved: PassThePopcorn performance has improved in bfcdc46 after 5 minutes.


fix(form): add correct render props ref type

This fixes the problem we talked about at Lisci.

It infers automatically based on the type. I don't know if you have access to Lisci anymore, but when I fixed locally my node_modules, it inferred correctly. You do not need to pass generic, it can infer based on the passed value. Also if you do not pass it, you have default value as well. :)

I merged it and try it currently locally. For me it does not work in this way. Are you sure, that you tried it with an non HTMLElement field?

Was the new version released? I updated to the latest @nx-kit/form, which is 0.1.23. I still see this in ControlledInput.types.ts in our node_modules:
import React from 'react';
export declare type RenderProps = {
    onBlur: () => void;
    onChange: (value: any) => void;
    value: React.Key;
    hasError: boolean;
    ref: React.Ref<HTMLElement>;
};
export declare type ControlledInputProps = {
    name: string;
    render: (props: RenderProps) => React.ReactElement;
    validation?: any;
};

I found out that I forgot one more thing in this PR. Update other stuff to pass the <T>.
import React from 'react';
export declare type RenderProps<T extends HTMLElement = HTMLElement> = {
    onBlur: () => void;
    onChange: (value: any) => void;
    value: React.Key;
    hasError: boolean;
    ref: React.Ref<T>;
};
export declare type ControlledInputProps<T> = {
    name: string;
    render: (props: RenderProps<T>) => React.ReactElement;
    validation?: any;
};

and
export declare const ControlledInput: ({ name, render, validation }: ControlledInputProps<T>) => JSX.Element;

I can use our MultiSelect with this:
export const MultiSelectField = forwardRef<HTMLSelectElement | null, MultiSelectFieldProps>(, which in your branch was previously export const MultiSelectField = forwardRef<HTMLElement | null, MultiSelectFieldProps>(. I removed the @ts-ignore and it works correctly. If I change the type back, it yells.

No, I had a little trouble with updating all react-aria packages. I will let you know here, if it's released.
But this looks wrong: export declare const ControlledInput: ({ name, render, validation }: ControlledInputProps<T>) => JSX.Element;
I have it now like this: export declare const ControlledInput: <T extends HTMLElement = HTMLElement>({ name, render, validation }: ControlledInputProps<T>) => JSX.Element;

@pegak It's now published on npm. Please update all @nx-kit libs. It should work like this now:
<Form.ControlledInput<HTMLSelectElement>>


[Epic] Multi-party approval

Epic
Multi-approval Approval

Task
Multi approval approval

Epic Title
Multi-Party Approval #76

User Story Title
Multi-party approval

Statement
As a user of NEXT, I want to define the eligible roles so I can enable multi-party approvals

Description

Business Need (the why):
multi-party approval functionality

design conversation needs to happen - documentation needs to come out of it.

This isn't a "true epic" in that it has no child stories.  There are other Epics related to Multi-party approval so we will use those instead.


Not able to downgrade the LocksmithAccessibleOption from "unlock" to "afterFirstUnlock"

In earlier version of my app the LocksmithAccessibleOption was default. Now I want to change it to "afterFirstUnlock". for that I added following code in my protocol
var accessible: LocksmithAccessibleOption? {return .afterFirstUnlock}
But after using this I'm not able to either delete or access the keychain item at all.
Help me here please.

I encounter this also. What I notice is that the keychain that was stored using the "unlock" accessible option can't be access or delete when you change to "afterFirstUnlock"
What I did is first check if data already exists using the "afterFirstUnlock" then if not I'll pull back by checking using the "unlock" accessible options. If the data is stored in the "unlock" accessible options I'll copy it over to "afterFirstUnlock" and delete the old data to avoid redundancy

"first check if data already exists using the "afterFirstUnlock" then if not I'll pull back by checking using the "unlock" accessible options. If the data is stored in the "unlock" accessible options I'll copy it over to "afterFirstUnlock" and delete the old data to avoid redundancy" -- > How I'm suppose to handle that. I'm using the protocol based approach. I'm attaching my code and please let me know how I can handle that:
private let UsernameKey = "com.xxxxx.usernameKey"
private let KeyKey = "com.xxxxx.keyKey"
private let Service = "xxxxx"
private let AuthAccount = "xxxxxx-Authorization"
struct Autorization {
let username: String
let key: String

private struct Access: ReadableSecureStorable, DeleteableSecureStorable, GenericPasswordSecureStorable {
    //var accessible: LocksmithAccessibleOption? {return .afterFirstUnlock}
    var service: String { return Service }
    var account: String { return AuthAccount }
}

}
extension Autorization {
static func readCurrentAuthFromKeychain() -> Autorization? {
    let access = Access()
    let result = access.readFromSecureStore()
    guard let data = result?.data, let username = data[UsernameKey] as? String, let key = data[KeyKey] as? String else { return nil }
    return Autorization(username: username, key: key)
}

static func deleteCurrentUserFromKeychain() throws {
    let access = Access()
    try access.deleteFromSecureStore()
}

}
extension Autorization: CreateableSecureStorable {
var data: [String : Any] {
return [
UsernameKey: username as AnyObject,
KeyKey: key as AnyObject
]
}
}
extension Autorization: GenericPasswordSecureStorable {
//var accessible: LocksmithAccessibleOption? {return .afterFirstUnlock}
var service: String { return Service }
var account: String { return AuthAccount }
}

I'm not sure how can I change your code, but here is part of my code:
In a helper file:
private let accessible = LocksmithAccessibleOption.afterFirstUnlock
private func loadData(forUsername username: String) -> [String: Any]? {
    // case1: try if we can get the value when using "accessibleOption"
    // case2: try using the default method, if we can find value
    // case3: we didn't have saved data for this username
    if let dataTemp = Locksmith.loadDataForUserAccount(userAccount: username, accessibleOption: accessible) {
        return dataTemp
    } else if let dataTemp = Locksmith.loadDataForUserAccount(userAccount: username) {
        // yup we have values with the old "accessibleOption", get it and assign to dataTemp
        try! Locksmith.deleteDataForUserAccount(userAccount: username) // remove it so we don't have redundant values for this key
        try! Locksmith.updateData(data: dataTemp, forUserAccount: username, accessibleOption: accessible) // assign it to the new "accesibleOption"
        return dataTemp // return the value
    } else {
        return nil
    }
}

then I created an extension:
extension Locksmith {
    fileprivate static func loadDataForUserAccount(userAccount: String,
                                                   inService service: String = LocksmithDefaultService,
                                                   accessibleOption: LocksmithAccessibleOption) -> [String: Any]? {
        struct ReadRequest: GenericPasswordSecureStorable, ReadableSecureStorable {
            let service: String
            let account: String
            var accessible: LocksmithAccessibleOption?
        }

        let request = ReadRequest(service: service, account: userAccount, accessible: accessibleOption)
        return request.readFromSecureStore()?.data
    }

    fileprivate static func updateData(data: [String: Any],
                                       forUserAccount userAccount: String,
                                       inService service: String = LocksmithDefaultService,
                                       accessibleOption: LocksmithAccessibleOption) throws {

        struct UpdateRequest: GenericPasswordSecureStorable, CreateableSecureStorable {
            let service: String
            let account: String
            let data: [String: Any]
            var accessible: LocksmithAccessibleOption?
        }

        let request = UpdateRequest(service: service, account: userAccount, data: data, accessible: accessibleOption)
        return try request.updateInSecureStore()
    }
}

I'm not sure if this is the ideal way, but it does works the way we need it.

"else if let dataTemp = Locksmith.loadDataForUserAccount(userAccount: username)" statement returns nil even if there is data in the keychain for the same account.


Custom templates - what am i missing?

Copy pasted the Student Teacher Dialog prompt from the README.md into a page in my journal
Made sure that the # Student Teacher Dialog line was the very highest level block
Unfortunately, /gpt is not showing a custom template
What am I missing?

Could you post a screenshot off of the template?
Make sure the code bock is indented beneath the template if it isn't already

I was also having the same issue, until I saw your previous comment and realized that we need to indent the prompt. This was not clear from the README. The following syntax works for me.
- # Student Teacher Dialog
  prompt-template:: Student Teacher Dialog
	- ```prompt
	  Rewrite text as a dialog between a teacher and a student:
	  ```

The identing thing was the problem. Copy-paste from the readme wasn't bringing that in, somehow.

One more thing, if I copy paste as per avijeet above, logseq saysFull content not displayed. For others, your're on the right track, but not quite there yet.

Awesome, glad that fixed it. I'll update the readme with the hyphens so it works with copy and paste.


how to enable cblas_cgemm3m in windows

when i use cblas_cgemm3m in vs2019,  it occurs compile error:error LNK2019: unresolved external symbol cblas_cgemm3m referenced in function,how to solve it?

Do you see anything about cgemm3m or cblas_cgemm3m in the build log ? The GEMM3M functions in OpenBLAS should be available on x86, x86_64, mips (and ia64 if that still exists).

i can see cgemm3m_(not cgemm3m ) in the openblas.lib file,but when i use it ,the app crashes。

What is your hardware ? It is possible that the recently rewritten cgemm3m kernels for Haswell and SkylakeX were not tested on windows, but as they use intel intrinsics I'd think the compiler should be able to handle any os-specific differences.  (cgemm3m kernels for older cpus use assembly directly, but in a syntax that VS does not understand, so you would get generic C kernels - again these should work)

my hardware is "intel core i9 Coffe lake" read  from CPU-Z ,openblas compile info is "OpenBLAS 0.3.9 DYNAMIC_ARCH NO_AFFINITY Haswell"。how can i direct use the api cblas_cgemm3m in the cblas.h?

It does look as if there is an error in our cmake support files (which are used on Windows) and the CBLAS interfaces to the CGEMM3M functions are not getting built by VS2019. Could you try changing the line in interface/CMakeLists.txt that reads
GenerateNamedObjects("gemm.c" "GEMM3M" "gemm3m" false "" "" false ${float_type})
to
GenerateNamedObjects("gemm.c" "GEMM3M" "gemm3m" ${CBLAS_FLAG} "" "" false ${float_type}) (change the first "false" to "${CBLAS_FLAG)") and rebuild OpenBLAS ?


Weather Unit ?u Isn't Respected For V2

I'm running the following command and trying to specify USCS/Farenheit, but that option isn't respected.
curl v2.wttr.in/Evergreen+Colorado\?u

Same issue for me. I'm sure the environment doesn't matter, but Debian 10.10

The same for v3 :)

Yes, this will be implemented. It is a duplicate of #376

@rjhilgefort @Mist-Hunter @rifen
I've added UCSC support for v2.wttr.in. Please test if it works for you.
It should switch automatically to USCS if you are in the US,
otherwise the mode should be activated with ?u, as usually.
$ curl v2.wttr.in/?u

If the USCS mode is active:

The temperature is displayed in Fahrenheit
The wind-speed is displayed in MpH

Nice, thanks! It has the correct units now which is fantastic. I don't know if something changed with my font/terminal, but now the icons overflow the box.

@rjhilgefort No, Rob, nothing changed except the weather.
That's a know bug (or feature):

#345

@chubin Awesome, good to know. I'll follow that issue, thanks!


from string doesn't works if it contains Chinese/Japanese characters

when I change the from string to '美食网<EMAIL_ADDRESS>it will generate errors like:
["apiMessage":protected]=>
 string(27) "Message generation rejected"
 ["apiCode":protected]=>
 string(4) "1902"
 ["apiDescription":protected]=>
 string(38) "No Sending Domain found in From header"
 ["message":protected]=>
 string(40) "Received bad response from Transmissions"
 ["string":"Exception":private]=>

But that works well if I replaced '美食网' to any other english word.
Thanks~

Hi @senseluo if you enclose the characters in double quotes like so:
'from' => '"美食网"<EMAIL_ADDRESS>
That should work. It's a known bug we need to address. Alternatively, you can use the following format as well (to avoid the ugly double-quote thing):
'from' => [
    'name': '美食网',
    'email'<EMAIL_ADDRESS>]

I should also mention we're going to make this automatic when we address #61.

I looked into making this automatic, but in the end decided not to for the following reasons:

Our underlying API clearly has an issue that should be resolved here
It is not guaranteed that the mailparse extension will be compiled when PHP is built so we can't just use that to solve the problem
We'd have to write our own logic if we truly wanted to support this properly

For those reasons, we decided that updating the examples and docs for now until the underlying API is fixed would be the way to go.
Thank you so much for reporting this @senseluo.


Safely edit markdown containing new resource shortcode parameters.

As an author, I'd like to be able make localized edits to pages without messing up other parts of the page.
Acceptance Criteria:

[x] Edit pages containing {{< resource uuid123 href_uuid="uuid456" >}} and {{< resource uuid123 href="https://www.mit.edu" >}} should preserve those shortcodes.

Out of Scope

Editing the href and href_uuid values of {{< resource >}} shortcodes is out-of-scope.

Background
Current ocw-next pages include link-wrapped resources such as [{{< resource some-uuid-here >}}](/link/to/thing). These are problematic, so we are converting the wrapping-link to a href_uuid or href parameter. Support for this on hugo-themes was added in https://github.com/mitodl/ocw-hugo-themes/pull/538 .
We need to support them in studio before migrating the data to new format.

Closed by #1192


LivingDocument class

Easy way to carry information around, describes what this is for... could be the name of the project?
Yeah, I'll create a new repo to track a cookiecutter of benchwarmers or whatever

calling this satisfied by github.com/dmarx/workbench


1.5: Fix coreaudio arg mismatch (+ brew check)

This fixes a crash in the 1.5 branch that occurs when Pyglet tries to load the coreaudio media codec, and includes the open PR #954 for additional Mac support.

Thanks for the PR! Shall we close the other one?

Thanks for the PR! Shall we close the other one?

The other PR contains the same change for Homebrew, but is targeting the master (2.0) branch as opposed to 1.5.


lilypond: 2.24.3 -> 2.24.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for lilypond is: Music typesetting system
meta.homepage for lilypond is: http://lilypond.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.24.4 with grep in /nix/store/a0lszdqikp0nwxp1iq0b2ix4s6df4g8g-lilypond-2.24.4
found 2.24.4 in filename of file in /nix/store/a0lszdqikp0nwxp1iq0b2ix4s6df4g8g-lilypond-2.24.4

Rebuild report (if merged into master) (click to expand)

18 total rebuild path(s)

18 package rebuild(s)

First fifty rebuilds by attrpath

asciidoc-full
asciidoc-full-with-plugins
clevis
denemo
disorderfs
kakounePlugins.rep
lenmus
lilypond
lilypond-with-fonts
luksmeta
python311Packages.abjad
python312Packages.abjad
rep
solfege
spring
springLobby
tang

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a0lszdqikp0nwxp1iq0b2ix4s6df4g8g-lilypond-2.24.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lilypond https://github.com/r-ryantm/nixpkgs/archive/eaaf0eba0b6f6687c79b6c6ba289999740968fe7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/eaaf0eba0b6f6687c79b6c6ba289999740968fe7#lilypond

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a0lszdqikp0nwxp1iq0b2ix4s6df4g8g-lilypond-2.24.4
ls -la /nix/store/a0lszdqikp0nwxp1iq0b2ix4s6df4g8g-lilypond-2.24.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages marked as broken and skipped:

spring
springLobby

2 packages failed to build:

python312Packages.abjad
python312Packages.abjad.dist

18 packages built:

asciidoc-full
asciidoc-full-with-plugins
asciidoc-full-with-plugins.dist
asciidoc-full.dist
clevis
clevis.man
denemo
disorderfs
kakounePlugins.rep
lenmus
lilypond
lilypond-with-fonts
luksmeta
python311Packages.abjad
python311Packages.abjad.dist
solfege
tang
tang.man

Maintainer pings
cc @MarcWeber @yurrriq for testing.

Result of nixpkgs-review pr 329816 run on x86_64-linux 1

4 packages marked as broken and skipped:

python312Packages.abjad
python312Packages.abjad.dist
spring
springLobby

18 packages built:

asciidoc-full
asciidoc-full-with-plugins
asciidoc-full-with-plugins.dist
asciidoc-full.dist
clevis
clevis.man
denemo
disorderfs
rep (kakounePlugins.rep)
lenmus
lilypond
lilypond-with-fonts
luksmeta
python311Packages.abjad
python311Packages.abjad.dist
solfege
tang
tang.man


[Feature Request] Client.expect gives the option of including body in the UnexpectedStatus error

Many API only gives the detailed error information in the body, including the body in the UnexpectedStatus would be really helpful.
Right now I am using my own implementation of expect like
def expect[A](req: Request[F], onError: Option[(String, Status) => Throwable] = None)
 (implicit d: EntityDecoder[F, A]): F[A] = {
   val r = if (d.consumes.nonEmpty) {
        val m = d.consumes.toList
        req.putHeaders(Accept(MediaRangeAndQValue(m.head),
      m.tail.map(MediaRangeAndQValue(_)): _*))
      } else req
   http.fetch(r) {
      case Successful(resp) =>
        d.decode(resp, strict = false).fold(throw _, identity)
      case failedResponse =>
        onError.fold(F.raiseError(UnexpectedStatus(failedResponse.status))) { f =>
          EntityDecoder[F, String].decode(failedResponse, false)
            .fold(identity, f(_, failedResponse.status).flatMap(F.raiseError)
       }
    }
}

If would be nice to have an idiomatic way to do this.

Do we want to go any further with this, or are we happy with the expectOr at this point?

Super happy!


Add net training test.

Pertaining to issue 24, this adds a test that trains the network to learn a simple audio clip (3 tones forming a chord, at least that's what it is supposed to be). The idea is to provide broad coverage of the code and sanity check that the WaveNet is trainable.
It runs in a couple seconds of seconds on my machine, but might not be Travis-friendly since I think it has to run on the CPU there.
If you uncomment the print statement you can see the loss drops very close to zero, so it has successfully overfit to the 0.10 second long clip.
It would be interesting to save the generated audio (and training audio) out to .wav file to see how well it sounds.
Edit: build fails at moment because travis can't find librosa.

@jyegerlehner: There was a bug in the ci install script. If you rebase on master, the tests should now run correctly.

Can you replace the use of get_arguments with default values? (Or maybe the idea was to use the current default args?)
The problem is that this will give an error when using nosetests to run the tests, as the nosetests parameters become the ones parsed by get_arguments.

Now get_arguments() is failing. I see this failure locally if I run
sh ci/test.sh

but succeeds if I run
python ci/test_net.py

I'm somewhat new to python and know nothing about argparse, so will have to flounder around a bit to figure this out. Or maybe I'll just rip out the get_arguments() call and put some hard-coded params in.

Or maybe I'll just rip out the get_arguments() call and put some hard-coded params in.
Sorry, that's what I meant with default values :)
argparse interferes with the testing framework, and putting hard-coded values in the test should be fine.

Sorry, that's what I meant with default values :)

Ah my fault; I posted before I saw that comment (needed to refresh my browser). It was perfectly clear. Sure, that's the easiest way to go. At the expense of a bit of lost coverage: I was thinking if we use the same defaults as the rest of the software, then this test does some sanity checking on them.

This is a really neat test, thanks for implementing it!

Running the test 1200 rounds or more instead of 50 rounds does not converge. The error jumps up to 4 or five and stays there...  I've noticed similar things when training on the voice data, so perhaps there is some problem.  Nice to have it manifest itself so quickly in a test, though.
@@ -57,9 +57,9 @@ class TestNet(tf.test.TestCase):
         with self.test_session() as sess:
             sess.run(init)
             initial_loss = sess.run(loss)
-            for i in range(50):
+            for i in range(5000):
                 loss_val, _ = sess.run([loss, optim])
-                #print "i: %d loss: %f" % (i, loss_val)
+                if i%10==0: print("i: %d loss: %f" % (i, loss_val))

         # Sanity check the initial loss was larger.
         self.assertGreater(initial_loss, max_allowed_loss)

That's a nice find.
I haven't encountered this myself when training yet, but I do see it when increasing the number of steps in this test (both on the GPU and CPU).


Add page with security tools

Closes https://github.com/equinor/isc-appsec/issues/276

May be adding availability criteria for the tools is a good idea.
For example : Fundamental level tools are available to all, Intermediate level tools are available with an easy to get approval or a specific group of people (may be only the SC network) and expert level tools are available with a more difficult to get approval or a specific group of people with thorough knowledge on the use of the tool.
Approval processes can also be added using hyperlinks.
The reasoning behind this is that tools like ZAP and Kali can cause unintended traffic if not used properly. This in turn can result in alerts to CDC team.
Under intermediate tools I would also add burp suite community edition.

I agree, but as of now we have only mentioned free software (and Snyk) which people don't need our approval to use. When it comes to licensed software with a cost attached, I agree that we need a routine for governing access (such as AccessIT). Until we have such a procedure in place I think the best we can do is give recommendations, and make sure only to recommend approved software (not shadow IT)


nan if variance is 0

All the eigenvector and eigenvalues, equal to nan if there is even one row in the dataset that has all 0s or all 2s.

Fixed with f00e071dfaedb2d8522e8a5779e927fd33b3e70e


MRG: Fix documentation

Closes #3753.
Ready for review/merge from my end.

Current coverage is 87.43% (diff: 100%)

Merging #3760 into master will decrease coverage by <.01%

@@             master      #3760   diff @@
==========================================
  Files           343        343
  Lines         60976      60976
  Methods           0          0
  Messages          0          0
  Branches       9320       9320
==========================================
- Hits          53315      53312     -3
- Misses         4917       4918     +1
- Partials       2744       2746     +2

Powered by Codecov. Last update 599ae12...2e3a0f7

thx


Add a connection type variable for Plugin implementations

This type variable is intended to represent the type of the
connection object, if any, that the plugin will use. This will
help make the implementations more robust by leveraging Java's
type checking instead of rudimentary type casts over the
connection objects.

@mohanarpit @trishaanand Exploring something here, trying to move towards a cleaner interface for plugin implementations. This occurred to me when working on datasource structure implementation. The connection object typecasting is being done a lot of times and this can help avoid any potential pitfalls there. Let me know what you guys this.


Test golden files without shadows

Now that we have golden file testing, it will significantly reduce the churn on our golden files if we test out golden files against scenes that contain no shadows.
Proposal

We add the ability to turn shadows off in the framework.  Ideas as to how to do this:

We introduce a debugZeroElevation global in the rendering layer that will cause all elevations to be zero, thus implicitly turning off shadows even though the code to draw the shadows is still active.

We introduce a debugNoShadows global in the rendering layer that will be consulted in places like PhysicalModelLayer.addtoScene() and RenderPhysicalModel.paint()

Something else...?

We make the flutter test harness turn shadows off by default in all widget tests.  Tests that specifically want shadows can update the flag in their test configuration (either via setUp() or flutter_test_config.dart).

We provide a small suite of golden file tests that explicitly enable shadows and test them, so that we can catch regressions in Skia or our own shadow rendering.

/cc @Hixie

This is fixed with #17363


Tracking reminder stays on screen after tracking is started

Title says it all, really. This is on macOS. When I get a tracking reminder and start tracking using hotkeys, the reminder should be cleared. But it doesn't. Instead I have to use my mouse to dismiss the reminder.

#3065 might fix this.


Where is the documentation?

how can i start with this project ???

Hi, it's still a working progress. We are still developing and have yet to release a stable version. I'll be sure to let you know whenever we have a RC.


[EN] Feedback for: /web/fundamentals/engage-and-retain/web-app-manifest/?hl=en

The material design on this page makes it hard to read. The purple header looks like something that is covering up content on the page. I want to close it but I can't.

Thanks for the feedback, we've got an update coming next week that should fix this.


.axaml file extension and msbuild metadata

@drunkvs keeps breaking our extension...

Just tested this out on https://github.com/AvaloniaUI/AvaloniaVS/pull/161. It seems to generally work, but adding a new UserControl I still get the following added to the .csproj:
  <ItemGroup>
    <AvaloniaXaml Remove="UserControl1.axaml" />
  </ItemGroup>

  <ItemGroup>
    <None Update="UserControl1.axaml">
      <SubType>Designer</SubType>
    </None>
  </ItemGroup>

Which causes the XAML compiler to not be run on UserControl.xaml.


Added reset of overflow-x in setoptions

The horizontal scroll bar state is only set during initial creation of the grid.  This change adds updating the overflow-x update to setOptions() so that it will add/hide the horiz scrollbar based on the current setting of forceFitColumns.

This looks good but we might consider submitting this to the original repo since this is a generic fix that they would probably take (assuming it is the right fix). Then we can consume it when they merge it in.

will submit to the original repo in parallel to doing this change locally


Get the player to confirm before downloading a large file

Maybe 2MB is fine, above that get the player to confirm before downloading a large file. This will involve changes to parchment-proxy, and the parchment loading code.

Duplicate of #10


Change installation instructions to recommend RemoteSigned instead of Unrestricted

The installation instructions in the README and at http://scoop.sh/ recommend setting powershell executionpolicy to Unrestricted. Wouldn't RemoteSigned be a safer default?
Many users probably don't understand the difference, and globally changing a user's scripting security defaults should probably have as little impact as possible.
On my machine I was able to install scoop and curl with RemoteSigned without a problem.

I thought RemoteSigned meant the scripts had to be signed? Some Scoop contributors looked into signing scripts a while ago, but we couldn't find a practical way to do this without compromising our ability to quickly and easily make changes and include outside contributions.
Let me know if I'm incorrect about how RemoteSigned works.

When using RemoteSigned only remote scripts need to be signed, for the wrapper scripts or scripts from applications they are considered local they they don't need to be signed. As the bootstrap script of scoop, since you are using iex to invoke, the execution policy actually doesn't apply: see No. 4 in https://blog.netspi.com/15-ways-to-bypass-the-powershell-execution-policy/.

Yes iex is used to install Scoop, but what about subsequently running scoop.ps1?

Set the execution policy to RemoteSigned will allow subsequently running scoop.ps1. I think the suggestion here is RemoteSigned is considered to be more safe than Unrestricted.

Okay thanks. I was sure I had tried remotesigned in the past and it didn't work, but it does seem okay. I will update the recommendation on the Scoop homepage.

I remember the bootstrap setup script currently stops if the current execution policy is not set to Unrestricted, so it might also need to be updated. You may have to compare it using a range instead of just checking for one; in PowerShell it is defined as an enum:
public enum ExecutionPolicy
{
	Unrestricted,
	RemoteSigned,
	AllSigned,
	Restricted,
	Bypass,
	Undefined,
	Default = 3
}

I personally would recommend just remove the check in the bootstrap script, in the documentation we should have already mentioned if the user is unable to run the bootstrap then they should run Set-ExecutionPolicy RemoteSigned -s cu -force.

Thanks for this. I've updated the install bootstrap script to check for RemoteSigned or lower security.
I've updated the docs, but I think it's good to have the check in the bootstrap as well—in case people don't read the docs first we can let them know how to proceed.
I've tested with a fresh user account with default settings and it appears to work well.

Awesome, looks good! I found a little more documentation in the README that could be updated too

Commenting on old issue as I recently encountered this due to effective ExecutionPolicy being Bypass. The -gt check will return true for Bypass, however this is essentially Unrestricted minus even prompting.
Yes, I know Bypass is a terrible terrible evil, but there was an extenuating circumstance. :P


Does not work as is

Hello,
When I clone the repo, change directory into it, and npm install && npm start I get the following error.  It does not seem to work as is, do I need to update any files?
FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
1: 00007FF69B3B7B7F v8::internal::CodeObjectRegistry::~CodeObjectRegistry+114079
2: 00007FF69B344546 DSA_meth_get_flags+65542
3: 00007FF69B3453FD node::OnFatalError+301
4: 00007FF69BC7B29E v8::Isolate::ReportExternalAllocationLimitReached+94
5: 00007FF69BC6587D v8::SharedArrayBuffer::Externalize+781
6: 00007FF69BB08C4C v8::internal::Heap::EphemeronKeyWriteBarrierFromCode+1468
7: 00007FF69BB05D64 v8::internal::Heap::CollectGarbage+4244
8: 00007FF69BB036E0 v8::internal::Heap::AllocateExternalBackingStore+2000
9: 00007FF69BB28266 v8::internal::Factory::NewFillerObject+214
10: 00007FF69B85A735 v8::internal::DateCache::Weekday+1797
11: 00007FF69BD08F91 v8::internal::SetupIsolateDelegate::SetupHeap+494417
12:<PHONE_NUMBER>C1F2A0

Hi,
This is normally cause by insufficient memory
Try running export NODE_OPTIONS="--max-old-space-size=8192" before running npm start

I'm doing this in PowerShell in a Windows 10 VM with 32gb of RAM.  Should I theoretically be able to just git clone the repo, cd into the folder, npm install and npm start run and it just work as is?

There is a good discussion about this on stackexchange if you want to get
into the details:
https://stackoverflow.com/questions/53230823/fatal-error-ineffective-mark-compacts-near-heap-limit-allocation-failed-javas
The package that causes the error is the cardano-serialization-lib btw
Try if running the suggested command removes the error. I have added it to
the troubleshooting section in the Readme
On Fri, Mar 4, 2022, 22:02 marcoarroyo41 @.***> wrote:

I'm doing this in PowerShell in a Windows 10 VM with 32gb of RAM. Should I
theoretically be able to just git clone the repo, cd into the folder, npm
install and npm start run and it just work as is?
—
Reply to this email directly, view it on GitHub
https://github.com/dynamicstrategies/cardano-wallet-connector/issues/2#issuecomment-1059563379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR54XQAA7MVE6IYJV7HULXDU6KCBFANCNFSM5P4OTOGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
.com>

I switched to Linux and ran that command, different error this time haha.  Is there any other configuration required or any commands besides just downloading the repo and npm install and start run?
Html Webpack Plugin:
Error: Child compilation failed:
Module.createRequire is not a function

child-compiler.js:169 childCompiler.runAsChild
[cardano-wallet-connector]/[html-webpack-plugin]/lib/child-compiler.js:169:1    8

Compiler.js:564 compile
[cardano-wallet-connector]/[webpack]/lib/Compiler.js:564:11

Compiler.js:1183 hooks.afterCompile.callAsync.err
[cardano-wallet-connector]/[webpack]/lib/Compiler.js:1183:17

Hook.js:18 Hook.CALL_ASYNC_DELEGATE [as _callAsync]
[cardano-wallet-connector]/[tapable]/lib/Hook.js:18:14

Compiler.js:1179 compilation.seal.err
[cardano-wallet-connector]/[webpack]/lib/Compiler.js:1179:33

Compilation.js:2784 finalCallback
[cardano-wallet-connector]/[webpack]/lib/Compilation.js:2784:11

Compilation.js:3089 hooks.afterSeal.callAsync.err
[cardano-wallet-connector]/[webpack]/lib/Compilation.js:3089:11

Hook.js:18 Hook.CALL_ASYNC_DELEGATE [as _callAsync]
[cardano-wallet-connector]/[tapable]/lib/Hook.js:18:14

I upgraded to Nodejs version 14 and that fixed it!

Great!
I've also updated the Readme that requires Nodejs version 14 or higher
On Fri, Mar 4, 2022, 22:22 marcoarroyo41 @.***> wrote:

I upgraded to Nodejs version 14 and that fixed it!
—
Reply to this email directly, view it on GitHub
https://github.com/dynamicstrategies/cardano-wallet-connector/issues/2#issuecomment-1059575072,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADNMMZDUL2K2AB4JEA5P6GDU6KEJXANCNFSM5P4OTOGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***
com>


AzureCNI/Custom VNET - Remove ip address network configuration race condition

What this PR does / why we need it:
Case handled:  AzureCNI/Custom Vnet
There is a documented race condition whereby it is impossible to ensure that master ip addresses are allocated serially as REQUIRED by k8 bootstrap scripts.  I hit this problem every time i.e. the cluster fails to start.
This PR guarantees the 'MasterCount' ip addresses are allocated serially beginning with firstConsecutiveStaticIP before dynamic ip allocation begins.
The generated dynamic ips stanza are made dependent on the static ip (masterCount) stanzas.
To implement this, both the static and dynamic ips are in separate arm stanzas.
The static network interfaces reference the lb and intlb(for masterCount> 1) as before.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
It passes all tests and works predictably with automation onto existing subnets.
Tested with v0.20.7 and v0.21.1
If applicable:

[ ] documentation
[ ] unit tests
[ x] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Strike
If possible, assign to the firstConsecutiveStaticIP configuration property an IP address that is near the "end" of the available IP address space in the desired subnet.
For example, if the desired subnet is a /24, choose the "239" address in that network space

And change to:
Ensure that firstConsecutiveStaticIP + (masterCount-1) ip addresses are free within the master subnet as required by k8 installation scripts.

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

Oh well, Adobe won't let me give CircleC access to all my private repositories.  But who would anyway?
Maybe someone who's forked over all their private repos can take this PR and submit it.
See ya.  I tried...


Enabling ALB Support

Description of changes:
This is a non breaking change, already tested and working in a production environment.
The ALB lambda integration is not well documented..
I Had to remove the  Access-Control-Allow-Credentials header when the request is coming from an ALB because  it was returning 502 when I tried with that header.
Also i found out(via test and errors) that the order of the returned object does matter as isBase64Encoded must be before the body, if it is not.
This should save the user the $3.5 per 1M if he does not need more features from APIGW, and it does also works with Cloudfront.
With this PR it will continue to work with APIGW without changes.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Why is this PR not being merged? it would save users lots of $ in High load scenarios..
I Am using this in Prod with 10M MAU

Thanks for your contribution, @atrope
We've added this feature to v5.1.0. Unfortunately, due to the internal policy, we don't merge the PR directly. You can see the detail in CHANGELOG and your name in README.
Once again, thanks for your contribution.


Add version command to observatiroum gateway

@rubenvp8510 Since we add this nice way to package the version with the binary, I suggest to drop by default a log line when some starts this container anyway? WDYT?

@periklis Thanks for the review, I've already added the changes requested. Please give it another review when you get a chance.

@periklis Please review when you get a chance! Thanks

@periklis Done with the changes! Thanks!

@rubenvp8510 Just for clarification, do we need the command at all if we by default log the version?

@rubenvp8510 Just for clarification, do we need the command at all if we by default log the version?

Is not required, but may be good to have the only thing you want is to know the version. and don't see any weird error like "configuration missing" or something.

@rubenvp8510 Just for clarification, do we need the command at all if we by default log the version?

Is not required, but may be good to have if the only thing you want is to know the version. and don't see any weird error like "configuration missing" or something.

For the sanity to keep long-term maintenance low and since we do not have other subcommands in this binary, I would suggest to skip adding the extra subcommand and keep the log line only.

@rubenvp8510 Just for clarification, do we need the command at all if we by default log the version?

Is not required, but may be good to have if the only thing you want is to know the version. and don't see any weird error like "configuration missing" or something.

For the sanity to keep long-term maintenance low and since we do not have other subcommands in this binary, I would suggest to skip adding the extra subcommand and keep the log line only.

Ok, I'll remove it then.

@periklis done!

@periklis Can you review this again please? Thank you


Remove extra content from embed before sending

Set include_extras=False by default and updated each failing test to have the expected output.
I didn't check the integration tests as VScode wasn't able to find the tests/data directory, so I figured I'd let those test here for now until I have more time to look into that.

Set include_extras=False by default and updated each failing test to have the expected output. I didn't check the integration tests as VScode wasn't able to find the tests/data directory, so I figured I'd let those test here for now until I have more time to look into that.

Not sure why that would be the case, but perhaps the first time you set up the testing for this project you didn't configure the right test directory?
VS Code backs the settings in a file at .vscode/settings.json. Mine looks like this after the initial setup:
{
    "python.testing.pytestArgs": [
        "tests"
    ],
    "python.testing.unittestEnabled": false,
    "python.testing.pytestEnabled": true
}


Not working since 2016 May 14

I just cloned, correct the prod.cfg with my data and try to run... if fails like the lines below.
[Tue 31/May 10:52] alexgv@PROC-PE0ZW ~/workspace/packtpub-crawler $ python script/spider.py --config config/prod.cfg
     __             __         __           __           __    __        __
    /\ \     __    /\ \       /\ \         /\ \         /\ \  /\ \    _ / /\
   /  \ \   /\_\   \ \ \     /  \ \       /  \ \____   /  \ \ \ \ \  /_/ / /
  / /\ \ \_/ / /   /\ \_\   / /\ \ \     / /\ \_____\ / /\ \ \ \ \ \ \___\/
 / / /\ \___/ /   / /\/_/  / / /\ \ \   / / /\/___  // / /\ \_\/ / /  \ \ \
/ / /  \/____/   / / /    / / /  \ \_\ / / /   / / // /_/_ \/_/\ \ \   \_\ \

/ / /    / / /   / / /    / / / _ / / // / /   / / // //\    \ \ \  / / /
/ / /    / / /   / / /    / / / /\ / // / /   / / // /_/     \ \ / / /
/ / /    / / // / /  / / /\ \ / \ \ _/ / // / /___      \ \ / /
/ / /    / / //_/// / /\ \ \  \ ___/ // / /\      \ \  /
//     // ///_/   /__/ /__________/       _/
Download FREE eBook every day from www.packtpub.com
@see github.com/niqdev/packtpub-crawler
[+] configuration file: config/prod.cfg
[-] <class 'requests.exceptions.ConnectionError'> HTTPSConnectionPool(host='www.packtpub.com', port=443): Max retries exceeded with url: /packt/offers/free-learning (Caused by <class 'httplib.BadStatusLine'>: '') | spider.py@41
Traceback (most recent call last):
File "script/spider.py", line 41, in main
packpub.run()
File "/home/alexgv/workspace/packtpub-crawler/script/packtpub.py", line 110, in run
self.__GET_login()
File "/home/alexgv/workspace/packtpub-crawler/script/packtpub.py", line 46, in __GET_login
response = self.__session.get(url, headers=self.__headers)
File "/usr/lib/python2.7/dist-packages/requests/sessions.py", line 467, in get
return self.request('GET', url, **kwargs)
File "/usr/lib/python2.7/dist-packages/requests/sessions.py", line 455, in request
resp = self.send(prep, **send_kwargs)
File "/usr/lib/python2.7/dist-packages/requests/sessions.py", line 558, in send
r = adapter.send(request, **kwargs)
File "/usr/lib/python2.7/dist-packages/requests/adapters.py", line 378, in send
raise ConnectionError(e)
ConnectionError: HTTPSConnectionPool(host='www.packtpub.com', port=443): Max retries exceeded with url: /packt/offers/free-learning (Caused by <class 'httplib.BadStatusLine'>: '')
[-] something weird occurred, exiting...
[Tue 31/May 10:52] alexgv@PROC-PE0ZW ~/workspace/packtpub-crawler $

Hi,
This error Max retries exceeded looks like a network error.
something in your network is changed? a proxy or a firewall? do you have a static ip?

It seams ok... here a curl connecting to the proper url:
[Tue 31/May 10:52] alexgv@PROC-PE0ZW ~/workspace/packtpub-crawler $ curl -# -v https://www.packtpub.com/packt/offers/free-learning > output.txt

Hostname was NOT found in DNS cache
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 3128 (#0)
Establish HTTP proxy tunnel to www.packtpub.com:443

CONNECT www.packtpub.com:443 HTTP/1.1
Host: www.packtpub.com:443
User-Agent: curl/7.35.0
Proxy-Connection: Keep-Alive

< HTTP/1.0 200 Connection established
< Connection: close
<

Proxy replied OK to CONNECT request
successfully set certificate verify locations:
CAfile: none
CApath: /etc/ssl/certs
SSLv3, TLS handshake, Client hello (1):
} [data not shown]
SSLv3, TLS handshake, Server hello (2):
{ [data not shown]
SSLv3, TLS handshake, CERT (11):
{ [data not shown]
SSLv3, TLS handshake, Server key exchange (12):
{ [data not shown]
SSLv3, TLS handshake, Server finished (14):
{ [data not shown]
SSLv3, TLS handshake, Client key exchange (16):
} [data not shown]
SSLv3, TLS change cipher, Client hello (1):
} [data not shown]
SSLv3, TLS handshake, Finished (20):
} [data not shown]
SSLv3, TLS change cipher, Client hello (1):
{ [data not shown]
SSLv3, TLS handshake, Finished (20):
{ [data not shown]
SSL connection using ECDHE-RSA-AES256-GCM-SHA384
Server certificate:
subject: OU=GT23863044; OU=See www.rapidssl.com/resources/cps (c)15; OU=Domain Control Validated - RapidSSL(R); CN=*.packtpub.com
start date: 2015-03-30 03:23:14 GMT
expire date: 2018-07-01 00:37:31 GMT
subjectAltName: www.packtpub.com matched
issuer: C=US; O=GeoTrust Inc.; CN=RapidSSL SHA256 CA - G3
SSL certificate verify ok.

GET /packt/offers/free-learning HTTP/1.1
User-Agent: curl/7.35.0
Host: www.packtpub.com
Accept: /

< HTTP/1.1 200 OK

Server nginx/1.4.5 is not blacklisted
< Server: nginx/1.4.5
< Date: Wed, 01 Jun 2016 21:58:25 GMT
< Content-Type: text/html; charset=utf-8
< Transfer-Encoding: chunked
< Connection: keep-alive
< Expires: Sun, 19 Nov 1978 05:00:00 GMT
< Cache-Control: public, s-maxage=172800
< Age: 30153
< Via: 1.1 varnish
< X-Country-Code: BR
<
{ [data not shown]
######################################################################## 100,0%* Connection #0 to host <IP_ADDRESS> left intact

Attached the page downloaded correctly.
output.txt

:disappointed_relieved: I've just verified and everything works fine for me, maybe this is due to a libraries that you upgraded?
I have also upgraded my requests version from 2.7.0 to 2.10.0 and still work.
Do you have the possibility to run the script in another environment/machine just to understand if is only your problem or not? If I can't reproduce the error I don't know how to help you, sorry.
If you have any idea or you need that I verify some library versions or other things, just ask me.

Hi @alexgv99, do you still have the problem? Can I help you somehow? Otherwise I will close the issue.
Thanks

Yes, I do have the same problem yet, but as I can't test anywhere else, I
mean, I tried at home, but in my house is Windows, not Linux, and I
couldn't configure the environment to run it... so I dropped... Go ahead
and close the thread... Thank you anyway for trying to help me.
Cordialmente.
Alex Gouvêa Vasconcelos
<EMAIL_ADDRESS>MSN<EMAIL_ADDRESS>http://about.me/alexgv99
2016-06-09 15:23 GMT-03:00 niqdev<EMAIL_ADDRESS>
Hi @alexgv99 https://github.com/alexgv99, do you still have the
problem? Can I help you somehow? Otherwise I will close the issue.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/niqdev/packtpub-crawler/issues/3#issuecomment-224983395,
or mute the thread
https://github.com/notifications/unsubscribe/AFuYZuK-Uf1Js37LVhALJau7dAuu0ogdks5qKFo_gaJpZM4Iqk4r
.

If it can help you can have a look at this on windows otherwise maybe wait for this.
Thanks