Import order incorrect

Hi
Currently, the way the import is handled cause that the application will not work with pre-commit tools like isort.
For example, in your tutorial, you establish that the init.py for /app should be (tag v0.4)
from flask import Flask
from flask_sqlalchemy import SQLAlchemy
from flask_migrate import Migrate
from config import Config

app = Flask(__name__)
app.config.from_object(Config)
db = SQLAlchemy(app)
migrate = Migrate(app, db)

from app import routes, models

Using isort changes it to
from flask import Flask
from flask_migrate import Migrate
from flask_sqlalchemy import SQLAlchemy

from app import models, routes
from config import Config

app = Flask(__name__)
app.config.from_object(Config)
db = SQLAlchemy(app)
migrate = Migrate(app, db)

This will break the entire app.
flask.cli.NoAppException
flask.cli.NoAppException: While importing "microblog", an ImportError was raised:

Traceback (most recent call last):
  File "/home/the-box/Github/flask-mega-tutorial/.venv/lib/python3.9/site-packages/flask/cli.py", line 240, in locate_app
    __import__(module_name)
  File "/home/the-box/Github/flask-mega-tutorial/microblog.py", line 1, in <module>
    from app import app
  File "/home/the-box/Github/flask-mega-tutorial/app/__init__.py", line 5, in <module>
    from app import models, routes
  File "/home/the-box/Github/flask-mega-tutorial/app/models.py", line 2, in <module>
    from app import db
ImportError: cannot import name 'db' from partially initialized module 'app' (most likely due to a circular import) (/home/the-box/Github/flask-mega-tutorial/app/__init__.py)

Yes. Flask is particularly sensitive to circular imports, sometimes the imports need to be carefully designed to avoid this. I suggest you add an exception so that isort does not alter the import order in this file.

I have the same problem. How do we go about it?

@SamwayneKE I'm not sure I understand what you mean. Not using isort is the solution.

I have an error as above. NoAppException.
Cannot import email from app.emails

@SamwayneKE I can't really help with the little information you have provided, sorry. This is likely a standard circular dependency error so you have to review your imports.

This is my error!
This is my imports in routes.py
from flask import render_template, flash, redirect, url_for, request
from werkzeug.urls import url_parse
from app import app, db
from app.forms import LoginForm, EditProfileForm, EmptyForm, RegistrationForm, PostForm, ResetPasswordRequestForm
from flask_login import current_user, login_user, logout_user, login_required
from app.models import User, Post
from datetime import datetime
from app.email import send_password_reset_email
flask.cli.NoAppException
flask.cli.NoAppException: While importing "biblechats", an ImportError was raised:
Traceback (most recent call last):
File "/home/samuel/Desktop/biblechats/venv/lib/python3.8/site-packages/flask/cli.py", line 240, in locate_app
import(module_name)
File "/home/samuel/Desktop/biblechats/biblechats.py", line 1, in
from app import app
File "/home/samuel/Desktop/biblechats/app/init.py", line 52, in
from app import routes, models, errors
File "/home/samuel/Desktop/biblechats/app/routes.py", line 8, in
from app.email import send_password_reset_email
ImportError: cannot import name 'send_password_reset_email' from 'app.email' (/home/samuel/Desktop/biblechats/app/email.py)
Traceback (most recent call last)
File "/home/samuel/Desktop/biblechats/biblechats.py", line 1, in
from app import app
File "/home/samuel/Desktop/biblechats/app/init.py", line 52, in
from app import routes, models, errors
File "/home/samuel/Desktop/biblechats/app/routes.py", line 8, in
from app.email import send_password_reset_email
During handling of the above exception, another exception occurred:
File "/home/samuel/Desktop/biblechats/venv/lib/python3.8/site-packages/flask/cli.py", line 343, in call
rv = self._load_unlocked()
File "/home/samuel/Desktop/biblechats/venv/lib/python3.8/site-packages/flask/cli.py", line 330, in _load_unlocked
self._app = rv = self.loader()
File "/home/samuel/Desktop/biblechats/venv/lib/python3.8/site-packages/flask/cli.py", line 388, in load_app
app = locate_app(self, import_name, name)
File "/home/samuel/Desktop/biblechats/venv/lib/python3.8/site-packages/flask/cli.py", line 245, in locate_app
raise NoAppException(
flask.cli.NoAppException: While importing "biblechats", an ImportError was raised:
Traceback (most recent call last):
File "/home/samuel/Desktop/biblechats/venv/lib/python3.8/site-packages/flask/cli.py", line 240, in locate_app
import(module_name)
File "/home/samuel/Desktop/biblechats/biblechats.py", line 1, in
from app import app
File "/home/samuel/Desktop/biblechats/app/init.py", line 52, in
from app import routes, models, errors
File "/home/samuel/Desktop/biblechats/app/routes.py", line 8, in
from app.email import send_password_reset_email
ImportError: cannot import name 'send_password_reset_email' from 'app.email' (/home/samuel/Desktop/biblechats/app/email.py)
The debugger caught an exception in your WSGI application. You can now look at the traceback which led to the error.
To switch between the interactive traceback and the plaintext one, you can click on the "Traceback" headline. From the text traceback you can also create a paste of it. For code execution mouse-over the frame you want to debug and click on the console icon on the right side.
You can execute arbitrary Python code in the stack frames and there are some extra helpers available for introspection:
dump() shows all variables in the frame
dump(obj) dumps all that's known about the object
Brought to you by DON'T PANIC, your friendly Werkzeug powered traceback interpreter.

@SamwayneKE so you know how to debug this on your own then?

I think you just need to move
'from app import app, db' to the last line of the module, just to make sure that everything else is initialized.

@SamwayneKE so you know how to debug this on your own then?

Nop still the same error

@SamwayneKE it's because you have the import order incorrect (most probably on the init.py files), just verify which tag is the corresponding chapter you are in and make sure the imports are in that correct order.

@SamwayneKE it's because you have the import order incorrect (most probably on the init.py files), just verify which tag is the corresponding chapter you are in and make sure the imports are in that correct order.

exactly happens with me in init.py file my import order was misplaced so do remember imports order matters in flask

I have this problem, can anyone help? I'm stuck there.
Traceback (most recent call last):
File "E:\Pythoshka\Web-site\main.py", line 1, in
from website import create_app
File "E:\Pythoshka\Web-site\website_init_.py", line 1, in
from website import models
File "E:\Pythoshka\Web-site\website\models.py", line 2, in
from . import db
ImportError: cannot import name 'db' from partially initialized module 'website' (most likely due to a circular import)

I'm closing this issue, since it isn't related to the project. Please look for standard advice dealing with circular dependencies.


Only modify width and height when image is loaded or a canvas

Fixes #14575
Supersedes #14601

What happens if scale is set after loading, e.g.
      original.getImage(1).addEventListener('load', () => {
        const clone1 = original.clone();
        expect(original.getScale()).to.eql(clone1.getScale());
        original.setScale(3);
        expect(original.getScale()).to.be(3);
        const clone2 = original.clone();
        expect(original.getScale()).to.eql(clone2.getScale());
        done();
      });

Won't 3 be changed to [3, 3] ?

@mike-000 Yes it will, but that won't affect the visual representation of the icon.

@mike-000 Yes it will, but does it matter? It does not make a difference in the visual representation of the icon.

@mike-000 Yes it will, but does it matter? It does not make a difference in the visual representation of the icon.

In that case a potential future update might be to always store and return all image and text scales only as arrays regardless of how they is set, avoiding the need for a separate getScaleArray for rendering.

but does it matter?

It would if someone is using getScale() in an arithmetic expression

Well, if they are using type checking they would get a type checking error because the returned value can also be an array.

That is a big if.  Someone might be doing image.setScale(image.getScale() * 2) to highlight something.

It's standard when using an IDE like VS Code.

Someone might be doing image.setScale(image.getScale() * 2)

OpenLayers code does use .getScaleArray()[0] when doing something like that https://github.com/openlayers/openlayers/blob/main/src/ol/format/KML.js#L1358
The same should apply with text scale.  https://github.com/openlayers/openlayers/blob/main/src/ol/format/KML.js#L2701 is clearly expecting a numeric value, but in theory it could be an array (which would be invalid in the output) and using .getScaleArray()[0] would be safer (and would be needed if an array was always returned).

I really wish #14346 had never been merged the way it was merged. It really made the API of ol/style/Icon worse than it was already, and I feel sorry now that I had motivated @KaiVolland to work on that.
To prevent unexpected behavior, I added a commit that prevents setScale() from being called when width or height were set.
The underlying problem is that ol/style/Icon serves both the uses case where the icon's image is used for the icon as a whole, and where the image is a sprite sheet that multiple icons can be referenced from using size and offset. Now when width or height are specified, will the scale be calculated from their ratio to the image, or the icon? In fact, it is the image, although the scale is applied to the icon. I tried to add this fact to the description of the width and height properties.
@marcjansen @KaiVolland Maybe you guys want to review this. I hope this pull request does not break your applications.

I had to add yet another commit, to avoid unexpected behavior related to size (and offset).

A possible better alternative to this pull request would be to change the behavior to calculate the scale from width, height and size. This means that size would have to be set when width and height are provided. @KaiVolland Do you think you could live with that?

If the size was known in advance the change would not have been needed as it would have been possible to set the necessary scale.  The problem was similar to setting the correct scale for KML icons https://github.com/openlayers/openlayers/blob/main/src/ol/format/KML.js#L311-L313 where a KML scale of 1 is based on showing a minimum width or height of 32, but you do not know the dimensions to scale to that size until the image is loaded.

Yes, we all agree that the API is awkward, and we'll need to rework this whole thing, making breaking changes. Closing this in favor of #14619, so the new width and height settings calculate the scale in relation to the icon size, instead of the image size.


Initial issues when cloning the repo

I cloned the repo and ran into the following problems getting things running, all house cleaning stuff:

When using VSCode with format on save enabled, because prettier is configured via .eslintrc with trailingComma set to 'all', this results in the function call file=path.resolve in setup.js to be formatted with a trailing comma which is not supported before ES7. FIxed this by changing the confuration from 'all' to 'es5'

'prettier/prettier': [
      'error',
      {
        singleQuote: true,
        trailingComma: 'es5',
      },

node-fetch is a required dependency from setup.js but is not listed in package.json in the dependencies. Fixed by adding node-fetch as a depenency.

Tests are not found because they are located in the test directory which is not in the src directory. react-scripts expects all tests to be located in the src directory. Fixed by moving the test directory into the src directory

chai is a required dependency from spec.js but is not listed as a dev dependency. Fixed by adding chai as a dev dependency.

Once I made these tweaks everything seems to be working fine. Thanks for putting this package together, love that I can use relay modern without having to eject from create-react-app.

@mlesk thanks, much appreciated! If you like, you can fetch and merge this fix into your project:
git remote add react-static-boilerplate https://github.com/kriasoft/react-static-boilerplate.git
git checkout master
git merge react-static-boilerplate/master

Still having the following problems with this fix merged:

trailing comma in setup.js line 37,  removing this comma and setting trailing comma prettier option to es5 prevents this from happening.

yarn relay v0.27.5
$ node setup --download-schema && relay-compiler --src ./src --schema ./src/schema.graphql
/Volumes/Data/Dev/projects/current/x/setup.js:38
);
^
SyntaxError: Unexpected token )
at createScript (vm.js:53:10)
at Object.runInThisContext (vm.js:95:10)
at Module._compile (module.js:543:28)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
at Module.runMain (module.js:605:10)
at run (bootstrap_node.js:423:7)
at startup (bootstrap_node.js:147:9)
error Command failed with exit code 1.

@mlesk thanks, fixed in a5b0b8a  Yet, you may want to use Node v8+ to avoid this issue.

Yes, patch solved it. V8 does as well. Thanks


4.0.0: Cleanup

This epic captures the engineering work to go and remove all the vestiges of the old calibration flows and protocol API v1 from the system.

Looks closed to me!


ItemTree: Lower tuple types despite invalid type

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/5380

bors r+


ODSP: File creation error logging can be improved

Based on this investigation
fluid:telemetry:OdspDriver:createNewFile_cancel contains sharepoint correlation ID.
But when this error is propagated to fluid:telemetry:Container:ContainerClose, correlation ID is not there.
It would be great for errors to keep all of their properties, including correlation IDs.
I have not looked at details, i.e. what's the issue here.
Also cancel event should be an error event - we already do so in other places (I believe in summarization and loading from snapshot). Not sure if it's one-by-one fixing (like in this place), or logger should just make all cancel event (by default) errors.

I changed the cancel event to be an error. Also, for attach network call in driver, I added retries in container.ts if there is some error.
Also I don't think we close the container on attach failure?


Search highlight disappears while scrolling

After the latest update, search highlight disappears while scrolling, see the video. It's pretty annoying since I search for permissions almost all the time.
I use Google Chrome version 85.0.4183.102.

Yeah this was a side effect of the improvement I made to groups with really large sets of nodes.
Don't worry I will add a proper search feature in the future
In the meantime you could use this tool I made to search for permissions in your setup: https://lp-search.turbotailz.com

Yeah this was a side effect of the improvement I made to groups with really large sets of nodes.
Don't worry I will add a proper search feature in the future

Nice to hear, I'm looking forward to it.

In the meantime you could use this tool I made to search for permissions in your setup: https://lp-search.turbotailz.com

Oh, thank you very much! I'll definitely use this until the in-built search feature comes out.


Solution for 3.animation in css exercise

Because

This PR

Issue

Closes #XXXXX
Additional Information

Pull Request Requirements

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this PR follows the location of change: brief description of change format, e.g. 01-flex-center: Update self check
[x] The Because section summarizes the reason for this PR
[x] The This PR section has a bullet point list describing the changes in this PR
[x] If this PR addresses an open issue, it is linked in the Issue section
[x] If applicable, I have ensured that the TOP solution files match the Desired Outcome image

@kongnoun Good job finishing those exercises. As per step 6 of the exercises repo README, please do not open PRs for your own solutions in this repo.
You are welcome to commit to your own fork, but do not open a PR.

@kongnoun Good job finishing those exercises. As per step 6 of the exercises repo README, please do not open PRs for your own solutions in this repo.
You are welcome to commit to your own fork, but do not open a PR.
oh i forget thannk you so much ^~^, I am sorry for this mistake.


Call to undefined function Spatie\LaravelPackageTools\now()

With PHP8 and lumen 8:
In PackageServiceProvider.php line 56:
Call to undefined function Spatie\LaravelPackageTools\now()

We don't support Lumen for this package.

This is not a lumen error but php problem, there is no function now() in php 8

Laravel applications have now() by default.


Where are the definitions of enableInterrupt and disableInterrupt?

ENV:
Arduino: 2.3.2
Arduino-ESP32: 3.0.0-alpha
OS: macOS 11.7
Board: ESP32-C6-N4
These are a series of excellent examples.
But, When I compiled the Zigbee_Light_Switch, an error occurred:
In file included from /Users/HonestQiao/Documents/Arduino/ESP32C6/Esp32-C6-Bug-Arduino-Examples/examples/esp32c6bugzigbeemqttbridge/Zigbee_Light_Switch/Zigbee_Light_Switch.ino:31:
/Users/HonestQiao/Documents/Arduino/ESP32C6/Esp32-C6-Bug-Arduino-Examples/examples/esp32c6bugzigbeemqttbridge/Zigbee_Light_Switch/Zigbee.h: In function 'void switch_gpios_intr_enabled(bool)':
/Users/HonestQiao/Documents/Arduino/ESP32C6/Esp32-C6-Bug-Arduino-Examples/examples/esp32c6bugzigbeemqttbridge/Zigbee_Light_Switch/Zigbee.h:214:13: error: 'enableInterrupt' was not declared in this scope; did you mean 'detachInterrupt'?
  214 |             enableInterrupt((button_func_pair[i]).pin);
      |             ^~~~~~~~~~~~~~~
      |             detachInterrupt
/Users/HonestQiao/Documents/Arduino/ESP32C6/Esp32-C6-Bug-Arduino-Examples/examples/esp32c6bugzigbeemqttbridge/Zigbee_Light_Switch/Zigbee.h:216:13: error: 'disableInterrupt' was not declared in this scope; did you mean 'detachInterrupt'?
  216 |             disableInterrupt((button_func_pair[i]).pin);
      |             ^~~~~~~~~~~~~~~~
      |             detachInterrupt

exit status 1

Compilation error: 'enableInterrupt' was not declared in this scope; did you mean 'detachInterrupt'?

I searched the Arduino-ESP32 library and found no trace of these two functions.
@allexoK

Thanks @HonestQiao, the examples are based on official Arduino Zigbee examples so it would be unfair for me to take all the credit:)
I checked out the problem, seems like I actually pushed the wrong example(switch coordinator instead of light-bulb end device), now it should be fine, here is Zigbee_Light_Bulb.

Thanks @HonestQiao, the examples are based on official Arduino Zigbee examples so it would be unfair for me to take all the credit:) I checked out the problem, seems like I actually pushed the wrong example(switch coordinator instead of light-bulb end device), now it should be fine, here is Zigbee_Light_Bulb.

Thank you very much for those work.
I have also made a few minor optimizations and submitted a pull request : #2

My ESP32-C6 boards:
Light Blub: (Small Board)
DFRobot Beetle ESP32 C6
Bridge: (Big Board)
DFRobot FireBeetle 2 ESP32 C6

@HonestQiao Thanks for adding WiFi support!


Add non-const instrumented constructor to BoxPainter.

Updated the doc

This PR should be postponed till transformation or macro implementation.


Inference failed with error Invalid \escape: line 3 column 373 (char 424)

Describe the issue as clearly as possible:
When using Outlines to generate JSON schemas, I get this error approx. %5 of the time: Invalid \\escape: line 3 column 373 (char 424) even with very simple schemas: {prompt: str} and for the same prompt.
Steps/code to reproduce the bug:
import outlines

model = outlines.models.transformers("TheBloke/zephyr-7B-beta-GPTQ", device="cuda")
json_schema = json.dumps({
  "type": "object",
  "properties": {
    "poem": {
      "type": "string"
    }
  }
})
generator = outlines.generate.json(model, json_schema)

sequence = generator("Write a long poem")
print(sequence)

Expected result:
{"poem": "blablabla"}

Error message:
Invalid \\escape: line 3 column 373 (char 424)

Outlines/Python version information:
outlines==0.0.13

Context for the issue:
The error comes from the JSON library. Our guess is that the string generated through outlines is not accepted by the json library.

Any chance you could use a debugger and output the string that’s generated before parsing?

Better yet, could you set the rngs and get one to reproduce the issue?

This seems to be an issue with the model exceeding the context length before it has the time to output }. Can you try by constraining the length of the string field using maxLength?


error:ares_setup.h(83) and ares_build.h

win10 x64,vs2015update3
Severity	Code	Description	Project	File	Line	Suppression State
Error	C1083	Cannot open include file: 'ares_build.h': No such file or directory	ares	d:\develop\c++\grpc\third_party\cares\cares\ares_setup.h	83

I also met the same issue when build grpc 1.4.x with vs2015

is this still seen?

https://groups.google.com/a/google.com/forum/#!forum/grpc.io might be a better place for this


Really high CPU usage

For context, my system: Ryzen 7 3700x, 32GB 3200mhz and a RTX 4090. Yes I am aware of a CPU bottleneck in some games
The problem lies in the usage of my CPU while running RVC, it's usage balances around 40% sometimes reaching 50%. This is an issue because if my CPU usage goes in the 90-100% usage range RVC becomes very unstable/laggy.
My settings:

Wrong software

Isn't this w-okada/voice-changer? Instead of posting here you should open a issue in there.

That is entirely

I entirely overlooked that, my bad!


[FEATURE] changing the style of Start learning button

Description
Please check the video. Can I add this style to this button. Then I will make a PR.
Screenshots
https://user-images.githubusercontent.com/70086051/171104016-7d320def-f90b-4b6d-a227-548f3f70a594.mp4
Additional information
No response

@anubha-v-ardhan I have created an issue for this.

@anubha-v-ardhan Can you merge this PR?


Reduce default connection time to 10s (from 32s)

This helps to address https://github.com/docker/cli/issues/1739, where an
invalid DOCKER_HOST setting could result in a 64s delay (that's twice the
delay here because the client was trying to hit the /_ping endpoint twice,
which was addressed in https://github.com/moby/moby/pull/39206)
I made a previous attempt to fix this purely on the Docker cli side
(https://github.com/docker/cli/pull/1872) however that had the side effect of
adding the timeout across the board and not just for the dial phase, which
caused a regression for docker logs -f (https://github.com/docker/cli/issues/1892)
and so was reverted (https://github.com/docker/cli/pull/1893).
The new value of 10s is just based on a gut feeling, no initial connection
should be taking that long in the real world unless something about the network
link is pretty broken (e.g. bad dns perhaps), in which case affected users are
surely pretty used to retrying things, better to fail faster in the normal case.
Also drop the comment since the linked issue just shows that the original
number, just like the new number, was arrived at fairly arbitrarily based on
gut feelings (rather than anything empirical) so the reference is not really
terribly useful.
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

I debated whether to fix this here or to arrange to somehow overwrite/override the value in e.g. github.com/docker/docker/client. I decided that doing it at the root (i.e. here) made most sense.

Hrm, monkey patching this change into the CLI doesn't seem to have worked, something else may be going on. I'll investigate further.

something else may be going on.

mostly user stupidity (I was testing docker from $PATH not ./build/docker, oops!), plus the double from https://github.com/moby/moby/pull/39206 which still affects the tree I was testing.
But so far as this change goes I believe it is effective and can go in.


Bumped salt version to 2019.2.2

Please update saltstack to the latest version. There have been many issues fixed since 2019.2.1

Thanks!


Add 'AudioManager' function + Main Menu

I have created a simple main menu screen for the game, along with an audio manager system to handle sounds in a much better and efficient way.
This pull request contains:

AudioManager.gd (Set as Autoload in Project settings)
Main menu + background music
A Start Game button that makes sounds (it works but doesn't do anything else since there is no level(s) available yet)
A Sounds folder containing a Music folder & UI (User Interface) folder

I'm not sure if we need an AudioManager, it's not part of the minimum viable project plan. Update this code to include only the main menu. We might have use of an AudioManger in the future though.


Development: Update Apollon version

Checklist
General

[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context
Update version of Apollon from v2.11.2 to v2.11.3
Description
This bugfix version of Apollon fixes an issue with long text insertion in Apollon, the problem stated by @TheTrueVester at one of the Testing Session [Table: Findings > Assigned Issue > Issue Nr: 24]
Steps for testing
As stated in "Steps to reproduce" section of Testing session.
OR

Create a modeling exercise
Insert any element in canvas and try setting its name (or attributes/methods) with more than 100 characters.
Observe that the application stops taking input once 100 characters are inserted.

Changelog: https://github.com/ls1intum/Apollon/releases/tag/v2.11.3

Code change looks good, but I couldn't test it:

Ahh, it could be coz of the same branch name I used to update the previous versions of Apollon.
Could you try deploying it with a different release version name?
general-update-apollon-2.11.3-1 for instance?

I accidentally updated the version in the last general client updates, thanks for testing that everything works!


fix mentions not copy to clipboard

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/40477
$ https://github.com/Expensify/App/issues/40403
PROPOSAL: https://github.com/Expensify/App/issues/40477#issuecomment-2064649852, https://github.com/Expensify/expensify-common/pull/741#issuecomment-2209458519
Tests

Open any workspace chat
Send a message contain mention user and mention report and other text  @MENTION_USER text after #MENTION_REPORT text after
Right click on the sent message then Copy to clipboard.
Paste the copied text in comopser and Verify the message pasted successfully with mentions

[x] Verify that no errors appear in the JS console

Offline tests

Same as Tests step
QA Steps

Same as Tests step

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
I can't record the android video because android have an issue that focus out the composer and prevent me from type the message. I will check later if fixed

Android: mWeb Chrome
https://github.com/Expensify/App/assets/41129870/0ec294f1-ecb1-4f32-b57f-2fc95ba74cc7

iOS: Native
https://github.com/Expensify/App/assets/41129870/58a5b6e6-6656-4824-ab3d-4f65ba475ca7

iOS: mWeb Safari
https://github.com/Expensify/App/assets/41129870/4a176cb8-adb7-45c9-8ea4-cd716dc0d51c

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/41129870/c9a6a11b-4cda-47b4-aebb-67c72f7e7183
https://github.com/Expensify/App/assets/41129870/2ab64e6b-4221-40ee-85b3-ed0ec799a35b

MacOS: Desktop
https://github.com/Expensify/App/assets/41129870/59c28529-962e-409f-b848-2e70c9ced243

@eh2077 I can't record the android video because android have an issue that focus out the composer and prevent me from type the message. I will check later if fixed

Android: Native
https://github.com/Expensify/App/assets/41129870/72aa2bf5-4413-4c91-9a1b-e767930e4401

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/lakchote in version: 9.0.7-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/thienlnam in version: 9.0.7-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.7-8 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 9.0.8-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


About accelerate inference

hi , thanks for your repo. when I try to accelerate inference, I use llm.half，and change the input to int32 or float16, and the type of lm_imput  is also float16, but the inference speed of LLM has not improved.  Do you have any suggestions?

are you using gpu or cpu? maybe cpu do not support fp16 speedup

are you using gpu or cpu? maybe cpu do not support fp16 speedup

All inference stages are based on GPU,  and I directly used this code to run it on A10


Provide a way to omit component creator declarations when possible to deduce one.

Dagger2 can generate a Builder declaration that accepts all necessary modules and dependencies and produces a component instance, when an explicit declaration is omitted. While this is a convenient and type-safe way to reduce boilerplate code, Yatagan can't do just that - the reflection backend implementation obviously can't generate an API by itself, it can only implement an existing interface.
Thus let's try to think of some way to allow users to omit builder declarations for components (and maybe non-root components too). It's probably impossible to do this in a type-safe manner though.

Using runtime
A pretty obvious way to go about it would be to allow something along the lines of:
interface AutoBuilder<T> {
    /** would be used to provide every necessary argument with runtime checks */
    fun provideInput(input: Any): AutoBuilder<T>
    fun create(): T
}

/** factory-like */
fun Yatagan.createAuto(componentClass: Class<T>, vararg inputs: Any): T {
    // query argument types and pass them to either generated builder or reflection engine
}

/** builder-like */
fun Yatagan.builderAuto(componentClass: Class<T>): AutoBuilder<T> {
    // provide an implementation of a AutoBuilder<T> for the correspondent backend.
}

This is, obvously, not type-safe and would move a class of errors from compile time to runtime. There's a problem of type erasure though, which might seem to make this solution inapplicable - we won't be able to differentiate between, say, List<ClassA> and List<ClassB> inputs at runtime. But actually, we're probably good here, as no generics are possible in the case where auto-create or auto-builder is applicable: only component dependencies and "instancefull" modules can be parameters. Otherwise an explicit @Component.Builder declaration is required. Component dependencies and modules are declared in annotation arguments, that don't support generics, only raw classes, which are perfectly discernible at runtime. So this approach is a possible way to go, need to think about it some more.
Using codegen
Another way to tackle the problem would be to provide a code-generator (KSP-only?) to always generate a type-safe builder. This might penalize RT build times, which is something we don't want TBH.


Lido Community Staking integration

Description
The integration includes a new kind of deployment - Lido-compatible node:

SSV and Obol nodes for Simple DVT Staking Module integration
Prepare code for Community Staking Module integration when it is developed

General

FEE_RECIPIENT is Lido Execution Layer Rewards Vault contract
MEV_RELAYS consists of values from Lido MEV Boost Relay Allowed List contract
(In case of new keys generation) WITHDRAWAL_CREDENTIALS is Lido Withdrawal Vault contract

SSV node for Simple DVT
Integration uses the current implementation that already exists in eth-docker
Obol node for Simple DVT
There was no integration between Obol and eth-docker before, so this is a new option.
The integration contains:

ENR generation
Checking cluster definition
DKG ceremony running

Couple preliminary comments while you continue to work on this PR

Compose V1 is no longer supported. Remove version and make sure HOST_IP/SHARE_IP syntax is adjusted
ipv6.yml is gone, this is now handled by a statement in each yml. Please adjust accordingly.
I vastly prefer named volumes over using bind mounts. If .charon can be replaced with a named volume let's; if not please explain why
I have not dug into the Dockerfile yet for this stuff, just a quick note that I prefer using non-root users inside containers and that I like giving the user the option to source compile as desired, with Dockerfile.binary and Dockerfile.source

@yorickdowne thank you for your comments!

Compose V1 is no longer supported. Remove version and make sure HOST_IP/SHARE_IP syntax is adjusted
ipv6.yml is gone, this is now handled by a statement in each yml. Please adjust accordingly.

Acked 👀

I vastly prefer named volumes over using bind mounts. If .charon can be replaced with a named volume let's; if not please explain why

This directory only stores cluster info such as validators_keys and cluster-lock\definition, similar to how deposit-cli does it. I'm not sure if we should put it in a named volume, I'd like to keep the expected behavior here

I have not dug into the Dockerfile yet for this stuff, just a quick note that I prefer using non-root users inside containers and that I like giving the user the option to source compile as desired, with Dockerfile.binary and Dockerfile.source

I would be very grateful if you could write which dockerfiles you mean exactly 🙏  I'd look into what we can do here

There's an ssv-dkg.yml now in main. I used some of your code, and made different choices in other places. Please adjust this PR to it.
You can use query_dkg to ask the user whether to run DKG, and the output directory is .eth/dkg_output - handled by ethd and ssv-dkg.yml

Note that sed -i'' needs to be changed to sed -i'.original' to work on macOS, with attendant changes in .gitignore and a rm to remove the .original file
How's this coming along?

Note that sed -i'' needs to be changed to sed -i'.original' to work on macOS, with attendant changes in .gitignore and a rm to remove the .original file
How's this coming along?

Hi! Still in development, especially Obol integration part

@yorickdowne Hello there!
It looks like now the fork is just trying to keep up with upstream, no new features from our side (for a while, I guess).
This PR is ready for review 🚀
cc: @dgusakov

Thanks for preparing it @vgorkavenko!
From my side, I can add that there is a potential to create vanilla Obol integration based on the implemented Lido<>Obol integration. But this is definitely a separate topic.
@yorickdowne your comment and review are very welcome! CSM is expected to go live on the mainnet in Nov 2024. It is a good time to merge Lido fork to upstream now.

I think I can merge this, and then iterate on it. I'm not sure .charon and validator_ejector should be where they are, arguably that fits under .eth, at least the ejection stuff. I am also not sure what .eth_backup does.
Then there's the attestation VC parameters for Obol, which could be detected by presence of Obol, instead of having an .env variable for it.
And, those are all minor quibbles and can be tackled later, if and as desired.
Great work, and thank you!


Limit mount of docker config

Issue #, if available:
Description of changes:
The way we currently mount the docker config file takes over the entire ~/.docker/ directory and makes it readonly since that is how configmaps are mounted.  This makes it impossible to change other docker settings during builds, like when using buildx and it tries to store its config there.  Using this subpath option should fix this.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/approve


[Bug Report] Select BUG: Multiple Select can not hide dropdown when click arrow icon when set filterable attribute

Element UI version
2.4.8
OS/Browsers version
MacOs/Chrome newest
Vue version
2.5.17
Reproduction Link
https://jsfiddle.net/mmx38qxw/6694/
Steps to reproduce

use <el-select>
set multiple and filterable attribute both
click arrow icon to show dropdown
click arrow icon to hide dropdown

What is Expected?
dropdown should be hidden
What is actually happening?
dropdown not be hidden

It's not a bug. ElSelect is waiting for you to type something when it's is focused.

It's not a bug. ElSelect is waiting for you to type something when it's is focused.

So it is such a design that user click arrow icon can not hide dropdown menu when ElSelect is focused? I do think it is not a good design, coz not expect usual habits

可以尝试下，开启多选，再开启搜索，此时要想关闭只能点击页面其他地方才能关闭，这个交互很诡异


Would git be possible on LibTerm

I am wondering if there are any plans to add git to LibTerm. Would it be added in an app update? Or would someone have to make a git command in python?

It would be very difficult to compile. There is a library to do Git with Python: https://github.com/dulwich/dulwich. But the command itself should be rewrote.

Ok. Thanks I was just curious


v++ not able to synthesize the memory interface of top-level function

Hi,
I was trying to run the PLRAM example in the repo, and ran into some issues when compiling it with v++. When I compile the source directly, as https://github.com/Xilinx/Vitis_Accel_Examples/blob/master/cpp_kernels/plram_access/src/mmult.cpp#L45-L51, the tool first complains that the size of the input array is unknown, so I changed the top-level function signature as followed:
void mmult(const int a[64], // Read-Only Matrix A
           const int b[64], // Read-Only Matrix B
           int c[64],       // Output Result
           int a_row,    // Matrix A Row Size
           int a_col,    // Matrix A Col Size
           int b_col     // Matrix B Col Size
           ) {

After I changed the function, the v++ complains another issue:
===>The following messages were generated while  performing high-level synthesis for kernel: mmult Log file: /work/shared/users/phd/sx233/Vitis_Accel_Examples/cpp_kernels/plram_access/_x.hw.xilinx_u280_xdma_201920_1/mmult/mmult/vivado_hls.log :
INFO: [v++ 204-61] Option 'relax_ii_for_timing' is enabled, will increase II to preserve clock frequency constraints.
INFO: [v++ 204-61] Pipelining loop 'readA'.
INFO: [v++ 204-61] Pipelining result : Target II = 1, Final II = 1, Depth = 2.
INFO: [v++ 204-61] Pipelining loop 'readB'.
INFO: [v++ 204-61] Pipelining result : Target II = 1, Final II = 1, Depth = 2.
INFO: [v++ 204-61] Pipelining loop 'outer_loop1_outer_loop2_inner_loop'.
INFO: [v++ 204-61] Pipelining result : Target II = 1, Final II = 1, Depth = 8.
INFO: [v++ 204-61] Pipelining loop 'writeC'.
INFO: [v++ 204-61] Pipelining result : Target II = 1, Final II = 1, Depth = 2.
INFO: [v++ 200-790] **** Loop Constraint Status: All loop constraints were satisfied.
INFO: [v++ 200-789] **** Estimated Fmax: 309.02 MHz
ERROR: [v++ 213-400] This design has interfaces which are not supported by Vitis. Check interface pragmas to ensure the design only uses one s_axilite interface including the 'return' port and one or more m_axi interfaces with offset=slave set to the s_axilite interface; there should not be any other port interface used.
ERROR: [v++ 200-445] Unexpected error generating RTL model: generate_json error: Found incompatible interfaces
ERROR: [v++ 60-300] Failed to build kernel(ip) mmult, see log for details: /work/shared/users/Vitis_Accel_Examples/cpp_kernels/plram_access/_x.hw.xilinx_u280_xdma_201920_1/mmult/mmult/vivado_hls.log
ERROR: [v++ 60-599] Kernel compilation failed to complete
ERROR: [v++ 60-592] Failed to finish compilation
make: *** [_x.hw.xilinx_u280_xdma_201920_1/mmult.xo] Error 1

Hi,
Can you please specify which branch are you running it for. Also which VITIS and XRT version did you source. Was the example running for sw_emu? I tried the example for sw_emu and observed that the same is working for 2020.1 for xilinx_u280_xdma_201920_1.

The XRT version:
{
  "BUILD_VERSION" : "2.3.1301",
  "BUILD_VERSION_DATE" : "Fri, 25 Oct 2019 03:04:42 +0000",
  "BUILD_BRANCH" : "2019.2",
  "VERSION_HASH" : "192e706aea53163a04c574f9b3fe9ed76b6ca471",
  "VERSION_HASH_DATE" : "Thu, 24 Oct 2019 19:27:30 -0700"
}

v++ version
****** v++ v2019.2 (64-bit)
  **** SW Build 2708876 on Wed Nov  6 21:39:14 MST 2019
    ** Copyright 1986-2019 Xilinx, Inc. All Rights Reserved.

I was compiling for the HW.

I tried the example for the master branch sourcing 2020.1 Vitis and XRT. The examples is working for me for all the flows i.e. sw_emu, hw_emu and hw. I ran the example with no changes.
I have also tried for the 2019.2 branch sourcing 2019.2 Vitis and XRT and both the emulation flows are working for me.
Kindly source the respective Vitis and XRT. Also can you please try the original example (I checked for the original example) first for sw_emu and hw_emu and notify if you are facing any issues.

Thanks, @virata-xilinx!
I tried the 2019.2 branch and it worked well. So it seems that Vitis 2019.2 requires users to specify the interface programs, otherwise, the synthesis would fail?
Also, I ran to another (performance) issue with the interface pragma -- when I synthesize an application without the interface pragma using VIvado HLS, the program can achieve II=1. The application is a simple 2D CONV operator. However, when I add the interface pragmas and compile it with v++, the II becomes much worse. And from the HLS log, the tool throws the warning message as follows:
INFO: [SCHED 204-61] Pipelining loop 'Loop 1.1'.
WARNING: [SCHED 204-69] Unable to schedule bus request on port 'gmem1' (/work/shared/users/phd/sx233/alveo-test/kmeans/kernel.cpp:35) due to limited memory ports. Please consider using a memory core with more ports or partitioning the array.
INFO: [SCHED 204-61] Unable to satisfy pipeline directive: Unable to pipeline the region.

HI @Hecmay ,
Thanks for reporting this issue.
If possible, could you please post your Vitis (or HLS) related queries to Xilinx Public Forum to get quick response from Xilinx Experts and support staff?
https://forums.xilinx.com/t5/Vitis-Acceleration-SDAccel-SDSoC/bd-p/tools_v
-Heera

Thanks @heeran-xilinx. I will try. The technical support on Xilinx forum is far from responsive, and oftentimes my question won't get any response for quite a long time (like around half a year)... that's why I opened issues here.

Hi @Hecmay ,
Thanks for your valuable feedback on Technical support. I will convey this to Xilinx internal.
If possible, please post this query there and share the Query link here.
-Heera


text following circular path

Hello!
I was wondering if it is possible to recreate effect like on following example? Of text following a curved path.
<svg xmlns="http://www.w3.org/2000/svg"
     width="300" height="100" viewBox="0 0 300 100">

  <path id="MyPath" stroke="lightblue" fill="none"
	d="M 50,50 C 100,0 200,100 250,50"/>

  <text style="font: 24px sans-serif;">
    <textPath href="#MyPath">Text on a path.</textPath>
  </text>

</svg>

reference: https://www.w3.org/TR/SVG/text.html
more examples:
http://thenewcode.com/482/Placing-Text-on-a-Circle-with-SVG
https://css-tricks.com/snippets/svg/curved-text-along-path/
https://codepen.io/rdfriedl/pen/rOegaP
Thanks a lot for hints!

The textPath element has not been implemented yet.  You can subclass DrawingBasicElement or DrawingParentElement yourself to make any SVG tag (and maybe make a pull request).
This should get you started:
import drawSvg as draw
import xml.sax.saxutils as xml

class _TextPathNode(draw.DrawingBasicElement):
    TAG_NAME = 'textPath'
    hasContent = True
    def __init__(self, text, path, **kwargs):
        super().__init__(xlink__href=path, **kwargs)
        self.escapedText = xml.escape(text)
    def writeContent(self, idGen, isDuplicate, outputFile, dryRun):
        if dryRun: return
        outputFile.write(self.escapedText)

class TextOnPath(draw.DrawingParentElement):
    TAG_NAME = 'text'
    def __init__(self, text, fontSize, path, **kwargs):
        super().__init__(font_size=fontSize, **kwargs)
        self.append(_TextPathNode(text, path))

d = draw.Drawing(300, 100, origin=(0, 0), displayInline=False)

path = draw.Path(stroke='lightblue', fill='none')
path.M(50, 100-50).C(100, 100-0, 200, 100-100, 250, 100-50)
d.append(path)

text = TextOnPath('Text on a path.', 24, path)
d.append(text)

d

Thanks for a starter! I'll be happy to contribute!

The textPath element has not been implemented yet. You can subclass DrawingBasicElement or DrawingParentElement yourself to make any SVG tag (and maybe make a pull request).

Does this apply to the 'basic drawing elements' example? I can't get path text or multi-line text working (lines 32-33). The error states required positional arguments 'x' and 'y' are missing.

@lthumbe You probably have an old version of drawSvg.  The pull request linked right above this added the feature for the version 1.8.1 recently published (and I forgot to close this issue).  From your terminal run pip show drawSvg to check the version.  To update run pip install --upgrade drawSvg.
Let me know if this works then I'll close this issue.

@cduck Upgrade fixed it. Thanks!

Closing because text following a path was implemented by #22.


[VL] Support regr_intercept aggregate function

What changes were proposed in this pull request?
Support regr_intercept aggregate function.
How was this patch tested?
Add test case VeloxAggregateFunctionsSuite.regr_intercept

CH CI fails for new PRs, just created an issue: https://github.com/apache/incubator-gluten/issues/5274

Run Gluten Clickhouse CI


Add carry (move and focus) option

Add option to move current container to a new workspace and change focus to that workspace.

Thanks for the PR! Nice feature, I like it!


Kubernetes in non <IP_ADDRESS> subnet - services with wrong cluster IP

Is this a BUG REPORT or FEATURE REQUEST? (choose one): Feature request
Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm): Kubernetes latest
What happened: created a cluster using an existing subnet with custom subnet CIDR (not the <IP_ADDRESS> that ACS assumes). everything basically works, but services are still created under <IP_ADDRESS> range.
What you expected to happen: services be created under the cluster's range, or allow me to specify a custom 'kubeServiceCidr'.
How to reproduce it (as minimally and precisely as possible):

Create a VNET and subnet <IP_ADDRESS>/16.
Create Kube cluster in that VNET including 'kubernetesConfig.clusterSubnet' and 'firstConsecutiveStaticIP'. cluster nodes are created correctly under the correct IPs.
Once done, create a new deployment, pods get a correct IP in the range.
expose the deployment using a kubernetes service with type 'cluster-ip'. service ip is created under <IP_ADDRESS>/16 and therefore are non-routable, and might conflict with existing subnets.

Anything else we need to know:
This was apparently fixed in GH-473 which added serviceCidr to cluster definition but was then reverted back in GH-508 which was partially fixed again in GH-632 (serviceCidr is still hardcoded).
GH-632 says that "This PR is complementary to #473" and that "this PR can be applied in any order to re-enable that functionality."
So should now that GH-632 was applied - should we merge GH-473 again on top of it?

BTW, as a workaround, I guess I can modify this: https://github.com/Azure/acs-engine/blob/master/parts/kubernetesmastervars.t#L87 ?
anything else I need to do for this workaround to work?

Assume this one can be closed due to the merge of #546


[NRF51_DK] us_ticker.c doesn't support IAR build

The RTC1_IRQHandler in us_ticker.c is implemented by inline assembler and there is no IAR toolchain support.  Detected by RC1 OOB test.
C:\Users\toywat01\Documents\GitHub\mbed-os-example-blinky>mbed compile -m NRF51_DK -t IAR
Building project mbed-os-example-blinky (NRF51_DK, IAR)
Scan: .
Scan: FEATURE_BLE
Scan: FEATURE_UVISOR
Scan: FEATURE_IPV4
Scan: mbed
Scan: env
Compile: rt_MemBox.c
Compile: rt_Memory.c
Compile: us_ticker.c
[ERROR]
  #error Compiler not supported.
   ^
"C:\Users\toywat01\Documents\GitHub\mbed-os-example-blinky\mbed-os\hal\targets\hal\TARGET_NORDIC\TARGET_MCU_NRF51822\us_ticker.c",382  Fatal error[Pe035]: #error directive: Compiler not supported.
Fatal error detected, aborting.

https://github.com/mbedmicro/mbed/blob/master/hal/targets/hal/TARGET_NORDIC/TARGET_MCU_NRF51822/us_ticker.c#L382

cc @pan-

@toyowata This is expected for the RC1 because IAR support was not enabled.
It will be for RC2.

@pan- Thanks. I will close this issue, once I confirm the fix in RC2.

@pan- Confirmed the fix.  Closed ticket.


Request deletion fails

We have an issue with Singularity 0.4.1, you can reproduce by doing:

create a request,
create a deploy inside this request,
remove request,
re-create request: it fails saying the request already exists.

Even when deleting /singularity zkNode, flushing Mysql database and restarting Singularity, data remains. Where is the data stored ? How can i "really" delete a request ?

I think there might be two things going on here:

If you try to re-create it while it is being cleaned up it will fail. There is a queue which processes removal of requests. Once it has been drained you should be able to re-create the request.

You should be able to trivially purge the data - most operational data is in ZooKeeper.

I'm going to make sure having stopped Singularity before doing an "rmr  /singularity" on zookeeper, to make sure there is nothing left.


Unable to parse object spread

When 6to5 is run using --whitelist, spread parsing fails.
let y = {c: 3, d: 4}
let x = {a: 1, b: 2, ...y}

$ 6to5 -l classes test.js
SyntaxError: xx.js: Unexpected token (2:21)
  1  | let y = {c: 3, d: 4}
> 2  | let x = {a: 1, b: 2, ...y}
     |                     ^

I was expecting the output to be the same as input.

Object spread is an experimental ES7 feature with the transformer objectSpread.

But why the following works:
let x = {}
let y = {}

$ 6to5 -l classes test.js

// as expected
let x = {}
let y = {}

objectSpread should be treated like other transformers: if not specified, parsing should be ok, but the transformation not applied.

Moreover:
let y = {c: 3, d: 4}
let x = {a: 1, b: 2, ...y}

Produces an error:
$ ./node_modules/.bin/6to5 -l objectSpread xx.js
SyntaxError: xx.js: Unexpected token (2:21)
  1  | let y = {c: 3, d: 4}
> 2  | let x = {a: 1, b: 2, ...y}
     |                     ^

Yeah, you need to enable experimental support.
$ 6to5 -l objectSpread --experimental xx.js

Thanks it works as expected.

I'm unable to parse object spread as well. Setting is given as:
        {
               loose: 'all',
               blacklist: ['es6.modules'],
               optional: ['es7.objectSpread'],
               experimental:true
        }

with gulp-6to5, but it's not working:
ERROR in ./source/scripts/main.js
Module build failed: SyntaxError: /srv/home/gsklee/projects/dashboard/source/scripts/main.js: Unexpected token (336:18)
  334    |     }).done(function(data) {
  335    |       console.log(data);
> 336    |       let { x, y, ...z } = { x: 1, y: 2, a: 3, b: 4 };
         |                   ^
  337    |
  338    | console.log(z);
  339    |     });

Is extra package/runtime required for this experimental feature?

@gsklee It's a bit confusing by the es7.objectSpread actually also requires the es6.destructuring transformer to be enabled. This issue comes up quite a bit so in the next patch I'm going to force es6.destructuring to be enabled when you have es7.objectSpread whitelisted.

@sebmck I did have destructuring enabled, I only disabled es6.modules.

Hmmmm, I've imported '6to5/polyfill' in my project and am doing this within my gulp task:
             .pipe($['6to5']({
                 loose: 'all',
                 blacklist: ['es6.modules'],
                 optional: ['es6.destructuring','es7.objectSpread'],
                 experimental: true
              }))

But still get the warning:
ERROR in ./source/scripts/main.js
Module build failed: SyntaxError: /srv/home/gsklee/projects/dashboard/source/scripts/main.js: Unexpected token (18:12)
  16   | const ENDPOINT_SERVER = '';
  17   |
> 18   | let { x, y, ...z } = { x: 1, y: 2, a: 3, b: 4 };
       |             ^
  19   | console.log(z);
  20   |
  21   | class Header extends React.Component {

What gives?

@gsklee Not sure if it'd make a difference but the option you're looking for is whitelist, not optional.
I can't seem to replicate this
script.js
let { x, y, ...z } = { x: 1, y: 2, a: 3, b: 4 };

$ 6to5 --blacklist es6.modules --whitelist es6.destructuring,es7.objectRestSpread --experimental script.js
var _objectWithoutProperties = function (obj, keys) { var target = {}; for (var i in obj) { if (keys.indexOf(i) >= 0) continue; if (!Object.prototype.hasOwnProperty.call(obj, i)) continue; target[i] = obj[i]; } return target; };

var _ref = { x: 1, y: 2, a: 3, b: 4 };

let x = _ref.x;
let y = _ref.y;
let z = _objectWithoutProperties(_ref, ["x", "y"]);

Does it work if you remove the blacklist/whitelist/optional options?

Found the source of the issue but it's a little bit cryptic.
This is my gulpfile.js:
             .pipe($['6to5']({
               loose: 'all',
               blacklist: ['es6.modules'],
               optional: ['es7.objectRestSpread'],
               experimental: true
              }))
             .pipe($.webpack(require('./webpack.config.js')))

And my webpack.config.js:
  module: {
    loaders: [{
      test: /\.js$/,
      exclude: /node_modules/,
      loader: '6to5'
    }]
  },

I fixed the problem by changing the webpack config to loader: '6to5?experimental'. I thought when the file stream enters the second pipe, all ES6 syntax (save import/export) would already be replaced? So this is kinda beyond me.

@gsklee The es7.objectRestSpread transformer isn't optional, it's automatically enabled when you use experimental so you can safely drop that option. Why are files being parsed by 6to5 twice anyway?

@sebmck I'm using webpack for and only for modularization, I personally find gulp's imperative manner easier to handle than config-based building tools.

Am I right and the only way to use spread object destructuring inside if statement is var?
if (true) {
  var {x, ...y} = {x: "X", a: "A", b: "B"};
} else {
  var x, y;
}

console.log(y);

OMG, it seems the only way to remove prop in React is spread destructuring (*) and the only way to do this is to run Babel in experimental mode with var statements inside if which are planned to removal at least in SaneScript. We are going on a very thin cracking ice.

– props can't be deep cloned because they contain links to some very heavy objects and with shallow cloning mutable props can be... mutated.

@ivan-kleshnin Not sure I understand the question, you've skipped too far ahead. What are you trying to do and why do you think you can't do it?

Ok, sorry. I want to remove one exact property from object by object spread destructuring. See my code example. But this should be inside one of the branch of if statement. And the result is going to be used after this statement. Is there other way to do this without var statement?

var x, y;

if (true) {
  ({x, ...y} = {x: "X", a: "A", b: "B"});
}

console.log(y);

Thanks! Note to readers: It works only with the latest Babel 5.3.*.
Syntax is... :unamused:  whatever.

No that has worked since object spread/rest was implemented in Babel.

Well, if you care, I just had this error with Babel 5.2. Update to 5.3 solved this.
Had no other problems with that 5.2 install.
/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/babel/helpers/parse.js:70
    throw err;
          ^
TypeError: /Users/ivankleshnin/JavaScript/test.js: Cannot read property 'start' of undefined
    at Parser.pp.toAssignable (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/lval.js:29:56)
    at Parser.pp.parseMaybeAssign (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:111:44)
    at Parser.pp.parseParenAndDistinguishExpression (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:419:28)
    at Parser.pp.parseExprAtom (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:338:19)
    at Parser.parseExprAtom (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/plugins/jsx.js:593:162)
    at Parser.pp.parseExprSubscripts (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:217:19)
    at Parser.pp.parseMaybeUnary (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:199:19)
    at Parser.pp.parseExprOps (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:155:19)
    at Parser.pp.parseMaybeConditional (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:138:19)
    at Parser.pp.parseMaybeAssign (/usr/local/lib/node_modules/babel/node_modules/babel-core/lib/acorn/src/expression.js:106:19)

You can do it with let as well:
let x, y;

if (true) {
  ({x, ...y} = {x: "X", a: "A", b: "B"});
}

console.log(y);

I think it's not a let vs. var issue; it's always a good practice to collect your variable declaration whenever possible.

@gsklee yes, I just never met ({x, ...y} = {x: "X", a: "A", b: "B"}); syntax before.


⚠ Bofrost.de has degraded performance

In 2e65a3c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 802 ms

Resolved: Bofrost.de performance has improved in e9d8bcf.


[AMQP] Re-Attach Link on Detach

This PR adds a feature to re-attach a link that has been detached.

/azp run python - servicebus - tests

/azp run python - eventhub - tests

API change check
API changes are not detected in this pull request.


smtp error,

I'm forked from @CryptorClub  add<EMAIL_ADDRESS>but I received error log
Error on connectMx for:  { exchange: 'alt4.gmail-smtp-in.l.google.com', priority: 40 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1083:14)
errno: 'ETIMEDOUT',
code: 'ETIMEDOUT',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 25 }

Open port 25. @OKPayCurrency

DO blocked port 25,🤯

Okay thanks, now work with new host no DO😁, THANKS GUYS


GDScript: Initialize all defaults beforehand in implicit constructor

Set all the default values for typed variables before actually trying to initialize them, including @onready ones.
This ensures that if validated calls are being used there will be a value of the correct type, even if the resolution is done out of order or deferred because of @onready.
Fix #73893
Fix #66144 Note: this still won't work as expected, since members are resolved in order, but it won't crash anymore.

A very quick perusal makes it make sense to me. Are there types other than array that it may make sense to test here?

Thanks!


So i hack WebGLFilterManager's pushFilter and popFilter method, in pushFilter method i create a new stencilManager instance and save the old one, in popFilter i destroy the new stencilManager and restore the old one, then this workaround works rightly as i expect.
I think this is a bug for PIXI, so i hope @photonstorm or @englercj can help me checking whether this fixing is right or not, thanks.
var game = new Phaser.Game(800, 600, Phaser.WEBGL, 'phaser-example', { preload: preload, create: create });

function preload() {
	game.load.image('phaser', 'assets/sprites/phaser2.png');
	game.load.script('filterX', '../filters/BlurX.js');
	game.load.script('filterY', '../filters/BlurY.js');
}

function create() {
	var logo = game.add.sprite(game.world.centerX, game.world.centerY, 'phaser');
	logo.anchor.setTo(0.5, 0.5);

	var blurX = game.add.filter('BlurX');
	var blurY = game.add.filter('BlurY');

	logo.filters = [blurX, blurY];
	var group = game.add.group();
		group.addChild(logo);
		group.mask = game.add.graphics();
		group.mask.beginFill(0xff);
		group.mask.drawRect(0,0,500,500);
}

After i hack WebGLFilterManager's code, i can get the right effect like below:

https://github.com/photonstorm/phaser/pull/1842/


Cannot read property 'track_file_stream_url' of undefined on Windows

So I finally got catz to work, after pasting a link and the app resolving it (showing the playlist name) there seems to be nothing appearing afterward. Just a blank screen with 8tracks tag and title of playlist.

Can you check the console for errors?

No errors at all,
`$ npm start

catz@ start C:\Program Files\nodejs\catz
concurrently --kill-others "webpack --config webpack.config.dev.js --watch" "electron ."

[1]
[0] Live Reload listening on port 35729
[0]
[0] Hash: 7d9ef3e7b3f1fd633108
[0] Version: webpack 1.13.2
[0] Time: 4897ms
[0]  Asset    Size  Chunks             Chunk Names
[0] app.js  121 kB       0  [emitted]  main
[0]    [0] multi main 40 bytes {0} [built]
[0]     + 32 hidden modules
`
Although I was missing an optional dependency which I posted on another ticket.

Closing the app gave me this error.
https://paste.ee/p/xjYoM

Check the console in the app. I think it's under view.

Here you go:
Uncaught (in promise) TypeError: Cannot read property 'track_file_stream_url' of undefined(…

This has come up before. But I haven't been able to reproduce it.

How can I help you? I can provide more logs if you want.

Try pulling the latest commit.

Pulling it now, big thanks! Will report back asap.

Just a shot in the dark unfortunately /:

No worries, will give it a try. Building it now.

Same error:
Uncaught (in promise) TypeError: Cannot read property 'track_file_stream_url' of undefined     at EightTracks.songFailure (eval at <anonymous> (C:\Program Files\nodejs\catz\catz\app\bundle\app.js:120:2), <anonymous>:209:60)     at EightTracks._callee$ (eval at <anonymous> (C:\Program Files\nodejs\catz\catz\app\bundle\app.js:120:2), <anonymous>:147:85)     at tryCatch (C:\Program Files\nodejs\catz\catz\node_modules\regenerator-runtime\runtime.js:62:40)     at GeneratorFunctionPrototype.invoke [as _invoke] (C:\Program Files\nodejs\catz\catz\node_modules\regenerator-runtime\runtime.js:336:22)     at GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (C:\Program Files\nodejs\catz\catz\node_modules\regenerator-runtime\runtime.js:95:21)     at step (eval at <anonymous> (C:\Program Files\nodejs\catz\catz\app\bundle\app.js:120:2), <anonymous>:33:191)     at eval (eval at <anonymous> (C:\Program Files\nodejs\catz\catz\app\bundle\app.js:120:2), <anonymous>:33:361)     at process._tickCallback (internal/process/next_tick.js:103:7)

I have node v6.9.1 on Windows 10 Home in a virtual machine and everything works fine ):

That's weird :( I just followed everything you said from the instruction. Do you think I should clone using node or using git bash?

I just cloned using git directly. You can download it over there -> https://git-scm.com/

Yes, that's what I mean about git bash, I'm going to update my Git and clone it directly (which is what I'm currently doing). Again, thank you!

Update:

Reinstalled Git, and updated to the latest version
Updated my Java
didn't use yarn

Still getting the same error.
TypeError: Cannot read property 'track_file_stream_url' of undefined
Although there are no more errors when exiting the app (not using yarn).

Hopefully this would help:

Soundcloud download is working
8Tracks playlist has hyphens (although I tried non-hyphened and it didn't work as well)

Downloading another playlist in Soundcloud:
C:\Users\kjaro\catz\node_modules\react\lib\ReactDOM.js:75 Download the React DevTools and use an HTTP server (instead of a file: URL) for a better development experience: https://fb.me/react-devtools
C:\Users\kjaro\catz\node_modules\browser-id3-writer\dist\browser-id3-writer.js:275 Uncaught (in promise) TypeError: Cannot use 'in' operator to search for 'byteLength' in null
    at Writer.setFrame (C:\Users\kjaro\catz\node_modules\browser-id3-writer\dist\browser-id3-writer.js:275:133)
    at Song.tagMp3 (eval at <anonymous> (C:\Users\kjaro\catz\app\bundle\app.js:126:2), <anonymous>:244:106)
    at eval (eval at <anonymous> (C:\Users\kjaro\catz\app\bundle\app.js:126:2), <anonymous>:227:31)
    at Song.tagStuff (eval at <anonymous> (C:\Users\kjaro\catz\app\bundle\app.js:126:2), <anonymous>:211:16)
    at Song._callee$ (eval at <anonymous> (C:\Users\kjaro\catz\app\bundle\app.js:126:2), <anonymous>:121:37)
    at tryCatch (C:\Users\kjaro\catz\node_modules\regenerator-runtime\runtime.js:63:40)
    at GeneratorFunctionPrototype.invoke [as _invoke] (C:\Users\kjaro\catz\node_modules\regenerator-runtime\runtime.js:337:22)
    at GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (C:\Users\kjaro\catz\node_modules\regenerator-runtime\runtime.js:96:21)
    at step (eval at <anonymous> (C:\Users\kjaro\catz\app\bundle\app.js:126:2), <anonymous>:71:191)
    at eval (eval at <anonymous> (C:\Users\kjaro\catz\app\bundle\app.js:126:2), <anonymous>:71:361)

Oh boy. This looks interesting. Link to the soundcloud playlist?

I actually forgot, but I was only able to download the latest playlist from
one curator then the old one's, it gives me an error.
On Mon, Nov 14, 2016, 15:59 Cade Scroggins<EMAIL_ADDRESS>wrote:

Oh boy. This looks interesting. Link to the soundcloud playlist?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cadejscroggins/catz/issues/65#issuecomment-260270696,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIiYtMLMkmePfrsrYvazXJPRltC_tPeFks5q-BTvgaJpZM4KvY28
.

https://soundcloud.com/complexmag/sets/best-of-playlist-september-2016
The November playlist is the only playlist that I could download from the curator.

Okay so only one song doesn't download from that playlist. And It's because the album art for it 404d.

Ohh, alright. Although I still can't download from 8tracks.

I feel you. For some reason this promise isn't returning anything. Normally it returns the whole playlist or part of the playlist if it's large. If it returns part of the playlist, this conditional succeeds and a timer is set based on how long the most recent song fetched was. However, in your case, since it's not returning anything, this assignment fails because there aren't any songs at all.
So, yeah. I have no idea why that promise wouldn't be resolving the songs (as I can't reproduce any of this).

I understand, I tried rebuilding it 3 times. I'll probably stick with SoundCloud for now. Thank you so much!

Can you try the latest push? I suggest re-cloning and following the new instructions. Thanks.

It seems to still be an issue. I'm pretty confident that, for some reason, certain electron chrome builds restrict the number of network requests that can be made asynchronously. This is an issue for the 8tracks downloader because, in order to get the initial bunch of songs (and not just the first four), the app sends as many requests as possible in a short amount of time. Chrome seems to be blocking those requests (resulting in this error).

Alright. I had a thought. Can someone test this?

@nikohd

Turns out, this happens because 8tracks streams music from youtube in certain countries.. A proper error should be displayed. Otherwise, wontfix.

I was able to reproduce this error on 8tracks URLs on latest commit, node etc when using an Australian IP.  When using a SOCKS proxy or VPN for a Canadian IP the app runs without error (and beautifully).  Happy to help out testing. Thanks!

@stevenvandervalk thanks for confirming this. enjoy!


UMAP reduction missing for Adipogenesis

Hi, I'm interested in plotting the umap projections for Adipogeneis subluster. I see "pca", "harmony" and "phate" in the subset  eWAT_Adipogenesis.Rds but not "umap"

Hi,
We didn't use the UMAP embedding for the primary analysis of adipogenesis. You can create a UMAP embedding using the harmonized PCs. For example:
Adipogenesis.object <- RunUMAP(Adipogenesis.object, reduction = "harmony", dims = 1:20)
You will need to find out how many dimensions to use. In the above code I suggested 20, which in our hands often works decently, but it can be worth it to explore how your analysis changes by using a different number of dimensions for embedding.
I'm closing this issue now, but please create a new one or re-open it if its not working for you.


Expose list_spaces

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Create an index.js file to import icons

Hi, would it be possible to generate an index.js file in the output dir ?
This way we can import icons like this :
// index.js

export { default as Icon1 } from "./Icon1.svelte";
export { default as Icon2 } from "./Icon2.svelte";

// svelte component

import { Icon1, Icon2 } from "icons/index.js"

And thanks for this project !

@mvuidev Hi there. Currently, this library does indeed generate an index.js that exports all Svelte components.
An example is svelte-codicons. At the bottom of the page, there is an index.js file.

@metonym Hey, you're right, there was a problem with one of my svg files and the script throwed an error, that's why the index file wasn't created.
Sorry ! It's all good :)


Compatibility with Meteor v3.0

Updated dependencies.
Migrated all mongo methods to their *Async counterparts.
Migrated from http to fetch.
Separate AccountsTemplates.oauthServices methods on client and server, with the latter being now async.
Made AccountsTemplates.oauthServices usable again, as it was relying on the long deprecated Accounts.loginServiceConfiguration, which we now replaced with ServiceConfiguration.configurations.

Merged and published to version 1.17.1


Improve User Table and Contacts Page

Create a contacts page for the public, and one for current employees. There is some concern about sharing contact info for all staff so I suggest limiting the public contacts page to managers and support staff.

Get feed back from JV.


Verifying did identity with V3 does not seem to support custom RPC provider

I am trying to verify documents using V3 and a custom RPC provider (having network id "cyan-imda").

verificationBuilder takes an array of verifiers and some options. Thanks to these options, we can pass a custom ethers provider.

For each verifier, it calls verifier.verify(document, verifierOptions), so we can assume that our custom provider is passed to each verifier.

But looking at didIdentityProof verifier, we see that it calls verifyV3 without passing our custom provider.

Then verifyV3 calls verifySignature().

verifySignature immediately calls getPublicKey().

getPublicKey immediately calls resolve().

resolve tries to use the default resolver, which is configured with the default network and default RPC provider.

==> the error is thrown: "No conf for networkId: cyan-imda".
==> the error is propagated and finaly catched by the error wrapper withCodedErrorHandler.
==> verification fails.

Hi, thanks for the detailed report.
I guess you would expect a resolver to be built that way:
const providerConfig = {
  networks: [
    { name: "whatever", provider: ethersProvider }
  ]
}

We never tried to use ethers provider with ethr-did-resolver and the example mention only about web3. So I'm not sure whether ethers is fully supported but we can try.
However, provider is mainly meant for document store and token registry verification. For DID I think we should introduce a new resolver option maybe.

Please correct me if I am wrong Benoit @jaffaskree , I think the main problem we are facing here is that the provider is hardcoded to mainnet
https://github.com/Open-Attestation/oa-verify/blob/ad06d88e963e95464c349af4f2c2dbb1bc07cb6d/src/did/resolver.ts#L8
This would mean that calls for resolving DID for any other network will fail. Can we instead pass the provider from options and only default to mainnet when there is no provider present in the options.

@hitesh-sm Yes.
@Nebulis I didn't mean that the provider should necessary be ethers-like. Using ethers provider is not really the problem.
What I had in mind is: it would be great to be able to pass our own config to the resolver in order to use custom network.
Currently, when calling verificationBuilder, the custom config seems to be dropped at the time you wrap the verify function inside withCodedErrorHandler.
In didIdentityProof.ts, line 110, verify receives 2 params: document and verifierOptions. Then, at line 113, it calls verifyV3 but pass only one arg: document.
At this point, it would be great if the verifierOptions were  passed to verifyV3.
Then, at line 95, verifyV3 calls verifySignature. At this point, it should also pass the verifierOptions. And so on, until the resolver is called with, this time, the custom options.
Another way to achieve this would be to allow us to directly pass a resolver to the verificationBuilder.
Something like:
interface EthrResolverConfig {
  networks: Array<{
    name: string;
    registry: string;
    rpcUrl: string;
  }>;
}

export function createResolvers(ethrResolverConfig: EthrResolverConfig): Resolver {
  return new Resolver({ ...ethrGetResolver(ethrResolverConfig), ...webGetResolver() });
}

export const verificationBuilder = <Document>(verifiers: Verifier<Document>[], resolver: Resolver) => (
  document: Document,
  promisesCallback?: PromiseCallback
): Promise<VerificationFragment[]> => {
  const promises: Promise<VerificationFragment>[] = verifiers.map((verifier) => {
    if (verifier.test(document, resolver)) {
      return verifier.verify(document, resolver);
    }
    return verifier.skip(document, resolver);
  });

  promisesCallback?.(promises);
  return Promise.all(promises);
};

// example:
const ethrResolverConfig: EthrResolverConfig = {
  networks: [
    {
      name: "cyan-imda",
      registry: "0xaE5a9b93Ac9....",
      rpcUrl: "https://cyan-imda.something....",
    },
  ],
};

const resolver = createResolvers(ethrResolverConfig);

const verifyBesu = verificationBuilder(
  [openAttestationHash, openAttestationDidIdentityProof, openAttestationDidSignedDocumentStatus],
  resolver
);

I'm fine with the resolver option.
Looks like you have a good idea how do to it. Feel free to open a PR. Otherwise you will have to wait they one of us do it for you.
Let me know if you need some help.

I opened a PR.
This is how I use it:

import { v3 } from '@govtechsg/open-attestation';
import {
  isValid,
  openAttestationDidIdentityProof,
  openAttestationDidSignedDocumentStatus,
  openAttestationHash,
  verificationBuilder,
  createResolver,
} from '@govtechsg/oa-verify';
import { ethers } from 'ethers';

const resolver = createResolver({
  networks: [{ name: 'my-network', rpcUrl: 'https://my-private-chain/besu', registry: '0xaE5a9b9...' }],
});

const verifyBesu = verificationBuilder<v3.SignedWrappedDocument<v3.OpenAttestationDocument>>(
  [openAttestationHash, openAttestationDidIdentityProof, openAttestationDidSignedDocumentStatus],
  {
    provider: new ethers.providers.JsonRpcProvider({ url: 'https://my-private-chain/besu' }),
    resolver,
  }
);

async function validate(document: v3.SignedWrappedDocument<v3.OpenAttestationDocument>): Promise<boolean> {
  const frags = await verifyBesu(document);
  const valid = isValid(frags);
  return valid;
}

Thanks for your contribution :smile_cat:

My pleasure ;)

:tada: This issue has been resolved in version 7.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


There is issuse in heder.php

login.php and  registration.php not connect with heder.php

i will sole this >>>


Documentation of exposed Haskell packages for code.world

Hello! Thanks for maintaining this project!
I just want to ask something about the documentation.
Do we have a complete list of exposed Haskell packages for use on the online site? (https://code.world/haskell#). I noticed that I could import Data.Vector (vector) and Control.Monad.State (mtl).
The current reference and appendix sections on the guide page are a good start to see the documentation for code-world and other related high-level APIs, but I believe it would be better to let the user know the list of other "low-level" exposed packages/modules so they can get the general idea of what's available to use and search the doc for them somewhere else in case they'd prefer to use those modules directly for whatever reason.
Thanks :D!

https://github.com/google/codeworld/blob/master/codeworld-available-pkgs/codeworld-available-pkgs.cabal is the canonical source


rudy covers only 1/4 of the diearea

Describe the bug
the Rudy map looks correct, but is displayed in the lower quadrant of the diearea
UNITS are 1000
DIEAREA ( -479600 -479600 ) (<PHONE_NUMBER>00 ) ;

Expected Behavior
Rudy map should match the diearea
Environment
M1 MacOS
OpenROAD v2.0-14310-g69872cd76 (Jun 25 2024)
Features included (+) or not (-): +Charts +GPU +GUI +Python

To Reproduce
openroad -gui -threads max risc2.or
read_lef lef/risc2.lef.gz
read_def def/risc2.def.gz
global_placement
detailed_placement
iccad04risc2.tgz
Relevant log output
No response
Screenshots
No response
Additional Context
No response

Unrelated to Rudy, but using the same lef/def.
Let's try mpl2 to get at better macro placement:

macro_placement
[WARNING MPL-0100] No macros found.

The MACRO's are PLACED and of CLASS RING.

FYI @stefanottili This warning in from the original macro placer mpl, not mpl2.

changing CLASS RING to CLASS BLOCK get's me a MACRO placement, but then detailed placement fails with
NesterovSolve] Iter:  420 overflow: 0.105 HPWL:<PHONE_NUMBER>
[NesterovSolve] Finished with Overflow: 0.099651
[WARNING DPL-0038] No 1-site fill cells detected.  To remove 1-site gaps use the -disallow_one_site_gaps flag.
[INFO DPL-0034] Detailed placement failed on the following 62 instances:
[INFO DPL-0035]  lx1/lx0/IRAM/MUX_INSTW_UP/U27
...
[ERROR DPL-0036] Detailed placement failed.

I'm building OpenRoad with mpl2, so why wouldn't it be used ?
[ 74%] Built target mpl2_lib
[ 74%] Built target mpl2

read_lef lef/risc2.lef.gz
read_def def/risc2.def.gz
macro_placement
global_placement -density 0.95
detailed_placement

Am I corred to assume that mpl2 won't derive hierarchy information from def using the DIVIDERCHAR info ?
So I need to have hierarchical verilog to run rtl_macro_placer ?

jupp, confirmed, rtl_macro_placer coredumps when just reading lef/def.
Side notes:
a) why not rtl_macro_placement as in macro_placement/global_placement/detailed_placement ?
b) why not hier_macro_placement as there is no rtl involved, just gate level netlist hierarchy ?
read_lef lef/risc2.lef.gz
read_def def/risc2.def.gz
rtl_macro_placer

[INFO ODB-0227] LEF file: lef/risc2.lef.gz, created 13 layers, 16 vias, 147 library cells
[INFO ODB-0127] Reading DEF file: def/risc2.def.gz
[INFO ODB-0128] Design: frisc
[INFO ODB-0130]     Created 629 pins.
[INFO ODB-0131]     Created 32622 components and 196041 component-terminals.
[INFO ODB-0132]     Created 2 special nets and 65244 connections.
[INFO ODB-0133]     Created 34034 nets and 128506 connections.
[INFO ODB-0134] Finished DEF file: def/risc2.def.gz
Floorplan Outline: (-479.6, -479.6) (480, 480),  Core Outline: (-479.6, -478.4) (480, 479.2)
Traversed logical hierarchy
	Number of std cell instances: 32615
	Area of std cell instances: 521777.56
	Number of macros: 7
	Area of macros: 349004.25
	Area of macros with halos: 349004.25
	Area of std cell instances + Area of macros: 870781.81
	Core area: 918912.94
	Design Utilization: 0.95
	Core Utilization: 0.92
	Manufacturing Grid: 10

Signal 11 received
Stack trace:
 0# handler(int) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 1# _sigtramp in /usr/lib/system/libsystem_platform.dylib
 2# mpl2::HierRTLMP::createDataFlow() in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 3# mpl2::HierRTLMP::createDataFlow() in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 4# mpl2::HierRTLMP::runMultilevelAutoclustering() in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 5# mpl2::HierRTLMP::run() in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 6# mpl2::MacroPlacer2::place(int, int, int, int, int, float, int, float, int, int, int, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, int, bool, char const*) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 7# mpl2::rtl_macro_placer_cmd(int, int, int, int, float, int, float, int, int, int, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, float, int, bool, char const*) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 8# _wrap_rtl_macro_placer_cmd(void*, Tcl_Interp*, int, Tcl_Obj* const*) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
 9# TclNRRunCallbacks in /opt/homebrew/Cellar/tcl-tk/8.6.14/lib/libtcl8.6.dylib
10# TclEvalEx in /opt/homebrew/Cellar/tcl-tk/8.6.14/lib/libtcl8.6.dylib
11# Tcl_Eval in /opt/homebrew/Cellar/tcl-tk/8.6.14/lib/libtcl8.6.dylib
12# gui::TclCmdInputWidget::executeCommand(QString const&, bool, bool) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
13# gui::startGui(int&, char**, Tcl_Interp*, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, bool) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
14# ord::tclAppInit(Tcl_Interp*) in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad
15# Tcl_MainEx in /opt/homebrew/Cellar/tcl-tk/8.6.14/lib/libtcl8.6.dylib
16# TclInitNamespaceSubsystem in /opt/homebrew/Cellar/tcl-tk/8.6.14/lib/libtcl8.6.dylib
17# main in OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad

@stefanottili Could you create a new issue reporting the MPL2 crash? Discussing MPL2 in the rudy issue might the discussion very confusing here.

@stefanottili, I am testing your design with the latest master branch and no longer see the issue. I believe PR https://github.com/The-OpenROAD-Project/OpenROAD/pull/5293 fixed it:

I will close this now, but please feel free to open it again if the error happens again.