feat: in-app embed settings

Description
Adds a code snippet for embedding with iframes.
Fixes https://github.com/appsmithorg/appsmith/issues/18292
Media
https://user-images.githubusercontent.com/67054171/209044270-d3b2d03d-3283-4edc-8561-d5381dfac4ff.mov
Code changes

Most of the changes in un related cypress files are formatting changes made by the pre-commit hook  For e.g https://github.com/appsmithorg/appsmith/pull/19023/files#diff-8512d8d522be6cbd62765ee75c2ec23fd1c103de0834808b60f0108b1f36a6daR8
Chrome asks for permission while copying to clipboard on cypress added a command to grant the same https://github.com/appsmithorg/appsmith/pull/19023/files#diff-3eda263cbeef4ddbace0cb7244ba4ccbf097968e297f271bf403a60967e75b1dR33
Will get a new release from design system before merging https://github.com/appsmithorg/appsmith/pull/19023/files#diff-b8d2936a034f3ef827722f0cd18b81d5522b816bfe559ce499e8841f2142aeafR49
Updating margins of tab when there is no message. For group selection on ee we pass a message prop keeping the margins  as it is in that case. https://github.com/appsmithorg/appsmith/pull/19023/files#diff-2a32a9d6d34facd2317a42d447073d12dd2a944903979bf6bd7824c230a34570R30
Created a hook to reuse logic as we have two slight variations of the same UI https://github.com/appsmithorg/appsmith/pull/19023/files#diff-7a5ea9c499be5787a047fe5fb52c1e97c747689b5c0af83de09e39c5730922f1R39
Moved switch styles to components as we use the same styles in page settings and embed settings https://github.com/appsmithorg/appsmith/pull/19023/files#diff-614654ec58d14187e37f4c41ceb094d4a660a191a6b8b9cea77810b1737c633aR4
Removed title etc for the invite modal opened from edit mode header https://github.com/appsmithorg/appsmith/pull/19023/files#diff-88af4adc81bc30e3f31926240eb347829dc88ba7bd516ea08585b44e1c1d9ba4R475
Changes to test file in embed settings to get the deploy url from clipboard, previously it was picking from an input https://github.com/appsmithorg/appsmith/pull/19023/files#diff-f1b14022438d58b8616b2fbff67a259726a5b50e1952f458e449e198ee351ec7R33
Removed duplicate scroll which appears outside the invite modal(visible in release and prod)https://github.com/appsmithorg/appsmith/pull/19023/files#diff-8c6a7a48042ef3b85d76eaad387ec54cd18df8eef5c99985e29f66bef8694ed8L578
Updated to latest design system version as there is a dependency on it.

Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manual
Self hosted dp details is on https://github.com/appsmithorg/appsmith/issues/18292#issue-1457319773
Cloud dp link can taken from the PR
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing
Issues are raised are from https://github.com/appsmithorg/appsmith/issues/18292#issuecomment-1357089835 comment on wards
Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=6877342

/ok-to-test sha=93fcfa3

/ok-to-test sha=b9f25d1

/ok-to-test sha=818ed06

/ok-to-test sha=c7e4455

/ok-to-test sha=7b04118

https://github.com/appsmithorg/appsmith/actions/runs/3769857348

Added more tests

@akash-codemonk - Show navigation bar is not working in sync between the AppSettings pane & Share modal.. Is this expected?
https://user-images.githubusercontent.com/91450662/209430097-dc7df368-a849-4a8a-afa3-0d9f13263fe0.mov

@akash-codemonk have we also tested this share modal in EE codebase? If not, I highly recommend you to do so. Please get it tested on EE codebase along with your changes.

@akash-codemonk have we also tested this share modal in EE codebase? If not, I highly recommend you to do so. Please get it tested on EE codebase along with your changes. Also, make sure this is tested on home page + RBAC screens.

@ankitakinger Yes have checked it before and have mentioned in the PR description. The only place where it could affect are margins for the tabs in the modal. I am keeping it as it is for ee as it has a message(i.e it has other content above tabs) https://github.com/appsmithorg/appsmith/pull/19023/files#diff-2a32a9d6d34facd2317a42d447073d12dd2a944903979bf6bd7824c230a34570R30. Tabs are used in ee for adding user groups/roles. I have rechecked again now.

/ok-to-test sha=d7e59be

@akash-codemonk - Toggling off ShowNav from App pane is not actually hiding the Navigation bar in Deployed mode.. is this expected? But it syns value with Share modal.
https://user-images.githubusercontent.com/91450662/209446128-64412a3c-763a-4d6a-9088-7a666090ed7c.mov

@akash-codemonk - Toggling off ShowNav from App pane is not actually hiding the Navigation bar in Deployed mode.. is this expected? But it syns value with Share modal.
Screen.Recording.2022-12-24.at.11.01.24.PM.mov

It won't change the view of the deployed version. This is only for previewing the final result of the generated snippet. To preview you need to click on the preview embed link in modal.

@ankitakinger @eco-monk @Aishwarya-U-R Gentle ping for approval if all looks good, so that I can merge it before freeze.

LGTM.!
I'm on mobile couldn't find an option to approve.

LGTM. Please wait for Aishwarya and Anand's approval as well though.

@Aishwarya-U-R has reviewed already. https://github.com/appsmithorg/appsmith/pull/19023#pullrequestreview-1229683059


C++20

Issue Type: Feature Request
Please enable C++20 for microsoft build tools as a default version of C++.
VS Code version: Code 1.50.0 (93c2f0fbf16c5a4b10e4d5f89737d9c2c25488a3, 2020-10-07T06:10:52.432Z)
OS version: Windows_NT x64 10.0.18363

This repository does not contain C++ language features for VS Code. Please file an issue in this repository

\extC++


Alias device.getManifestBySlug as config.getDeviceTypeManifestBySlug

Also deprecates device.getManifestByApplication().
Change-type: minor
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!

@balena-ci I self-certify!

@balena-ci I self-certify!


Null Context fixes

Get rid of passing Context instance explicitly in cases when MwmApplication is accessible because Application itself is an instance of Context.
Closes #86

Decided that having static instance have more negative aspects than benefits


Backend (dependency check)

dependency check - add scan results to Postgres

It is possible to run dependency check via docker. I don't see any sense in it as the project files are stored on the backend side. This is done to reduce the load on the server side. And this method is better suited for management and backup. OG


[impove][admin] Fix the createMissingPartitions doesn't response correctly

Motivation
If the topic metadata doesn't exist, it will create a new PartitionedTopicMetadata with 0 partition. If user use the cli to create missing partitions, it will response OK, which may confuse user.
Modifications
When the metadata partition is 0, throw the NotFound exception to tell the user the topic not existed.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Codecov Report
Attention: Patch coverage is 66.66667% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 73.70%. Comparing base (bbc6224) to head (f8cd82a).
Report is 69 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #22311      +/-   ##
============================================
+ Coverage     73.57%   73.70%   +0.13%
- Complexity    32624    32798     +174
============================================
  Files          1877     1887      +10
  Lines        139502   139473      -29
  Branches      15299    15293       -6
============================================
+ Hits         102638   102801     +163
+ Misses        28908    28711     -197
- Partials       7956     7961       +5

Flag
Coverage Δ

inttests
26.90% <0.00%> (+2.32%)
:arrow_up:

systests
24.38% <0.00%> (+0.06%)
:arrow_up:

unittests
72.96% <66.66%> (+0.12%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

...pulsar/broker/admin/impl/PersistentTopicsBase.java
66.11% <66.66%> (+0.66%)
:arrow_up:

... and 104 files with indirect coverage changes


Various issues and fixes

Hi, I tried to run your project using Windows 11 Powershell 7.4 and ran into various issues.  I was able to debug some of them, so I thought I'd jot down the steps I took:
1) pipx run --spec codeqai codeqai configure
This didn't work for me, the setup launched but Codeqai was unavailable after.  (My understanding of pipx run is that it's a temporary, run-once sandbox venv only.)
pipx install codeqai, followed by codeqai configure worked instead.
2) UnicodeDecodeError: 'charmap' codec can't decode byte 0x8f in position 1137: character maps to
Whenever you're using open(), I believe you should add encoding='utf-8'.  This solves this issue.
For example: with open(env_path, "w", encoding='utf-8') as env_f: in app.py.
3) Command '['C:\\Users\\<USER>\\.local\\pipx\\venvs\\codeqai\\Scripts\\python.exe', '-m', 'pip', 'install', 'faiss-gpu (Only if your system supports CUDA))']' returned non-zero exit status 1.
Unless I'm mistaken, on line 170 of vector_store.py, you're passing the literal string faiss-gpu (Only if your system supports CUDA) to pip install.  You'd want fiass-gpu instead.  However, I still couldn't install fiass-gpu as it returned a no compatible packages error.  fiass-cpu worked fine.
4) When I reran codeqai search/sync/etc I get "IndexError: list index out of range". in C:\Users\<USER>\.local\pipx\venvs\codeqai\lib\site-packages\codeqai\vector_store.py", line 34,.
This seems to be because "documents" is empty.  Going back to app.py, the files var after files = repo.load_files()  has an array of docs, but documents does not after documents = codeparser.parse_code_files(files)
After some debugging, this seems to be because treesitterNodes in codeparser.py by line 36 is empty.  However, programming_language has content (Language.JAVASCRIPT /n Language.JAVASCRIPT), TreesitterMethodNode has <codeqai.treesitter.treesitter_js.TreesitterJavascript object at ....  (x2), and file_bytes also has the expected file data.
I'm unfamiliar with Treesitter to be able to debug any further as to why treesitter_parser.parse(file_bytes) is returning an empty array in this case.
Hope this can help.
P.S.
Didn't include this in the list as it may be my local ENV, but for some reason I was unable to run codeqai via pipx in python 3.10.5.  It repeatedly wanted to use pyenv-win 3.9.6, even though that was nowhere on my system.  I had to install 3.9.6 to be able to continue.  This may be a local env issue from an old installation however.

Hi @strawberrymelonpanda, thanks a lot for these valuable findings! Let me go through them. Unfortunately I am on Mac and does not own a windows machine. But lets try to solve this!

pipx install codeqai is also my preferred way of choice but on a device a saw some issues which got resolved with pipx run --spec codeqai codeqai configure. Also in some other open source projects I saw pipx run as the preferred approach. Maybe pipx install is the way to go. Unfortunately I never had an issue with both on my own device and cannot reproduce it. I use also pyenv btw. But I am on Mac.
Good catch! This is also not popping up on Mac. If you like you can raise a PR to fix this :)
Another good catch! I have no Cuda (since, what else? Mac haha) and never tried that installation, seems it is broken. If you like you can also fix this in a PR if you like :)
This is interesting, I will investigate this further. To me this always worked, but I will try new scenarios!

Thanks a lot for your extensive feedback!

No worries, I expected you were running Mac or Linux and some of these issues are certainly Windows specific.  I expect that some of them would also be resolved by using WSL2, but I just thought I'd give it a try as-is to see how far I could get.
For the UTF-8 issue, that one is definitely a Windows thing (unfortunately!) as far as I know, but thankfully it should also be harmless to Linux and Mac to specify an encoding on open at least.
As for submitting PRs, I'll have to pass for now since I haven't actually been able to use Codeqai yet, aside from getting past the crash points. If I can figure out what's going on with the Treesitter parse call and get everything in a working state, I'll see about circling back if they're still issues.
For now I just figured it was best to have the information out there in case someone picks it up. 😄
FYI: If you're interested in taking a look at some point, Microsoft offers a free, ready to go Windows ISO for VMs over at https://developer.microsoft.com/en-us/windows/downloads/virtual-machines/ for development.  I understand if you don't want to get into that overhead of testing multiple systems though.
The project looks great by the way, nice to see Llama.cpp and Instructor embeddings being used locally for this.  The sync update in particular is a great idea.  Good luck!

Alright, but in any case thanks for our insights! ❤
I have an idea regarding point 4. Currently my treesitter parser only parses methods with documentation. simple Dataobjects or class headers are skipped. Maybe you tried to sync git diff and got some updates but the treesitter parser did not found any, since your git diff only consist of changes outside of methods. This is the first idea I have in mind, but I will take a closer look. Guess it is an easy fix!

I have an idea regarding point 4. Currently my treesitter parser only parses methods with documentation.

I think this may be the issue as you suspect.  I saw the note in the Readme that it works better with documentation, but I thought I'd throw it at some code I had just to try it out.  So if this is expected behavior that's fine, all that needs fixed is the "IndexError: list index out of range" in that case.
For a sanity check and a common point of reference, I ran Codeqai search against the current main branch of Codeqai and it worked. 👍
However, after adding the "encoding" changes in a new branch and committing them, I ran codeqai sync and got a new error, which may still be related to what you were saying:
File "C:\Users\<USER>\.local\bin\codeqai.exe\__main__.py", line 7, in <module>
[...]
if document.metadata["filename"] in self.vector_cache:
TypeError: argument of type 'VectorCache' is not iterable

By the way, unless I comment out the spinners, when it errors out it loops on ⠇ 💾 Syncing vector store... after an error rather than quitting.  (and in Powershell, won't respond to ctrl+c, so the term has to be closed)
Not a big deal, but thought I'd mention.

Thanks for reporting! I fixed the
File "C:\Users\<USER>\.local\bin\codeqai.exe\__main__.py", line 7, in <module>
[...]
if document.metadata["filename"] in self.vector_cache:
TypeError: argument of type 'VectorCache' is not iterable

error with https://github.com/fynnfluegge/codeqai/releases/tag/0.0.7 ✅
Will also take a closer look to point 4 from your first comment 🙌

on osx 14.2.1 from the cloned repo I got this:
pipx run --spec codeqai codeqai configure
Fatal error from pip prevented installation. Full pip output in file:
    /Users/tyoc213/Library/Logs/pipx/cmd_2024-01-15_18.12.19_pip_errors.log

pip failed to build package:
    multidict

Some possibly relevant errors from pip install:
    error: subprocess-exited-with-error
    multidict/_multidict.c:458:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:503:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:538:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:780:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:839:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:875:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:922:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    multidict/_multidict.c:970:37: error: incompatible pointer to integer conversion initializing 'int' with an expression of type 'void *' [-Wint-conversion]
    error: command '/opt/homebrew/bin/clang' failed with exit code 1
Error installing codeqai.

I had to modify codeparser.py to add encoding='utf-8' to  "with open(code_file, "r", encoding='utf-8')" to get it to work

@EasyTop Thank you, are you so nice to raise this as a PR? 🙂 All Windows users will appreciate this!

I would like to pick up all points here and will collect them into a troubleshooting section in Readme.md.
The encoding error should be fixable with a PR.

I tried this on windows and pipx says it successfully installed codeqai, it also creates a shortcut to place codeqai in path, but the appointed location is empty. There isn't even a pipx folder in my AppData.

Updated the Installation section https://github.com/fynnfluegge/codeqai?tab=readme-ov-file#-installation and also added a Troubleshooting section. Hope that helps!

Feels like enough of this has been addressed through PRs and README updates that it's time to close it. 👍
More specific issues should probably be created for anything outstanding.


Search from CLI

Is your feature already implemented in the latest master?
No
Is your feature request related to a problem? Please describe.
Right now the current method for getting search-like information from Spotify via the CLI is to use spotify_player get item <TYPE> --name. Unfortunately this only returns a single result rather than the list of results that the TUI returns when searching.
Describe the solution you'd like
A new CLI command called search which allows users to do the same kind of search that the TUI does.
Describe alternatives you've considered
N/A
Additional context
From poking through the codebase (I'm not super familiar with Rust so I'm sure I've missed something here) it looks like the CLI does item retrieval calls out directly to the artist, track, playlist, and track methods in rspotify.  Would it be too much work to wire up the search method as well?

Hi, I'd like to give this a try if that's okay with you? Thanks!

yes, please do. Lmk when you have a PR up


include SequenceRecords from second-in-pair reads

fixes #693

SGTM! Squash down and I'll merge after tests pass?

k, will do. Now that the build has started, can the squash/force-push mess it up? Is that how we end up with some versions succeeding and others failing?

Nah, the current build will be unaffected. I wouldn't worry about the squash/force-push issue; it's a transient bug and can be resolved just by rerunning.

squashed!

Thanks! Can you rebase?

rebased!

lgtm

bump!

rebased once more

Merged! Thanks @ryan-williams!


Feature request:  suite-level set block

Use case:  my chart has several values which are required to render (using the helm required function).  They are often not related to the unit under test.
I would like to see a set block at the suite level, which applies the values to all tests in the suite and treats set blocks at the test level as overrides in case of a conflict.

I second this myself.

I think I'd really like visibility into these values at the suite level, rather than storing configuration across multiple files, so if it's not too much trouble I'd still like to see a suite-level set block with the same syntax as the test-level block.


commonjs package manager support (fix #8)

& fix enclose undefined angular reference

thanks!

happy to help :smile:


initial changes for elion deployment

Added changes for Elion deployment. Modified config/profiles/elion.config, config/rnaseq.config and modules/rsem/rsem_alignment_expression.nf. Created a new variable called REFERENCE_CACHE to redirect to different local instances depending on profile. This work only for rnaseq pipeline. Tested on elion using bowtie2 and star.

The proposed change makes sense and I do see the value making the root input directory flexible. The code looks OK but we will need to go through our internal testing to make sure as a final sanity check on our end. Prior to testing, the Sumner profile needs the REFERENCE_CACHE modification.
Prior to merge, the configs files of all major workflows will need to be updated. I would like to avoid having a handful of configs on a different system. @gabyrech would you be able to take a first pass at that?

The proposed change makes sense and I do see the value making the root input directory flexible. The code looks OK but we will need to go through our internal testing to make sure as a final sanity check on our end. Prior to testing, the Sumner profile needs the REFERENCE_CACHE modification.
Prior to merge, the configs files of all major workflows will need to be updated. I would like to avoid having a handful of configs on a different system. @gabyrech would you be able to take a first pass at that?

Sure! I can take that.

Closed in favor of different approach


New IDEA: Automation of adding service providers to app.php

Hello, Right now when a composer package is added, we have to look at its documentation and add the corresponding ServiceProvider and Facets in app.php.
Why can't it be automated by using an another command , which will be executed right after we run composer update . I have experience in laravel, still then i am new in PHP.
The command will just read the laravel.json file (for eg.), belonging to each and every package and add the data into app.php if it does'nt exist.

There is already a package that does this. This was discussed then, and it's not getting into the laravel core.

Someone came into my dream and stole my idea. :-) Thanks. Whats the name of the package anyway?


baetyl-1507: Add nodeport check for app.

Codecov Report

Merging #411 (ba00a0c) into master (15a4fcb) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #411      +/-   ##
==========================================
+ Coverage   65.71%   65.73%   +0.01%
==========================================
  Files          91       91
  Lines        6572     6575       +3
==========================================
+ Hits         4319     4322       +3
  Misses       1640     1640
  Partials      613      613

Impacted Files
Coverage Δ

api/application.go
75.31% <100.00%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

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


added Blockly to launcher

Created a Blockly icon and added it next to the file name when we open a new Blockly Editor, to the launcher and to the command palette.

Thank you, Denisa. Looks great!
I added a small fix for an empty workspace, what I'm doing is creating a JSON object with an attribute called variables when the document is empty. This way Blockly doesn't complain.


fix bug in body-fixed XYZ Euler angles

This changes the Euler angle calculation to match the one in the PX4/Firmware ecl library;
so that the order of the Euler angles is RPY (XYZ), instead of the previous (pitch, roll, yaw).

Looks good. I think this function needs to be updated as well: https://github.com/PX4/jMAVSim/blob/master/src/me/drton/jmavsim/CameraGimbal2D.java#L116
I didn't even know jMAVSim has gimbal simulation.

@kd0aij Can you update the gimbal sim?

@bkueng @LorenzMeier That code matches the description here:
https://github.com/PX4/Firmware/blob/master/msg/manual_control_setpoint.msg#L25
I don't know why the "x" axis in the manual control message would be considered the pitch axis (and y roll), but that's the way it is. But there are no comments in jmavsim explaining the intention here either :)

@kd0aij yes, the for the manual_control_setpoint it's swapped too. It's because it comes from the mavlink message, and we cannot really change that.
But I don't think it's related to the jMAVSim gimbal code, because the gimbal uses the actuator control outputs, which is defined by the mixer config. Plus gimbal sim channels can be configured. So I still think gimbal sim needs to be updated to be consistent with the rest of jMAVSim.

@bkueng Sounds like you know more about this than I do. Feel free to change it any way you want.

continued in #37


Added env CHAINLIT_DEBUG on run_chainlit(target), default to False.

Added env CHAINLIT_DEBUG(default False) when run_chainlit to enable to set log debug mode.

Isn't it already done by the CLI run command? Unless you are not starting Chainlit with the CLI?

FWIW I do a combination of setting the env and setting config directly:
Probably would be nice use one approach or the other consistently.
from chainlit.config import config
from chainlit.cli import run_chainlit

...

# Specify a different port to run the server on
os.environ["CHAINLIT_PORT"] = str(settings.get("port"))

# Will prevent auto opening the app in the browser
config.run.headless = True

if settings.get("cluster") == "dev":
    # Reload the app when the module changes
    config.run.watch = True
    # Set the log level to debug
    config.run.debug = True

run_chainlit("my_chainlit_app.py")

Hey , thanks for review !
When debugging through the IDE run_chainlit, some? environment variables are not read, hence this PR.
However, when analyzing the code further, as suggested by @kevinwmerritt , it would really be better if execution via both CLI and run_chainlit maintained the same behavior.
Maybe something to look at here: config.py#L484
I will close this PR for a better evaluation.
Thanks !


improve layout in sidebar mode

Some css tweaks:

use @component-padding from ui-theme for padding instead of 10px
set minimum width
all buttons in a row so output gets to use the entire width of the panel

Before

After

Very nice. Thank you for this!


Add documentation workflow to module-release-process.md

Update md
Cleanup workflows; it appears we have duplicates here: https://github.com/Azure/enterprise-azure-policy-as-code/actions

Cleaned up workflows


Hiding zoom controls

Is there a good and easy way to hide the zoom controls in the upper left corner of the map?
I'm able to do it by replacing
controls: ol.control.defaults({
    zoom: interactive
}),

with
controls: [],
in mapview.js.
However, placing this altered mapview.js in my project does not override it, this could be due to NetBeans class loaders, I don't know.
The reason for wanting this is that I already have a zoom control, the one in the lower left corner of:

Hi Patrick,
using your own mapview.js will not work, as this is loaded from the same path where the html is loaded from; and placing that in a custom folder where it is found by the classloader first would mean that you need to put all the html and js files there - including OpenLayers. I would not guarantee that this is working.
My idea would be to change the initialization of the MapView Java object (and of the underlaying Javaascript object). At the moment there are the two parameters for the projection and the interactive flag; This is messy and I would like to encapsulate this into a separate configuration object which is then internally transferred into an configuration object for the Javascript class. This would leave room for extending and adding configuration options.
I'll put this on my list, but it might be the next weekend before I find time to implement this.
btw, although I know that some people are using mapjfx, this is the first screenshot I see of an application that integrates it, looks nice

implemented in versions 1.27.0 and 2.8.0
tl;dr
configure your MapView instance with:
        mapView.initialize(Configuration.builder()
            .showZoomControls(false)
            .build());

Works great, thanks!


Consider adding the onShown, onHidden, onclick and onCloseClick to a Toast

Currently a toast receives an OnClick Action in the configuration and has 2 events

OnUpdate: used to signal the container to rapaint itself
OnClose: it should correspond to OnHidden

Should these events be global or toast specific?

No one seemed interested into these events so closing the issue for now


[BP-1.20][FLINK-36468][parquet] Use Flink Preconditions util instead of Parquet

1.20 Backport of https://github.com/apache/flink/pull/25488
What is the purpose of the change
Use Flink Preconditions util instead of Parquet
Brief change log
org.apache.parquet.Preconditions -> org.apache.flink.util.Preconditions
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure


Customizable "Dashboard Projects.json" file location

:wave: Hello.
Thanks for a great extension.
Would it be possible to configure the location of Dashboard Projects.json file?
Currently, it gets stored in ~/tmp/Dashboard Projects.json.
What if users could configure the path of this file with a setting? Following the established naming scheme, it could look something like this:
{
    "dashboard.customProjectsJSONPath": "/customizable/path/to.json",
}

This option would accommodate people who keep all of their VSCode settings under version control, and people who do not want to clutter their home directory.
What do you think?

Hey!
Sounds like a good idea, especially as it does not impact users that do not want to change anything.
I'll have a look on this! :)

This has been implemented in version 1.3, released today. :)


0.14.0 - modify the 'menu' width

Change the 'menu' width to 2/3 of the current one.

Blocked by: https://github.com/TurtleRover/tcs/issues/58


Can ELM be added to Syntax Highlighting?

If ELM can be added to https://github.com/PrismJS/prism/tree/gh-pages/components, it'll be great.
ELM examples:
https://github.com/elm-lang/core/blob/master/tests/Main.elm
ELM will also make the pip in the following command a different color which is beneficial to the docs I'm making with docsify:
pip install something

I don't know how to create one of those files myself so I can't send a PR

There is already a pending PR (#1174) for this, waiting for changes to be done.

Okay thanks!


why you status bar button has icon

Hi, why you status bar button has icon? where to define it? thanks

https://github.com/microsoft/vscode-extension-samples/tree/master/statusbar-sample
Stack Overflow is a better place for questions like this.


Routes for creating Groomer Business Profile routes

Description
Add get route to receive all necessary information for groomer profile page
Add post route to create entry in business_profile table and groomer_cover_images table
###Type of Change

[ ] This is a bugfix.
[x] This is a new feature.
[ ] This is part of a feature.

Organization

[ ] Have you linked this pull request to a Trello card?
[x] Have you named the branch in the format of feature/describe_feature_or_change OR bug/describe_bug_this_is_fixing

Review

[ ] Is this a work in progress?
[x]  Is this ready to be reviewed?
[ ]  Have you added two reviewers to this pull request?

 Create Endpoints for Business Profiles

 Create Endpoints for Business Profiles


Discussion: Approaches to scene2d Layout/Widget measuring

This would probably be a better fit for a forum, but this place seems to be a better fit to reach other contributors
After seeing cae2342f08a368d15bed2d8d923f4f79f903d6f1 and fighting with similar issues myself:
How about we added Android-style onMeasure() instead of getMin/Pref/MaxSize?

Working with wrapped Labels without it is a pain (and probably with some other widgets as well),
it clearly defines when should a widget measure itself
and generally, from my experience, it is more predictable to use.

I don't like many Android constructs, but this one is IMO pretty good. iOS seems to be doing something similar, under the hood of Autolayout.
It would be a breaking change, but it shouldn't be hard to write some temporary bridges to ease the transition. I know that there are some big applications written in scene2d.ui, so the costs of rewriting could prevent this from happening (maybe libGDX 2.0). But even if that would be the case, I'd like to know what the general opinion is about these two approaches.
Of course this is just about the general idea, it probably wouldn't have the exact Android API. I'd volunteer to write the PR for this, given enough positive feedback. Implementation details could be further discussed there.

It's been a while since I looked at it, but from my experience Android layout is completely FUBAR, rivaled only by the madness in UIView. I would not look to either API for inspiration.
I have struggled with many UI toolkits over the years and there are absolutely none which I think get layout correct. scene2d.ui's layout is at the core of the library and it works fabulously, IMHO, especially Table, which has no equal. It is pretty much the simplest solution, easy to understand, handles all the necessary cases, and is easy to customize. I'm happy to discuss alternatives, but it is unlikely I can be convinced a different system is better, even putting aside the conversion effort of all the widgets and apps built with the current system (eg, Spine). It may be more fruitful to discuss your pain points with scene2d.ui's layout.
Keep in mind that scene2d.ui and scene2d are separate, so there is nothing stopping you from writing a different layout system on top of scene2d.
Note that cae2342f08a368d15bed2d8d923f4f79f903d6f1 was done because b9ec77edf9192aac9b0d6c1bd0e5c62ca5266ad7 now ensures the layout is indeed valid after validate returns. Otherwise the min/pref/max of scene2d.ui and the way layout works has been solid for a very long time.
A wrapped Label is the only widget which calls invalidateHeirarchy from within layout. It is really not terribly complex: a wrapped label's preferred height depends on its width. For example, a smaller width means the text wraps more often and so it will have a larger preferred height. A measure method is unnecessary. If you want the pref height of a wrapped label for a given width, simply set the label's width, then call getPrefHeight.
No other widgets have a dependency between their current size and their pref size and so are even simpler: the pref size is the size the widget would like based on its current state, regardless of its current size.

Thanks for the thoughtful response!
I agree with most/all of your points (especially the one about awesomeness of Table).
However, the repeated layout() in validate() and setting the width of Label to influence its preferred width still feel a bit hacky to me. For example, I am not sure if there is a clean way to use the set-width-of-Label-first-trick for Labels inside Table, for situations when you want the Label to be at most 300 wide, but 280px when wrapping at 300px would lead to preferred width of 280px.
My line of thinking is, that this would be easily solved, if getPrefW/H was supplied with some constraints about the environment in which it will be placed, for example max width/height or something. Which is basically what Android's onMeasure does. And if we had that, the min/max/pref system as it is now would be probably superseded by this new one.
It is true that this currently only really affects one widget, but on the other hand, text handling is, in UI library, pretty important.
I'll probably try to implement a proof of concept to see if it would even solve the problems I think it would, and then I will report back. I'll close the issue so it doesn't get in the way, but the discussion can continue.

Interesting, in that case I might need to recalibrate my hackiness detector.

for situations when you want the Label to be at most 300 wide, but 280px when wrapping at 300px would lead to preferred width of 280px.

I don't follow -- a wrapped label doesn't have a preferred width (it is zero). A wrapped label's width must be set based on something external.

It is currently not easily achievable  which is probably why it sounds weird, so I'll try to explain it better:
I have tried to create tooltips with arbitrarily sized text. Without wrapping, the Label in Table got the exact size it needed. However, when the text was too long, it didn't look good (and didn't fit on the screen), so I have decided that I wanted to wrap at 300px. But for this to work, I had to do something like this:
label.setText(tooltipText);
if (label.getPrefWidth() > 300) {
    label.setWrap(true);
    table.add(label).prefWidth(300); //or just width()
} else {
    table.add(label);
}

This was ugly, had to be done on every text change, but worked. However the tooltip was then always 300px wide, even though it could be smaller, as none of the wrapped lines was exactly 300px wide - all were smaller, sometimes significantly, which created strange extra space in the tooltip.
Setting the width of the label before adding it to the table just collapsed the label to be 0 width. pack()ing the table afterwards also didn't help, because I had to force the Cells (pref)width
to already be some size. Also, I have tried to do the set-width-first-trick in custom Cell.width Value, but that seems to be impossible to implement without getting down to measuring the width of letters myself, because even though computePrefSize() supports it, getPrefWidth() just returns 0f when wrap is enabled, thus there is no way to actually get that value.
So unless I am missing something, it is not easily possible to achieve the "wrap at 300px and shrink" behavior with current API, but it would be natural if the getPrefWidth() didn't have special case for wrapped Labels and always sized itself according to the preferences of its container.

This whole thing is mostly a corner case, and it could be fixed by implementing custom Value for these cases (with appropriate changes to Label). But it makes me wonder, if this could all be prevented with a different approach entirely.

I do exactly what you describe with wrapped label tooltips in Spine:

It's done like this:
Label label = ...
label.setWrap(true);
label.setWidth(maxWidth);
label.validate();
Table table = ...
table.add(label).width(Math.min(maxWidth, label.getGlyphLayout().width));

This doesn't consider a background on the label, but my label doesn't have a background. Maybe Label could expose the pref width in a new method, separate from getPrefWidth. It doesn't make sense to allow getPrefWidth to return the pref width for a wrapped label. Presumably someone is calling getPrefWidth so they can set the width, so there is a chicken and egg problem since the pref width has a dependency on the current width. This could result in different pref widths when layout is done multiple times in succession without explicitly setting the width to some value first.
Layout is the thing all other toolkits get horribly wrong, either with a wacky or complicated API and/or with really poor layout managers. There may be something better, but scene2d.ui's system stays out of the way for the most common cases and still handles complex cases just fine. Sorry I don't have much ideas for a new system, I really like the current system!
I find I don't use percentages very often, instead relying on table columns.
Getting a bit off topic, but one thing I would like is for UI animations to be easier. I've found trying to use actions for UI animations works OK at first, but falls apart when there are multiple actions, actions need to be cleared, etc. In a recent project I wrote a class that stores a group's children's positions, validates the layout, stores the "target" position, then puts the children back to the original position. Every frame the children are moved toward their target position. The result is that when I change the UI (eg, add or remove a widget), widgets animate from their old position to their new layout position. You can see it in action in this app:
http://esotericsoftware.com/gloomhaven-helper
The code is sort of general purpose, you can see it here:
http://n4te.com/x/4298-PkCz.txt
Each child needs to implement HasAnimator, then it is used like this:
// Parent widget needs to avoid invalidate if animating and call updateChildren.
parent = new HorizontalGroup() {
	public void invalidate () {
		if (!Animator.enabled || !Animator.childrenAnimating(this)) super.invalidate();
	}

	public void draw (Batch batch, float parentAlpha) {
		Animator.updateChildren(this);
		super.draw(batch, parentAlpha);
	}
};
...
// Later, change the UI in ways that change child positions, but don't validate.
parent.addActor(something);
parent.removeActor(somethingElse);
// Then just call this.
Animator.storeChildren(parent, 0);

It requires enough wiring up that I'm not sure it belongs in scene2d.ui, but it is neat.
Oh, the missing piece for the above is this gem:
http://n4te.com/x/4299-vs2q.txt
Here is a version that works on a float:
http://n4te.com/x/4300-1DcV.txt
I love this function. It moves from the current value toward the target value, moving faster when farther away, without needing to track start/end times. I use it all over the place to easily animate values, eg:
float desaturation; // Animates between 0 and 1 based on isDesaturated().

public void act (float delta) {
	super.act(delta);
	float target = isDesaturated() ? 1 : 0;
	desaturation = animate(desaturation, target, 0.5f, 1, 2, delta);
}

Looking forward to seeing your prototype! :D
An animation system is maybe tougher than it appears. Sometimes animations can be combined, other times they need to be cancelled, other times they need to be fast forwarded to the end. I have a feeling the most flexible solution is some sort of animation controller that handles just one type of animation, then application code can define what happens as the user takes more actions.
Using HarfBuzz for shaping would be awesome. We don't have RTL support, but we already have on-the-fly glyph generation, enabling typing in CJK or most other LTR languages, even when the glyphs typed are not known beforehand.


QNZM

Use the preview tab to see the example given below. Edit the relevant information and preview the changes before submitting. You may delete these instructions.

Key
Value

ID
QNZM

Source
Vaaran Bhai Gurdas Ji

Page
28

Line
gur isKI dw krmu eyhu; gur Purmwey gurisK krxw [

Correction
THIS ≠ THAT

PROOF (EDITION)
Image:

I don't see the issue here:
Image:

@Sarabveer can you please confirm with more research? It's the only one that doesn't follow the rest of the page.

@bhajneet it should be put together


[C4GT] Asset: Separate preparation of creation of Asset from dispatch

Description
Separate preparation of Asset Entry and dispatch of extrinsic to chain. This would help when bulk transaction takes place so we just prepare all the required extrinsics and then execute these transaction in batches by dumping them into a promise using
a batch sign & submit utility ex, authorizeBatch which can take array of extrinsics, instead of just one.
Goals

[ ] Creation of new export function named prepareCreateExtrinsic(), which will contain creation of asset entry and Did.AuthorizeTx().
[ ] Integration of above method into existing dispatchCreateToChain()

Implementation Details
Developers/Contributors can refer to example PRs: #188, #182

Product Name
CORD.JS
Organization Name
Dhiway
Domain
Blockchain, SDK Development, API
Tech Skills Needed
Typescript
Mentor(s)
@amarts @vatsa287
Complexity
[Low]
Category
[Performance Improvement], [Feature]
Sub Category
[API], [Refactoring], [Backend], [Beginner friendly]

Can I proceed with this @vatsa287 ?


Extracting qlog files from Firefox

Hi all. I've been trying to extract the qlog files from Firefox following your articles and blogs. But until now, I still cannot do it.
For the server-side, I used quic-go v0.28 (the latest release at the time I post this). I was able to acquire the qlog files from the server. But when using qvis, I can only get the information up to the QUIC layer. I cannot get information regarding the HTTP/3 layer (e.g., GET requests, type of frames, etc.)
So I started to think maybe if I get the qlog files from the client-side, I could get the HTTP/3 information? I then followed the description in your article:

Set the network.http.http3.enable_qlog in Firefox via about:config
Trigger the QLOGDIR environment variable via terminal:

export QLOGDIR=$HOME/Desktop

But I don't know why it failed to return any qlog file.
Do you have any idea?
Thank you very much in advance.

Hey there @chanh1964,
Seems like my article isn't entirely correct/clear here, I'm sorry about that...
Firefox does not support QLOGDIR, but instead makes its own directory based on the firefox process ID in the Operating System's temporary directory.
For example, on my Windows system, this is where I could find the qlogs after enabling the network.http.http3.enable_qlog pref:

The qlog_ suffix will change depending on your firefox instance. And of course the location will be different on linux/mac. But the qlogs seem to be somewhat usable and are written fast enough (you don't need to close firefox or actively wait for the connections to close or anything).
This is consistent with the code from the original commit adding the qlog preference:
https://hg.mozilla.org/integration/autoland/rev/b53e9a8ffb1f#l4.30
Finally, with regards to HTTP/3, I'm afraid only a few stacks really implement qlog for HTTP/3... it looks like that's not in the Firefox qlogs either; they also only do QUIC. The ones I know about that do H3 logging are Cloudflare's quiche and the Python aioquic library, so maybe you can look into those if that's really what you need (though they also don't log everything)?
If you have any more questions, please let me know!
With best regards,
Robin

Hi @rmarx,
I think you have answered all my questions.
Thank you very much.
Best regards,
Chanh


L5 support

Will this package support L5 or only L4 ?

Yes, I plan to make any adjustments needed to support L5. I'm hoping to do this within the next week or so (maybe sooner).

Thanks a lot for your answer and looking forward to it, we have been so happy to use your package here.

@avlastenok I just pushed a bunch of updates for L5 which you can find on the develop branch, please give it a whirl and let me know how everything works out. I'd like to get some more testers on L5 before I start tagging V2
I'll keep this ticket open for now

That really sounds awesome will give it a try later today.

@avlastenok did you get a chance to try it out?

Sorry I did mean to but work took up all my time, I will see to it tomorrow when I need to start migrating a project from L4 to L5 which also uses your package so will give some details on how that goes.

@avlastenok, no rush, just checking in :)

So far so good, got it all working as it should just one little thing as of laravel 5 the Form package is no longer in the core so when people try your examples out you might want to just add the fact that they need to pull in the illuminate/html package.
But beside that I have no issues we just successfully migrated to L5 with this included.

Thanks @avlastenok, good idea. I half assumed that most people know that L5 doesn't have the form/html packages anymore, but I guess it's a good idea to make sure. I'll put a note in the readme tomorrow. I'm glad that everything else is working as expected though. If no one else reports any L5 issues, I'll probably tag it V2 by the end of the week, so keep an eye out for that too.

Yes i'm sure your right that most people know that by now but someone brand new to laravel might not have that knowledge yet so was more as a helper for them.
Thanks for all your work on this package we really appreciate it.

No problem at all! I'll probably be tagging v2 at some point tomorrow too

just tagged v2


Suggest not using substr and new Date

Within the following line, I would suggest replacing the Date allocation with a call to Date.now() and the use of substr (which is deprecated) with the equivalent substring
https://github.com/johnny-quesada-developer/easy-web-worker/blob/6c9b2317c18652c38bcfbf4a8f2d6489b80b677b/src/EasyWebWorkerFixtures.ts#L2

Hi @gabrielecirulli, you are absolutely right! substr has been marked as legacy, so I am more than happy to implement your suggestion in version 4.0.3.
Thank you so much for taking the time to review it and, hopefully, for giving the package a try.
I haven't prepared a contribution readme yet, but I will do so soon. In the meantime, please feel welcome to share any ideas or collaborations that could improve this package and help it reach more people.
Regards, and again, thank you so much.

suggestion was implemented and published in version 4.0.3


Added support for layoutmsg params and added the param 'newfocus' for 'swapwithmaster' and 'focusmaster'

Describe your PR, what does it fix/add?
It adds the possibility to add parameters to the layoutmsg dispatcher for the Master layout.
It also adds the parameter newfocus for the layoutmsg dispatchers swapwithmaster and focusmaster,
which can have the following values for swapwithmaster:

master - focus at the new master after swap
child - focus the new child after swap
auto - keep the focus on the previously focused window (default and also current behavior)

For focusmaster it can have the following values:

master - focus the master, or keep the focus at master if it is already focused
auto - swap the focus with the first child, if the current focus was master, otherwise focus master (default and also current behavior)

Fixes #674 with for example the following binding:
bind = $mod,P,layoutmsg,swapwithmaster,master

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
The PR itself doesn't introduce any breaking compatibility.
I think CVarList should have size() == 0 when the given parameter for the constructor is an empty string (while it is 1 and the first parameter is the empty string).
I haven't changed this behavior in fear that anything breaks that depends on the current behavior.
I'm working around it with a check like this right now: CVarList::size() > 0 && vars[0] != "".
Is it ready for merging, or does it need work?
Yes it's ready for merge, though we might need to update the hyprland-wiki as well with the new Master layout layoutmsg para, should I add a PR there?

I'd rather this being separated with   instead of , to avoid bugs with parsing.
a la swapwithmaster master
Yes, make a wiki pr plox.

Changed it to ' '
But one thing that might be an unwanted effect, is that multiple spaces count as multiple parameters (empty strings again), I doubt that this should be intended behavior?

Yes, make a wiki pr plox.

Alright doing that now

But one thing that might be an unwanted effect, is that multiple spaces count as multiple parameters (empty strings again), I doubt that this should be intended behavior?

well why should something like swapwithmaster                    master be valid anyways...?

Well just wanted to point that out, if people accidentally use multiple spaces, but I explicitly added a comment about that in the docs right now, I don't have a problem with that.

tanks!


'git pull' hangs on 'Initializing Keybase... done'

On one of my machines, running 'git pull' hangs after printing 'Initializing Keybase... done'. The strange thing is that it doesn't happen on my other machine.
My log id: e28a6ab060c45a5f3a4c0b1c
The machine on which it hangs runs macOS El Capitan, Keybase client 1.0.45-20180305194145+3a1c65a05.
I noticed in keybase status that a bunch of git-remote-keybase processes were hanging...
Other users:
    git-remote-keybase git (redacted) [pid: 87793, version: 1.0.2-20180305194232+2120dde5]
    git-remote-keybase git (redacted) [pid: 87849, version: 1.0.2-20180305194232+2120dde5]
    git-remote-keybase git (redacted) [pid: 87915, version: 1.0.2-20180305194232+2120dde5]
    git-remote-keybase git (redacted) [pid: 86422, version: 1.0.2-20180305194232+2120dde5]

...probably because of my multiple 'git pull' attempts. Killing these processes did not help.

@FooBarWidget it looks like you're having DNS issues.  From your log:
Connection: error dialing transport: dial tcp: lookup bserver-1.kbfs.keybase.io: no such host

Are you behind a VPN or filewall that might block keybase.io subdomain DNS queries?

I wasn't. But it looks like my OS had some other kind of problem which affected the network stack. A reboot mysteriously solved the issue.
Regardless of the problems in my OS, should git-remote-keybase not abort with an error instead of haging?


Foliage painting may be broken

Reported on the forum here. User was trying to paint foliage on Cesium World Terrain but found it didn't work. I haven't been able to triage this, but I'm opening an issue so that we can take a closer look.
While searching for related issues / PRs I found PR https://github.com/CesiumGS/cesium-unreal/pull/414 about collision settings that were impacting the foliage painter. Maybe this is a similar problem?

Foliage painting works as of the latest release in UE 5.3. I'm asking for more information, but I suspect there is something off with the user's setup, rather than the feature itself.


Feature: add smoke tests

we should add smoke tests so we now the examples are actually working. We should run those via the CI (as before)

I agree. I'll see if I can make you a member of the cirrus-ci management.

well i could do that in github using workflows too. Maybe for the example project that's not a bad option?

Sure thing!! Feel free to do it :).

Nice, will do.
Finished moving the docker example project to this repo and just release it https://github.com/jodconverter/docker-image-jodconverter-examples/releases/tag/4.4.5-5 - so we are aligned.

Didn't know about github workflow! This is nice!


Fix newSubscription with null tokens

Fix for https://github.com/laravel/cashier/issues/270 - attempting to update card with a null token

What is your situation of wanting to create a Stripe customer without a token?

The scenario is that I have a multi-tenant app where a customer is a company rather than an individual user. As part of our pre-sales activities, the sales team creates a tenant (company) in our software which in turn creates users and other items which are 'owned' by the tenant. The tenant's users then have access to the software on the plan which the pre-sales activities have identified as being most suitable for the tenant for a fixed period of time (the trial period). It is important that we don't force collection of billing details at this point, because at the time the customer is created we are still moving through the sales journey, but flexible enough that we can add the card details later to continue the subscription.
It is also important that an individual user doesn't own the billing relationship because in an enterprise the billing will either be manual via a PO or via a company card, where multiple users will need to be able to own the billing relationship.
I had considered managing trials through the Models directly but we want/need Stripe to be the source of truth for subscription management.


Change Base URI to brickschema.org

Addressed by https://github.com/BuildSysUniformMetadata/Brick/pull/9
Acceptance of PR predicated on the example buildings being updated with the new URI

done: https://github.com/BuildSysUniformMetadata/Brick/commit/6f20dd82dc9d78fb8c3b15f33467f1854cf7f29c


Added record type sort in manage zones tab

Fixes #541.
Changes in this pull request:

Added sort(Ascending and descending) for record type column in Manage records tab.
Added sort for record type in API. (/recordsets?maxItems=100&recordTypeSort=asc) .

Note :- Sort cannot applies for both record type and record name simultaneously . if record type sort is applied, record name sort will be ignored.

API looks good locally, but sorting in the manage records tab in the portal doesn't work for me. I'm able to click the Type column, but nothing happens, and I don't see the arrow icon like for the Name column. See screenshot

Can you please double check the same once. I could see the sort arrow icon in type field.


At/np 306

Modifies a little some scenarios that in theory user can interact and because of that has some visual modification. This pull request changes the script and the references for the following components:

Buttons
Navigation
Radio Group
Radio Group small
Targeted Content

Some screenshots of the new references:

Going to solve those conflicts first, then reopen the Pull Request.


Setup repo issue templates

Task
To ensure all issues (epics, sub-tasks and bugs) are created in consistent and complete manner issue templates must be created
Definition of Done
once issue templates are created for:

[ ] Epics
[ ] Sub-tasks
[ ] Bugs

Out of Scope
only the above templates are in scope
Discussion required
Who is going to review these templates?
To get around the review issue for this task previous projects templates (gazetteer repo?) that have been previously reviewed could be used for this project
However review will be an ongoing project going forward.
I will open an issue for peer review as well as raise at the next topo-data standup

Closed via PR (#4)

Closed via PR (#4)


Run rook ceph together with sig storage

What this PR does / why we need it:
This PR enables rook-ceph-default coming with kubevirtci and focusses all sig-storage and rook-ceph tests for the sig-storage lanes (i.e. TARGET=k8s-1.19-sig-storage).
It also moves a test into tests/storage folder and removes a duplicate sig label.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm

/test pull-kubevirt-e2e-k8s-1.19-sig-storage
/test pull-kubevirt-e2e-k8s-1.20-sig-storage

/test pull-kubevirt-e2e-k8s-1.19-sig-storage
/test pull-kubevirt-e2e-k8s-1.20-sig-storage

So I would not put live migration in the context of sig storage, only reason it has rook ceph because the test needed RWX storage.

So I would not put live migration in the context of sig storage, only reason it has rook ceph because the test needed RWX storage.

Ah, ok, so I can remove the extra rook-ceph focus as @xpivarc requested, right?

I would say yes, if that is the only place remaining with rook-ceph context.

So I would not put live migration in the context of sig storage, only reason it has rook ceph because the test needed RWX storage.

Ah, ok, so I can remove the extra rook-ceph focus as @xpivarc requested, right?

But in that case we need rook-ceph for sig-compute, right?

So I would not put live migration in the context of sig storage, only reason it has rook ceph because the test needed RWX storage.

Ah, ok, so I can remove the extra rook-ceph focus as @xpivarc requested, right?

But in that case we need rook-ceph for sig-compute, right?

I don't understand. In other lanes rook-ceph tests are not explicitly skipped? If the test had this requirement shouldn't it be skipped then? And how at all can it work then? Or what do I overlook here?

So I would not put live migration in the context of sig storage, only reason it has rook ceph because the test needed RWX storage.

Ah, ok, so I can remove the extra rook-ceph focus as @xpivarc requested, right?

But in that case we need rook-ceph for sig-compute, right?

I don't understand. In other lanes rook-ceph tests are not explicitly skipped? If the test had this requirement shouldn't it be skipped then? And how at all can it work then? Or what do I overlook here?

So, as I see it, if that test is not explicitly sig-storage, but requires RWX storage (which rook-ceph provides), the easiest would be to leave it like in the current state (i.e. adding the focus for rook-ceph). WDYT?

So the test will be skipped if rook-ceph is not found in the test https://github.com/kubevirt/kubevirt/blob/a3b51ae43590948ec5294213bd13c65a73d57e4d/tests/migration_test.go#L1754-L1757 but there are several tests in that context that don't require rook-ceph at all.

@dhiller
Actually looking at the context. Only one of the 3 tests requires rook and it is skipped if not available.
At the same time this test is included in storage lane.

So the test will be skipped if rook-ceph is not found in the test
https://github.com/kubevirt/kubevirt/blob/a3b51ae43590948ec5294213bd13c65a73d57e4d/tests/migration_test.go#L1754-L1757
but there are several tests in that context that don't require rook-ceph at all.

OK, then the rook-ceph context needs to get deleted and the rook-ceph marker needs to get added to the test that requires it. @awels  SGTY?

@awels @xpivarc like this? https://github.com/kubevirt/kubevirt/pull/5874/commits/c4ae8d691fd2dbe2e056cafc74eff3994920999e

Yeah, that one test is kinda straddling both worlds. its technically a sig-compute (live migration) but it requires rook ceph because it is doing a LM on a RWX block device

Yeah, that one test is kinda straddling both worlds. its technically a sig-compute (live migration) but it requires rook ceph because it is doing a LM on a RWX block device

Hm, then we should maybe reconsider applying only one label to every test. Should I maybe re-add the sig-compute label again? Although this might confuse test writers, and also sig people, as i.e. the test will not appear in sig-compute but only in sig-storage. I think we need to add a comment explicitly stating that somehow but unsure how to phrase this...

/lgtm

/test pull-kubevirt-e2e-k8s-1.21-sig-storage
/test pull-kubevirt-e2e-k8s-1.20-sig-storage
/test pull-kubevirt-e2e-k8s-1.19-sig-storage

@xpivarc @rmohr @awels thanks for your input, addressed comments, PTAL!

/test pull-kubevirt-e2e-k8s-1.21-sig-storage
/test pull-kubevirt-e2e-k8s-1.20-sig-storage
/test pull-kubevirt-e2e-k8s-1.19-sig-storage

/test pull-kubevirt-e2e-k8s-1.21-sig-storage
/test pull-kubevirt-e2e-k8s-1.20-sig-storage
/test pull-kubevirt-e2e-k8s-1.19-sig-storage

/retest

@rmohr addressed comments, PTAL, thanks!

/override pull-kubevirt-e2e-kind-1.17-sriov
That provider doesn't exist anymore in the kubevirtci coming with kubevirt now.

/retest

/retest


Use jQuery's show/hide instead of setting visibility directly

See scripting, ui, etc.

This is going to depend on #105 mainly.

jQuery is gone now, instead we have show/hide/showv/hidev (for display vs visibility.)
We need to figure out now if we can just switch to using display for all of them (as it makes more sense, invisible things don't need boxes.)


is_playing return always false when AnimationSprite is a child

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Windows 7 64bit, Godot 3 beta
Issue description:
is_playing() returns always false when AnimatedSprite is a child
Steps to reproduce:

create a scene with a AnimatedSprite as root and as child

create a looping animation an both
create a script with this content

extends AnimatedSprite

func _ready():
	set_process(true)

func _process(delta):
	$is_playing.text = "is_playing: "+str(is_playing())
	$AnimatedSprite/is_playing.text = "is_playing: "+str($AnimatedSprite.is_playing())

Link to minimal example project:
animationSprite_bug.zip

too much is_playing in this report..


Resolve simple puppet-lint notices

A run of puppet-lint flagged these easy fixes.

Hi @jcpunk
It looks like there's some syntax problems here looking at this test failure

Also, just looking again, there's a number of Rubocop violations from the change too: https://travis-ci.org/github/puppetlabs/puppetlabs-postgresql/jobs/729322793#L422-L913
Autocorrect resolved a few for me (bundle exec rake rubocop:auto_correct), however the rest need manual intervention. If you wouldn't mind taking a look again and addressing, we'll be happy to merge 👍

In theory those bits are fixed now.

I've been looking at this a bit, and I fear I'll need some help to get it over the line....

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d4cc7d7). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1191   +/-   ##
=======================================
  Coverage        ?   60.96%
=======================================
  Files           ?       14
  Lines           ?      351
  Branches        ?        0
=======================================
  Hits            ?      214
  Misses          ?      137
  Partials        ?        0

Continue to review full report at Codecov.

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

happy to help you @jcpunk.
Thank you for your contribution!
kind regards,
@adrianiurca


[Dev][Dropdow]Add styles

Current dropdowns with select/options are not following the design guidelines, as styling the select element is not trivial. The element uses the operating system's default values.
To fix it it may be necessary to add a library called react-select

We are now using the provided Dropdowns from Infima, the embedded CSS toolkit inside Docusaurus.


Is that possible I could pass self-defined variables into extra headers to FreeSwitch? or any better way.

These are typically directed to the google group.
I am thinking to pass some extra self-defined variables when using invite method.
I referred to the document https://sipjs.com/api/0.5.0/ua/#invitetarget-options and thought to use
invite("target"[,options])
I have tried
var options = {extraHeaders: ["X-Header: 2000", "X-pin: 2000","X-destination: xxxxxxxx"] };userAgent.invite('9000', options );
and in dialplan, try to get print it out using <action application="log" data="INFO ${extraHeaders}"/> 				<action application="log" data="INFO ${options}"/> 				<action application="log" data="INFO ${X-Header}"/> 				<action application="log" data="INFO ${extraHeaders.X-Header}"/>
all cannot work.
I also tried
var extraHeadersArray = new Array();     extraHeadersArray.push("2000");     extraHeadersArray.push("2000");     extraHeadersArray.push("xxxxxxxx");
then
options = {         extraHeaders : extraHeadersArray     };  userAgent.invite('9000', options);
Cannot work either. So can I ask whether there are some better ways to do this? I need to pass some variables to FreeSwitch to do some further logic.

This is not a bug. Please use our google group.

I thought I have open a new question. But I did not see the label. I will go to group. Thanks.


Plots from temporary experiments are first updated when finished

If you start a temporary experiment by using dvc exp run --temp, the plots will show the metrics of the last completed experiment in the workspace instead of the current temporary experiment.
When the temporary experiment finishes, it disappears from the plots and needs to be re-added, showing the actual metrics then.
It would be good to see the metrics of currently running temporary experiments in realtime in the Plots section.

Hi @ostromann,
Thanks for raising the issue.
There is a limitation in that DVC does not collect metrics for experiments running outside of the workspace (unless they are checkpoint based). This is being actively worked on at https://github.com/iterative/dvc/issues/8478. We have #3178 in this repo for the subsequent work needed.
By any chance are you using DVCLive to log metrics?

Thanks!
Yes, I do use DVCLive to log metrics.

Ok, as a temporary workaround, you can get live updates by connecting to Studio, switching from --temp to --queue and starting the queue with the extension. Live updates will show up in Studio for all experiments run from the queue.

Docs are here.
Hope this helps. LMK if you get stuck.
Note: I think it you start dvc exp run --temp with the environment variables STUDIO_TOKEN & STUDIO_REPO_URL included then you should still get live updates.

As far as I understand, that rules out DVC Studio for me, right now

There is an on-prem option for Studio (it's paid, but just in case your org can consider this). In no way saying we are not fixing this, we are working on this, it's prioritized.

@dberenbaum this issue is why we want to prioritise live plot updates for experiments running in the queue/temp directories. Collecting data for the separate queue/temp revisions would be problematic for the reasons mentioned in #3676. We would have to re-implement the "merging" logic, from DVC in the extension, make separate CLI calls, cache data etc. This would be a significant effort in the extension and IMO less valuable than implementing the ability to call plots diff with these revisions in DVC. I'd be happy to implement it on the DVC side if we decide it would be the best way forwards.
LMK what you think, 🙏🏻.

Hi @ostromann,
I realise it has been a long time since we gave you an update on progress. The good news is that the changes required to get live updates for plots running outside of the workspace were released with DVC 2.58.1. In version 0.9.0 of the extension, you can select experiments running outside of the workspace and view their plots.
Thanks for your patience. 0.9.0 should be in the marketplace soon.


WIP feat(test): improvement of performance --maxWorkers=4 and reworking the getTestingRenderedComponent

This PR introduces changes in the config for running tests on CircleCI by adding the maxWorker=4 option. Also, there are changes introduces in the getTestingRenderedComponent method for using shallow + dive instead of mount + find. Diffrences between performance of the UTs from this and the https://github.com/stardust-ui/react/pull/223
#223
PASS  test/specs/lib/FocusZone-test.tsx (7.599s)
PASS  test/specs/lib/mergeThemes/mergeThemes-test.ts (15.826s)
PASS  test/specs/lib/factories-test.tsx (16.349s)
PASS  test/specs/lib/AutoControlledComponent-test.tsx (16.381s)
PASS  test/specs/components/Image/Image-test.tsx (6.076s)
PASS  test/specs/components/Menu/MenuItem-test.tsx (30.639s)
PASS  test/specs/components/Menu/Menu-test.tsx (19.755s)
PASS  test/specs/components/Button/Button-test.tsx (22.247s)
#225
PASS  test/specs/lib/FocusZone-test.tsx (5.934s)
PASS  test/specs/lib/mergeThemes/mergeThemes-test.ts (8.221s)
PASS  test/specs/lib/AutoControlledComponent-test.tsx (8.314s)
PASS  test/specs/lib/factories-test.tsx (8.334s)
PASS  test/specs/components/Menu/MenuItem-test.tsx (14.436s)
PASS  test/specs/components/Menu/Menu-test.tsx (10.248s)
PASS  test/specs/components/Button/Button-test.tsx (12.757s)

Closing this for now in favor of: #223


tf.io.gfile.rename not working for directories in S3

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.12
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
The following code used to work in Tensorflow < 2.6. Upon Tensorflow 2.6, we had to import tensorflow_io. However, the tf.io.gfile.rename function which used to work on directories in S3 no longer works. I would like to update to a newer version of Tensorflow but this issue is preventing our organization from doing so, as some libraries we use use tf.io.gfile.rename to change folder names during training.
tf.io.gfile.rename should work on directories according to the Tensorflow documentation
Standalone code to reproduce the issue
import tensorflow as tf
import tensorflow_io as tfio

SOURCE_DIR = 's3://.../old_name/'
DEST_DIR = 's3://.../new_name/'

tf.io.gfile.rename(SOURCE_DIR, DEST_DIR)

Relevant log output
Traceback (most recent call last):
  File "/home/eigen/.config/JetBrains/PyCharm2023.1/scratches/tf2_s3_rename_test.py", line 9, in <module>
    tf.io.gfile.rename(SOURCE_DIR, DEST_DIR)
  File "/home/eigen/venvs/eigen-ml-tf2-12/lib/python3.10/site-packages/tensorflow/python/lib/io/file_io.py", line 622, in rename_v2
    _pywrap_file_io.RenameFile(
tensorflow.python.framework.errors_impl.FailedPreconditionError: Source is a directory or empty file

@j99ca Could you please make sure that the source and destination paths are appropriate?  As per the doc here , it should raise errors.OpError if it fails?
Thank you!

@sushreebarsa I have indeed confirmed the behaviour. It works fine when I revert the tensorflow version number. Can you replicate it in S3?

Hi,
Could you please close your old issue here https://github.com/tensorflow/tensorflow/issues/60165 to track the status at single place.

@sachinprasadhs I have closed the other story

@sachinprasadhs is there any resolution to this issue? It is still preventing us from migrating to a newer version of tensorflow above 2.5

Has anyone found a resolution or workaround for this issue?


WIP: Serialization and deserialization

Codecov Report

Merging #4 (621bb28) into master (3db9837) will increase coverage by 3.53%.
The diff coverage is 95.58%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   90.90%   94.44%   +3.53%
==========================================
  Files           5        7       +2
  Lines          22       90      +68
==========================================
+ Hits           20       85      +65
- Misses          2        5       +3

Impacted Files
Coverage Δ

Sts1CobcSw/Serialize/Serialize.hpp
66.66% <66.66%> (ø)

Tests/UnitTests/Serialize.test.cpp
100.00% <100.00%> (ø)

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


Tolerate 128bit trace ids by throwing out high bits

This bridges support for 128bit trace ids by not failing on their
receipt. Basically, this throws out any hex characters to the left
of the 16 needed for the 64bit trace id.
see: https://github.com/openzipkin/b3-propagation/issues/6

👍


Enabling the Purple Theme in Inji

Problem Statement
We should be able to create a build with both themes.
Resolution
The implementation for the purple theme is already available. But a few new UI components are not in sync with that theme. We should make sure that the purple theme works with the new UI changes.

Enable Theme selection based on the environment variables
Enable Theme selection for github actions
Selection of the theme in github action takes higher in priority than what is declared in .env file
When the selected theme via environment variable/.env file isn't recognised then DefaultTheme is applied
The logic of selection MUST take care of case-insensitiveness for theme selection

Does it mean that same build will support both Themes? And user change the Theme?

Does it mean that same build will support both Themes? - Yes at the build time
And user change the Theme? - Yes, user means, the person who is building the apk
This avoids maintaining two different branches and code diversion between purple theme and default theme.

Purple Theme is Enabled in the Inji

The Application Theme Can be Toggle using Github Workflow Action Input "Application Theme".
Which can either be "orange" or "purple", which Literally mean their respective Themes.

This change is delivered as part of 09-03-2023 release
INJI PR link: https://github.com/mosip/inji/pull/630

For Purple Theme:

Changed the OTP verification screen mobile logo colour from orange to purple
Changed the cursor colour from orange to purple in all places

We have an option to choose the theme (orange/purple) while triggering the build for the android apk.


[Feature] Implement RayService In-place update test in Golang

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Implement test_in_place_update in Golang, and add a file rayservice_in_place_update_test.go under https://github.com/ray-project/kuberay/tree/master/ray-operator/test/e2e.
https://github.com/ray-project/kuberay/blob/4fc17996bb7aedb06d4fc02727f1ca24c86f4090/tests/test_sample_rayservice_yamls.py#L205
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

@kevin85421  I 'd like to help with this


ci: scheduled&manual workflow to update bin.walrus.site

Successful workflow run: https://github.com/MystenLabs/walrus-docs/actions/runs/10006893810/job/27660483590
Closes #64

It's a bit unfortunate that we need a personal access token in the calling workflow
This is is common practice within the Mysten org right now.  AFAIK they're being used where workflows touch multiple repositories.

      - name: Copy site-builder binary
        run: "cp bin/site-builder site/site-builder-ubuntu-x86_64"

      - name: Update Walrus Site
        run: >
          RUST_LOG=site_builder=debug,walrus=debug,info
          site-builder
          --config walrus-sites/site-builder/assets/builder-example.yaml
          update --list-directory site ${{ vars.WALRUS_SITE_BIN_OBJECT }}

When the devnet workflow is being executed, the jobs for each arch are running in parallel, @mlegner can you confirm that only the  ubuntu-x86_64 job needs to call out to this workflow?


changes in indian flag

Is your feature request related to a problem? Please describe.
it is not responsive .the design and layout can be better
Describe the solution you'd like
we can make it responsive and also interactive by adding some hover effects and animations
Describe alternatives you've considered
By implementing these changes,  Indian flag page should be more visually appealing, interactive, accessible, and performant.
Additional context
pls assign this to me under vsoc

Assigned @shreya-2410


Data loss:  corrupted record at 0

Ubuntu 20
Python 3.8.10
Tensorflow 2.5
NVIDIA Titan RTX and NVIDIA Titan X
CUDA Version: 11.4

When I run the following command:
python deeplab2/trainer/train.py --config_file=./deeplab2/configs/cityscapes/panoptic_deeplab/resnet50_os32_merge_with_pure_tf_func.textproto --mode=eval --model_dir=. --num_gpus=1
I get the following error:
2022-03-25 15:25:18.774693: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudart.so.11.0
I0325 15:25:20.391919<PHONE_NUMBER>05536 train.py:65] Reading the config file.
I0325 15:25:20.395706<PHONE_NUMBER>05536 train.py:69] Starting the experiment.
2022-03-25 15:25:20.397703: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcuda.so.1
2022-03-25 15:25:20.449437: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: NVIDIA TITAN RTX computeCapability: 7.5
coreClock: 1.77GHz coreCount: 72 deviceMemorySize: 23.65GiB deviceMemoryBandwidth: 625.94GiB/s
2022-03-25 15:25:20.450306: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 1 with properties:
pciBusID: 0000:03:00.0 name: NVIDIA TITAN X (Pascal) computeCapability: 6.1
coreClock: 1.531GHz coreCount: 28 deviceMemorySize: 11.91GiB deviceMemoryBandwidth: 447.48GiB/s
2022-03-25 15:25:20.450328: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudart.so.11.0
2022-03-25 15:25:20.453842: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcublas.so.11
2022-03-25 15:25:20.453883: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcublasLt.so.11
2022-03-25 15:25:20.455577: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcufft.so.10
2022-03-25 15:25:20.455783: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcurand.so.10
2022-03-25 15:25:20.456216: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcusolver.so.11
2022-03-25 15:25:20.456952: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcusparse.so.11
2022-03-25 15:25:20.457090: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudnn.so.8
2022-03-25 15:25:20.460701: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1871] Adding visible gpu devices: 0, 1
2022-03-25 15:25:20.461012: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-03-25 15:25:20.743150: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: NVIDIA TITAN RTX computeCapability: 7.5
coreClock: 1.77GHz coreCount: 72 deviceMemorySize: 23.65GiB deviceMemoryBandwidth: 625.94GiB/s
2022-03-25 15:25:20.743871: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 1 with properties:
pciBusID: 0000:03:00.0 name: NVIDIA TITAN X (Pascal) computeCapability: 6.1
coreClock: 1.531GHz coreCount: 28 deviceMemorySize: 11.91GiB deviceMemoryBandwidth: 447.48GiB/s
2022-03-25 15:25:20.747358: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1871] Adding visible gpu devices: 0, 1
2022-03-25 15:25:20.747408: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudart.so.11.0
2022-03-25 15:25:21.602245: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1258] Device interconnect StreamExecutor with strength 1 edge matrix:
2022-03-25 15:25:21.602280: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1264]      0 1
2022-03-25 15:25:21.602287: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1277] 0:   N N
2022-03-25 15:25:21.602291: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1277] 1:   N N
2022-03-25 15:25:21.606291: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1418] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 21512 MB memory) -> physical GPU (device: 0, name: NVIDIA TITAN RTX, pci bus id: 0000:01:00.0, compute capability: 7.5)
2022-03-25 15:25:21.607156: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1418] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:1 with 11425 MB memory) -> physical GPU (device: 1, name: NVIDIA TITAN X (Pascal), pci bus id: 0000:03:00.0, compute capability: 6.1)
I0325 15:25:21.608190<PHONE_NUMBER>05536 train_lib.py:104] Using strategy <class 'tensorflow.python.distribute.one_device_strategy.OneDeviceStrategy'> with 1 replicas
I0325 15:25:21.875683<PHONE_NUMBER>05536 deeplab.py:57] Synchronized Batchnorm is used.
I0325 15:25:21.877165<PHONE_NUMBER>05536 axial_resnet_instances.py:144] Axial-ResNet final config: {'num_blocks': [3, 4, 6, 3], 'backbone_layer_multiplier': 1.0, 'width_multiplier': 1.0, 'stem_width_multiplier': 1.0, 'output_stride': 32, 'classification_mode': True, 'backbone_type': 'resnet', 'use_axial_beyond_stride': 0, 'backbone_use_transformer_beyond_stride': 0, 'extra_decoder_use_transformer_beyond_stride': 32, 'backbone_decoder_num_stacks': 0, 'backbone_decoder_blocks_per_stage': 1, 'extra_decoder_num_stacks': 0, 'extra_decoder_blocks_per_stage': 1, 'max_num_mask_slots': 128, 'num_mask_slots': 128, 'memory_channels': 256, 'base_transformer_expansion': 1.0, 'global_feed_forward_network_channels': 256, 'high_resolution_output_stride': 4, 'activation': 'relu', 'block_group_config': {'attention_bottleneck_expansion': 2, 'drop_path_keep_prob': 1.0, 'drop_path_beyond_stride': 16, 'drop_path_schedule': 'constant', 'positional_encoding_type': None, 'use_global_beyond_stride': 0, 'use_sac_beyond_stride': -1, 'use_squeeze_and_excite': False, 'conv_use_recompute_grad': False, 'axial_use_recompute_grad': True, 'recompute_within_stride': 0, 'transformer_use_recompute_grad': False, 'axial_layer_config': {'query_shape': (129, 129), 'key_expansion': 1, 'value_expansion': 2, 'memory_flange': (32, 32), 'double_global_attention': False, 'num_heads': 8, 'use_query_rpe_similarity': True, 'use_key_rpe_similarity': True, 'use_content_similarity': True, 'retrieve_value_rpe': True, 'retrieve_value_content': True, 'initialization_std_for_query_key_rpe': 1.0, 'initialization_std_for_value_rpe': 1.0, 'self_attention_activation': 'softmax'}, 'dual_path_transformer_layer_config': {'num_heads': 8, 'bottleneck_expansion': 2, 'key_expansion': 1, 'value_expansion': 2, 'feed_forward_network_channels': 2048, 'use_memory_self_attention': True, 'use_pixel2memory_feedback_attention': True, 'transformer_activation': 'softmax'}}, 'bn_layer': functools.partial(<class 'tensorflow.python.keras.layers.normalization_v2.SyncBatchNormalization'>, momentum=0.9900000095367432, epsilon=0.0010000000474974513), 'conv_kernel_weight_decay': 0.0}
I0325 15:25:22.024067<PHONE_NUMBER>05536 deeplab.py:96] Setting pooling size to (33, 65)
I0325 15:25:22.024253<PHONE_NUMBER>05536 aspp.py:135] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:22.024336<PHONE_NUMBER>05536 aspp.py:135] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
2022-03-25 15:25:25.563853: W tensorflow/python/util/util.cc:348] Sets are not currently considered sequences, but this may change in the future, so consider avoiding using them.
I0325 15:25:25.565743<PHONE_NUMBER>05536 controller.py:391] restoring or initializing model...
restoring or initializing model...
WARNING:tensorflow:From /home/me/.local/lib/python3.8/site-packages/tensorflow/python/training/tracking/util.py:1315: NameBasedSaverStatus.__init__ (from tensorflow.python.training.tracking.util) is deprecated and will be removed in a future version.
Instructions for updating:
Restoring a name-based tf.train.Saver checkpoint using the object-based restore API. This mode uses global names to match variables, and so is somewhat fragile. It also adds new restore ops to the graph each time it is called when graph building. Prefer re-encoding training checkpoints in the object-based format: run save() on the object-based saver (the same one this message is coming from) and use that checkpoint in the future.
W0325 15:25:25.584475<PHONE_NUMBER>05536 deprecation.py:330] From /home/me/.local/lib/python3.8/site-packages/tensorflow/python/training/tracking/util.py:1315: NameBasedSaverStatus.__init__ (from tensorflow.python.training.tracking.util) is deprecated and will be removed in a future version.
Instructions for updating:
Restoring a name-based tf.train.Saver checkpoint using the object-based restore API. This mode uses global names to match variables, and so is somewhat fragile. It also adds new restore ops to the graph each time it is called when graph building. Prefer re-encoding training checkpoints in the object-based format: run save() on the object-based saver (the same one this message is coming from) and use that checkpoint in the future.
I0325 15:25:25.598557<PHONE_NUMBER>05536 controller.py:397] initialized model.
initialized model.
I0325 15:25:25.601044<PHONE_NUMBER>05536 controller.py:277]  eval | step:      0 | running complete evaluation...
 eval | step:      0 | running complete evaluation...
2022-03-25 15:25:25.957597: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:176] None of the MLIR Optimization Passes are enabled (registered 2)
2022-03-25 15:25:25.977684: I tensorflow/core/platform/profile_utils/cpu_utils.cc:114] CPU Frequency:<PHONE_NUMBER> Hz
I0325 15:25:27.446719<PHONE_NUMBER>05536 api.py:446] Eval with scales ListWrapper([1.0])
I0325 15:25:28.432155<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:28.461583<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:28.487152<PHONE_NUMBER>05536 api.py:446] Eval scale 1.0; setting pooling size to [33, 65]
I0325 15:25:32.265706<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:32.291409<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
WARNING:tensorflow:From /home/me/.local/lib/python3.8/site-packages/tensorflow/python/ops/array_ops.py:5043: calling gather (from tensorflow.python.ops.array_ops) with validate_indices is deprecated and will be removed in a future version.
Instructions for updating:
The `validate_indices` argument has no effect. Indices are always validated on CPU and never validated on GPU.
W0325 15:25:32.952551<PHONE_NUMBER>05536 deprecation.py:528] From /home/me/.local/lib/python3.8/site-packages/tensorflow/python/ops/array_ops.py:5043: calling gather (from tensorflow.python.ops.array_ops) with validate_indices is deprecated and will be removed in a future version.
Instructions for updating:
The `validate_indices` argument has no effect. Indices are always validated on CPU and never validated on GPU.
I0325 15:25:34.675148<PHONE_NUMBER>05536 api.py:446] Eval with scales ListWrapper([1.0])
I0325 15:25:34.700171<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:34.724930<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:34.748688<PHONE_NUMBER>05536 api.py:446] Eval scale 1.0; setting pooling size to [33, 65]
I0325 15:25:35.657829<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
I0325 15:25:35.683211<PHONE_NUMBER>05536 api.py:446] Global average pooling in the ASPP pooling layer was replaced with tiled average pooling using the provided pool_size. Please make sure this behavior is intended.
2022-03-25 15:25:36.482976: W tensorflow/core/framework/cpu_allocator_impl.cc:80] Allocation of 16801800 exceeds 10% of free system memory.
2022-03-25 15:25:36.522768: W tensorflow/core/framework/cpu_allocator_impl.cc:80] Allocation of 16801800 exceeds 10% of free system memory.
2022-03-25 15:25:36.522816: W tensorflow/core/framework/cpu_allocator_impl.cc:80] Allocation of 16801800 exceeds 10% of free system memory.
2022-03-25 15:25:36.524742: W tensorflow/core/framework/cpu_allocator_impl.cc:80] Allocation of 16801800 exceeds 10% of free system memory.
2022-03-25 15:25:36.576277: W tensorflow/core/framework/cpu_allocator_impl.cc:80] Allocation of 16801800 exceeds 10% of free system memory.
2022-03-25 15:25:36.971095: E tensorflow/core/grappler/optimizers/meta_optimizer.cc:808] layout failed: Invalid argument: Size of values 3 does not match size of permutation 4 @ fanin shape inDeepLab/PostProcessor/StatefulPartitionedCall/while/body/_85/while/SelectV2_1-1-TransposeNHWCToNCHW-LayoutOptimizer
Traceback (most recent call last):
  File "deeplab2/trainer/train.py", line 76, in <module>
    app.run(main)
  File "/home/me/.local/lib/python3.8/site-packages/absl/app.py", line 312, in run
    _run_main(main, args)
  File "/home/me/.local/lib/python3.8/site-packages/absl/app.py", line 258, in _run_main
    sys.exit(main(argv))
  File "deeplab2/trainer/train.py", line 71, in main
    train_lib.run_experiment(FLAGS.mode, config, combined_model_dir, FLAGS.master,
  File "/media/me/storage/src/github/STEP/deeplab2/trainer/train_lib.py", line 200, in run_experiment
    controller.evaluate(steps=config.evaluator_options.eval_steps)
  File "/home/me/src/github/STEP/models/orbit/controller.py", line 282, in evaluate
    eval_output = self.evaluator.evaluate(steps_tensor)
  File "/home/me/src/github/STEP/models/orbit/standard_runner.py", line 346, in evaluate
    outputs = self._eval_loop_fn(
  File "/home/me/src/github/STEP/models/orbit/utils/loop_fns.py", line 75, in loop_fn
    outputs = step_fn(iterator)
  File "/home/me/.local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py", line 889, in __call__
    result = self._call(*args, **kwds)
  File "/home/me/.local/lib/python3.8/site-packages/tensorflow/python/eager/def_function.py", line 950, in _call
    return self._stateless_fn(*args, **kwds)
  File "/home/me/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 3023, in __call__
    return graph_function._call_flat(
  File "/home/me/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 1960, in _call_flat
    return self._build_call_outputs(self._inference_function.call(
  File "/home/me/.local/lib/python3.8/site-packages/tensorflow/python/eager/function.py", line 591, in call
    outputs = execute.execute(
  File "/home/me/.local/lib/python3.8/site-packages/tensorflow/python/eager/execute.py", line 59, in quick_execute
    tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
tensorflow.python.framework.errors_impl.DataLossError: 2 root error(s) found.
  (0) Data loss:  corrupted record at 0
	 [[{{node MultiDeviceIteratorGetNextFromShard}}]]
	 [[RemoteCall]]
	 [[IteratorGetNext]]
	 [[DeepLab/PostProcessor/StatefulPartitionedCall/while/LoopCond/_110/_104]]
  (1) Data loss:  corrupted record at 0
	 [[{{node MultiDeviceIteratorGetNextFromShard}}]]
	 [[RemoteCall]]
	 [[IteratorGetNext]]
0 successful operations.
0 derived errors ignored. [Op:__inference_eval_step_10512]

Function call stack:
eval_step -> eval_step

@robonrrd  I am having the same issue, were you able to resolve it?

Please check if you have successfully generated the TFRecord for your target dataset.
You could at least see if the TFRecord contains something to make sure.

Closing the issue, as there is no activity for a while.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1831 ms

Resolved: Bofrost.at performance has improved in c468faf after 6 minutes.


Add programs involving recursion in the following languages

Programs involving recursion in
C
C++
Java
Python
JS
Add them in their respective folders

I can do it , assign to me !

I can do it , assign to me !

Just add any 1 program and that too in any 1 language

Okay

can i do this using java?

I would like to write code in c++ @ankitapuri

Can I write the height of the binary tree problem using recursion in java?

Thank you

Can I write 'print subsets of sum k in an array' using recursion in java?

Hey @ankitapuri I would like to add reverse of a string using recursion problem!!
Could you pls assign this issue to me?

Hi @ankitapuri I'd love to do this using Java. Could you please assign it to me?

Hi, I would like to add code for finding palindrome in cpp

I just added two programs of Recursion in Java.

Hi @ankitapuri Please review my pr for string reverse in c++ using recursion -> https://github.com/ankitapuri/DSA-guide/pull/83
Thanks

Hello, I would like to add the following two Recursion programs in Java :
1 : Convert Decimal to Binary.
2 : Print all possible combinations of N-dice Rolls

Hey @ankitapuri I would like to add reverse of a string using recursion problem!!
Could you pls assign this issue to me?

specify the language as well

@ankitapuri I have added many problems in recursion in java like subset, string permutation, josephus problem, checking palindrome allow me to push the code.
Thanks

I want to work on this, assign this to me

Hi @ankitapuri I'd love to do this using Java. Could you please assign it to me?

@ankitapuri added the Java code for Tower of Hanoi

Hey @ankitapuri, I can contribute a bunch of recursion solutions here,
does this require assignment or is it ok to directly push a PR?

Don't add so many codes please it takes a lot of effort to review it

I want to contribute Fibonacci sequence using recursion in JavaScript.

Hello @ankitapuri, I would like to add the solution of the Staircase Problem in Java.

I am closing this issue for some time
those who were assigned can still continue


fix slow import: only import quimb.tensor when needed

partially addresses #947: reduces import qualtran time from ~3.2s to ~1.4s

@mpharrigan ptal

is it accurate to say that the change is only strictly required in controlled.py, since all the other locations aren't actually pulled in when you import qualtran.?

is it accurate to say that the change is only strictly required in controlled.py, since all the other locations aren't actually pulled in when you import qualtran.?

Yep. The remaining changes speed up import qualtran.bloqs.basic_gates by ~2s.