The dependency Axios has a security breach

Dependencies:
<EMAIL_ADDRESS>      └─┬<EMAIL_ADDRESS>        └──<EMAIL_ADDRESS>
Axios versions from 0.8.1 through 1.5.1 are affected by a moderate severity Cross-Site Request Forgery (CSRF) vulnerability, and the issue has been patched in version 1.6.0.
I talked to Alexis B who have made Xboxreplay and he will try to update to a new 5.0.0 version asap

2 notes: CSRF is only a thing in browsers, and it's only a problem if loading data from arbitrary URLs. For example, user is logged into a website, then someone on a Github comment posts a link to that website, then someone clicks it and the website does some action automatically on their behalf because their auth data was cached. Referrer checks, CORS, special headers, etc can block that.
Since prismarine-auth doesn't formally support the browser (due to CORS restrictions), and since the xboxlive auth dep shouldn't be loading dynamic websites, there is no security impact for prismarine-auth. However, just to note that dep is only used at all if password auth is used, not done by default. We may remove in the future, but updating to a new major version is breaking and would require code changes.


Add saving images for testing purposes

Task

JIRA

Description

Additional Notes (optional)

Checklist

[x] I am following the code style guide
[ ] The code includes tests of new features
[x] The code passes static analysis
[x] README.md is up to date
[x] Version number is up to date
[x] ProGuard configuration files are up to date
[x] All temporary TODOs and FIXMEs are removed
[ ] I have tested the solution on these devices:

Device A, Android X.Y.Z
Device B, Android Z.Y
Virtual device W, Android Y.Y.Z

Merge

[ ] Committer
[ ] Reviewer
[ ] Project lead

SonarQube analysis reported 2 issues

 1 major
 1 info

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 CameraPresenter.kt#L223: Line detected that is longer than the defined maximum line length in the code style.


WBP-1G bk7231 N on pcb DIY Smart Mini Switch aliexpress

Hello, a firmware dump from our user:
readResult_BK7231N_QIO_2023-18-3--11-51-13.zip
I don't know if it's already supported, please check and add if possible! Thanks

Added as Tuya Generic WBP-1G Smart Switch


cpu dashboard doesn't work well with workload prioritization

We have been show foreground CPU time for some time now, which has been a life saver.
To compose the query, we sum the cpu time of the main and statement groups. However, once we use Workload Prioritization, the statement group  isn't used anymore. Other, per-workload groups are used instead.
We should change that to reflect the usage of workload prioritization. One possibility is to show one graph per class, including the internal ones, or hide the internal ones.

Seems like a duplication of: #612
If so, please close this one and comment there (which @avikivity is not, though he opened the issue)

That issue actually refers to our already existing per-CPU dashboard - maybe Avi wasn't aware of that.
However our dashboard currently only tracks foreground time.
This issue is about the fact that if we use workload prio, even foreground time doesn't work (because the statement class is no longer used). One possible solution is to stop tracking foreground time - and instead showing a separate graph per class for all classes. This would work with workload prioritization as well, and also include more information.
But that's not the only possible solution.
And BTW, we will very likely have a similar issue in the I/O dashboards. So I'd like to keep this issue open so we make sure all of our dashboards work well with Workload Prioritization, regardless of what the solution is.

I agree about the IO dashboard, I'd like to break the current functionality and combine both to a single dashboard


Add markdown to error/warning descriptions

Update the descriptions in the phase_validations.py to contain markdown instead of just plain text.  Based on the FIG descriptions at https://www.consumerfinance.gov/data-research/small-business-lending/filing-instructions-guide/2024-guide/#4
Use these to generate the csv that is stored in https://github.com/cfpb/sbl-content/blob/main/fig-files/validation-spec/2024-validations.csv?plain=1
Update pytests

example_descs.md


[BUG] Unable to donwload the mnist dataset

System information

Windows
pip
FiftyOne v0.14.2, Voxel51, Inc.
Python 3.7.9

Describe the problem
Unable to download the mnist dataset
Code to reproduce issue
import fiftyone as fo
import fiftyone.zoo as foz

foz.load_zoo_dataset("mnist", split="test")

Other info / logs
Downloading split 'test' to 'C:\Users\hw630\fiftyone\mnist\test'
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-12-7ee5d8bb7cd7> in <module>
      1 # Load the data from fiftyone
----> 2 voc_2012_train = foz.load_zoo_dataset("mnist", split="test")
      3 # voc_2012_train = foz.load_zoo_dataset("voc-2012", split="train", label_types=["detections"])

d:\python37\lib\site-packages\fiftyone\zoo\datasets\__init__.py in load_zoo_dataset(name, split, splits, label_field, dataset_name, dataset_dir, download_if_necessary, drop_existing_dataset, overwrite, cleanup, **kwargs)
    201             overwrite=overwrite,
    202             cleanup=cleanup,
--> 203             **download_kwargs,
    204         )
    205         zoo_dataset = info.get_zoo_dataset()

d:\python37\lib\site-packages\fiftyone\zoo\datasets\__init__.py in download_zoo_dataset(name, split, splits, dataset_dir, overwrite, cleanup, **kwargs)
    119         splits=splits,
    120         overwrite=overwrite,
--> 121         cleanup=cleanup,
    122     )
    123

d:\python37\lib\site-packages\fiftyone\zoo\datasets\__init__.py in download_and_prepare(self, dataset_dir, split, splits, overwrite, cleanup)
   1002                     num_samples,
   1003                     classes,
-> 1004                 ) = self._download_and_prepare(split_dir, scratch_dir, split)
   1005
   1006                 # Add split to ZooDatasetInfo

d:\python37\lib\site-packages\fiftyone\zoo\datasets\torch.py in _download_and_prepare(self, dataset_dir, scratch_dir, split)
     91             download_fcn,
     92             get_class_labels_fcn,
---> 93             sample_parser,
     94         )
     95

d:\python37\lib\site-packages\fiftyone\zoo\datasets\torch.py in _download_and_prepare(dataset_dir, scratch_dir, download_fcn, get_class_labels_fcn, sample_parser)
    534         )
    535     else:
--> 536         raise ValueError("Unsupported label class %s" % label_cls)
    537
    538     # Write the formatted dataset to `dataset_dir`

TypeError: not all arguments converted during string formatting

What areas of FiftyOne does this bug affect?

[ ] App: FiftyOne application issue
[x] Core: Core fiftyone Python library issue
[ ] Server: Fiftyone server issue

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another
member of your organization be willing to contribute a fix for this bug to the
FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance
from the FiftyOne community.
[x] No. I cannot contribute a bug fix at this time.

This is a duplicate of https://github.com/voxel51/fiftyone/issues/1451 and should be fixed as of fiftyone==0.14.3.
Upgrading should resolve this:
pip install --upgrade fiftyone


Bumps appengine-plugins-core version to 0.2.10 (#1326)

cherry picked from 2017 branch commit 3c714c4. updates core lib in preparation for devappserver updates
Note the branching for this PR:
target branch : v17.2.2 -> branched off of the last release tag for 2016
we will release a interim maintenance release from v17.2.2.

I know we have a bunch of release branches now and the naming is a bit confusing. This is only temporary but if you have a suggestion to improve the situation I'm open.


Spelling correction

Add here your WMS ID
11492
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Hi @amarpahwa,
Please solve your conflicts.

@amarpahwa please resolve the conflicts by June 10, EOD, otherwise we will close the PR


MD034: Email autolink with incomplete domain in angle brackets incorrectly reported as a violation

Similar to: https://github.com/igorshubovych/markdownlint-cli/issues/339
When running the markdownlint-cli2 Pre-commit hook with the --fix option, a pair of angled brackets are added around text that is detected as a URL infinitely.
Minimal Markdown Example:
# Foo

Bar<EMAIL_ADDRESS>

Expected behavior:
Run N:
# Foo

Bar<EMAIL_ADDRESS>

Actual behavior:
Run 1:
# Foo

Bar<EMAIL_ADDRESS>

Run 2:
# Foo

Bar<EMAIL_ADDRESS>

Run 3:
# Foo

Bar<EMAIL_ADDRESS>

et c.

OK<EMAIL_ADDRESS>
Bad<EMAIL_ADDRESS>


Improve Main Dashboard UI

This project was initially built using WinForms. The demo on the initial page reflects that old WinForms UI.
However, the project has since then moved from WinForms to WPF, and the new UI is not what it used to be. Something was put together quickly to just test the logic -- not much time was spent thinking of the UI. So now it now looks like this:

It would be nice to re-work the new WPF UI to be similar to the old one. Any new ideas, styles, or designs are more than welcome!

Is this issue accepts PR as hacktoberfest 2022, then I can give it a go, Can you please assign this task to me?

@Hermes179 Assigned :) 🚀

Hi @Capm96,
I was wondering if you still need help on this project?? If so, please let me know and I can work on this task. It looks like it was previously assigned to Hermes, but I don't see that any work has been done. Please let me know!


Script to generate IntelliJ Run Configuration for Controller and Invoker

Script which generates the run configuration for Controller and Invoker to enable launching them from within IDE. This allows faster development and debugging as time spent in rebuilding and restarting docker containers can be avoided
The script can be run via below command (requires OpenWhisk running)
./gradlew -p tools/dev intellij

The script also supports customizing the generated config with local overrides via custom config file (see README.md for details)
For now using this script Controller can be launched from within the IDE. Invoker posses some problem

Can this PR be reviewed?

@rabbah Can you review this

Updated the readme based on review feedback. Future PR would not miss on the dots ... :dart:


Xcode Swift Package Manager only adds version 1.0.2

I do not seem to find a way to add the 1.1.0 version with Xcode SPM. I only manage up to 1.0.2.
Are the SPM required files up-to-date or is this just a user error?
Steps to reproduce:

create a new SwiftUI app project
try to add the NavigationStack to this project with Swift Package Manager. The latest version I am able to import is 1.0.2. It does not matter what selection or version numbers I choose (Up to next major version, Up to next minor version, Exact version with 1.1.0, or Range)

Thanks for pointing that out, seems I used a wrong tag version number format for SPM ("1.1" instead of "1.1.0"). Should be fixed now.


Dev cultural revolution

The cultural revolution will prevent new class inequalities

Can you solve the merge conflits ?

I will merge this one once the previous PR has been merged.


plot_dataset_distillation.py example creates but doesn't use images_test

https://jaxopt.github.io/stable/auto_examples/implicit_diff/plot_dataset_distillation.html#sphx-glr-auto-examples-implicit-diff-plot-dataset-distillation-py

superceded when we moved to notebooks


Is Micro Manager Python Support Python3?

Python 2.7 will reach the end of its life on January 1st, 2020.
Do we hope Micro Manager support Python3, Is that in the plan?

I'm not affiliated with Micro-manager. This repo is just a remembrance from my past.
Yes. Source code already supports python3. Which python version is being used defined during compilation. Please contact Micromanager dev team for python3 builds.

ありがとう


fix: avoid query statement plan to make TiDB OOM

regex search statement plan is not very necessary, and sometimes it causes TiDB OOM.

Codecov Report

Merging #1385 (e6f0755) into master (571ea1d) will increase coverage by 1.38%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1385      +/-   ##
==========================================
+ Coverage   26.21%   27.59%   +1.38%
==========================================
  Files         202       93     -109
  Lines       15394     9972    -5422
  Branches       99        0      -99
==========================================
- Hits         4035     2752    -1283
+ Misses      11088     7040    -4048
+ Partials      271      180      -91

Flag
Coverage Δ

be_integration_test_nightly
?

be_unit_test
27.59% <0.00%> (+0.01%)
:arrow_up:

e2e_test
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Extended certificate callbacks.

VerifyCertificate and VerifyChangedCertificate callbacks now have more arguments for more detailed certificate information.
VerifyCertificate now also called, if the hostname does not match with argument set accordingly.

+1

rebase due to merge conflicts

@awakecoding @bmiklautz @hardening Rebased this pull, any objections / suggestions for it?

Well the patch itself looks good, anyway isn't there a way to mutualize the code that print the certificate warning and co ? (The code looks exactly the same in xfreerdp and wlfreerdp...)
A global +1 anyway.

@hardening Yes, for the command line clients that would be possible (maybe client-common?)

I do get some compiler warnings when compiling:
freerdp/FreeRDP-master/libfreerdp/crypto/crypto.c: In function ‘crypto_cert_subject_common_name’:
freerdp/FreeRDP-master/libfreerdp/crypto/crypto.c:438:24: warning: pointer targets in passing argument 1 of ‘_strdup’ differ in signedness [-Wpointer-sign]
  common_name = _strdup(common_name_raw);
                        ^
In file included from freerdp/FreeRDP-master/winpr/include/winpr/crt.h:30:0,
                 from freerdp/FreeRDP-master/libfreerdp/crypto/crypto.c:24:
freerdp/FreeRDP-master/winpr/include/winpr/string.h:51:17: note: expected ‘const char *’ but argument is of type ‘BYTE *’
 WINPR_API char* _strdup(const char* strSource);
                 ^

I didn't build and test android/ios but the PR looks good for me and xfreerdp works fine. I really do like that I can temporarily accept an host \o/.
Same remark as @hardening. For command line clients it would really make sense to have this code unified. libfreerdp-client (client/common) sounds like a good place.
Except the build warnings above and my remark +1 from my side.

@hardening @bmiklautz Could anyone have a look at this one?

I was about to +1 this PR, but now it needs rebasing !

@hardening Ok, rebased the whole stuff and adjusted the android specific parts.

@hardening @bmiklautz can we merge this one?


IML FS moves to unavailable state if one MDT is unmounted

If Lustre is setup to have multiple MDTs, it seems that the filesystem should not move to the unavailable state unless MDT0 has been unmounted.

I'm not sure it's totally wrong.  Maybe an extra state of "degraded"?  Some directories or files will be unavailable if an MDT other than 0 is missing

Yes, I think degraded would be fine iff MDT != 0 is unmounted. We can then rely on that state + available as usable states for scans.

Given the complexity of adding a state, I'm going to rescind my suggestion and just ignore non-zero MDTs for the "unavailable" state.


$emailAddress must be a string. Got<EMAIL_ADDRESS>

Default should be $notifiable->email->value instead of $notifiable->email.

Hi @isaacdarcilla,
The readme was out of date. Actually we use the routeNotificationFor('mail') method. This is the correct interface method to use for notifiables, and it's expected to return the email address as a string. You might want to check your implementation of this method in the User class (or your notifiable model).
https://laravel.com/docs/10.x/notifications#customizing-the-recipient
In case you don't want to touch your model code, you can always add a custom "->to" directive and pass the email address that way instead. If you provide at least one "to" address to the SendGridMessage instance, the library will use that instead.
I'm closing this issue for now, but feel free to re-open if necessary.

Thank you for  the details.

Hi @isaacdarcilla,
This might not affect the specific issue that you encountered, but I just realised the routeNotificationForMail can also return an array (['email_address' => 'name']) as well.
I opened a PR for this, and will release this soon.


Advanced Search "Request Description" filter button

Describe the bug in current situation
Search result issues when request description button is selected
Link bug to the User Story
#1688
Impact of this bug
Moderate - Advanced Search will not function properly with this bug and all users will be impacted
Chance of Occurring (high/medium/low/very low)
High
Preconditions
Go to TEST, login as an IAO or Ministry user
Steps to Reproduce
Steps to reproduce the behavior:

Go to Advanced Search
Click on "Search" bar and input the keyword "test"
Click on "Request Description" filter button
4: See error: "Apply Search" button should become active but does not, or sometimes it will but a pop up text box appears below the search bar (in this case it says "Add test") and when I click on it, the Apply Search button becomes inactives again

Expected behaviour
Apply search button becomes active after inputting one or more key words and selecting one of the filter buttons

Thanks for the fix.


Iterative scheme for Task processing

Introduces the iterative scheme:
User can declare number of iteration that given JPetUserTask should perform or even the infinite loop is acceptable with the special condtion that will stop the infite loop.
The condition is just a special boolean option ("stopIteration_bool")  that can be set in the JPetUserTask terminate method, and if it is set to true, then, the iteration will be interrupted.
In addition:
-Add make_unique implementation missing in C++11
-.Refactoring of JPetTaskIO:  separate JPetOutputHandler and JPetTaskIOTools. Also, add option that in case of empty outputFileType string passed to the constructor, the task will not write anything to the output.
-Also, some changes of JPetScopeLoader
-Create JPetTaskFactory and remove the Task creation logic from JPetManager
-Add bunch of unique pointers.
-Cleanup and simplification of JPetManager
-Change the way JPetManager returns error, now it throws exceptions that should be catched by the main functions.
-Remove JPetParamInterface and simplify large parts of coe
-Add bunch of tests

Please grep for "c-string" in the comments describing function parameters and update these comments as most of these params are no longer c-strings.

Also, think about the case, where several modules have stop condition setup (naming clash)

I fixed the comments.


BUG: meson fails to properly detect numpy, pybind11 and (sort of) pythran when cross compiling

The new dependency detection in scipy/meson.build works suitably for native builds of SciPy, but does not correctly detect build-arch dependencies for cross compilation. The meson rules invoke the host Python interpreter, there called py3, to import the numpy, pybind11 and pythran packages (all installed for the host arch) and deduce the proper paths for includes and, in the case of NumPy, infer a search path for the npymath and npyrandom libraries.

NOTE: the Pythran search is incorrect but doesn't really cause a problem because the it is only used on the host to render files; the include path returned by the problematic search is never referenced by any subsequent meson build logic. The entire incdir_pythran block could be removed without adverse effects.

If the requisite dependencies are installed for the host arch (whether or not they are also installed for the build arch), the detection will return paths for the host arch, which might cause subtle problems in header files and fails outright when the linker attempts to link object files for the build arch with the native npymath or npyrandom libraries. If the dependencies are installed only for the build arch, the interpreter will fail entirely and meson will never even configure the build.
Resolution attempts
I have tried a couple simple workarounds, to no avail:

Install only the dependencies only for the build arch. On Void Linux, we set a lot of environment variables to tell the host Python to use the sysconfig data for the build arch and also add the build root to PYTHONPATH, allowing the host Python to find these modules and grabbing relevant information (field sizes, shlib suffixes, etc.) for the build arch rather than the host. This would work with the existing meson detection for pybind11 and pythran, but does not work for numpy because import numpy triggers a bunch of shared object loads and the build arch libraries are incompatible with the host. (A more targeted import, such as from numpy.__config__ import get_include, might work in this situation, but I haven't bothered to try.)
Install dependencies for host and build arches, use the detection to find paths to the host, but then manually prepend the build root to the returned paths. This almost works on Void Linux because we install packages for the build arch under a /usr/<triple> prefix that otherwise mirrors the native layout. In generally, most of the compilation commands include the right -I flags to find headers for the build arch. However, some commands still include paths to the host interpreter, and the find_library calls to identify npymath and npyrandom somehow still pick up the host versions and trigger a linker failure. I don't know enough about how meson sets the Python environment when searching for it to understand how these host paths are creeping in or why find_library still seems to prefer the host paths even though I add the correct paths to the search paths in that function. (I wouldn't really expect find_library to dig into the numpy tree to find the libraries, so it seems an additional search path is creeping in before I add one explicitly.)

Possible fixes
Although I'm speculating, it seems a few approaches could be taken to resolve this issue, in order of decreasing "niceness":

Convince numpy (and, probably, pybind11) to ship pkg-config files. This is probably desirable, was mentioned in the [related meson issue], and sidesteps a lot of problems. The trouble with pybind11 is that it wants to be entirely self-contained within the Python package tree; however, even if it ships a .pc file within its package tree instead of in a system-specific path, Void can probably work around the issue with relative ease. (Void already wraps pkg-config for cross builds so that it loads descriptors for the build arch and manipulates the paths appropriately.) The determination for pythran should just be dropped altogether (also, rather than invoking a Python interpreter to read SCIPY_USE_PYTHRAN from the environment, Pythran should just be a meson build option). The trouble here is backwards compatibility; if SciPy will build with old versions of NumPy or pybind11, it will still need fallback detection. Hence...
Existing logic can be improved, even if only as a fallback for older versions of dependencies. I'm not sure what this should look like, but reading variables from the environment might make sense (e.g., NUMPY_ROOT, PYBIND11_INCLUDE_DIR); when these variables are defined, they are used as-is; otherwise, the existing interpreter invocations can provide sensible defaults for native builds. Of course, this assumes that find_library can be made to find NumPy libraries for the build arch even though it now prefers the host versions.
The existing search for a Python interpreter could allow a custom path rather than always using the default, which seems to be the same interpreter that is running meson. This might allow some clever wrapping of the interpreter but is probably an incomplete (and maybe completely ineffective) solution. For example, no amount of sensible wrapping will allow print(numpy.get_include()) to dump some modified path; however, a successful from numpy.__config__ import get_include using the build-arch NumPy might make this workable if ugly.

Related issue
This issue was opened in response to https://github.com/mesonbuild/meson/issues/9598#issuecomment-1201475862 as a means to track SciPy specifics and provide a link target for inclusion in https://github.com/scipy/scipy/issues/14812.
cc: @rgommers

@ahesford thanks a lot for the detailed bug report. We are just working through resolving the same issue for Buildroot in gh-14812. Still like to commit a temporary fix there, but at least there was success when carrying one patch with a hardcoded path for the two static libraries coming from NumPy. This is a cleaner and more dedicated issue, so I may continue here.

but reading variables from the environment might make sense (e.g., NUMPY_ROOT, PYBIND11_INCLUDE_DIR); when these variables are defined, they are used as-is; otherwise, the existing interpreter invocations can provide sensible defaults for native builds.

That seems like a useful thing to do as a workaround at least.

3. For example, no amount of sensible wrapping will allow print(numpy.get_include()) to dump some modified path; however, a successful from numpy.__config__ import get_include using the build-arch NumPy might make this workable if ugly.

I'm going to really try to not need to run the target Python interpreter for the final solution.

Convince numpy (and, probably, pybind11) to ship pkg-config files. This is probably desirable, was mentioned in the [related meson issue], and sidesteps a lot of problems.

Yes, I do want to do this for numpy.

as a means to track SciPy specifics and provide a link target for inclusion in #14812.

I added Void Linux to the issue description there.

The trouble here is backwards compatibility; if SciPy will build with old versions of NumPy or pybind11, it will still need fallback detection. Hence...

My goal is for that fallback detection to live inside Meson; scipy/meson.build should only contain dependency('numpy') and similar look-ups. That will work with older numpy versions too.

Convince numpy (and, probably, pybind11) to ship pkg-config files. This is probably desirable, was mentioned in the [related meson issue], and sidesteps a lot of problems. The trouble with pybind11 is that it wants to be entirely self-contained within the Python package tree; however, even if it ships a .pc file within its package tree instead of in a system-specific path, Void can probably work around the issue with relative ease. (Void already wraps pkg-config for cross builds so that it loads descriptors for the build arch and manipulates the paths appropriately.)

By sheer coincidence, I had already submitted https://github.com/pybind/pybind11/pull/4077
Note that pybind11 already installs cmake files, and Meson can pick those up, although again it installs them to the self-contained python package tree, which is a bit painful. There is also a Meson wrap, so dependency('pybind11') will find something via cmake, if you manually set some cmake variables, and otherwise download a private copy. I'm generally hoping to make this better (particularly, you can install pybind11 via cmake ...... && make install, which installs to the system instead and which I believe is vastly superior).

Re: pybind11, it was only recently that they moved everything into the package tree; earlier releases installed in system locations. I can see why they might want to keep things self contained, but having the headers installed in a system location does make things a bit easier from our perspective.

Randomly I just ran into a pkg-config related file for libnpymath that actually does get installed. Doesn't help much, because there's no corresponding one for libnpyrandom.a:
$ cat numpy/core/lib/npy-pkg-config/npymath.ini
[meta]
Name=npymath
Description=Portable, core math library implementing C99 standard
Version=0.1

[variables]
pkgname=numpy.core
pkgdir=/home/rgommers/code/tmp/buildroot-scipy/oscimp_br2_external/buildroot/output/host/aarch64-buildroot-linux-gnu/sysroot/usr/lib/python3.10//site-packages/numpy/core
prefix=${pkgdir}
libdir=${prefix}/lib
includedir=${prefix}/include

[default]
Libs=-L${libdir} -lnpymath
Cflags=-I${includedir}
Requires=mlib

[msvc]
Libs=/LIBPATH:${libdir} npymath.lib
Cflags=/INCLUDE:${includedir}
Requires=mlib

Re: pybind11, it was only recently that they moved everything into the package tree; earlier releases installed in system locations.

Both options are available. pybind11-global installs to the system locations (intended for use in pyproject.toml). You can get both via pybind11[global].

I tried to get rid of libnpymath.a and libnpyrandom.a completely (see https://github.com/numpy/numpy/issues/20880#issuecomment-1206846026), but that's too complex unfortunately - at least on the timescale in which we want to get this fixed in scipy 1.9.1.

I tried to get rid of libnpymath.a and libnpyrandom.a completely (see numpy/numpy#20880 (comment)), but that's too complex unfortunately - at least on the timescale in which we want to get this fixed in scipy 1.9.1.

For the record: I'm actually giving this a serious go, because it would be much healthier for both NumPy and SciPy to actually get this done. The changes are just too large to be backported to 1.9.x, so we need a workaround there.

I've really only looked at libnpymath for two minutes, but it leaves me scratching my head.  While NumPy might need this compatibility layer for itself, I fail to understand why this should be exposed as a public library. The purpose of the NumPy C API is to allow compiled Python extensions (like SciPy) to interact with NumPy data. Let those extensions worry implementation disparities for low-level functions on their own.
In short, I agree with the argument in numpy/numpy#20880 that libnpymath should be stripped way back and become nothing more than a convenience header easily vendored by SciPy.

I fail to understand why this should be exposed as a public library.

A matter of history I'd say. Pre C99 one needed all that code, and C99 support in MSVC is quite recent. And that code requires a good amount of build system support, which comes from numpy.distutils and would be very difficult to reimplement (at least before we had Meson).

There was also a problem with Windows compilers, they didn't always provide functions that could be passed into the ufuncs, but rather used inlined instructions.  Hence the renaming with the npy_* prefix. That was back around the first NumPy releases. It was a while before we could get things working on Windows.

Now, since https://github.com/pybind/pybind11/pull/4077 has landed in it's latest release. I was thinking of adding support for finding the include path via the pkg-config path, but I noticed that it doesn't really contain the prefix of the python package. And. as mentioned here earlier, it's not installed in a typical pkg-config path - meaning that in order to make pkg-config find it, a user would need to define PKG_CONFIG_PATH appropriately.
Would you be willing to accept a simple PR that will add support for environmental variables as default include paths? As suggested earlier?

it doesn't really contain the prefix of the python package

What do you mean? Is it something that we can fix? The only requirement is that it is inside site-package's pybind11 folder in the normal package (it can be anywhere in the global package).

it doesn't really contain the prefix of the python package

What do you mean?

I'll admit that I haven't noticed and/or tested the global installation option when writing my previous comment. In anycase, for the none global installation, the <prefix>/lib/python3.10/site-packages/pybind11/share/pkgconfig/pybind11.pc file has:
prefix=${pcfiledir}/../../
includedir=${prefix}/include

Name: pybind11
Description: Seamless operability between C++11 and Python
Version: 2.10.1
Cflags: -I${includedir}

Which is nice but not a huge improvement because this pkgconfig directory is not located in a standard location - meaning that even if scipy would have read the includedir pkg config variable, a user trying to cross compile scipy with a non global pybind11 installation will need to add the correct path to PKG_CONFIG_PATH when running meson setup.
However, I tested the global installation with Nix and I encountered issues which perhaps it'll be better to outline in pybind's issue tracker. I will say the the global installation did generated a useful pybind11.pc file with absolute values to the prefix and includedir.
So to summarize, the new pybind11-global can be useful for scipy, but it still needs some work. I'd still like to get feedback for the idea of simply reading plain environment variables here in scipy's meson files.

All of my comments about pybind11 were a mess. Now I noticed that running make install when building pybind11 (not pybind11-global), does create a correct pybind11.pc file at a standard location, which is great for Nix! Sorry for the noise..

My pybind11 PR was about adding pkg-config at all.
It included support for relocation because that's what the pybind11 team wants, and I agreed it's fair enough to want it -- but for distro packaging I'd strongly recommend installing via cmake, not setup.py (or else do both) and use system-integrated pkg-config files (and cmake config files as well :p) so that this all ends up in the system location.
At some point I'm probably going to add a custom dependency lookup method for pybind11 to Meson -- it will try for system pkg-config or cmake, but also handle pybind11-config, and that will allow SciPy to do dependency('pybind11'), which will look for system versions installed via e.g. NixOS, and then if those cannot be found it will also happily use a virtualenv-installed pyproject.toml build dependency that injects pybind11-config into the build $PATH.
This should then work nicely in a wide variety of circumstances, including cross compilation. In fact it already mostly does that, and the exception is "detect it via pip install in a virtualenv", so I'm waiting until I can offer a SciPy patch that both improves support and simplifies the code. ;)

The only requirement is that it is inside site-package's pybind11 folder in the normal package

@henryiii I looked into this before, concluded that it's kinda pointless to install pkg-config files in site-packages (I'd like to install them for NumPy), and that it should in principle be possible to install them to the correct location under <prefix>/include/ by using https://peps.python.org/pep-0427/#the-data-directory. Does that seem feasible to you?

@rgommers,
I don't disagree with you, but pybind11 already has pybind11_global for the exact same situation except replacing the word "pkg-config" with "cmake" everwhere.
It installs a second copy of all data files, IIRC, and then depends on pybind11 for the -config script and importable module. That's kinda extraneous but relatively speaking not by all that much.

No, pip install package should not install things outside of site packages. If a user does this without being inside a virtual environment, it messes with the base system. Try this in brew then type "brew doctor", and it will complain about mysterious files that it does not control showing up in the root. This is why pybind11 has a pybind11-global copy - the normal pybind11 doesn't touch anything outside of site-packages. The "global" exists entirely for this purpose - it is allowed to put things outside of site-packages.
There are two better options, IMO, though. One is to use an entry-point for pkg-config. Tools like meson-python and scikit-build-core could check for this entrypoint, and add the folders specified. I briefly had this for cmake files, but removed it while we design a proper name and system. But it did work, there's a PR for pybind11 and history in scikit-build-core where it was working.
Second option requires pkg-config's search to be smart enough to detect <package>/... as a pkg-config file - CMake does this (<package>/share/...), so if you add site-packages to the correct CMake variable, it can detect pybind11's config. This is a bit more limited (it has have the same name as the package it's in), but works in CMake's case, and is how scikit-build-core finds pybind11 currently. Not sure if pkg-config has the same search option.

exact same situation except replacing the word "pkg-config" with "cmake" everwhere

Is the pkg-config not in that one too? It should be, IMO.

Sorry, I meant "the original rationale for pybind11_global was this, except s/pkg-config/cmake/ since cmake was all there was at the time, so when I implemented pkg-config I just treated it the same as cmake".
pybind11_global does indeed have pkg-config files in the data directory, and that's part of the tests we hooked up. :)

but pybind11 already has pybind11_global for the exact same situation

Ah, I didn't realize that that's an actual package on PyPI. Thanks for elaborating.

No, pip install package should not install things outside of site packages. If a user does this without being inside a virtual environment, it messes with the base system.

That's explicitly the point of the .data dir in PEP 427 though. So is that one of those "good idea in principle, but you must not use this, and no we won't bother to update the PEP" things?

I think different people have different ideas of what it should mean, probably.
For example, I would consider the data directory an officially sanctioned method for messing with the base system on the grounds that it is my desire to mess with the base system.
Obviously, pip install package installs things outside of site-packages and people are okay with this, because console_scripts is installed outside of site-packages. It's just installed to the "scripts" directory instead of the "data" directory, so it's arbitrarily declared to be acceptable to install to the scripts directory specifically. After all, scripts are really common. Common enough that I suppose brew doctor doesn't consider it something to complain about.

I wouldn't say there aren't places where you can use it (pybind11_global does use it!), but it's somewhat discouraged due to the ability to do exactly what it was supposed to do - install to any arbitrary place at the root of your python environment - which includes / if you are not in a venv. That's why we have a "safe" pybind11 package and a "you know what you are doing" pybind11_global package.


[Integration] Add cleanup command to integration tests

Use crc cleanup in integration tests (at the end of each feature file). Check output at least once and use it for its functionality in the rest of the feature files.

fixed by #1177


visa icon added

Hello all,
I added the Visa icon
Paul

:tada: This PR is included in version 1.1.0-rc.11 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bump Dependencies #4727366199

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/4727366199

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Cannot connect to MySQL in Circle test environment

Even though I have set up --net=host, mysql2 still cannot connect to the circle_test database due to reasons still unknown. Currently I disabled the test component to let deployment go through, but we need to come up with a solution.

Found the reason: in test and prod we could not connect to mysql with mysql sockets since they are not generated by those systems. Therefore we need to force Rails to use TCP by using the ip addresses instead. Solved in commit 35.


fix markdown in README and docs/docker.md

My local vim incorrectly displayed the markdown.

@nepet Are the tests manually canceled? I think we can merge this one since it's a minor markdown change.

You are correct this pr does not need to be tested. To avoid spending CI time on "doc-only" prs one can add e.g .[skip ci] to the commit message https://docs.github.com/en/actions/managing-workflow-runs/skipping-workflow-runs.
Go ahead and merge If you like :+1:

Ah TIL! Will do that for future docs PRs.


How is Peak Memory per GPU lower for Full finetune compared to LoRA?

Great observation! So the numbers quoted in the README are related to the default configs.
Our single device full-fine-tune recipe has a few optimizations that the default LoRA config doesn't enable. Eg: we have optimizer_in_bwd=True which fuses the optimizer step with backward and reduces the memory footprint associated with gradients (see https://pytorch.org/tutorials/intermediate/optimizer_step_in_backward_tutorial.html for more detail). We also make use of the PagedAdamW from bitsandbytes in the full-finetune recipe compared to standard AdamW in LoRA.
There aren't any technical reasons that stops us from enabling these for LoRA. But full-finetune definitely needed more memory-optimization love to get up and running on single GPU with 24GB hence these defaults. We'll have a detailed tutorial on this topic coming out soon :)
Note: There a small gotcha here - you can't use optimizer_in_bwd with gradient accumulation (no gradients to accumulate!) and so that's something to keep in mind.

Thanks for explaining, @kartikayk. I think "gotcha" needs to be highlighted!


Update schema to reflect migrations

These colums need to be nullable and migration reflects this but
schema does not currently
To fix problems locally you can run this
rails db:migrate:down VERSION=20220708123323

rails db:migrate:up VERSION=20220708123323

What
Link to story
Describe what you did and why.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

@cpjmcquillan  @RoseSAK this should fix your local machines if you get errors saving proceedings via UI


Drop dependency from mruby-enumerator to mruby-enum-ext

The previous mrbgems with only mruby-enumerator without mruby-enum-ext as a dependency may not work.
I agree to increase independence.
However, users should be widely informed on the next release.
Actually, mruby-cfunc uses #each_slice method, but CFunc::Struct#define does not work because mruby-enum-ext is not specified.
% ./build/host+mruby-cfunc/bin/mruby -e '
	class Foo < CFunc::Struct
	  define CFunc::Int, :bar,
	         CFunc::Double, :quux
	end'
trace (most recent call last):
        [0] -e:2
        [1] -e:4
        [2] build/mrbgems/mruby-cfunc/mrblib/cfunc_rb.rb:426:in define
build/mrbgems/mruby-cfunc/mrblib/cfunc_rb.rb:426: undefined method 'each_slice' (NoMethodError)


Improve example in Navigation/NavigationView

In a previous update, One can add/remove item(s) in NavigationPane but there was no implementation of this feature in the example application.
So, the following changes are made

Add the PaneAction named Add New Item which adds a new item in NavigationPane.
Refactor the code to achieve this, i.e. Moved items outside the Widget Build function and added PaneAction in footerItems.
Refactor the code snippet according to implementation.

Before

After

Pre-launch Checklist

[x] I have updated CHANGELOG.md with my changes
[x] I have run "dart format ." on the project
[x] I have added/updated relevant documentation

Hey, I have made changes and amended the commit. Also, double quotes have been changes to single quotes in Code Snipppet too.


Trying to get current user locale

Hello
I'm trying to adapt my calendar to the current user locale. I tried :
var userlocale = <%= GetCurrentCulture %>;
$('#datetimepickerfin').datetimepicker({
    locale: userlocale,
    showTodayButton: true,
    defaultDate: moment({ h: 23, m: 59 })/*Heure par defaut : le jour même à 23:59*/
});

But the calendar just doesn't show up.
I tried
locale: navigator.language || navigator.browserlanguage
But I got the installation language of the browser, which doesn't help me as I have a multilingual web page and the datetimepicker still displayed in french although I have set the prefered language to german.
Anyone had the same thing ?
Yann.

what does var userlocale = <%= GetCurrentCulture %>; actually render in the browser as?
the local must match one of Momentjs' local files


Adds the company for our sponsors

Fix #113

@doublejess does this fix your issue, and can you replace your placeholder sponsor.login?


[Automation] P0 Scenarios

@maryjaf @jainkrati what is the context for this issue? is it still relevant?


Request entity not found

Exporting one or many fusion tables returns an error:
"Error! Requested entity was not found."
Reported by @briandavidthom
I have experienced the same issue. It does not depend on different tables. All tables are covered.
It happens after a long spinner, not our regular progress screen.
My guess is that has to do with the fusion tables API or the drive API.

Hm, the message Requested entity was not found is thrown by fire base. Might be an issue there?

It seems the error occurs when there is already an ft-archive folder created form a different application.
Possible solution:
include the creater (the app that created the folder) in this query:
https://github.com/google/fusion-tables-drive-export/blob/master/server-src/drive/find-file.ts#L49

Question remains: why do we get an error form datastore?
We should fix that as well.

Additional info from Russel:

One of the OAuth scopes is that the tool can write to files that this particular OAuth client has created... I had to switch cloud projects at one point (remember the app was not deploying) and so the OAuth client changed.
And the new client cannot write to the sheet ft-archive-index that the old client had created.
The solution is to rename the existing sheet from ft-archive-index to .

The error isn't thrown by datastore. It's the error message of a 404 error from the sheets API.


Add a nonblocking LED display driver (draft 2)

Supersedes PR #14
This version is based on the new HiResTimer implementation in nrf51-hal (Cargo.toml specifies nrf51-hal = "0.6.0" but it really needs today's master to build).
This is the core of the code from https://github.com/mattheww/microbit-blinkenlights, as discussed in #13.
I've left out the font and scrolling support to start with.
Notes:

I haven't done anything with the existing 'blocking' led.rs module.

The rustdoc is using RFC1946-style links, which aren't stable but do work on docs.rs . I can remove those if you don't want to require nightly rustdoc.

This version is based on the new HiResTimer implementation in nrf51-hal (Cargo.toml specifies nrf51-hal = "0.6.0" but it really needs today's master to build).

I've just published a new 0.6.2 version.

The rustdoc is using RFC1946-style links, which aren't stable but do work on docs.rs . I can remove those if you don't want to require nightly rustdoc.

I'm not sure which version is currently utilised by crates.io but I would assume it's nightly so that should be fine.
Other than that, looks excellent to me. Thanks a lot!

As I understand it docs.rs uses nightly rustdoc, yes.

Love the example. 👍

@therealprof This looks like it's ready to merge (and the example does look great)

Hmm, completely forgot about this. @mattheww Can you resolve the merge conflicts?

I've resolved the merge conflicts.

I've merged it manually to get rid of the merge commit, thanks.


GH Action publishes release artifacts when any tag is pushed

The github release action publishes release artifacts (publishing to helm stable) when any tag is pushed.
https://github.com/stackabletech/operator-templating/blob/main/template/.github/workflows/publish_release_artifacts.yml.j2#L10
This should not be. Only when a tag is in main should anything actually be released.

Especially since we might switch to a Git workflow where we might have a release branch (or version branches for LTS purposes that might never be merged to main again).

I just talked with Razvan and this is not an isolated issue, so I'll close it for now.


Scaling / sizing label

it is posible to add an option to set the whole sizing of the label? so it would be possible to print directly on a labelprinter and not on a a4 paper.

I have the exact same question, we've bought a labelwriter for printing qr codes directly in the hosting center on durable labels, and we do not have access to a printer and can't place a printer in the hosting center, except a small label printer.

I would also like this feature very much. I have a Brother PT-E550W and would really like to be able to print on 9 mm wide tape. There should be some scale-to-height functionality.

Has anyone got usable template for 12mm (0.47'') continuous labels. I use scaling in print settings for now

Has anyone tried Brother's SDK?
https://developerprogram.brother-usa.com/labelling-and-mobile-developer-program

I recently built an HTML label version which works quite promisingly under Firefox. I have to integrate it into the QRCode plugin soon and wanted to suggest it as a pull request.


Added Train Cloud

This is really cool! I'm not sure it could be used on the labels in real life, though, due to the dimensions, but we;ll see.


Basic animations

Hello, I have a conceptual idea and I would like to know if I could carry it out with Blazor3d. I have not worked much with the 3d theme, I have a basic gltf model and I would like to do basic programmatic animations, how to rotate certain parts of the model, could this be done? I have seen animations of the entire object, but can it be done on a specific part?

Hi. I believe it can be done.
But it requires a huge effort to do this.
At the moment, after loading any 3d file into the scene, only JavaScript side (ThreeJS) knows all about the loaded file structure (which node is parent, which are children, their locations, rotations, scaling, etc). It notifies the .Net side about the object id and name, but not about its internal structure.
So, firstly, you need to implement this.
The next thing to do is to implement basic animations (changing positions, rotations, etc)
After that, you can programmatically analyze the loaded model structure and apply required animations for required nodes
It can be done, but at the moment I have no time to do this.

Thanks for answering, I'll be watching the project


Apple Pay reload loop on confirm page / CheckoutConfirmPageLoadedEvent executed twice

Used Shopware Version: <IP_ADDRESS>
Description
We have found a conflict between the PayPal plugin and the SwagCommercial plugin.
When Apple Pay is selected, the Confirm page always reloads.
Steps to reproduce

Activate PayPal Apple Pay
Activate SwagCommercial
Add product to shopping cart and click through to Confirm Page
Select Apple Pay

Actual result
If you select the payment method Apple Pay with a browser that does not support Apple Pay, the page is constantly reloaded.
Expected result
If you select the Apple Pay payment method with a browser that does not support Apple Pay, an error message is displayed stating that the browser is not supported.
Reason
In the SwagCommercial plugin, the CheckoutController or confirm page is decorated:
Shopware\Commercial\B2B\OrderApproval\Storefront\CheckoutControllerDecorator
This causes the event CheckoutConfirmPageLoadedEvent to be executed twice and PreventErrorReload is removed to early in the PayPal plugin.
Here is the relevant snippet from the CheckoutControllerDecorator:
$page = $this->confirmPageLoader->load($request, $context);
// ...

$notAllowToOrder = $this->getApprovalRuleMatched($cart->getErrors());
if (!$notAllowToOrder) {
	// ...
	return $this->decorated->confirmPage($request, $context);
}

Workaround
As a workaround, we have adapted the Decorater via Composer Patch as follows:
public function confirmPage(Request $request, SalesChannelContext $context): Response
{
	return $this->decorated->confirmPage($request, $context);
	// ...
}

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

[public]

FixVersion Change: <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>.
[created from NEXT-37580, comment 510412]

[public]

FixVersion Change: <IP_ADDRESS>, <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>, <IP_ADDRESS>.
[created from NEXT-37580, comment 510455]


Can't load the 'dictionary' source

The Neovim version:

Other sources work normally and 'dictionary' can't work:

Here's my settings:

The .dic file is downloaded from github, I don't know if it would be the cause.

I have changed dict to dic but still does not work.

Yes. I have made a destructive change and released v2.0.0. The new setup instructions are in the doc; you can also use the previous version with a tag designation.

@uga-rosa I'm having trouble with the new release. Not sure what I'm doing wrong.
old config
require("cmp_dictionary").setup({
  dic = {
    ["*"] = "~/en.dict",
  },
})

new config
require("cmp_dictionary").switcher({
  spelllang = {
    en = "~/en.dict",
  },
})


Add project name in terminal tab

sometimes，i open workspace that contains lots of project,i can't switch terminal easily.

Did you find the Terminal: Rename command in Command Palette?

yeah, i got it, thanks.


Appstudio update hacbs-test-project-vmts

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-vmts-on-pull-request-
creating pipelinerun hacbs-test-project-vmts-on-pull-request- in namespace rhtap-demo-xwgq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Delete docker-compose.yml symlink

Fix #67
A symlink to a docker-compose.yml file was added to the project but
should be removed. Because the link points to a location that doesn't
exist on a local dev's hard drive, the file always shows up as being
modified -- preventing operations such as rebase. There's no need to
add a symlink to a git repo in any case.
Signed-off-by: Trevor Conn<EMAIL_ADDRESS>

Hmm the diff here looks odd on github, as it shows up as a binary file added, but locally the git commit diff looks correct.

Yeah, I noticed that too. @tingyuz Can you review/approve please?

The same happened with my PR as well. I guess github can't handle binary correctly.


Lets make supporting of unique fields

Example:
jsonApi.define({
  resource: "example",
  handlers: new jsonApi.MemoryHandler(),
  attributes: {
    title: jsonApi.Joi.string(),
    name: jsonApi.Joi.string().unique(), // When we create or update this field - check for unique and show warnings
  }
});

@theninj4, @pmcnr-hx, what do you think?

There are two perspectives to consider here:

When using a stock resourceHandler (eg, the Mongo handler or the RelationalDB handler) - For these situations the best place to enforce uniqueness will be directly on the data store, be adding some kind of unique index on the necessary properties.

When using a custom resourceHandler (eg, consuming another API or internal system) - For these situations the best place to enforce uniqueness will be within the custom resource handler, where the author will be best placed to know how to efficiently decide if a given value is acceptable.

Adding something to the Joi schema may add value in terms of autogenerating documentation, but I think it'll lead to a less-than-ideal implementation if we were to try and use it to magically change the behaviour of a handler.


Read a text file from server using GET

How do I read a file from server using GET method? Is it supported in this library?

Hi,
This library does not implement the http protocol. You can still fetch files, but you need to send the http commands your self. See examples of http GET here:
http://www.tutorialspoint.com/http/http_requests.htm
Regards
Jonas Ekstrand

25 juli 2016 kl. 08:51 skrev Prithviraj Shetty<EMAIL_ADDRESS>How do I read a file from server using GET method? Is it supported in this library?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Cannot Camel Case Anonymous Object Within JsonExtensionData Property

I am using JSON.NET v10.0.2 and trying to serialize this JSON:
{ "id": 1, "Name": "Foo 1", "Description": "Foo 1 Description", "MyObject": { "Property1": "Value1" }, "MyArray": [{ "Property1": "Value1" }] }
Using this model:
[JsonObject(NamingStrategyType = typeof(CamelCaseNamingStrategy), NamingStrategyParameters = new object[] { true, true, true }]
public class MyClass
{
  [JsonProperty(PropertyName = "id")]
  public int Id { get; set; }
  [JsonExtensionData]
  public Dictionary<string, object> Data { get; set; }
}

The result that's coming back is:
{ "id": 1, "name": "Foo 1", "description": "Foo 1 Description", "myObject": { "Property1": "Value1" }, "myArray": [{ "Property1": "Value1" }] }
When I expected the result to be:
{ "id": 1, "name": "Foo 1", "description": "Foo 1 Description", "myObject": { "property1": "Value1" }, "myArray": [{ "property1": "Value1" }]  }
This seems to be a bug within the JsonSerializer. Any assistance?

This isn't a bug. Ask on Stackoverflow for help.

Have the same issue.
On Stackoverflow said ask here.


Image changes

logo is no longer required
an image can added instead of logo
updated version
added facebook
added option to change text for phone number, email and website
added option to change the font size of the company name

Thank You for the efforts ❤


Bulbs No Longer Found

Hey there - I know you've moved on to the Philips Hue ecosystem, but I'm trying to get my Wiz lights working with Homebridge. They were working fine in HOOBS, but after switching over to the official Homebridge Raspberry Pi OS, and installing your plugin, nothing is showing up. Could you possibly see if anything has changed with their API, or if something was broken in the more recent Homebridge updates? I appreciate your help.

Hey, unfortunately I can't provide much support at this point. https://github.com/victori/homebridge-wiz-lan seems to have a working build updated for Costco bulbs, so feel free to try that out.

Sorry. Completely new to this. His build seems be the exact same install as yours.  How do I install his updates?
Thanks!

Closing due to new V3.0.0 update which should fix most things. Feel free to open another issue if problems persist!


chore: bdd-spring-1575250867 to 0.0.1

chore: Promote bdd-spring-1575250867 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Added missing keycodes for FromStr

FromStr impl was missing Key0 and Numpad keys

Thanks for catching this! Greatly appreciated.


udp message from ipv4 to ipv6

is it possible to send a udp message to a node in a thread network?
i can send a udp message from a node on my computer, this computer is connect to a ipv4 only network. this is done by using tayga. but how can i responde to this?

OTBR implements NAT64 using TAYGA. As is typical with stateful NAT, communications must be initiated by Thread devices so that the NAT can setup the network translation mapping. An IPv4 host responding to a UDP/IPv4 message initiated by a Thread device should be able to respond to the packet's IPv4 source address and UDP source port.

ok, thanks. but the source address from the ping and the upd message are from raspberry pi.
what am I doing wrong then? Tayga gives the node in the thread network an ipv4 address from the dynamic pool. but the source address is still from the raspberry pi.

ok, thanks. but the source address from the ping and the upd message are from raspberry pi.
what am I doing wrong then? Tayga gives the node in the thread network an ipv4 address from the dynamic pool. but the source address is still from the raspberry pi.

That is how stateful NAT is typically configured. The NAT uses the IPv4 address assigned to its external interface as the external IPv4 address and uses the external UDP port to map to an internal IPv6 address an UDP port.


Is it possible to wait evaluation success?

Can horseman wait while evaluate function return true. For websites using ajax to generate content

That's exactly what waitFor is.

@awlayton yes, sorry for dummy issue. I just found it myself


Update sphinx version and switch to github actions

We were still on the now defunct travis and on a pretty old sphinx version.
No changes to the content.

@maxnoe Thanks -- should I merge this now?


Generate .gts not .g.ts to avoid confusing intellisense

Also update the pxt.json generation

I think pxt.json is fine - we generate pxt.g.json only if nothing is there


Types aren't recognized until I open them in another tab

VSCode Version: 1.49.0
OS Version: Win 10

Steps to Reproduce:

clone https://github.com/sweetalert2/sweetalert2-parcel-demo
yarn install
open app.ts

Current behavior

Expected behavior

The WEIRD part
As soon as I open node_modules/sweetalert2/sweetalert2.d.ts in another tab it works as expected:

Does this issue occur when all extensions are disabled?: Yes

What if you remove the .js in the import?

What if you remove the .js in the import?

Same.

@mjbvz small detail to ad here (it might help): when I close node_modules/sweetalert2/sweetalert2.d.ts the unexpected behavior is starting again and Swal isn't recognized.

From https://github.com/sweetalert2/sweetalert2/issues/2056

I think it may be caused by VSCode not loading the data tag file sweetalert2.d.ts, but Javascript also clearly stipulates that Swal is a class and a local file, which makes no sense.

From https://github.com/sweetalert2/sweetalert2/issues/2056#issuecomment-701328165

Another proof that it's the VS Code issue is that in WebStorm all works as expected:

tsconfig.json was missing in that project:
{
  "include": [
    "app.ts",
    "node_modules/sweetalert2/sweetalert2.d.ts"
  ]
}

https://github.com/sweetalert2/sweetalert2-parcel-demo/commit/ea7846d216fe2eb152e0b98df4dc059f200ba765


customizing the join us and more options button to make it even with other buttons

Description
The join us and more options button needs some changes!
Screenshots
The button's hover effect should be changed from :

To:

The second screenshot looks a bit better then the first one!
same changes can be done on the more options button

can I work on the above mentioned changes?

Go for it!
For support join our discord community: https://discord.com/servers/codein-community-831388874118660156

Hi, can i work on this issue ??

hey @prakhar-7129 this task is already assigned to me and I am going to create a PR within 10-15mins so kindly look for other issue!😅


Implement shifting of uid/gid in xattrs

This issue is to track code changes to shared/idmapset to implement the shifting of those uid/gids that are stored in xattrs. This includes:

Filesystem ACL entries
Unprivileged filesystem capabilities

@albertodonato can you take a look at this one?
The filesystem capabilities isn't a thing right now but the filesystem ACLs are.
We need Shiftrootfs and Unshiftrootfs to also apply to filesystem ACLs so that we don't end up with partial shifts of the container storage.


notmuch-bower: 0.10 -> 0.11

Motivation for this change
https://github.com/wangp/bower/blob/0.11/NEWS
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @erictapen

When trying to run bower it doesn't find notmuch:
# ./bower
/bin/sh: line 0: exec: notmuch: not found
Errors in configuration file:
process returned with exit code 127
could not derive default from_address from .notmuch-config

@mmahut It doesn't make sense to use notmuch-bower without notmuch so I guess that is not a problem.
For me this works, thanks for the bump, @dtzWill!

@erictapen should that be taken as a dependency then?

@mmahut From the notmuch-bower Readme:

Bower is designed such that it can be run on the local machine but call out to notmuch on a remote machine (that holds your mail) via ssh. The advantage is that you can start helper programs on the local machine (e.g. a web browser or image viewer), and add or save attachments on the local filesystem, even if your mail archive is stored on a different machine.

So I guess that usecase doesn't need a locally installed notmuch. In case you still think notmuch should be installed could you open another issue for this? The behaviour you describe has been the case for all former versions of notmuch-bower.

@erictapen I see, thank you for explanation!

Built and tested locally :+1:

@dtzWill thanks!


Fix upload_screenshots.rb requiring the options hash (#10173)

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
#10173 describes the issue.
I've tested it using the local version fastlane to actually deliver an app I'm working on.
Description
A previous commit edited deliver's upload_screenshots.rb in a way that broke the function. This change fixes it while keeping the reasoning behind the previous change (I hope!).

I signed it!

Thanks for making your first contribution @Orgmir! Welcome to the fastlane community! 🚀

Hey, thanks Orgmir for this patch. Currently facing the issue this fixes, just wondering when a release is planned?

Hey @kulor, we will likely get a release out first thing tomorrow if that is okay with you? 🚀

Thanks guys! @kulor You can manually edit the file with the same changes I did while we are waiting for a new release.


Tid fix domain name

This only an example of how to setup domainName instead of interfaceDescriptions at registration on capif.
This code solve the issue we found on testing that we must use DomainName parameter on AEF publication becasue on k8s environment we need to setup host name and port to access instead ip and port.

I can't really test this right now but the changes look sensible, if it is working in your setup then it should be ok.
I made a small update, but it should not change the behavior in any way. Merging.


Is there any way to specify a custom link widget?

I'm able to specify custom node widgets but custom link widgets aren't working.
On line 388 of DiagramWidget#onMouseDown, we're instantiating new LinkModel(). Shouldn't this be coming from one of the instance factories?

Fixed it here: https://github.com/emumba-com/react-js-diagrams/commit/034f7282da51bd90814e36ac70f7b32bd55ffdf2

@woodenconsulting You could've asked for a PR instead of copy pasting :)

@umarashfaq I went with a slightly different approach, please see demo5. When a link instance factory is defined that will be used by default rather than setting via a prop. Please feel free to create PRs for future issues, it's much appreciated!


Remove user scope from filips.FirefoxPWA 2.5.0

user scope for WiX installers generally does not work well because of a bug/design issue with WiX installers in that the installer installs per-user but incorrectly writes the uninstall entries to HKEY_LOCAL_MACHINE that makes WinGet think it's a per-machine install. This causes users not being able to upgrade the package after installation through WinGet. For best UX, it's best to remove the user scope from the manifest

Microsoft Reviewers: Open in CodeFlow

Is there any way to fix the installer, so it works properly both in user and machine scope?

This is an issue with almost every MSI installer that is made using WiX. You can find related discussion over at https://github.com/microsoft/winget-cli/issues/3011

So, this means that WiX-made installers cannot be installed in a user scope with winget, unless this issue is fixed in WiX Toolset or winget-cli gets some other way of detecting installation scope?

So, this means that WiX-made installers cannot be installed in a user scope with winget, unless this issue is fixed in WiX Toolset or winget-cli gets some other way of detecting installation scope?

Unfortunately yes. WinGet detects scope from where the installer writes its uninstall entries in Registry (HCKU corresponds to user, HKLM -> machine). The issue is that if a user installs a WiX installer in user scope, subsequent updates would fail because WinGet looks for a compatible installers in the upgrade scenario that matches the architecture and scope of the current installation. Since the current installation would be detected as "machine-scoped", WinGet would fail with No applicable upgrade found


Compute encodings using AMP in TransformerTorchEncoder

Describe the feature
Encodings could be computed using Automatic Mixed Precision for TransformerTorchEncoder when the model is on GPU - this practically doubles the computation speed (on GPUs that support it), while having almost zero effect on the precision of the results.
Your proposal
Just adding mixed precision is easy: the following line from jina.hub.encoders.nlp.TransformerTorchEncoder
        with self.no_gradients():

would be replaced with
        with self.no_gradients(), torch.cuda.amp.autocast():

Checking can be done if the model is on GPU or not. Possibly it would also need to be checked whether autocast is availible or not - as it was introduced fairly recently (I think 1.6.0, not sure if the minimal requirement in Jina covers it). But where it is supported this would definetly be a welcome performance boost :)

@tadej-redstone was this addressed by your last refactoring?

Not yet - I'll be adding this and a quantization option in another PR
On Wed, Dec 23, 2020, 10:06 AM Joan Fontanals<EMAIL_ADDRESS>wrote:

@tadej-redstone https://github.com/tadej-redstone was this addressed by
your last refactoring?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jina-ai/jina-hub/issues/1283#issuecomment-750024003,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQUQGD7JNAAN4TWOJDL2BD3SWGXHXANCNFSM4UWZFSQA
.


Gallery Mode with wrap-around = false doesn't work properly.

Describe the bug
When using the double carousel mode, show in the docs as the gallery demo (https://ismail9k.github.io/vue3-carousel/examples.html#gallery). The gallery thumbnails do not correctly navigate to the appropriate slide when setting wrap-around to false, if the slide index is outside of the min/max slide settings.
To Reproduce

Copy the gallery example from the docs.
Set wrap-around to false and set items-to-show to 4.
To the slide, add a button that sets the v-model value of the slide to the slide index.
Click through the slides. At the end of the list, note that the button does not allow you to select the last couple of slides.

Expected behavior
When wrap-around is disabled, the gallery thumbnails should still be selectable.

Here is a codepen that demonstrates the issue:
https://codepen.io/dalemac89/pen/XWGVKNK
To see the issue, load the codepen above and from the bottom carousel click slides 7, 10 and 13. Notice how when clicking slide 13 it then changes you back to slide 12.
Weirdly, the problem doesn't present itself if there are fewer than 10 slides.

For anyone coming across this issue in the future, the workaround for this is to use two separate data properties, one for each of the carousels. When clicking the button in the gallery, update both carousel values. The carousel will internally reset the value for the gallery carosuel without updating the main carousel value.
This will introduce another issue as when you change the slide on the main carousel you'll no longer be updating the value for the gallery carousel. I used the slide-start event on the main carousel to then update the value of the gallery carousel.
This workaround resolved my issue, but it would be cleaner if a proper fix could be found.

The original reproduction you provided is now working, so this issue has been fixed


Minimization or angular-timezone-select as a bower_component in a angular-fullstack project breaks.

This is not directly an issue of angular-timezone-select, which I think is great.
The issue is that in my yo angular-fullstack project, this one bower_component breaks when minimized, etc.  i.e. grunt server:dist breaks.
However if I just insert the proper files by hand:
link rel="stylesheet" href="bower_components/angular-timezone-selector/dist/angular-timezone-selector.min.css"
script src="bower_components/moment-timezone/builds/moment-timezone-with-data-2010-2020.js"
script src="bower_components/chosen/chosen.jquery.min.js"
script src="bower_components/angular-timezone-selector/dist/angular-timezone-selector.min.js"
Then grunt serve:dist runs properly.
So what are the special settings need to properly perform the uglify, minimization?
I already have:
uglify: {
options: {
mangle: false
},
},
Thanks!

It turns out the issue was related to a problem had nothing to do with angular-timezone-select.  So I am closing.


Fix dependency alert

https://github.com/omgnetwork/plasma-contracts/network/alerts

Can't seem to access this


Quests randomly resetting

(Required) What is your Quests version / build number (do not say "latest")?
Dev build 196
(Required) Please describe the problem and how to replicate it. Any console errors?
No console errors, players randomly lose their progress.. This hasnt happened on a small quest only the bigger ones such as killing 500 zombies. It doesnt always occur it's random.
(Optional) Can you provide an example quest from quests.yml using Pastebin.com?
https://pastebin.com/PzM9ndje

Can someone help me with this..?

@Sjilli After a little experimenting, I believe you may be dealing with a bug that was fixed recently. Please give the latest dev build a try (202 at time of writing) and let us know if the issue persists (if so, check for new errors).

Alright, if I replace my current quests.jar with this one, will it replace my quests and I'll have to re-add them?
Thanks in advance.

@Sjilli No, the build I linked will not replace any Quests you have made, so long as you do not delete quests.yml

Brilliant, thank you very much!

I had the same bug now, it happened to one player only though. Some daily quests got reset after restart, even thou it was not 24 hours yet

Since I first tackled this nearly a year ago, I've been unable to recreate this in test environment across multiple versions of Quests, Spigot, and Java. Today, I gave it one last go, but could not replicate the problem. It's entirely possible this issue has been resolved as the result of another ticket, or it may be OS dependent (I've been using Windows 7).
It's time this issue be closed. It may be reopened if new evidence/leads are brought forward. For future reference, the quests.yml file I used for testing is linked below.
https://pastebin.com/Hc5MW8ZY


chore: version packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#1261 f599246 Thanks @manudeli! - feat(react-query): remove experimental jsdoc of infiniteQueryOptions,<Mutation/>

Updated dependencies [f599246]:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#1261 f599246 Thanks @manudeli! - feat(react-query): remove experimental jsdoc of infiniteQueryOptions,<Mutation/>

Updated dependencies []:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#1261 f599246 Thanks @manudeli! - feat(react-query): remove experimental jsdoc of infiniteQueryOptions,<Mutation/>

Updated dependencies []:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.39%. Comparing base (f599246) to head (f59b71f).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1262   +/-   ##
=======================================
  Coverage   73.39%   73.39%
=======================================
  Files          81       81
  Lines         827      827
  Branches      183      183
=======================================
  Hits          607      607
  Misses        196      196
  Partials       24       24

Components
Coverage Δ

@suspensive/utils
47.16% <ø> (ø)

@suspensive/react
100.00% <ø> (ø)

@suspensive/react-dom
96.66% <ø> (ø)

@suspensive/react-native
100.00% <ø> (ø)

@suspensive/react-query
72.09% <ø> (ø)

@suspensive/react-query-4
0.00% <ø> (ø)

@suspensive/react-query-5
0.00% <ø> (ø)

@suspensive/react-query-next-experimental
∅ <ø> (∅)

@suspensive/react-query-next-experimental-4
6.15% <ø> (ø)

@suspensive/jotai
0.00% <ø> (ø)

@suspensive/cache
98.01% <ø> (ø)

@suspensive/react-image
80.39% <ø> (ø)


[Bug] Issues when not using vela-system namespace

Describe the bug
Upstream Chart can not be installed into custom namespace.
We are using a quite strict naming policy for the namespaces used in our clusters, e.g. app-some-name for application namespaces and sys-some-name for cluster services. This implies we can not use the preferred namespace name vela-system as described in the installation guide.
With some manual tweaking of the helm charts templates we got KubeVela running on our dev cluster, but we want to use the upstream helm chart in the future.
We also encountered the problem, that the vela CLI tool is not capable of listing showing any traits, components, or apps when using a custom namespace.
To Reproduce

Install steps as described in the documentation (Advanced with use of cert-manager)
Using different namespace for the helm chart: helm install --create-namespace -n sys-kubevela --set admissionWebhooks.certManager.enabled=true kubevela kubevela/vela-core configured as a flux2 HelmRelease configuration
Observe that KubeVela will not start.
Run helm template and manually set missing namespace fields and overwrite hardcoded vela-system references, apply modified manifests.
Observe that everything can by applied, test application runs
vela CLI tool not functioning:$ vela show ingress
Error: could not find ingress in namespace default, or vela-system

kubectl vela plugin works as a namespace can be specified as a parameter.

Expected behavior
Helm Chart can be installed with custom namespace, tools are working correctly with a given namespace as a parameter, e.g. vela show -n namespace-name ingress.
Screenshots
none
KubeVela Version
1.0.6
Cluster information
Kubernetes Version 1.19.9
Additional context

Hardcoded vela-system namespace in:$ rgrep -n -e vela-system ./charts/vela-core/*
./charts/vela-core/crds/core.oam.dev_applications.yaml:5:    cert-manager.io/inject-ca-from: vela-system/kubevela-vela-core-root-cert
./charts/vela-core/crds/core.oam.dev_applications.yaml:15:          namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:10:      namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:29:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:32:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:47:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:50:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:53:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:139:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:147:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:155:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:165:                  app.kubernetes.io/instance: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:352:                  app.kubernetes.io/instance: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:567:                  app.kubernetes.io/instance: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:758:                  app.kubernetes.io/instance: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:1273:                  app.kubernetes.io/instance: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:1871:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:2012:                namespace: vela-system
./charts/vela-core/templates/addons/fluxcd.yaml:2031:                namespace: vela-system
./charts/vela-core/templates/test/test-application.yaml:45:          kubectl -n vela-system wait --for=condition=Applied application helm-test-vela-app --timeout=3m
./charts/vela-core/templates/test/test-application.yaml:50:          kubectl -n vela-system wait --for=condition=available deploy helm-test-express-server --timeout 3m
./charts/vela-core/templates/test/test-application.yaml:54:          while ! [ `kubectl -n vela-system get ing helm-test-express-server | grep -v NAME | wc -l` = 1 ]; do
./charts/vela-core/templates/defwithtemplate/raw.yaml:8:  namespace: vela-system
./charts/vela-core/values.yaml:91:systemDefinitionNamespace: vela-system

No namespace specified for Issuer and Certificate in certmanager.yaml

Thanks for reporting, I assume KubeVela should be able to deploy in any specified namespace. Maybe it's a bug and needs to be investigated

Let me provide more info: All files in ./charts/vela-core/templates/addons are ConfigMap for vela CLI.
And now the namespace of configMap will change to helm chart style( {{.Values.systemDefinitionNamespace}} ). ref to #1915
Some of the addon like fluxcd as you see. The plenty vela-system string are data in configMap, which doesn't have any effect for helm (and now it's was move to flux-system)

/assign

@o-otte  Hi, I try to reproduce your problem and found you did't specify the systemDefinitionNamespace field.
Therefore, to run kubevela normally, you can install as bellow:
1.  Install official cert-manager
kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.4.0/cert-manager.crds.yaml

2. Install chart in remote repository,
helm install --create-namespace -n sys-kubevela --set admissionWebhooks.certManager.enabled=true --set systemDefinitionNamespace=sys-kubevela  kubevela kubevela/vela-core

or Install chart in local repository,
helm install --create-namespace -n sys-kubevela --set admissionWebhooks.certManager.enabled=true --set systemDefinitionNamespace=sys-kubevela  kubevela ./charts/vela-core

you can also use  `--dry-run` args to see that the namespace "sys-kubevela" has be specified for Issuer and Certificate.

And the vela show CLI currently only support show namespace in "vela-system" and "default"
We intend to fix it to support kubectl vela plugin works as a namespace can be specified as a parameter.
like "vela show -n namespace-name ingress".
Or would you like to contribute a PR to fix this so that other people in community will be able to solve this similar scene problems more smoothly.  @o-otte

@wangyuan249 Hi and thank you for hint with the systemDefiinitionNamespace field! I wasn't aware of it. Nevertheless, I have a question regarding the hardcoded "vela-system" namespace inside the ./charts/vela-core/crds/core.oam.dev_applications.yaml CRD on lines 5 and 15. For us it seems to produce errors if cert-manager related resources (Issuers, certificates) are deployed in another namespace. The problem did not show on the setup, but on the first use of the webhook. The problem could be solved by changing the namespace in the aforementioned CRD. I think it could be relevant to your PR #1922
Regarding the PR for the CLI tool, I don't have much time, but I'm willing to help on that. Can you provide my with some info on how and where to start on that specific topic?

@o-otte  thanks for your feedback.
When installing a helm chart, the first step is to install pre-resource such as crd or other Dependent sub-chart, the second step is our def template.  While in the first step, the crd could not perceive this variable systemDefinitionNamespace

Therefore, currently my solution on installing self-def namespace cert is to change the origin path config/crd/pathes/...yaml

and use make  manifests to gen crd in chart

And the specific logic for vela show in shown in figure.

I think this should be supported in v1.3.x. please have a try. Feel free to re-open this issue if there's still any probelm.


add sureflap:1.0.5 to stable

Automated adapter checker
ioBroker.alias-manager

[ ] :heavy_exclamation_mark: [E154] common.dependencies must contain {"js-controller": ">=2.0.0"} or {"js-controller": ">=3.0.0"}

ioBroker.fuelpricemonitor

[ ] :heavy_exclamation_mark: [E405] Icon must be in the following path: https://raw.githubusercontent.com/HGlab01/ioBroker.fuelpricemonitor/main/
[ ] :heavy_exclamation_mark: [E407] Meta URL(latest) must be equal to https://raw.githubusercontent.com/HGlab01/ioBroker.fuelpricemonitor/main/io-package.json
[ ] :heavy_exclamation_mark: [E426] Icon(stable) must be in the following path: https://raw.githubusercontent.com/HGlab01/ioBroker.fuelpricemonitor/main/
[ ] :heavy_exclamation_mark: [E428] Meta URL(stable)  must be equal to https://raw.githubusercontent.com/HGlab01/ioBroker.fuelpricemonitor/main/io-package.json

ioBroker.lg-ess-home

[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2021 Morluktom<EMAIL_ADDRESS>at the start of LICENSE

ioBroker.sureflap

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

Hi,
sorry that it took that long, some review topics:

Note: Because of the admin 4.0.9 globalDep and controller dep in your io-package you do nt need the manual encrypt/decrypt in your index_m file ... but also do not hurt if it is in
Note: https://github.com/Sickboy78/ioBroker.sureflap/blob/master/main.js#L89 ... we have setObjectNotExistsAsync :-) But ...
Note: You can also just subscribe to the relevant objects where actions are allowed instead of "any" own statechange ... like "control.*" or such
It seems to me that you do several API requests and then set data. If this can take in sum (for all rewquests and stuff) > 500ms then in case of an unload you could run into the effect that you clear the updateTimer (but this has no effect because is irrelevant becasue you do all the api requests ... and at the end you set the timer anew - but unload was already finished, so nooen would clear the timer again. So in compact mode your adapter runs forever spamming the log with a lot errors. Ideally remember that unload was called and add checks after each "http request" before processing the data and especially before setting update Timeout again.

Please check out especially the last note and add a guard ... other then that adapter is fine
Please poke again when done, Thank you

Hi @Apollon77 ,
I have removed the deprecated manual encryption (point 1) and added a guard, to make sure, no timers are called after unload  (point 4).
Point 2 and 3 need a bit more extensive code changes, I'll do them with the next bigger refactoring.
Hope, thats ok for now.
Best Regards

@Sickboy78 Please change the year of the copyright to 2022 or 2021-2022
https://github.com/Sickboy78/ioBroker.sureflap/blob/14d6d4ee6720267b814705b5b8e9a570363c0dba/LICENSE#L3

@ldittmar81 done

@Sickboy78 All good and thank you for your


Added usage example

To make future first usages easier I added a small example app.
It's just an Nginx instance returning a constant string via SSL. Certificates are registered for the domain yourhostname.com on the email<EMAIL_ADDRESS>Also, I adjusted the folder structure of this project so that the example does not interfere with the actual source code. There is a src folder where all source files were moved to and an example folder with all files requred by the example app.
Hope this will be useful in illustrating the usage of this project.

Yeah, sure, this seems reasonable.  Thanks!


Problems installing on Ubuntu

Hello! Thanks a lot for pygraphblas, it makes life easier)
But for some reason docker doesn't suit me and I need to install pygraphblas directly. I tried to run the install-ubuntu.sh, but got the errors like this:
build/temp.linux-x86_64-3.7/_pygraphblas.c: In function ‘_cffi_var_GrB_BAND_INT16’:
build/temp.linux-x86_64-3.7/_pygraphblas.c:38345:12: error: ‘GrB_BAND_INT16’ undeclared (first use in this function); did you mean ‘GxB_LAND_INT16’?
38345 |   return &(GrB_BAND_INT16);
      |            ^~~~~~~~~~~~~~
      |            GxB_LAND_INT16
build/temp.linux-x86_64-3.7/_pygraphblas.c:38345:12: note: each undeclared identifier is reported only once for each function it appears in
build/temp.linux-x86_64-3.7/_pygraphblas.c: In function ‘_cffi_var_GrB_BAND_INT32’:
build/temp.linux-x86_64-3.7/_pygraphblas.c:38350:12: error: ‘GrB_BAND_INT32’ undeclared (first use in this function); did you mean ‘GxB_LAND_INT32’?
38350 |   return &(GrB_BAND_INT32);
      |            ^~~~~~~~~~~~~~
      |            GxB_LAND_INT32
build/temp.linux-x86_64-3.7/_pygraphblas.c: In function ‘_cffi_var_GrB_BAND_INT64’:
build/temp.linux-x86_64-3.7/_pygraphblas.c:38355:12: error: ‘GrB_BAND_INT64’ undeclared (first use in this function); did you mean ‘GxB_LAND_INT64’?
38355 |   return &(GrB_BAND_INT64);
      |            ^~~~~~~~~~~~~~
      |            GxB_LAND_INT64
build/temp.linux-x86_64-3.7/_pygraphblas.c: In function ‘_cffi_var_GrB_BAND_INT8’:
build/temp.linux-x86_64-3.7/_pygraphblas.c:38360:12: error: ‘GrB_BAND_INT8’ undeclared (first use in this function); did you mean ‘GxB_LAND_INT8’?
38360 |   return &(GrB_BAND_INT8);
      |            ^~~~~~~~~~~~~
      |            GxB_LAND_INT8
build/temp.linux-x86_64-3.7/_pygraphblas.c: In function ‘_cffi_var_GrB_BAND_UINT16’:
build/temp.linux-x86_64-3.7/_pygraphblas.c:38365:12: error: ‘GrB_BAND_UINT16’ undeclared (first use in this function); did you mean ‘GxB_LAND_UINT16’?
38365 |   return &(GrB_BAND_UINT16);
      |            ^~~~~~~~~~~~~~~
      |            GxB_LAND_UINT16

This happens when installing the setup.py. Can you please help me?
P.S. python 3.7.5, Ubuntu 18.04.3 LTS

Does it help if you change the SuiteSparse version to v3.3.0 in install-ubuntu.sh?
https://github.com/michelp/pygraphblas/blob/7465ef6fcc77c9901869b70ddf1d77a86570c336/install-ubuntu.sh#L11

I changed the version as you specified, now the installation completes without errors. But when I launched a simple script (consisting only of importing pygraphblas)
from pygraphblas import *

I got the following traceback:
Traceback (most recent call last):
  File "/home/simleton/Projects/graph-blas/main.py", line 1, in <module>
    from pygraphblas import *
  File "/home/simleton/.local/lib/python3.7/site-packages/pygraphblas-3.2.0-py3.7-linux-x86_64.egg/pygraphblas/__init__.py", line 1, in <module>
    from .base import *
  File "/home/simleton/.local/lib/python3.7/site-packages/pygraphblas-3.2.0-py3.7-linux-x86_64.egg/pygraphblas/base.py", line 1, in <module>
    from _pygraphblas import lib, ffi
ImportError: /home/simleton/.local/lib/python3.7/site-packages/pygraphblas-3.2.0-py3.7-linux-x86_64.egg/_pygraphblas.abi3.so: undefined symbol: GxB_TIMES_PAIR_INT32

I also tried running ldconfig (peeped from your Dockerfile), but it didn`t help. Can you tell me please what I can try to do?

I wasn't able to reproduce the issue.
It might be possible that the previous SuiteSparse install is interfering with the new one.
Can you please remove the .so files (sudo rm /usr/local/lib/libgraphblas.so*) and reinstall it (sudo make install && sudo ldconfig)?

I tried to do this, but it did not help. But when I tried to install requirements and pygraphblas with sudo, and run python3 also with sudo, no errors occurred. Installation with root privileges creating files in the /usr/local/lib/python3.7/dist-packages/pygraphblas-3.2.0-py3.7-linux-x86_64.egg folder, without root privileges - in the /home/simleton/.local/lib/python3.7/site-packages/pygraphblas-3.2.0-py3.7-linux-x86_64.egg. So maybe you know what the problem is with running as a regular user?

@marci543, thank you very much for your help!) My last question was a bit stupid, sorry. Removing previous versions of libraries really helped and everything is working correctly now. There were libgraphblas* files in another directories, also the libsuitesparse-dev was installed. So after removing this package and the rest of graphblas files all working properly (I also reinstalled pygraphblas). So I am very glad that it works and this issue can be closed.
P.S. pygraphblas API looks very cool


Maybe adjust default value of the -m argument for easier upgrade experience

Due to changes in commit cba8b22ad5, the compile time constant for maxRecords is changed to a runtime constant.  The compile time constant was 2500.  The default (and minimum) size with the argument is 1600.  When we upgraded to 9.0, we got the following message every minute or so in diags.log
[Sep 25 21:00:46.121] [ET_NET 16] WARNING: too many stats/configs, please increase max_records_entries using the --maxRecords command line option

We fixed this by adding the -m 2500 option to our systemd service file.
However, for a superior out of the box experience on upgrade, shouldn't the new default be the same as the old compile time constant?

@15ljindal  encountered the same issue. In our case, we have another script to call the trafficserver script to start ATS.
It seems that the current trafficserver script doesn't take any parameters from the caller, which I think we can make a change, so that we don't need to maintain a local patch for our customized ATS parameters.

Please close the issues associated with a PR.