[DO NOT MERGE] MLflow Integration

Description
LlamaIndex is a supported flavor in MLlfow with the 2.15 release. This PR adds MLflow to the integration docs.
Fixes # (issue)
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No

Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

@B-Step62 and @BenWilson2 for visbility.

@michael-berk @B-Step62 I saw v2.15 of mlflow got released, is this safe to merge/wrap up?


Feature request: do git init with create command

Would be nice if tsdx could also run this git init command for me while calling tsdx create mylib.
I know there are people out there who use other version control systems, but for them, could we introduce a flag --noGitInit or something like that?

We do have some git specific references in the code as well as use of gitignore in templates and use of git hooks via Husky, so I think it's reasonable to do so and optimize for the case that is likely the vast majority of usage. Although the first ref is behind an if, gitignore is harmless, and there's #419 to install without Husky which is very highly requested.
This feature I think will run into the same issue as #419 for monorepos as well as existing repos more generally. So a PR for this should check if already inside a git repo and only perform git init if not inside one.
Per https://github.com/formium/tsdx/issues/438#issuecomment-578110253, the eventual plan is to have a command like tsdx add that allows for easier adoption in existing repos as well as partial adoption of TSDX so would probably suggest using that over the flag sprawl that would be various --no flags for every use-case. create can be reserved for an opinionated workflow. There are some trade-offs with that too (e.g. how granular is add?) so there's more to think about on that front though.

Related: https://github.com/facebook/create-react-app/issues/8280


Mass Mint

Final Mass Mint 1.0 designs

I will provide some key description here how the feature should work.
Figma contains:

Detailed interactions and logic is available Figma.
all the needed screens
interactive prototype

Please take the prototype with a grain of salt, not everything is perfect and 1:1 to final design how it should look, so in case you dont know something, dont hesitate to ask me and I will try to respond ASAP.
There are some predefined flows so not everything is 100% interactive. I will provide a short video where I will cover all the logic in the prototype so you dont have to explore it all by yourself. However feel free to explore as you wish and need as well :)
Main functionality
Onboarding

onboarding is displayed only once when
entering Mass Mint for the first time,
when entering for the 2nd time, no onboarding displayed
, user is taken straight to mass mint
user can scroll between slides by pressing Next button or Arrows
user has option to go back to onboarding from mass mint feature
button “done” is displayed on the last slide
“skip” button is displayed on every slide except the last one

Mass Mint

STEP 1: SELECT COLLECTION

user has to select a collection where he wants to mass mint NFTs
dropdown shows all collections which user has
last item in the list is "+ Create New Collection" which takes user to already existing and implemented Create Collection flow

its always displayed at the bottom

user also cannot interact with any other mass mint element, untill he selects a collection first

STEP 2: UPLOAD PICTURES

supported files: .zip
Supported art formats: BMP, GIF, JPEG, PNG, SVG, TIFF, WEBP, MP4, OGV, QUICKTIME, WEBM, GLB, FLAC, MP3, JSON
user can upload a .zip file containing the art
user can either:

tap and choose a files (tapping opens default OS finder)
drag and drop the file into the field

uploading has an animation
once uploaded, user can still make a change by pressing "Choose File" button which opens the finder again and then choose another .zip file

STEP 3: UPLOAD DESCRIPTION FILE (optional step)**

Supported files: .TXT, .CSV OR .JSON
same logic as for Upload Pictures
in case user plans to mint many NFTs, he can create a description file containing information for each NFT

in our case it would contain:

FILE NAME (e.g. awesome_art_1.jpeg)
NAME (e.g. Soldier#1)
DESCRIPTION (e.g. This is the rarest soldier of the collection)
PRICE (e.g. 200 KSM)

file needs to have correct structure to be able to connect the information to uploaded art files @yangwao we can discuss the structure and technical implementation of this to provide more information

STEP 4: OVERVIEW TABLE
Mint Button:

by opening this table Mint button becomes tappable

mint button shows how many NFTs will be minted

Table:

table shows uploaded art files
each file is displayed as a separate row
row with columns description is fixed and does not move
user can scroll in the frame with NFTs starting with#1
would be nice if the height of the table is resizable (nice to have)

List Item in Table:

Name

taken from the name of the uploaded files
in case user uploads description file, name and other information is taken from that file
if missing, “*Name Required” is displayed
whole name textfield can also be tapped to open edit panel

Description

taken from description file
if empty, display “Description missing”
whole description textfield can also be tapped to open edit panel

Price

taken from description file
if empty, display “*Price Required”
whole price textfield can also be tapped to open edit panel

States

OK - this is desired state so that the file can be minted

Name, Price and Description are filled

Incomplete

file is missing required information:

Name
Price
or Both

If some files in the table have this state, user cannot mint unless he resolves them. If he tries to mint “Incomplete mint - popup” is displayed

Description

file which is missing only description
this state does not restrict a user from minting

Operation

Edit

users can edit each file by clicking edit button
this opens a side edit panel where all changes to the image can be done

Delete

users can also delete an item from the table
confirmation popup is displayed after delete button is tapped

STEP 5: MINT

user needs to sign all the necessary transactions
when minting is finished user is redirected to the collection to which he minted the NFTs

OTHER
Edit panel

edit panel slides in from the right, check prototype
when missing required information (Name, price) textfields are red and placeholder says “*Required”
description is limited at 500 characters
save button can be tapped when changes were made and no *Required information is missing
user can always go back by tapping close button without any changes

Dialogs
There are 4 main dialogs:

Missing Required Information

this dialog is shown when there are some files with Incomplete state
user cannot mint unless he resolves the incomplete files
can be resolved by:

filling in required information
deleting incomplete files

Final Mass Mint Overview - missing description

when Description missing, it does not restrict user from minting
he is however informed that some files are missing description
shows how many NFTs will be minted

Final Mass Mint Overview - Everything OK

shows how many NFTs will be minted

Delete file

displayed when user taps delete file button
if name is missing, display file number from the table

Ref

#614

IMO  #4942 should be done before this one

Can someone look on this? 😭🥹
The current minting experience is pretty bad and till we fix it back it will take while

https://github.com/kodadot/nft-gallery/issues/4752

@kodadot/internal-dev 🙄🙏

Tasks

[ ] Onboarding #5176
[ ] Select Collection #5177
[ ] Upload Pictures #5178
[ ] Upload Description file #5179
[ ] Overview Table #5180
[ ] Edit Panel  #5181
[ ] Dialogs #5182

@kodadot/internal-dev
@prachi00
Feel free to grab some

cc
@sacha-l

🥳 Closing!
@sacha-l you can give it a shot here, and feel free open a bug issue if you spot something! 😉
https://canary.kodadot.xyz/stmn/massmint

We will make it available for the public, and we will do the silent release 👀


Added EnumDataTypeAttributeRelay.cs and EnumDataTypeAttributeRelayTes…

Added EnumDataTypeAttributeRelay.cs for suport EnumDataTypeAttribute
Added unit tests

@elasskiy I made some changes to the PR. I think the relay is still missing some tests though.
I'll try to have another look in the next few days.

@elasskiy I made some changes to the PR. I think the relay is still missing some tests though.
I'll try to have another look in the next few days.

@aivascu

I think the relay is still missing some tests though.

I can help with tests. Can you send me list of missing tests.

@aivascu

I think the relay is still missing some tests though.

I can help with tests. Can you send me list of missing tests.


Fixed 'English' to start with capital letter

@Triang3l Would you mind recreating your branch from current master? Or if you prefer, I could do that for you. (Shall I fake your authorship so you maintain credit for fixing it?)

Change already applied to master.


allow custom default width for sideframe

is this somehow already possible?
reason: opening cms page tree doesnt show further actions (not in menu, add subpage, etc.), and filer for example is very strange to use (after uploading a file, the clipboard is not visible, and the file disapeared..)
I see use cases for a small initial width, but it would be nice to have this easily customized...?

Basically it's this setting https://github.com/divio/django-cms/blob/develop/cms/static/cms/js/modules/cms.sideframe.js#L17
It's quite doable to make it customisable in some way

as I get it, it's created here:
https://github.com/divio/django-cms/blob/develop/cms/static/cms/js/modules/cms.toolbar.js#L439
and here (if it was open before reload):
https://github.com/divio/django-cms/blob/develop/cms/static/cms/js/modules/cms.toolbar.js#L83
one could add a data-width per sideframe button/item to
https://github.com/divio/django-cms/blob/develop/cms/templates/cms/toolbar/items/button_sideframe.html
and/or a default value, but I don't see how this would best get into cms.toolbar.js? Maybe with a custom Toolbar? As I don't think a django settings.CMS_TOOLBAR_DEFAULT_SIDEFRAME_WIDTH would make it... guidance welcome. and, does it need selenium tests?

Adding a generic "data" attribute to the Button class (https://github.com/divio/django-cms/blob/develop/cms/toolbar/items.py#L361) is an option for custom toolbars, for the default one, tou still need to override the current default, or remove the current buttons and add it in your toolbar code.
As of now frontend tests are not required as we're in the process of completely overhauling them

In 3.2 by default sideframe takes 80% of the screen.

nice! thx.

Am 23.09.2015 um 09:04 schrieb Vadim Sikora<EMAIL_ADDRESS>In 3.2 by default sideframe takes 80% of the screen.
—
Reply to this email directly or view it on GitHub.

Fixed in 3.2


JVM crash on ARM64

Summary

JVM inside Docker container is crashing in the C1 Compiler Thread after around 5-10 of running on AArch64 Amazon Linux 2 host
Steps to reproduce
Not sure what to provide here.  This happens on one of three containers using the same JVM version, same Spring Boot version, same everything only the class name in current compilation ask is only used on container that is crashing.  I'd be happy to provide this class privately.  Let me know what else I can provide
Expected results
JVM does not crash
Actual results
Hard crash
Triaging info
Java version:
openjdk version "11.0.10" 2021-01-19
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.10+9)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.10+9, mixed mode)
What is your operating system and platform?
Container is based on arm64v8/adoptopenjdk:11-jdk-hotspot
Host: AArch64 Amazon Linux 2 - 4.14.219-161.340.amzn2.aarch64 #1 SMP Thu Feb 4 05:54:27 UTC 2021 aarch64 aarch64 aarch64 GNU/Linux
How did you install Java?
arm64v8/adoptopenjdk:11-jdk-hotspot container comes with it
Did it work before?
Switched to Arm64
Did you test with other Java versions?
yes, java 11.0.10 openj9 works fine; only hotspot having issues
hs_err_pid7.log
replay_pid7.log

No rush on this, I'm using the OpenJ9 build currently.  This was more an FYI ticket.

Gets fix in April: https://bugs.openjdk.java.net/browse/JDK-8247766. Nightly builds should have it.
Might need a backport to 8.

Good enough for me.


Documented FIPS mode change for .NET Framework 4.8

Documented FIPS mode change for .NET Framework 4.8
//cc @bartonjs

Thanks for the review, @bartonjs. I've incorporated all of your suggestions. I've also classified this as a major change; is that correct?

I've also classified this as a major change; is that correct?

Is there a place where I can see the definitions :smile:?

Yes, @bartonjs. It's at https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/application-compatibility#net-compatibility-diagnostics. It strikes me, though, that since it shouldn't actually require modification of code or recompilation (code would have to take a dependency on the exception), it perhaps shouldn't be classified as major.

It depends... going forward (4.7.2 -> 4.8) it's Edge Case.  Almost all apps that worked will continue to work (they'd only fail if they were depending on the exception).
Going backwards (4.8 -> 4.7.2) it's... Minor?  Based on the total number of machines and the total number of apps, the amount that are in FIPS-mode and it's relevant is proportionally small.

I think that Edge Case is a good choice based on 4.7.2 -> 4.8 migration.


Update via CLI

Is it a duplicate question?
Please search in Issues without filters: https://github.com/louislam/uptime-kuma/issues?q=
Info
Uptime Kuma Version:
Using Docker?: Yes
Docker Version: 1.7.3
Node.js Version (Without Docker only):
OS: Debian Bullseye
Browser:
Hello, i haven‘t seen which database Uptime uses. Can i do for a backup of the DB the sqlite3 dbname .backup dbbackup command?

It is kuma.db.

So Sqlite or MySQL or something Else?

So Sqlite or MySQL or something Else?

Sqlite


Ignore UTRs when filtering de-novo library

Making filtration stage less strict.
We should avoid removing repeats from library, when it overlaps an UTR.
Models with repeats in UTR are valid most of the time.

tested on athalia_rosae_gca000344095v2 at me1


build.rs build script fails

When building with cross (but not with cargo) I get weird build errors when running my build.rs:
https://travis-ci.org/droundy/fac/builds/254944182
Sorry to be terse.  I'm still debugging my travis configuration, but it's clear that cross was breaking it.

The clue is in these lines: your build.rs tried to write a version.rs file. But cross mounts the path as read-only as listed in the README under caveats/gotchas.

Is there a way to generate source code that only writes to $OUT_DIR?

@droundy check rustup, it does something similar to what you are doing to get the commit hash but it writes a temporary file to $OUT_DIR, instead of to the src directory. Here's how it reads back that file into source code.

@droundy were you able to get this working? Would this be okay to close?

Yes, I did get this working.


feat: algolia search support

This is definitely a feat rather than a chore.

Aha, +1 !!! my reaction is the same!!! in https://github.com/vesoft-inc/nebula-docs-cn/pull/2237


Pawn TextMate-compatible grammar (take 2)

Replaces #2512

@ziggi sorry it's taken a while to get this merged. This change will be live on GitHub when we create a new release of Linguist (sometime in the next couple of weeks).

@arfon very nice, thank you!


Enhance searches on  place of enumeration.

Currently searches are based on the first two levels of the enumeration heirachy (county and place).  This has some problems.  In the Channel islands, for example, the independent country of Sark is a third level place, and does not appear (see #273). The names for second level places change. A free text search for third level place names (FC1) does not indicate whether the spelling entered reflects the spelling used.  Search needs developing in two ways (a) third level searching for enumerative places, and (b) searching for place through places not necessarily mentioned in the cencus

To note (for no action at this point) In searching:
FreeBMD allows searching by their equivalents of 2 and 3 (all the data they have)  FreeREG allows searching by 2 and 7 (all the data they have).
FreeCEN1 allows searching by 2, or 4, or a free text search for 7
FreeCE2  allows searching by 2 and then by , but allocates each 4 to a level 2 which may be 'wrong' as a level 3 can cross county boundaries..

I am not sure why this was closed as #960 only concerns London, not similar situations in other parts of the UK (eg Manchester and Liverpool).   And neither would resolve the Sark issue / general need for more nuanced searching (i.e. at level 7, too).

This story is still open. It was 960 that was closed.


colour of text not maintained after opening in inkscape

The text colour is as programmed after the svg export and opening the download file and displaying in Chrome. However if I then open the sag exported file in inkscape everything is fine except the text colour is set to black. Line colours etc work fine though.
I suspect it might be something to do with the font - I am using Arial??

Hi @wrcstewart
Can you provide us with a sample?
Thanks!


Do YAML comments need a space after the #?

This is working:
videoediting:
  - l1
  - l2
# putOnTop:
#  - Door: ZuCoffee

Data I am getting: { videoediting : [ "l1","l2" ] }

This is not working:
videoediting:
  - l1
  - l2
#putOnTop:
#  - Door: ZuCoffee

Data I am getting: { videoediting : [ "l1","l2" ], #putOnTop : undefined }

Do YAML comments need to have a space after #?

All the examples I've seen use a space, but the spec appears to require that comments are only preceeded by whitespace. But that doesn't make sense either! I will need to poke at a reference implementation

k, checked a few other implementations and it seems # varies a bit 🙄 I'll implement the requested change when I can as it's the more permissive option.

Fixed in 70cd9e141355c87fea3e17e0919b3ece1fd11090 and available in https://github.com/JujuAdams/SNAP/releases/tag/5.1.1
Thank you!


Get the new replay list data

With the newest update Epic added replayed. I pretty sure the replays themself only can be viewed inside the client.
But the replays have som nice stats with them (win / kills and stuff)

I will have look later tonight. Check if any new endpoints exist for this

I will check it this WE. If you or @TheMaster, have the new endpoint you can create a pull request. I will add it ;).

Sure I'm at work at the moment so i can't do it now. Will try and check later today.

Just had a quick look for new endpoints for stats/replays. Couldn't find anything new or useful :(

Replay data is collected while playing and stored locally.

Store Locally, impossible to get replay from other account so issue closed.


fix(spring): add javadoc comments for configurable properties

This PR is a follow-up on https://github.com/GoogleCloudPlatform/spring-cloud-gcp/pull/1364#issue-1468601100, and adds Javadoc comments to configurable properties.
Todos:

[ ] Once 1143 is merged, rebase this branch onto autoconfig-gen-draft-2 and update base branch of PR

Rebasing on top of #1143, since previous approach for retry properties comments is no longer applicable

Managed to get this PR stuck in rebasing mess...closing and trying again in https://github.com/googleapis/gapic-generator-java/pull/1171.


Option to not be @'d in the comments

Is your feature request related to a problem
I have a lot of Terraform matrix jobs and when they run in a PR I get @'d in the comments... a lot.
Describe the solution you'd like
It'd be awesome if I could disable @'ing in the action, so I don't get spammed with notifications when my PRs run.
Describe alternatives you've considered
Turning my phone off. 😆

Thanks @jagregory, this issue/request is way too relatable!
Trying to figure how best to address it:

Simplest would be to add something like a "mention-author" option to toggle @'ing. But that's yet another input parameter to add to the list.
Alternatively, add an option to specify the line like, "By  at  ()". BUT that's still another input parameter, and way more complex to interpolate!
Got some other ideas?

As an aside, would you mind sharing a little more about your experience with this Action? External feedback is rare, especially from matrix users, so keen to know how you're getting along.

I have one pipeline which can sometimes trigger up-to 40 matrixed terraform jobs 😅
I was thinking the first option would be the simplest, just template the inclusion/exclusion of the @ in the message body so the author doesn't get tagged. Anything more than that feels like it would get complicated quite quickly.
As for feedback, it's been a very helpful action and I tried a couple of alternatives but found this one the easiest to use and produces the most useful output. I particularly like the short summary of changes which is great for an at-a-glance look, and the inclusion of a diff as well so I don't have to pour over the full plan to understand what's changing. Specifically for matrix use: I don't love that there's a comment created per job but I understand why that is. If there was a way to "fan-in" after a matrix job to post a single summary comment that would be amazing, but I think that's a lot of complexity and might not be worth the effort.
Thinking out loud (and back to the original issue) perhaps an option to only tag the author in a comment when a diff has changes? That would be probably be more useful than just disabling the tagging entirely. Or possibly a variation on #361 but more about posting comments than uploading plans.

Happy to share this has been shipped with v13.0.0, where your contribution has been credited!
Please consider ⭐ this project, if you find it useful.

For your original request, you can either set tag-actor: on-change to only be tagged upon diff/error, or never to prevent tagging altogether (default is always).
For the follow-up request, you can either set comment-pr: on-change to only add PR comment upon diff/error, or never to avoid commenting altogether (default is always).
While matrix summary is currently out-of-scope, I wonder if the 1-line summary in the PR status checks are of use to you (screenshot attached), or even the workflow summary page?


Add key to rendered components, so React doesn't optimize

This is an interesting fix, where we're now passing a key to the rendered components, so that React doesn't try to optimize and recognizes that the components are different. This is an issue for having multiple Graph components for example, where if there's no key, React will try to optimize and will use the same state for both Graphs. This is what's happening in dcc #379, and this PR will fix that.
Link to the React docs: https://reactjs.org/docs/lists-and-keys.html#keys

Can we add an integration test?

@chriddyp I didn't add any integration tests here because these are React internals - what should it test? Should it test that the React component's constructor() is called for each new Graph component? Or test that React passes a key prop correctly to the components? I guess I can add a test that tests if click data is being passed through to a callback, when there's multiple Graph components, but that feels more like a DCC test to me.

I guess I can add a test that tests if click data is being passed through to a callback, when there's multiple Graph components, but that feels more like a DCC test to me.

That's what I'd recommend testing. Basically, we have an example that is broken right now (tabs in a graph) because of dash-renderer. This PR proposes that it fixes this example, so we should add a test that mimics that example to "prove" that it indeed fixes the issue.

Or test that React passes a key prop correctly to the components?

This in and of itself isn't important for me to test as we have to infer that adding a key will prevent something from using the same instance which will prevent odd event handling bug code like this. So, I'd rather test a step further back with the actual tabs and graphs.

Could we reorder the commits so that the test is first? That way we can make sure that the test is failing on CircleCI and then is subsequently fixed with the next commit. Once that it is done, 💃

Since this is taking a little longer to get through review, could we create a pre-release so that any community members or customers that are affected by this bug can try out the prerelease?

Could we reorder the commits so that the test is first? That way we can make sure that the test is failing on CircleCI and then is subsequently fixed with the next commit.

I agree we should see this test failing without the fix before concluding the fix is correct -- the test is very remote from the code fix and we have to infer a lot. Barring reordering the commits to get the desired failure --> success chronology, just revert the fix (it's just one line after all) for one build and add it back.
In the future we should strive to make the test failure / success more explicit in our PRs.


Design optimised out?

@bleuintegral please check your design. The final GDS only has 8 cells that do any work:

might be worth adding a test.
See https://github.com/TinyTapeout/tt03-verilog-demo for an example

I did build a testbench and a test.py that runs through a handful of test cases, and they all pass. This is a very simple design, just doing matrix multiplication on 2x2 matricies of 1s and 0s. That really comes down to just some multiplication and addition, which with 1s and 0s is probably being simplified down to just combinational logic. I can add some additional test cases, but if they continue to pass, is there anything else I need to do to verify that the design isn't optimized out?


Introduce BucketMigrator

Will be used in #975

Neat idea. Is the plan to transition the changeTrieNodeEncoding to be a BucketMigrator in a future PR?

We could do that, but I am more interested in writing new ones with this:)


Fix/under over flows

Changelog

Fixes #20
Updates parseWei to ensure underflows don't occur, by utilizing evm-bn package
Updates all classes with better getters
Adds display getter
Removes parsed getter from percentage, and wei
Adds tests for all getters
Updates Percentage with more explicit natspec + getters

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


dash.wav and dot.wav

The CMake file and main.cpp make references to dash.wav and dot.wav.  But these files aren't included in the repo.

Yes. I used files from my mac that I do not have the rights to. Just pick any short notification wav and name it dash.wav and dot.wav

I'm on Mac too, where can I find these 2 files?

As Kirk replied to my query you will need to substitute your own files.
One way is to copy over two short audio files and call one dash.wav and the other dot.wav.
For me,  the audio wasn't important, so I just commented out the references entirely.

Yeah, that was what I did too. I misunderstood Kirk, I though he meant these 2 files were from macOS, so I asked.


Add missing res folder in INSTALLATION.md

Does any other open PR do the same thing?

(please answer here)
No, not right now in opened PRs.
What issue is this PR fixing?
(please link the issue here)
Updating troubleshooting information.
How did you test this PR?

(please answer here)
Just updated an installation.md file to prevent confusion.

😻


Change spatial traversal to fix perf regression from #897

Fix perf regression from #897
On Saturn with the standard HACC 37M problem (eps=0.042), the traversal time goes down (compared to master (fbd25845)):
minpts = 2 (query+cluster timer):
FDBSCAN: 0.147 -> 0.114
FDBSCAN-DenseBox: 0.154 -> 0.126
minpts = 10
FDBSCAN
neigh: 0.058 -> 0.057
query: 0.137 -> 0.115
FDBSCAN-DenseBox
neigh: 0.057 -> 0.057
query: 0.161 -> 0.129
This patch should restore most of the lost performance due to APIv2 refactoring:

I've run the bvh benchmark on Saturn for Serial and CUDA, and surprisingly, it showed no difference in times. Maybe that's why we did not catch the regression.
master_fbd25845.json
pr976_e61159fb.json

What is "Saturn" again?

Nice. Just less branching effectively?

It's a bit of mystery for me. Possibly. I'm pretty sure I played around with different traversals in #897 but could not find a good one. It could be that Power9 was regressing or something.

What is "Saturn" again?

One of the University of Oregon OACISS center systems.

HIP build failure was "No space left on device"
I am ok to merge as is.

All pass (HIP did not run).


Refs #23919 -- Used "raise from" instead of cause in reraising backend-specific database exceptions.

@timgraham Thanks for review. I added PostgreSQL specific test. That is also tested in #8396.

Looks okay but the test should be in a separate commit that precedes the refactor to make it clear that the code change isn't causing any behavior change.


azure-cli returns exit code 0 (success) and no warning when deleting non-existing VM

azure-cli returns exit code 0 (success) and no warning when deleting non-existing VM.
Tested on: Cloud Shell - azure-cli (2.0.30) / Ubuntu - azure-cli (2.0.29)

If you try to delete a non-existing VM in an existing group, it won't print any warning and return 0

johan@Azure:~$ az vm delete --resource-group cloudera_copy --name xyz --output json --yes; echo $?
0

Shouldn't it return a warning and exit code 1 ? Such as:
Virtual machine 'xyz' could not be found
1

This is the intended behavior, in the same way that show commands in the CLI usually return nothing and exit code 0 when the user tries to show a  non-existent thing.


Devs SBAT reference design doc to see historical returns on locked gold

Expected Behavior
Devs SBAT have a plan to see historical returns on locked gold for different lockup periods.
Could involve extending notification service, as with the gold/dollar exchange rate querying, or separating this into another polling package
Current Behavior
No plan

Because of the new Proof of Stake proposal only offering rewards to Celo locked gold holders who vote for a validator group, it is unclear whether locked gold will be added to the app, as it would require selecting a validator group for app users. Closing this for now and will reopen if we end up needing this after all.
cc @nityas


Add pre-commit

pre-commit is a tool for assessing code quality.
It uses git hooks to run checkups of high code quality, so many projects uses it.
I propose to add very basic setup of pre-commit to check simple things (e.g. consistent line endings, line endings at the end of files, etc)

@alexandrebarachant since #93 is not resolved, my hope for review is melting =(

@alexandrebarachant since #93 is not resolved, my hope for review is melting =(

thx @v-goncharenko


biological interpretation of occupancy tracks

Hi Alicia,
I am having troubles interpreting some of the occupancy data in a biological context. I was hoping you could help me with it.

This is a plotProfile (generated using deeptools) from the nucleosome occupancy (.occ.bedgraph) tracks for four different conditions.
First of all, I was going through your previous answers, and it seems like it's okay to compare occupancy profiles between conditions as the signal is normalized to read depth, provided the signal:noise ratio is comparable, right?
Specific to the experiment:
It's sort of a time course experiment, where condition 1 is the control, and I am looking at two different genomic regions (region 1 and region 2).
Here's my interpretation of the data:
In condition 2 the nucleosome occupancy is re-distributed, and generally occupancy around the +/-1.5kb of the TSS is higher than in condition 1...there seems to be a non-specific re-distribution, leading to loss in resolution of occupancy profiles.
The profiles seems to re-estabilish in conditions 3, and 4 (4 more so than 3).
Also, between the two regions compared, region 1 seems to re-estabilish occupancy profiles sooner, i.e., in condition 3, compared to region 2
I'm not sure if this is reasonable interpretation of the data...any thoughts would be helpful!
Thanks!

hi @bhatis2,
In general terms, the occupancy profile should be okay to compare between conditions provided signal:noise ratio is comparable.  If interested in a particular region or set of regions, I highly recommend looking at "control" regions to see whether changes are specific to region of interest, e.g. if interested in a particular tss, looking at random other tss.
The occupancy is also not very reliable if there are insufficient reads, so if comparing specific regions I'd also look to see that there isn't a drastic reduction in reads such than in one condition there are barely and reads. Taking a look at the upper and low bound nucleosome occupancy tracks can also be useful for that.
For your specific plots, I think your  interpretation seems reasonable -- that in condition 2 there is not as strong positioning of the +1 and -1 nucleosome.


 Also will make things much easier to do via docker since we can provide json files or env args.

NOTE: This is a breaking change - current users will have to create a tribeca.json file if they wish to upgrade. Sorry, but I need to rip this band-aid off now.

Removes ZMQ dependency which was screwing up some users

Removes OkCoin since it's most likely not working.

Also update README

Not


Update isv-overview.md

Added recently certified solutions by BTS solutions for Contact Center and Attendant Console.
Sorted the list alphabetically for an easier search/view online.

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

@yhezroni Could you please sign CLA?


linting core app

IN PROGRESS
What types of exceptions are we actually trying to catch in these cases?
core/forms.py
  Line: 30
    pylint: bare-except / No exception type(s) specified (col 8)

core/management/commands/reindex_elasticsearch.py
  Line: 43
    pylint: bare-except / No exception type(s) specified (col 12)

core/templatetags/core_tags.py
  Line: 66
    pylint: lost-exception / return statement in finally block may swallow exception (col 12)

Any opinions on how to handle these cases?
core/resolver.py
  Line: 53
    pylint: unused-argument / Unused argument 'private' (col 41)
  Line: 122
    pylint: unused-argument / Unused argument 'private' (col 38)

core/utils/tasks/public.py
  Line: 27
    pylint: unused-argument / Unused argument 'request' (col 30)
    pylint: unused-argument / Unused argument 'state' (col 39)
    pylint: unused-argument / Unused argument 'context' (col 46)

If anyone wants to pick this PR up, feel free. @smcoll said he would try to return to this later if not.

Moved this to #2911 to finish up work on a different branch.


[MBL-698] [Regression Fix] Gate Segment "Identify" Event

📲 What
During QA we were seeing the identify call in segment after denying consent. This call should be gated just like all other segment calls that include sensitive user data.
🤔 Why
If someone doesn't want us to track their data, then we shouldn't.
🛠 How
This call happens in KSRAnalytics -> public func identify(newUser: User?) on line: 528 and can be gates like the other calls…
guard let _ = self.advertisingId else { return }
✅ Acceptance criteria

[x] If a user opts out of tracking, or hasn't explicitly allowed tracking, then no Segment events that contain user data should be sent out. This includes the initial "identify" event on log in

SwiftFormat found issues:

File
Rules

Library/Tracking/KSRAnalytics.swift:533:1
warning: (indent) Indent code in accordance with the scope level.

Library/Tracking/KSRAnalyticsTests.swift:1726:1
warning: (indent) Indent code in accordance with the scope level.

Library/Tracking/KSRAnalyticsTests.swift:1730:1
warning: (indent) Indent code in accordance with the scope level.

Library/Tracking/KSRAnalyticsTests.swift:1731:1
warning: (wrap) Wrap lines that exceed the specified maximum width.

Library/Tracking/KSRAnalyticsTests.swift:1731:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/Tracking/KSRAnalyticsTests.swift:1732:1
warning: (indent) Indent code in accordance with the scope level.

Library/Tracking/KSRAnalyticsTests.swift:1732:1
warning: (wrapArguments) Align wrapped function arguments or collection elements.

Library/Tracking/KSRAnalyticsTests.swift:1735:1
warning: (indent) Indent code in accordance with the scope level.

  Generated by :no_entry_sign: Danger


[muparser] update to 2.3.4

Describe the pull request

What does your PR fix?
Update muparser to 2.3.4

Which triplets are supported/not supported? Have you updated the CI baseline?
!uwp, No

Does your PR follow the maintainer guide?
I think so

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Thank you!


Review Rate Base FERC1 Table

Okay @jrea-rmi ! Once we merge in PR #3450, I believe we now have much cleaner detailed tables and rate base table. Can you give these a review as best you can and let me know if you find anything that seems like a red flag? I've attached the two core graphs below and here are pickled dfs of the main inputs and the rate base table itself:

out_ferc1__yearly_rate_base.pkl.zip
out_ferc1__detailed_balance_sheet_assets.pkl.zip
out_ferc1__detailed_balance_sheet_liabilities.pkl.zip
out_ferc1__detailed_tags.pkl.zip
forest_as_table_core_ferc1__yearly_balance_sheet_assets_sched110.pkl.zip
forest_as_table_core_ferc1__yearly_income_statements_sched114.pkl.zip

Rate base table: by input table & utility

I'd be happy to give you a tour of these or set you up with anything you need to best get a sense of these tables. Even high level gut reactions are vert helpful!
Once we all feel good about the contents we can move on #3310
Compiled Found Weird Items:

2004: both the assets and liabilities table are abnormally low that year.

This is so cool! Really exciting to have this get close to being available to everyone through PUDL!
Here are notes on what I've found so far:

accumulated depreciation by function has mostly zero values in 2021, which then get covered up with accumulated_depreciation_subdimension_correction. But the real data values exist in the raw xbrl table, and it'd be good to fix that rather than using corrections.
the accumulated depreciation table differentiates hydraulic_production_conventional and hydraulic_production_pumped_storage. But the plant in service table only gives original cost for a combined hydraulic_production. I would make tags_aggregatable_plant_function="hydraulic_production" for all of these. The plant_function field still differentiates the components of accumulated depreciation, but then it's easier to aggregate to a consistent technology category.
the records from the plant in service table are missing plant_function
xbrl_factoid accumulated_deferred_income_taxes should have tags_rate_base_category==net_ADIT
xbrl_factoid other_regulatory_assets should have tags_rate_base_category==net_regulatory_assets
records with tags_rate_base_category=="other_electric_plant" but utility_type=="gas" or "common" or "other" should have a different tag. Maybe other_gas_plant and other_plant? Maybe the common part should be split into components with an estimate.

The records with utility_type ="total" and tags_aggregatable_utility_type=NA have values that correspond to electric, gas, and other utility types (water, etc.). In RMI's use of this table, we would take the summary of utility plant table, calculate the fraction of assets that are electric/gas/other, and multiply all of the values for these records by that fraction to estimate the value that's included in electric/gas/other rate base. Without doing this conversion, users can only get either total rate base across all utility functions, or a partial rate base  Would you consider adding this estimated conversion of total lines into their electric/gas/other components, so users can evaluate electric/gas/other rate base?

hey @jrea-rmi  thank you so much for these notes! Many of these have pretty straightforward fixes that I'll jump on them now.

[ ] 2021 accumulated_depreciation:

[ ] identify fix: #3309
[ ] fix problem

[ ] double hydro tags

[x] identify fix: this will be straightforward bc the manually compiled tags override whatever comes from the data. I can go and add  hydraulic_production for the accumulated depreciation table.
[ ] fix

[ ] plant_function missing from plant in service table

[ ] identify fix
[ ] fix

[ ] accumulated_deferred_income_taxes rate base tag

[x] identify fix: manually add this tag!
[ ] fix

[ ] other_regulatory_assets should have tags_rate_base_category==net_regulatory_assets

[x] identify fix: manually add this tag!
[ ] fix

[ ] tags_rate_base_category=="other_electric_plant" but utility_type=="gas" or "common" or "other"

[ ] identify fix: we set up all of the tags either use any of the
[ ] fix

[ ] total/partial records

[ ] identify fix: I think breaking down the total and/or partial records out into would be a great improvement! I'd love to chat with you about how y'all did this and scope it a bit before jumping in (was this ratio developed for every year/utility or for every xbrl_factoid from little slice of the detailed tree?)
[ ] fix (i'll probably want to break this out into its own issue and tackle it independently of these simpler fixes)

the records from the plant in service table are missing plant_function

@jrea-rmi do you need the main plant_function column to be filled? We added tags_aggregatable_plant_function for this table from y'all's tags so that column contains values. Is that enough or do you need the values in the main plant_function column - these main dimension columns come from the transformed table while the tag columns are a mix of new tags, overridden dimensions and filled in values.  you probably know this but the info about the plant_function is mostly embedded in the xbrl_factoid values but its not super consistent how it is reported across the table so we never extracted it systematically.

records with tags_rate_base_category=="other_electric_plant" but utility_type=="gas" or "common" or "other" should have a different tag. Maybe other_gas_plant and other_plant? Maybe the common part should be split into components with an estimate.

one more question for you @jrea-rmi ! what if this category was just other_plant? and you could distinguish between the utility types by the utility_type or tags_aggregatable_utility_type columns?

the records from the plant in service table are missing plant_function

@jrea-rmi do you need the main plant_function column to be filled? We added tags_aggregatable_plant_function for this table from y'all's tags and that column contains values. Is that enough or do you need the values in the main plant_function column - these main dimension columns come from the transformed table while the tag columns are a mix of new tags, overridden dimensions and filled in values. you probably know this but the info about the plant_function is mostly embedded in the xbrl_factoid values but its not super consistent how it is reported across the table so we never extracted it systematically.

tags_aggregatable_plant_function is fine for us. It may be confusing to users, and I could see an argument for keeping that column and dropping the main plant_function column for simplicity, but I'll know how to use it so fine for me personally.

records with tags_rate_base_category=="other_electric_plant" but utility_type=="gas" or "common" or "other" should have a different tag. Maybe other_gas_plant and other_plant? Maybe the common part should be split into components with an estimate.

one more question for you @jrea-rmi ! what if this category was just other_plant? and you could distinguish between the utility types by the utility_type or tags_aggregatable_utility_type columns?
but I certainly can just add a specific rate base category per utility_type if that is preferable

both options work for me. Maybe the single category is best; making an other_other_plant is v confusing. Also noting possible confusion with other_production_plant category, but I don't know how to get around that.
So yes, I like your idea.


ENH: add input validation for dendrogram and a test.

Reference issue
To fix #11492
What does this implement/fix?
add input validation of Z and labels dimension consistency for dendrogram and a test.

Thanks @AtsushiSakai
There's a PEP8 CI failure:
 pycodestyle scipy benchmarks/benchmarks
scipy/cluster/hierarchy.py:3276:18: E271 multiple spaces after keyword
1       E271 multiple spaces after keyword

@rgommers Thank you for your reviewing. I fixed these based on your comments. PTAL.

@EverLookNeverSee, those failure are a known problem.  The Azure tests have been failing like that for the last day or so: https://mail.python.org/pipermail/scipy-dev/2020-March/024106.html
Tyler pinged a couple github staff (https://github.com/scipy/scipy/pull/11748#issuecomment-605728037), and got a response a little while ago.

@EverLookNeverSee, those failures are a known problem. The Azure tests have been failing like that for the last day or so: https://mail.python.org/pipermail/scipy-dev/2020-March/024106.html
Tyler pinged a couple github staff (#11748 (comment)), and got a response a little while ago.

@WarrenWeckesser Thank you for this tip.

LGTM now, merged. Thanks @AtsushiSakai!


Add a high level datetime object to handle conversion from SCET/OBT to UTC

Codecov Report

Merging #41 (312b05d) into master (151167f) will decrease coverage by 0.04%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
- Coverage   92.39%   92.34%   -0.05%
==========================================
  Files          25       26       +1
  Lines        1301     1320      +19
==========================================
+ Hits         1202     1219      +17
- Misses         99      101       +2

Impacted Files
Coverage Δ

stixcore/datetime/datetime.py
89.47% <89.47%> (ø)

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 151167f...312b05d. Read the comment docs.

Codecov Report

Merging #41 (312b05d) into master (151167f) will decrease coverage by 0.04%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
- Coverage   92.39%   92.34%   -0.05%
==========================================
  Files          25       26       +1
  Lines        1301     1320      +19
==========================================
+ Hits         1202     1219      +17
- Misses         99      101       +2

Impacted Files
Coverage Δ

stixcore/datetime/datetime.py
89.47% <89.47%> (ø)

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 151167f...312b05d. Read the comment docs.


coin3d: unstable-2022-07-27 -> 4.0.2

Description of changes

Update to 4.0.2
Fix darwin build

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4000

Friendly ping to @gebner and @viric on this 🙂

@JohnRTitor you've been helpful in reviewing and merging a few other of my PRs recently, maybe you have some time and interest to devote your expertise on this one?

Thanks for your help on this, @JohnRTitor, very much appreciated! 😃


Add plugin reference flutter_map_marker_cluster in readme

Hi, i created plugin flutter_map_marker_cluster.
I think that it will be usefull for many people, so i would add reference.

splendid!


Trackpad support in Mac/OSX

🚀 Feature
Have the option to use gestures on the trackpad for interacting with the canvas.
Motivation
Do I need to explain why this is cool?
Pitch
Sparkles in the eyes.
Alternatives
Be lame and boring.
How to do this:
This is how one can access the trackpad from python:
http://blog.sendapatch.se/2009/november/macbook-multitouch-in-python.html
I have not tried, but I have done this in the past in Java.
It is doable for sure. Basic gestures would be translate,
pinch and zoom.
Other functions for which we could have gestures:

Zoom back to fit all...
1-1 pixel perfect.
'rotate' layer list (some sort of quick swap) to quickly
alternate visualising different layers...

Vispy references "Touch Events" in some of their docs, see here, but I'm not sure how much has been implemented there, or if it makes sense to implement this in parallel.
I do worry slightly about having too much functionality that would not be cross-platform / available to people using a desktop and mouse, but it is still a good idea to explore

We are going to investigate support for wacom tablets for image annotation, as requested in #190. Will be interesting to see if there is any support from the Stylus Event in vispy

@kevinyamauchi I have gotten a wacom tablet and am trying controlling the paintbrush in the labels with it. Integration was absolutely no problem, we didn't need to do anything with tablet events etc. to get the basic functionality working. If we want to get the most out of the tablet though I think we will want to map buttons on the tablet to certain events like changing which mode you are in or controlling the pan / zoom. Hopefully when we work on #210 we can also make customization like this possible. I look forward to giving you a demo!

If we want to extract custom button presses from the tablets then we will need to use the QTabletEvent.

Hooking into the touch event seem to be relatively straitforward, update with the following:
class VispyCanvas(SceneCanvas):
    ...
    self.events.touch.connect(self._process_touch_event)

    def _process_touch_event(self, event):

        if event.type == 'pinch':
            print(f"{event.type=}")
            print(f"{event.pos=}")
            print(f"{event.last_pos=}")
            print(f"{event.scale=}")
            print(f"{event.last_scale=}")
            print(f"{event.rotation=}")
        elif event.type == 'begin':
            pass # store state and start whatever drag/zoom you like
        elif event.type == 'end':
            pass # commit change.

or even just this if no processing required:
--- a/napari/_qt/qt_viewer.py
+++ b/napari/_qt/qt_viewer.py
@@ -233,6 +233,10 @@ class QtViewer(QSplitter):
         self.canvas.connect(self.on_mouse_wheel)
         self.canvas.connect(self.on_draw)
         self.canvas.connect(self.on_resize)
+        self.canvas.connect(self.on_touch)
+
+    def on_touch(self, evt):
+        print('touch event', evt)

We likely want to ask vizpy to add totalScaleFactor() and totalRotationAngle() otherwise we'll have to accumulate the deltas between event and there will be precision loss as there are many event with small changes. Maybe even let us access the raw event if they can....

Opened https://github.com/vispy/vispy/issues/1964

Was this closed by https://github.com/vispy/vispy/pull/1968?

Was this closed by https://github.com/vispy/vispy/pull/1968?

@Carreau ?

FWIW As far as I can tell on my Mac (12.6) trackpad gestures don't do anything.

Just to update this, vispy now has support for native gestures (https://github.com/vispy/vispy/pull/2456) thanks to @aganders3
Pinch-to-zoom works in vispy perfectly for me, but not yet in napari.

I need to dig a bit deeper. The key / mouse events were not touched for the reason that this would be done at a later stage.

It works with pre-https://github.com/napari/napari/pull/5701 napari. Adding "gesture_zoom" to the cases in this function will re-enable it but I've been meaning to find time to think about whether that's the right solution. I'm not sure simply adding it there would allow for e.g. binding scroll and pinch to different actions for example.

Thanks for checking @aganders3 and sorry for the false accusation @melonora !
@ksofiyuk do you have any thoughts on this?

It might be okay to enable it that way for now. It's not really part of the API so might not be a breaking change to alter the behavior later or make it configurable.

I am trying to do annotations with our homemade plugin and it is lagging as hell when I draw labels. I don't have this problem when using the mouse. I am running Napari on Ubuntu and will soon try on Windows to see if it's better (128Gb RAM, 3080 RTX, 16 cores CPU).

@brisvag seems like #5710 was the one I was looking for, tried my luck here since it was related to a Wacom tablet. I will try the main brain ASAP and report here if that solves the problem. Thanks a lot!

It solved the problem, thanks again @brisvag!

Nice, no problem :) We're working on the next release, so hopefully that should be released in the coming weeks!

BTW, if you do end up testing Wacom on Windows, do also let us know how that goes!

Can this be closed?

Definitely!
I tried on window (you need to install the driver) and it works fine. The good part is that using the drive it's possible to map the pen buttons to whatever you want and hence create custom workflows.

@psobolewskiPhD are you interested in the rotate gesture for 2D or 3D or both?
I think implementing for 3D (we use the vispy AcballCamera) might not be too hard.
For 2D we use the PanZoomCamera, which does not support rotation. This would probably mean creating a new PamZoomRotateCamera or something (probably not too hard either). We probably don't want this to only work with trackpads (there are even platform limitations, from what I remember of implementing this) so we'd probably need to come up with another way to control the rotation as well.

I've not thought about it that deeply, so my initial thoughts were about 2D. In 3D rotation is pretty easy with drags...
In terms of other controls, I think some sort of   would be the most natural for rotation of canvas/camera?

Oh yeah, good point. I think gesture rotation could be other unexpected as a new feature.

Just one question, perhaps better to check what is not supported and make an issue for what is not supported as tasks?

Here's the macOS gestures, just as a starting point:
https://support.apple.com/en-us/102482
Going row-by-row, L->R:

[x] tap and 2-finger tap to click/right-click work great
[ ] 2-finger double tap ("smart zoom") doesn't do anything

could make this trigger the viewer home button to reset zoom

[x] 2-finger swipe up/down triggers scroll wheel (zoom at the moment)
[x] pinch-to-zoom works great on my MacBook pro.
[ ] 2-finger rotation doesn't do anything

what would rotation gesture do? In 2D would need to activate the layer transform and rotate the layer. This could be useful?
in 3D it could indeed rotate, but that's easy enough with just drag gesture.

[ ] 2-finger horizontal swipe (macOS default scroll left/right, switch pages) doesn't do anything

maybe this could be show-hide other layers?

[x] 3-finger drag also works without a hitch
[ ] 3-finger tap doesn't do anything

maybe this could mode-switch to selection mode?

The others all trigger macOS specific functions and I for one would not want to have them over-written.

The 2-finger tap, "smart zoom", is not implemented in vispy, but I think could be added here:
https://github.com/vispy/vispy/blob/85c4bf944d4f38acc67c215548d3701adb8d9772/vispy/app/backends/_qt.py#L560


⚠ GitLab Server has degraded performance

In d543f96, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6882 ms

Resolved: GitLab Server performance has improved in aae585c.


Docs: replace right/left with logical properties for RTL-supported examples

See /class=.*(right|left)/ matches in this repo. Changing this will make it easier to use the doc examples in rtl projects. Also, it'll make it easier to make an rtl version of the docs. Simply using logical properties solves most problems.

https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_logical_properties_and_values/Floating_and_positioning#mapped_properties_and_values
https://tailwindcss.com/docs/top-right-bottom-left#using-logical-properties

Eg for the first result (of the search linked above), replacing left-1/2 with start-1/2 is all that's needed.

Community contributions are desired for this one, specifically someone who is confident in RTL & LTR to verify everything looks golden in both!

Closing as we don't have time to tackle this ourselves at the moment. If someone submits a PR we can reopen this.


Remove the length validation

Revert #10
Because I tested the original library and the following works,
const id = init({ length: 150 });
console.log(id());
// p0m6ycspza37iwloxpn5zw0nqtbkbcycha5mm6jlqzsu33qrlh84kmkvi41qt2w5khcddvb7a50wsczoy1o795rs75b3ifld3u

So

this does not match with the original spec.
since we are returning error, user cannot use this inline, they will have to define it first and then pass

myFunction(CreateId()) // this won't work

id, err := CreateId()
myFunction(id) // only this will work. This is cumbersome and for most cases, not required.

Added panic instead of error because the gain from validation was too small as compared to usability issue


Parley cannot compute caret geometry if layout text is empty

The following screen capture shows the issue. This is using a PlainEditor and drawing the selection/caret as in the vello_editor example. Note that there is initially no caret even though the input is focssed. The caret shows once some text has been typed in the field and disappears again when the text is deleted.
https://github.com/user-attachments/assets/a3881822-e5fc-48c9-aff9-5d25b6481bbf

Yeah, this is similar I think to the issue of caret position downstream of a hard line break.

This is actually because the layout is empty therefore we don’t do any font selection and have no metrics from which to generate cursor geometry. My upcoming code has a hack to address this by forcing layout of a single space (which used to be there but was removed at some point). There’s definitely a better way to do this but I plan to defer that to the larger refactor.

Ah yes, I removed that in #67 because, well, it seemed like a hack (and improvements in #67 meant that it would no longer panic when the string was empty). I guess the space hack will work but we'll need to keep track that we're in that "mode" so that we don't allow the space to be selected, etc.
There is actually another "mode" that I want which could probably work similarly: placeholder text. Which I am imagining working by having a second text string and set of styles that would be used as a fallback when the main string is empty.

Fixed by #170


Is there code folding option in Rich text editor ?

Is there an option for code folding in rich text editor. ? If its already there be a demo program to explain it ?

Duplicate of #107

Closing as this is a duplicate issue.


Make publishing to maven optional

Add an option to disable maven publishing. Also make the URL configurable (with a default to GTNH)

Yes it would be nice.
It would be nice to add an authorization configuration for publishing to Maven (preferably take it from environment variables).
Currently need to edit build.gradle to achieve this...

Solved in a recent update via gradle.properties


feat(tree): Add material styled tree

@tinayuangao rebase?

looks like it has been rebased onto master instead of onto tree

Rebased. Please take a look. Thanks!

Could you add a material tree demo to the demo app? It's kind of hard to reason about the CSS without being able to see a demo

Updated the document. Please take a look. Thanks!


Missing requirement association between epics and tasks

When testing the full migration flow we found that the requirement is not always set between tasks and requirements.
Specifically it seems like there is an issue when the target requirement is an epic, or maybe when it is more than one step away in the hierarchy.
I will investigate this and see if I can come up with a solution.

I have managed to reproduce the error. It seems like Epic link is a custom field in Jira for tasks, which makes it so that the epic does not register as a parent requirement during the migration of a task. There needs to be a workaround for this. Will continue working on it.


[R4R] - {sdk}: align standard-bridge with selected chain

Goals of PR
Core changes:

Defines a mapping between L1_MNT_ADDRESS and L2_CHAIN_ID so that we can check against the correct MNT address in the standard-bridge.

Thank you very much for your contribution. Please refer to our contributing guide and make your PR to develop. @grezle


Fix BQ connection parsing with dataset info

Description
Cosmos version 0.7.4 currently does not correctly parse BigQuery connection information with dataset information.
Cosmos expects dataset as a top level parameter in connection json.
However, based on the documentation of apache-airflow-providers-google here, dataset can not appear as a top level parameter in connection json.
This PR fixes this by updating the mapping in cosmos, so that it expects dataset parameter in extra field of connection.
Related Issue(s)
closes #365
Breaking Change?
None that I am aware of
Checklist

[ ] I have made corresponding changes to the documentation (if required)
[X] I have added tests that prove my fix is effective or that my feature works

@MonideepDe thank you for the contribution!


Favor uid over username in User directive.

I notice that I can use either of these syntaxes:
  run-as: nonroot
  run-as: 65532

... and we in fact favor the former in our examples.
I believe that certain runtime security features in Kubernetes don't work properly with the string variant, but do work with the uid variant.  I believe the reason is that if user is a uid, all you need is the config to evaluate policy, where if it is username then you have to fetch the image contents to access /etc/passwd to determine the uid, which in general means downloading arbitrary amounts of data.
If we can, then I'd suggest we bias towards doing the "right" thing regardless of input, and using the uid variant.  Regardless, we should probably switch the images where we use this to get the uid result.

Just to clarify, the action here is just to update the examples, we don't want to change any behaviour or apko code?

Yeah, I think that's right.

Seems like a nice enhancement for the examples, but we can do the UID mapping for strings too.  Would be nice to get it into apko 0.4.


Add Travis CI

I've added Travis CI. You can see it passing on my fork:
https://travis-ci.org/OR13/truebit-os/builds/378927391
In order to do this, I had to change .gitmodules to use https instead of ssh, and fix a path issue in the mocha test.
Having CI will help encourage contributions :)

Hey there! I believe we met at Devcon. Thanks for the PR, totally agree about it being helpful for contributions.


Add button to service app menu to open Copilot for Xcode app [ENHANCEMENT]

Suggestion to simplify access to the Copilot for Xcode.app by adding a "Open Copilot for Xcode" option to the service app's menu bar.
Equivalent to running:
open "/Applications/Copilot for Xcode.app"

Mock-up of new menu...

@intitni
I would like to work on this Issue.

@SNQ-2001 That would be great. The menu is defined in AppDelegate of the ExtensionService target. Please try to derive the path to the host app from the Bundle.main.xxxxPath, do not hard code the path if possible.


Users with only "Issue rights" ?

is there a way, that a user add issues without see the sourcecode ?
Can i give user any rights ?

@Metti9 no. Create an empty repository just for issues, if that is what you want.

I think this is duplicated/part of https://github.com/gogits/gogs/issues/1411.
Or you may also be interested in https://github.com/gogits/gogs/issues/649.


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2545 ms

Resolved: Samsung performance has improved in 70bacfe.


i3 4.12

Please consider merging #209 along with this PR.

Merged in https://github.com/Homebrew/homebrew-x11/commit/86827dc8dd27d523f7f83a2d9db885cb5fa91be7. Thank you for your contribution to Homebrew @afh; we appreciate it! :smiley_cat:

Thanks for merging!


Compile db2util on Linux

Original report by Anonymous.

This might be massively out of scope of this project.
My research has shown that there is no easy or practical way to use some of the nice new features of our beloved IBM i from the outside, lets say from a Linux machine. Lets say you wanted to leverage some of the new QSYS2 functions to do in depth monitoring of an IBM i box from one of the Open Source monitoring solutions. There seems to be no way of doing this.
This tool running on a Linux machine would be exactly what is needed.
PS: I know that it is possible to create services on the IBM i with node oder python to get access to those functions, but what is needed is not yet another process or server but instead use the basic system functionality.

The need to remove ODBC as a requirement expands this proposal far beyond the scope of db2util. Closing for that reason (feel free to reopen if a business need remains)


Server Not Installed Properly

Describe the bug
The server is not installed properly on launch
To Reproduce
Steps to reproduce the behavior:

Run in Docker Desktop for macOS

Expected behavior
Server running
Actual behavior
Server does not run
OS information (IMPORTANT)

OS:  macOS 15.1.1
Palworld Version:  UNKNOWN (Doesn't install)
Docker Image Version:  1.0.0 & dev @ 4a1f191

Hardware information (IMPORTANT)

CPU: M1
Memory:  16GB

docker-compose.yml contents (IMPORTANT)

docker-compose.yml
services:
  palworld:
    image: ghcr.io/thijsvanloef/palworld-server-docker:dev
    restart: unless-stopped
    stop_grace_period: 60s # Set to however long you are willing to wait for the container to gracefully stop
    ports:
      - 8211:8211/udp
      - 27015:27015/udp
      # - 8212:8212/tcp  # Port for REST API if REST_API_ENABLED: true
    env_file: "~/Cloud/iCloud/Projects/DockerCompose/palworld.env"
    volumes:
      - "~/Server/palworld/:/palworld/"

PUID=1000
PGID=1000
PORT=8211 # Optional but recommended
PLAYERS=16 # Optional but recommended
SERVER_PASSWORD="X" # Optional but recommended
MULTITHREADING=true
RCON_ENABLED=true
RCON_PORT=25575
TZ="America/Los_Angeles"
ADMIN_PASSWORD="X"
COMMUNITY=false  # Enable this if you want your server to show up in the community servers tab, USE WITH SERVER_PASSWORD!
SERVER_NAME="X"
SERVER_DESCRIPTION="Palworld server for X & friends"
ALLOW_CONNECT_PLATFORM="Steam" # Defaults to "Steam" if not set, set this to "Xbox" if you want to host a server for Xbox players. CROSSPLAY BETWEEN XBOX-STEAM IS NOT YET SUPPORTED

Container/Host Logs (IMPORTANT)

Logs
****EXECUTING USERMOD****
usermod: no changes
Server installation not detected.
****Starting Installation****
Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
Logging directory: '/home/steam/Steam/logs'
Looks like steam didn't shutdown cleanly, scheduling immediate update check
[  0%] Checking for available updates...
[----] Verifying installation...
UpdateUI: skip show logoSteam Console Client (c) Valve Corporation - version<PHONE_NUMBER>
-- type 'quit' to exit --
Loading Steam API...IPC function call IClientUtils::GetSteamRealm took too long: 555 msec
Fatal error: futex robust_list not initialized by pthreads
src/common/framefunction.cpp (238) : Assertion Failed: CFrameFunctionMgr::~CFrameFunctionMgr: non static FrameFunction[Bootstrapper HTTP Client] still registered
cp: cannot stat './PalServer.sh': No such file or directory
sed: can't read ./PalServer-arm64.sh: No such file or directory
chmod: cannot access './PalServer-arm64.sh': No such file or directory
./PalServer-arm64.sh does not exist.
Server Not Installed Properly

Additional context
Tried deleting the only bound directory to reset, but luck
Bug Report Checklist

[x] I searched for my issue on the Known issues page
[x] I searched for my issue in the Closed issues
[x] I have included all relevant information required above.
[ ] I am willing to fix this bug

Hi @Juice805 This is a known issue with mac hosts, you can try setting the following environment variables to see if it works:

BOX64_DYNAREC_BIGBLOCK=0

BOX64_DYNAREC_SAFEFLAGS=2

BOX64_DYNAREC_STRONGMEM=3

BOX64_DYNAREC_FASTROUND=0

BOX64_DYNAREC_FASTNAN=0

BOX64_DYNAREC_X87DOUBLE=1

Thanks for the reply! Unfortunately it adding the variables did not fix the issue

Hi! Try using DepotDownloader instead of steamcmd by setting USE_DEPOT_DOWNLOADER=1.

I'm currently playing around with this on my Mini M4 as well.
steamcmd fails due to the known bug, which prevents a complete download.
DepotDownloader got suck somewhere in the image itself, not downloading anything,
so I simply used it outside of the docker to download the files.
The problem is....that it didn't download linux64/steamclient.so due to that,
forcing me to copy it from my Steam Deck.
After solving the download issue another problem popped up when running the server itself:
[+] Running 1/0
 ✔ Container palworld-server  Recreated                                                                            0.1s
Attaching to palworld-server
palworld-server  | ****EXECUTING USERMOD****
palworld-server  | usermod: no changes
palworld-server  | chown: changing ownership of '/palworld/linux32/libstdc++.so.6': Operation not permitted
palworld-server  | ****Checking for available container updates****
palworld-server  | The container is up to date!
palworld-server  | ****GENERATING CONFIG****
palworld-server  | Using Env vars to create PalWorldSettings.ini
palworld-server  | ****Compiling PalWorldSettings.ini****
palworld-server  | Compiling PalWorldSettings.ini done!
palworld-server  | ****GENERATING CRONTAB****
palworld-server  | BACKUP_ENABLED=true
palworld-server  | Adding cronjob for auto backups
palworld-server  | time="2024-12-18T23:15:02Z" level=warning msg="process reaping disabled, not pid 1"
palworld-server  | Cronjobs started
palworld-server  | ****Starting Server****
palworld-server  | ./PalServer-arm64.sh -queryport=27015 -publiclobby
palworld-server  | time="2024-12-18T23:15:02Z" level=warning msg="process reaping disabled, not pid 1"
palworld-server  | time="2024-12-18T23:15:02Z" level=info msg="read crontab: /home/steam/server/crontab"
palworld-server  | The file already exists: /palworld/Pal/Binaries/Linux/steamclient.so
palworld-server  | [S_API] SteamAPI_Init(): Loaded local 'steamclient.so' OK.
palworld-server  | Shutdown handler: initalize.
palworld-server  | 5.1.1-0+++UE5+Release-5.1 1008 0
palworld-server  | Disabling core dumps.
palworld-server  | Setting breakpad minidump AppID = 2394010
palworld-server  | [S_API FAIL] Tried to access Steam interface SteamUser021 before SteamAPI_Init succeeded.
palworld-server  | [S_API FAIL] Tried to access Steam interface SteamFriends017 before SteamAPI_Init succeeded.
palworld-server  | Waiting for REST API(8212) port to open to show player logging...
palworld-server  | [S_API FAIL] Tried to access Steam interface STEAMAPPS_INTERFACE_VERSION008 before SteamAPI_Init succeeded.
palworld-server  | [S_API FAIL] Tried to access Steam interface SteamNetworkingUtils004 before SteamAPI_Init succeeded.
palworld-server  | Waiting for REST API(8212) port to open to show player logging...
palworld-server  | IPC function call IClientUtils::GetSteamRealm took too long: 160 msec
palworld-server  | Game version is v<IP_ADDRESS>518
palworld-server  | Waiting for REST API(8212) port to open to show player logging...
palworld-server  | Waiting for REST API(8212) port to open to show player logging...
palworld-server  | REST API started on port 8212
palworld-server  | Running Palworld dedicated server on :8211
palworld-server  | Waiting for REST API(8212) port to open to show player logging...
palworld-server  | [2024-12-18 23:15:27] [LOG] REST accessed endpoint /v1/api/players OK


Add new API to return the accelerator name.

Closes #203.
Each vendor should modify the function mlperf_backend_accelerator_name and return the appropriate accelerator name.

For TFLite backend, @anhappdev or @d-uzlov will implement the method in another PR.
For other backends, each vendor should submit a separated PR.


Update to salt 3000.9

@bors-servo r+

:pushpin: Commit 97c9a2a has been approved by jdm

:hourglass: Testing commit 97c9a2afedb3da16b5f9a5ccd1c2a08414420cab with merge 8faaf63fa66846a458f7cb6c0d4fd3b66be13bfb...

:sunny: Test successful - checks-travis
Approved by: jdm
Pushing 8faaf63fa66846a458f7cb6c0d4fd3b66be13bfb to master...


Build issues with deasync

Environment
azure-pipelines-task-lib version: 4.6.0
Issue Description
Fails to npm install. Seems to be related to deasync being a dependency now, and they are not working well on node 20.
Expected behaviour
Works just like 4.5.0, and builds fine on node 20.
Actual behaviour
Fails to restore packages on node 20.
Steps to reproduce

clone https://github.com/microsoft/setup-msstore-cli
Update azure-pipelines-task-lib to 4.6.0
npm i
Fail

Logs

Hi, the addition on "deasync": "^0.1.28", to the package-json made our ndoe_modules 3 times bigger and our 30MB extension to 80MB extension.
Can you look into it?
@DenisRumyantsev

@azchohfi, that is the problem related to missing binaries as I see.
There are no node20 binaries deasync-bin repository.

@azchohfi I noticed binaries (node v19-21) that were uploaded 3 days ago.
I'm closing this ticket for now.
@sverdlov93 upgrading azure-pipelines-task-lib from 4.5.0 to 4.6.0 makes node_modules bigger by 7MB. Could you please double-check that your problem depends on deasync module? If your problem persists, feel free to create another issue.

The specific issue was related to Jest itself. I had to add 'node' do the moduleFileExtensions in the jest.config.js.
Still, I'm facing a different issue now. deasyn doesn't seem to work with Jest:
https://github.com/abbr/deasync/issues/156

Ok, solution is to avoid the deasync code paths completely.
The problem is that whenever the deasync lib is called, the promises start to not work anymore. Solution? Don't go through the deasync code paths. How? Avoid the constructor that calls the sync methods.
Instead of doing:
let tr: ttm.MockTestRunner = new ttm.MockTestRunner(testPath, taskJsonPath)
await tr.runAsync()

Do:
let tr: ttm.MockTestRunner = new ttm.MockTestRunner()
await tr.LoadAsync(testPath, taskJsonPath);
await tr.runAsync()

It will completelly avoid calling deasync, as it won't call getNodePathSync


Script generates MD links.

Includes thumbs an GH links, but not themes.gohugo.io
Currently writes to /tmp.
This is for review; once I have my questions answered in the issue I'll push more changes.

Now this should be good.

Looks very good. Great job.
This is very useful for people looking for a theme, esp. until we get the theme site up and running.


paket cache clean

Hi there,
It would be nice to add something like "paket cache clean" - just like npm and bower do - so it is easy to wipe every packages that was downloaded into the user workspace - I'm talking about the NuGet Cache folder.
Behind the scene, I suppose we only need to do something like:
del %LOCALAPPDATA%\NuGet\Cache\*.nupkg /q
See, http://stackoverflow.com/questions/30933277/how-to-clear-nuget-package-cache-using-command-line
Thank you and have a nice day people!

"paket clear-cache" is the answer.
Doh!


Mantain feature parity with tailwind-merge v2.5

New tailwind-merge release:

https://github.com/dcastil/tailwind-merge/releases/tag/v2.5.0

We are up to v2.5.4 now. I have been busy lately but will get to this eventually


Yarn upgrade

"webpack": "^5.50.0",
"webpack-cli": "^4.7.2"
"cypress": "8.2.0"
Also fixed one failing test as was checking cypress upgrade

My general approach when going up major versions (the first number in 1.2.3) is to look through the release notes. Major versions mean breaking changes, and well-maintained libraries will provide a migration guide. For example, this is the migration guide for Cypress v8. If it is a small change, you will probably get lucky and not need to do anything, but it is worth still going through them to double check.


fix: minor type issues in starlight

Description

Closes # N/A didn't create an issue since the PR is so tiny
What does this PR change? Fixes some types
Did you change something visual? No

Here's the type issues I was getting, skipLibCheck unfortunately doesn't work for TS files (only .d.ts) so this is a real blocker when trying to lint/compile my project which uses starlight.

Hello! Thank you for opening your first PR to Starlight! ✨
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any issues you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🤩

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.


Previous replacements get clobbered in contenteditable regions in WebKit/Blink browsers

Reproducable on the demo page (see below) (even thought its using old versions of jQuery and At.js). But it also exists if you're using current jQuery and At.js (reproduced with 2.1.4 and 1.3.2 respectively). This seems to only happen with contenteditable fields, not textareas. Steps to reproduce on the demo page:

Go to demo page http://ichord.github.io/At.js/ in Chrome or Safari
Click on the contenteditable field to place the cursor after the :h
Select the Heart emoji from the dropdown when it appears to insert it (I did it by pressing Enter key)
Press backspace followed by space to delete and re-add the space injected after the emoji image.
Type : to display the dropdown and choose the first one in the list ("smile" for me) and insert it (I pressed Enter)
The Heart is replaced with the Smile emoji.

However, this only seems to happen following a replacement that IS NOT prefixed with an "At" symbol in the insertTpl. To illustrate, repeat steps 4-6 by alternating between : replacements and @ replacements. If you do 2 of each, you will only be left with the @-prefixed replacements.
I'm guessing there's a boundary detection issue with the replacement algorithm.
Now that I think I've identified the details of the issue, I will (hopefully) be posting a PR for this in the next few days.
This does not happen with replacements that are prefixed with the "At" symbol in the insertTpl template. So, in addition to the above, if you do steps 1-4, and then type an @ and select one of the options, the previous replacement is preserved. But if after the @-prefixed replacement, you repeat steps 3-6 with the : matcher, the emoji will

Upon further testing, it appears that what's happening in the browser is that, after inserting the first replacement, when I press backspace, the caret enters into the freshly .atwho-inserted element. Then when I press spacebar to insert a space, the caret stays in the .atwho-inserted element, appending the space character inside. Then when I type the "At" symbol to bring up the popup and select the replacement I want, the library injects the replacement into the .atwho-inserted element, clobbering its existing contents. However, in Firefox, it properly inserts the new "At" character in its own new .atwho-query element.

Upon further further testing, it looks like this is a browser-level issue that needs to be worked around. What's happening is this: Given the following:
<div contenteditable style="outline: 0px none;border: 1px solid lightgrey; width: 20em; height: 10em;" spellcheck="false" dir="auto">
  <b>Hello</b> World!
</div>

First, I click into the field to place the caret at the end (after "World!"). Then I backspace to delete "World!" to the point that the caret abuts against "Hello". And here's where things get interesting:

In Firefox (the one that behaves correctly), at the point, the caret is still outside of the b tag. If I begin typing, the text is normal.
In Chrome/Safari (the ones that don't behave correctly), the caret is now INSIDE the b tag. If I begin typing, the text is bold.

So, to the bug, when I backspace once after a replacement is inserted, Chrome and Safari capture the caret inside the .atwho-* around the replacement and the contents get clobbered from this https://github.com/ichord/At.js/blob/v1.3.2/dist/js/jquery.atwho.js#L688

@ichord I'm having issues trying to create a workaround for a number of reasons. The company I work for who's using this plugin is willing to contract you to prioritize fixing this issue. I can share the research I've done toward a solution to help you get a head start. Would you be interested/willing to discuss this possibility? We need it ASAP.

What's your idea to solve this problem?
Chrome and Safari will put the caret into the closest element, even if you use the Range API to move it outside the wrapper; And then all you typed would be inside the wrapper.

Well, I'm wrong. We can use the API to move it outside the wrapper, but still not work out.

I found a bunch on Stack Overflow about this issue and some possible solutions using the Range and Selection APIs to programmatically move the caret out of the element. However, I was having a hard time ingerating it into your code, primarily because I don't know CoffeeScript so I was working in the compiled JS locally.
As I said, my boss would be willing to contract you (with pay of course) to fix this specific bug because it's currently causing us problems. Do you have time and would you be willing to take that on?

@ichord I work with Matt. It would be great if you could give us some pointers and lead us to the direction on how we can solve this problem. We really need to solve it. So any suggestions would be helpful.
Thanks,
Puja

@Puja-K @webbower I tried to solve this problem yesterday, but not worked out.
@webbower Could you show me the what you got for now? I emailed you.

@ichord Here are my findings. These were the most informative links I had come across:

Provided by a colleague who had run into this before: http://pastie.org/10464855
http://stackoverflow.com/questions/6249095/how-to-set-caretcursor-position-in-contenteditable-element-div#6249440

The solution in the SO link worked in isolation (just a contenteditable by itself without having At.js bound to it) but I couldn't integrate it into your code.

Hi @ichord. Curious if you might have made any progress.
One other thing came to mind: when I was trying to implement this, the problem we are facing is around when to stay in the bounds of the query or when to move the cursor outside of it. So I was implementing it under a config flag about how to handle whitespace inside of the query. true would keep the caret inside the query and allow spaces in the query match (which would require special code in Firefox to move the caret inside the query tag), while false would move the caret outside the query tag when whitespace was entered at the tail end of the query.

Hi @ichord. Checking in again. Sorry for being a pest about this.

It's ok, @webbower . I am sorry for not working on it, and thanks for providing those solutions.
The problem is not how to move the caret outside, is when to move.
It's a weird problem, and I spent a whole afternoon trying to solve it 16 days ago and got nothing.
I got tried with it.
I will refactor it to keep it simple to reduce problems if necessary。

When you say "when to move the caret," do you mean at the code level (on which key event) or in terms of the user experience. If it's the user experience, that's what my earlier check-in comment had some thoughts on. If the plugin is configured to not allow spaces in the mention, then move the caret out when any whitespace key is pressed at the end of the mention. If spaces are allowed in the mention, then maybe move the caret when spacebar is pressed a second time in a row at the end, or when tab or Enter/Return are pressed at the end.
Thank you for looking into this.

You are right, it's user experience thing.
Allowing spaces is a basic requirement so that we have to support it.
We should not limit how many spaces could be put in. It's bad to tell user to press Tab and Enter/Return jump out to the 'box', and we have to explain what is the 'box'.

So, are you declining to fix this bug? It's causing us serious issues on our site. Many of our users are complaining about how the bug manifests itself. As I said before (and she just re-confirmed), our CEO would be willing to contract/pay you to fix this bug. If you are open to discussing it with her directly, send an email to engineering at peerlyst dot com.
So, brainstorming heuristics for how to implement this: I think it's fair to assume that, if a mention is selected from the suggestions dropdown and inserted, that that is a blessed output and so the wrapper should probably be marked as blessed. At that point, if the caret is at the end of the content within the blessed wrapper, pressing any whitespace key on the keyboard should move the caret outside the blessed wrapper and insert the whitespace after of the blessed wrapper.
Conversely, moving the cursor up to the tail of the mention should move the caret inside the mention wrapper (which doesn't happen in Firefox) to give them a chance to edit or change it (maybe they selected the wrong entry in the dropdown and wanted the one that was longer than what they selected). In that case, typing anything that isn't whitespace inside the blessed wrapper would un-bless it and re-trigger the suggestions dropdown behavior (like you currently have with the .atwho-query and .atwho-inserted classes).
In the case of typing the trigger symbol (@, :, #, etc) and a few characters after to show the suggestions, I think you can safely ignore whitespace within the query when determining what options to show based on what has been typed so far. Those suggestions could include whitespace for the output, and since they are wrapped in .atwho-inserted, they would become blessed wrappers and modifying them in any way would un-bless them (which I believe At.js already does). I think people would be ok with not matching whitespace when determining the suggestions since the usual use cases involve only typing a few characters and then autocompleting the rest. Plus, in our case, we're not even persisting the trigger symbol in the output. We're replacing it with a hyperlinked output that also supports tooltips.
Or, maybe you should do the suggestion matching based on the caret position and just do a straight replacement of the matched content with whatever the implementor wants (either straight text or including HTML), unless you tried that originally and couldn't get it to work.

I would like to fix this issue. But I'm moving to a new place this week, and got things to be done.
I don't have a whole day or half day time to work on it.
And I did send an email to you, but there is no response.
Here is my email, contract me if you would like to.

Sorry I can't reply to your idea right now, I have to get back to work right now.

Oh. I must have missed your email. Sorry about that. I will have my CEO reach out to you. Thank you,

how about this:

Yay !! Seems like you have fixed one of the scenario'. That's awesome. Thank you. Have you pushed your changes to git? How can we get the latest code changes and test it within our application.
Sent from my iPhone

On Nov 3, 2015, at 5:33 AM, Harold.Luo<EMAIL_ADDRESS>wrote:
how about this:
—
Reply to this email directly or view it on GitHub.

Awesome! That looks good. Once it's pushed, I'll try it out. Thanks!

@Puja-K @webbower  I have pushed it to the dev branch. You can pull it down and switch to the dev branch to try it out.
I'm going to optimize it a little bit tonight and release a new version.

I'm planning to fix as much bugs as i can and release a new version on this weekend.
If you don't use version manager, the codes is already in the master branch and good to go.
If not, I'm afraid that you have to wait the new release; sorry about this.

Not a problem. Thank you for doing this. We all really appreciate it.


Deployment

Hi, thanks for your amazing work. I want to deploy your model for a project. However, I have difficulties in understanding the model file with ".th" extension. Typically, we pickle the model and use the pickled model for deployment using Flask. However,  I do not know how to approach ".th" model file. I will appreciate it if you can provide me any clarification or hint in this regard. Thank you!

U can download from links in Pretrained models section

Hi, thanks for your amazing work. I want to deploy your model for a project. However, I have difficulties in understanding the model file with ".th" extension. Typically, we pickle the model and use the pickled model for deployment using Flask. However, I do not know how to approach ".th" model file. I will appreciate it if you can provide me any clarification or hint in this regard. Thank you!


No license on project

This project was maded with love, and need a license for use.

I agree the MIT license


Add a response message upon CLA acceptance

We would like to automate adding and removing labels on PRs when the bot informs the author about signing requirements. We can do this on the initial report since a comment is posted by the bot, but once the user "signs" the CLA there is no confirmation that the CLA was signed. If we had a message from the bot indicating the PR author has successfully signed the CLA then we could automate removing the label ourselves.

@denelon the confirmation that the CLA was signed is on the PR checks. See the image below:

@DragosDanielBoia is there a way to make a fabric bot rule trigger an action based on that?

@DragosDanielBoia is there a way to make a fabric bot rule trigger an action based on that?

There is no way, today we don't look at the checks on the PR. We are going to open source the new FabricBot so people can start contributing.

As per CLA policy
Install [Microsoft GitHub Policy Service] (https://github.com/apps/microsoft-github-policy-service)
If you run on GH Enterprise Cloud, you have to give us (by creating an issue on this repo) the name of your enterprise.
Create a .github repo.
Add platform context. yml under policies folder. You can push this directly.
Add cla. yml under policies folder. Create a separate PR for this, the policy service will create a comment example bellow. After you merge the PR, CLA policy will be activated across the entire org. image


3 months later, any updates?

Hi, does there any updates on codes or data?

We have released the code and datasets in this repo: https://github.com/3d-hand-shape/hand-graph-cnn/


Readme features

This readme doesn't explain enough to me, so I've added some basic stuff to the Getting Started section and another section at the end of the readme.

Yeah, thanks man for adding that stuff. I didn't really know I needed it, but now it's great to have it.

@Guelakais thanks for submitting this PR. Could you rework this PR so that it only contains your changes to the README file? I don't know why @ytakano chose to fork this project instead of submitting a PR, but unfortunately we can't merge their changes unless they submit them.


Love your work, I use the editor in all my projects!!

Hey, sorry to post it here as I did not see any way to contact you. I really like your work, so I am going to feature it in the tomorrow's issue of my my daily newsletter,  https://dailysandbox.com

Did you managed to solve HTML to markdown transform in correct way? The library itself saving HTML, which is weird.
I guess its better to save result in markdown, and then parse it on front-end, but convertation from library is incorrect, because its not actually apply something, its applying own classes, which not passed to markdown and any result you can take.

I am just wonder how you do that. Because i cant take something good on frontend

@MagisterUnivers The Quill Editor uses its own delta format to store the content. By default, it is not supposed to convert it to markdown.

@artknight You are absolutely right. I just left it work with HTML format and dangerouslyInsertHTML + DomPufiry. It is enough.
But if you want simple, fast, component based library for text-editor and markdown - use MDXEditor. I refactored it to that library, and i am so satisfied.


Rule_manager - Unable to rename remote rules

@threat-punter Is renaming a remote rule not possible via the rule_cli? I tried to rename a rule in our system (changed from "office365_LogTypes" to "Office365_LogTypes") and got the below error. I updated the yaral rule_name, the file name and the rule config entry.
13-Mar-24 07:09:41 UTC | INFO | load_rules | Loaded 7 rules from /opt/actions-runner/_work/chronicle-siem-detection-rules-dev
13-Mar-24 07:09:41 UTC | INFO | update_remote_rules | Attempting to retrieve latest version of all rules from Chronicle
13-Mar-24 07:09:41 UTC | INFO | get_remote_rules | Attempting to retrieve all rules from Chronicle
13-Mar-24 07:09:42 UTC | INFO | get_remote_rules | Retrieved 57 rules
13-Mar-24 07:09:42 UTC | INFO | get_remote_rules | Retrieved a total of 57 rules
13-Mar-24 07:09:42 UTC | INFO | get_remote_rules | Attempting to retrieve rule deployment state for 57 rules
13-Mar-24 07:10:21 UTC | INFO | update_remote_rules | Checking if any rule updates are required
13-Mar-24 07:10:21 UTC | INFO | update_remote_rules | Local rule name Office365_LogTypes not found in remote rules
Traceback (most recent call last):
File "/opt/actions-runner/_work/_tool/Python/3.10.12/x64/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/opt/actions-runner/_work/_tool/Python/3.10.12/x64/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/opt/actions-runner/_work/chronicle-siem-detection-rules-dev/rule_cli/main.py", line 268, in
update_remote_rules()
File "/opt/actions-runner/_work/chronicle-siem-detection-rules-dev/rule_cli/main.py", line 85, in update_remote_rules
rule_updates = Rules.update_remote_rules(http_session=http_session)
File "/opt/actions-runner/_work/chronicle-siem-detection-rules-dev/rule_cli/rules.py", line 557, in update_remote_rules
rule_id = local_rule.rule_id
AttributeError: 'Rule' object has no attribute 'rule_id'

Hi @curiousghost,
Yes, you should be able to rename a rule in Chronicle via your codebase by renaming the rule in the .yaral file name, in the YARA-L rule itself, and in the rule_config.yaml file.
Can you pull the latest version of the code from this GitHub repo and try again? Please let me know how it goes.
The output from the --update-remote-rules command should look something like this.
13-Mar-24 11:17:48 MDT | INFO | update_remote_rules | Local rule name google_workspace_mfa_disabled_1 not found in remote rules
13-Mar-24 11:17:48 MDT | INFO | update_remote_rules | Rule google_workspace_mfa_disabled_1 (ru_12345678-1234-1234-1234-1234567890ab) has been renamed - Creating new rule version for existing rule
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Logging summary of rule changes...
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules created: 0
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules new_version_created: 1
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | new_version_created google_workspace_mfa_disabled_1 (ru_12345678-1234-1234-1234-1234567890ab)
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules enabled: 0
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules disabled: 0
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules alerting_enabled: 0
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules alerting_disabled: 0
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules archived: 0
13-Mar-24 11:17:49 MDT | INFO | update_remote_rules | Rules unarchived: 0

Hi @threat-punter ,
Yes, this works with the latest version of the code. Thanks :)

Perfect!