Package conflict with CCL through conda

I have checked this on two different systems.
CCL and NaMaster conflict on the version of gsl:

CCL requires any newest version
NaMaster requires older versions

This means that once either of the two is installed through conda, the other one won't install because of this conflict.
Here is the output of  conda install -c conda-forge pymaster when pyccl is installed, and equivalently, the output of conda install -c conda-forge pyccl when pymaster is installed:
Package gsl conflicts for:
gsl
pyccl -> gsl[version='>=2.6,<2.7.0a0']
namaster -> gsl[version='>=2.4,<2.5.0a0|>=2.5,<2.6.0a0']

Why is the newest version of gsl that NaMaster can work with the alpha release of 2.6? Surely it could work with just 2.6?
@beckermr tagging you because you got infinite kudos for putting NaMaster on conda-forge :)

I am rebuilding the conda package. It was simply out of date. Try again in a bit!

Rebuild is merged. This issue can be closer. Should be ready in a few hours.

I attempted another install of NaMaster through conda and apparently it still tries to install gsl=2.5, instead of gsl=2.6+.
I did this twice, one time trying with just conda install -c conda-forge namaster, and another one (after I had looked up conda search namaster to see which is the latest version) with conda install -c conda-forge namaster=1.0. It is still conflicting with gsl, and here are the packages it is trying to install:
  chealpix           conda-forge/linux-64::chealpix-3.30.0-h14c3975_1000
  fftw               conda-forge/linux-64::fftw-3.3.8-nompi_h7f3a6c3_1109
  gsl                conda-forge/linux-64::gsl-2.5-h294904e_1
  libsharp           conda-forge/linux-64::libsharp-1.0.0-h14c3975_1000
  namaster           conda-forge/linux-64::namaster-1.0-py37heade8f8_1

Am I doing something wrong? Where is the fixed version located?

At the moment, I have found a workaround by building CCL and creating a symbolic link in my conda environment pointing to the pyccl directory in the CCL build, so it is not very urgent.

Conda forge globally uses a single version of gsl. It is currently 2.5. This ensures that all codes can use the same env. If the install works with 2.5, then there is no bug. It sounds like it does, right?

Ahhh wait. I see the issue. It is pyccl. Let me fix it

Alright. It turns out pyccl was using the wrong version of gsl for technical reasons related to conda. I think this is fixed now. Try again in a bit and sorry for the trouble!

Alright, thanks a lot, it w̶o̶r̶k̶s̶ (doesn't work) now. No package conflicts but CCL rc3, which is the most recent version of CCL on conda is broken. This is not an issue with NaMaster though, so I am closing this and opening a new issue on CCL's page.

What is broken with it?

Everything works great for me
(anl) localhost:~ Matt$ conda install pyccl
Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /Users/Matt/miniconda3/envs/anl

  added / updated specs:
    - pyccl

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    camb-1.0.6                 |   py37hb7221d7_3         878 KB  conda-forge
    gfortran_impl_osx-64-7.3.0 |       hf4212f2_2         9.3 MB  conda-forge
    gfortran_osx-64-7.3.0      |       h22b1bf0_6          21 KB  conda-forge
    gsl-2.5                    |       ha2d443c_1         2.7 MB  conda-forge
    isl-0.19                   |                0         1.3 MB  conda-forge
    pycamb-1.0.6               |           py37_3          33 KB  conda-forge
    pyccl-2.0.0rc3             |   py37h96385ee_2         642 KB  conda-forge
    ------------------------------------------------------------
                                           Total:        14.8 MB

The following NEW packages will be INSTALLED:

  camb               conda-forge/osx-64::camb-1.0.6-py37hb7221d7_3
  cctools            conda-forge/osx-64::cctools-921-h5ba7a2e_4
  clang              conda-forge/osx-64::clang-9.0.0-h28b9765_1
  fastcache          conda-forge/osx-64::fastcache-1.1.0-py37h01d97ff_0
  gfortran_impl_osx~ conda-forge/osx-64::gfortran_impl_osx-64-7.3.0-hf4212f2_2
  gfortran_osx-64    conda-forge/osx-64::gfortran_osx-64-7.3.0-h22b1bf0_6
  gmp                conda-forge/osx-64::gmp-6.1.2-h0a44026_1000
  gmpy2              conda-forge/osx-64::gmpy2-2.1.0b1-py37h1744cca_0
  gsl                conda-forge/osx-64::gsl-2.5-ha2d443c_1
  isl                conda-forge/osx-64::isl-0.19-0
  ld64               conda-forge/osx-64::ld64-409.12-h3c32e8a_4
  libllvm9           conda-forge/osx-64::libllvm9-9.0.0-h770b8ee_2
  mpc                conda-forge/osx-64::mpc-1.1.0-h3aece09_1006
  mpfr               conda-forge/osx-64::mpfr-4.0.2-h44b798e_0
  mpmath             conda-forge/noarch::mpmath-1.1.0-py_0
  pycamb             conda-forge/osx-64::pycamb-1.0.6-py37_3
  pyccl              conda-forge/osx-64::pyccl-2.0.0rc3-py37h96385ee_2
  sympy              conda-forge/osx-64::sympy-1.4-py37_0
  tapi               conda-forge/osx-64::tapi-1000.10.8-h770b8ee_3

Proceed ([y]/n)? y

Downloading and Extracting Packages
pycamb-1.0.6         | 33 KB     | ############################################################################################################################################################# | 100%
gfortran_impl_osx-64 | 9.3 MB    | ############################################################################################################################################################# | 100%
gsl-2.5              | 2.7 MB    | ############################################################################################################################################################# | 100%
isl-0.19             | 1.3 MB    | ############################################################################################################################################################# | 100%
pyccl-2.0.0rc3       | 642 KB    | ############################################################################################################################################################# | 100%
gfortran_osx-64-7.3. | 21 KB     | ############################################################################################################################################################# | 100%
camb-1.0.6           | 878 KB    | ############################################################################################################################################################# | 100%
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
INFO: deactivate-gfortran_osx-64.sh made the following environmental changes:
+LDFLAGS=-Wl,-pie -Wl,-headerpad_max_install_names -Wl,-dead_strip_dylibs -Wl,-rpath,/Users/Matt/miniconda3/lib -L/Users/Matt/miniconda3/lib
(anl) localhost:~ Matt$ ipy
Python 3.7.3 | packaged by conda-forge | (default, Jul  1 2019, 14:38:56)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.8.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import pyccl as ccl
   ...: import numpy as np
   ...:
   ...: # Create new Cosmology object with a given set of parameters. This keeps track
   ...: # of previously-computed cosmological functions
   ...: cosmo = ccl.Cosmology(
   ...:     Omega_c=0.27, Omega_b=0.045, h=0.67, sigma8=0.8, n_s=0.96,
   ...:     transfer_function='bbks')
   ...:
   ...: # Define a simple binned galaxy number density curve as a function of redshift
   ...: z_n = np.linspace(0., 1., 200)
   ...: n = np.ones(z_n.shape)
   ...:
   ...: # Create objects to represent tracers of the weak lensing signal with this
   ...: # number density (with has_intrinsic_alignment=False)
   ...: lens1 = ccl.WeakLensingTracer(cosmo, dndz=(z_n, n))
   ...: lens2 = ccl.WeakLensingTracer(cosmo, dndz=(z_n, n))
   ...:
   ...: # Calculate the angular cross-spectrum of the two tracers as a function of ell
   ...: ell = np.arange(2, 10)
   ...: cls = ccl.angular_cl(cosmo, lens1, lens2, ell)
   ...: print(cls)
[8.66780883e-09 1.17822402e-08 1.30401125e-08 1.35000276e-08
 1.35645508e-08 1.34173983e-08 1.31528650e-08 1.28233003e-08]

CAMB works too
In [2]: import pyccl as ccl
   ...: import numpy as np
   ...:
   ...: # Create new Cosmology object with a given set of parameters. This keeps track
   ...: # of previously-computed cosmological functions
   ...: cosmo = ccl.Cosmology(
   ...:     Omega_c=0.27, Omega_b=0.045, h=0.67, sigma8=0.8, n_s=0.96)
   ...:
   ...: # Define a simple binned galaxy number density curve as a function of redshift
   ...: z_n = np.linspace(0., 1., 200)
   ...: n = np.ones(z_n.shape)
   ...:
   ...: # Create objects to represent tracers of the weak lensing signal with this
   ...: # number density (with has_intrinsic_alignment=False)
   ...: lens1 = ccl.WeakLensingTracer(cosmo, dndz=(z_n, n))
   ...: lens2 = ccl.WeakLensingTracer(cosmo, dndz=(z_n, n))
   ...:
   ...: # Calculate the angular cross-spectrum of the two tracers as a function of ell
   ...: ell = np.arange(2, 10)
   ...: cls = ccl.angular_cl(cosmo, lens1, lens2, ell)
   ...: print(cls)
[9.31280743e-09 1.26656483e-08 1.40114330e-08 1.44869501e-08
 1.45274865e-08 1.43334458e-08 1.40087568e-08 1.36118779e-08]

I was about to open an issue on CCL's page, but since you're asking:
Try running ccl.halo_bias(cosmo, 1e14, 1), for example, and you get the following error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/koukoufilippasn/anaconda3/envs/spyder4b/lib/python3.7/site-packages/pyccl/massfunction.py", line 70, in halo_bias
    lib.halo_bias_vec, cosmo, halo_mass, a, overdensity)
  File "/home/koukoufilippasn/anaconda3/envs/spyder4b/lib/python3.7/site-packages/pyccl/pyutils.py", line 281, in _vectorize_fn4
    check(status, cosmo_in)
  File "/home/koukoufilippasn/anaconda3/envs/spyder4b/lib/python3.7/site-packages/pyccl/pyutils.py", line 29, in check
    raise CCLError("Error %s: %s" % (error_types[status], msg))
pyccl.errors.CCLError: Error CCL_ERROR_MF: ccl_massfunc.c: ccl_halo_b1(): No b(M) fitting function implemented for mass_function_method: 1

PS: Only CCL rc3 can be installed alongside NaMaster with conda, if someone tries to install an older version (currently the only other available is rc2), they still get package conflicts with gsl.

WeakLensingTracer works fine for me too.

Your mass function may not have a bias function. Which one are you using?

In [1]: import pyccl as ccl
   ...: import numpy as np
   ...:
   ...: # Create new Cosmology object with a given set of parameters. This keeps track
   ...: # of previously-computed cosmological functions
   ...: cosmo = ccl.Cosmology(
   ...:     Omega_c=0.27, Omega_b=0.045, h=0.67, sigma8=0.8, n_s=0.96)
   ...:
   ...: # Define a simple binned galaxy number density curve as a function of redshift
   ...: z_n = np.linspace(0., 1., 200)
   ...: n = np.ones(z_n.shape)
   ...:
   ...: # Create objects to represent tracers of the weak lensing signal with this
   ...: # number density (with has_intrinsic_alignment=False)
   ...: lens1 = ccl.WeakLensingTracer(cosmo, dndz=(z_n, n))
   ...: lens2 = ccl.WeakLensingTracer(cosmo, dndz=(z_n, n))
   ...:
   ...: # Calculate the angular cross-spectrum of the two tracers as a function of ell
   ...: ell = np.arange(2, 10)
   ...: cls = ccl.angular_cl(cosmo, lens1, lens2, ell)
   ...: print(cls)
[9.31280743e-09 1.26656483e-08 1.40114330e-08 1.44869501e-08
 1.45274865e-08 1.43334458e-08 1.40087568e-08 1.36118779e-08]

In [2]: ccl.halo_bias(cosmo, 1e14, 1)
Out[2]: 1.8196490114414752

Only tinker10 and shethtormen have bias functions.

Yes, if I remember correctly @damonge helped me to hard code the bias function before building CCL, and this is still work in progress. I use the 'tinker' mass function.

And I've seen that on CCL's page it has been suggested that mixed models should be allowed in case anyone wants to try new stuff. So don't worry about it for now. Thanks a lot for your help!

So the tinker mass function does not have a bias function
Use tinker10 and it will work.

Yes, yes, but mixed models is another issue which is currently on the table for CCL. (For example, one might want to define a 'tinker' cosmology object, but use the 'tinker10' bias function.
Thanks again Matt!

Sure sure understood. We are trying to tag v2 so I wanted to make sure nothjng was actually broken.


Joke denied

When I submitted a joke, it showed an alert saying that it has a foreseeable future.
The joke is, "Do programmers hate nature? Perhaps. It has too many bugs!"

That's funny! Maybe this alert is a joke by itself? :D

The submission of jokes is disabled until version 3.0 is released, sorry for the inconvenience


add mahotas in REQUIRED

Thresholding uses Otsu's method, specifically the version in mahotas:
https://mahotas.readthedocs.io/en/latest/thresholding.html

fixed by #11


Authentication using devise (Backend)

The user logs in to the website, only by typing the username

This task is assigned to Fernando


Gem is non-functional as-is

Multiple issues with this gem that make it non-functional, as-is:

Client::Transactions methods add_lines, create_or_adjust_transaction, create_transaction, delete_lines, and refund_transaction try to use post() incorrectly, it doesn't take both a model argument and options.
Incorrect encoding used with Faraday request.

Other issues:

Documentation implies that model should be an object, but request() code is written expecting a JSON string.  Request should call to_json before setting the body or documentation should be updated.

If anyone cares, my fork at https://github.com/tmilker/AvaTax-REST-V2-Ruby-SDK has a running set of commits to fix these issues and others.  It is starting to get a little opinionated(BigDecimals instead of floats, for example), so cherry pick what you like.

Hey, I just saw this issue - would you mind opening a PR and I'll get your changes merged in?

Actually @tmilker I went ahead and cherry-picked your commits and pushed a new version of the gem. Thanks!


Migrate layouts styles to emotion and remove tailwind

:page_facing_up: Description:

Migrate AuthLayout styles to emotion
Migrate MainLayout styles to emotion
Remove Tailwind and update all the files that had references to it

:popa:

Could you please remove postcss-cli from the dependencies?

Could you please remove postcss-cli from the dependencies?

Sure 😎


Cannot resolve size_t

Hello. I am using the clion bazel plugin successfully but it cannot resolve some symbols for the standard library like size_t!  Any ideas what I may need to tweak to get this working? I am using a custom cc toolchain with llvm (very similar to tensorflow build toolchain).
Specifically, my editor is saying: Unknown type name 'size_t'

+1

We have the same issue. The root cause for us is that in our tool chain paths such as isystem_directories and builtin_include_directories are relative to the execroot. The bazel plugin captures the information with its aspect, but when CLion tries to parse the code the working directory is not execroot, it is the repo root. Hence, it will not be able to find headers. This can be seen through strace:
Using strace to figure out what file accesses are done by CLion is revealing:
[pid 31102] stat("/home/dag/ext/werkstatt/external/clang_3_9_1_x86_64_linux/lib/clang/3.9.1/include", 0x7f79268a96b0) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/external/clang_3_9_1_x86_64_linux/lib/clang/3.9.1", 0x7f79268a9580) = -1 ENOENT (No such file or directory)
[pid 31102] stat("external/clang_3_9_1_x86_64_linux/lib/clang/3.9.1/include", 0x7f79268a96b0) = -1 ENOENT (No such file or directory)
[pid 31102] stat("external/clang_3_9_1_x86_64_linux/lib/clang/3.9.1", 0x7f79268a9580) = -1 ENOENT (No such file or directory)
[pid 31102] stat("external/clang_3_9_1_x86_64_linux/lib/include", 0x7f79268a96b0) = -1 ENOENT (No such file or directory)
[pid 31102] stat("external/clang_3_9_1_x86_64_linux/lib", 0x7f79268a9580) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/.clang-tidy", {st_mode=S_IFREG|0664, st_size=4889, ...}) = 0
[pid 31102] open("/home/dag/ext/werkstatt/.clang-tidy", O_RDONLY|O_CLOEXEC) = 6
[pid 31102] stat("/home/dag/ext/werkstatt/platform/fusion/src/core/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/platform/fusion/src/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/platform/fusion/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/platform/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/.clang-tidy", {st_mode=S_IFREG|0664, st_size=4889, ...}) = 0
[pid 31102] open("/home/dag/ext/werkstatt/.clang-tidy", O_RDONLY|O_CLOEXEC) = 6
[pid 31102] stat("/home/dag/ext/werkstatt/platform/fusion/src/core/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/platform/fusion/src/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/platform/fusion/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31102] stat("/home/dag/ext/werkstatt/platform/.clang-tidy", 0x7f79268a9a50) = -1 ENOENT (No such file or directory)
[pid 31042] lstat("/tmp/clion-clang-tidy", {st_mode=S_IFDIR|0775, st_size=4096, ...}) = 0
[pid 31042] open("/tmp/clion-clang-tidy", O_RDONLY) = 12
[pid 31042] lstat("/tmp/clion-clang-tidy", {st_mode=S_IFDIR|0775, st_size=4096, ...}) = 0
[pid 31042] rmdir("/tmp/clion-clang-tidy") = 0
It is looking for headers in
/home/dag/ext/werkstatt/external/clang_3_9_1_x86_64_linux/lib/clang/3.9.1
and
external/clang_3_9_1_x86_64_linux/lib/clang/3.9.1
(etc)
These  are not the valid locations, because the files are really in the execroot not in the repo.

Hi there! We're doing a clean up of old issues and will be closing this one. Please reopen if you’d like to discuss anything further. We’ll respond as soon as we have the bandwidth/resources to do so.


[WIP] Namespaces

This is almost complete. My head really hurts right now. If you want to fix the last few things (just run the tests and see), that'd be awesome.
There's about 3 tests still failing.
This fixes #51.

Another thing: maybe you can find an easy way to automatically add use statements for everything. Maybe there is an IDE that supports doing this, and doing it in batch mode.

Wow, thanks!
I'll try to fix the last issues later today, but this is already a great start!

I've just fixed the last few issues, now all unittests pass.


Feature Request: styling for 1st-level items

Feature request
Hi @minop1205, is it possible to add the ability to style the top-level list items?
For realisation like this:

Changing the style only for top-level items is possible by using the classes property or the depth option of render.
The classes property allows you to define the class name of the root ul, so for example, you can style only the items in the top-level hierarchy with the following selector
.rootList > li > div {
  color: #f00;
}
``` .

Click here for a sample.
https://codesandbox.io/s/full-features-issue81-1-e7ysh

Also, since option is passed to the render callback function, you can use the `depth` parameter in it to know the depth of the node. So it is possible to assign a dedicated className for depth === 0 in a custom node.

Here is a sample
https://codesandbox.io/s/full-features-issue81-2-ibjjp

thanks for the detailed answer, did not think about it


Fixing Font sizes

hey @Milind-Palaria buddy you can't create a new issue for this, you have  got the issue to fix the entire Issue for UI and includes responsiveness and and font size, if i accept the PR just to increase the number, repo will be excluded from hacktober


How to disable Scrollify on mobile device

Hello
I am using your wonderful Scrollify.js
But I need to disable it on mobile device.
I am newbie with javascript and php.
Need help with complete code.
Thank you
My code:
<script>
			  $(function() {
			      $.scrollify({
			          section : ".pagescroll",
			          standardScrollElements: ".modal",
			      });
	 		});
    </script>

You can add this snippet, and scrollify snap function will be disabled on all devices that is below the specified screen width.
if ($(window).width() <= 1024) {
$.scrollify.disable();
}

When I use it this way
if ($(window).width() <= 1024) {
      $.scrollify.disable();
}
$(function() {
			      $.scrollify({
			          section : ".pagescroll",
			          standardScrollElements: ".modal",
			      });
	 		});

on mobile it is stop scrolling whole page.
That function is dissabling scroll not scrollify.js
Thank you

$.scrollify.disable(); disables scrollify only. If scroll is disabled then you might have a CSS issue causing that.

thank you
now it is working.
I had some problems with CSS.


[infra-proxy-service] node list HTTP & gPRC API

:nut_and_bolt: Description: What code changed, and why?

Add the ndoe list API with attributes name, fqdn, IP address, environment, uptime and check-in time etc.
Minor updates in affected node API to pass URL instead of name and version.

:chains: Related Resources
Fixes: #4418
:+1: Definition of Done

Infra service responding with node list API.

:athletic_shoe: How to Build and Test the Change
Steps to build:

rebuild components/infra-proxy-service
rebuild components/automate-gateway

Steps to test:

Add the chef-server test env direction given at step 3 https://github.com/chef/automate/blob/master/dev-docs/adding-data/adding_test_data.md#adding-data-to-infra-views

Hit the node list HTTP API

curl -sSkH "api-token: $(get_admin_token)" https://a2-dev.test/api/v0/infra/servers/chef-server-dev-test/orgs/chef-org-dev/nodes?pretty

{
  "nodes": [
    {
      "id": "c745433f-9517-4c9a-8525-65360bde854e",
      "name": "node-learn_chef_apache2",
      "fqdn": "vagrant.vm",
      "ip_address": "<IP_ADDRESS>",
      "check_in": "1.5901204929347618E+09",
      "uptime": "50 minutes 09 seconds",
      "platform": "ubuntu",
      "environment": "_default",
      "policy_group": ""
    }
  ]
}

:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable
Signed-off-by: Vivek Singh<EMAIL_ADDRESS>

@stevendanna pagination stuff makes sense to me but it seems go-chef lib itself does not support the pagination first, we required to support pagination in go-chef lib then in further PR we can add it to here.
Ref: https://github.com/go-chef/chef/blob/c278260aa167af1515bb3e2c282c2f02a2597543/search.go#L129-L138


Use with AWS EKS clusters

Great tool!
It would be great to have this support AWS's EKS.
Currently, it fails on connections issues.

Same for Azure


Design System > Elements > Enhancements in Video Player

Design System > Elements > Enhancements in Video Player
https://www.figma.com/design/vziFLZAgMFi8wA5SlikLh5/Raaghu-Design-System?node-id=1602-32577&node-type=frame&t=ncBHc3XyPjDjE2We-0
Docs	1]  Video Player development different than design
Default	2] Three video players needs to be implemented, Default,  vimeo, Youtube

Fixed on staging


outdated version.cmake content

version.cmake inside release tags contains outdated version values.
repro:
trying to build v<IP_ADDRESS> under gentoo (https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-db/clickhouse/clickhouse-<IP_ADDRESS>.ebuild) will produce app image with <IP_ADDRESS> version string, cause ebuild pulls https://github.com/yandex/ClickHouse/archive/v<IP_ADDRESS>-stable.zip which contains outdated version: https://github.com/yandex/ClickHouse/blob/v<IP_ADDRESS>-stable/dbms/cmake/version.cmake

Fourth number in version doesn't mean anything in releases. It's uniquely determined by third number. But I'll try to fix this in our CI.

I am not gentoo ebuild maintainer. Stable release creation is not gentoo specific issue. I think any v*-stable.zip archive should contain appropriate version string, so version.cmake commit like version tagging is just another preparation before release (there is no uglification).

Fixed for stable and prestable releases since v<IP_ADDRESS>-stable.


feat(form-group): add size prop and theme options

what

Added a size property to the FormGroup component

why
To better fit with larger or smaller inputs!

Note
Force-pushed a commit to remove pnpm-lock.yaml from it

thanks for catching those @Haythamasalama!

Would it make sense to also pass the size prop down to the form element like the AvatarGroup and ButtonGroup?

Would it make sense to also pass the size prop down to the form element like the AvatarGroup and ButtonGroup?

I realized that the FormGroup associated with one input doesn't seem to be important to pass. However, I don't think it's necessary to have a different size for the FormGroup and the input inside it every time. It would be easier for us to add a size parameter to the FormGroup instead of passing it to both every time.

Thanks again @Haythamasalama! Made the controls for the props hidden on the documentation and passed down the size property to the input component.

Thanks again @Haythamasalama! Made the controls for the props hidden on the documentation and passed down the size property to the input component.

 It looks great! ✨👌

I've made a few adjustments, let me know if this works for you!

looks good!


is not a main package

when i try to install the package it gave me these errors:
go install github.com/gopxl/pixel/@latest
go: downloading github.com/gopxl/pixel v1.0.0
go: github.com/gopxl/pixel/@latest: argument must be a clean package path
go install github.com/gopxl/pixel/v2@latest
go: downloading github.com/gopxl/pixel/v2 v2.2.0
package github.com/gopxl/pixel/v2 is not a main package
go get github.com/gopxl/pixel/v2
go: go.mod file not found in current directory or any parent directory.
'go get' is no longer supported outside a module.
To build and install a command, use 'go install' with a version,
like 'go install example.com/cmd@latest'
For more information, see https://golang.org/doc/go-get-install-deprecation
or run 'go help get' or 'go help install'.

This is expected, pixel is not meant to be an executable, but a library that you build your own executables from.
Check out the wiki for how to use the library

Ight thanks


Count a view generates exception

When I'm trying to do a count() in a view, the exception java.lang.IllegalStateException is thrown, showing the message "No count expression can be created".
This exception was threw by com.querydsl.sql.RelationalPathBase class, method count().
I think the reason is that during the export of the view, a primary key isn't generated.
I'm using querydsl v4.0.9 and PostgreSQL v9.5.2.

To resolve, was needed to put by hand the primary key in the generated code of the view.

A think a view doesn't define any primary key, but you can alternatively also count via view.pkColumn.count().
What kind of fix would you suggest?


Revert "fix: 🐛 Removed the Manual Github pages Build & deploy action"

Reverts mansiruhil13/Bobble-AI#143

hi @mansiruhil13 why u revert it ?

hi @mansiruhil13 why u revert it ?

to make changes in hackoctober fest

okh can you explain to me what reverting the PR does ?

hi @mansiruhil13 why u revert it ?

to make changes in hackoctober fest site and pull request successfully merged and closed
@mansiruhil13
does merging then revert PR changes discards our code or accepts our code into the repo

okh can you explain to me what reverting the PR does ? and how does hacktoberfest is affected ?

Reverting a PR on GitHub means undoing the changes that the pull request introduced. It brings the code back to how it was before the PR was merged, without deleting the original PR. Instead, a new commit is created to reverse the effects of the merged changes. This is useful if something went wrong after merging the PR or if you need to roll back the changes for any reason.

hi @mansiruhil13 why u revert it ?

to make changes in hackoctober fest site and pull request successfully merged and closed

@mansiruhil13 does merging then revert PR changes discards our code or accepts our code into the repo

so what happens to my code in your project, it's not merged then ??

hi @mansiruhil13 why u revert it ?

to make changes in hackoctober fest site and pull request successfully merged and closed

@mansiruhil13 does merging then revert PR changes discards our code or accepts our code into the repo

so what happens to my code in your project, it's not merged then ??

actually i have not much idea

hi @mansiruhil13 why u revert it ?

to make changes in hackoctober fest site and pull request successfully merged and closed

@mansiruhil13 does merging then revert PR changes discards our code or accepts our code into the repo

so what happens to my code in your project, it's not merged then ??

okh i don't understand this but still if you think it is okh then I have no problems.

but is it not problem for you to have many failed workflow actions, piling up. It is not good right ?
@mansiruhil13

but is it not problem for you to have many failed workflow actions in the repo actions tab, piling up. It is not good right ?
@mansiruhil13

i don't know about this


Connect to API endpoints instead of directly to Elasticsearch

I'm not sure if I'm missing something, but it seems vue-innersearch can only be used if an Elasticsearch server is available to the public. Suffice it to say that there are obvious, prohibitive drawbacks to doing this in production:
Note:
Is it safe to expose the Elasticsearch Search API directly through your application's API?
https://stackoverflow.com/questions/39103547/is-it-safe-to-expose-the-elasticsearch-search-api-directly-through-your-applicat
Can InnerSearch can be configured otherwise? If so where can we find an example?

vue-innersearch can only be used with the ElasticSearch API. If you want to secure your ElasticSearch API there are few options :

to use a nginx server as a proxy server for example ( using Authoriation headers )
use X-Pack security activities for Elastic ( check doc )


Added 'previous command' command: "/!" (Issue #159)

Command: !
Description: Runs the last command executed by you.

Hey, looks mostly good. Ignore the conflict, the binaries can be a pain - I think I broke it on you actually. GitHub won't let me pull it in.
But anyways, only 1 issue. Seems you also need to remove the last command when the player leaves the server. After that we can pull er' in :)

hahha yep my bad, thanks. ill fix it up now :smile:


chore: add next 14 revalidate example [TOL-1602]

Added an example for live preview using next 14

[x] instead of reloading the path, revalidate the cache for the path and update that way

Could you give is a quick intro on how it works in the PH? :)


Jupyter Notebook creates a .txt file - and doesn't let me save as .ipynb. Manually creating a .ipynb file doens't open a jupyter notebook.

Environment data

VS Code version: 1.59.0
Extension version (available under the Extensions sidebar): Python v2021.8.1105858891, Jupyter v2021.8.1195043623
OS and version: Windows 10 64bit
Python version (& distribution if applicable, e.g. Anaconda): Python 3.9.6 64bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv

Expected behaviour
Using command palette  'Jupyter: Create blank new Jupyter notebook' it should open a .ipynb file with all the features of a jupyter notebook.
(source: https://code.visualstudio.com/docs/datascience/jupyter-notebooks)

Actual behaviour
Using command palette 'Jupyter: Create new blank notebook' it opens a jupyter notebook with all the features and the proper jupyter notebook layout, but with extension .txt.

I tried to save it as .ipynb and returns this error and don't let me save it:

When I create a file manually with 'filename.ipynb' it creates a file with the jupyter icon, but without all the features of a jupyter notebook.. it is a simple plain script file like a .py file.

I already have done a lot of web search looking for some solution, but without success.
already tried reinstall vscode and downgrade some versions of the python and jupyter extensions, but all without success.
Any help to solve this would be appreciated.

I have the same issue, furthermore it refuses to open current .ipynb files- well it opens them but in txt format. Uninstalled everything vscode. Re-installed vscode and extensions and the issue persists.


fix(close): ensure in-flight messages are drained

Track in-flight requests and add an onDrain deferred that resolves when all in-flight ack/modAcks are done.
Fixes #951

@googlebot I signed it!

Thanks for the contribution, @jeffijoe ! I'll review in just a sec.

If you are wondering about the force-pushes, I am just rebasing to latest master.

Yep, no worries on the rebase!

I think the rebase is actually confusing our CI bots, so you might want to just use merge to get the tests running automatically.

Kokoro seemed to have issues even with my first non-rebased commit though? I can push a dummy commit if you want and see if it resolves anything?

Sure, let's try it for science :)
It looks like the checks are passing at this point. Hopefully we'll have our automated sample failure bot going on this repo soon.

Apparently the actual issue with the CI here is that Kokoro is having issues. I'll try to get this merged tomorrow.


Small barcode detection issue

Hello,
We are trying to build a Xamarin mobile android application that will detect the datamatrix (2D)  barcodes.  We have the item size and length of barcodes is 12mm to 12 mm. it seems barcodes longer than 3 cm are detected very good. But if they become smaller than 2 cm it's pretty difficult to detect them (even at TryHarder mode) because they are probably too small for 6 inches distance (from camera to the target) to be detected and when you put the camera closer it can't (auto)focus at the target.
Is there a way to configure the library so it can detect smaller barcodes at the same recommended 6 inches distance or maybe focus at smaller distance (12mm X 12mm)? Maybe some kind of Macro mode?
Thank you.

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Clone Button Rearrangement

Feature Description
Currently, the Clone and Download Buttons use a lot of horizontal space and only the download/clone in editor buttons are "hidden" in a Dropdown-Menu. Being used to the GitHub UI, this UI feels a bit messy and unlogical. Why does the full clone section have to be visible all the time? Why are the download buttons hidden behind "..."?
Therefore, I'd suggest rearranging the clone/download section to look similar like on GitHub, see attached Screenshots.
This would have the following benefits:

Unless needed, the Clone/Download UI is out of the way, creating more blank space.
Inside the dropdown menu, there is more space for the Clone/Download UI, so buttons and links can be made larger

Also, inside the dropdown, a second tab can be added (similar to how GitHub does it, see screenshot) to simplify the UI even further, especially with more items added, like PR #22378

Screenshots
Current Implementation:

Suggested Implementation (Mockup, details may change):

Mockup for a two-tab layout:

One thing I do not like about this is that it takes two clicks to copy a SSH URL in GitHub UI, while this can be done in one click in the Gitea UI.
Also, I'm not sure we are really starved for space like GitHub is with their very condensed layout. We do have more space available to show the URL.


Setting static variable in a constructor

Why i can't set static variable in a model constructor ?
My Post model has a public static $truncateText = false; and i want to set this variable in constructor like this
   `$model = new $this->modelClass([
        'scenario' => $this->scenario,
        'truncateText' => true,
    ]);

    return new ActiveDataProvider([
        'query' => $model::find()
            ->status()
            ->car($id)
            ->user(Yii::$app->user->identity->getId())
            ->hasComments()
            ->hasLikes(),
    ]);`

But i get the Exception
Setting unknown property: frontend\modules\v1\models\post\Logbook::truncateText
So, i trying to set it manually
    `$model = new $this->modelClass([
        'scenario' => $this->scenario,
    ]);
    $model::$truncateText = true;

    return new ActiveDataProvider([
        'query' => $model::find()
            ->status()
            ->car($id)
            ->user(Yii::$app->user->identity->getId())
            ->hasComments()
            ->hasLikes(),
    ]);`

And it works, of course, but in my opinion the installation of static properties in the constructor is much cleaner.
This is a small example to how to set the static property using ReflectionClass
public function configure($config = []) {         $class = new \ReflectionClass(static::class);         foreach ($config as $name => $value) {             $class->setStaticPropertyValue($name, $value);         }     }

Currently you must create your own setter in order to set static class property using Yii::createObject.
I guess is by design. Let's wait for other opinios.

Yes. It's by design.

@samdark OK! My solution that helped me. Maybe it could be useful to someone
`/**

Class StaticConstructor
@package frontend\modules\v1\components
*/
trait StaticConstructor
{
public function __set($name, $value)
{
if (array_key_exists($name, get_class_vars(static::class))) {
$reflectionProperty = new \ReflectionProperty(static::class, $name);
if ($reflectionProperty->isStatic()) {
$reflection = new \ReflectionClass(static::class);
$reflection->setStaticPropertyValue($name, $value);
} else {
parent::__set($name, $value);
}
} else {
parent::__set($name, $value);
}
}
}`

` /**

@static @method Post __set()
*/
class Post extends ActiveRecord
{
use PostTrait;
use StaticConstructor;`

/** @var Post $model */         $model = new $this->modelClass([             'scenario' => $this->scenario,             'truncateText' => true, // static property that already defined in class             'truncateTextCount' => 300, // static property that already defined in class         ]);

My solution that helped me. Maybe it could be useful to someone

Most obvious solution if you want to set static properties via Yii::createObject() is definition of the non-static setter for it:
class Post extends ActiveRecord
{
    public static $truncateText = false;

    public fucntion setTruncateText($value)
    {
        $this->truncateText = $value;
    }

Although the issue overall does not make any sense.


loadTexture with spritesheet with bitmapData

A bug in the API
A problem with the documentation?
A problem with my own code?

Since 2.4.7 BitmapData.loadTexture used for Sprite with spritesheet does not update the frams but changes the Sprite to a single frame. Works in 2.4.6 and before except for 2.4.5
I did add frameData but didn't get that to work so perhaps this is more a question of understanding how to add framedata properly that was managed before under the hood.
First the sprite is loaded.
     game.load.spritesheet("bird", "assets/animals/bird.png?v=1", 128, 128, 4);

Later during runtime when an object is created
   Phaser.Sprite.call(this, game, posX, posY, cardInfo.animal,1);

    var shaped = game.make.bitmapData(256, 256);
    var shapeName = cardInfo.shape.toString();
    shaped.alphaMask(cardInfo.animal.toString(), shapeName);

    this.loadTexture(shaped);

Took some time to get my head around but I just have to figure out how to add and match frames to the bitmap.


2D Picking

Fixes #2447. Remove globe depth rendering and fix pick culling volume in 2D.

Pick in 2D when rotated is failing, but other than that this fixes the issue.

Code is OK with me.  @mramato can merge when happy.

@mramato OK, this is ready.


Reduce default verbosity

Building from cea5ae8:
$ ta done 22
modified task 05550979-63a6-498d-87b4-2bb89404bebd
modified task 9034ba03-6355-468e-b3f0-9dd2d3d87c3c
modified task 9590b11f-4cf9-4f77-879d-f5879c05c76c
modified task 91678ad0-929d-4aff-97be-620758af6758
modified task 21956370-2c0d-4249-99c8-6104b6d839c3
modified task a57b53be-4f82-42e9-aceb-6e3a8e7ca3b8
modified task 87df9f11-0364-4e2b-b6e0-6b448529271b
...

Perhaps let's not print the whole chain of modified states? Some users might have 100+ tasks.

In that example, I think it has actually modified every task that is listed - ta done 22 currently updates tasks which match empty filter, and replaces the description with 22 - which is issue #177
ta 22 done would only output a single "modified task" - ideally it would say something more useful (#233)

Yep, I think this is handled in those two issues (one of which has been waiting for review for a few weeks)

one of which has been waiting for review for a few weeks

..oops 🏃


Google Analytics - IP Excluded provides the same numbers from May-Dec 2019 as the one without IP Excluded

Does the Google Analytics - IP Excluded actually exclude IP from our office location? The numbers from May-Dec 2019 are the same from Google Analytics-IP Excluded and as those from Google Analytics-Arrival Advisor.
Example 1: The number of users from May-Dec 2019 are exactly the same (highlighted in red)from both the "IP Excluded" and the regular "Arrival Advisor" analytics:

Example 2: Sessions by Device and Top Screens Views from the last 90 days also show the same numbers from both the "IP Excluded" (screen shot 2a) and the regular "Arrival Advisor" (screen shot 2b) analytics:
Screen shot 2a:

Screen shot 2b:

Acceptance Criteria

[ ]

Steps for QA:

I created a no filter view on the app now, it should show whether if a filter is wrongly applied or if none of the filters are working.

Upon checking the results with the No filter view and compared to the existing IP Excluded and ArrivalAdvisor view it was apparent that the stats on all three views are the same. In essence, none of the filters are working correctly.
The reason could be best described with this Google support page here. As far as I know we don't collect the IP addresses and send them to Google as part of every google analytics event, this might explain why the filters don't work.
We might have to enact a policy of enabling the easter egg on staff phone, and during testing to reduce the noise. We might also have to move easter egg to the home page to cut down on the number of requests to enable easter egg on the search page.

The fix is to use the easter egg.


3d Skin Layer Support

3d Skin Layer makes your hand just not work correctly

yeah its buggy

See https://github.com/IthanMendoza/Fresh-Moves/wiki/Fix-Moving-1st-Person-Hand


Eject Mass

is there a way you can make the same eject mass collision as gota?
:c

i want my server as gota as possible

not possible stop making useless issue pls close @AlexHGaming

@Exobyte918 Lol I'm not a collaborator.
You can ask @MastaCoder. :)

And for @freaky767, this is out of scope of the project in which we need to make vanilla as close as possible. If you were to make a gota server, ask somebody to create a gota.io private server and can support the protocol.

ok


Surface geometry "resolution" is sometimes wrong

From pds-webserver created by rfrenchseti: SETI/pds-webserver#13
From pds-opus created by rfrenchseti : SETI/pds-opus#455
On 8/30/2018 5:04 PM, Rob French wrote:

I've been playing around with surface geometry searches, and I'm finding things that I don't understand.
For example, take COISS_2076-1727054847_1727147261-N1727119421_1, which is a distant shot of Mimas approximately 80x80 pixels in a 1024x1024 image:
Intended Target Name:
Mimas
Body Center Resolution:
5.12952
Body Center Distance:
859864.88
Finest Observed Resolution (Min):
5.1293
Finest Observed Resolution (Max):
5.12937
Coarsest Observed Resolution (Min):
29.95767
Coarsest Observed Resolution (Max):
40.56602
This resolution is believable - at 400 km diameter, Mimas should be about 80 pixels.
Now take COISS_2060-1644743986_1644781734-N1644777828_1, which is an extreme closeup of Mimas where Herschel Crater fills more than the 1024x1024 image size.
Intended Target Name:
Mimas
Body Center Resolution:
1.55525
Body Center Distance:
16294.177
Finest Observed Resolution (Min):
1.53693
Finest Observed Resolution (Max):
1.5439
Coarsest Observed Resolution (Min):
1.56995
Coarsest Observed Resolution (Max):
2.45734
How can image #1 be 859864/16294 = 53X farther from Cassini, and yet the body center resolution is only 5.129/1.555 = 3.3X different? In fact I don't believe the body center resolution at all. I think in image #2 Mimas is at least 4000 pixels wide, so the resolution should be at least 4000/80 = 50X different (which agrees with the distance ratio). Yet this resolution would have me believe Mimas would fit in less than 400 pixels, much smaller than the FOV.

Now then, here are the metadata links for the two examples I sent:
https://pds-rings.seti.org/viewmaster/metadata/COISS_2xxx/COISS_2076/COISS_2076_moon_summary.tab/N1727119421_1
CENTER_RESOLUTION                    = 5.12952
CENTER_DISTANCE                      = 859864.88
https://pds-rings.seti.org/viewmaster/metadata/COISS_2xxx/COISS_2060/COISS_2060_moon_summary.tab/N1644777828_1
CENTER_RESOLUTION                    = 1.55525
CENTER_DISTANCE                      = 16294.177
The data here agree with what I saw in OPUS, showing that the import pipeline is correct.
I went back in time using the last CDAPS navigation I did on 1/12/18, where I keep all the metadata used to generate the navigation. The metadata for N1727119421 roughly agrees with what we're seeing now, to within a few percent. But the metadata for N1644777828_1 is completely different! In fact the old version says the resolution is 0.0976, not 1.555. This is roughly 1/53 the resolution of the first image, which is what I would expect based on the distance.
Conclusion: The current surface geometry files are massively wrong in some cases, and whatever is going wrong happened since Jan 12.

This has been fixed for COISS but still needs to be done for other instruments.

From Matt:
We can test this keyword against the PIXEL_SCALE keyword, which was calculated by the Imaging Team and delivered to the Imaging Node.
I first tried to click over to the image’s delivered label from the metadata page to which Rob linked.  This does not have a direct link that I could find (perhaps it should), but I clicked down the tree a few levels and then up again to “data” and found it:
https://pds-rings.seti.org/holdings/calibrated/COISS_2xxx/COISS_2060/data/1644743986_1644781734/N1644777828_1_CALIB.LBL
Unfortunately, the label appears to include only the keywords that were created at the project level, and not those created by the Imaging Team.
I then decided to find the image in the Imaging Node’s atlas.  Even though I already knew by name the image I wanted, this took me at least 20min and is strongly tempting me to sympathize with the critics of PDS.  Part of the problem is that Imaging Node does not appear to offer the Imaging Team keywords on their own, but only shows them as part of the search tool.  So I had to construct a search that would include the image we want, which I finally succeeded in doing:
https://pds-imaging.jpl.nasa.gov/search/?fq=ATLAS_INSTRUMENT_NAME%3A"iss+nac"&fq=START_TIME%3A[2010-02-13T00%3A00%3A00Z+TO+2010-02-13T23%3A59%3A59Z]&q=*%3A*&start=72
Apparently, added columns do not come through in this URL, so from this link you must follow these steps:

click on “Sort View”
click on the pull-down menu for “Add field to sort by” and select PIXEL_SCALE.
click on the down-arrow under “Start Time” so that it will go back to sorting by descending start time.  We are still on page 4, since that is included in the URL (and we have to go by pages because the time constraints do not allow resolution finer than one day!).
Finally, text search this page for “1644777828”, and you’ll find the image we’re discussing.  You can now see that the PIXEL_SCALE keyword is calculated at 0.0964 km/px.

Conclusion:  Rob is correct to say that he has found an error in our calculated metadata.
Best, -Matt

See Sec. <IP_ADDRESS> in https://docs.google.com/document/d/1JVEhksHlru2jk8ZW6WPyZeGxReG0ccB-3GEQ2vyX4vg/edit


NaN propagation

From IEEE754-2008, 6.2.3:
"If two or more inputs are NaN, then the payload of the resulting NaN should be identical to the payload of one of the input NaNs if representable in the destination format. This standard does not specify which of the input NaNs will provide the payload."
The convention seems to be to take the first operand. For example:
union uval {
   float fval;
   int32_t ival;
};

volatile union uval a, b, c;

int main()
{
    a.ival = 0xff8abcde;
    b.ival = 0xff812345;
    c.fval = a.fval + b.fval;
    printf("%08x\n", c.ival);
}

On a desktop machine, this returns 0xffcabcde (the most significant bit of the significand is set). On Nyuzi, it returns the hardcoded NaN value 0x7fffffff.

The RISC-V spec (v2.1, section 7.3) makes a compelling case for not supporting this:
"Except when otherwise stated, if the result of a floating-point operation is NaN, it is the canonical NaN. The canonical NaN has a positive sign and all significand bits clear except the MSB, a.k.a. the quiet bit...We considered propagating NaN payloads, as is recommended by the standard, but this decision would have increased hardware cost. Moreover, since this feature is optional in the standard, it cannot be used in portable code."


[nginx] Don't log 404s to the error log

In the Chef API, 404s are typically not errors. Rather they are often
the expected response about an object that doesn't exist.
The log_not_found directive stops nginx from logging 404s as errors;
however, they should still be logged to the request log
(i.e. access.log).
References:

http://nginx.org/en/docs/http/ngx_http_core_module.html#log_not_found

Signed-off-by: Steven Danna<EMAIL_ADDRESS>

Note: This is untested.  Opening this to continue the discussion around 404 logging in the Chef Server more centrally.

This looks pretty reasonable. Just kicked off a build in CI:
https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/125#6c27fc3f-20ea-4353-a57b-cc07c378b5bb


Language Client Services Overriding Problem.

I'm having the same issue as this: https://github.com/TypeFox/monaco-languageclient/issues/225. And unable to find a solution for this problem.
Doesn't seem to be a way to dispose previously installed Client Language Services triggering the 'error: Language Client services has been overridden' after installing via MonacoServices.install(editor)
This issue: https://github.com/TypeFox/monaco-languageclient/pull/156/files#diff-544ff078317aa067b00c24927238a8131ce5f71ba00704ec4b927e40b6935d32R55
is only referring to disposing within Services. This is an issue because if you install it through MonacoServices and not Services, you won't have access to the disposable function.
Question is, is there any potential problems of overriding the Language Client Services and also is there a way or workaround to suppress the Error?

What about:
const services = MonacoServices.create(monaco);
const disposable = Services.install(services);

?

unfortunately doesn't work. When it attempts to dispose via disposable.dispose() the Error:
Error: Language Client services has not been installed appears.
I attempted to install both Services and MonacoServices as well but get the Error the language services has been overridden. I suppose the error isn't really affecting the expected functionality as far as I can tell, just an ugly error message in console.


feat(cheatcodes): add --fork-retry + --fork-retry-backoff equivalent to vm.create*Fork

Component
Forge
Describe the feature you would like
Most of our integration tests are multichain, meaning we use vm.createFork to set up the forks. We can't use the --fork-url option because we want to ensure that the fork IDs have the same order, regardless of which chain the tests belong to.
For example, we always instantiate the forks in the following order:

ID 0: Moonbeam fork
ID 1: Base fork
ID 2: Optimism fork

When running a Base integration test, the setUp function first fetches the Moonbeam governor and executes any live proposals (that can be cross-chain).
This is why we cannot use --fork-url with forge test.
Please allow the --fork-retry and --fork-retry-backoff flags to be used without requiring the --fork-url.
Additional context
No response

May I take this issue on?

Can I work on this, please?

Can I work on this, please?

I appreciate your enthusiasm but please do not claim tickets you are not intending to work on
You are already assigned to https://github.com/foundry-rs/foundry/issues/7248#issuecomment-2454222443 and others

@zerosnacks  can i work on this? The #9201 is open, but i believe most of the works  is done

Thanks Kien, all good - assigned to you

@zerosnacks sorry for taking this long, i  forget that i'm assigned to this issue , i've been busy this month, never actually got the time to do this issue
i've just un-assign myself for others to take the issue

@zerosnacks hi! i'm interested in working on this :)

Thanks @anukul! Assigning to you - let me know if you have any questions

There are three versions of vm.createFork (similar for vm.createSelectFork):
// Creates a new fork with the given endpoint and the _latest_ block and returns the identifier of the fork
function createFork(string calldata urlOrAlias) external returns (uint256)
// Creates a new fork with the given endpoint and block and returns the identifier of the fork
function createFork(string calldata urlOrAlias, uint256 block) external returns (uint256);
// Creates a new fork with the given endpoint and at the block the given transaction was mined in, and replays all transaction mined in the block before the transaction
function createFork(string calldata urlOrAlias, bytes32 transaction) external returns (uint256);

@zerosnacks thoughts on overloading it with a retryArgs parameter, for each existing version?
/// Retry arguments for RPC requests to forks
struct RetryArgs {
    /// Maximum number of retries
    uint256 max_retries;
    /// Initial retry backoff (in milliseconds) on encountering errors
    uint256 backoff;
}

Hi @anukul,
Looking at it again I think we possibly do not need to add these overrides as in foundry.toml one can add an [rpc_endpoints] section which includes the retries and backoff behavior as follows:
[rpc_endpoints]
mainnet = { endpoint = "${_CONFIG_MAINNET}", retries = 3, retry_backoff = 1000, compute_units_per_second = 1000, auth = "Bearer ${_CONFIG_AUTH}" }

One can then reference this by its alias (vm.createSelectFork("mainnet");) in the test and all the settings would be applied to the fork.
cc @anajuliabit would this resolve your issue? This is the preferred way to perform multichain tests over --fork-url
cc @grandizzy just to check, am I overlooking something?

@zerosnacks good point, yeah, they can be configured like this. Though from a quick test it seems like these configs are not propagated when building provider, should be pretty easy to do such

I see, this is a bit more complex - would you be interested in looking into this @anukul?
The general idea I think could be to expand the ResolvedRpcEndpoints, changing the endpoints into a BTreeMap<String, Result<RpcEndpointConfig, UnresolvedEnvVarError>> so it results in and then modify the evm_opts in create_fork_request with the settings.
https://github.com/foundry-rs/foundry/blob/59f354c179f4e7f6d7292acb3d068815c79286d1/crates/cheatcodes/src/config.rs#L32-L33
https://github.com/foundry-rs/foundry/blob/59f354c179f4e7f6d7292acb3d068815c79286d1/crates/config/src/endpoints.rs#L408-L415
https://github.com/foundry-rs/foundry/blob/59f354c179f4e7f6d7292acb3d068815c79286d1/crates/cheatcodes/src/config.rs#L188
https://github.com/foundry-rs/foundry/blob/59f354c179f4e7f6d7292acb3d068815c79286d1/crates/cheatcodes/src/evm/fork.rs#L329-L330
Related PR: https://github.com/foundry-rs/foundry/pull/8570 for the auth field, looks like it is not connected up either. This should be folded into the RpcEndpointConfig.

Sure, happy to do it!


git-lfs: configure repository and install git-lfs purely

Motivation for this change
When running nix run nixpkgs.git-lfs -c git-lfs install, the update git config and the generated hook do not refer to git-lfs in its full path resulting in GUI applications and CLI commands outside of nix-shell to fail.
Patch git-lfs to install the git-lfs hook and configuration using the absolute path to the git-lfs command. Doing so makes the git-lfs installation pure as it won't depend on an impure PATH.
Things done

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

[ ] NixOS
[ ] macOS
[ ] 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 nixpkgs-review --run "nixpkgs-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
[x] Fits CONTRIBUTING.md.

closes #85264

Result of nixpkgs-review pr 120270 at 13e3fb16 run on aarch64-linux 1

2 packages built successfully:

git-lfs
github-backup

We can't use a wrapper here?

but I don't really want to patch this in

Can you elaborate on this?

Can you elaborate on this?

I would like to avoid this kind of patch if possible but I don't know a better way right now to avoid it.

there's a home-manager module which install this correctly for you in your ~/.gitconfig. Not sure if we should be augmenting the base application in this fashion.

there's a home-manager module which install this correctly for you in your ~/.gitconfig. Not sure if we should be augmenting the base application in this fashion.

In this context, we use git-lfs at work and the repository set up git-lfs within a shell hook. In some situations, some developers find themselves dealing with errors loading the shell and then git refuses to even change a branch, let alone pull from origin. I could install git-lfs globally, but it's difficult to get those updated without manual intervention.

I think this might be a bad idea in general: this will tie a repo to an old git-lfs - does the git-lfs install command update the existing hook if there's one already there? How will users know they need to run it again?
If you just install the hook every time using a shell hook (and it overwrites the old path), then I think it makes sense to have this feature, but behind a flag or something so that it isn't the default behaviour.

@lukegb yes I realized that a flag is needed so I'm going to mark this PR as a draft as I'm considering doing a proper flag for it and file it upstream first.


Remove engines from package json

What is this PR and why do we need it?
The build is currently failing so this reverts some changes which, while not very suspicious, are something npm could mistakenly complain about.
Pre-Merge Checklist (if applicable)

[x] Unit and Feature tests have been added/updated for logic changes, or there is a justifiable reason for not doing so.

Closing because the build issue was resolved in another way (by regenerating an npm token).


Remove const vars used for logical tensor indices

The below list of files contain const vars which are used as logical tensor indices.  Each of these variables can be converted to a constexpr.  Making this conversion would have the following results:

load immediate instructions on most CPU architectures
no memory access for the index value
single CPU cycle to place the index value into a register
recovery of 352 bytes of code-space memory (Flash on some platforms, RAM on others)

This would also require special handling for Hexagon.  The Hexagon library has references to these const vars.  Therefore, just for Hexagon, an additional source file would be added to the Hexagon third_party source directory containing the const variable symbols.  Once the TFLM codebase is updated to use constexpr and Hexagon has updated to the latest TFLM codebase, this additional source file can be removed.  It is theorized that Hexagon will not need to make any changes to their internal code.
`tensorflow/lite/micro/kernels/activations_common.cc
33,1: const int kActivationsInputTensor = 0;
34,1: const int kActivationsOutputTensor = 0;
tensorflow/lite/micro/kernels/add_common.cc
31,1: const int kAddInputTensor1 = 0;
32,1: const int kAddInputTensor2 = 1;
33,1: const int kAddOutputTensor = 0;
/lite/micro/kernels/circular_buffer_common.cc
30,1: const int kCircularBufferInputTensor = 0;
31,1: const int kCircularBufferOutputTensor = 0;
tensorflow/lite/micro/kernels/conv_common.cc
26,1: const int kConvInputTensor = 0;
27,1: const int kConvWeightsTensor = 1;
28,1: const int kConvBiasTensor = 2;
29,1: const int kConvOutputTensor = 0;
tensorflow/lite/micro/kernels/depthwise_conv_common.cc
30,1: const int kDepthwiseConvInputTensor = 0;
31,1: const int kDepthwiseConvWeightsTensor = 1;
32,1: const int kDepthwiseConvBiasTensor = 2;
33,1: const int kDepthwiseConvOutputTensor = 0;
tensorflow/lite/micro/kernels/elementwise.cc
36,1: const int kElementwiseInputTensor = 0;
37,1: const int kElementwiseOutputTensor = 0;
tensorflow/lite/micro/kernels/fully_connected_common.cc
29,1: const int kFullyConnectedInputTensor = 0;
30,1: const int kFullyConnectedWeightsTensor = 1;
31,1: const int kFullyConnectedBiasTensor = 2;
32,1: const int kFullyConnectedOutputTensor = 0;
tensorflow/lite/micro/kernels/hard_swish_common.cc
31,1: const int kHardSwishInputTensor = 0;
32,1: const int kHardSwishOutputTensor = 0;
tensorflow/lite/micro/kernels/leaky_relu_common.cc
28,1: const int kInputTensor = 0;
29,1: const int kOutputTensor = 0;
tensorflow/lite/micro/kernels/logical_common.cc
25,1: const int kLogicalInputTensor1 = 0;
26,1: const int kLogicalInputTensor2 = 1;
27,1: const int kLogicalOutputTensor = 0;
tensorflow/lite/micro/kernels/logistic_common.cc
29,1: const int kLogisticInputTensor = 0;
30,1: const int kLogisticOutputTensor = 0;
tensorflow/lite/micro/kernels/mul_common.cc
29,1: const int kMulInput1Tensor = 0;
30,1: const int kMulInput2Tensor = 1;
31,1: const int kMulOutputTensor = 0;
tensorflow/lite/micro/kernels/pooling_common.cc
27,1: const int kPoolingInputTensor = 0;
28,1: const int kPoolingOutputTensor = 0;
tensorflow/lite/micro/kernels/sub_common.cc
32,1: const int kSubInputTensor1 = 0;
33,1: const int kSubInputTensor2 = 1;
34,1: const int kSubOutputTensor = 0;
tensorflow/lite/micro/kernels/svdf_common.cc
43,1: const int kSvdfInputTensor = 0;
44,1: const int kSvdfWeightsFeatureTensor = 1;
45,1: const int kSvdfWeightsTimeTensor = 2;
46,1: const int kSvdfBiasTensor = 3;
47,1: const int kSvdfInputActivationStateTensor =
49,1: const int kSvdfOutputTensor = 0;`

freshen


Outdated pod-identity-webhook version includes a bug that disallows injection of AWS_STS_REGIONAL_ENDPOINTS=regional

VPCE requires SetRegion and SetEndpoint parameters explicitly defined to be automatically used over the global (default) STS endpoint. The AWS-Pod-Identity-Webhook operator on is still the initial version (v0.1.0) which currently cannot set these parameters, because of https://github.com/aws/amazon-eks-pod-identity-webhook/pull/120. Regionality should be either set true by default or the operator should be updated to a version that allows env injection w/ sa annotation.
Given this bug that was solved later in EKS, when is this component intended to be updated? The deployed version in our  4.13 cluster is the v0.1.0, but this repository has branches for releases that seemingly covered this issue.
Our requirement is to be able to restrict cross-account IRSA conditionally and utilize an interface endpoint for regional sts.

https://github.com/openshift/aws-pod-identity-webhook/pull/166 will land upstream rebase in the 4.15 branch when it opens. We'll look into backporting once merged.

@abutcher that's great. We appreciate also your attention to this issue and look forward to its swift resolution. Thx for your assistance.

@abutcher is it also possible to backport it to 4.14?


[FR] Dark mode for Vue SFC Playground

What problem does this feature solve?
Currently, Vue SFC Playground only has a light mode, would be great if dark mode is supported as well.
What does the proposed API look like?
A dark mode switch.

see https://github.com/vuejs/vue-next/issues/3668


Merge Old https://github.com/DioxusLabs/cli/pull/124 PR to main repo

Basically just a straight copy, besides a few spots where there were some gaps in the old and new code.
Hastily put together so please review, any suggestions are welcome, all credit to mrxiaozhuox
Edit: Link 124

Thanks for trying to move this over, but we are going to use WASI for the plugin system instead of Lua. I could try to invite you to that branch if you still want to work on plugins. (discussion)

Closing this in favor of #1673


Solar Inverter communication with max 232 not work properly for all inverter

Hi, I am working on a small project depending on Arduino, max 232, and solar inverter.
I do the wiring and upload the code, the problem is I have two brands of solar inverters, one of them a commercial blue sun inverter,
the second one is the Falcon inverter.
So, I could send a message and get a response from the Falcon inverter, but the blue sun inverter did not respond to all messages.
I check the Voltage of TX, and RX of the Rs232 cable of the blue sun inverter and it seems correct as -13V for TX and  0V for RX same as the Falcon inverter.
I went to my friend and got the same problem with his solar inverter, is the problem with the wiring, or does each solar inverter brand has a unique commutation protocol?
SolarInverterCode.zip

Why do you expect both inverters are using the same protocol?

Hello @syssi, I think that because I did a lot of research on the internet and all results indicate that it uses the rs232 protocol.
like this file:
RS232 communication Protocol for InfiniSolar 3K.pdf

Please try to use the PC application Watchpower on each inverter. If Watchpower is working you should be able to replicate the communication. Furthermore Watchpower has a debug log so you can have a look at the traffic here.

Thank you my friend for replying.
I tried Watch Power when I plug the Falcon inverter with a USB cable, it appears in the Watch Power app but when I plug second inverter it does not appear on my PC.

Sounds like the second inverter isn't supported by Watchpower because it uses another protocol.

Today, I went to another friend who works in an electronics store, he give me permission to try on different brands of inverters,
The inverter that works default protocol is SAKO, FALCON, and PowerLand, RENOVO, but the inverters that not works are MUST, BlueSun.

is there any possible way to know what is the protocol that the inverter works on ??
I have the catalog and all the paper, but there is nothing about communication only the use of rs232.

is there any PC application for Solar inverter that use different protocol?

Is there any difference if I use esp2866 instead of Arduino?

Some Arduinos are using a logic level of 5V. The ESP has a logic level of 3.3V. UART-TTL communication should be pretty similar.

is there any PC application for Solar inverter that use different protocol?

I cannot help here.

thank you, my friend.
I will update the state of the issue if I get any solution.


enhancement: Disable Prompt Submission via Enter Key on Mobile Devices

Is your feature request related to a problem? Please describe.
When using the application on mobile devices or smaller screens, there is no easy way to create a new line within text areas as the Enter key submits the prompt. In contrast, in OpenAI web UI, when window size is smaller enter doesn't submit with enter key..
Describe the solution you'd like
On mobile devices or screens below a certain width (e.g., 768px), the Enter key should not trigger the submission of the prompt.
Describe alternatives you've considered
I have explored alternative solutions, such as utilizing different keyboard options (Google Keyboard and Samsung Keyboard), but none of them provided a suitable workaround for this issue.
Additional context
I primarily use this application hosted on my server as a Progressive Web App (PWA) on my Android smartphone. It is uncertain whether this issue is specific to my device or a more widespread concern.
Here is a potential implementation approach that could be added to the file "src/lib/components/chat/Messages.svelte":
isMobileDevice = false;

// Check if the screen size is small
if (window.innerWidth <= 768) {
  isMobileDevice = true;
}

// ... (existing code) ...

on:keypress={(e) => {
  if (!isMobileDevice) { // Only handle Enter key if not a mobile device
    if (e.keyCode == 13 && !e.shiftKey) {
      e.preventDefault();
    }
    if (prompt !== '' && e.keyCode == 13 && !e.shiftKey) {
      submitPrompt(prompt, user);
    }
  }
}}

I was not confident in the tech stack used here, so I did not request any merge. Feedback on the approach and suggestions welcome.

Great suggestion! I'll take a look!


FLAS-3: Improve Dark Mode Aesthetics and Toggle Button Design

Description of the Change
This pull request addresses the issue of the dark mode feature not being properly implemented, as described in issue #3. The changes focus on improving the aesthetics of the dark mode and redesigning the toggle button to resemble iPhone-style toggles.
Code Changes

Updated style.css to enhance the dark mode background and form element styles. The background color for input and textarea elements in dark mode is now set to a darker shade, with text color adjusted for better readability.
Redesigned the toggle button in style.css to have a rounded appearance with a sliding effect, making it more visually appealing and user-friendly.
Modified base.html to replace the button element with a div for the toggle, aligning with the new design requirements.

Related Issues

fixes #3

Checklist

[ ] Add tests to demonstrate the correct behavior of the change.
[ ] Update relevant documentation in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.

Looks like the html element has a hardcoded background colour, remove this so that all styles come from the body or elements


Double flattening does not work

Hi,
When using a JSON input file with 2 levels, I try to use the new Flatten functionality twice.
Unfortunately this does not work properly:
or he incorrecly detects the sublevels as being an array instead of the proper data type
or he fails with user input error when trying to execute
or he complaints that a mapping is not filled in, while it is. Reselecting the mappings does make this issue go away.
I can provide a sample input file and the data flow if required.
Regards,
Tjomme

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

ID: 9dad308d-58b5-79db-ab11-ebf3aeee3e94
Version Independent ID: 5a28dc38-39c8-3e58-9ae2-52a6345e6442
Content: Flatten transformation in mapping data flow - Azure Data Factory
Content Source: articles/data-factory/data-flow-flatten.md
Service: data-factory
GitHub Login: @kromerm
Microsoft Alias: makromer

@TjommeVergauwen Thanks for bringing this to our attention. Could you please share some sample that way we could reproduce the scenario.
Thanks.

Hi,
The JSON source file 'SpotifySearch-Queen.json' contains 3 hierarchies we want to flatten: A first level on Items and a second level for images and genres.

Without the second split on images and/or genres all works fine.
Source code is attached:
50520-ADF_Flatten.zip
Is this OK?

Thank you for sharing the additional details, @TjommeVergauwen .  I am actively looking into the issue and hope to get back to you soon.

@TjommeVergauwen I have reproduced your issue and will escalate internally.
When I hear back I will let you know.  Until then, I think I found a work-around.  It is possible to flatten items & genres, or items & images in the same step.

In the above screenshot I show the results of flattening items & genres together.  I did this by specifying unroll by: artists.items.genres and unroll root: artists.items as show below.

I also simplified the flow by replacing the conditional split with "new branch" (see below pic)

"New branch" accomplishes what you were doing with the conditional split, without the extra logic.
Please let me know if you found this useful.

Hello @MartinJaffer-MSFT ,
This is a great suggestion!
Unfortunately I'm running into error  "4501","Hit unexpected exception and execution failed." when trying this with multiple branches.
As a test, I cloned the data flow and removed one the first branch and then the other branch. Each of them work perfectly. It's only when combined that I get this error.
Another annoying thing is that the flatten step regularly looses it's column mappings. That is, the column mappings are still there, but produce a 'missing mapping' error when trying to publish. Re-selecting the same mappings again does solve the issue.
Thanks for checking this.

This is quite disappointing.  I will attempt to reproduce the parallel not working, and add it to my escalation.  With neither flatten in parallel nor flatten in series working, this becomes a blocking issue.  The loss of column mappings I ran into in a different issue.
Another approach to try would be adding a duplicate source in the data flow instead of using the 'new branch' or 'conditional split'.  By not letting the streams touch each other, the success or failure could provide insights.

@TjommeVergauwen it might get to work now as I had a similar issue and MS fixed a front-end bug that prevented the double flatten in my case.
Ciao, Roberto

@RobMcZagBDS Thank you for letting us know.  @TjommeVergauwen  I will close the issue now.  Please tag me in comment if you are still facing the problem.


Highlight square

Hi, is there a way to highlight squares in different colors? I've tested markers but it's not the same I think, maybe I'm missing a way to use them.

I needed to color squares red and green. I added these two classes to the end of the markers.css
  .cm-chessboard .markers .marker.bad-square {
      fill: #ff0000;
      opacity: 1.0;
  }
  .cm-chessboard .markers .marker.good-square {
      fill: #7CFC00;
      opacity: 1.0;
 }

Then, for each "square" I would
const useClass = (isBad) ? "bad-square" : "good-square"
const myOwnMarker = {class: useClass, slice: "markerSquare"}
chessboard.addMarker(myOwnMarker,square)

I would recommend to define two markers for this:
const goodMarker =  {class:  "good-square", slice: "markerSquare"}
const badMarker =  {class:  "bad-square", slice: "markerSquare"}

If you do so, you can also remove markers.
chessboard.removeMarkers(goodMarker)
chessboard.addMarker(goodMarker, square)
chessboard.addMarker(badMarker, square)


Adding instructions for porting a course to Coursera!

Purpose/implementation Section
What changes are being implemented in this Pull Request?
I'm adding instructions about how to add a course in Coursera using this template. Tried to add screenshots and be descriptive where possible. Hopefully it is clear!
This will close #123 !
Also snuck in a line so we can close #164

These look great!! Just adding a reminder here about asking Ira about his document.

Also I am having a bit of trouble with running the coursera render script, but perhaps I need an updated version - will check on that

Also I am having a bit of trouble with running the coursera render script, but perhaps I need an updated version - will check on that

If you want to send me the quiz that you are having trouble converting, I'm happy to look into it.


baseUrl with file scheme (local file)

is there a possibility using file:// scheme on baseUrl (localfile)?

imageproxy was always specifically designed to proxy images that are already publicly available on the web.  Adding support for proxying file URLs would be a significant amount of work since the http package doesn't support it, and I wouldn't want to add the additional complexity to imageproxy even if the http package did support it.  Thanks for the interest though.

I know, this project created to proxy images that are already publicly available on the web, but i use it mainly for managing/resizing image, thats why iam asking for it.
Btw, thanks for clarification.

@emaniacs We are interested in the same. Did you ever find a solution?
@willnorris I don't know Go, but which API call do you use to fetch the original url? Maybe the API call knows how to handle custom protocol urls?


Browserify support

Add support for http://browserify.org/.

Added UMD support (http://davidbcalhoun.com/2014/what-is-amd-commonjs-and-umd/) to support both AMD & CommonJS.


Theme name change from CodingInSylens to just Sylens

Hello there jean, its me again I was wondering if I can request a theme name change from CodingInSylens to Sylens.

Hi! I renamed you theme as requested. You may need to set it again in the settings module.

Thank you so much jean! I just saw the name change update when I select the theme.


Upload caching

I think it'd be very beneficial to enable caching with an ETag.
https://github.com/JoeGandy/ShareX-Custom-Upload/blob/6571ae901bb1c006a0bc082ec7792d3835302b66/src/rewrite.php#L60-L62
Adding these lines below between L60 and L62 perform wonderfully. Can't do a PR right now, sorry!
This way, if a file changed, the updated file would be served as the ETag (SHA256 hash of the file)
header('Cache-Control: public, max-age=360, stale-while-revalidate=120');

$etag = "test";
$etag_algo_used = "";

$time_start = microtime(true);

if (PHP_INT_SIZE === 8) {
    # 64-bit, so we use SHA512 for faster performance
    $etag = hash_file("sha512", $file_path);
    $etag_algo_used = "SHA-512";
} else {
    # 32-bit (or other?), so we use SHA256 for faster performance
    $etag = hash_file("sha256", $file_path);
    $etag_algo_used = "SHA-256";
}

$time_end = microtime(true);

header('ETag: "'.$etag.'"');

# get time difference in ms
$time_diff = round(($time_end - $time_start) * 1000, 4);
header('X-ETag-Generation-Milliseconds: '.$time_diff);
header('X-ETag-Algorithm: '.$etag_algo_used);

I really like this idea! Thanks for suggesting it and for writing the code. I think it's worth adding as an optional config option, but it's definitely worth having! I'll open a PR to add this.

Hey, quick question. What's the point of providing the ETag generation algorithm and generation time. Those aren't standard properties and ETags should really be used as opaque identifiers—just a random meaningless string.

What's the point of providing the ETag generation algorithm and generation time.

Debugging. If someone has a very low power server, like a Pi 2, they might wonder why the server takes a while to respond.
It'd be a good idea for this to be togglable in settings. I like it because I'm curious and like providing extra info when I can, just because... why not?

Those aren't standard properties

Any header starting X- is considered a custom header. X-Forwarded-For isn't standard, but large companies like Cloudflare use them.

ETags should really be used as opaque identifiers—just a random meaningless string.

Not at all... Check MDN. https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/ETag

The method by which ETag values are generated is not specified. Often, a hash of the content, a hash of the last modification timestamp, or just a revision number is used.

Bearing in mind that the user is using the site to get the hosted content, providing a hash of the content isn't bad at all. They could get a hash by downloading it themselves anyway.

From the MDN page on HTTP caching:

The ETag response header is an opaque-to-the-useragent value that can be used as a strong validator. That means that a HTTP user-agent, such as the browser, does not know what this string represents and can't predict what its value would be.

But I'm not here to argue, you make a fair point. By non-standard I meant that I can't find anything that uses it (like a caching server), but yeah I guess it's nice to have more information. I'll send generation algo by default, but I think the generation time could be in something like a debug mode, as you said. Debug mode could potentially be used in the future for adding other verbose information, but sending hash generation time by default seems like a bit of a waste of bandwidth.

I mean the browser doesn't know what the ETag represents, unless it starts doing hash cracking lol.
I agree about the statements about debug mode!


startStep now computes an initial RewardProvenance.

Several of the fields were not being populated in the reward provenance. In particular, the startStep function was computing several of them, but was not in the provenance monad.

@nc6 I added a description. I'll be better remembering to leave good notes in PR when doing code reviews, thanks!


[router, utils] Allow coin gecko API key to be passed in

If routers expect high volume, they should be able to pass through a coingecko API key for calculating fees. Additionally, if the coin gecko service gets rate-limited, the utils package where it is used should clearly display that.

Coingencko api is already free, https://www.coingecko.com/en/api
So, i don't think we need to be worried about it.


Shouldn't 1.14 really be 2.0?

There seems to be a complete change in the API (promises are great, don't get me wrong), and the 'mapInitialized' event I relied upon seems have gone?

Hi @simonh1000
'mapInitialized' is still working as well as $scope.map, but those are deprecated.
1.14 is the prepration of angular 2 migration, but not 2 yet.
All the features you are using should be available, unless I miss something.

$scope.map is deprecated, but still available
$scope.$on('mapInitialized', function(event, map) {..} is deprecated, but stiill available
$scope.showInfoWindow(id) is deprecated, but still available
map tag is deprecated, but still available


List of Identifiers

Isn't the list of identifiers for a given project working?
Has it ever worked or is it just a planned feature?

Real bug, thanks for reporting.  Fix in next round.  This is because some Wikidata identifier is causing a problem.

This is fixed, coming soon to live.


New feature: Get-ExchangeServerCertificates

Feature request: #417
First draft
Added a check to query the certificates from an Exchange server. We calculate how long they're valid and if you have a public key of at least 2048 bit.
We also keep the original certificate object for further use in the future.

Please review @dpaulson45

Need to make some changes to the current code set. Moving over to a Dev branch for work.


Implementing WAIT command

See https://github.com/nrk/predis/issues/298 for my motivation on this.
When using redis-cluster with a version >= 3.0, the WAIT command comes in handy to increase data safety with important writes by making sure that the write has been acknowledged by n (number of slaves).
This PR implements the command for versions >= 3.0
Since this is the first time I was digging into the Predis src, it might be that I missed some obvious things, although the tests are passing for me.
The tests have been set up locally with a redis-cluster created by the cluster-create util in the redis distribution.I still need to investigate on how to set this up in travis though, maybe someone of you can help. :) I have added a Makefile to set up a redis cluster (v3.0) within travis.

First of all thanks for your PR @tPl0ch! It would be awesome if you could extract the part that sets up redis-cluster on Travis CI into a separate PR because I'm definitely interested to have that merged ASAP :-)
Back to the actual issue, as I've said in my inline comment I think that using a command handler to return a value that is not a key (in the case of WAIT the first argument is just the number of slaves) does not make much sense. But we could still play along these lines by changing our internal assumptions slightly.
Right now Predis\Cluster\ClusterStrategy::getSlot(), which is the actual method used by aggregate connections to obtain a valid slot for the specified command, is expected to return either:

a slot if it can be calculated from a valid key or multiple keys
NULL when the command is not supported... which until now just meant that it cannot be mapped to one specific slot.

We might just change that to augment the possible return values by including boolean FALSE to indicate that the command can be used in the context of a certain cluster implementation even though it cannot be mapped to a specific slot. At this point when  FALSE is returned our implementation could stick to the last used node or just a random one to issue the command.
Thoughts? To tell you the truth I'm not a fan of having more than one type of value returned by Predis\Cluster\ClusterStrategy::getSlot() (or any method for that matter) but from a pragmatic POV I think it does not add much complexity to the library internals.

@nrk I made a little prototype change now: https://github.com/nrk/predis/commit/ab4296db8128c46456b24af919f9b5e7e3b5faae
Thoughts on this?

I guess I'll need some time to think about it and play with some actual code before carrying on with the changes, in the end it still feels a bit too hackish :-)

Any updates on it? Changes looks promising


Error: decoding error response: invalid character 'p' after top-level value

Your environment

Terraform Provider Version: v1.1.2

Connect Server Version:

OS: ubuntu / docker

Terraform Version: v1.0.3
What happened?

I'm trying to create a password to my test vault and it returns:
module.secrets.onepassword_item.sql_password: Creating...
╷
│ Error: decoding error response: invalid character 'p' after top-level value
│
│   with module.secrets.onepassword_item.test_password,
│   on modules/secrets/main.tf line 18, in resource "onepassword_item" "sql_password":
│   18: resource "onepassword_item" "sql_password" {
│
╵

What did you expect to happen?
It should successfully create a password when running terraform apply
Steps to reproduce

create a vault for testing
setup the secrets automation workflow, api tokens and such
have following terraform config:

terraform {
  required_providers {
    onepassword = {
      source = "1Password/onepassword"
      version = "~> 1.1.2"
    }
  }
}

provider "onepassword" {
  url = "http://op-connect-api:8080"
}

data "onepassword_vault" "staging_vault" {
  name = "staging-test"
}

resource "onepassword_item" "sql_password" {
  vault = data.onepassword_vault.staging_vault.id

  title    = "SQL password - test"
  category = "password"
  password_recipe {
    length  = 40
    symbols = false
  }
}

run terraform apply
get the error mentioned above

If the error is valid which it very well might be, the error message should be more descriptive.

The reason for this was since I was using data.onepassword_vault.staging_vault.id instead of data.onepassword_vault.staging_vault.uuid. data.onepassword_vault.staging_vault.id returns vault/some-id instead of some-id which causes the create to fail.
However, at least IntelliJ is not able to highlight or auto complete the uuid part of the data.onepassword_vault.staging_vault.uuid. So there is two underlying problems: Error message was not descriptive and some fields are not picked up by auto complete and shines in red although they are applied just fine.

The reason for this was since I was using data.onepassword_vault.staging_vault.id instead of data.onepassword_vault.staging_vault.uuid. data.onepassword_vault.staging_vault.id returns vault/some-id instead of some-id which causes the create to fail.
However, at least IntelliJ is not able to highlight or auto complete the uuid part of the data.onepassword_vault.staging_vault.uuid. So there is two underlying problems: Error message was not descriptive and some fields are not picked up by auto complete and shines in red although they are applied just fine.

This one works for me.


[Patch Request]: Ultimate College of Winterhold and The Great Cities of JK's North

Checklist

[X] This patch doesn't already exist
[X] This patch would only contain ESPs or mesh fixes

Request away!
Hi, I wrote this as a request before, figured i'd file it here since I saw a new page went live. Basically, The Great City of Winterhold V4 has patches for Immersive College of Winterhold and Obscure's College of Winterhold, but not Ultimate College of Winterhold, which has a patch for the older Great Cities of Winterhold version with Windhelm architecture. Navmesh conflicts. The author of the ICoW and OCoW patches for TGCoW v4, Janquel, is unlikely to be able to get to the patch due to the growing mountain of patches he's currently maintaining. I've seen a few other users mention they're looking for this one. If you could ever get around to it, it'd be greatly appreciated.

Ah yeah, I'll look into it if it isn't too much work. I've never used that mod before

robertgk2017  of Rob's Bug Fixes fame might be approaching this. If he doesn't get around to it within a month I'll see what I can do, but honestly I looked into it in both the CK and in-game and the nav-meshing seemed completely fine.

Since I don't think this needs a patch, unless I'm not seeing the conflict or something, I'll be closing this. Feel free to elaborate, I'll open this up if there is the need for a patch. It seems like Janquel just kind of cleaned the navmeshes a bit, so not having a patch really doesn't seem like a dealbreaker. Would prefer to save my time :P

According to comments on the Nexus, TGCoW v4's patch for Immersive College of Winterhold is mostly correct, but doesn't edit navmesh around the new notice boards added by OCoW, which NPCs get stuck on. Using Obscure's patch fixes this, but causes NPCs to try to walk through the new ICoW buildings added to the courtyard. theoretically, taking the ICoW patch for TGCoW v4 and altering the navmesh so its aware of the new stuff added to the courtyard (the signs, say) should do the trick.

Hmmm, i'll ask rob for his progress and if he doesnt have anything i'll take a second look maybe

Rob's still approaching this, I'll keep the issue up until it's uploaded


[ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpreter

What is this PR for?
New interpreter, SAP BusinessObjects (Universes). See documentation
What type of PR is it?
[Feature]
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3194
How should this be tested?
see screenshots
Screenshots (if appropriate)
Autocomplete + get results

Errors

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

@felixcheung
there are unit tests for convert queries (from Zeppelin format to SAP format) and unit tests for auto completeion.

Ready to review

Cool! Looks great to me.

@zjffdu @felixcheung
do you have any other questions?

@Leemoonsoo @felixcheung @zjffdu
if no more questions maybe merge it?

need to specify the format of the "Combined filters (and/or)"

I'd like to merge this PR to master if we have no further reviews. @masyan Is this PR closed on purpose?

@Leemoonsoo
okay, I had one question in support of SAP, but they don't answer

CI green https://travis-ci.org/masyan/zeppelin/builds/342323724

@Leemoonsoo
found bug of parser

@Leemoonsoo
I fixed the parser. CI red because of unrelated code. Ready to merge

Thanks @masyan for great contribution. Looks good to me.
I'll merge this to master and branch-0.8 if no further comments.

@masyan Do you have your JIRA id?

@Leemoonsoo
https://issues.apache.org/jira/secure/ViewProfile.jspa?name=masyan

Thank you. Marked ZEPPELIN-3194 as resolved.


postgres_backend: stop using async_trait

We have async fn in traits now, let's use it.
Preliminary refactoring while working on https://github.com/neondatabase/neon/issues/7427
and specifically https://github.com/neondatabase/neon/pull/8286

there is some lifetime issue that makes one of the regression tests fail, see #7301 . I gave up in the end because I felt I spent too much time on a simple refactor.

Hm, I'm getting simply
# .neon/safekeepers/sk1/safekeeper-1.log
thread 'WAL service worker' has overflowed its stack
fatal runtime error: stack overflow

So, I fixed it by bumping executor thread stack size 🤷

So, the increase in memory consumption by this would be 2MiB * runtime thread count.
The default tokio runtime has a pool size of ncpus + 512, I think.
Safekeeper has many tokio runtimes.
I don't know which ones use the spawn_blocking pool.
NB: tokio::fs uses spawn_blocking pool under the hood.
@arssher , @koivunej , thoughts on the risk?
So,

I assume that would only be virtual memory use, rather than resident memory usage, unless all threads (and not just the async worker threads) are hitting the upper half of the stack

Hmmm this makes sense. async_trait does boxing of the returned future, so if you remove it, the data gets stored on the stack then.
Maybe one could add explicit boxing to the handler impl for the safekeepers instead of bumping the stack size?
Personally I'm also fine with just bumping the stack size if the problem goes away and there is no issue with it in prod.
however there is a potential issue here:

So, the increase in memory consumption by this would be 2MiB * runtime thread count.

In theory, a 2 MB stack size would mean 512 times 2 MB, which makes 1 GB. There is overcommit, so it's not actually present physically until it's written to, until then it points to a zero page. however over the lifetime of the process we will write to the stacks of these threads, and IIRC there is no reverse operation to this, so the memory will remain physically present even if the RAM is not used any more.
For boxing, only the space is used that is actually allocated at a current point in time, and the same space can be reused by a later creation of the future. I hope Box::new(...) solves the issue already :laughing:, because while it usually does, it's not guaranteed to forego the stack.

abandoning this, the async fn that we're calling are actually not that high traffic, so, not that high allocation rate. Not worth the headache / complication with shepherding this into prod.


Speed up filters

This is a big project, requiring implementing all/some of the flters in Elasticsearch aggregations using nested types.  It may not be possible.

This is (at least partially) addressed.  Aggregations would still be faster, but again not sure if it's possible.


Can't test changes made to the fork

When building with the given script, even with no changes to the source, it will return a "Failed to execute main script" error.
Aside from that, testing main.py returns a FileNotFound error, as  dev_working_dir\DreamAPI.log does not exist.
It's possible I'm doing something wrong, if so please let me now!
The log file created by the executable is just this:
[%A totally reasonable time to be awake forking%][DEBUG]	Setting log level to DEBUG

I have made some improvements to code portability and added instructions to run the project from source: https://github.com/acidicoala/dream-api#running-from-source
Could you test it to see if the instructions work for you?

Same issue, here's the traceback:
Traceback (most recent call last):   File "C:\Users\%MyUserName%\Desktop\dream-api-1.1.1\src\main.py", line 5, in <module>     from gui.app_window import ApplicationWindow   File "C:\Users\%MyUserName%\Desktop\dream-api-1.1.1\src\gui\app_window.py", line 5, in <module>     from setup.config import config_path, config   File "C:\Users\%MyUserName%\Desktop\dream-api-1.1.1\src\setup\config.py", line 6, in <module>     from util.log import log   File "C:\Users\%MtUserName%\Desktop\dream-api-1.1.1\src\util\log.py", line 31, in <module>     log = Log()   File "C:\Users\%MyUserName%\Desktop\dream-api-1.1.1\src\util\singleton.py", line 6, in __call__     cls._instances[cls] = super(Singleton, cls).__call__(*args, **kwargs)   File "C:\Users\%MyUserName%\Desktop\dream-api-1.1.1\src\util\log.py", line 17, in __new__     fileHandler = FileHandler(log_path, 'w')   File "C:\Users\%MyUserName%\.pyenv\pyenv-win\versions\3.9.0a4-win32\lib\logging\__init__.py", line 1144, in __init__     StreamHandler.__init__(self, self._open())   File "C:\Users\%MyUserName%\.pyenv\pyenv-win\versions\3.9.0a4-win32\lib\logging\__init__.py", line 1173, in _open     return open(self.baseFilename, self.mode, encoding=self.encoding, FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\%MyUserName%\\Desktop\\dev_working_dir\\DreamAPI.log'
Odd thing that of the double \ , even so, the file does not exist either way, is that file perhaps a trace from another one made during testing?

Are you running the python src\main.py from the project's root directory?

Indeed

I've added the dev_working_dir in one of the latest commits, so no wonder you did not get it before. Also, I would advise against using global python interpreters and instead creating per-project virtual environment, so that dependencies from different project don't conflict with each other.

gotcha!