Issue with Long Path in Windows

Description
I was trying to compile all the sources from an examples repository using the Windows 2016 machine on my fork using this configuration.
While compiling, I'm receiving this error with some examples with long names:

C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\VC\VCTargets\Microsoft.CppBuild.targets(321,5): error MSB3491: Could not write lines to file "x64\Release\content_filtered_topic_string_filter_CXX11_sources\content_.ED6EA502.tlog\content_filtered_topic_string_filter_CXX11_sources.lastbuildstate". The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters. [D:\a\rticonnextdds-examples\rticonnextdds-examples\examples\connext_dds\build\content_filtered_topic_string_filter\c++11\content_filtered_topic_string_filter_CXX11_sources.vcxproj]

It's supposed that the Long Path is enabled (see issue #1052), but with the previous error, it looks that it isn't.
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[X] Windows Server 2016
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
20210815.1
https://github.com/ManuelJNunez/rticonnextdds-examples/runs/3399643219
Is it regression?
No
Expected behavior
The Long Path should be activated, and the compilation should be completed with no errors due to the path limit.
Actual behavior
There are some errors due to the path limit.
Repro steps
See the logs of the build. I can't help you with this because we are using private libraries but I'm trying to reproduce it with a simple example.

Hello @ManuelJNunez,
I've verified that longpath feature is enabled on Windows 2016, but just in case: Could you please add the following job at the begging of your workflow?
- name: Check LongPathsEnabled
  run: |
     (Get-ItemProperty "HKLM:System\CurrentControlSet\Control\FileSystem").LongPathsEnabled

We're looking forward to your reply

Hello @Darleev
I added the step to the workflow configuration. You can find the logs here.
Thank you for your response :)

Hello @ManuelJNunez,
I'm going to close the issue until further clarifications from your side.
Feel free to contact us at any time.


Fix #248 Add HttpMethodRouteConstraint

Constraint code ported from WebAPI2. Tests are new.

Move IRouteConstraint to abstractions
Fix namespace of a constraint
Some general style cleanup
use RouteValueDictionary in the public API

/cc @pranavkm

Other than minor stuff :shipit:

:shipit:


How to collect the logs to kafka by fluentd

Any updates? Need a sample fluentd-kafka-daemonset yaml.

Doc says "Default YAML uses latest v1 images like fluent/fluentd-kubernetes-daemonset:v1-debian-kafka".
There is no YAML file using kafka image and no samples to start using these docker images with Kafka.
Would be useful to have a sample YAML for using with Kafka.

Is there any example for that?


Random connections unless "Remember Me" checked

We need to test the connection on different browsers, 2 of us have trouble

After many test, I think I the problem of a database setting.


Enable autofail error logging for OSIE deprovision script

This enables the autofail() trap function and gives us error and timeout logging in the same way we've recently added for the osie.sh script.

Going to spend a day testing this, will merge when ready.


Not installing on Debian Jessie RC1

I'm not certain this is meant to work under Debian, but when following the guide "Installation on deb based system" I run into an error:
Setting up lwp (0.8-25-g62597b1) ...

missed /etc/lwp/lwp.conf file
cannot read config files. Exit!
dpkg: error processing package lwp (--configure):
subprocess installed post-installation script returned error exit status 102
Processing triggers for libc-bin (2.19-17) ...
Errors were encountered while processing:
lwp
E: Sub-process /usr/bin/dpkg returned an error code (1)

I manually copied the example file by "cp /etc/lwp/lwp.example.conf /etc/lwp/lwp.conf" and tried again but it still fails:
Setting up lwp (0.8-25-g62597b1) ...

Auth type: database
dpkg: error processing package lwp (--configure):
subprocess installed post-installation script returned error exit status 102
Errors were encountered while processing:
lwp
E: Sub-process /usr/bin/dpkg returned an error code (1)

Is there anything I can do?
Thanks!
Sven

Hi @svennieslony
I use those project on some ubuntu server but I don't see any problem to run in on debian or centos distro, so also if I suggest to use ubuntu I pretty happy to fix stuff to be more distro independent as possible.
On your error now, during the installatio progess the debian hook will call this script https://github.com/claudyus/LXC-Web-Panel/blob/master/debian/lwp.postinst to generate the secret session token if it doesn't exists.
Can you try to directly run /usr/bin/lwp --generate-session-secret as root to see if the same error is reported?
It should not involve the config file or db during this operation. This is really strange behaviors.

Hi Claudyus,
This runs fine:
/usr/bin/lwp --generate-session-secret

Auth type: database

I can also start lwp by running /usr/bin/lwp

Auth type: database
Running on http://<IP_ADDRESS>:5000/

lwp is is accessible as expected. It seems like something in the posttest script is not working. I will try to do some debuging...
Thanks for the quick reply!

delete me


Move string utils out of main class

Improves the formatting of complex numbers in the debugging helpers.  Also moves them out of the main class and adds unit tests to keep codecov happy.

Codecov Report

Merging #10 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   94.11%   94.16%   +0.04%
==========================================
  Files          32       34       +2
  Lines        2633     2655      +22
==========================================
+ Hits         2478     2500      +22
  Misses        155      155

Impacted Files
Coverage Δ

src/helpme.h
96.79% <ø> (ø)
:arrow_up:

src/string_utils.h
100% <100%> (ø)

src/matrix.h
96.41% <100%> (ø)
:arrow_up:

test/unittests/unittest-string.cpp
100% <100%> (ø)

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 797b524...d1f9c02. Read the comment docs.


There is no workflow defined for ...

Hello @Aslemammad,
We once disabled the pkg-pr-new publish job in the workflow and now we decide to add it back. But pnpx pkg-pr-new publish now gives us the There is no workflow defined for ... error. Is this a bug or are we using it in a wrong way?
The failed job: https://github.com/biomejs/biome/actions/runs/9389095954/job/25855841148#step:10:16

cc @ematipico

Oh that's bad, on the vite pr we have the same issue, I'll try to find a way for this, thank you for the report!
I'll try forking and debugging. Thanks.

just fixed this, it helped me rethink a bit of the architecture, thank you so much again.

just fixed this, it helped me rethink a bit of the architecture, thank you so much again.

Cool! Huge thanks. I see the fix is related to #96, we should wait for it to merge, right?

Yea yea, we'll have a release soon!

just released 0.0.8!


Current Sphinx version incompatible with numpydoc

I'm trying to build the documentation using Sphinx.
With sphinx-build properly running with Python 3, I've tried make html in the Documentation directory as per the instructions
I get the following error:
~/projects/econ-ark/HARK/Documentation$ make html
sphinx-build -b html -d ../../HARK-docs/doctrees   . ../../HARK-docs/html
Running Sphinx v2.2.0
Inserting /home/sb/projects/econ-ark/HARK

Extension error:
Could not import extension numpydoc (exception: cannot import name 'Directive')
Makefile:63: recipe for target 'html' failed
make: *** [html] Error 2

Looking up this error, it looks like it's the result of incompatibility between the current version of Sphinx (v2.2.0) and the numpydoc extension:
https://github.com/readthedocs/readthedocs.org/issues/4057
The recommended fix from that issue, which is quite old (from May, 2018) is the fix the Sphinx version in requirements.txt to an early version, sphinx==1.6.7

Indeed, it looks like numpydoc explicitly requires sphinx <2.0
https://github.com/simpeg/discretize/issues/167
I'll try adding this requirement and see if it helps with the sphinx docs build.

Reverting to sphinx=1.6.7 and trying make html gets this new error:
$ make html
sphinx-build -b html -d ../../HARK-docs/doctrees   . ../../HARK-docs/html
Running Sphinx v1.6.7
Inserting /home/sb/projects/econ-ark/HARK
/home/sb/anaconda3/lib/python3.6/site-packages/sphinx/util/compat.py:40: RemovedInSphinx17Warning: sphinx.util.compat.Directive is deprecated and will be removed in Sphinx 1.7, please use docutils' instead.

RemovedInSphinx17Warning)
Extension error:
Could not import extension recommonmark (exception: No module named 'recommonmark')
Makefile:63: recipe for target 'html' failed
make: *** [html] Error 1

This is because recommonmark is not included as a dependency in requirements.txt

Fixing this requirements issue and rerunning make html results in this error:
$ make html
sphinx-build -b html -d ../../HARK-docs/doctrees   . ../../HARK-docs/html
Running Sphinx v1.6.7
3.6.8 |Anaconda custom (64-bit)| (default, Dec 30 2018, 01:22:34)
[GCC 7.3.0]
Inserting /home/sb/projects/econ-ark/HARK
/home/sb/anaconda3/lib/python3.6/site-packages/sphinx/util/compat.py:40: RemovedInSphinx17Warning: sphinx.util.compat.Directive is deprecated and will be removed in Sphinx 1.7, please use docutils' instead.
  RemovedInSphinx17Warning)
loading pickled environment... not yet created
/home/sb/projects/econ-ark/HARK/Documentation/notebooks/DCT-Copula-Illustration.rst: WARNING: document not readable. Ignored.
loading intersphinx inventory from https://docs.python.org/objects.inv...
intersphinx inventory has moved: https://docs.python.org/objects.inv -> https://docs.python.org/3/objects.inv
[autosummary] generating autosummary for: CONTRIBUTING.md, HARKmanual.md, HARKmanual/ContributingToHARK/ContributingToHARK.md, HARKmanual/HARKmanual.md, generated/HARK.BayerLuetticke.Assets.One.FluctuationsOneAssetIOUs.rst, generated/HARK.BayerLuetticke.Assets.One.FluctuationsOneAssetIOUsBond.rst, generated/HARK.BayerLuetticke.Assets.One.SharedFunc.rst, generated/HARK.BayerLuetticke.Assets.One.SharedFunc2.rst, generated/HARK.BayerLuetticke.Assets.One.SteadyStateOneAssetIOUs.rst, generated/HARK.BayerLuetticke.Assets.One.SteadyStateOneAssetIOUsBond.rst, ..., generated/HARK.BayerLuetticke.Assets.Two.SharedFunc3.rst, generated/HARK.BayerLuetticke.Assets.Two.SteadyStateTwoAsset.rst, generated/HARK.BayerLuetticke.Assets.Two.defineSSParametersTwoAsset.rst, generated/HARK.BayerLuetticke.ConsIndShockModel_extension.rst, index.rst, notebooks/Gentle-Intro-To-HARK.ipynb, notebooks/README.rst, packaging.md, readme.md, submodules/README.rst
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.Assets.One.Luetticke_wrapper': no module named HARK.BayerLuetticke.Assets.One.Luetticke_wrapper
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.Assets.Two.EX3SS_20_python2.p': no module named HARK.BayerLuetticke.Assets.Two.EX3SS_20_python2.p
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.Assets.Two.EX3SS_20_python3.p': no module named HARK.BayerLuetticke.Assets.Two.EX3SS_20_python3.p
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.BayerLuetticke_wrapper': no module named HARK.BayerLuetticke.BayerLuetticke_wrapper
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.notebooks.DCT': no module named HARK.BayerLuetticke.notebooks.DCT
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.notebooks.DCT.ipynb': no module named HARK.BayerLuetticke.notebooks.DCT.ipynb
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.notebooks.OneAsset': no module named HARK.BayerLuetticke.notebooks.OneAsset

Exception occurred:
  File "/home/sb/projects/econ-ark/HARK/HARK/BayerLuetticke/notebooks/TwoAsset.py", line 162, in <module>
    get_ipython().run_line_magic('matplotlib', 'auto')
NameError: name 'get_ipython' is not defined
The full traceback has been saved in /tmp/sphinx-err-p53q6tm1.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
Makefile:63: recipe for target 'html' failed
make: *** [html] Error 1

As this seems to be due to an issue with a specific notebook and its rendering into the docs, and not the dependency incompatibility issues, I'm going to make a separate issue for this.

Generically, there is no strong reason for any particular notebook to be
included either in the docs or in the Travis testing.  Our approach has
been mostly "if a notebook doesn't cause problems, great, include it; if
not, leave it out."   So, feel free to leave out BayerLuetticke, if it is
causing problems.
On Tue, Oct 22, 2019 at 11:06 AM Sebastian Benthall <
<EMAIL_ADDRESS>wrote:

Fixing this requirements issue and rerunning make html results in this
error:
$ make html
sphinx-build -b html -d ../../HARK-docs/doctrees   . ../../HARK-docs/html
Running Sphinx v1.6.7
3.6.8 |Anaconda custom (64-bit)| (default, Dec 30 2018, 01:22:34)
[GCC 7.3.0]
Inserting /home/sb/projects/econ-ark/HARK
/home/sb/anaconda3/lib/python3.6/site-packages/sphinx/util/compat.py:40: RemovedInSphinx17Warning: sphinx.util.compat.Directive is deprecated and will be removed in Sphinx 1.7, please use docutils' instead.
RemovedInSphinx17Warning)
loading pickled environment... not yet created
/home/sb/projects/econ-ark/HARK/Documentation/notebooks/DCT-Copula-Illustration.rst: WARNING: document not readable. Ignored.
loading intersphinx inventory from https://docs.python.org/objects.inv...
intersphinx inventory has moved: https://docs.python.org/objects.inv -> https://docs.python.org/3/objects.inv
[autosummary] generating autosummary for: CONTRIBUTING.md, HARKmanual.md, HARKmanual/ContributingToHARK/ContributingToHARK.md, HARKmanual/HARKmanual.md, generated/HARK.BayerLuetticke.Assets.One.FluctuationsOneAssetIOUs.rst, generated/HARK.BayerLuetticke.Assets.One.FluctuationsOneAssetIOUsBond.rst, generated/HARK.BayerLuetticke.Assets.One.SharedFunc.rst, generated/HARK.BayerLuetticke.Assets.One.SharedFunc2.rst, generated/HARK.BayerLuetticke.Assets.One.SteadyStateOneAssetIOUs.rst, generated/HARK.BayerLuetticke.Assets.One.SteadyStateOneAssetIOUsBond.rst, ..., generated/HARK.BayerLuetticke.Assets.Two.SharedFunc3.rst, generated/HARK.BayerLuetticke.Assets.Two.SteadyStateTwoAsset.rst, generated/HARK.BayerLuetticke.Assets.Two.defineSSParametersTwoAsset.rst, generated/HARK.BayerLuetticke.ConsIndShockModel_extension.rst, index.rst, notebooks/Gentle-Intro-To-HARK.ipynb, notebooks/README.rst, packaging.md, readme.md, submodules/README.rst
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.Assets.One.Luetticke_wrapper': no module named HARK.BayerLuetticke.Assets.One.Luetticke_wrapper
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.Assets.Two.EX3SS_20_python2.p': no module named HARK.BayerLuetticke.Assets.Two.EX3SS_20_python2.p
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.Assets.Two.EX3SS_20_python3.p': no module named HARK.BayerLuetticke.Assets.Two.EX3SS_20_python3.p
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.BayerLuetticke_wrapper': no module named HARK.BayerLuetticke.BayerLuetticke_wrapper
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.notebooks.DCT': no module named HARK.BayerLuetticke.notebooks.DCT
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.notebooks.DCT.ipynb': no module named HARK.BayerLuetticke.notebooks.DCT.ipynb
WARNING: [autosummary] failed to import 'HARK.BayerLuetticke.notebooks.OneAsset': no module named HARK.BayerLuetticke.notebooks.OneAsset
Exception occurred:
File "/home/sb/projects/econ-ark/HARK/HARK/BayerLuetticke/notebooks/TwoAsset.py", line 162, in
get_ipython().run_line_magic('matplotlib', 'auto')
NameError: name 'get_ipython' is not defined
The full traceback has been saved in /tmp/sphinx-err-p53q6tm1.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at https://github.com/sphinx-doc/sphinx/issues. Thanks!
Makefile:63: recipe for target 'html' failed
make: *** [html] Error 1
As this seems to be due to an issue with a specific notebook and its
rendering into the docs, and not the dependency incompatibility issues, I'm
going to make a separate issue for this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/406?email_source=notifications&email_token=AAKCK73GFVVQ4HHQRS7M3B3QP4JHZA5CNFSM4JCRRSM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB6CU7Y#issuecomment-545008255,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKCK74VGWQBOMWR3AOWZ3LQP4JHZANCNFSM4JCRRSMQ
.

--

Chris Carroll


chore(master): release 2.5.0

:robot: I have created a release beep boop
2.5.0 (2024-06-13)
Features

deptrac and infra layer (3a90d44)

Bug Fixes

stryker dashboard mutation score (3360705)
stryker dashboard mutation score (4faa023)

Documentation

add contributing guidelines (6f37090)
update readme.md (6f8f7b4)

Dependencies

deps: update googleapis/release-please-action action to v4.1.3 (2a5014e)
deps: update shivammathur/setup-php action to v2.30.5 (3c7535c)

Continuous Integration

add deptrac support (ef7afdd)
add rector (a3b7dd4)
add rector workflow (3c1be89)
minor updates (9b31ba3)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.5.0
:sunflower:


Third party server support for 7.1.1?

I understand that ES is not going to support communication with servers maintained by 3rd parties.
Looks like the elasticsearch-ruby already added the check here
https://github.com/elastic/elasticsearch-ruby/commit/8a0e6d8f98e1f9670bcc749bc615796fc9b68ddb#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4ed
I assume that the same will happen to this repo.
we are currently using the elastic search cluster of amazon and it will take some time for us to migrate to the official ES cluster.
for the time being, my rails app has this:
gem 'elasticsearch-model', '7.1.1'
gem 'elasticsearch-rails', '7.1.1'

Please let us know if we are covered for the time being and hopefully, nothing will break?

Hi @vamsipavanmahesh,
Since elasticsearch-model depends on elasticsearch ~> 7, I'd suggest locking this dependency in your app's Gemfile to be less than 7.14, the version where the check was introduced:
gem 'elasticsearch-model', '7.1.1'
gem 'elasticsearch-rails', '7.1.1'
gem 'elasticsearch', '< 7.14'

This way you'll get the latest version of elasticsearch-ruby before 7.14 if you update your gems.

Thanks @picandocodigo for your inputs :)


Fixes #1258

Description
Fixes #1258
Wraps string stored in strings.xml with CDATA to prevent PHP's SimpleXML return null when string contains other tags.

Codecov Report

Merging #1261 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1261   +/-   ##
======================================
  Coverage    3.78%   3.78%
======================================
  Files         125     125
  Lines        5866    5866
  Branches      578     578
======================================
  Hits          222     222
  Misses       5629    5629
  Partials       15      15

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 51221f6...0a45e0c. Read the comment docs.

Am not very familiar with how wiki translations work.
@nicolas-raoul @misaochan Can you guys confirm if this will fix the issue. :)

@Nikerabbit has mentioned in #1258 that CDATA does indeed fix the problem. To test this, I think we just need to look at the relevant strings in the app and confirm that our own parsing works and that the strings look OK.


An unexpected error occurred while processing your document.

we  are getting following error while uploading document on Watson tried by API endpoints.
Response
{
  "document_id" : "cdb7c1ec-3c5e-462e-b721-ec098d9df2a0",
  "filename" : "filename.pdf",
  "file_type" : "pdf",
  "sha1" : "8f9fa3dd0382172131d5e47b860f30c68152bee4",
  "status" : "failed",
  "status_description" : "Document failed to be ingested and indexed",
  "notices" : [ {
    "notice_id" : "internal_error",
    "document_id" : "cdb7c1ec-3c5e-462e-b721-ec098d9df2a0",
    "created" : "2019-03-06T09:08:49.064Z",
    "severity" : "error",
    "step" : "pdf",
    "description" : "An unexpected error occurred while processing your document."
  } ]
}

Environment Details
Storage: 6.97 MB / 200 MB
Documents: 5 / 1,000
Collections: 1 / 2

Hello @harikeshoodles. Need to ask a couple questions:

are you using the API on your own, or just running the app in this code pattern?
this code pattern uses json files, and this is a pdf, which means you are using your own data. Correct?
can you try and add the one document using the Discovery service tooling within IBM Cloud? (See Step #3 of the README for an example)

No response - closing.


Upgrade dependencies to work with current versions.

The "apt" module was the main one that required updating- the others did not have any changes and just needed to have their versions upgraded.

Looks good, thanks!  Make the tests past and I'll merge the PR.
Stephen.

Any luck with the tests?

I've got some working tests that I'll apply, thanks for the PR!


Add font fallback on social images

Description
Just added a sans-serif fallback for social images to check if there is a problem with Arial font. Also wrapped the function inside a try-catch to check if some error is logged (even if the images are created). Also I changed the generate function to be a named function to improve the trace for any error.

hm strange, i'm actually getting an error with the font on the Netlify deploy with that 🤔
https://cards-dev.twitter.com/validator

not sure if you can see this but not getting any errors interestingly enough https://app.netlify.com/sites/next-wordpress-starter-dev/deploys/60eeea4767d91f0007a344b2

hey @GuilleAngulo just making sure you saw my note here

hey @colbyfayock sorry for the late reply. As it is difficult to say what could be happening, I'll make a separate deploy on my Netlify account to make some tests and see if I can figure out why the fonts are not working.


About the ssh protocal listening port opened by bridge.

Hi there,
See the following example on my Ubuntu 22.04.4 LTS:
werner@x13dai-t:~$ bridge -b :18080 -p - -p socks5h://<IP_ADDRESS>:16668
2024/07/17 21:50:50 INFO DIAL "-" <- "socks5h://<IP_ADDRESS>:16668" <- LOCAL <- "http://:18080|socks5://:18080|socks4://:18080|ssh://:18080" LISTEN
 chains="{Bind:[{Probe: LB:[http://:18080 socks5://:18080 socks4://:18080 ssh://:18080]}] Proxy:[{Probe: LB:[-]} {Probe: LB:[socks5h://<IP_ADDRESS>:16668]}] IdleTimeout:0s}"

As you can see, bridge says it's listening on ssh://:18080, but how can I use it this way?
Regards,
Zhao

bridge -b :10022 -p -

# Convert ssh proxy to socks with ssh command
ssh -D 1080 -C -N -p 10022<EMAIL_ADDRESS>
Or
# Convert ssh proxy to socks5 with bridge
bridge -b socks5://:1080 -p - -p ssh://<IP_ADDRESS>:10022

# test
curl -x socks5://<IP_ADDRESS>:1080 https://example.org/

# Convert ssh proxy to socks with ssh command
ssh -D 1080 -o "StrictHostKeyChecking no" -o "UserKnownHostsFile /dev/null" -C -N -p 10022<EMAIL_ADDRESS>

Why use root instead of a regular user here?

This does not matter, it can be any user, because the server does not auth it.


Create new scope Search and implement Binary Search

Create new scope Search (Algorithms), implementing fundamental search algorithms.
Also implement Binary Search.
Reference: https://en.m.wikipedia.org/wiki/Binary_search_algorithm

Fixed in #119 by @kdimolikas.


Enable preview feature and update Java version

Updating Java version after Java19 was released past month.

Merged the commit with the flaky tests disabled, and green now 👍


How to get the actual rank using trainer.predict()?

Describe the Question
I am trying to get the rank out of a trained model (using trainer). However, when I do trainer.predict() I get back a numpy array of shape num_qids x 1. The number of query ids .predict returns is depending on the dataloader dl passed on trainer.predict(dl).
In other words, as I understand I get a score (probably the first metric I've defined on metrics?) for each query id. However, what I need is a ranked list of  documents for each query id, rather than a single score.
How can I get that? I could find no solution through the tutorials.
My code looks like:

    trainer.run()

    # Evaluation
    print('Validation results:')
    print(trainer.evaluate(valid_dl))
    print('Test results:')
    print(trainer.evaluate(test_dl))

    val_preds = trainer.predict(valid_dl)
    test_preds = trainer.predict(train_dl)

val_preds.shape
>> Out[18]: (150, 1)
valid_dl.label.shape
>> Out[19]: (150,)

Having more details on evaluate, it would be useful

I think when you use .predict, you will get a list of scores. You can sort on those scores and then get the order of it using np.argsort. Then you can get the corresponding document using the order you obtained. This is my understanding and hopefully it is helpful.

Hi, thanks for your message. You are correct. The problem was that I was
using a dataset creation function of MZ that sampled (positives, negatives
and shuffled) to create the dataset, so there was a mismatch in input and
output.
On Tue, Jun 16, 2020, 17:32 shimengfeng<EMAIL_ADDRESS>wrote:

I think when you use .predict, you will get a list of scores. You can sort
on those scores and then get the order of it using np.argsort. Then you can
get the corresponding document using the order you obtained. This is my
understanding and hopefully it is helpful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NTMC-Community/MatchZoo-py/issues/136#issuecomment-644840213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACZZRYIO573JRFR4ZZ4XV4TRW6F7ZANCNFSM4LTVLMQQ
.


Fix OSMemoryBarrier deprecation warning on OS X

Remove deprecated OSMemoryBarrier in favour of an implementation based on std::atomic.
See the warning below on macOS 10.12:
In file included from ./port/port_posix.h:47:
./port/atomic_pointer.h:59:3: warning: 'OSMemoryBarrier' is deprecated: first deprecated in macOS 10.12 - Use std::atomic_thread_fence() from <atomic> instead [-Wdeprecated-declarations]
  OSMemoryBarrier();
  ^
/usr/include/libkern/OSAtomicDeprecated.h:749:9: note: 'OSMemoryBarrier' has been explicitly marked deprecated here
void    OSMemoryBarrier( void );
        ^
1 warning generated.

A variant of this was landed in 594cc987af2e0af6417c4ac2b947ee8cdad59e5e. Thank you very much for your contribution!


Remove case5 for ACP due to restoration error

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature-limits@48ab82a). Click here to learn what that means.
The diff coverage is 100%.

@@               Coverage Diff                @@
##             feature-limits     #20   +/-   ##
================================================
  Coverage                  ?   96.8%
================================================
  Files                     ?      10
  Lines                     ?    1282
  Branches                  ?       0
================================================
  Hits                      ?    1241
  Misses                    ?      41
  Partials                  ?       0

Impacted Files
Coverage Δ

test/power_models_acp.jl
100% <ø> (ø)

src/model.jl
98.92% <100%> (ø)

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 48ab82a...f495946. Read the comment docs.


tests, enforce TLSv1 for curl,  get favicon from atom feed

Closing this for now. Need some time to update and test this stuff.


fix(build): esm mapping

Currently, our mapping use absolute paths instead of relative, and the keys do not match the import statements:

This PR changes it to be like this:

I am in the process of verifying if this really works


Add Windows support

Although Rice oficially doesn't support Windows, it works with the following changes.
https://jasonroelofs.com/rice/

@zakjan huh interesting. I'm not much of a C++ guy - I assume #include <iostream> will still work with *nix compilers?

Yes, compiles on Linux, Mac, Windows

You can check the compiler input in Github Actions output at https://github.com/lutaml/expressir/pull/33

Good enough for me :)

@zakjan is this a good time to release a new version of the gem?

Yes, I'm done, thanks :)

Published along with #3, #4, and #5 as v1.1.0.


Mod crashes on reload after newest update

This error only happens on reload, not when initially loading mods.

i'm having the same issue

Should be addressed now.


How to RunComplie on updated template using same key?

string template = "Hello @Model.Name, welcome to RazorEngine!";
var result = Engine.Razor.RunCompile(template, "templateKey", null, new { Name = "World" });

Now i update my existing template to as below. I get my template from database.
string template = "Hello @Model.Name, welcome to My World!";

Whenever i do that i get an error The same key was already used for another template.
What is the best way to fix this issue?

I found this link about template manager .
I update the code to use MyTemplateManager() and it's working fine.
var config = new TemplateServiceConfiguration();
config.Debug = true;
config.TemplateManager = new MyTemplateManager(template);
Engine.Razor = RazorEngineService.Create(config);
Engine.Razor.RunCompile(template, null, viewModel);

I am not sure if it's creating new key every time or using the same key and updating the new template.

HI Prakash,
Did you define the class MyTemplateManager youself?
Could you help to share the code or do you have document for this?
Regards

did you find any workaround of this issue ?


Implement JSON unmarshaling for Any

The Any type has already support for marshalling, but the unmarshaling of it was left unimplemented.
This PR simply implements the clearly missing feature. I tired to stay as close as possible in terms of approach and coding style as similar code in this file.
We get the test almost for free.

Hey @dsymonds,
I noticed you were the author of the Any marshaling part and you left unmarshaling as TODO.
This was originally tracked in #170. Should I open a new issue or can we just continue here?
cheers,
mkm

@calbach I noticed you add support for this in 18c9bb3261723cd5401db4d0c9fbc5c3b6c70fe8
which also handles unmarshaling well known types .
I believe this makes this PR no longer necessary, thus I'm abandoning it.


[Task]: Remove the stale index lock file which created at the time of pod restart

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Sometimes when the pods restart on kubernetes during a git operation, a stale lock file is created. As of now, user need to manually remove this file to continue with git operations. Ideally We should remove this file automatically

Can we instead solve this at the application-level? Like, before performing a git operation, check for an existing lock file, and if it exists, delete it and then perform the operation? That way, if a lock file is leftover for any other reason, we're still resilient to all such cases.

Event Name git_stale_lock_file_deleted
Properties

default properties(segment source name, version, instance_id)
repo_url
lock_file_age (how old the lock file is - in seconds)


Feature: Possibility to set group as default for first time logged users via SSO credentials

🔖 Feature description
Hi, Would you please consider to add simple checkbox next to the group to mark this group as default for all new users logged via SSO, in my case Azure
🎤 Pitch
All new users logged and created using SSO by default receive super admin privileges. Will be nice to limit this as not every one should be able be admin when first time see Devtron
✌ Solution
Add something like "Default group for SSO" checkbox
🔄 Alternative
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#7085

Hey @jakubdzikowski thanks for opening this issue, we will be picking this in our upcoming sprint. You can expect this in our next release.

Dropping the default group, supports Azure AD group mapping: https://docs.devtron.ai/global-configurations/authorization/sso-login/microsoft#auto-assign-permissions


Videos path fix experiment

closes #201

is this a draft pr or is it ready to merge

is this a draft pr or is it ready to merge

Darft

A, ok

I love how the crash handerler works, making the "Can turn your bad pc into a better pc if you are having trouble lagging hard in the base one." mostly true; however, did you actully forget to add the offical fnf songs to the base of your no lag engine?

I love how the crash handerler works, making the "Can turn your bad pc into a better pc if you are having trouble lagging hard in the base one." mostly true; however, did you actully forget to add the offical fnf songs to the base of your no lag engine?


When using Node 21 and Docker I get the following error Error: Cannot find module<EMAIL_ADDRESS>

Is there an existing issue for this?

[X] I have checked for existing issues https://github.com/getsentry/profiling-node/issues
[X] I have reviewed the documentation https://docs.sentry.io/
[X] I am using the latest SDK release https://github.com/getsentry/profiling-node/releases

How do you use Sentry?
Sentry Saas (sentry.io)
SDK Version
1.3.5
Link to Sentry event
No response
What environment is your node script running in?
Express app inside docker container using node:alpine image.
How is your code deployed and bundled?
It's plain javascript express server, and I'm running it inside a docker container as follows
FROM node:21-alpine

#TODO add proper user configurations in this dockerfile
COPY package*.json ./

# Use the default working directory for Node.js
WORKDIR /usr/src/app

RUN npm install

COPY . .

EXPOSE 8080

CMD [ "node", "index.js" ]

My integration for Sentry and the ProfilingIntegration looks like this
// rest of the conde
import * as Sentry from '@sentry/node';
import { ProfilingIntegration } from '@sentry/profiling-node';
import packageJson from './package.json' assert { type: 'json' };

const PORT = process.env.PORT || 8080;
const HOST = process.env.HOST || '<IP_ADDRESS>';

//TODO Go through express configuration and check the best configurations  on OWASP
const app = express();

//Sentry integration
Sentry.init({
  dsn: process.env.SENTRY_DSN,
  integrations: [
    // enable HTTP calls tracing
    new Sentry.Integrations.Http({ tracing: true }),
    // enable Express.js middleware tracing
    new Sentry.Integrations.Express({ app }),
    new ProfilingIntegration(),
  ],
  // Performance Monitoring
  tracesSampleRate: 1.0, //  Capture 100% of the transactions
  // Set sampling rate for profiling - this is relative to tracesSampleRate
  profilesSampleRate: 1.0,
  environment: process.env.ENVIRONMENT,
  release: packageJson.version,
  debug: true
});
//rest of the code

package.json looks like this
  "dependencies": {
    "@aws-sdk/client-s3": "^3.288.0",
    "@sentry/node": "^7.99.0",
    "@sentry/profiling-node": "^1.3.5",
    "axios": "^0.27.2",
    "cors": "^2.8.5",
    "dotenv": "^16.0.2",
    "express": "^4.18.1",
    "npmlog": "^6.0.2"
  },
  "devDependencies": {
    "@typescript-eslint/eslint-plugin": "^6.19.1",
    "@typescript-eslint/parser": "^6.19.1",
    "eslint": "^8.56.0",
    "nodemon": "^2.0.19",
    "prettier": "3.0.3"
  }

Steps to Reproduce

Installed @sentry/node and @sentry/profiling-node
Configured following documentation
Setup docker file
Build docker container running docker build --no-cache -t efs .
Run docker container using node:alpine image with the following command docker run -p 8080:8080 efs

Expected Result
Docker container runs on latest image
Actual Result
❯ docker run -p 8080:8080 efs
(node:1) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
info S3Client Initializing with us-east-2
node:internal/modules/cjs/loader:1149
  const err = new Error(message);
              ^

Error: Cannot find module<EMAIL_ADDRESS>Require stack:
-<EMAIL_ADDRESS>    at Module._resolveFilename (node:internal/modules/cjs/loader:1149:15)
    at Module._load (node:internal/modules/cjs/loader:990:27)
    at Module.require (node:internal/modules/cjs/loader:1237:19)
    at require (node:internal/modules/helpers:176:18)
    at importCppBindingsModule (/node_modules/@sentry/profiling-node/lib/index.js:19266:10)
    at Object.<anonymous> (/node_modules/@sentry/profiling-node/lib/index.js:19268:34)
    at Module._compile (node:internal/modules/cjs/loader:1378:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1437:10)
    at Module.load (node:internal/modules/cjs/loader:1212:32)
    at Module._load (node:internal/modules/cjs/loader:1028:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [<EMAIL_ADDRESS>]
}

Node.js v21.6.1

And it's correct, because, in fact there's no sentry_cpu_profiler-linux-arm64-musl-120.node file in my node_modules.
Also I checked the code for this library and even though I don't understand completely what's happening inside, but there's no check for that 120 anywhere to be found in src/cpu_profiler
This also happened in my M1 Macbook pro when running node 21
(node:2350) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
info S3Client Initializing with us-east-2
node:internal/modules/cjs/loader:1149
  const err = new Error(message);
              ^

Error: Cannot find module<EMAIL_ADDRESS>Require stack:
-<EMAIL_ADDRESS>    at Module._resolveFilename (node:internal/modules/cjs/loader:1149:15)
    at Module._load (node:internal/modules/cjs/loader:990:27)
    at Module.require (node:internal/modules/cjs/loader:1237:19)
    at require (node:internal/modules/helpers:176:18)
    at importCppBindingsModule (/Users/osauceda/Documents/x/efs/node_modules/@sentry/profiling-node/lib/index.js:19266:10)
    at Object.<anonymous> (/Users/osauceda/Documents/x/efs/node_modules/@sentry/profiling-node/lib/index.js:19268:34)
    at Module._compile (node:internal/modules/cjs/loader:1378:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1437:10)
    at Module.load (node:internal/modules/cjs/loader:1212:32)
    at Module._load (node:internal/modules/cjs/loader:1028:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
<EMAIL_ADDRESS>  ]
}

Node.js v21.6.1

It's worth mentioning that this only happens when I run node 21, but whenever I change to node 18, the profiler works as expected

We're having the same issue and downgrading to node 18 works as you said. We do however prefer to use node 21 which is a problem.

Same issue with Node 20

This is expected.

We currently ship prebuilt binaries for a few of the most common platforms and node versions (v16-20).
https://github.com/getsentry/profiling-node/tree/v1.3.5?tab=readme-ov-file#prebuilt-binaries

@lukeocodes we do not support non LTS (odd) versions of nodejs. We will release a new version when node 22 reaches LTS


Chat - New message icon not show when receive message when user not at LHN

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.58-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
**Email or phone of affected tester (no<EMAIL_ADDRESS><EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Pre-condition: 2 account with conversation
0. Openn app in 2 accounts

Account A, please open another page ( Eg, Profile)
Account B, send message to A
Observe at account A

Expected Result:
Show "new message" icon, notify for user about new message
Actual Result:
New message icon not show when receiving message when the user is not at LHN
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/ad50c2ce-bebe-4b48-bf47-6cc5b2615c48

View all open jobs on GitHub

@muttmuure  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Yeah I agree with that, the dot on the icon should match any red or green brick road dots found in the chats list.


[HUDI-3183] Wrong result of HoodieArchivedTimeline loadInstants with TimeRangeFilter

https://issues.apache.org/jira/browse/HUDI-3183
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Hoodie use ArchiveFileVersionComparator  to sort archive files as an optimization to skip reading unnecessary archived files.
But with the wrong regex pattern, there's no sort actually(always return 0) and lead to a wrong loadInstants result.
https://github.com/apache/hudi/blob/b5f05fd153df29a8be377404a14a0ced2f00b4bf/hudi-common/src/main/java/org/apache/hudi/common/table/timeline/HoodieArchivedTimeline.java#L219
https://github.com/apache/hudi/blob/b5f05fd153df29a8be377404a14a0ced2f00b4bf/hudi-common/src/main/java/org/apache/hudi/common/table/timeline/HoodieArchivedTimeline.java#L297
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

Hi @nsivabalan Thanks a lot for your help!


How do referenceMentions offsets work?

In the paper contents JSON, under metadata.referenceMentions, there are objects with startOffset and endOffset fields. After some exploration, I was unable to determine what they are offsets of and what they mean. Are those the boundaries of the content in the context field? Where? Are they character offsets, or somehow word offsets?

Here is an example

In this example, The reference appear at context[startOffset: endOffset]

Oh, thanks. I thought this would be something completely different. I had my mind on locating where the context appeared in the paper to see the surrounding paragraph, but I solved that in other ways.
Thanks for the reply!


FDG Curl module

@fossam You'll have to tell me if this has any gaps around your use case. I left it pretty open (which I think is still ok from a security standpoint since I disabled fdg chaining for this module)

Merging. I'll create a new PR for any additions we need as a result of @fossam's discussion with me.


Update defaults for each pulldown menu

Now that everything will follow the same format, can we update the defaults for what is shown in the "X axis", "Y axis", "color by", "size by" pulldowns?
How it's written now, the new defaults should be:
X-axis: PC1
Y-axis: PC2
color: grp
size: support
"support" will still be a DAPC-only thing, but that's fine in my mind. I bring it up in the manuscript as an option for DAPC only.

I think this should be set in the config file (re: #29) rather than having hard coded defaults. I'll keep this here to track the desired content of that file.

Note that grp may also be absent, as we now allow other methods than DAPC. As far as I can tell, the only fields that will be always there are:

key: unique individual ID
PC1: 1st principal component
lat: latitude
lon: longitude

Yeah, that sounds good @thibautjombart
The defaults aren't that important, since you can always just switch whats displayed when you start up an instance of the app. I also make a note of using "grp" to refer to population in the manuscript.


Feature fp solver

Summary
This PR addresses https://github.com/stan-dev/math/issues/1370, namely to add fixed point algebra solver based on KINSOL implementation, in order to solve system
x = f(x; theta)
The implementation differs from current Newton solver(also based on KINSOL) in that no AD is performed during the iteration from i to i+1
x_(i+1) = f(x_(i); theta)
After convergence(so that x = f(x; theta) approximately), gradient dx/d(theta) is obtained by performing one jacobian function call and one linear system solving. This is similar to current implementation of algebra solver in how Jxy is calculated.
Main the performance advantage is a result of

no AD during iteration
Anderson acceleration (the implementation has fixed Anderson depth 4).

The function has signature
  template <typename F, typename T1, typename T2>
  Eigen::Matrix<T2, -1, 1>
  algebra_solver_fp(const F& f,
                    const Eigen::Matrix<T1, -1, 1>& x,
                    const Eigen::Matrix<T2, -1, 1>& y,
                    const std::vector<double>& x_r,
                    const std::vector<int>& x_i,
                    const std::vector<double>& u_scale,
                    const std::vector<double>& f_scale,
                    std::ostream* msgs = nullptr,
                    double f_tol = 1e-8,
                    int max_num_steps = 200)

where

x: init point of iteration
y: parameter
x_r: real data
x_i: int data
u_scale: scaling vector for unknown(ref. KINSol function documentation)
f_scale: scaling vector for residual(ref. KINSol function documentation)
msgs: output stream
f_tol: scaled residual tolerance.
max_num_steps: max nb. of iterations allowed.

Tests
algebra_solver_fp_test.cpp contains
TEST_F(FP_exp_func_test, solve)
TEST_F(FP_2d_func_test, solve)
TEST_F(FP_exp_func_test, gradient)
TEST_F(FP_2d_func_test, gradient)
TEST_F(FP_2d_func_test, gradient_with_var_init_point)
TEST_F(FP_2d_func_test, algebra_solver_fp)
TEST_F(FP_2d_func_test, exception_handling)

based on two test fixtures: FP_exp_func_test for 1D system and FP_2d_func_test for 2D system.
Side Effects
N/A
Checklist

[x] Math issue #1370

[x] Copyright holder: Metrum Research Group
By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Thanks for the updates. The main remaining point of discussion is how to handle u_scale and f_scale, see my comments.

Thanks for addressing my comments. Once the continuous integration passes, you may merge the PR.

Has this thing made it to the Stan language? I am not sue I have seen it in the docs.

no.

Is it not meant to be exposed? Or is there already an issue for it?

It's meant to be exposed, just not my priority as currently I only need it internally, and I don't really have a good idea on documenting it(and the other two solvers) for general users, as requested here. Feel free to create an issue tho.


Resolves #31

Added handling for all status codes.

Changes Unknown when pulling 635ec82670568e70b30a7308e4773e79f7b6f1de on issue-31 into ** on master**.


refactor: 重构 prefixCls，AppContainer 等

💻 变更类型 | Change Type

[ ] ✨ feat
[x] 🐛 fix
[x] ♻ refactor
[ ] 💄 style
[x] 🔨 chore
[ ] ⚡ perf
[ ] 📝 docs

🔀 变更说明 | Description of Change

📝 补充信息 | Additional Information

Codecov Report
Attention: 43 lines in your changes are missing coverage. Please review.

Comparison is base (ed9ed53) 81.41% compared to head (ede2e43) 81.32%.
Report is 2 commits behind head on main.

Files
Patch %
Lines

src/ProBuilder/components/NavBar/style.ts
4.34%
22 Missing :warning:

src/ProBuilder/components/Canvas/Component.tsx
7.14%
13 Missing :warning:

docs/pro-editor/demos/controlledPresence.tsx
0.00%
1 Missing :warning:

src/ContextMenu/MenuItem/style.ts
0.00%
1 Missing :warning:

src/ContextMenu/index.tsx
0.00%
1 Missing :warning:

src/IconPicker/features/IconfontScript.style.tsx
50.00%
1 Missing :warning:

src/ProBuilder/components/NavBar/index.tsx
0.00%
1 Missing :warning:

src/ProBuilder/components/Stage/index.tsx
50.00%
1 Missing :warning:

src/ProBuilder/components/Stage/style.ts
50.00%
1 Missing :warning:

src/SortableTree/components/TreeItem/index.tsx
94.44%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
- Coverage   81.41%   81.32%   -0.10%
==========================================
  Files         405      399       -6
  Lines       25978    25861     -117
  Branches     1163     1156       -7
==========================================
- Hits        21150    21031     -119
- Misses       4828     4830       +2

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


ContextFlyout not working properly after double-click

Describe the bug
After double-click the ContextFlyout is dead, the flyout doesn't show up anymore.
Clicking outside and then single click again fixes it.
Issue appears again after you double-click.
To Reproduce
Double-click
Screenshots
Screencast from 26-05-2023 16:09:51.webm
Desktop (please complete the following information):
Debian
Additional context
First time noticed it here https://github.com/zkSNACKs/WalletWasabi/issues/7753 after that, could also repro here https://github.com/zkSNACKs/WalletWasabi/pull/10767

It looks like you have a heavily customized control there. Is this reproducible with stock Avalonia controls in some way?
You also don't include any of the Avalonia version information which is pretty important right now.

The version is 0.10.21. The behavior as I know should be the same as in vanilla Avalonia since only the theme is different. Can try to make a repro and perhaps a fix with it.


Remove try... convert of integers (E3002)

In the property Type check, when the property value should be an integer (but it's not), there is an attempt to convert it:
https://github.com/awslabs/cfn-python-lint/blob/master/src/cfnlint/rules/resources/properties/Properties.py#L56
Since both YAML and JSON just handle these types out the box, the try..except can in theory be removed (since the type's don't match, a string is not an integer).
Personally I like the check to be more "strict', who do other think?
Example (JSON):
https://github.com/awslabs/cfn-python-lint/blob/master/test/templates/quickstart/vpc-management.json#L816
Example (YAML):
https://github.com/awslabs/cfn-python-lint/blob/master/test/templates/good/resource_properties.yaml#L464
These values can easily be written as actual integers
      - CidrIp:
          Ref: SSHLocation
        FromPort: 22
        IpProtocol: tcp
        ToPort: 22

I actually have the reverse - an invalid integer where a String is required by CFN itself, that cfn-lint doesn't catch. The CloudFormation Designer doesn't catch it either.
In a https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-elasticbeanstalk-configurationtemplate-configurationoptionsetting.html all properties must be Strings.
For example, an entirely reasonable-looking, but wrong use of an integer:
{
                        "Namespace": "aws:autoscaling:asg",
                        "OptionName": "MinSize",
                        "Value": 12
},

Does anyone know if the same bit of code is related, or do I need to hunt for a bug in the Specification?

@finchd and this fails when you launch to the stack?  I think it’s a similar issue to what is being described.  I think it’s a fair request by both to say that if it’s a string it’s a string and if it’s a number it’s a number and try not to translate it.   We should be capturing your issue if that stack failed to deploy because of that issue. I will need to track that down if that’s the case.  I’m not sure this is a spec issue as it looks like they use that generic property for string and number based values.

Yes, it definitely fails to create the stack: "Value of property Value must be of type String"
aws cloudformation validate-template succeeds with no errors either.

After chatting with @kddejong tonight, I'm in the camp of strong enforcement of types.  It may trigger a lot of errors/warnings in old templates, but it's an opportunity to polish them up.  It'll be like when I first added the Flake8 linter to Atom. :)

This change would catch @finchd error and satisfy this issue for @fatbasstard .  I'm good with it too.  Interesting that @finchd found a spot where integers aren't turned into strings as there are other spots where they are.   I don't want to get into the boat of writing exceptions to rules based on certain scenarios so I also like the idea of sticking to the spec the way its written.
I have one last question for this.  This change may be annoying to some because the template could still work (in some cases).  So should we move this to a new rule with a new number?  That way people can disable it if they want.

I have one last question for this. This change may be annoying to some because the template could still work (in some cases). So should we move this to a new rule with a new number? That way people can disable it if they want.

They can also disable the current one, right?
Well, really love the "being strict", so 🎉 on that!

Any change this (https://github.com/awslabs/cfn-python-lint/pull/83) is release before the next release? https://github.com/awslabs/cfn-python-lint/pull/84 will


blender_script renders data very slowly

Hello! I would like to use blender_script to render input and target views. But it took me more than 3 days to render just 10000 objects. How long does it cost for you to render the whole dataset?

Hi, the rendering requires a GPU and it needs faster IO of the model file. This could be the bottleneck for the rendering speed. I use about 2 weeks to render 800k models using 8 V100 GPUs.

Great thanks! You remind me to use multi-GPU and multi-process.

Have you solved your problem? @thucz
Are you using Cycle Engine for rendering?

Hi, @thucz. Have you solved your problem? Are you using Cycle Engine for rendering?

Not completely. Although I used multi-gpu, it still takes a lot of time. I'm using Cycle Engine


Add PostgreSQL 16, remove Postgres 10 and 11

Description
Add PostgreSQL 16 to the versions of choice.
Checklist:

[x] I've made sure that tests are updated accordingly (especially if adding or updating a template option)
[x] I've updated the documentation or confirm that my change doesn't require any updates

Rationale
Psycopg 3 supports PostgreSQL from version 10 to 16. Fix #4920

@nixsiow thanks for your PR. According to the Django docs

Django supports PostgreSQL 12 and higher

So we can remove version 10 & 11, don't you think?

Thanks! I've made a new commit. Hopefully I've done it right. :)

You have done 99% right, the only minor detail you missed is the commit message (add postgresql 15) but I will squash it and fix the commit message ;)
Thanks for your contribution!


Character count remaining characters message not visible

This issue is from a May 2019 external accessibility audit report.
WCAG Reference: Usability feedback only, there is no WCAG related guidelines.
Issue ID: DAC_22
URL: http://govuk-frontend-v3.herokuapp.com/full-page-examples/feedback
Screen Shot

There is a character count that has been added to the text areas, however, the first ‘What are you trying to do?’ form field’s character count is not functioning and is not visible on the page. This is due to it being hidden using the CSS property of visibility:hidden; Although this may have been intentionally hidden, ZoomText users that rely on audio feedback were able to hear this information which was confusing as it was not visible to them on the page.
Current Code Ref(s)

  What were you trying to do?  <textarea class="govuk-textarea js-character-count" id="what-were-you-trying-to-do" name="what-were-you-trying-to-do" rows="5"></textarea> You have 100 characters remaining

.govuk-character-count__message--disabled { visibility: hidden; }

ZoomText user comments

“Asked to enter more characters than required, I noted that there was no visible instruction next to the form to make the user aware of this limit. Also, by randomly swiping across the bottom of the form, I noted an audible label informing me of a character limitation, but this was not visible. Having entered excessive characters, a visible message did inform a user of this.”

Solution
Ensure that, if the count is to be hidden, that it is hidden from all users, which may require the use of display:none and aria-hidden.

Closed in #1442


Follow-Up for #411:  Should unavailable Catalogs be ignored during resolution?

Mentioned in https://github.com/operator-framework/operator-controller/pull/411#discussion_r1332190866 by @joelanford:

Follow-up: we probably should not ignore unavailable catalogs. If we do, we introduce non-determinism to resolution.
If a Catalog that is needed for resolution is unavailable (that's all Catalogs right now because there's no way to disable catalogs globally or for a particular Operator), I think we should fail resolution, report the error, and then requeue when the Catalog changes state.

Yeah, we need to think through implications. We likely need some way for an Operator to specify which catalogs should be used to resolve it.


[Docs] Explain how to enable plugins in Xcode and Xcode Cloud

Motivation
By default, adopters hit this error both in Xcode and Xcode Cloud and might not be familiar with how to resolve it.
Modifications
Expand the FAQ to cover how to enable plugins in Xcode and Xcode Cloud.
Result
Better docs, adopters can find a solution easier.
Test Plan
N/A

It's potentially worth adding a comment to the script to point out that the misspelling of validation in the plist key is "correct" as Xcode Cloud/Xcode expects it.
I've had many devs before try to "fix" the typo and then wonder why it doesn't work.

It's potentially worth adding a comment to the script to point out that the misspelling of validation in the plist key is "correct" as Xcode Cloud/Xcode expects it.
I've had many devs before try to "fix" the typo and then wonder why it doesn't work.

Ha! Nice spot. I'm slightly embarrassed that, while I've used this many times, I never even noticed :)
Yes, if you open a PR, we'd take it.

Ha! Nice spot. I'm slightly embarrassed that, while I've used this many times, I never even noticed :)

I'd like to take credit for noticing it, but all the credit has to go to the spell checker. :D I have it turned on in Xcode, so it points out spelling mistakes to me all the time.


DataSkippingReader - EqualNullSafe support

Resolves  https://github.com/delta-io/delta/pull/974#discussion_r825686186
Add EqualNullSafe support

Expression
Behavior

EqualNullSafe(Literal(null, _), a)
this will be optimized to IsNull(a) by rule NullPropagation

EqualNullSafe(NonNullLiteral(v, _), _)
this has the same behavior as EqualsTo(NonNullLiteral(v, _), _)

Not(EqualNullSafe(NonNullLiteral(v, _), _))
this has the same behavior as Not(EqualsTo(NonNullLiteral(v, _), _))

Not(EqualNullSafe(Literal(null, _), a))
this will be optimized to Not(IsNull(a)) by rule NullPropagation and then optimized to IsNotNull(a) by rule BooleanSimplification

Hi @cfmcgrady - thanks for this PR. We're busy preparing for the 1.2 release right now ... so we will get to this PR after 1.2.

cc @tdas take a look?

Will do. Maybe @zsxwing can look at it too. Sorry we are busy with  1.2.1 release, so we will get to this shortly.

Found a bug when data files contain null values.

fixed.

Thanks all.


ImportExport can't finish and export jobs hangs permanently with status "Processing" when using storage bucket adapters

PHP Version
8.1
Shopware Version
<IP_ADDRESS>
Expected behaviour
When exporting products with selected profile export job should export and then combine all partial export files finishing with status "Successful" and providing download ling for full export.
Actual behaviour
After all partial exports are done we get critical error when using google cloud storage adapter for filesystem:
Error thrown while handling message Shopware\Core\Content\ImportExport\Message\ImportExportMessage.
Removing from transport after 3 retries.
Error: failed: stream_copy_to_stream(): supplied resource is not a valid stream resource

Relevant trace (first to last):
./vendor/shopware/core/Content/ImportExport/Message/ImportExportHandler.php:46
./vendor/shopware/core/Content/ImportExport/ImportExport.php:269
./vendor/shopware/core/Content/ImportExport/Processing/Writer/AbstractFileWriter.php:64
./vendor/shopware/core/Content/ImportExport/Processing/Writer/AbstractFileWriter.php:38 < CRITICAL ERROR

Method stream_copy_to_stream fails in: .vendor/shopware/core/Content/ImportExport/Processing/Writer/AbstractFileWriter.php:38
Because second argument $this->tempFile is a already closed resource.
I spend some time debugging this and this is the order of events that is breaking the code:
last export chunk is done by exportChunk method called in ImportExport.php:260 this method is calling flushmethod of AbstractFileWriter object and in this method  $this->filesystem->writeStream is called on $this->tempFile AbstractFileWriter.php:43.
I guess problem is if filesystem is using guzzle client (like bucket adapters) $this->tempFile resource will be closed after writeStream line.
Then when finishing of export needs to be done finishmethod of AbstractFileWriter is called on ImportExport.php:269 and that calls flush again with already closed $this->tempFile resource resulting with error on line AbstractFileWriter.php:37.
My temp fix was to add this line:
if(!is_resource($this->tempFile)) {$this->tempFile = fopen($this->tempPath, 'a+b');}

on line AbstractFileWriter.php:44 after write $this->filesystem->writeStream is called, this is working but I didn't spend time checking would this break something else.
How to reproduce
Have google storage bucket filesysetm adapter and try exporting products on shopware <IP_ADDRESS>

Hi,
thx for your issue we will put it into our new evaluation process and have a look at it again.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37914.
[created from NEXT-37914, comment 488806]


[Pokémon Yellow EU/FR] Trading always gives a glitched Rhydon

Describe the bug.
Trading App version: 1.3
Flipper Firmware: 0.94.1
Game: Pokémon Yellow, French version
I can't trade: it forces me to receive a glitched Rhydon, instead of the Pokémon I want.
This Rhydon is glitched: on the first trade I made, I couldn't open my Pokémon list anymore (white screen and couldn't quit) after the broken trade.
The video below shows the second trade where I am forced to exchange this Rhydon which breaks my save for a Rhydon which doesn't break my save but is still glitched (no name, strange stats, broken icons)
Note: I tried with and without resistors on CLK, the issue is the same.
Reproduction
Video of the issue (Rhynoferos = Rhydon and Bulbizarre = Bulbasaur, the Pokémon I chose in the app): https://youtu.be/sYfjd7b4AdA

Choose a Pokémon to trade on the flipper
Start a trading session
On the game's trade screen, the flipper has 2 visible Pokémons: a (glitched) Rhydon first and, below it, the Pokémon I've chosen
Flipper always select this Rhydon to trade so you will receive a glitched Rhydon no matter what.

Target Framework
No response
Logs
No response
Anything else?
No response

Interesting.
Does this happen no matter what Pokemon and stats you select?
Are you using just the defaults for Bulbasaur?
Do you only have Pokemon Yellow or do you have Red/Blue as well?
I've only ever done testing on USA Blue, but, I think @EstebanFuentealba did most of the work against Yellow (not sure exactly what region) and hasn't run in to issues as far as I am aware.
I've been working on an overhaul of the Trade routines, cleaning up the areas where it is fragile. Are you able to build and test from my WIP branch and let me know if it continues? https://github.com/kbembedded/Flipper-Zero-Game-Boy-Pokemon-Trading/tree/trade-features
For what its worth, I don't use the series resistor, and the way the code is set up it should never be an issue (that is, the Flipper side will never try and drive the clock pin).

First, there is a detail that I just thought of and that could be a lead: my save/Pokémon trainer's name has a one of the Pokémon special characters, it's "☥ANUBIS" (Well in reality it's"♀ANUBIS" but with so few pixels, there is no difference)

Does this happen no matter what Pokemon and stats you select?

Yup, my first trade attempt was an overpowered Mew, with custom name and a Peck attack

Are you using just the defaults for Bulbasaur?

Yep, for this second attempt, all defaults

Do you only have Pokemon Yellow or do you have Red/Blue as well?

Unfortunately no, besides Yellow, I only have the next gen atm (Gold)

Are you able to build and test from my WIP branch and let me know if it continues? https://github.com/kbembedded/Flipper-Zero-Game-Boy-Pokemon-Trading/tree/trade-features

Sure I'll try

For what its worth, I don't use the series resistor, and the way the code is set up it should never be an issue (that is, the Flipper side will never try and drive the clock pin).

Got it
Btw combats with this glitched Rhydon are quite fun to watch: HP bars extend beyond limit, text doesn't mean anything, it even corrupts Pikachu and people 😁
Imgur Gallery

First, there is a detail that I just thought of and that could be a lead: my save/Pokémon trainer's name has a one of the Pokémon special characters, it's "☥ANUBIS" (Well in reality it's"♀ANUBIS" but with so few pixels, there is no difference)

You might be on to something. I took closer look at your video. The Flipper's trainer name has ♀ prepended to it.
I think this is something I recently fixed and my WIP branch should work for you.
tl;dr: (I can go in to more detail on the issue if we confirm that my WIP has any effect on the bug) the code to go from synchronizing RNG between the linked units to start swapping real data between them very loosely matches control bytes; the ♀ symbol would match this and the Flipper might be thinking its part of the preamble still.
Looking at the data struct, if the data the Flipper sent was shifted by one, that would push the 0x01 party_count value in to the first byte of the party_members. 0x01 is the hex value of a Rhydon. The rest of the data just kind of goes off the rails.

Btw combats with this glitched Rhydon are quite fun to watch: HP bars extend beyond limit, text doesn't mean anything, it even corrupts Pikachu and people 😁 Imgur Gallery

Those are fantastic. I ran in to a bunch of hilarious effects when doing work on this a while ago. One of them actually crashed my game if I opened Start->Pokemon. I eventually figured out that the HP value was so large, the game would try to render an HP bar that long, and it basically clobbered the whole screen before just hard-locking the system. I luckily was able to box it up and release it and my game no longer had issues.
This was also a fun one:

I think this is something I recently fixed and my WIP branch should work for you.

I confirm. I built your fork, and it fixed my issue (after removing the resistors because it wasn't detecting the flipper). No more Rhydon, no more "♀Flipper", only the Pokémon I've chosen.
To be sure, I exchanged my glitched Rhydon for the default Bulbazaur
Then I catched and named a Pokémon with a lot of special chars and I exchanged it for an overpowered weird MewTwo (Fire type and with Peck attack).
Everything worked as expected, nice work! :tada:

tl;dr: (...)(I can go in to more detail on the issue if we confirm that my WIP has any effect on the bug) the code to go from synchronizing RNG between the linked units to start swapping real data between them very loosely matches control bytes; the ♀ symbol would match this and the Flipper might be thinking its part of the preamble still. If that is the case, that means any trainer names with a number, or a couple other specific symbols, as the first character would also suffer from this. If you had that symbol in any other character position, I don't think it would be a problem on the latest release.
Looking at the data struct, if the data the Flipper sent was shifted by one, that would push the 0x01 party_count value in to the first byte of the party_members. 0x01 is the hex value of a Rhydon. The rest of the data just kind of goes off the rails.

Thanks for the explaination, it makes sense.

This was also a fun one:

Image

lmao this reminds me of the Sesame Street "8 to infinity" meme

I think this is something I recently fixed and my WIP branch should work for you.

I confirm. I built your fork, and it fixed my issue (after removing the resistors because it wasn't detecting the flipper). No more Rhydon, no more "♀Flipper", only the Pokémon I've chosen. To be sure, I exchanged my glitched Rhydon for the default Bulbazaur Then I catched and named a Pokémon with a lot of special chars and I exchanged it for an overpowered weird MewTwo (Fire type and with Peck attack).

:tada:
Glad to hear it.
@EstebanFuentealba keep this issue open for now for anyone else running in to it. I'll mark it as being resolved by a new PR once I open it.

... exchanged it for an overpowered weird MewTwo (Fire type and with Peck attack).

An interesting note on types if you were not aware. The Flipper can set them to whatever, but if you view the Pokemon's stats, it will always show the Pokemon's defaults. The save data keeps a writable copy of types that is normally only ever updated when the Pokemon evolves or uses an attack that could temporarily change its type, Transform, for example. Since Mewtwo doesn't evolve, as long as you never use an attack with a type change effect, it will stay as Fire/Peck (I assume Peck is Flying?)
Before I understood that quirk of types, I created a Ghost/Fire Mew and was baffled when it was unaffected by physical attacks lol.

lmao this reminds me of the Sesame Street "8 to infinity" meme

:rofl:

it will stay as Fire/Peck (I assume Peck is Flying?)

The Mewtwo I made is a Psy/Fire type (it only shows Type 1: Psy as espected) and I replaced one of the attacks with Peck

I also just learned that, Trainer and Pokemon names cannot contain numbers. It just so happens a few symbols end up with their upper nibble being hex F. So there are some other tweaks I also need to make. In theory though, the gameboy should render them, but, it will cause issues with the patch list since that does not cover names. I always wondered why, and its because they should never have the byte that the patch list patches.


Follow up #3392 (refactor)

#3392 improved the push int API and in doing so correctly kept the diff minimal by maintaining the logic around push_opcode. However, partly because of rustfmt this code is a bit odd to read, also the opcode calculation requires one to look at opcodes to see whats going on.
Consider using this instead
    /// Adds instructions to push an unchecked integer onto the stack.
    ///
    /// Integers are encoded as little-endian signed-magnitude numbers, but there are dedicated
    /// opcodes to push some small integers.
    ///
    /// This function implements `CScript::push_int64` from Core `script.h`.
    ///
    /// > Numeric opcodes (OP_1ADD, etc) are restricted to operating on 4-byte integers.
    /// > The semantics are subtle, though: operands must be in the range [-2^31 +1...2^31 -1],
    /// > but results may overflow (and are valid as long as they are not used in a subsequent
    /// > numeric operation). CScriptNum enforces those semantics by storing results as
    /// > an int64 and allowing out-of-range values to be returned as a vector of bytes but
    /// > throwing an exception if arithmetic is done or the result is interpreted as an integer.
    ///
    /// Does not check whether `n` is in the range of [-2^31 +1...2^31 -1].
    pub fn push_int_unchecked(self, n: i64) -> Builder {
        match n {
            -1 => self.push_opcode(OP_PUSHNUM_NEG1),
            0 => self.push_opcode(OP_PUSHBYTES_0),
            1..=16 => {
                let n = n as u8;
                let op_pushnum = Opcode::from(n + (OP_PUSHNUM_1.to_u8() - 1));
                self.push_opcode(op_pushnum)
            }
            _ => self.push_int_non_minimal(n)
        }
    }

Closed by #3411


Initialize without activate call

Attempting to fix #354 by initializing the window regardless of platform

Voila, frontend launches now
So glad it was something that simple
I'll merge this in to get the ball rolling
I do however still have a problem now with the Windows dist in that it won't activate the backend; can investigate in follow-up
Don't know if it's related but I do now still see the good old message

Argh. Usually that means the path to the executable is incorrect.
Thankfully we have great error-checking between the frontend and electron now—so we can log everything to see if this is the case.


Refactor: some fetcher-related code

This PR extracts the types related to fetcher from the components to ExplorerService and index.d.ts, preparing for the next step of refactoring to the models level.

Rebased.

Rebased.

Rebased.


Can't extract html, css, js and images from pug file

I'm using pug v2.0.4, webpack v5.75.0 and node v14.15.4 for dev reasons. This is the interested code:
---
title: "Core - Dashboard Builder"
classSidebarHome: active
pageTitle: "Dashboard"
classHeader: unauthorized
---
extends layouts/layout
block content...

And this is my webpack.config.js:
const path = require('path');
const PugPlugin = require('pug-plugin');

module.exports = {
    mode: 'none',
    entry: {
        index: './src/pug/index.pug',
    },
    output: {
        path: path.resolve(__dirname, 'dist'),
        publicPath: '/',
        filename: 'assets/js/[name].[contenthash:8].js'
    },
    module: {
        rules: [
            {
                test: /\.pug$/,
                loader: PugPlugin.loader, // default method is 'render'
            },
            {
                test: /\.(css|sass|scss)$/,
                use: ['style-loader', 'css-loader', 'sass-loader']
            },
            {
                test: /\.(png|jpg|jpeg|ico)/,
                type: 'asset/resource',
                generator: {
                    // output filename of images
                    filename: 'assets/img/[name].[hash:8][ext]',
                },
            }
        ]
    },
    plugins: [
        new PugPlugin({
            extractCss: {
                // output filename of CSS files
                filename: 'assets/css/[name].[contenthash:8].css'
            },
        }),
    ],
    devServer: {
        static: './src'
    }
};

Instead this is my src folder structure:

src [root]
img [2nd-tier]
js [2nd-tier]
pug [2nd-tier]
sass [2nd-tier]

When I run webpack (aka npm run build) the result is an unexpected text error near "Core..." and also if I remove the "variables" from pug file because they are not variables I can't extract assets from entry file and src folder.
What's the problem? I would like to understand where I'm wrong.

Sorry it was my fault. I have other problems with webpack and pug-plugin but this issue can be closed or moved to another issue that I'll open. Bye

Hallo @riccardi1997,
can you please create a small repo with reproducible issue in next issue.

Hello @webdiscus, yes I can.
I need to open an issue as I said but I would like to send you an invitation for access to my repository. Can you accept it so I can post the new issue and quote you?

Hi @riccardi1997,
Please, describe new issue in new ticket or change the title and text in this issue, because this issue is a mistake.

I edited the original title and question, I hope it's clearer now otherwise ask and I will respond. Sorry for the initial mistake

@riccardi1997
I can't understand you problem:
When I run webpack (aka npm run build) the result is an unexpected text error near "Core..." and also if I remove the "variables" from pug file because they are not variables I can't extract assets from entry file and src folder.

Please create a repo with reproducible issue then I can see you code.

If you agree I can give you the access to the repository. I will send you an invitation in a while

@riccardi1997
I don't have an invite.
Please tell me what is the name of the invited repo or github.

I'm migrating to webpack and this is because I'm using pug-plugin but I'm new to webpack and I can't extract assets as explained. New invitation will arrive in a bit

@riccardi1997
I have completely (90% all Pug and Sass files) refactored your release/dev branch for Webpack + pug-plugin.
By me local it works.
I tried to push the new branch in the repo, but it does not work. How can I push the new branch?

This is wrong syntax:
---
title: "Core - Dashboard Builder"
classSidebarHome: active
pageTitle: "Dashboard"
classHeader: unauthorized
---
extends layouts/layout
block content...

Must be:
layout.pug
//- use this block to add custom variables from Pug page
block variables

include mixins

doctype html
html(lang="en")

page.pug
extends layouts/layout

append variables
  -
     title = "Core - Dashboard Builder"
     classSidebarHome = "active"
     pageTitle = "Dashboard"
     classHeader = "unauthorized"

extends layouts/layout
block content...

Thank you very much, @webdiscus. Anyway you should be able to commit, try now and let me know.
I owe you one if that's correct

@riccardi1997
not works. It's very strangely. I can do git push --set-upstream origin webpack-pug-plugin and the pushing state will be displayed:
Enumerating objects: 248, done.
Counting objects: 100% (248/248), done.
Delta compression using up to 10 threads
Compressing objects: 100% (209/209), done.
Writing objects: 100% (247/247), 24.07 MiB | 4.80 MiB/s, done.
Total 247 (delta 48), reused 165 (delta 37), pack-reused 0

Then its hang, nothing happens. No permission warning.

@riccardi1997
Push works only with --force:
git push --set-upstream --force origin webpack-pug-plugin
Please, check new webpack-pug-plugin branch in your repo.
npm i
npm start

Well done, It works, thanks @webdiscus!
Before to close the issue, can you tell me if I can keep webpack dir only or src folder also?
And now I don't need of gulp files anymore, right?

@riccardi1997
you nice UI framework works now from ./webpack directory w/o dependences outer.
Following old/unused directories and files can be deleted:

./build (new version compile in the ./dist directory, if you need keep it then remove dist from .gitignore)
./gulp (is unused)
./src (old, is unused)
./gulpfile.js (in unused)
./static.json (in unused)

To keep clean directory structure move all files and dirs from ./webpack to ./ and then delete empty ./webpack.
Yes, you don't need of gulp files anymore, delete its.
Yet one thing... Don't forget add the postcss-loader to webpack config to use the autoprefixer , if you need it.
Note: you don't need any minimiser for CSS or JS. Webpack minify its self very well.

@riccardi1997
can  the issue be closed?

@webdiscus
Yes, u can


📃:  Lane Line Detection Using OpenCV and Machine Learning

:red_circle: Title : Lane Line Detection Using OpenCV and Machine Learning
:red_circle: Aim : Detect Vehicle Lane using OpenCV and Machine Learning
:red_circle: Brief Explanation : Using OpenCV and pretrained OpenCV models we would be predicting the vehicle lane. Testing can be done using video.
Screenshots 📷

NA

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Aviral Garg
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.) SSOC Contributor

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@aviralgarg05 go ahead.


Update pushstate.js

Added anchor target check. If existing dont push state!!! If I have target _blank on an anchor it ignores it.

Closing this one due to missing tests.


Source Maps not Working

Picking up after #33, which was awaiting reproduction, I have created a repo that reproduces the issue. Been 5 days since I posted in that issue with no apparent attention. So starting a new issue.
Basically when a test error occurs, the reported error line is incorrect.
https://github.com/Lalem001/karma-rollup-preprocessor-issue

I wouldn't suspect the chokidar file watcher, which means there's only about 40 lines of code that would contain the bug. Maybe something with the cache?
I’ll take a closer look soon, thanks @Lalem001


Adding name or description to key table

The API I'm writing doesn't have users, and is mostly headless. I'd like to be able to specify a name or description for the key (which would just be a varchar(255) column on the api_key table) when generating them. Is there an easy way for me to add this without modifying the code in the vendor dir? I'd like to add a param to the generate artisan command to specify the name. If not, you could consider this a feature request I guess.
Thanks!

How u can solve this is now to make a migration with $tbl->text('<column name>')


getParamsSchema is not typed in c.req.valid("param")

I found that getParamsSchema is not typed properly in the route handler.
export const list = createRoute({
  tags,
  path: `${collectionRootPath}/cards`,
  method: "get",
  request: {
    params: getParamsSchema({ name: "collectionId" }),
  },
  responses: {
    [HttpCode.OK]: jsonContent(z.array(selectCardSchema), "List of cards."),
  },
});

export const getOne = createRoute({
  tags,
  path: `${rootPath}/{id}`,
  method: "get",
  request: {
    params: IdUUIDParamsSchema,
  },
  responses: {
    [HttpCode.OK]: jsonContent(selectCardSchema, "The requested card."),
    [HttpCode.NOT_FOUND]: jsonContent(
      createMessageObjectSchema(HttpPhrase.NOT_FOUND),
      "Card not found.",
    ),
    [HttpCode.UNPROCESSABLE_ENTITY]: jsonContent(
      createErrorSchema(IdUUIDParamsSchema),
      "Invalid ID.",
    ),
  },
});

Bellow is a exemple with IdUUIDParamsSchema, where id is properly set and typed.

But with getParamsSchema I don't get any completion and the type isn't properly set.

I'm not a Typescript expert enough to be able to fixed this myself sadly 🥲 ahah.

Unless it is hidden inside collectionRootPath, this does not include collectionId in the path:
path: `${collectionRootPath}/cards`,

If collectionRootPath has collectionId in it, you might try declaring it with as const;
const collectionRootPath = "/things/{collectionId}" as const;


Performance goes down with full namespace

From<EMAIL_ADDRESS>on 2010-09-16T10:53:23Z
performance: after having the program open for about 1 hour I need to close it for not runnung into a near freeze
%reset in Console helps
This happens when large arrays are loaded and thus retained in the variable explorer.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=373

From pierre.raybaut on 2010-09-25T00:57:24Z
Status: Done

From pierre.raybaut on 2010-09-22T13:13:23Z
I think that it may be related to "Variable explorer" refresh process. If so, you may disable the option "Show arrays min/max" to improve things.
I've disabled this option by default in the new config.py to avoid this kind of problem (I had the same problem today after opening a dozen of very large arrays).
Summary: Performance goes down with full namespace
Status: Accepted


Event not fired when plotting points for certain symbols

When the map is in draw mode, the IDrawEventListener.onDrawUpdate() listener is not called when plotting points for certain symbols. If a point is moved while the map is in draw mode, the event will fire, however, simply plotting a point does not fire this event. Below is a partial list of symbols which exhibits this behavior.
GFM*BCF---****X Ferry
GFF*LTF---****X Final Protective Fire
GFM*OEF---****X Fix (Obstacle Effect)
GFT*F-----****X Fix Task
GFT*A-----****X Follow and Assume
GFT*AS----****X Follow and Support

That's right.  In other words, I don't understand what the expectation here is.  Let's say you are drawing the first item in the list: Ferry.  It is a two-point graphic.  When you start, it lets you put the two points anywhere.  When you update, you can move either of those points, and the update event fires properly.  If you click elsewhere, no event fires, because it won't let you add a third point.  Are you expecting that the click will move one of the original points to the new location?

Even on the initial two points for Ferry, the update event will not fire when those two points are plotted, only when they are moved. For other graphics, the update event fires both when the initial point is plotted and when it is moved.

That makes more sense.  As I look through the code, I am wondering if you get an event if you delete it.  Could you test that.

For area graphics or multiline segments where I am able to delete points, the update event does fire when points are deleted. For Ferry, I was unable to delete any points.

That may be ok, because it has exactly two points and you can move either to change the graphic.
I have now fixed your original issue of firing the event when a point is added.  The fix will be in version 3.0.2-SNAPSHOT.


[BUG] Clicking project pin on the Explore Projects map view doesn't filter to the project, shows 0/0 most of the time

Describe the bug
When clicking on projects around the map, the project cards are not always shown. There seems to be an issue particularly with low number projects (<100?). My recent project (377) has the correct behavior. Also sometimes the pins disappear (first screenshot). The main HOT task manager Explore Projects does not have this behavior.
To Reproduce
Steps to reproduce the behavior:

Go to https://tasks.openstreetmap.us/explore
Toggle to show the map
Zoom to a task are (ex: Arizona)
Click projects in that area (377 has the right behavior, most projects show nothing, project 90 showed the wrong card)

Expected behavior
I expect the filter to show the correct project card on the left pane
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome

Weird bug, thanks for reporting! The issue seems to arise from the fact that TM isn't actually filtering by project number, it's just searching #73 or whatever as text. If you search 73 then the right project comes up, but other stuff might as well. Url encoding of the # might be a problem. The issue might not come up on HOT because all the projects there have four-digit codes already? There shouldn't be any other different in the implementation.


Slowing down when approaching the API Call Limit

Due to Shopify API Call limits I would like to add some slow down function to not exceed them. Before every call() it checks for callsLeft, and wait 0,5s if just 5 calls left.

I like the idea here.  But I don't think we should always sleep.  It  may be better to return an appropriate error or something else.  For instance, sleeping and an http worker thread would probably not be best, better to just end the request.
So, with that in mind, why don't you make your code here only happen if the ShopifyClient has this setting turned "on".  For instance:
$shopifyClient = new ShopifyClient(
// This is just an example of what you could name it:
$shopifyClient->enableThreadSleepOnAPICallLimit();

// then future api calls would sleep only if the previous call was made to turn the setting 'on'

I'm open to your suggestions here, but let's keep this as an option, rather than something that always happens.
Thanks for the PR, keep them coming!
Colin


Feature/route check

Creating this for @jcornfusionalliance
Chris, does this help fix the bug?

I think he might've resolved it on the feature/jat branch.
And my branch doesn't exactly work quite yet.
I pulled out some existing Auth logic to centralize that validation around a single observable stream broadcasted from rootscope, but was fighting against some persistence to session storage in the controllers.
Sent from my iPhone
On Oct 2, 2017, at 11:11 AM, BW <notifications<EMAIL_ADDRESS>wrote:
Creating this for @jcornfusionalliancehttps://github.com/jcornfusionalliance
Chris, does this help fix the bug?

You can view, comment on, or merge this pull request online at:
https://github.com/fusionalliance/autorenter-angular1/pull/65
Commit Summary

built some route checking functionality but it is still not working right. done for the day tho
created streams to guard against routes
created user service to house all user functionality login logout info etc
refactored login logic, not all done yet
login functionality currently broke, but certain features working better. need to revert to inspect how to authorize a call to backend to login

File Changes

A package-lock.jsonhttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-0 (7426)
M src/app/app.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-1 (1)
M src/app/app.route.config.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-2 (15)
M src/app/data-access/login-data-service.factory.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-3 (7)
M src/app/data-access/login-data-service.factory.spec.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-4 (1)
A src/app/data-access/user.service.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-5 (61)
D src/app/login/login.component.htmlhttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-6 (42)
D src/app/login/login.component.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-7 (8)
M src/app/login/login.controller.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-8 (26)
M src/app/login/login.htmlhttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-9 (2)
M src/app/logout/logout.controller.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-10 (12)
M src/app/main/main.controller.jshttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-11 (36)
M src/app/nav-bar/nav-bar.component.htmlhttps://github.com/fusionalliance/autorenter-angular1/pull/65/files#diff-12 (2)

Patch Links:

https://github.com/fusionalliance/autorenter-angular1/pull/65.patch
https://github.com/fusionalliance/autorenter-angular1/pull/65.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/fusionalliance/autorenter-angular1/pull/65, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ael1Gz_Wu9sgt5urhBKAYuib4rpKEN2Uks5soP0HgaJpZM4Pqydq.