Issue 831 - Repeater major changes

Major CSS & JS cleaning and refactoring
Major UX/UI update (to offer something similar to Menu and Widgets)
Dynamic Row Labels : user can now override the “row” text string as well as define a text input as label (i.e.:title)

Related to #831

looks good...
Merged in develop. :+1:
We can do more testing etc there. That's the branch most devs use, so if there's an issue we'll be able to catch it early on.
Great work...

Any documentation on how to change the labels? I'd like to be able to set the "Row #" as well as "Add new X" labels fully. Not just the "Row" if that makes any sense!

@Quaked updating the documentation is also on my to-do list :-) but you'll find some details here : #831

@Quaked feel free to submit a PR to the documentation with this ;-)

@guillaumemolter point being I don't know. Right now I'm using a field to populate the "title", but I'm not aware of a way to change the actual text "Add new". This is what I've got and it works, but I still need to change the 'Add new' text:
Kirki::add_field( self::$cfg, array(
	'type'        	=> 'repeater',
	'label'       	=> esc_attr__( 'Sektioner', Settings::$textdomain ),
	'section'     	=> 'front_sections',
	'priority'    	=> 10,
	'settings'    	=> 'sections',
	'fields'		=> $section_fields,
	// this replaces *row* in; "Add new row" and title "Row #" until replaced by field 'section_header'
	'row_label' 	=> array(
		'type' 			=> 'field',
	    'value' 		=> 'section',
	    'field' 		=> 'section_header',
	),
));

Ok sorry I thought my comment in #837 was clear, obviously it is not.
So you currently have 3 options:

Don't pass the 'row_label' parameter. It will result in the repeater using the default value:
array( 'type' => 'text' , 'value' => 'row' )
Result: Each row will have a "Row X" label and the add button label will be "Add new row"
Change the text label by passing the row_label parameter with a value:
array( 'type' => 'text' , 'value' => 'slide' ) N.B.: I'm pretty sure that type=text is not even required in this case
Result: Each row will have a "Slide X" label and the add button label will be "Add new slide"
Use dynamic field labels by passing the row_label parameter with a value:
array( 'type' => 'field' , 'value' => 'slide', 'field' => 'a-field-id' )
Result: As long as a-field-id is empty the row will have a "Slide X" label and will then switch to a-field-id value once not empty. The add button label will be "Add new slide".

Other use cases are currently not supported.

Ah. I believe you were clear, just not in regards to the text Add new which unfortunately doesn't seem to be changeable. That's the issue really, my theme options aren't in English! As an example, 'row' is 'rad' in Swedish, so I'm getting "Add new rad". Maybe a future update will sort this out as some other parts (like 'Add Image' in the image field) aren't translatable either.

So you can definitly translate the "add new" as it is added using l10n . The only current constraint is that "row/rad" has to be at the end of the label but it doesn't seem to be an issue in your case (Lägg till ny rad).
You'll have to ask @aristath for more details on how translation works in Kirki.

@Quaked You can translate those using poEdit just like any other WordPress plugin, or using the web-ui from https://translate.wordpress.org/projects/wp-plugins/kirki
Otherwise you can always do something like this:
function my_custom_translation_strings( $config ) {
    $config['i18n']['add-new'] = 'LALALA';
    return $config;
}
add_filter( 'kirki/config', 'my_custom_translation_strings' );

You can see the strings available for translation this way here: https://github.com/aristath/kirki/blob/2.3.0/includes/class-kirki-l10n.php#L101-L217


[EmitC] Rewrite import ops before call ops in conversion

The conversion for the call/call.variadic op expects functions to exist which get generated by the import op conversion.

This should be ready now I think.


[BUG] The offer to create the img/ directory uses relative path

Please confirm

[x] I am running the latest version of this plugin

Describe the bug
It looks to me like the img/ directory gets created from where vim was launched, not the notes path.
To Reproduce
Steps to reproduce the behavior:

Start nvim outside of the notes directory
say yes to create the img directory.
note that the new directory is outside of the notes path.

Expected behavior
The directory should get created at the base of the notes structure, not the cwd of where vim was launched.
Operating system (please complete the following information):

OS: Linux

I can confirm this behavior.

Indeed, this is because the other directories (templates, weekly and dailies) are specified as absolute path in the config, while for images it should be a subdirectory.
So the check function that creates the directory should work a bit differently for images.
https://github.com/renerocksai/telekasten.nvim/blob/f20fb7862b35be4795ad76ed08d1011f94f59d0b/lua/telekasten.lua#L162-L172
An other way around would be to have the images directories as a full path as well in the config (but then the image search functions should be modified I guess)

Thanks for the response.  I've updated my config to include the absolute path to the image_subdir.

@zalegrala I do not think this should be considered closed. Using an absolute path in the config for image_subdir is definitely going to cause issues when you will try to paste image from the clipboard. I do not know the best approach to solve this properly, so you may want to keep it open while we wait for @renerocksai 's input.

Sorryyyyy! That's my bad when I introduced the dir check yesterday. Will fix ASAP.
Please do not put absolute paths into your configs for image_subdir!

This should fix it.


ideas for multigene display

Had a quick chat with Kim this morning. He mentioned it might be possible to unclutter this further by not displaying the "gene":

so in tis instance it would be fine not to display the gene, except
i) we need to hyperlink it
and
ii) not all genes have the gene name in the genotype name (although all of these do)
So, we were wondering how common it is not to have the gene name in the genotype, and for these the gene could be in parentheses after the name. However this would mena hyperlinking part of the geneotype to link to the gene pages which might look strange?
@mah11
@Antonialock
thoughts?

[ ]   sort  within geneotype allele rows alphabetically

[ ]   one of the above alleles has (wild type) but more generally it is hidden...I wonder why that one is different... maybe it is  different somehow in  the session?

Re gene name display, hyperlinking etc. - would it be terrible to just display the allele names on gene pages, and have the genotype link to a genotype page for all the gory details? People should be getting familiar with that sort of thing via Canto ...

Good plan.
Minimally I'd still like to see     cdc2-33(A177T)  not just   cdc2-33 though, (I think?... but maybe its not completely necessary in this view.... especially if we do other clever stuff with amino acid variation linked to phenotypes in a protein browser view, but I'm getting carried away.....)
I can't remember what we do in Canto now I'll need to look!

Yeah, we could def miss the "allele descriptions" out of the collapsed view couldn't we....
http://www.pombase.org/spombe/result/SPBC11B10.09#fission_yeast_phenotype-multi_allele

yes, allele description hidden in compact (with mouseover?) and show in full view sounds good

[ ]  Don't hyperlink an allele to its own gene  page

relayed to
https://github.com/pombase/website/issues/53

I don't think we need this ticket?


New-DcnClone: use dbatools for db enumeration

Fixes #216

I wonder if that is something we want to do. We already have the server object which already contains all the database objects. Using Get-DbaDatabase opens another connection to the database server which in theory isn't needed

Could definitely to it custom, but the DBA tools function does a lot of heavy lifting to ensure it doesn't error out, so re-doing that work might not be worth it.

I wonder if that is something we want to do. We already have the server object which already contains all the database objects. Using Get-DbaDatabase opens another connection to the database server which in theory isn't needed

Could definitely to it custom, but the DBA tools function does a lot of heavy lifting to ensure it doesn't error out, so re-doing that work might not be worth it.

Good call. If I write my script and already have a connections I try to refrain from having multiple connections because when you're iterating lots and lots of objects, then those connections  can really add up.
In this case I think it could be benefitial.

Thank you for putting in the effort


Use the existing base_path if present

To generate unique base paths, the Need API added numbers to the
end. Therefore, this base path needs to be used, and you can't just
generate the base path from the goal.

👍


Added timeout to test-pylenium action

The bug
UI tests sometimes fail during CI (fixes #307)

Description of the Change
I added a timeout argument after selenoid starts up on the runner.

name: Start Selenoid Server
# https://github.com/marketplace/actions/start-selenoid-server?version=v2
uses: Xotabu4/selenoid-github-action@v2
+    with:
+        selenoid-start-arguments: |
+        --args "-timeout 60s"

Verification
Tested on my own branch by changing the action trigger to ensure UI tests all passed. Then changed it back to its previous state once this was validated.

Release Notes
N/A

New to open source, open to any suggestions on how to improve this and/or future PRs!

This is great! Thanks for taking the time to create a PR. Let's see how the pipeline(s) do 😄

Poop... looks like the pipeline failed in the UI Tests step. Here's the job for more details: https://github.com/ElSnoMan/pyleniumio/actions/runs/5357935823/jobs/9723194411?pr=308
It shows that the page crashed for one of the tests which is relatively vague for our situation. Some ideas:

Increase the timeout to see if that stabilizes it
See what other config options for Selenoid exist in the GH Action that we can use

💡 I'm open to other ideas as well because nothing is "forcing" us to use Selenoid or this specific action

Sounds good, I'll start by increasing the timeout and then look for potential other options. Thanks for your time!

Just wanted to check in on this - I added a commit to the PR that increased the timeout. Are there additional steps I should take?

@abzdel I'm super sorry for just getting to this. You will need to merge main into your branch and then send a new commit. If all passes, we're good to merge

No worries @ElSnoMan - just merged in main and added two commits (something in CI failed on the first run, but not when I pushed again - not sure why). Please let me know if any other changes are necessary


Create own torch_geometric.data.Data class with extra features

Hi,
first of all thank you very much for your awesome library.
We all appreciate it your work a lot.
My question concerns the torch_geometric.data.Data class:
Let's say I have a graph and I have some extra features which cannot be described with the attributes of the torch_geometric.data.Data class.
How would I add these extra features of edges, nodes or the entire graph to the Data class?
Below is a conceptual example of what I would like to achieve.
It basically boils down to the Dataloader spitting out more information than "just" x, y, edge_index, edge_attribute etc.
Specifically, I would like to receive the extra_data1 and extra_data2 during the training loop, as highlighted in the AugmentedData class.
In fact, during my testing the Dataloader produces seemingly right batches with all the relevant information including extra_data1 and extra_data2.
The Dataloader seemingly grabs all the class attributes correctly and apparently there is no need to implement an extra getitem method?
I just wanted to make sure what the proper way is to implement such extra data.
Thanks in advance for your time and thanks a lot for your effort!
import torch
import torch.nn.functional as F

import torch_geometric
from torch_geometric import transforms
from torch_geometric.data import InMemoryDataset, Data

import os

class AugmentedData(torch_geometric.data.Data):

	def __init__(self,
	             _x                 = None,
	             _edge_index        = None,
	             _edge_attribute    = None,
	             _y                 = None,
	             _pos               = None,
	             _norm              = None,
	             _face              = None,
	             _extra_data1       = None,
	             _extra_data2       = None)

		super().__init__()

		self.x                 = _x,
		self.edge_index        = _edge_index,
		self.edge_attribute    = _edge_attribute,
		self.y                 = _y,
		self.pos               = _pos,
		self.norm              = _norm,
		self.face              = _face,

		# !!!!!!!
		self.extra_data1       = _extra_data1,
		self.extra_data2       = _extra_data2
		# !!!!!!!

class AugmentedDataSet(InMemoryDataset):
	def __init__(self,
	             root=None,
	             name=None,
	             transform=None,
	             pre_transform=None):

		super().__init__(root, name, transform, pre_transform)

		self.data, self.slices = torch.load(self.processed_paths[0])
		os.remove(self.processed_paths[0])

	@property
	def raw_file_names(self):

		return ['xzy_raw']

	@property
	def processed_file_names(self):

		return ['xzy_processed']

	def download(self):
		pass

	def process(self):

		data = [AugmentedData(...), AugmentedData(...), AugmentedData(...)] # Code obviously not working

		data, slices = self.collate(data)

		torch.save((data, slices), self.processed_paths[0])

data = AugmentedDataSet()

dataloader = torch_geometric.data.DataLoader(data)

Hi, you do not need to inherit from Data. Data can hold any attribute:
data = Data(x=x, extra_data1=extra_data1)
data.extra_data2 = data.extra_data2

In addition, no modifications are needed for the Dataset.

Nice! Thanks a lot!


feat(DIA-797): remove breadcrumb label from saved artworks screen

This PR resolves DIA-797
Description
There are some minor differences in how the navigation headers for the saves, follows, and alerts screens work that are more obvious now that those surfaces can be accessed easily from the redesigned profile screen.
This PR addresses the minor difference that the saves screen header has a "profile" breadcrumb next to the back nav button and the other two screens don't.

Before

After

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a [feature flag], or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an [app state migration], or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates
Cross-platform user-facing changes

Removed back nav breadcrumb from saved artworks screen

This PR contains the following changes:

Cross-platform user-facing changes (Removed back nav breadcrumb from saved artworks screen)

  Generated by :no_entry_sign: dangerJS against 917daf023ef7c3e59d675c2a0ec78bfd795fc932

Still a good cleanup!


Add location codes to views

From this discussion realized that location codes should be added to the API endpoints

You just need to do a pull/push refresh to get the latest CHANGELOG conflict resolved, then good to go.


[Bug] Hit none partition cache, but hit range is still right

Proposed changes
Doris supports two kinds of cache mode: sql_cache and partition_cache. sql_cache takes sql string as key and cache the whole data. partition_cache splits the data into many partition data and caches them differently. Therefore a query may hit part of the partition_cache data. If a query hits the left part of the data, we call the hit range is left. If a query hits the right part of the data, we call the hit range is right. And if a query hits the whole part of the data, we call the hit range is full.
A query does not hit any partition cache, but the algorithm still returns hit range right. It should return hit range none.
Related issue: #5136
Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] Documentation Update (if none of the other choices apply)
[] Code refactor (Modify the code structure, format the code, etc...)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on (Fix #ISSUE), and have described the bug/feature there in detail
[x] Compiling and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] If this change need a document change, I have updated the document
[x] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM


when I host online the bot doesn't play the music, the bot enters and exits

hi, I'm writing this issue because I don't know how to solve it, and I don't know if anyone has the same problem as me, basically when I host my discord bot online, the music doesn't start, the bot enters it says there is music and exits saying that's over, if I host it locally from my pc everything works
(sorry for my english)

i have the same exact problem

Also happened recently

Your server does not have FFmpeg, so you will need to install FFmpeg on it. Despite the fact that package.json does include ffmpeg and ffmpeg-static, that is not enough and you will need to install the binaries.
For example, here are the steps I used on my Rocky Linux server:
https://computingforgeeks.com/install-use-ffmpeg-on-rocky-alma-9/
sudo dnf install epel-release
sudo dnf config-manager --set-enabled crb
sudo dnf install --nogpgcheck https://mirrors.rpmfusion.org/free/el/rpmfusion-free-release-$(rpm -E %rhel).noarch.rpm -y
sudo dnf install --nogpgcheck https://mirrors.rpmfusion.org/nonfree/el/rpmfusion-nonfree-release-$(rpm -E %rhel).noarch.rpm -y
sudo dnf install ffmpeg ffmpeg-devel

Try different virtual machines and if you are using the npm install command, download modules such as ffmpeg, ffmpeg-static, distube individually, not as a bulk download.


codecov: make project coverage status informational

What problem does this PR solve?

Issue Number: ref https://github.com/pingcap/tiflow/issues/10473
What is changed and how it works?
To make the codecov status of project just informational in github commit statuses on components and flags level, not fail the
commit statuses.
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
[x] No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/retest


Use yaml.UnmarshalStrict() during strict validation

Earlier this week the https://github.com/go-yaml/yaml/pull/307 landed which adds checking of duplicate keys when unmarshaling yaml, which would have saved me a few hours with a buggy pipeline that I wrote.
This PR changes to the behaviour of verify-pipeline to use yaml.UnmarshalStrict(), only when the existing --strict option is set. This also has the benefit of picking up any field names in a yaml file that don't match a struct (when unmarshalled into a map, anything is allowed) which can help detect typos.
Note that in order for these tests to pass, you will need to vendor in a new version of gopkg.in/yaml.v2. I'm not sure how they plan to update their branch - in my testing of below I updated to the tip of the devel branch at https://github.com/go-yaml/yaml, which I verified passes the full set of integration tests (testflight).
diff --git a/src/gopkg.in/yaml.v2 b/src/gopkg.in/yaml.v2
index e4d366fc..031c9222 160000
--- a/src/gopkg.in/yaml.v2
+++ b/src/gopkg.in/yaml.v2
@@ -1 +1 @@
-Subproject commit e4d366fc3c7938e2958e662b4258c7a89e1f0e3e
+Subproject commit 031c922227a592b2b562a1833438308381f9a8bf

(maybe you'll want to wait for a newer release from them?)
Anyway, submitting the PR now with some basic tests.
It might be interesting to consider using yaml.UnmarshalStrict() everywhere that yaml.Unmarshal() is used, but that's a bigger change to contemplate.

@vito - looks like a new version of gopkg.in/yaml.v2 has been released which has the changes we need, so this ought to be able to be merged now.
e.g. https://gopkg.in/yaml.v2 points to https://github.com/go-yaml/yaml/tree/v2.1.1 which includes https://github.com/go-yaml/yaml/commit/77b384eced7745af978888311ea3c67e57c7ed96

Hello any news ?

Thanks! All good.

Man! beer++ if we meet one day!


Fixed bug where client is nil.

Sometimes the client is nil and it throws an error. This PR Fixes this.

thanks for the contribution. can you please keep the code formatting the same? it's hard to see the changes in a diff as it looks like everything has changed.
Thanks!


Timeout error fix

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 21 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
109

Relevant Lines:
109

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

21 of 21 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
109

Relevant Lines:
109

💛  - Coveralls


Fix version bump workflow

Description
Removing the manual dispatch on upload workflow action, and the version env var from the commit message, needs more work and testing.
General Checklist

[ ] Tests added for this feature/bug
if it was a bug, test must cover it.
[ ] Conform by the style guides, by using formatter
[ ] Documentation updated
[ ] (optional) Please add a label to your PR

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.894%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3671

Relevant Lines:
4462

💛  - Coveralls


[Bug report] Line number and code misalignment

Bug report
Description
Line numbers and codes are misaligned on both iOS and macOS Safari, especially when window scaling is performed, seems caused by unwanted line wrap.

Steps to reproduce

Reproduction link / repo: https://v2.vuepress.vuejs.org/reference/cli.html#dev

Expected behavior
Line numbers and codes are misaligned when your scale down the window width.

Screenshots
https://user-images.githubusercontent.com/15633984/134792883-7d9e196c-afb8-4d59-8ab3-44eb368dc719.mov

Environment info

Browser: Safari 15.0(176<IP_ADDRESS>.2)
VuePress: v2.0.0-beta.26

Weird. Could you help to check on other browsers?

Safari do have some weird issues, such as mine https://github.com/vuepress-theme-hope/vuepress-theme-hope/issues/611

Weird. Could you help to check on other browsers?

Chrome is fine

It seems add display: -webkit-box here would be a workaround:
https://github.com/vuepress/vuepress-next/blob/459602898cee7c3ecd5d3c05c83a7db97729a14f/packages/%40vuepress/theme-default/src/client/styles/code.scss#L206-L210

But the spacing between code and scrollbar looks strange.

Pretty sure this issue is related with this one.

I set code block to use display: inline-table in https://github.com/Unbinilium/vuepress-next/commit/7cd08305ee72239b292a4f5998e6a692672215fa , this issue seems resolved, but not tested on the other browsers.
https://user-images.githubusercontent.com/15633984/135033000-bc4b6385-1643-4238-b8f0-12264676d26f.mov

A similar or same issue also appears on other browsers.
You can see a slight shift, for example at https://v2.vuepress.vuejs.org/reference/default-theme/config.html#navbar
Edge (Chromium) 94.0.992.38

Firefox 92.0.1


Unknown bug with marking item as complete/incomplete

I discovered a strange behavior when sometimes, when I press on an item to mark it as complete or incomplete, it creates a duplicate item and sometimes overwrites other items. But I haven't been able to reliably reproduce the error.

Try using list name as list id (enforce list name uniqueness)
Alternatively, go back to numbers and have an algorithm that finds the lowest missing number (having a static number that keeps incrementing will be an issue).

Bug is not related to list id. Something else is going on.

Commit: https://github.com/johneastman/todo/commit/fefb38485dda3ced99700c854e5d5cb71b45b061


Use public domain resources for examples.

[X] Has this change been discussed on the forum or in an issue before? https://en.sfml-dev.org/forums/index.php?topic=13420
[X] Does the code follow the SFML Code Style Guide?
[X] Have you provided some example/test code for your changes? Sort of. It is a bug fix.
[X] If you have additional steps which need to be performed list them as tasks!
[ ] Modify or remove examples/asset_licenses.md. (Forgive the typo.)
[ ] Answer any further questions.

Description
This PR fixes some questions about non-free resources which are used in the examples for SFML. It was also mentioned that the "SFML Pong" example could be an issue, so I changed it to "SFML Tennis".
A large number of resources were replaced, and all of the replacements are public domain. A list of the assets + their licenses/creators can be found in example/asset_licences.md (again, sorry for the typo, :P).
This PR is related to the issue  #1627
Tasks

[X] Tested on Linux
[ ] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Build the examples, and test them.

non-free resources which are used in the examples for SFML

I'm surprised of this statement. An effort was made in SFML 2, to use only free resources. Since you're talking about the non-free resources inside the SFML SDK, I assume you have a list of them, together with their respective license?

@LaurentGomila This is the classic case of "it's not free unless explicitly declared as such". Some distributions have very strict requirements on the packages they make available through their package repository because they don't want to have to deal with any kind of litigation.
Saying something is or should be freeware is not enough, there has to be an explicit notice from a credible source and if we use these resources we will also have to state where they came from and under what license they are offered.
The README for Sansation contains the following text:
Sansation - freeware font family
Version 1.31
Bernd Montag © 2011 - All Rights Reserved

This font family is freeware and can be used freely for personal and commercial purposes.
Although a PayPal donation in return is very much appreciated.

You may share this font on CDs, websites,... with the following restrictions:

	- Modification of the font files is only allowed for personal use - don´t distribute a modified version of the font files!
	- Do not rename the font files!
	- Do not sell the font files!
	- Do not pass the font files without this textfile!
	- Make sure you have downloaded the latest update from www.dafont.com for best optical results.

If you have further questions - please contact me.

<EMAIL_ADDRESS>
This unfortunately doesn't pass as "free" in the context of FOSS.
PONG also seems to be a registered trademark: https://tsdr.uspto.gov/#caseNumber=76148525&caseType=SERIAL_NO&searchType=statusSearch
In my opinion we should really just replace the resources with ones that are obviously and indisputably free. Which resources these should be is mostly a matter of taste.

binary1248 is absolutely right on. So

Which resources these should be is mostly a matter of taste.

and that taste is up to you guys. Let me know if you guys like or hate what I've done with it.

For reference here are the resources Debian excludes, with here having some more reasoning:
Files excluded from Debian package
================
The Debian documentation package ships some samples that won't run out
of the box. This is because the data files aren't free (and for some
of them it's questionable if it's even allowed to distribute them). To
run these samples you might need to find replacements for the following
files:

  examples/android/app/src/main/assets/canary.wav
  examples/android/app/src/main/assets/orchestral.ogg
  examples/iOS/resources/canary.wav
  examples/iOS/resources/orchestral.ogg
  examples/opengl/resources/background.jpg
  examples/opengl/resources/texture.jpg
  examples/shader/resources/background.jpg
  examples/sound/resources/canary.wav
  examples/sound/resources/orchestral.ogg
  examples/win32/resources
  examples/win32/resources/image1.jpg
  examples/win32/resources/image2.jpg

The Debian source package does not ship any of the xcode templates since
they also include non-free resources like the ones above.

@nabijaczleweli thanks. I can confirm that all those resources are replaced with public domain ones. (Including the XCode template ones)

SFMLBuildMaster: Build this please.

This should resolve all the size issues. I did fix up the markdown file, and left the few changes to the examples. They can be removed, or moved to a different PR. They are just some small things I thought could be improved as I went through the examples.
I would also ask you don't merge this PR quite yet as I found a few more assets that need to be resolved in tools/...

Ok, I think I got everything in place. It would be great if someone could test the XCode templates because I don't have access to a machine to try it on.

Can you please rebase your branch on master so it's easier to read? Right now the year changes in license make it hard to separate the actual content of this review from the fluff.

@eliasdaler in case you want to take another look 🙂

There seems to be some issue with the resources regarding the iOS build. Will try and look into it later.

After this patch is merged, we'll need to check that SFML docs on sfml-dev.org use non-free resources as well
It's not really critical or essential there, since we're not packaging the page somewhere and fair-use still has its place, but if anyone wants to invest the time, they're free to do so. 🙂


Style with new colors

Have tested both on android and iOS and looks good, at least its a base to work from with the new colors and fonts.


fix copyto vs. stageto GET parameter for COPY

First off: thanks for this awesome project!
Everything works beautifully, except for one small thing: The readme states to use the stageto GET parameter for both the COPY and STAGE method. The code in app.py#L120 only accepts copyto for COPY.
This adapts the README to match the implementation.

Coverage remained the same at 84.183% when pulling c2451c80a5bbbb9c5311af31bdf3235257f63338 on luto:patch-1 into 01350cfb9547910ad1c3f4854d345be628ee7991 on arnehilmann:master.

Thnx for proofreading :o)


Rename have_x to has_x to improve how logic reads

Description
Minor change to the function names so the logic reads:
if pybamm has something

instead of
if pybamm have something

Key checklist:

[x] No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[x] All tests pass: $ python run-tests.py --all (or $ nox -s tests)
[x] The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Still need to do the changelog


All size options are super wide

Even with size="small" the control is much wider than I would expect for typical single/double-digit number entry. Could we have more size options, or better yet a width property?

The provided sizes are for most of the users, for your case, please override the CSS styles for your expected size in your project.


added timestep range for activation

Hey there!
I'm submitting this PR so to add the possibility to decide when the perturbed attention will be active!
Which makes it look like this:

I have updated my own nodes and added the same type of sliders on a modified version of the SAG nodes so to be able to use the 3 of them at diverse stages of the generation.
The following image is a suggestion / test workflow:

So here they are connected like this:

The perturbed attention is up from 100% to 50% of the sigma (which is like 4-5 steps)
The guidance is up from 50% to 6.86% of the sigmas as well as the SAG
Then there is just the positive conditioning.
The idea is to be able to somehow take advantage of all of them at the best timing. This arrangement here is probably not the absolute best but it allows to get a bit of all while still going faster at the end (I get an average of 4it/s instead of 3).
Check out my repository to get the latest update if you want to try it! :)
Best regards,

On a side note it makes it possible to do this:

So to set the scale of your node at 100 since the node right after fights the over-blowing values.
Which I don't know yet if is really beneficial. I mean have tested and it works, I've got a prompt with a giraffe buying groceries which would never make me a giraffe up until I did this. But I would not swear my GPU on it.

Thanks for the contribution!
I'll merge it into a separate branch to test it and probably make adjustments related to forge, and then I'll merge it into main


Add new style

Closed previous PR (#65) so PR can be made from a branch instead of master.
New style based on davewhipp CSS.

Monochrome theme
Headings brought in line with content
Headings are heavier and capitalised, with H2's partly underlined
Basic responsiveness
Updated README to reference three themes

thank you for this style. hope it gets approved. it looks extra good :)


FIFA 14 WORLD CLASS SOCCER [PCSG00201]

App summary

App name: FIFA 14 WORLD CLASS SOCCER
App serial: PCSG00201
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 2854
Commit hash: https://github.com/vita3k/vita3k/commit/1ce445ce
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: nishinji
OS: Windows 10
CPU: Ryzen 9 3950x
GPU: RTX 3060ti
RAM: 32 GB

Issues
Screenshots

Log
vita3k.log
Recommended labels
nothing

J


fix(fetch() api): fix multiple fetch() call on same component

Closes #7107

Types of changes

[x ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Resolves: #7107. Just add a return on fetch already pending to prevent multiple calls

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

In addition, as suggested from @olddeda we can solve the issue also in this way
// file vue-app/template/utils.js
export function addLifecycleHook(vm, hook, fn) {
if (!vm.$options[hook]) {
vm.$options[hook] = []
}

vm.$options[hook].push(fn)

if (!vm.$options[hook].includes(fn)) {

vm.$options[hook].push(fn)

}
}

If you need this patch, I can make another pull request.

The hook option seems better indeed @asterd
Happy that you make another PR :)

Thanks, @asterd for all helps. I've added both proposed suggestions by #7129 (credits preserved of course :heart: )


Push notifications keep arriving after logout

Rocket.Chat.Android version: 1.0.14
Seems like I'm not the only one: https://github.com/RocketChat/Rocket.Chat.Android/issues/77#issuecomment-305454719

+1
Happens to me too

That's probably because we're not clearing the token into the server when user logouts. Can you confirm @leonardoaramaki, please?

I can confirm :)

@rodrigok @SeanPackham Can you guys update the Real Time API with logout method, so we can call it on the native apps when user tries to logout? I assume that this call should clear the push token.
Current docs is blank: https://rocket.chat/docs/developer-guides/realtime-api/method-calls/logout/#logout

["{\"msg\":\"method\",\"method\":\"logout\",\"params\":[],\"id\":\"16\"}"]

that's the raw query
So pretty much call method: logout with no params and it'll log you out

That call I don't think will remove your push token though.
Looking at the methods we have available... it looks like raix:push strangely does not have a way to remove push tokens... https://github.com/raix/push/blob/master/lib/server/server.js

We should check if the user is logged in on the Push Receiver on the App, we should also check if the push is for the current user, let's say the User logout from one username and logins with another, he should not receive (or at least show notifications) for the old username.

Yes on the Android side that would for sure work, long term we should also try and get the ability to remove a token too.

Yes, this is something we need to handle for both iOS and Android
@rodrigok Do we have a way to clear the device token from the server? On iOS, we do save the device identifier (https://github.com/RocketChat/Rocket.Chat.iOS/blob/ff09823ed7ab0672519cacc14913cc75b2b203f8/Rocket.Chat/Managers/PushManager.swift#L24), so in theory, we could remove this from the server with a simply API call.

I think we could definitely open a PR against raix:push to add a method to remove.  But currently there isn't a way.
https://github.com/raix/push/issues/298
https://github.com/raix/push/issues/61

@geekgonecrazy I think updating to an invalid token is OK. Maybe we set a standard value now, like unknown so in the future we can clear it?


switching to modules and yeet js

This has been a pr wip for 12 days. How close is it to being ready?


why HostConfig remove Deprecated

I use the start API  to add a  new volume to a container after it was created. just add a binds info to the HostConfig parameter. it test ok.
At https://docs.docker.com/engine/deprecated/ ,I find HostConfig in the API of start container  will be removed,is there any other method to add volume after container created? thanks

/cc @cpuguy83 @mlaventure perhaps something related to https://github.com/docker/docker/pull/22563?

Definitely should not rely on this, and certainly not if HostConfig is already set.

Not related to #22563, he is trying to add a new volume entry (i.e. -v equivalent) not a new device :).
Otherwise, updating the container configuration post create has never been explicitly supported AFAIK; that'd explain why HostConfig in start was deprecated I guess (can't say I remember when that happened though)

@cpuguy83  any other method to add a new volume to a created container?

@thaJeztah @mlaventure   I want  to add a new volume to a created container. Now ,i use the HostConfig parameterin the Start API.(just stop the running container,start the container with the binds parameter in HostConfig)
but i found the parameter will be removed.
does docker have the feature to add a new volume to a created container?

https://github.com/docker/docker/pull/22570   docker1.12 has removed this feature,so i closed this issue.


Please Publicize

Can you please publicize this and make it consumable?
If you do not, then nobody really can use it.  (except researchers who stumble on it like me)
It needs to either:

be merged with the main flash attention repo or
be given a unique name (perhaps FlexHeadFA or something) and added to pypi so that we can 'pip install flexheadfa'

This can be a big contribution to open source AI development, if you would make it accessible.
Thank you,
Eric Hartford

☝

@ehartford  Thank you for your suggestion, Eric!
We actually created a PR to the main FlashAttention repository several months ago, with the intention of merging it there. However, since the PR has been stalled for quite some time, giving it a unique name and publishing it to PyPI might indeed be a better approach.
We’ll work on this and make it accessible soon. Thank you for highlighting its potential contribution to the open-source AI community!

Can you please reference the PR to Flash attention?I will try to bring attention to it

Can you please reference the PR to Flash attention?I will try to bring attention to it
Is it this one? Dao-AILab#1166

Yes, and we've created a related issue to query about this yesterday. https://github.com/Dao-AILab/flash-attention/issues/1358

Thankfully he made an explicit decision

Thankfully he made an explicit decision
We’ll work on publishing it to PyPI then. :)

Hi @ehartford , We’ve just published it to PyPI! You can install it using: pip install flex-head-fa --no-build-isolation （We went with the name you suggested :)). Please note that this version is not yet synced with the latest main FlashAttention repository. We’ll work on updating it soon.
Thank you for your suggestion.

Thank you!  Looking forward to using it!


Installed jest and created a sample test #779

Setup jest library and created a sample test in /jest-test
resolves #779
How to verify:

npm install
npm test

Also configured it to work with husky


feat(diagnose): check if container is still starting

in which case, print a specific message

Codecov Report

Merging #76 (c215262) into main (ab80527) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
+ Coverage   78.19%   78.26%   +0.07%
==========================================
  Files          16       16
  Lines         908      911       +3
==========================================
+ Hits          710      713       +3
  Misses        144      144
  Partials       54       54

Impacted Files
Coverage Δ

pkg/diagnose/diagnose_pod.go
77.77% <100.00%> (+0.63%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Initial storybook setup

Add storybook to non-prod deployments of the editor similar (if not identical) to the projects-ui setup for consistency.
It'd also be worth considering adding:

[ ] documenting generic styles
[ ] setting up common components

There's no requirement on this ticket to storybook everything, just enough to get the ball rolling with a couple of examples so we can add new/updated components as and when.

host on /storybook


Bugfix: mapId: string -> mapId: number

Map is is actually a number but was incorrectly marked
Fixes #51

update package version


iterparse function need xml file to be parameter

The xml source data we get usually are just xml string, before passing it to etree.iterparse()   and generating the xml tree, we need to covert it into xml file type, or the function will throw IOError coz it can not find the xml source.

Oops - I switched to requests but didn't realise that this will stop the content from being a file-like object and turn it into a string.  This means most of the manager/query tests that return content as an StringIO object are actually invalid.
All the tests need to be updated to have api.content return a simple string, and then there is only one use case which is that _fragments gets a string and not a file-like.

See PR #13.


In-memory engine: RangeManger is designed too complicated to be implemented correctly

The complex of the RangeManager
The RangeManager handles multiple data ranges' cache status, it used 7 BtreeMap data structures to simulate different status of each range, and move them between different BTreeMap due to status changing. There is a global RWLock to lock this RangeManger. This design makes it difficult to implement correctly since there are many threads may concurrently handle range caches, and this design mess up status changing, ranges management and concurrent control together. We should use a divide and solve design to simplify it.
pub struct RangeManager {
    // Each new range will increment it by one.
    id_allocator: IdAllocator,
    // Range before an eviction. It is recorded due to some undropped snapshot, which block the
    // evicted range deleting the relevant data.
    historical_ranges: BTreeMap<CacheRange, RangeMeta>,
    // `ranges_being_deleted` contains ranges that are evicted but not finished the delete (or even
    // not start to delete due to ongoing snapshot)
    pub(crate) ranges_being_deleted: BTreeSet<CacheRange>,
    // ranges that are cached now
    ranges: BTreeMap<CacheRange, RangeMeta>,

    // `pending_ranges` contains ranges that will be loaded into the memory engine. To guarantee
    // the completeness of the data, we also need to write the data that is applied after the
    // snapshot is acquired. And to ensure the data is written by order, we should cache the data
    // that is applied after the snapshot acquired and only consume them when snapshot load
    // finishes.
    // So, at sometime in the apply thread, the pending ranges, coupled with rocksdb
    // snapshot, will be poped and pushed into `pending_ranges_loading_data` (data here means the
    // data in snapshot and in further applied write). Then the data in the snapshot of the
    // given ranges will be loaded in the memory engine in the background worker. When the
    // snapshot load is finished, we begin to consume the write batch that is cached after the
    // snapshot is acquired.
    //
    // Note: as we will release lock during the consuming of the cached write batch, there could be
    // further write batch being cached. We must ensure the cached write batch is empty at the time
    // the range becoming accessable range.
    //
    // Note: the region with range equaling to the range in the `pending_range` may have been
    // split. This is fine, we just let the first child region that calls the prepare_for_apply
    // to schedule it. We should cache writes for all child regions, and the load task
    // completes as long as the snapshot has been loaded and the cached write batches for this
    // super range have all been consumed.
    pub(crate) pending_ranges: Vec<CacheRange>,
    // The bool indicates the loading is canceled due to memory capcity issue
    pub(crate) pending_ranges_loading_data: VecDeque<(CacheRange, Arc<RocksSnapshot>, bool)>,

    ranges_in_gc: BTreeSet<CacheRange>,
    // Record the ranges that are being written.
    //
    // It is used to avoid the conccurency issue between delete range and write to memory: after
    // the range is evicted or failed to load, the range is recorded in `ranges_being_deleted`
    // which means no further write of it is allowed, and a DeleteRange task of the range will be
    // scheduled to cleanup the dirty data. However, it is possible that the apply thread is
    // writting data for this range. Therefore, we have to delay the DeleteRange task until the
    // range leaves the `ranges_being_written`.
    //
    // The key in this map is the id of the write batch, and the value is a collection
    // the ranges of this batch. So, when the write batch is consumed by the in-memory engine,
    // all ranges of it are cleared from `ranges_being_written`.
    ranges_being_written: HashMap<u64, Vec<CacheRange>>,
    range_evictions: AtomicU64,
}

Solution
We should simplify the data ranges management of RangeManger with a divide and solve strategy:

There is just one BTreeMap to store data ranges.
There is a global RWLock to protect the data ranges, only acquire write lock when adding new ranges or removing old ranges. Retrieve the CachedRange should acquire the read lock first and release lock ASA return the CachedRange structure from the BTreeMap, all following operations' ordering and correctness is guaranteed by CachedRange's atomic variable. This can guarantee the performance for these hot code paths.
Each range is an individual state machine, it should have an atomic variable to represent the status of this range.

pub struct CachedRange {
    // region: Region
    state: AtomicU32,
}

pub struct RangeManager {
    lock: RWLock,
    cached_ranges: BTreeMap<CachedRange, ...>,
}

closed by #17331


Rate limit control plane ingress

Why
Should rate limit control-plane ingress to protect both the control-plane itself and the vsock conn to enclave
How
Use semaphores to restrict number of connections/minute
Feature flagged the rate limiting, printing quantity of requests/minute if not enabled.

Single e2e test failing in ci but not locally, haven't traced it back to these changes yet..


Is there a setup guide for using this plugin with L3MON4D3/LuaSnip?

The README says it should be compatible but I don't see any setup instructions.

Sorry about that, you're right, there should be some setup example. Here the link where they explain how to use VsCode snippets in with LuaSnip that https://github.com/L3MON4D3/LuaSnip/blob/72323c10fe2d91695f7593e572496ab9f004afee/Examples/snippets.lua#L263-L274

I copied the last line there
require("luasnip/loaders/from_vscode").lazy_load() -- You can pass { path = "./my-snippets/"} as well
But see an error in luasnip when I navigate to the snipped in the list that shows up.
Error executing vim.schedule lua callback: .../pack/packer/start/nvim-compe/lua/compe_luasnip/init.lua:27: attempt to concatenate a table value

Oh yeah that's a problem with the compe source for luasnip, should get fixed by hrsh7th/nvim-compe#457

Thanks, you can close the ticket. Will follow that issue.

I still can't select the snippet

I am not sure how to confirm the selection. Ctrl J, Y or any other key doesn't work. I tried arrow keys, shift, tab, they only complete the keyword, not the snippet associated with it.
Also, it's not clear what is being shown in the floating window. Why is there a [cpp] and what is the double line underneath?

You'll most likely have to bind compes' confirm, see :h compe. As for the [cpp], that's the current doc-window for the luasnip source in compe (not very pretty, I know :sweat_smile:).

Thanks @L3MON4D3, that worked, I was under the impressions there'd be some defaults. Do you know, within a snippet, how to jump to different sections? In Vim and earlier plugins, C-J would take me to the 4 different sections in a for snippet.

Luasnip doesn't provide defaults, but you can see what to set here


Audio captcha is insecure

Someone created a tool to automatically solve google audio recaptcha's using voice recognation services. (https://github.com/eastee/rebreakcaptcha)
Could you fix this or implement an option to disable audio captcha's?

Closing super old issues. Please re-raise if still relevant.


Upgrade whereabouts CNI to v0.5.4 and align secondary network IPAM with additional pluginArgs

Register Whereabouts CNI as a valid IPAMType for secondary network configuration. Whereabouts is the IPAM used to configure secondary/virtual networks for Service function chaining needs at this time.
Signed-off-by<EMAIL_ADDRESS>

Codecov Report

Merging #3987 (546489e) into main (2f9134b) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3987      +/-   ##
==========================================
- Coverage   64.58%   64.56%   -0.03%
==========================================
  Files         295      295
  Lines       43740    43756      +16
==========================================
- Hits        28251    28249       -2
- Misses      13215    13231      +16
- Partials     2274     2276       +2

Flag
Coverage Δ

*Carryforward flag

kind-e2e-tests
51.15% <ø> (ø)

Carriedforward from 2f9134b

unit-tests
44.31% <100.00%> (-0.01%)
:arrow_down:

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/cniserver/ipam/ipam_delegator.go
46.83% <ø> (-7.72%)
:arrow_down:

pkg/agent/cniserver/ipam/antrea_ipam.go
74.35% <100.00%> (-1.40%)
:arrow_down:

...r/ipseccertificate/ipsec_certificate_controller.go
61.56% <0.00%> (-1.96%)
:arrow_down:

pkg/ipam/poolallocator/allocator.go
53.49% <0.00%> (-0.97%)
:arrow_down:

...catesigningrequest/ipsec_csr_signing_controller.go
64.11% <0.00%> (+2.45%)
:arrow_up:

/test-e2e

/test-all

/test-all

/test-all


contrib/boosted_trees: minor spelling tweaks

tensorflow/contrib/boosted_trees/python/kernel_tests/prediction_ops_test.py:123: [C0301(line-too-long), ] Line too long (81/80)

@frankchn rebased to match master/HEAD again.  can merge this into #17815 if that would make things simpler.  I would normally rather do a lot of little commits (one for each contrib path) rather than touching a bunch of files like that PR is doing, do you have a preferred style?

@brettkoonce Not really, we have a rotation of who handles PRs such as this, so everyone's preferences would be different. Just do whatever you prefer :)

Unrelated test failure. Merging.


Kakao Maps SDK is not available

Issue:  When trying to show the string based location value to turn into geocode location and map it in kakaomap, i am getting the following console errors issues:

KakaoMap.tsx:36 Failed to load Kakao Maps script.

KakaoMap.tsx:53 Kakao Maps SDK is not available.

[x] Fix kakao map location showing based on different location name labeled on map.

Note: I have  enabled my project domain as a locahost  inside the:

My Application >  App Settings > Platform
-  Site domain:  http://localhost:5173

Reference:

Kakao Map Initialization Code:  c0219a3

Fixed the issue:
commit : b745e1b


Google Play Services wont run unless you update...

Hey guys,
just bought a HtC OnePlusOne Android phone running 6.0.1 solely for CarStream.  I've rooted the handset, installed AA 3.5 (also tried 3.0 and 2.9) and installed GPS 12.6.
If I launch AA, an error pops up
"Something Went Wrong
Google Play Services doesnt seem to be working right now."
On my lock screen there is a notification:
"Update Google Play Services
Google Play Services wont run unless you Update"
If I update GPS to latest 12.8, AA works, but that defies the purpose because CarStream player doesnt work on 12.8.
any help would be appreciated!

Read all the comments under the issue number 211. Step by step instructions are there.

Read all the comments under the issue number 211. Step by step instructions are there.

Did all 20 steps that you wrote in number 211 but when I'm in step 12 starts to appear a notification that says: "Update Google Play Services... Google Play Services wont run unless you Update". I ignore it and install & open AA 3.5 but when I try to open says: "Something Went Wrong... Google Play Services doesnt seem to be working right now." I've already check if GPS is updated but nope... I have installed 12.6.88 GPS... please help! :(

Same here. I installed GPS 12.6.88 and AA 3.5. I also use the disable service apk app to block GPS from updating.  But AA refused to open instead it pop up the notification saying "something went wrong Google play service doesn't seem to be working right now" . Does anyone know how this can be fixed?

USing GPS 12.6.85 and AA 3.2.58 for long time on op3t. Since yesterday AA refuses to load unless i update some google apps including AA.i havent updated to the latest oos as that auto updates gps. anyone else experiencing this? any suggestions or combintions of software versions still working? thx..


Form (Preview): client-side defaults

Describe the feature in detail (code, mocks, or screenshots encouraged)
Loading from server side can be really slow and hinder UX — it would be great if we can pass new default values to the form from the client side after we load data with onMount. Is this a feature you would consider adding?
What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response

This is probably more relevant on the Formsnap docs. Closing for now


How can we recreate application gateway?

If application gateway cannot be moved then how are we suppose to recreate an application gateway that contains 50+ http listner, 100+ Rules, and 20+ backend pools, 20 SSL Certificates??
Is there any way to copy the template of an existing application gateway and recreate it into another subscription??

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

ID: 76c1c185-685f-27e0-e634-844b510b1834
Version Independent ID: 09375778-5354-ab44-c5c6-94c2aebe34bc
Content: Move Azure resources to new subscription or resource group
Content Source: articles/azure-resource-manager/resource-group-move-resources.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@Anmolgan81 Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

We need a solution for this as soon as possible, one of our customer has a 6 Application Gateway and he cant afford a downtime of 1 Hour, what is the best way to recreate the application gateway with the same rules and certificate as fast as possible?
Or if Moving the application gateway is going to be enabled in few days??

@Anmolgan81 - you can export a template for a resource group. You can get either the actual templates used for the deployments or a snapshot of the current state. However, the exported template usually requires some customizing before it can be redeployed to a new resource group. And, not all resources can be exported to template. Exporting is available through PowerShell, Azure CLI or the portal. Here is the Powershell doc.
Is the customer trying to move these resources to a new subscription or a new account offering? The move operation only works within the same account. If you are actually trying to migrate to a different account offering, see Change your Azure Pay-As-You-Go subscription to a different offer or Migrate Azure subscriptions from pay-as-you-go to Azure CSP.

Are you talking about automation script of the resource group??
Am afraid that cannot be possible as currently the customer has 200 plus resources inside 1 resource groups hence I cannot get the automation script for that resource group.
How can I get the snapshot of the current state.
Can you give me some examples as how can edit a template for Application Gateway?
Or do you have some links that deploys an application gateway with custom rules and certificates that I can enter at the time of provisioning so that the application gateway come up online with all of its SSL certificate and listenrs and all the required backend pool as well?

Hi @tfitzmac I cant use the Export powershell command as I have more then 200 Resources inside one of the resource group.
Can you suggest me some other method of exporting the template, modifying and then creating app gateway??

@Anmolgan81 - the deployment history has the templates that were used to create the resources. You might be able to construct a new template by assembling several templates from the history.

Otherwise, the quickstart gallery might have some examples like -gateway with public ip

I tried that but wasnt able to succedd
Get Outlook for iOShttps://aka.ms/o0ukef

From: Tom FitzMacken<EMAIL_ADDRESS>Sent: Saturday, May 5, 2018 6:37:38 AM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How can we recreate application gateway? (#8037)
Otherwise, the quickstart gallery might have some examples like -gateway with public iphttps://github.com/Azure/azure-quickstart-templates/blob/master/101-application-gateway-public-ip/azuredeploy.json
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/8037#issuecomment-386768394, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKpxsjsmLF88pZKeVpLxkuP3ENru_dks5tvPtZgaJpZM4Twwxy.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Technologies: www.ifi.tech

@Anmolgan81 - are you still looking for a solution to this issue? If so, I think the best option is to work with support.

#please-close

Yeah am still looking for some answers.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Tom FitzMacken<EMAIL_ADDRESS>Sent: Tuesday, June 5, 2018 10:50:46 PM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How can we recreate application gateway? (#8037)
@Anmolgan81https://github.com/Anmolgan81 - are you still looking for a solution to this issue? If so, I think the best option is to work with support.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/8037#issuecomment-394791597, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKp4dKZSGb4YFCVTSmNFXAWS0T8GNHks5t5r3ugaJpZM4Twwxy.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Technologies: www.ifi.tech

Either open a support case or send an email to<EMAIL_ADDRESS>Include the subscription ID and link to this issue.

#please-close

I have done that long ago but no response from them on this
Get Outlook for iOShttps://aka.ms/o0ukef

From: Tom FitzMacken<EMAIL_ADDRESS>Sent: Friday, June 8, 2018 10:28:29 PM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How can we recreate application gateway? (#8037)
Either open a support casehttps://docs.microsoft.com/en-us/azure/azure-supportability/how-to-create-azure-support-request or send an email to AzCommunity<EMAIL_ADDRESS>Include the subscription ID and link to this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/8037#issuecomment-395822701, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKp7TkDUFGDulXxcS5L8v4-vW7RBVYks5t6q01gaJpZM4Twwxy.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Technologies: www.ifi.tech

The values you mentioned in your original post are all arrays on the Application Gateway resource. Create a template for the gateway. You can view the current values by using the REST API. Take those values and add them to your template.


Refactored bcounter_mgr and corresponding tests

Probably needs review first

Transfers were To and From are equal (called self transfers) are always allowed and translated to an increment
The CRDT antidote_crdt_counter_b prohibits self transfers if no local increment exists and if one exists then self transfers are essentially equal to increments
The new version is a bit more restrictive on transfers and allows only transfers that go to an existing DC (a descriptor exists)
(Like the previous version) Increments and decrements are only allowed locally and the target DC that is defined in the update operation is ignored

Le 15 sept. 2020 à 16h56, Kevin Bartik<EMAIL_ADDRESS>a écrit :
(Like the previous version) Increments and decrements are only allowed locally and the target DC that is defined in the update operation is ignored

Wouldn’t it be better to raise an error if target DC ≠ local DC?
				Marc

I can add a check that returns {error, no_permissions} if an update operation contains an invalid DC
It forces the client to make the right call which has advantages (always intended behavior) and disadvantages (client must find out the DCID when they want to perform an operation on a bcounter)

Best of both worlds:
by default the system fills in the correct DCID when none is provided
When one is provided, check that it is correct
				Marc

Le 15 sept. 2020 à 17h04, Kevin Bartik<EMAIL_ADDRESS>a écrit :
I can add a check that returns {error, no_permissions} if an update operation contains an invalid DC
It forces the client to make the right call which has advantages (always intended behavior) and disadvantages (client must find out the DCID when they want to perform an operation on a bcounter)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/AntidoteDB/antidote/pull/439#issuecomment-692776616, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABS23H3NY6PSBI5SROS2IR3SF57BDANCNFSM4RNH37ZA.

I am still a bit confused why a local transfer would result in an increment. What is the semantic reasoning behind this?

I am still a bit confused why a local transfer would result in an increment. What is the semantic reasoning behind this?

This can be seen by the internal implementation of the antidote_crdt_counter_b
-spec increment(id(), pos_integer(), antidote_crdt_counter_b()) -> {ok, antidote_crdt_counter_b()}. increment(Id, V, {P, D}) ->     {ok, {orddict:update_counter({Id, Id}, V, P), D}}.
-spec transfer(id(), id(), pos_integer(), antidote_crdt_counter_b()) -> {ok, antidote_crdt_counter_b()}. transfer(FromId, ToId, V, {P, D}) ->     {ok, {orddict:update_counter({FromId, ToId}, V, P), D}}.
In case of a transfer the available permissions are always checked beforehand and if not enough permissions exist then the operation fails with {error, no_permissions}
The behavior with self transfer is inconsistent because it will add permissions (like increment) only if enough permissions exist in the first place.
In a normal transfer operation no permissions are added, they are only shifted. The self transfer (like the increment) actually increases the globally available permissions.

Best of both worlds: by default the system fills in the correct DCID when none is provided When one is provided, check that it is correct Marc

I would like to do this but this requires further changes (probably in the antidote_crdt repo) because antidote checks the update operation that is provided by the client. Only operations that pass this check actually arrive at the bcounter_mgr. Internally the antidote_crdt_counter_b requires an id() (we always use a DCID) and the is_operation() check requires that such an id() is provided.
Overall the client must provide something for the id() field and currently, it is ignored and always replaced by the local DCID (in case of transfers the FromID is always the local DCID, and the ToID is defined by the client (it is checked and only a valid DCID will pass))

@marc-shapiro
I will create a solution that fits your requirements (full support will require changes to antidote_crdt_counter_b => allow operations without an id() but that can be done later)
For now, the client can set the DCID to undefined which will use the local DCID
Otherwise, if the client uses an invalid DCID an error will be returned

@bieniusa @marc-shapiro
I will create another pull request that resolves the conflicts with master and runs the CI


readdirSync not showing the correct results after file changes

The first time I do fs.readdirSync it gives me the correct result. After I delete one file and use chokidar to trigger this unlink event fs.readdirSync still gives me the same results.  Is this the correct behavior?  If yes, is there anyway I can get the updated list of files?  Thanks.

Could you provide an example JS program that also prints messages before and after each file operation, in order to easily check order of execution?
If the issue only applies in a program that has been transformed by brfs, please also provide the tramsformed source. Since this issue is posted in brfs, have you verified that it's actually fs.readdirSync that "still gives […] the same results"? Hasn't it been transformed to a constant representing the file system state at transform time?


Flaky test: TestRunWithJsonFileLogDriverAndLogPathOpt

=== RUN   TestRunWithJsonFileLogDriverAndLogPathOpt
    run_test.go:287: assertion failed: error is not nil: open /tmp/TestRunWithJsonFileLogDriverAndLogPathOpt348967069/001/nerdctl-testrunwithjsonfilelogdriverandlogpathopt/nerdctl-testrunwithjsonfilelogdriverandlogpathopt-json.log: no such file or directory
--- FAIL: TestRunWithJsonFileLogDriverAndLogPathOpt (3.38s)

@AkihiroSuda  this never occurs in my local :/ . Unit and integration tests log-rotate seems to be stable. I am using GH CI to reproduce the error (which is random) so this will take more than expected time. Sorry for that.

https://github.com/containerd/nerdctl/pull/1650


⚠ Bofrost.lu has degraded performance

In 26b02e9, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 837 ms

Resolved: Bofrost.lu performance has improved in 7b7e789.


Quickstart pretrain error

I use pytorch2 and latest code.
$ composer train/train.py \

train/yamls/pretrain/mpt-125m.yaml
data_local=my-copy-c4
train_loader.dataset.split=train_small
eval_loader.dataset.split=val_small
max_duration=10ba
eval_interval=0
save_folder=mpt-125m

Initializing model...
cfg.n_params=1.25e+08
Building train loader...
Building eval loader...
Building trainer...
Traceback (most recent call last):
...
RuntimeError: Cannot writeback when the parameter shape changes
Expects torch.Size([1769472]) but got torch.Size([2304, 768])
ERROR:composer.cli.launcher:Rank 0 crashed with exit code 1.

Hi @semsevens and @DayOfThePenguin , LLM Foundry and MPT were all built with torch 1.13 and that's still the recommended install version (and the version in our Docker image mosaicml/pytorch:1.13.1_cu117-python3.10-ubuntu20.04)
We are actively working on torch2 support (and you can see some active PRs :) but it we are still a week or two away from a stable release tag with torch2 support.
@DayOfThePenguin could you clarify the system on which you had to use torch >=2 ? That is very surprising to me and I would love to know more.

Note: I've only gotten this error when using a single GPU with FSDP.
If you are using one GPU, you shouldn't be using FSDP.

Closing this issue for now, but @DayOfThePenguin if you can send a repro for this issue we would very much appreciate it!

I can only run the released MPT models with torch >= 2 because of a similar low-level API difference in the matmul call signature.

I was only able to reproduce with the old version of the Triton attention implementation (before it was updated ~10 days ago).  With the changes, I'm able to run all models with torch 1.13.1!


Pairs with difference k - POTD | 30-Oct-2024

Pairs with difference k
Link
Difficulty: Easy
Given an array arr[] of positive integers. Find the number of pairs of integers whose difference equals a given number k.
Note: (a, b) and (b, a) are considered the same. Also, the same numbers at different indices are considered different.
Examples:
Input: arr[] = [1, 5, 3, 4, 2], k = 3
Output: 2
Explanation: There are 2 pairs with difference 3,the pairs are {1, 4} and {5, 2}
Input: arr[] = [8, 12, 16, 4, 0, 20], k = 4
Output: 5
Explanation: There are 5 pairs with difference 4, the pairs are {0, 4}, {4, 8}, {8, 12}, {12, 16} and {16, 20}.

Constraints:

1 <= arr.size() <= 106
1 <= k <= 106
0 <= arr[i] <= 106

Solved
def countPairsWithDiffK(self,arr, k):
    i,j,n=0,1,len(arr)
    arr.sort()
    ans=0
    while j<n:
        diff=arr[j]-arr[i]
        if diff==k:
            countI,countJ=1,1
            while i<n-1 and arr[i]==arr[i+1]:
                countI+=1
                i+=1
            while j<n-1 and arr[j]==arr[j+1]:
                countJ+=1
                j+=1
            ans+=(countI*countJ)
            i+=1
            j+=1
        elif diff<k:
            j+=1
        else:
            i+=1
    return ans


logoutPopup parameter type incorrect.

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
3.1.0
Wrapper Library
Not Applicable
Wrapper Library Version
None
Public or Confidential Client?
Public
Description
Parameter type for function PublicCLientApplication.logoutPopup is incorrect. Its currently defined as EndSessionRequest  but should be defined as EndSessionPopupRequest. The incorrect parameter type prevents mainWindowRedirectUri and popupWindowAttributes from being specified.
Note: StandardController.logoutPopup is defined with EndSessionPopupRequest:
Error in VSCode

Error Message
Argument of type '{ mainWindowRedirectUri: string; }' is not assignable to parameter of type 'EndSessionRequest'.
Object literal may only specify known properties, and 'mainWindowRedirectUri' does not exist in type 'EndSessionRequest'.ts(2345)
Msal Logs
No response
MSAL Configuration
{
  // required
  auth: {
    // must match info in dashboard
    clientId: import.meta.env.VITE_AUTH_CLIENT_ID,
    authority: import.meta.env.VITE_AUTH_AUTHORITY,
    knownAuthorities: [
      import.meta.env.VITE_AUTH_AUTHORITY,
    ],

    // login redirect; must match path in dashboard
    redirectUri: `${location.origin}/auth/redirect/login`,
  },

  // optional
  system: {
    loggerOptions: {
      logLevel: LogLevel.Verbose,
      loggerCallback,
    }
  }
}

Relevant Code Snippets
async logout () {
    // ref: https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-browser/docs/logout.md#logoutpopup
    return msal.logoutPopup({

      // required to make the application return to the home page
      mainWindowRedirectUri: '/'

    })
  },

Reproduction Steps

Clone repo https://github.com/davestewart/msal-vue-demo
upgrade msal-browser
auth.ts will produce an error

Expected Behavior
For properties mainWindowRedirectUri and popupWindowAttributes to assignable when calling logoutPopup
Identity Provider
Azure B2C Basic Policy
Browsers Affected (Select all that apply)
Chrome, Firefox, Edge, Safari, Internet Explorer
Regression
<EMAIL_ADDRESS>Source
External (Customer)

Thanks, marking this as a bug. Please feel free to open a pull request with the change if you need this fixed sooner rather than later.

If you're stuck on this in the meantime this works:
import { type EndSessionPopupRequest } from "@azure/msal-browser"

 const logoutRequest: EndSessionPopupRequest = {
    mainWindowRedirectUri: "/",
  }
  instance.logoutPopup(logoutRequest)


[BUG]:  project code requires adding  modules "OLinkProtocolLibrary", "ApiGearOLink" dependencies when using Olink from code

Description of the bug
project code requires adding  modules "OLinkProtocolLibrary", "ApiGearOLink" dependencies when using Olink from code.
the ApiGear and YourInterfaceApi should be enough, they should also provide  "OLinkProtocolLibrary", "ApiGearOLink" modules as public
Steps To Reproduce
create the project in which you're using olink client class
get e.g. your interface client subsystem
try to run application
there are linking errors for message converter and node base
Additional Information
No response

Issue still exists:
with this line PublicDependencyModuleNames.AddRange(new string[] { "Core", "CoreUObject", "Engine", "InputCore", "ApiGear", "Api" })
I get an error:
Cannot open include file: 'olink/clientnode.h': No such file or directory  in a generated file ApiTestApi0OLinkClient.h
with this line all works
WithPublicDependencyModuleNames.AddRange(new string[] { "Core", "CoreUObject", "Engine", "InputCore", "ApiGear", "OLinkProtocolLibrary", "ApiGearOLink", "Api" });


[ML] fix: Add function call that was removed accidentally

Description
This PR re-adds a function call that was accidentally removed in #31553.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Error with ffmpeg config for batch inference

Hi, @Rudrabha @prajwalkr thanks for the amazing code and congrats on your paper!!
I am trying to run inference but I get the following error message with ffmpeg:
[buffer @ 0x6f2b80] Error setting option pix_fmt to value -1.
[graph 0 input from stream 1:0 @ 0x6f2e20] Error applying options to the filter.
Error opening filters!
If there some config requirement that I am missing?
Looking forward to your response.

Thanks for trying the code out! Could you please copy paste the exact command which you had run?
Also, I am assuming the error occurs in line 223:
command = 'ffmpeg -i {} -i {} -strict -2 {}'.format(args.audio,                        path.join(args.results_dir, 'result.avi'), path.join(args.results_dir, 'result_voice.mp4'))

Yes, the error occurs in line 223.
The command I had run:
 CUDA_VISIBLE_DEVICES=1 python batch_inference.py --checkpoint_path logs/lipgan_best_residual.h5 --face demo_images/test_sample2.jpg  --audio preprocess/temp.wav --results_dir ./results/ --fps 25

What is your FFMPEG version?

FFmpeg version 2.8.15-0ubuntu<IP_ADDRESS>

Please paste the full output log here. I am guessing it is an issue with the audio/video file or a version issue. Hard to say without the full log.

Error Summary:
Could not find codec parameters for stream 0 (Video: mpeg4 (DIVX / 0x58564944), none, 256x256): unspecified pixel format
[buffer @ 0x1622a00] Unable to parse option value "-1" as pixel format
[buffer @ 0x1622a00] Error setting option pix_fmt to value -1.
[graph 0 input from stream 1:0 @ 0x1622440] Error applying options to the filter.
Error opening filters!
Full Logs

Predictions shape:  (169, 96, 96, 3)
100%|###########################################################################################################################################################################################################| 1/1 [00:11<00:00, 11.90s/it]
ffmpeg version 2.8.15-0ubuntu<IP_ADDRESS> Copyright (c) 2000-2018 the FFmpeg developers
built with gcc 5.4.0 (Ubuntu 5.4.0-6ubuntu1~16.04.10) 20160609
configuration: --prefix=/usr --extra-version=0ubuntu<IP_ADDRESS> --build-suffix=-ffmpeg --toolchain=hardened --libdir=/usr/lib/x86_64-linux-gnu --incdir=/usr/include/x86_64-linux-gnu --cc=cc --cxx=g++ --enable-gpl --enable-shared --disable-stripping --disable-decoder=libopenjpeg --disable-decoder=libschroedinger --enable-avresample --enable-avisynth --enable-gnutls --enable-ladspa --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libcdio --enable-libflite --enable-libfontconfig --enable-libfreetype --enable-libfribidi --enable-libgme --enable-libgsm --enable-libmodplug --enable-libmp3lame --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-librtmp --enable-libschroedinger --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libtwolame --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx265 --enable-libxvid --enable-libzvbi --enable-openal --enable-opengl --enable-x11grab --enable-libdc1394 --enable-libiec61883 --enable-libzmq --enable-frei0r --enable-libx264 --enable-libopencv
libavutil      54. 31.100 / 54. 31.100
libavcodec     56. 60.100 / 56. 60.100
libavformat    56. 40.101 / 56. 40.101
libavdevice    56.  4.100 / 56.  4.100
libavfilter     5. 40.101 /  5. 40.101
libavresample   2.  1.  0 /  2.  1.  0
libswscale      3.  1.101 /  3.  1.101
libswresample   1.  2.101 /  1.  2.101
libpostproc    53.  3.100 / 53.  3.100
Guessed Channel Layout for  Input Stream #0.0 : mono
Input #0, wav, from 'preprocess/temp.wav':
Metadata:
encoder         : Lavf56.40.101
Duration: 00:00:07.12, bitrate: 256 kb/s
Stream #0:0: Audio: pcm_s16le ([1][0][0][0] / 0x0001), 16000 Hz, 1 channels, s16, 256 kb/s
[avi @ 0x1619460] Could not find codec parameters for stream 0 (Video: mpeg4 (DIVX / 0x58564944), none, 256x256): unspecified pixel format
Consider increasing the value for the 'analyzeduration' and 'probesize' options
./results/result.avi: could not find codec parameters
Input #1, avi, from './results/result.avi':
Metadata:
encoder         : Lavf58.17.101
Duration: N/A, start: 0.000000, bitrate: N/A
Stream #1:0: Video: mpeg4 (DIVX / 0x58564944), none, 256x256, 25 fps, 25 tbr, 25 tbn, 25 tbc
File './results/result_voice.mp4' already exists. Overwrite ? [y/N] y
[buffer @ 0x1622a00] Unable to parse option value "-1" as pixel format
Last message repeated 1 times
[buffer @ 0x1622a00] Error setting option pix_fmt to value -1.
[graph 0 input from stream 1:0 @ 0x1622440] Error applying options to the filter.
Error opening filters!

Please share the generated result.avi here. Also, see if the error gets resolved if you change result.avi in line 223 to result.mp4.

Please Find Attached
The generated result.avi looks fine though.
result.avi.zip

Does the error get resolved if you change the avi to mp4?

Please Find Attached
The generated result.avi looks fine though.
result.avi.zip

Please also send the wav file so that I can run the ffmpeg command on my system and check

No, the issue does not get resolved on changing .avi to .mp4
Please find attached the wav file
temp.wav.zip

I was able to merge with FFmpeg 3.4.6 using the exact same command. Merged file attached below. Please upgrade your FFMPEG version.
result_voice.zip

Thanks!! I'll try it.


Implement Contact Consent in Sign-Up Process

We want to be able to add community members to the Newsletter Mailinglist by default. This legally requires admission of consent during sign-up.

Update: We should add checkboxes for newsletter and for privacy policy


YOLOv4-tiny is released

@hhk7734 Hi,
Nice work with pypi: https://pypi.org/project/yolov4/
Did you manage to realize CIoU-loss and Mosaic?
Also YOLOv4-tiny is released:

source code: https://github.com/AlexeyAB/darknet
cfg: https://raw.githubusercontent.com/AlexeyAB/darknet/master/cfg/yolov4-tiny.cfg
weights: https://github.com/AlexeyAB/darknet/releases/download/darknet_yolo_v4_pre/yolov4-tiny.weights
discussion: https://github.com/AlexeyAB/darknet/issues/6067

There is required only 1 feature:

Add groups= and group_id= to the [route] layer.

[route]
layers=-1
groups=2
group_id=1

So if input is WxHxC, it divides input into 2 groups WxHx(C/2) (there are 2 groups: 0 and 1), and loads the 2nd group_1 WxHx(C/2).
If there are many layers specified in layers= parameter, then this will be done for each of the input layers specified in layer=, then results will be concatenated across channels.

@AlexeyAB  Hey~! Hi :sunglasses:
hunglc007/tensorflow-yolov4-tflite has some problems with training.
After solving this problem, I will manage BoF functions.
I will try to add YOLOv4-tiny when the above works are finished. Thanks for letting me know.

@hhk7734 Hi,

hunglc007/tensorflow-yolov4-tflite has some problems with training.

Did you solve this issue?

Also how many FPS did you get for YOLO4-tiny on Coral board(TPU)? https://github.com/hhk7734/tensorflow-yolov4#objective

hunglc007 solved the problem, so I applied the solution.
Failed to compile the whole. Some sequence or operations are not compiled. The part that failed compilation was replaced by numpy. So, TPU consumes 55 ~ 60ms, the others are 35 ~ 40ms. FPS is 10~11.

CIoU

utility: train: refactor bbox_ciou() 3a3d6522b5d59
utility: train: move to tf.train 1bcce65e28156301
tf: train: remove tf.math.divide_no_nan db12202497a091

Tiny

model: yolov4: implement YOLOv4Tiny 65dcfdb33206c8710

Mosaic

tf: dataset: implement mosaic  5c8cd15d1ccf66c0d

Requests were reflected. :)

@hhk7734 Great!
Do you know the reason? https://github.com/hunglc007/tensorflow-yolov4-tflite/issues/165

I studied artificial intelligence since May. So there are many things that I do not know.

@hhk7734 Very interesting to see v4 tiny on Edge TPU. I have two questions

What ops where not mapped to the TPU?
Did you quantize (post-training/training-aware) to INT8?

Thanks


Add xlnt port

xlnt is a C++ library I wrote for reading and writing XLSX files. I would like to add support for installing xlnt using vcpkg with this pull request. Please let me know if there's anything I need to do to improve this port. (Closes tfussell/xlnt#85).

Hi @tfussell, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

I think it would be best if dependencies of your library would be provided by vcpkg. Unfortunately we don't have ports of libstudxml and botan, so IMHO they should be added first.
It would be also nice if your port automatically inserted XLNT_STATIC macro to your headers after build.
As a sidenote not related to port: I noticed that constructor for  document_security class isn't defined anywhere - shouldn't it be defaulted?

Thanks for the feedback @codicodi. It would be nice to use vcpkg for all of the dependencies, but I'm targeting Windows, OSX, and Linux so I think I'll have to stick with Git submodules for the time being. I'm not sure how other vcpkg ports have dealt with this.
You're right about the XLNT_STATIC definition. I'll see if I can get that working.
As for document_security, defaulting the constructor is a good idea. Xlnt does have some unused methods missing definitions since my goal was to implement the core functionality and then let the community provide any advanced fucntionality they might need. The spec is just too big for me to get everything done on version 1.0.

We do need to use system dependencies when they are available, to enable

Upgrading/security patching
Static linking (to avoid symbol conflicts with the system copies)
Sane debugging (#include <expat> does the same thing everywhere)

For now, this means cryptopp, expat, zlib, and utfcpp.
Looking at [1], I think the least intrusive way to do this would be to add a non-default cmake macro that would disable the xlnt.third-party target and instead run a series of find_library calls that grab the appropriate prebuilt versions and assemble them into a cmake pseudo-target.
[1] https://github.com/tfussell/xlnt/blob/master/third-party/CMakeLists.txt

Thanks for taking a look at my port @ras0219-msft. It does make sense to use system libraries when possible. I was simply trying to reduce friction when using my library by building all of the third-party libraries directly. I've nearly sorted out how to use find_package to locate vcpkg libraries. I'll get the port modified and update the pull request accordingly soon.

@tfussell Any progress here?

I'm working hard to get this working. This cross-platform support is turning out to be complicated. I hope to have an update for you by the weekend. Sorry it's taking so long. @ras0219-msft

No problem at all, I just wanted to check in.
Please take your time to make sure it's in a state you can maintain going forward. I'd be happy to help on any decisions you're unsure about.

@Mixaill Thanks for the suggestion. I've reconfigured the xlnt portfile based on your recommendation and some other example ports I found. @ras0219-msft It took more work than I anticipated, but I believe my library and associated port now meet all of your requirements. static/dynamic and x86/x64 builds are all working on my end. Let me know if you need me to do anything else for this to be merged.

Looks great, thanks for the hard work!


Core team status on user profiles

We plan to move archive our core team Gitter chat room and move our core team over to our new contributor chat room - which is open to anyone who wants to contribute to our open source community through:

open source codebase contributions
YouTube videos
Medium or Wiki articles
Events through our Campsites
curriculum development, and more

Instead of communicating core team status through presence in our core team room, core team members will instead have a special text on their Free Code Camp profile, sort of like this:

Core team membership will be on a year-by-year basis, so core team contributors who get busy and don't have time to contribute any more will automatically be recognized as past core team members, while ones who stay on will continue to accumulate annual core team distinctions.
Here is what we need to do:

[ ] create an array attribute coreTeam on the User model, defaulting to []
[ ] update the profile view to show the text "Core Team N" where N is the year for each year that a camper is on the core team. For example:

coreTeam: ["2015","2016"]

would show the text:
Core Team 2016
Core Team 2015
in reverse chronological order, below earned certificates.
Once this pull request has been accepted, I will manually go into the production database and set this status to true for core team members.

@QuincyLarson, would it not be better to use badges?

I think badge looks more beautiful. I'd like to work on this one. I'll use "text" for now, waiting for @QuincyLarson 's reply.


[JOSS Review] Automated tests

This issue is part of the JOSS review process available here: https://github.com/openjournals/joss-reviews/issues/5128

According to JOSS review criteria:
  Authors are strongly encouraged to include an automated test suite covering the core functionality of their software.

As far as I can tell, MolPainter does not contain any automated or manual tests. I strongly encourage to include tests, at least for the drawing part (to make sure the system is correctly created with lipids and/or solute) and for Molsolvator (to make sure the solvation process is  running smoothly).
The pytest testing framework is a good tool to implement those kind of tests. Additionally, once they are implemented, it would be useful to run those tests automatically through a continuous integration service like Github Actions.

We have added manual tests for molpainter that can be run with python -m pytest. The test for MolPainter checks if it can properly load a JSON file. The test for MolSolvator checks if it can properly reproduce a solvent box given an initial seed for the RNG.

Thanks for the tests although it's quit minimalist.
In Molpainter, the exporter needs to be tested to make sure the PDB produced is correct compared to the GUI grid, same thing for the JSON save file and the solute feature.

Yeah, we planned to add tests of that nature in 1.1.4. Thanks for suggesting specifics.

okay.


chore(main): release 0.1.6

:robot: I have created a release beep boop
0.1.6 (2024-08-16)
Other

bump plugin versions (#395) (ae540cb)

Dependency Updates

deps: update module github.com/slack-go/slack to v0.14.0 (#393) (b3ecb8b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.6
:sunflower:


wip: fixing easy cla

fix to the easy cla process. wip. working with @henkvancann on this. a couple things pending:

TODO: determine if secrets are set correctly
TODO: need to figure out how to test this without a merge

@henk: we need to see what tokens are set on the repo. Need to check in with Judith on this.

@henk: It's a little hard to test this with a fork. I can't run the workflow directly over actions as part of a forked branch. It might make sense for me to directly make a branch on this repo so I can trigger the action manually.

@henk: ... It might make sense for me to directly make a branch on this repo so I can trigger the action manually. ...

It's also not my repo. So I can't give you those user rights, I am afraid. Judith is the woman to go to.

@henk I have permissions to do it. But I want to make sure you and others don't have any issues that. I'd guess it'd be fine, but figured I would check beforehand.

@JFleenor Can you take a look at this:
Total 5 (delta 1), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (1/1), completed with 1 local object.
To https://github.com/trustoverip/acdc
 ! [remote rejected] easy-cla-fix -> easy-cla-fix (refusing to allow a Personal Access Token to create or update workflow `.github/workflows/glossary.yml` without `workflow` scope)
error: failed to push some refs to 'https://github.com/trustoverip/acdc'

I can't really test this flow against the acdc repo without hitting the EasyCLA directly. That means I need access to push to a branch on the repo.

Andor, I've tried to bring in sufficient sense of urgency to do this simple task of giving you access, but I haven't succeeded yet.

Hey @henkvancann there's been a thread going on. I still need access, and I've been talking to an LF member to get me it.
To https://github.com/trustoverip/acdc
 ! [remote rejected] easy-cla-fix -> easy-cla-fix (refusing to allow a Personal Access Token to create or update workflow `.github/workflows/glossary.yml` without `workflow` scope)
error: failed to push some refs to 'https://github.com/trustoverip/acdc'

I'll ping him again and see if I can get access.

I have a workaround which I will attempt.

@henkvancann can you merge this. I'm still having access issues, so it's probably best we just merge and revert if there's an issue.

@henkvancann can you merge this. I'm still having access issues, so it's probably best we just merge and revert if there's an issue.

Hi Andor, I am afraid not, I don't have permission at ToIP. But I have set up a test environment here: https://github.com/WebOfTrust/WOT-terms/actions where I pasted in your PR improvement by hand in glossary.yml,
and I've copied in all the glossary items too: https://github.com/WebOfTrust/WOT-terms/wiki
The Action script still doesn't finalize correctly, but at least I've got all the necessary permission over there to solve the problem. Maybe you can see where it goes wrong.

The glossary.html appears in a commit though: https://github.com/WebOfTrust/WOT-terms/tree/d673f5119d41962ea35d705165f15b57a4d304d7 but is not apparent in gh-pages branch


Ability to add HTML in values of source property of Autocomplete

WDYT about add some HTML in values of source property of Autocompletecomponent ?

Sorry, no more work on the Autocomplete because it is going to be rewritten. The current implementation is very limited and we need something more flexible. Anyway I'm curious about why do you find it useful?

I want to show component like in Dropdown component.

@grrseguin It's going to be possible. The content of the suggestion will be completely given by users by nesting


fix bug on moving clims slider

Description

This PR fixes a bug that caused an error on moving the clims slider.
Type of change

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

References

How has this been tested?

[x] examples/add_image.py

Final checklist:

[x] My PR is the minimum possible work for the desired functionality

What error exactly?

Appologies, should have put in more detail. If you currently try and move the slider you generate the following error
WARNING: Traceback (most recent call last):
  File "/Users/nicholassofroniew/Github/napari/napari/layers/_image_layer/view/controls.py", line 36, in clim_slider_changed
    layer._qt_controls.clim_slider_update()
TypeError: clim_slider_update() missing 1 required positional argument: 'event'

This PR adds an argument None to this callback

Agreed, we might want to revisit this