From what environment should I call SLURMCluster? Parsing Issue in sbatch?

I am running on a university cluster with a SLURM submission scheduler. I installed conda and sourced the pangeo environment without error.
I git cloned this repo, opened python and tried
(pangeo) [b.weinstein@gator4 dask-jobqueue]$ python
Python 3.6.4 | packaged by conda-forge | (default, Dec 23 2017, 16:31:06)
[GCC 4.8.2 20140120 (Red Hat 4.8.2-15)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from dask_jobqueue import SLURMCluster
>>>
>>> cluster = SLURMCluster(project="ewhite")
>>> cluster.start_workers(10)
sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

[2, 3, 4, 5, 6, 7, 8, 9, 10, 11]

Where ewhite is the group tag (see below).
In previous submission scripts, I would typically start with something like
#!/bin/bash
#SBATCH --job-name=NEON_LIDAR   # Job name
#SBATCH --mail-type=END
#SBATCH<EMAIL_ADDRESS>#SBATCH --account=ewhite
#SBATCH --qos=ewhite-b

#SBATCH --ntasks=1                 # Number of MPI ranks
#SBATCH --cpus-per-task=1            # Number of cores per MPI rank
#SBATCH --mem=4000
#SBATCH --time=00:59:00       #Time limit hrs:min:sec
#SBATCH --output=NEON_LIDAR.out   # Standard output and error log
#SBATCH --error=NEON_LIDAR.err

which works without a problem.
Attempts
I changed
#SBATCH -A %(project)s

to
#SBATCH --account %(project)s

to match previous submission scripts. No change.
So I tried removing the line, which yielded
>>> cluster.start_workers(10)
sbatch: error: Invalid argument: 00:30:00

sbatch: error: Invalid argument: 00:30:00

sbatch: error: Invalid argument: 00:30:00

...

This is an error in the walltime argument. This makes me believe that it doesn't have to do with the argument value, but how all the arguments are parsed and passed to bash. The most likely explanation is that i'm calling this script from an incorrect location, and the variable expansion or quotes are not being parsed in the way I anticipate.

If its useful, similar error here was reported here, sbatch would not accept spaces. I'll trying to look around and see how/if this could be connected.

Maybe we should be enclosing the executable in quotes since we can't
control the base_path ?
On Tue, Mar 20, 2018 at 5:26 PM, Ben Weinstein<EMAIL_ADDRESS>wrote:

If its useful, similar error here
https://github.com/nextflow-io/nextflow/issues/218 was reported here,
sbatch would not accept spaces. I'll trying to look around and see how/if
this could be connected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-jobqueue/issues/15#issuecomment-374762468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszC-wYXPNmutzo7hf4Cz2T9aLSL_aks5tgXQAgaJpZM4SyplE
.

I'm still playing with it. Its not clear to me exactly where it is failing. I added a logger to see from sys out,
logger = logging.getLogger(__name__)
ch = logging.StreamHandler(sys.stdout)
ch.setLevel(logging.DEBUG)
formatter = logging.Formatter('%(asctime)s - %(name)s - %(levelname)s - %(message)s')
ch.setFormatter(formatter)
logger.addHandler(ch)

but I never see the command called.
It yields
(pangeo) [b.weinstein@gator4 dask-jobqueue]$ python
Python 3.6.4 | packaged by conda-forge | (default, Dec 23 2017, 16:31:06)
[GCC 4.8.2 20140120 (Red Hat 4.8.2-15)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from dask_jobqueue import SLURMCluster
>>>
>>> cluster = SLURMCluster(project='ewhite')
>>> cluster.start_workers(3)
2018-03-20 18:07:43,667 - dask_jobqueue.core - ERROR - sbatch: error: Invalid argument: ewhite

2018-03-20 18:07:43,687 - dask_jobqueue.core - ERROR - sbatch: error: Invalid argument: ewhite

2018-03-20 18:07:43,707 - dask_jobqueue.core - ERROR - sbatch: error: Invalid argument: ewhite

but its clear that core should first log the cmd, and that will give us a sense for the parsing.
Line 79
        logger.debug("Submitting the following calls to command line")
        for cmd in cmds:
            logger.debug(' '.join(cmd))

I've tried manually removing the spacing, but without see the cmd parse, i'm guessing a bit. All thoughts welcome.

Can you try the solution in https://github.com/dask/dask-jobqueue/pull/18 ?
On Tue, Mar 20, 2018 at 6:12 PM, Ben Weinstein<EMAIL_ADDRESS>wrote:

I'm still playing with it. Its not clear to me exactly where it is
failing. I added a logger to see from sys out,
logger = logging.getLogger(name)
ch = logging.StreamHandler(sys.stdout)
ch.setLevel(logging.DEBUG)
formatter = logging.Formatter('%(asctime)s - %(name)s - %(levelname)s - %(message)s')
ch.setFormatter(formatter)
logger.addHandler(ch)
but I never see the command called.
It yields
(pangeo) [b.weinstein@gator4 dask-jobqueue]$ python
Python 3.6.4 | packaged by conda-forge | (default, Dec 23 2017, 16:31:06)
[GCC 4.8.2 20140120 (Red Hat 4.8.2-15)] on linux
Type "help", "copyright", "credits" or "license" for more information.

from dask_jobqueue import SLURMCluster
cluster = SLURMCluster(project='ewhite')
cluster.start_workers(3)
2018-03-20 18:07:43,667 - dask_jobqueue.core - ERROR - sbatch: error: Invalid argument: ewhite

2018-03-20 18:07:43,687 - dask_jobqueue.core - ERROR - sbatch: error: Invalid argument: ewhite
2018-03-20 18:07:43,707 - dask_jobqueue.core - ERROR - sbatch: error: Invalid argument: ewhite
but its clear that core should first log the cmd, and that will give us a
sense for the parsing.
Line 79
https://github.com/dask/dask-jobqueue/blob/9dbdf4b60e074ce3d4cd7de75bc003063343320c/dask_jobqueue/core.py#L77
    logger.debug("Submitting the following calls to command line")
    for cmd in cmds:
        logger.debug(' '.join(cmd))

I've tried manually removing the spacing, but without see the cmd parse,
i'm guessing a bit. All thoughts welcome.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-jobqueue/issues/15#issuecomment-374774953,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszBpd0yIyg0MD3t-kLQZN5_qyc5mRks5tgX7ZgaJpZM4SyplE
.

Same behavior.
(pangeo) [b.weinstein@gator3 dask-jobqueue]$ python
Python 3.6.4 | packaged by conda-forge | (default, Dec 23 2017, 16:31:06)
[GCC 4.8.2 20140120 (Red Hat 4.8.2-15)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from dask_jobqueue import SLURMCluster
>>>
>>> cluster = SLURMCluster(project='ewhite')
>>> cluster.start_workers(3)
sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

sbatch: error: Invalid argument: ewhite

[2, 3, 4]
>>> quit()
(pangeo) [b.weinstein@gator3 dask-jobqueue]$ cat dask_jobqueue/slurm.py
import logging
import os
import socket
import sys

from distributed import LocalCluster
from distributed.utils import get_ip_interface

from .core import JobQueueCluster

logger = logging.getLogger(__name__)

dirname = os.path.dirname(sys.executable)

class SLURMCluster(JobQueueCluster):
    """ Launch Dask on a SLURM cluster

    Examples
    --------
    >>> from pangeo import SLURMCluster
    >>> cluster = SLURMCluster(project='...')
    >>> cluster.start_workers(10)  # this may take a few seconds to launch

    >>> from dask.distributed import Client
    >>> client = Client(cluster)

    This also works with adaptive clusters.  This automatically launches and
    kill workers based on load.

    >>> cluster.adapt()
    """
    def __init__(self,
                 name='dask',
                 queue='',
                 project=None,
                 threads_per_worker=4,
                 processes=8,
                 memory='7GB',
                 walltime='00:30:00',
                 interface=None,
                 death_timeout=60,
                 extra='',
                 **kwargs):
        """ Initialize a SLURM Cluster

        Parameters
        ----------
        name : str
            Name of worker jobs. Passed to `#SBATCH -J` option.
        queue : str
            Destination queue for each worker job.
            Passed to `#SBATCH -p` option.
        project : str
            Accounting string associated with each worker job. Passed to
            `#SBATCH -A` option.
        threads_per_worker : int
            Number of threads per process.
        processes : int
            Number of processes per node.
        memory : str
            Bytes of memory that the worker can use. This should be a string
            like "7GB" that can be interpretted both by PBS and Dask.
        walltime : str
            Walltime for each worker job.
        interface : str
            Network interface like 'eth0' or 'ib0'.
        death_timeout : float
            Seconds to wait for a scheduler before closing workers
        extra : str
            Additional arguments to pass to `dask-worker`
        kwargs : dict
            Additional keyword arguments to pass to `LocalCluster`
        """
        self._template = """
#!/bin/bash

#SBATCH -J %(name)s
#SBATCH -n %(processes)d
#SBATCH -p %(queue)s
#SBATCH -A %(project)s
#SBATCH -t %(walltime)s
#SBATCH -e %(name)s.err
#SBATCH -o %(name)s.out

export LANG="en_US.utf8"
export LANGUAGE="en_US.utf8"
export LC_ALL="en_US.utf8"

"%(base_path)s/dask-worker" %(scheduler)s \
    --nthreads %(threads_per_worker)d \
    --nprocs %(processes)s \
    --memory-limit %(memory)s \
    --name %(name)s-%(n)d \
    --death-timeout %(death_timeout)s \
     %(extra)s
""".lstrip()

        if interface:
            host = get_ip_interface(interface)
            extra += ' --interface  %s ' % interface
        else:
            host = socket.gethostname()

        project = project or os.environ.get('SLURM_ACCOUNT')
        if not project:
            raise ValueError("Must specify a project like `project='UCLB1234' "
                             "or set SLURM_ACCOUNT environment variable")
        self.cluster = LocalCluster(n_workers=0, ip=host, **kwargs)
        memory = memory.replace(' ', '')
        self.config = {'name': name,
                       'queue': queue,
                       'project': project,
                       'threads_per_worker': threads_per_worker,
                       'processes': processes,
                       'scheduler': self.scheduler.address,
                       'walltime': walltime,
                       'base_path': dirname,
                       'memory': memory,
                       'death_timeout': death_timeout,
                       'extra': extra}
        self.jobs = dict()
        self.n = 0
        self._adaptive = None
        self._submitcmd = 'sbatch'
        self._cancelcmd = 'scancel'

        logger.debug("Job script: \n %s" % self.job_script())

Am I correct in assessing the problem as it adding a space, " ewhite"? I'll be back tomorrow morning.

Am I correct in assessing the problem as it adding a space, " ewhite"? I'll be back tomorrow morning.

I don't know.  I don't personally have much experience here.  You might try creating a SLURM cluster, seeing the template that it produces, and then trying to use that template manually, making sufficient tweaks as necessary.  This might allow you to diagnose the problem with the dask-jobqueue project out of the pitcture..

Understood, will report back today. I'm certain it's something minor.
Hopefully a PR soon.
On Wed, Mar 21, 2018, 05:24 Matthew Rocklin<EMAIL_ADDRESS>wrote:

Am I correct in assessing the problem as it adding a space, " ewhite"?
I'll be back tomorrow morning.
I don't know. I don't personally have much experience here. You might try
creating a SLURM cluster, seeing the template that it produces, and then
trying to use that template manually, making sufficient tweaks as
necessary. This might allow you to diagnose the problem with the
dask-jobqueue project out of the pitcture..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-jobqueue/issues/15#issuecomment-374919274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJwrDA6qKriBh8D9HnkBlshhbBY5HYtks5tgkZ9gaJpZM4SyplE
.


[ROAD-218] Feat: improved description panel header

Added common header for description panel (like in IJ).

@aldanchenko can you rebase this branch please, so that it's reviewable? Thank you 🙌


Added ARM release

Hi!
I needed an arm build and thought that it might have come useful to someone else as well.
Thanks

Awesome! Thanks. I can't really test the builds for those architectures, but I guess it won't hurt adding them to the release process.

Thanks! I'm using a manual build for arm7 since some months without issues. I'm not saying it's a full fledged test, but I had no issues up to now.


excerpt update for kinectron

this is the last time i'm using this branch...

adding a few extra updates to this pull request -- accessibility improvements 🎩


Add LMPHP...

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Hi @BaseMax  ,
Thanks for starting this pull request. In order to review it for acceptance, we need to make sure that all of the prerequisites are satisfied. I see that you didn't check this:

I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee)

This is a requirement so that we can maintain a high level of independence in this project. Please update if you are able to verify that you meet that requirement.
Thanks!

Hi Justin Kenyon @kenyonj,
I has been check that and submit a edit.
Best Regards,
Max.

Unfortunately, since you are the creator of the project this precludes you from submitting a new topic entry.

🤐

OK,
I did not know it.
I hope one person who has use this library will submit it.
Best Regards,
Max.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-07-20)
⚠ BREAKING CHANGES
This project has been renamed to zeal. To migrate, replace zealot with zeal in your
project's requirements. In your Django settings, replace ZEALOT_ALLOWLIST, ZEALOT_RAISE, etc.
with ZEAL_ALLOWLIST, ZEAL_RAISE, and so on.
In your code, replace from zealot import ... with from zeal import ....
Miscellaneous Chores

rename to zeal (cc429a2)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


no HTML autocomplete or snippets in .astro files

Just installed this extension in a mostly naked ST install (4180).
I'm trying to create a link and it's like it doesn't even recognize the HTML context.

The language is correctly selected as I can see HTML (Astro) in the bottom right:

Astro files are (and need to be) scoped source.astro for astro language server to detect and handle them correctly.
text.html.astro was is added to also enable ST's HTML specific features (completions and snippets).
Please note: To enable auto completions changes of https://github.com/sublimehq/Packages/pull/4085 are required, because HTML's current default_completions_selector excludes all sorts of source scope.
You can wait for a ST release, which ships this change or manually apply required change to user specific HTML.sublime-settings.

Thanks!

Sorry to bother again but I can't figure it out.
I found some comments of yours on the forum saying this is needed to add the source scope in the HTML.sublime-settings:
{
	"default_completions_selector": "text.html - source - string"
}

But I'm still not getting any HTML completions in the the .astro files.

This old suggestion doesn't take into account html being embedded into source or even being scoped source as in case of Astro.
The reason it doesn't work is - source, which excludes source.astro as well.
As of Astro 2.10.0, defaults from mentioned PR would work as they only exclude - text.html source, which matches CSS/JS script tags only.
    "default_completions_selector": [
        // Plain HTML
        "text.html - text.html meta.embedded - text.html meta.interpolation - text.html source - (text.html.markdown - markup.raw.code-fence text.html), ",
        // Embedded HTML blocks
        "text.html.embedded - text.html.embedded meta.embedded - text.html.embedded meta.interpolation - text.html.embedded source",
    ]

Thanks again!


fix bug related to argument and output

Set default hap threshold and default geno threshold #146 #133 #132

@AprilYUZhang Why this pull request is closed before merging in?

I completely forgot why, but it should be merged in principle. So I reopened.

@AprilYUZhang @XingerTang this PR is now openned for a long time. Remind me where we stand with this one?

@gregorgorjanc This PR is ready to merge. :)


[Question]: Issues with Client Development on Windows and Remote Database Connection

Describe your problem
Hello, I am currently using both the Python API and the HTTP API to connect to a standalone database service. However, it seems that the default connection is only allowed through "localhost." How can I customize the configuration file? I couldn't find any relevant documentation regarding this parameter and its path. Also, is it true that versions of the SDK after 0.2.0.dev7 are only available for Linux and not Windows? How can I develop code on Windows to connect to a 0.3.0.dev4 database? Thank you.

Yes, Python SDK is only for Linux now. On windows, you may use Python SDK to connect infinity 0.3.0 by using WSL , or you can use HTTP API to connect Infinity. In fact, HTTP API have much more functions than Python SDK.

Please check the document: https://github.com/infiniflow/infinity/blob/main/docs/references/pysdk_api_reference.md#connect, you can you following code to connect the remote server with IP <IP_ADDRESS>:

import infinity
infinity_object = infinity.connect(infinity.NetworkAddress("<IP_ADDRESS>", 23817))

BTW, Infinity python SDK also supports to open a local directory use following code:
import infinity
infinity_object = infinity.connect("/var/infinity")

We migrated all infinity server functions into python SDK. That is why python SDK is only available for Linux.

3q

(infinity_env) infiniflow@infiniflow:~/infinity_data$ docker run -d --name infinity -v /home/infiniflow/infinity_data/:/var/infinity --ulimit nofile=500000:500000 --network=host infiniflow/infinity:nightly
047a2adaec8a797296b9c7a8c33f9db021fff5cdcf8111354b16a5d02defc8bf
(infinity_env) infiniflow@infiniflow:~/infinity_data$ docker container ls -a
CONTAINER ID   IMAGE                         COMMAND                  CREATED         STATUS         PORTS     NAMES
047a2adaec8a   infiniflow/infinity:nightly   "bash -c 'cp -rf /re…"   6 seconds ago   Up 4 seconds             infinity
(infinity_env) infiniflow@infiniflow:~/infinity_data$ cd ~/Documents/development/infinity
(infinity_env) infiniflow@infiniflow:~/Documents/development/infinity$ python example/simple_example.py
shape: (3, 3)
┌─────┬─────────────────────────────────┬───────────────────┐
│ num ┆ body                            ┆ vec               │
│ --- ┆ ---                             ┆ ---               │
│ i32 ┆ str                             ┆ list[f64]         │
╞═════╪═════════════════════════════════╪═══════════════════╡
│ 1   ┆ unnecessary and harmful         ┆ [1.0, 1.2, … 0.9] │
│ 2   ┆ Office for Harmful Blooms       ┆ [4.0, 4.2, … 4.5] │
│ 3   ┆ A Bloom filter is a space-effi… ┆ [4.0, 4.2, … 4.5] │
└─────┴─────────────────────────────────┴───────────────────┘

import infinity
from infinity.common import NetworkAddress

try:
    # open a local directory to store the data
    # infinity_instance = infinity.connect("/var/infinity")

    # connect to server with <IP_ADDRESS>
    infinity_instance = infinity.connect(NetworkAddress("<IP_ADDRESS>", 23817))

I just tried the latest docker image of as above. Use example/simple_example.py updated with IP address as above code. And it works.
You'd better to check if there are network issue, such as port conflict, incorrect ip address, etc. Use netstat or lsof to check if Infinity is listening on the port.
Infinity installed with deb/rpm can load config file, but docker running infinity can't load config file yet, this problem will be solved in the next Dev release.
infinity -f infinity_conf.toml

嗯嗯，谢谢大佬

Yes, Python SDK is only for Linux now. On windows, you may use Python SDK to connect infinity 0.3.0 by using WSL , or you can use HTTP API to connect Infinity. In fact, HTTP API have much more functions than Python SDK.
Please check the document: https://github.com/infiniflow/infinity/blob/main/docs/references/pysdk_api_reference.md#connect, you can you following code to connect the remote server with IP <IP_ADDRESS>:

import infinity
infinity_object = infinity.connect(infinity.NetworkAddress("<IP_ADDRESS>", 23817))

BTW, Infinity python SDK also supports to open a local directory use following code:
import infinity
infinity_object = infinity.connect("/var/infinity")

We migrated all infinity server functions into python SDK. That is why python SDK is only available for Linux.

Hello, since the HTTP server's listening IP is set to localhost, it cannot be accessed from external networks. Does this situation actually exist? Should it be changed to the IP specified in the infinity_conf.toml file?
https://github.com/infiniflow/infinity/blob/e62d9dba35872ed595be920d4c7335b055d219bc/src/network/http_server.cpp#L3336
By the way, in version 0.3.0-dev7, the provided example of the search API seems incorrect: https://github.com/infiniflow/infinity/pull/874#issue-2205075924. Could you provide more examples of the HTTP API, as some of them differ from the SDK API?

Yes, Python SDK is only for Linux now. On windows, you may use Python SDK to connect infinity 0.3.0 by using WSL , or you can use HTTP API to connect Infinity. In fact, HTTP API have much more functions than Python SDK.
Please check the document: https://github.com/infiniflow/infinity/blob/main/docs/references/pysdk_api_reference.md#connect, you can you following code to connect the remote server with IP <IP_ADDRESS>:

import infinity
infinity_object = infinity.connect(infinity.NetworkAddress("<IP_ADDRESS>", 23817))

BTW, Infinity python SDK also supports to open a local directory use following code:
import infinity
infinity_object = infinity.connect("/var/infinity")

We migrated all infinity server functions into python SDK. That is why python SDK is only available for Linux.

Hello, since the HTTP server's listening IP is set to localhost in http_server.cpp, it cannot be accessed from external networks. Does this situation actually exist? Should it be changed to the IP specified in the infinity_conf.toml file?

Yes, it shall be fixed by #1759

https://github.com/infiniflow/infinity/blob/e62d9dba35872ed595be920d4c7335b055d219bc/src/network/http_server.cpp#L3336
By the way, in version 0.3.0-dev7, the provided example of the search API seems incorrect: #874 (comment). Could you provide more examples of the HTTP API, as some of them differ from the SDK API?
https://github.com/infiniflow/infinity/blob/e62d9dba35872ed595be920d4c7335b055d219bc/src/network/http/http_search.cpp#L137-L140

Yes, we are preparing to release v0.3.0. And the document will be updated accordingly.


Debora elkin search simlar entities

Reusable asset to add the ability to search for similar entities (based on a set of criteria) from an entity detail page

@DeboraElkin - Thanks for your submission!  This is great!
Most of the comments so far are related to style and making the code more "teachable".  My main concerns are improving the code so that it will be easy for someone to pick learn from your contribution and use it.
Once you've submitted changes someone will test everything to make sure it works and then it will probably be good to go.

I've incorporated as much of the feedback as I could (I've left out using array.map and to_query, as explained above.
Let me know if there are any other changes that should be made.

Looks good to me.  I haven't had a chance to test it yet.

@michaelkeeling , were you able to test the code? I'm keen for the contribution to be made public (if it's deemed worthy of it, of course...)


When Panning Map, Don't Move the Pin Every Time

This is specifically on the obs submit form, when picking location for where the obs is coming from. Right not the Pin will move each time you touch the map with one finger.

I have a fix for this, but when working on it I came across some user experience things that maybe could be improved on. I wanted to get feedback before making the changes though.
Right now, it's not clear that closing the modal with the X button is going to erase the location as well. I think the standard practice is that it should close out, and keep everything as it was. I think we can make this more clear in 2 ways.

Add another button to the modal (like an eraser or trash can) that the user presses to clear the pin on the map, they would then hit the checkmark to save the clear

Add a "Clear location" button under the location form field, if the user has a location set

Let me know what you think. I'm also fine leaving it as is. It was just something that I found a little wonky when debugging this.

@kevinherdez these are fair points!
a little history on the view: at first, we only had the check mark and you could only select the location and there was no way to clear it. when we added the ability to clear, we went back and forth on this quite a bit (if the X made sense/etc, if it should be a different icon, we havent considered having the clear button outside or the map as far as I remember since we didnt want to make major changes to the obs form itself), and in the end the consensus was it would be good to get a design opinion on what user experience is best but we didn't at the time have a designer who can help out. With all that, totally open to changing it!
if you have a vision of how it may work better, and its not a lot of work, I'd say - put it together so we can play with it. If its more then minimum amount of work, lets hold off for now -- we may have some designer help later on as well, so it is possible this whole experience may need a redesign. will leave a tag for @charlotteguard  to keep me honest :)

That sounds good! I'll go ahead and make a separate issue for this.

Made the new issue here #793


Log messages being sent to the wrong file

When switching files, the log writer will instantly switch between output files. This however has the effect that messages sent before the file change occurred, but received by the log writer afterwards, will be written to the wrong file.
The LogWriter should keep a list of the recent files with their timestamps of when the change occurred, and then for each message it should figure out, based on its timestamp, to which file the log message should be written.

To keep in mind - annoying for crashes.


Shiny Celebi Broken from Latest PKHeX Commit

Celebi @ Leftovers
IVs: 30 Spe
EVs: 252 HP / 220 Def / 36 Spe
Ability: Natural Cure
Level: 100
Shiny: Yes
Bold Nature

Calm Mind
Recover
Psychic
Hidden Power [Ice]

Auto mod can't seem to do this anymore. The regular pkhex can if a legal file of gen 2 celebi is selected.
251 ★ - Celebi - 45E70036F843.zip
Auto PKHeX 4341.zip

Cant replicate on my end.

New base PKHeX commit breaks this


Update userguide_deployment.adoc

@pierre Also merging for having a link -- however we should have a discussion about this section.


docs(examples): Add Next.js example project

Add example project for universal error reporting on a Next.js app.
Going by some of the techniques in this thread:
https://github.com/zeit/next.js/issues/1852

A couple of the links in the readme 404


Add bridge rendering

Fixes #169
Bridges are given a fill extrusion just like buildings, but with a red tinge and different heights for different values of layer. The same red tinge is applied to an extra casing around linear road and rail bridge features.
Example locations
production, localhost

production, localhost

production, localhost

production, localhost

Cool addition. Just to clarify, this is for rendering man_made=bridge areas only, not a halo around ways tagged bridge=yes right? Or is it both?

Cool addition. Just to clarify, this is for rendering man_made=bridge areas only, not a halo around ways tagged bridge=yes right? Or is it both?

It’s both. At low zoom levels, only the casing layer tends to be visible, but at high zoom levels, the casing is still subtly visible within the bridge area.

Just wanted to note here that @adamfranco's track PR implements a "guardrail"/casing approach to track bridges: https://github.com/ZeLonewolf/openstreetmap-americana/pull/717#issuecomment-1416502830. The considerations are slightly different since tracks wouldn't have a casing, but obviously I think we'd want to unify the approaches coherently.

with a red tinge

Will that become yellow for toll roads?

with a red tinge

Will that become yellow for toll roads?

No. My intent was to make bridges look like buildings, but slightly different. I wasn't considering changing the color of the bridge based on the tagging.
Besides, I'm leaning towards replacing this with a knockout/"guardrail" effect as above, so the color would be a moot point.

From #169:

A solution to this would be to compose two different knockouts per bridge: one based on the bridge area and another based on the roadway that’s always wider than the roadway. In turn, there can be two guardrail layers based on the same features. To keep the bridge area–based guardrail layer from looking like an outline, the roadway-based knockout layer can cover up the bridge area–based guardrail layer at the abutments. To avoid redundant guardrails, both guardrail layers would appear beneath both knockout layers.

I much prefer this idea. I've gone ahead reimplemented bridges to be in a knockout/guardrail style. It conflicts a little with my desire for bridges to be slightly transparent at high zoom, but I managed to fudge the guardrail opacity so that it's barely noticeable under a bridge.
Example locations
production, localhost

production, localhost

production, localhost

production, localhost

I like this effect much better. One minor note, it looks like something odd happens with the overlapping knockouts when the road/bridge curves: https://preview.ourmap.us/pr/934/#map=17.74/34.027658/-118.453153.

In OSM, the bridge ways are glued to the bridge area. I'm guessing this is tricky, and certainly not a blocking issue.

Also, poking around in this PR has already provided some useful layering QA opportunities 😎 . I don't know if any other style correctly layers bridge areas, nice!
roadway wrongly at layer=2, bridge area at layer=1

bridge area at layer=1, roadway wrongly missing layer

One minor note, it looks like something odd happens with the overlapping knockouts when the road/bridge curves:
...
In OSM, the bridge ways are glued to the bridge area. I'm guessing this is tricky, and certainly not a blocking issue.

It is indeed tricky. I don't really have a solution for it.

production, preview, localhost

The road casing is redundant to the bridge area. Can we deemphasize the road casing at this zoom level? In localities without separately mapped bridge areas, the bridges would be less prominent, but this could be an incentive to map bridge areas.

The road casing is redundant to the bridge area. Can we deemphasize the road casing at this zoom level? In localities without separately mapped bridge areas, the bridges would be less prominent, but this could be an incentive to map bridge areas.

Here's my attempt. I didn't touch the opacity of expressway casing because it doesn't look good with link roads.

Thanks for trying that. Unfortunately it didn’t quite turn out as clear as I had hoped. I’m starting to wonder if the knockout approach is really appropriate at zoom levels where roads are filled. Looking back at https://github.com/ZeLonewolf/openstreetmap-americana/issues/169#issuecomment-1043669724, I do see a pattern where maps only bother with a knockout when the road is stroked.
On this street-scale map of Chicago, only the overpasses carrying minor streets get a knockout with guardrails. Even railroad tracks (in gold on this map) don’t get a knockout, presumably because they already contrast well enough against the green freeways:

This Minnesota Department of Transportation map treats a freeway as a pair of independent stroked roads (similar to what aaroads-wiki#2 does with expressways). The freeways are red but have black guardrails:

This black-and-white map of Springfield, Ohio, does combine road casings with bridge knockouts and guardrails. It’s exquisite linework. I think the only thing keeping the casings from looking redundant is that the guardrails are guardrails – they don’t form a closed polygon:

This map of New York City is possibly the only one in my collection that tries to depict the shape of each overpass – but still not as closed polygons. It would’ve been a lot more confusing with polygons:

This is a tough one, and it scratches at whether we even render these polygons.  I do think we should at the highest zooms at least.  I wonder if perhaps a color treatment more reminiscent of pavement might better satisfy what we're trying to show.

I achieved the following effect with:
export const bridgeFill = "hsl(248, 25%, 85%)";
export const bridgeBackgroundFill = "hsl(248, 25%, 36%)";

The slightly bluer effect seemed more pavement-y.
I'm not really a fan of how the background fill creates breaks when two adjacent bridge areas have separate levels.  I tried a straight knockout effect, which looks decent on the sides but has an even more pronounced effect on the bridge joins:

Bridge areas are now hidden at z<15, and the guardrail is now a knockout at z>15.


Kubernetes_sd_config scraping everything

Bug Report
What did you do?
Put the prometheus.io scraping annotations on a service and tried to monitor it.
What did you expect to see?
A job in prometheus with the targeted service and nothing else.
What did you see instead? Under which circumstances?
The prometheus was trying to scrape all the services with or without the annotation "prometheus.io/scrape: true"
In my case some of the services do not need to be scraped, unless they got the right annotations.
Environment

System information:
Running on Kubernetes 1.13

Prometheus version:
2.13.1

Prometheus configuration file:

global:
  scrape_interval:     15s

  external_labels:
    monitor: 'example-monitor'

scrape_configs:
- job_name: 'example-metric'
  scrape_interval: 5s

  metrics_path: /metrics/
  scheme: https
  basic_auth:
    username: admin
    password: adminadmin
  tls_config:
    insecure_skip_verify: true

  kubernetes_sd_configs:
  - role: services
  relabel_configs:
  - source_labels: [__meta_kubernetes_service_label_app]
    separator: ;
    regex: config-example
    replacement: $1
    action: keep
  - source_labels: [__meta_kubernetes_endpoint_port_name]
    separator: ;
    regex: https
    replacement: $1
    action: keep
  - source_labels: [__meta_kubernetes_namespace]
    separator: ;
    regex: (.*)
    target_label: namespace
    replacement: $1
    action: replace
  - source_labels: [__meta_kubernetes_pod_name]
    separator: ;
    regex: (.*)
    target_label: pod
    replacement: $1
    action: replace
  - source_labels: [__meta_kubernetes_service_name]
    separator: ;
    regex: (.*)
    target_label: service
    replacement: $1
    action: replace
  - source_labels: [__meta_kubernetes_service_name]
    separator: ;
    regex: (.*)
    target_label: job
    replacement: ${1}
    action: replace
  - separator: ;
    regex: (.*)
    target_label: endpoint
    replacement: https
    action: replace

Really struggling to understand what i did wrong.
Thank you !
*

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Decide the schedule for a phased implementation of the permanent URL in citations

Discussion today with @richardofsussex suggests: add ' FreeBMD1 url' to the database and a 'search by FreeBMD1 url" to the search page.
?Also add info on that feature to the 'url not found' message
Nb the FreeBMD1 url is a hash, so corrections, adding a postem, etc. means it will change.

If it is going to be significantly quicker to implement the prettified url than citations, then prettified url should go first.


Documentation> Check out this PDF link> "Not Found" error message is appearing upon launching.

Describe the bug
"Not Found" error message is appearing upon launching the link "Check out this PDF".
To Reproduce
Steps to reproduce the behavior:

Go to https://react-magma.cengage.com/version/2.5.7/contribution-guidelines/
Click on "Check out this PDF" link
Verify the error

Expected behavior
Appropriate pdf should appear upon launching the link.
Screenshots
https://somup.com/c3nQQAZNtU
Desktop:

OS: [Win 10]
Browser [Chrome]
Version [Chrome version: Version 98.0.4758.81 (Official Build) (64-bit)]

The issue is fixed.
See Screen-cast : https://www.screenpresso.com/=6ULPf
Hence, Closing the ticket.
Thanks!


Python completer can display warnings and errors in the console

This is a follow-up to the problem described in https://github.com/commontk/CTK/pull/1051#pullrequestreview-1164299439

from a user standpoint I might be questioning if I'm doing something wrong now that these errors are more prominently displayed

The error messages are reported if a property getter of a Python-wrapped Qt object reports an error (see for example a freshly created slicer.qMRMLTableView(), without setting a table node in it).
This is because the auto-completer calls ctkAbstractPythonManager::dir_object function, to get a list of attribute names and values. The values are used to determine if the value is "callable" and if it is then () is added to the completion.
Potential solutions:

change all Qt-wrapped classes in Slicer to not return errors in property getters
change the auto-completer to get the list of attribute names and determine if they are callable without executing the getter function for each property

change all Qt-wrapped classes in Slicer to not return errors in property getters

Considering that independently of ctkAbstractPythonManager::dir_object, calling dir() on an instance of qMRMLTableView returns errors, this would make sense.
>>> t = slicer.qMRMLTableView()

>>> dir(t)
['AboveItem', ..., 'windowTitleChanged', 'windowType', 'wordWrap', 'x', 'y']
^ qMRMLTableView:: bool qMRMLTableView::transposed() const  failed: invalid node
^ qMRMLTableView:: bool qMRMLTableView::firstRowLocked() const  failed: invalid node
^ qMRMLTableView:: bool qMRMLTableView::firstColumnLocked() const  failed: invalid node

>>> t.transposed
False
^ qMRMLTableView:: bool qMRMLTableView::transposed() const  failed: invalid node

I have a prototype for temporarily disabling error logging during ctkPythonConsoleCompleter's update method (will send a pull request today or tomorrow).
I think Jupyter uses static analysis for getting completions, and that does not execute the property getter methods, so should not log any errors.


How to parse and obtain each url's stress test result records from the output csv file that is generated through command 'vegeta encode'?

Question
I was glad these days as I had confirmed that multiple URLs can be successfully tested under certain stress at the same time, and I want to say that it's really cool!
But after that, I found a new question since I cannot get the corresponding test records of every URL from the CSV file outputted by executing the command 'vegeta encode'. This is because there is no way to distinguish what results belong to a certain or another URL in the CSV file even though we can get the whole output. However, there is no any URL for those successful test records in the file.
What I want to know is how to address the above problem.

You can always post process the results, group by URL into different groups, and then feed each of those grouped result sets to vegeta report.


Enable opting out of workspace settings / .vscode/settings.json

Unlike https://github.com/microsoft/vscode/issues/40233 – which is about separating local workspace settings from shared workspace settings – I would primarily want a way to disable workspace settings in its entirety.
Most basic solution of all:
Have a user setting that disables the use of workspace settings.
Why do I want this?
I sometimes get very confused when I open up a repository shared with someone else and all of the sudden my editor starts eg. formatting on save etc when I work with files there – and even when I figure that out there's no way for me to avoid it other than convincing my collaborators to remove the workspace settings from the repository.
I can never think of a scenario where I personally would prefer to use settings from someone else, but I can see how others may like it.
Having an option to opt out of the shared settings would help avoid a lot of conflict and friction within teams and projects.
Is this a duplicate?
Unlike eg. https://github.com/microsoft/vscode/issues/40233 and https://github.com/microsoft/vscode/issues/89737 (which was marked a duplicate of https://github.com/microsoft/vscode/issues/40233) this isn't about individual rules, this is about opting out of workspace settings entirely.

Primary intention of workspace/repo settings is to allow users collaborating with that workspace to have similar and stream lined experience. Ignoring it can lead to issues like breakages. I do not think it is a good idea to allow such a feature in the product that can break the team experience. If a user does not like or change some settings/guidelines, it is recommended to make that change in the team guidelines.

The current situation breaks the team experience as it forces me to have to convince my colleagues to change settings or for me to do workarounds, like patching the checked out repo or swapping to another editor (or disabling the extensions that eg are configured to do auto formatting)
A team experience is enforced through CI and scripts, not through UI.
Also:
When forking a GitHub repository to do PR:s I will get the team settings of a fully unknown team and will get random surprises in how the editor works depending on which repository ai’m currently contributing to, with no way of really knowing what changes and when. It disrupts my muscle memory and can’t be fixed by “talking to my team”.
I should be able to make PR:s with VSCode without being forced upon the settings of other teams. (I maybe can achieve this by marking the repo as not trusted, but that’s very disruptive and far further reaching. It’s not that I don’t trust them, it’s that I can’t instantly adapt my muscle memory to each and every project I contribute to)
At least leave it open for some discussion? And then close it as “not planned” rather than “completed” when closing? 🥺

team settings should help in catching any CI failures upfront and make it efficient

Only holds true when one is also enforcing the editor of choice and especially when contributing to open source projects editors will not be forced upon contributors, so not sure why settings should be.

For future reference to others finding this issue:
One can use the assume-unchanged git feature to remove it locally while having git treat it as unchanged (found out thanks to suggestions on Mastodon!):
git update-index --assume-unchanged .vscode/settings.json


chore(deps): update common-dev-assets digest to a949ce0

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
dd1294a -> a949ce0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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

GitHub release
v2.0.0

Your semantic-release bot :package::rocket:


Error Compiling

Hi,
I installed everything following the install instruction.
First I got 3 Error messages, that there are missing files.
..\async-mqtt-client-master\src the File "ESPAsyncTCP.h" was missing.
copied from ..\libraries\ESPAsyncTCP-master\src
..\async-mqtt-client-master\src the File "async_config.h" was missing.
copied from ..\libraries\ESPAsyncTCP-master\src
..\libraries\WS2812FX\src the File "Adafruit_NeoPixel.h" was missing.
copied from ..\libraries\Adafruit_NeoPixel
Now I get the following problem
I attached a text file for the Error Message
ErrorMessage_01.txt

Hi @Haschu87,
I'm not sure if you're having the same problem I had, but when I tried to build with the latest version of VSCode on Linux, platformIO would seeming delete stuff as things were being compile causing errors about missing this file or bad assembly...  seems like some kind of strange race condition, it would take me between 3 and 5 compile attempts to build a firmware.bin file.
Here's a script I wrote to work around it, you can try it out, if it doesn't work after 10 attempts you might have a real dependency issue:
#!/usr/bin/env bash

t=0; until platformio run --target upload --environment nodemcuv2
do
  t=$((t + 1))
  echo
  echo "$(date +%H:%M) Trying again... (${t})"
  echo
  if [ -e .pio/build/nodemcuv2/firmware.bin ]; then
     echo "binary built!! quiting..."
     break;
  fi
done
echo Took ${t} tries..


I don't think there's any reason to have an "en" subdir when English is already the messages.pot source.

My review of the @jdlcdl's commits is up to e13f5dc0094bb668f4fb4685132f4f96f0083376 so far.

@jdlcdl can you add a commit to delete the *.mo files from resources/babel?
I don't think there's any reason to have an "en" subdir when English is already the messages.pot source.

Correct, "en" is not at all necessary.
I added it for my personal strong opinion that MSGID might be treated as a name that is not changed by programmers within code for the following reasons:

Terms could be changed for english just like any other language... with a separation of gettext files instead of actual code that requires much better review.  It could become the job of a ui/copy-writer/communicator instead of an engineer.
Conventions for these messages might be set once and never change again.  Instead of "Change address validated." in the code, it might be "PSBT: change_is_valid" and we'd just treat it like any other variable name that doesn't change often.

Terms could be changed for english just like any other language... with a separation of gettext files instead of actual code that requires much better review.  It could become the job of a ui/copy-writer/communicator instead of an engineer.

Ah, yes, I forgot about this.
That creates a level of separation / inconvenience for translators though, ya? Instead of translating "This is my text" there's a lookup task to see what SOMESCREEN_SUCCESS_MSG is in English. Or can the *.po extractions include that mapped English content by default?

That creates a level of separation / inconvenience

Yes, it would both for translators and for coders chasing "bug reports" from users in english since they'd have to check the msgstr to find the msgid in the catalog instead of just searching for the string in code.
My opinion probably comes from old conventions where every single message/error had a numeric id in code, and being able to alter what users see as strings in less-delicate-than-code files.  Besides that small win, you're pointing out valid down-sides to having an english gettext catalog when without it, it still works perfectly fine for msgids in code to change when new terminology in english takes place (with caveat that translations might need to be carried forward or could get lost).

I think for now the msgid approach feels like a 40% win / 60% burden.
We're not going to get everything right on this first pass / initial release, and probably the most complicated part of all this is the logistics and coordination with the not-necessarily-technical translation volunteers.
So I'm in favor of deferring a msgid approach. There'll be some inefficiency and pain if we do want to refactor to it later, but such is life.

So glad you kept this branch alive, @jdlcdl!

So glad you kept this branch alive, @jdlcdl!

I'm so glad that you started it, and are back on it again.  I bumped my head against many walls that you are gracefully solving one after the next, so to have kept it alive was well worth it.
It's hard to express the strange feeling of regret I have for my absence recently while you are moving forward on this branch.  Lately, my lack of time is in my control -- purposely prioritized towards family, in the short term, while I can be effective for them.  That you are mastering progress here is a great relief every time I check back in.


Implement GET /1/playlist/(playlist_mbid)

This builds upon the feature/new-endpoints branch, adding the endpoint to actually fetch playlists (as other existing endpoints actually don't return tracks, as far as I'm aware). I also annotated some jspf fields that aren't always provided by ListenBrainz.
This is my first actual Rust bit of code, so there might be issues with it - although I tried to mimic what's around it so it should most likely be fine. I just realised when opening this PR that my editor formatted a bit too much - happy to revert those changes if there's any actual interest into getting this merged, but I'm not sure what the status of this upstream branch actually is, as it hasn't been updated in quite while?

I would rather not merge any formatting changes into the branch, given the formatter has affected other functions.
I'm just wondering how we can avoid merging the aforementioned changes, though...

Formatting changes removed - how do you feel about me opening a PR just to apply repo-wide formatting changes to avoid this kind of friction in the future? As far as I understand, cargo fmt is the most common way of formatting Rust code, so most people's editors will follow that?

Formatting changes removed - how do you feel about me opening a PR just to apply repo-wide formatting changes to avoid this kind of friction in the future? As far as I understand, cargo fmt is the most common way of formatting Rust code, so most people's editors will follow that?

I spotted one more fmt change.
I will leave that decision up to @InputUsername. The formatting is generally done by rustfmt, yes - we should also look into EditorConfig.

Other than formatting, I'm happy with the PR. Will approve when the final review is resolved.

I'm not sure what the status of this upstream branch actually is, as it hasn't been updated in quite while?

I honestly kind of forgot about the branch as I haven't needed any of the additional endpoints myself, and judging by the issues others have not either. Looking at it now, this branch actually seems to be based on version 0.4.1 which is over 3 years old :sweat_smile:
Still happy to try to at least get a subset merged into main though. @shymega I guess we can just try to merge whatever is currently implemented into main and keep the issue open?

how do you feel about me opening a PR just to apply repo-wide formatting changes to avoid this kind of friction in the future? As far as I understand, cargo fmt is the most common way of formatting Rust code, so most people's editors will follow that?

I have a sneaking suspicion main is already formatted correctly, it's just this branch that is really old, but feel free to create a PR if we've missed anything!

Nits addressed, thanks!
Thanks for the update on the status of this branch too. As you probably have guessed, I have an interest in some of those playlist-related endpoints, so would definitely be keen on seeing this merged and eventually released. Happy to help in any way I can!
Regarding formatting, it seems you're right and main is properly formatted, I'll disregard that then :-)

LGTM!
@shymega I think the way forward is merging this PR, then rebasing+merging feature/new-endpoints into main but keeping the branch? Then we can release whatever is there now and implement the other endpoints at some other time.

Hey @InputUsername sorry, was busy. I've reviewed the remaining PRs, but I think rebasing and merging is a good step forward, as long as we don't rebase on main. We can always cherry-pick if necessary.

Hey @InputUsername sorry, was busy. I've reviewed the remaining PRs, but I think rebasing and merging is a good step forward, as long as we don't rebase on main. We can always cherry-pick if necessary.

@shymega No worries! I've already taken the liberty of merging main into feature/new-endpoints so after the other MRs we should be ready for the next release (0.8.0?) with some of the new endpoints, how does that sound to you?


message view: Add loading indicator to bankruptcy modal.

Fixes: #9629

GIF:

Merged, after also removing the old notice text (which for some reason wasn't being displayed anyway).
Thanks @Anupam-dagar!


POST Method not Allowed via postman untill i call protected function verbs()

Hi team,
I am creating API using yii2 (basic).
when i tried to call any post method, i am getting error

"Method Not Allowed. This URL can only handle the following request methods: GET, HEAD.",

after googling few article, I found a way to solve this.
protected function verbs() {
	   $verbs = parent::verbs();
	    $verbs =  [
			'index' => ['GET', 'POST', 'HEAD'],
			'view' => ['GET', 'HEAD'],
			'create' => ['POST'],
			'update' => ['PUT', 'PATCH']
		];
	   return $verbs;
}

Now Problem is, I have to paste this code in every controller. (which is not ideal and i don't want)
So is there any way to set as global?

what your request action

Show your controller code

@rustamwin @Auramel
My request action is POST (sending request vi POSTMAN)
Here is controller code.
<?php

namespace app\controllers;

use Yii;
use yii\base\Model;
use yii\rest\ActiveController;

class SessionController extends ActiveController {

	public $modelClass = 'app\models\Session';

       //when i add verb function, its work but don't want to add everytime in each controller.
	/*
	protected function verbs() {
	   $verbs = parent::verbs();
	    $verbs =  [
			'index' => ['GET', 'POST', 'HEAD'],
			'view' => ['GET', 'HEAD'],
			'create' => ['POST'],
			'update' => ['PUT', 'PATCH'],
			'delete' => ['DELETE'],
		];
	   return $verbs;
	}
	*/
	public function init() {
    }
	public function actions() {
	}

	public function actionIndex($params = array()) {
		if($_SERVER['REQUEST_METHOD'] == 'POST'){
			// logic goes here
		}
	}
}
?>

And here is config/web.php
'urlManager' => [
            'enablePrettyUrl' => true,
            'showScriptName' => true,
            'rules' => [
                [
				'class' => 'yii\rest\UrlRule',
				'controller' => ['users', 'todo'],
				],
				'<alias:\w+>' => 'site/<alias>',
               ],
        ],

@sudhir600 add session controller to urlManager rules

I added but same error.
'controller' => ['users', 'todo', 'session'],

so add verbs to controller
protected function verbs() { 	   $verbs = parent::verbs(); 	    $verbs['index'] =  ['GET', 'POST', 'HEAD']; 	   return $verbs; 	}

this is already i am doing since i asked this question.
What am i trying to get it, why i need to write verns () function in each controller?
is there any way to write at once and overright for all controller?

Okay, I look into \yii\rest\ActiveController::verbs(); Here is:
    /**
     * {@inheritdoc}
     */
    protected function verbs()
    {
        return [
            'index' => ['GET', 'HEAD'],
            'view' => ['GET', 'HEAD'],
            'create' => ['POST'],
            'update' => ['PUT', 'PATCH'],
            'delete' => ['DELETE'],
        ];
    }

So, POST method allowed by default only for create action. If you want add your METHOD - you should overwrite verbs() method or something else (I didn't see the documentation).
If you don't want to write it each controller I offer create BaseActiveController with your verbs() and extends from it

Maybe, someone offer a better solution...

What am i trying to get it, why i need to write verns () function in each controller?
is there any way to write at once and overright for all controller?

create a base controller and extend other controllers

Thanks @rustamwin. your suggestiins worked (By using base controller)
Those, who are facing the same issue, use this code.
in components folder, create a file "baseController.php" and paste below code.
<?php
namespace app\components;

class BaseController extends \yii\rest\ActiveController{
	public $modelClass = ''; // no single table dependent

	public function init(){
        parent::init();
    }

	public function actions() {
	}

	protected function verbs() {
	   $verbs = parent::verbs();
	    $verbs =  [
			'index' => ['GET', 'POST', 'HEAD'],
			'view' => ['GET', 'HEAD'],
			'create' => ['POST'],
			'update' => ['PUT', 'PATCH'],
			'anyOtherAction' => ['DELETE'],
		];
	   return $verbs;
	}
}
?>

in userController (or every controller)
class SessionController extends \app\components\BaseController {
  // your code goes here
}


Contributing

Hi,
I would like to create a PR for a small enhancement, however there is no contributing.md so could you give me some direction on your PR process.
(The PR I want to create is simply to add  a link to the market on the order pages so market column entries link directly to the relevant trade page 'trade/${market-pair}' instead of being plain text.
This allows the user to navigate directly to a specific market instead of having to go via then menu & then selecting market from dropdown. Its a small change but I  find this quite useful & I think other users might too.)

Please submit a PR. We will review it.


Fix for scatter plot issue.

Currently if you try to use
lc.scatter(c='C3')
It breaks because it tries to add a colorbar to something that isn't colorbar-able. This fixes the issue by checking if the color kwarg is a string.

Thanks @christinahedges!  Do you think we can add lc.scatter(c='C3') to the existing plotting unit test?

Codecov Report

Merging #250 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #250   +/-   ##
=======================================
  Coverage   89.04%   89.04%
=======================================
  Files          24       24
  Lines        3168     3168
=======================================
  Hits         2821     2821
  Misses        347      347

Impacted Files
Coverage Δ

lightkurve/lightcurve.py
93.59% <100%> (ø)
: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 fab7054...2fe09ab. Read the comment docs.

@barentsen can we merge this?

Looks good!  Thanks @christinahedges !


jsx to js

Can we use it as js and not jsx? I don't know or like jsx very well.

electron-vite-boilerplate is better for you.


Close Pull Request doesn't work

Looks like there was a missing type check.

Not sure what the repro here is, but I checked out a PR and ran the GitHub Pull Request: Close Pull Request command and it did nothing.
Output:
[Info] Found GitHub remote
[Info] Review> current branch tyriar/kv is associated with pull request #495
[Info] Review> Resolving pull request
[Info] Review> Fetching pull request data
[Info] Review> register comments provider
[Info] Review> display pull request status bar indicator and refresh pull request tree view.

I was using version 2020.8.39781 in a WSL workspace

Not sure if this made it into the afternoon release (1pm) but this looks fixed on macOS. Double checking on WSL.

Yep! should have made it into the release then


lumberg create

May I know how to set a package when creating a new cpanel user?
result = server.account.create(
  :username => 'sample',
  :domain   => 'sample.com',
  :password => 'samplesamplesample',
  :savepkg => false,
  :reseller => false,
  :language => 'en',
  :pkgname => 'package_name',
  :contactemail =><EMAIL_ADDRESS>)

I have an error
{:message=>"Sorry, you must choose a package", :params=>{:status=>0, :options=>nil, :statusmsg=>"Sorry, you must choose a package", :rawout=>nil}, :success=>false}

Thank you In advance :)

Actually, you're not using the correct param. You should be using :plan => ''. pkgname is for creating a new package. See this page for more details http://docs.cpanel.net/twiki/bin/view/SoftwareDevelopmentKit/CreateAccount.


Мore localization flexibility

Thanks for the helpful library!
But the list of translations is limited. It would be nice if the Countries class could be passed to the widget constructor as a parameter for more localization flexibility.

Any updates on this one? I need to add hebrew translations to counties names


Add external logging support

Currently we only log to stdout which are accessible via the commit-boost logs command. We should add support to persist logs to file, with log rotation enabled

resolved by #48


DNS Port not open

Problem/Motivation
Installed with no issues, but port is not open in the docker

(Why the issue was filed)

Expected behavior
nslookup double.com <IP_ADDRESS> (my server)
Query returns <IP_ADDRESS>
nslookup bing.com <IP_ADDRESS> (my server)
Query also fails

(What you expected to happen)

Actual behavior
nslookup double.com <IP_ADDRESS> (my server)
Query fails

(What actually happened)

Steps to reproduce

Install hassio
Install pi-hole

(How can someone else make/see it happen)

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

Never mind, went away after reboot.


Hugo minify breaks sitemap.xml

What version of Hugo are you using (hugo version)?
v0.75.1 linux/amd64 BuildDate: 2020-09-15T14:24:46Z
Does this issue reproduce with the latest release?
Yes. (Latest is v0.75.1 as of now)

With hugo --minify, hugo minifies everything including sitemaps.
Validatos give error: XML declaration allowed only at the start of the document.
Generally 1st line is meant only for xml tags declatarion : <?xml version="1.0" encoding="utf-8" standalone="yes"?> but minify makes everything in one line.
This also affects google search console's sitemap submittions and shows error as Sitemap could not be read.
p.s. Also is there any way we could disable minification for certain path or something ?

https://gohugo.io/getting-started/configuration/#configure-minify

Just in case you missed it (it's pretty invisible) there is a search field in the documentation site in the navigation bar. Maybe that field should be made white so be more prominent. Searching for "hugo --minify" turns up the link above that solves your issue.

https://gohugo.io/getting-started/configuration/#configure-minify

Oh that's great! completely missed it. Thank you :)


Amazon app store version out of date

I just downloaded DuckDuckGo Android on Amazon app store and noticed that the app version is 3.1.0. Is this something you might be able to update on Amazon app store?

Thanks for reporting this. We aren't currently publishing to the Amazon app store but will investigate if we can start again soon.

@arunsathiya we are not publishing our app to the Amazon App. However we publish it into the F-droid app store -> https://f-droid.org/en/packages/com.duckduckgo.mobile.android/
Thanks.


Result of New-SSHShellStream cannot be traced back to SSHSession object

Hi
Feature request:  would it be possible to incorporate something into either SSHSession or SSHShellStream objects that allow them to be related to each other?
I'm finding it difficult detecting whether an SSHShellStream session is still connected, and also whether it's connected to the host I'm expecting it to be connected to.
If I could relate a SSHShellStream to an SSHSession, I could for example use the "connected" property of the SSHSession to get a reasonable estimate of whether I should try to send commands over the SSHShellStream.  For now, I'm tearing down and creating a new SSHShellStream too often, which is a slow operation.

would something like this be what you would like?
PS C:\> $sshstream = New-SSHShellStream -SessionId 0
PS C:\> $sshstream | fl

SessionId     : 0
Session       : Renci.SshNet.SshClient
DataAvailable : True
CanRead       : True
CanSeek       : False
CanWrite      : True
Length        : 79
Position      : 0
CanTimeout    : False
ReadTimeout   :
WriteTimeout  :

PS C:\> $sshstream.Session

ForwardedPorts           ConnectionInfo                        IsConnected KeepAliveInterval
--------------           --------------                        ----------- -----------------
{}                       Renci.SshNet.Connecti...                     True 00:00:10

That would be most useful, yes!

ok I will push that to 1.7.2 and make a github release and update the module in the PowerShell gallery


book: use cargo add when adding dependencies

i also think there are more places in the book where dependencies are introduced.

Noted, and updated.

Thanks!

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:
Thanks for your kind patience for guiding me through this.


rollup packages are not deployed

hello
I have used this package to write a tool which installed Windows embedded updates (collected by WEDU)
recently the updates started to come as rollup big updates.
unfortunately there seems to be a problem and the rollup updates are not installed
here is a log extract about the update:
2017-05-10 15:26:57, Info                  DISM   API: PID=3616 TID=3648 Input parameters: Session: 178, PackagePath: C:\SD_Updates\Updates\OS\AutoUnattend_Files\offlineServicing\amd64~package_for_rollupfix~~~~7601.23545.1.4~1.0\Windows6.1-KB3185278-x64.cab, IgnoreCheck: 0, PreventPending: 0, CancelEvent: 0x00000000000005A8, Progress: 0x000000001B00B98C, UserData: 0x0000000000000000 - DismAddPackageInternal 2017-05-10 15:26:57, Info                  DISM   API: PID=3616 TID=3648 C:\SD_Updates\Updates\OS\AutoUnattend_Files\offlineServicing\amd64~package_for_rollupfix~~~~7601.23545.1.4~1.0\Windows6.1-KB3185278-x64.cab is not a directory, assuming that this is not an expanded package - DismAddPackageInternal 2017-05-10 15:26:57, Info                  DISM   API: PID=3616 TID=3648 Successfully enqueued command object - CCommandThread::EnqueueCommandObject 2017-05-10 15:26:57, Info                  DISM   API: PID=3616 TID=936 ExecuteLoop: CommandQueue signaled - CCommandThread::ExecuteLoop 2017-05-10 15:26:57, Info                  DISM   API: PID=3616 TID=936 Successfully dequeued command object - CCommandThread::DequeueCommandObject 2017-05-10 15:26:57, Info                  DISM   DISM Provider Store: PID=1304 Getting Provider DISM Package Manager - CDISMProviderStore::GetProvider 2017-05-10 15:26:57, Info                  DISM   DISM Provider Store: PID=1304 Provider has previously been initialized.  Returning the existing instance. - CDISMProviderStore::Internal_GetProvider 2017-05-10 15:26:57, Info                  DISM   DISM Package Manager: PID=1304 Initiating Changes on Package with values: 4, 7 - CDISMPackage::Internal_ChangePackageState 2017-05-10 15:28:20, Info                  CSI    00000001 Shim considered [l:256{128}]"\??\C:\Windows\Servicing\amd64_microsoft-windows-servicingstack_31bf3856ad364e35_6.1.7601.17514_none_678566b7ddea04a5\pkgmgr.exe" : got STATUS_OBJECT_PATH_NOT_FOUND 2017-05-10 15:28:20, Info                  CSI    00000002 Shim considered [l:250{125}]"\??\C:\Windows\WinSxS\amd64_microsoft-windows-servicingstack_31bf3856ad364e35_6.1.7601.17514_none_678566b7ddea04a5\pkgmgr.exe" : got STATUS_SUCCESS 2017-05-10 15:28:20, Info                  DISM   DISM Package Manager: PID=1304 CBS has requested a shutdown. - CDISMPackageManager::Internal_Finalize 2017-05-10 15:28:20, Info                  DISM   DISM Package Manager: PID=1304 DISM has detected a DISM component change. Requesting a shutdown. - CDISMPackageManager::Internal_Finalize
this only happened to the rollup updates I have downloaded

KB3185278
KB3185330
KB3197868
KB3207752
KB3212646
KB4012215
KB4015549

the path is valid and so is the package, tested by 7zip.
thank you

i found it was a problem at the tool i wrote, sorry

No problem, sorry I didn't reply.  Glad its working for you now.


package_info readme file is wrong: dependency constraint single quote

Hello, just to note that in the package readme on github and pub the dependency constraint to allow a smoother ecosystem migration uses single quote instead of double quotes:
WRONG
package_info: '>=0.4.0+16 <2.0.0'
CORRECT
package_info: ">=0.4.0+16 <2.0.0"

In which file do you read this?

Hello!
For example here:
https://pub.dev/packages/package_info

I would like to close this issue with my PR.

I'm not sure where you saw that ' was wrong; both versions are supported. See https://dart.dev/tools/pub/dependencies.


Add Luminous Scans

Source name
Luminous Scans
Source link
https://luminousscans.com/
Language
English
Other details
Please add this!! thank you!
Acknowledgements

[x] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist on the website extensions list or the app.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

🙏🙏🙏🙏🙏

🙏🙏🙏🙏🙏
Aniyomi only add anime extension not manga/ manhwa go to tachiyomi GitHub( but they already removed that extension when translations group requested so it not be added again you should read this  https://github.com/tachiyomiorg/tachiyomi-extensions/blob/master/REMOVED_SOURCES.md )


The Lord of the Rings: The Return of the King (2003) low FPS

The game is slow and has an unstable FPS of 60-25
System information

GPU: Nvidia GTX 1070
Driver: 451.67
Wine version: i use Windows
DXVK version: 1.7

Log files
ROTK_d3d11.log
ROTK_dxgi.log

Please try the latest master. You can find a build here: https://git.froggi.es/doitsujin/dxvk/-/jobs
If it's still slow, make an apitrace so we can take a look at what the game is doing.
Here's a guide how you make one: https://github.com/Joshua-Ashton/d9vk/wiki/Making-a-Trace
You'll notice that the download link in that guide is dead. Use this one instead: https://ci.appveyor.com/api/projects/jrfonseca/apitrace/artifacts/build/apitrace-latest-win32.7z?branch=master&job=msvc32

Please try the latest master. You can find a build here: https://git.froggi.es/doitsujin/dxvk/-/jobs

Tried the latest master 89d36e1d. No change, same problems.
Log files
ROTK_d3d11.log
ROTK_dxgi.log

If it's still slow with that, make an apitrace so we can take a look at what the game is doing.

Put the files d3d11.dll and dxgi.dll
The game won't start, no log files are generated.

I was able to start the game with d3d8.dll
Here is the .trace file
https://mega.nz/file/EZg02KpA#9Iz2KMM1V7aJVXHEeEN9ADHSaej4MPn5QaO9jLKwMsw

Launched the game with dgvoodoo2 d3d8.dll
Then I took the files from apitrace: d3d11.dll, dxgi.dll, dxgitrace.dll
The game started, I managed to choose the language, and then the logo and everything was written down. I waited 10 minutes and no reaction.
Here is the update .trace file
https://mega.nz/file/xExjXJBY#OzXeMGIo4o6PonzFTEc5idPftGG1zjOqHlukih1iISU

Thank you very much. As of the latest 1.7.2 from https://git.froggi.es/doitsujin/dxvk/-/jobs, the game works fine. There are no FPS drawdowns.


Enhance the Alert Message

The current version of the website, doesn't have proper Alert messages.

Go through each and every functionality and add the proper Alert message with appropriate message & appropriate styling of the respective to the functionality of that feature.
Feel free to reach out for any queries.

Instructions :

Make sure to mention in detail what all changes or enhancements you have made in the PR
Provide the Screen shots of the changes implemented.
Make sure the changes made are compatible on all screen sizes and devices and are professional to use.
Resolve any conflicts in the code by checking once the PR is generated.

@TheOpenInnovator
do you want message like when we hover over a functionality and it shows a message about the functionality? or anything else?

please clarify the doubts and assign this task to me. @TheOpenInnovator

Hey @gt951436  Pls add some note in this website : https://theopeninnovator.github.io/Zen-Note/ and try to delete the entered Note. Then an Alert Message will generate.
Based on that create similar kind of alert messages for this Productivity for whichever functionality required with proper styling and sizes. Make sure to add any functionality or other  changes which can make the overall website better and interactive.
Feel free to reach out  for any queries.


Add new SI unit for velocity (m/s)

Name (Abbreviation) of the unit you want to create?
mps
What is its base SI unit?
new base unit
Would you like to provide more information about your unit?
No response
Contributor Guide

[X] I will comply with the Contributor Guide

Code of Conduct

[X] I have read and agree to the project's Code of Conduct

@thielec this is a test comment


Binary Content Matching is odd - cross platform/arch

Smells
Cross CI provider and cross arch has shown a few inconsistencies in content-type matching, which is clearly demonstrable here. Haven't thought of a solution just yet, but this at least highlights and allows us to test more combinations than before
Encountered in https://github.com/pact-foundation/pact-js-core/pull/444#issue-1692947149
If you have code that looks like this, there is a definite code-smell
https://github.com/pact-foundation/pact-js-core/blob/facfef810036d0b80fb3c2fbcafd8efa255b6171/test/consumer.integration.spec.ts#L23-L34
The condition is used in these tests
https://github.com/pact-foundation/pact-js-core/blob/dac0e46eafc5cdf633f459adc591e9cc5ba26c24/test/consumer.integration.spec.ts#L77-L87
https://github.com/pact-foundation/pact-js-core/blob/dac0e46eafc5cdf633f459adc591e9cc5ba26c24/test/message.integration.spec.ts#L94-L109

See https://github.com/pact-foundation/pact-reference/issues/171 for why we have an OS switch here
Windows: does not have magic mime matcher, uses content-type
OSX on CI: does not magic mime matcher, uses content-type
OSX: has magic mime matcher, sniffs content
Linux: has magic mime matcher, sniffs content

So based on our compat table of

OS
Amd64
Arm64

Linux (glibc)
✅
✅

Mac
✅
✅

Windows
✅
❌

Linux (alpine)
❌
❌

✅ = supported
❌ = not supported
We have a few different ways we can test to cover different arch combinations

Locally
Locally with GitHub Actions / Act
GitHub Actions
Locally with Cirrus CLI
Cirrus CI

OS
Amd64
Arm64
Amd64 (GH)
Amd64 (GH Act)
Amd64 (Cirrus)
Arm64 (Cirrus CI)
Amd64 (Cirrus CLI)
Arm64 (Cirrus CLI)

Linux (glibc)
❓
❓
❓
❓
❓
❓
❓
❓

Mac
❓
❓
❓
❌
❓
❓
❓
❓

Windows
❓
❌
❓
❌
❓
❌
❓
❌

Linux (alpine)
❌
❌
❌
❌
❌
❌
❌
❌

❓ = needs test evidence
❌ = not supported, or untestable on platform

@YOU54F Running into a similar issue on Mac Arm64:
error<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
error Found incompatible module.

hi this isnt the same issue, you are running an old version of pact-node which has never supported arm64

you can run pact-node with rosetta or vote on this issue to bring native arm support to pact-node
or migrate to pact-js-core which supports arm64 on macos and linux

fyi this related issue is where rosetta was supported for pact-node on macos
https://github.com/pact-foundation/pact-js-core/issues/264
youll need a version bump to 10.x as described in the linked issue to get it working with rosetta
👍🏽

@YOU54F Thank you sir, let me look into this

So it appears installing shared-mime-info in arm64 containers will result in the same behaviour on Linux
Debian
apt install shared-mime-info

Alpine
apk add shared-mime-info

Fixed as part of https://github.com/pact-foundation/pact-reference/pull/429
will be sorted in 0.4.21 release of the pact_ffi which can then be consumed in here

Fixed in

Pact-JS-Core from v15.0.0
Pact-Core from from v13.0.0


Admin\Operator - External User can see operator's user access grid

Description:
As an external user, I can see the users that have access to my operator
Acceptance Criteria:
Given I am on the dashboard
When I click on users and access requests
Then I am brought to the users and access requests grid
Figma
Development Checklist:

[x] copy/paste over this feature and tests from reg1
[x] update any links that broke (might need to add dashboard in some places)

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[x] User story is included
[x] User role and type are identified
[x] Acceptance criteria are included
[x] Wireframes are included (if required)
[x] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[x] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[x] Reviewed and approved by Product Owner

Notes:
Dependencies

Blocked by
Blocking

@zoeyli-46 could we update the Figma link to the correct wireframe. I'd like to refine and prep this ticket for next sprint. Thanks!

Done!

Having a "Search" field to filter the table was not required for Reg part 1. Is it necessary for Reg part 2? Should we implement it now?
cc: @patriciarussellCAS @andrea-williams

If it will factor into the decision at all, I don't think search is necessary (I just have it included on every grid in the designs) because it's not likely that operators will have that many users that they need to search for a user.
If anything, the "sort" functionality is probably more important so that the user can "sort" the table so that they can see the requests at the top, unless we implement it in a way that by default requests show up at the top.

Sort functionality already implemented!

Thanks @Sepehr-Sobhani and @zoeyli-46 - this works as described.
My only review note is that "BCeID Business" should actually be called "Business BCeID" (that's the official name, and how it's displayed on our main BCIERS log in page). Is it possible to make that minor change now before I approve?
I agree that we likely don't need a search function here but the ability to sort by columns will be sufficient.

Thanks @Sepehr-Sobhani and @zoeyli-46 - this works as described. My only review note is that "BCeID Business" should actually be called "Business BCeID" (that's the official name, and how it's displayed on our main BCIERS log in page). Is it possible to make that minor change now before I approve?
I agree that we likely don't need a search function here but the ability to sort by columns will be sufficient.
Sepehr - can you confirm that all the relevant items in the DOD are complete?

Since this feature is already in DEV, I will create a quick task ticket to cover that renaming tweak.
Regarding the DOD, I've checked all the relevant ones except the one about QA because I'm unsure if it applies now.


Reduce index lookups

Ahh yes, auto build hiccup. I managed to re-run the build, and it passed. I had to log in to see the re-run button.


Fixed versioning

If using lerna and wanting fixed mode versioning is there a way to get this to respect that?

I'm afraid not. Fixed versioning doesn't really fit into the semantic-release-monorepo workflow; it is designed to run against each individual package.
That said, it shouldn't be too difficult to achieve something like that. You might use regular semantic-release and replace its publish plugin with one that runs lerna's publish using the release version that semantic-release calculates.

Cool thanks. I ended up getting my project working with independent versioning for now. The key was that my CI (Drone) wasn't putting the repo path into the PATH and as such my npmrc with my custom auth wasn't being picked up.


Open Editors and Explorer tree does not remain synced

Latest Insiders
vscode-python 2018..8.0-alph0
im working on a network drive with a large project.
Its not specific to a particular file. I see the issue in 2 scenarios.
Ctrl-P to open a recent file will open the file, but the Explorer tree will not sync to it.
Switching files in Open Editors will not sync the Explorer tree.
This scenario does work.
Scrolling to find a file in Explorer tree, and open it. In this case when you select the file in the Open Editors the Explorer tree will sync.
You end up with a list of Open Editors.. some of which will sync in Explorer and some wont.

Can you reproduce if you disable all extensions?
If yes, please provide a step by step reproducable case. Or a even better a short gif showing the issue because your description is not clear


knife bootstrap - curve name mismatched

erratic@laptop ~ $ knife bootstrap -x root <IP_ADDRESS>
Connecting to <IP_ADDRESS>
ERROR: Net::SSH::Exception: curve name mismatched (`������1
x��I��1C����� b����' with `')
erratic@laptop ~ $

erratic@laptop ~ $ knife --version
Chef: 11.14.6
erratic@laptop ~ $

erratic@laptop ~ $ ruby --version
ruby 1.9.3p547 (2014-05-14 revision 45962) [x86_64-linux]
erratic@laptop ~ $
erratic@laptop ~ $ gem --version
2.2.2

erratic@laptop ~ $ gem list --local

*** LOCAL GEMS ***

bigdecimal (1.1.0)
bundler (1.7.2)
bundler-unload (1.0.2)
chef (11.14.6)
chef-zero (2.2)
coderay (1.1.0)
diff-lcs (1.2.5)
erubis (2.7.0)
executable-hooks (1.3.2)
ffi (1.9.3)
ffi-yajl (1.0.2)
gem-wrappers (1.2.5)
hashie (2.1.2)
highline (1.6.21)
io-console (0.3)
ipaddress (0.8.0)
json (1.5.5)
libyajl2 (1.0.1)
method_source (0.8.2)
mime-types (1.25.1)
minitest (2.5.1)
mixlib-authentication (1.3.0)
mixlib-cli (1.5.0)
mixlib-config (2.1.0)
mixlib-log (1.6.0)
mixlib-shellout (1.6.0.rc.0)
net-ssh (2.9.1)
net-ssh-gateway (1.2.0)
net-ssh-multi (1.2.0)
ohai (7.2.4)
plist (3.1.0)
pry (0.10.1)
rack (1.6.0.beta)
rake (<IP_ADDRESS>)
rdoc (3.9.5)
rest-client (1.6.7)
rubygems-bundler (1.4.4)
rvm (<IP_ADDRESS>)
slop (3.6.0)
systemu (2.6.4)
wmi-lite (1.0.0)
erratic@laptop ~ $

sshd_config:
PasswordAuthentication no
#PermitEmptyPasswords no

# Change to no to disable s/key passwords
ChallengeResponseAuthentication no
UsePAM no

The problem comes from Net::SSH advertising support for ED22519 host keys without actually supporting it. This is also causes a bunch of other issues when trying to use SSH servers with more modern algorithims. See https://github.com/net-ssh/net-ssh/issues/191 for more details.
Net::SSH 2.9.2 has been released with bugfixes, removing various algorithims from it's advertised list until the code is actually ready.

12.1.0 should fix this then.

Will there be a new chefdk release soon after 12.1.0?

this should have been fixed by picking up a new net-ssh.


How can i prevent some specific notifications?

I want to detect notifications for custom data and decide whether to show the banner or to prevent it, but before I can detect the notification - the app shows a banner.
How can I block some messages what I want?

send data notification and then call presentLocalNotification when you receive it.

@evollu thanks for your reply!!! I did it and it is working fine.


Molecule pretrain without SMILES

Dear developers,
Your example molecule lmdb data for the pre-train task have 3 required attributes: 'atoms', 'coordinates', 'smi'. But my own dataset only has 3D conformations, which means I don't have 'smi'. What should I do to remove this 2D graph from my pre-train pipeline? One way is to transform the 3D conformation into SMILES. But I think it's not suitable for my dataset, cause the 3D data is already enough for the pre-training.
Thank you.

One way is to transform the 3D conformation into SMILES. But I think it's not suitable.

yes, you can remove 2d graph dataset, also SMILES is not necessary, you can easily remove smi key from task and loss.

Nice one. I'll try it first, and continue to post here if there are new problems.

@Naplessss Hello, how are dist_mean and dist_std calculated?


Use the name of the class as step name.

When re-using the same cluster for submitting different jobs it can be
confusing using the default "Spark Step"-name. It would be clearer when
using the name of the class that is being used in the
sparkSubmit-statement

Cool! Would you like to edit the email in your commit to match the one you use on Github?

Hey, thanks for noticing that my e-mail wasn't correct... I've amended the commit but It looks like two people made that commit now. Maybe I'm doing something wrong here.
Btw, awesome plugin!


Question: getting dereferenced source schemas?

I have "Schema A" which defines some properties and "Schema B" which uses some $refs to Schema A. I add these schemas to an AJV instance such that when I do ajv.validate('schema-b', obj) the references are resolved and the schema does the validation. That's all working just fine. Now I have a use case where I need to use the source Schema B in another AJV instance that does not include Schema A. So if I do otherAjv.addSchema( originalAjv.getSchema('schema-b').schema ) the references will not be resolvable.
So, my question is: can I force a compile of all schemas in originalAjv such that when I get a source schema out of it all of the references will be de-referenced?

Compiling all schemas would not resolve the references, $ref just delegates the validation to another schema that is compiled on demand - the original schema remains unchanged. FAQ covers why it is the case.
So when you compile schema B, schema A gets also compiled become B uses it. But when you simply add schema object B to another instance, it cannot compile as it cannot find A anymore.
So you need to separately add all dependencies into the second instance.
Alternatively you could use asynchronous compilation, loading from one instance to another on demand (whatever the use case may be).


Update Readme to include instructions to install pods

Overview

Although it seems obvious, newer developers may not know to run install pods after installing this library. This PR adds a simple line to the readme file instruction the user to run pod install.
Test Plan

It's an update to a readme file. No testing needed.

@cmcaboy Thanks for the suggestion!
Though, the README already includes instructions for running npx pod-install to install Pod files for iOS, which is simpler and preferred way of linking the package.
React Native v0.60+
For iOS, use cocoapods to link the package.

run the following command:

$ npx pod-install

@Naturalclar The way the docs are laid out, it looks like npx pod-install is only applicable for react native 0.60 and above. There is nothing for 0.59 and below. pod instal is still required for 0.59 and below.

@cmcaboy For React Native versions 0.59 and below, autolinking of native modules are not available yet, so neither npx pod-install or pod install will work. You will need to run react-native link @react-native-community/clipboard .

@Naturalclar Running react-native link @react-native-community/clipboard adds the pod to your podfile, but it does not run pod install. The user has to explicitly run pod install after the link command in order to install the module. This is why I created the PR.

@cesargdm Thank you for approving this PR!


Feature/count vectorizer

This pull request implement countVectorizer.
const vectorizer = new CountVectorizer(["The quick brown fox jumped over the lazy dog."]);
const toVector = vectorizer.transform(["the brown brown","the the brown"]);

console.log(toVector)
// output
// [
//  [2, 0, 0, 0,0, 0, 0, 1],
//  [1, 0, 0, 0,0, 0, 0, 2]
//  ]

console.log(vectorizer.wordOrder)
//output
// {
//    brown: 0,
//    dog: 1,
//    fox: 2,
//    jumped: 3,
//    lazy: 4,
//    over: 5,
//    quick: 6,
//    the: 7
//  }

why is it showingMerging is blocked? request changes by @yorkie are now addressed in the new push

@steveoni sorry, I have approved this :)

@steveoni sorry, I have approved this :)

ok. I was confused though because normally after making some changes based on feedback, it does not show merging is blocked.
@yorkie

Yea, PR will be blocked util the change request is removed, I will update ASAP next time :)


Can't Access Azure Kinect via k4aviewer in Docker Container

I built a docker image based on Ubuntu 18.04 and OpenGL, installed k4a-tools, libk4a1.4-dev and setup the udev rules, which should satisfy all prerequisites to start Azure Kinect. However, when I launch k4aviewer, the terminal will return an error like:
[error] [t=33] /__w/1/s/extern/Azure-Kinect-Sensor-SDK/src/usbcommand/usbcommand.c (370): find_libusb_device(). Unable to open LIBUSB at index 0

Here was what showed in k4aview GUI:

After I clicked the Dismiss button, this would show:

However, if I run k4aviewer outside the docker container, it can access Azure Kinect and work perfect. I really need Azure Kinect to run in docker container which isolated from my laptop environment. Anyone has idea that how to make the camera work in docker environment?
Desktop (please complete the following information):

OS with Version: Ubuntu 18.04
SDK Version:  1.4
Firmware version:

These errors appear when the K4aViewer can not find a Kinect connected to the PC. Ensure a device is connected to avoid this error and ensure you have rules.d updated.

These errors appear when the K4aViewer can not find a Kinect connected to the PC. Ensure a device is connected to avoid this error and ensure you have rules.d updated.

Hey @wes-b,
Thanks for replying. I can definitely make sure that Azure Kinect connect to my PC. Here is what showed after run lsusb in docker container:

It's exactly the same as what showed in my host machine.
Actually I also tried simultaneously open 2 k4aviewer, one from docker container, one from my host machine. Azure Kinect would work on the host machine, but would report the error like what I said above in docker container.

These errors appear when the K4aViewer can not find a Kinect connected to the PC. Ensure a device is connected to avoid this error and ensure you have rules.d updated.

I just made the docker container successfully connect to Azure Kinect. To make it work, I should add --privileged to give extended privileges to the container when run docker image, like sudo docker run --privileged ...
However, the k4aviewer GUI still said it can not initialize audio backend (like the image I showed above).
Here is the GUI without audio parts in docker container:

It seems that the docker container may miss some software support (like audio driver) to run Azure Kinect audio parts. Do you got any idea about that?

Hi, @msr-peng did you manage to find a solution for microphone/audio part?

Hi Guys, I tried to simulate your jobs with a basic Ubuntu 18.04 image and failed to run k4aviewer with the error:
X11: The DISPLAY environment variable is missing,
I tried to set DISPLAY variable with 0.0 and 1.0,  but neither works with below error:
X11: Failed to open display
Any comments will be helpful!! And also if you could kindly share your docker image, that will be best!

What worked for me.
Docker container: Debian GNU/Linux 11
xhost local:docker # or :root
lsusb | grep 097 # must see 5 devices in host and in container

docker run options (thanks to materdd's comment):
--gpus 'all,"capabilities=compute,utility,graphics"' # instead of --gpus all
--privileged
--volume="/dev:/dev" # optional for USB devices
-e DISPLAY=$DISPLAY
-v /tmp/.X11-unix:/tmp/.X11-unix

Test viewer.
k4aviewer # must work in host computer: cameras (infrared, depth, color), IMU data, Microphone data
DISPLAY=:1 k4aviewer # in container - viewer opens up in host GUI (without microphone working)


Potential Improvement for environment.yml and Dockerfile

Thanks for setting up the environment @bfrizzell01. Just spotted 2 small issues:

deepchecks version confirmation in environment.yml
Automate the installation for deepchecks in Dockerfile

Please see comments on environment.yml and Dockerfile for detail.

Thanks @MasonZhang-MZ, I've addressed your comments: https://github.com/UBC-MDS/mushroom_classifier/pull/12


Fastlane Match Codesign for Frameworks is using the Login keychain, not the keychain I specified

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject

Fastlane Match

No

No
Question Description

I use Fastlane Match to codesign my App
The App has two Frameworks (internal, not third party)
When Fastlane Match codesigns the App, the Frameworks are codesigned with a Development Certificate. The codesign syle in the target settings for these frameworks is 'Automatic'. I tried specifying profiles, but that is not allowed.
We run this process in a CI environment with several build machines, and as part of this new process, we need to use a custom keychain, and we can't use the Login keychain.
When I run match, I specify the keychain_name, keychain_password parameters.
These arguments work fine for the main app with the app_identifiers I provide.
But when the codesign happens for the frameworks, I will receive a prompt to unlock the Login keychain, and this breaks our Jenkins pipeline.
Any suggestion?

You might be getting this problem if you did not configure the keychain you want to use as the default and unlocked.  you want to create the keychain with these params..
  desc "Create keychains to store certificates"
  lane :keychains do
    create_keychain(
      name: ENV["MATCH_KEYCHAIN_NAME"],
      password: ENV["MATCH_PASSWORD"],
      default_keychain: true,
      unlock: true,
      timeout: 3600,
      add_to_search_list: true
    )
  end

Thank you! That works


Req. generic masking shifts

I need to overload Swift's generic masking shifts to fix an issue in the standard library's implementation for non-power-of-two bit width integers. But, that said, I want to keep the overloads to a minimum. Adding requirements to ANKFixedWidthInteger, with a custom implementation in ANKFullWidth, makes it so core integers continue to use the standard library's implementation while ANKFullWidth receives the fix. Using ANKFullWidth in the context of FixedWidthInteger will result in the standard library's behavior.
static func &<<=(lhs: inout Self, rhs: some BinaryInteger)
static func &<< (lhs: Self, rhs:  some BinaryInteger) -> Self
static func &>>=(lhs: inout Self, rhs: some BinaryInteger)
static func &>> (lhs: Self, rhs:  some BinaryInteger) -> Self

I added protocol witness tests for ANKFixedWidthInteger in 762a850678967f99532feeba044167ba724dcbb9. Removing the aforementioned requirements will therefore cause some test cases to fail. If removing the requirements does not cause any test cases to fail, then  has fixed Swift's standard library implementation.


Offer a pre-commit hook that runs the .NET tool

.NET pre-commit hooks rely on dotnet pack, which uses the .NET SDK version specified in global.json. Relax the .NET SDK version constraint from ~7.0.100 to ^7.0.100, maximizing compatibility.

Thank you for the prompt review! It sounds like you are talking about .pre-commit-config.yaml, which indeed belongs in downstream repositories. This pull request only pertains to .pre-commit-hooks.yaml, which resides upstream and does not configure a repository to use pre-commit hooks. Nor does it serve as a reference for downstream repositories to configure their pre-commit hooks, but rather implements CSharpier as a pre-commit hook for consumption by downstream repositories. Pre-commit maintains a centralized list of a very small fraction of the repositories offering hooks. Downstream repositories can implement hooks locally as a workaround in cases where the upstream repository does not wish to support pre-commit. This largely defeats the goal of the tool though (to the extent that no such hooks may be disseminated via the aforementioned list). When the hook is supported upstream, this enables pre-commit to automatically manage development dependencies for a wide variety of programming languages and tools, including .NET tools. In the case of CSharpier, a local hook may require setting up .config/dotnet-tools.json and an additional hook to run dotnet tool restore. There are some pitfalls to avoid if you don't want to push hassle onto the end developer that I won't get into, and everyone ends up iteratively figuring it out for themselves (or not realizing it's possible), so no two solutions are the same. On account of the convenience of being able to run thousands of different tools written in any of the most popular programming languages on any repository by adding four lines of YAML to a config file, pre-commit has spread like wildfire, reaching 9.6k stars since its initial release in 2017.