What does this extension do?

Subject says it all.

@mark-hahn Thanks for your interest.
To be honest, I thought only I would use it, so I didn't write it down properly, but since I got an issue, I wrote an explanation.
If you still can't find how to use this extension, please let me know.

It looks like something I could use.  I just have to figure out how to use
it.
On Tue, Feb 15, 2022 at 6:26 AM Fumiaki MATSUSHIMA @.***>
wrote:

@mark-hahn https://github.com/mark-hahn Thanks for your interest.
To be honest, I thought only I would use it, so I didn't write it down
properly, but since I got an issue, I wrote an explanation.
If you still can't find how to use this extension, please let me know.
—
Reply to this email directly, view it on GitHub
https://github.com/mtsmfm/vscode-jscad-preview/issues/1#issuecomment-1040336141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGGDP2DWDU4KHGZEGIF6XLU3JPAHANCNFSM5OMM46VQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Can't load AudioTranscriber()

Gives me error : "argument of type 'NoneType' is not iterable"
When i do this :
loader = AudioTranscriber()
My versions are :
llama-hub==0.0.16
llama-index==0.7.17

You can try to update your llamahub and llama-index versions,
it's not reproducible more.


Add parameter 'stripDefaultValueProps'

This makes it possible to render the minimal amount of JSX necessary to
instantiate a component. All props that have default values will be
exluded from the rendered JSX.

Closing, with the intent to make a cleaner refactoring of this.


Do not automatically remove unused dependencies in the versions.properties file when update

I don't want to delete unused dependency properties every time I update with ./gradlew refreshVersions. Is there an API that can help me disable this feature?

@oh-Rin There's no such API, but you can add a dummy configuration where you have the dependencies that would otherwise be removed as a workaround.

@oh-Rin There's no such API, but you can add a dummy configuration where you have the dependencies that would otherwise be removed as a workaround.

This may be a bit tricky, because I share a versions.properties for multiple different projects, so I don’t want to delete dependencies used by other projects when one of the projects is updated

It's less tricky if you have a "master" project that has all the dependencies in a dummy configuration, that is the only project where you run refreshVersions.

It is true, but I prefer refreshVersions to be used in all projects, which will undoubtedly be more convenient...Is this possible?

It's less tricky if you have a "master" project that has all the dependencies in a dummy configuration, that is the only project where you run refreshVersions.

The solution I currently think of is to record versions.properties and convert to java.util.Properties before updating
Then add the missing dependencies back after the update, but I encountered a problem, java.util.Properties can’t know "#available=***" of the dependency
In the end it looks like this:

It is true, but I prefer refreshVersions to be used in all projects, which will undoubtedly be more convenient...Is this possible?

Are you using Gradle's includedBuilds (aka. composite builds)?

It is true, but I prefer refreshVersions to be used in all projects, which will undoubtedly be more convenient...Is this possible?

Are you using Gradle's includedBuilds (aka. composite builds)?

No, for some special reasons, I can’t use composite builds, because some projects I work with others, but we all want to use the same dependencies... well the situation is a bit complicated

Ok, I’m very lucky, I solved this problem in a simple way, now i can work.

My situation should be special, this question can be closed now

Would be best to post an actual snippet rather than a screenshot for other to copy paste the relevant parts if they have the same use case since you might not be the only one ever having such a use case.

Thanks for your suggestion, but I am not sure if my solution is correct or other problems, this is my code snippet for build.gradle.kts:
object VersionsProperties {
  val file = File(rootDir, "versions.properties") // or in other custom locations
  var last = current
  val current get() = file.readLines()
}

tasks.findByPath(":refreshVersions")?.apply {
  doFirst {
    VersionsProperties.last = VersionsProperties.current
  }
  doLast {
    val old = VersionsProperties.last
    val new = VersionsProperties.current.toMutableList()
    var locked = true
    // 通过对比找出新的 versions.properties 文件中被删除的依赖
    old.forEach { line ->
      // 如果到达了下一个依赖组
      if (!line.startsWith("#") && line.contains("=")) {
        // 代表后面的行不能够记录了
        locked = true
      }
      // 如果依赖组丢失则记录，并且开锁
      if (!new.contains(line)) {
        new.add("\n" + line)
        // 代表依赖组后面的 "available" 数据可以被记录
        locked = false
      }
      // 允许记录当前依赖组的 "available" 数据
      if (!locked && line.contains("# available=")) new.add(line)
    }
    // 更新依赖文件
    VersionsProperties.file.writeText(new.joinToString("\n"))
    VersionsProperties.last = new
  }
}

In version 0.10.0, unused version keys are not removed. We plan to add an informative metadata comment when unused in a future release.


SQLA - form_columns does not allow '.'

See comment here: https://github.com/flask-admin/flask-admin/pull/1174#issuecomment-172404695
form_columns doesn't allow specify a column in a related model in the same way as column_list, column_filters, column_sortable_list ('<relation>.<column name in related model>')
It gives the following error message: ValueError: Invalid model property name <class '__main__.Tyre'>.car.desc
The fix might request using rec_getattr in tools.py.

No, can't use rec_getattr, as getting SQLAlchemy relation path is much more complicated than that.
On other hand, there's helper that you can use: https://github.com/flask-admin/flask-admin/blob/master/flask_admin/contrib/sqla/view.py#L350
Hopefully (or eventually) this helper will be able to find join path between two properties, but for now it can explicit trace path through relations.

Made some progress here: https://github.com/flask-admin/flask-admin/compare/master...pawl:fix_1174?expand=1
Mostly moved get_field_with_path into sqla/tools.py and used it in get_form. It's showing the field now, but it isn't getting populated.

I thought some more about this and realized it's more complicated when there isn't already a related record to edit (or if there are multiple related records). Hence, the reason for inline_models.
Maybe there a should just be an exception that tells the developer to use inline_models.


[Bug-11650][worker] #11650 fix SQL type task, stop task does not take effect

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #11668 (3b177aa) into dev (23d8aaf) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #11668      +/-   ##
============================================
+ Coverage     39.80%   39.81%   +0.01%
- Complexity     4716     4717       +1
============================================
  Files          1002     1002
  Lines         37872    37853      -19
  Branches       4230     4222       -8
============================================
- Hits          15074    15072       -2
+ Misses        21195    21180      -15
+ Partials       1603     1601       -2

Impacted Files
Coverage Δ

...ler/server/worker/processor/TaskKillProcessor.java
0.00% <0.00%> (ø)

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
27.90% <0.00%> (ø)

...cheduler/server/master/utils/DependentExecute.java
0.00% <0.00%> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

...phinscheduler/plugin/task/chunjun/ChunJunTask.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/api/ShellCommandExecutor.java
0.00% <0.00%> (ø)

...nscheduler/service/process/ProcessServiceImpl.java
31.80% <0.00%> (+0.02%)
:arrow_up:

...server/master/runner/task/SwitchTaskProcessor.java
2.54% <0.00%> (+0.02%)
:arrow_up:

...ver/master/runner/task/ConditionTaskProcessor.java
4.81% <0.00%> (+0.05%)
:arrow_up:

... and 5 more

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

Hi, @fengjian1129 @ruanwenjun

Hi, @fengjian1129 Could you please show the worker log  (especially the log related to TaskKillProcessor) when you successfully stop the sql task as below:

And for reference, i think it cannot stop the sql task correctly is because the null pointer exception in #11175
Here is the log:
[INFO] 2022-08-29 07:15:09.412 +0000 org.apache.dolphinscheduler.server.worker.processor.TaskKillProcessor:[90] - [WorkflowInstance-0][TaskInstance-0] - task kill command : TaskKillRequestCommand(taskInstanceId=27)
[INFO] 2022-08-29 07:15:09.413 +0000 org.apache.dolphinscheduler.server.worker.processor.TaskKillProcessor:[177] - [WorkflowInstance-0][TaskInstance-0] - kill task by cancelApplication, task id:27
[ERROR] 2022-08-29 07:15:09.418 +0000 org.apache.dolphinscheduler.remote.handler.NettyServerHandler:[131] - [WorkflowInstance-0][TaskInstance-0] - process msg Command [type=TASK_KILL_REQUEST, opaque=80, bodyLen=21] error
java.lang.NullPointerException: null
        at org.apache.dolphinscheduler.server.worker.processor.TaskKillProcessor.sendTaskKillResponseCommand(TaskKillProcessor.java:127)
        at org.apache.dolphinscheduler.server.worker.processor.TaskKillProcessor.process(TaskKillProcessor.java:106)
        at org.apache.dolphinscheduler.remote.handler.NettyServerHandler.lambda$processReceived$0(NettyServerHandler.java:129)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

@rickchengx @ruanwenjun I also submitted a PR # 9680, which needs to be the premise to fix this bug, because the application ID of yarn needs to be parsed in the program execution log and get appids,Please see the impact method in the following figure. I will send the detailed operation process here, but it will take some time. Give me one day.

@rickchengx @ruanwenjun
1.At present, an error "null point" is reported when the dev branch stops the task

Because appIds in taskExecutionContext class is null, it is SQL type , processId =0 as shown below:

2.After I fixed the bug, there was no such problem,Because I need to stop the process or yarn, get appid and ProcessId, and then judge whether there are appids and ProcessId = 0, as shown below:

@EricGao888 @ruanwenjun @rickchengx Please help me check whether it is clear that I have passed the local test of these codes

@ruanwenjun please help review it

AFAIK, you only need to change the line you marked to avoid NPE, you don't need to do any other change.


build(Learn) Chapter 2: Varaibles, class, methods

Learn Chapter 2

[x] Variable
[x] Method
[x] Class
[x] Instance Variable
[x] irb
[x] Class and Method

God Book

[x] string
[x] regular expression
[x] Match
[x] Constrains


Grammatically confused

One Settings/System the Formats check box says Ignore Asserts and tooltips. Is the word supposed to be Ignore Asserts or Assets?

It is "Ignore Asserts". It is an option to allow users to bypass intentionally 'corrupted' files.
Asserts are used in SoulsFormats when reading data to ensure it is valid, however some people corrupt files by placing junk data within sections that have asserts, causing SoulsFormats to fail to read the data. SoulsFormats is stricter than the game(s), so the corrupted file still works, but cannot be read.


Getting an error from vanity when trying to update redis

We just updated our redis instance.
Now we're getting an error any place we have a call to vanity.
For example:
      mixpanel.track('Acquisition Event', {
        'Trigger Type': '8 seconds',
        'Show New Version': <%= ab_test(:sg_20170624_homepage_redesign) %>,
        'Page': 'Homepage'
      })
    }, 8000); // 8 seconds```

The error points to the ab_test line and reads as:

```>> ab_test(:sg_20170624_homepage_redesign)
!! #<Redis::CommandError: ERR unknown command '[]'>```

Hey! f06d90b5bddb2c91df44a79d6c3070c57080e7da on master should fix the issue. Haven't done a release yet, but if you could use master for a bit, that's probably the quickest fix.

@toddmetheny did master fix the issue?

@phillbaker Ah---I missed that comment. We switched to use split instead. Wish I had seen it.


Proxy support for LDAPS authentication plugin use

Hey,
I'm using pGina with LDAP plugin over SSL.
How can I provide proxy URL\IP (with proxy username and password) instead of the LDAPS url?
Thanks,
Moses

Proxy is not currently supported - I presume you mean a SOCKS style proxy?


testdata: normalize line ending for go files

According to .gitattributes, all go files in this repository should
use LF as the line ending.
I have checked the commit history, it turns out that there is not
intention to use CRLF for these files.
Updates #53664.

Abandoned due to https://github.com/golang/go/issues/53664#issuecomment-1173167979.


doesn't work on macos15

Workflow version
2024.1
Alfred version
5.5[2257]
macOS version
15.0
Debugger output
[3:02:46.757 pm] Logging Started...
[3:02:47.583 pm] Banner Be Gone[Hotkey] Processing complete
[3:02:47.594 pm] Banner Be Gone[Hotkey] Passing output '' to Run Script
[3:02:47.966 pm] ERROR: Banner Be Gone[Run Script] /Users/vpchris/Library/Caches/com.runningwithcrayons.Alfred/Workflow Scripts/B1DE93A5-E593-4AF1-8A70-A5012311535B: execution error: Error: Error: Invalid index. (-1719)

More details
No response

Thank you. Version 2024.2, already live in the Gallery, updates this for Sequoia.

Just noticed that v2024.2 doesnot work on macOS 14.7, it says requires macOS 15+. Glad I can roll back to v2024.1, gonna stay at macOS 14.7 for a while considering macOS 15 is not ready for me.

Correct. That is covered in the forum update post.


fix: proof configurable on proofRecord

Signed-off-by: Berend Sliedrecht<EMAIL_ADDRESS>

Codecov Report

Merging #397 (e0a1255) into main (69684bc) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #397   +/-   ##
=======================================
  Coverage   86.53%   86.54%
=======================================
  Files         236      236
  Lines        4894     4896    +2
  Branches      773      775    +2
=======================================
+ Hits         4235     4237    +2
  Misses        659      659

Impacted Files
Coverage Δ

packages/core/src/modules/proofs/ProofsModule.ts
89.53% <ø> (ø)

...s/core/src/modules/proofs/services/ProofService.ts
87.22% <100.00%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Relative position between left and right hands

Thanks for you great work!
The rendering result is very good, but why is the relative position between the hands so poor?

Our HaMeR network estimates the hand pose from the local bounding box of the hand. The translation is inferred as a function of the provided focal length. The demo uses a large focal length that approximates weak perspective projection. If you use a more reasonable focal length value (e.g., f=1000), you will get something that looks like this:

Please check also the explanation in this issue.


Milestone 4 improvement

Hi Team,
For the 3rd part of milestone 4, I addressed the improvement suggestions by @jyflowang and @jim-franklin to separate raw and processed data files into different folders. I pushed my changes to the quarto branch but I can't seem to create a pull request from it. I'd appreciate it if any of you could help merging the changes to the main branch.

I would like to hold off on merging the branches just yet, all these files and folders are created using the code itself, so I want to make those changes to the code so that this could take place automatically. Running make clean would remove all the manually created files, so I will push new changes to the code where this can be automated

Sounds good.


cannot run npm install

I followed the manual installation but cannot execute yarn install or npm install:
pi@RPI3B:~ $ cd MirrorOS
pi@RPI3B:/MirrorOS $ npm install
bash: npm: command not found
I have the latest Raspbian Stretch on Raspberry Pi 3

Hi, you may need to install nodejs to run npm. Check out nodejs.org
On 20 Sep 2017 09:54, "goprojojo"<EMAIL_ADDRESS>wrote:

I followed the manual installation but cannot execute yarn install or npm
install:
pi@RPI3B:~ $ cd MirrorOS
pi@RPI3B:/MirrorOS $ npm install
bash: npm: command not found
I have the latest Raspbian Stretch on Raspberry Pi 3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/s6joui/MirrorOS/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACFsZLnRP6STKj-eqqj7JDh-5LT3BTojks5skMTJgaJpZM4PdekB
.

Thanx for your quick reply. I've installed node.js ( sudo apt-get install nodejs npm ) but still having issues running npm:
pi@RPI3B:~ $ sudo npm install
npm ERR! install Couldn't read dependencies
npm ERR! package.json ENOENT: no such file or directory, open '/home/pi/package.json'
npm ERR! package.json This is most likely not a problem with npm itself.
npm ERR! package.json npm can't find a package.json file in your current directory.
npm ERR! System Linux 4.9.41-v7+
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! cwd /home/pi
npm ERR! node -v v4.8.2
npm ERR! npm -v 1.4.21
npm ERR! path /home/pi/package.json
npm ERR! syscall open
npm ERR! code ENOPACKAGEJSON
npm ERR! errno -2
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/pi/npm-debug.log
npm ERR! not ok code 0
pi@RPI3B:~ $

First you should cd to the directory where you cloned the MirrorOS
repository.
2017-09-20 14:36 GMT+02:00 goprojojo<EMAIL_ADDRESS>
Thanx for your quick reply. I've installed node.js ( sudo apt-get install
nodejs npm ) but still having issues running npm:
pi@RPI3B:~ $ sudo npm install
npm ERR! install Couldn't read dependencies
npm ERR! package.json ENOENT: no such file or directory, open
'/home/pi/package.json'
npm ERR! package.json This is most likely not a problem with npm itself.
npm ERR! package.json npm can't find a package.json file in your current
directory.
npm ERR! System Linux 4.9.41-v7+
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! cwd /home/pi
npm ERR! node -v v4.8.2
npm ERR! npm -v 1.4.21
npm ERR! path /home/pi/package.json
npm ERR! syscall open
npm ERR! code ENOPACKAGEJSON
npm ERR! errno -2
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR! /home/pi/npm-debug.log
npm ERR! not ok code 0
pi@RPI3B:~ $
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/s6joui/MirrorOS/issues/3#issuecomment-330837847, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACFsZFvBLry__ygx6h5LlhFBmm_EY2sFks5skQa_gaJpZM4PdekB
.

I did as you told but still no avail:
pi@RPI3B:~ $ cd MirrorOS
pi@RPI3B:~/MirrorOS $ npm start

<EMAIL_ADDRESS>start /home/pi/MirrorOS
./system/starter.sh

npm ERR! Linux 4.9.41-v7+
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "start"
npm ERR! node v6.11.3
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: ./system/starter.sh
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script './system/starter.sh'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the MirrorOS package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ./system/starter.sh
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs MirrorOS
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls MirrorOS

Hi, I am also facing an issue similar to this:
pi@raspberrypi:~/MirrorOS $ npm install
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No license field.
npm ERR! 404 Not Found
npm ERR! 404
npm ERR! 404 'types/node' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it
npm ERR! 404 It was specified as a dependency of 'electron'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, or http url, or git url.
npm ERR! System Linux 4.9.80-v7+
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! cwd /home/pi/MirrorOS
npm ERR! node -v v4.8.2
npm ERR! npm -v 1.4.21
npm ERR! code E404
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/pi/MirrorOS/npm-debug.log
npm ERR! not ok code 0
Thanks

Hi solved the problem.
Its an issue with the OS .
Actually I switched to raspbian jessie and now I am able to install dependencies and start the program .

Solution:
Use the latest NPM version. You can install it with node following these instructions: https://nodejs.org/en/download/package-manager/#debian-and-ubuntu-based-linux-distributions


Accept TOS to register, create institutions, projects (development)

Purpose
Add Terms of Service validations to:

User Registration
Creating a new Institution
Creating a new Project

Adds a handful of components: EmailInput, PasswordInput, TextInput, TextArea, and NumberInput
Related Issues
Closes #1155
Screenshots
User Registration

Creating an Institution

Project Creation

I need to remove master.  Please open again later.


Develop backend to send API requests with authorization

React does not permit to fetch from APIs without exposing keys.
Develop express backend to proxy requests.

Will not implement. Maybe future new version.


[WiP] bump specter version

Same story as here https://github.com/marick/suchwow/pull/2
tests run via lein midje all succeed. Version bumping left to your hands.
Depends on the suchwow PR version bumping. (everything succeeded using lein checkouts)

Fixed independently of this pull request.


Enduring Might's duration may be wrong?

        "duration": {
            "expiry": "turn-start",
            "sustained": false,
            "unit": "rounds",
            "value": 0
        },

As you can see the duration's value is 0.
So after you drag it to the token it would immediately become expired and the resistance would not work.
Is it works as intended?

It's used as a reaction, which will typically will be outside one's turn. If used that way, it doesn't immediately expire. That said, it's certainly possible to have an occasion to use it in the midst of a turn, so there's currently a system limitation making that impossible.

@stwlam Thank you for your explanation.
I checked Champion's Retributive Strike' resistance effect.
It have the same machanism and it's duration value is 1
    "_id": "DawVHfoPKbPJsz4k",     "img": "systems/pf2e/icons/spells/dancing-shield.webp",     "name": "Effect: Champion's Resistance",     "system": {         "description": {             "value": "
The ally gains resistance to all damage against the triggering damage equal to 2 + your level.
"         },         "duration": {             "expiry": "turn-start",             "sustained": false,             "unit": "rounds",             "value": 1         },
Is there any reason for this ?


Use/Allow Themeable Images for Action Icons

Actions panel currently does not theme images. Therefore images for action icons must be default colored such that they appear well for all themes. We've been moving towards images which change colors based on theme though, but we can't use them as action icons. If we just immediately changed to theming them we would mess up images which aren't meant to be themed. We should think about what we want to do about this....

Related: https://github.com/microsoft/AzureStorageExplorer/issues/5688

Extensions to swap to use the themed icons

[ ] storage account
[ ] blob
[ ] file
[ ] queue
[ ] table
[ ] disks
[ ] app configuration


Various cleanup in tests

coverage: 87.3% (-0.009%) from 87.309%
when pulling 6a2e63c57657cd743a3545b0c8559ceeeb85c5ba on hazendaz:pom-cleanup
into 5665cc35484c6806d477d859e389a178663c8776 on mybatis:master.


project path reappear after deletion

project paths will reaper after deletion.
2nd level cache must be cleared before opening project detail

internal bug, not an issue


Disallow manually reporting posts on child metas

Given that SmokeDetector doesn't detect posts on child metas, and manually reported posts on child metas seem to be triggering a bug (see #2097 ), I propose that we detect and ignore attempts to manually report posts on per-site metas.

Fix the bug, don't remove the feature.


quickcheck test suite doesn't build with text-icu 0.8

Preprocessing test suite 'quickcheck' for unicode-transforms-0.4.0..
Building test suite 'quickcheck' for unicode-transforms-0.4.0..
[1 of 2] Compiling QuickCheckUtils  ( test/QuickCheckUtils.hs, dist/build/quickcheck/quickcheck-tmp/QuickCheckUtils.dyn_o )
[2 of 2] Compiling Main             ( test/Properties.hs, dist/build/quickcheck/quickcheck-tmp/Main.dyn_o )

test/Properties.hs:20:35: error:
    Not in scope: type constructor or class ‘ICU.NormalizationMode’
    Perhaps you meant ‘T.NormalizationMode’ (imported from Data.Text.Normalize)
    Module ‘Data.Text.ICU’ does not export ‘NormalizationMode’.
   |
20 | toICUMode :: NormalizationMode -> ICU.NormalizationMode
   |                                   ^^^^^^^^^^^^^^^^^^^^^

test/Properties.hs:23:18: error:
    Not in scope: data constructor ‘ICU.NFD’
    Module ‘Data.Text.ICU’ does not export ‘NFD’.
   |
23 |        T.NFD  -> ICU.NFD
   |                  ^^^^^^^

test/Properties.hs:24:18: error:
    Not in scope: data constructor ‘ICU.NFKD’
    Module ‘Data.Text.ICU’ does not export ‘NFKD’.
   |
24 |        T.NFKD -> ICU.NFKD
   |                  ^^^^^^^^

test/Properties.hs:25:18: error:
    Not in scope: data constructor ‘ICU.NFC’
    Module ‘Data.Text.ICU’ does not export ‘NFC’.
   |
25 |        T.NFC  -> ICU.NFC
   |                  ^^^^^^^

test/Properties.hs:26:18: error:
    Not in scope: data constructor ‘ICU.NFKC’
    Module ‘Data.Text.ICU’ does not export ‘NFKC’.
   |
26 |        T.NFKC -> ICU.NFKC
   |                  ^^^^^^^^

test/Properties.hs:30:27: error:
    Not in scope: ‘ICU.normalize’
    Perhaps you meant one of these:
      ‘T.normalize’ (imported from Data.Text.Normalize),
      ‘ICU.serialize’ (imported from Data.Text.ICU)
    Module ‘Data.Text.ICU’ does not export ‘normalize’.
   |
30 |     T.normalize mode t == ICU.normalize (toICUMode mode) t
   |                           ^^^^^^^^^^^^^

Upstream commit about this change: https://github.com/haskell/text-icu/commit/6e803c5f02711c218215081fecbd97ba7ccf77bb


Cut the first question into two

From first review:

A drop-down for the 14 cryptocurrencies available on Coinbase
A search for the all currencies available on Coinbase. You can get the list with this link and find how to do it on daskeyboard-applet--weather-international.
With that, you will be able to build your current currency key by adding "-" between these two values.

So looking into this a bit and it seems like this change will more than likely just cause more trouble for the user for the following reasons:

Coinbase does not provide an api for getting their currently supported cryptocurrencies which makes the maintenance of this list a manual effort. Since this suggestion was made in June there have been 2 other coins added already. On top of that they provide (currently) 14 other tokens or coins that they supply data for but not trading.
A dropdown for fiat currencies as the second would work with whats provided. However in the future I would like to enable users to check per-say their LTC price compared to BTC (LTC-BTC) which wouldn't really work unless I had an api for supported cryptocurrencies.

For now I think letting the user type in their crypto-fiat pair is the way to go. In the future I plan to move this over to a different API. Perhaps coinbase pro's currency api which is all currency pairs?


Remove internal zmq_poll implementations for eventloop integration

Instead, use edge-triggered zmq.FD.
This enables integration with any eventloop supported by asyncio, tornado, etc. (closes #894)
After experience of numerous bugs in zmq.green, this can result in hangs if not done super carefully, but it seems worth it at this point with the proliferation of asyncio implementations.

[x] handle user retrieving zmq.EVENTS, which can consume events on the FD.
[x] update docs
[x] update examples
[x] verify that it doesn't break anything in IPython/Jupyter.

Thank you @minrk! 👍

yay, this is yuuge :)


Revert "Merge pull request #65516 from aschwaighofer/guard_using_clang_for_objc_protocols_by_deploy_target"

This reverts commit 1a7f342a4e8ac2302cc8181757c9d57dc865fa9f, reversing changes made to 404b925d8b6c1f323900ea765132924150b1e54b.
Proper fix for this to follow.

@swift-ci test


option to not add "ic_" infront of filename

hi :)
in the generic icon generator, It automatically adds "ic_" infront of the filename, I'm trying to stop it.
some help would be nice, thanks.

yesssss, found it in src\html\icons-generic.html


⚠ Samsung has degraded performance

In eb23717, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 7188 ms

Resolved: Samsung performance has improved in 0bf6228.


Add ability to specify filesystem type

Right now ext4 is hardcoded.  We need to add the ability to specify other filesystems when creating volumes.

XFS, BTRFS, and Ext(2-4) are now available with the volume create option "filesystem"
https://github.com/linode/docker-volume-linode#create-options


B18988 int<PHONE_NUMBER> update 5 digit id in move history to be shipment locator

Agility ticket
B18988
Summary
In the "Move History" for a move, the shipment id (the value appearing after HHG/PPM Shipment #) was changed to "shipment locator".  Shipment locator = move locator + "-" + sequence number for the the move e.g. "ABC123-01".
How to test
HHG

create a HHG move
login as a SC
find the move and open it
go to the "Move History" tab
all occurrences of "HHG shipment #" should have move locator value + "-" + "01"

PPM

as a user

create a PPM move

as an officer user

login as a SC
find the move and open it
edit allowances
submit move

as the user

click upload documents
enter vehicle info
add pro gear
add an expense
submit ppm documentation

as an office user

login as an SC
go to closeout queue
open move
go to the "Move History" tab
all occurrences of "PPM shipment #" should have move locator value + "-" + "01"

Screenshots
HHG

PPM

Changes look good but I think you might be missing a few files.
For example: src/constants/MoveHistory/EventTemplates/ApproveShipment/createStandardServiceItem.jsx

I thought I'd merged that fix in.  That was the HHG fix I did recently.  :(. Thx for catching that.

Looks like a few more missing ones. If you search for .shipment_id_abbr.toUpperCase(), you should be able to find more.

Sorry about the sloppy work on my part.
Thx for the search string.
Searched the code and updated all relevant spots.
Changes have been committed.  Int branch has been updated and pushed.

Found a couple other ones that need to be updated:

updateServiceItemStatus.jsx

Here's the event that you can use to verify, you can just go to the bottom of the MTO page and reject any service item then re-approve it to get it to show up:

editSitEntryDateModal.jsx

For this one you need to find a service item for Domestic Origin 1st day SIT

Then approve it, you'll then get this:

Thats a good find @brooklyn-welsh
Along with those changes I think src/pages/Office/MoveHistory/LabeledDetailsWithToolTip.jsx Line 49 should also be changed to:
shipmentDisplay += ` shipment #${changedValuesToUse.shipment_locator}`;

Thats a good find @brooklyn-welsh
Along with those changes I think src/pages/Office/MoveHistory/LabeledDetailsWithToolTip.jsx Line 49 should also be changed to: shipmentDisplay += ` shipment #${changedValuesToUse.shipment_locator}`;

Oh, that was actually the file I was meaning to quote for the SIT modal, glad you posted that, I'll edit my previous comment

Found a couple other ones that need to be updated:

updateServiceItemStatus.jsx

Here's the event that you can use to verify, you can just go to the bottom of the "Move Task Order" tab for any move and reject any service item then re-approve it to get it to show up:

LabeledDetailsWithToolTip.jsx

For this one you need to find a service item for Domestic Origin 1st day SIT
Then click "edit" and enter a new SIT entry date, you'll then get this:

Code updates have been done and pushed to INT.


[Research]: simulate VR in unreal

Object, subject, context, hypotheses
it chould be possible to develop vr apps without hardware. Need to find ways to do it.
usefull links
https://vreue4.com/
https://www.reddit.com/r/unrealengine/comments/apsdwv/emulate_vr_without_vr_equipment_with_kbm_only/

SimpleHMD
https://www.reddit.com/r/unrealengine/comments/evlvs1/testing_a_vr_project_without_a_headset/

Create project from VR template in UE5.
If there is no VR in the VRspectator blueprints, you should change the base value of bSpectatorEnabled to True.
If it is necessary to test mechanics on behalf of a character, it is recommended to create custom events
If on whose behalf the mechanics are tested is not important, you can create them directly in VRspectator and then transfer them to the character.
The basic template project inputs support both keyboard and mouse and VR controls.


Ribbon integration with hystrix using HystrixObservableCommand

This pull request has been created to show how to integrate Ribbon with Hystrix using HystrixObservableCommand. It also fixes #1772, when not retryable client is used (RetryableOkHttpLoadBalancingClient, RetryableRibbonLoadBalancingHttpClient).

Codecov Report

Merging #1793 into master will increase coverage by 0.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master    #1793      +/-   ##
==========================================
+ Coverage   75.38%   75.39%   +0.01%
==========================================
  Files         203      200       -3
  Lines        6240     6219      -21
  Branches      781      776       -5
==========================================
- Hits         4704     4689      -15
+ Misses       1196     1194       -2
+ Partials      340      336       -4

Impacted Files
Coverage Δ

...x/ribbon/apache/RibbonLoadBalancingHttpClient.java
75% <ø> (-0.76%)
:arrow_down:

...etflix/zuul/filters/route/RibbonRoutingFilter.java
81.11% <100%> (+4.44%)
:arrow_up:

...l/filters/route/support/AbstractRibbonCommand.java
86% <85.18%> (+11.71%)
:arrow_up:

...ork/cloud/netflix/zuul/filters/ZuulProperties.java
84.68% <0%> (-1.81%)
:arrow_down:

.../cloud/netflix/ribbon/RibbonAutoConfiguration.java
90% <0%> (-0.91%)
:arrow_down:

...ramework/cloud/netflix/zuul/ZuulConfiguration.java
86.66% <0%> (-0.44%)
:arrow_down:

...work/cloud/netflix/ribbon/SpringClientFactory.java
100% <0%> (ø)
:arrow_up:

...ix/ribbon/RibbonApplicationContextInitializer.java

...loud/netflix/ribbon/RibbonEagerLoadProperties.java

...x/zuul/ZuulRouteApplicationContextInitializer.java

... and 1 more

Continue to review full report at Codecov.

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

I'm afraid these changes significantly change the way things work and I'm of the mind to not accept them.


Release a new version with latest changes

It seems the version on crates.io does not contain features like .binary() on Response or the WebSocket fixes (#13)? I would like if a new version with these changes were published on crates.io :slightly_smiling_face:

I'm planning on it. I was hoping to get #11 done before publishing. I'll publish as soon as possible (and add example later)

0.3.0 is now released


HV-1305 Use HTML5 doclet when generating JavaDoc with Java 9

https://hibernate.atlassian.net/browse/HV-1305

Using HTML5 for JavaDoc generation with JDK 9. I've added -html5 to parameters and also replaced a couple of <tt>...</tt> with {@code} or {@link} as otherwise it will give a next error:
error: tag not supported in the generated HTML version: tt

https://www.w3schools.com/tags/tag_tt.asp

Nice, thanks, @marko-bekhta!


Make MvcFileIncluder also look for hyphenated file names

This make possible the use of file names separated by hyphens, without losing retrocompability. This way a developer could use my_action.php as well as my-action.php, meeting the WordPress Coding Standards for file names.

Nice addition. Thanks!


fix slinku'i/cmevla conflict in vlatai

Fix some erroneous detection of cmevla that sort of look like slinku'i, by adding another negative lookahead to vlatai's slinku'i detector.

I think this was fixed in v0.7, which was released the day after this pull request, and seems to have drawn on discussions with durka.


Enable Windows Forms high DPI features

This pull request adds high DPI support (resolves #23).

PerMonitorV2 (left) and DpiUnaware (right). Fonts and icons are scaled automatically. Custom render code has been adjusted to make sizes consistent across DPI settings.
Moving fences between monitors with different DPI settings might not work perfectly but I would expect users to leave their fences mostly on one screen.

One bug here still needs to be fixed. The settings store logical window sizes but scaled window sizes get written back. On high DPI screens windows are currently getting larger with every start of the application 😂

Nasty fix. However, even in Vocup that's working fine. People might need to resize their window after dragging it to another screen but I think that's acceptable.

No, I don't think so. Sorry for not looking at the new changes, I'm kinda busy 😅. I'll test the new stuff later today and merge it.

I've just tested it and there still is an issue with the scaling. When I set the scaling on my right monitor to 150% and run NoFences on the main monitor (which is at 100%) - the fence shrinks down a lot on the 100% monitor. Could this be an issue with the Width and Height also being in "regular" pixels.

Resizing when moving between monitors does not work perfectly. However, I don't believe it can be done better with Windows Forms without extensive use of native calls which will not implement. Vocup runs on .NET 6.0 and also has issues when moving between screens with different scaling. Some fixes are announced for .NET 7.0 so we can see if this get better by then.

The problem also occurs when I don't move it between screens, but just have two screens at different resultions. But I'll merge it for now!


Feature/replace model list stage filter with deployment toggle

Description

Remove stage filter in model list
Add deployment toggle to model list

Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #163 (a75ebac) into feature/model-registry (603d536) will increase coverage by 0.03%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                    Coverage Diff                     @@
##           feature/model-registry     #163      +/-   ##
==========================================================
+ Coverage                   75.46%   75.50%   +0.03%
==========================================================
  Files                          76       75       -1
  Lines                        1284     1286       +2
  Branches                      228      243      +15
==========================================================
+ Hits                          969      971       +2
+ Misses                        261      260       -1
- Partials                       54       55       +1

see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


android support

support for android environments is highly desirable

to be able to instantiate Reflections within android rt
to be able to query the Store
easy integration with gradle (+ instructions)

some options: 1) with a different metadata scanner than javassist that supports dalvik, or 2) as a solution for generating scanned metadata on build time and collecting on runtime with Reflections.collect()
2 seems simpler and better for android rt performance, though it's more a workaround.
a one liner integration for gradle or similar is preferred.

I would like to work on this issue please assign me this issue.

I can try to work on this. I see it is tagged as a good first issue so I will take a look


Provide an Automatic-Module-Name

Is your feature request related to a problem? Please describe.
It is currently not possible to create modular applications which depend on google-cloud-kms it is not module-ready.
Describe the solution you'd like
Define an Automatic-Module-Name in MANIFEST.MF to make google-cloud-kms module-ready.
Describe alternatives you've considered
The only alternatives are giving up creating modular applications or repackaging google-cloud-kms to add the required metadata.
Additional context
Providing Automatic-Module-Name has no impact on non-modular applications or applications targeting Java 8.

If there is no objection to this addition, I would gladly provide PRs for all required modules.

We are currently using a trick where we split every affected module in two, the actual module depending on an "impl" automatic module. We define an Automatic-Module-Name on "impl" so that tools recognized it as module ready. Then, all classes which contain a reference to Google Cloud stuff goes to "impl". This is incredibly annoying, and is inducing people to give up on architecture and put unrelated things in the same "impl".


Reverse Supercell Mapping

Adding function to map SC onto UC positions
@jmmshn check tests and function

Pls hold off on further development for a few hours. I am implementing a major reorganization of pymatgen-diffusion. Once I am done, you can pull in the master branch and continue working.

I am done with my rewrite. Feel free to pull main master into your PR.

Coverage increased (+0.6%) to 75.799% when pulling 36fcecad09bfa3af7b700edfaf170aa665e5c4ad on hmlli:neb_setup into a056d6bf7bcf34a123dbc699e43915d8b36f0730 on materialsvirtuallab:master.

@shyuep This is now ready to merge, please check and merge. Thanks!

Thanks. I will soon add a contributor page so that we can acknowledge these contributions.


OCPBUGS-29119: this is a test commit

This is a test

/hold

/jira refresh

/jira refresh

/jira refresh

/jira refresh

/close


SystemFontFamilies does not return all FontFamilies in Chinese System

I got an exception thrown in the GetRenderer method when I am trying to change the font name from Arial to 宋体.

In the VS debugger, I can confirm the Fonts.SystemFontFamilies only contains 204 elements without 宋体, but it indeed exists in my system and can be created through var songti = new Font('宋体', 9). The songti variable shows it could be found at System.Drawing.FontFamily, containing 253 elements.
https://github.com/ForNeVeR/wpf-math/blob/b5d0af65a80e7095addcde247005a8ac1d01b0b6/src/WpfMath/TexFormula.cs#L107
So, using SystemFontFamilies in GetRenderer is aimed to avoid some expectable error or just a bug? Is it possible to change it to the System.Drawing.FontFamily?

I'm classifying this as a feature request, since I believe that eventually we'll come to a conclusion that the existing API should be improved for users to be able to pass any fonts directly (not by name).

Hi @ForNeVeR , thanks for your concern.

I am so sorry about confusing the conception of installed and system font. The file of 宋体 is a system font which filename is SIMSUN.TTC. The system automatically installs it during OS install, which means it will exist in all Chinese Windows. Now I am confirmed the SystemFontFamilies contains a font which Source is SimSun, and the 宋体 exists in the FontFamily.FamilyNames.Values.

It is probably that this API provides all available alias of Font for different languages.
For example, the MS Gothic font provides name 'MS Gothic' and 'ＭＳ ゴシック' in FamilyNames with respect to en-us and ja-jp.

From the above test, I guess the implementation of the GetSystemFont method can be rewritten as:
        internal static SystemFont GetSystemFont(string fontName, double size)
        {
            var fontFamily = Fonts.SystemFontFamilies.First(ff => ff.FamilyNames.Values.Contains(fontName));
            return new SystemFont(size, fontFamily);
        }

It is working fine in my system and may is a viable solution 😄 .

Thanks a lot for help! I've fixed the issue and will prepare a release soon.


React docs on useEffect

The docs have an example about using useEffect with cleanup, here.
If I carry out the sequence, then I would get
run effect -> subscribe to update from ChatApi -> setIsOnline -> cleanup -> run effect  -> subscribe to update from ChatApi -> setIsOnline
I am unsure as to why this does not lead to an infinite loop without use of the dependency array?
One understanding / theory that I have is:
The first time the subscription cause setIsOnline(true). Next time (after one cycle of unsubscribe/effect), the setIsOnline gets argument as true and hence further update to the component does not occur. This is assuming that this form of setIsOnline does not re-render if the isOnline has not changed (it is still true).
I am curious as to why the piece of code is correct.
Here it is as copied from the useEffect documentation
import React, { useState, useEffect } from 'react';

function FriendStatus(props) {
  const [isOnline, setIsOnline] = useState(null);

  useEffect(() => {
    function handleStatusChange(status) {
      setIsOnline(status.isOnline);
    }
    ChatAPI.subscribeToFriendStatus(props.friend.id, handleStatusChange);
    // Specify how to clean up after this effect:
    return function cleanup() {
      ChatAPI.unsubscribeFromFriendStatus(props.friend.id, handleStatusChange);
    };
  });

  if (isOnline === null) {
    return 'Loading...';
  }
  return isOnline ? 'Online' : 'Offline';
}

useEffect without any values to observe does not run second time.
IF you are trying to re run your code on infinite loop
useEffect(()=>{
setIsOnline(!isOnline)
}, [isOnline])

But most probably you will get error saying maximum depth reached and code or app will break

Could you point to documentation where its mentioned that useEffect does not run a second time if no values to observe (in dependency array) are given? I am not sure I came across that. Just a thing that was baffling in the documentation for me.


Implement edge_nal::TcpAccept trait from edge-net

This PR provides an implementation of edge_nal::TcpAccept to use esp-mbedtls with edge-net.
As for now, the trait is simply a wrapper over the edge_nal::TcpAccept implementation from edge-nal-embassy.
I also provided an example that uses edge-http in server mode for testing, and ensuring compatibility.
Further down the line, other traits will also get their implementation, in subsequent PRs, as I (or any contributor) come around a use case for them.

I'll write this here to document the behavior, but there is currently an issue with how esp-wifi, esp-mbedtls, edge-net act together.
Specifically, if a client sends a request with the header "Connection": "Close", the server implementation of edge-net requests a read() from esp-mbedtls (It won't if the body type is something else). Since the client has sent all its data, async_internal_read() will hang on reading data from socket. This is due to esp-wifi's read impl that blocks on read until there is data.

I'll write this here to document the behavior, but there is currently an issue with how esp-wifi, esp-mbedtls, edge-net act together.
Specifically, if a client sends a request with the header "Connection": "Close", the server implementation of edge-net requests a read() from esp-mbedtls (It won't if the body type is something else). Since the client has sent all its data, async_internal_read() will hang on reading data from socket. This is due to esp-wifi's read impl that blocks on read until there is data.

I'm not sure I follow 100% but regardless, are you using the master version of edge-http?
There is a fix in master I did recently which avoids a similar (yet, the opposite) problem, where if there are multiple requests scheduled over a single TCP connection, the processing of the first request might read a bit into the second request.


added type parameter to the relations

The type parameter is necessary to choose the service build the right
resources. The type is not always the same as the relation name.
For instance say there is a Person that has a relationship called 'friends' to other Persons and a relationship called 'children' to other Persons as well. The specification in the resource would now be:
friends: hasMany('friends', 'persons')
offspring: hasMany('children', 'persons')

The previously you would get an exception when the name of the relation was not the same as the type.
The type is now the second argument in the specification of the relation in the resource, but it is optional. If the type is indeed the same as the relation name, it can be omitted.

I'll take a look tomorrow, thanks.

@Rahien yeah I see how the type value helps when the reasource name is not the same as the type.
I'm wondering about how to set that up so it's clearly documented, for example:
friends: hasMany('friends', 'persons')

vs.
friends: hasMany({ resource: 'friends', type: 'persons' })

so instead of two parameters there would still be one parameter passed to hasMany or hasOne, and i the argument is a String then the resource name and the type value are equal, alternatively if the argument is an Object then the resource and type are set and used accordingly.
Any thoughts?

Yes! That makes a lot of sense, I changed it in the latest commit.
Thanks for the feedback!

@Rahien I'm trying to figure out a test case for this usage of a resource with a different type name. do you have any JSON examples you can share from your API? Are you using JSONAPI::Resources gem? I'm guessing you use case is with Polymorphic associations, correct?

@Rahien  I think your use case is that a service may be named 'persons' which can return various resource types, e.g. Children and Friends. And, those types are relation names as well. But when used as a relation only the 'persons' service (endpoint) is used to find the resources.
This use case seems doable but it will require some extra setup when generating resources with the ember-cli generator tool. You'll need to generate a model for the Child and Friend types; but not services using those resource names (children and friends services/endpoints do not exist, only persons). And, you may need to generate a 'persons' resource - which may only be an abstract model used when extending to create Child or Friend resources (model prototypes).

I'd like to create a test to accompany this pull request but am not familiar with how your app is using this optional resource/type value for using a common server for various resources. I'd need some JSON docs and a the resource (model) prototypes to setup a test.

Let me start by thanking you for all your help! Now, the actual use case that I am trying to solve is the front end of a taxonomy matching service. A user receives a list of topics and the topics have mappings to topics of another taxonomy. These mappings are directional and receive a match type selected by the user, e.g. exact, close, narrow, broad (the last two explain why the match is directional).
So, the mappings have two relationships to topics, a "from" and a "to" relation, both pointing to a resource of the kind "topic". There is no reason to use a different model or service to instantiate these resources (though I can see how polymorphic relations could be challenging). In my case both relations are of the type "topic", but they have different names and different semantics.
Here is an example json file that is returned by the back end that I am using (generated by https://github.com/mu-semtech/mu-cl-resources), this returns the list of mappings for a topic:
{"data": [
    { "attributes": { "matchtype": "exact" },
      "id": "MatchOne",
      "relationships": {
          "from": {
              "links": {"related": "/mappings/MatchOne/from", "self": "/mappings/MatchOne/links/from"}
          },
          "to": {
              "links": {"related": "/mappings/MatchOne/to", "self": "/mappings/MatchOne/links/to"}
          }
      },
      "type": "mappings"
    },
    { "attributes": { "matchtype": "broader"},
      "id": "MatchTwo",
      "relationships": {
          "from": {
              "links": {"related": "/mappings/MatchTwo/from", "self": "/mappings/MatchTwo/links/from"}
          },
          "to": {
              "links": {"related": "/mappings/MatchTwo/to", "self": "/mappings/MatchTwo/links/to"}
          },
      },
      "type": "mappings"
    }
]}

the topics themselves have the following resource file (coffeescript):
`import Ember from 'ember'`
`import Resource from './resource'`
`import { attr, hasOne, hasMany } from 'ember-jsonapi-resources/models/resource'`

Mapping = Resource.extend
  type: 'mappings'
  service: Ember.inject.service 'mappings'

  score: attr()
  matchtype: attr()

  from: hasOne { resource: 'from', type: 'topic' }
  to: hasOne { resource: 'to', type: 'topic' }

`export default Mapping`

So there are two hasOne relations with the same type "topic". This was previously impossible as the name of the resource was assumed to be the same as the topic.
Is there anything else that is still unclear in the use case?

@Rahien I did some work to figure out what is needed to support polymorphic associations, see https://github.com/pixelhandler/ember-jsonapi-resources/pull/37 however that is not directly related to your use case.

from: hasOne { resource: 'from', type: 'topic' }
to: hasOne { resource: 'to', type: 'topic' }

These reference relations `to` and `from` both are always using a type of `topic` If I understand correctly the use case simple needs to have a resource name that is used in the relationship object of the resource's payload and use a specific service (endpoint) with a different name (type). Basically giving more configuration options.

I am still curious if just creating a service (the uses the application adapter and serializer) for the `from` and `to` relations would work since they are only used as relations and don't need a resource (model) prototype. See the example I added in the pull request for polymorphic association support where I used an 'imageable' service that is only used since that is the name of the relation. I added the service, adapter and initializer for the imageable (polymorphic) association between a picture resource that can belong to either an employee or a product.

I'm wondering what would be best to have a simple service that is used only for the findRelated adapter method for every relation (no need to add extra configuration) or do provide the proposed options to provide a type that is used for the service when the relation is fetched via the service#findRelated call.

hey guys, fwiw, I had a similar case where the same entity could be of a different type when in a relationship. In it a participant is also an author to a proposal, and a me (current user) resource.
I got bit by EJR not accepting a type and relation names being different, and almost posted a feature request here as well. But then decided to try and solve this on the backend, by having different endpoints and resources for these relationships. So, in rails, the Participant model now backs three different resources - ParticipantResource, AuthorResource and Me resource. Each of these are defined, using JSONAPI::Resources, with a model_name 'Participant'.
This clicked for me, seems intuitive to have the same resources defined on the front end and backend, instead of having to translate the types so a front-end resource maps to a different one on the backend. Makes it easy for the front-end to decide what resources it needs. Also gave me the freedom to return different payload for each of the resource representations on the backend.
This assumes one has control of the API, though, to keep the resource modeling in sync between the two. I'm not sure what the JSONAPI authors think about the common enough scenario of front-end devs having to adapt to APIs that don't map to their local modeling. My impression is the philosophy has been for the front-end to drive the API design from the beginning?

@Rahien one more thing I am curious about… in your use case will the to and from relations sometimes use a different type? (I.e. sometimes the topic for a mapping but maybe another subject for a grouping) If the to and from resources do always use the topic service/endpoint then the to and from adapters can define how the url is used.
export default ApplicationAdapter.extend({
  type: 'from',
  url: '/topics',
  fetchUrl: function(url) {
    /* manipulate if needed */
    return url;
  }
});

Hmmm regarding the url, indeed at the moment the type of both relations is always 'topic'. I see what you mean, for every relation, you would create an adapter and a service. I guess that works, but isn't the overhead of defining an extra adapter and service every time a bit much? It does away with extra configuration options being passed in, but there is so much extra code involved when using it... Just referring to an existing service in the model seems less confusing to me.
Still: personal opinion, so it's your call of course!

@Rahien I agree it's a nice option to specific another type as an alias for a relation. I just needed to better understand the use case so I could come up with a test for it. I'll work on adding a test.


ci: updates docs push action by fetching tags & setting the correct tag

This should (finally) fix the docs pushing - due to #1230  the deployment broke because the actions/checkout@2 does not fetch the tags per default anymore.
I've updated the CI script to fetch the last 20 commits and the tags and updates the VERSION detection a bit (the --match would still require the full history) - to detect such issues earlier I've added a simple echo of the variables.

You're welcome :) - master is green and has been deployed successfully! 👏


More mouse button

I'm using Elecom EX-G Pro, and it has 8 buttons, including left/middle/right click. Instead of back/front buttons, I want to assign the function to another button.

Thank you for the request, the trackball looks awesome. Remapping buttons to other functionality than scrolling is already covered by other software, I therefore consider it out of scope of TrackballScroll. I recommend autohotkey for this, remapping only requires a 1-line configuration.

Remapping buttons to other functionality than scrolling

No, I want to assign scrolling to another mouse button, like 5th X-Button.

Thank you for the clarification, in that case, the issue is valid for TrackballScroll.
The Windows libraries support only 5 buttons (left, right, middle, x1, x2). I found a comment which suggests that additional mouse buttons actually send keyboard inputs.
Source: https://stackoverflow.com/q/27655619 (comment from Jan 5, 2015)
#28 already covers keyboard inputs and then should also cover the additional mouse buttons.


Remove unused score functions in the stats_helper.py file

OpenStack Release
Liberty+
Description
There are a number of methods in the stats_helper module that are not used by the OpenStack code.  They were left over from a previous version of the library that was used in a different project. This is now dead code and should be removed.

See this fix in master in commit: https://github.com/F5Networks/f5-openstack-agent/commit/667dfc91f640495a389c2b9accc01cc2d4c73bfe


Edition 2021 update and some slight fixes.

Switched to 2021 feature set.
Fixed lifetime issues in the client.
Satisfied clippy.

Does clippy explicitly care about using explicit return statements?  If not, I prefer to keep that particular syntax out of my codebase since it's not as clear as a nice return statement 🙂.

Omitting return statements is considered idiomatic rust and clippy does catch it.
https://rust-lang.github.io/rust-clippy/master/index.html#implicit_return
CI for rust repositories often includes cargo clippy -D warn, which would return an error.
I don't mind either syntax and if you prefer the return statements, the "pregma" can be insterted at the root of the crate.

Actually my bad, I've linked to the wrong clippy lint.
I meant to send this one. It does report as a warning by default.
https://rust-lang.github.io/rust-clippy/master/index.html#needless_return

There's a couple clippy things I will deal with later, for now this can be merged and I'll fix the clippy pragmas for future stuff.


maintainers/teams: add node

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@marsam, @lilyinstarlight, would either of you like to join us?

Yeah sure, go ahead and add me
Thank you @winterqt!

Sure, please add me to the list. Thanks!


[MGDOBR-1131]:Restructuring framework by moving common classes of int…

…egration tests into a new module
MGDOBR-1131
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] The layers in the kustomize folder have been updated accordingly.
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

Can this be closed?

Yes, closing


Feature Request: Bonding/Pairing on ios and Android

Would be great to have a way to pair a device from the code without manually doing it from the system settings page.

Hi @electro-logic,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!

public abstract Task BondAsync(IDevice device);

Hello, BondAsync is not working on Android

Would be great to have a way to pair a device from the code without manually doing it from the system settings page.

are you using the latest version?

Hello, BondAsync is not working on Android

It should. What are you trying to do? And what happens when you do it? Do you get any error messages or exceptions?

I tried again today
await Permissions.RequestAsync<Permissions.Bluetooth>();
var adapter = CrossBluetoothLE.Current.Adapter;
adapter.DeviceDiscovered += Adapter_DeviceDiscovered;
await adapter.StartScanningForDevicesAsync();
// ...
await adapter.BondAsync(e.Device);
await adapter.ConnectToDeviceAsync(e.Device);

and seems working fine. Thank you anyway.


Dynamic elements

When I tried to add dynamically elements, failed because the library does not support it.
I tried to fix but does not work even using $scope.$apply to reload.
Any idea?

It's possible to add dynamically elements. It was added some versions ago.
Can you reproduce or explain what you're trying to do ?

Hello,
I have the same problem when using http.get to assign values to it. The return value for http.get is JSON object. When i directly assign it hardcoded, it works well, just when assign after i get return data from http.get is failed.
The error caught is as shown below
TypeError: Cannot read property 'length' of undefined
Thanks & Regards,
Kevin M.

Are you using the latest version ? This issue was fixed a couple of versions ago. Can you build an example ? Or send me some sort of code snippet so I can have a look ?

imported files

i pull it via bower, should be latest 1?
How i call the API and assign value
$http.get("http://www.xionapi.com/MenuSelectCafe.aspx?id=" + $stateParams.cid)
.success(function(response) {
$scope.imenu = response;
});
The frontend

It should. Just to be sure, check in the package.json if the "version": "0.1.8" if not, update it

hello, yeah it is 0.1.8, i wonder what's the problem, the item can be in JSON object format right?

can you provide me the code for ionic view example? let me fork it so i can learn from it.
Thank you very much for your time

hello,
I think i've made it. I'm able to dynamically set the value in. What changes i've made is simply declare the variable as object before assign it.
Thank you for your time and effort.
Kevin Mun

Can you share your solution ? (Just so others can see it if in the future they have the same issue)

Hello,
yes of course.
need to declare the variable as object before assign value to it.
$scope.imenu = [];
$http.get(http://www.myapi.com/myapi.aspx)
.success(function(response) {
$scope.imenu = response;
});
Thanks & Regards,
Kevin Mun

Makes sense. Was passing an undefined scope variable to the directive. Thanks


Update Dockerfile

Standalone Dockerfile updated

Could you please send this to 0.4.0 branch. Please make sure versions are correct.


Fix Compose previews not working

This PR fixes the issue where the Compose Previews stop working because of an illegal cast of BridgeContext to Activity
Preview (Before Fix)
Compose preview not working because of the illegal cast ❌

Preview (After Fix)
Compose preview working fine ✅

@stevdza-san Could you please have a look into this?


fix: 🐛 canTransfer method for 5.4 with dual version

Description
bug was introduced with dual version support because RPC methods are not automatically generated. 5.4 was expecting newer return type. fix works by aliasing the new type name to old response on 5.4
Breaking Changes
None
JIRA Link
N/A
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 22.0.0-beta.5 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 22.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can't open AWS Publishing panel - Unable to Publish to AWS  -  System.ComponentModel.Win32Exception: Access denied

Describe the bug
Hi, When trying to open the Publish To AWS dialog in Visual Studio 2019, the following message shows:

In the logs I can see the following error:
2022-11-10 14:03:24,137 [47] ERROR Amazon.AWSToolkit.Publish.Package.PublishToAws - Failed to open the Publish document
System.Exception: Unable to start deploy tooling: Se han producido uno o varios errores. ---> System.AggregateException: Se han producido uno o varios errores. ---> System.ComponentModel.Win32Exception: Acceso denegado
en System.Diagnostics.Process.Kill()
To Reproduce
Steps to reproduce the behavior
example:

Go to 'Compile'
Click on 'Publish xxx to AWS'
See error

Expected behavior
I should see the Publish To AWS helper Dialog. I need to be able to publish using the Fargate tool.
Computer (please complete the following information):

Windows Version: Windoes 10 Enterprise
Visual Studio Version: 2019
AWS Toolkit for Visual Studio Version: <IP_ADDRESS>

Hi @marisagrasso , can you please upload your log file so that we can see if there are any additional details that could help diagnose the issue?

Closing this issue since we haven’t heard back from you in a while. If you are still encountering this problem please open a new issue with detailed steps to reproduce it.


Feature Request:  MySQL Port option

Would really appreciate the ability to set a non-default port ie. 3307 as the current port is already in use.

Done https://github.com/ImagicTheCat/vRP/commit/181048e7fc74fd44a815dd4018f40d390e48b6a9.


modify schema to support integers in the urls

It looks as though this removes underscores from the regex also, which is probably unintended.
Could you elaborate on the use case you have and/or what you expect/desire to happen to give me a bit more context? Thanks!


Add link to GitHub repo

in the header, there should be a link to link back to this repository

Hi @Khazifire ! I'd like to take this up, please assign this to me. Thanks !

@samyukthasudhakar  I have assigned the issue to you, Thanks!
for the icon you can use (remix icons )  --- its already installed in the project

#14

Thank you @samyukthasudhakar for your contribution, I will review it


Makefile

Codecov Report

Merging #1215 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1215      +/-   ##
=========================================
+ Coverage    3.86%   3.87%   +<.01%
=========================================
  Files          35      35
  Lines       18069   18023      -46
=========================================
  Hits          698     698
+ Misses      17245   17199      -46
  Partials      126     126

Impacted Files
Coverage Δ

apis/voyager/v1beta1/zz_generated.deepcopy.go
0% <0%> (ø)
:arrow_up:

pkg/ingress/nodeport.go
0% <0%> (ø)
:arrow_up:

pkg/ingress/internal.go
0% <0%> (ø)
:arrow_up:

pkg/ingress/loadbalancer.go
0% <0%> (ø)
:arrow_up:

pkg/ingress/hostport.go
0% <0%> (ø)
:arrow_up:

apis/voyager/v1beta1/openapi_generated.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Improve keyboard navgiation in result list

The keyboard navigation was pretty much non-existent in the results list of the explorer. We need at least "something" to get this working. This address issue #232
This implements the bare minimum of keyboard navigation through the results list.
As you arrow "down" or "right" throw the results list, it will expand the groups and select the next result.
As you arrow "up" it will expand the group and select the previous result.
As you arrow "left" it will collapse the group that the selection is moving from and then select the previous result (in the previous group) unless it is the top group.
This also improves the "expanding" or "collapsing" of a group by using the jquery selector to find the groups that need to change rather than enumerating through all the results (which improves perf a bit).
@jeffersonking Is currently investigating a new way to implement the explorer view (which is more modern) so this fix is to address current concerns.

Could we discuss the config preserveKeyboardFocus? How many people do we know want it false?
Considering existing UI like the Problems panel and Explorer, the predominant pattern is to preserve keyboard focus. I don't think those are configurable either.

Don't disagree with that statement at all. The ONLY reason I made it a setting is that the existing behavior was to NOT preserve focus, so I made an option for anyone that may want that behavior.
Let me check on any VSCode settings.

In reply to: 617885676 [](ancestors = 617885676)

What does the "Spec" say the behavior was supposed to be?  That would need to be fixed first.  As a result, what Gabriel has is a way to maintain previous behavior.

In reply to: 617890878 [](ancestors = 617890878,617885676)

What does the "Spec" say the behavior was supposed to be? That would need to be fixed first. As a result, what Gabriel has is a way to maintain previous behavior.
In reply to: 617890878 [](ancestors = 617890878,617885676)

Don't disagree with that statement at all. The ONLY reason I made it a setting is that the existing behavior was to NOT preserve focus, so I made an option for anyone that may want that behavior.
Let me check on any VSCode settings.
In reply to: 617885676 [](ancestors = 617885676)

Could we

Don't disagree with that statement at all. The ONLY reason I made it a setting is that the existing behavior was to NOT preserve focus, so I made an option for anyone that may want that behavior.
Let me check on any VSCode settings.
In reply to: 617885676 [](ancestors = 617885676)

Could we double-check this claim: "that the existing behavior was to NOT preserve focus"?
I have 2.14.0 (from the Marketplace) installed and I am not seeing that.
Furthermore, #231 is titled "Add an option for taking focus to source when moving between results". Implying focus was not being taken to source and thus being preserved in the past.

Trying out this change, but blocked by #255 where I can't click on the result to begin with.

Interesting. I'll look into #255. I'm using SARIF files where it either contains artifact content(embedded) or I have the source files.

In reply to: 617936956 [](ancestors = 617936956)

Interesting. I'll look into #255. I'm using SARIF files where it either contains artifact content(embedded) or I have the source files.
In reply to: 617936956 [](ancestors = 617936956)

Ok, I'll temp test with those types of logs.

Went and looked at the "original" code, I changed the behavior, so removing the setting and retaining original behavior.

In reply to: 617894660 [](ancestors = 617894660,617890878,617885676)

Went and looked at the "original" code, I changed the behavior, so removing the setting and retaining original behavior.

In reply to: 617904822 [](ancestors = 617904822)


Add failing test

Failing test for #669.

This is actually correct behavior. LineLenght fixers work following way:

If the line is too long, break it
It it's too short, inline it

Code in the test is basically:
-$this->call(
-     $one,
-     $two
-);
+$this->call($one, $two);

It's fixed correctly by the fixer.
There might be personal preference on indentation, but that's not what this fixer should cover.
Possible some ArrayIndent or Braces Fixer/Sniff

Would it be possible to add some configuration to at least not make my code uglier? I can't use this fixer in its current state.
My practice is that whenever any argument takes multiple lines (array, closure, long function call) then each argument should start at its own line. This makes the use-case from my test completely different than simple $this->call($one, $two);.

What kind of configuration do you have in mind?

Not exactly sure, depends on how the fixer works. But the simple idea that comes to mind is an option to only break long lines but not do the reverse. While not ideal it would at least enable me to use the fixer.

Something like?
break_long_lines: true
inline_short_lines: false

Yeah. :+1:

It would only require adding 3 configure() methods with if/else + 1 test.
I don't have time for it, but can guide your trought it.

Sounds good. Give me some notes how to do it or where to look for inspiration and I'll try.

PR code is worth dozend of github comments
This pull-request added 1 configuration option to already existing fixer:
https://github.com/Symplify/Symplify/pull/314/files#diff-5bab0be1e11c555c36f4bf5bdd9dc645R100
Link goes directly to 2 new method that needs to be added. Fixers are configured via Symfony\OptionsResolver, so that's why.
Also test with non-default configuration was added, to prove it works. All is in the PR.

Cool. Can you give me some advice how to upgrade to 4.0 alpha? I'd like to try it on my project and then look into this.

Sure. You can see this issue: https://github.com/Symplify/Symplify/issues/679
There is PR to update Symplify 4-dev on my website: https://github.com/TomasVotruba/tomasvotruba.cz/pull/266
And definitely check CHANGELOG.md: https://github.com/Symplify/Symplify/blob/master/CHANGELOG.md#v400alpha1unreleased
It's all there


lowvram may not be working properly after ComfyUI update last week

https://github.com/comfyanonymous/ComfyUI/issues/3044
I'll look into it this week. lowvram implementation changes will remain a thorn in my side until the end of time it seems lol, feeling like Sisyphus.

Should be fixed now; will close issue when get confirmation.


New Docker image does not run because JDK7

On new image (for example http://spark-notebook.io/dl/docker/0.7.0/2.11/2.1.1/2.6.0-cdh5.10.1/true/true )
when we run and try to go to the http interface, we have a oops, the
unsupported major.minor version 52.0
on logs.
So it is more critical than issue https://github.com/spark-notebook/spark-notebook/issues/868 where only dependencies is not good : now the docker image is not usable.
Regards

hi, 0.7.0 is a rather old release.
you may try the newest code from master (scala 2.11 only) - if I'm not mistaken, it should use jdk8.
Use:
docker pull andypetrella/spark-notebook:0.9.0-SNAPSHOT-scala-2.11.8-spark-2.1.1-hadoop-2.7.2-with-hive
docker run -p 9001:9001 andypetrella/spark-notebook:0.9.0-SNAPSHOT-scala-2.11.8-spark-2.1.1-hadoop-2.7.2-with-hive

* for master, currently you need to change 0.7.0-SNAPSHOT -> 0.9.0-SNAPSHOT in the instructions sent to you by spark-notebook.io

P.S. we still need to add newly released 0.8.3 to spark-notebook.io interface and update the UI...

so basically it's not JDK7/8 issue, anymore...
what needs some love:

[ ] spark-notebook.io
[ ] maybe improving default docker config slightly...

so basically it's not JDK7/8 issue, anymore... so closing this ticket.
what needs some love:

[ ] spark-notebook.io
[ ] maybe improving default docker config slightly...


Transfer jupyterlab-rjsf

Target

The code repository: https://github.com/deathbeds/jupyterlab-starters/tree/main/packages/jupyterlab-rjsf
The npm package on npmjs.com (if available): https://www.npmjs.com/package/@deathbeds/jupyterlab-rjsf
The python package on pypi.org (if available):
The conda-forge recipe on https://github.com/conda-forge (if available):
The documentation website (if available):

Reason

Checklist

[ ] You are owner of the repository you want to transfer
Which role do you want to have in the jupyterlab-contrib organization (check one of):

[x] Help with all jupyterlab-contrib repositories
[ ] Help with the transferred repository only
[ ] I don't have time anymore to keep helping the project

The transferred repository (check one of):

[ ] Must keep the extension name
[ ] Cannot use the original name
[x] I don't care

[x] The code is available under a Open Source license
[x] A license file is present in the code repository
[x] The extension is working on at least JupyterLab 3.x
[x] The repository contains a README file describing:

[ ] The provided feature(s)
[ ] A picture illustrating the feature(s)
[ ] The available options/settings (if applicable)
[ ] How to install the extension
[ ] optional a Binder link to test the feature online

[ ] optional The frontend code (Typescript/Javascript) is tested
[ ] optional The backend code (Python) is tested
[x] I certify the extension is not using exclusively paid-plan of any commercial service
[x] I will help the transfer process on the code repository AND the built packages

You must enable two-factor authentication on GitHub

cc. @bollwyvl
We would like to move jupyterlab-rjsf https://github.com/deathbeds/jupyterlab-starters/tree/main/packages/jupyterlab-rjsf into a place we can maintain as a community. Right now that package is buried inside a monorepo that makes it difficult for us to contribute and update.
We depend on this in multiple packages, and the fact that it still builds against JupyterLab 3 packages makes it difficult to work with.
I suggest we move that under jupyterlab-contrib and continue the maintenance there.

I'll echo my position from during the implementation of the settings form: this feature (if not perhaps this implementation) belongs in JupyterLab itself, available to all extension authors. This could be delivered either via rjsf, or, and likely better, as a lumino component agnostic to the actual leaf node renderer.
Considering the former: as rjsf is already a dependency (and part of the public API of) JupyterLab, a low-react lumino widget could be made available (in e.g. ui-components), with an rjsf Theme which fully participates in the jupyterlab theming system, design language, configurable markdown renderer, etc. It should be possible to add additional schema renderers, templates, etc. to a default registry. In the specific case of settings, that metaschema would expand to provide the full uiSchema object, allowing for very precise control of individual fields, rather than the "big stick" of an entirely custom react widget.
Such an implementation could then go further, making more key jupyterlab UI features available as JSON-configurable widgets. Examples include: LSP-enabled CodeEditor components, lumino DataGrid, etc.
In light of the react/lumino dissonance, the other direction (implementing an accessible, themable, lumino-first JSON schema renderer) would likely be more maintainable long term, and would make deeper reuse more plausible.

My experience has been that supporting complex extensions in the 4.x line has become increasingly difficult. If an organization has the means to support this work in any form, going directly for lab 4.4.x will provide the most benefits for the most number of extension authors and thereby users. I doubt there is much to convince about: unless replaced with a feature-complete lumino implementation, rjsf (and clearly codemirror, marked) aren't going anywhere in the codebase in the 4.x line, and making these features available via declarative JSON (instead of JSX soup) would be a boon.


Test storage readers functions after reverts (additional checks)

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[x] Other (please describe): Tests

Does this introduce a breaking change?

[ ] Yes
[x] No

This change is

crates/papyrus_storage/src/body/events_test.rs line 119 at r1 (raw file):

    let event_index = EventIndex(
        TransactionIndex(block_number, TransactionOffsetInBlock(0)),
        EventIndexInTransactionOutput(2),

Why 2?
Code quote:
        EventIndexInTransactionOutput(2),

crates/papyrus_storage/src/state/state_test.rs line 311 at r1 (raw file):

    let new_data = StarkFelt::from(1);
    let updated_storage = IndexMap::from([(updated_storage_key, new_data)]);
    let nonce1 = Nonce(StarkFelt::from(0));

Consider from 1 to be consistent with elsewhere
Code quote:
let nonce1 = Nonce(StarkFelt::from(0));

crates/papyrus_storage/src/state/state_test.rs line 323 at r1 (raw file):

        .begin_rw_txn()
        .unwrap()
        .append_state_diff(BlockNumber(0), state0.clone(), vec![])

Consider renaming state_diff0 (and 1 similarly)
Code quote:
state0

crates/papyrus_storage/src/state/state_test.rs line 349 at r1 (raw file):

    txn.commit().unwrap();

    assert_matches!(deleted_data, Some((_thin_state_diff, _class_definitions)));

Shouldn't we check for the specific values here?
Code quote:
Some((_thin_state_diff, _class_definitions)))


feature: System database support for AWS S3

Feature scope
Configuration (settings parsing, validation, etc.)
Description
It would be great to have Meltano able to write runs data to AWS S3. S3 is a supported "state backend" (so Meltano can write state there) but it's not a supported "system database" (so Meltano cannot write runs data there).
We currently run Meltano using a Postgres system database and have become accustomed to having the runs table data available. However we'd like to retire this Postgres database and have been standing up a Meltano project using the S3 state backend. I am now realizing that this means we will no longer have runs data available to us.

Hey Matt, thanks for filing!
Can you say more about how you're using the runs table? Is it something that you glance at occasionally, or are some of your workflows dependent on it?

I'm trying to think what this would look like. State backends are essentially key-value stores so it's easy to use object storage for that, but the runs table is more transactional.
See for example how the job/run model is used within a transaction to keep a heartbeat
https://github.com/meltano/meltano/blob/bcbe3eb465e258efa818a97a43f8fe637a24220e/src/meltano/core/job/job.py#L249-L280
and to control concurrency
https://github.com/meltano/meltano/blob/bcbe3eb465e258efa818a97a43f8fe637a24220e/src/meltano/core/block/extract_load.py#L477-L499
I'm happy to discuss spec and implementation proposals, and even review PRs, but this is something that we probably won't prioritize ourselves.

That said, one option that may be available today is to rely on the default SQLite system db and use something like Litestream[^1] to sync the database with S3.
[^1]: https://litestream.io Litestream - Streaming SQLite Replication

Another idea that just came to mind is to search for or implement a sqlalchemy dialect that's sqlite + s3, so it could be used like the example in https://github.com/meltano/meltano/issues/7143#issuecomment-1690040370.
The individual components seem to be out there:

https://github.com/uktrade/sqlite-s3vfs
Although this is concerning

No locking is performed, so client code must ensure that writes do not overlap with other writes or reads. If multiple writes happen at the same time, the database will probably become corrupt and data be lost.

https://rogerbinns.github.io/apsw/vfs.html

https://github.com/edgarrmondragon/sqlalchemy-sqlean/blob/206f86df0a325a9d853d5afb066d26cbcddd41d7/src/sqlean_driver/__init__.py (Example of a SQLite driver implementation)

Can you say more about how you're using the runs table? Is it something that you glance at occasionally, or are some of your workflows dependent on it?

We don't have anything today that depends on it specifically. We query it manually, occasionally, for debugging purposes.
As we build out our "second-generation" Meltano platform, though, we would like to better implement "reporting and analytics on our ELT workflows" - ideally we'd be able to e.g. surface in a dashboard which ELT jobs have run recently, succeeded, failed, etc.
We run Meltano in Kubernetes via Argo Workflows and we have the Argo Workflows workflow archive set up, so all Argo Workflows executions are recorded in a database today. A Meltano run corresponds to exactly one Argo Workflows run, so we still have good information available on what Meltano jobs ran when, succeeded, failed, etc.
The part I'm thinking about specifically as a potential limitation is not having the "payload" field from the meltano runs table available. It seems like it'd be useful to have that explicitly persisted - it seems to provide the value of the replication key for each stream in the run, at the start of the run.

The other thought I had is that we're moving from Postgres into Snowflake for our warehouse. System database support for Snowflake would accomplish the same goal for us (continue to leverage a persistent state backend without running a Postgres database).

We don't have anything today that depends on it specifically. We query it manually, occasionally, for debugging purposes.
As we build out our "second-generation" Meltano platform, though, we would like to better implement "reporting and analytics on our ELT workflows" - ideally we'd be able to e.g. surface in a dashboard which ELT jobs have run recently, succeeded, failed, etc.
We run Meltano in Kubernetes via Argo Workflows and we have the Argo Workflows workflow archive set up, so all Argo Workflows executions are recorded in a database today. A Meltano run corresponds to exactly one Argo Workflows run, so we still have good information available on what Meltano jobs ran when, succeeded, failed, etc.
The part I'm thinking about specifically as a potential limitation is not having the "payload" field from the meltano runs table available. It seems like it'd be useful to have that explicitly persisted - it seems to provide the value of the replication key for each stream in the run, at the start of the run.

Thanks for adding context! That makes sense. The payload field can indeed give some insight into "state evolution" of a tap and its streams, which can be valuable.

The other thought I had is that we're moving from Postgres into Snowflake for our warehouse. System database support for Snowflake would accomplish the same goal for us (continue to leverage a persistent state backend without running a Postgres database).

Yeah, that's been asked in Slack before. The database_uri is a SQLAlchemy URL, so in theory you could point it to a snowflake instance by setting it to 'snowflake://<user_login_name>:<password>@<account_name>/<database_name>/<schema_name>?warehouse=<warehouse_name>&role=<role_name>'^1.
Now, I recall that doesn't work because at least one of the migration scripts is not compatible with snowflake's sql so changes would be required there (see https://github.com/meltano/meltano/issues/6529 and https://github.com/meltano/meltano/pull/6167). Do log an issue for it if a Snowflake support for systemdb would make this transition easier for you, and of course PRs would be welcome 😄.

After some further considerations we've decided that it's actually not feasible to retire our Postgres database. We plan to continue to use it for the Meltano system database (and for some other metadata capturing, Argo Workflows archive etc).
So, from my perspective this issue could be closed.


panic

interface conversion: interface {} is nil, not *lru.entry
lru.go:108
an error will be reported when deleting the old one when adding

same thing here.
groupcache.lru is not thread safe.
we need to use the lock by ourselves.


Failed to load resource: The operation couldn’t be completed. (WebKitBlobResource error 1.) in Angular 8

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:
Operating system and its version:
PDF.js version:
Is a browser extension:

Steps to reproduce the problem:
1.
2.
What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Failed to load resource: The operation couldn’t be completed. (WebKitBlobResource error 1.) safari browser

Without all information in the template, this is nothing we can do here.


use_su_login not working as expected?

Describe the bug
See: https://github.com/Fizzadar/pyinfra/issues/412#issuecomment-685092335

This is potentially expected behaviour - at least on MacOS I can't get su -l to execut as expected when passing a command:
root$ su -l nick
nick$ echo $PYENV_ROOT
/Users/nick/.pyenv

vs
root$ su -l nick -c 'echo$PYENV_ROOT'
     # nothing output

Interestingly this works as expected on freebsd:
vagrant ssh freebsd12
Last login: Thu Sep  3 15:19:51 2020 from <IP_ADDRESS>
hi
[vagrant@freebsd12 ~]$ su -l ^C
[vagrant@freebsd12 ~]$ sudo su
[root@freebsd12 /usr/home/vagrant]# whoami
root
[root@freebsd12 /usr/home/vagrant]# su -l vagrant
hi
[vagrant@freebsd12 ~]$ logout
[root@freebsd12 /usr/home/vagrant]# su -l vagrant -c ''
hi
[root@freebsd12 /usr/home/vagrant]# su -l vagrant -c 'echo hi'
hi
hi

And with pyinfra:
$ pyinfra @vagrant/freebsd12 server.shell 'echo hello' sudo=true su_user=vagrant -vvv
...
--> Starting operation: Server/Shell ('echo hello',)
[@vagrant/freebsd12] >>> sudo -H -n su vagrant -c 'sh -c '"'"'echo hello'"'"''
[@vagrant/freebsd12] hello
    [@vagrant/freebsd12] Success

# with su login enabled

$ pyinfra @vagrant/freebsd12 server.shell 'echo hello' sudo=true su_user=vagrant use_su_login=true -vvv
...
--> Starting operation: Server/Shell ('echo hello',)
[@vagrant/freebsd12] >>> sudo -H -n su -l vagrant -c 'sh -c '"'"'echo hello'"'"''
[@vagrant/freebsd12] hi
[@vagrant/freebsd12] hello
    [@vagrant/freebsd12] Success

So I can only conclude that the MacOS variant of su does not respect the -l flag when executing a command directly. Annoying gotcha perhaps?

Based on the above - I no longer think this is a bug, but a gotcha with various systems. I think the best next step here is to add a warning when using use_su_login highlighting the potential issues.

Warning added in https://github.com/Fizzadar/pyinfra/commit/bb60661046e552c993857432c0bc8db5e58f1924 which will be released imminently in v1.1.2.