Kitty no longer tiled

Hey guys,
This stopped working for me after a restart, I imagine apple made some sort of update. Really a big part of my workflow. How can I get this to a state where its working again?
After some looking around on some other threads there was a solution where kitty window was too small to be picked up by yabai,
changed initial kitty window from 800X600 to 2400X2400, still not managed
I installed from HEAD.
My Mac-os version:
12.1

Followed all sip instructions. I also tried to delete the tmp/yabai* but was unable to locate it.

No problems with Kitty here.  I just upgraded from 12.1 to 12.2 this morning and everything is still fine.

Hey Mantis, can you elaborate on what steps you followed to get this working?

That's just the thing.  I wish I could be of more help but I didn't need to do anything special.  I do run Yabai from HEAD, using brew to install/update/build, but it just works.
I just poked around in your dotfiles and I don't see anything that looks off, but have you tried with empty kitty.conf and .yabairc files?
All I do for kitty is load a color theme and set my font
https://github.com/Liquidmantis/dotfiles/blob/main/kitty/kitty.conf
My yabairc is straightforward, too
https://github.com/Liquidmantis/dotfiles/blob/main/yabai/yabairc

also not tiling Kitty for me... didn't change any yabairc or kitty.conf lately. seems like a macOS update broke something
kitty 0.24.4 created by Kovid Goyal

yabai-v4.0.0

skhd version 0.3.5

System Integrity Protection status: unknown (Custom Configuration).

Configuration:
	Apple Internal: disabled
	Kext Signing: enabled
	Filesystem Protections: disabled
	Debugging Restrictions: disabled
	DTrace Restrictions: enabled
	NVRAM Protections: enabled
	BaseSystem Verification: enabled

This is an unsupported configuration, likely to break in the future and leave your machine in an unknown state.

No idea, works fine for me on both my MacBooks.

Having same problem too, yabai can't manage my kitty windows.


no such table gpkg_extensions

when writing the project to geopackage I get an error
Traceback (most recent call last):
File ".qgis2/python/plugins\QgisGeopackage\qgis_geopackage.py", line 112, in write
gpkg.write(project_path)
File ".qgis2/python/plugins\QgisGeopackage\qgpkg.py", line 208, in write
'INSERT INTO gpkg_extensions VALUES (?,?,?,?,?)', extensions)
OperationalError: no such table: gpkg_extensions
Maybe the plugin should create this table if it detects it is not available?

Fixed in 9c00ad6


SITK_4D_IMAGES is broken

@kaspermarstal reported
https://github.com/SimpleITK/SimpleITK/commit/adc03d972935f348d16ae16139f2fec611cd44aa#commitcomment-22107008
The recent changes to rename user exposed CMake variable to constantly have the SimpleITK prefix over SITK has broken the 4D image option.

closed by #177


\footcites leads to an error

Consider this MWE
\documentclass{article}
\usepackage{filecontents}
\begin{filecontents*}{\jobname.bib}
@Book{test1,
  author = {A. Author},
  date = {2018}
}
@Book{test2,
  author = {B. Buthor},
  date = {2017}
}
@Book{test3,
  author = {C. Cuthor},
  date = {2017}
}
\end{filecontents*}
\usepackage[%
 backend=biber,
 bibstyle=ext-authortitle,
 citestyle=ext-authoryear,
]{biblatex}
\addbibresource{\jobname.bib}
\begin{document}

\footnote{\cites%
[1--3]{test1}%
[4--6]{test2}%
[7--10]{test3}}

% Does not work
\footcites%
[1--3]{test1}%
[4--6]{test2}%
[7--10]{test3}

\end{document}

It seems like \footcites is not working !?

Thank you very much for the very quick debug! It works very well! So I will close this issue...


Add Dockerfile

Added Dockerfile to build c3po NF images
Added Makefile targets to build and push images
Added install_deps.sh
Fixes: #8, #9
Signed-off-by: Saikrishna Edupuganti<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@stolsma Thanks for reviewing!! Updated :)

retest this please

Can we merge this patch?

@omec-project/c3po-maintainers

retest this please

retest this please

@suresh-marikkannu this is dependent on #16 and #27

@krsna1729 Here is the standard docker build method we use in CORD project. Please update Makefile accordingly so that docker images in dockerhub can be updated automatically whenever new patch is merged or new version is released. https://guide.opencord.org/developer/test_release_software.html#publish-docker-container-images-to-public-dockerhub-job-docker-publish

retest this please

@hyunsun i think this and makefile pull request are failing because the CI needs to be updated to use .ci scripts in this repo instead of doing somehting manual.

https://github.com/omec-project/omec-project-ci/pull/36

@krsna1729 Thanks. I really hope this PR gets merged as soon as possible.


Support GHE when apiUrl points to it

What was fixed?  (Issue # or description of fix)
Closes #193

much cleaner! nice job


Single Linked Lists

This repository is for non-code contributions, you can contribute this to any repo related to CP.


Does not compile with new pytorch versions

pip3 install torch==1.10.0+cu113 torchvision==0.11.1+cu113 torchaudio==0.10.0+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html
conda install pytorch=1.10 torchvision torchaudio cudatoolkit=11.3 -c pytorch
conda install pytorch==1.9 torchvision torchaudio cudatoolkit=11.1 -c pytorch -c conda-forge -y

FAILED: /home/Desktop/Mesa/build/temp.linux-x86_64-3.7/native.o
c++ -MMD -MF /home/Desktop/Mesa/build/temp.linux-x86_64-3.7/native.o.d -pthread -B /home/miniconda3/envs/mesa2/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I/home/miniconda3/envs/mesa2/lib/python3.7/site-packages/torch/include -I/home/miniconda3/envs/mesa2/lib/python3.7/site-packages/torch/include/torch/csrc/api/include -I/home/miniconda3/envs/mesa2/lib/python3.7/site-packages/torch/include/TH -I/home/miniconda3/envs/mesa2/lib/python3.7/site-packages/torch/include/THC -I/home/miniconda3/envs/mesa2/include/python3.7m -c -c /home/Desktop/Mesa/native.cpp -o /home/Desktop/Mesa/build/temp.linux-x86_64-3.7/native.o -DTORCH_API_INCLUDE_EXTENSION_H '-DPYBIND11_COMPILER_TYPE="_gcc"' '-DPYBIND11_STDLIB="_libstdcpp"' '-DPYBIND11_BUILD_ABI="_cxxabi1011"' -DTORCH_EXTENSION_NAME=native -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++14
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
/home/Desktop/Mesa/native.cpp: In function ‘at::Tensor gelu_backward_cpu(const at::Tensor&, const at::Tensor&)’:
/home/Desktop/Mesa/native.cpp:92:24: error: ‘gelu_backward_cpu’ is not a member of ‘at::native’; did you mean ‘glu_backward_cpu’?
   92 |     return at::native::gelu_backward_cpu(grad_output, input);
      |                        ^~~~~~~~~~~~~~~~~
      |                        glu_backward_cpu
/home/Desktop/Mesa/native.cpp: In function ‘at::Tensor gelu_backward_cuda(const at::Tensor&, const at::Tensor&)’:
/home/Desktop/Mesa/native.cpp:98:24: error: ‘gelu_backward_cuda’ is not a member of ‘at::native’; did you mean ‘glu_backward_cuda’?
   98 |     return at::native::gelu_backward_cuda(grad_output, input);
      |                        ^~~~~~~~~~~~~~~~~~
      |                        glu_backward_cuda
/home/Desktop/Mesa/native.cpp: In function ‘at::Tensor softmax_backward_cpu(const at::Tensor&, const at::Tensor&, int64_t, const at::Tensor&)’:
/home/Desktop/Mesa/native.cpp:190:24: error: ‘softmax_backward_cpu’ is not a member of ‘at::native’; did you mean ‘col2im_backward_cpu’?
  190 |     return at::native::softmax_backward_cpu(grad_output, output, dim, self);
      |                        ^~~~~~~~~~~~~~~~~~~~
      |                        col2im_backward_cpu
/home/Desktop/Mesa/native.cpp: In function ‘at::Tensor softmax_backward_cuda(const at::Tensor&, const at::Tensor&, int64_t, const at::Tensor&)’:
/home/Desktop/Mesa/native.cpp:198:24: error: ‘softmax_backward_cuda’ is not a member of ‘at::native’; did you mean ‘col2im_backward_cuda’?
  198 |     return at::native::softmax_backward_cuda(grad_output, output, dim, self);
      |                        ^~~~~~~~~~~~~~~~~~~~~
      |                        col2im_backward_cuda
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
  File "/home/miniconda3/envs/mesa2/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 1723, in _run_ninja_build
    env=env)
  File "/home/miniconda3/envs/mesa2/lib/python3.7/subprocess.py", line 512, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['ninja', '-v']' returned non-zero exit status 1.

These versions seem to work:
conda install pytorch==1.8.0 torchvision==0.9.0 torchaudio==0.8.0 cudatoolkit=10.2 -c pytorch
conda install pytorch==1.8.0 torchvision==0.9.0 torchaudio==0.8.0 cudatoolkit=11.1 -c pytorch -c conda-forge
pip install torch==1.7.1+cu101 torchvision==0.8.2+cu101 torchaudio==0.7.2 -f https://download.pytorch.org/whl/torch_stable.html

Hi @styler00dollar,
Thanks for using Mesa and reporting this issue.
Our experiments reported in the paper are based on PyTorch 1.7 with CUDA 10.1. I have also successfully built Mesa with PyTorch 1.8 and PyTorch 1.9. However, it seems PyTorch 1.10 has changed its C++ API for GELU and Softmax. We will investigate this issue further.
Kind regards.

Hi @styler00dollar，
This is an update on this issue. We have fixed the compatibility with PyTorch 1.10 and successfully built Mesa with it. Please do not hesitate to contact us if you have any further issues.
Cheers,
Zizheng

Quick testing shows that it does indeed compile on 1.10 and 1.9 now. The above commands work. Thanks.


ScrollToFixed generating extra div problem

Hi, firstly I'd like to say that the plugin is great and thank you ver much for sharing.
However I am having a slight problem with it, I have seem to be having an extra div being generated when jthe function assigns the 'fixed' position attribute.
You can takek a look at the problem here: http://faiththeagency.com/easyphone/frequently-asked-questions.php
Not sure if anyone else has had this problem at all?
Could someone please help?

The referenced resources must have been cached as the fiddle is no longer working due to the original url no longer being available (http://youreasyphone.com/frequently-asked-questions.php).  Too many errors due to missing references.  If you implement what I posted in the first comment, you should be fine.

Thank you very much for your response, yes as you can tell I've had a bit of a change around and a problem with where the site is hosted. I'll give your solution a try asap.
Thanks again.

Hi bigspotteddog,
I have applied your workaround and it works perfectly! Thank you again for your help.
Danny.

Cool!  Glad I could help.  Best of luck.

The below one worked for me.
//Scroll to fixed sidebar
$("#fixed-sidebar-wrapp").scrollToFixed();
//Remove extra div
$("#fixed-sidebar-wrapp").next().remove();


wpan0 interface is not up

I am setting up ot-br-posix on raspberry pi.
After completing all steps described in the border router build guide ( https://openthread.io/guides/border-router/build). My wpan0 interface is not up.
I have connected RCP and it is visible as /dev/ttyACM0 interface.
Is there any step to start wpan0 interface?

There isn't extra step to start the wpan0 interface. Could you verify if otbr-agent service is running?
service otbr-agent status

No, it is not runnig.
Below is the output of it:
lines 1-6/6 (END)   otbr-agent.service - Border Router Agent    Loaded: loaded (/lib/systemd/system/otbr-agent.service; enabled; vendor preset: enabled)    Active: activating (auto-restart) (Result: exit-code) since Mon 2022-01-24 14:52:07 IST; 4s ago   Process: 27468 ExecStartPre=/usr/sbin/service mdns start (code=exited, status=0/SUCCESS)   Process: 27491 ExecStart=/usr/sbin/otbr-agent $OTBR_AGENT_OPTS (code=exited, status=1/FAILURE)  Main PID: 27491 (code=exited, status=1/FAILURE)

Have you ever tried https://openthread.io/codelabs/openthread-border-router?

No.
Let me try it and get back to you on this.

hi,
I have tried that guide freshly but still I am facing the same error.

Could you share your syslog of otbr-agent?
For example,
tail -n 1000 /var/log/syslog | grep otbr-agent

Closing stale issue.


Importing distutils.spawn raises DeprecationWarning

nco.nco imports distutils.spawn so it can find the path to an executable. This now raises a DeprecationWarning. disutils.spawn.find_executable can be replaced with shutil.which()

I'll take that as a "No". I just pushed this change to the repository. Should probably make a new patch release. Will wait a few weeks to see if this change passes tests and does not cause any immediate problems for users.

This change fails the self-test for py3.7 and succeeds on all later Python versions. Probably should eliminate the py37 test.

Fixed by #73


Deprecate old scope management APIs

We have  soft-deprecated sentry_sdk.push_scope and sentry_sdk.configure_scope in Sentry SDK v2.0.0 by asking users to move away from these methods. It is time for us to officially deprecate these functions.

All subtasks complete


Timer Feature

I have implemented the timer feature in my local. I can raise a PR if you create a Branch for me and allow me to raise a PR. Thanks

@SENTHILnew  - Thanks for adding the timer feature. This project is public/open source repo, you should be able to create a branch and create PR. Let me know if you face any issues.

I am not able to create a branch

I found the branch you created (add-timer) Ill use that. Thanks

yes, you can use add-timer branch

Can you fork the repo and create the PR to merge into this repo

#93


Wrong lint results when running lint after moving fragment declaration to another file

Issue looks very similar to https://github.com/dimaMachina/graphql-eslint/issues/1232. I"m running eslint.lintFiles("Queries") repeatedly on a changing set of graphql files. It seems that when a fragment is moved from one file to another between runs, the subsequent runs fail e.g. unknown fragment check.
It looks like the following line keeps a global cache which affects subsequent runs: https://github.com/dimaMachina/graphql-eslint/blob/master/packages/plugin/src/siblings.ts#L29
Is there any way to reset that cache?
Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[ ] 1. The issue provides a reproduction available on GitHub, Stackblitz or CodeSandbox

For example, you can start off by editng the
'basic' example on Stackblitz.

Please make sure the graphql-eslint version under package.json matches yours.

[ ] 2. A failing test has been provided

[ ] 3. A local solution has been provided

[ ] 4. A pull request is pending review

Describe the bug

To Reproduce Steps to reproduce the behavior:

Expected behavior

Environment:

OS:
@graphql-eslint/eslint-plugin:
Node.js:

Additional context

Might be related to https://github.com/dimaMachina/graphql-eslint/issues/2511 ?


Fix parse problem in EwsUtilities

This is a fix for parsing Strings into different types. The isAssignableFrom() usages had the classes the wrong way round. This is fine when the classes are the same, but not for eg Number and Integer. This error caused the ExchangeServerInfo parsing from a SOAP header to fail.

Hi @tomravn, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@tomravn, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

@tomravn are you planning any changes?

Sorry, I got pulled onto another project for a few days.
I've added the extra tests.
Before that though I made the project OSGi ready since we're using it in an OSGi container and needed it to have the correct MANIFEST.MF entries. I'm a bit new to pull requests but can see now that I should have made the related changes on their own branch and setup a pull request from there instead! I can see from your history that someone previously made the project OSGi ready and it was merged in but later reverted although I couldn't see why. This bundle has been tested in Apache Karaf and works fine so maybe it's worth merging that as well?

We've just fixed this as well. We also fixed all the other calls of isAssignableFrom() in that method, not only the one for Integer.class is broken. Could you fix those as well?

They were all changed in the first commit

Great, looking forward to the merge then :)

@tomravn thanks for the changes. plz note that we decided to not include osgi components to the main project as these should be handled on a different branch (e.q.). See #177 for further information and discussion. For that reason changing the dependencies should be handled with a distinct PR and may be included in a 2.X+ release. If you want to reopen discussion on that topic plz create or comment on the existing issues related to this topic.

@tomravn since there seems to be a little bit of urgency on this topic (due to several issue reports) will you devide the pr as mentioned? I think this should be merged within the next days.

created another PR during the lag of feedback given here. #305
Closing this for now.

Sorry for the lack of response. I'm not full time on our project that requires this.
I've moved our OSGi changes to a different branch and reverted them on the branch this PR was for. I know this is closed and there's now an alternative PR open but this is OK for merging if that's still possible

hi @tomravn thanks for your reply. The alternative has already been merged. Once we are fine with branching we will also release an osgi-version of ews-java-api


Allow for bower version --no-git-tag-version.

This is really poorly documented in npm, but it'd be nice to have the option to not tag when running bower version, a la npm version --no-git-tag-version. For any project with multiple manifests, I think it's nice to be able to have one commit and tag for a version bump.
The option isn't really all that descriptive since it's really more like --no-git-commit. --no-git-tag-version keeps it in step with npm though.

Really need this, why no response to this pr?

+1 on this functionality. It is very annoying when developing a build system when you cannot disable automatic git tagging and commits.

version field is now ignored by bower, it doesn't make sense to update version in bower.json without adding git tag


only run phpcs if the file has no syntax errors.

running phpcs on a file which contains a parse error generates a huge
number of warnings from the phpcs library.
You can see an example of what phpcs outputs in such cases here: https://gist.github.com/6c6535c3fc22c754ca6b
Therefore - don't run phpcs on files which have parse errors; don't freeze vim for minutes at a time when syntastic is already aware that the file is fundamentally borked

Pulled - good idea :-)

U iz fast :)


Many empty session entries generated

Node version: v12.14.1
"sails": "^1.2.3",
"sails-hook-apianalytics": "^2.0.3",
"sails-hook-organics": "^0.16.0",
"sails-hook-orm": "^2.1.1",
"sails-hook-sockets": "^2.0.0",
"sails-mongo": "^1.0.1",
"sails-postgresql": "^1.0.2",

Hello, I am using connect-mongo as session adapter
Any login (which stores a session internally) will generate tones of empty session entires, see attached screenshot is the result of login/logout/login.

I dont see any related issue in mongo-connect so I think this should be a sails issue. Have you ever encountered such problem? This will totally spam my session store.

Sidenote : this happen only in dev mode, not production

@LouAdrien Just a hunch- are you using Passport.js?

@LouAdrien Also, can you tell me a bit more about your prod environment and how you're doing SSL?  (also the app's sails.config.session.cookie.ssl and sails.config.http.trustProxy settings in dev vs. prod)
Thanks!


Column Header Tooltips

Column headers should provide the option to display definitions via tooltips on hover.

+1
You can use md-tooltip for the implementation of this.

+1


removed tiles background in starting

What does this PR do?
On the starting of the game the tiles have a box shadow which revealed the white background of the tiles.
Screenshots of changes made to the webapp.
I removed the box shadow and removed the add style line in the loop that added that box styling to all the other boxes.
I verified it by playing the game and it worked all okay as I just changed the styling not any actual logic.
Related PRs and Issues
It is related to the PR issue #92 which also was raised to remove the background of tiles. but removing the box shadow was alo important

Have you updated the README.md file?

[ ] yes
[x] no

Have you made any big breakthrough change?

[ ] yes
[x] no

Is the code beautified?

[x] yes
[ ] no

Is the code clear to understand woth proper variable naming and comments?

[x] yes
[ ] no

sorry I am new to contribution and learning and i hope i did all the things you asked for

Sure no probs. Thanks for the efforts


Support Postgresql

that would be great feature for more db support.

I thinking about create Nuget packages to support a wide range of databases. And who forks / download can choose whatever database he want.
But the database must be supported by EF Core.

Done! Now it's possible to use PostgreSQL


Fix #14

Suggest you to squash the commits, I added/fixed comments in 2 more commits ^^

It's actually a WIP, as I'm waiting for your feedback. If everything is OK for you, I'll add examples in the README.

@matcornic It looks like you're close.  Would you mind squashing them on your end once this is ready?

@matcornic It looks like you're close.  Would you mind squashing them on your end once it's ready?

@matcornic So far it's looking pretty good to me!

@jaytaylor squashed and ready !

@matcornic Way to go, this is excellent work!
Thank you, it's been a pleasure.
Fixes #14.


Get list of recent chats

Is this possible?

It's not currently supported in osa-imessage but it should be possible to add by querying the ~/Library/Messages/chat.db database. I don't have a ton of time to write that feature but PRs are always welcome.


Maintenance work

React Boilerplate
Self-explanatory... Updated all the outdated dependencies.

I changed the history imports following a warning saying our usage was deprecated.
I also removed eslint-plugin-compat (which was a recent addition). In v3, instead of being able to tell the plugin about your polyfills in a concise way (i.e. fetch, promise), you now have to specify individual methods so it becomes extremely verbose and I have a feeling it will be confusing to many users.

Coverage remained the same at 99.669% when pulling 3354296b002821be37290cafb86f00e5889b1a6e on maintenance into 434a12a82d1ef3bc28c52f4d2b0fc023bd88dcb2 on dev.

Coverage remained the same at 99.669% when pulling 3354296b002821be37290cafb86f00e5889b1a6e on maintenance into 434a12a82d1ef3bc28c52f4d2b0fc023bd88dcb2 on dev.


Add Large Tensor Test for linalg_syrk

Description
Check if syrk_batch works correctly with large tensors. This test will fail with the current code base; https://github.com/apache/incubator-mxnet/pull/18752 should fix it.
TODO:

make test function naming consistent with other large tensor tests (need to rebase after any of those tests are merged)
merge this only after the fix has been merged

This test passes on both BLAS int32 and 64 builds.
ubuntu@ip-172-31-43-103:~$ MXNET_TEST_COUNT=10000 nosetests --logging-level=DEBUG --verbose -s mxnet/tests/nightly/test_large_array.py:test_linalg_operators
test_large_array.test_linalg_operators ... [23:14:57] ../src/storage/storage.cc:198: Using Pooled (Naive) StorageManager for CPU
ok

----------------------------------------------------------------------
Ran 1 test in 637.245s

OK

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@Zha0q1 can you fix this issue with your PR "This branch is out-of-date with the base branch"

@Zha0q1 can you fix this issue with your PR "This branch is out-of-date with the base branch"

Fixed.

Few comments ... overall code is good

Can you just check with smaller input run and let me know the results. That should be good enough .... overall LGTM!

This has been tested with both small and large input tensors

@Zha0q1 also add your name to Contributors.md in the upcoming PR [let's not retrigger CI for that].
Thanks a lot for your contributions! :)

@Zha0q1 also add your name to Contributors.md in the upcoming PR [let's not retrigger CI for that].
https://github.com/apache/incubator-mxnet/blob/master/CONTRIBUTORS.md
Thanks a lot for your contributions! :)

will do!


Feature Request: Update URL with file path.

For example, if I'm looking at coverage/index.html and I open up the to_do.rb file to see the coverage, I would like the URL to be updated to reflect the specific file that I'm looking at. Something like coverage/index.html#specs/models/to_do.rb.
This would allow two main things:

A history so you can go back on the browser to previous files, etc.
Reload the page after re-running the specs for that file and see the updated coverage.

Thoughts?

I take it back, sorry! This is using a unique(ish) SHA1 of the file path, so reloading the page works just fine. Closing this now. Thanks.


cd.yml: remove non-existing job output

The api_version output was added in a previous commit and is
no longer used.
Signed-off-by: Martin Vrachev<EMAIL_ADDRESS>

Codecov Report
Patch and project coverage have no change.

Comparison is base (3e2624e) 93.77% compared to head (827f2e2) 93.77%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #275   +/-   ##
=======================================
  Coverage   93.77%   93.77%
=======================================
  Files          16       16
  Lines         610      610
=======================================
  Hits          572      572
  Misses         38       38

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

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


Update version to 2.0.0

Coverage remained the same at 83.216% when pulling e27c0da041190925d84661c6b45ee4bacbc4a0f9 on scottmo/versionUpdate_2_0_0 into c08b4f66110c2b9a645594f5c4259b15d1a5bd05 on master.


PARQUET-1038: Key value metadata should be nullptr if not set

cpplint failure

This is super odd BTW, I am not sure where the test failure on the Arrow side was introduced

Is this good when the build runs? You should enable appveyor on your fork if you haven't already

Yep, as soon as the appveyor build is green this is ready to merge.

Running on my fork here: https://ci.appveyor.com/project/cpcloud/parquet-cpp

Actually, let me add a test that exercises actual file metadata instead of the somewhat contrived schema based test I have here.

Ah, I'm actually doing that already :) This ready to go as soon as appveyor passes.

@wesm This is passing here: https://ci.appveyor.com/project/cpcloud/parquet-cpp/build/1.0.2

This patch broke coveralls

https://coveralls.io/repos/93701/builds?page=12

Hm, I remember failing CI without that hack.

I'll revert that commit and see if it still fails.


Refactor langchain

Overview

Fix bug in vectorstore span names and add tests for span names
Extract shared code into functions.
Utilize attrs with value of None being dropped.
Add tests for chain span names.
Make sure the span is always exited before we record it's duration.

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

Project coverage is 81.86%. Comparing base (b950070) to head (580c72f).
Report is 3 commits behind head on develop-ai-limited-preview-3.

Files
Patch %
Lines

newrelic/hooks/mlmodel_langchain.py
97.05%
3 Missing and 1 partial :warning:

Additional details and impacted files
@@                      Coverage Diff                      @@
##           develop-ai-limited-preview-3    #1097   +/-   ##
=============================================================
  Coverage                         81.86%   81.86%
=============================================================
  Files                               192      192
  Lines                             21253    21183   -70
  Branches                           3711     3687   -24
=============================================================
- Hits                              17398    17341   -57
+ Misses                             2783     2773   -10
+ Partials                           1072     1069    -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


don't register anonymous modules

This change is a follow up from our conversation on Slack regarding potentially not registering anonymous modules to the runtime instance that originally created them.
One key consideration to take into account is that, with the exception of host modules from the imports such as wasi_snapshot_preview1, modules would by default be anonymous since the default module config has no name.
To minimize the impact of this change, we can maintain the historical behavior of the helper Instantiate methods, by inheriting the name from the module binary, which means the only way to create a anonymous modules is using InstantiateModule.
I haven't yet spent time working on documentation as I think the priority would be to agree that this is the change we want to bring to wazero, there is probably a need for a test or two as well. This PR is a starting point to bring material to the conversation.
PS: as a side note, I was pleased to see how little had to change to make it work, wazero really is a high quality code base 🙌

"the default module config has no name" is only in effect if the name section doesn't define one. please see #1258 for why I really don't want special naming conventions resulting in safety control differences. It is similar to the problem in java of using exceptions for control flow. Let's find another way so we can have both.

Remember right now, we are 4 days from 1.0 and all of our examples, say this:
	// Create a new WebAssembly Runtime.
	r := wazero.NewRuntime(ctx)
	defer r.Close(ctx) // This closes everything this Runtime created.

We have code all over GitHub that would have to change as well code no one can see. And if that doesn't change, people will have resource leaks. I can't get behind this as something that happens without an explicit config opt in. If you and @mathetake do this anyway, it is ok. please help bear the burden of contacting all known users.. similar to what I did for far safer assumption breaks.

If you and @mathetake do this anyway, it is ok. please help bear the burden of contacting all known users.. similar to what I did for far safer assumption breaks.

I agree, I don't think this is the right model, we need to find something that plays better by the rules that wazero has established already 👍 it must be additive, not modify the way existing code behaves.
I needed to see it to get convinced 😄

can we see the perf diff with go test -run='^$' -benchmem -bench 'BenchmarkCompilation|BenchmarkInitialization' -count=5 ./internal/integration_test/bench/...?

Currently the benchmarks show no differences, but I think that's expected since we're allowing the instantiation of anonymous modules, so there were no benchmarks to exercise it before?

maybe runInitializationConcurrentBench needs to change to not specify a module name?

maybe runInitializationConcurrentBench needs to change to not specify a module name?

I would still expect very little difference in benchmarks. The only savings in this implementation is the cost of inserting an element into a map.

diff --git a/internal/integration_test/bench/bench_test.go b/internal/integration_test/bench/bench_test.go
index 74c9c017..73f76ffb 100644
--- a/internal/integration_test/bench/bench_test.go
+++ b/internal/integration_test/bench/bench_test.go
@@ -6,7 +6,6 @@ import (
        _ "embed"
        "fmt"
        "runtime"
-       "strconv"
        "sync"
        "testing"

@@ -123,14 +122,14 @@ func runInitializationConcurrentBench(b *testing.B, r wazero.Runtime) {
        wg.Add(b.N)
        b.ResetTimer()
        for i := 0; i < b.N; i++ {
-               go func(name string) {
-                       m, err := r.InstantiateModule(testCtx, compiled, config.WithName(name))
+               go func() {
+                       m, err := r.InstantiateModule(testCtx, compiled, config)
                        if err != nil {
                                b.Error(err)
                        }
                        m.Close(testCtx)
                        wg.Done()
-               }(strconv.Itoa(i))
+               }()
        }
        wg.Wait()
        b.StopTimer()

try this in the bench... without this, bench result won't be different...

little bit unfair, but in any situation, we have to pay the similar price in reality right?

for sure!

plus without the change, the bench doesn't measure the anonymous instantiation at all.

https://github.com/tetratelabs/wazero/commit/bb57f1abaa57971213b426e04bc4952ae9c0fe6a this kind of sharding might improve the perf, but I guess the change here is a good starting point.

Here are the numbers with the modified benchmarks:
goos: darwin
goarch: arm64
pkg: github.com/tetratelabs/wazero/internal/integration_test/bench
                                    │ /tmp/wazero.bench.main │        /tmp/wazero.bench.2         │
                                    │         sec/op         │   sec/op     vs base               │
Initialization/interpreter                      12.27µ ± 12%   12.67µ ± 7%        ~ (p=0.394 n=6)
Initialization/interpreter-multiple             36.55µ ±  7%   18.35µ ± 3%  -49.79% (p=0.002 n=6)
Initialization/compiler                         8.881µ ±  4%   9.167µ ± 3%   +3.22% (p=0.015 n=6)
Initialization/compiler-multiple               16.117µ ± 27%   9.756µ ± 2%  -39.47% (p=0.002 n=6)
Compilation/with_extern_cache                  108.52µ ±  6%   94.55µ ± 1%  -12.88% (p=0.002 n=6)
Compilation/without_extern_cache                2.254m ±  5%   2.136m ± 3%   -5.24% (p=0.026 n=6)
geomean                                         50.04µ         40.17µ       -19.74%

                                    │ /tmp/wazero.bench.main │         /tmp/wazero.bench.2          │
                                    │          B/op          │     B/op      vs base                │
Initialization/interpreter                      132.4Ki ± 0%   132.4Ki ± 0%       ~ (p=1.000 n=6) ¹
Initialization/interpreter-multiple             133.3Ki ± 0%   132.9Ki ± 0%  -0.29% (p=0.002 n=6)
Initialization/compiler                         133.1Ki ± 0%   133.1Ki ± 0%       ~ (p=1.000 n=6) ¹
Initialization/compiler-multiple                137.8Ki ± 0%   137.4Ki ± 0%  -0.28% (p=0.002 n=6)
Compilation/with_extern_cache                   53.25Ki ± 0%   53.23Ki ± 0%  -0.05% (p=0.002 n=6)
Compilation/without_extern_cache                1.070Mi ± 0%   1.070Mi ± 0%  +0.01% (p=0.011 n=6)
geomean                                         163.2Ki        163.0Ki       -0.10%
¹ all samples are equal

                                    │ /tmp/wazero.bench.main │         /tmp/wazero.bench.2          │
                                    │       allocs/op        │  allocs/op   vs base                 │
Initialization/interpreter                        29.00 ± 0%    29.00 ± 0%        ~ (p=1.000 n=6) ¹
Initialization/interpreter-multiple               46.50 ± 1%    42.00 ± 0%   -9.68% (p=0.002 n=6)
Initialization/compiler                           29.00 ± 0%    29.00 ± 0%        ~ (p=1.000 n=6) ¹
Initialization/compiler-multiple                  36.00 ± 3%    32.00 ± 0%  -11.11% (p=0.002 n=6)
Compilation/with_extern_cache                     979.0 ± 0%    979.0 ± 0%        ~ (p=1.000 n=6) ¹
Compilation/without_extern_cache                 11.33k ± 0%   11.33k ± 0%        ~ (p=0.076 n=6)
geomean                                           158.1         152.4        -3.59%
¹ all samples are equal

With RunParallel, we can compare the performance with different GOMAXPROCS versions, here's the different between GOMAXPROCS=1 and GOMAXPROCS=4
goos: darwin
goarch: arm64
pkg: github.com/tetratelabs/wazero/internal/integration_test/bench
                                    │ /tmp/wazero.bench.branch │     /tmp/wazero.bench.parallel     │
                                    │          sec/op          │   sec/op     vs base               │
Initialization/interpreter                        12.30µ ± 13%   11.57µ ± 2%        ~ (p=0.394 n=6)
Initialization/interpreter-multiple               35.65µ ±  1%   16.68µ ± 2%  -53.23% (p=0.002 n=6)
Initialization/compiler                           8.874µ ±  4%   9.274µ ± 4%   +4.51% (p=0.002 n=6)
Initialization/compiler-multiple                  16.97µ ± 10%   10.43µ ± 2%  -38.56% (p=0.002 n=6)
Compilation/with_extern_cache                    106.07µ ±  1%   90.39µ ± 2%  -14.78% (p=0.002 n=6)
Compilation/without_extern_cache                  2.141m ±  1%   1.641m ± 2%  -23.39% (p=0.002 n=6)
geomean                                           49.66µ         37.47µ       -24.54%

                                    │ /tmp/wazero.bench.branch │      /tmp/wazero.bench.parallel      │
                                    │           B/op           │     B/op      vs base                │
Initialization/interpreter                        132.4Ki ± 0%   132.4Ki ± 0%  +0.00% (p=0.002 n=6)
Initialization/interpreter-multiple               132.9Ki ± 0%   132.9Ki ± 0%  -0.05% (p=0.002 n=6)
Initialization/compiler                           133.1Ki ± 0%   133.1Ki ± 0%       ~ (p=1.000 n=6) ¹
Initialization/compiler-multiple                  137.4Ki ± 0%   137.4Ki ± 0%  -0.05% (p=0.002 n=6)
Compilation/with_extern_cache                     53.25Ki ± 0%   53.31Ki ± 0%  +0.12% (p=0.002 n=6)
Compilation/without_extern_cache                  1.070Mi ± 0%   1.073Mi ± 0%  +0.22% (p=0.002 n=6)
geomean                                           163.1Ki        163.1Ki       +0.04%
¹ all samples are equal

                                    │ /tmp/wazero.bench.branch │     /tmp/wazero.bench.parallel      │
                                    │        allocs/op         │  allocs/op   vs base                │
Initialization/interpreter                          29.00 ± 0%    29.00 ± 0%       ~ (p=1.000 n=6) ¹
Initialization/interpreter-multiple                 43.00 ± 0%    42.00 ± 0%  -2.33% (p=0.002 n=6)
Initialization/compiler                             29.00 ± 0%    29.00 ± 0%       ~ (p=1.000 n=6) ¹
Initialization/compiler-multiple                    33.00 ± 0%    32.00 ± 0%  -3.03% (p=0.002 n=6)
Compilation/with_extern_cache                       979.0 ± 0%    979.0 ± 0%       ~ (p=1.000 n=6) ¹
Compilation/without_extern_cache                   11.33k ± 0%   11.33k ± 0%  +0.01% (p=0.022 n=6)
geomean                                             153.8         152.4       -0.90%
¹ all samples are equal

Yep, the title was outdated, I updated it.

Thanks @achille-roussel and @ckaznocha!


Project file not found in the repository

Sometimes, when I open an hardhat repo, hardhat-vscode doesn't find my project file

It can happen regardless the choosen directory
for a given directory, sometimes it works, sometimes no.
If I reboot, or if I close vscode, sometimes it solves the problem, sometimes no...

info shows that the extension does find the file, but it's not "loaded" in vscode.

It's very annoying because the "go to definition" stuff doesn't work at all when this happens.
hardhat-vscode v0.6.14
vscode 1.76.2
Linux Mint xfce 21.1
Am I missing something ?
Is this a bug ?
Can I enable some additional debug info on the extension's loading to figure what's going on and provide additional information, since I can't give a mwe ?

Now it works !
I've opened some others repos, in different paths, and when I reopen the first one it just works... Not even rebooted.
r

I notice the files analyzed at the end of "info" is not the same, 34 is nearly the right one for this project, and 1013 is possibly the whole content of my dev folder.

Ok it seems heavily related to this one https://github.com/NomicFoundation/hardhat-vscode/issues/270
because /home/fabien/Bureau/dev is a symlink of /home/fabien/dev
Feel free to close this.


Switch to mosaic for all plotting functionality

Fixes #255 and fixes the plotting issues from #242.

Checklist

[ ] User's Guide has been updated
[ ] Developer's Guide has been updated
[ ] API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
[ ] Documentation has been built locally and changes look as expected
[ ] Testing comment in the PR documents testing used to verify the changes

@andrewdnolan, could you change "Fixes #255" to "Partially addresses #255" since we don't want to add python 3.13 testing in this PR.  If you leave things as they are, merging this PR will automatically close #255.


Settings screen not opening, when screen resolution set to 5120x1440

I need to switch to a lower screen resolution like 3840x1080 or 2560x1440 on a 49 inch wide screen monitor to successfully open the settings (right click on the tray icon). This happens not only in 0.20 version, but also in earlier versions too.

I'm running on 5120x1440 as well and have no problem with 0.19.2

I'm on 1920x1080 and I've not been able to open the settings menu for the past few versions

@Hasifant what is your DPI?  I'm on a 4k at 3840x2160.
Still happening on 0.23.2?
@Box-Of-Hats can you share your logs?  I'm betting this is a different issue.  I know with 0.23.x we added a lot of hardening around settings.

Hi!
No, the issue was gone with one of the higher versions and it did not appear again until now.
Btw, my resolution is 5120x1440
Thx
Oliver
Am 16.10.2020 um 19:17 schrieb Clint Rutkas<EMAIL_ADDRESS>
@Hasifanthttps://github.com/Hasifant what is your DPI? I'm on a 4k at 3840x2160.
Still happening on 0.23.2?
@Box-Of-Hatshttps://github.com/Box-Of-Hats can you share your logs? I'm betting this is a different issue. I know with 0.23.x we added a lot of hardening around settings.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/5418#issuecomment-710252882, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHIUIORHN3BOKMVW3RXERRDSLB53JANCNFSM4PRQE4HQ.

@crutkas In the most recent version of PowerToys, I am no longer having this issue.
Thanks for checking up :)

@Hasifant Just to be 100% sure, the issue went away and is happening again?  Does 0.23.2 have the issue?

Dear Clint,
The issue went away and did NOT show up again,
Thank you very much for following up.
BR Oliver
Am 17.10.2020 um 01:23 schrieb Clint Rutkas<EMAIL_ADDRESS>
@Hasifanthttps://github.com/Hasifant Just to be 100% sure, the issue went away and is happening again? Does 0.23.2 have the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/5418#issuecomment-710694597, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHIUIOQTV5ADEA2RGMW2T3LSLDIYFANCNFSM4PRQE4HQ.


fix: allow pulling keys from environment as fallback

Patch for failing workflow (generating OpenAPI spec for documentation): https://github.com/cpacker/MemGPT/actions/runs/8776883099/job/24126445760
  File "/home/runner/work/MemGPT/MemGPT/memgpt/server/server.py", line 216, in __init__
    assert self.credentials.openai_key is not None, "OpenAI key must be set in the credentials file"
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: OpenAI key must be set in the credentials file

#1290


Add support for terminal cursor movement with Ansi/VT100 codes

It will be really usefull if we can also do some cursor movements with the Ansi codes, described in:
http://www.termsys.demon.co.uk/vtansi.htm
This way we can do cool terminal animations, text progressbars etc.
Very cool in combination of node modules like Charm.
https://github.com/substack/node-charm
Please consider implementing this.
Thanks!

I don't think it will be possible to add cursor movement from the server. Adding cursor up/down will be impossible because underneath terminal use cmd plugin that sit below the output and you can't move it around.
But even that you can still add animation by setting command line, check user typing animation, there is better to do this type of animation on the client then on the server, besides terminal is command based so you can't modify single command from the server, it execute your function on enter.


perf: move the notification to a dedicated thread to increase the throughput

FYI: https://github.com/MrCroxx/foyer/pull/619#discussion_r1685461451
#619

Close for there is no more notifications.


[Bug]: HTTP client transform fails in case of proxy

Apache Hop version?
2.6.0
Java version?
java.version=<IP_ADDRESS>
Operating system
Linux
What happened?
Using HTTP client transform through an http proxy results in a "Server Hangup" response (502)
Disabling preemptive authentication by editing the code and transform works.
Issue Priority
Priority: 3
Issue Component
Component: Transforms

Is there any solution to this? I'm not sure what is meant by "Disabling preemptive authentication".

Could you try one of our latest builds. Could be solved by #4195
You can fetch a current build here these do require Java 17

I just tried the latest snapshot, but the problem persists.

To be clear, which lines did you disable to get it working?
Because I see code labeled as Preemtive authentication:
https://github.com/apache/hop/blob/5e0ecaa4b33a18db0542aa94dc66d52876d6beb3/plugins/transforms/http/src/main/java/org/apache/hop/pipeline/transforms/http/Http.java#L154-L158
But disabling that is the same as removing the proxy


Guatemala area

I created the messages and area for Guatemala.

🚀 Deployed: https://welcome.osm.be/north-america/guatemala/list


Adds initialFen attribute to GameStateEvent.Full

Hi,
This pull request adds the initialFen attribute returned by https://lichess.org/api/bot/game/stream/{gameId} to GameStateEvent.Full event.
It seems it is the only way to get the initial FEN of a game (which is useful when you want to get the full history of a chess960 game after it was started. Typically to make your bot able to restart and continue to play ongoing games).
I was hoping that GameStartEvent could help me, unfortunately its fen attribute does not contain the initial FEN, but the current FEN.

Hi!
The initial FEN can be found in the VariantType in the GameType in the Full event.
I've added an example where a Chess960 FEN is retrieved from the Full event.
https://github.com/tors42/chariot/blob/main/src/test/testchariot/it/boardapi/Play960.java#L91-L97
    private String initialFenFromFullEvent(Full full) {
        return switch(full.gameType().variant()) {
            case Chess960(Some(String fen)) -> fen;
            case FromPosition(Some(String fen)) -> fen;
            default -> Board.fromStandardPosition().toFEN();
        };
    }

Or if using Java 17,
one could do chess960.jsh:
if (! Files.exists(Path.of("chariot.jar"))) {
    Files.copy(URI.create(
       "https://repo1.maven.org/maven2/io/github/tors42/chariot/0.0.84/chariot-0.0.84.jar"
       ).toURL().openStream(), Path.of("chariot.jar"));
}
/env --class-path chariot.jar

import chariot.model.Opt;
import chariot.model.Some;
import chariot.model.VariantType;
import chariot.model.VariantType.Chess960;

VariantType variant = new Chess960(Opt.of("nbrqkrbn/pppppppp/8/8/8/8/PPPPPPPP/NBRQKRBN w KQkq - 0 1"));

if (variant instanceof Chess960 chess960
    && chess960.fen() instanceof Some some) {

    System.out.println("Initial FEN: " + some.value());
}

/exit

$ jdk-17/bin/jshell chess960.jsh
Initial FEN: nbrqkrbn/pppppppp/8/8/8/8/PPPPPPPP/NBRQKRBN w KQkq - 0 1

Thanks a lot for replying so quickly and accurately.
(Again) I should have investigated further ...


feat(fmt): function call args statement

Solution

add support for formatting Expression::FunctionCall, Expression::FunctionCallBlock & Statement::Args

intentionally leaving it as a draft for a discussion.
@jpopesculian we should probably revise how next_chunk_needs_space function works. it broke when I added the support of function calls

when i was thinking about this, i was thinking that the function call start and the parentheses needs to be a single chunk. even if we changed the whitespace if the identifier and the parentheses are not chunked together, its possible that the end up on separate lines. the way i was thinking about handling this was to add an extra argument to visit_expr to append the ( to the last chunk

@jpopesculian i did think about it and didn't think of any. but my general understanding is if there is any other that we missed, we'll patch it when implementing support for it

@mattsse yup


Uncaught Error: spawn /Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update...

[Enter steps to reproduce below:]

Opened repo from terminal line with atom ./
Error occured

Atom Version: 1.4.1
System: Mac OS X 10.10.3
Thrown From: Atom Core
Stack Trace
Uncaught Error: spawn /Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/MacOS/Atom Helper ENOENT
At events.js:141

Error: spawn /Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Frameworks/Atom Helper.app/Contents/MacOS/Atom Helper ENOENT
    at exports._errnoException (util.js:837:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:178:32)
    at onErrorNT (internal/child_process.js:344:16)
    at doNTCallback2 (node.js:441:9)
    at process._tickCallback (node.js:355:17)
    at EventEmitter.ipc.sendSync (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/atom.asar/renderer/api/lib/ipc.js:21:31)
    at BrowserWindow.Object.defineProperty.get [as loadSettings] (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/atom.asar/renderer/api/lib/remote.js:160:27)
    at Object.windowState (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/window-load-settings-helpers.js:17:39)
    at Object.stringify (native)
    at exports.setWindowLoadSettings (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/window-load-settings-helpers.js:27:52)
    at ApplicationDelegate.module.exports.ApplicationDelegate.setRepresentedDirectoryPaths (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/application-delegate.js:172:14)
    at /Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:844:44
    at Function.module.exports.Emitter.simpleDispatch (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:125:28)
    at Project.module.exports.Project.addPath (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/project.js:255:29)
    at AtomEnvironment.module.exports.AtomEnvironment.openLocations (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1029:26)
    at EventEmitter.outerCallback (/Users/ethandavis/Library/Application Support/com.github.atom.ShipIt/update.nv4gG26/Atom.app/Contents/Resources/app.asar/src/application-delegate.js:244:18)
    at emitTwo (events.js:92:20)
    at EventEmitter.emit (events.js:172:7)

Commands

Config
{}

Installed Packages
# User
No installed packages

# Dev
No dev packages

Thanks for reporting!  This looks like a duplicate of #9559 though, so please subscribe there for updates.


Feat build versioning api

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.25 :warning:

Comparison is base (566bac7) 90.62% compared to head (48a39cc) 90.37%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #323      +/-   ##
==========================================
- Coverage   90.62%   90.37%   -0.25%
==========================================
  Files          54       58       +4
  Lines        2293     2432     +139
==========================================
+ Hits         2078     2198     +120
- Misses        148      160      +12
- Partials       67       74       +7

Impacted Files
Coverage Δ

cmd/wallet-sdk-gomobile/did/swift_workaround.go
100.00% <100.00%> (ø)

...d/wallet-sdk-gomobile/localkms/swift_workaround.go
100.00% <100.00%> (ø)

.../wallet-sdk-gomobile/openid4ci/swift_workaround.go
100.00% <100.00%> (ø)

cmd/wallet-sdk-gomobile/version/version.go
100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

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


JSON example for configuring batch period is incorrect?

Setting the batch "period" attribute to a number, for example 30, in the appsettings.json file causes the sink to log nothing. Setting the attribute to something like "00:00:30" seems to work. No errors are logged.
The README example (https://github.com/serilog/serilog-sinks-mssqlserver#json-microsoftextensionsconfiguration) may need updated to reflect format above or the sink should convert the number to a TimeSpan. An error should be thrown if the value is invalid. There were no errors shown with SelfLog enabled so this took a while to track down.

List the names and versions of all Serilog packages used in the project:

Serilog 2.7.2-dev-01033
Serilog.AspNetCore 2.1.2-dev-00028
Serilog.Extensions.Logging 2.0.2
Serilog.Settings.Configuration 3.0.1
Serilog.Sinks.Console 3.1.2-dev-00779
Serilog.Sinks.MSSqlServer 5.1.3-dev-00232

Target framework and operating system:

[ x] .NET Core 2.1
[ ] .NET Framework 4.7
[ ] .NET Framework 4.6.x
[ ] .NET Framework 4.5.x
OS:
Windows

Provide a simple reproduction of your Serilog configuration code:

"Serilog": {
    "Using": [ "Serilog.Sinks.Console", "Serilog.Sinks.MSSqlServer" ],
    "MinimumLevel": {
        "Default": "Information",
        "Override": {
            "Microsoft": "Warning",
            "System": "Warning"
        }
    },
    "WriteTo": [
        { "Name": "Console" },
        {
            "Name": "MSSqlServer",
            "Args": {
                "connectionString": "connection string here",
                "schemaName": "dbo",
                "tableName": "Logs",
                "autoCreateSqlTable": false,
                "batchPostingLimit": 1000,
                "period": 30,
                "columnOptionsSection": {
                    "disableTriggers": true,
                    "addStandardColumns": [ "LogEvent" ],
                    "removeStandardColumns": [ "Properties" ]
                }
            }
        }
    ]
},

        public static IWebHostBuilder CreateWebHostBuilder(string[] args) =>
            WebHost.CreateDefaultBuilder(args)
                .ConfigureLogging(logging =>
                {
                    logging.ClearProviders();
                    logging.SetMinimumLevel(LogLevel.Debug);
                })
                .UseStartup<Startup>()
                .UseSerilog((hostingContext, loggerConfiguration) =>
                    loggerConfiguration.ReadFrom.Configuration(hostingContext.Configuration)
                );

Just encountered this myself. Luckily I came here to look before I started to dig deeper. Thanks

Just encountered this myself. Unfortunately it was days after I started digging deeper. Thanks

The sample seems to be correct now. Closing this issue.


/api/ipam/ip-addresses/ query by virtual_machine or virtual_machine_id returns "Bad Request"

Python version:  3.5.4
NetBox version: 3.5.3

Steps to Reproduce

https://NetBox/api/ipam/ip-addresses/?virtual_machine=fluffymonkey or https://nNetBox/api/ipam/ip-addresses/?virtual_machine_id=123

Expected Behavior
Return of:
{
    "count": 0,
    "next": null,
    "previous": null,
    "results": []
}

Observed Behavior
{
  "virtual_machine": [
    "Select a valid choice. %(value)s is not one of the available choices."
  ]
}

{
    "virtual_machine_id": [
        "Select a valid choice. %(value)s is not one of the available choices."
    ]
}

We should fix the error string, but the response code is correct. The request is for resources which belong to an object which doesn't exist.

@jeremystretch : Sorry, confused. So, if I wanted to query the API for all IP addresses "belonging" to VM named "FluffyMonkey", https://NetBox/api/ipam/ip-addresses/?virtual_machine=fluffymonkey is not the correct query? https://netbox.datto.net/api/ipam/ip-addresses/?device=fluffy_monkey works (rather, it returns a count of 0, but that's the expected behaviour)

There are a couple ways to handle this:

Return an empty set because no device/VM with the given name was found. This can be confusing e.g. in the event of a typo in the device/VM name, because it's the same effect as specifying a valid device/VM which simply doesn't have any IP addresses assigned to it.
Return an unfiltered list of all IP addresses, effectively nullifying the filter because it was passed an invalid object. While perhaps semantically correct, this is probably even more prone to confusion.
Return a 400 error, indicating a bad request. IMO this is the most obvious indication that an error has been made, even if the request isn't malformed.
Return a 404 error. Similar to above. Maybe more appropriate.

@jeremystretch : Sorry, very confused.
Looking at https://github.com/digitalocean/netbox/blob/3acc8ca3ab0307487aa5d61a8c8b091900b594b2/netbox/ipam/api/serializers.py#L169 strongly implies that an ipaddress object has the capability to have either a nested device or virtual_machine structure. Cool! But this also makes me think that I should be able to query the API for IP address(es) associated/assigned to either a device (by name or by numeric ID), or to a virtual machine (again, either by name or numeric ID).
If I botch either the ID or the name, am I unreasonable in expecting either an empty set to be returned a-la"you asked for IPs belonging to device ID 13234. None are. Enjoy.", or "Hey, the supplied ID wasn't found at all"?
In fact, returning the empty set is what the API currently does when I query by device or device_id,  but not by virtual_machine or virtual_machine_id. At a minimum, this seems inconsistent and quite confusing.
Am I misunderstanding something?

If I botch either the ID or the name, am I unreasonable in expecting either an empty set to be returned a-la"you asked for IPs belonging to device ID 13234. None are. Enjoy.", or "Hey, the supplied ID wasn't found at all"?

IMO we should have different behavior for querying an existing device/VM with no IPs and querying a non-existent device/VM.

In fact, returning the empty set is what the API currently does when I query by device or device_id, but not by virtual_machine or virtual_machine_id. At a minimum, this seems inconsistent and quite confusing.

Agreed. This is due to the custom filter_device() method on IPAddressFilter, which was needed to match on virtual chassis interfaces.

In fact, returning the empty set is what the API currently does when I query by device or device_id, but not by virtual_machine or virtual_machine_id. At a minimum, this seems inconsistent and quite confusing.

Agreed. This is due to the custom filter_device() method on IPAddressFilter, which was needed to match on virtual chassis interfaces.
Fair enough. I don't have a strong feeling on which of the two behaviours is more appropriate, it was the difference that is most confuzzling :)

Looks like this was resolved at some point and just never closed out. The API requests cited in the description work as intended now.

/virtualization/interfaces/?virtual_machine_id= and /dcim/interfaces/?device_id= should be made equal in behavior regarding this behavior. As of d44f3290cbd7, the former results in the error given in this issue, while the latter returns an empty set.


Override properties are not affecting the redis configuration: redis_password  & redis_logfile

When adding the following properties to the vars/main.yml and including the file in the main.yml that
i.e.

vars_files:
vars/main.yml
tasks:
include: tasks/master.yml

and using the DavidWittman.redis role
i.e.

name: "Configure the master redis server."
hosts: redis-master
roles:
DavidWittman.redis
sudo: yes

The following properties in vars/main.yml do not seem to have an effect on the redis configuration file:

redis_password: somePasswordThatYouWillNeverGUessBecauseItIsSoExcruciatinglyLong
redis_logfile: '/var/log/redis.log'

Which look like the following after the Ansible run is compete.

requirepass foobared
logfile ""

Sorry the markdown is interfering with the code above but you should get the gist of it!

@stephansarver can you provide the full playbook which you're using? Wrap it in triple backticks (```playbook```) for syntax highlighting.
Also, you should not be manually editing any variables within this role; they should all be set within your playbook.

OK so I don't know what is different butt it must be a PEBKAC issue. Since I blew away the original stuff I could not get working here is the test files that do work:
`
`
`
---
- hosts: redis-master
  user: vagrant
  sudo: yes
  vars_files:
    - vars/main.yml
  roles:
    - DavidWittman.redis
# The following functionality seems to be missing to add this property change!
#  tasks:
    # Add the masterauth configuration so when this master fails and comes back up it can be turned into a slave automatically.
#    - lineinfile: dest=/etc/redis/6379.conf regexp="^# masterauth <master-password>" line="masterauth {{ master_auth_pass }}"
    # Restart the service after the configuration changes.
#    - service: name=redis_6379 state=restarted

`
`
`

~~~vars/main.yml~~~
`
`
`
---
#https://github.com/DavidWittman/ansible-redis
redis_version: 3.0.7
redis_password:  "{{ master_auth_pass }}"
redis_logfile: '/var/log/redis.log'
redis_sentinel_logfile: '/var/log/redis_sentinel.log'
redis_sentinel_monitors:
- name: "{{ master_name }}"
  host: "{{ master_addr }}"
  port: "{{ master_port }}"
  quorum: 2
  auth_pass: "{{ master_auth_pass }}"
  down_after_milliseconds: 30000
  parallel_syncs: 1
  failover_timeout: 180000
  notification_script: false
  client_reconfig_script: false

`
`
`

~~~inventorry~~~

`
`
`
[redis-master]
<IP_ADDRESS> master_name=master1 master_addr=<IP_ADDRESS> master_port=6379 master_auth_pass=somelongpassword

`
`
`

This issue should be closed!


[BZ-1324017] fix dealing with URL suffix on Windows

TLDR; URLs can only contain forward slashes - '/' and
filesystem pathas contain os-specific hashes. The code
needs to deal with URLs and paths slightly differently!

@mariofusco could you please take a look?

+1


Update Git to include v2.26.1

Re-pushed to include the correct version number after pushing the v2.26.1.vfs.1.1 tag.

How did the perf look for this version?

I haven't tested the perf because the only change should be the security fix.


feat: Add shortcuts component

This PR adds a toolbar in the render to show hotkeys help.
Also add the following new hotkeys:

M to activate the positioning gizmo
R to activate the rotating gizmo
X to activate the scaling gizmo
ctrl + D to duplicate entities

https://github.com/decentraland/js-sdk-toolchain/assets/3170051/854d57c1-61aa-4e30-8947-f42391be8079
Closes: https://github.com/decentraland/sdk/issues/1072

i noticed one issue when i ctrl + D to duplicate it also moves the camera (because WASD moves the camera). Maybe we could check on the camera bindings if the ctrl key is also pressed and prevent the movement, or stop the propagation of the event from the duplicate handler to avoid reaching the camera handler, i dunno

i noticed one issue when i ctrl + D to duplicate it also moves the camera (because WASD moves the camera). Maybe we could check on the camera bindings if the ctrl key is also pressed and prevent the movement, or stop the propagation of the event from the duplicate handler to avoid reaching the camera handler, i dunno

Hey @cazala -- thanks for the review! issue fixed 🙌


DRILL-5369: Add initializer for ServerMetaContext

ServerMetaContext had no default constructor. The lack of it
might cause m_done to be set to true, same for other variables.
Add a default constructor to explicitly initialize its members.

Thanks @laurentgo for the fix!

+1


Social Sign in using Facebook, Google or any other clients.

Hi guys, I came up with a flow regarding the Social login process using Fb, Google or any other such accounts.
Here is the flow diagram:

The official fb and gapi for client sdk.
To make use of the following diagram flow, we need to use the following packages in our Node server:
passport-facebook-token
passport-google-token
I have followed this logic before in applications where I have worked on and it works perfectly fine. If the idea is accepted will document this in readme file as a standard guide section for social sign up.
Let me know what you guys think.

@twishasaraiya @anurag-majumdar
So i got links to JavaScript code that we could use to have Google and Facebook integration in our web app. If the mods and co-mods I could try to implement in our app.
Google Sign in integration
Facebook Sign in integration

@parmeet1402 This is exactly what I was talking about. In one of my github projects I have used both the SDKs which you gave in the links. I can help setup the API key and secrets as I have experience with them as well as how the app should interact with the sign in strategies.

@anurag-majumdar Oh Okay gotcha man! I haven't implemented it before. It would be really great if you would let me do it with you, I would be able to learn, how to do it in reality.

@parmeet1402 Sure! Since this is a collaborative project, I think the mods and co-mods will break this functionality into chunks and all of us interested can work on this. This authentication will require code changes in both front and back end.

@anurag-majumdar Yup we would be required to make some changes.

@parmeet1402 Exactly! We can help each other in code and give guidance as to how to approach a specific part of the process, once each part of the process gets assigned.

Thank you guys for bringing up this issue. Okay since the user can play without logging in, we will be providing only one social login for now(it can be anything facebook,google etc). Besides that I would like to say that since I haven't implemented REST API before so we will wait for others moderators.
@skyshader , @danivijay what do you think??

@twishasaraiya you are right about the social login part. We can keep only one for this app. I have implemented these features before in 1 day's time when there was a lot of pressure in my projects and am quite comfortable with this area. Lets discuss more with other co-mods regarding this.
I also believe that we should break up the tasks so that most of the people get a chance to contribute. After all this is a collaborative project. If it had not been collaborative I can implement these features from front to back in the least time possible. 😃

Hey guys, so we discussed this before on issue #8
And for first cut we decided to go with a basic username, password authentication. For the reasons check issue #8
However, after the first cut, surely we are focusing on Login with Google and we can close this discussion till then? Let me know your thoughts.

using google auth api is easy and yes @anurag-majumdar  access token + refresh token .You might want to edit that for people unaware of the accesstoken expiry and refresh procedure i.e new to auth2.0

I can implement the Google Login through Firebase SDK.

@ayushdev
Google Login using FIrebase SDK in React?
Count me in! I was working on it recently!
I could help out with it and gain some experience from you on how to make it work.
How do we collaborate?

I too have used google auth api and integration with this is easier. So I vote for google too.

@ayushdev @LoneWolf36 As we got our own back-end, I think we can implement the same without 3rd party, ie Firebase, if necessary.
As a first step, we are implementing basic auth with email and password. I think social logins can be added via passport, as @anurag-majumdar mentioned on the first post.

@LoneWolf36 @danivijay As the app is not so complicated, we can implement that using our own backend. I am in favour of whatever tools get the work done in a short time.

I agree with @ayushdev on this 😄

I would like to work on this part too. Let's divide it into chunks I guess.


Complete FontFile ref and update example

I complete the ref items according to TextServer ref and update example with new properties path

I had not come across the doc of the Control object and this way of overriding theme, I have always used the editor's property path 😳. I found the same outdated example in the FontVariation class ref

I found the same outdated example in the FontVariation class ref

Feel free to fix it the same way :)

I found the same outdated example in the FontVariation class ref

Feel free to fix it the same way in this PR :)

in this PR ? Give me a few minutes to understand how to do this :)

Yes, you need to update it anyway to apply my suggestions. You can create a second commit to add more changes, and then you need to squash them and force-push the branch. See our documentation if you need help with that.

Sorry I'm not very good with Git, by the time I understood how to follow your expected workflow (only 1 commit per PR) I accepted your suggestions from the github interface, which created a commit that I couldn't figure out. BUT, the new modification was (i hope) well done.

Thanks! And congrats for your first merged Godot contribution :tada:


Polyfill for mediaQuery breaks in IE11

What should happen?
Foundation should load all of its JavaScript (in this case, trying to initialize an accordion) in IE11 without throwing any exceptions.
What happens instead?
In IE11, get error SCRIPT65535: Invalid Calling object in the console window and the accordion does not work.
Possible Solution
The cause seems to be the polyfill for window.matchMedia.  There is a local variable matchMedia, to which window.matchMedia is being assigned, and calling this as a function in foundation.util.mediaQuery.js is what seems to trigger the error.
The preferable solution would be to remove the polyfill from foundation altogether (just use the native window.matchMedia), and leave a note in the setup guide specifying that for older browsers a third-party polyfill will be required (https://github.com/paulirish/matchMedia.js)... Foundation should not be baking in its own polyfills.
Steps to Reproduce
How to reproduce:

Define markup for an accordion.
Call $(document).foundation()
Navigate to site in IE11
Observe that clicking on the accordion does not cause it to expand or collapse, and there is an error in the console.

Context

I'm just trying to use the Accordion plugin.  It works in Chrome, but not IE11 b/c of the aforementioned polyfill.
Your Environment

Foundation version(s) used: 6.4.3
Browser(s) name and version(s): Internet Explorer 11
Operating System and version (desktop or mobile): Windows 10 Enterprise (desktop)

Checklist (all required):

[x] I have read and follow the CONTRIBUTING.md document.
[x] There are no other issues similar to this one.
[x] The issue title is descriptive.
[x] The template is correctly filled.

Never had this issue in current projects which use accordions.
Can you test Foundation Sites 6.5.0 RC1?

Could reproduce it with a project here.
It seems atLeast() is the cause, at least here. Can you test if is() works?
And can you share the used markup and code?

Probably related: https://github.com/tus/tus-js-client/issues/77#issuecomment-308701823

Newest release is here: https://github.com/paulirish/matchMedia.js/blob/master/matchMedia.js

My UI/UX team member decided to go with an alternate solution, so I'm afraid I didn't save the code or markup changes and have since moved on to other things.  But I can tell you that I used webpack to bundle the code (as did the person who reported the issue in your aforementioned "probably related" link... perhaps that's the common denominator?), and for me the error bubbled up from within the _getCurrentSize() function.  And the markup I used was just copied & pasted from the first sample here.

We had a similar issue and could also trace it back to the _getCurrentSize() function. The issue is that the matchMedia is not mapped back to the window.matchMedia property which is likely causing an error in IE due to strict mode.
patch-package
--- a/node_modules/foundation-sites/js/foundation.util.mediaQuery.js
+++ b/node_modules/foundation-sites/js/foundation.util.mediaQuery.js
@@ -63,6 +63,8 @@ let matchMedia = window.matchMedia || (function() {
}
})();

+window.matchMedia = matchMedia;
+
var MediaQuery = {
queries: [],

@@ -109,7 +111,7 @@ var MediaQuery = {
var query = this.get(size);

if (query) {
- return matchMedia(query).matches;
+ return window.matchMedia(query).matches;
}

return false;
@@ -160,7 +162,7 @@ var MediaQuery = {
for (var i = 0; i < this.queries.length; i++) {
var query = this.queries[i];

- if (matchMedia(query.value).matches) {
+ if (window.matchMedia(query.value).matches) {
matched = query;
}
}

foundation-sites+6.4.3.patch.zip

I also think we should also use polyfills from polyfill.io by the Financial Times
https://github.com/Financial-Times/polyfill-service

@DanielRuf I don't think you want to lock users into using any particular polyfill solution.  I'd rather Foundation not use any polyfills at all because some people may only be targeting evergreen browsers (in which case a polyfill only adds unnecessarily to the download size), and not everyone is going to want to create an external dependency on the polyfill.io site (some may prefer to bundle the polyfill with their own code).  It should be documented that Foundation has a dependency on this or other newer features of JavaScript/ECMAScript which are only supported in newer browsers, and you can suggest 3rd-party polyfill solutions but it should ultimately be up to the developer as to what polyfill, if any, they want to use.

on the polyfill.io site

Please check the repository and see package.json, this is on npm.

Well, currently we internally use a matchMedia polyfill (which has bugs).


Strategy for working around TypedArray calling Resources.loadDrawable() directly?

Hi, I took a look at the problem of TypedArray calling directly Resources.loadDrawable() and I was wondering what ideas you had to overcome it?
The way I see, there are only two options (and both pretty much suck):

Override Resources.loadDrawable() by using a modified Android.jar (like in this tutorial)
Fake a previously cached drawable. Like the system does, call Resources.cacheDrawable() (through reflection, as it's obviously private), that way it won't try to inflate it but will use it directly... Oh and don't forget to keep another reference to it somewhere as the cache uses WeakReferences! Little problem: the implementation was slightly different in Honeycomb (I haven't actually checked every version)... Still possible, but what a pain!

Neither option guarantees that it will work everywhere though, as OEMs or custom ROMs might have changed the implementation enough to break it... Thoughts?

Hi, some news about the ExtendableResources approach? Instead of searching for an all-working solution, is it possible to support just the supposedly most often scenario: action bar + AppCompat (my case of course :). It still will be worth enough!

Hi guys, we forget to checkout how do HoloEveryWhere Team solves this problem, also they are solves cache issue#1

It would appear that an official appcompat.v7.vectordrawablecompat had been started but is a MIA for some time now
https://github.com/android/platform_frameworks_support/commit/34cbdb2e17271b31d1511d2773edd241fafede7d

Apologies for not being on top of this repository recently (between jobs), as @trevjonez pointed out an official VectorDrawable backport is coming to support library, it is great to have first party support for this but unfortunately makes efforts like this redundant, therefore I will be deprecating Mr. Vector hoping they'll release it sooner rather than later. Feel free to fork and continue development if you prefer to do so, once more thanks for your contributions.


Fixes 2081 - Added breaking change article to toc

Fixes #2081 - Added breaking change article to toc

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

File
Status
Preview URL
Details

docs-conceptual/azps-6.6.0/toc.yml
:white_check_mark:Succeeded
View (azps-6.6.0)

docs-conceptual/azps-7.2.0/toc.yml
:white_check_mark:Succeeded
View (azps-7.2.0)

docs-conceptual/azps-7.3.2/toc.yml
:white_check_mark:Succeeded
View (azps-7.3.2)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Print requested V-Sync mode when Print Fps is enabled

master version of https://github.com/godotengine/godot/pull/56110.
This can be used to diagnose why the printed FPS is locked to the monitor refresh rate.
Note that the OS may still override the actual V-Sync mode, but this should provide first steps for troubleshooting already.
See https://github.com/godotengine/godot/issues/56089#issuecomment-998144899.

Thanks!


IE browser just hangs when launched fresh

Thanks again for your work on this launcher. I've configured and I'm able to run this successfully.
The issue that I ran into is that my tests only run if the VM is:

started
and the IE browser is already opened.

Karma waits for tests to run as the VM tries to launch the IE browser fresh. When it launches fresh, somehow the tests are not connecting. I get these screens and errors:

Have you ran into this? Or can you think of ways to address a fix?

@d48 that is really bizarre, I have not ran into this before.  Everything that I can find relating to acr_error.html seems to be dependent on the installation of IE being broken somehow.  Are you able to navigate to any other sites in that browser?

Thanks @austin94. Yea, it's strange. If I close out the running process and re-run the tests, then IE runs it successfully. It's seems to want to connect before it launches the browser and just hangs. I wonder if introducing a delay before it runs the tests may help

There is currently a 10 second delay which will continue to check to see if the VM is loaded and then   open the browser.  You can see the implementation here https://github.com/austin94/karma-virtualboxany-launcher/blob/master/index.js#L68-L78.  Sometimes this can result in a weird state but normally it works fine or wouldn't open the browser at all.  Could you try to replicate with the VM running, but with all IE processes killed?

I was able to run the tests fresh with just IE10 virtualbox vm running the tests.
Then when running with IE in my browser list for IE11 on my host, it caused the virtualbox vm to hang. I'll check if it's any configuration when accompanied with that. If I find anything I'll post again. Thanks for your help.


Conceal enhancement

Conceal is great but:

We should be able to define which parts should be concealed
For example i like superscript and subscript to be concealed but not things like \left

The ability to automatically deactivate conceal when we switch from live preview to source mode

Seconding point 1: I really want to use conceal but the fact that some invisible characters are hidden entirely (such as \quad) is really quite a dealbreaker for me
Maybe I'll come up with a fix myself if it starts bothering me enough but something that would fit my usecase is a list of control sequences to ignore in the settings


[fix] croaringbitmap compile support USE_AVX2=0

Now third party package CRoaring will turn AVX2 dynamically.
We need to FORCE disable AVX2 while release no-avx Doris package.
Proposed changes
Issue Number: close #10100
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

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...

P0 regression failed

P0 regression failed

Every body's P0 regression failed. It looks like the BE for regression is crashed.


Introduce hyper::ext::Http1RawMessage

Target is 0.14, not master?

Lol I didn't even realise that Hyper 1 is out.

Did you want to update the target to master/v1?

Yeah I plan to, just didn't get around to it yet that's all.


[28312] Execution returned code=0 in 7.714 seconds

When I am trying to run any manim code, it return this massege and the preview screen does not appear. After I have tried to reinstall the extension and even VSCode, it return to being normal for a while but after some minutes, it comes to this siuation again. I don't know whether this situation is really an error or not. If so, how can I solve it?

Can you please show me the message received?


Validator fails to restart

After running for few days with high load (CU 42M +) validator fails to restart with an error:
[2024-05-24T00:34:03.140014735Z WARN solana_rpc::rpc] Bank with Finalized not found at slot: 0
[2024-05-24T00:34:03.140074965Z WARN solana_rpc::rpc_health] health check: behind by 415 slots: me=1002936, latest cluster=1003351
0: rust_begin_unwind
at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/std/src/panicking.rs:645:5
1: core::panicking::panic_fmt
at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/panicking.rs:72:14
2: solana_accounts_db::epoch_accounts_hash::manager::Manager::wait_get_epoch_accounts_hash
3: solana_runtime::bank::Bank::hash_internal_state
4: solana_runtime::bank::Bank::freeze
5: solana_ledger::blockstore_processor::process_blockstore_from_root
6: solana_core::validator::ProcessBlockStore::process
7: solana_core::validator::ProcessBlockStore::process_to_create_tower
8: solana_core::validator::Validator::new
9: solana_validator::main
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
[2024-05-24T00:34:03.152454200Z ERROR solana_metrics::metrics] datapoint: panic program="validator" thread="main" one=1i message="panicked at accounts-db/src/epoch_accounts_hash/manager.rs:72:35:
The epoch accounts hash cannot be awaited when Invalid!" location="accounts-db/src/epoch_accounts_hash/manager.rs:72:35" version="1.18.12 (src:b9c13825; feat:4215500110, client:SolanaLabs)"

I don't have it anymore, but I'll setup a test cluster and reproduce it
On Wed, Jun 12, 2024 at 3:18 PM Brooks @.***> wrote:

Can you share the whole log beginning from the restart?
—
Reply to this email directly, view it on GitHub
https://github.com/anza-xyz/agave/issues/1491#issuecomment-2163992863,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEEDZD74RC3YCE5AWWOZDLZHDCMHAVCNFSM6AAAAABIIPOQUGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRTHE4TEOBWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@jacklevin74 - I'm inclined to close this issue out since we haven't gotten steps to repro back, and the v1.18 line is hopefully reaching EOL in a couple weeks. We can of course reopen if you're able to repro again & with more recent version. Sound reasonable (I'll close in ~1 week if I don't hear back from ya)

Let’s go ahead and close it, Thanks
On Wed, Oct 9, 2024 at 1:54 PM steviez @.***> wrote:

@jacklevin74 https://github.com/jacklevin74 - I'm inclined to close
this issue out since we haven't gotten steps to repro back, and the v1.18
line is hopefully reaching EOL in a couple weeks. We can of course reopen
if you're able to repro again & with more recent version. Sound reasonable
(I'll close in ~1 week if I don't hear back from ya)
—
Reply to this email directly, view it on GitHub
https://github.com/anza-xyz/agave/issues/1491#issuecomment-2403635998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEEDZHQAWRYUFI74ERSQLTZ2W64FAVCNFSM6AAAAABIIPOQUGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBTGYZTKOJZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Hover effect for the header

Fixes #107
I changed the color and font weight on hover so it looked more interactive.
I didn't have any other idea and tried to change font family and it didn't look good so if you have any other idea tell me.

I think it looks great like this! Thank you so much for your contribution!


Added unit tests

More unit test coverage and tests for case note amendments.

Codecov Report

Merging #19 (efa6181) into main (5f6fd3f) will increase coverage by 12.16%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##               main      #19       +/-   ##
=============================================
+ Coverage     73.37%   85.54%   +12.16%
- Complexity       75      102       +27
=============================================
  Files            31       30        -1
  Lines           432      429        -3
  Branches         20       18        -2
=============================================
+ Hits            317      367       +50
+ Misses           96       52       -44
+ Partials         19       10        -9

Impacted Files
Coverage Δ

...e/digital/hmpps/config/DeliusConnectionProvider.kt
92.85% <100.00%> (+92.85%)
:arrow_up:

...igital/hmpps/datetime/ZonedDateTimeDeserializer.kt
100.00% <100.00%> (+50.00%)
:arrow_up:

...hmpps/exceptions/ProbationAreaNotFoundException.kt
100.00% <100.00%> (+100.00%)
:arrow_up:

...ital/hmpps/integrations/delius/entity/StaffTeam.kt
88.88% <100.00%> (+1.38%)
:arrow_up:

.../digital/hmpps/integrations/nomis/NomisCaseNote.kt
89.47% <100.00%> (+13.15%)
:arrow_up:

...gital/hmpps/integrations/delius/entity/CaseNote.kt
70.14% <0.00%> (+1.49%)
:arrow_up:

.../digital/hmpps/integrations/delius/entity/Staff.kt
92.59% <0.00%> (+14.81%)
:arrow_up:

...ntegrations/delius/service/OfficerCodeGenerator.kt
92.30% <0.00%> (+15.38%)
:arrow_up:

...e/digital/hmpps/integrations/delius/entity/Team.kt
100.00% <0.00%> (+16.66%)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


Dependency Updates

Signed-off-by: Frank Jogeleit<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3505

Relevant Lines:
3713

💛  - Coveralls


Notes page. Extract "card" to the dedicated component

The same way as we did it for Videos.

Done.


Mono: Exported variables from base classes are in the wrong order.

Can we change the order of exported values so that the base class values are on top and the sub class values are on bottom? It would be much more consistent this way.
The following project can be used. The base class is BehaviorTreeNode which has one exported variable "enabled". The first BehaviorTreeNode is located at "Navigation->BehaviorTree->Loop".
https://github.com/NathanWarden/godot_ai_csharp
This is how it currently looks.

Here's another class:

@neikeq If you think this makes sense to change and it would save you time I'll change it.
Thanks :)

The entire Inspector dock is set up so that base classes are on the bottom (Node -> Spatial -> Script Variables in your example). What do you mean when you say that suddenly reversing this trend would be more consistent?

Thanks for asking... It would be more consistent because your variables in base classes would always be in the same place. Right now everything is in reverse order.
For instance:
public class BaseClass
{
    [Export] public bool variable1;
    [Export] public bool variable2;
}

public class SubClass : BaseClass
{
    [Export] public bool variable3;
    [Export] public bool variable4;
}

public class SubSubClass : SubClass
{
    [Export] public bool variable5;
    [Export] public bool variable6;
}

Right now these variables show up as...
variable5
variable6
variable3
variable4
variable1
variable2
What I'm proposing would change it to:
variable1
variable2
variable3
variable4
variable5
variable6
In my case I have an "enabled" variable. So each node's behavior is enabled or disabled. But, it's really strange looking for my most common variables at the bottom of the list, and my specific variables at the top.

@leoddd I do see what you're saying though :/ But, do the variable orders really need to follow a convention for component types? Well, not a deal breaker either way :)

Actually, fixing this would make it consistent with GDScript...
If you make a script:
ClassA.gd
extends Node
export var var1 = true

ClassB.gd
extends "ClassA.gd"
export var var2 = true

The variables show in the inspector as:
Var 1
Var 2
So my proposal will make it consistent :)

Opened a PR for this at https://github.com/godotengine/godot/pull/13774


[Productize Flash Attention performance #1] add support for attention in match-target-size

upstream PR: https://github.com/triton-lang/triton/pull/4240

upstream pr merged: https://github.com/triton-lang/triton/pull/4240


Moves the abandon button to left side of the form.

Tested and works in chrome and recent Firefox.


error when trying to search online news

You've closed my issue without providing a solution: https://github.com/mediacloud/web-search/issues/482
I've added the following lines to the mcweb/.env
MEDIA_CLOUD_API_KEY=4a6e7196b5fb0ed06d55e5ef62f9b94f382b01a5
MEDIA_CLOUD_API_URL=https://search.mediacloud.org/
But still it shows the same error, you can also check it from here: https://media.eventassay.com/search

Hello, my apologies for the confusion. I do not believe your use case will work without a Legacy Media Cloud API key (issued pre 8/22). The current api key that is found in your profile does not work with the Media Cloud Legacy API. We no longer have access to the previous system to generate Legacy Media Cloud API keys. Essentially, if you did not have an api key for the Media Cloud Legacy system, over a year ago, then you will not be able to query Media Cloud Legacy locally.
Any searches you would like to make through the Media Cloud Legacy API should be able to be made through the current live application, found at search.mediacloud.org.
But if you would like to follow up with me via email<EMAIL_ADDRESS>I would be happy to help and discuss further.


How do you rate limit in trafficserver?

I'm wondering is it possible to rate limit in trafficserver and if so how?
In varnish you can use vsthrottle.

There is a plugin called "fq_pacing" which will rate limit individual TCP connections.  Is this sufficient?

There is a plugin called "fq_pacing" which will rate limit individual TCP connections. Is this sufficient?

Thanks! Not exactly. It doesn't appear you can white list specific ips from the rate limit or configure a rate limit per domain with it.
For example we do:
	# Allow higher limits for static.mh.o, we can handle more of those requests
	if (req.http.Host == "static.miraheze.org" || req.http.Host == "static-new.miraheze.org") {
		if (vsthrottle.is_denied("static:" + req.http.X-Real-IP, 500, 1s)) {
			return (synth(429, "Varnish Rate Limit Exceeded"));
		}
	} else {
		# Do not limit /w/load.php, /w/resources, /favicon.ico, etc
		# T6283: remove rate limit for IABot (temporarily?)
		if (
			(req.url ~ "^/wiki" || req.url ~ "^/w/(api|index)\.php")
			&& (req.http.X-Real-IP != "<IP_ADDRESS>" && req.http.User-Agent !~ "^IABot/2")
		) {
			if (req.url ~ "^/w/index\.php\?title=\S+\:MathShowImage&hash=[0-9a-z]+&mode=mathml") {
				# The Math extension at Special:MathShowImage may cause lots of requests, which should not fail
				if (vsthrottle.is_denied("math:" + req.http.X-Real-IP, 120, 10s)) {
					return (synth(429, "Varnish Rate Limit Exceeded"));
				}
			} else {
				# Fallback
				if (vsthrottle.is_denied("mwrtl:" + req.http.X-Real-IP, 12, 2s)) {
					return (synth(429, "Varnish Rate Limit Exceeded"));
				}
			}
		}

On going #7623 (New rate_limit plugin for simple resource limitations ) is what you're looking for?

@paladox We do have a rate limit plugin: https://docs.trafficserver.apache.org/admin-guide/plugins/rate_limit.en.html


working on newconstructs and reorg

Codecov Report

Merging #235 (dab8c4f) into main (cb2cb51) will increase coverage by 1.22%.
The diff coverage is 97.52%.

:exclamation: Current head dab8c4f differs from pull request most recent head d50293d. Consider uploading reports for the commit d50293d to get more accurate results

@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
+ Coverage   68.18%   69.40%   +1.22%
==========================================
  Files         103      106       +3
  Lines        5928     6178     +250
  Branches      437      488      +51
==========================================
+ Hits         4042     4288     +246
- Misses       1769     1773       +4
  Partials      117      117

Impacted Files
Coverage Δ

pyEX/premium/newconstructs/__init__.py
75.00% <75.00%> (ø)

pyEX/premium/stocktwits/__init__.py
31.25% <75.00%> (+31.25%)
:arrow_up:

pyEX/premium/valuengine/__init__.py
75.00% <80.00%> (-5.00%)
:arrow_down:

pyEX/tests/test_api_client.py
92.10% <92.10%> (ø)

pyEX/__init__.py
95.23% <100.00%> (ø)

pyEX/client.py
56.92% <100.00%> (+28.10%)
:arrow_up:

pyEX/premium/__init__.py
100.00% <100.00%> (ø)

pyEX/premium/auditanalytics/__init__.py
60.00% <100.00%> (ø)

pyEX/premium/brain/__init__.py
64.55% <100.00%> (ø)

pyEX/premium/extractalpha/__init__.py
52.21% <100.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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


Modular Exponential [JavaScript]

This issue is about Modular Exponential.

[x] Issue Title is in form Algo/DS Name [Language].
[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate.
[ ] This Algo/DS is worth adding to this repository.
[x] Assign this issue to me (I want to work on this).
[x] I will send my PR only after I'm assigned to this issue by any maintainer.

@Anshuman-Verma: Please complete previously assigned issues first.

@Anshuman-Verma Can I work on this one ?

All yours @aashutoshrathi :smile:

[x] Assign me this issue.