Disable some SonarPython checks on Code Climate

This disables three SonarPython checks to reduce noise and make it more likely that the things it flags are useful/relevant. The checks disabled are:

number of parameters in a function: we generally allow many parameters, because we often have many with defaults
cognitive complexity: these are, arguably, not actually a useful measure (see also #595)
FIXME comments: we have # FIXME(#someissue) comments recording future work by connecting to our issue tracker

This silences 39 issues (out of 145 total, and 72 from SonarPython specifically) on Code Climate.
See: #954

Codecov Report

Merging #968 into develop will increase coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop    #968     +/-   ##
=========================================
+ Coverage     84.3%   84.4%   +0.1%
=========================================
  Files           58      58
  Lines         4997    4945     -52
=========================================
- Hits          4213    4176     -37
+ Misses         784     769     -15

Impacted Files
Coverage Δ

stellargraph/datasets/datasets.py
67.2% <0%> (-1.8%)
:arrow_down:

Continue to review full report at Codecov.

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


Validation fails with null value

Bicep version
Bicep CLI version 0.14.85 (f4a4d485ba)
Describe the bug
When using New-AzResourceGroupDeployment an error is reported regarding the name property containers.name, the value is reported as "null" even when using a string directly and not via a parameter.
Error that is reported

New-AzResourceGroupDeployment : 12:44:53 - Error: Code=ContainerAppInvalidPropertyValue; Message=Property
'containers.name' has an invalid value ''. A value must consist of lower case alphanumeric characters, '-' or
'.', and must start and end with an alphanumeric character.

To Reproduce
:
Replacing appImageName and/or webImageName with any param or string returns the same error.
Additional context
Add any other context about the problem here.

containerApp.txt
I'm quite new to bicep, so there is possibly (likely) other issues I need to deal with but I keep hitting the validation error

@cwp-michaell this looks potentially suspect (array with an empty object):
      initContainers: [
        {

        }
      ]

Could you try replacing with (empty array):
      initContainers: []

Argh.
That worked. I only had that there to remind me to look at possible use in the future.

Property
'containers.name' has an invalid value

Could do with updating to

Property
'initcontainers.name' has an invalid value.

It would be more useful.


peoplestackwebexperiments-pa.clients6.google.com

Hi
This domain is non malicious
Required for Google Photos Face Grouping

I seem to remember encountering that outside of Google Photos, but I'll remove it.


在九天毕晟上开了一个GPU运行环境，跑train程序，最后报错，什么原因？

Traceback (most recent call last):
File "/root/aaa/ChatGPTBook/PromptProj/train.py", line 261, in
main()
File "/root/aaa/ChatGPTBook/PromptProj/train.py", line 257, in main
train(model, device, train_data, test_data, args, tokenizer)
File "/root/aaa/ChatGPTBook/PromptProj/train.py", line 140, in train
model_to_save.save_pretrained(output_dir)
File "/opt/conda/envs/pytorch1.8/lib/python3.9/site-packages/transformers/modeling_utils.py", line 2376, in save_pretrained
safe_save_file(shard, os.path.join(save_directory, shard_file), metadata={"format": "pt"})
File "/opt/conda/envs/pytorch1.8/lib/python3.9/site-packages/safetensors/torch.py", line 281, in save_file
serialize_file(_flatten(tensors), filename, metadata=metadata)
File "/opt/conda/envs/pytorch1.8/lib/python3.9/site-packages/safetensors/torch.py", line 460, in _flatten
shared_pointers = _find_shared_tensors(tensors)
File "/opt/conda/envs/pytorch1.8/lib/python3.9/site-packages/safetensors/torch.py", line 72, in _find_shared_tensors
if v.device != torch.device("meta") and storage_ptr(v) != 0 and storage_size(v) != 0:
RuntimeError: Expected one of cpu, cuda, xpu, mkldnn, opengl, opencl, ideep, hip, msnpu, xla, vulkan device type at start of device string: meta

just use the requirements.txt ,pip install 。


当图中没有prompt提示的物品时，会把最大的物品当做prompt提示的物品

输入的prompt为sun /cat/ dog 等。图中圈出来的都有问题：

下面这个是原图：

@rentainhe can you help me or give me some advice ？

@rentainhe can you help me or give me some advice ？

There does appear to be an issue with the control over counterexamples in the Grounding-DINO model. This may be due to the model's weights. It might be worth trying better weights to see if it alleviates such a problem.

@rentainhe can you help me or give me some advice ？

There does appear to be an issue with the control over counterexamples in the Grounding-DINO model. This may be due to the model's weights. It might be worth trying better weights to see if it alleviates such a problem.

Thx, i will try with another model weight.

@rentainhe can you help me or give me some advice ？

There does appear to be an issue with the control over counterexamples in the Grounding-DINO model. This may be due to the model's weights. It might be worth trying better weights to see if it alleviates such a problem.

Sorry, could you please confirm again if you are referring to replacing the model or adjusting the parameters in GroundingDINO_SwinB.cfg.py?
Need i change models blow or adjusting config?

My text prompt is 1cat . photos blow are all recogize failed.

我个人实践，在openset上用Grounding-DINO在上做开放目标检测，有些理解

text prompt尽量多测试，并且用地道英语（1cat我都不太能理解），一般框都挺准，但可能和text对不上
box thresh可以调高些，但text thresh过高会出现断词的现象
对于box占全图过大的case就过滤掉
加些启发式联合过滤，比如衣服一定有人脸
openset的zero short 不可避免地会有误检，只能说在大数据范围内准确率还有个60%多，剩下的还得double check

我个人实践，在openset上用Grounding-DINO在上做开放目标检测，有些理解

text prompt尽量多测试，并且用地道英语（1cat我都不太能理解），一般框都挺准，但可能和text对不上
box thresh可以调高些，但text thresh过高会出现断词的现象
对于box占全图过大的case就过滤掉
加些启发式联合过滤，比如衣服一定有人脸
openset的zero short 不可避免地会有误检，只能说在大数据范围内准确率还有个60%多，剩下的还得double check

好的，感谢~


Create README with documentation on the use of the functions and the repo

Describe all the steps to make this notes drawer run with mkdocs

In ref #3


Error: A timeout of 5 minutes is exceeded. Please ensure your EC2 instance has access to the Internet.

I'm sure that this is a configuration issue on my end but I'm not sure what the problem is.
Link to my yml:
https://github.com/choderalab/super-duper-guacamole/blob/27b75e91103c5804e1d056908cb4c97110b5a7eb/.github/workflows/self-hosted-test.yml
Link to github action log:
https://github.com/choderalab/super-duper-guacamole/runs/2273873236?check_suite_focus=true
I made the inbound traffic rules wide open to help troubleshoot this:
Inbound rules
Type | Protocol | Port range | Source | Description - optional
-- | -- | -- | -- | --
All traffic | All | All | <IP_ADDRESS>/0 | wide open for testing
All traffic | All | All | ::/0 | wide open for testing

Outbound rules
Port range | Protocol | Destination | Security groups
-- | -- | -- | --
443 | TCP | <IP_ADDRESS>/0 | GitHubActionSelfHostedRunner

I'm able to SSH onto the EC2 instance that gets spun up -- any other ideas on how to test?

Hey @mikemhenry, thanks for your question!
The issue could be related to the type of EC2 instance you selected: t4g.micro is based on ARM64 CPU architecture, and the action currently uses x64 compatible GitHub Action Runner package as you can see here: https://github.com/machulav/ec2-github-runner/blob/main/src/aws.js#L13
As a result, I would expect that the runner is just can not be started on your EC2 instance and register itself in your GitHub repository. Try to use x64 compatible EC2 instance.
I hope it helps!

I'm sure that is the issue! I am looking at this again today. Do you intend on supporting multiple architectures? Perhaps it defaults to x86 but could support arm64? It is a pre-release but I am interested in helping you test if needed.

Hi, I'm running into this even on an intel based instance. This is my yaml.
My networking is also wide open for testing, both inbound and outbound. Any idea how one would go about debugging this?

Hey @davinchia,

The instance is currently running in a private VPC/subnet with a private IP. Does the instance need a public IP?

If you run it in the private subnet, please make sure you use NAT Gateway with this subnet, so your instance with private IP has access to the Internet and can pull the jobs from GitHub. Otherwise, the instance with the private IP won't be able to connect to the internet.
I hope it helps!

I'm sure that is the issue! I am looking at this again today.

I hope you resolve the issue!

Do you intend on supporting multiple architectures? Perhaps it defaults to x86 but could support arm64?

Yeah, it's definitely a good point to support multiple architectures! Created a new issue for that #37

It is a pre-release but I am interested in helping you test if needed.
Cool, thanks a lot! I will notify you when it's ready for testing.

If you run it in the private subnet, please make sure you use NAT Gateway with this subnet, so your instance with private IP has access to the Internet and can pull the jobs from GitHub. Otherwise, the instance with the private IP won't be able to connect to the internet.

@machulav you were right - worked once I attached a NAT gateway!


 does not narrow

I'm not entirely sure why you would want to extend unknown. But if you end up doing it, it seems a bit surprising that the narrowing behavior is not the same as for unknown itself.

TypeScript Version:  4.0.0-dev.20200509

Search Terms: narrow unknown
Code
function works(arg: unknown): void {
  typeof arg === "object" && !!arg && Object.keys(arg);
}

function fails<T extends unknown>(arg: T): void {
  // No overload matches this call.
  // Overload 1 of 2, '(o: {}): string[]', gave the following error.
  //   Argument of type 'T' is not assignable to parameter of type '{}'.
  //     Type 'unknown' is not assignable to type '{}'.
  // Overload 2 of 2, '(o: object): string[]', gave the following error.
  //   Argument of type 'T' is not assignable to parameter of type 'object'.
  //     Type 'unknown' is not assignable to type 'object'.
  typeof arg === "object" && !!arg && Object.keys(arg);
}

Expected behavior:
Both examples compile.
Actual behavior:
Only works compiles.
Playground Link:  https://www.typescriptlang.org/play?ts=4.0.0-dev.20200509#code/GYVwdgxgLglg9mABAdzgJwNYGcAUBDNAcwC5FwMw5kwBKUgNzhgBNEBvAKEUSgE8AHAKZxgiAoUQBeaYgBEcAEYArQdFmIAZBsQBCHeM3aA8stVQAdBkG9c4mgG4OAXw4dQkWAkTA8MADZYADwAKoiCAB5QgmDMWGRgFFRgAHz4RKTBdIiMLOxciAD0BYgAcnCIcPSCaH5weKwAtnhQEAAWgnFQrTBxEHh+fub5RYhGVTV1rACMFaIATAA0iADkOHCkbE5ZWFBoMGCEANoAustLhHhVPO3ecANU+xLVaOhD3CPcAIJEIA3RULMeAJBCtgstED1EJQAXgsFgYIQwHgFH4QVByvwCHg-lE0IC+EIVptlm9CsVuNxgsCVuRKNRwZDoWI4QikSi0eUCSDlsTSSMxtVavVEHNAYsVmtSIoVNBtrtHiczogLlcuiDgHdashHmE0C80HzyYhvoRfv98dTlmCIXEmbD4YjkaieBisTjqhbCctpWYScMjZTLbSkgzbXAYSzHeyXUCvT7oH7uFyRGIiFIZPJTGpDLp9GmtKMsxYrDY0oQHM4gA
Related Issues:

In general a T extends U won't behave exactly like U in any number of cases. Different things having different behavior is why we have different things.

@RyanCavanaugh I had a very similar issue to @asmundg . I was surprised to learn that T extends unknown can't narrow.. what do you see as the intended use case of <T extends unknown> ? Why not just use <T>? I found that this does compile when you remove the extends unknown (even in strict mode), but that said I can't figure out what the point of extends unknown is.

@benwiley4000 Did you manage to find out what the point is of extends unknown?

@sebasthuis My current understanding is that it's used to trigger distribution and create a distributive type as explained in the v3.0 release notes.


Mark folder updated when an updated item is added

Description
If an item is updated, update the folder entries it belongs to.
Also contains a minor refactor - unexport FolderEntry to folderEntry
Does this PR need a docs update or release note?

[x] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[ ] :no_entry: No

Type of change

[x] :sunflower: Feature
[ ] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Test
[ ] :computer: CI/Deployment
[ ] :hamster: Trivial/Minor

Issue(s)

#1812

Test Plan

[ ] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: vkamra:x: rajatkatejaYou have signed the CLA already but the status is still pending? Let us recheck it.


Create ACK standard between modules

Needs to separate out an ACK library for handling Call and response between modules

ACK needs to be updated to function any time it's send (what a pain). This will make the Master Module Debugging easier.

Might be a good idea to pull this out into it's own little class so that I only have to update mistakes once.


dex: use big endian of complement in the base liquidity index

Is your feature request related to a problem? Please describe.
@zbuc and I noticed that the base liquidity heuristic for path search surface the worst asset first. This is because the key is encoded as prefix || base_asset || BE(base_liquidity) which transfer the numerical ordering over into a lexicographic ordering. But that's not what we want!
Describe the solution you'd like
Instead of doing we should compute BE(!base_liquidity) i.e. get the big-endian encoding of the complement so that the position with the most base liquidity get surfaced first.
Describe alternatives you've considered
It is tempting to extend cnidarium to allow prefix queries to specify a custom iteration order. But I think that fixing the state-key is the better solution because: 1/ a unique iteration order means you don't have to remember which one is the correct for the particular query you are making 2/ this is an easy migration and a useful exercise for us to do.
Random idea, a tiny schema library could provide out-of-the-box encoding wrappers e.g. DescendingOrder(Amount) -> BE(!amount).
Additional context
We should have a couple simple unit-tests that assert this works.

Along the way to this, it'd be great to have an isolated unit test that the ordering encoding is what we expect


Opening name disappears after a game ends. But then it appears on mouse hover.

Weird behavior. Will have to investigate cause. Probably needs a refresh on game ending.

fixed in v2.0.13


Chart component

This is a frontend component that implements the design system
tabs and table alongside a google chart. It is intended for
representing time based data primarily pictorially but with
a tabular backup to support accessibility and allow an alternative
view of the figures.
If running the app for review, see implementions on the page at /dev as the component guide currently doesn't work properly.

@pmanrubia @alex-ju I believe I've addressed everything you both raised so it would be good to get an ok to merge on this.
I'm not sure why I'm getting errors on build - the file it's highlighting as having issues matches the one in master now.


Was trying to create a new AI but something went very wrong with the population requirements

I was trying to create a 'new AI' using the AIC and a new AIV replacing Phillip, but it turned out he needed 28 houses, something obviously has gone very wrong. I don't understand what caused this problem, as population per iron mine, per woodcutter and per quarry values in the notepad edit of output.aic were set to less than Frederick's values. (This AI has a total of 12 farms, 5 ale farms, 8 woodcutters, 3 quarries and 1 iron mine)

Has anyone else had this problem when creating aivs - the AI requiring many more houses than they should?

what about farms per population it also needs to be pointed out that hopsfarms are a bit wonky as they don't count towards the max farm value it would probably be a good idea to share a cutout of the aiv file itself as the screenshot doesnt really tell much of a story

I have that exact same issue and I found some pattern, but no solution:

I'll place extra houses around the castle in case the ai needs them as I know the ai doesn't build them unles required.
Two AIs will work just fine until their economy reaches a certain point.
Then suddenly, one AI will stop producing pawns. It's not AI or castle related, I did some matches with 2 new ais and in every match, exactly one of them got it, but I couldn't see a pattern.
The Suddenly, more and more buildings will get the "no worker" cross sign.
The AI is still doing great with popularity and money, but not long.
The AI lacking pawns suddenly begins spamming houses, probably because it realizes it has no pawns. However, this doesn't help as not a single pawn spawns besides populations up to 400 and every building being out of work!
Meanwhile, the other AI has no such problems, but keeps their economy and recruitment running smoothly.
The AI spawning no pawns slowly but surely gets steamrolled because it doesn't have anymore workers or recruits, at this point it doesn't matter that popularity is to 0 because no money and food is coming in.

I have honestly no idea what's going on. I mean yeah, the economies are relatively big (20 farms, 8 iron mines, 4 stone quarries and 20 weapon producers alongside beer production), but nothing that wouldn't happen a normal game with at least 4 AIs!
Interestingly, when I'm doing a 2v2, only 1 AI gets the pawns to work normally, all other 3 are doomed, too.
I could provide pics/aiv/aic if needed. What leads me to belief that it's not related with aics or aivs is that it happens to the AIs seemingly at random, an ai struggling can be fine the next round. There really seems to be an issue with the pawns though!

this is probably because somewhere a unit was recruited that doesnt exist, rendering popualtion useless, or just kinda making it vanish.


keyboard ui: Fix arrow key navigation for user profile menu.

If you enter the user profile menu (such as by hitting u) the arrow keys don't work to select menu items.  We should fix this.

@shubham-padia This might be something that's easy for you to fix.  (And I'd poke around other menus while you're at it to see if similar bugs exist).

@zulipbot claim


how do I bash / sh to the container?

how do I bash / sh to the container?
I mount the volume, but I can't access, how do I troubleshoot it and check for it? and how do I update the apache configuration?

Hi,
bash is not installed on this image, but you can use sh
To sh into an already running container you can use something like
docker exec -it CONTAINER_NAME sh

You can add custom apache configurations in the /etc/apache2/conf.d/custom directory

Excellent! Thanks for your help!


Add missing dependecies

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
I was working on another conda-forge recipe, when I the build failed because of some pip-check errors. It appears that the following two dependencies are necessary for py7zr.
requirements:
  run:
	...
	# pyppmd and zipfile-deflate constraints are necessary for py7zr
    - pyppmd >=0.17.0,<0.18.0
    - zipfile-deflate64 >=0.2.0

Please update the conda-forge package dependencies accordingly.
Refer to: https://github.com/conda-forge/staged-recipes/pull/18423#discussion_r833280222
Installed packages
NA

Environment info
NA

Thanks for the issue report!
This is very interesting, indeed. Can you put in a PR, please? I would be more than happy to review it. 😄


Change lifecycleChecklist - query parameters into short form

When changing the life cycle status of API using publisher REST APIs [1], lifecycleChecklist can be specified in order to achieve following:

Deprecate old versions after publish the API: Setting this to true will deprecate older versions of a particular API when it is promoted to Published state from Created state.
Requires re-subscription when publish the API: If you set this to true, users need to re subscribe to the API although they may have subscribed to an older version.

We need to make this parameters in short form, as the user need to specify a long sentence as query parameter.
[1] https://docs.wso2.com/display/AM260/apidocs/publisher/index.html#!/operations#APIIndividual#apisChangeLifecyclePost

If fixed, we need to handle the migration scenario as well


Dependency Manager Customizations

AILURE: Build completed with 2 failures.
1: Task failed with an exception.

Where:
Build file<EMAIL_ADDRESS>line: 85

What went wrong:
A problem occurred evaluating project ':react-native-async-storage_async-storage'.

Plugin with id 'com.facebook.react' not found.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.

What went wrong:
A problem occurred configuring project ':react-native-async-storage_async-storage'.

compileSdkVersion is not specified. Please add it to build.gradle

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

invalid issue


Disableable Subsystems

Allows us to easily remove a subsystem when the robot does not have it

Jackson, let's wait to merge this until release 1.1 is merged back into develop.  It will make things a bit easier

This has been tested on the swerve Robot


Update README.md

Hi, please add portcullislabs as a Cisco org. I also fixed a small typo.

Fixes #7


Bluetooth: SMP: Fix failing to pass SM/MAS/PROT/BV-01-C

smp_pairing_complete does actually clears flags so setting
SMP_FLAG_TIMEOUT must come after that.
Fixes #22786
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

@agata-ponitka could you please retest with this PR? Thanks.

It is OK with this PR. Thanks.


various improvements to helm chart

added configuration of LDAP
added log level configuration
added service annotations
added storage class specification
fix bad registry configmap : https://github.com/kubic-project/caasp-services/issues/37

This is awesome, thank you for looking into this, especially ConfigMap!
Since this is a larger change and there were some PRs before you I would like to get some of those merged before I do a full review.
One note for now - persistent storage is disabled by default in community charts unless it is absolutely necessary for the app to function.

i've made both correction, rebased and squashed the commits
i wouldn't mind adding a section in notes.txt but honestly i'm not sure what should be said about ldap in there as the usage ain't much different. I guess replacing the Create admin user account with alternate text saying the first login will be promoted to admin account ?

notes.txt should include basic information of what was deployed and how to get started (it does now) and important information that the chart can't do for you automatically. Some charts will also give a report of any add-ons you enabled.
Like I said it was just a suggestion if anything came to mind that could help someone get started using the feature. I guess it is normally implied that the first user created is an admin.


Added tip on pre-empting instance retirement by stopping and starting the instance.

Added some extra detail, cheers.


Solitary genes

Hi,
Does an option exist to show not only complete systems, but also solitary genes such as solitary methyltransferases ?
In other words, can we have access to the full landscape of defense genes, even if they are not included in complete systems?
Best,
Auriane

No option exists currently, other than inspecting the raw output of HMMER (*.domtblout), although this will give you all the hits pre-e-value filtering. We could add a feature for returning all solitary genes, but our e-value cutoffs have not been optimised for this purpose - and the results would likely require further post-filtering.
I'll re-open this issue if this ever get added.


release 15.4

the patches are no longer necessary for this release

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Allow for "No Delete" policy of certain synced objects through tagging

Is your feature request related to a problem? Please describe.
Currently if a NetBox device or VM is manually added and then picked up by this script, it will follow the same rules as all other synced objects including being deleted if it becomes orphaned.
Describe the solution you'd like
Provide an option to skip the deletion of any objects that contain a certain tag, perhaps "Manual".
Additional context
This was originally raised as part of a discussion in #83.

Merged into develop.


Preloaded modern apps appear on Cairo's taskbar

Some times, on start, I have the same few apps shown in task bar:

Calculator
Photos
Modern settings
Windows Store

They are background tasks, the apps are preloaded but their windows are invisible.
But the apps are listed in the Cairo's taskbar which is wrong behavior.

This seems to happen only when loading Cairo manually after the system has been up for a while in my experience. For some reason the Windows API returns these as active windows even though they are not, and they are not being marked as hidden. I will continue to look for a solution for this, but in the mean time, you can select them and then close the window that appears and the button will be removed.

@mdmk  how did you get program icons on the left like a quick launch?

@nickisghosty You need the app grabber to function in order to pin apps.


Specify total population size?

If we know that the CSV is actually a sample from a population, then we can make a stronger privacy guarantee.

I think this can be de-prioritized. I know population amplification is in DPCreator, but I've heard arguments from people that this is rarely a relevant question, because data is rarely truly a simple sample. There is always some sampling bias from collecting it.


Dynamically change baseFileName of VTKFileInspector

Hello,
Is that possible to dynamically change the baseFileName attribute of a VTKFileInspector (available through inheritance from PerformanceInspector), so I can add a number suffix that will allow me to inspect several ICP being run by the same program but with different initial transformations? I was thinking that I would be able to dynamic_cast the inspector pointer to PerformanceInspector and then change baseFileName, but PerformanceInspector seems not to be available through the installed header files (it is defined into InspectorsImpl.h).

I'm currently using something like the following for that purpose :
  Parametrizable::Parameters param;
  boost::assign::insert(param) ("baseFileName", baseFileName);
  boost::assign::insert(param) ("dumpDataLinks", "1");
  boost::assign::insert(param) ("dumpReading", "1");
  boost::assign::insert(param) ("dumpReference", "1");
  inspector.reset(PM::get().REG(Inspector).create("VTKFileInspector", param));

where inspector is a public member of ICPChainBase and therefore also ICP (see https://github.com/ethz-asl/libpointmatcher/blob/master/pointmatcher/PointMatcher.h#L644).
So instead of changing baseFileName this just creates a new VTKFileInspector with the correct baseFileName. This isn't very expensive and shouldn’t hurt therefore unless it is done at high frequency.  Does that help?

(Sorry for the late reply.)
Yes. Reseting the pointer to a new inspector did the trick. Thank you!


NSAttributedString and HTML support

Hi !
First of all, thanks for this SwiftUI multi-line TextView implementation, that's great to be able to use it as easy as it was under UIKit ;)
I've a question, and maybe I've missed something... but when I use this Textview with an NSAttributedString containing some HTML, style is not updated but Markups are gone...
Maybe I'm doing something wrong...
Do you know if this has already worked for you ?
I mean something like:
// and then in my view
struct MyView: View {

  var content: String = "This is a <b>bold</b> text."

  var body: some View {
      if let contentHtml = content.htmlAttributedString {
          TextView(.constant(contentHtml))
              .isEditable(false)
              .isSelectable(false)
              .allowsRichText(true)
      }
  }
}

When View is displayed, I would expect the word bold to be in bold but here, <b> and </b> markups are removed but no bold anywhere...
Have any idea about that ?
Thank for your answer and have a nice weekend
Jérémie
// somewhere in my code (in a viewModel and in a String extension for example)
extension String {
    func htmlAttributedString() -> NSAttributedString? {
        guard let data = self.data(using: .utf8) else {  return nil  }

        return try? NSAttributedString(
            data: data,
            options: [.documentType: NSAttributedString.DocumentType.html],
            documentAttributes: nil
        )
    }
}

Yeah I don’t believe this view currently supports AttributedString’s actually. I have seen several issues related to this.
I have a complete re-writ in-progress atm and that should resolve these kinds of issues.
That being said HTML support with attributed string’s is extremely slow.
Rather, I’d suggest you checkout my shaps80/MarkdownText package which enables almost full markdown spec support using only native SwiftUI so its extremely performant, animatable, etc…


Compatibility with PWC-Net?

Hi @gengshan-y ,
Thank you for your great work. I just have a question. Have you tested the compatibility of your implementation with PWC-Net?

Hi, I haven't tested, but I think the inference pipeline should be able to work with any flow input.
Note the expansion layers only depend on optical flow and image features (from VCN image encoder). One could replace line 499-516 with PWC-Net and make sure the flow fields matches the resolution of original VCN outputs (where flow is computed on 1/4 image resolution).

Hi, I haven't tested, but I think the inference pipeline should be able to work with any flow input.
Note the expansion layers only depend on optical flow and image features (from VCN image encoder). One could replace line 499-516 with PWC-Net and make sure the flow fields matches the resolution of original VCN outputs (where flow is computed on 1/4 image resolution).

Hi @gengshan-y ,
Thank you very much for your reply. Indeed, I have tried PWC-Net with your expansion pipline. However, I just have questions about your data augmentation.

I noticed that in your augmentation. you used PCA Aug, Chromatic Aug and random rotation, scaling and cropping. But, the input image pair you used are just original image with partial cover, right? Then your GT exp and mid are augmented? Will this be inconsistent?
How would image partially covering benefit the performance of your expansion pipeline? For me, your expansion performance relies on good flow prediction and quality of correlation at top level. So I am just curious about this.

Best,

Hi @gengshan-y ,
Thank you very much for your reply. Indeed, I have tried PWC-Net with your expansion pipline. However, I just have questions about your data augmentation.

I noticed that in your augmentation. you used PCA Aug, Chromatic Aug and random rotation, scaling and cropping. But, the input image pair you used are just original image with partial cover, right? Then your GT exp and mid are augmented? Will this be inconsistent?
How would image partially covering benefit the performance of your expansion pipeline? For me, your expansion performance relies on good flow prediction and quality of correlation at top level. So I am just curious about this.

Best,

From what I remember,

The ground-truth optical expansion is computed on the fly from ground-truth optical flow (not the estimated ones). So there wouldn't be inconsistency wrt expansion.
During training, the input images and flow are augmented together (color+geometric+occlusion augmentation). Here, only the geometric augmentation is applied to optical flow. One could think of it as changing camera intrinsics for both frames, which does not affect the perceived 3D geometry. Therefore I don't see inconsistency wrt motion-in-depth.
At training time, the occlusion augmentation avoids getting over-fitted to clean flow measurements. (Note the ground-truth flow is not changed by such occlusion augmentation)

Hi @gengshan-y ,
Thank you for your reply. Indeed, 3D geometry is irrelevant to image augmentation. Just have one last question. Is that possible for your to provide the VCN model's AEPE on Synth Driving dataset with pretrained weight flow-things, if you happens to observe that?
Best,

Hi, I don't have that number. It would be straightforward to modify eval_flow.py to support evaluation on synthetic driving dataset, but I don't have cycles to work on it right now, thanks.

Hi @gengshan-y ,
Thank you for your reply. I will try that!
Best,


How is targetIndex affecting in multi-targeting? // Problems with recognising targets in multi-targeting

I've been developing a WebAR application with React and A-Frame and I have stumbled across some problems with multi-targeting:
There are various targets (on a physical paper), such as a large lake, a bank and a house near the lake. The lake has mindar-image-target="targetIndex: 0", so as I understand it, it should be weighted higher (= recognised before other markers) than the bank (targetIndex 1) and the house (targetIndex 2). However, if you try to scan the lake from a distance, in the vast majority of cases the bank, which is right next to the lake, is recognised as the marker and not the lake (this also happens if we reduce the marker from the big lake to just one smaller marker in the middle of the lake). So does the targetIndex have no influence on which marker is "higher rated" = firstly recognised?
@hiukim How do you handle different targets that are close to each other or partly maybe even overlapping in multi-targeting?
Has anyone else also experienced similar problems with multi-targeting?
Thank you in advance.


Fix roles submit button disable state

Description
Fixes a bug in Roles with submit button
Related issue(s)
Fixes #771

/test pre-busola-integration-test-k3s

/test pre-busola-integration-test-k3s

/test pre-busola-integration-test-k3s

/test pre-busola-integration-test-k3s

/test pre-busola-smoke-test-docker

/test pre-busola-integration-test-k3s

/test pre-busola-smoke-test-docker

Now on edit, we make every call twice, can we reduce it?

Hmm, it does so as ResourceFormWrapper mounts (not renders) all its children twice (so it resets all the states, refs etc. to default). I would start a new bug with a request to optimize that behavior. Locally I can introduce only a nasty workaround


invalid JSON

Old build was https://travis-ci.org/exercism/ceylon/builds/276065890 that passed using 3.5.0 of configlet.

New build https://travis-ci.org/exercism/ceylon/builds/279014098 fails, using 3.5.1.


Fixed broken link on blog index page

The ACM logo at the top of the blog index page is a broken link (looking for /blog/acmlogo.svg instead of /acmlogo.svg), so I fixed this by adding a / to the start.

thanks will!


Compatible with autoform v5.1.x?

Current version of bs-datepicker is restrained to autoform v5.0.0, tested with autoform v.5.1.2 and everything appears to work.

It should work, but I haven't tested even with 5.0 yet. It depends on 5.0+, so you can update to any 5.x.x.

I received the following response when upgrading from autoform 4.2.2 to 5.1.2
meteor add<EMAIL_ADDRESS>
 => Errors while adding packages:

While selecting package versions:
error: Potentially incompatible change required to top-level dependency: aldeed:autoform 5.1.2, was 4.2.2.
Constraints on package "aldeed:autoform":
*<EMAIL_ADDRESS><- top level
*<EMAIL_ADDRESS>|| 5.0.0 <- aldeed:autoform-bs-datepicker 1.1.1

So I then I ran
meteor update --allow-incompatible-update                                                                                ⏎
This project is already at Meteor 1.1, the latest release.

Changes to your project's package version selections from updating package versions:

aldeed:autoform*  upgraded from 4.2.2 to 5.1.2

* These packages have been updated to new versions that are not backwards
compatible.

So everything does appear to work, but the dependency is currently 5.0.x and not 5.x?

Perhaps I'm just reading the constraints messages incorrectly?

4.0.0 || 5.0.0 means 4.x.x or 5.x.x, so there shouldn't be any conflict. Looks like you are on Meteor 1.1, which has a new constraint solver I think, so maybe a bug in that? I haven't updated anything to 1.1 yet.

My money is on a bug in Meteor 1.1. I'd suggest submitting your issue in the meteor repo. Seems like there's some problem with that new --allow-incompatible-update flag where it's requiring that and ignoring the fact that multiple dependency versions are listed.

I should have read the release notes closer...
"Don't downgrade packages listed in .meteor/packages, or upgrade to a new major version, unless the new flag --allow-incompatible-update is passed as an override."
https://github.com/meteor/meteor/blob/devel/History.md#version-solver
So it's not datepicker package, just the fact that I was going from 4.x to 5.x.

OK, I see, yeah if you were on 4.x when you tried the update I guess you need the flag. Not the most descriptive error message.


The installation dependency called playwright-chromium takes a long time

Clear and concise description of the problem
Hello, In the latest version of the beta (2.7.0-beta.1), I found that a dependency called playmate had been added, which took so long to install that I wondered if the download had been interrupted and wished there were alternatives or advice on how to install it faster.
Suggested solution
No response
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Vite doesn't have any effect on dep installation speed


RLMException", reason: 'The Realm is already in a write transaction

Hello! I'm doing an object property update here, and I get the app crash with the error "RLMException", reason: 'The Realm is already in a write transaction'. Realm I use in many places of the application, when the data comes from the web sockets, that is, the data models that are immediately stored in Realm. I first encountered this problem, how best to fix it? Thanks!
 // save to realm

        let chatRealmGeneralManager = ChatRealmGeneralManager()
        let chatMessageRealmManager = ChatMessageRealmManager()
        if let checkChatConversationModel = chatRealmGeneralManager.getChatConversationModel(chatUserPersonalConversation.id) {
            do {
                let realm = try Realm()
                try realm.write {
                    // MARK: - This is very important
                    checkChatConversationModel.lastActivityTimeStamp = timeStamp // **app crash here**
                }
            } catch {
                debugPrint("error", error.localizedDescription)
            }

            chatMessageRealmManager.updateChatMessageRealmModel(chatMessageRealmModel, completion: nil)
            chatMessageRealmManager.updateChatConversationInfo(chatMessageRealmModel, chatConversationModel: checkChatConversationModel, completion: nil)
        } else {
            let chatConversationModel = ChatConversationModel()
            chatConversationModel.lastActivityTimeStamp = timeStamp
            chatConversationModel.id = chatUserPersonalConversation.id
            chatConversationModel.typeIndex = ChatConversationModel.ChatType.oneToOne.index
            chatConversationModel.lastActivityTimeStamp = chatMessageRealmModel.creationTimeStamp
            chatMessageRealmManager.updateChatMessageRealmModel(chatMessageRealmModel, completion: nil)
            chatMessageRealmManager.updateChatConversationInfo(chatMessageRealmModel, chatConversationModel: chatConversationModel, completion: nil)
        }

I changed my code. It works, but I understand it does not guarantee that I can be completely sure that there will not be an app crash in this place.
    // save to realm
        DispatchQueue.main.async {
            if let checkChatConversationModel = chatRealmGeneralManager.getChatConversationModel(chatUserPersonalConversation.id) {
                do {
                    let realm = try Realm()
                    try realm.write {
                        // MARK: - This is very important
                        checkChatConversationModel.lastActivityTimeStamp = timeStamp
                    }
                } catch {
                    debugPrint("error", error.localizedDescription)
                }

                chatMessageRealmManager.updateChatMessageRealmModel(chatMessageRealmModel, completion: nil)
                chatMessageRealmManager.updateChatConversationInfo(chatMessageRealmModel, chatConversationModel: checkChatConversationModel, completion: nil)
            } else {
                let chatConversationModel = ChatConversationModel()
                chatConversationModel.lastActivityTimeStamp = timeStamp
                chatConversationModel.id = chatUserPersonalConversation.id
                chatConversationModel.typeIndex = ChatConversationModel.ChatType.oneToOne.index
                chatConversationModel.lastActivityTimeStamp = chatMessageRealmModel.creationTimeStamp
                chatMessageRealmManager.updateChatMessageRealmModel(chatMessageRealmModel, completion: nil)
                chatMessageRealmManager.updateChatConversationInfo(chatMessageRealmModel, chatConversationModel: chatConversationModel, completion: nil)
            }
        }

Thanks for getting in touch with us. If you are inside a write transaction, you are not allowed to start another write transaction:
try! realm.write {
    // Will throw
    try! realm.write { /* ... */ }
}

This must be happening somewhere in your code. Do you start a write transaction anywhere in getChatConversationModel()? Do you have a custom setter for lastActivityTimeStamp? Are any of the functions, methods, or property getters/setters you are calling in the code snippets above calling realm.write?

Hi @austinzheng! Thank you for response. No, I do not have realm.write there, I also know that this can not be done, and of course I have not written anything like this in the code.
Also, I threw off completely the function, and the Conversation model completely
  func getChatConversationModel(_ id: String) -> ChatConversationModel? {
        let realm = try! Realm()
        let conversation = realm.objects(ChatConversationModel.self).filter("id == %@", id).first
        return conversation
    }

class ChatConversationModel: Object, Mappable {

    enum ChatType {
        case oneToOne, group, nonFriend

        var index: Int {
            switch self {
            case .oneToOne:
                return 0
            case .group:
                return 1
            case .nonFriend:
                return 2
            }
        }
    }

    dynamic var id = ""
    dynamic var typeIndex = ChatType.oneToOne.index
    dynamic var lastMessage: ChatMessageRealmModel?
    dynamic var lastActivityTimeStamp = 0.0
    dynamic var modelVersion = AppStaticSettings.versionNumber

    /// for group chat equal card photos
    dynamic var cardInfo: ChatConversationCardInfoModel?

    override class func primaryKey() -> String? {
        return "id"
    }

    convenience required init?(map: Map) {
        self.init()
    }

    func mapping(map: Map) {
        if map.mappingType == .fromJSON {
            id <- map["id"]
            typeIndex <- map["typeIndex"]
            lastMessage <- map["lastMessage"]
            lastActivityTimeStamp <- map["lastActivityTimeStamp"]
            modelVersion <- map["modelVersion"]
            // for group chat
            cardInfo <- map["cardInfo"]
        } else {
            id >>> map["id"]
            typeIndex >>> map["typeIndex"]
            lastMessage >>> map["lastMessage"]
            lastActivityTimeStamp >>> map["lastActivityTimeStamp"]
            modelVersion >>> map["modelVersion"]
            // for group chat
            cardInfo >>> map["cardInfo"]
        }
    }

}

@alexsanderkhitev Perhaps is the code called from within the Realm notification block? If so, an exception occurs because the notification block is called within a write transaction.

@kishikawakatsumi No, this block is called by another event (when the user types a message, I use Chatto framework for this). At the moment, I do not use the Realm notification block in the application.

Is it possible for you to create a reproduction case for us to examine? It might be you uncovered a bug, or that there is some subtle interaction that our conversation so far has not discussed.

@austinzheng Hello, sorry for late response. I currently have a loaded schedule, if there is time, I will definitely make an example that repeats this script (exactly where the message is sent in the chat and how it is recorded in Realm).

Thanks, take your time. Feel free to send it to<EMAIL_ADDRESS>or attach/link to it in this ticket whenever you have it.

Closing due to lack of activity. Feel free to open a new ticket if you have further concerns.


WARNING! PCG DID NOT CONVERGE AFTER SHAKE_CG_ITER=   500          STEPS!

I am getting this error during annealing step while using MiMiC. Could you please suggest how to resolve this issue ?

Not relevant here.


Higher input resolution?

Hi, the input face size is 64*64 which is too small, have you tried higher resolution for better image quality?

Hi!
We did try 128x128 at some point, but I do not have a completed model at the moment. It should be quite straightforward to add one more downsampling/upsampling layer to the encoder/decoder networks. But training will be ~4 times slower because of the rendering. And you might also need to adjust the perceptual loss, eg, using a deeper layer.


Feature/Core - Log checker message when collecting Issues

Used in https://github.com/Avaiga/taipy-core/pull/492 to run only the _MigrationChecker on production.

You're right. I'll close the PR and move the log method to core for the MigrationConfigChecker.
Thank you for the review


missing file

Error complains file "donate.dart" does not exist.

Error complains file "donate.dart" does not exist.

The file Donate.dart contains credentials like ad ids and stuff so i put it in . gitignore . On which page/file is the error thrown from ? Because currently it doesn't contain references to donate.dart.

Closing this issue for now.


[Epoch] alternate .m3u8 format not found

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that a supported site is broken
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
video marked as geo-restricted (United States)
Provide a description that is worded well enough to be understood
Looking at this page:
https://www.theepochtimes.com/gluten-public-enemy-documentary_5060209.html
Trying to download with yt-dlp (with United States IP address) produces output shown in subsequent section.
The page source contains a .m3u8 file with a very different format than typical for this extractor.
https://epochtv.youmaker.com/live2/content/96659/play-96659.m3u8
Using yt-dlp with the raw .m3u8 file (generic / hlsnative extractor) works perfectly (and doesn't even enforce the geo-restriction).
I believe that the regex in the extractor needs updated to find .m3u8 files in the above format
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--restrict-filenames', '-o', '%(title)s-%(id)s-%(uploader)s.%(ext)s', '-w', '-F', '-v', 'https://www.theepochtimes.com/gluten-public-enemy-documentary_5060209.html']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d]
[debug] Python 3.11.3 (CPython x86_64 64bit) - Linux-5.15.88-with-glibc2.36 (OpenSSL 1.1.1t  7 Feb 2023, glibc 2.36)
[debug] exe versions: ffmpeg 4.4.3 (setts), ffprobe 4.4.3
[debug] Optional libraries: certifi-3021.03.16, pycrypto-3.17
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[Epoch] Extracting URL: https://www.theepochtimes.com/gluten-public-enemy-documentary_5060209.html
[Epoch] 5060209: Downloading webpage
[Epoch] 5060209: Downloading m3u8 information
ERROR: [Epoch] 5060209: Failed to download m3u8 information: HTTP Error 404: Not Found (caused by <HTTPError 404: 'Not Found'>); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 694, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/epoch.py", line 47, in _real_extract
    formats, subtitles = self._extract_m3u8_formats_and_subtitles(
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 1941, in _extract_m3u8_formats_and_subtitles
    res = self._download_webpage_handle(
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 882, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 839, in _request_webpage
    raise ExtractorError(errmsg, cause=err)

  File "/usr/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 821, in _request_webpage
    return self._downloader.urlopen(self._create_request(url_or_request, data, headers, query))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/yt_dlp/YoutubeDL.py", line 3742, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/urllib/request.py", line 525, in open
    response = meth(req, response)
               ^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/urllib/request.py", line 634, in http_response
    response = self.parent.error(
               ^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/urllib/request.py", line 563, in error
    return self._call_chain(*args)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/urllib/request.py", line 496, in _call_chain
    result = func(*args)
             ^^^^^^^^^^^
  File "/usr/lib/python3.11/urllib/request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found

Initially, force the generic extractor:
$ python3.9 -m yt_dlp -v -F --force-generic 'https://www.theepochtimes.com/gluten-public-enemy-documentary_5060209.html'
[debug] Command-line config: ['-v', '-F', '--force-generic', 'https://www.theepochtimes.com/gluten-public-enemy-documentary_5060209.html']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (source)
[debug] Lazy loading extractors is disabled
[debug] Git HEAD: d4e6ef407
[debug] Python 3.9.16 (CPython i686 32bit) - Linux-4.4.0-210-generic-i686-with-glibc2.23 (OpenSSL 1.1.1t  7 Feb 2023, glibc 2.23)
[debug] exe versions: ffmpeg 4.3, ffprobe 4.3
[debug] Optional libraries: Cryptodome-3.11.0, certifi-2019.11.28, secretstorage-3.2.0, sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1788 extractors
[generic] Extracting URL: https://www.theepochtimes.com/gluten-public-enemy-documentary_5060209.html
[generic] gluten-public-enemy-documentary_5060209: Downloading webpage
WARNING: [generic] Forcing generic information extractor
[generic] gluten-public-enemy-documentary_5060209: Extracting information
[debug] Looking for embeds
[debug] Identified a JSON LD
[generic] Extracting URL: https://epochtv.youmaker.com/live2/content/96659/play-96659.m3u8#__youtubedl_smuggle=%7B%22force_videoid%22%3A+%22gluten-public-enemy-documentary_5060209%22%2C+%22to_generic%22%3A+true%2C+%22http_headers%22%3A+%7B%22Referer%22%3A+%22https%3A%2F%2Fwww.theepochtimes.com%2Fgluten-public-enemy-documentary_5060209.html%22%7D%7D
[generic] gluten-public-enemy-documentary_5060209: Downloading webpage
[debug] Identified a direct video link
[generic] gluten-public-enemy-documentary_5060209: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for gluten-public-enemy-documentary_5060209:
ID   EXT RESOLUTION │   TBR PROTO │ VCODEC        VBR ACODEC    ABR
───────────────────────────────────────────────────────────────────
522  mp4 426x240    │  522k m3u8  │ avc1.4D4015  522k mp4a.40.2  0k
838  mp4 640x360    │  839k m3u8  │ avc1.4D401E  839k mp4a.40.2  0k
1236 mp4 842x474    │ 1237k m3u8  │ avc1.64001E 1237k mp4a.40.2  0k
2315 mp4 1280x720   │ 2316k m3u8  │ avc1.64001F 2316k mp4a.40.2  0k
4509 mp4 1920x1080  │ 4510k m3u8  │ avc1.640032 4510k mp4a.40.2  0k
$

Eventually, perhaps just delete the Epoch extractor.

suggested PR #5552


fix: named params contribute to batch parameter count

The batch validation code was only allowing numbered params contribute to the total parameter count of a query, which prevented batch queries with only named parameters from successfully generating code.
Fixes #1625

merge conflicts with myself


With mantine v7

Will make new PR

Please delete this PR, I will make a new one

thx for this <3 🙏


Display issue (cropped answers & whitespace) when reviewing DMPs when…

… answers in
readonly mode.
Fixes issue #1683.
When the review mode is readonly, then we display the answer in in a paragraphs.
(Could not replicate whitespace issue to test if this fixes it too. But will continue to look at it.)
Before change

After change

looks good to me too. thanks @johnpinto1


Custom style was not works well in Production

In local, the notification works well with custom css .my-notification and type success will reproduce class my-notification success
but in production, the notification reproduces the wrong class my-notificationsuccess which leads to failure to render the style.
All is well just the style is issue. Has anyone ever experienced this and have resolved it?
Local

Text successfully copied to clipboard.

Production

     <name>

Text successfully copied to clipboard.

     </name>

H! I fixed it in 3.1.2 version, please update your dependency

Yeah, it works well now in production. Have a nice day🙌


Doesnt work in right to left layouts

the lib is nice but when the language changes to arabic the messages don't appear

Same issue here


Flutter web: Uncaught (in promise) TypeError: Cannot set properties of undefined (setting 'nativeCommunication')

Environment

Technology
Version

Flutter version
>=3.2.0 <3.19.3

Plugin version
6.0.0

Android version

iOS version

macOS version

Xcode version

Google Chrome version
123.0.6312.59

Device information:
Description
I updated from 5.8.0 to 6.0.0 but now I can't get the project to run on web. No problems in apps.
Expected behavior:
Flutter web should not crash
Current behavior:
Crashes on startup
Steps to reproduce
I tried to remove the imports and references in the code but still didn't work. When I uninstalled flutter_inappwebview, the project could then run in web again again.
 When I do, I get this console error as well:
Failed to load resource: the server responded with a status of 404 (Not Found) u:1 Refused to execute script from 'http://localhost:64112/assets/packages/flutter_inappwebview_web/assets/web/web_support.js' because its MIME type ('text/plain') is not executable, and strict MIME type checking is enabled.

I finally figured it out; we have <base href="/u/"> set so I had to update the path to be /u/assets/packages/flutter_inappwebview_web/assets/web/web_support.js. I was pretty sure I already tried this but must have done something wrong then, perhaps I missed adding the "/" in the beginning.

Please re-open the issue because this is just a patch.

我终于搞明白了；我们已经<base href="/u/">设置好了，所以我必须将路径更新为/u/assets/packages/flutter_inappwebview_web/assets/web/web_support.js。我很确定我已经尝试过了，但一定是做错了什么，也许我忘了在开头添加“/”。 请在安装指南中更新这一点，这样像我这样不习惯使用 javascript 的笨蛋就不会错过这个细节。

I also encountered the same problem when running the web, which simply crashed. It's strange. How can I solve it?


isUserLoading remains true after loadUser() if no user in storage

I have an app that has various requirements around authentication. Some pages require authentication; unauthenticated users should automatically start the signinRedirect.
I am calling loadUser immediately after creating the store (same as the example app).
isLoadingUser gets set to true so I know that I need to wait before firing the redirect, but if no user was in storage then there's no indication that the load is finished and I don't know that it's time to do the redirect.
One of the project's existing test cases is "should dispatch USER_EXPIRED when no user is present" - is this the desired behavior? Firing that action would reset the isLoadingUser flag to false and would be good for my app. Is there any reason not to fire USER_EXPIRED in this situation? The test case uses an expired vs a null user, which I don't think correctly describes my scenario.
I can get a pull request together if USER_EXPIRED should be fired when loadUser is called with an empty storage.

I managed to get this working more like I expected by skipping redux-oidc's loadUser and chaining off the getUser method. In one of the .then methods I could call redux-oidc's userExpired and/or signinSilent.
This results in some duplicate actions being fired if OidcProvider is mounted before the usermanager promises are resolved. I removed my OidcProvider and bound the userManager.events to redux-oidc actions in the last .then method.

Yes, USER_EXPIRED should be fired if no user is present or the user has expired. This currently doesn't happen when the user is null. I will fix this in the next update. Thanks for the bug report!

This was fixed in beta.13. Please check it out and see if this helps you.


Schema representation - 'importing' types

The specification states the following:

A schema or protocol may not contain multiple definitions of a fullname. Further, a name must be defined before it is used (“before”  in the depth-first, left-to-right traversal of the JSON parse tree, where the types attribute of a protocol is always deemed > to come “before” the messages attribute.)

This means that the following schema is valid (the A type is defined in b_field_one and the type is not defined with b_field_two):
{
  "name":"B",
  "type":"record",
  "fields":[
    {
      "name":"b_field_one",
      "type":{"name":"A","type":"record","fields":[]}
    },
    {
      "name":"b_field_two",
      "type":{"name":"A"}
    },
  ]
}

This crate makes it easy to load a schemata in the form of a sequence (Schema::parse_list) of schema where:

The ordering rule is relaxed (no need to load the schemata in a particular order)
The defined-once rule is applied to the whole sequence, instead of per schema

However I can't find a way to output an individual schema in a schemata that is 'complete' (ie does not depend on other schemas, and otherwise complies with the rules).  For example:
   let schema_str_1 = r#"{
        "name": "A",
        "doc": "A's schema",
        "type": "record",
        "fields": [
        ]
    }"#;
    let schema_str_2 = r#"{
        "name": "B",
        "doc": "B's schema",
        "type": "record",
        "fields": [
            {"name": "b_field_one", "type": "A"},
            {"name": "b_field_two", "type": "A"}
        ]
    }"#;
    let schema_strs = [schema_str_1, schema_str_2];
    let schemata = Schema::parse_list(&schema_strs)?;

    for d in schemata {
        println!("{}",d.canonical_form());
    }

Gives the following canonical schemas (for A I think this is OK, B is problematic as there is no definition for A):
{"name":"A","type":"record","fields":[]}
{"name":"B","type":"record","fields":[{"name":"b_field_one","type":"A"},{"name":"b_field_two","type":"A"}]}

I believe the canonical form for B should actually be:
{"name":"B","type":"record","fields":[{"name":"b_field_one","type":{"name":"A","type":"record","fields":[]}},{"name":"b_field_two","type":"A"}]}

Do we have a way of producing this 'correct' form? This is necessary for fingerprint calculation and some schema registry interactions.

I've raised a PR to provide this here: https://github.com/apache/avro-rs/pull/66

Sorry about the noise - I've added a fix for nested Refs (where a schema depends on another that depends on another), and re-opened the PR.  Added a test for this as well.
The goal here is ultimately to support interop with schema registries where each schema is stored independently of other schemata in the registry.
I haven't changed the current functionality (Schema::canonical_form()), EXCEPT, you'll notice a change in the two test cases:

test_avro_3370_record_schema_with_currently_parsing_schema_named_enum()
test_avro_3370_record_schema_with_currently_parsing_schema_named_fixed()
In both cases, the 'expected' canonical form is (I believe) incorrect, as they both include two (duplicate) definitions for a type.


Syntactical rewrite using c++ templates to avoid heap fragmentation

avoid using free/malloc as it causes heap fragmentation and crashes long term (https://arduino.stackexchange.com/questions/682/is-using-malloc-and-free-a-really-bad-idea-on-arduino/712) when repeatedly calling constructor/destructor (this modif avoids that)
use uint8_t in stead of int (saving memory)
use bitSet and bitClear functions

Thank you very much for contributing. I'll review the PR within the next weeks once I have access to hardware to test it.


MSDKUI-1914: [Android] Improve guidance components layouts

Improved Guidance components UI, Added several options in DevApp for GuidanceManeuverview, GuidanceNextManeuverView, GuidanceEstimatedArrivalView etc
Signed-off-by: Vijay Yadav<EMAIL_ADDRESS>

Codecov Report

Merging #105 into master will increase coverage by 1.62%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #105      +/-   ##
============================================
+ Coverage     73.07%   74.69%   +1.62%
+ Complexity     1641      995     -646
============================================
  Files           129       69      -60
  Lines          5656     3936    -1720
  Branches        871      436     -435
============================================
- Hits           4133     2940    -1193
+ Misses          855      749     -106
+ Partials        668      247     -421

Impacted Files
Coverage Δ
Complexity Δ

...va/com/here/msdkui/guidance/GuidanceSpeedView.java
89.69% <100%> (+0.1%)
23 <0> (ø)
:arrow_down:

...e/msdkuiapp/routing/RoutingInstructionsFragment.kt

...java/com/here/msdkuiapp/common/PermissionsUtils.kt

.../com/here/msdkuiapp/routing/OptionPanelFragment.kt

...ere/msdkuiapp/routing/RouteManeuverListFragment.kt

.../here/msdkuiapp/routing/SubOptionPanelPresenter.kt

...m/here/msdkuiapp/routing/SubOptionPanelFragment.kt

...msdkuiapp/guidance/GuidanceNextManeuverFragment.kt

...com/here/msdkuiapp/routing/OptionPanelPresenter.kt

...m/here/msdkuiapp/position/AppPositioningManager.kt

... and 50 more

Continue to review full report at Codecov.

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


function name precedence

Currently the openformula functions are listed individually.  This is necessary to disambiguate them from JMESPath functions.
Expected Behaviour
The grammar should have precedence set up correctly so that openformula and JMESPath functions can co-exist without ambiguity

We've reconciled to a single set of functions without using any precedence


[smo] Issue with Firebase initialization iOS

Fixed crash for projects where firebase is added to pubspec already

Just some initial history of this bug I found. I have a project where I already have firebase auth, analytics etc. I've added your awesome lib to test notifications and found out that it's crashing on iOS. Crash report said that firebase default app can't be initialized twice. So to enable ability to use all firebase products with your package, this check becomes required.

Great job! I didnt got this error on my tests. Thank you so much!
Soon ill release a new version with more complete iOS version.


Step 8 under "Register your app for the Windows Store" is wrong

The Dev Center pages have changed, and there is no WNS page under Services. There is one under App Management, but it doesn't match the description here at all. It requires a .cer file to be uploaded in fact. I'm kind of lost now.

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

ID: 5b48be73-2f29-6b2a-c837-751d3862bcdb
Version Independent ID: efd00d73-1ff5-8689-8415-fe1630795123
Content: Get started with Azure Notification Hubs for Universal Windows Platform apps
Content Source: articles/notification-hubs/notification-hubs-windows-store-dotnet-get-started-wns-push-notification.md
Service: notification-hubs
GitHub Login: @jwhiteDev
Microsoft Alias: jawh

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

@femsulu please assign to @dimazaid

@BryanTrach-MSFT please assign this one to me.

@spelluru is this issue resolved? Have the WNS docs been updated since this went in?

@jwargo - No. Sorry. I have to take this as an action item.

It looks like this issue has been addressed, so I am closing this. Thanks!


[ROCKETMQ-272] Fix sync slave timeout when using SYNC_MASTER

Jira: https://issues.apache.org/jira/browse/ROCKETMQ-272
The timeout logic doesn't work correctly.
Thread waiting in GroupTransferService may frequently waked up by ReadSocketService in HAConnection.
So the transfer logic may return soon and wake up the thread waiting for the HA handling, which will make the timeout value in HA handling useless.
This patch repairs the timeout logic in syncing, and also introduces an option syncSlaveTimeout in MessageStoreConfig to distinguish from the disk flush timeout option.

Coverage increased (+0.02%) to 39.051% when pulling 6f2501a24a701368b6213fd5acb3355ebdaafeb6 on evthoriz:debug-ha into ca14a2d474b6c71143944ec95f7c28e23e15632d on apache:develop.

@vongosling @zhouxinyu  The CI environment is not correctly set. Would you guys have a loot at that?

@evthoriz
Would you please also point out that what kind of scenario  will this issue  be reproduced?

@Jaskey This issue can be easily reproduced with the config I provided in Jira. Please take a look at the description there.

@evthoriz , so the scenario is for huge message.
Please modify the jira title to make it clearer.
But actually, I sent huge message before, but not able to reproduce it, so huge messages scenario
is only the necessary condition, it may also depends on other factor.
Let's get more people involved. @vongosling @shroman @lizhanhui

@Jaskey Another key condition is the MASTER-SLAVE replicating mode, which is SYNC_MASTER in the case.
However, the option syncFlushTimeout is useless not only for big messages, but also those small messages. Small Messages just not trigger the bug, but it still exists.

@vongosling @zhouxinyu @shroman @lizhanhui Anybody willing to review this pr？

@evthoriz may you mock a test for this scenario?

@dongeforever thanks for your reply. I tried, but it seems that there's no easy way to mock a test for this, since the ha mechanism involves lots of salve replicating behavior. Do you have any good ideas?

I will close the pr, if you happened to the same question, please let me know.


WIP: Implement ocr-numbers

This is not ready for merge. Just getting it up here so that we can start the discussion/bikeshedding.
I've created the tests and the stub file. No working implementation yet.
I'm really not sure what implementations people will use for this one.
Will be interesting to see.
The one goofy thing I've done here is turn off rustfmt for each test. I
don't see a way to turn it off at the file or directory level. I've
confirmed that currently a directory-level
ignore is not
supported. But maybe there's a file-wide directive that I'm missing?

Any ideas on what a common approach to this will be? Knowing that will help figure out where this problem should be placed.
I'm thinking of doing the example with either an approach like Swift or hashing each input and comparing it a list of known hashes (which is more or less the same, I guess).

Any ideas on what a common approach to this will be?

My general thought when looking over Ruby implementations was that an implementation similar to the Swift implementation was the most common. Mine was the only solution I saw that counted segments and determined the character based on the on/off segment locations.

Well, matching against slices apparently requires a gated feature, so I probably won't use that.
https://doc.rust-lang.org/book/slice-patterns.html

I've been working on a production issue the past few weeks. Sorry that this one has stagnated.
I'm wondering what the Rusty way is for breaking the input into 4-character chunks.
For example, when I get this input:
 "       _     _        _  _ \n".to_string() +
"  |  || |  || |  |  || || |\n" +
"  |  ||_|  ||_|  |  ||_||_|\n" +
"                           ";

I'll send each 'character' through the conversion function, right? To do that I need to get the first 4 characters of each line.
But indexing strings by characters is not something that Rust supports. So I can't just say "Get me the first 4 characters of this string"
So I'm not quite sure how to proceed. nth()?
Hopefully someone has a good idea.

Untested ideas:

as_bytes if willing to make an assumption that each character is one byte.
chars to get an iterator over the characters, then do something with the iterator

It's pretty gross, but it currently passes everything but the last test. So it's a start.
I spent some time looking at the examples in other tracks and most of them end up doing something similar, so I guess I'm on the right track.

Ok. Tests passing. Finally. Not sure why this one was so hard for me. I don't think I quite get Rust iterators yet.
Starting to think about problem placement. Obviously this one was tough for me, but that might just be my dumb brain. My dumb brain aside, solutions to this will probably use

iterators & looping
lines
chunks (and therefore slices)

The problem of recognizing a single character is pretty straightforward.  The problem there is that you need to get the 3 characters per line, while retaining which 'character' they belong to. Rust makes this slightly harder because you can't index strings, or simply chunk through them.
Because of that, I think this exercise should go later in the track. I'm thinking after wordy. Or maybe even after anagram since the non-indexing of strings is certainly an aspect of Rust that I find new.
I also want to re-order the tests so that they have a smoother curve:

Errors
Individual numbers 0-9
Garbled individual numbers
Multi-character, single line
Multi-line, multi-character

The current order puts the test of a multi-character binary too early. Once this test is passed, nearly all of the remaining tests (except for multi-line) are trivial.

Now that I've gotten this working, I think the API is wrong. This exercise is another one that is well suited for the From trait, right?
Like Roman Numerals I'm thinking the tests should look like
 let input = " _ \n".to_string() +
             "| |\n" +
             "|_|\n" +
             "   ";

    assert_eq!("0", Ocr::from(input).unwrap().to_string());

Since the conversion can fail, it really makes me wish the TryFrom trait were stable so that it could be used instead!
Someone may rightly point out that converting a roman numberal could have failed too (> 3999, < 0), but I guess the difference is the tests never tested a failing roman numeral conversion, whereas we will be testing failed OCR reads.
It's not certain to me that we can use From because of this consideration. Maybe if implementing From for Result<something, something> - not sure how common this is.

I have discovered when attempting this exercise myself:
If we keep the convert function, I would suggest changing the convert(input) into convert(&input), since I don't think it needs to take ownership. And this allows me to write convert(input: &str) instead of the current convert(input: String)

Since the conversion can fail, it really makes me wish the TryFrom trait were stable so that it could be used instead!

Ah, right, because From requires a return of Self. I missed that. Thanks.
Converting to a borrow also makes sense.

I finished implementing. The harder part is definitely splitting up the characters, iterating over the lines in parallel. I imagine that minesweeper also would have needed string indexing (though I guess there is a possible solution that doesn't). Placement immediately before or immediately after minesweeper could be appropriate.

Until this PR is out I can't submit my solution to exercism.io so I'll put it on https://gist.github.com/petertseng/7ec8c8aec510e3685613d8fb36b34d24 for now (note for anyone reading in the future, I'm deleting the gist once I can submit it).

Ok. Updated and rebased.

Well I'm satisfied with what I see here.
If you intend to do the BTreeMap thing I can take a look at that as well when you've done it.
I might have also considered asking that the 4 and 3 be made into constants, but I'm also not going to be super rigorous with non-mission-critical code.

:+1:


Aeotec Nano Switch is now unknown product with 8.5.0

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Update from 8.4.1 to 8.5.0
What do you observe?
All nano switches are labelled as unknown product:

What did you expect to happen?
Switches to work properly like they did in 8.4.1
Device information
node_52.json.txt
https://devices.zwave-js.io/?jumpTo=0x0086:0x0003:0x0074:3.0
https://github.com/zwave-js/node-zwave-js/blob/b70d32963bb3c40d7586b49a46ed1f123c8c0940/packages/config/config/devices/0x0086/zw116.json#L12
How are you using node-zwave-js?

[X] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version:
node-zwave-js branch: 8.5.0
zwavejs2mqtt branch: 5.9.0
Did you change anything?
no
Attach Driver Logfile
interview_log.log

That's not a driver log


Error when launching DNX WEB on Docker with the RC1-final boilerplate

I have created a repository of the sample broken code https://github.com/alexchesser/aspnet-docker-sample In theory, I would expect an out-of-the-box boilerplate to run correctly on the microsoft/aspnet docker image (note that the yo aspnet generators were updated immediately before attempting to build)
Steps to run the code should be:

checkout repo on machine with docker-tools installed
run docker-compose up -d
revel in glory and amazement at an MVC boilerplate running on a linux docker VM
actual results are abject failure and woe.

Note: on a windows powershell, the code within the WebApplication runs near flawlessly with the dnx web command (there appear to be some image errors on the carousel, but those are irrelevant to this issue)
Note that I HAVE also tried replacing my volume mounted code in the dockerfile with a COPY command, per the instructions in http://blogs.msdn.com/b/webdev/archive/2015/01/14/running-asp-net-5-applications-in-linux-containers-with-docker.aspx
However, I have found that this does not make a difference AND in the case of setting up a local-dev environment, running the code from the mounted filesystem allows for a smoother process of editing and iterating on code. (in NodeJS projects with livereload or hot-module-swap this is a killer-feature)

Seems to be the case that you can't build a DNX project without copying the files to it.


fix: allow localhost and svg namespace urls

#7

Hi @jason-ha  we resolved this with the new release of v0.1.7. I didn't merge the PR as I wanted to add unit tests but the changes have been made.
Thanks for your contribution!

Closed PR as the functionality it tried to provide was added

@A-Katopodis, can you re-open and merge? Main branch doesn't appear to handle svg namespace, unless I am reading the code incorrectly. (I just looked to see if main branch had the same changes and it does not.)

@jason-ha the new changes actually include the full exclusion of the xlmns attribute. What you want is the exclusion of the svg namespace regardless of where is present? Because from my understanding it should only exist in the xlmns attribute

@A-Katopodis, I see it now. Added in a prior change. The comment saying:
TODO: add more typical false positives such as XML schemas after more testing
should be updated.


Documentation is missing screenshots.

For this project to gain traction, there need to be screenshots for every page that has an example application. Especially the ones that us the .kv files.
I'm willing to contribute screenshots, if someone will point me to where to submit them.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/99049-documentation-is-missing-screenshots?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

This does sound like a good idea.   I could see rewriting doc/build/html/gettingstarted/examples.html to have nice screenshots and better descriptions.   Should I go ahead and do it?
Just kidding, I'll just put in a Pull Request rather than asking permission. ;)

OK, I'm getting closer to this one.   If anyone has sensibilties about the layout, please let me know now.
The idea is to have a gallery of screenshots, with descriptions pulled from the example docstrings,
There are two screenshots showing how that looks.  Clicking on any screenshot goes to a detail
page that linky's the doc string and puts the source code for the example and related files on that page.

Great, I love it :+1:

Awesome work!

There is still some work to be done on this issue:

Document (add docstring) and upload screenshots for another 30 examples
Update contributions/other pages to explain how documentation is generated
Update or replace the GettingStarted / Examples page
Automatically filter and link to major Kivy APIs based on example imports.
Don't trigger rebuilding all examples on every make.

I'm willing to do the actual work, but no longer willing to deal with the PR cycle.   If someone is willing to handling the objections on each PR, I can either pass them the code or just let them work on it after the intial pull request.   For example, PR #3027, PR #3023, and PR#3029 move towards this issue, but require soothing to get merged.  I'm looking for someone who likes being the buffer.

I really love our new gallery. Thx for it. The only way it could be improved is to change the order of examples. My personal view is that the very first example shouldn't be a 3d rendering with openGL....
I saw that the galery is in alphabetical order, and generated with sphinx. I'm not experienced with sphinx. My question: Is renaming our examples an easy way to reorganize the gallery?
Under renaming I mean for example:
instead of "app_with_build.py"  putting a number in front of it : "1_app_with_build.py"

My first version actually did that, naming screenshots like
"01__path__file__ext.py".  It's very doable, changing existing file names
and such.  Unfortunately, I'm no longer directly making changes to kivy:
it frustrates me that every pull request comes back with requests for
useless changes.   I you care enough, I'll make the change and let you
fight with getting it accepted.
On Tue, Feb 17, 2015 at 12:37 AM, picibucor<EMAIL_ADDRESS>wrote:

I really love our new gallery. Thx for it. The only way it could be
improved is to change the order of examples. My personal view is that the
very first example shouldn't be a 3d rendering with openGL....
I saw that the galery is in alphabetical order, and generated with sphinx.
I'm not experienced with sphinx. My question: Is renaming our examples an
easy way to reorganize the gallery?
Under renaming I mean for example:
instead of "app_with_build.py" putting a number in front of it :
"1_app_with_build.py"
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/710#issuecomment-74633662.

Are these only PEP8 - related issues?

The issues feel more conflicts about appreciation and wanting
conversations.   Any PEP8 issues get a bounce, and reviewers are upset that
I often use GitHub's Mac tool which fails on commit hooks.  Other bounces
include odd things like too much space in the github archive, possible
experiments someone else will be running, and so on.   I currently have
four open PRs with one 'closed' but still in progress.   At least one PR
has been resubmitted several times, never on substantive changes.   I try
to not to have more than five PRs open, as it starts to cause more
management than coding.
Currently, I'm just going through the rest of the examples and will submit
a single PR to close most of the remaining issues in the examples
directory.
For the screenshots, ordering, how about:

screenshots with 'ordering tags' are always first.
'ordering tags' are optional tags before the directory name and can only
be digits.
This adds the restriction that an untagged screenshot may not come from a
directory named only with digits, but
that's acceptable.
The changes are changing the pattern to:
pattern = re.compile(r'^((.+)(.+)([^-]+)).png')
also, adjust the group numbers, set a ['order_tag'] in the fileinfo,
and change the sort line to use order_tag first.
What screenshot order did you have in mind?   Would a grid of thumbnails
be a better solution?

Charles
On Mon, Feb 23, 2015 at 12:28 AM, picibucor<EMAIL_ADDRESS>wrote:

Are these only PEP8 - related issues?
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/710#issuecomment-75504910.

:+1:
And I like the thumbnail grid as well.
As a first idea I would suggest:

Application example using build() + return
Gallery of Examples » Application built from a .kv file
Application from a .kv in a Template Directory
4.Showcase of Kivy Features
Kivy Catalog
Camera Example
Lines Extended Demo
Line (SmoothLine) Experiment
Circle Example
Widget animation
Canvas stress
Rotation Example
Bezier Example
Repeat Texture on Resize
Texture Wrapping and Coordinates Example
Multitexture Example
Stencil demo
Shuffled Camera Feed Puzzle
Mesh test
Tesselate Demonstration
Suite of Application Builders
Basic Picture Viewer
Touch Tracer Line Drawing Demonstration
3D Rotating Monkey Head
Multistroke Recognition Database Demonstration

I think it would be also good to auto-generate screenshots with kivy's screenshot method, without transparent borders and window decorations, as those elements feel outdated in 1-2 years and they are not representative because kivy is cross-platform.

In general, generating the screenshots would be better for the reasons
dessant mentions, as well as for extra unit tests.  The currently generated
screenshots are usually 'further into the app', meaning some user input
should already have taken place.   To automate the screenshot of an app
that starts with a blank screen is an interesting problem; I'm not clear
what's used for event replaying.   So, a good suggestion but not yet
practical.
On Thu, Feb 26, 2015 at 3:35 AM, dessant<EMAIL_ADDRESS>wrote:

I think it would be also good to auto-generate screenshots with kivy's
screenshot method, without transparent borders and window decorations, as
those elements feel outdated in 1-2 years and they are not representative
because kivy is cross-platform.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/710#issuecomment-76163651.

I think I see what you are doing:  trying to make a progression of
applications that guide a new programmer through the library.  Less like a
tutorial and more like a book broken up by chapters?
So, how about adding a hierarchy and some 'just read me' text.  That is,
slice your list something like this:
Building Applications
Readme note on purpose of examples and pointing to documentation
1. Application example using build() + return
2. Gallery of Examples » Application built from a .kv file
3. Application from a .kv in a Template Directory
Playgrounds
Readme note on purpose of examples and pointing to documentation
4.Showcase of Kivy Features
5. Kivy Catalog
Graphics
Readme note on purpose of examples and pointing to documentation
6. Camera Example
7. Lines Extended Demo
8. Line (SmoothLine) Experiment
9. Circle Example
10. Widget animation
11. Canvas stress
12. Rotation Example
13. Bezier Example
14. Repeat Texture on Resize
15. Texture Wrapping and Coordinates Example
16. Multitexture Example
17. Stencil demo
18. Shuffled Camera Feed Puzzle
19. Mesh test
20. Tesselate Demonstration
21. Suite of Application Builders
22. Basic Picture Viewer
23. Touch Tracer Line Drawing Demonstration
24. 3D Rotating Monkey Head
24. Multistroke Recognition Database Demonstration
Is this where you are heading?
On Thu, Feb 26, 2015 at 2:27 AM, picibucor<EMAIL_ADDRESS>wrote:

[image: :+1:]
And I like the thumbnail grid as well.
As a first idea I would suggest:

Application example using build() + return
Gallery of Examples » Application built from a .kv file
Application from a .kv in a Template Directory
4.Showcase of Kivy Features
Kivy Catalog
Camera Example
Lines Extended Demo
Line (SmoothLine) Experiment
Circle Example
Widget animation
Canvas stress
Rotation Example
Bezier Example
Repeat Texture on Resize
Texture Wrapping and Coordinates Example
Multitexture Example
Stencil demo
Shuffled Camera Feed Puzzle
Mesh test
Tesselate Demonstration
Suite of Application Builders
Basic Picture Viewer
Touch Tracer Line Drawing Demonstration
3D Rotating Monkey Head
Multistroke Recognition Database Demonstration

—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/710#issuecomment-76154883.

Yes. Definitely. It should have a "learning-curve". But I wouldn't name it as chapters of a book. I would name it what it is: "learning by examples". Every example in the same "chapter" is building on the previous example.
I've learned a lot playing around with some small basic examples. (I think it is not just me.) And that is why I like the matplotlib gallery as well. It allows you to see the very basic elements from you can build up your own plot / report / pdf / whatever.
As a newbie with knowledge in python I would really like to experiment with small basic examples. And it is great to have something on the homepage to enhance these kind of interactions...

@picibucor @merriam
As a newbie I second the two above posts. I best learned by copying examples (and asking questions on IRC). At the end of  the pong tutorial, it's mentioned that you should try examples and open the catalog (in "where to go from there?" section).
Providing a logical structure for a kind of crash course, will be a very nice addition to increase the potential for new "core" contributors. Anything done to ease the learning curve in this sense is probably a big bonus on the long term (cause I feel we lack man power). :+1:

I think we need some more basic examples. SoI've added a new container-example. It is ment to be in the first section, after the playgrounds...
You could find it in my repo:
https://github.com/picibucor/kivy/tree/master/examples/container

my first PR: #3117

Cool!
I like the idea of some examples showing, with explicit comments, how the
properties work and relate to the .kv language.
Good work!
Charles
On Mon, Mar 9, 2015 at 2:40 AM, picibucor<EMAIL_ADDRESS>wrote:

my first PR: #3117 https://github.com/kivy/kivy/pull/3117
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/710#issuecomment-77823977.

I added the guided approach to looking at .kv files, but first down in Kivy Catalog.   The program example/demo/kivycatalog/main.py goes through a bunch of examples using .kv language files.  I added a category spinner so that examples can be found and paged through a little more comfortably:
This should show you what I mean:

It's off in PR #3112, which won't get merged.   Someone could pull that section and resubmit, though I expect to drastically expand the number of examples in kivy_catalog over the next few days.  Might be better to wait.

@merriam

Make atomic commits.
Don't bring in fixes that are not related to the issue your pr solves fix.
We have a commit hook. Use it.
We have even school kids who are able to do this so I believe even you can too.
Simply put if any one can not follow the guidelines then we can't merge their pr.

@merriam Do you need help chunking up PR #3112 to get it merged? Or are any of these: PR #3027, PR #3023, and PR #3029; higher priority?
@akshayaurora I will help him make better commits that will follow Kivy's guidelines.

found a project with great documentation:
http://scikit-learn.org/stable/index.html

@picibucor can you elaborate? What makes it so excellent? I did look at it, and it does have a lot of links for help. But what specifically do you see as excellent?

@FeralBytes , @picibucor: some stuff from https://github.com/kivy/kivy/pull/3112 was already taken out and pulled separately, it would be great to also pr the refactored screenshots, if someone finds the time.

@FeralBytes and others
I recommended matplotlib/gallery because it is highly visual, the diagrams in the examples are well-separated in the code, and they are mostly pretty straightforward.
By matplotlib not only the examples are easy to understand, but your problem as well: You want to visualise a given dataset.
How to do it?
0. Have a set of data

Choose a type of diagram from the gallery
Reorganise your dataset as it is in the example
Copy-paste the chosen example, and replace some variable-names to your own dataset-variable name
Optionally you could have several types of diagrams on a plot, a layout of several plots, edit ticks/labels/layout, add button/slider/... (The more it is used in general, the easier it is to understand)

The examples in the matplotlib/gallery are parallel. They are only categorised, because the examples are not blocks built on each other. You could choose and use one without knowing about the others.
Why is scikit-learn is a good example?
The problem kivy is solving is more complex. There are certain building blocks that are essential to know, blocks that are building on other blocks, and there are parallel parts as well.
I was searching for a documentation that manages this kind of complexity. This is where we scikit-learn comes in. Machine learning is a very complex topic as well.
Please check this: http://scikit-learn.org/stable/tutorial/machine_learning_map/index.html
There are parallel parts, which contains other parallel parts or blocks built on other blocks. And we could use a similar kind of structure by the kivy/gallery_of_examples as well.
For example:

Maybe having such pictures showing the structure of what you are learning at the moment /where you are in the learning curve  would be also a good idea.
Other things I like by scikit-learn:
--> code-text-pic-ratio :
Many pictures, small amount of texts and atomic examples. The longer the texts and/or code-examples are, the more motivation you need to sit down in front of it (even if you know the half of it).
Watch the following video on the topic: https://www.youtube.com/watch?v=fqUSjHjIEFg .
I recommend using atomic examples. By a max. 8-12 line code your motivational barrier will low enough. If an example is longer than that, it could be chunked up (as the tutorials) and the differences between versions could be color-code.
--> large examples are hidden "behind" diagrams
Large examples could be marked somehow, signalling: "Extra motivation is needed".
-->examples are downloadable
easier to play around
Basically that was it... Thx for reading.

@picibucor :+1:
Kivy is big
After a while, some people might take for granted that you know some of the basics, because they are written in the docs (or examples, etc...) but the truth is until you actually have to use them, you don't learn even if you have read about it.
Let's say we had a map like the picture above, it would be slightly easier to check that you didn't miss something important during learning. As is, it's all text, code and experience. Nothing like a crash course.
Better examples (with pics)
Another reason, is also that some common assumptions are being broken.
One good example is ScrollView. The online doc covers a case that is not the most common one. A ScrollableLabel.

Wiki: https://github.com/kivy/kivy/wiki/Scrollable-Label
gist: https://gist.github.com/tshirtman/c8ed74df889c8452afb9

Providing examples that are actually relevant to day-to-day production code (possibly with relevant pictures) is a plus.
Class hierarchy (maybe)
Maybe, it's not that hard to create a schema with the class hierarchies. Or you can also expand the doc with that class hierarchy in plain text like the MSDN online documentation. That would be a starting point even if not perfect.
The problem with non-automated work (pictures)
Schemas, while quick to create often require manual intervention. Considering the steady pace of change of Kivy, it might be better to avoid such work if it cannot be automated. Because frankly, nobody will have time to update them (depends on frequency though). But I still like the ability to go through tutorials and example from a visual map.
NOTE
Off-topic. Someone from the core must define the perimeter of examples beyond the online doc, tutorials, including examples and wiki articles that constitute the real basics. Lastly, if anyone is interested I am building a list of basic issues and improvements on my blog.

I made a gist for generating screenshots (as an idea):
https://gist.github.com/picibucor/fa44131c5b75c9b00230
Does ImageGrab run on other OS-s as well?

@dessant :
Some screenshots added: #3374

Some of the parts if this issue have been fixed. This issue is getting to big to follow.
Opening a new one to start fresh on this.


[ISSUE #6962]operation.md Format adjustment

Which Issue(s) This PR Fixes

Fixes #6962
Brief Description

How Did You Test This Change?

Codecov Report

Merging #7106 (564d0ec) into develop (ab61183) will decrease coverage by 0.03%.
Report is 2 commits behind head on develop.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #7106      +/-   ##
=============================================
- Coverage      42.74%   42.71%   -0.03%
+ Complexity      9303     9295       -8
=============================================
  Files           1139     1139
  Lines          81259    81259
  Branches       10636    10636
=============================================
- Hits           34734    34713      -21
- Misses         42188    42204      +16
- Partials        4337     4342       +5

see 21 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update to 0.16.1, Pytorch 2.1, simplify recipe

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

thank you for noticing that, but that hasn't change in this PR, it was also there before:
2023-11-13T08:40:45.2161860Z WARNING (torchvision): run-exports library package conda-forge::cudnn-<IP_ADDRESS>-h838ba91_3 in requirements/run but it is not used (i.e. it is overdepending or perhaps statically linked? If that is what you want then add it to `build/ignore_run_exports`)

I feel like the recipe simplification is out of scope. I reverted that change.
@carterbox while the cudnn is weird, it doesn't seem any different from before. I am inclined to merge unless there are other issues.

I was just pointing it out. It doesn't seem to be a direct dependency of this package, but I didn't read through the git history of this recipe / upstream to know why it was added as a dependency in this recipe.