[SPARK-46943][SQL] Support for configuring ShuffledHashJoin plan size Threshold

What changes were proposed in this pull request?
Introduce the configuration spark.sql.shuffledHashJoinThreshold, the default is the maximum value of Long, which can limit the maximum value of the size converted to SHJ.
Why are the changes needed?
When we enable spark.sql.join.preferSortMergeJoin=false, we may get the following error.
org.apache.spark.SparkException: Can't acquire<PHONE_NUMBER> bytes memory to build hash relation, got 478549889 bytes
	at org.apache.spark.sql.errors.QueryExecutionErrors$.cannotAcquireMemoryToBuildLongHashedRelationError(QueryExecutionErrors.scala:795)
	at org.apache.spark.sql.execution.joins.LongToUnsafeRowMap.ensureAcquireMemory(HashedRelation.scala:581)
	at org.apache.spark.sql.execution.joins.LongToUnsafeRowMap.grow(HashedRelation.scala:813)
	at org.apache.spark.sql.execution.joins.LongToUnsafeRowMap.append(HashedRelation.scala:761)
	at org.apache.spark.sql.execution.joins.LongHashedRelation$.apply(HashedRelation.scala:1064)
	at org.apache.spark.sql.execution.joins.HashedRelation$.apply(HashedRelation.scala:153)
	at org.apache.spark.sql.execution.joins.ShuffledHashJoinExec.buildHashedRelation(ShuffledHashJoinExec.scala:75)
	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage11.init(Unknown Source)
	at org.apache.spark.sql.execution.WholeStageCodegenExec.$anonfun$doExecute$6(WholeStageCodegenExec.scala:775)
	at org.apache.spark.sql.execution.WholeStageCodegenExec.$anonfun$doExecute$6$adapted(WholeStageCodegenExec.scala:771)
	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsWithIndex$2(RDD.scala:915)

Because when converting SMJ to SHJ, it only determines whether the size of the plan is smaller than conf.autoBroadcastJoinThreshold * conf.numShufflePartitions.
When the configured numShufflePartitions is large enough, it is easy to convert to SHJ. The executor build hash relation fails due to insufficient memory.
https://github.com/apache/spark/blob/223afea9960c7ef1a4c8654e043e860f6c248185/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala#L505-L513
Does this PR introduce any user-facing change?
No
How was this patch tested?
Production environment verification
Add UT
Was this patch authored or co-authored using generative AI tooling?
No

@cloud-fan @ulysses-you Please help review when you have time. Thank you!

Does spark.sql.adaptive.maxShuffledHashJoinLocalMapThreshold satisfy your requirements ? It is more accurate to convert SMJ to SHJ in AQE by checking if the actually partition size less than threshold.


added shadows for homepage header and homepage buttons

Description
added shadows for homepage header and homepage buttons
Fixes # (issue)
How to test?
Please provide a short summary how your changes can be tested?
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

Thanks @Poorooshka for your review. I have created one more PR for those issues, please check. And we do not need create account now as we are only logging in through google account.


Add appveyor support

closes #25

Coverage remained the same at 100.0% when pulling 5bbe32aca78bdfd92e568aae1425c807944c09d1 on appveyor into 1ba5bd81dd30200b9da8577ccdaea6a6b8c78448 on master.

Coverage remained the same at 100.0% when pulling 5bbe32aca78bdfd92e568aae1425c807944c09d1 on appveyor into 1ba5bd81dd30200b9da8577ccdaea6a6b8c78448 on master.

Coverage remained the same at 100.0% when pulling aad79ec10b282c6ba2bad5e2c540115753d04537 on appveyor into 1ba5bd81dd30200b9da8577ccdaea6a6b8c78448 on master.

Coverage remained the same at 100.0% when pulling aad79ec10b282c6ba2bad5e2c540115753d04537 on appveyor into 1ba5bd81dd30200b9da8577ccdaea6a6b8c78448 on master.


[flutter_tools] make precache force blow away stamp files

Description
update flutter precache --force to delete all stamp files. In the event that a user is hitting a cache issue, this should be easier than re-downloading all artifacts or manually blowing away the cache.
This is probably how it should have worked in the first place

@christopherfujino for thoughts on whether this conflicts with the existing cache work

@jonahwilliams great idea, i was thinking about a new command for this functionality, but this makes sense.


Update permissions for namespaces/finalizers

This change was necessary for me to get the headless service to spin-up on OpenShift 4.9
To get modelmesh-serving working on OpenShift 4.9
Modifications:  update rbac to give permissions so the service can be created
Result:  The service comes up as expected on OpenShift 4.9

@crobby thanks for trying out on OpenShift 4.9 and providing the fix!

Thanks!
/approve


ModuleNotFoundError: No module named 'falkon.sparse.sparse_helpers' when testing installation

Hello,
I followed the installation steps and did not run into any error while compiling the library.
However, when trying the kernel ridge regression notebook, I cannot load the module.
I get the following error trace:

import falkon
Traceback (most recent call last):
File "", line 1, in
File "/home/vignac/falkon/falkon/init.py", line 3, in
from . import kernels, sparse, center_selection, preconditioner, optim
File "/home/vignac/falkon/falkon/kernels/init.py", line 1, in
from .kernel import Kernel
File "/home/vignac/falkon/falkon/kernels/kernel.py", line 6, in
from falkon.mmv_ops.fmm_cpu import fmm_cpu_sparse, fmm_cpu
File "/home/vignac/falkon/falkon/mmv_ops/fmm_cpu.py", line 10, in
from falkon.sparse.sparse_tensor import SparseTensor
File "/home/vignac/falkon/falkon/sparse/init.py", line 2, in
from .sparse_ops import sparse_norm, sparse_square_norm, sparse_matmul
File "/home/vignac/falkon/falkon/sparse/sparse_ops.py", line 4, in
from falkon.sparse.sparse_helpers import norm_sq, norm_
ModuleNotFoundError: No module named 'falkon.sparse.sparse_helpers'

Is it just a path that is incorrect, or do you think it is a bigger problem with the installation?
Thanks,
Clement

Packages version:
nvcc: 11.4
g++: 7.5.0
cmake: 3.18.2
$ pip list
Package           Version

certifi           2021.10.8
cycler            0.11.0
falkon            0.6.3
joblib            1.1.0
kiwisolver        1.3.2
matplotlib        3.4.3
numpy             1.21.4
Pillow            8.4.0
pip               21.2.4
psutil            5.8.0
pykeops           1.4.2
pyparsing         3.0.5
python-dateutil   2.8.2
scikit-learn      1.0.1
scipy             1.7.2
setuptools        58.0.4
six               1.16.0
threadpoolctl     3.0.0
torch             1.10.0+cu113              (works fine on gpu)
typing-extensions <IP_ADDRESS>
wheel             0.37.0
Installation traces: everything seemed to be fine:
$ pip install ./keops
Processing ./keops
DEPRECATION: A future pip version will change local packages to be built in-place without first copying to a temporary directory. We recommend you use --use-feature=in-tree-build to test your packages with this new behavior before it becomes the default.
pip 21.3 will remove support for this functionality. You can find discussion regarding this at https://github.com/pypa/pip/issues/7555.
Requirement already satisfied: numpy in /home/vignac/.conda/envs/falkon/lib/python3.9/site-packages (from pykeops==1.4.2) (1.21.4)
Building wheels for collected packages: pykeops
Building wheel for pykeops (setup.py) ... done
Created wheel for pykeops: filename=pykeops-1.4.2-py3-none-any.whl size=478011 sha256=dac861f7bd93a552854c4566aaf688a1deaaf737518a862f4818e04bc8b8d16d
Stored in directory: /tmp/pip-ephem-wheel-cache-bq1e39qr/wheels/36/47/f5/4be78e0d60dfe330cfb4652a2e21c469d4f6ea7bb0d0d767df
Successfully built pykeops
Installing collected packages: pykeops
Successfully installed pykeops-1.4.2
$ pip install .
Processing /home/vignac/falkon
DEPRECATION: A future pip version will change local packages to be built in-place without first copying to a temporary directory. We recommend you use --use-feature=in-tree-build to test your packages with this new behavior before it becomes the default.
pip 21.3 will remove support for this functionality. You can find discussion regarding this at https://github.com/pypa/pip/issues/7555.
Requirement already satisfied: torch>=1.4 in /home/vignac/.conda/envs/falkon/lib/python3.9/site-packages (from falkon==0.6.3) (1.10.0+cu113)
Collecting scipy
Downloading scipy-1.7.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (39.8 MB)
|████████████████████████████████| 39.8 MB 5.5 MB/s
Requirement already satisfied: numpy in /home/vignac/.conda/envs/falkon/lib/python3.9/site-packages (from falkon==0.6.3) (1.21.4)
Collecting scikit-learn
Downloading scikit_learn-1.0.1-cp39-cp39-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (24.7 MB)
|████████████████████████████████| 24.7 MB 108.8 MB/s
Collecting psutil
Downloading psutil-5.8.0-cp39-cp39-manylinux2010_x86_64.whl (293 kB)
|████████████████████████████████| 293 kB 125.6 MB/s
Requirement already satisfied: typing-extensions in /home/vignac/.conda/envs/falkon/lib/python3.9/site-packages (from torch>=1.4->falkon==0.6.3) (<IP_ADDRESS>)
Collecting threadpoolctl>=2.0.0
Using cached threadpoolctl-3.0.0-py3-none-any.whl (14 kB)
Collecting joblib>=0.11
Using cached joblib-1.1.0-py2.py3-none-any.whl (306 kB)
Building wheels for collected packages: falkon
Building wheel for falkon (setup.py) ... done
Created wheel for falkon: filename=falkon-0.6.3-cp39-cp39-linux_x86_64.whl size=1270129 sha256=a55a9db44d82a77908f9c190aae18500e199d29563d74434bbeeff2ed977fe41
Stored in directory: /tmp/pip-ephem-wheel-cache-qj6av2bg/wheels/42/2f/de/817b4dc8ce9bdfe9d8d5b31d82288a66442e83b0509995d8a1
Successfully built falkon
Installing collected packages: threadpoolctl, scipy, joblib, scikit-learn, psutil, falkon
Successfully installed falkon-0.6.3 joblib-1.1.0 psutil-5.8.0 scikit-learn-1.0.1 scipy-1.7.2 threadpoolctl-3.0.0

Hi! This might be an easy fix: have you tried to import falkon from within the same folder where you installed it?
If so, try again from another folder! (Or if it's important to work in the same folder install with pip install --editable).
If not let me know and I'll look into it more

Sorry, I forgot to answer!
It indeed works if I run it inside the falkon folder. Does it mean that I am supposed to include all my code inside this folder as well?


[WIP] Add support for OpenSfM

@kevinddchen  not sure if you're still working on this or if pending review: I fixed a typecheck as Im hoping to process some data through your script (thanks!) https://github.com/nerfstudio-project/nerfstudio/pull/1073

@antoinemacia I am not working on this at the moment--I hear this script is going through / has gone through a refactor? Either way, this PR will likely be superseded with a different one. I'm just keeping it here for future reference. Maybe someone with more knowledge can chime in here (@akristoffersen ?)
But if you want, you are free to merge into this branch

Got it thanks @kevinddchen  - Ill just merge it in to avoid having 2 PR floating on the same subject
Im actually just after processing equirectangulars images I just stitched to use in your recently merged work, without feeding it into OpenSfM

@antoinemacia I'm not sure on what's going to happen with this branch, but yesterday I pushed a branch on alex/open_sfm to merge a newer main commit into this branch. I wasn't able to get it working due to openSfM failing to install, so you're mileage may vary though.
Feel free to commit to either branch, or make your own off of them though. Originally we were looking at openSfM as an alternative to Colmap, but we'd need to look into it more to be sure.

Closing, this is pretty out of date now. If anyone wants to pick it up, feel free to reopen.

I am trying to use the results of OpenSfM instead of COLMAP as input. So I found this issue. Is the development of this feature discontinued?


net: add IP.IsPrivate (RFC 1918 & RFC 4193)

The net package seems to have many helpers to report what an IP is. e.g:

IsLoopback()
IsMulticast()
IsInterfaceLocalMulticast()

However there are no helpers to report if a IP address is in the private ranges (RFC 1918 & RFC 4193).
If no one has any objections I can make a method in net/ip.go named IsPrivate() that will check for the ranges listed in both RFCs using the same syntax the other helpers use.

How many more kinds of groups are there besides loopback, multicast, private, etc? That is, what else might we be missing? I'd like to understand if this is the last one or just the next on a very long sequence. It seems OK if we're almost done.

/cc @mikioh

One problem with an isPrivate() method is that those private ranges only apply on the internet. Lots of code runs inside cloud VPCs and other private networks which use the whole IPv4 address space, so what would be a public network address on the internet is actually private.

ping @mikioh, @aaranmcguire: roughly speaking, are there 2 more of these IP address classes that we need to add, or 20 more?

I don't know any more classes of addresses, however I'm not a network person so there may be more I'm unaware of.

It sounds like people are generally OK with IsPrivate, meaning either RFC 1918 (including with the standard IPv4-in-IPv6 embedding) or RFC 4193.
Let's stay away from IsTranslator and IsLocalUnicast for now.

This is just a reply to @rsc for sharing the background.

Why? We can easily include the "Unique Local Address" and "ULA" and "RFC 4193" in the docs, but is "private" an inaccurate description?

Not so pedantic, two concerns:

the existing methods for IP address identification are based on addressing architecture, and the  blocks described in RFC 1918 and RFC 4193 can be considered as globally (unique, sometimes non-unique) local unicast addresses which are detached from global routing operation and are subtypes of global unicast addresses as described in RFC 4291 (please don't say it's just for IPv6 instantly, IPv6 cannot ignore the transition to IPv6 from IPv4 as its nature). I'm not sure whether introducing APIs from the operational point of view is a good idea because I don't see any difference between this proposal and a proposal like net: add IsAmazonAZ, IsGoogleExternal and IsAzureElastic to IP.
privately use is one of the applications for the address blocks. I just wonder how do API users feel from !IP.IsPrivate().

generally OK with IsPrivate

I'd recommend renaming IsPrivate with IsPrivateUnicast. IsPrivate looks a bit vague and we already have IsGlobalUnicast.

What's the status of this? Can it be merged now?

Kindly paging @MaxSem the author of CL https://go-review.googlesource.com/c/go/+/162998/, @mikioh left some comments on your CL sometime ago, please take a look. Thank you!

As I see it, there were disagreements about the proposed function name among project members. I'm happy to change it, but I'd appreciate clarity as to the correct name :)

Would be great to see this brought in!

https://en.wikipedia.org/wiki/Reserved_IP_addresses uses the term Private Network (Used for local communications within a private network).
https://www.iana.org/assignments/iana-ipv4-special-registry/iana-ipv4-special-registry.xhtml uses the term Private-Use.
https://www.iana.org/assignments/iana-ipv6-special-registry/iana-ipv6-special-registry.xhtml uses the term Unique-Local.

Not sure if we care about what other languages are doing, but:
Ruby's ipaddr uses the term private for ip4/ip6.
Pythons ipaddress uses the term private as well.

Python ipaddress definition of private means reserved:
https://github.com/python/cpython/blob/master/Lib/ipaddress.py#L1539
https://github.com/python/cpython/blob/master/Lib/ipaddress.py#L1323

It looks like this useful CL has stalled for more than a year. I think it's ready to go except for some naming concerns. Right now the CL uses the name IsLocal, but the consensus on this thread (before the CL was sent) seemed to be IsPrivate. @mikioh suggested IsPrivateUnicast instead -- @rsc, thoughts about IsPrivate vs. IsPrivateUnicast?

What about IsOrganizationLocalUnicast?
https://en.wikipedia.org/wiki/Multicast_address#IPv6

IsOrganizationLocal is probably the same as site local scope, that has been deprecated in https://tools.ietf.org/html/rfc3879.

There is nothing wrong with IsPrivate, most network types will refer to addresses checked by the implementation as private or local addresses during day to day conversation.
net.ParseIP("<IP_ADDRESS>").IsPrivateUnicast()

If the function's name contains "Unicast", I expect the call below to return false because it is a broadcast address, and packets sent to it are not destined for one network interface.
Implementation wise, distinguishing between unicast and broadcast addresses without a subnet mask is impossible. PrivateUnicast is just a confusion waiting to happen.

Totally agree with @as, today without the netmask, it's not possible to identify an address as unicast/broadcast/network.
But Private range, Multicast range, LinkLocal range and Lookback range are clearly identified.

Is there anything still blocking this?

The CL was abandoned when the PR was closed. Someone will need to send a new CL and get it reviewed.

I'm going to create a Pull ...

If I have to do something just tell me :)
-> #42793

would also love to see this merged 👍

I think the comment in the IP.IsPrivate method is wrong.
For the local IPv4 addresses, the comment cites RFC 4193, but this document only specifies local IPv6 addresses.

@perillo, you are correct, thank you; it was just a typo. I had highlighted the change in my comment https://go-review.googlesource.com/c/go/+/272668/7..11/src/net/ip.go#b135

But I missed it in the later revision. Please feel free to send a change and tag me and I’ll review it.

@odeke-em thanks to you for the contribution.
I have also noted another small issue.  Unlike RFC 1918, RFC 4193 in section 3 does not say "The Internet Assigned Numbers Authority (IANA) has reserved ...".  Instead in section 8 (IANA Considerations) it just says "The IANA has assigned the FC00::/7 prefix to "Unique Local Unicast".
See https://tools.ietf.org/html/rfc4193#section-3 and https://tools.ietf.org/html/rfc4193#section-8.
Thanks

Did I miss something?!?

@perillo sorry it was me geting these typos in :/
should I send a pull with your suggestions?

@6543 yup, please send a PR for them. Thank you @perillo for the report, thank you @6543 for all your work and looking forward to more interactions with you when you send that change.


Concepts Section Feedback

I like the section on Cell Types - are these the only cell types that can be used in RasterFrames?
CRS:
a. Is it useful to mention that GeoJSON files sometimes don't have a "crs" member and just default to 4326?
b. I don't know how hard this would be to do, but some kind of visual that shows a single region reprojected to a couple different CRS and demonstrates how different projections distort the image would be interesting to me
It would be cool to have visuals showing scene, extent, tile, etc. and how they are related
The difference between extent and projected extent isn't totally clear to me
Projected raster - I don't think the "raster" concept is introduced yet. Might be useful to talk about that briefly first.

Some very good ones here. Credit @courtney-whalen
I would tend to shy away from generating our own fundamental cartography/gis content like 2b, but it could be useful to try to find some links. This is maybe too basic but easy to understand and light hearted, no trig https://geoawesomeness.com/best-map-projection/
Our this is pretty good in term of principles and viz . https://gisgeography.com/map-distortion-tissots-indicatrix/

I will break out item 3 it is a great idea but a heavier lift. Something quite similar was mentioned in #250

Addressing the rest in #249


Add python bindings based on Pyside2

This PR will add python bindings for rviz using Pyside2 for a couple of classes. The goal is to make rviz usable as a qwidget in python.
Some design decisions:

Why Pyside2 and not SIP? Honestly, documentation for Pyside is much better and resources are easier found. Plus the license of Pyside is much more business friendly.
Why have class VisualizerFramePy? It seems super complicated to wrap WeakPtr, or at least I could not get it to work. So VisualizerFramePy has the ROS Interface integrated and one does not need to pass it in. Open to help here.
This PR will not target Windows. I just do not have the knowledge. But it should be possible to do the generation for Windows as well.
We are using this in a proof of concept project only - so testing will be limited.

Open points:

[ ] Restructuring and tidying up
[ ] Documentation on how to use
[ ] Add more necessary classes
[ ] Testing
[ ] Add correct dependencies to package.xml
[ ] Add pyside and shiboken to rosdistro systemdeps

This PR should wait for the conclusion of https://github.com/ros-visualization/python_qt_binding/issues/114 before proceeding.  But I just wanted to make one point:

This PR will not target Windows. I just do not have the knowledge. But it should be possible to do the generation for Windows as well.

Unfortunately, Windows support must get completed before we can consider merging this PR.  Windows is one of our Tier-1 platforms, and so features that go in must also support it.  Hopefully when we resolve the issue above this will become easier.

Unfortunately, Windows support must get completed before we can consider merging this PR. Windows is one of our Tier-1 platforms, and so features that go in must also support it. Hopefully when we resolve the issue above this will become easier.

Okay. Problem is, I am not really equipped to test this on Windows, as I am not to familiar with ROS2 on Windows. It would cost me a serious amount of time to do that. Generally, it would only be a thing of getting the correct paths for shiboken and pyside Libs and Execs into CMake - I think I can manage that. There is a python script for Pyside by Qt that is thought for Windoof and Linux to extract the necessary info and an example CMakeLists.txt. But testing this on Windows and managing the dependencies plus doing potential changes is something I cannot do right now.
I see 2 options:

Move rviz_python_bingings package to a seperate repo - and think about merging it in here later. At least people can start using it.
Someone with time and a ROS2 on Windows setup helps with testing and implementing potential changes.

@clalancette
Okay, I have added intial Windows support -- to the best of my knowledge based on the examples provided by qt -- not tested.
Windows Dependencies: People will need to install pyside2 shiboken2 and shiboken2-generator from qt's release repository using pip. Instructions in readme. They will also need to add the installed dlls to PATH. If not we'll need to copy them to CMAKE_INSTALL_PREFIX/lib in a deployment step - this is kinda shady. Any opinions here?
pip install \
    --index-url=https://download.qt.io/official_releases/QtForPython/ \
    --trusted-host download.qt.io \
    shiboken6 pyside6 shiboken6_generator

Does anyone have a windows setup to test this? It is really hard to set this up in my company due to IT restrictions...

This depends on https://github.com/ros/rosdistro/pull/34243. Once merged checks should pass.


evaluator: field reference causes error "cannot add field field: was already used"

What version of CUE are you using (cue version)?

$ cue version
cue version v0.9.0-alpha.3

go version go1.22.2
      -buildmode exe
       -compiler gc
  DefaultGODEBUG httplaxcontentlength=1,httpmuxgo121=1,tls10server=1,tlsrsakex=1,tlsunsafeekm=1
     CGO_ENABLED 1
          GOARCH arm64
            GOOS darwin
             vcs git
    vcs.revision 92bc0b7aa6d53f9c655840204ee9c55e25e275a4
        vcs.time 2024-04-30T10:18:11Z
    vcs.modified false

Does this issue reproduce with the latest stable release?
yes
What did you do?
ran the following test:
exec cue vet -c example.cue

-- example.cue --
package example

#Task: X={
	if X.field != _|_ {}

	...
}

#A: #Task & {
	$type: "a"

	field: {}
}
#B: #Task & {
	$type: "b"

	field: {}
}
#C: #Task & {
	ref: #A | #B
}

a: #A & {}
c: #C & {
	ref: a
}

What did you expect to see?
a passing test
What did you see instead?
a failing test:
> exec cue vet -c example.cue
[stderr]
c.ref: 2 errors in empty disjunction:
c.ref: cannot add field field: was already used:
    ./example.cue:12:2
c.ref: cannot add field field: was already used:
    ./example.cue:17:2
[exit status 1]
FAIL: /var/folders/xj/q6k0bchn5414yzgryc_m1hgw0000gn/T/testscript2396058271/repr.txtar/script.txtar:1: unexpected command failure
error running repr.txtar in /var/folders/xj/q6k0bchn5414yzgryc_m1hgw0000gn/T/testscript2396058271/repr.txtar

Likely a duplicate of #2337, but having another test case is good, similar to #2595.


There is no csv file uploaded .could you please upload the csv file

Okay will add dataset link (csv file) in the readme file

added the dataset in the readme file you can check it now
Dataset: https://www.kaggle.com/datasets/johnsmith88/heart-disease-dataset


Fix specs on win32

This either fixes or disables specs that don't succeed on win32.
With this and #8668 win32_std_spec.cr should be all green :+1:
Finished in 6.84 seconds
4920 examples, 1 failures, 5 errors, 13 pending

Rebased to squash fixups.


How can I change the base color configuration ?

components.json
How can I change the baseColor ? If I try to change it nothing happens
  "$schema": "https://ui.shadcn.com/schema.json",
  "style": "default",
  "rsc": true,
  "tailwind": {
    "config": "tailwind.config.js",
    "css": "app/globals.css",
    "baseColor": "gray",
    "cssVariables": true
  },
  "aliases": {
    "components": "@/components",
    "utils": "@/lib/utils"
  }
}

P.S I changed baseColor in components.json

The baseColor has two functions:

when the cssVariables is set to false it's used to change the colors in the component's classNames
when the cssVariables is set to true it's used to generate the css variables inside the globals.css file

If you don't use the cssVariables when you add a new component, its base color will be the same of the one inside the configuration file. So if you want to change the base color you could just reinstall all the components (remember to add the --overwrite option).
Otherwise, if you need the css variables, you must change the css file manually or run the init command again.

Neutral
Gray
Zinc
Stone
Slate

How to add color's other than the options provided such as Green, Blue, Yellow, etc.., while installing or later

@bm9avan inside your components.json file,  do you have cssVariables: true set?
If so, you need to adjust your main css file (with @layer base { ... inside).
Then you go to https://ui.shadcn.com/themes and click "Customize" and select a color. Next to the customize button, click "Copy code" and replace it with the layer in your css file.


pom.xml: java.version & maven-compiler-plugin

I. java.version
java.version is standard property.
In Maven context it's contain Java Runtime Environment version which was used to run Maven itself.
Override it in pom.xml - it's very common mistake.
I suggest to use jdk.version instead (and use this new jdk.version property in the maven-compiler-plugin configuration )
As an example of why this plays a role, take a look at the following maven-enforcer-plugin config example (which is also good to have in pom.xml, actually ):
<plugin>
  <artifactId>maven-enforcer-plugin</artifactId>
  <version>${plugin.enforcer}</version>
  <executions>
    <execution>
      <id>enforce-maven</id>
      <goals>
        <goal>enforce</goal>
      </goals>
      <configuration>
        <rules>
          <requireJavaVersion>
            <message>Current JDK version ${java.version}: should be ${jdk.version}, or higher!</message>
            <version>${jdk.version}</version>
          </requireJavaVersion>
        </rules>
      </configuration>
    </execution>
  </executions>
</plugin>

II. maven-compiler-plugin optimize
optimize configuration option deprecated long time ago:
https://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html#optimize
Is there any particular reason for using it? If it so - need a comment which describe it.
Otherwise, it is better to delete it from pom.xml because it is confusing.

Thanks @Gmugra ! There is no particular reason for the pom being as it is - I think. It's been adjust by various people over the years. Feel free to submit a PR :)


Issue #673, #647 Integration tests use specific binary and run with embedded bundle

Addresses Issue #673 so the tests work with embedded bundle
Updates how we pass arguments to make integration.
User can specify the location of the binary they want to use with tests: Issue #647.

More details:

Unless the --bundle-location flag is used, the bundle is assumed to be embedded.
Arguments pertinent to integration tests are now passed via Go flags. The following 4 are defined:

--bundle-location: default is embedded
--bundle-version
--pull-secret-file
--crc-binary

The crc binary is assumed to be called crc. The way the binary is specified for the tests is by prepending PATH with the user provided binary location. The reason for not wrapping all crc commands is greater transparency/readability: this way the commands will be in plain view in the feature files.

retest this please


feat: Expose getDefaultIntegrations to use in other sdks

In sentry-cordova we need to be able to overwrite/use the browser default integrations so we need to expose them. Same is probably valid for sentry-electron later.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1393      +/-   ##
==========================================
+ Coverage   84.77%   84.81%   +0.04%
==========================================
  Files          31       31
  Lines         742      744       +2
  Branches      110      110
==========================================
+ Hits          629      631       +2
  Misses        109      109
  Partials        4        4

Impacted Files
Coverage Δ

packages/node/src/index.ts
100% <100%> (ø)
:arrow_up:

packages/node/src/sdk.ts
90% <100%> (+2.5%)
:arrow_up:

Continue to review full report at Codecov.

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


YouTube video as background

I was wondering if it's possible to use YouTube videos with this library?

Hi @rickerd, the plugin is only made for self-hosted videos. When I made it there were other libraries for using YouTube, and I'm sure they're probably still around.


Switching to PCL IRIs

We should switch IRIs, which are currently based on BDS IDs, to one that follows OBO patterns.
To do this, we will use PCL IDs -> PCL_XXXXXXX
@hkir-dev to look at what is needed to be done for this.
BDS ID will then be moved to skos:prefLabel
Current prefLabels can be removed as they are already synonyms

We need a PCL ID range and a stable IRI mapping approach.
Stable IRI mapping example for range 0010000 - 0018500:
0010000 - 0010100  : helper entities (Cluster, taxonomy, has_age, has_sex, ...)
0010100 - 0010400  : mouse classes (CS201912132_72 -> PCL_0010172)
0010400 - 0010700  : human classes (CS201912131_104 -> PCL_0010504)
0010700 - 0011000  : marmoset classes (CS201912132_72 -> PCL_0010772)
0011000 - 0012500  : other species (up to 10) classes, based on taxonomy.yaml order
0012500 - 0012900  : mouse indvs
0013300 - 0013700  : human indvs
0014100 - 0014500  : marmoset indvs
0014500 - 0018500  : other species (up to 10) indvs

Plan:
propose deprecate all current terms in pCL and Shawn will add ‘term replaced by’ & comments to map to our terms

Switched


Modernize and refactor stackstorm.js

This PR breaks up src/stackstorm.js into src/stackstorm_api.js and src/stackstorm.js (which is now just a wrapper).
Additionally, the stackstorm_api.js module is refactored into a StackStormApi JS "class", and the closed over global functions in the original anonymous closure have been broken out into their own constituent public methods.
Notes
The stop function was changed slightly. Instead of opening a new connection to ST2's event stream and then immediately closing it:
    api_client.stream.listen().then(function (st2stream) {
      st2stream.removeAllListeners();
      st2stream.close();
    });

it now saves the event stream handler and uses that to close and stop the event stream handler:
  if (self.st2stream) {
    self.st2stream.removeAllListeners();
    self.st2stream.close();
  }

Before, authenticate() was run conditionally:
  if (env.ST2_API_KEY || env.ST2_AUTH_TOKEN || env.ST2_AUTH_USERNAME || env.ST2_AUTH_PASSWORD) {
    // If using username and password then all are required.
    if ((env.ST2_AUTH_USERNAME || env.ST2_AUTH_PASSWORD) &&
        !(env.ST2_AUTH_USERNAME && env.ST2_AUTH_PASSWORD && env.ST2_AUTH_URL)) {
      throw new Error('Env variables ST2_AUTH_USERNAME, ST2_AUTH_PASSWORD and ST2_AUTH_URL should only be used together.');
    }
    authenticated = authenticate();
  }

And now it that check is separated from the authenticate() call:
if (env.ST2_API_KEY || env.ST2_AUTH_TOKEN || env.ST2_AUTH_USERNAME || env.ST2_AUTH_PASSWORD) {
    // If using username and password then all are required.
    if ((env.ST2_AUTH_USERNAME || env.ST2_AUTH_PASSWORD) &&
      !(env.ST2_AUTH_USERNAME && env.ST2_AUTH_PASSWORD && env.ST2_AUTH_URL)) {
      throw new Error('Env variables ST2_AUTH_USERNAME, ST2_AUTH_PASSWORD and ST2_AUTH_URL should only be used together.');
    }
  }

I compensate for this in stackstorm.js almost perfectly:
module.exports = function (robot) {
  var stackstormApi = new StackStormApi(robot);

  return stackstormApi.authenticate().then(function () {

But, that isn’t quite good enough for some of the tests. So in order to test things like the environment variables ST2_API and HUBOT_2FA being handled, you now have to [specify proper values for ST2_AUTH_URL, ST2_AUTH_USERNAME, and ST2_AUTH_PASSWORD]:
test-st2bot-envvars.js:69
    restore_env = mockedEnv({
      ST2_API: 'https://nonexistent-st2-auth-url:9101',
      ST2_AUTH_URL: 'localhost:8000',
      ST2_AUTH_USERNAME: 'user',
      ST2_AUTH_PASSWORD: 'pass'
    });

test-st2bot-envvars.js:102
    restore_env = mockedEnv({
      ST2_AUTH_URL: 'localhost:8000',
      ST2_AUTH_USERNAME: 'user',
      ST2_AUTH_PASSWORD: 'pass',
      HUBOT_2FA: 'true'
    });

Coverage Differential
Test coverage went from:
---------------------|----------|----------|----------|----------|-------------------|
File                 |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
---------------------|----------|----------|----------|----------|-------------------|
All files            |    89.35 |    87.13 |    87.76 |    89.05 |                   |
 src                 |    62.16 |    60.55 |    63.33 |    62.16 |                   |
  stackstorm.js      |    62.16 |    60.55 |    63.33 |    62.16 |... 02,403,404,437 |
...
---------------------|----------|----------|----------|----------|-------------------|

to:
---------------------|----------|----------|----------|----------|-------------------|
File                 |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
---------------------|----------|----------|----------|----------|-------------------|
All files            |    89.58 |    87.28 |    87.76 |    89.29 |                   |
 src                 |       65 |    61.95 |    63.33 |       65 |                   |
  stackstorm.js      |      100 |      100 |      100 |      100 |                   |
  stackstorm_api.js  |    63.73 |    61.95 |    60.71 |    63.73 |... 52,454,455,457 |
...
---------------------|----------|----------|----------|----------|-------------------|

(unchanged coverage results omitted)
Further Work
Now that StackStormApi has been turned into a JS class, it should be easier to write unit tests for each of its public methods, and to update it to use new style JS classes with the class and extends keywords.

[ ] Add unit tests for StackStormApi public methods
[ ] Move things into the stackstorm.js wrapper

[ ] the process.on unhandled rejection handler registration
[ ] the robot.error handler registration
[ ] the robot.respond handler registration
[ ] the robot.router.post endpoint (see #133's stackstorm.js:51)
[ ] any other endpoints (I expect at least one to be added for ChatOps authentication as part of ChatOps RBAC)

[ ] Decouple stackstorm_api.js from stackstorm.js a bit, similar to how it is implemented in #133

[ ] have StackStormApi inherit from EventEmitter
[ ] emit st2.chatops_announcement and st2.execution_error events (consumed in stackstorm.js:68 and stackstorm.js:72, respectively)

[ ] also possibly emit/handler an event for two-factor authorization requests (see HUBOT_2FA)

[ ] Return the promises in loadCommands, start,

Note that I disagree with how commandFactory was turned into an event emitter (stackstorm.js:84) so I'm not including it in this checklist.

GitHub and Travis aren't talking to each other properly right now.
The Travis CI results are here: https://travis-ci.org/StackStorm/hubot-stackstorm/builds/560748297. The build is passing. ✅


[Horovod] ModelCheckpoint and EarlyStopping CBs hit errors with Torch 1.13+

Bug description
Since PyTorch 1.13, we have observed that ModelCheckpoint and EarlyStopping callbacks would hit an undefined symbol error with Horovod strategy.
Details and examples are in https://github.com/horovod/horovod/commit/e392eb9daf09d2c987497686ae82b62c924c9c27
It is reproducible with Torch 1.13 alone, but I think underneath, reduce_op fromDDP should be not mixed with Horovod. This line in PTL hits the error.
https://github.com/Lightning-AI/lightning/blob/master/src/pytorch_lightning/strategies/horovod.py#L179
How to reproduce the bug
from torch.distributed import ReduceOp

op = None
op in (ReduceOp.SUM, None)

Error messages and logs
        Traceback (most recent call last):
            File "<stdin>", line 1, in <module>
            TypeError: __eq__(): incompatible function arguments. The following argument types are supported:
            1. (self: torch._C._distributed_c10d.ReduceOp, arg0: c10d::ReduceOp::RedOpType) -> bool
            2. (self: torch._C._distributed_c10d.ReduceOp, arg0: torch._C._distributed_c10d.ReduceOp) -> bool
        Invoked with: <torch.distributed.distributed_c10d.ReduceOp object at 0x7fba78c9e0b0>, None

Environment

#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10): 1.13+
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
Comments and suggestions are welcome.
cc @awaelchli

Thanks @chongxiaoc! Were you able to fix the issue by replacing the reduce op with the correct one from Horovod?

Hi @awaelchli , I believe there was a historical reason of adding torch DDP sum op into horovod part code.
From this commit:
https://github.com/Lightning-AI/lightning/commit/b37b58a73e74e173f4efc674a9ac2d4bb978e422

Thank you for digging deeper!


Include TTL in DomainRecord update request

Was missing from #174

@yassirh could you perhaps merge this? 😄

Are there existing tests for this method? If so, please update them.

If not, let me know, and I'll merge.

Oh, I see there are tests, but not covering this method? Please could you add one?

Yeah noticed that as well, should have updated it in the first place. Updated it now. Not used to working with phpspec, so hope everything is okay :)

@GrahamCampbell could you have another look with the added test coverage?


Auto-build container

@NeuraLegion/devops we now use this repo as the basis for the "on-prem agent" solution.
Let's publish it automaticlly to dockerhub neuralegion/repeater instead of the old one.
We can fix docs on the way.

It's already building to neuralegion/agent. Let's wait for some stabilizing and merge it with repeater

@rielas this is stable, please consult with @derevnjuk on changes for example files if needed, anyhow, this is ready to move.

It's already in repeater repo


where are product endpoints

Hi @ZelvaJan,
I am exploring the project and read the documentation, I have few question:

I could not find any endpoints for products.
there seems to be lots of endpoints in the app, like navigation list, shop etc. but only few are listed in documentation.
I am planning to build a backend api on top of laravel, but could not find enough information on how to start.
the version 1.0 on play store does not works and give internal server error, but the v2.0 is working. but the v1.0 source code on github is available. from where can i find the v2.0 source code?

I also tried to contact on email mentioned in contact section, but did not get any response from there. Please give me answers to these questions asap because I have to develop this project for someone.

Partially solved by connecting to apiary mock server https://bfashionapibfversion.docs.apiary.io/# which give you some idea about endpoints.
I just helped to develop the android v1.0 application, i am not able to help with business contact or with v2.0 source code.


Improve Jaeger Python traces (more spans/snapshot tag).

Adds more spans and metadata.
Tested in internal repo.

Not sure what's going on with CI here but lgtm


vintage iphones are not necessarily supported

This comes from a now-deleted comment on the recent HN thread:

Why is the iPhone 5S listed as supported? I don’t know that I would described it as a supported device (even if its most recent OS gets an occasional critical security update).
Apple lists the 5S as “vintage.” They won’t repair it, they won’t even replace the battery.

We already mark 5S as Discontinued, but perhaps we should also classify a separate Hardware Support tier? Our definition of support is "does this iphone receive critical security updates", but hardware support for repairs is another definition that users care about.

Good idea, perhaps with a custom column. Do you know a good source where this is documented?

https://support.apple.com/en-us/HT201624#iphonevintage currently lists the following:

iPhone 4 (8GB)
iPhone 5
iPhone 5S
iPhone 6
iPhone 6 Plus
iPhone 6s (32GB)
iPhone 6s Plus (32GB)
iPhone SE

But this is already tricky, since there might be the same model marked as vintage or not depending on the variant. 4-8GB is vintage, but 4-16GB or 4-32GB variants aren't. We could mark them as vintage, but clarify in the description perhaps.


netsparker2ivil script first import

I decided to put the ivil importer here though scan script is still ugly and not ready for production yet

@arkenoi can you fix these errors:
t/03_license_applied.t .. 89/?
#   Failed test 'Author Alex Smirnoff is duely credited in file: ./bin/netsparker2ivil'
#   at t/03_license_applied.t line 163.
#                   '# Copyright 2014, 2015, 2016 ArkanoiD, Frank Breedijk, Glenn ten Cate, Artien Bel
# '
#     doesn't match '(?^:Alex Smirnoff)'
#   Failed test 'Copyright year of file: ./bin/netsparker2ivil'
#   at t/03_license_applied.t line 169.
#          got: '2014'
#     expected: '2016'

If they are fixed I can accept the PR.

fixed

fixed


fix(backend): indicate "greater than or equals" limit (10K) count in lineage search results

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Will go ahead and close for now. Please reopen to submit this PR!


Date Time Picker: Time zones may need to be at a higher granularity than 1 hour

There are time zones at 30 minutes and even 15 minutes bounds: https://en.wikipedia.org/wiki/List_of_UTC_time_offsets

Yup I'd agree, historically we (Tier3 > CTL) have only supported hour break outs but I'm not opposed to adding the additional "missing" ones

With x4 the existing granularity, a different paradigm than a <select> with a tons of options might have to be investigated, too...

is it really x4? or are there just a couple? a quickly looking at the list it seems like there would only be 10 maybe 12 more...

Yeah, it could as well be a heterogenous list of just all existing time zones on the planet.

Is there a customer use case for this currently?


Feature Request

Hi There, can you please include manual subtitle search option. It's like in MX Player. Please this would help us a lot finding a more suitable and synced subtitle instead of default or none. Renaming is quite cumbersome. I m using the Archos video player for my 55inch 4K TV.

You expect us to pay for the service for which support like this is given????? Thank you


Update to Node 8

Since it looks like our previous development has been happening with Node 8 anyway, update the existing documentation to make that official. Also add an .nvmrc for convenience.

Codecov Report

Merging #251 into main will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #251   +/-   ##
=======================================
  Coverage   59.26%   59.26%
=======================================
  Files          95       95
  Lines       11767    11767
=======================================
  Hits         6974     6974
  Misses       4793     4793

Continue to review full report at Codecov.

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


"Images Used" profile metric not showing correct count

Summary:
My profile in the Commons app shows the "Images Used" metric as 0/5 despite the fact that a few of my photos are used in Wikipedia articles (listed below).
Photos

https://commons.wikimedia.org/wiki/File:Golden_Hill_corner.jpg
https://commons.wikimedia.org/wiki/File:Nebelung_cat_looking_out_door.jpg
https://commons.wikimedia.org/wiki/File:Euclid_Trolley_Station.jpg

Steps to reproduce:

Check profile for images used in Wikipedia articles( mine are listed above).  Listed in "file usage on other wikis" section.
Compare number of images against Commons profile.

System logs:
I neglected to capture logs at the time but can reproduce if they are desired.
Device and Android version:
Android version: 12
Device manufacturer: Google
Device model: Pixel 4a
User name: Mds08011
Commons app version:
App version name: 3.1.1~1c9267ca0
Screen-shots:

Would you like to work on the issue?
No.

Thanks for reporting this! The same issue occurs for me on current master.

Hey @misaochan, I am able to reproduce this.
I think the problem is from the server side. As the API returns the JSON in an invalid format due to which all values are initialized as zero.

@Pratham2305 thanks for investigating! Could you link to a sample query string and the response, please?

Could you link to a sample query string and the response, please?

Query:
https://urbanecmbot.toolforge.org/commonsmisc/feedback.py?user=Azanti
Response:
Content-type: application/json
()
{"status": "ok", "uniqueUsedImages": 3, "articlesUsingImages": 7, "database": "commonswiki", "deletedUploads": 15, "featuredImages": {"Quality_images": 0, "Featured_pictures_on_Wikimedia_Commons": 0}, "labs": false, "thanksReceived": 0, "user": "Azanti", "imagesEditedBySomeoneElse": 5}

I think the first two lines in the response are unnecessary.

Also, are you interested in working on this?

Yes, but I am not sure about the server-side so should I just make changes in the above response to extract the valid JSON?

Yes, you can try that and post the results. If it works, please feel free to submit a PR. :)


Added the ability to configure keys using Environment Variables

Added the ability to use environment variables for configuration.
Also I added skip event checking, because otherwise the program does not allow me to use a gamepad or mouse for binding, although I have bound the corresponding buttons.

Could you reopen? It sounds like a useful addition

It would be great for README to point in the right direction on how to configure this for a desired key

Hi, sure. I want to do some tests, yesterday when I tried to run it for some reason it didn't work.


Update image to the latest version

Here I think you could update to the latest available version as specified in the documentation:
https://github.com/emqx/emqx-operator/blob/main/docs/en_US/getting-started/getting-started.md#deploy-emqx-5

Hi @alessandroros Thank you for your contribution.
Could you please format you commit message to pass the git lint check, you can check the CONTRIBUTING document


Disable broken AWS ALB ingress status check

For now, the AWS ALB ingress status check is broken, which is described in this issue https://github.com/kubeflow/testing/issues/779,
Disable the status check until above issue fixed
/cc @Jeffwan @yanniszark

Latest stable version is 12.0. I would suggest to upgrade k8s python client and build new image.
Removing the logic means we don't even check entrypoint which make test case not that helpful.

Send another PR to fix the issue
https://github.com/kubeflow/kfctl/pull/432

/close
close the PR in favor of  https://github.com/kubeflow/kfctl/pull/432


Pine64 A64+ board support, based on Armbian/Bullseye

Pine64 A64+ board support

Believe there should be better to use additional action for pine-a64-plus/overlays/armbian/boot/boot.scr generation rather than put the generated file as is -- it may differ from the source due human mistake.


Target profile page rewrite: Replace uses of OtTable with OtTableRF

Replace uses of OtTable with OtTableRF throughout the code base.

Closing this ticket as when I run a search for OtTable in the platform-app repo, it returns no results.
@mirandaio, please re-open if there are other instances that need to be changed


Multiple call to .then() will trigger the request multiple times

Environment
Knex version: master
Database + version: not relevant
OS: not relevant
Bug
According to the Knex documentation about .then():

Coerces the current query builder chain into a promise state, accepting the resolve and reject handlers as specified by the Promises/A+ spec. As stated in the spec, more than one call to the then method for the current query chain will resolve with the same value, in the order they were called; the query will not be executed multiple times.

Important part: the query will not be executed multiple times.
However, any call to .then() on a Knex query will re-trigger the request: https://github.com/knex/knex/blob/d463284349c0a653bd9c953be8e911ad654aeb01/lib/builder-interface-augmenter.js#L23-L24
This can be proved by listening to the query hook;
 knex
    .on('query', (query) => {
      console.log(query, "started");
    })
    .on('query-response'=>{
      console.log(query, "ended");
   });

const query = knex('table').select('foo');
// query will run once
query.then(() => {});
// query will rerun from scratch (the call to .then() is implicit through await)
await query;

I wasn't sure if I should open this here or on the documentation, since strictly speaking, the documentation is incorrect. However, the behavior from the documentation is also what I'd expect: calling .then() multiple time on the same promise shouldn't rerun the promise, and should instead return the "cached" result if it exists.
An easy workaround is to capture the "returned" promise instead for reuse, but it seems this shouldn't be necessary according to documentation.
const query = knex(...);
const cachedQuery = query.then(r => r);

// This will query once
cachedQuery.then(() => {})
// This won't query again, and will reuse the first results if already available, or block until they become available.
await cachedQuery();

Think I just got tripped by the same bug. Although I am using the async/await with transaction instead of .then`
const trx = await knex.transaction()
const statement = trx('user').insert({ name: 'hello' })
await statement;
await statement; -- will execute the statement again.


Can we contribute for translate to Spanish?

Is possible contribute for translate to Spanish?

Hi @johnfelipe ,
thanks for the suggestion and your offer.
I'm not sure what you would like to translate to Spanish:

editor UI (https://maputnik.github.io/editor/)?
documentation (e.g. the wiki https://github.com/maputnik/editor/wiki)?
website (https://maputnik.github.io/)?

Regarding the editor:
The whole style specification is in English and part of the UI (JSON editor, naming of the fields, ...) and I don't think it makes sense to translate these parts because it might be not clear afterwards to which part of the spec something refers.
If we keep everything related to the spec in English, there is not much left to translate.
Also there is currently no functionality implemented to switch the language.

Maybe has sense to translate the helptext when you pass the cursor over fields in the editor. I could collaborate for spanish too.
But if there's not a way to switch...

Maybe has sense to translate the helptext

These are also taken from the style spec.
https://www.npmjs.com/package/@mapbox/mapbox-gl-style-spec

I could collaborate for spanish too

@11sma thanks for your offer.

Closing for now, I generally don't see a lot of value in translating the UI, docs might be a different case, but also there I'm not sure there's a real value.


Travis Setup (and some doc updates)

/cc @jayzes

We should add the "Code of Conduct" block to the README too (like https://github.com/ello/streams#code-of-conduct) - looks good otherwise though!


Using custom request

Hi, Is there a way to access the request session that makes the call to salesforce in a view?
I am trying to use this package with OpenId credentials.  Ultimately would like to do something like this:
def my_accounts(request):
    session = requests.Session()
    session.headers.update(...)
    Account.objects(session=session).all()

Thanks

I’m not fully up to date on what OpenID integration SFDC provides, but I’m not sure this would be a supported use case.
You’d still want to associate the connection with an “Application” in Salesforce, so you’d still end up getting OAuth keys generated.
Also, I personally prefer relying on the security code instead of whitelisting IPs, and I’m not sure how this would work if you are skipping the Salesforce authenticator.
@hynekcer may have deeper insight...

Thanks for the quick response.
I am using this authentication flow here
But I'm not suggesting managing the authentication flow in this package.  I'm just looking for a way to update the request session from a view post authentication.
Currently I'm using:
from simple_salesforce import Salesforce
def my_accounts(request):
    sf = Salesforce(instance=request.user.instance, session_id=request.user.session_id)
    sf.query("SELECT ...")
    ...

But using simple-salesforce means writing lots of SOQL.
As a backup plan I may go this route
from simple_salesforce import Salesforce
def my_accounts(request):
    sf = Salesforce(instance=request.user.instance, session_id=request.user.session_id)
    accounts = Account.objects.all()
    sf.query(accounts.query)
    ...

Interesting...I have an idea but I'm not near my dev workstation, so you might have to experiment a bit.
The requests session object in question is deep inside the RawConnection object that simulates the Python DB API. I think you can do something like this:
from django.db import connections

conn = connections['salesforce']
conn.sf_session.headers.update(...)

This should work fine within a view or something, since the auth/connection happens at the beginning of the Django run by default. There's some experimental and edge features in django-salesforce (like lazy loading and dynamic authentication) that probably would clobber this header, so that's worth remembering if your usage becomes more sophisticated.

This is exactly what "dynamic authentication" does:
connections['salesforce'].sf_session.auth.dynamic_start(access_token, instance_url=instance_url)
# some work...
connections['salesforce'].sf_session.auth.dynamic_end()
# forgot the access token and optionally use the auth values that were valid before .dynamic_start(...)

It is regularly tested by tests/test_dynamic_auth/tests.py.
It should be thread safe to use different tokens/organizations in different in individual threads. That was tested manually four years ago, but no automated test exists.

Three years ago I wrote also a modified general SalesforceAuth that can work with a custom authentication class configurable by DATABASES settings, not on only the default SalesforcePasswordAuth or a direct work with tokens by auth.dynamic_start(), e.g. a class for interactive authentication by "Force.com CLI" by web browser, useful for development with Salesforce, without saving a password anywhere. It is not a part of production repository still and it is little obsoleted because SFDX is probably a better option now. If that solution would be committed then the dynamic auth probably will have to be enabled in settings to work.

Thanks @philchristensen and @hynekcer, that does trick.


Remove-BcEnvironment status is null

Describe the issue
When using Remove-BcEnvironment we get this output
The property 'status' cannot be found on this object. Verify that the property exists.

Scripts used to create container and cause the issue
Remove-BcEnvironment -bcAuthContext $authContext -environment $Environment

Full output of scripts
Submitting environment removal request for BusinessCentral/SandboxNextVersion
Environment removal request submitted
Removing..Remove-BcEnvironment Telemetry Correlation Id: 14fb5bfe-86ed-4dfc-a72a-7be475aec837
##[error]The property 'status' cannot be found on this object. Verify that the property exists.
...

Additional context
I think this line does not have a status at some point
https://github.com/microsoft/navcontainerhelper/blob/f5c4cd7165a69e60e2e9e00d0fdafaeb65a976ce/Saas/Remove-BcEnvironment.ps1#L63

Could you share the full output of the job, either here or in an email to freddyk at microsoft dot com?

Hi Freddy,
sent you an email with detailed output.

The new implementation of DELETE (softDelete) apparently breaks callers using the old API Version.
You can fix this (until the next version) by adding -apiVersion '2.18' or -doNotWait to the Remove-BcEnvironment call.
I will change the default apiVersion in ContainerHelper

Hi Freddy,
thank you!
You had a small typo in your comment - we need to set apiVersion to "v2.18" instead of "2.18".
Remove-BcEnvironment -bcAuthContext $authcontext -Environment  -apiVersion 'v2.18'
worked. Thanks!


Help Wanted Form submission

Need to have a form where users can post help wanted.  This should send a "post" event to
Post: /portal/commands/v1/profile/posting
Header: Content-Type: application/json
Header: access-token:
Body:
{
      "meta": <object of meta stuff the client might want to keep>,
      "game_id": <string name of the game, like "world of warcraft">,
      "details": <string description of the work that needed to be done>,
      "conf_type": "XRP",
      "conf_amt": <string formatted number, this is the "upfront" payment amount>,
      "comp_type": "XRP",
      "comp_amt": <string formatted number, this is the "job done" payment amount>,
      "bonus_type": "XRP",
      "bonus_amt": <string formatted number, this is the "If we win vs loses" payment amount>,
      "bonus_req": <string description of what is needed to get the bonus payment>,
      "user_count": <integer number of users this job will require>"
      "type": "individual" or "guild"  (are you looking to hire some rando folks or a rando guild)
}

Complete with commit: 521e25c04e78c6a77ce37e3e3d945724290cdf18


Update libstorage documentation to reflect new company name

The team name is now "{code} by Dell EMC" instead of "EMC {code}"

Ha - I notice that even today both rexray and libS RTD have an EMC {code} reference in the nav header, all the way on the right, that link to a github.io page. Still needs fixed! Surprised we've missed that.

Perhaps this issue is a duplicate?

Closing.


Update sqs to 2.29.39

About this PR
📦 Updates software.amazon.awssdk:sqs from 2.29.31 to 2.29.39
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk", artifactId = "sqs" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk", artifactId = "sqs" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #132.


Request for auto select single item on lookup

When I type into the lookup associated input and there is only one item to select I still have to select it in the menu.
Is it an option to have an 'auto-select' option in the lookup component that automatically select an item when the lookup options (loadOptions) return only one value?

I do not understand what you mean. And maybe you do not understand what I mean to.
If you look at the example and you type Option 1 then you get a menu with only one option. I would be nice if this is automatically selected as the option and you do not have to click it.

When you type your filter the options function gets called. Normally, that function just returns the array with one option in it but you can also set a value for example before returning the array.

Is this still relevant?


Update typesafe:config to 1.3.3

Updates com.typesafe:config from 1.0.2 to 1.3.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

I'm fairly sure typesafe-config preserves bincompat, especially given it's a Java library.  If no one chimes in I'll merge on Monday.


Card Cloner crashes client

LaserIO version
1.6.7
Minecraft Version
1.20
Forge Version
No response
Modpack & Version
Direwolf20 1.20
Do you have optifine installed?
No
Describe the issue
Card Cloner crashes client
Steps to reproduce
Craft the Card Cloner
Place on hot bar
Hover over the Cloner
Press Shift
Client crashes after a second or two.

Expected behaviour
Nooooo crashy crashy.  I actually expected to see how to use the card, since I wanted to copy/paste RF card settings.
Screenshots
No response
Log files
https://pastebin.com/raw/RAi3BQ7G
Additional information
No response

I just happened to catch this in the FTB launcher
[20:06:58] [Render thread/WARN] [co.re.re.sc.gr.st.ItemGridStack/]: Could not retrieve item tooltip of laserio:card_cloner
Not sure what triggered that.

also experienced this in All The Mods 9
did you have card overclockers in the card you cloned from?
LaserIO version
1.6.7
Minecraft Version
1.20.1
Forge Version
47.2.16
Modpack & Version
All The Mods 9 - 0.2.32
Do you have optifine installed?
No
Describe the issue
Playing on multiplayer server, not on LAN.
Log files
https://pastebin.com/Py2iMG5N

That IS a possibility?  I did shift-click on one of the power cards that were configured with 4 overclockers, tried going to another card but I didn't notice anything.  I then saw the popup about shift, acted on it, and the client crashed.  I don't think the server said anything, but didn't think to look.

Got the same error in crash log.

Closing.  Already in #220


refactor linux script

This should get the linux side going.
Notes:
I commented out the steamcmd detection and always download it for now.  Reason is the download_depot calls drop files in a different location depending on the steam installation.  Feel free to fix it if you want - relevant parts are commented out.
Added a check to reuse the downloaded steamcmd if it's already there.
Made the steamcmd call take the username as a param from python input.  Then the steamcmd call itself will prompt for the password, and, if needed, the steamguard code.  This will make it compatible with steamguard-via-email since you don't need to have a steamguard code in advance.
I also added a quick check to make sure the user is not root.
Needs more testing end-to-end.

also I revived the script text file because it keeps that stuff off to the side if an end user ever needs to change one of the depot commands.


Dev/tsweeney/docfix/5

Adding the skopeo-containers package to the list or required packages

Ugh, will have to do the merge tomorrow.  Need to scat now.

The file that contains /etc/containers/policy.json might be a different one on other distributions, but it's a reasonable guess.  LGTM.

Can we rebase this to avoid including the merge commit?

Rebase looks good.  Once CI comes back we're okay to merge.

@rh-atomic-bot r+ f36fee20b70c804234121721fd45ffebbcfcf94f


Invalid multibyte character encoding

Getting lots of server errors along these lines:
Exception on /z3950/search/oclc/html [GET] Traceback (most recent call last):
  File "/var/www/pybossa/env/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
  File "/var/www/pybossa/env/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
  File "/var/www/pybossa/env/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
  File "/var/www/pybossa/env/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
  File "/var/www/pybossa/env/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
  File "/var/www/pybossa/env/local/lib/python2.7/site-packages/Flask_Z3950-0.2.2-py2.7.egg/flask_z3950/view.py", line 61, in search_html
    data = dataset.to_html()
  File "/var/www/pybossa/env/local/lib/python2.7/site-packages/Flask_Z3950-0.2.2-py2.7.egg/flask_z3950/dataset.py", line 46, in to_html
    records = [pymarc.Record(data=r) for r in self.record_data]
  File "build/bdist.linux-x86_64/egg/pymarc/record.py", line 74, in __init__
    utf8_handling=utf8_handling)
  File "build/bdist.linux-x86_64/egg/pymarc/record.py", line 314, in decode_marc
    data = marc8_to_unicode(data, hide_utf8_warnings)
  File "build/bdist.linux-x86_64/egg/pymarc/marc8.py", line 27, in marc8_to_unicode
    raise UnicodeDecodeError("marc8_to_unicode", marc8, 0, len(marc8), "invalid multibyte character encoding")
UnicodeDecodeError: 'marc8_to_unicode' codec can't decode bytes in position 0-16: invalid multibyte character encoding

UnicodeDecodeErrors caught since v.0.2.3


CSS Custom Props (css variables) syntax highlighting

I would like to request a little more distinction in the syntax highlighting for css variables.

Definitely! This should be more legible. I'm happy to work on this but I'm low on time, so if I do it, it won't be for a couple of months. Open to PRs.

I've been searching for a good reference on how to select these in the settings, but I am struggling to figure it out using TextMates' documentation. Can you share the source you referenced in building this theme? Thanks!

@sdras I've got it working now, here is the regular dark theme, can you comment before i move on to the other themes?
I've opted for the purple color you used and also set it to italic when actually being used and non-italic when just being declared.
Also could you tag this as assigned to me and in progress?

@chiangs That looks really good, maybe I like the second option more. Are you working on fixing this issue? If not I would't mind taking a look.

@Arnau-Ninerola Yes I am actively working on it, I just wanted @sdras to have a look first and assign to me. I think I like the second one better as well.

@sdras PR opened


Link string doesn't appear for multiple photos post

When you create photo post, you can use only one link string, I think.
So I tried to make one link appear for multiple photos, but  no links was appeared.
Is there any way to make link or links to appear for multiple photos post?
import pytumblr

consumer_key = 'xxxxxxxxxxxxxxxxxxxxx'
consumer_secret = 'xxxxxxxxxxxxxxxxxxxxx'
oauth_token = 'xxxxxxxxxxxxxxxxxxxxx'
oauth_secret = 'xxxxxxxxxxxxxxxxxxxxx'

client = pytumblr.TumblrRestClient(consumer_key, consumer_secret,
                                   oauth_token, oauth_secret)
data = ["img1.png", "img2.png", "img3.png", "img4.png"]
# ↓this does not work.
client.create_photo(blogname, state="published", data=data, link="https://github.com")
# ↓this does not work.
links = ["https://github.com"]*len(data)
client.create_photo(blogname, state="published", data=data, link=links)
# ↓this works.
client.create_photo(blogname, state="published", data=data[0], link=links[0])

I don't believe there is a way to do this, you can't do this when creating a post on the website


[FEAT/BUG] Automatically convert arguments of path/query to string

Bug part: I was trying to do something like .add_path(['user', 123]) and it didn't raise an exception during the execution of the method but only when i called .get, which is imho late.
Feature request: Everything should be converted to str automatically (not raising error) or at least usual types like int, float. I think it should be user's responsibility to provide desired str output and everyone knows that resulting url is always str.

Nice catch! Will get to fixin'.

For now, I've only made it raise an AttributeError if it receives a list and it has something other than str inside.
I'm still thinking of a good way to implement the automatic conversion part.

In my wrapper function that i am using to bypass some problems that i encoutered (and reported) i use something as simple as [str(p) for p in path_parts].


react-app-rewired: Permission denied

Describe the bug
React-App-Rewired runs into permission denied, when running in a docker container.
This happens on start up, so you can't start the app with react-app-rewired.
(Note: react-scripts only work fine)
To Reproduce
Steps to reproduce the behavior:

take your react-app-rewired app and copy it to a docker container
run npm install
run npm start (or npx react-app-rewired start) depending on your setup (packages.json scripts section or gulpfile e.g.)
See error

node@7fdd14c9ecdf:/opt/project$ npm install

added 2471 packages, and audited 2472 packages in 36s

found 0 vulnerabilities
node@7fdd14c9ecdf:/opt/project$ npm start

><EMAIL_ADDRESS>start
> react-app-rewired start

sh: 1: react-app-rewired: Permission denied
npm ERR! code 126
npm ERR! path /opt/project
npm ERR! command failed
npm ERR! command sh -c react-app-rewired start

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/node/.npm/_logs/2021-02-09T17_23_55_570Z-debug.log

Verbose Log output
node@7fdd14c9ecdf:/opt/project$ cat /home/node/.npm/_logs/2021-02-09T17_23_55_570Z-debug.log
0 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'start' ]
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing config:load:defaults Completed in 1ms
4 timing config:load:file:/usr/local/lib/node_modules/npm/npmrc Completed in 1ms
5 timing config:load:builtin Completed in 1ms
6 timing config:load:cli Completed in 1ms
7 timing config:load:env Completed in 0ms
8 timing config:load:file:/opt/project/.npmrc Completed in 1ms
9 timing config:load:project Completed in 2ms
10 timing config:load:file:/home/node/.npmrc Completed in 0ms
11 timing config:load:user Completed in 0ms
12 timing config:load:file:/usr/local/etc/npmrc Completed in 0ms
13 timing config:load:global Completed in 0ms
14 timing config:load:cafile Completed in 0ms
15 timing config:load:validate Completed in 0ms
16 timing config:load:setUserAgent Completed in 1ms
17 timing config:load:setEnvs Completed in 0ms
18 timing config:load Completed in 6ms
19 verbose npm-session ee28113559a2a390
20 timing npm:load Completed in 12ms
21 timing command:run-script Completed in 8ms
22 timing command:start Completed in 10ms
23 verbose stack Error: command failed
23 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/index.js:64:27)
23 verbose stack     at ChildProcess.emit (node:events:379:20)
23 verbose stack     at maybeClose (node:internal/child_process:1065:16)
23 verbose stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:296:5)
24 verbose pkgid<EMAIL_ADDRESS>25 verbose cwd /opt/project
26 verbose Linux 4.15.0-88-generic
27 verbose argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
28 verbose node v15.6.0
29 verbose npm  v7.4.0
30 error code 126
31 error path /opt/project
32 error command failed
33 error command sh -c react-app-rewired start
34 verbose exit 126

Expected behavior
The app starts like on windows.
Desktop (please complete the following information):
Server: Docker Engine - Community
Engine:
Version:          18.09.6
API version:      1.39 (minimum version 1.12)
Go version:       go1.10.8
Git commit:       481bc77
Built:            Sat May  4 01:59:36 2019
OS/Arch:          linux/amd64
Experimental:     false
=> official docker hub image: https://hub.docker.com/layers/node/library/node/15.6.0/images/sha256-a65a1706f56a9ce393bb6059f49cb05bfab487226487be5da71f205eeb8c05ca
Additional context
Add any other context about the problem here.
We tried to change the permissions of the user, we tried to change the ownership of folder,
nothing seems to work.
snippet from package.json
"scripts": {
    "start": "react-app-rewired start",
    "build": "react-app-rewired build"
  },
  "dependencies": {
    "@material-ui/core": "^4.11.2",
    "@material-ui/icons": "^4.11.2",
    "@material-ui/lab": "^4.0.0-alpha.57",
    "@monaco-editor/react": "^4.0.9",
    "@st/image-viewer": "^6.4.1492",
    "@st/react-ui-components": "^1.2.60",
    "@types/redux-thunk": "^2.1.0",
    "codemirror": "^5.59.1",
    "element-resize-detector": "^1.2.1",
    "intl-pluralrules": "^1.2.2",
    "material-design-icons-iconfont": "^6.1.0",
    "monaco-editor": "^0.22.3",
    "monaco-editor-webpack-plugin": "^3.0.0",
    "pepjs": "^0.5.3",
    "react": "^17.0.1",
    "react-codemirror2": "^7.2.1",
    "react-contextmenu": "^2.14.0",
    "react-dom": "^17.0.1",
    "react-intl": "^5.10.13",
    "react-json-inspector": "^7.1.1",
    "react-loader": "^2.4.7",
    "react-redux": "^7.2.2",
    "react-reflex": "^3.1.1",
    "react-resize-detector": "^5.2.0",
    "react-router-dom": "^5.2.0",
    "react-scripts": "4.0.1",
    "reactjs-popup": "^2.0.4",
    "redux-saga": "^1.1.3",
    "redux-thunk": "^2.3.0",
    "styled-components": "^5.2.1",
    "superagent": "^6.1.0",
    "toastr": "^2.1.4",
    "web-vitals": "^0.2.4"
  },
  "devDependencies": {
    "@testing-library/jest-dom": "^5.11.8",
    "@testing-library/react": "^11.2.2",
    "@testing-library/user-event": "^12.6.0",
    "@types/element-resize-detector": "^1.1.2",
    "@types/jest": "^26.0.20",
    "@types/node": "^14.14.20",
    "@types/react": "^17.0.0",
    "@types/react-codemirror": "^1.0.3",
    "@types/react-dom": "^17.0.0",
    "@types/react-loader": "^2.4.4",
    "@types/react-redux": "^7.1.15",
    "@types/react-resize-detector": "^5.0.0",
    "@types/react-router-dom": "^5.1.7",
    "@types/superagent": "^4.1.10",
    "@types/toastr": "^2.1.38",
    "@types/uuid": "^8.3.0",
    "esm": "^3.2.25",
    "gulp": "^4.0.2",
    "npm-check": "^5.9.2",
    "react-app-rewired": "^2.1.8",
    "sass": "^1.32.2",
    "sass-loader": "^10.1.0",
    "teamcity-service-messages": "^0.1.11",
    "typescript": "^4.1.3"
  }

Any guidance in the right direction, would be appreciated! Thanks!

In your Dockerfile, use the syntax USER root on the top, or provide sufficient execute/read/write permissions to the docker-created user inside the container on the working directory using RUN chmod a+wrx .


created for Scion 2 (Kickstarter)

this was made to playtest the game..
tho there ATM no official char sheets drawn up ... so it might be totally wrong in places (and some sections have altered names)

Hi @Hotbloodhypno, because this is a separate sheet, its files need to be placed in a separate directory than Scion/, which already has the first sheet.  Scion2/ would do just fine.
Also, since there is no publicly available form of the sheet, I can accept the code into the repository for use in playtesting (and for tracking, then release down the line), but you will need to use a pro account to copy the code and put it in your game as a Custom sheet; we don't support adding sheets to the dropdown list unless they are publicly available in some form.

ok.. thanks for the info...
i will resend the scion 1 alternate sheet...  in its own folder..
as to the actual attribute names...  nothing changed .. it should be  (mostly) be exactly the same...   except where.. the option menus at the top (choose pantheon.. and god!) use a script/writer.. to infuence a few lines on the pages  i.e.  you choose god/pantheon  it auto sorts your Virtues, and the pantheon Purview ...
was trying to save time and page turning for players

From: Jeffrey Lamb<EMAIL_ADDRESS>Sent: 30 April 2018 22:06
To: Roll20/roll20-character-sheets
Cc: Hotblood; Mention
Subject: Re: [Roll20/roll20-character-sheets] created for Scion 2 (Kickstarter) (#3711)
Hi @Hotbloodhypnohttps://github.com/Hotbloodhypno, because this is a separate sheet, its files need to be placed in a separate directory than Scion/, which already has the first sheet. Scion2/ would do just fine.
Also, since there is no publicly available form of the sheet, I can accept the code into the repository for use in playtesting (and for tracking, then release down the line), but you will need to use a pro account to copy the code and put it in your game as a Custom sheet; we don't support adding sheets to the dropdown list unless they are publicly available in some form.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Roll20/roll20-character-sheets/pull/3711#issuecomment-385543086, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbNTvKSjyMM9ltJNv3QHNi_eEDuTzeRAks5tt4rVgaJpZM4TsJbP.

You should put the Scion 2 sheet (this one) in its own folder, leaving the first one as an update so long as those changes don't totally drop the attributes as you've said.  The first one doesn't need those separate _1E files either.  Since one of those is a CSS file and it's not referenced in the sheet.json, I'm going to assume it's needed so I'll hold off on merging both of these until those issues are resolved.

You still need to move this version to a new directory before I can merge this, as it is a separate sheet for a new version of the game.

have been trying to create a folder for it and post it up to my on line github..  but cant seem to get it to co-operate
its sitting on my system in its own folder.

From: Jeffrey Lamb<EMAIL_ADDRESS>Sent: 08 May 2018 13:33
To: Roll20/roll20-character-sheets
Cc: Hotblood; Mention
Subject: Re: [Roll20/roll20-character-sheets] created for Scion 2 (Kickstarter) (#3711)
You still need to move this version to a new directory before I can merge this, as it is a separate sheet for a new version of the game.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Roll20/roll20-character-sheets/pull/3711#issuecomment-387403455, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbNTvDSNvEoYxGeAga4oOc8MYUapu6u6ks5twZ6XgaJpZM4TsJbP.


fix: todo-plugin ignores images and files that are bigger than/equal 200k

Hey, I just made a Pull Request!
This fixes #5997 by ignoring images and files that are bigger than 200Kb

:heavy_check_mark: Checklist

[x ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ x] All your commits have a Signed-off-by line in the message. (more info)

Currently looking for feedback on this PR, especially the size check. Could also be that the size check might be out of scope for this issue. If so, let me know and I will remove it.

Hi, I can confirm that what @Jerry4013 is saying is working. Changing the gitlab reader from zip to tar would fix the issue. With that change I could read out the official gitlab repo in a minute or so. Do, you want me to include it in this MR? I was wondering if the extension skipper and size check are then still needed?
For the size check you want to move it out of the backend-common, where would you like define it then? For the excludedExtension constant you said you would like to have it part of the fromConfig function. Do you envision something like this:
static fromConfig(config: Config, options: Omit<Options, 'integrations'>, skippedExtensions: Array<string>) {
    return new TodoScmReader({
      ...options,
      integrations: ScmIntegrations.fromConfig(config),
      skippedExtensions: ... // don't know yet from we should get the values
    });
  }

Let me know what you think.

Hey @djamaile - there are a few comments remaining on refactoring this, but we are eager to get the contribution in. Will you have the time to make those changes?

Hi @freben ! I have been a bit sick this week. I will be able to work on this at the end of the weekend. Feel free to merge @Jerry4013 PR, I can always rebase on top of it or we can close this if you feel it is needed anymore.

Ah, hope you get better soon!
We'll probably want both - they help in different ways!

Just a small ping - do you think you will have an opportunity to continue on this?

I took the liberty of refactoring this to try to get it across the line


DoctrineBundle is not registered in your application

I ran into this issue today when I upgraded one of our Symfony 4.2 applications to run on PHP 7.4 and updated dependencies accordingly.
In the composer.json we have symfony/orm-pack installed, which installs doctrine/doctrine-bundle properly at version 1.12.* or so.
We also have DoctrineBundle defined in config/bundles.php properly.
Otherwise the application worked OK in PHP 7.4 and new dependencies, but I noticed that one of our controllers was throwing errors that displayed:
DoctrineBundle is not registered in your application

After digging, I noticed that the bundle was in fact registered when running console debug:container. ManagerRegistry instances were properly injected when passed in as constructor parameters, so it seemed to work.
What was different with the failing controller, was that it used $this->getDoctrine() instead of injecting a ManagerRegistry. The controller extends Symfony's AbstractController.
The problem went away when I swapped the getDoctrine() call to be a ManagerRegistry injection instead.
I do think that this has something to do with a relatively stale Symfony 4.2 being in use, but wanted to post this issue to

Have a reference available in case someone else runs into a similar issue,
Let you know in case this might be an actual fixable bug in later Symfony versions as well.

I encountered the same problem with symfony = 4.4.8 and  orm-pack =1.0.8 versions.
orm-pack has internal doctrine/doctrine-bundle dependency which resolved at version 1.11.2
I managed to solve the problem by adding the following doctrine-bundle dependency to composer.json "doctrine/doctrine-bundle": "^1.12"

This should be fixed now.


doc: fix link to kodi-19.0 announcement

Motivation for this change
Logically re-apply 7afaacf9a8 ("doc: fix link to kodi-19.0 announcement"),
because it was lost in the conversion from docbook to markdown, in commit
32c2dd304d ("docs: nixos release notes to CommonMark (2105)").
Things done

[ ] 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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

CI failed because... apparently we're maintaining the release notes both in markdown and docbook, and if the two formats have different content the build fails? (/me confused.)


reverse commit 10d303c224dbc7e63d8eab7fa297cb204f6de300

according to the couchdb docs[1], slashes in document ids
need to be url escaped just like any other special character.
The earlier commit escaped everything except slashes.  This
commit reverses that change so that all special characters
are appropriately escaped.
1: http://wiki.apache.org/couchdb/HTTP_Document_API#Document_IDs

@indexzero, sorry I haven't been monitoring this in a while.  Will let you know when I test it.


Reviving DNxHR support?

Howdy all - Just curious if there's been any ongoing discussion around DNxHR since Thomas Gritzan did his work a few years ago (thread, repo)? I'd love to make use of bmxtranswrap for generating DNxHR opatom files, and looked at pulling Thomas' branch into the current project - it seems mostly fine, though the files aren't working in Avid at the moment.
I couldn't find much more discussion about DNxHR though, so I'm curious if there's a philosophical reason it isn't covered in this project?

There's no philosophical reason for not supporting it. It's not a format that I've needed in my work and so can't spend time resolving issues and completing submissions; it's up to the contributor to address the comments I had in that thread. It's also more complicated now that I don't have the same access to Avid editors, which makes it harder to validate files.


Sort address book entries alphabetically

Description
The desktop wallet and the Android version of the mobile wallet sorts address book entries alphabetically. It would be nice if the iOS app does the same.
Steps to Reproduce
Watch entries in the address book on iOS and notice that they are sorted in the order they are added.
Expected Result
They are soreted 0-9 and then A-Z.
Actual Result
They are sorted in the order they are added.
Versions

Software Version: Testnet 0.1.68
OS: iOS 14.5
Browser: Not applicable
Mobile device: iPhone XS

Ignore case when sorting.

Findings from testing:
It seems to work as intended.


"extension-module" feature causing problems

See https://github.com/fusion-engineering/inline-python/issues/27
Enabling features = ["extension-module"] on pyo3 in a crate that uses inline-python gives a linker error, because "extension-module" is then also enabled within inline-python-macros, which contains the proc macro that will use pyo3 to compile the Python code to bytecode. That crate is not an extension module, so leaving the symbols undefined will not work, as it won't be loaded by Python, but by Rustc.
This makes me think that "extension-module" should not be a feature, as cargo features should be strictly additive. That the end user is making an extension module, doesn't always mean that everything in the dependency tree will end up in an extension module, although this might very well be limited to procedural macros.
However, I don't know of a solution. Even if the ffi part would be split off into a separate crate, it'd still need to be compiled with different linker flags for the procedural macro than for the final crate. Having a separate crate just for linking to the Python libraries might work, but will probably cause confusion and other problems.
Any ideas?

This is mostly a duplicate of #771, with the added detail that it's not about a bin crate here, but about a (proc_macro) lib crate.
This might make a solution involving rustc-bin-link-arg trickier.


How about the performance compared with the original U-Net/nnUNet encoder?

I have used transUnet to train the lung segmentation images from Kaggle, but I find the performance is similar to AttUnet. Maybe my hardware is poor to train it. :(

I have used transUnet to train the lung segmentation images from Kaggle, but I find the performance is similar to AttUnet. Maybe my hardware is poor to train it. :(

Yep, I also train lung segmentation images from Kaggle, and i find the performance is not better than Unet. My hardware is 2080ti, so sad.

Hi @lisherlock : Can you share your code? or do you have published for code at kaggle?


[REVIEW]

Todo List
https://github.com/numanjvd82/todo-list
Wanted to made it with context API but I didn't know at the time. :)
You can also join the discord community here
Feel free to check out other cool repositories of EddieHub Community here

Todo List
https://github.com/numanjvd82/todo-list
Wanted to made it with context API but I didn't know at the time. :)
You can also join the discord community here
Feel free to check out other cool repositories of EddieHub Community here

Hey @numanjvd82 it looks good you work was very good but i have give you one suggestion that plss improves the readme it looks more cool if you readme was good then other peoples was also known what is this full project you can take this example for sample sample this is my project look at the readme sample 2

Yeah sure i would update as soon as  i can shave some time for github.

When I create/delete a todo, entire page shifts down when the toast is displayed. It's not a big issue but it could be made better in terms of user experience, if the toast didn't impact the other components. You could probably use the z-index property with position: fixed in your CSS to make the toast seems detached from rest of the page.

Maybe also add some prerequisites Fr running in local like node installed etc.

Requested Changes 😉

[ ] Live Preview Can Be Also The Repository Website URL
[ ] Add more to the readme: refer #1553
[ ]  Repository Description
[ ] Code of conduct and contributing
[ ] License
[ ] Issue/PR Templates
[ ] @shubhendumadhukar's suggestion: https://github.com/EddieHubCommunity/support/issues/1671#issuecomment-850800884
[ ] Logo as simple as #1561
[ ] CI Build GitHub Actions
[ ] Take a look at #1726
[ ] Commit style: either https://conventionalcommits.org or https://gitmoji.dev

Feedback

[x] Webpage Looks Cool 🤩
[x] Responsiveness 😍
[x] Project Tech Stack ❤

Copy This Checklist Paste It Into An Comment Then Keep It Updated With Your Workflow. I will update mine once I see your checklist is updated 😃

Actually i made a todo-list-v2 where i implemented context api and the toast also has a position fixed wit

Requested Changes 😉

[ ]  Live Preview Can Be Also The Repository Website URL
[ ]  Add more to the readme: refer #1553
[ ]   Repository Description
[ ]  Code of conduct and contributing
[ ]  License
[ ]  Issue/PR Templates
[ ]  @shubhendumadhukar's suggestion: #1671 (comment)
[ ]  Logo as simple as #1561
[ ]  CI Build GitHub Actions
[ ]  Take a look at #1726
[ ]  Commit style: either https://conventionalcommits.org or https://gitmoji.dev

Feedback

[x]  Webpage Looks Cool 🤩
[x]  Responsiveness 😍
[x]  Project Tech Stack ❤

Copy This Checklist Paste It Into An Comment Then Keep It Updated With Your Workflow. I will update mine once I see your checklist is updated 😃

Thank you for your feedback. I actually made a todo-list-v2 where i implemented context api and made my code cleaner and scalable for future additions.
I will also try to update my project repo using your checklist.
Thank you again for your time.

When I create/delete a todo, entire page shifts down when the toast is displayed. It's not a big issue but it could be made better in terms of user experience, if the toast didn't impact the other components. You could probably use the z-index property with position: fixed in your CSS to make the toast seems detached from rest of the page.

check out my todo-list-v2 i fixed that issue with the toast and also a search bar for searching todos.

Thank you for sharing, we will review on a live stream soon 👍 It looks like you are getting great community feedback already 💥


Pure transparent (aka Aero Glass) theme support

Is your feature request related to a problem? Please describe.
No, it's not related problem
Describe the solution you'd like
I wish if Aero Glass (without blur) is added for next release, I want a completely transparent titlebar
There was a program called Aero Glass in Windows 10, but it doesn't work in newer versions.
Describe alternatives you've considered
I don't know
Additional context
Nope :(

We don't plan to support DWM hooking methods currently.

@TK50P check this out: https://github.com/Maplespe/DWMBlurGlass


Packaging for PyPI or conda?

The current toolchain to install this project is quite heavy on the user. Is there a plan to provide packaged versions of this project on a conda channel or via PyPI?

The current toolchain to install this project is quite heavy on the user. Is there a plan to provide packaged versions of this project on a conda channel or via PyPI?

Yes indeed, there will be a PyPi package soon, which will simplify the installation process tremendously.


[Prunning] Check guarantee type then run the contained

https://github.com/apache/arrow-datafusion/blob/8524d58e303b65597eeebc41c75025a6f0822793/datafusion/core/src/physical_optimizer/pruning.rs#L245-L260
I think here we should check guarantee type first then run the real running method here.

@alamb  @yahoNanJing  I think here only Guarantee::In and Guarantee::NotIn should run the  contained  method,  other type will discard the result

Sorry see there are only two types in LiteralGuarantee