Overhaul frame props plugin

This PR adds the following:

Categorisation (see here)
Dynamically add tables if any props of the corresponding category are found
Add some special handling for props from common plugins/JET functions
Separate all the LUTs and the like to their own files for easier modifying and a better oversight
"Prettified" is default now. Switch instead asks if the user wants to see the raw data

more in line with what was asked in #173

General refactoring
Prop sorting (reserved props always up top)
Double-click to copy the value of the selected row
Strips prop keys/values in the prettify window
Handle lambdas in prop prettifier function for more specialised outputs

Screenshots:

Categories and props
The following categories were added, and these props get sorted into them automatically:
Video
For the most part clip-wide properties that shouldn't change (much) per frame

'_Matrix', '_Transfer', '_Primaries'
'_ChromaLocation', '_ColorRange'
'_DurationNum', '_DurationDen', '_AbsoluteTime'
'_SARNum', '_SARDen'
'_FieldBased' (still considering moving this or adding a check to see if it's progressive or TFF/BFF and assigning it based on that)
'_FrameNumber'
'_Alpha'
'Name'
'idx_filepath' (vssource)

Metrics
Generic metrics, such as from PlaneStats or SceneChange detection plugins

'PlaneStats*', 'psm*', 'VFMPlaneStats*' (vs-zip, VIVTC)
'_SceneChangeNext', '_SceneChangePrev', 'SceneChange' (wwxd, scxvid)
'MMetrics', 'VMetrics' (dmetrics)
'dgi_*' (vssource's DGIndex)
'AutoBalance*' (stgfunc)
'pkt_*' (lvsfunc, possibly vstools in the future?)
'psnr_', '_ssim', 'ciede2000' (VMAF)
'_SIMULACRA2' (ssimulacra2)

Field
Direct IVTC/deinterlacing-related props

'_Combed', '_Field'
'VFM*', 'VDecimate*' (VIVTC)
'TFM*', 'TDecimate*' (TIVTC)
'wobbly_*' (lvsfunc, possibly vsdeinterlace in the future?)

Other
Anything else that doesn't get caught by the filters.
Prettified props:
The following props get prettified:
Basic

'_Combed' => 'Is Combed': 'Yes'/'No'
'_Field' => 'Frame Field Type': 'Bottom Field'/'Top Field'
'_SceneChangeNext' => 'Scene Cut': 'Curent Scene'/'End of Scene'
'_SceneChangePrev' => 'Scene Change': 'Current Scene'/'Start of Scene'
'idx_filepath' => 'Path to File': {value} (displayed as posix path)

VIVTC/TIVTC/dmetrics

'VFMMics'/'TFMMics' => 'VFM Mics'/'TFM Mics': 'x | y | z | a |b' (display 'N/A' if < 0)
'VFMMatch'/'TFMMatch' => 'VFM Match'/'TFM Match': x (any value between p, c, n, b, and u)
'VFMSceneChange' => 'VFM Scene Start/End': 'No'/'Yes'
'VDecimateDrop => 'VDecimate Drop Frame': 'No'/'Yes'
'VDecimateMaxBlockDiff' => 'VDecimate Max Block Diff': {value}
'VDecimateTotalDiff' => 'VDecimate Absolute Total Diff': {value}
'MMetrics' => 'DMetrics MMetrics': {value}
'VMetrics' => 'DMetrics VMetrics': {value}

Packet Sizes

'pkt_size' => 'Packet Size': {value} (display 'N/A' if < 0)
'pkt_scene_avg_size' => 'Packet Scene Average Size': {value} (display 'N/A' if < 0)
'pkt_scene_min_size' => 'Packet Scene Minimum Size': {value} (display 'N/A' if < 0)
'pkt_scene_max_size' => 'Packet Scene Maximum Size': {value} (display 'N/A' if < 0)

Adding the rest later...

i'd also make it possible to show VideoFrame at this point

I really like that idea, but I'm not sure I'll do that as part of this PR.

If there's no qualms about this PR besides the videoframe pop-out thing (which can easily be moved to a different PR), I'll be merging this in the following days.


kubectl-helm-minikube - Exposing Minikube ingress in Visual Studio Code Port Forward

Hey, I'm getting a bit lost (going round and round on google searches) on how to achieve a forwarding outcome. I've successfully got a dev container in my repo that comes up with Minikube (+ingress addon) that I can start and apply a deployment, service and ingress too. My devcontainer.json is approx like the below:
{
	"name": "k8s",
	"image": "mcr.microsoft.com/devcontainers/base:bullseye",
	"features": {
		"ghcr.io/devcontainers/features/docker-in-docker:2": {
			"enableNonRootDocker": "true",
			"moby": "true"
		},
		"ghcr.io/devcontainers/features/kubectl-helm-minikube:1": {
			"version": "latest",
			"helm": "latest",
			"minikube": "latest"
		}
	}
}

The outcome I want is for someone to be able to clone my repo and all they need is Visual Studio Code,WSL and Docker and they can boot the dev container, attach and then apply workloads to the cluster and from their windows desktop open the browser and access them via their ingresses.
My understanding on why this isn't working is that the Minikube ingress comes up using the minikube ip. So kubectl get ingress -o wide gives me:
NAME              CLASS   HOSTS              ADDRESS        PORTS   AGE
example-ingress   nginx   hello-world.info   <IP_ADDRESS>   80      121m

<IP_ADDRESS> being the minikube ip
Port forwarding in Visual Studio Code appears to be limited to forwarding to localhost or <IP_ADDRESS> not <IP_ADDRESS>... I know I can run kubectl port-forward but kinda want to avoid scripting that and just have networking already up and running. I don't think Minikube can come up as ports in the dev container can it? How can I achieve this outcome?

Hi 👋
We have information on how to make it work on kubernetes-helm#ingress-and-port-forwarding. Let us know if you are still running into issues, thanks!
We also have a kubernetes-helm-minikube Template.

Feel free to reopen if you have more questions, thanks!


is it possible to record video conferencing in server side?

is it possible to record video conferencing in server side?
when a "Record" button is pressed, the room will start recording.
it will only stop recording when "Stop" button is pressed or there's no one in the room anymore.
I tried digging in the source code of ion,
this is the only thing related i found.
https://github.com/pion/webrtc/blob/master/examples/save-to-disk/main.go#L97
but I think this code will always record the connection.
I hope it is possible to selective recording.
I hope to use it in flutter.
any help will be appreciated

This is not supported yet. We have a webm saver plugin but it still needs some work. It doesn't support conditional recording either. It is either enabled/disabled by config during initialization. If you are interested in working on this, that would be great!

supported by avp now

Hi can you elaborate more?
so conditional recording is now supported?
can you put a link or any hint how to make this work?
or which file it is now?
is it now available in Flutter demo too?

can you write the link of the commit please ?


fix: ScrollableColumn is an antipattern

Description
From the docs: Creates a scrollable vertical array of children.
The need for the widget is not clear to me yet, but it is a very very poor choice for a scrollable array of children.
This widget is a composition of a SingleChildScrollView, with an IntrinsicHeight wrapped Column. This is ripe for performance issues. Offering this widget to users to provide children to place inside of a SingleChildScrollView, on top of the speculative layout pass that IntrinsicHeight adds, will not lazily load children on demand.
This is what a ListView is for.

Hey @Piinks are you planning to share a recommended approach to mitigate this issue within the Flutter framework or are you hoping we generate and share a proposal for your review?

I know there are multiple use cases, so I cannot provide a recommendation for all of them. In my opinion, the replacement should be a simple ListView, but I am happy to consult if an example where this does not work can be provided. Thanks!


Manage Email Addresses process is not clear

Clicking Associate on the Manage Email Addresses page responds with:
"Your association request has been handled. Please check your inbox! Depending on grey-listing etc, it may take up to 15 minutes before your confirmation email arrives."
This implies that there is nothing further to do. However that is not the case as the email states:
"You (or someone else) has requested to merge this email address with the account ... in Pony Mail
If you wish to complete this merge, please visit https://..."
It's also not clear what 'merge this email address' means.
It's confusing for the mail to use 'merge' when the page uses 'associate'

Here is a patch to use associate rather than merge in the e-mail:
merge.patch.txt

could you submit this in the form of a pull request instead?


issue with jdbc The connection is broken and recovery is not possible

I have  been using the following jdbc connection string in our aks cluster to sql server. And the connection works but after idle I see this error connection is broken and recovery is not possible. Is there any keepalive setting which I can use to keep the session active similar to oracle jdbc (SO_KEEPALIVE=true)(TCP_KEEPIDLE=60)(TCP_KEEPINTVL=30)(TCP_KEEPCNT=15)). I am not able to find the same in microsoft documentation.
jdbc:sqlserver://xxxxx.database.windows.net;database=xxxxxdb;encrypt=true;trustServerCertificate=true;loginTimeout=120;authentication=ActiveDirectoryPassword
```

Hi thanks for your question.
I would like to understand more about what is happening. Can you provide more details about the failure scenario (e.g. is it happening during a long running job)? Also could you please provide logs that capture this error event? What version of the JDBC driver are you using? Which OS are you using and what is the Java version? Is Java connection pooling being used?
The connection recovery depends on specific unrecoverable scenarios which are documented here - https://learn.microsoft.com/en-us/sql/connect/jdbc/connection-resiliency?view=sql-server-ver16#connection-recovery. I suspect we will see one of these events in the logs.
The JDBC driver does not support a Keep Alive feature. However you can manually configure such a feature in the OS depending on your OS. Here are the instructions for Configuring the Keep Alive on Windows - https://learn.microsoft.com/en-us/sql/connect/jdbc/connecting-to-an-azure-sql-database?view=sql-server-ver16#connections-dropped
Please note, if you are setting the Keep Alive on the OS, it will be a global setting impacting all connections.
I look forward to hearing more about this issue.

Currently we are using mssql-11.2.0.jre11.jar . It is running in pod rhel8 image in AKS cluster. We are using Java connection pool . Noticed that this happens when it is idle and all of a sudden transaction takes place.
We also add the following settings connectRetryCount=10;connectRetryInterval=10
jdbc:sqlserver://xxxx-xxxx-dev.database.windows.net:1433;database=XXXXXX;encrypt=true;trustServerCertificate=true;loginTimeout=120;connectRetryCount=10;connectRetryInterval=10;authentication=ActiveDirectoryPassword

The keepalive setting in the pod is as follows
net.ipv4.tcp_keepalive_intvl = 75
net.ipv4.tcp_keepalive_probes = 9
net.ipv4.tcp_keepalive_time = 7200

This is error we see in the log of the application
 [Database] SqlServer error: code = '0', SQL state = '': while committing: The connection is broken and recovery is not possible. The connection is marked by the client driver as unrecoverable. No attempt was made to restore the connection.
 0 | 2024-01-02 10:48:36 | XXXXX | DBG0000 | F | could not commit pending transactions: The connection is broken and recovery is not possible. The connection is marked by the client driver as unrecoverable. No attempt was made to restore the connection.

basically what I noticed is under idle condition the driver becomes unrecoverable. We need to restart to connect again to the database

I find it slightly confusing that you are reporting that the connection is idle when the issue happens. The error message indicates that there is an open transaction pending when the connection is lost. That would imply that there is an ongoing DB operation. Could you please look into what DB operations(s) is in the transaction.
The driver does have built-in keep-alive packets, but this feature is not configurable. If the keep-alive packets are enabled you will see the following in the logs:
"Setting KeepAlive extended socket options"
If keep-live packets are not available you will see the following in the logs:
"KeepAlive extended socket options not supported on this platform"
Assuming keep-alive packets in either the driver and\or the OS are available, that would mean that there is another process that is interrupting the connection. For example if you are using a Proxy connection when there is a 30 minute inactive connection the Azure Gateway would kill the connection (https://techcommunity.microsoft.com/t5/azure-database-support-blog/azure-sql-idle-sessions-are-killed-after-about-30-minutes/ba-p/3268601). At this point it appears that this is a likely explanation for this issue.

It's been some time since we last heard from you. As such we are going to close this ticket.
If you have any further questions, please feel free to reopen the ticket and we will be glad to help.


Basic CWL no-container test

According to docs, this simple test should be enough to run CWL tests with --no-container flag.

Have you tried in the cluster? If running with --no-container you probably need the local installation of bcbio-nextgen (with tools and so on), which you don't have in Travis.

Yes, @guillermo-carrasco, I just managed to make it work in the cluster after the last commits on PATH and HOME env vars and also tweaking cwl's directory testinput-bcbio_system.yaml file so that it points to tools installed in cluster paths:
  #picard: {dir: /usr/local/share/java/picard}
  picard: {dir: /cm/shared/apps/bcbio/20150720-devel/tools/share/java/picard}
  #rnaseqc: {dir: /usr/local/share/java/RNA-SeQC}
  rnaseqc: {dir: /cm/shared/apps/bcbio/20150720-devel/tools/share/java/RNA-SeQC}

Now I see where @chapmanb was aiming at with the from_world function in bcbio.cwl.create.py. I'll either make this test pass on travis with and/or without docker or expand the from_world function.
I think I'll focus on the docker CWL part passing on travis, what do you think, @chapmanb?

It would be great to make it easier to run CWL tests but is probably too early to include this in travis since it doesn't even work with Docker locally. I'd add the tests, leave it out of Travis, then try to get a reasonable subset working locally before trying to push into Travis. Would this work?


@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/python-crontab-feedstock#10.


remove leading underscore from lock_counter member

I haven't yet figured out why this change is needed for me - Perhaps the API of the filelock library changed recently?  I have filelock version 3.12.0,  Python 3.10.11

https://py-filelock.readthedocs.io/en/latest/api.html#filelock.BaseFileLock.lock_counter

Hmm, ya
https://github.com/tox-dev/py-filelock/commit/b4713c946cf8bcd65276bffbccef8f269d504dbd
The bigger question is why do you have v3.12.0. The Pipfile.lock says 3.10.2

My project uses terra as a submodule/editable dependency and has its own Pipfile.lock, which does say 3.12.0.  If you wanted to ensure an older version of filelock is installed you probably need to specify something in the requirements.

Ya. I guess we should set a version limit on filelock in the setup.py file

Closing this PR because it would require updating terra's Pipfile, Pipfile.lock, which is not desirable.  Instead, users of terra should make sure that their version of lockfile is < 3.12


Created a vue + react project when asked for a Vue project

Describe the bug
I used the prompt:
"I want to build a vue + typescript + vite + shadcn-vue with centered layout (not the 100% of the width) and background color for its different sections so it looks like a triped table but with the entire layout" and it created a Vue + React project
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-dfsbzm
Steps to reproduce

Go to the repro link
Check the terminal and the generated code (you wiill see tsx components and react imports)

Expected behavior
It should generate only a Vue with Typescript project using vite + shadcn-vue
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 53236314,
  "usedJSHeapSize": 50538382,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = David-Pena
Chat ID = 258600754fa9

Additional context
No response

I had exactly the same problem here's my vue, supabase, nuxt-ui project that, event after multiple prompts, bolt would not remove the React code: https://bolt.new/~/sb1-6nfxdm


Change to proof of agency status label

Summary
Make the proof of agency status label on the partner profile consistent between bank and partner views
Why
Makes communication between bank and partner slightly less confusing
Details
Wherever we have a "Proof of Agency Status"  or "501(c)(3) IRS Determination Letter",  change it to "501(c)(3) IRS Determination Letter or other Proof of Agency Status”
This will be on the partner profile view and edit screens for both the bank and the partner
Criteria for completion

[ ] changes as described
[ ] update any tests that use that text as well
[ ] may require updates to the user documentation

@cielf I would like to work on this. Currently setting up the project for the first will reach out again if I manage to run everything and the issue is still open.

If you have trouble,  please feel free to join our Sunday office hours at 11 Eastern Time -- https://us06web.zoom.us/j/89418249390?pwd=pdxbtMU3DPvyEc5BVOjbhO70WUhsCF.1     or ask questions in Slack.

can you please assign this to me

Hey @Naraveni -- we mostly keep the Good First Issues for first timers  (this one is just a very simple text change) -- to have something to do while they get their system set up and exercise the whole cycle for the first time.
I see you've had a couple of successes so far --   and we've got about 14 unassigned Difficulty-Beginners.   May I suggest you give one of those a try as a next step?

Hi, I would like to contribute to this issue. Is there any body working on this?

It's yours to work on!  I'll assign you momentarially.


Space after BEGIN or END {} blocks breaks syntax

syntax inside the { block } is broken in atom

END {
    if (line || !fieldIndex) {
        exit !fieldIndex
    }
}

ok in atom

END{
    if (line || !fieldIndex) {
        exit !fieldIndex
    }
}

Can you give me more details? I can't see any difference:

Must be the theme I was using. I will post a screenshot as soon as I have a chance for reference.
Thanks.

Seems to be a little glitch with Atom's Dark theme.

I switch themes fairly often and this is not a major problem for me TBH.


[cherry-pick] Fix bugs toBranch 2.1

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

run starrocks_fe_unittest

run starrocks_fe_unittest


[BERT] Switch to use CheckpointReader to load tensors from checkpoint.

Switch to use CheckpointReader to load tensors from checkpoint (it was using TensorFlow API directly). Also add a simple command line parameter for the example app, BERT-CoLA, which takes "BERT", "ALBERT", "RoBERTa" to switch between models.
swift run BERT-CoLA RoBERTa ends up with this error.
Training BERT for the CoLA task!
[Epoch 1]
Fatal error: Input to reshape is a tensor with 1 values, but the requested shape has 786432: file /Users/danielzheng/swift-debug/tensorflow-swift-apis/Sources/TensorFlow/Bindings/EagerExecution.swift, line 301
Illegal instruction: 4

This happens in master as well. So, it is not an issue introduced by the change.
BERT base and ALBERT base can start training.

I made this change to load different models easier and also verify different language models. I think BERT is in best shape. ALBERT's checkpoint is pointing at an TfHub checkpoint, therefore needs some special treatment. I don't know how RoBERTa's checkpoint was generated. The original checkpoint was in PyTorch format.
I open this PR to see if those code changes can be useful to anyone. It is fine to leave it out of master.


how to finetune using emma framework?

Hi~
Do you have some suggestions for fine-tuning base on the emma framework?  data generation, train, etc.
You said:

The overall performance of the MLLMs without fine-tuning for end-to-end trajectory planning is inferior compared to fine-tuning-based approaches, such as those presented in EMMA.

the next step should be fine-tuning.

Hi,
Thank you very much for your interest in our OpenEMMA project!
This is indeed a valuable question. Based on our current work, we are actively exploring the pipeline for fine-tuning MLLMs. Based on our experience, I’d like to share some of our findings that may help address your query:

Supervised Fine-Tuning (SFT) should be employed;
Both the input historical status and output predictions can be represented in a textual format.
Using continuous frames as the visual input often yields better results compared to a single image;
Predicting speed and curvature tends to be easier and more reliable for MLLMs than directly predicting the entire trajectory;
Incorporating Chain-of-Thought (CoT) reasoning helps improve the models' performance. This can be automatically generated using advanced models likeGPT-4o;
We did not observe significant benefits from using the multi-view frames compared to the front-camera view;
Tackling 3D grounding remains an exceptionally challenging task for current MLLMs.

I hope these insights help address your question. Please don’t hesitate to reach out if you have any additional questions or suggestions!

Do your group have plan to develop these features? @ShuoXing98


HV-1299 Upgrade TCK to 2.0.0.Alpha2

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

Won't build until https://github.com/hibernate/hibernate-validator/pull/708 is merged.


rebar_plugin windows paths issue

Hi, there's an issue with the pathname comparison here
https://github.com/evanmiller/ChicagoBoss/blob/master/skel/priv/rebar/boss_plugin.erl#L130
is_base_dir() ->
    rebar_utils:get_cwd() == rebar_config:get_global(base_dir, undefined).

I get drive letters one in upper- and one in lowercase and the comparison fails. So the whole build-process fails.
I resolved the issue for myself by wrapping the two strings in filename:nativename() but I'm not sure if this is a proper solution.

Try this
https://github.com/evanmiller/ChicagoBoss/commit/d09a2b0a76f23332e1f80b2a408ed5eaf9421506
On Tue, Feb 21, 2012 at 8:26 AM, Falco Hirschenberger
<EMAIL_ADDRESS>wrote:

Hi, there's an issue with the pathname comparison here
https://github.com/evanmiller/ChicagoBoss/blob/master/skel/priv/rebar/boss_plugin.erl#L130
is_base_dir() ->
   rebar_utils:get_cwd() == rebar_config:get_global(base_dir, undefined).

I get drive letters one in upper- and one in lowercase and the comparison fails. So the whole build-process fails.
I resolved the issue for myself by wrapping the two strings in filename:nativename() but I'm not sure if this is a proper solution.

Reply to this email directly or view it on GitHub:
https://github.com/evanmiller/ChicagoBoss/issues/82

Yes it works, thanks.


color doesn't update

Hi there
Sorry to bug you, I really like the ease of use of TaskBoard so far!  I've looked through all of the issues and solved most of my install problems, but the one remaining issue is that the color doesn't update -- no matter whether I use automatic actions or if I just manually try to set the color of an item.  I can update the individual tasks, no problem.
I made sure the TaskBoard directory has the correct privileges, re-looked at issues #3, #7, and #159.
Using Apache 2.4 on Windows 8 (just a prototype machine for trying these things out), the only difference I can see to what you listed on other issues is that I am missing these options within Apache:
LoadModule mpm_prefork_module modules/mom_mpm_prefork.so
and this within PHP 5.6:
extension=zip.so
Did I grab the wrong version of Apache or PHP, or both?

Are you on v0.3.0? PHP and Apache versions seem fine.
Can you bring up F12 developer tools and let me know if any error messages show up when you try changing a task's color?

Closing due to lack of response.
Let me know if this needs to be re-opened.


fix(ui): Issues around Add Data Quality Test #7498

Describe your changes :

I worked on the Fixing UI: Issues around Add Data Quality Test

Only the admin can add a test suite
Active cursor onFocus of description
Removed disabled state of select test suite dropdown onclick of add new test suite
Closes #7498

Type of change :

[x] Bug fix
[x] Improvement

Frontend Preview (Screenshots) :
https://user-images.githubusercontent.com/71748675/191276650-81abb006-bd4a-4d01-a5a7-8948031d5cd6.mov

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Looks good to me. Can we remove tailwind classnames please?

I have updated the code, Can you have a look?


Volunteers Page: Kiosk Mode: Login of New User

Enhancement: In Kiosk Mode: ADD login of new user? Currently it appears that the ID associated with actions done in the Kiosk Mode will be the ID of the person who initially logged into the device. For reference, in kiosk mode, who's account/ID number is logged in the "Entered By" field?

The tricky thing here is that it would then require all volunteers at the console to remember their password.
Either that or we have an 'operated by' sort of selection box that would just accept a badge number/name/etc... and use that without question.

Accepted


jdupes: 1.27.3 -> 1.28.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/jdupes/versions.
meta.description for jdupes is: Powerful duplicate file finder and an enhanced fork of 'fdupes'
meta.homepage for jdupes is: https://github.com/jbruchon/jdupes
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.28.0 with grep in /nix/store/0khxagiafaa2y8j49wp4k78q0nz04lpa-jdupes-1.28.0
found 1.28.0 in filename of file in /nix/store/0khxagiafaa2y8j49wp4k78q0nz04lpa-jdupes-1.28.0

Rebuild report (if merged into master) (click to expand)

2016 total rebuild path(s)

2016 package rebuild(s)

First fifty rebuilds by attrpath

CuboCore.coretoppings
OVMFFull
_1password-gui
_1password-gui-beta
_64gram
adapta-gtk-theme
adl
adriconf
aegisub
aether-lv2
affine
akkoma
alfaview
alfis
almanah
altair
alvr
am2rlauncher
amule
amule-daemon
amule-gui
amule-web
anbox
ani-cli
anilibria-winmaclinux
anime-downloader
anki
anki-bin
antares
antimatter-dimensions
anytype
apache-directory-studio
apostrophe
appgate-sdp
appimage-run
appimage-run-tests
apvlv
aquamarine
arc-icon-theme
arcan
arcan-all-wrapped
arcan-wrapped
archi
archipelago
archipelago-minecraft
archivebox
arduino-ide
armcord
arrayfire

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0khxagiafaa2y8j49wp4k78q0nz04lpa-jdupes-1.28.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jdupes https://github.com/r-ryantm/nixpkgs/archive/d64a847d3645bea16b07c096bba8dba4c327efa7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d64a847d3645bea16b07c096bba8dba4c327efa7#jdupes

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0khxagiafaa2y8j49wp4k78q0nz04lpa-jdupes-1.28.0
ls -la /nix/store/0khxagiafaa2y8j49wp4k78q0nz04lpa-jdupes-1.28.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

Looks like https://github.com/NixOS/nixpkgs/blob/2818172c0afad162d629198e76b2988ea439b4af/pkgs/development/python-modules/sure/default.nix#L50 broke eval


[ETHOSN] Improve inferring new shape of the Reshape operator

This pr improves the inference of the new shape of the Reshape operator for the Arm(R) Ethos(TM)-N NPU.
cc @lhutton1

@lhutton1 @leandron

After reading https://github.com/apache/tvm-rfcs/pull/88 I think we should quickly update the commit body to be more specific about the fix, WDYT?

After reading apache/tvm-rfcs#88 I think we should quickly update the PR body to be more specific about the fix, WDYT? (No need to push a new commit, I can change the commit message so it matches the PR)

It is good to me to update the PR body.

Thanks @NicolaLancellotti!


[5.7] Revert "work around php bug"

Now that PHP 7.3.2 has been released, we should be able to revert the temporary fix.

I think we would also need to disallow the incompatible php versions in composer.json.

@sisve I'm actually not sure if that's a good idea, especially not now in 5.7. The majority of people didn't had problems with PHP 7.3.0 or 7.3.1, just a certain amount of people.

Yeh, I don't think we should change the min version. We've never done this in the past, for every bug in 7.x that affected Laravel.

Yeh, I don't think we should change the min version.

That is, the min version of 7.3 allowed.

I think I'll just leave this for a bit longer since people could still be running on those broken PHP versions.

Sure thing 👍


Merge ais state and agent state

Merge ais state and agent state
Create single step function in the agent, which is jitted, instead of jitting AIS, and jitting the gradient update separately.

Done


allows using variable name for TAG name

It is not an ideal implementation since it may cause unintended usages. However, it does the job without using an additional flag.
Closes #3

Have you actually verified this works?

Yes it does, I tested on here https://github.com/cilerler/test-github-actions

Ah, now I understand what it does :).
I'm not sure if passing a name of the var to be interpreted later is the best approach here - seems like it might be difficult to explain, and I'm not sure it offers any benefits over using a standard name for the variable.

Definitely not, the way you implemented is good to go!  Thanks again.


Is there a way to invoke jscs auto fix as part of "save" of a file?

Is there a way to invoke jscs auto fix as part of "save" of a file?

I'm not sure if the Brackets API allows this. At the moment, you can run it from the context menu.


⚠ Bofrost.at has degraded performance

In caf6af8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1459 ms

Resolved: Bofrost.at performance has improved in f84b494.


Refactor ldap_rest_users User Entity

Refactor ingestion/src/metadata/ingestion/sink/ldap_rest_users.py to work with the proper User Entity from pydantic models, instead of relying on a custom-defined MetadataUser.
EDIT: I actually think that this class is not being used at all. Double-check and remove it if necessary!

@pmbrull I will take this one (my first issue, just to get my hands dirty)

Thanks, @omriAl!
Please, let me know if you have any questions or issues while working on this. Basically, the point of the task at hand is to have you prepare the local instances of the metadata server and get a bit familiar with the code.
Thanks

@pmbrull Thanks, As mentioned, I will also deprecate MetadataUser class.
It's used only on tests, which should be modified anyways after this change.

@pmbrull Hi.
currently, org.openmetadata.catalog.api.teams.CreateUser schema does not match the User class.
As I see this, we have 2 options - either changing the User class to have the same attributes as the schema, or changing the org.openmetadata.catalog.api.teams.CreateUser schema, which I'm guessing will affect other places as well.
what do you think is the better approach here?
seems like option a is preferable, but just wanted to make sure.
Thanks!

Hi, @omriAl, sorry what do you mean by CreateUser not matching the User class?
In the JSON schemas, we have a couple of definitions for each Entity. The entity itself, which describes the properties that define an Entity (in this case user), and then the Create Entity (in this case CreateUser. They are a bit different from the Create, for example, does not require the id field, which is automatically generated when running the POST operation to the backend, and it takes the definitions from the Entity schema.
Is this what you are referring to? Sorry if I misunderstood! Or is there some field not properly defined?
Thanks!

There are some discrepancies between the User class and user.json schema.
for example first_name & last_name doesn't appear in user.json but are attributes of the User class.
and the other way around - user.json has description, but User class does not.
It seems like the best way here is to modify the user class to match the user.json schema.
I just want to verify that there is no objection to that.

@pmbrull There are some discrepancies between the User class and user.json schema. for example first_name & last_name doesn't appear in user.json but are attributes of the User class. and the other way around - user.json has description, but User class does not.
It seems like the best way here is to modify the user class to match the user.json schema. I just want to verify that there is no objection to that.

Aaaah sorry I completely misunderstood. What User class are you taking? We should be working with the one under metadata/generated, which id automatically creared based on the JSON schemad

Closed with https://github.com/open-metadata/OpenMetadata/pull/1473


Restore compatibility with Python 3.5

ZipInfo.is_dir() was added in Python 3.6.

Closes #18 -- although gitlab-art should probably have Python version compatibility defined in setup.py (see scuba) and CI tests (see scuba).

Tested and works in debian:9.5


Requirement for MFA to be disabled for admin accounts?

Is it necessary to disable MFA for the admin account which is used for running PowerShell script to create/configure  device accounts for Surface Hub 2S? This is not mentioned in the technical documentation but if I use a admin account with MFA enabled to run those commands then I get an access denied error.

@neeldaya. will you mention  the article from where you read it

This is the documentation for creating device accounts for SurfaceHub 2S.
If I run these commands with an admin account with MFA enabled then it
fails. I have to disable MFA for such admin account to run these commands.
Unfortunately this is not described anywhere.
On Tue, Dec 10, 2019 at 2:24 PM VARADHARAJAN K<EMAIL_ADDRESS>wrote:

@neeldaya https://github.com/neeldaya. will you mention the article
from where you read it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/windows-itpro-docs/issues/5639?email_source=notifications&email_token=AIWPMVD6WTSRG3NRHASHJYLQX6J7TA5CNFSM4JY6G26KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGPGV2I#issuecomment-564030185,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIWPMVAJGBKBU76HBXNB2RLQX6J7TANCNFSM4JY6G26A
.

--
Regards,
Dayanand

@neeldaya. Thanks for the reply. I am using  standardalone computer so I could not able to give exact solution. Keep waiting for the solutions from th experienced windows users.
In meanwhile you can post your questions or queries on answers.microsoft.com or techcommunity.microsoft.com

@neeldaya - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

Sorry, I forgot to mention the link,
https://docs.microsoft.com/en-us/surface-hub/surface-hub-2s-account

@neeldaya - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
Please follow this link to contact support for Windows 10 products:  https://support.microsoft.com/hub/4338813/windows-help?os=windows-10
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


MatTabGroup height is always dynamic

Bug, feature request, or proposal:
Bug
What is the expected behavior?
According to the documentation,

By default, the tab group will not change its height to the height of the currently active tab. To change this, set the dynamicHeight input to true. The tab body will animate its height according to the height of the active tab.

What is the current behavior?
As far as I can tell, the height of the mat-tab-group element is always dynamically set according to the height of the currently active tab. The only difference is that when dynamicHeight = true there is a smooth height transition when switching tabs. What I want in my own app is for the height of the group to be fixed and for the content to scroll when necessary, which I assumed would be the case if dynamicHeight = false. Maybe I am misunderstanding the documentation - if so then I think its meaning should be clarified.
What are the steps to reproduce?
Here is a plunkr showing what I mean: http://plnkr.co/edit/gI0A56uTDzoLjYAosbDR?p=preview
Toggle the dynamicHeight property using the slide toggle. The height of the md-tab-group element will change regardless of the value of this property.
What is the use-case or motivation for changing an existing behavior?
Implementation should be aligned with documentation or documentation should be unambiguous.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 4.4.3/Master, Material beta 11/Master, Windows 10, Chrome 61
Is there anything else we should know?

If you you would like to have dynamicHeight=false and prevent resizing of the mat-tab-group, you will need to set a specific height for the mat-tab-group.  Without the height being set, the browser will expand the element to contain the contents.
With dynamicHeight=true, the expectation is that it will be resizing throughout use.  The component provides a transition animation and adds overflow-y: hidden to the mat-tab-body as it should not enter a scrollable state when changing.

@josephperrott Thanks, but I still don't fully understand - if I set a height on mat-tab-group then the height will be fixed regardless of dynamicHeight: http://plnkr.co/edit/1uGqpx1wHrJGMnDFX1dZ?p=preview

Correct, dynamicHeight does not make it dynamic as much as it handles the animations for it being dynamic.  I think of it as being a marker for telling the tab group it is dynamic rather than telling it that it should be dynamic.

Closing as this is working as expected.

@josephperrott If this is the intended behaviour, why would you ever want to set it to false? I still don't think the documentation of this property really aligns with the implementation. The line

By default, the tab group will not change its height to the height of the currently active tab.
quite strongly suggests to me that the tab group's height will be static unless dynamicHeight is true.

Agreed, there is an inconsistency between the following

Overview description

Dynamic Height
By default, the tab group will not change its height to the height of the currently active tab. To change this, set the dynamicHeight input to true. The tab body will animate its height according to the height of the active tab.

API reference

dynamicHeight | Whether the tab group should grow to the size of the active tab.

@benelliott's plunker
http://plnkr.co/edit/gI0A56uTDzoLjYAosbDR?p=preview

IMO, the behavior makes sense per @josephperrott's explanation (particularly the no overflow part), but the docs suggest a different behavior.

If this is the intended behaviour, why would you ever want to set it to false

Still badly named and badly documented.
What we need is matchHeights or equalizeHeights or normalizeHeights or something to that effect. That's not what this is. This is animateHeight and that's all.

If this is the intended behaviour, why would you ever want to set it to false

If you don't want it to animate and want an instantaneous jump.
This property is still badly named and badly documented. What we often need is matchHeights or equalizeHeights or normalizeHeights or something to that effect.
That's not what this is. This is animateHeight and that's all this is.

But then it should be renamed as such. This isn't dynamicHeight, it is animatedHeight.
And the docs should be fixed to reflect that.

yea, the whole thing just seems weird.
If you want to animate, then you want it to animate right.
It should be called: heySoThePageDoesNotGetJackedUpForASplitSecondIncludeThisPieceOfCode


failed to validate auth token. issue a new one

It keeps throwing this error in the log, i have redone the token several times on different computers with the same result

I can get the token from Dev>Application>Cookies>gp_access_token

@BigWebstas I have the same issue and started digging. When I've checked status code in .validate() step it was 404 but it is propably due to url mismatch (now url looks like https://gopro.com/media-library/ but this class tries to get media from https://api.gopro.com/media/search). When I've updated the url to media-library, then I've got 403 instead, which indicates wrong api_token :/
@itsankoff do you have any suggestion how to overcome this issue?

Indeed. I receive the same issue.

same

same

+1 to the thread above.

@BigWebstas @adam-kozlowski-280 @PiotrKrzyzek @Louiscars190 @rfsbraz @szaracz:
Folks the issue is fixed. The problem was that GoPro cloud changed their authentication mechanism from pure JWT to more sophisticated JWE which encrypts the content of the JWT token (kindof).
So the new version 1.4.3 is tested but keep in mind that now you need to provide additional parameter which is USER_ID.
Closing the issue for now. If you experience any other issues, let me know, or feel free to contribute to the project


[Bug] AGPS update loading icon doesn't disappear

When updating AGPS from the App Loader, once it is complete the upload icon continues to show the loading wheel, even though it has shown as complete in the lower banners.

I noticed the same issue with Languages app also 🤔

I imagine it's related to custom apps then... Maybe the promise doesn't complete?


Performance improvement on ChainedBlock - Implement SkipList

Ported over skip list logic from bitcoin core and related unit tests. Significantly improves efficiency of ChainedBlock.GetAncestor(), ChainedBlock.GetLocator(), and ChainedBlock.FindFork() methods currently. Further optimizations to follow.
Partially addresses #659

@mikedennis I'm unable to see ChainedBlockVerifySkipListForGetAncestor test in the list of tests in MSVS Test Explorer, I'm thus unable to run it

I think a bug in visual studio... I moved the tests up inside the class and now it shows up :confused:

I will let @Aprogiena commit when he is happy with the test.

@fassadlr @aprogiena are you guys familiar with skip lists implementation, or is this your own stab at it?
Just asking, I haven't looked into it yet to make a judgement.

@bokobza the GetAncestor code is port from Bitcoin Core, which reads as
CBlockIndex* CBlockIndex::GetAncestor(int height)
{
    if (height > nHeight || height < 0)
        return nullptr;

    CBlockIndex* pindexWalk = this;
    int heightWalk = nHeight;
    while (heightWalk > height) {
        int heightSkip = GetSkipHeight(heightWalk);
        int heightSkipPrev = GetSkipHeight(heightWalk - 1);
        if (pindexWalk->pskip != nullptr &&
            (heightSkip == height ||
             (heightSkip > height && !(heightSkipPrev < heightSkip - 2 &&
                                       heightSkipPrev >= height)))) {
            // Only follow pskip if pprev->pskip isn't better than pskip->pprev.
            pindexWalk = pindexWalk->pskip;
            heightWalk = heightSkip;
        } else {
            assert(pindexWalk->pprev);
            pindexWalk = pindexWalk->pprev;
            heightWalk--;
        }
    }
    return pindexWalk;
}

the only thing we are trying to do is make that code more readable without changing the functionality
so it does not matter if one is familiar with how skip lists work (although i am), this is just making equivalent (or semi-equivalent) changes

Excellent! that's what I was about to say: if none of us know exactly how to implement skip list, then let's just do what Core does without trying to be too fancy.

Why does skip list is chosen over SortedDictionary<TKey, TValue>?
Take a look at this SO question.

I like @fassadlr version, I don't think it's too long. I've got some good unit tests so I'm ok making a change like this. Thanks for the doing that, I looked at the boolean and thought it looked complex enough that breaking it apart would be more complicated than the original but you guys did a nice job and I think that does make the code a lot more readable.

ok go for it

just pls double check that whatever you end with is not changing a functionality (i.e. that it still works)

Ok see what you think of this. @fassadlr version with a couple of my own improvements.

Looks good @mikedennis! Way easier to read now 👍

i'm having a lot of tests failed in memory pool and consensus - null reference in GetAncestor ... i think you forgot to double check ...


Static IP address not working

Describe the bug
Static IP address not working
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Screenshots
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context

No fault found


make spring-cloud-sleuth-zipkin-stream a module

In zipkin, we sometimes get support issues where people are creating custom zipkin servers as step 1. Just like monitoring users don't start by making a custom Prometheus server, we should find a way to stop encouraging this here. The end goal is to simplify the first tracing experience of sleuth users and lessen the support burden on OpenZipkin.
In short, custom builds are neither the fastest past to learning how to do tracing, nor a good way to leverage the OpenZipkin community. The fact that Zipkin is built with Spring Boot is only a convenience for server developers and extenders. It has never been a day 1 story.
Why are people creating custom zipkin servers?
Zipkin server developers themselves and a handful of advanced users will need custom servers. Beginners do not need a custom server, but they are much more often the ones asking for help making one. Most of this comes from historical @joshlong videos, which start zipkin by making a custom one with start.spring.io. I spoke with josh about this, and hopefully new videos will stop suggesting that. The only other source of custom builds from beginners comes from spring-cloud-sleuth-zipkin-stream. Once we remove this here we stopped promoting custom builds as step 1 in spring videos, and stopped requiring custom builds here, we'd really have no problem anymore.
But.. I like start.spring.io!
start.spring.io is good for sleuth, ex how do you get spring tracing in your app. On the other hand, it would be surprising to think beginners should need a custom Zipkin server. This has had numerous support impact including people asking for 1-1 sessions on how auto-configuration works, not getting their classpath or eclipse to work, etc. Our non-java developers know how to help with the supported means to configure zipkin, albeit from a jar or a docker pov. Many people can help with commodity support. However, Spring Boot support is neither necessary, nor commodity in our channel. In many cases, this leads to people personally chatting with me only to later find out they had little idea what tracing even is, and were only making a custom server because of a video or more rarely the stream server. This ends up low value activity, which adds cognitive debt to the beginner as starves the OpenZipkin project of my time. Personal time aside, most people in the Zipkin channel are not java users. Pasting classpath dumps or eclipse problems is distracting there, especially when a custom server isn't needed.
Ok.. I get helping zipkin would be nice, but what's in it for Sleuth?
Between the time we started in sleuth and now, a lot of progress has been made with collectors. By revisiting spring-cloud-sleuth-zipkin-stream, not only can Zipkin have less support burden, but also sleuth I think, too. For example, users who ask for things like Elasticsearch support wouldn't need special instructions. People wouldn't need to read or write blogs in order to get things to work. They wouldn't end up on Zipkin channel with few people able to help them with Eclipse. The first time experience with tracing will be easier and less confusing as the steps to get started would no longer include understanding how to build a custom telemetry server.
Hmm.. ok, but what's next?
I'd like to use this issue to help track migrating scs-zipkin-stream to a "normal" 3rd party collector, such as we have with zipkin-aws or zipkin-azure (possibly stackdriver-zipkin will also use this process in the future).
This would result in scs-zipkin-stream being a "normal" collector, even if we still make an exec jar here for convenience.

Here are some counter-points and alternatives to rebasing as a zipkin collector
Is scs-zipkin-stream primarily covering for a lack of RabbitMQ support upstream?
At least in almost 4 years of issues and mailing lists, Zipkin has never had a request for RabbitMQ support. However, rabbit is often used in Josh's demos and maybe some Zipkin sites. I have seen one other project use RabbitMQ. If there's an invisible need for RabbitMQ, we could bubble that up.
scs-zipkin-stream features are tied to spring cloud releases (and implicitly spring boot)
Currently, scs-zipkin-stream looks and acts like a zipkin collector. However, it uses the sleuth model. While this model is relatively stable, it is tied to classes and change requests tied to spring cloud releases and those are also tied to specific spring boot releases. While the sleuth model is used, it could cause more complication to primarily tie to zipkin's spring boot, which isn't coordinated with spring cloud releases.
scs-zipkin-stream is more about spring cloud stream and sleuth than zipkin.
While there is functional overlap with zipkin collector, understanding spring cloud stream is the primary thing to "get" with this vs a usual zipkin collector. For example, this bleeds into how messages are or aren't enveloped over transport etc, and what features are compatible with others. Keeping this primarily pinned to other spring cloud projects (vs an external one like zipkin), could help ensure that from a feature POV, things stay aligned across spring cloud stream and sleuth.
Final notes..
There are ways to reduce support burden and increase data store support besides re-using zipkin-aws or similar integration. For example, eventually new spring videos may stop suggesting start.spring.io for starting zipkin server (even better if zipkin server was removed). This would get most of the low-value troubleshooting out of the way. Also, we could do some things here to formalize or test more common back-ends, like Elasticsearch, so that users don't need blogs or 1-1 support to figure out how to do this. In other words, an alternate route could be of high value, and not require a major change.

Is scs-zipkin-stream primarily covering for a lack of RabbitMQ support upstream?

No. It is primarily to allow apps to quickly pivot away from any particular middleware (e.g. start with Rabbit and move to Kafka with just a change of classpath). This is a feature of Spring Cloud Stream, not really of Sleuth per se.
Note that Cloud Foundry does not have a native Kafka service, but it does have Rabbit. So that explains why we want Rabbit, but also would like to make it as transparent, and replaceable as possible.
I don't think users care what the format of the data is on the wire, so moving to a native Zipkin format would make perfect sense, as long as we retain interop with older clients (IIRC this wasn't possible when we started sleuth because Zipkin was still tied to Thrift). Equally, moving the whole thing to openzipkin wouldn't matter, as long as it can synchronize releases to support major Spring Cloud/Boot updates.

Thanks for the insight, Dave. I'll chat with Marcin about this since seeing
him tomorrow as well.
Cheers!

The problem was fixed by keeping the module in Sleuth but we've removed the creation of Zipkin Http from initilizr so as not to make people confused.


removed convoy gluster but still visible in GUI

I removed the convoy gluster stack under system, but when i navigate to infrastructure > storage it still shows up there (see screenshot)

Useful
Info

Versions
Rancher v1.1.3 Cattle: v<IP_ADDRESS> UI: v1.1.12

Access
github admin

Orchestration
Cattle

Route
storagepools.pools

Currently, we don't have the ability to delete storage pools.
https://github.com/rancher/rancher/issues/2918


added my name to guestbook.md

Great course!

Thank you!

Thanks a los. Great course
El 05/12/2017 20:44, "Sean Kross"<EMAIL_ADDRESS>escribió:

Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/seankross/the-unix-workbench/pull/135#issuecomment-349435877,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQRlaGmqjKieNZwkIBSCI_sdlkluC7kMks5s9atAgaJpZM4QzgFa
.


Microsoft/users/aujensen/vs2019 vs task (#9273)

Backport of VS2019 support for VSBuild and MSBuild tasks, because I've been asked to backport for RTW

@austin-jensen We should have someone from your team approve.


Adding Refactory

:clipboard: Refactory and update code
Overview
The following pull request includes a refactoring of many functions as well as updating the code to avoid repeated code, where possible, config-driven was applied to avoid the use of nested switches/ifs.
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

BridgeFees : The repeated and duplicated code was removed, a map was made over the different options that simply renders the content on demand.
Sevens : A restructuring of the current map was applied, unnecessary conditionals were avoided, and duplicate code was prevented."
Connect: Unnecessary if statements were removed, and the onClick functions were separated into an external function.
Earn : A function was created to avoid code repetition, the if statements were simplified, especially the nested ifs.
CDMCompletionModal : The current code was transformed into a config-driven approach, avoiding code repetition and multiple if/conditional statements.
Faucet: Unnecessary code was reduced, ternary operators were used to simplify nested conditionals.
Gas services : Object destructuring was applied and individual promises were replaced with a complete promise.
GraphQl Services: The comments were removed and the use of ternaries was implemented to simplify the code.
NetworkServices: The use of config-driven was implemented and unnecessary nested conditionals were removed.
WalletServices: The use of nested if statements was removed to simplify the code reading as well as the functionality of the methods.
Amount Convert : Config-driven was applied to eliminate the use of if statements, give more meaning to the method, and remove unnecessary code.
Calculation : The use of switch was removed and the mathematical operations were componentized to give better performance and maintainability to the code.
CoinImage: The use of Swift was replaced by a config-driven version.
Common.js : A function was created to filter the list of available tokens, making it more readable and reusable, as well as simplifying the base code.
Nft Images : The code was simplified and unnecessary if statements were removed.

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

Codecov Report
Patch coverage has no change and project coverage change: -2.61 :warning:

Comparison is base (13b9b0c) 53.68% compared to head (5514579) 51.07%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #667      +/-   ##
===========================================
- Coverage    53.68%   51.07%   -2.61%
===========================================
  Files          156      171      +15
  Lines         6454     7553    +1099
  Branches      1272     1536     +264
===========================================
+ Hits          3465     3858     +393
- Misses        2989     3695     +706

Flag
Coverage Δ

coverage
51.07% <ø> (-2.61%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 19 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Missing Debug a custom action section

Missing Debug a custom action section. The information is not documented anywhere else. In the previous docs for Composer 1.x it was mentioned to:

In the Attach to Process window, select Microsoft.BotFramework.Composer.WebApp.exe from the
Available processes section. Select Attach.

But the Microsoft.BotFramework.Composer.WebApp.exe does not exist when using Composer 2.0

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b45a275f-7b99-3611-589c-76774e23b35f
Version Independent ID: 5c85f403-d998-ff31-a08e-6bb9d3a8cdb9
Content: Create custom actions - Bot Composer
Content Source: composer-docs/how-to-create-custom-actions.md
Service: bot-composer
GitHub Login: @clearab
Microsoft Alias: anclear

Following up with a subject matter expert.

Thank you. The problem still exists. I tried it today with Composer 2.1.1.

Closing due to age. If this is still an issue, please re-open this request with updated details.


feat: Chat example

Demonstrates a few features:

How to create a custom UI
How to use commands and clientrpc
How to create a custom user

What's up with the UI in those screenshots?
Why not use Unity's built in style? It does look a lot nicer :)

What's up with the UI in those screenshots?
Why not use Unity's built in style? It does look a lot nicer :)

Those screenshots are outdated.

@vis2k the point is demonstrating a UI using UnityUI instead of obsolete OnGUI.   This is the out of the box look for Unity UI


add findLongestPrefix

I like Data.List.stripPrefix. regex-applicative calls their generalization of it findLongestPrefix.

Might I ask what is your use case for this?
It's not clear to me whether you're looking for a stripPrefix or stripLongestPrefix.

I use findLongestPrefix in this haskell source preprocessor. Elsewhere in that file I use Data.List.stripPrefix. That HListPP is probably obsoleted by extensions like OverloadedRecordDot, but I'm working on a similar preprocessor adding c syntax. Because Regex.List.find already conflicts with Data.List.find, I think you could add
stripPrefix :: RE c a -> [c] -> Maybe [c]
findPrefix :: RE c a -> [c] -> Maybe (a, [c])

In my case, I would use Regex.List.findPrefx because I have to keep track of the input that was consumed.

Thanks for the link to your code. I've also thought about this function for a bit.
To give a simple answer, one possible definition of findPrefix is
findPrefix :: RE c a -> [c] -> Maybe (a, [c])
findPrefix re = reParse (liftA2 (,) re manyList)

It is easy to define by a user who needs it.
But I am reluctant to add it to the library, because doing this operation repeatedly would be inefficient. And it does seem to encourage that pattern. It is inefficient because it reconstructs the tail of the list after the match. Doing this repeatedly is quadratic in the length of the list.
So why not add a separate runner that tries to preserve the tail, like regex-applicative's findLongestPrefix?
Because

There is currently only one way to run a regex on a list, which is parse, and that runs on the whole list. Any other function present for convenience (reParse, find, replace, etc.) uses it under the hood. This simple model is easy to understand and I like it.
Even a separate runner like regex-applicative's findLongestPrefix can take linear time in the worst case, depending on the regex and the list. Neither of these libraries lend themselves well to the style of "parse a prefix of a list, then continue to parse the rest". (Side note: Data.List.stripPrefix does fit well in this pattern, it is only linear in the prefix, and does not touch the rest of the list)

Now, if I had to implement your preprocessor's takeQual with parser-regex, I would not use the above definition of findPrefix because of the described inefficiency.
Instead, I would try one of two things:

Extract the token separately from parsing it.

takeQual x = case span isValidIdentChar x of
  (y, rest) -> case reParse qualIdent y of
    Nothing -> (y, rest)
    Just a -> (addLabel a, rest)

isValidIdentChar = ...

Or, do away with the manual parsing bits and put everything in a regex

re = fmap concat $ many $ asum
  [ string
  , char
  , comment
  , addLabel <$> qualIdent
  , anySingle
  ]

comment = toMatch $ list "{-" *> manyListMin *> list "-}"
string = ...
char = ...

I've looked around a bit more on this issue, and I learnt that this is a common technique for tokenization/lexing of languages. It seems that the quadratic complexity does not usually pose a problem, perhaps because 1. the language is static and if lexing takes forever you would notice, and 2. you would need a large enough source file to see the difference.
While I can now see better why someone would want findLongestPrefix, I still don't like the idea of putting a potential footgun in the library.
For lexing in particular, parsing using many tokenRe might be an alternative. The only thing you lose this way is streaming, i.e. generating tokens one at a time.
Should someone really need it, it is possible to define findLongestPrefix for lists using the primitives in Regex.Base. It is not possible for Text currently, but I will expose the internals in the next release to make this possible.

I'll leave a concrete example of how it's quite easy to shoot yourself with findLongestPrefix.
import Data.List
import Text.Regex.Applicative

re = sym 'a' <|> (many (sym 'a') *> sym 'b')

-- O(n^2)
parse :: String -> String
parse = unfoldr (findLongestPrefix re)

n = 100000

main = print $ parse (replicate n 'a')

The same is true for various lexing libraries. Here is an alex program with the same behavior.
{
}

%wrapper "basic"

tokens :-
  a   { \_ -> 'a' }
  a*b { \_ -> 'b' }

{
-- O(n^2)
parse :: String -> String
parse = alexScanTokens

n = 100000

main = print $ parse (replicate n 'a')
}

The readme has:

parser-regex attempts to be a more fully-featured library built on the ideas of regex-applicative.

Maybe it should be something like:

parser-regex attempts to be a more principled but less fully-featured library built on the ideas of regex-applicative.


Reference

We need to add a reference to the notification serverhttps://concordium.atlassian.net/wiki/spaces/EN/pages/1502019590/Message+data+payload

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Cleanup global styles

Tested - looks good.
Note: not all fields are translated into Chinese (Tokens, Permissions settings). This bug is also on the developer version.

I wouldn't say it's a bug, it's just that we lack translations for all of the text on the wallet.

Actually, the broadest change is to renaming common to styles that modifies a lot of components but it should be easy to rebase on top of that.


replace 1st person model with Real Camera

more a suggestion than an issue, but if you used Real Camera in the pack you would be able to remove 1st person model (which in my experience has usually ended up being a bigger resource drain than physics has) and shoulder surfing(covered by Real Camera's classic mode so remove or leave in at your discretion, not everyone who want's shoulder surfing style play is gonna want to have to turn it on in mod settings to get it), and possibly even not enough animations (though that one is up to preference since I don't know if real camera does anything to fill in gaps in the third person animations)

Sorry I kinda stopped playin Wynncraft for a while so I didnt think about checking on my pack xD. Ill look into it and let u know in a bit. Might be a while before me and Loyd update tho bc we dont rly play Wynncraft anymore (he vanished like my father or somethin idfk).

Nvm bro is fast af. Updating now

Yup, it should be released everywhere within the next 1-2 hours!

These updates are being pushed to Modrinth, Curseforge, and ScytedTV Resources. Curseforge may take a little time to verify the changes, but it should be available everywhere else.


Editor Menu Buttons remain in the pressed state

Godot version:
3.0 current master 03823f1 (NOT in 3.0 beta, where it works well)
OS/device including version:
Winows 8.1
Issue description:

[ ] When you click the Menu buttons in the editor, they get "pressed" and become blue (default theme), but they remain like this even when not pressed anymore.

[ ] If a menu is open, the next click in the engine window will close the menu, but will be ignored.

I think that bug appeared approximately 24h ago. It is theme independant.
Steps to reproduce:
Open any project, and click on the Menu Button Scene, Project, Debug, Editor, Help

Duplicate of #14854

Oops! Sorry about that...


Create NuGet package

NuGet package stakx.WIC (version 0.1.0) has been published: https://www.nuget.org/packages/stakx.WIC/.


Early failing ignite new should have rollback

There's nothing useful in an early failing ignite new

The TestProj folder has an npm-debug.log which is useless.  We should have just cleaned up after ourselves.

This won't fail anymore because we've published ignite-ir-boilerplate-2016 on npm.


refactor: use core drag drop event data for non-nested effects

A rehash of #100, preferring to use the same AE data transfer as core when not dragging a nested effect.
This change allows non-nested effects to be handled by Core's own drag and drop handlers.
When coupled with the Drop Effects on Items module, this allows convenient effects to be placed directly onto items. When further coupled with Item Effects to Chat D&D5e, these effects can be applied directly from chat with no further handling on this module's part.
Video demo of what this change allows with those two modules also enabled in the world:
https://user-images.githubusercontent.com/7644614/153972573-c855947b-3a28-484b-b681-566665c910bc.mp4

DND5e now handles dragging and dropping active effects from character or item sheets onto the macro bar, which creates a macro that enables or disables that effect. This is causing two macros to be created (one from dnd5e and one from CE) because of this PR converting an effect from CE to a Foundry ActiveEffect on drag.
As seen in dnd5e:
// in Hooks.ready in dnd5e.js
  Hooks.on("hotbarDrop", (bar, data, slot) => macros.create5eMacro(data, slot));

// in macros.js in dnd5e
export async function create5eMacro(data, slot) {
  const macroData = { type: "script", scope: "actor" };
  switch ( data.type ) {
    case "Item":
      if ( !("data" in data) ) return ui.notifications.warn(game.i18n.localize("MACRO.5eUnownedWarn"));
      foundry.utils.mergeObject(macroData, {
        name: data.data.name,
        img: data.data.img,
        command: `game.dnd5e.macros.rollItem("${data.data.name}")`,
        flags: {"dnd5e.itemMacro": true}
      });
      break;
    case "ActiveEffect":
      if ( !("data" in data) ) return ui.notifications.warn(game.i18n.localize("MACRO.5eUnownedWarn"));
      foundry.utils.mergeObject(macroData, {
        name: data.data.label,
        img: data.data.icon,
        command: `game.dnd5e.macros.toggleEffect("${data.data.label}")`,
        flags: {"dnd5e.effectMacro": true}
      });
      break;
    default:
      return;
  }

  let macro = game.macros.find(m => (m.data.name === macroData.name)
    && (m.data.command === macroData.command) && (m.data.author === game.userId));
  if ( !macro ) macro = await Macro.create(macroData);
  game.user.assignHotbarMacro(macro, slot);
  return false;
}

I don't want to break this integration in your demo by reverting back to just passing the effectName around, but I also don't have a solution since I can't stop the dnd5e hook from firing nor can I override the create macro function. Any ideas?

Relevant PR that "fixes" the double macro issue but breaks this integration: https://github.com/DFreds/dfreds-convenient-effects/pull/155


Pol 934

I'm happy to merge this PR. As you probably know i'm not using CsQuery any more and so this is really just an administrative activity for me, since I can't really review it. If you are interested in becoming a collaborator so you can commit directly to master let me know.

Thank you, but I created the pull request by mistake. We forked the repo and are applying occasional bugfixes to our forked version. I meant to create a pull request against that. It's very rare that we make any changes. The fork is available at http://www.github.com/siteimprove/csquery in case it has any interest.


Fix E2502 triggering on AWS::EC2::LaunchTemplate

E2502 is triggering for AWS::EC2::LaunchTemplate as well.
Question: could it ignore AWS::EC2::*?
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Not sure why the build is failing after this change.
FAILED (failures=1)
ERROR: InvocationError for command '/home/travis/build/awslabs/cfn-python-lint/.tox/py27/bin/python -m unittest discover' (exited with code 1)
___________________________________ summary ____________________________________
ERROR:   py27: commands failed

Hi! Thanks for the contribution!
Yeah, there’s a sporadic bug in one of the  tests that manifest from time to time. Re-running tests now and suspect it will run clean. I’ll create an issue on the broken test and do a code review on this when I get to my desk.

@cmmeyer no problem. Glad it was just a blip.


TSRCollector fails on non English Language. Idrac8 fails. Some 15g Fails.

Fails on non english Language. Idrac8 fails. Some 15g Fails.

Updated to use SID for language compatibility


Finding unracked items

Environment

Python version:  3.5.2
NetBox version:  2.5.5

Proposed Functionality
I would like to be able to identify all devices in a site which are "unracked" - i.e. they have been assigned to a rack but have no rack position recorded.  Examples would be devices on shelves.
Currently the the way I am doing this is to browse to the detail page for each rack in turn, and look at the the list of unracked devices down the page.  This is tedious across many racks.
I have thought of some possible approaches:

A new search filter, e.g. /dcim/devices/?position=null.  This would make the search column even busier than it already is.
In the Devices list, change the "rack" column to show "rack and position": e.g. instead of "407" show "407 U34 F".  Then if you sort on this column, you'd see all the devices grouped by rack, and the unracked ones grouped together too
In the Rack Elevations view, show a list of the unracked devices underneath each rack

Use Case

Rack Elevations are a handy visual guide to the equipment in each rack, but are missing information about unracked equipment so don't provide a complete list of all the devices
Quick data quality check - identify devices which should have been given a position

Database Changes
None
External Dependencies
None

@candlerb, /dcim/devices/?rack_id=null already does that. If you're using the selector on the right side of the page, then search for None.

I'm not looking for rack_id=null.  I want to search for devices which have been assigned a rack, but not a position within the rack.
These are the ones listed as "Non-Racked Devices" on the detail page for a given rack /dcim/racks/<id>/

There seems to be a limitation in django_filters where it won't allow a null value for a NumberFilter. Not sure how best to get around that.

This now works if I manually construct a URL like /dcim/devices/?site=<name>&position=null to find these devices in a site.  But there doesn't seem to be a way to do this via the UI in the Search box. (Netbox 2.5.9)


Add debug single test

Summary

Test plan

Not sure about this really, there's quite a bunch of "debugging" in VSCode now, I don't use it too often so hard to tell. @Andarist maybe you can advise here?

Seems to be pretty similar to what I have been using with success:
    {
      "type": "node",
      "request": "launch",
      "name": "Jest Current File",
      "program": "${workspaceFolder}/node_modules/.bin/jest",
      "args": ["${fileBasenameNoExtension}", "--config", "jest.config.js"],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen",
      "disableOptimisticBPs": true,
      "windows": {
        "program": "${workspaceFolder}/node_modules/jest/bin/jest"
      }
    }

@Andarist - that was the original one (found in some googling...). However, I had to change ${fileBasenameNoExtension} to ${file} since there I had  2 tests in the format of a-b-c and b-c, and when trying to focus on b-c it was also attempting to run a-b-c.
I found this setup extremely useful and wanted to share the love, but I do respect @thymikee concern.

I'm fine with this if it's useful to people 🙂
But maybe we could change it to some tab format so it doesn't take up as much space? https://docusaurus.io/docs/markdown-features/tabs
And then have "single file" (and not) as separate tabs?


Automatically add needed pluigns to Jmeter test

Description
Nowadays, we should always have CSV writers as part of the test to have metrics available in Maestro. It might be confusing, especially for newcomers why we need to do it and also can be misleading as without a writer the test would be stopped after 2 minutes.
Solution
I see two things that we could improve here:

[ ] Make waiting for metrics file timeout optional, disabled by default. In that case we always will show running time and users can use different backend with Maestro (like InfluxDB + Grafana) which is so popular.
[ ] Add configuration where the writer would be added to the test automatically. In that case, it will fully invisible for users. All tests would not require any configuration to run on the Maestro.

This is sort of related to #54. We already testing backend listener to send metrics directly from Jmeter. It makes sense for me to close this issue.
We already introduced adding Backendlistener to the running test in #220


/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/ImGuiColorTextEdit/TextEditor.cpp:932:85: error: cannot initialize a parameter of type 'ImGuiKey' with an rvalue of type 'char'
                else if (!IsReadOnly() && !alt && ctrl && !shift && !super && ImGui::IsKeyPressed('['))
                                                                                ^~~
/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/imgui/imgui.h:924:51: note: passing argument to parameter 'key' here
    IMGUI_API bool          IsKeyPressed(ImGuiKey key, bool repeat = true);                     // was key pressed (went from !Down to Down)? if repeat=true, uses io.KeyRepeatDelay / KeyRepeatRate
                                                  ^
/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/ImGuiColorTextEdit/TextEditor.cpp:934:85: error: cannot initialize a parameter of type 'ImGuiKey' with an rvalue of type 'char'
                else if (!IsReadOnly() && !alt && ctrl && !shift && !super && ImGui::IsKeyPressed(']'))
                                                                                ^~~
/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/imgui/imgui.h:924:51: note: passing argument to parameter 'key' here
    IMGUI_API bool          IsKeyPressed(ImGuiKey key, bool repeat = true);                     // was key pressed (went from !Down to Down)? if repeat=true, uses io.KeyRepeatDelay / KeyRepeatRate
                                                  ^
/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/ImGuiColorTextEdit/TextEditor.cpp:952:165: error: use of undeclared identifier 'ImGuiKey_KeyPadEnter'; did you mean 'ImGuiKey_KeypadEnter'?
                else if (!IsReadOnly() && !alt && !ctrl && !shift && !super && (ImGui::IsKeyPressed(ImGui::GetKeyIndex(ImGuiKey_Enter)) || ImGui::IsKeyPressed(ImGui::GetKeyIndex(ImGuiKey_KeyPadEnter))))
                                                                                ^~~~~~~~~~~~~~~~~~~~
                                                                                ImGuiKey_KeypadEnter
/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/imgui/imgui.h:1366:5: note: 'ImGuiKey_KeypadEnter' declared here
    ImGuiKey_KeypadEnter,
    ^
3 errors generated.
make[2]: *** [libs/CMakeFiles/IMGUI.dir/ImGuiColorTextEdit/TextEditor.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
[ 39%] Linking CXX static library libopentime.a
[ 39%] Built target opentime
/Users/omakazu/program/01_multi_project/0001_DaVinci_Resolve_Conform_Tool/zzz_test_project/raven/libs/nativefiledialog/src/nfd_cocoa.m:56:17: warning: 'setAllowedFileTypes:' is deprecated: first deprecated in macOS 12.0 - Use -allowedContentTypes instead [-Wdeprecated-declarations]
        [dialog setAllowedFileTypes:allowedFileTypes];
                ^
/Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSSavePanel.h:215:49: note: property 'allowedFileTypes' is declared deprecated here
@property (nullable, copy) NSArray<NSString *> *allowedFileTypes API_DEPRECATED("Use -allowedContentTypes instead", macos(10.3,12.0));
                                                ^
/Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSSavePanel.h:215:49: note: 'setAllowedFileTypes:' has been explicitly marked deprecated here
[ 40%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_opentime.dir/utils.cpp.o
[ 41%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_opentime.dir/test_opentime.cpp.o
1 warning generated.
[ 42%] Linking C static library libnativefiledialog.a
[ 42%] Built target nativefiledialog
[ 43%] Linking C static library libglfw3.a
[ 43%] Built target glfw
[ 43%] Linking CXX static library libImath-3_2.a
[ 43%] Built target Imath
[ 44%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/composable.cpp.o
[ 45%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/clip.cpp.o
[ 46%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/composition.cpp.o
[ 47%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/deserialization.cpp.o
[ 48%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/effect.cpp.o
[ 48%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/algo/editAlgorithm.cpp.o
[ 49%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/errorStatus.cpp.o
[ 50%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/externalReference.cpp.o
[ 51%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/gap.cpp.o
[ 52%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/freezeFrame.cpp.o
[ 54%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/mediaReference.cpp.o
[ 54%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/marker.cpp.o
[ 54%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/item.cpp.o
[ 55%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/imageSequenceReference.cpp.o
[ 56%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/generatorReference.cpp.o
[ 57%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/missingReference.cpp.o
[ 58%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/linearTimeWarp.cpp.o
[ 59%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/stack.cpp.o
[ 60%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/stringUtils.cpp.o
[ 61%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/serializableObjectWithMetadata.cpp.o
[ 62%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/serializableObject.cpp.o
[ 63%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/safely_typed_any.cpp.o
[ 64%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/serializableCollection.cpp.o
[ 65%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/stackAlgorithm.cpp.o
[ 66%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/transition.cpp.o
[ 67%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/track.cpp.o
[ 68%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/timeEffect.cpp.o
[ 69%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/CORE_VERSION_MAP.cpp.o
[ 69%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/typeRegistry.cpp.o
[ 70%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/timeline.cpp.o
[ 70%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/serialization.cpp.o
[ 71%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/unknownSchema.cpp.o
[ 72%] Building CXX object libs/opentimelineio/src/opentimelineio/CMakeFiles/opentimelineio.dir/trackAlgorithm.cpp.o
[ 73%] Linking CXX executable test_opentime
[ 73%] Built target test_opentime
[ 74%] Linking CXX static library libopentimelineio.a
[ 74%] Built target opentimelineio
[ 75%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_track.dir/utils.cpp.o
[ 76%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_serializableCollection.dir/test_serializableCollection.cpp.o
[ 77%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_editAlgorithm.dir/utils.cpp.o
[ 77%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_serialization.dir/test_serialization.cpp.o
[ 78%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_serializableCollection.dir/utils.cpp.o
[ 78%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_editAlgorithm.dir/test_editAlgorithm.cpp.o
[ 79%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_timeline.dir/test_timeline.cpp.o
[ 80%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_timeline.dir/utils.cpp.o
[ 81%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_track.dir/test_track.cpp.o
[ 82%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_clip.dir/test_clip.cpp.o
[ 83%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_clip.dir/utils.cpp.o
[ 84%] Building CXX object libs/opentimelineio/tests/CMakeFiles/test_serialization.dir/utils.cpp.o
[ 85%] Linking CXX executable test_serialization
[ 85%] Built target test_serialization
[ 86%] Linking CXX executable test_timeline
[ 87%] Linking CXX executable test_track
[ 88%] Linking CXX executable test_serializableCollection
[ 88%] Built target test_timeline
[ 88%] Built target test_track
[ 88%] Built target test_serializableCollection
[ 89%] Linking CXX executable test_clip
[ 89%] Built target test_clip
[ 90%] Linking CXX executable test_editAlgorithm
[ 90%] Built target test_editAlgorithm
make[1]: *** [libs/CMakeFiles/IMGUI.dir/all] Error 2
make: *** [all] Error 2

what error does it happen?

macOS Ventura 13.4
M1 Max
cmake version 3.28.2

Hmmm.. can you check these things:

Which version of Xcode do you have?
What does git status report?
Could you run git submodule update and try again?

I run git submodule update , it build sucessfully.
Thank you for helping me.

xcode : 14.3.1

You're welcome :)


github actions not updating website on commits

when content .md pages are updated either through github interface or pushed from a local repo through command line, the github actions runs, but pages are not being built & deployed (see last commit, eg, a314c6c)
See this run: https://github.com/cu-mkp/dyngleyfamily-editioncrafter-website/actions/runs/8333171340
is this deprecation at fault?
Annotations
1 warning
deploy
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, peaceiris/actions-hugo@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

To demonstrate, I've made a change to the about.md in the dev branch https://github.com/cu-mkp/dyngleyfamily-editioncrafter-website/commit/0ae36915344978359db75c4b7029e9dfd42b0280 which is no appearing in the generated website at https://cu-mkp.github.io/dyngleyfamily-editioncrafter-website/about/

btw, @melissabreynolds to make changes to the transcription, edit the TEI file in the gh-pages branch of the https://github.com/cu-mkp/dyngleyfamily-editioncrafter-data repo, then run the editioncrafter-cli  (https://github.com/cu-mkp/editioncrafter-cli) and add/commit/push, which will trigger the publication of that content. We can review the process and help setup editioncrafter-cli if you'd like.

Thanks for this info, @tcatapano. I've finally gotten around to creating the "About" and "Introduction" pages for the Dyngley Family site, which I know will need to be updated in the dev branch. I went ahead and made changes in the main branch first, but should I even bother? I thought I might simply create a pull request in the dev branch to merge the two and import my changes in "main" to "dev." Does that sound like a good or bad idea?

@melissabreynolds : I was about to say, yes, make the changes on dev, but I see you've already done that and things are working. Congrats! https://cu-mkp.github.io/dyngleyfamily-editioncrafter-website/introduction/ https://cu-mkp.github.io/dyngleyfamily-editioncrafter-website/about/

We can take on aligning the branches later...


Use multiformat to represent hash and public key (and signature?) values

Multiformat has an extensive list of data types and encodings, which would allow compact representation of relevant cryptographic primitives.  This is a reasonably widely used format (e.g. in IPFS for hash-valued content identifiers), and is reasonably well-defined, and is extensible.
Use multiformat instead of KERIHash and KERIVerifier, since the latter two are more limited in the kinds of hashes and key types they can represent.
Question: Does multiformat specify formats for cryptographic signatures?  E.g. ed25519-sha-512, secp256k1-sha-256, etc?  multicodec?  I'm not finding an obvious listing.

Followup notes:

https://crates.io/crates/multibase - popular Rust crate supporting multibase
https://crates.io/crates/multihash - popular Rust crate supporting multihash
https://github.com/cryptidtech/multisig and https://github.com/cryptidtech/multikey seem to be the only Rust implementations of multisig and multikey.

Other considerations for use of multiformat:

How widely are the various components (multibase, multihash, multikey, multisig) supported in other programming languages?  I.e. how difficult would it be to create a did:webplus implementation (even just the DID resolution portion) in a given target language, if multiformats were used?

Oh, sorry, I seemed to have "watched" this repo after the initial issue was opened and just saw this.
Background
Multiformats is pretty easy to implement a tiny subset of in different languages-- there ARE libraries in every major language that implement some chunky subset of the multicodec table, although in some languages the various subsets like multihash and multiaddr are implemented separately (with different processing steps and ergonomic/efficiency tradeoffs appropriately).
multibase and multikey are two special cases-- multibasing is an optional step (and technically is NOT a "subregistry" of multicodecs in the IANA sense of registries and subregistries).  In most languages/IPFS contexts, the multibase library is broken out from the multiformats/multiaddr/multihash library, and not exported where it isn't used.  think of it as a transport-layer convenience, where ASCII is needed; everywhere binary formats and streams are fine, it's not used.  since a few years before CIDv1, it's been the case in practice (and only recently in the official specs) that CIDs are officially a binary format, with multibase a recommended (but optional) transformation for contexts like the HTTP where ASCII makes more sense.
multikey is a special case for the simple reason that it was never structural in IFPS (due to IPNS and other ways of representing keys) so it was never really spec'd out.  the W3C CCG and DID WG specs use publicKeyMultibase which is not really specified in multiformats or used in IPFS.
the cryptidtech implementations are intended as a general-purpose crates, but I haven't reviewed them in detail.  i CAN say that cryptidtech is intimately aware of the KERI approach and I think it dave huseby would be more than happy to walk you through any nits or cornercases of expressing KERI signature formats or canonicalization through those crates.  I don't know if parity with KERI is a goal of those libraries specifically but the use-cases of webplus and keri are familiar to their designer so it wouldn't take him/them very long to answer well-formulated questions.
i trust them to be loyal to the spec and work for all the test vectors in the multiformats repos, but admittedly the conformance regimen for all this is... those test vectors.  mostly corner cases that arise in IPFS get filtered up to multiformats eventually, but multikey doesn't have that implementation history so i'm not as confident we've found all the corner-cases yet.  presumably if a multiformats WG forms at IETF there would be that feedback loop for any multi- libraries/microprotocols that members of the WG volunteer to implement and cross-test.
Concrete next steps

how difficult would it be to create a did:webplus implementation (even just the DID resolution portion) in a given target language, if multiformats were used?

not very hard, I think-- there is implementation history across many languages on the multihash and CID side of things, so the core mechanisms are pretty language-agnostic and mature at this point.  the harder part is just the universal quirks of representing keys and signatures, which doesn't necessarily go away if you use some other toolchain or family of formats from, e.g. CBOR or JW* traditions.

Does multiformat specify formats for cryptographic signatures? E.g. ed25519-sha-512, secp256k1-sha-256, etc? multicodec? I'm not finding an obvious listing.

this is... an ongoing debate among implementers.  some companies got together and proposed defining together a dictionary of IPLD-formatted (i.e. multiformats-encoded) signatures, but that project is still unstable and on hold at the moment, because those use-cases are mostly downstream of the UCAN project that is also at a bit of a breaking-change juncture.  I can point you towards libraries that mostly conform to it... but it's probably not helpful in the sense that you probably want to build webplus on a definitive v1 spec with test vectors :D
you could just use cryptidtech's multisig, or one of the varsig variants, or do your own thing, but note that this might change under you as well, as multisig hasn't had as much multi-stakeholder refinement as the other multis.  sad but true!


"support" attribute to designspace sourcedescriptor.

Following up on the pre-robothon 2018 talk for support layers in designspaces.
We discussed a couple of steps to enable supporting outlines to specific glyphs. The use case would be a variable font can have n masters for most glyphs, but one or more need some extra help and they can have n+i additional shapes. If you

A new support attribute in the source element indicates the name of the UFO3 layer the glyph geometry is coming from. (does it need a naming scheme or convention?)
The layer does not need to have a complete glyphset, only the glyphs that are present in the layer will be processed.
The layer does not need to be in a specific UFO - it can be in the default UFO, but can be any ufo really.
The designspace consumer should be able to handle a varying number of masters in each glyph.

To do:

I guess this requites a change to the specification:
https://github.com/fonttools/fonttools/blob/master/Doc/source/designspaceLib/readme.rst#attributes
Add support to varlib (what else cu2qu ? ) to support a varying number of masters per glyph.

Kerning is specifically not a part of this proposal because it is difficult to store kerning tables with glyphs that are not present in the font.

does it need a naming scheme or convention?

no, I think the support layer name could be abritrary, anything suitable for identifying it among the other layers. What marks it as "support" is the fact that it's been used as such in the designspace.

Add support to varlib (what else cu2qu ? ) to support a varying number of masters per glyph

yes. Plus, in ufo2ft compileTTF and compileOTF function one should also be able to specify from which layer to get the glyph set (currently it only compiles the glyphs from a font's default layer); and also it should skip generating OTL tables for these "incomplete" support master ttfs.

For those who were not there while we discussed this, the idea is that one can add additional <source> elements which may or may not reference the same filename UFO as other sources, and which have an additional support="some ufo3 layer" attribute which marks these as special masters containing support outlines only for specific glyphs.
The authoring tools should include these support layers to obtain the list of masters for each glyph (matched between different fonts and layers by their name).

yes, actually that makes sense. I don't see why only the extra "support" layers should be allowed to have sparse glyph sets, any source (besides the base master) could be like that.
I also think the name "layer" is clearer than "support".

except in the neutral or base master's default layer

Except in the layer specified to get the base master glyphs from, which is the default layer if no other layer is specified.

Agree. The name "support" doesn't make sense to me. Any source can come from another layer. Any source can be sparse. Initially, base source must have all glyphs.

Proposal for adding a layer attribute to the source element in the document
layerName attribute for SourceDescriptor
SourceDescriptor object reads and writes the layerName.

https://github.com/fonttools/fonttools/commit/1c6c35dc388f877f66c25dd5dac9986e40288103
This is not ready for a PR.

Done. https://github.com/fonttools/fonttools/pull/1253

Thank you Erik.
I'm thinking we should also bump the designspace "format" version, to mark the fact that sources now have this new optional "layer" attribute and designspace consumers need to be updated to deal with that.
It would be nice if the format string would contain both a major and a minor version instead of a single digit, so we could distinguish between new backward-compatible features (like this one) and breaking changes.
However, if we do change that, this very change could be a backward incompatible one, as other tools may assume format is meant to be read as an integer, not a dot-separated version string.
In light of that, it may be ok to bump the version from 3 to 4, and do the same every time we do any change. I understand we did not follow this rule in the past, e.g. when we added a <lib> element. But better late than never. If we don't use that format field, then it will become meaningless pretty soon.

Ok.

Ok to close?

I'd say, let's keep it open until we address the second part of the issue:

Add support to varlib (what else cu2qu ? ) to support a varying number of masters per glyph.

or if you prefer, go on and close this and open a new separate issue for the latter, thanks

Just trying to prune some of the long-open-stale-issues.


Minor code refactors

Ref https://github.com/prestodb/presto/pull/12370 except

Add missing throw statement (already fixed on our side)
Rename test resources
Update deprecated use of catchingAsync (already fixed on our side)

and with some additions

[INFO] There is 1 error reported by Checkstyle 8.16 with checkstyle/airbase-checks.xml ruleset.
[ERROR] src/test/java/io/prestosql/elasticsearch/ElasticsearchQueryRunner.java:[30,8] (imports) UnusedImports: Unused import - java.net.URI.

@kokosing thanks