quantum neural net example output is Nan when using more than 1 mode

Issue description
The quantum_neural_network.py example output is Nan when using more than 1 mode. (working as expected for 1 mode)

Expected behavior: Not Nan

Actual behavior: Beginning optimization Rep: 0 Cost: 7.0005 Fidelity: 0.0000 Trace: 1.0000 Rep: 1 Cost: nan Fidelity: nan Trace: nan Rep: 2 Cost: nan Fidelity: nan Trace: nan Rep: 3 Cost: nan Fidelity: nan Trace: nan

Reproduces how often: 100%

System information:

Copyright 2018-2020 Xanadu Quantum Technologies Inc.

Python version:            3.8.5
Platform info:             Linux-5.8.0-53-generic-x86_64-with-glibc2.10
Installation path:         /home/jonas/anaconda3/envs/strawberry/lib/python3.8/site-packages/strawberryfields
Strawberry Fields version: 0.17.0
Numpy version:             1.19.2
Scipy version:             1.4.1
SymPy version:             1.7.1
NetworkX version:          2.5
The Walrus version:        0.14.0
Blackbird version:         0.3.1-dev
TensorFlow version:        2.2.0

Source code and tracebacks
update line 135 of the example quantum_neural_network.py from modes = 1 to modes = 2

Hi @jonasbrami, thank you for catching this! I could reproduce the issue based on your suggestion and it seems that the issue is with computing the gradient of the
difference = tf.reduce_sum(tf.abs(ket - target_state))

line in the cost function. Changing the absolute value computation such that a relatively small constant is added and having tf.abs(ket - target_state + 1e-10) makes the resulting nan values disappear.
We're considering a fitting fix here.

Hi @jonasbrami, we've updated the example, it should work now for multiple modes. Let us know if something else comes up :slightly_smiling_face:


Try to install nox with the latest available (non-prerelease) version

@actions/tool-cache has evaluateVersions which could be used, but it barfs on invalid semvers, so we need to pre-filter available versions with semver.valid anyway.
Just use semver.maxSatisfying while at it. Let @actions/tool-cache bring in the semver dep it works with. Fall back to the old behavior in case none of the versions are valid semvers.
Closes https://github.com/daisylb/setup-nox/issues/431

Caveat: not actually tested, just prototyped with some similar code locally, which seemed to work.

Had to delete my fork to be able to fork fjwillemsen/setup-nox2 and file the same PR there: https://github.com/fjwillemsen/setup-nox2/pull/8
The one here is still as valid as it was, it just got autoclosed and I cannot reopen it.


Can't find legacy cleaved_peptide_region (SO:0100011)

I wanted to remove some unsupported annotations from
https://www.pombase.org/term/SO:0100011
(i.e some ISS that have a better annotations,  some are wrong, plus I used to assume that  all of the odd fusion proteins were cleaved on import, it seems that some are, but some are not)
However, I can't find them in:
manual_so_term_annotations.tsv

So far I've only moved the  signal_peptide (SO:0000418) annotations out of the contig files.
I think cleaved_peptide_region is annotated like:
FT                   /controlled_curation="term=sequence feature, cleavage
FT                   site; cv=sequence_feature; evidence=ISS;

OK, I'll wait until they are all migrated. It isn't the most urgent task and editing will be quicker in the .tsv

OK, I'll wait until they are all migrated.

Done!
pombe-embl/supporting_files/manual_so_term_annotations.tsv

fixed


ci: use github actions macos

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-05-03T09:43:17.730+0000

Duration: 6 min 35 sec

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


separate examples and downloadable repository

Currently all the things below are intermixed in a flat-ish hierarchy of bazel-zig-cc:

files that are designed to be loaded by third parties: WORKSPACE and toolchain/.
repository bookkeeping (gazelle, lint, release scripts)
tests/.

The original idea was to have the "minimal required things" be in the top level of the repository; meaning the WORKSPACE file would have only such contents: workspace(name = "bazel-zig-cc"); everything else should be hidden in sub-directories. However, I can't figure out a way to make bookeeping work -- which depends on Gazelle, Go and all that fun stuff. Basically, it's a conflict of what should be in the WORKSPACE.
Concretely:

Could the "book-keeping" WORKSPACE be in it's sub-folder, and the bookkeeping commands be run from there? To be clear, we want the bookkeeping commands to access the full repository.
Can this be made easier with bzlMod? Looking for bzlMod experts here.

If I understand correctly, when you import a repository, you don't actually load it's WORKSPACE file

It does:
bazel-zig-cc$ mkdir x; cp -r toolchain x/
go-code somewhere$ $ bazel build --override_repository=bazel-zig-cc=/code/bazel-zig-cc/x :usergroups
Starting local Bazel server and connecting to it...
INFO: Invocation ID: 284eb0bb-645a-4c0a-8db6-a085e798a5dd
ERROR: Error computing the main repository mapping: no such package '@bazel-zig-cc//toolchain': No WORKSPACE file found in /home/motiejus/.cache/bazel/_bazel_motiejus/80f026c00534678eecd7f80fa20fddc4/external/bazel-zig-cc

If I add a minimal (with the one-line file contents as quoted) or empty x/WORKSPACE, it will then Just Work.
So at least it needs a file present.

Yes, it needs the WORKSPACE file present, but it doesn't load the repos declared in it. So it's fine to make bazel-zig-cc's workspace the bookkeeping workspace, like @laurynasl-uber said.
An experiment you can do is

Create a simple repo that doesn't need rules_go, for example, only contains C code
Load bazel-zig-cc to compile the C code
Verify that neither rules_go or go_sdk is loaded (bazel query "//external:go_sdk")


Error with haishoku backend on wallpapers with limited colors

Sample image that triggers the error: https://i.imgur.com/1gKSbJ0.png
[I] image: Using image Untitled.png.
[I] colors: Generating a colorscheme.
[I] colors: Using haishoku backend.
Traceback (most recent call last):
  File "/usr/bin/wal", line 11, in <module>
    load_entry_point('pywal==3.3.0', 'console_scripts', 'wal')()
  File "/usr/lib/python3.7/site-packages/pywal/__main__.py", line 208, in main
    parse_args(parser)
  File "/usr/lib/python3.7/site-packages/pywal/__main__.py", line 164, in parse_args
    sat=args.saturate)
  File "/usr/lib/python3.7/site-packages/pywal/colors.py", line 144, in get
    colors = getattr(backend, "get")(img, light)
  File "/usr/lib/python3.7/site-packages/pywal/backends/haishoku.py", line 37, in get
    return adjust(cols, light)
  File "/usr/lib/python3.7/site-packages/pywal/backends/haishoku.py", line 31, in adjust
    return colors.generic_adjust(raw_colors, light)
  File "/usr/lib/python3.7/site-packages/pywal/colors.py", line 71, in generic_adjust
    colors[15] = colors[7]
IndexError: list assignment index out of range

Same issue!

If it helps, I've found another image that triggers this bug.

This bug starts in this place (in current code base).
I tried to see why (with REPL):
from haishoku.haishoku import Haishoku
palette = Haishoku.getPalette("https://i.imgur.com/1gKSbJ0.png")
print(len(palette))
# 8

palette = Haishoku.getPalette("https://user-images.githubusercontent.com/23202923/128759243-fecd7209-fed4-44fb-8019-13d086579260.jpg")
print(len(palette))
# 7

For some reason, the length returned by haishoku may be 7, not 8.
A chain of calls led me to:

Haishoku.getPalette
-- Haishoku.getColorsMean

Good find. Going a little deeper, in the call to getColorsMean, by the time the weighted mean colors are computed and the nested for loops have finished executing, the resulting colors_mean has only 7 entries:
# haishoku/haishoku/haishoku.py

  # get the weighted mean of all colors
  colors_mean = []
  for i in range(3):
      for j in range(3):
          for k in range(3):
              grouped_image_color = grouped_image_colors[i][j][k]
              if 0 != len(grouped_image_color):
                  color_mean = alg.get_weighted_mean(grouped_image_color)
                  colors_mean.append(color_mean)


Add derivation for google ceres-solver library

This PR adds a derivation to development/libraries for Google's ceres-solver nonlinear optimization library for C++.

No, it's not that important.


Remove: frontend dockerfile

Could you please elaborate in the description the need for this issue (just for the record)


[C++] Implement GetFileInfo selector for Azure filesystem

Describe the enhancement requested
Implement AzureFileSystem::GetFileInfo(const FileSelector& select).
This should follow slightly different code paths for flat vs hierarchical namespace to take advantage of the performance benefits hierarchical namespace can provide on listing operations.
https://github.com/apache/arrow/pull/38505 implements the first part of the filesystem that depends on hierarchical namespace so that should lay the ground work.
Related Issues:

https://github.com/apache/arrow/issues/18014 (is a child of)
https://github.com/apache/arrow/issues/38335 (depends on)

Component(s)
C++

@kou are you working on this? Can I claim this task?

@felipecrv No. Go ahead!


Consider integration of website with messenger bot

Hi guys, I'm back! (Lol, I admit, a little late with the game, done studying the SAT stuff for now)
Alright, I don't know how realistic this sounds, but I think this is doable. Clubs tend to have their own sign in sheets, which tends to be scattered haphazardly all over the place. We want them to be accessible (and the keyword is easily) by the user. Messenger is an extremely popular platform which practically every person in the school uses.
My proposal is to have add some links on the website to a Messenger bot. (Wilson Ho is mostly involved in creating this bot, and it has already been implemented in Math Club). We can expand this into a centralized system where all clubs can use one bot to record all membership activity. (and also promote events as well as other news!). Think of this as an extension of the website. I think this is also a more effective means of communicating the Lynbrook community as a whole (the messenger bot might be more popular than the app itself, since all the participants need to do is click a messenger link, rather than downloading an app from the store). We are essentially using Facebook's popularity to launch this bot to Lynbrook's students in a far more efficient manner.
The messenger bot we currently have implemented relies on MongoDB (written in JS). There is literally no frontend dev to it at all, just a JS backend server. Overall, its a lot easier to debug and deal with as well. It can do a lot more just beyond membership, too. It can include games, demos, links, and other helpful info.

Possibly, on hold for now

@oliver-ni  I think we might need to share some info the database. i.e. we might need to share the same database.


Implemented "Go to Implementation" and "Peek Implementation"

Closes #37

Fantastic @ivanz! :smile;
I'll be back in a couple of hours. If you want to make those small changes to the changelog I requested, I'll merge when I'm back.

Updated :)

Thanks again @ivanz. You rock!

Awesome stuff!? Is there a CI build to try this out?

No, we don't push CI builds. I'll need to create a beta release, which I was going to take a look at shortly. I need to get an omnisharp-roslyn build first to ensure that other recently-fixed issues are included in the first 1.8 beta build.

Cool. No worries


Feature: Multiplayer

Implement Google Multiplayer service.

Hey @RameshRavone
This module https://github.com/kloder-games/godot-gpgs actually plays well with your module. I just added the realtime multiplayer class and with abit of trial and error got it to work.

Thanks, I'll look into thing's and a PR would be nice,


Make it a ROS 2 package

Signed-off-by: Aaron Chong<EMAIL_ADDRESS>
converts fleet_adapter_mir into a ROS 2 package
move example configurations into configs
documentation updated

Upon testing with a physical MiR robot, some issues were found when interacting with the MiR API 2.0, fixes are more documentation will be added in https://github.com/osrf/fleet_adapter_mir/pull/13

merge change request


Added note about Get-ExchangeCertificate on connect-to-exchange-servers-using-remote-powershell.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/5256
@chrisda do you think we should add this note? This issue is also related to:
https://github.com/MicrosoftDocs/office-docs-powershell/issues/3493
https://github.com/MicrosoftDocs/office-docs-powershell/issues/4651
Thanks!

Docs Build status updates of commit 9941cbd:
:clock10: Full build: the average full build time is 12 min(s) 46 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 9941cbd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/connect-to-exchange-servers-using-remote-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 3c8611d:
:clock10: Full build: the average full build time is 12 min(s) 46 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3c8611d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/connect-to-exchange-servers-using-remote-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 1826166:
:clock10: Full build: the average full build time is 12 min(s) 46 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 1826166:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/connect-to-exchange-servers-using-remote-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


kgo: add TxnForceOffsets

Needed to allow changes to the offset Metadata without a change to Offset.
This is the missing piece for #559

Copying from my reply on the issue:
What do you think about instead changing the behavior in the source: you can modify the input request, add your offsets, and then update postcommit based on the TxnOffsetCommit req+resp on current line 291? This would also avoid adding a new API (and I'm less keen to bump a minor version for a bit), and instead issue this as a patch release?

@twmb I'm not 100% sure I understand what you are proposing. The issue I ran into is that PreTxnCommitFnContext never has a chance to modify the request if uncommitted is unchanged. Is there a way I can force a change in uncommitted that will always result in valid offsets?
The problem seems to arise because the uncommitted offsets API is only concerned with the numerical offsets and not the metadata. I don't see a way to force End to actually make a commit if I only want to modify the metadata. In some cases I could do something hacky like increment/decrement by 1, but in the case of a new, empty topic, I run into the problem that there may not be another valid offset to update to (and there are probably other reasons trying to hack around it like that is ill advised.)

@twmb #580 does it without new API and seems fine in my testing, however I did have to reorder at least the produceID load, so I'm not sure how safe a change this is.


Shopware throws an error right after installation

Hi there,
I'm trying to use this plugin. Therefore i installed the latest release zip (1.1.0). After the activation shopware throws an error. Both in the frontend and in the backend. See:
 Fatal error: Uncaught Error: Class 'Whoops\Run' not found in /home/vagrant/www/shopware/custom/plugins/WhoopsForShopware/WhoopsForShopware.php:30 Stack trace: #0 /home/vagrant/www/shopware/engine/Library/Enlight/Event/Handler/Default.php(91): WhoopsForShopware\WhoopsForShopware->onRouteStartup(Object(Enlight_Controller_EventArgs)) #1 /home/vagrant/www/shopware/custom/plugins/ShyimProfiler/Components/Event/EventManager.php(258): Enlight_Event_Handler_Default->execute(Object(Enlight_Controller_EventArgs)) #2 /home/vagrant/www/shopware/custom/plugins/ShyimProfiler/Components/Event/EventManager.php(78): ShyimProfiler\Components\Event\EventManager->parentNotify('Enlight_Control...', Object(Enlight_Controller_EventArgs)) #3 /home/vagrant/www/shopware/engine/Library/Enlight/Controller/Front.php(157): ShyimProfiler\Components\Event\EventManager->notify('Enlight_Control...', Object(Enlight_Controller_EventArgs)) #4 /home/vagrant/www/shopware/engine/Shopware/Kernel.php(182): Enlight_Controller_Front->dispatch() #5 /home/vagrant/www in /home/vagrant/www/shopware/custom/plugins/WhoopsForShopware/WhoopsForShopware.php on line 30

you have to run composer install in the directory of the plugin

In the last version was a mistake with the composer autoloading. I fixed the issue and made a new release. Could you try this?
https://github.com/shyim/whoops-for-shopware/releases/download/1.1.1/WhoopsForShopware-1.1.1.zip

Thanks, now it works!


Revisit the cpuType type definition

I already merged this PR, but on second thought we could define this as []struct{Type string, Arch Arch} so that a single YAML can be used on multiple archs?

Originally posted by @AkihiroSuda in https://github.com/lima-vm/lima/pull/643#discussion_r805548077

I don't know. Initially I thought that it would rarely be useful, but I can see it for defaults.yaml, so you can override the cpu type defaults for the different architectures.
If we want to change it, we should do it before the next release, so there is no backwards compatibility issue!

we could define this as []struct{Type string, Arch Arch} so that a single YAML can be used on multiple archs?

I'm going to work on this, but I think it should be a map[Arch]string instead of a slice. It makes no sense to specify multiple CPU types for the same arch. The defaults would be:
cpuType:
  aarch64: cortex-a72
  x86_64: qemu64

Let me know if I misunderstand!

I thought it is inconsistent with the images struct, but not a huge deal probably, so SGTM

I thought it is inconsistent with the images struct, but not a huge deal probably, so SGTM

For images it (used to) make sense because you could have a faster local location, and a slower download location for the same arch as a fallback. Now that we have built-in caching, this seems questionable, but probably not worth changing.


Upgrade ruby from 2.7 to 3.1 in bin/parse-changelog.sh

Tests are failing here due to an outdated GitHub certificate. This can be fixed easily but I wonder if we should bother since this project is deprecated. Perhaps we should just archive this repo?
CC @jtuttle @ismarc


Sub-grid bubble model

Description
We have integrated a sub-grid bubble model designed to address the dynamics of a cloud of bubbles within a host medium. The model currently operates under the assumption that the bubbles' positions remain static. This model was implemented in an old MFC version, so we migrated and fully adapted it to work in the current version. The dynamics of these Lagrangian bubbles are influenced by the hydrodynamic field of the host medium, with the presence of bubbles altering the properties of the Eulerian host. Comprehensive details regarding the physical model, governing equations, and the Euler-Lagrangian coupling approach can be found in the work of Kazuki and Colonius (https://doi.org/10.1016/j.jcp.2018.05.029).
Type of change

[x] New feature (non-breaking change which adds functionality)

Scope

[x] This PR comprises a set of related changes with a common goal

If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

[x] Test A: Dynamics of a bubble cloud
A monodisperse cloud of bubbles initially of 10 μm of diameter contained in a 5mm cubic space interacts with a single sinusoidal acoustic wave of 0.1 MPa of amplitude and 300 kPa of frequency. The time evolution of the pressure at the center of the cloud (0,0,0) was compared with figure 3a reported by Bryngelson et al. (https://doi.org/10.1016/j.ijmultiphaseflow.2019.03.028).

[x] Test B: Dynamics of an isolated bubble
An isolated bubble of 50 μm of diameter interacts with a single sinusoidal wave of 0.2 MPa of amplitude and 150 kHz of frequency. The time evolution of the bubble radius is compared to the analytical solution of the Keller-Miksis equation, similar to what is reported in figure 3a by Kazuki and Colonius (https://doi.org/10.1016/j.jcp.2018.05.029).

Test Configuration:
Test A: This test was implemented as an example case in the “examples” directory. The case.py file contains all the bubble properties and flags required. The functionality and physical meaning of these additional variables are commented in the ‘~/simulation/m_global_parameters.fpp’ file. The geometrical parameters of the bubbles are specified in ‘inputs/particles.dat’; the information it contains is x-location, y-location, z-location, x-velocity, y-velocity, z-velocity, bubble radius and interface velocity. The total number of particles is the number of rows in the file. Please run the post_process target with one processor (-N 1 -n 1), which will generate the ‘particles_data’ folder that stores the information from the bubble dynamics evolution at every t_step_save. Then, run the ‘convert_particlesData.m’ MATLAB script to populate a data file compatible with MATLAB which can be used to visualize the radius evolution of each bubble.
Test B: The case.py file from test A will run this test after correctly modifying the acoustic source amplitude and frequency. All the parameters in inputs/particles.dat are required to be equal to zero, except for the bubble radius which will be equal to 0.05. The instructions described for test A are suggested to follow in order to obtain the radius evolution of the bubble.

What computers and compilers did you use to test this:

Checklist

[x] I have added comments for the new code
[ ] I added Doxygen docstrings to the new code
[ ] I have made corresponding changes to the documentation (docs/)
[ ] I have added regression tests to the test suite so that people can verify in the future that the feature is behaving as expected
[x] I have added example cases in examples/ that demonstrate my new feature performing as expected
[ ] I ran ./mfc.sh format before committing my code
[ ] New and existing tests pass locally with my changes, including with GPU capability enabled and disabled
[x] This PR does not introduce any repeated code (it follows the DRY principle)
[x] I cannot think of a way to condense this code and reduce any introduced additional line count

If your code changes any code source files (anything in src/)
To make sure the code is performing as expected on GPU devices, I have:

[ ] Checked that the code compiles using NVHPC compilers
[ ] Ran the code on either V100, A100, or H100 GPUs and ensured the new feature performed as expected (the GPU results match the CPU results)
[ ] Enclosed the new feature via nvtx ranges so that they can be identified in profiles
[ ] Ran a Nsight Systems profile using ./mfc.sh run XXXX --gpu -t simulation --nsys, and have attached the output file (.nsys-rep) and plain text results to this PR
[ ] Ran my code using various numbers of different GPUs (1, 2, and 8, for example) in parallel and made sure that the results scale similarly to what happens if you run without the new code/feature

Converting this to draft as much of the checklist is still to be completed. Also there is significant code duplication in the source code (not DRY).

closing due to inactivity


auto-own createDraft drafts when assigning through proxies (#393)

Heads up, tests fail for the es5 implementation when assigning to new fields. I am not sure how to solve that.
TBH, as I understand the es5 version, it's unsolvable. Feels like a potential compromise and/or footgun to just not support it for es5 because at the same time, it'd be great to have this working in environments that can use native proxies.
Fixes #393.

I doubt this PR is a good idea. It confuses two different problems: casting and creating drafts. (And a it makes a static type checking problem a runtime problem, which is generally a really bad trade) Currently traveling so can't really elaborate, but please don't merge yet.

For the record, I'm not sure this exact solution is a good idea, either. It is just one of several possible solutions and I wanted to discuss over code and see what concrete problems shook out (like the es5 failures).
I thought it was just a static compilation problem at first, but it is also fixes a runtime problem: if you cast an object T to Draft, and it has any readonly fields at any depth, you could try to mutate them later (in a forEach callback, for example) and the typing would check out but it would fail at runtime on the inserted objects.
My first draft of this PR used a new asDraft function and only took ownership of drafts created by that function, but after thinking about it I figured the addition of a new public API cost more long term than this change to the behavior of createDraft.
No problem if you don't want to merge this as is. I just figured having a solution to look at would help make the discussion more concrete. Also this project is great and I wanted to pitch in to say thanks! But as a maintainer myself I know every change incurs maintenance cost, and evaluating unsolicited PRs is nontrivial cognitive burden so no worries!

Edit: just caught up on discussion on #393. Looks like you are already talking through the stuff I mentioned. Seems like the simplest first step is to close this because the solution (if any) will not change the behavior of createDraft 👍🏼


Could not find a version that satisfies the requirement azure-cognitiveservices-speech (from versions: none) - Raspberry Pi4

Describe the bug
I am trying to install Azure Cognitive Services Speech on Raspberry Pi4, but I am getting "Could not find a version that satisfies the requirement azure-cognitiveservices-speech (from versions: none)
To Reproduce
Steps to reproduce the behavior:

pip3 install azure-cognitiveservices-speech
Error occurs
Tried with: pip install azure-cognitiveservices-speech
Error occurs

Version of the Cognitive Services Speech SDK
Latest version
Platform, Operating System, and Programming Language

OS: Ubuntu 20.04
Hardware - ARM (64 bits)
Programming language: Python

Possibly similar issue as #854

Unfortunately, Python is not yet supported on ARM. You can find the supported distributions/platforms here.
I'm sorry for the inconvenience.

@MarcoAndrade26 I've added an item to our backlog to add ARM64 support to our python bindings. Please let us know if there is anything else you need help with.

We have added this work item to our future 1.16 release.

@chrisbasoglu Is there any timeline for 1.16 release? Thanks!

There is no exact date, but at the moment Speech SDK 1.15 is scheduled for the beginning of 2021 and 1.16 should follow a couple of months or so after that.


[Bug] Self intersecting cells

Sometimes, after cellpose, some cells may be self-intersecting, leading to a TopologyException
sopa/segmentation/shapes.py:52 in solve_conflicts   cell1.union(cell2).buffer(0)
GEOSException: TopologyException: Input geom 1 is invalid: Self-intersection

After intersections, some cells may become a GeometryCollection: we now consider this case scenario
Also, _ensure_polygon is now safer: it should really return a Polygon, else the cell is filtered (it should not happen though)


apollo 5.0 need to install apollo kernel?

System information

**OS Platform and Distribution (e.g., Linux Ubuntu 14.04):  16.04
**Apollo installed from (source or binary): source
**Apollo version (1.0, 1.5, 2.0, 2.5, 3.0): 5.0
As I see, Cyber in apollo 3.5 and 5.0 can realize realtime task schedule, Do I need to install apollo kernel in apollo 3.5 and 5.0？

Hi qwetqwe,
It is strongly recommended to install the Apollo Kernel , which provides the necessary kernel level support to run Apollo software stack.

Hi all, we have updated Low-Latency / Realtime Kernel Installation Guide
(English , Chinese) for Ubuntu 18.04 now. Please follow these docs to install PREEMPT/PREEMPT_RT kernels for Apollo.
Thank you.


Adds support for Air live reloading

Adds a basic .air.toml file to support using Air for live reloading
Fixes #240

Do we need any modifications to Makefile? 👀

True!
Just modified the Makefile to use air for "make run". Would it also make sense to include a command for installing air or something like that? If air is not installed, the command will fail.

@ppmdo Thank you Pablo!
Is air something we can install via go install command? If yes, we can simply add it to this command above:
https://github.com/EinStack/glide/pull/270/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52R16-R21

https://github.com/EinStack/glide/pull/277 - this is what I was talking about in the last comment 👀


Bug: Duplicated row synchronizes itself

When I duplicate a row and edit the content, the content is synchronized with the original row.
I suspect that the ID is also duplicated, which causes the unwanted behaviour.

Browser: Chrome
Plugin Version: 2.4.0
Kirby Version: 4.5.0

https://github.com/user-attachments/assets/d3a81999-e31b-4567-bb1b-49b035609751
The red and blue badges are for the spellchecker. Please don't be distracted by them.

I've released a patch that fixes this issue


ENV variables need to be in the .env.example file

Most of the env variables needed for the kafka plugin are not in the env.example file yet.
we need to add them

I've resolved some of these — only KAFKA_BROKERS is really required, and #18 fixes the validation for that.
I'll add #start/env validations for the other options, especially those introduced in #12


🐛 [cloud_firestore_odm] JsonKey has no effect on generated files

Bug report
When I use JsonKey on a field, the generated file doesn't use the name specified in it.
Steps to reproduce
Run the builder for this code
import 'package:json_annotation/json_annotation.dart';
import 'package:cloud_firestore_odm/cloud_firestore_odm.dart';
import 'package:cloud_firestore/cloud_firestore.dart';

part 'person.g.dart';

@JsonSerializable(createFieldMap: true)
class Person {
  Person(this.phoneNumber);

  @JsonKey(name: 'phone_no')
  final String phoneNumber;
}

@Collection<Person>('persons')
final personsRef = PersonCollectionReference();

The generated toJson() method uses 'phone_no' as expected but the code generated by cloud_firestore_odm uses 'phoneNumber'. Check out the generated wherePhoneNumber function, it has:
 _$PersonFieldMap['phoneNumber']!

Expected behavior
It should rather have
 _$PersonFieldMap['phone_no']!

Version:
json_annotation: ^4.7.0
cloud_firestore_odm: ^1.0.0-dev.38

build_runner: ^2.3.2
json_serializable: ^6.5.4
cloud_firestore_odm_generator: ^1.0.0-dev.38

This also causes .update({ propName }) to update propName instead of prop_name

The generated code is correct. The FieldMap key should be the unmodified property name.
If that's not correct, then it's a json_serializable bug. It should use the field name as key, not the JsonKey name.

@rrousselGit Is the .update() behavior intended then? Looks like a big to me

That's not what I said. I said that what's responsible for the problem isn't the ODM but json_serializable

Ah, got it mb

To be fair, I'm responsible for both :)
I'm the one who implemented FieldMap on json_serializable, so that's still my bad. I'll fix it

What is the issue again? Because I checked the currently generated code, and there is no issue.
As established before, the description of this issue is incorrect (doing _$PersonFieldMap['phoneNumber']! is expected).
And the generate FieldMap by json_serializable appears correct too:
const _$Person JsonFieldMap = <String, String>{
  'phoneNumber': 'phone_no',
};

So what's the issue?

@rrousselGit
Yes, that's correct. My bad. I think, I overlooked that _$PersonFieldMap['phoneNumber']! is a map and not the query clause. And that map it indeed points to 'phone_no' string which is correct.
Sorry, this is working as expected.


[#796]: Created RegexValidator for TextInputControl.

PR for #796.
Thank you for maintaining such a nice library!

Thanks @DeeVeX  👍  looking forward for more PRs :)


Package request: apple-color-emoji

Project description
apple color emoji in ttf format, ripped and ported straight from apple's proprietary products so i'm not sure if this will actually be added
the github project hosting the port has been up for almost 5 years at this point so i don't think apple really cares, considering how eager they seem to be to take down projects they don't like

Metadata

homepage URL: https://github.com/samuelngs/apple-emoji-linux
source URL: https://github.com/samuelngs/apple-emoji-linux
license: proprietary
platforms: linux, bsd? probably not darwin but whats the point

Apple's emoji set is under copyright, and they have in fact cracked down in the past: https://blog.emojipedia.org/apples-emoji-crackdown/
This should not be added to nixpkgs.

understandable. it is in the AUR thoughever, twice even. coming from arch having installed nixos 2 days ago this makes things feel a bit more limited desu
not to be an asshole or anything senpai but the article you linked only refers to 2 (two) cases, where relatively unpopular apps got rejected from the Apple App Store®™ back in 2017 for using emojis in the app as a design element, if you can even consider it a design element
considering that snapchat, an app thats been sponsored by/was partnering with apple, one of the most popular apps of the last decade, undisputably uses emojis as a major design element, i think its more likely that it was a case of a rogue employee or some exec making some stupid decisions and reverting them later
anyways im going off topic but in my personal opinion™ i think its worth adding apple emojis to nixpkgs because it saves like 2 minutes when reinstalling a machine but its no big deal if it has to be installed manually

whatever happens in the AUR is immaterial here, as this is not the AUR. that is basically unmoderated user content.

An option could be to create your own derivation or flake in one of your own repos. This would allow you to use it declaratively (the Nix way) without imposing the potential licensing trouble on the nixpkgs repo 😇

What if this was done using runCommandLocal? good src would have to be figured out, maybe it's somehow possible to download fonts from apple without pulling whole macos recovery image?

What if this was done using runCommandLocal? good src would have to be figured out, maybe it's somehow possible to download fonts from apple without pulling whole macos recovery image?

i'm sure apple's file servers support the Range http header so one might be able to figure out a byte offset where the font is located in the dmg file and a way to extract the font without having the entire dmg but it's way too much work for such a simple thing


Allow to update basename

I know this issue has come up before: https://github.com/ReactTraining/history/issues/644
The response to that was that:

The basename should not be considered dynamic; its purpose is to identify an application's "root" on the server.

However history's documentation states that:

If all the URLs in your app are relative to some other "base" URL, use the basename option.

A common use case for the html base tag also seems to put the language in the base url: https://stackoverflow.com/a/1889957/6593498
So I think using the basename to hold a language identifier (or any other possible dynamic information) can be a warranted use case, if every other URL is relative to that.
I also don't see that much of a technical difficulty to implement this.  It would require another public function to change the basename.
Downstream consumers of history (such as react-router) could also then allow to update the basename, but would require to somehow notify usages of createHref (e.g. <Link/>), to be updated.
Is there any strong opinion against this, would be willing to provide a PR for this, if this is not considered too out of scope.

Hey guys, I would like to see it also, please respond. 👍

This would be a very useful feature!
I created an issue for react-router project but they closed it with the same answer:

basename is not intended to propagate. It emulates the HTML  tag, which also does not update throughout the document when changed.

Hope for any reaction from project authors

+1
It would be great to have this feature implemented since it's a very common case when you need to have a dynamic path prefix.
Public methods like setBasename along with onBasenameChange could make the goal much easier to achieve.


problem with unfold

after
k_out = k_out.contiguous().view(batch, self.groups, self.out_channels // self.groups, height, width, -1)

it gives error,
RuntimeError: shape '[2, 1, 16, 34, 34, -1]' is invalid for input of size 294912

this is because of use of unfold on k_out before this, so height, width is not consistent.

I think a better alternative is to call F.unfold()
kernel_size, dilation=1, padding=0, stride=1
k_out = F.unfold(k_out, kernel_size=(self.kernel_size, self.kernel_size), stride=self.stride)
v_out = F.unfold(v_out, kernel_size=(self.kernel_size, self.kernel_size), stride=self.stride)

# Instead of
k_out = k_out.unfold(2, self.kernel_size, self.stride).unfold(3, self.kernel_size, self.stride)
v_out = v_out.unfold(2, self.kernel_size, self.stride).unfold(3, self.kernel_size, self.stride)

You should change the padding to 1, so the dimension w and h of the feature maps would keep the same after 1x1 convolusion.
class AttentionConv(nn.Module):
    def __init__(self, in_channels, out_channels, kernel_size, stride=1, padding=1, groups=1, bias=False):


Discussion: Overhaul ExceptionRenderer Methods

*Modify the Delegates to be func.
Delegate Definition / Signature:
private delegate void ExceptionDataTarget(StringBuilder sb, Exception ex);

Methods which will be affected by this
AppendMessage
AppendMethod
AppendStackTrace
AppendToString
AppendType
AppendShortType
AppendData
CompileFormat

Do you mean to change the exception layout renderer? And to introduce funcs instead of a big switch?
(PS func is always prefererred by delegate nowadays)

Yes. I meant ExceptionLayoutRenderer. The switch will still be there, but just replace the delegates with func.
Do you have any proposal of totally removing the switch case?
PS: I corrected the title.

In my opinion the current code is too much coupled.
But I'm doubting if removing the switch is overdesign, let me know:

create an enum for the Format values (please keep the string format backwardscompatibe)
create a method for parsing the format string to list of enum
create a static dictionary from enum to func
CompileFormat is now just a foreach over the result of 2 and using 3.

Benefits:

no switch (good for cyclomatic complexity)
configure from API would be nice with enum-list instead of string.

Let me try to get a proposed code change in.

@304NotModified Sorry for mis-stating in the initial thread of this issue. What I actually meant was to replace with Action delegates instead of func delegates.

Yes action is also OK. :)

Okie. I am working on it. Will try to get the code ready by this week.

:+1:

@304NotModified

configure from API would be nice with enum-list instead of string.

How do see we can test this once the enums are created?

What do you mean?
To be more clear, I propose to use a list<ExceptionInnerFormat> instead of a string.
We should keep it backwards compatible though

Yes I have added them into the Layour Renderer, but how do cover those lists for unit testing when we do the configuration through the API?

Let's chat about this when you have time. There is no problem with testing this IMO.

Sure. I will IM you now, if you have time.

Will you be having time to PR and push this in 4.3? if yes I can create PR for pushing this change into the master branch.

Yes I have. :)


Perfect Number in Python

@SrMordred : It would be great if you can add the description along with the issue number.
Feel free to use our contribution guidelines (https://github.com/fnplus/interview-techdev-guide/blob/master/CONTRIBUTING.md) for reference

Thanks for contributing


fixed ifdef to flag?() for binding

ifdef was removed and replaced with {% if flag?(:darwin) %} for :darwin and :x86_64
edit - I didn't realize till now that I ran the formatter on this file.  If you would like I can redo this without the formatter.

Thank you!


Logs detailed view

Fixes #638
I've added an optional parameter for the logs command called detailed.
Timestamps are now disabled by default.
A user must use the --detailed command if they want timestamps now.
Works with the --watch command as well.

[x] npm run test succeeds.
[x] npm run lint succeeds.
[x] Appropriate changes to README are included in PR.

I signed it!

@grant thoughts?

Looks fine to me. Asked Steve to review.
Would need a minor version (x.1.x) bump.

This requires a major version bump as it is not backwards compatible. Anyone currently expecting timestamps will need to make a change when upgrading.
Why not invert the logic so the option is --no-timestamps? That would be backwards compatible with the current behavior.

Ah right, that makes much more sense.
I'll make adjustments.

I used the terminology --simplified because it's similar terminology to the api documentation.
See Here

We are unable to make simplified the default view without making a major version bump.
There may be users who are dependent upon the timestamps.
We can definitely make it the default view if you would rather make a major version bump. I'll just revert my recent changes.

Thanks for the contribution!


Live Share Session View in Browser C++ CoDebug No Variables/Callstack

Issue Type: Bug

Create a local C++ project.
Start live share of C++ project, with co-debugging enabled (allowGuestControl = true).
Join live share via browser (preview).
Log in (avoid read-only status).
Set a breakpoint in the browser.
Start the debugger in the browser.
Watch as no variables are populated and the call stack is nearly devoid of useful information.

Visual Studio Code clients don't have the same issue, and the host can also see variables without a problem. It appears to only be an issue for clients within the browser view.
Extension version: 1.0.2666
VS Code version: Code - Insiders 1.49.0-insider (f694b07825a23005985b470e767a68927a83a553, 2020-08-19T05:47:42.278Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
AMD Ryzen 7 1800X Eight-Core Processor          (16 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.93GB (11.63GB free)

Process Argv

Screen Reader
no

VM
0%

We’re not able to prioritize this issue over the other higher-impact issues we receive every week, based on the votes and comments from others in the community and our understanding of the issue. However, rest assured that we love your input. If you feel it deserves to stay open, then clarify your use case and contact us to let us know how severe it is for you.


Add Amazon Clone

I have added Amazon Clone front end template which will help people to learn how big websites are designed, how css helps in the betterment of website look. People will get a idea of the coding that is making it a beautiful frontend-friendly website.

@DhanushNehru Please review my request sir


Slug behavior can't auto generate

Hello, I config in model follow bellow
use yii\behaviors\SluggableBehavior;
  [
    'class' => SluggableBehavior::className(),
    'attribute' => ['title', 'slug'],
    'slugAttribute' => 'slug',
    'ensureUnique' => true,
    'immutable' => true
  ],

When i typing in field slug string: tiếng việt có dấu. I see in database field slug is tiếng việt có dấu.
I think it should is tieng-viet-co-dau.
Thank you!

It's weird to make slug dependant from slug. I'd suggest to change it to
    'attribute' => 'title',
    'slugAttribute' => 'slug',

Then please show the following

Oh, wait a minute.

When i typing in field slug string

You have 'immutable' => true and you are filling slug manually. Current behavior is expected.
Fill title, leave slug untouched and all will work as expected.


View Total Moderation Actions on Moderator Profiles

Is your feature request related to a problem? Please describe.
No, this is not related to a problem.
Describe the solution/idea you'd like
A counter for the extension that shows how many actions a moderator has of all time on their profile.
Describe expected outcome from your solution
An easy way to see how many actions have been performed by visiting the profile page with the extension.

https://brainly.com/profile/LukeG1-16118329

Hi Sahin, I think this code can help you with the enhancement
class Actions {
	constructor(moderatorID){
		this.actionsUrl = `${window.document.location.origin}/moderation_new/view_moderator/${moderatorID}/page:99999`

		this.Init().then(() => console.log({
			"User": this.userNick,
			"Total actions": this.totalActions || 0
		})).catch(console.error)
	}

	async Init(){
		this.response = await this.requestPage()
		this.responseText = await this.response.text()
		this.responseHTML = new DOMParser().parseFromString(this.responseText, "text/html")
		this.userNick = this.responseHTML.querySelector("h1.alignCenter.marginTop.marginBottom > a.nick").innerText

		this.actionsElements = this.responseHTML.querySelector("div.numbers > span.current")
		if(!this.actionsElements) return

		this.totalPages = Number(this.actionsElements.innerText)
		this.lastPageTotalActions = this.responseHTML.querySelectorAll(".activities > tbody > tr").length
		this.totalActions = (this.totalPages - 1) * 15 + this.lastPageTotalActions
	}

	async requestPage(){
		let response = await fetch(this.actionsUrl, {
			redirect: "manual",
			method: "GET",
			credentials: "include"
		})

		if(!response || [0, 302].includes(response.status)) throw "You cannot view this moderator's actions"
		if(!response.ok && response.status != 200) throw "Request failed"
		return response
	}
}

To execute the code, just paste this in console (inside a brainly's page), and type new Actions(moderatorID) to view how many actions the moderator has.
It should work this way:


Need rtcp nack, pli and fir

Librtp is a very helpful library. It has helped me a lot. Thanks to the authors. But I noticed that librtp does not implement some rtcp commands link nack, pli and fir. Do you have any plans to improve it?

Ok I will push after I finish the test, It may take a long time.

Ok I will push after I finish the test, It may take a long time.

Hi,  @29654761 RTCP_RTPFB  and RTCP_PSFB  are undefined.
rtcp_fb.c:100:12: error: use of undeclared identifier 'RTCP_RTPFB'
rtcp_fb.c:100:32: error: use of undeclared identifier 'RTCP_PSFB'

It should be defined as below according to  RFC4585 6.1: Common Packet Format for Feedback Messages
#define  RTCP_RTPFB  205
#define  PSFB      206


FreqDist ordering based on partial order

Change the implementation of <= and >= for FreqDist
so they are both a partial order.
With the old implementation for a pair of two frequency
distributions each could be bigger than the other.
>>> a = FreqDist('aab')
>>> b = FreqDist('abb')
>>> a > b
True
>>> b > a
True

This is very unintuitive as it ignores both common sense
and the mathematical definition of order.
It should be noted that the new ordering scheme is a not
a total order and that both a <= b and b <= a are False.
Closes #1457

Agreed, like inclusion for plain sets, this should be a partial order.


Add patch to add msflint kernel driver as module

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] If you are adding/removing a .spec file that has multiple-versions supported, please add @microsoft/cbl-mariner-multi-package-reviewers team as reviewer (Eg. golang has 2 versions 1.18, 1.21+)
[ ] Ready to merge

Summary

Adding patch to add mstflint kernel driver
Change Log

Does this affect the toolchain?

YES
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: xxxx

Still need to review if we need to enable in ARM64 config


Introduce register_static_xxx! macros

This can further simplify user code.

@Hoverbear Friendly ping :)

Ping @Hoverbear


Names of tools on the details screen of pwa-directory should be active links (else make text black)

See: https://pwa-directory.appspot.com/pwas/5126173218570240

@alipattsev good catch, we want to link to the actual report, we should make them black until we do.
Thank you,
Julian.


getmyinvoices v-1

Please check the following conditions for your PR.

[ ] apiDefinition.swagger.json is validated using paconn validate command.
[ ] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@diamondsnipper21 Please sign CLA

Closing the PR as Partner want to submit the PR using his company's github account


Add a way to install custom plugins

Hi:
Please, could you add a way to install custom plugins at pod creation time? Probably a config file with plugin urls that allow the installation!!
Now I cant install plugins, I have grafana running and when I install plugin with grafana-cli plugin never appear in the UI, also, where are the plugins directory???

Do you mean Grafana plugins?  If so, you should be installing plugins at image creation time.  Create a new image with ours as the base image, and install during the docker build phase.

Ok, I make the same work and all ok, but the option of values with plugins to install is a good way.


Cache .m2

Haha, thanks @johnktims, you also helped me with showing a bug.

:D

Nice, thank you very much!


Stutter & freezing when adding wifi control.

Hi i'm having an issue on wifi where it works fine then it freezes it also seems to set the wrong pixels too. Ive attached a video and the code i've used.
I've adapted ArtNet to use these matrix panels.
#include <ArtnetWifi.h>
#include <Arduino.h>
#include <ESP32-HUB75-MatrixPanel-I2S-DMA.h>
MatrixPanel_I2S_DMA matrix;

// Wifi settings
const char* ssid = "Reece-PC";
const char* password = "";
const int numLeds = 2048; // CHANGE FOR YOUR SETUP
const int numberOfChannels = numLeds * 3; // Total number of channels you want to receive (1 led = 3 channels)
// Art-Net settings
ArtnetWifi artnet;
const int startUniverse = 0; // CHANGE FOR YOUR SETUP most software this is 1, some software send out artnet first universe as 0.
const int maxUniverses = numberOfChannels / 512 + ((numberOfChannels % 512) ? 1 : 0);
bool universesReceived[maxUniverses];
bool sendFrame = 1;
int previousDataLength = 0;

int matrixWidth = 64;
int matrixHeight = 32;
boolean ConnectWifi(void)
{
  boolean state = true;
  int i = 0;

  WiFi.begin(ssid, password);
  Serial.println("");
  Serial.println("Connecting to WiFi");

  // Wait for connection
  Serial.print("Connecting");
  while (WiFi.status() != WL_CONNECTED) {
    delay(500);
    Serial.print(".");
    if (i > 20){
      state = false;
      break;
    }
    i++;
  }
  if (state){
    Serial.println("");
    Serial.print("Connected to ");
    Serial.println(ssid);
    Serial.print("IP address: ");
    Serial.println(WiFi.localIP());
  } else {
    Serial.println("");
    Serial.println("Connection failed.");
  }

  return state;
}

int* computeLedPos(int ledNo)
{
  int x = 0;
  int y = 0;
  for (int i = 0; i < ledNo; i++)
  {
    if(y = 32) {
      ++x;
      y = 0;
    }
    ++y;
  }
  static int cords[2];
  cords[0] = x;
  cords[1] = y;
  return cords;
}

void initTest()
{

}

void onDmxFrame(uint16_t universe, uint16_t length, uint8_t sequence, uint8_t* data)
{
  sendFrame = 1;
  // set brightness of the whole strip
  if (universe == 15)
  {
    //FastLED.setBrightness(data[0]);
    //FastLED.show();
  }

  // Store which universe has got in
  if ((universe - startUniverse) < maxUniverses) {
    universesReceived[universe - startUniverse] = 1;
  }

  for (int i = 0 ; i < maxUniverses ; i++)
  {
    if (universesReceived[i] == 0)
    {
      //Serial.println("Broke");
      sendFrame = 0;
      break;
    }
  }

  // read universe and put into the right part of the display buffer
  for (int i = 0; i < length / 3; i++)
  {
    int led = i + (universe - startUniverse) * (previousDataLength / 3);
    if (led < numLeds){
      int x = led % matrixWidth;
      int y = led / matrixWidth;
      //Serial.print("X: ");
      //Serial.print(x);
      //Serial.print("Y: ");
      //Serial.print(y);
      //Serial.print("R: ");
      //Serial.print(data[i * 3]);
      //Serial.print("G: ");
      //Serial.print( data[i * 3 + 1]);
      //Serial.print("B: ");
      //Serial.print( data[i * 3 + 2]);
      //Serial.println();
      matrix.drawPixelRGB888(x, y, data[i * 3], data[i * 3 + 1], data[i * 3 + 2]);
      //leds[led] = CRGB(data[i * 3], data[i * 3 + 1], data[i * 3 + 2]);
    }
  }
  previousDataLength = length;

  if (sendFrame)
  {
    matrix.fillScreen(0);
    // Reset universeReceived to 0
    memset(universesReceived, 0, maxUniverses);
  }
}

void setup()
{
  Serial.begin(115200);
  ConnectWifi();
  matrix.begin();
  matrix.fillScreen(0);
  artnet.begin();
  initTest();
  artnet.setArtDmxCallback(onDmxFrame);
}

void loop()
{
  // we call the read function inside the loop
  artnet.read();
}

Only a few pixels at the top are being set too that maybe me doing the wrong calculation however as the orignal example used FastLED which sets leds using:
leds[led] = CRGB(data[i * 3], data[i * 3 + 1], data[i * 3 + 2]);
And i do:
     int x = led % matrixWidth;       int y = led / matrixWidth;       matrix.drawPixelRGB888(x, y, data[i * 3], data[i * 3 + 1], data[i * 3 + 2]);
Any help would be greatly appreicated.
https://user-images.githubusercontent.com/4342846/102909174-7a12a400-4470-11eb-9eca-b227002c78ab.mp4

I think it would be better to start with one of the examples provided, like PatterPlasma to make sure the matrix works fine. Than you can add WiFi code, etc..

I think it would be better to start with one of the examples provided, like PatterPlasma to make sure the matrix works fine. Than you can add WiFi code, etc..

The matrix works fine with opther examples, Also it works fine if i remove the WifI code and set each LED manually but for some reason the second i add WiFi iit stutters and freezes

Unfortunately this is likely a result of the ESP32 hardwareand RTOS possibly diverting CPU/DMA time to the WiFi portion of the silicon. There's nothing this library can do about it I'm afriad.

Unfortunately this is likely a result of the ESP32 hardware and/or RTOS possibly diverting CPU/DMA time to the WiFi portion of the chip. There's nothing this library can do about it I'm afriad.

Ahh isee, Off topic but could i offload the wifi portion to another esp and connect them directly together using i2c or something? or would i still face the same issue?

What software are you using to transmit?

Jinx! or Glediator for now but im already in the process of writting my own software for my use case. I just need to be able to get around 30fps transmited from PC -> ESP -> Matrix to be honest it dosen't need to be WiFi but Serail is just too slow even at the maximum baud rate.

You might be interested to know I reverse enginerred Glediator and attempted to hack it for this purpose: https://github.com/mrfaptastic/Glediator3
Perhaps you can contribute if you're good at Java? I think it's time for a simpler protocol to be honest. I attempted to implement a hack protocol in Glediator which was based on 16bit color only. 33% reduction in bandwidth and corresponding increase in speed, but colors wern't as good. Serial was hopelessly slow as well.

Ahhh yes i've used your version to be honest it worked right out the box! but as you say Serial is hopelessly slow if WiFi is an issue i may need to completely rethink this and take a new approach. I work in java from time to time but mostly code in netcore or C++/C#

tried to reproduce the issue but I was unable to even send some data to the controller. Jinx! just sends empty packets - with all correct headers but all zeros in data. Glediator doesn't work via UDP -sigh!
So, @ReeceKetley how do you send Art-Net data to the controller? 30 minutes of searching gave me 0 results on any other free/open software able to do render some Art-Net data via network.

tried to reproduce the issue but I was unable to even send some data to the controller. Jinx! just sends empty packets - with all correct headers but all zeros in data. Glediator doesn't work via UDP -sigh!
So, @ReeceKetley how do you send Art-Net data to the controller? 30 minutes of searching gave me 0 results on any other free/open software able to do render some Art-Net data via network.

You have to patch the pixels in output patch using fast patch (fyi its a bit buggy fast patch is a bit weird you have to try weird values to get it to patch all of the pixels at once also if you see the video i posted i couldn't get it to fully show all the pixels no idea why)

I'm in the process of hacking my decompiled version of Glediator3 to bits to use TPM2_NET as the transmission protocol from the PC to the ESP32. Are you forced to use ArtNet, or do you not care?
https://github.com/mrfaptastic/Glediator3

"Weird" is an understatement :) I was able to light about 150 pixels poking some random values. Colours and animation matches, although it is very slow. Well, it works nothing more to say.
But this weird software is a no-go.

Merry Christmas by the way you two.

I'm in the process of hacking my decompiled version of Glediator3 to use TPM2_NET as the transmission protocol from the PC to the ESP32. Are you forced to use ArtNet, or do you not care?
https://github.com/mrfaptastic/Glediator3

I really hold no love for ArtNet or Jinx! i find it very confusing to be honest. TPM2_NET and Glediator would be fine. Do you think TPM2_Net would have better luck with the panels? I did make my own protocol system to use but as i compressed the data it made things even slower to be honest. @vortigont yeah it's really not ideal. I think as a comunity we could develop something thats better. Im doing some more poking around to see what i can do.
@mrfaptastic Merry Christmas to you too dude happy hacking!

https://user-images.githubusercontent.com/4342846/103111496-b3503d00-4645-11eb-846d-673eab387fd3.mp4
I can play doom on it now.

Is that for real? If so that's the best thing to come out of 2020.

That's awesome! :) @ReeceKetley pls. pls more details! :)))

So thats using @mrfaptastic Glediator TPM2 sketch and some c# code that i wrote to get the screen buffer of an emulator then quickly send it va TPM2 Serial colours are reduced to gain more speed but if you stand like 10 feet away its playable decent FPS

Closing this as I created an example that uses WiFi. I had to hack the TPM2 protocol and split a frame over multiple packets (UDP payload limit is about 1400 bytes) and re-assemble).
Easily got 30 fps over WiFi. Silky smooth.
https://github.com/mrfaptastic/Glediator3/tree/master/esp32-arduino

I have updated the Glediator3 jar to make it much easier to use with the Arduino ESP32 TPM2_NET sketch. Literally three steps now.
https://github.com/mrfaptastic/Glediator3

hi. I've tried the sketch above, and it worked. but the problem is the delay / scanning of the led. I've tried in ELM software, Resolume and jinx, in ELM software I also tried lowering the fps, but it's still the same, after I observe it turns out that in universe 6 to 12 there is a delay, universe 1-5 is ok. I'm attaching the video
https://user-images.githubusercontent.com/6624207/165770198-d660f858-2698-4d2a-8c12-feddc9b66707.mp4


[HINTS] Add implementation for hint on assert_lt_felt (math.cairo)

Add implementation for hint on assert_lt_felt (math.cairo)
Checklist

[ ] Linked to Github Issue
[x] Unit tests added
[x] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #278 (37c40ac) into main (4c455d7) will increase coverage by 0.07%.
The diff coverage is 99.45%.

@@            Coverage Diff             @@
##             main     #278      +/-   ##
==========================================
+ Coverage   96.26%   96.33%   +0.07%
==========================================
  Files          24       24
  Lines        7355     7538     +183
==========================================
+ Hits         7080     7262     +182
- Misses        275      276       +1

Impacted Files
Coverage Δ

src/vm/hints/hint_utils.rs
80.87% <94.73%> (+0.54%)
:arrow_up:

src/vm/hints/execute_hint.rs
100.00% <100.00%> (ø)

tests/cairo_run_test.rs
100.00% <100.00%> (ø)

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 4c455d7...37c40ac. Read the comment docs.


[1.1] Add "say" method to send messages to users

Right now it's only possible for BotMan to react to incoming messages (through webhooks).
But some applications will need the ability to originate messages to the user based on some internal logic.
BotMan needs to be extended with a say method.
This method accepts a channel.
Depending on the messaging driver the channel is:

A Slack User-ID
A Facebook User-ID
A Skype User-ID
A SMS recipient numebr
A Telegram User-ID

The idea would be that you can use the say method without the need to specify the driver. In this case, BotMan will try to deliver the message through all drivers that are configured.
The second option would be, to use a Driver class as an argument for the say method.
Example syntax:
No driver specified
$botman = BotManFactory::create($config);
$botman->say('Hello!', 'channel_id_123456789');

With additional messaging service parameters
$botman = BotManFactory::create($config);
$botman->say('Hello!', [
    'channel' => 'channel_id_123456789'
]);

With a predefined messaging driver
$botman = BotManFactory::create($config);
$botman->say('Hello!', 'channel_id_123456789', FacebookDriver::class);

Thoughts? Ideas? :)

See https://github.com/mpociot/botman/commit/bc291d8e95052b0e9b01f626dcd3fa11145ad79a


Live demo version does not work

When opening https://stackblitz.com/edit/angular2-cookie-law?file=src%2Fapp%2Fapp.component.html on Firefox or Chrome it tries to load the page forever and no content is presented.

@damianszczepanik is this still happening? I can correctly open the link with no problems

Link is working correctly but content is not uploaded on merge to master

I'm not sure what you mean by that. I just created a new demo for Angular 7 using the latest 7.0.1 version of this package and everything looks fine on my end. https://citymapper.com/trip/Tkb8pp8vgh5

Looks fine, maybe temporary problem with hosted app


Support get view's id from IdMap

By calling DynamicView.getIdMap(),
user can get child view via findViewById(idMap.get("id_name").intValue());

Hi @BrandonTsai ,
Thanks for your suggestions.
I think we need a better approach to return the map of <String,Int> because as the createView is a static it is not good to keep the ids in a variable in the class. We need to return in similar way we return the view.
What do you think about that ?
Nikos

Hi @tsiougkosn ,
How about return a DynamicView object which contain the view and the ids when user call DynamicView.createView()?
For example:
DynamicView dynamicView = DynamicView.createView(this, jsonObject, SampleViewHolder.class);
View sampleView = dynamicView.getView();
HashMap<String, Integer> idMap = dynamicView.getIds();


[Question] Connection to a websocket server with external client

Hi,
Is it possible to connect to a asp.net core websocket server with a client that is outside the solution ?
I tried to connect to the server with the same client that was inside the wwwroot folder, without any success . Also I tried to expose the IIS express to my home private network (to prevent possible loopbacks), and it worked for http request but still not for the wobsocket request.
I also tried to deploy it to an external hosting, still without any success.
I need to mention that I tried my client with a connection to ws://echo.websocket.org and it worked like a charm.
You are my only hope, and thanks in advance!

WebSockets has no addition security requirements beyond those of HTTP. What errors do you get?

After some research it worked.
But I will share, to spare the time of others on same issue.
I believe that it's an issue of old asp.net core version,
I had no error, it simply didn't connect to the server.
When I changed the project to an newer asp.net core version it didn't worked once again because
I tested it with Microsoft edge - which it was a big mistake!
Microsoft edge do not allow loopbacks!
I am sorry for the spam.


Getting Started - small error

I recently followed the Tests section of the Getting started. I believe the steps are missing a make call after the cmake.

Right! Let me fix.


NavigationView move focus wrongly before animation finish

Describe the bug
When i change navigation and move focus before selection indicator finish moving, the focus will skip the item between the animation.
Steps to reproduce the bug

Open WinUI 2 Gallery
Go to NavigationView page
Click the fisrt navigation sample "menu Item2"
Click down
Click Enter
Click Up as fast as you can before animation finish

Expected behavior
Focus change one by one. Not skipping anything.
Screenshots
https://user-images.githubusercontent.com/7922655/224882026-2836b39d-3c25-4453-9901-105a34a08b01.mp4
NuGet package version
None
Windows version
Windows 10 (21H2): Build 19044
Additional context
Xbox gamepad have the same behavior too.

I was unable to repro this in the latest WinUI3 Gallery. If you are still seeing this there, please leave a comment.

@ojhad I can still see this using the latest WinUI3 Gallery. You should press up or down just after pressing enter. The focus will skip the icon before the animation finish.


[question]  find xsd by xsi:schemaLocation

its possible validate xml files usign the xsi:schemaLocation attribute from the xml file?
I have an xml which has this attribute but also has a tag with the very same attribute.
The principal tag, points to one xsd, I have downloaded and I'm using it for validate but when I run my node project, always shows error in the child tag which have the xsi:schemaLocation pointing to another xsd, so....
Its possible put multiple xsd files to validate one xml or its possible just put null the second parametter and let search the xsd online?
<?xml version="1.0" encoding="utf-8"?>
<cfdi:Comprobante ... xsi:schemaLocation="http://www.sat.gob.mx/cfd/3 http://www.sat.gob.mx/sitio_internet/cfd/3/cfdv33.xsd" Version="3.3" ...... >
   .......
   <tfd:TimbreFiscalDigital xsi:schemaLocation="http://www.sat.gob.mx/TimbreFiscalDigital http://www.sat.gob.mx/sitio_internet/cfd/TimbreFiscalDigital/TimbreFiscalDigitalv11.xsd" >......</tfd:TimbreFiscalDigital>
</cfdi:Comprobante>

To my knowledge this is not possible at the moment. We expect a schema to be passed that we use for validation.
Contributions to improve the current state are welcome. I'll not personally be able to follow up on this one unfortunately though.


github: check token scopes even if authorized

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

A homebrew-cask user attempted to submit a bump PR but got this error because their personal access token did not have the public_repo scope:
Error: Unable to fork: Not Found!

This PR changes utils/github to check token scopes even if the token is authorized, which results in this more helpful error message:
Error: Your macOS keychain GitHub credentials do not have sufficient scope!
Scopes required: public_repo
Scopes present:  none
Create a personal access token:
  https://github.com/settings/tokens/new?scopes=gist,public_repo&description=Homebrew
echo 'export HOMEBREW_GITHUB_API_TOKEN=your_token_here' >> ~/.zshrc
Error: Unable to fork: Not Found!

The authorization check was added in https://github.com/Homebrew/brew/commit/6135da800e4f93ded3a5a331d0efa691f4822afb
Related:  https://github.com/Homebrew/homebrew-cask/issues/97297

Review period will end on 2021-01-11 at 01:48:24 UTC.

Review period will end on 2021-01-11 at 01:48:24 UTC.

Good call 👏🏻

Good call 👏🏻

Review period skipped due to critical label.

Review period skipped due to critical label.


Bug fix waitingMTMVTaskFinished failed at test_mtmv_ssb_ddl

Proposed changes
fix waitingMTMVTaskFinished failed at test_mtmv_ssb_ddl
Exception in mtmv_p0/ssb/test_mtmv_ssb_ddl.groovy(line 33):

    }

    def ssb_mtmvs = ["ssb_flat", "ssb_q11", "ssb_q12", "ssb_q13", "ssb_q21", "ssb_q22", "ssb_q23",
                     "ssb_q31", "ssb_q32", "ssb_q33", "ssb_q34", "ssb_q31", "ssb_q42", "ssb_q43"]

    for (String mvName in ssb_mtmvs) {
        sql "drop MATERIALIZED VIEW IF EXISTS ${mvName}"
        println "run mtmv ddl: ${mvName}"
        sql new File("""${context.file.parent}/ddl/${mvName}_create.sql""").text
        waitingMTMVTaskFinished(mvName)
^^^^^^^^^^^^^^^^^^^^^^^^^^ERROR LINE^^^^^^^^^^^^^^^^^^^^^^^^^^
    }
}

Exception:
org.junit.ComparisonFailure: expected:<[SUCCESS]> but was:<[PENDING]>
  at org.junit.Assert.assertEquals(Assert.java:117)
  at org.junit.Assert.assertEquals(Assert.java:146)

Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

LGTM

run buildall


0.5 warning and test fixes

Builds on top of @yuyichao's 0.6 fixes branch.

Changes Unknown when pulling dd3ea4e905695275ad6506d54f7366327af8ea7c on s/0.5-fixes into ** on master**.

Changes Unknown when pulling dbb918fea67bb4578b612fb87d743246f94f198c on s/0.5-fixes into ** on master**.


Add version requirements

Coverage remained the same at 100.0% when pulling 8fcba0a26f0f252f23d136f7d5372474405405e1 on installation-instructions into f2fa950f8a4b9cac17aa63976107030659dd4468 on master.


Upload file error: EOF

An EOF error occurs when I upload a file that is larger than about 1GB（local mode）, even though the file is still sent successfully. This doesn't happen when I send files around 500MB. The POST request returns to the EOF about halfway through the file, but the request has been interrupted. After the result is returned, the file is still sent by the API.

This error is related to the time it takes the API to process the request. If multiple files are sent at the same time, but each file is sent slowly, this error can also occur on small files.Is this because the API forcibly pinches long time connections?

The err is "POST http://localhost:8081/botapi/method(such as sendvideo) EOF".
This error occurs in the middle of sending a file that is larger than about 1G(file URL). I use Golang to write my robot program, and this error often occurs when a server disconnects or for some other reason accidentally disconnects(in other Go program).
I wrote a simple file receiver server and sent more than 2GB files or connected for more than 1 hour to test whether the client was working properly, send a POST request to my own server and the error did not happen (there was no actual function, just the completion of the request).

Bot API server can't return anything called "EOF".

Yes, this error is caused by my program.But why does my program produce this error? Because the connection dropped unexpectedly. And it's a local connection break, there's no complex routing problem. And if the server is replaced by my own test server, there will be no errors. So I suspect the tg-bot-api forced the disconnect.

Bot API server has timeout of 600 seconds for reading incoming requests. It is very unlikely that sending a local file could take so much time.

Probably is after this time has produced the error. When I send a 1.5G file at 1 M/s, it takes about 1500 seconds.In my case, the transmission speed is between 1M/s and 6M/s, so more than 600 seconds is very likely.

The upload speed needs to be improved.
Between, you can send a file by a local path in a local mode.

I already send my file by a local path, but it's slowly。My server is in the Netherlands，and when I send data to the api.telegram.org, it can reach more than 30M/s. Maybe the local API will be speed-limited, just like a user account?

I will try to change the timeout time to a suitable range to solve this problem. Thank you for your answer.
I also hope the authorities provide some kind of interface to change the timeout if you think it is feasible.

No, local API isn't speed limited. This depends solely on your network connection.

OK, thank you very much.


An error occurred while Style/ExtraSpacing cop was inspecting

1 error occurred:
An error occurred while Style/ExtraSpacing cop was inspecting /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/app/views/devise/registrations/edit_users_permissions.html.haml.haml_lint.tmp20160627-93752-vo0mai.
Errors are usually caused by RuboCop bugs.
Please, report your problems to RuboCop's issue tracker.
Mention the following information in the issue report:
0.34.2 (using Parser <IP_ADDRESS>, running on ruby 2.2.2 x86_64-darwin14)
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
/Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml: Style/SpaceBeforeFirstArg has the wrong namespace - should be Lint
Warning: unrecognized cop Style/ConditionalAssignment found in /Users/kieranklaassen/Dropbox (REC Sound Design BV)/rails/occasion/.rubocop.yml
undefined method `children' for false:FalseClass
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/linter/class_attribute_with_static_value.rb:35:in `block in contains_class_attribute?'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/linter/class_attribute_with_static_value.rb:28:in `each'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/linter/class_attribute_with_static_value.rb:28:in `contains_class_attribute?'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/linter/class_attribute_with_static_value.rb:19:in `visit_tag'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:18:in `visit'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `block in visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `each'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:23:in `visit'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `block in visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `each'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:23:in `visit'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `block in visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `each'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:23:in `visit'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `block in visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `each'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:27:in `visit_children'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/haml_visitor.rb:23:in `visit'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/linter.rb:28:in `run'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/runner.rb:58:in `block in collect_lints'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/runner.rb:57:in `map'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/runner.rb:57:in `collect_lints'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/runner.rb:21:in `block in run'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/runner.rb:20:in `map'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/runner.rb:20:in `run'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/cli.rb:82:in `scan_for_lints'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/cli.rb:51:in `act_on_options'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/lib/haml_lint/cli.rb:23:in `run'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/gems/haml_lint-0.15.0/bin/haml-lint:7:in `<top (required)>'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/bin/haml-lint:23:in `load'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/bin/haml-lint:23:in `<main>'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/bin/ruby_executable_hooks:15:in `eval'
/Users/kieranklaassen/.rvm/gems/ruby-2.2.2/bin/ruby_executable_hooks:15:in `<main>'
Report this bug at https://github.com/brigade/haml-lint/issues

To help fix this issue, please include:
- The above stack trace
- Haml-Lint version: 0.15.0
- Haml version: 4.0.7
- RuboCop version: 0.34.2
- Ruby version: 2.2.2

Hey @kieranklaassen, thanks for the report.
A small code sample that reproduces the issue would be invaluable in helping solve this. Thanks!
Also, does the issue occur with the latest version of HAML-Lint? (0.17.1 at time of writing)

Closing due to lack of response. Feel free to reopen with a code sample!


[bug] print_monitor_cv.py model not iterable

I've tried print_monitor_cv.py model.pkl but I've got
Traceback (most recent call last):
  File "~/pylearn2/pylearn2/scripts/print_monitor_cv.py", line 84, in <module>
    main(**vars(args))
  File "~/pylearn2/pylearn2/scripts/print_monitor_cv.py", line 38, in main
    for model in list(this_models):
TypeError: 'MLP' object is not iterable

so I changed this part:
        this_models = serial.load(filename)
        for model in list(this_models):
            # ...

to
        # ....
        this_models = serial.load(filename)

        try:
            this_models = list(this_models)
        except TypeError:
            this_models = [this_models]

        for model in this_models:
            # ...

PR?

At first, I was not sure why you would use print_monitor_cv.py on a pkl file with only one model inside, but it makes sense if you specify more than one pkl, and there is already a special case when there is only one model at all.
Then, I think such a PR would be a good idea. Instead of relying on TypeError to check if this_models is iterable or not, maybe you can use something like:
if not isinstance(this_models, collections.Iterable):
    this_models = [this_models]
for model in this_models:
    ...

Ask forgiveness not permission? :)
Ok, I'll do that.


Change Text size

Hi, how do I change textsize?

@sicksand This feature will reflect in next update mostly tomorrow night. I will ping you once it is done.

Cool! Will close this.

I will close this once it's done.

closing this for now .this will be tracked with #38 issue which is like superset to this


Implement catApiService.dynamicEndpoint similar to catApiServiceProvider.endpo...

... similar to catApiServiceProvider.endpoint for dynamic endpoint definitions.
We need dynamic endpoints that cannot be created in the config phase and don't fit to the child endpoint idea either.

See Issue #5 ;-)


[DOC-10371] Add unit to cpu-profile-duration flag

[DOC-10371] Add unit to cpu-profile-duration flag
Ref: https://cockroachlabs.slack.com/archives/C01PHNMUFLN/p1716909335135489
Previews:
src/current/v23.1/cockroach-debug-zip.md
src/current/v23.2/cockroach-debug-zip.md
src/current/v24.1/cockroach-debug-zip.md

He gave the solution with the report. :)


escape Unicode sequences when copying bookmarks

When trying to import just exported string which contains portals with Unicode names, Bookmarks fails to decode the JSON and I get the 'Import failed' error.
This patch fixes it by properly escaping all Unicode characters on import.

Up. Do you want to me change anything to get it merged?


Simplifying dimension merging

Currently, dimension merging is processed by two stages. One for dictionary, one for index. If this can be processed by  single stage, total processing time could be deceased.

Reduced dim conversion time from 11.6 sec to 7.1 sec, with 12 index with 500K rows each.

:+1:

:+1: This looks good to me, but I think someone else familiar with this code needs to do a review

+1

can you squash the commits ?
If no one have any question i will merge.

@binlijin squashed. thanks.

@binlijin do you mind holding off on merging? I just want 1 more pair of eyes to review this
@xvrl can you take a look?

@fjy ok

@binlijin It's becoming more and more painful to rebase.  Do you really have a time to look into this?

@navis, i am ok with the PR，but fj tell need @xrvl to take a look.

@navis, i am ok with the PR，but fj tell need @xrvl to take a look.

@navis yes, there's multiple teams working on the same code, which is why we need proposals to coordinate, and all the changes are important

@navis
Looks good to me, I don't have any additional comments beyond what @fjy has already noted.
Can you rebase? I'll do another review pass on this after.

Rebased on master, barely. I'll address comments.

:+1: looks good to me after addressing the Seeker renaming comments

@himanshug @xvrl do you want to take a look? I think this is getting close to ready and will merge unless there's more comments

:+1:  pls squash.

squashed


[BOT PR] to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: istio-testing
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
904a4b62c57197e9edb0d4dec548949d753307ab
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
904a4b62c57197e9edb0d4dec548949d753307ab
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
904a4b62c57197e9edb0d4dec548949d753307ab
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
904a4b62c57197e9edb0d4dec548949d753307ab
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
904a4b62c57197e9edb0d4dec548949d753307ab
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
904a4b62c57197e9edb0d4dec548949d753307ab
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


 We should aim to fully use vibe.d-json and fast json for everything possible, and remove any code that uses std.json.

done


added release of meshery v0.3.11

fixes #766
Signed-off-by: ashiscs<EMAIL_ADDRESS>

Ready to be merged @leecalcote sir!!


Option unwrap on none value

Hi I got this error when doing merge-feature-branches
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /Users/runner/work/lezeh/lezeh/lezeh-deployment/src/lib/cli.rs:167:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

no clue on which repo or which pull request but some pull requests got merged anyway.

Hi @prildy ,
looking at the error log, are u building/running lezeh directly from the source code? at what commit hash did u run lezeh? The error is because I mistakenly force unwrapping optional, if it's confirmed (the same issue that I pointed) then it should be fixed on master and 0.8.2
https://github.com/sendyhalim/lezeh/blob/master/lezeh-deployment/src/lib/cli.rs#L167

Just discussed in private that the bug is from 0.8.0 which should be fixed by 0.8.2, I'm closing this issue for now. Feel free to reopen.


Terminal accessibily help screen isn't read out to user

macos
enable voice over
trigger the terminal accessibility help view F1 > Show Terminal Accessibility Help
the contents of the help screen isn't read out, only a short summary

(screen capture below shows what's being read by voice over)

/duplicate #173911


--list flag accepts anything as user input

[x] Using latest version as provided on the master branch
[x] Searched for similar issues including closed ones

What is the purpose of your issue?

[x] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description

You can pass anything from an image to spotdl.py itself which may result in unwanted or even dangerous behavior, especially if the file is getting deleted after being processed.
At the very least have this file_path.lower().endswith(('.txt'))
If you want to go further, use regex to see if there are spotify links, if you want to support text input also then that could be an additional feature.

I looked into this issue a bit, and I found that this can be done in 2 ways using the python-magic library.

I can add a type=txt_mime parameter when we define the argument:
group.add_argument("-l", "--list", type=txt_mime, help="download tracks from a file")

and have a function txt_mine() defined as:
def txt_mime(arg):
    """ Checks whether arg is a mime of a text file. Raises ValueError if not. """
    if magic.from_file(arg, mime=True) != "text/plain":
        raise ValueError("{0} is not of a valid file format".format(arg))

    return arg

So the error message of a bad input (Example: spotdl.py) would be:
usage: spotdl.py [-h] . . .
spotdl.py: error: argument -l/--list: invalid txt_mime value: 'spotdl.py'

After the arguments have been parsed, I can check with an if statement:
file = parsed.list
if magic.from_file(file, mime=True) != "text/plain":
    raise argparse.ArgumentTypeError("{0} is not of a valid file format".format(file))

So the error message of a bad input (Example: spotdl.py) would be:
Traceback (most recent call last):
File "spotdl.py", line 24, in <module>
     raise argparse.ArgumentTypeError("{0} is not of a valid file format".format(file))
argparse.ArgumentTypeError: spotdl.py is not of a valid file format

The benefit of the 2nd method is that we can output more detail about the error, whereas the first method returns the same error message regardless of what arguments I put into ValueError().
Would we prefer the 2nd method? Or is there another way to go about warning the user of a bad input type?

I like the 2nd method but instead of raise argparse.ArgumentTypeError we could use parser.error.

@ritiek issue can be closed


503

https://nodenormalization-sri.renci.org/get_normalized_nodes?curie=CHEMBL.COMPOUND:CHEMBL76729

Hi @edeutsch  have you seen this recurrently?  I haven't been able to reproduce it.

it was down for several hours at the time of posting. It does seem to be fine again now, thanks.


[SPARK-38215][SQL] InsertIntoHiveDir should use data source if it's convertible

What changes were proposed in this pull request?
Currently spark sql
INSERT OVERWRITE DIRECTORY 'path'
STORED AS PARQUET
query

can't be converted to use InsertIntoDataSourceCommand, still use Hive SerDe to write data, this cause we can't use feature provided by new parquet/orc version, such as zstd compress.
spark-sql> INSERT OVERWRITE DIRECTORY 'hdfs://nameservice/user/hive/warehouse/test_zstd_dir'
         > stored as parquet
         > select 1 as id;
[Stage 5:>                                                          (0 + 1) / 1]22/02/15 16:49:31 WARN TaskSetManager: Lost task 0.0 in stage 5.0 (TID 5, ip-xx-xx-xx-xx, executor 21): org.apache.hadoop.hive.ql.metadata.HiveException: java.lang.IllegalArgumentException: No enum constant parquet.hadoop.metadata.CompressionCodecName.ZSTD
	at org.apache.hadoop.hive.ql.io.HiveFileFormatUtils.getHiveRecordWriter(HiveFileFormatUtils.java:249)
	at org.apache.spark.sql.hive.execution.HiveOutputWriter.<init>(HiveFileFormat.scala:123)
	at org.apache.spark.sql.hive.execution.HiveFileFormat$$anon$1.newInstance(HiveFileFormat.scala:103)
	at org.apache.spark.sql.execution.datasources.SingleDirectoryDataWriter.newOutputWriter(FileFormatDataWriter.scala:120)
	at org.apache.spark.sql.execution.datasources.SingleDirectoryDataWriter.<init>(FileFormatDataWriter.scala:108)
	at org.apache.spark.sql.execution.datasources.FileFormatWriter$.org$apache$spark$sql$execution$datasources$FileFormatWriter$$executeTask(FileFormatWriter.scala:269)
	at org.apache.spark.sql.execution.datasources.FileFormatWriter$$anonfun$write$1.apply(FileFormatWriter.scala:203)
	at org.apache.spark.sql.execution.datasources.FileFormatWriter$$anonfun$write$1.apply(FileFormatWriter.scala:202)
	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
	at org.apache.spark.scheduler.Task.run(Task.scala:123)
	at org.apache.spark.executor.Executor$TaskRunner$$anonfun$10.apply(Executor.scala:408)
	at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1360)
	at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:414)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

Why are the changes needed?
Convert InsertIntoHiveDirCommand  to InsertIntoDataSourceCommand can support more features of parquet/orc
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added UT

Gentle ping @cloud-fan  @viirya Could you take a review? it's a useful feature.

Also ping @dongjoon-hyun @HyukjinKwon

Gentle ping @cloud-fan GA passed.

thanks, merging to master!

@AngersZhuuuu Hi，in the case of inserted dir has same path as selected table location, this may cause error. https://issues.apache.org/jira/browse/SPARK-38215


Fixed #26296 -- fixed error message for second values() call after ex…

Fix of https://code.djangoproject.com/ticket/26296#ticket
Need some grammar checking (not native speaker).

Thanks for the patch, could you also submit a test for extra -> extra_select change?

I adding a test in https://github.com/django/django/pull/6273 but noticed this change results in a regression in another unit test: tests/many_to_one/tests.py. Since extra() is discouraged and the docs say we aren't fixing bugs with it, I'll just close this rather than sink more time into it. Thanks for trying!