Change package from xenon.clickhouse to com.clickhouse

Summary
Change package from xenon.clickhouse to com.clickhouse
Checklist
Delete items not relevant to your PR:

[ ] Unit and integration tests covering the common scenarios were added
[ ] A human-readable description of the changes was provided to include in CHANGELOG
[ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

... to com.clickhouse

may be to com.clickhouse.spark, to avoid conflicts with the Java client


Obtaining cable location information

Hi,
I'm a student and I'm trying to use MuJoCo for Master's research.
I'm looking for some help with Obtaining cable location information.
I am trying to write code in Python to get the location of a cable, but I don't know how to do it. I would like to know if there is a function or other function that can be used for this.

Hi, the cable is noting but a collection of bodies (try to export the XML after you load a cable, you will see that the composite is expanded into multiple bodies). So accessing their positions is done in the same way as for all other bodies in MuJoCo, i.e. from mjData, for example using the variable qpos. See the colab tutorial for more information.

Using print(physics.named.data.geom_xpos), we could get the coordinates of the object, but the coordinates of the initial point will always be displayed, even if the object is manipulated and its position changes. Is there any way to display the coordinates accordingly when the object is moved?


test_roundtrip failing builds

https://github.com/xpublish-community/xpublish/blob/c4134dbc074ec1fa5a2cc7e55f458b5f98a6a34e/tests/test_zarr_compat.py#L140-L156
Is causing builds with unrelated changes to fail recently.
It looks like the issue is due to the time encoding.
FAILED tests/test_zarr_compat.py::test_roundtrip[2018-01-01-2021-01-01-6H-180-360-True-gregorian-False] - AssertionError: Left and right Dataset objects are not identical

Differing coordinates:
L * time         (time) datetime64[ns] 2018-01-01 2018-01-01 ... 2020-12-31
    bounds: time_bounds
R * time         (time) datetime64[ns] 2018-01-01 ... 2020-12-31T12:00:00
    bounds: time_bounds
Differing data variables:
L   tmin         (time, lat, lon) float32 dask.array<chunksize=(4384, 180, 360), meta=np.ndarray>
R   tmin         (time, lat, lon) float32 dask.array<chunksize=(4384, 180, 360), meta=np.ndarray>
L   time_bounds  (time, d2) datetime64[ns] dask.array<chunksize=(4384, 2), meta=np.ndarray>
R   time_bounds  (time, d2) datetime64[ns] dask.array<chunksize=(4384, 2), meta=np.ndarray>
L   tmax         (time, lat, lon) float32 dask.array<chunksize=(4384, 180, 360), meta=np.ndarray>
R   tmax         (time, lat, lon) float32 dask.array<chunksize=(4384, 180, 360), meta=np.ndarray>

A warning is getting thrown earlier
  /home/runner/work/xpublish/xpublish/xpublish/utils/zarr.py:144: UserWarning: Times can't be serialized faithfully to int64 with requested units 'days since 1980-01-01'. Resolution of 'hours' needed. Serializing times to floating point instead. Set encoding['dtype'] to integer dtype to serialize to int64. Set encoding['dtype'] to floating point dtype to silence this warning.
    encoded_da = encode_zarr_variable(dvar, name=key)

Which is from here:
https://github.com/xpublish-community/xpublish/blob/c4134dbc074ec1fa5a2cc7e55f458b5f98a6a34e/xpublish/utils/zarr.py#L132-L155
But is really coming from the Xarray Zarr backend.
https://github.com/pydata/xarray/blob/25e6e084aa18c49d92934db298a3efff9c712766/xarray/backends/zarr.py#L288-L318

Nothing has changed in encode_zarr_variable in the last few years, so it may be deeper.

Digging deeper. conventions.encode_cf_variable calls times.CFDatetimeCoder() which then calls encode_cf_datetime() which has had some recent changes.
https://github.com/pydata/xarray/blob/25e6e084aa18c49d92934db298a3efff9c712766/xarray/conventions.py#L154-L195
https://github.com/pydata/xarray/blob/25e6e084aa18c49d92934db298a3efff9c712766/xarray/coding/times.py#L803-L823
https://github.com/pydata/xarray/blob/25e6e084aa18c49d92934db298a3efff9c712766/xarray/coding/times.py#L670-L747
https://github.com/pydata/xarray/pull/8201 landed in Xarray v2023.09.0 which then caused a new bug that looks similar https://github.com/pydata/xarray/issues/8271 which should be fixed by https://github.com/pydata/xarray/pull/8272
If I anti-pin Xarray to resolve to another version (xarray!=v2023.09.0), tests at least seem to pass locally.

If I test locally against upstream (git+https://github.com/pydata/xarray.git in dev-requirements.txt) that also works.
For now I'm going to anti-pin Xarray in requirements.

Which explains why the dev builds worked.


Add kind checking in the REPL

Although it is pretty cool to check the AST and type using :p and :t respectively, it would be cool to check the kind of an expression using :k.

Fixed with this commit


WIP: add captum

Pre-submission checklist:
Please check each of these after submitting your pull request:

[X] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[X] Is your project closed source or, if it is open source, does your project have at least 250 GitHub stars?
[X] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[X] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~10 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

@andife I fixed the entry and it is now added and live. thanks.


SciPy's QZ decomposition leads to failure

A PR for the fix is already committed to SciPy https://github.com/scipy/scipy/pull/8178. In the meantime modify the tests that avoids QZ convergence failures.

Fixed.


maximal job mode

Currently, panama run --jobs N starts N jobs per primary particle. So if we have 5 components, we potentially have 5*N particles.
On a multi-user system we want to be able to limit the number of total jobs we have. So we need to implement a --max-job option to run the components after each other, not all at the same time.
While at it, a class-based refactoring of run.py is needed. See branch max-threads

the "maximal job mode" is now the default and only behaviour since #39 . everything else does not make sense, since different primaries need different time, so parallelizing that does not really make sense, since we would start out with e.g. 10 jobs, and then result in 5 jobs after a short time which take a long time. Then we would have 5 spare cpus not used.


feat(frontend, mock-ase): secure admin API with Hydra

Changes proposed in this pull request

Adds a Hydra image to the list of docker containers that are spun up
Adds a middleware to the admin API in backend that verifies the token with a request to the Hydra container.

Returns 401 if invalid token, continues to next() if valid

Adds a boilerplate login/consent screen to the Admin UI, to be replaced with ORY Kratos or something more sophisticated with https://github.com/interledger/rafiki/issues/2200
Implements a basic Hydra authorization flow with the login & consent screens

Captures the authorization from this flow in a new session storage instance in the Remix app

Re-initializes the Admin UI Apollo Client each time it makes a request, so that it may pass in an auth token once one has been acquired
Uses updated error handling to handle & communicate Unauthorized responses from the Apollo client

Context

Fixes #2218 and #2497.
Adds a Hydra container, a basic authorization flow for it, and a middleware in the API to check for a valid authorization token for a given API request.
Checklist

[x] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[x] Make sure that all checks pass
[ ] Bruno collection updated

Questions:

How do I test this? Just by using the frontend?
Is there a way to get a token to be used in e.g. Bruno?
Do the tokens auto-refresh?

I guess my general question is:
What is the flow for an ASE that integrates with Rafiki and uses the Admin APIs? Where do they get the token from, where do they put it, and what happens when that expires?

Questions:

How do I test this? Just by using the frontend?
Is there a way to get a token to be used in e.g. Bruno?
Do the tokens auto-refresh?

I guess my general question is: What is the flow for an ASE that integrates with Rafiki and uses the Admin APIs? Where do they get the token from, where do they put it, and what happens when that expires?

You can test this by using the Admin UI frontend, the Hydra flow can be kicked off by navigating to this localhost URL, then once the flow is completed, try viewing any of the pages on the Admin UI that make GraphQL requests - these should not work without a working token.
I could expose the token in the UI when the flow is complete so that it may be copy-pasted. I think you could potentially get a token using only API requests but I'm not 100% sure.
The token does not auto-refresh. This PR is big enough as it is so I thought it might be worth adding separately.

To answer the general question - the flow for an ASE would be to get a token through a Hydra flow, which gets stored in a session cookie in the Admin UI. When it expires, the Hydra authorization flow will have to be performed again to gain a new token.
The token currently isn't well-exposed to a potential ASE right now. Currently this PR secures the admin API and implements a mechanism to gain authorization within the admin UI.


[ART-1564] Fix group.yml update on advisories job

Previous attempt was wrong (https://github.com/openshift/aos-cd-jobs/pull/2131)
@tbielawa you were absolutely right! thanks a lot for pointing that out.

/lgtm


fs.rename() not being "redirected" correctly

Hi there,
First of all, many thanks for such a great package!
I've encountered an issue with the rename() function that's returned via the link function. All of the other file system methods are working correctly, however, this one does not seem to maintain the path redirect.
For example:
const { link } = require('linkfs');
const fs = require('fs');

const linkedFs = link(fs, [[`${__dirname}/test`, `${__dirname}/dist`]]);

linkedFs.ReadStream = fs.ReadStream;
linkedFs.WriteStream = fs.WriteStream;

/* ---------------------------
 *
 * Correctly loads `./dist/package.json`
 *
 * ------------------------ */

console.log(linkedFs.readFileSync('./test/package.json', 'utf8'));

/* ---------------------------
 *
 * Incorrectly renames file to `./test/package-test.json` instead of `./dist/package-test.json`
 *
 * ------------------------ */

linkedFs.rename('./test/package.json', './dist/package-test.json', () => console.log('COPIED'));

Do you have any idea why this might be?
Many thanks for any help in advance, very much appreciated!
All the best

@streamich I've opened a PR here that fixes this. I've added a test case as well:
https://github.com/streamich/linkfs/pull/333

I've forked the repo and added support for this:
https://www.npmjs.com/package/@ricsam/linkfs
https://github.com/ricsam/linkfs
npm install @ricsam/linkfs


VisualStudioVersion could be removed

<VisualStudioVersion Condition="'$(VisualStudioVersion)' == ''">10.0</VisualStudioVersion>
In sdk-based csproj this is always not empty (because sdk-based csproj'es is always VS 2017 or later, and VisualStudioVersion is always set in VS 2012 or later)

thanks!


DMA Support

Looks like this is done - but perhaps just need to properly test the zero copy interface.

Fully tested the zero copy interface on hardware as well. DONE!


Get updated Controller (Kontroller) working with integration tests.

Also run the tutorials to make sure BalancingAgent works.

All tests are passing except those that have issues filed.


Error: Cannot find module 'add-eventlistener-with-options' from ...

Bumped from v1.25.0 to 1.25.3 in my https://github.com/binarykitchen/videomail-client package and now unit tests fail.
Here a stack trace, not very helpful I am afraid:
[17:02:44] Starting 'test'...
Error: Cannot find module 'add-eventlistener-with-options' from '/Users/michael-heuberger/code/videomail-client/src/wrappers/visuals'
internal/streams/legacy.js:59
      throw er; // Unhandled stream error in pipe.
      ^

Error: javascript required
    at Stream.<anonymous> (/Users/michael-heuberger/code/videomail-client/node_modules/browser-run/index.js:35:34)
    at _end (/Users/michael-heuberger/code/videomail-client/node_modules/through/index.js:65:9)
    at Stream.stream.end (/Users/michael-heuberger/code/videomail-client/node_modules/through/index.js:74:5)
    at Stream.method [as end] (/Users/michael-heuberger/code/videomail-client/node_modules/duplexer/index.js:47:39)
    at Stream.<anonymous> (/Users/michael-heuberger/code/videomail-client/node_modules/throughout/index.js:7:25)
    at _end (/Users/michael-heuberger/code/videomail-client/node_modules/through/index.js:65:9)
    at Stream.stream.end (/Users/michael-heuberger/code/videomail-client/node_modules/through/index.js:74:5)
    at Stream.onend (internal/streams/legacy.js:44:10)
    at emitNone (events.js:111:20)
    at Stream.emit (events.js:208:7)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

The interesting part is probably
internal/streams/legacy.js:59
      throw er; // Unhandled stream error in pipe.

and
Error: javascript required

@binarykitchen This should be fixed as part of https://github.com/addityasingh/add-eventlistener-with-options/releases/tag/v1.25.5

good, thanks


move to sdk012

Signed-off-by: Kuan Feng<EMAIL_ADDRESS>

/test test multicloud-operators-subscription

/lgtm


Added lemmy-stats-exporter to tools section

lemmy-stats-exporter is a tool I built to export various metrics for a Lemmy instance from its PostgreSQL database over to InfluxDB, allowing instance admins to have an easy way to track these metrics over time and plug that into something like Grafana to create charts and dashboard of said metrics 😁

Thanks!


chore: Update README

Proposed Change
This PR updates the readme to include more details about BindPlane and our mission to elevate OpenTelemetry and make it easier to adopt.
Checklist

[ ] Changes are tested
[ ] CI has passed

The Supported Operating Systems are not enumerated and only exist in our technical documentation.
Add a link to the supported list of Operating Systems here:
https://observiq.com/docs/advanced-setup/installation/install-agent
Suggest that you add a line after: "For more detailed installation instructions view the Getting Started guide."
For Supported Operating System Versions see... https://observiq.com/docs/advanced-setup/installation/install-agent

@ryancgoins @jhowellbm @craigleeit -- Closing this PR in favor of https://github.com/observIQ/bindplane-otel-collector/pull/2062 that includes updates regarding naming changes to BindPlane Distro for OpenTelemetry Collector.
FYI - Craig, I've updated the section to add the link you suggested.


Fixed a syntax error

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Modified spyder/app/mainwindow.py line 213. Corrected the show_compatibility_message with show_plugin_compatibility_message
Issue(s) Resolved

Fixes #
Exception thrown at startup when failed compatibility check of a plugin:
Traceback (most recent call last):
File "C:\Users\elog-admin\AppData\Local\Programs\Spyder\pkgs\spyder\app\mainwindow.py", line 753, in
lambda plugin_name, omit_conf: self.register_plugin(
File "C:\Users\elog-admin\AppData\Local\Programs\Spyder\pkgs\spyder\app\mainwindow.py", line 213, in register_plugin
self.show_compatibility_message(message)
File "C:\Users\elog-admin\AppData\Local\Programs\Spyder\pkgs\spyder\app\mainwindow.py", line 1069, in getattr
return super().getattr(attr)
AttributeError: 'MainWindow' object has no attribute 'show_compatibility_message'
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:
abulgher

@abulgher, this is a duplicate of pull request #18910, so I'm going to close it. But thanks a lot for your interest to improve Spyder!


Re-think Error Design

Axios library returns err.response, err.request and err.config, which are very fat and make stdout impossible to read. Need to figure out if we have to return some of these values to the user or not. And if so, we have to define the output + design.
The v2 JSON API returns this on error:
errors: [{id: ..., help: ...., code: ....,  details:...}]

SDK deserializes these fields into:
{ ValidationError: Validation error, cannot save post.
    at makeRequest.catch (ghost/ghost-sdk/packages/admin-api/lib/index.js:290:33)
    at process._tickCallback (internal/process/next_tick.js:68:7)
  name: 'ValidationError',
  context: 'Validation failed for \'posts[0]\'',
  type: 'ValidationError',
  details:
   [ { keyword: 'required',
       dataPath: '.posts[0]',
       schemaPath: '#/properties/posts/items/required',
       params: [Object],
       message: 'should have required property \'title\'' } ],
  help: null,
  code: null,
  id: '2f5ebe00-3917-11e9-8a93-53537def9b36' }

This is cool. But if you get a library error, the error looks like:
{ Error: connect ECONNREFUSED <IP_ADDRESS>:2368
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1104:14)
  errno: 'ECONNREFUSED',
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 2368 }

Error is different. This is hard to document & hard to work with.
Need to figure out what todo.

One thing I've run into which trolled me hard for a while was .response being deleted from the error thrown by makeRequest.
I was throwing custom errors from our overridden makeRequest function and was attaching the request as I needed conditional logic based on the response status code. Spent ages trying to work out why my response object wasn't available on the error without thinking to check that this package could be modifying errors rather than simply passing them through.
Does it make more sense to pass errors through as-is in this library and leave it up to the consumer's error handling/logging to do what it needs to do?

Does it make more sense to pass errors through as-is in this library and leave it up to the consumer's error handling/logging to do what it needs to do?

I think this is the best thing to do - and lines up with exposing primitives task


Java Heap Space Error caused by an 'Out Of Memory' Error

At the end of the analyzer-result.yml file I see:
      'Gradle::app/build.gradle:':
      - timestamp: "2020-02-25T16:08:48.310903Z"
        source: "Gradle"
        message: "Resolving dependencies for 'build.gradle' failed with: BuildException:\
          \ Could not fetch model of type 'DependencyTreeModel' using Gradle distribution\
          \ 'https://services.gradle.org/distributions/gradle-5.4.1-all.zip'.\nCaused\
          \ by: LocationAwareException: Java heap space\nCaused by: OutOfMemoryError:\
          \ Java heap space"
        severity: "ERROR"
    has_issues: true

I've this setting in my global .gradle/gradle.properties file:
org.gradle.daemon=true
org.gradle.parallel=true
org.gradle.configureondemand=true
org.gradle.jvmargs=-Xmx6g -XX:MaxPermSize=4096m -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8

I think I give enough memory to Gradle or do I need to set the setting above in some other gradle.properties file? If I understood it correctly then, it fails to resolve the dependencies for 'build.gradle' because of that OutOfMemoryError.
I'm using:

Fedora 31 OS
OpenJDK 12
gradle -v output:

------------------------------------------------------------
Gradle 6.2
------------------------------------------------------------

Build time:   2020-02-17 08:32:01 UTC
Revision:     61d3320259a1a0d31519bf208eb13741679a742f

Kotlin:       1.3.61
Groovy:       2.5.8
Ant:          Apache Ant(TM) version 1.10.7 compiled on September 1 2019
JVM:          10.0.2 ("Oracle Corporation" 10.0.2+13)
OS:           Linux 5.5.5-200.fc31.x86_64 amd64

I looks like using ort analyze ... command downloads and uses gradle-5.4.1-all during the whole process instead of mine. So, maybe I need to put my changes into the gradle.properties file of the downloaded Gradle v5.4.1. What is the location of this file after it has been downloaded by the tool?

The ORT Analyzer uses the version of Gradle which has been configured to be used by the project being analyzed, via the Gradle wrapper. That is, in contrast to Maven, where you usually use the globally installed mvn command, Gradle projects usually specify by themselves which
version of Gradle they are supposed to be built with, similar to like they declare usual build dependencies. So from that perspective everything is fine, and Gradle downloads its distributions to ~/.gradle/wrapper/dists.

This post has some more memory setting hints for Gradle 5.

This post has some more memory setting hints for Gradle 5.

Thanks, it looks like I'll have to edit the gradle.properties file to set the needed Java memory parameters of the scanned project. I'll do it tomorrow and report the results.

This post has some more memory setting hints for Gradle 5.

Adding and modifying (for my needs) the suggested line in this comment to the scanned project gradle.properties file solved the OutOfMemoryError issue. This has bumped the size of my analyzer-result.yml file from ~404KB to ~581,7MB.


PEP8 changes

PEP8 changes

Django already lints with flake8 and a discretionary max line length of up to 120, there is no need to change these few lines to 80 chars.

Correct, the first commit isn't desired. The second commit might be useful, but I don't think the proposed naming of messages is a good one. I don't have another suggestion offhand.


Add protection middlewares

:speech_balloon: Describe your changes

Added middlewares to protect routes
Added tests for auth middlewares

:dna: Type of change

[ ] Bug fix
[x] New feature
[ ] Documentation update

:white_check_mark: Checklist before requesting a review

[x] I have performed a self-review of my code
[x] The code coverage is 100%

:bulb: Additional informations

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (307cd56) 100.00% compared to head (8b29abd) 100.00%.

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

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        14    +1
  Lines          199       211   +12
  Branches        31        35    +4
=========================================
+ Hits           199       211   +12

Impacted Files
Coverage Δ

src/middlewares/auth.ts
100.00% <100.00%> (ø)

src/routes/auth.ts
100.00% <100.00%> (ø)

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

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

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (307cd56) 100.00% compared to head (ddc734f) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        14    +1
  Lines          199       211   +12
  Branches        31        35    +4
=========================================
+ Hits           199       211   +12

Impacted Files
Coverage Δ

src/middlewares/auth.ts
100.00% <100.00%> (ø)

src/routes/auth.ts
100.00% <100.00%> (ø)

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

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

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (307cd56) 100.00% compared to head (ddc734f) 100.00%.

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

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        14    +1
  Lines          199       211   +12
  Branches        31        35    +4
=========================================
+ Hits           199       211   +12

Impacted Files
Coverage Δ

src/middlewares/auth.ts
100.00% <100.00%> (ø)

src/routes/auth.ts
100.00% <100.00%> (ø)

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

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

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (307cd56) 100.00% compared to head (0ca4342) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        14    +1
  Lines          199       211   +12
  Branches        31        35    +4
=========================================
+ Hits           199       211   +12

Impacted Files
Coverage Δ

src/middlewares/auth.ts
100.00% <100.00%> (ø)

src/routes/auth.ts
100.00% <100.00%> (ø)

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

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


[Snyk] Upgrade react-chord-diagram from 1.6.0 to 1.7.0

Snyk has created this PR to upgrade react-chord-diagram from 1.6.0 to 1.7.0.

As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2020-12-31.

Release notes

Package name: react-chord-diagram

1.7.0 - 2020-12-31

1.6.0 - 2020-03-14

    from react-chord-diagram GitHub release notes

Commit messages

Package name: react-chord-diagram

180ee92 Fix for no onclick
12e6cec Merge pull request #13 from langfordG/dependabot/npm_and_yarn/ini-1.3.8
fdc2b7c Bump ini from 1.3.5 to 1.3.8
3e6333c Merge pull request #11 from langfordG/dependabot/npm_and_yarn/elliptic-6.5.3
8b96ca2 Merge pull request #12 from langfordG/dependabot/npm_and_yarn/http-proxy-1.18.1
edbcfb4 Merge pull request #10 from langfordG/dependabot/npm_and_yarn/lodash-4.17.19
2460b8a Merge pull request #9 from langfordG/dependabot/npm_and_yarn/websocket-extensions-0.1.4
d2343f6 Bump http-proxy from 1.18.0 to 1.18.1
a295920 Bump elliptic from 6.5.2 to 6.5.3
469739e Bump lodash from 4.17.15 to 4.17.19
75a9f6b Bump websocket-extensions from 0.1.3 to 0.1.4
04a2ebd Update README.md

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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


fix(udf): fix decimal values

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR fixes #11823.
The problem is that Arrow has no data type equivalent to the DECIMAL (unconstrained numeric) in RisingWave/PostgreSQL. The decimal types in Arrow are fixed-point, while our decimal type is floating-point. Previously we used decimal(38,0) in arrow to represent decimal values. This is clearly wrong because it only accepts integer numbers.
This PR switches the arrow type for decimal to "large-binary". Values are stored as their string representations in the array, like [b"1.23", b"0", b"-inf"]. This may not be efficient in encoding, but is least error prone. And theoretically, this design can accept arbitrary-precision decimal values.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[x] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

This solution is too hacky to me. Converting to arrow not only used in udf, but also used in inserting to iceberg/parquet.

This solution is too hacky to me. Converting to arrow not only used in udf, but also used in inserting to iceberg/parquet.

But arrow does not have a type for floating-point decimal. It's impossible to use a certain type decimal array to store them.

This solution is too hacky to me. Converting to arrow not only used in udf, but also used in inserting to iceberg/parquet.

But arrow does not have a type for floating-point decimal. It's impossible to use a fixed type decimal array to store them.

Yeah, that's a problem. I think we need a discussion before deciding a solution.

This change breaks decimal handling in iceberg. Should be fixed in the future.


Don't trigger start animation when clicking on an button within anchor

Considering this case
<a href="/username">
    <button
        onClick={(e) => {
            e.stopPropagation();
            follow();
        }}>
        Follow
    </button>
</a>

Issue: When the button is clicked, the progress bar triggered
Expected: When the button is clicked, the progress bar should not do anything. There should be a way to prevent the progress bar to not being triggered

I am in the same boat (as mentioned here: https://github.com/TheSGJ/nextjs-toploader/issues/8)
@krehwell did you find any solution for your problem?

@baba43 ends up using this package instead: https://github.com/Skyleen77/next-nprogress-bar
works flawlessly for us

Should also work in https://github.com/TheSGJ/nextjs-toploader/issues/61#issuecomment-1853335459

I had similar case and this seems to fix the issue
e.nativeEvent.stopImmediatePropagation();
so instead of e.stopPropagation() try calling it.

I had similar case and this seems to fix the issue
e.nativeEvent.stopImmediatePropagation();  so instead of e.stopPropagation() try calling it.

Worked for me! 💯


Release 3.1.0

Release 3.1.0
New features

Register to events of other modules / apps to forward its content via TCP/IP

Improvements

Update triggerEvents / eventsToForward if parameters were loaded
Updated docu and added "Known issues" in README that data of registered events to be forwarded via TCP/IP are always transmitted as converted data type "string"

Bugfix

Did not convert data of registered events to strings before trying to transmit them via TCP/IP

Following tasks were checked

[ ] Module is coded and structured according to the "Developing guideline for modules" described within the CSK documentation
[x] All functions/events/parameters are documented within the manifest documentation
[x] The manifest description of the main CROWN includes main information about the purpose of the module and how to use it in general
[x] API docu based on the manifest was created and stored within the "docu"-folder of the repository
[x] Internal LUA code documentation exists for variables and non served functions
[x] All relevant infos are logged via the SharedLogger 'ModuleLogger'
[x] Module supports persistent data feature based on 'CSK_Module_PersistentData'
[x] Module supports user management based on 'CSK_Module_UserManagement'
[x] No open "ToDos" within the code or at least clearly explained comments why they exist...
[x] "Version" key in app manifest was updated following semantic versioning (and use '0.x.y' for test / experimental modules which are not yet ready to be officially released)
[x] Meaningful IDs used for UI elements
[x] Module was tested on an AppSpace device (at least on SICK AppEngine) with no error message
[x] README.md is up to date (incl. info of device + firmware the module was tested with)
[x] CHANGELOG.md is up to date

Hello Roy!
I have some minor comments to this pull request:

File "CHANGELOG.md":

Change "modules / apps" to "modules/apps" (or vice versa) as I have seen it without spaces at other positions or other modules. Would also suggest to use "SensorApps" instead of "apps".
Furthermore, please mention that only the first parameter of the event will be considered to be forwarded. Maybe this should be better placed within the docu.

File "CSK_Module_TCPIPClient/project.mf.xml":

Line 31:
change from "The same can be done via the UI step by step (see setTrigger(), setEventName(), addTriggerEventPairViaUI() ). +"
to "The same can be done via the UI step by step (see setTrigger(), setEventName(), or addTriggerEventPairViaUI()). +"
Line 215:
change from "Select a event to forward content to TCP/IP server via UI."
to "Select an event to forward the content of its first parameter to TCP/IP server via UI."
Line 219:
change from "Add a event to register to and to forward content (as string) to TCP/IP server."
to "Add an event to register to and to forward the content of its first parameter (as string) to TCP/IP server."

File "CSK_Module_TCPIPClient/scripts/Communication/TCPIPClient/TCPIPClient_Controller.lua":

Line 164:
change from "if selectedEventToForward == nil then"
to "if ( selectedEventToForward == nil or selectedEventToForward == "" ) then"
The selectedEventToForward can also be empty after string.sub(). Both cases can also be separated as above.
Change the log messages of the cases above to differentiate between check for nil and check for empty.
Line 185:
Where does the local variable "eventToForward" get its initial value? At least not in line 21. Is it done via the function "setEventToForward( ... )"?
Line 177 & 193:
What happens if success returns with false? The GUI will not consider this, only the log file will show the result.
The order within these two functions have been explicitly chosen this way to have the register/deregister before/after the GUI will be updated?

File "docu/CSK_Module_TCPIPClient.html":

Line 784:
Is the function "transmitCommando()" named this way on purpose or should it be "transmitCommand()"? In the latter case, you need to change the name of the function within the controller starting on line 269 and all its further appearances.
Line 788:
change from "To do so make use of "addEventToForward" (via script) or "addEventToForwardViaUI" (event needs to be preset via"setEventToForward")."
to "To do so make use of "addEventToForward" (via script) or "addEventToForwardViaUI" (event needs to be preset via "setEventToForward")."
Line 790:
change from "3) Wait for incomining triggers"
to "3) Wait for incoming triggers"
Line 969 & 1281:
These lines will change automatically with the changes in the manifest?

By the way if I am not wrong, this module is a single instance module? Will there come also a multi instance module?
Best regards, Ingo

Thx for the review Ingo!
I updated most of the issues.
3.4)
Yes, currently there is only feedback via the Log. Can be improved in future versions. If needed I would suggest to create an "Issue" for this feature within the repo. But I guess for the current version it is ok to keep it like it is.
4.1) "transmitCommand" would be better, you are right. But renaming would force a new major release, as other modules could not use this command anymore. From my point of view it is ok to keep it as it is....but maybe can be changed in future.
4.4) Yes, docu was updated via manifest.
And yes, this is currently a single instance module. Please create a new discussion if you need a module with multiple instances.


No validation set detected with validation_split:0

Hi Jake,
I have now  over 3h video footage from different animals doing different behaviors and I got the message No validation set detected with validation_split:0 so I changed it to 0.1.
Fit the model works  with these settings:
model.fit(
batch_size=12, #less than n_train from data generator
validation_batch_size=1, # less than n_val from data generator
callbacks=callbacks,
#epochs=1000, # Increase the number of epochs to train the model longer
epochs=200,
n_workers=1,
steps_per_epoch=500,
)
and the output from the training generator looks like this:
{'n_train': 19,
'n_validation': 2,
'validation_split': 0.1,
'downsample_factor': 3,
'output_shape': (48, 48),
'n_output_channels': 88,
'shuffle': True,
'sigma': 5,
'output_sigma': 0.625,
'use_graph': True,
'graph_scale': 1,
'random_seed': 1,
'augmenter': True,
'datapath': '/home/einat/deepposekit-data/datasets/whipspider/annotation_data_release.h5',
'dataset': 'images',
'generator': 'DataGenerator',
'n_samples': 21,
'image_shape': (384, 384, 1),
'keypoints_shape': (43, 2)}
Output from fit the model
Epoch 1/200
13/500 [..............................] - ETA: 2:36:46 - loss: 178.4359 - output_0_loss: 71.6774 - output_1_loss: 56.4588 - output_2_loss: 50.2997
Epoch 1/200
405/500 [=======================>......] - ETA: 29:31 - loss: 52.7862 - output_0_loss: 19.2698 - output_1_loss: 17.0771 - output_2_loss: 16.4393
Does this looks good?
Cheers, Philip

Yes, if you set validation split to 0 then there will be no validation set. If the loss is decreasing then training is working. You should assess the quality of the results using the notebooks for step 4.


PetShop: Implement search functionality

Implemented basic search functionality.
Added noSearch image.
Refactored functions to minimise code-duplication.

Done, thank you for your contribution Ujwal.🚀


Swap windows with mouse

Since the mouse resize WinEvent is shared with mouse moving, I may as well implement a handler for moves where the window that a dragged window is placed over is treated as a swap target.

Closed with #9.


Feature request : allow printing in console

Hi, if I paste the following code
import data.nat
open nat

eval 2 + 3

into the console at https://leanprover.github.io/tutorial/index.html and click the compile button, nothing is printed out. I just get :
-- Processing...
-- Saved at cookie.
-- Done(0.784sec)

Would it be possible to have the results of eval printed out in the console? Thanks

When you click the compile button, you should see a small "i" appear next to the line number for the eval. If you hover over that "I", you can see the output.
Is that sufficient for you?

@avigad Fantastic thanks, yes that is good enough for me.

BTW, if the output is big, then "i" hover text is not very useful.
The problem is that the "balloon" does not fit in the screen.
This is recurrent issue when I'm making demos :-(
Here is a simple problematic example
import data.vector
print definition vector.map2

@avigad Hi, reopening the feature request following the comment by @leodemoura . Maybe an alternative would be to have the "i" output scrollable. Thanks

I think we need to have a setting pane (with an icon something like this) which allows us to set options. It should include at least the two things (and more):

Memory Size : int (64MB is default)
Output to Console : bool

We should save the changes in the cookie not to repeat setting things up. I will implement them when I have some time. Please feel free to add more options to have.

@leodemoura's example is pretty big and I'm not sure what is a good way to show this. Scrollable div might provide a bad UX because the editor itself is also scrollable and the two might interfere. I'll think about it more. Any suggestion is welcome of course.

@soonhokong Hi, that sounds like an excellent idea. Thanks

@soonhokong An additional option that might be useful : word wrap (or alternatively horizontal scroll on command window)
For me check (λ(Q R S : Type) (v : R → S) (u : Q → R) (x : Q), v (u x)) A B C g f a goes off the screen
Thanks

Another thing about only showing output in a floating "balloon" is that it is hard (impossible?) to copy and paste from it.

Now there is a setting pane where you can set this as an option.


UIAPPS-164 Start custom widget with 3rd-party data

Motivation

We want to make a template for a custom widget that interacts with 3rd-party data. This should continue to grow and change as features evolve.
Jira Issue: https://datadoghq.atlassian.net/browse/UIAPPS-164

Changes

We create an example we can use as a template that solves a specific use case: an App where there's a custom widget that interacts with 3rd-party data.
We're following a lot of the setup from the custom widget with 1st-party data from https://github.com/DataDog/apps/pull/120.

Testing

This is likely best tested once we have support for templates in the Datadog UI. We are almost assuredly going to need to make adjustments to this template afteward.

Releases
Choose one:

[x] No release is necessary.
If you're only updating examples/documentation, this is likely what you want.
[ ] All packages that need a release have a changeset.

Current dependencies on/for this PR:

master

PR #125   (this PR)

This comment was auto-generated by Graphite.

Mind giving it another look?

Thanks for the review!


Tooling plans and technical proposal

What?
I wrote a draft of a technical proposal for tooling and a tentative roadmap, as well as a strategy to succeed.
Please, review and leave your comments.

@rmelian yes, Rust also has this property.

@MikeRalphson allow me to spent 5 mins reading about RUST :)

Have 10!

I read here https://matthias-endler.de/2017/go-vs-rust/ Rust has a steeper curve than Golang

@rmelian interesting article. It twice mentions C interoperability and FFI as strengths of Rust and never mentions the subject for Go...
From here:

Rust was designed to interoperate with C. Go can as well, but gives up a lot to achieve this goal, and it’s not really its focus.

I'm not sure yet what is meant by "gives up a lot"...

I feel we would be more productive with Golang than with Rust, as @MikeRalphson has some basics, also @fmvilas. In my case, I already started learning it so If go with Rust(I only know the name) it will take us a lot more time to do something real with it.

I see Rust as an option to keep an eye on for future implementations. I'm not worried so much about the capabilities of the language we choose. Almost every language offers what we want to achieve. However, if we want to invite people to contribute to AsyncAPI, Rust will make it harder. Not because of the language, but because the community and adoption are not there yet. I'm conscious this is a chicken-egg problem: if nobody uses Rust, the community will not grow and it will make others decide not to choose it.
That said, the parser is such a key piece in the tooling I don't feel comfortable trying something that's not widely adopted. However, there will be other pieces of software where we may want to use Rust, like schema format parsers. Nothing prevents us from starting with them and eventually migrating the parser to Rust if that's what we want.

It is actually not included in the release. This was an unintentional release caused by the release automation initial run. 1.0 version of the spec was released a long time ago. The release mentioned by the bot was removed.


Fix auto merge conflict 4298 [skip ci]

fix merge conflict #4298
NOTE: this must be merged as a merge commit

build


TEST PR

jenkins: depends on https://eos2git.cec.lab.emc.com/OnRack/on-build-config/pull/46

BUILD on-dhcp-proxy #13 : FAILURE

BUILD peter-test » on-dhcp-proxy-test #25 : FAILURE

BUILD on-dhcp-proxy #17 : FAILURE

BUILD on-dhcp-proxy #5 : FAILURE


fix: swapped CRF and max bit rate in logs

This commit fixes a regression that was introduced in 389681329d951d6f64a48259f9798c06899b346f, where the wrong format specifier was updated for extended CRF, leading to CRF and MBR values being swapped in logs.

Before:
$ SvtAv1EncApp -i blue_sky_1080p25.y4m --mbr 1100k -b output.ivf
[…]
Svt[info]: SVT [config]: BRC mode / rate factor / max bitrate (kbps)			: capped CRF / 1100 / 35.00
                                                                                ^^^^^^^^^^^^
                                                                                swapped

After:
$ SvtAv1EncApp -i blue_sky_1080p25.y4m --mbr 1100k -b output.ivf
[…]
Svt[info]: SVT [config]: BRC mode / rate factor / max bitrate (kbps)			: capped CRF / 35.00 / 1100

Thanks! Fixed in 7e412a42


Wrong suffix for placements and decisions

Examples in  /main/help/using/offers/api-reference/offer-delivery-api/decisioning-api.md have incorrect prefixes for placements and decisions, such as

xcore:offer-placement:ffed0456
xcore:offer-activity:ffed0123

These old "xcore" prefixes do not work anymore. Please change to the new "dps" prefix.

hi @jvaldas, thanks for your feedback.
We'll update docs shortly.
Internal reference: DOCAC-11164


Unable to read from work_peer ::ffff:<IP_ADDRESS> 7075: Connection reset by peer (104)

Description of bug:

Steps to reproduce the issue:
1.build docker image with 'test' option:
./docker/node/build.sh -n test
source from: https://github.com/nanocurrency/raiblocks/issues/443
2.run two docker containers with the image built in the first step, input different port number:
docker run --name nano1 -p 7001:7075/udp -p 7001:7075 -p <IP_ADDRESS>:7011:7076 -v $(pwd):/root/RaiBlocksTest raiblocks-node-test
docker run --name nano2 -p 7002:7075/udp -p 7002:7075 -p <IP_ADDRESS>:7012:7076 -v $(pwd):/root/RaiBlocksTest raiblocks-node-test
3.use RPC protocol to add work peers and bootstrap:
curl -d '{"action": "work_peer_add","address": "::ffff:<IP_ADDRESS>","port": "7075"}' <IP_ADDRESS>:7011
curl -d '{"action": "bootstrap", "address": "::ffff:<IP_ADDRESS>", "port": "7075"}' <IP_ADDRESS>:7011
'::ffff:<IP_ADDRESS>' is the ip address of one docker container.
4. when I try to send a RPC request to one node, I got this:
Describe the results you received:
when I add work peers, I got this:
[2018-10-22 05:52:14.056383]: 0x7f18b4000980 {"action": "work_peer_add","address": "::ffff:<IP_ADDRESS>","port": "7075"} [2018-10-22 05:52:14.057011]: RPC request 0x7f18b4000980 completed in: 660 microseconds [2018-10-22 05:57:06.793769]: Starting bootstrap attempt [2018-10-22 05:57:06.794616]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.795618]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.796194]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.796995]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.797559]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.798149]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.798809]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.799354]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.799986]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.800613]: Bootstrap stopped because there are no peers [2018-10-22 05:57:06.801241]: Exiting bulk push client [2018-10-22 05:57:06.801916]: Exiting bootstrap attempt [2018-10-22 06:00:52.837150]: Found a representative at [::ffff:<IP_ADDRESS>]:7075 [2018-10-22 06:00:52.838427]: Frontier sending finished [2018-10-22 06:02:06.007801]: Frontier sending finished [2018-10-22 06:02:06.795187]: Starting bootstrap attempt [2018-10-22 06:02:06.797426]: Received 1 frontiers from [::ffff:<IP_ADDRESS>]:7075 [2018-10-22 06:02:06.798485]: Completed frontier request, 0 out of sync accounts according to [::ffff:<IP_ADDRESS>]:7075 [2018-10-22 06:02:06.799370]: Completed pulls [2018-10-22 06:02:06.800043]: Bulk push finished [2018-10-22 06:02:06.800702]: Exiting bulk push client [2018-10-22 06:02:06.801444]: Exiting bootstrap attempt
when I send wallet add request, I got this:
[2018-10-22 06:22:36.893584]: Beginning pending block search [2018-10-22 06:22:36.895099]: Pending block search phase complete [2018-10-22 06:22:37.262700]: 0x7f18bc000a90 {"action": "wallet_add", "wallet": "1F68B3E5A07C36482E8BB611531E4B55B70129B32F277F0A5B89C7992A10C939", "key": "34F0A37AAD20F4A260F0A5B3CB3D7FB50673212263E58A380BC10474BB039CE4"} [2018-10-22 06:22:37.264385]: Unable to read from work_peer ::ffff:<IP_ADDRESS> 7075: Connection reset by peer (104) [2018-10-22 06:22:37.265631]: Work generation complete: 2118 us [2018-10-22 06:22:37.267023]: RPC request 0x7f18bc000a90 completed in: 4488 microseconds
Describe the results you expected:
I expect the two nodes can connect and sync.
Additional information you deem important (e.g. issue happens only occasionally):
Environment:

OS: macOS High Sierra
Darwin Kernel Version 17.5.0
Node version: 8
logs

Regarding the reserved ranges, in version 18+ it will be possible to override this exclusion using a config option created as part of pull request #1370 called "allow_local_peers".

Seems like https://github.com/nanocurrency/raiblocks/pull/1370 solves this as the IP used is in a reserved range.


chore: add optimism rpc url to envs

Fixes integration tests by:

Replacing optimism fallback rpc url from https://optimism.gateway.tenderly.co (was failing) to https://optimism.llamarpc.com (more stable according to https://chainlist.org/chain/10)

Defining OPTIMISM_RPC_URL in CI and vitest setup.

The last step would be defining the OPTIMISM_RPC_URL env in the repo secrets.

@johngrantuk Do you have permissions to add OPTIMISM_RPC_URL to the secrets?

@johngrantuk Do you have permissions to add OPTIMISM_RPC_URL to the secrets?

Yeah, I've added it now. Thanks for fixing it!


added supoort for metrics MDS

Description
MDS support for metrics analytics
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://github.com/opensearch-project/dashboards-observability/assets/107440906/b025fe73-8111-4c51-8f7b-ef825cfcaa53


An error ocurred while starting the kernel

An error ocurred while starting the kernel
I run the spyder ide , in the ipython-console shows like this below:
An error ocurred while starting the kernel
Traceback (most recent call last):
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 231, in
main()
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 212, in main
from utils.ipython.spyder_kernel import SpyderKernel
ImportError: No module named ipython.spyder_kernel
Traceback (most recent call last):
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 231, in
main()
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 212, in main
from utils.ipython.spyder_kernel import SpyderKernel
ImportError: No module named ipython.spyder_kernel
Traceback (most recent call last):
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 231, in
main()
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 212, in main
from utils.ipython.spyder_kernel import SpyderKernel
ImportError: No module named ipython.spyder_kernel
Traceback (most recent call last):
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 231, in
main()
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 212, in main
from utils.ipython.spyder_kernel import SpyderKernel
ImportError: No module named ipython.spyder_kernel
Traceback (most recent call last):
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 231, in
main()
File "/home/lucask/anaconda2/lib/python2.7/site‑packages/spyder/utils/ipython/start_kernel.py", line 212, in main
from utils.ipython.spyder_kernel import SpyderKernel
ImportError: No module named ipython.spyder_kernel
I have tried use this command :  conda update setuptools
but it still shows this problem , anyone knows how to deal with it ?
Many Thanks!!!

Do you have a module called utils in your current working directory?

Closing because of lack of response.

@ccordoba12 - I have this issue too and yes I do have a module called utils in my wd. Does this mean I can't use spyder for any project with a utils module?

@JamesOwers, yes, that's right. We'll fix this in Spyder 4, sorry for the inconvenience in the meantime.

Thanks for response ok - I've moved my module to my_utils and refactored code...but I imagine this would be very annoying for established libraries being developed e.g. sklearn! What sort of date are you looking at for Spyder 4?

There's no clear release date, sorry. Please read
https://github.com/spyder-ide/spyder/wiki/Anaconda-stopped-funding-Spyder
for the details.

I also get
main()
File "/usr/lib/python3/dist‑packages/spyder/utils/ipython/start_kernel.py", line 239, in main
from spyder.utils.ipython.spyder_kernel import SpyderKernel
ModuleNotFoundError: No module named 'spyder.utils.ipython.spyder_kernel
when starting an ipython console after updating spyder to the newest version.
I no longer have a utils.py file in my working directory and still have the the problem

@bmilde, from your error above, it seems clear that you installed a new Spyder version over or next to an older one. That could have corrupted your Linux installation, which will force you to reinstall. You also need to uninstall the old version (no idea how, you were the one that installed it) to let the new one work correctly.

Thank you! The problem was indeed related to a corrupt installation from
having both the ubuntu version of spyder (after a recent dist-upgrade) and
the pip version installed. After removing the apt version, spyder works now
again!
Btw, does the limitation to have no local utils module on the current
wokring dir still exist, or is this solved now?
Carlos Cordoba<EMAIL_ADDRESS>schrieb am Mi., 20. März 2019,
10:18:

@bmilde https://github.com/bmilde, from your error above, it seems
clear that you installed a new Spyder version over or next to an older one.
That could have corrupted your Linux installation, which will force you to
reinstall. You also need to uninstall the old version (no idea how, you
were the one that installed it) to let the new one work correctly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/5233#issuecomment-474748796,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJRgelLr7rKQII3W78_1s5azZ2aW5Sllks5vYfzvgaJpZM4PW9vo
.

Btw, does the limitation to have no local utils module on the current wokring dir still exist, or is this solved now?

It should be solved now.


Use both website name and content textId for lookup

Pre-Flight checklist

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

What are the relevant tickets?
Fixes #429
What's this PR do?
We were using textId as the only lookup value for our frontend content detail data. However we also need to include the website name to make it a unique key. This updates the key to
How should this be manually tested?

Find or create two courses based on the ocw-course starter. Ideally they should be right next to each other in the site listing
Add some content to the metadata for each of the two sites. It doesn't matter what fields you update, it just needs to be distinct for each site.
Go to /sites and find the link for the first of the two sites. Click on that link, then click on "Metadata". The metadata fields should look correct.
Click the back button repeatedly until you get back to /sites. Click on the link for the second site then click on "Metadata". You should see the correct data for the website, which is different from the metadata for the first site.
Additionally, go to a page listing page and make sure things still work in general

Codecov Report

Merging #435 (bfcf0e4) into master (437fa8f) will decrease coverage by 3.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #435      +/-   ##
==========================================
- Coverage   90.34%   87.27%   -3.08%
==========================================
  Files         138       66      -72
  Lines        4745     2852    -1893
  Branches      741      338     -403
==========================================
- Hits         4287     2489    -1798
+ Misses        410      317      -93
+ Partials       48       46       -2

Impacted Files
Coverage Δ

static/js/util/factories/website_collections.ts

static/js/components/forms/SiteContentForm.tsx

static/js/test_util.ts

.../js/components/forms/WebsiteCollectionItemForm.tsx

static/js/components/widgets/MenuField.tsx

static/js/pages/SiteCreationPage.tsx

...atic/js/components/forms/WebsiteCollectionForm.tsx

static/js/components/forms/FormError.tsx

static/js/components/SiteContentListing.tsx

static/js/pages/WebsiteCollectionsPage.tsx

... and 57 more

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 437fa8f...bfcf0e4. Read the comment docs.


Plugin UI window not showing

Some plugin windows never show but only show a title bar (x42-comp for example), starting these UI's makes synthpod_ui output the following:
(UI)  [Error] _mod_uis_send: buffer overflow

Some plugins (zynaddsubfx for example) render around half of the times after starting the plugin UI, which gives no output from synthpod_ui when the UI does not render.

other examples of x11 plugins that don't show up are all the plugins from openAV.

zyn fusion, x42-comp and openAV plugins all show and render fine here (on Parabola GNU/Linux).
Do they show fine in other hosts on your side ?
Is this the zyn fusion UI you're referring to, or the oldone ?

(UI)  [Error] _mod_uis_send: buffer overflow

Yes, I can confirm this. There's something wrong here about shared memory communication, but has no influence on plugin rendering.
Will need some investigation...

Is this the zyn fusion UI you're referring to, or the oldone ?

The buffer overflows are coming from x42-comp stereo

Do they show fine in other hosts on your side ?

Yep, they show up fine in carla / ardour

Is this the zyn fusion UI you're referring to, or the oldone ?

I'm running zyn-fusion (new ui) git version of today (3.0.5-1)
I'm running arch with xfce window manager. The plugin windows are visible in window manager's app switcher, but the preview seems to point to random memory as that's a garbled mess of artifacts.

(on Parabola GNU/Linux).

Cool stuff, never heard of that before!

Ok, so let's test whether the UIs show just standalone and keep synthpod out of the question.
synthpod runs UIs in a separate process, so we can actually test them without running synthpod.
In 6ddde21fcb9fe2ea5738ce9340ae846dd465533f there's some new debugging output, which writes out the command line, which runs the UI.
 Out of curiosity, what tiling WM are u using?

Parabola is Arch minus all non-free stuff, so our sytems should be pretty similar (which puzzles me even more).
WM is awesome [1]
[1] https://awesomewm.org/

so, I've switched to xfce and the OpenAV UIs show with the wrong size, e.g. not high enough and thousands of pixels wide. So can kind of reproduce.

There was a structure which was not properly initialized.
Fixed in 25c9f0c18e39d65c37f88918df8407a18a7bc286
Please try it out.

Beautiful GUI's! Beautiful GUI's everywhere!
Thanks a bunch for switching WM & testing this! Problem solved.


Approaches to integrating with start-app

Since start-app is such a nice starting point for app development, it's natural, when writing another module, to want to be able to integrate with start-app in a nice way. That is, it's nice to make it easy for your module to be used in conjunction with start-app. I've run into a bit of a puzzle doing that, which leads to both a specific and a general question.
The background (which I'll try not to dwell on) is that I'm working on a module to help with single-page-app routing of the location hash -- where the "routing" goes in both directions -- that is, where changes to your model may generate changes to the browser's location hash, and changes to the browser's location hash may generate actions within your app. (So, this builds on TheSeamau5/elm-history and is related to TheSeamau5/elm-router, but has somewhat different goals).
Now, the implementation is, of course, more complex than this, but what my module ultimately provides (conceptually) is two things.

History.hash -> Signal action

That is, we wire up the signal graph so that changes to the browser's location hash are mapped to actions your app performs.

Signal model -> Signal (Task () ())

Where changes to your model are mapped to changes to the browser's location hash.
Now, it turns out to be important that these two things are inter-related, in the sense that we need to avoid infinite loops -- that is, we need to avoid a situation where changing the model triggers a change to the location hash, which in turn triggers a change to the model, ad infinitum. That's not hard to avoid, but it does mean that the two things the module provides are inter-related -- that is, they can't be done entirely separately -- they are generated together.
So, how to nicely integrate this with start-app? Now, each of these things are easy to integrate with start-app if handled separately. One of the things which StartApp.start provides is the Signal model, so I can suggest using that as input to my module. And, one of the things which StartApp.start takes as a parameter is inputs : List (Signal action), so I can suggest supplying what my module provides as part of that list.
However, you may have spotted the chicken-and-egg problem already. In order to produce the Signal action, my module needs the Signal model. However, in order to get the Signal model from StartApp.start, my module needs to provide the Signal action as a parameter. So, it won't work -- and I'm pretty sure that this is a necessary consequence of what I'm trying to achieve.
Now, one way to untie the knot would be for start-app to expose the Mailbox action which it creates internally. If that were available as part of the result of StartApp.start, then it would be possible to send the actions generated by my module to that address, without the chicken-and-egg problem.
So, I suppose the specific question I have is whether start-app might consider exposing its Mailbox action. Now, I should say that there may be perfectly sound reasons why start-app doesn't want to do that -- I can imagine what some of those reasons might be.
The more general question is how module authors ought to think about start-app. Should we treat start-app as, well, a starting point, that people might move beyond fairly readily, if needed? Or, should we try really hard to maintain integration with start-app, including suggesting changes to start-app if that's needed? I should say that I'm not advocating one way or the other -- I can see plausible reasons for either approach.

The module I mentioned above is now published, in case anyone is interested.
https://github.com/rgrempel/elm-route-hash
However, I should make it clear that this issue isn't primarily about the details of that module. It's more a question of how module authors ought to think about start-app in general -- that is, to what lengths ought we go in order to integrate with start-app (vs. asking users to move beyond start-app when needed).

Just to follow up with a little more context, here are the specific changes to StartApp that would expose the Address:
https://github.com/rgrempel/elm-route-hash/commit/77228f25de1e05f419839ed9f63a51e046f84493#diff-0017ac34b434e83f80d5140dc35b727c
So, in part the question is whether a change of this type might be considered for StartApp. But, the wider question is what role we should envision for StartApp within the Elm community as it develops. That is, should we should conceive of StartApp as something which, in the normal course of events, developers will make local copies of and modify in a variety of ways? Or should we conceive of it as something which ideally is used unchanged?

I'm confused about this too. start-app is marketed as the blessed path, but the community seems to treat it as a crutch for beginners. I'm hitting the same issue trying to implement elm-router. None of the examples use start app and I have no idea how to create an app without it.

FWIW, I think that start-app should indeed cater for more advanced use. We could indeed copy it and create a custom startApp, but then it forces us to keep an eye on the original one, and sync back the changes.
btw:

I'm hitting the same issue trying to implement elm-router

@seanhess , you should check elm-router-hash which has wonderful docs + examples based on start-app/ Elm architecture.

@rgrempel maybe worth creating a PR for this - so it would be easier for others to see the small changes that you suggest

Thanks @amitaibu I will. I was initially turned off by having to fork start-app, and that it seemed over complicated, but I'll give it another shot.

I ended up figuring out how to make elm-route-hash work with an unmodified StartApp -- see
https://github.com/rgrempel/elm-route-hash/commit/887b03300899600cdebab83522582a7246029d20
So, I'll close this issue -- it may still be of some general interest, but I no longer have an example that illustrates it.


Propose for inclusion in swift-lang

I think this library and pattern is 'swifty' enough to be part of Swift-lang itself!
Perhaps the syntax would change to be like an auto closure.
I speculate the compiler could do something with value-types being treated as immutable after being returned in some cases.

Awesome, sounds interesting. What could I do?

Start a thread on the mailing list to see what other people think, and start writing the proposal using the swift evolution template. Best of luck

Thank you! I'll take a look soon :smile:

If you do create a proposal, it would be better if Swift would do this without having to use the ".then" part (or maybe it was named something else).
The reason for that is because I love to use "Promises" for any async operations (take a look at a lib like PromiseKit http://promisekit.org/ ). Promises usually use .then(), so I'm not sure if there would be conflicts when using this lib.

@devxoul I'd suggest you to take a look at this thread https://lists.swift.org/pipermail/swift-evolution/Week-of-Mon-20151228/005122.html

@adrfer, Cool. Someone already done it. How can I lend weight to this suggestion?

@devxoul perhaps, after reading the related threads, take a look at @erica's https://gist.github.com/erica/eb32feb22ba99629285a
Sure you guys can put together and submit a great proposal 👌

@adrfer Thanks for making everyone aware of each other! @devxoul Done it? Far from it, but please do contribute on the mailing list. @erica's proposal looks nearly complete but I could not find it submitted on the Swift Evolution Repo and has not been edited for a few weeks (Need some collaborators?). We each have slightly different models with different goals. Here is the Then example code in the three different styles:
Mailing List "Inline Initializer" Restricted to initializers similar to then. However it acts as an initializer compared to code after (possible advantages). Also avoids $0 and has some ideas for handling self.
let label = UILabel {
    textAlignment = .Center
    textColor = .blackColor()
    text = "Hello, World!"
}

@erica's "Method Cascading" More comprehensive "method cascading" proposal. Applies to much more than initializers. Introduces "with" keyword. self is the label object.
with let label = UILabel() {
    textAlignment = .Center
    textColor = .blackColor()
    text = "Hello, World!"
}

Then Possible syntax proposal, not sure how you were envisioning it but this is similar to how you had it. Maintains self.
let label = UILabel() then {
    label.textAlignment = .Center
    label.textColor = .blackColor()
    label.text = "Hello, World!"
}

So can we come to an agreement, create a proposal, and submit it for review? Perhaps we can build off of the "Method Cascading" and bring in maintaining self and drop the with keyword? Perhaps the with is only required when self is desired?

Then, or the concept of Inline Initializer, really looked very promising at first glance to me. Because in many projects I set properties of a UI in its lazy initialization.
However, soon I found out that compared to lazy initialization of a var, then is almost useless in most use cases because you cannot reference self in a then block but you can do that in lazy initialization blocks.
For example, I can reference another instance var in a controller or a view, or assign the controller itself self to be the datasource or the delegate of a UITableView or UICollectionView, when using lazy initialization blocks.

Chris Lattner asked that it be submitted as a bug report instead of advancing as a proposal. It was. https://bugs.swift.org/browse/SR-160 https://bugs.swift.org/browse/SR-160
-- E

On Jan 4, 2016, at 2:20 AM, Nicholas T<EMAIL_ADDRESS>wrote:
Then, or the concept of Inline Initializer, really looked very promising at first glance to me. Because in many projects I set properties of a UI in its lazy initialization.
However, soon I found out that compared to lazy initialization of a var, then is almost useless in most use cases because you cannot reference self in a then block but you can do that in lazy initialization blocks.
For example, I can reference another instance var in a controller or a view, or assign the controller itself self to be the datasource or the delegate of a UITableView or UICollectionView, when using lazy initialization blocks.
—
Reply to this email directly or view it on GitHub https://github.com/devxoul/Then/issues/2#issuecomment-168618644.

@wcatron, thanks for asking. Of course I hope it to be adopted to Swift grammar :Smile:

It would be much less a PITA to have something like Kotlin's with() plus IDE support.


[As per document] Bank fee should be Two Points

Bug Description
Bank fee should be Two Points
Steps to Reproduce

Go to Send Points
In resulting popup check "Bank fee"

Actual Result
Bank fee is 1 point
Expected Result
[As per document] Bank fee should be Two Points
OS version
Windows
Screenshot

"Lastly, banks will collect transaction fees from all blocks sent through them. The transaction fees are determined by the individual banks. This will give users the ability to choose banks with the most reliable service and who offer the lowest transaction fees."
It does have to be 2. Banks can decide the transaction fees for themselves.


Non-zero estimate_u_values

Closes #161
Trivial fix to ensure estimate_u_probs returns at least 1/target_rows for all u values (equivalent to 1 comparison found for that gamma level)
Warning to users on manually setting any m_probabilities or u_probabilities to zero.

Nice.  Can you:
(1) increment splink version in toml file
(2) expand the comment in the code to include a hyperlink to the issue: https://github.com/moj-analytical-services/splink/issues/161