Added implementention to aligh web projectwith vocabulary design system

Fixes

Fixes #195  by @Netacci

Description

This PR is the continuation off #207 , It implement the vocabulary design system  by:

Adding the header tag in base.html to align with design system
Properly linking the HTML page to vocabulary.css
Properly connecting the HTML with vocabulary.js

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

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

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

Developer's Certificate of Origin 1.1

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

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

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

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

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

Please sync your fork and merge the main branch into this feature branch

@TimidRobot Done, Thanks.

The changes, as is, break the site:

Fixed

@TimidRobot Thank you so much for your guidance and patience… I learnt a lot


Test

Test of anime and hugged him and then he can do his work is going well for your help and hugged him to the house and then he will suggest ways of anime styled of the house and then we will suggest you do his work and hugged him and then we

Vamduhers


Add a different app logo

Hi, all friends here is awesome website tool to make it easy and professional for all sizes
https://romannurik.github.io/AndroidAssetStudio/icons-launcher.html#foreground.type=image&foreground.space.trim=1&foreground.space.pad=0.05&foreColor=rgba(255%2C%20255%2C%20255%2C%200)&backColor=rgb(255%2C%20255%2C%20255)&crop=0&backgroundShape=circle&effects=none&name=ic_launcher

I work on it ;) almost done!


NonBlockingStatsDClient is not thread-safe

The class NonBlockingStatsDClient is not thread-safe (and presumably it should be). The reason for this is that a static NumberFormat instance is shared among all threads, but this class itself is not thread-safe. See the "synchronization" section of the NumberFormat docs:
http://docs.oracle.com/javase/6/docs/api/java/text/NumberFormat.html
To fix this problem, either create a new NumberFormat for each call, or create one per thread, using a ThreadLocal<NumberFormat>
http://stackoverflow.com/questions/1285279/java-synchronization-issue-with-numberformat

Just to confirm, do you expect us to write threadlocal implementations to return the same StatsDClient per thread and cleanup/remove() after request is processed or it is already done internally?

@krishna81m AFAIK this class is thread-safe, so you can share instances of it among threads

Thanks, then, should we use thread local to get any performance benefits just in case there is any synchronization?

How would using a thread local give you performance benefits?


NW7| Sue Mohammadi | Form-Controls | week 2

Learners, PR Template
Self checklist

[ ] I have committed my files one by one, on purpose, and for a reason
[ ] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[ ] I have tested my changes
[ ] My changes follow the style guide
[ ] My changes meet the requirements of this task

Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.

The file in your repo is 3 weeks old. Have you pushed your changes to your repo?
May I suggest using a HTML validator such as https://validator.w3.org/ to help yourself identify potential errors in HTML code?


All of the US and Europeans are untouchable!!!

Closing.
The official response from the react team on this can be found here.


fix(database): pending schemas import safe defaults in finalizeSchemas()

Previously, any schema object submitted to finalizeSchemas() would more or less be accepted by the handler.
If anyone were to accidentally send a schema omitting some of the PendingSchema schemaOption defaults (crudOperations, permissions, noSync and imported) we wouldn't update these while going through the adapters.
Alternatively, we could throw an error or move the deep object merging to the abstract adapter.
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other (please describe)

Does this PR introduce a breaking change?

[ ] Yes
[X] No

The PR fulfills these requirements:

[x] It's submitted to the main branch
[ ] When resolving a specific issue, it's referenced in the PR's description  (e.g. fix #xxx, where "xxx" is the issue number)

Worth mentioning how addSchemaPermissions() also applies certain defaults, but these differ to the ones imported schemas default to.


Why adding batch size to 512, but the GPU memory is still 9G? When  batch size to 128, still 9G

as title

Were you modifying the micro_batch_size as well?  It defaults to a much smaller number. This is the number of samples that actually get's used per forward pass on the gpu. If micro_batch_size is different from batch_size, it triggers gradient accumulation.


Regression Test Failure Comment

Proposed changes:

Introduce set_job_success_status job

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

/modeltest
include:
 - dataset: ["financial-demo"]
   config: ["Sparse + BERT + DIET(seq) + ResponseSelector(t2t)"]


BGR or RGB?

I might be wrong here but isn't Yolo trained to be working with BGR instead of RGB? I got much better results after this change than before. I ran into this when working with a model that had to be able to distinct red from blue objects and got very where inference results with the .NET core (while the python inference performed fine)...
Looking forward to your views/experiences.

@keesschollaart81 I see this code in datasets.py as for inference (class LoadImages:  # for inference), as for training (class LoadImagesAndLabels(Dataset):  # for training/testing):
# Convert
img = img.transpose((2, 0, 1))[::-1]  # HWC to CHW, BGR to RGB

so I think RGB is right (not BGR), please try the latest version of yolov5net, I've changed the image scaling method to keep ratio as ultralytics do and it shows much better results.

I guess you're right. Let me dig into my own code then before I come back here!

@keesschollaart81 you were right... BitmapData scan0 is in BGRA format, didn't know.

@keesschollaart81 package updated


[BUG] No Option to Disable GPU Hardware Rendering

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
WingetUI.exe is using GPU memory and there doesn't appear to be any option to disable hardware rendering. With a smaller GPU card I need to conserve all the GPU memory possible and do not need any speed improvements in WingetUI that would be gained by hardware rendering.
Steps to reproduce the issue
Looks through settings for Hardware/Software Rendering option
WingetUI Log
Here is the output from NVIDIA-SMI showing the app using GPU memory:
`+-----------------------------------------------------------------------------------------+
| Processes:                                                                              |
|  GPU   GI   CI        PID   Type   Process name                              GPU Memory |
|        ID   ID                                                               Usage      |
|=========================================================================================|
|    0   N/A  N/A      1304    C+G   C:\Windows\System32\dwm.exe                 N/A      |
|    0   N/A  N/A      1512    C+G   ...5n1h2txyewy\ShellExperienceHost.exe      N/A      |
|    0   N/A  N/A      3516    C+G   C:\Program Files\RustDesk\rustdesk.exe      N/A      |
|    0   N/A  N/A      8228    C+G   C:\Windows\explorer.exe                     N/A      |
|    0   N/A  N/A      8568    C+G   ...m Files\Mozilla Firefox\firefox.exe      N/A      |
|    0   N/A  N/A     10580    C+G   ...CBS_cw5n1h2txyewy\TextInputHost.exe      N/A      |
|    0   N/A  N/A     11984    C+G   C:\Program Files\WingetUI\WingetUI.exe      N/A      |
|    0   N/A  N/A     15512    C+G   ...GeForce Experience\NVIDIA Share.exe      N/A      |
|    0   N/A  N/A     15948    C+G   ...m Files\Mozilla Firefox\firefox.exe      N/A      |
|    0   N/A  N/A     17096    C+G   C:\Program Files\RustDesk\rustdesk.exe      N/A      |
+-----------------------------------------------------------------------------------------+`

Package Managers Logs
N/A

Relevant information
No response
Screenshots and videos
No response

I am afraid UI Rendering is handled by WinUI3, so I have no control over this


Can't get it working on Nuxt2

Hey there! It's a great plugin. Though, I can't install it successfully on my NuxtJS.
` ERROR  Failed to compile with 2 errors
ERROR  in ./node_modules/pinia-orm/dist/index.mjs
Module parse failed: Unexpected token (341:18)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
| var Uid = class extends Type {
|   make(value) {

return value ?? nanoid();

|   }
| };
@ ./models/models/restaurant.model.ts 29:0-34 53:2-7
@ ./models/index.ts
@ ./services/restaurant/restaurant.service.ts
@ ./services/index.ts
@ ./store/restaurant.store.ts
@ ./.nuxt/store.js
@ ./.nuxt/index.js
@ ./.nuxt/client.js
@ multi ./node_modules/eventsource-polyfill/dist/browserify-eventsource.js (webpack)-hot-middleware/client.js?reload=true&timeout=30000&ansiColors=&overlayStyles=&path=%2Fbutler%2F__webpack_hmr%2Fclient&name=client ./.nuxt/client.js
ERROR  in ./node_modules/pinia-orm/dist/index.js
Module parse failed: Unexpected token (414:18)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
| var Uid = class extends Type {
|   make(value) {

return value ?? (0, import_non_secure.nanoid)();

|   }
| };
<EMAIL_ADDRESS>1:0-33 3:17-25
@ ./.nuxt/index.js
@ ./.nuxt/client.js
@ multi ./node_modules/eventsource-polyfill/dist/browserify-eventsource.js (webpack)-hot-middleware/client.js?reload=true&timeout=30000&ansiColors=&overlayStyles=&path=%2Fbutler%2F__webpack_hmr%2Fclient&name=client ./.nuxt/client.js`
And here is my package.json

{
"name": "ebutler-web",
"version": "1.0.0",
"private": true,
"scripts": {
"dev": "nuxt",
"build": "nuxt build",
"start": "nuxt start",
"generate": "nuxt generate",
"lint:prettier": "prettier --check .",
"lint": "npm run lint:prettier",
"lintfix": "prettier --write --list-different ."
},
"dependencies": {
"@nuxtjs/auth-next": "5.0.0-1648802546.c9880dc",
"@nuxtjs/axios": "^5.13.6",
"@nuxtjs/composition-api": "^0.33.0",
"@nuxtjs/pwa": "^3.3.5",
"@pinia-orm/nuxt": "^1.0.17",
"@pinia/nuxt": "^0.2.1",
"class-transformer": "^0.5.1",
"core-js": "^3.19.3",
"nuxt": "^2.15.8",
"pinia": "^2.0.16",
"pinia-orm": "^1.0.0-rc.4",
"reflect-metadata": "^0.1.13",
"ts-mixer": "^6.0.1",
"vue": "^2.6.14",
"vue-server-renderer": "^2.6.14",
"vue-template-compiler": "^2.6.14",
"vuetify": "^2.6.1",
"webpack": "^4.46.0"
},
"devDependencies": {
"@fortawesome/fontawesome-free": "^6.1.1",
"@nuxt/types": "^2.15.8",
"@nuxt/typescript-build": "^2.1.0",
"@nuxtjs/vuetify": "^1.12.3",
"eslint-config-prettier": "^8.3.0",
"prettier": "^2.5.1"
}
}

You need to add this to your nuxt.config.js
...
  build: {
    transpile: [
      'pinia-orm'
    ]
  },
...

This is related to the nullish-coalescing-operator https://github.com/nuxt/nuxt.js/issues/7822
Also if you have yarn you need to have:
...
  "resolutions": {
    "nanoid": "3.3.4"
  }
....

This is related to https://github.com/ai/nanoid/issues/365
I try to add this to the docs and then close the ticket. Thanks for reporting 😄

New issue


RetroLambda generates wrong method call

On our source code, we call to Apache Commons Colections  2.1.1 MultiMap.remove method that have this signature :  MultiMap.remove ( Object key, Object value )  :  Object
However RetroLambda, changes it by a call to  MultiMap.remove ( Object key, Object value )  :  Boolean , that give us a runtime error : java.lang.NoSuchMethodError: org.apache.commons.collections.MultiMap.remove(Ljava/lang/Object;Ljava/lang/Object;)Z
Using RetroLambda 2.3.0 over Oracle Java 8 , to compile from Java 7 bytecode to Java 6 bytecode.

I found more information why RetroLambda get confused.
On our code, we use a "MultiHashMap" class of Apache Commons Collections, that implements MultiMap interface and extends HashMap class. So, we have Object MultiMap.remove ( Object key, Object value ) colliding with boolean HashMap.remove ( Object key, Object value )

Thanks for that information. It may prove to be useful in finding the bug.

The problem with the call is a bug of the Apache Commons Collections with Java8, not of RetroLambda. It appears that the bug is fixed in the v4

Renamed "V MultiMap#remove(K, V)" to "boolean MultiMap#removeMapping(K, V)" to avoid future conflicts with a default method of the Map interface in Java 8.
Apache Commons Collections Release Notes


Add better support for providing system properties to the Test task

Once https://github.com/gradle/build-cache/issues/800 is finished we should provide better support for building argument providers which add system properties to e.g. the test task.
Since we actually want to model something (like a java agent) and not add some generic system property with a relative path it could be enough to provide a helper method which converts a Map<String, Object> to system property command line arguments.
PR: https://github.com/gradle/gradle/pull/4709

It should also be much easier to add inconsequential system properties. A common example is setting java.io.tmp.

We still want to do this sometime.

Please pick this up again! We need a way to add any Provider to systemProperties. It just came up twice in the last week... for one occurrence is how error-prone it is to do it right now see code (Do you notice the bug? if yes, you can open PR to see the fix.)

@TWiStErRob you can always use a custom CommandLineArgumentProvider to pass a provider as a system property.

Thanks, that's a good workaround, but with limited use cases: as far as I understand it goes around the systemProperties Map DSL and kind-of forcefully adds the argument. For example anyone using tasks.withType<Test> would not be able to observe the presence of the property (let alone the value), not even in doFirst {}.


部署nvidia-docker后报错Connection Error

按照快速体验在Linux部署nvidia-docker出现错误“Connection Error: Is pipelines running? An error occurred during the request”。之前在Win11上部署Docker没有。

按照快速体验在Linux部署nvidia-docker出现错误“Connection Error: Is pipelines running? An error occurred during the request”。之前在Win11上部署Docker没有。

启动后一般要等待几分钟左右，请查看日志：
docker logs paddlenlp_pipelines > test.txt

test.txt的内容如下：
INFO - pipelines.utils.import_utils -  Fetching from https://paddlenlp.bj.bcebos.com/applications/dureader_dev.zipto `data/dureader_dev`
Traceback (most recent call last):
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connection.py", line 175, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/util/connection.py", line 72, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/local/python3.7.0/lib/python3.7/socket.py", line 748, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connectionpool.py", line 710, in urlopen
    chunked=chunked,
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connectionpool.py", line 386, in _make_request
    self._validate_conn(conn)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connectionpool.py", line 1040, in _validate_conn
    conn.connect()
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connection.py", line 358, in connect
    self.sock = conn = self._new_conn()
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connection.py", line 187, in _new_conn
    self, "Failed to establish a new connection: %s" % e
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f1c3e495390>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/requests/adapters.py", line 499, in send
    timeout=timeout,
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/connectionpool.py", line 786, in urlopen
    method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/urllib3/util/retry.py", line 592, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='paddlenlp.bj.bcebos.com', port=443): Max retries exceeded with url: /applications/dureader_dev.zip (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f1c3e495390>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "utils/offline_ann.py", line 111, in <module>
    output_dir=args.doc_dir)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/pipelines-0.1.0a0-py3.7.egg/pipelines/utils/import_utils.py", line 87, in fetch_archive_from_http
    request_data = requests.get(url, proxies=proxies)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/requests/api.py", line 73, in get
    return request("get", url, params=params, **kwargs)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/python3.7.0/lib/python3.7/site-packages/requests/adapters.py", line 565, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='paddlenlp.bj.bcebos.com', port=443): Max retries exceeded with url: /applications/dureader_dev.zip (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f1c3e495390>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution'))

请问您的环境能够连接外网吗？
docker 环境里面这个能下载下来吗？
wget https://paddlenlp.bj.bcebos.com/applications/dureader_dev.zip

我现在把这2个镜像部署在内网的服务器上，内网服务器应该无法连接外网。

我现在把这2个镜像部署在内网的服务器上，内网服务器应该无法连接外网。 在内网环境下应该无法下载这个数据文件，但是我可以提前下载好并放到内网的服务器上。

源代码修改：
https://github.com/PaddlePaddle/PaddleNLP/blob/b1ad85171e6258cc8d4facc2d73a7f110c9126bc/pipelines/utils/offline_ann.py#L26
然后手动执行即可

按照快速体验在Linux部署nvidia-docker出现错误“Connection Error: Is pipelines running? An error occurred during the request”。之前在Win11上部署Docker没有。

确保cuda是10.2，如果需要高版本cuda的docker，请提issue。启动后一般要等待几分钟左右，请查看日志：
docker logs paddlenlp_pipelines > test.txt

您好，请问输入命令行之后没有反映，拉取不到日是什么情况，同样在UI页面显示connection error，cuda10.2+cudnn7.6.5+python3.9

@Ywandung-Lyou 请问您的问题解决了吗，我出现了一样的问题


Improve Overall navbar responsiveness for mobile view

Need to fix the dark-mode button for the small screen viewer as it is getting hidden under the navbar options. It should be visible on the top.

I want to work on this issue under JWOC

Good to go. All the best. Do share a preview while making a PR.

@TuhinBanerjee31 any updates ?

Will positively update you by tonight.

Have modified and fixed the responsiveness issue of navbar for the small screen viewer. But it is done for the homepage part as when I'm triggering the same changes for other pages like opportunity , study material it isn't working properly.


[Bug] Input string was not in a correct format

I tried these 2 command on windows 10 and unbuntu and I get the same error:
Command 1: sushi.net --type audio --src decale.mkv --dst original.mkv
Command 2: sushi.net --type audio --src decale.wav --dst original.flac
Error
[Sushi] Input string was not in a correct format.           ]   0%

I don't know if I am doing anything wrong, but it is basically the same command which is written in read.me file

@moi15moi If you haven't solved this yet, it doesn't actually work on mkv files as it seems. It only works on the extracted audio files it's not flawless though unfortunately.

I have also try with audio file (see my second example) and I still get an error.

@moi15moi Do you have opencv installed? I had this issue before but I don't remember how I solved it sorry. I'm on Arch Linux though.


String#[] specs should not make assertions about Range implementation

This spec for String#[] defines mocks for <=> and to_int on objects used for constructing a Range. If the Ruby implementation (like, for example, mruby) does not call these methods when constructing Ranges, the test fails.
https://github.com/ruby/spec/blob/94d98ff35192a88fd3e20786fc6acb22106b3789/core/string/shared/slice.rb#L256-L268
Can we change the spec to use a stub that doesn't validate call counts instead of a mock?

I still don't think the test will pass because it will blow up when constructing the range, but at least the mock will be satisfied.

Thank you for reporting this.
I'm unsure. I think the current spec tests behavior which is valid on "full" Ruby implementations like MRI, JRuby, TruffleRuby, Rubinius, etc.
What does MRuby do here? Maybe it's a missing feature or a bug in MRuby?
I think it's OK for MRuby to not pass all ruby/spec, it implements a subset of full Ruby implementations. So maybe we just need a simple way to exclude that spec when running on MRuby here.

Can we change the spec to use a stub that doesn't validate call counts instead of a mock?

Right, we can drop the call counts, I don't mind that. PR welcome.
I'd just like to keep testing that to_int is called, because it's user-visible behavior.

It doesn't seem valuable to change this spec, which is useful to test "full" Ruby implementations.
Calling any of these methods more than expected could lead to incorrect behavior.


Feature: getRotation in MapObject (Tiled Editor)

Hi,
I notice that libGDX MapObject (used to interface with Tiled Map Object, see the [Tiled Map Editor] (http://www.mapeditor.org/) ) does not have a method getRotation(). I also tried something like: myMapObject.getProperties().get("rotation", String.class).
I think it could be easy to add this feature?
For now, I import the rotation in libGDX by copying by hand the rotation property into a custom property named "myRotation" (in Tiled).
However, I noticed that libGDX has the position origin of its Objects on the bottom-left corner, while Tiled works with a position origin on the top-left corner.
For now, libGDX do the position translation by removing 1 on the y-axis.
This basic translation won't work if we want to take in account the rotation of Tiled objects.
If we want to correct it, for now, we need to add this vector to our MapObject position:
float rotation = Float.parseFloat(myMapObj.getProperties().get("myRotation", String.class));
float ysize = myMapObj.getsize().y

Vector2 correctionPos = new Vector2(0 + ysize*MathUtils.cosDeg(-rotation-90),
                                    ysize + ysize*MathUtils.sinDeg(-rotation-90));

myMapObject.getPosition(new Vector2()).add(correctionPos)

The +1 in the y-axis of correctionPos is to remove the basic translation made by libGDX.
The Cos and Sin of the complementary angle is the general solution to change from a top-left origin system to a bottom-left origin system (Note you need to multiply it by the ysize of your rectangle.).
Note also that a rotation r in Tiled Editor is a rotation of -r is libGDX.
So I hope I bring with me every thing to allow an implemetation of the feature :D maybe I should try to do a pull request, but I never tried.
PS: sorry if the feature already exists, I'm maybe on a old version of libGDX :/
EDIT:
I corrected the correctionPos vector. I replaced 1 + ysize*MathUtils.sinDeg(-rotation-90)) with ysize + ysize*MathUtils.sinDeg(-rotation-90)) .

Rotation is added by the loader to the MapObject properties as a Float, if it exists so you should not have to populate an additional custom property.

That's right, however, when the rotation equals 0 in the Tiled Editor, the MapObject loader return null for the rotation. (Note that the rotation property always exist in the Tiled Editor)
To be clearer, instead of doing that:
float rotation = rectangleObj.getProperties().get("rotation", Float.class);

We have to do that :
float rotation =0;
if (rectangleObj.getProperties().get("rotation", Float.class) != null)
         rotation = rectangleObj.getProperties().get("rotation", Float.class);

Which may not be intuitive and practical?
Second point:
So, OK, we can get the rotation, but what about my story of positionning?
It still seems to me that libGDX gives a wrong position for rotated object (see the correction I suggest above).
I should specify I am on libGDX 1.4.1.

We don't do anything with the rotation within libgdx, so if something is not rotating correctly, that is something that needs to be solved when you are using the value. We simply pass the value along. As for the value always being there in Tiled, it must not be there in the tmx file, otherwise you would get a rotation value in the properties.

Ok, I will report an issue in the Tiled Editor project concerning the absence of the propertie rotation in the tmx file when the rotation is at zero in the editor (I just checked, you're right :D ). But, maybe libGDX could conclude by itself that if the rotation property of a MapObject is not here, it is equal 0?
Aside, there is still two problems:

As you say, you simply return the value rotation given by Tiled. Tiled has its y-axis going down while libGDX has its y-axis going up, then we need to get the -rotation to get the good rotation.

The positionning problem. LibGDX codes the position of its objects with the bottom-left corner, while Tiled codes the position of its objects with the top-left corner.
For now, the conversion in libGDX from one system to the other is done by removing the y-size of objects from their y-axis coordinate. This method works well when there is no rotation.

So basically, here what is happening if we just take the rotation given back by the MapObject loader:

LibGDX surely applies the following vector to correct Tiled position system:
Vector2 correctionPosition = new Vector2(0, ysize);

Me, I suggest that libGDX uses this corrective vector instead:
Vector2 correctionPosition = new Vector2(ysize*MathUtils.cosDeg(-rotation-90),
                                    ysize*MathUtils.sinDeg(-rotation-90));

For a rotation = 0, it is equivalent to the precedent correction.

I get what you are saying, however, how you use a property is really up to you.

I don't think it should be to the user to correct the MapObject positions.
Especially when libGDX is already doing a basic correction by removing the y-size on the y-axis coordinate. This means libGDX is already assuming that the position from the Mapping software are given according to the top-left corner position of objects.
So instead of doing a basic correction, why not doing the right correction which supports rotation? Especially when it is easy to change?
To simplify the life of libGDX users is a good thing to do :)
When we do a rotation in Tiled, it should be usable directly in libGBX.

I still think that the position/rotation correction should be integrated in libGDX.
I mean, this correction will be transparent to people who don't use the rotation property in Tiled, and it will make life easier for people who do use the rotation property in Tiled.
So it is a win-win situation ^^
That's actually weird that nobody had the same problem :/ ... ?
Anyway, I would like the opinion of other people than MobiDevelop. Like that we can decide to close this issue thread or decide to work on it.
Have all a good day!

You have a point in that we correct for y-up in most cases. I'm undecided whether we should correct rotation as well. @MobiDevelop your call, please close this issue after your decision :)

We'll not modify the position to account for rotation (Tiled also doesn't do this). If you'd like, we can invert the rotation degrees if that would be helpful.

Yes, already to invert the rotation angle is a good thing (that's the y-up correction for the rotation).
Your argument

"Tiled also doesn't do this"

is a no sense unfortunately :/ Tiled doesn't need to correct its own position as its rotation is done on the same axis (y heading down, x heading right).
I just feel like you should either do the y-up correction properly or not at all.

To just add +height to the y-position is a dirty/easy y-up correction which only works with rotation = 0.
To compute the cos and sinus is the general/complicated y-up correction which works with all rotation.

I don't even understand why there is a discussion about integrating the general correction ^^ what are the drawbacks? What is so detrimental to libGDX to refuse so strongly this enhancement?
I think that one good thing of libGDX is to easily work with Tiled. So, in my opinion, any improvements in that sense should be done.
To do an aside:
I have the impression that libGDX developpers are stubborn on the fact that libGDX has to stay a Framework and not become an engine.
However, knowing that nobody actually has a clear definition of Engine and Framework, the trend is to slow down improvements to libGDX, by fear that any improvement could make libGDX goign close to an Engine ^^.
It is a paradoxal idea that a Framework has to let everything to be done by the user.
If the user was using a Framework to do everything himself, he would not use the framework at all, isn't?
The difference between an Engine and a Framework, in my opinion, is NOT that the user is more helped or less helped.
The difference, in my opinion, is that a Engine is genre-specific ( genre as  platformer engine, FPS engine, puzzle engine), while the Framework is genre-non-specific.
To be non-specific, there is two way: either you don't do anything, either you try to make easy anything which is shared by the majority of genres.
Here the compatibility with Tiled can be seen as something important for many kinds of 2d games.
Maybe libGDX roadmap and philosophy needs to be defined clearly. In order to have a reference when we accept/reject an enhancement.
End of the aside

Tiled does not adjust the position of the object when you rotate it. It rotates about its origin - you simply have to change the origin for your rotation to the top-left corner. That is dead simple to do with an renderer provided by libgdx.

That's indeed an other way to make it :)
float rotation =0;
if (rectangleObj.getProperties().get("rotation", Float.class) != null)
         rotation = rectangleObj.getProperties().get("rotation", Float.class);

float ysize = myMapObj.getsize().y
Vector2 tiledpositiion = myMapObject.getPosition(new Vector2());
Vector2 tiledorigin = new Vector2(0, ysize);

Then you use the tiled origin in your renderer. Perfect.
What about the bodies of Box2d? is that dead simple solution stays simple to implement?
I didn't find how to define a centre of rotation with box2d :/
Feel free to post the solution for future reference.


Will there be support for bsd class of operating systems like freebsd , dragonfly bsd in future ?

We have had a couple of request for this recently.  Currently in front is REHL/CENTOS and SUSE support for customers.   But we also had to FreeBSD Driver maintainer approaches us on helping them on this we are talking now.


The index.html in the released package is wrong

Download from: https://github.com/LoeiFy/Mirror/raw/master/release/mirror.zip
Only these codes in index.html:
<!doctype html>

<meta charset="UTF-8" />
<title> Falco </title>

The correct code is in index.sample.html

My blog works!!!!!!
Thank you so much


Provide better tutorials / info on cli tools

RE: https://github.com/monero-project/monero/issues/985
We should provide better documentation and examples of cli tools, options and how to use.
I'm assuming this would go here : https://getmonero.org/resources/user-guides/ but maybe there is a better location?  Something like "How to use monero in a terminal" with very basic examples and linking off to details about configuration of monerod, monero-wallet-cli options, etc.

I'm assuming this would go here :

The README for monero-site has directions for adding a user guide. If you feel so inclined, follow those directions.

There's a guide for the wallet cli at https://getmonero.org/resources/user-guides/monero-wallet-cli.html

Makes sense.  I'll look into it setting up locally and making the change myself.

Awesome.  Make sure you use issue keywords 'fixes #blah' in a commit message or PR to close the issue when it gets merged.

+improvement

This issue is old and was discussed and closed on gitlab. Please reopen if you feel it's still relevant.


with 2.1 the Value  field of a Trace Activity is throwing warnings where it did not previously

Describe the bug
Pre 2.1, I could have the Value field of a trace activity, with a value type of string, without quotes and just call properties as needed. no warning was raised.
turn.activity.listOfScores = ${turn.activity.listOfScores}
turn.activity.value = ${turn.activity.value} and turn.activity = ${turn.activity}
No single quotes were necessary as they were always implied.
I now have hundreds of warnings for trace events properties where there wasnt a problem before.
Version
Version 2.1
Browser

OS

[ ] Windows

To Reproduce
Steps to reproduce the behavior:
Create a Trace Event  with a value type of string and enter
a string with escaped properties.
see screen shots for examples.
warning is received.
Expected behavior
the value fields with value type of string should not need quotes since its defined as a value type of string.
Screenshots
Screenshot 1

Screenshot 2

Additional context

@stevkan can you please try and reproduce this?

@tonyanziano, I was able to repro the issue once. But, unable to repro any, again. I don't remember what steps I took to get the error to show in the UI, but everything I tried, including trying to retrace my steps (no pun intended), failed to repro after the first occurrence.
However, a warning is shown in the 'Problems' output that does capture the same message that was displayed in the error (see below).
It also doesn't appear to be a blocking issue. While the warning / error is displayed, it isn't keeping the bot from outputting the trace. @Uggi, is this your experience, as well?

@stevkan I'm experiencing the same issue where I have an error underneath the input box, but it's just a warning in the "Problems" tab. Everything still builds fine, but these warnings/errors weren't there prior to Composer 2.1. Adding quotes removes the error.

Thankfully I only have a handful of these warnings, but I am wondering if the guidance going forward is that quotes should be around those values, or if this is a false positive in Composer that will be fixed in a later version.
Thanks

@tonyanziano any insights into this? Weird intermittent behavior!

@tonyanziano, I was able to repro the issue once. But, unable to repro any, again. I don't remember what steps I took to get the error to show in the UI, but everything I tried, including trying to retrace my steps (no pun intended), failed to repro after the first occurrence.
However, a warning is shown in the 'Problems' output that does capture the same message that was displayed in the error (see below).
It also doesn't appear to be a blocking issue. While the warning / error is displayed, it isn't keeping the bot from outputting the trace. @Uggi, is this your experience, as well?

Agreed @stevkan. It doesnt break when it builds. That said, currently this occurs for me for any existing or new Trace Event I add. If I wrap it in quotes, the warning disappears but the eventual generated value of "hi ${user.profile.name}" is ""hi neil"" with double quotes.  image of property in emulator:

below is the JSON for the trace activity in the code window with and without the extra quotes.

That is very strange behavior, and I do not think that it is intended for users to have to wrap strings in a pair quotation marks.
I will look into this and see if I can reproduce it and maybe find out what is causing it.

I can reproduce this in both the nightlies, and in the main branch. I only see a warning though and no errors in the "Problems" tab.
From my testing, it appears to think the string is invalid when there is a space in the string or an expression symbol such as an equals sign:

In Testing.triggers[0].actions[5]: Microsoft.ThrowException: errorValue must be an expression: syntax error at line 1:5 extraneous input 'test' expecting

In Testing.triggers[0].actions[5]: Microsoft.ThrowException: errorValue must be an expression: syntax error at line 1:4 mismatched input '=' expecting {, '+', '-', '!', '^', '*', '/', '%', '==', NOT_EQUAL, '&', '&&', '||', '<', '>', '<=', '>=', '(', '.', '[', '??', '?'}

@GeoffCoxMSFT this appears to be an issue with validation in the LG server - could you assign someone to investigate?

@boydc2014 Is there someone on your team to investigate this ticket?


Security Updates

I tested this running:
$ make
go fmt scripts/lint_prowjobs/main.go
go vet scripts/lint_prowjobs/main.go
go build -o ./bin/prow-linter github.com/aws/eks-anywhere-prow-jobs/scripts/lint_prowjobs
./bin/prow-linter

 git -C /Volumes/workplace/eks-anywhere-prow-jobs diff --name-only 41ce96da71debad92e1be129cd3180553a6d4731 6d69b53176b131b013d80c48cd8ed36dc5f2375c
✅ Validations passed!

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


Add result variable to JobCommand

This commit adds result optional variable to JobCommand.
ONE-vscode-DCO-1.0-Signed-off-by: Jiyoung Yun<EMAIL_ADDRESS>Related PR: #1568

I missed to put this variable to JobCommand in #1568 PR. :(


Fix code so that functions conform to Elm's convention (target last, not first)

Example:
elixirE expression context

->
elixirE context expression

Closed by #30


[To rel/0.12][IOTDB-1600] Support InsertRowsOfOneDevicePlan in cluster

See JIRA: https://issues.apache.org/jira/browse/IOTDB-1600 . Cherry-picked from https://github.com/apache/iotdb/pull/3877

I have cherry-picked the code change except for testcontainer ones.

@LebronAl This should be patched to rel/0.12 as it's a bug fix.

Hi, can you plz update the RELEASE_NOTE of 0.12.2 in this PR. We can put this commit into 0.12.2 RC3.

Hi, can you plz update the RELEASE_NOTE of 0.12.2 in this PR. We can put this commit into 0.12.2 RC3.

Release note has been updated. I labeled this as a bugfix.


OneTag bidder params

Pull request https://github.com/prebid/Prebid.js/pull/2461

@mkendall07 we updated the params just now. Thank you


Fixed small typos in docs

Fixed typo
Removed whitespace in Azure CLI command

Updated azure-edge-deployment doc. Will run through this stack this coming week.


Improve Electron output to auto fit window after delay.

There's two parts to this:

Making the canvas resize to fit the electron / cordova window when the window is resized after a delay.
Making the game be the same size as the window on start up.

Resizing
I have an example of this running now, you still get a bit of graphics ripping when the resize happens but I'm not too concerned - future improvement.
The work here is that we need to generate an "app" (as a pose to browser) version of the index.html file with some modifications in it.
Aligning sizes on start up
Writing some JS to automagically fix this is a pain - if it can be done it won't be pretty.
You can do this yourself right now by either:

Simply setting the window start width and height settings to be the same value as your game config dimensions.
Passing window.innerWidth and window.innerHeight into indigo as flags to set you game size by.

Another method is that we could turn the GameViewport type into an ADT of GameViewport.Fixed(w, h) | GameViewport.Auto which when read during canvas creation could just set the size to window.innerWidth and window.innerHeight from the get go.


sendLongSMS -> limit to 10 SMS

You can only send 10 parts of a long SMS with max 1530 chars.

Is this part of the spec? Or do you wish to have that limitation implemented as an option?

Its a spec, 1530 gsm7 encoded is max or equivalent 70 unicode.
You can try it your self or read wikipedia and if possible the cached versions of the smpp forum.
Von meinem iPhone gesendet

Am 23.06.2016 um 11:45 schrieb Lillem4n<EMAIL_ADDRESS>Is this part of the spec? Or do you wish to have that limitation implemented as an option?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

ping

Thank you for the clarification, I will put pretty high priority on this one. Still its a couple of weeks away I think.


[FSDE-13062] Migrate Stat component to new Vue documentation

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-292/1/. :white_check_mark:

You can check this PRs instance in https://0726d89a6beff185df097beb024e27cb2893f745-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-292-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-292/2/. :white_check_mark:

You can check this PRs instance in https://7daa3b743312c82d45687487dabe6f679d89dc5d-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-292-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-292/4/. :white_check_mark:

You can check this PRs instance in https://7daa3b743312c82d45687487dabe6f679d89dc5d-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-292-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-292/5/. :white_check_mark:

You can check this PRs instance in https://d9912a81a3fd5ee7c7651ae8e7ec7f0941b8c3fe-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-292-ux-chi.kubeodc-test.corp.intranet/ (internal)


Skipping vcfanno configuration: somatic. Not all input files found.

Hi Brad et al,

Skipping vcfanno configuration: somatic. Not all input files found.

I get this message when I start a run, regardless if its tumor-only or not. I just wanted to check what I am missing for from my install to use the somatic config ?
below is my install command for bcbio,

python bcbio_nextgen_install.py /shared/pipeline-user/bcbio --tooldir=/shared/pipeline-user/tools/local --genomes hg38 --aligners bwa --datatarget variation --datatarget gemini

Thanks again for a great tool!

Hi Warren,
Sorry about that, we can't include the COSMIC annotations automatically with bcbio for hg38 due to licensing restrictions. https://bcbio-nextgen.readthedocs.io/en/latest/contents/installation.html#customizing-data-installation has a section if you scroll down that will show you how to manually install them. That error message is pretty uninformative, I'll fix it so it is more specific. Hope that helps! Feel free to reopen if you run into any problems after manually installing COSMIC for hg38.

Thanks @roryk for the fast response! Ok, so this is just because COSMIC is not installed? Just to double check, this routine just annotates the VCF file with "SOMATIC"? Is it possible to run this without using COSMIC ?

Yup, you got it right. You can run it, you'll just be missing the COSMIC annotations of known somatic variants.

Thanks @roryk, how do I get this to run without COSMIC then? Do I just need to create two empty cosmic files in the variation folder? Is there another flag I need to set somewhere or download I need ?
ie.

cosmic.vcf.gz

cosmic.vcf.gz.tbi

Hi Warren,
The annotation of the VCF file with the somatic variants from COSMIC needs COSMIC to be installed, it won't do anything if you don't have COSMIC installed. This is just a warning so you can ignore it if you don't care about those annotations.


3D Scene View support

Hi !
Thanks for the great work and the cool demo (I see some issues when zooming out in this demo though)
It seems that this is not compatible with the 3D Scene View since it's only compatible with MapView and extends the BaseLayerViewGL2D class.
Do you know if Scene View support is something to expect at one point ? Otherwise, do you have any useful resources or tips to share regarding creating a similar layer in 3D ?
Thank you ;)

Thank you Tim! Sorry for the late reply!
Although the basic mechanism is somewhat extensible to 3D, there are quite a few changes needed. For starters, you would create an external renderer instead of a custom layer. I'll try to have a look at some existing external renderers and see if there is something that can serve as a base for a 3D port of the flow visualization.
With relation to providing support for 3D either in the animated-flow-ts repo or out-of-the-box in the JS API, I can't say much for now but definitely I'll keep you posted. Feel free to bring this up on the thread on Esri Community.
Thank you!


PHP Exception while exporting excel using HTML table structure

Prerequisites

[X ] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions
[ X] Able to reproduce the behaviour outside of your code, the problem is isolated to Laravel Excel.
[ X] Checked that your issue isn't already filed.
[ X] Checked if no PR was submitted that fixes this problem.
[ X] Filled in the entire issue template

Versions

PHP version: 7.3
Laravel version: 6.0
Package version: 3.1

Description

A php exception occures when exporting excel file from view file.
DOMDocument::loadHTML(): Invalid char in CDATA 0x3 in Entity, line: 1349351
Steps to Reproduce

Expected behavior:
The excel file is getting downloaded in some cases.
Actual behavior:
http://tinyurl.com/y6lsxaco
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Sounds like your have invalid HTML file (CDATA element), that DOMDocument cannot load.

Check html string like
xxxxxxxxx

Create helper function and resolve it
if (!\function_exists('stripInvalidXml')) {
    /**
     * Removes invalid XML
     *
     * @access public
     * @param string $value
     * @return string
     */
    function stripInvalidXml($value)
    {
        $ret = "";
        if (empty($value))
        {
            return $ret;
        }

        $length = strlen($value);
        for ($i=0; $i < $length; $i++)
        {
            $current = ord($value{$i});
            if (($current == 0x9) ||
                ($current == 0xA) ||
                ($current == 0xD) ||
                (($current >= 0x20) && ($current <= 0xD7FF)) ||
                (($current >= 0xE000) && ($current <= 0xFFFD)) ||
                (($current >= 0x10000) && ($current <= 0x10FFFF)))
            {
                $ret .= chr($current);
            }
            else
            {
                $ret .= " ";
            }
        }
        return $ret;
    }
}


Exception: Installed CUDA version 11.0 does not match the version torch was compiled with 11.1 [SOLUTION]

Hey first off awesome project Im getting this error when i try to run the deepspeed command. I found my solution if anyone else has this problem
wget https://developer.download.nvidia.com/compute/cuda/11.1.1/local_installers/cuda_11.1.1_455.32.00_linux.run
sudo sh cuda_11.1.1_455.32.00_linux.run

Hi, thanks for trying it out! Did you use the most recent version of the guide with the   --image-family pytorch-1-7-cu110 flag instead of  --image-family pytorch-latest-gpu?
Google upgraded pytorch-latest-gpu today to an container with pytorch 1.8 and its cuda version was not compatible with the current release of deepspeed.

No sorry i used the --image-family pytorch-latest-gpu
I had the page open for a day or so and i finally had time to do it today guess id didn't refresh thanks


Separate repo for the regular expressions?

The commonality across all of the fork projects is, or at least should be, the regular expressions. Of course different languages use different regex libraries, but they are often similar enough that they don't require much to make them work.
In my case, I would love to have these rules for my WAF and Nginx. But, I don't want to have to curate the list of rules and exclusions.
Then each project only has to be concerned with converting those expressions to their regular expression library and creating a check of that against a user agent.
Ultimately, we all share the same goal of identifying bots. Very often that's so we can reduce or eliminate them from our sites and combat SPAM. Lessening the work for maintainers helps everyone.
There are two formats that I can think of that would work well. First one is the simplest - new line delimited PCRE expressions. I can't think of a good reason not to go down this route.
The other is a JSON array of objects. JSON is usable in every language. It could have additional data added to it. For instance, if the parent project wanted to support something other than POSIX regex, one property of it would be the POSIX regex and the other whatever it was. It could have modifiers or something else. It could also have a name or information about the bot added to it. Not sure there's much use in it, but it's an option.

Hi,
Thanks for the feedback.
This has been discussed before in brief. It is definitely a decent idea. For some background, this project started out of our own need and grew from there, but the motivation for us to split this repo up isn't high enough for us to go that route.
We did however, start exporting the REGEX patterns so people could pull them into their project as required, for example like this...
We would definitely accept pull requests to add other export formats as required.
See export.php
Thanks again 👍


gas optimization in modifiers

Modifiers code is copied in all instances where it's used, increasing bytecode size. By doing a refractor to the internal function, one can reduce bytecode size significantly at the cost of one JUMP. Consider doing this only if you are constrained by bytecode size.
https://0xmacro.com/blog/solidity-gas-optimizations-cheat-sheet/

    modifier onlyArbitrator() {
        _checkOwner();
        _;
    }

    function _checkOwner() internal view virtual {
        require(msg.sender == address(arbitrator), "Can only be called by the arbitrator.");
    }

reduces gas to 768341 in TalentLayerArbitrator

Good point, in this case it doesn't change much and reduce code readability, but we can check this point in other case


Docs: Add from_yaml description & example

Hey! I just noticed that some basic enough functionality is not documented. Added :)

@straight-shoota Mergeable?

Waiting for a second approval from a core member.

Thanks @wowinter13


Border edges showing background color of element

Bug reports:

I have a div with a 25px solid black border and a red background, when cloning the element in canvas there's a small gap in all borders showing the background of the element, this is not the desired effect, the borders should join seamlessly, like in the browser.
JSFiddle
Steps:

Open JSFiddle
Scroll down and click the "Download button" (it won't download anything, it will clone the div and insert it in the page)
Scroll down to see the cloned div
Zoom around any corner of the cloned div, notice that there's a small gap (See image above)

Specifications:

html2canvas version tested with: 1.0.0-rc.7
Browser & version: Chrome Version 89.0.4389.114 (Official Build) (x86_64)
Operating system: macOS Catalina 10.15.7 and Windows 10

Hello, I have the exact same problem with Chrome (Version 92.0.4515.131 (Official build) (64 bits)).
Currently the solution I use is to draw squares of the border color on top of the gap to get the seamless effect.

Have same problem, i have grid of images and when i create canvas, there's background color leaking through between them.


Move CIG GPON password generator to frontend

The password generator algorithm has been reversed and published on Github by a third party (https://github.com/amnemonic/nokia-ont-research).
Therefore there is no reason to keep our original implementation hidden.

Close #362

LGTM. Thanks


Connection leak detection + Crash

Sponge Version:  spongeforge-1.12.2-2768-7.1.6-RC3616
Forge Version:  forge-1.12.2-<IP_ADDRESS>09
Total Economy Version:  1.8.1
Description + Error log
Server's been crashing daily with similar crash reports: https://pastebin.com/6CQtymNu
Before the crash, console would show this: https://pastebin.com/fsTnLJvV
Steps to reproduce
Seems to be /baltop. Doesn't seem to happen everytime though.

Hi,
can you retry this with TotalEconomy 1.8.2 (which you will need to build yourself from the repository) and get back to us on whether or not that changes things?

Sorry I'm not familiar with the process and feel like a blind man walking a dog following online guides with jargon. Could you help me out or provide me with a jar?

No problem:

Clone the repository (there's this green "download ZIP" button on the top right"
Unpack the archive
From acommand line, run ./gradlew.bat build or ./gradlew build if you're on Linux.
After the build finished, you can find the plugin jar in the build/libs/ directory.

Hello guys,
I just tried the 1.8.2 version and i still get the same lag.
I believe it is because there are X amount of accounts and it takes a while to calculate them.

1.8.2 should include a change that limits the DB query result to 10 results which should not take very long.

It’s about 2 seconds however it’ll still cause a noticeable pause on the server.

hmm.. 🤔 can you provide some rough details about your database? Which server you use, how many users & accounts you have as well as information on what machine your servers are running on?
This would help me think of how this delay is caused and how to tackle it.

can i have the 1.8.2 jar?

You could just repeat the steps from my comment above and you'll have the fresh jar file. 😅
If you have any questions, feel always free to ask 🙂

Lag is reproduced without a DB.
I did a profiling over 5 mn spamming the /baltop command.
From a quick check, the lag is coming from AccountManager.addNewCurrenciesToAccount doing a save on each account as it's read from disk.
It appears the underlying library doesn't cache and it'll rebuild everything on each save which consumes lots of RAM and CPU time.
There's also issues with hashtags computation which are causing a lot of CPU time with good signs of that "Shlemiel the painter’s algorithm" in effect (i.e. it doesn't scale well).
Long story short, I suggest to only save when there's actually a change detected here https://github.com/ericgrandt/TotalEconomy/blob/1fd1348091db8d14d1f7aa2f12f8cfd4ab2763e8/src/main/java/com/erigitic/config/AccountManager.java#L450 and here https://github.com/ericgrandt/TotalEconomy/blob/1fd1348091db8d14d1f7aa2f12f8cfd4ab2763e8/src/main/java/com/erigitic/config/AccountManager.java#L471 .
Here's a screenshot from that profiling:

any updates on this issue im having the same problems

Hey @Nbgreen, this plugin is being completely rewritten for the latest versions of Minecraft and Sponge, so this should no longer be an issue once that releases. As well as that, I don't plan to bring the baltop command back in this rewritten version.

Ah what is the most up to date version for 1.12.2 MC.
Also that’s sad to here.
/baltop gives players a good reason to use the economy. they like to know who has the most money and try to beat those people

@Nbgreen this is the most recent version, though it is 4 years old: https://ore.spongepowered.org/Erigitic/Total-Economy/versions/1.8.1. The baltop command may make a come back, just no plans at this point.


jupyter extension - report correct directory when downloading datums

INT-1183

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Kevin Yang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Http4sServlet init method should call GenericServlet init

GenericServlet is doing some minimal setup in its init - preserving ref to ServletConfig in private field.
If the method is not called, then it will stay uninitialized (null) and when calling log method on implemented servlet it will throw exception:
java.lang.NullPointerException: Cannot invoke "javax.servlet.ServletConfig.getServletContext()" because the return value of "javax.servlet.GenericServlet.getServletConfig()" is null

@arixmkii I agree. Do you want to open a PR for that?  If so, please target series/0.21.

@arixmkii I agree. Do you want to open a PR for that?  If so, please target series/0.21.

@rossabaker did one for the main branch. Will redo to 0.21.

@rossabaker did one for the main branch. Will redo to 0.21.

Ha, you're typing on one issue while I'm typing on the other. Yeah, if you target series/0.21, we can release it as part of 0.21.17, and then it'll merge forward into 0.22 and 1.0 series as well.

Ha, you're typing on one issue while I'm typing on the other. Yeah, if you target series/0.21, we can release it as part of 0.21.17, and then it'll merge forward into 0.22 and 1.0 series as well.

PR for 0.21 series #4309

PR for 0.21 series #4309


NIFI-11860 fixed bug in ResultSetRecordSet affecting decimalScale whe…

…n resultSetScale was zero.

Summary
NIFI-11860
Tracking
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check
[x] JDK 17

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

I agree with the change and the request for a unit test to illustrate this. I will merge once the unit test is in place.

Thanks, I will add the test tomorrow and let you know.

When creating the ResultSetRecordSet object, the  default defaultScale  was 0.
Previously if the resultSetScale was 0 then the default (0) defaultScale was applied.
Therefore I created a new test (testCreateSchemaWhenScaleIsNonDefault), where the scale is set. With the old code (decimalScale = resultSetScale > 0 ? resultSetScale : defaultScale;) the test fails, whereas with the fix (decimalScale = resultSetScale >= 0 ? resultSetScale : defaultScale;) it passes.
I hope this is sufficient.

+1 LGTM, thanks for the fix! Merging to main and support/nifi-1.x


Add "Tao verified" checkbox on edit Dojo page

For CDF admins, display a "Tao verified' checkbox on the edit dojo page.
Enabling this officially "CDF" certification will display a Tao logo on the Dojo page.
Need to figure out what the Tao logo on listing page will look like.

See https://github.com/CoderDojo/community-platform/issues/736 for Technical spec on how to add a field to a Dojo listing.
See here https://github.com/CoderDojo/cp-zen-platform/blob/master/web/public/templates/dojos/dojo-detail.dust for Dojo listing template.

Still not ready for dev?

@Aryess Ready for dev!

Ok I'll try this one

This is live!


[FIX] Issue #389: bad oca_dependency version

This fixes issue #389 , the python-chess library stopped support for python2 after 0.23.x

Right :+1: I made the same recently, but for 10th branch https://github.com/it-projects-llc/website-addons/commit/70fe589752ad41499f879a4c87d21474d41c1720


support post-release versions, publish v1.15.0.post1

Contributes to https://github.com/rapidsai/build-planning/issues/57.
libucx.load_library() defined here tries to pre-load libcuda.so and libnvidia-ml.so, to raise an informative error (instead of a cryptic one from a linker) if someone attempts to use the libraries from this wheel on a system without a GPU.
Some of the projects using these wheels, like ucxx and ucx-py, are expected to be usable on systems without a GPU. See https://github.com/rapidsai/ucx-py/pull/1041#discussion_r1594729142.
To avoid those libraries needing to try-catch these errors, this proposes the following:

removing those checks and deferring to downstream libraries to handle the non-GPU case
modifying the build logic so we can publish patched versions of these wheels like v1.15.0.post1

Notes for Reviewers
Proposing starting with 1.15.0.post1 right away, since that's the version that ucx-py will use. I'm proposing the following sequence of PRs here (assuming downstream testing goes well):

this one
another changing the version to 1.14.0.post1
another changing the version to 1.16.0.post1

Once this merges, @pentschev or @jameslamb could you test installing a ucxx wheel along with this specific version of libucx (the ucxx wheels will allow using 1.15.0 at runtime right now I believe) to ensure that things work the way we want on both CPU-only and GPU-enabled machines?

Yes I can do this.

CI is stuck waiting for arm64 runners. Once those run and (hopefully) pass, I'll merge this and test with ucxx + these new wheels.

@vyasr @pentschev the CI pipeline from this PR has been stuck waiting for a runner for 2+ hours (build link), so the new wheels aren't up on the nightly index yet (https://pypi.anaconda.org/rapidsai-wheels-nightly/simple/libucx-cu12/).
To get around that, I did a minimal test of ucxx by downloading the new wheels from S3 + running https://github.com/rapidsai/ucxx/blob/branch-0.38/python/examples/basic.py.
#!/bin/bash

set -e -u -o pipefail

# check if there's a GPU attached
nvidia-smi || true

# download ucx wheel with the fixes
rm -f ./ucx-wheels-cu12.tar.gz
wget \
    -O ucx-wheels-cu12.tar.gz \
    https://downloads.rapids.ai/ci/ucx-wheels/branch/main/a0bf56f/ucx-wheels_wheel_cpp_ucx_cu12_x86_64.tar.gz

mkdir -p /tmp/delete-me/ucx-wheels/
tar \
    -xvzf ./ucx-wheels-cu12.tar.gz \
    -C /tmp/delete-me/ucx-wheels/

# install it
pip install /tmp/delete-me/ucx-wheels/*.whl

# Install the latest `ucxx` wheel.
pip install 'ucxx-cu12==0.38.*,>=0.0.0a0'

# try importing ucxx and libucx
python -c "import ucxx._lib.libucxx as ucx_api"
python -c "import libucx; libucx.load_library()"

# try running the example
python ./python/examples/basic.py

Ran it with and without a GPU visible to the processes.
# with GPU
docker run \
    --rm \
    --gpus 1 \
    -v $(pwd):/opt/work \
    -w /opt/work \
    -it rapidsai/citestwheel:cuda12.2.2-ubuntu22.04-py3.10 \
    bash ./test.sh

# no GPU
docker run \
    --rm \
    -v $(pwd):/opt/work \
    -w /opt/work \
    -it rapidsai/citestwheel:cuda12.2.2-ubuntu22.04-py3.10 \
    bash ./test.sh

Saw it succeed (after applying the modifications from https://github.com/rapidsai/ucxx/pull/229) on both.
I think that's enough evidence to move forward with publishing the other versions (1.14.0.post1 and 1.16.0.post1). But to be sure, tomorrow I'll try with the 1.15.0.post1 wheels in CI for https://github.com/rapidsai/ucx-py/pull/1041.

these wheels are working with ucx-py builds in both GPU and non-GPU environments 🎉
https://github.com/rapidsai/ucx-py/pull/1041

Awesome!


Rewrite Cleanup

Please do the following to test before you approve.

Check out master.
Run ALGInstallation instance cleanup to reset Algernon to the master version.
Use Algernon
Run the upgrade snippet below.BaselineOfAlgernon compile: 'projectClass ^ Smalltalk at: #MetacelloCypressBaselineProject ifAbsent: 	[super projectClass]'.
[ Metacello new
baseline: 'Algernon';
repository: 'github://HPI-SWA-Teaching/Algernon-Launcher:cleanup/packages';
onConflictUseIncoming;
get;
load. ] on: MetacelloSkipDirtyPackageLoad do: [:ex | ex resume: false. ].

Test that Algernon still works and that no errors appear (except the merge one [1]) during the upgrade.

Repeat the same process, but checkout development and run
 Metacello new
       baseline: 'Algernon';
       repository: 'github://HPI-SWA-Teaching/Algernon-Launcher:cleanup/packages';
       onConflict: [:ex | ex allow];
       load

[1] Merge error that is okay during the upgrade

You are about to load new versions of the following packages that have unsaved changes in the image.  If you continue, you will lose these changes.
Algernon

@MrModder (Tutor) I have a question regarding Baselines when I'm on master and I do
 Metacello new
       baseline: 'Algernon';
       repository: 'github://HPI-SWA-Teaching/Algernon-Launcher:cleanup/packages';
       onConflict: [:ex | ex allow];
       load

The postLoad is not run and I can't even find it in the code browser even though it's in the cleanup branch. Do you have any idea why that could be?

I encountered not only the in [1] mentioned error, but had the same warning for BaselineOfAlgernon and AlgernonTests.
I do not know wheather this is caused by the software or if I did not follow the instructions correctly. @BenBals If you ask me, the instructions are not very clear (or I didnt get them ^^). My recommendations/questions for the instruction are listed below, I will gladly test the steps again.

mention to checkout master/dev through the git Brower(at least I did forget this the first time)
Please ellaborate on "use algernon", just use it anyhow and test a single word or so?
split step 5 in two and make the step to check for errors first, the test algernon. And while doing so, maybe specify "test" (UAT or running tests? What is expected to be tested? Where do you expect failures and therefore where should we put the finger in the wound?)
For the repetition I was unsure about the snippet. I just used that snippet instead of the one specified above, but I wasnt sure if i was supposed to run the "old" snippet first and run the new snippet afterwards.

As I said, I will gladly test again, but I think I am not on the same level as that I can fulfill your testing the way you want right now.

When running on master I get the Pop-Up with [1] 3 times, running on dev I get it 2 times. But that ain't a big problem imho.
In both cases Algernon runs fine and I get no additional errors.

When running on master I get the Pop-Up with [1] 3 times, running on dev I get it 2 times. But that ain't a big problem imho.
In both cases Algernon runs fine and I get no additional errors.

This is the expected behavior. The reason is that compared to master the BaselineOfAlgernon package was changed, but those changes are already in dev.

@kej-jay Thanks for your feedback! I've updated the instructions accordingly. Please test again and let me know if anything is unclear.

Tested again. For master I got Baseline.., AlgernonTest and Algernon warnings each once, but that shouldn't be a problem. When on dev I did not have any warnings.

Awesome, thanks for your review, guys. I really want to get this right, so I appreciate your input.


please make declarations independent from generator

Hi,
thank you for making you work available.
I only want to get the declarations, and for now I think I don't need the entire generator. Is it possible to package it a little bit differently so that the declarations do not have a dependency on the generator and are thus more independent?
So than I could avoid adding additional repositories to my gradle file.
I Could build it just fine without additional repositories, but idea intelijii was marking everything red, because the generator module had certain dependencies which I would not get.
Thanks

Hi @runkelstein,
When you're depending on com.github.cypressious.kotlin-webextensions-declarations:webextensions-declarations, you're only getting the declarations. They don't have any dependencies on the generator.

Hi  cypressious,
ok than maybee is a problem with my gradle file or inteliji. I can build it without the generator, but intelijii marks it still red:

Ok, now I see it, I have to change that line
compile("com.github.cypressious", "kotlin-webextensions-declarations", Versions.kotlinWebextensions);
to
compile("com.github.cypressious.kotlin-webextensions-declarations", "webextensions-declarations", Versions.kotlinWebextensions);
then it is working as expected

Perfect, thx for reporting your solution.


Display Backlinks on Note Page

Is your feature request related to a problem? Please describe.
For the dendron Jekyll theme, it would be nice to have the option of displaying the backlinks on the page as we do in Dendron itself. Being able to configure this to the users liking would be excellent.
Describe the solution you'd like
When a note links to the current note that you are viewing, some contextual information on the current not displaying the backlinks is helpful when exploring your own or someone else's knowledge graph.
Heres an example:

Describe alternatives you've considered
There are a lot of backlinks in Dendron it self or Roam Research. Just simple links displaying the pages that link to it.
Additional context
Here's an example implementation in @maximevailancourt's digital garden jekyll template

This has shipped and is available if you publish recently.


fix(email-notifications): Changes Suffix of Email Notification CSV Reports to ZIP

SUMMARY
Fixes https://github.com/apache/superset/issues/25647
In short, attachments are being sent as "CSV" when they are in fact ZIP.
TESTING INSTRUCTIONS
Notification can be set up for a chart with CSV output. The Attachment will now be a ZIP containing the Alert Query as well as the Chart Requested
I'm seeking assistance with testing this or writing automated tests to make sure that the attachments created are of the correct type matching the extension.

@eschutho shouldn't the artifact only include the associated report CSV, i.e., sans the alert query?

Friendly ping to @john-bodley and @eschutho for review if you're able - I'm reviewing issues and this seemed worth bumping for review as it is a small PR that would close an issue.

@dpgaspar do you remember if it was explicitly a requirement during initial implementation to also get the alert query result? Or is this an unintended side-effect?

@RussellSpitzer I noticed that the output is zip if there's more than one CSV. I'm trying to wrap my head around some of these tests (or lack thereof), so bear with me a while longer..

Closed a related issue to this one, but yes, it sounds like a single CSV file should get the .csv extension, and multiple CSV files come through as Zip, and thus should get the .zip extension. I don't think this PR quite handles that logic.

I've just run into this issue as well, could I propose something like:
import zipfile
import io

"..."

#ln 170
if self._content.csv:
    csv_extention = "csv"
    try:
        with zipfile.ZipFile(io.BytesIO(file_content), 'r') as zip_file:
            csv_extention = "zip"
    except zipfile.BadZipFile:
        pass
    csv_data = {__("%(name)s.%(ext)", name=self._content.name, ext=csv_extention): self._content.csv}

Noting that this PR needs a rebase, as well. Hopefully @RussellSpitzer is still interested in getting this across the finish line 🤞

Moving this PR to draft mode, whilst it needs a rebase/update. This (and the related issue) might be closed in time if not given any further updates.


'All References' to 'app' smoketest failure

Repo

Run smoketest on latest insiders

Bug
  1) Running Code
       Editor
         finds 'All References' to 'app':
     Error: Timeout: get elements '.monaco-editor .zone-widget .zone-widget-container.peekview-widget.reference-zone-widget.results-loaded .body .ref-tree.inline .monaco-list-row .reference' after 20 seconds.
      at /Users/matb/projects/vscode/test/smoke/out/vscode/code.js:158:23
      at Generator.next (<anonymous>)
      at fulfilled (out/vscode/code.js:8:58)

The UI seems to be showing the correct thing, so this just looks like a test failure to me


Old session data is read by dynamic menu if set before during the conversation

When setting ctx.session.something inside a conversation before sending a dynamic menu, the dynamic menu will see an old value of the session.
Try this CodeSandbox:

Button is "-", when it should be "Hi".

Just updated the sandbox with that workaround, same result.

Can you check if #15 fixes the problem?

Yes!!! Bye grammyjs/router :P


pathoc -n seems broken

Steps to reproduce the problem:

pathoc -n 2 google.com get:/
07-06-16 12:22:24: >> 'GET':/
<< 302 Found: 261 bytes

What is the expected behavior?
Two requests printed.
What went wrong?
Only one request is printed.

Mitmproxy Version: -master
Operating System: osx

This is caused by this line, where 0da3e51e1c08eed2c8054d4a71bc591268b19af1 made args.requests[0] a generator that is exhausted after the first pass. While looking into this, I realized that pathoc get:/:x10 is similar to pathoc -n 10 get:/, with the major difference being that the former does everything on the same connection. We probably want to document that somehow.


Potentially add infinite scrolling/auto loading to the activity feed page

Closing because I don't think the pagination method should change from page to page.


How to reach dbt support: unify instructions

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
When triaging bug reports in dbt-core and adapters like dbt-bigquery, etc, I sometimes need to redirect the bug report to the team that supports Cloud. Here's the relevant content:

If you are a dbt Cloud user and need help with one of the following issues, please reach out to us by using the speech bubble (💬) in the dbt Cloud interface or at<EMAIL_ADDRESS>

We have two primary pages that come up at the top of Google search results for "dbt cloud support":

https://docs.getdbt.com/docs/dbt-support
https://docs.getdbt.com/guides/legacy/getting-help#dbt-cloud-support

What part(s) of the page would you like to see updated?
The awkward thing is that nearly every time, I'm wanting to refer to the 2nd one, which contains "legacy" in the URL path. This wording gives the connotation that the content is outdated or superseded.
I perceive multiple issues:

This page doesn't directly answer my question, but this one does
The non-legacy page actually links to the legacy one :spiderman_pointing:
Whenever I share a link that contains the word "legacy" in the path, it gives me an uneasy feeling

Potential solution:

Could we create a single common snippet and then include it on both pages?

Open to other ideas too!
Additional information
No response

Could we create a single common snippet and then include it on both pages

I like this solution a lot!

hey @dbeatty10 , we've made changes to the dbt support page - does it solve this issue? i do think we can link to the dbt support page from the https://docs.getdbt.com/community/resources/getting-help#dbt-cloud-support page maybe?

Closing issue as the page no longer includes or lives under legacy and points to the main support page.


Added a new collaborator to About page

Before you start

[x] Have you checked if a similar PR has already been requested?
[ ] Have you built and ran the app?

Related/Fixed issues
Address related/fixed issues

None
Description
Details of changes are listed here
Added @DeveloperWOW64 as contributor and updated @onein528's name
Motivation and Context
Items resolved/related issues by this PR.
None
Assets (if appropriate):
A list of assets (screenshots, mockups) relevant to this pull request.
None

LGTM. you customized your avator, but it will be replaced by GitHub Collaborators API.

ok

LGTM. you customized your avator, but it will be replaced by GitHub Collaborators API.

What do you mean by this?
are you planning to put in the GH Collab API?

Yes.

In addition to introducing the GitHub API to the account page of the settings, we will also introduce it to the About page.


Rename imagery to raster overlay

Resolves #612
This is fundamentally simple. By and large, any "imagery" or related string has been changed to "raster_overlay" or a casing variant.
There are still 25 instances. These are exclusively in .md documentation and a few user-facing strings in Python code. Sometimes "raster overlay" sounded overly technical for end users. This is debatable.
Previously, we had Imagery and ImageryLayer in variable and class names. I kept that. It leads to long variable names. The worst is overlayRasterOverlayLayerCount, but after first trying it without "Layer" I think this is best.

Previously, we had Imagery and ImageryLayer in variable and class names. I kept that. It leads to long variable names. The worst is overlayRasterOverlayLayerCount, but after first trying it without "Layer" I think this is best.

I slightly prefer without "Layer", but since it doesn't impact the USD schemas or UI I don't see a reason to hold up the PR.

Previously, we had Imagery and ImageryLayer in variable and class names. I kept that. It leads to long variable names. The worst is overlayRasterOverlayLayerCount, but after first trying it without "Layer" I think this is best.

I slightly prefer without "Layer", but since it doesn't impact the USD schemas or UI I don't see a reason to hold up the PR.

I found it helpful to have different variables names for raster overlays at the USD level and those in the material. Otherwise you get rasterOverlayPath, which can ambiguously refer to the prim path or MDL node path. All the "layer" symbols deal with the objects instantiated for the material. But "layer" doesn't imply that. Maybe we could swap the "layer" string with something material-specific. Maybe "Node."

I don't think the ambiguity is the big deal. Maybe just make the change in a new PR and see how it feels?
Also I realized that some of these changes are user facing like cesium_raster_overlay_layer_float4 so the simpler terminology is definitely preferable there.


Update awssdk:auth, awssdk:core, ... to 2.17.154

Updates

software.amazon.awssdk:auth
software.amazon.awssdk:core
software.amazon.awssdk:regions

from 2.17.141 to 2.17.154.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #59.


Updated input / output array to support std::vectorcv::cuda::GpuMat

Is it possible that it's a fluke that this test failed?  Before 1e1f20c this test passed.

Yes, it doesn't look like this change affect the failing test.

@jet47, can this be merged in now?

:+1:


Not offering netlify link

Describe the bug
I've asked it to deploy multiple times without an offer for the link to get the netlify website added to my netlify account.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-pfpqff
Steps to reproduce
Ask it to deploy
Expected behavior
Quicklink is given with a link to obtain ownership that takes me to netlify
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
performance.memory = {
  "totalJSHeapSize": 275680191,
  "usedJSHeapSize": 238940843,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = BrummettDrew
Chat ID = f5b64ace5ff5

Additional context
No response

Hi, are you clicking the Deploy button at the top-right of the page. Are you receiving any specific error message when you click Deploy?

Hey! Sorry for the vague request. No specific error, no! And since that
instance it's working again. It was just excluding the usual claim link and
only giving the view website option in the deploy report (where it shows
status of deployment, vite and such) I have it working now.
I'll try to provide more info as things come up. Amazing tool and greatly
appreciate it!
On Mon, Nov 25, 2024, 5:53 PM Brian @.***> wrote:

Hi, are you clicking the Deploy button at the top-right of the page. Are
you receiving any specific error message when you click Deploy?
—
Reply to this email directly, view it on GitHub
https://github.com/stackblitz/bolt.new/issues/3157#issuecomment-2499197813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMCT3LN5EJUQSXDO3MBDMTD2COS5XAVCNFSM6AAAAABSO3QH5OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJZGE4TOOBRGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Python "fully supported" vs "security fixes only" stages

Link to product page on endoflife.date
https://endoflife.date/python
Details of incorrect and correct details you have found
Python differentiates between "fully supported" and "security fixes only" stages. The latter one has no pre-compiled binaries. For example, the latest Python 3.9 is Python 3.9.17, although only Python 3.9.13 can be downloaded as a prebuilt (installable) package.
What is the source website for the product and for its version information?
For example for Python 3.9: https://www.python.org/downloads/release/python-3917/ (See "No installers" paragraph)
Additional context
I can send a PR for this if you agree with the concept.

We are a community Wiki, and contributions for such corrections are always welcome.
Personally, I’m surprised at this stance to actively distribute insecure software. We definitely should document this.

Personally, I’m surprised at this stance to actively distribute insecure software.

Are you suggesting deleting the old, insecure downloaders from the python.org server?
Security fixes are source only, but that's after 18 months of full support with installers.
Looking at 3.9, the final bugfix with installers was 2022-05-17. By this time, 3.10 had been out for seven months (since 2021-10-04), was the fully supported release with installers, and was already up to 3.10.4 (2022-03-24). People should really have been upgrading to 3.10, not a 3.9 micro.
And especially today, you shouldn't be installing 3.9, because 3.11 has been out for nine months, since 2022-10-24, with 3.11.5 set for release any moment now.
I think the idea for source-only releases of security-only fixes is to reduce the maintenance cost of building installers, while at the same time letting those people who can't or won't upgrade (for whatever reason) be able to patch their systems. And third parties, such as Linux distros and pyenv for Mac, do create installers of security-only releases.

I agree with you @hugovk, but I also think this should be clearly documented on https://endoflife.date/python. Knowing this, and whether an "official" installer is available for the latest version, are valuable information for endoflife.date users.

I'll create a PR about this.
Sometimes we (Python developers) have to support fully EOL-ed versions,
too. There are significant codebases in Python 2.7 as well. It is not an
ideal world, and it is frustrating, but this is the reality. It is good to
have security patched sources for such a long time (5 years from first
release).

Thanks @presidento !

Thanks!
By the way, from 3.13 (due October 2024), it's switching from 1.5 years bugfix + 3.5 years security, to 2 years bugfix + 3 years security.


Incorrect rotation when setting agent to a pre-computed translation & rotation in Matterport3d

Habitat-Sim version
v0.2.1
Following a previous thread, I'm trying to set the agent to follow a pre-computed sequence of translations & rotations and save RGB observations (essentially I only care about the camera but not the agent, and the camera will be placed at these translations & rotations). However, while the translations look correct, the rotations seem to be off that the agent is looking towards the floor, while it's supposed to look at the wall.
My code snippet:
sim_settings = make_default_settings()
sim_settings["sensor_pitch"] = 0
sim_settings["sensor_height"] = 1.

make_simulator_from_settings(sim_settings)

visual_sensor = sim._sensors["color_sensor_1st_person"]

# set sensor initial position
visual_sensor._spec.position = np.array([0, 0, 0])
visual_sensor._spec.orientation = np.array([0, 0, 0])
visual_sensor._sensor_object.set_transformation_from_spec()

# load motion sequence
motion_seq = np.load(args.npz_path)
trans = motion_seq["joints"][:, 0, :]
root_ori = motion_seq["root_orient"].reshape(-1, 3, 3)

# correct rotation to -y as gravity axis
quat_mp3d_to_habitat = quat_from_two_vectors(np.array([0, 0, -1]), geo.GRAVITY)
rot_mat_mp3d_to_habitat = quaternion.as_rotation_matrix(quat_mp3d_to_habitat)

for t in range(trans.shape[0]):
    camera_pos_t = trans[t]
    root_ori_t = root_ori[t]

    root_ori_t_x = -root_ori_t[:, 0]
    root_ori_t_y = root_ori_t[:, 2]
    root_ori_t_z = root_ori_t[:, 1]
    camera_rot_t = np.stack((root_ori_t_x, root_ori_t_y, root_ori_t_z), axis=1)

    camera_pos_habitat = rot_mat_mp3d_to_habitat @ camera_pos_t
    camera_rot_habitat = rot_mat_mp3d_to_habitat @ camera_rot_t

    sim.get_agent(0).scene_node.translation = camera_pos_habitat
    sim.get_agent(0).scene_node.rotation = mn.Quaternion.from_matrix(camera_rot_habitat)
    observation = habitat_sim.utils.viz_utils.observation_to_image(sim.get_sensor_observations()['color_sensor_1st_person'], \
        				            observation_type='color')

The extra axes swapping I did to produce camera_rot_t was related to how I got these orientations. I have verified that they are correct. This image shows the visualization of the sequence of camera_rot_t and camera_pos_t in the original matterport scene. As can be seen the rotation points towards the wall.

The actual observations I'm getting from this snippet are like below, which seem to be right in terms of translation but wrong in rotation.

Any help would be much appreciated.

Maybe @aclegg3 @erikwijmans? Any insights would be much appreciated.

What does camera_rot_t represent? Is it it transform points from camera space into world space or from world space into camera space? The rotation of the agent in habitat is the former -- camera space into world space -- so you may need to invert that matrix.
If you don't know, I'd try inverting and seeing what happens as everything else looks correct (assuming the axis swapping is correct).

Thanks for your reply @erikwijmans camera_rot_t is actually world2cam. I inverted it but it's still off - now the rotation in x-y plane is inverted but the camera still points towards the floor.
The axis swapping should be correct. The first figure basically visualizes using camera_rot_t as the camera poses.

What directions do the axes of the camera represent? In habitat the camera looks along -Z, +Y is up, and +X is right.

Yeah as you can see this rot_mat_mp3d_to_habitat accounts for this change and has been applied to both translation and rotation (works fine for translation). In the original mp3d frame the front vector is +Y, so theoretically after this correction it should be -Z.

And sorry camera_rot_t  is actually cam2world (the camera pose), as can be seen from the first picture. So it shouldn't need an invertion.

I don't see anything wrong with the code so either I am missing some bug in the code or the axes in the rotation matrix don't represent what you think.
It's not uncommon for the rotation component to have a different coordinate convention from translation component (this is sometimes done to make interop with certain renders easier). I'd try just adding axis swap rotations to make things look correct for one image and see if that generalizes to other cases.

Yeah I could just heuristically swap the axes to get reasonably looking observations. Was just worried if it's not following the correct logic and might cause problems later. I guess I'll go with this for now. Thanks for your help anyways.


VCF specifications: GQ of type Integer

Hi,
The VCF specification says that FORMAT/GQ should be an integer, and not a float as outputted by LongShot. While bcftools doesn't complain about it, pysam does throw an error when parsing the file...
[E::vcf_parse_format] Invalid character '.' in 'GQ' FORMAT field at chr10:38970
Traceback (most recent call last):
(...)
  File "pysam/libcbcf.pyx", line 4108, in pysam.libcbcf.VariantFile.__next__
OSError: unable to parse next record

I can round these now to work around this issue, so no real trouble. I just wanted to inform you.
Cheers,
Wouter

Thanks for reporting this. It should be fixed soon.

Took a long time, but finally fixed (master branch).

Thanks!

Hi,
I have a some more things related to this that should be easy to fix:

The VCF header now specifies the GQ as Integer but the values are still floating point numbers. As a consequence, bcftools complains about the "." in the GQ fields.
bcftools complains about the DP field not being specified in the header. Adding the following line fixes it:

##FORMAT=<ID=DP,Number=1,Type=Integer,Description="Read depth">

Thanks,
Birte

These issues have been fixed in a new release (0.4.2). Thanks.


Render Material using a PhysicalModel with optional compositing

@abarth

From a performance perspective, this new patch is fantastic. It did nothing but improve the benchmarks!


Meta v1.2 Scope out needs to submit package to cran

Goal
Installing from CRAN is easier for a lot of users. We should scope out what we would need to do to get this package on cran.
Tasks

[ ] separate github actions for R CMD check with and without --as-cran
[ ] #188
[ ] write skip_on_cran() functions for any tests that use cmdstan, similar to in primarycensoreddist() https://github.com/epinowcast/primarycensoreddist/blob/d293504f4107af6f68654842b551b5b26856ce2d/tests/testthat/test-stan-primary_censored_ode.R#L1
[ ] add examples to every function, or if an internal function than remove export and tag with @noRd
[ ] make title capital letters

I am not convinced this should be a particularly high priority, particularly since we currently have a non-CRAN package (cmdstanr) as an essential dependency (see my comments in #188).


Error "NoneType" object is not subscriptable!

Always i try to do img to 3d i got this error!! any know how to solve it?

Could you please try again? We fixed an issue maybe related to this.

I test it and i still got the exactly same problem!?

I am not sure what happened to you. Do you have this issue all the time?

I am not sure what happened to you. Do you have this issue all the time?

yes when i do a generate, i got always that error! I´m not able to use the node!

I am not sure what happened to you. Do you have this issue all the time?

yes when i do a generate, i got always that error! I´m not able to use the node!

I  cannot reproduce your case. Could you please the latest version and attach your log? There is an error in _receive_one function and I need your help.


feat: add builder mode when editing a modelfile

I know that the modelfile support is still in development, so not sure if this is a bug or a feature request. When you go into the modelfiles section and create a new modelfile, you have the option to toggle between builder mode and raw format. If you've previously created a modelfile and go back in to edit it, you don't see the option for builder mode, raw format is selected by default.
It would be nice to have the edit functionality behave like the create functionality. Not a deal breaker by any stretch, just a small quality of life improvement.

Hi, It indeed is WIP at the moment, will add the builder mode to edit page as well in the near future. Stay tuned, Thanks!

I'm bumping this issue as it hasn't had any activity since Dec 2023. This issue is still valid for the most part, as the Modelfile Format option is unavailable/missing from the UI when going back to edit a previously created Modelfile. The Builder Mode is selected by default now, unlike what was mentioned before:

"you don't see the option for builder mode, raw format is selected by default."

This also prevents modifications to advanced model parameters (directly from the Modelfile itself) with an already created Modelfile too. That is because Modelfile Advanced Settings is only available in Raw Format mode. This isn't the main issue I'd like to point out, as yes, advanced model parameters are indeed "advanced" settings. With this being said, I am pointing out why it could still be seen as an issue that the option for switching models in an already made Modelfile is missing.

PR for #665 should include this feature as well.

Added to our dev branch!


Make sure auto-cleanup doesn't purge chained backups that are being relied on

Auto-purge currently deletes the oldest backup.
This breaks differential and incremental chains.
Differentials are easy - if the oldest full backup is being relied on, delete the oldest non-full.

With incrementals, the entire chain is linked - restoring a partial backup means also restoring every other partial backup since the last full backup, whereas differentials only require the last full backup.
As such, we have two options :

Delete an entire chain.
Don't delete anything, and ignore the set limit.

We could have a user config for this...?

I think the best idea is to introduce a user configuration option.
This option allows users to choose how auto-cleanup behaves. They can either delete the entire backup chain, including linked incremental backups, or they can ignore the set limit and manage backup storage manually.
This user configuration option provides flexibility and ensures that users can preserve their backup chains or take full control over backup management. At cost of expensive documentation.

I think it should take into account the max number of chains and how many chains there currently are. I don't ever see a case where a user would be happy with an entire chain being deleted if it's the ONLY chain, basically nuking all backups.
So, if there's more than one incremental chain, delete the oldest one if necessary
If there's only one incremental chain, do nothing, and report it, with a recommendation to reduce chain length and/or increase max space

So :

With differentials, delete the oldest partial backup if the oldest full backup is being relied on. Do not adjust the chain length, so as to allow new full backups to be made again in due time.

 (What's a good way?)

Okay, this is now done - though, we do not have a good way to alert users.
@MisterMyst @superhealing I don't suppose you've got comments on how we alert the user? I feel just writing to the log isn't very helpful..

Maybe you could write it both to log, and also store the message to be displayed to an opped user at their logon?
Also, not sure if this has been offered yet, but if you have a discord bot/webhook to give backup results inside of a discord channel, you could use that as another channel to relay this information to an admin. (or ofc other connectors, to maybe Matrix, IRC, or similar.)