Why Docker?

Using this action, I come to see that GitHub Docker actions are generally inefficient.
For e.g., if I understand correctly, it seems lake an intermediate container is created by GitHub for ever LABEL in the dockerfile  😬
Is there a specific reason(s) that the action was created as a Docker action rather than a composite action?
Would be interested to try to help to convert to composite action if there are no specific reasons for being a  Docker action and there is no objection.

Yeah, Docker generally builds in layers, but seems like GitHub is doing something a little different when building an action.
See an example here: https://github.com/exercism/c/runs/4097894299?check_suite_focus=true
Scroll up and expand the step for "Build<EMAIL_ADDRESS>Docker starts by pulling layers as you have mentioned:
  Step 1/12 : FROM lycheeverse/lychee:0.7.1@sha256:ff7c1b7490ed7e3cf5afc42ddf7ed32321291ae3e8c313214a074bb171117a3b
  docker.io/lycheeverse/lychee:0.7.1@sha256:ff7c1b7490ed7e3cf5afc42ddf7ed32321291ae3e8c313214a074bb171117a3b:
  Pulling from lycheeverse/lychee
  a330b6cecb98: Pulling fs layer
  40a0245be6dc: Pulling fs layer
<PHONE_NUMBER>99: Pulling fs layer

But then (and maybe I misunderstand this) GitHub is running a new container for each instruction in the Dockerfile:
  Step 2/12 : LABEL maintainer="Matthias Endler<EMAIL_ADDRESS>   ---> Running in bd4b79e0ea4c
  Removing intermediate container bd4b79e0ea4c
   ---> 4c8d000f406e
  Step 3/12 : LABEL repository="https://github.com/lycheeverse/lychee-action"
   ---> Running in 5693ae0d3a1a
  Removing intermediate container 5693ae0d3a1a
   ---> a0470fb08afa
  Step 4/12 : LABEL homepage="https://github.com/lycheeverse/lychee-action"
   ---> Running in a4438f1b0327
  Removing intermediate container a4438f1b0327
   ---> a9d5ee4e142f
  Step 5/12 : LABEL com.github.actions.name="Link Checker"
   ---> Running in 17447d43370a
  Removing intermediate container 17447d43370a
   ---> f82cba5035cb
  Step 6/12 : LABEL com.github.actions.description="Quickly check links in Markdown, HTML, and text files"
   ---> Running in f3a07dc7782b
  Removing intermediate container f3a07dc7782b
   ---> d37542ba4401
  Step 7/12 : LABEL com.github.actions.icon="external-link"
   ---> Running in 1d7ed59f7eaa
  Removing intermediate container 1d7ed59f7eaa
   ---> ea74aaf6e140
  Step 8/12 : LABEL com.github.actions.color="purple"
   ---> Running in 4c0c08724849
  Removing intermediate container 4c0c08724849
   ---> 75d84c6cc6c7
  Step 9/12 : COPY README.md /
   ---> ba10efdead6e
  Step 10/12 : COPY LICENSE-MIT LICENSE-APACHE /
   ---> 47c967a514d1
  Step 11/12 : COPY entrypoint.sh /entrypoint.sh
   ---> eb6c41209b6d
  Step 12/12 : ENTRYPOINT ["/entrypoint.sh"]
   ---> Running in 58e561523ac6
  Removing intermediate container 58e561523ac6
   ---> 95aa5c38f9d6

I immediately don't see a good alternative for binaries however, I will need to do some research!

I think intermediate containers are normal.

Every command, in the Dockerfile, that could potentially alter the state of the image (such as RUN, since Docker cannot know if our command has side effects) produces an "intermediate image". That is, every time such a step is encountered an image will be created that holds the state produced by all the previous commands.

https://blog.hipolabs.com/understanding-docker-without-losing-your-shit-cf2b30307c63
Guess I'd get the same output locally.
That said, I think there should be a way to not build the Docker image at all and only used a cached version.

Idea on how to use the Github Actions cache with Buildkit (end of the document):
https://github.com/docker/build-push-action/blob/master/docs/advanced/cache.md

Looks like this can be closed.

Believe it or not I haven't forgotten about this  😄 .
I'm still not sure how to continue though.
Is the question on a binary due to Lychee being implemented in Rust, and Rust being a compiled language, and composite Actions seem to really be fit for interpreted scripts (such as Python, JS, Bash, etc) ?

Not sure I understand. Which question are you referring to?
You can't run raw binaries as Github actions from what I can tell and the intermediate Docker layers are normal. Building the Docker image is already pretty fast so I don't know what we should change here. 🤔

Sorry, I'm being really basic here; Im trying to understand why there is a question about a binary?
I think (but am not sure) it is because:

Lychee itself releases as a binary and so this Action uses a Docker file that uses that binary
Lychee is released as a binary because it is implemented in Rust which is a compiled language
Compiling a program in a composite action is expected to be slower than using the binary in a Docker action

Do I understand correctly?
I'm not saying your conclusion is wrong, I'm just trying to understand more about Lychee, the Lychee Action and Actions in general 🙂

Your three points are correct.


Make easy to customize default pagination

Customizing pagination in a page list template is currently difficult. This change provides options to Pagination#renderDefaultView method.
Customization example:
<?php if ($showPagination): ?>
    <?php
    $pagination = $list->getPagination();
    if ($pagination->getTotalPages() > 1) {
        echo $pagination->renderDefaultView([
            'prev_message' => '«',
            'next_message' => '»',
            'css_active_class' => 'current-item',
            'css_prev_class' => 'prev-item',
            'css_next_class' => 'next-item',
            'css_container_class' => 'ccm-pagination'
        ]);
    }
    ?>
<?php endif; ?>

Related forum post:
http://www.concrete5.org/community/forums/5-7-discussion/page-list-pagination-styling/

+1

I have improved the way this works and incorporated something similar to this. Check out the latest commit. You can pass arguments into the renderDefaultView but it's easier to swap out the default view without having to worry about setting up route collections for the pagination library, etc... I've also separated pagination into "application" and "dashboard" contexts so you can swap out pagination on the front-end while keeping the backend bootstrap3-driven.

Great! Glad to hear that.


How to fill Title and Version field via Actix plugin ?

Actix plugin allow to add summary and description for each call but I don't see any way to fill the info part of generated OpenAPI file.
    "openapi": "3.0.0",
    "info": {
        "title": "",
        "version": ""
    },

Is it possible ?

Absolutely ! thank you !


aqueduct create fails

Creating a new project based on template aqueduct create fails. with Bad state: No element. Looks like the aqueduct package isn't found even though pub cache directory is in path and aqueduct has been activated globally.
Bad state: No element
**** Stacktrace

#0      ListMixin.firstWhere (dart:collection/list.dart:150:5)
#1      CLIAqueductGlobal.aqueductPackageRef (package:aqueduct/src/cli/commands/create.dart:342:10)
#2      CLIAqueductGlobal.templateDirectory (package:aqueduct/src/cli/commands/create.dart:347:12)
#3      CLIAqueductGlobal.getTemplateLocation (package:aqueduct/src/cli/commands/create.dart:351:12)
#4      CLITemplateCreator.handle (package:aqueduct/src/cli/commands/create.dart:51:27)
#5      CLICommand.process (package:aqueduct/src/cli/command.dart:158:20)

#6      CLICommand.process (package:aqueduct/src/cli/command.dart:134:12)
#7      main (file:///C:/Users/kismet-cai/AppData/Local/Pub/Cache/hosted/pub.dartlang.org/aqueduct-3.2.1/bin/aqueduct.dart:9:27)
#8      _startIsolate. (dart:isolate-patch/isolate_patch.dart:299:32)
#9      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

Hi @kismet-cai,
How did you solve this issue?
Thanks in advance!

I recall that I had to edit the PUB_CACHE environment variable to point to the correct location.


Electrs is not compatible with Embassy backups

I got my shared folder set up and initiated a backup. electrs was fourth on the list and has been spinning for at least 30 min.

Right, I haven't implemented backups yet.  Still to do:
Fix proxy autoconfig rules
Fix health check
Fix backups
Fix readme
Fix instructions
Do something about memory usage

backups work as of 0572772c6e4a210ed5db9f5552fb4ac6b109dc4e


feat(note): add scientific and Helmholtz notation getters

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 12 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.03%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
315

Relevant Lines:
335

💛  - Coveralls


Tracing abstraction

The abstraction is based heavily off of http://htrace.incubator.apache.org.
The terminology is based on google dapper.

Trace: The overarching unit of sleuth.  A tree of spans.  Has an id.
Span: Nodes in a trace tree.  Edges represent parent/child relationship.  Spans without a parent are root spans.  Has a name, id, trace id, parent span ids, start and end times.
Annotations: Data associated with a span
Sampler: Determines when to trace

API
Trace is in interface with DefaultTrace is an implementation.  It's job is to start Spans appropriately.

TraceScope startSpan(String name)
TraceScope startSpan(String name, Span parent)
TraceScope startSpan(String name, Sampler<T> s)

TraceScope is a class that defines behavior for the life of a Span.

public TraceScope(ApplicationEventPublisher publisher, Span span, Span savedSpan)
public Span detach()
public void close()

You'll notice the ApplicationEventPublisher in the constructor.  Currently two spring application events are published: SpanStartedEvent and SpanStoppedEvent, which are self explanatory.  If have implemented listeners that for setting the trace id and span id via Slf4j MDC so those values can be shown in logs.  This replicates the 4financeit correlationid that was printed in the logs.  This is also where publishing spans to various pipelines can be implemented: Zipkin, cloudfoundry, jdbc etc...  Because it is event driven, multiple entities can consume spans.
Sampler interface.  Allows customization of when to trace.  A default Sampler is an argument to DefaultTrace.
Sampler

boolean next(T info)

Instrumentation
Instrumenting code is the next important step in tracing.  There is instrumentation for hystrix, spring scheduling, spring mvc controllers, servlet filter, RestTemplate and Zuul.  There are two major ways of instrumenting code, via AOP and via lifecycle hooks (spring mvc, RestTemplate).
@mstine @dsyer @joshlong
@marcingrzejszczak I'd like you to review and ask any questions you might have about how this is different from the implementation you committed.  I believe everything you had is still there.  What you called Correlation-Id is now analogous to Trace-Id, you just wouldn't care for the Span-Id's if you didn't want to.  It's pluggable now.

I'm on it - 70 files changes... dis gonna be good :P

Maybe I'm wrong but we have also this:
https://github.com/4finance/micro-infra-spring/blob/master/micro-infra-spring-base/src/main/groovy/com/ofg/infrastructure/web/correlationid/CorrelationIdAspect.java
Where we try to ensure that if someone is using Servlet 3.0 support with the Callable<> in Controllers that we pass the proper correlationid. I don't see it here - is it deliberate or did you solve it by some other means?

I wondered about Callable in Controllers, right now I trace ALL controllers.  But you're concerned with the thread there right?

I've replaced references to correlationId with references to tracing.

Yeah exactly - we've also traced all controllers since we had a filter. However a new thread is spawned and you loose correlationID info. That's why we did the aspect.

OK, I'll test that and make sure Callables returned from controllers still work.

That would also be an issue when Observable support https://github.com/spring-cloud/spring-cloud-netflix/pull/246 is in spring cloud netflix.

Definitely...

Htrace has some wrappers that could be used there with developer education.

Awesome! That would be great!

@marcingrzejszczak added fix for returning Callable from a Controller.

With Callables and Controllers it seems that everything from our side is there.
BTW @dsyer 2 questions:

what next could be merged from our side?
From our perspective it would be cool if you released some 0.0.1 non snapshot version of both Zookeeper and Sleuth. That way we would plug it in into our stuff to see what's missing and how it's actually working.

WDYT?

@marcingrzejszczak I'll look at a milestone soon.

BTW @spencergibb  - why did you remove all the spock tests? :P you could have reused them


fix how filesystem deals with windows UNC paths

dlt version
0.4.7
Describe the problem
Windows UNC paths ie \\localhost\c$\my_dir are mangled both by filesystem source and a destination.
Expected behavior
They can be used like any other path
Steps to reproduce
in the code below it is clear that UNC path got completely mangled
import os
import posixpath
from typing import Iterator
import dlt
from dlt.sources import TDataItems
try:
    from .filesystem import FileItemDict, filesystem, readers, read_csv  # type: ignore
except ImportError:
    from filesystem import (
        FileItemDict,
        filesystem,
        readers,
        read_csv,
    )
TESTS_BUCKET_URL = "file://////cifs-fin/oracle_erp_extracts/FNPD/Avetta/In"
def load_csv() -> None:
    """Demonstrates how to scan folder with csv files, load them in chunk and merge on date column with the previous load"""
    pipeline = dlt.pipeline(
        pipeline_name="standard_filesystem_csv",
        destination='snowflake',
        dataset_name="file_data",
    )
    data_file = readers(
        bucket_url=TESTS_BUCKET_URL, file_glob="VendorProvince.csv"
    ).read_csv()
    load_info = pipeline.run(data_file.with_name("data_file"),write_disposition="replace")
    print(load_info)
    print(pipeline.last_trace.last_normalize_info)

Error:
  File "c:\bgisrepos\dlt_pipeline_flows\.venv\Lib\site-packages\dlt\extract\pipe_iterator.py", line
270, in _get_source_item
    pipe_item = next(gen)
                ^^^^^^^^^
  File "C:\bgisrepos\dlt_pipeline_flows\filesystem\readers.py", line 30, in _read_csv
    with file_obj.open() as file:
         ^^^^^^^^^^^^^^^
  File "c:\bgisrepos\dlt_pipeline_flows\.venv\Lib\site-packages\dlt\common\storages\fsspec_filesystem.py", line 237, in open
    opened_file = self.fsspec.open(
                  ^^^^^^^^^^^^^^^^^
  File "c:\bgisrepos\dlt_pipeline_flows\.venv\Lib\site-packages\fsspec\spec.py", line 1293, in open
    f = self._open(
        ^^^^^^^^^^^
  File "c:\bgisrepos\dlt_pipeline_flows\.venv\Lib\site-packages\fsspec\implementations\local.py", line 197, in _open
    return LocalFileOpener(path, mode, fs=self, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "c:\bgisrepos\dlt_pipeline_flows\.venv\Lib\site-packages\fsspec\implementations\local.py", line 322, in __init__
    self._open()
  File "c:\bgisrepos\dlt_pipeline_flows\.venv\Lib\site-packages\fsspec\implementations\local.py", line 327, in _open
    self.f = open(self.path, mode=self.mode)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'C:/bgisrepos/dlt_pipeline_flows/cifs-fin/oracle_erp_extracts/FNPD/Avetta/In/VendorProvince.csv' (edited)

Operating system
Windows
Runtime environment
Local
Python version
3.11
dlt data source
No response
dlt destination
Filesystem & buckets
Other deployment details
No response
Additional information

reproduce the error
fix the error
we need to add test where UNC is used along current file:// bucket and run them on windows. mind that any windows path can be expressed as UNC path ie \\localhost\c$\my_dir

@rudolfix, okay, I think I reproduced it:

Figuring out what's wrong there.

@rudolfix, I wonder what's the meaning of this line:
https://github.com/dlt-hub/dlt/blob/20664b2917d3d6b11c3572c3109f13335fb8d416/dlt/common/storages/fsspec_filesystem.py#L287

@IlyaFaer this strips leading "//" from the front but IDK why.
I think this part:
# make that absolute path on a file://
        if bucket_url_parsed.scheme == "file" and not file.startswith("/"):
            file = f"/{file}"
        file_name = posixpath.relpath(file, bucket_url_no_schema)
        file_url = bucket_url_parsed._replace(
            path=posixpath.join(bucket_url_parsed.path, file_name)
        ).geturl()

destroys some of the local paths
you'll need to debug that more

@rudolfix, yes, I brought a couple of fixes into your code, but I see that fsspect package code is also breaking the path in some way. Checking what's wrong there yet...


[Concurrency] Implement global actor isolation checking for conformances

Witnesses and requirements need to agree on their global actor
annotations. However, this is not true for 'async' or '@asyncHandler'
witnesses, for which it does not matter what the actor annotation is
because part of the contract is that the function will execute on the
appropriate actor.

@swift-ci please smoke test


Scala versioning

Objectives: Simplify, preserving ability to have multiple Scala versions, make a single Scala version very easy.

Remove cross-Scala versioning from rules
Add typelevel Scala test
Consolidate bootstrapping scala compilation into basic_scala_library
Create @scala (default), @scala_annex_scala_2_12, and //external:scala_annex_scala

Looks good so far-- I need a bit more time to read through the whole thing. Additionally this needs a rebase with master.

Any more thoughts before I merge?

LGTM


updating my passphrase

please test this

please test this

please test this

please test this

please test this

build failure has nothing to do with this patch.


Reworked job_control.py

@mmacduff can you test the new job_control.py to see if it still works in the image

Closes #26


Python 3 bug in postSonarr.py?

I believe I have found a Python 3 bug. I am using Sonarr with your script on and Arch Linux install. I have followed the instructions in the README.md file. The outputted mp4s are not being tagged and I get the following info in the info.log; see below.
When I change the first line of postSonarr.py from #!/usr/bin/env python to #!/usr/bin/env python2 everything works fine. In Arch Linux 'python' defaults to python 3.
Let me know if there is any other info that could help you or if this is simply me not knowing what I am doing.
Cheers
2015-12-28 12:18:20 - SonarrPostConversion - INFO - Sonarr extra script post processing started.
2015-12-28 12:18:20 - readSettings - WARNING - Audio bitrate was invalid, defaulting to 256 per channel.
2015-12-28 12:18:20 - SonarrPostConversion - INFO - Processing /media/tv/Blindspot/Season 1/Blindspot - S01E02 - A Stray Howl WEBDL-1080p.mkv.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Reading video stream.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Video codec detected: h264.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Pix Fmt: yuv420p.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Reading audio streams.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Audio detected for stream #1: ac3 [eng].
2015-12-28 12:18:20 - mkvtomp4 - INFO - Creating audio stream 0 from source stream 1.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Reading subtitle streams.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Subtitle detected for stream #2: srt [eng].
2015-12-28 12:18:20 - mkvtomp4 - INFO - Creating subtitle stream 0 from source stream 2.
2015-12-28 12:18:20 - mkvtomp4 - INFO - Starting conversion.
2015-12-28 12:18:31 - mkvtomp4 - INFO - /media/tv/Blindspot/Season 1/Blindspot - S01E02 - A Stray Howl WEBDL-1080p.mp4 created.
2015-12-28 12:18:32 - SonarrPostConversion - INFO - Tagging /media/tv/Blindspot/Season 1/Blindspot - S01E02 - A Stray Howl WEBDL-1080p.mkv with ID 295647 season 1 episode 2.
2015-12-28 12:18:32 - tvdb_mp4 - ERROR - Failed to connect to TVDB, trying again in 20 seconds.
Traceback (most recent call last):
File "/var/lib/sonarr/mp4_automator/tvdb_mp4.py", line 26, in init
self.tvdb_show = Tvdb(interactive=False, cache=False, banners=True, actors=True, forceConnect=True, language=language)
File "/var/lib/sonarr/mp4_automator/tvdb_api/tvdb_api.py", line 433, in init
import requests_cache
ImportError: No module named 'requests_cache'
2015-12-28 12:18:52 - tvdb_mp4 - ERROR - Failed to connect to TVDB, trying again in 20 seconds.
Traceback (most recent call last):
File "/var/lib/sonarr/mp4_automator/tvdb_mp4.py", line 26, in init
self.tvdb_show = Tvdb(interactive=False, cache=False, banners=True, actors=True, forceConnect=True, language=language)
File "/var/lib/sonarr/mp4_automator/tvdb_api/tvdb_api.py", line 433, in init
import requests_cache
ImportError: No module named 'requests_cache'
2015-12-28 12:19:12 - tvdb_mp4 - ERROR - Failed to connect to TVDB, trying again in 20 seconds.
Traceback (most recent call last):
File "/var/lib/sonarr/mp4_automator/tvdb_mp4.py", line 26, in init
self.tvdb_show = Tvdb(interactive=False, cache=False, banners=True, actors=True, forceConnect=True, language=language)
File "/var/lib/sonarr/mp4_automator/tvdb_api/tvdb_api.py", line 433, in init
import requests_cache
ImportError: No module named 'requests_cache'
2015-12-28 12:19:32 - tvdb_mp4 - INFO - Tagging file: /media/tv/Blindspot/Season 1/Blindspot - S01E02 - A Stray Howl WEBDL-1080p.mp4.

Looks like your python3 installation is missing the requests-cache package. Running pip install requests-cache should fix this error.

I have installed requests-cache and I am now no longer getting the prevision error. Thank you for your help.
However I am now getting a different error and PLEX is not getting notified to update. Please see the output of info.log below. Like before if I change back to python2 I no longer get this error.
I am using "PLEX home" so I am using the token option. I double check and everything in autoProcess.ini looks correct. Not sure if this has anything to do with it or not.
Is this a case of me not knowing what I am doing again?
Thank you for your help
2015-12-29 22:06:09 - SonarrPostConversion - INFO - Sonarr extra script post processing started.
2015-12-29 22:06:09 - readSettings - WARNING - Audio bitrate was invalid, defaulting to 256 per channel.
2015-12-29 22:06:09 - SonarrPostConversion - INFO - Processing /media/tv/Blindspot/Season 1/Blindspot - S01E01 - Pilot WEBDL-1080p.mkv.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Reading video stream.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Video codec detected: hevc.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Pix Fmt: yuv420p.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Reading audio streams.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Audio detected for stream #1: aac [eng].
2015-12-29 22:06:09 - mkvtomp4 - INFO - Creating audio stream 0 from source stream 1.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Reading subtitle streams.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Subtitle detected for stream #2: ass [eng].
2015-12-29 22:06:09 - mkvtomp4 - INFO - Creating subtitle stream 0 from source stream 2.
2015-12-29 22:06:09 - mkvtomp4 - INFO - Starting conversion.
2015-12-29 22:29:47 - mkvtomp4 - INFO - /media/tv/Blindspot/Season 1/Blindspot - S01E01 - Pilot WEBDL-1080p.mp4 created.
2015-12-29 22:29:47 - SonarrPostConversion - INFO - Tagging /media/tv/Blindspot/Season 1/Blindspot - S01E01 - Pilot WEBDL-1080p.mkv with ID 295647 season 1 episode 1.
2015-12-29 22:29:48 - tvdb_mp4 - INFO - Tagging file: /media/tv/Blindspot/Season 1/Blindspot - S01E01 - Pilot WEBDL-1080p.mp4.
2015-12-29 22:29:48 - tvdb_mp4 - INFO - Trying to write tags.
2015-12-29 22:29:48 - tvdb_mp4 - INFO - Tags written successfully.
2015-12-29 22:29:48 - mkvtomp4 - INFO - Relocating MOOV atom to start of file.
2015-12-29 22:29:48 - qtfaststart - INFO - Removing free atom at 32 (8 bytes)
2015-12-29 22:29:48 - qtfaststart - INFO - Patching stco with 60637 entries
2015-12-29 22:29:48 - qtfaststart - INFO - Patching stco with 60636 entries
2015-12-29 22:29:48 - qtfaststart - INFO - Patching stco with 1 entries
2015-12-29 22:29:48 - qtfaststart - INFO - Writing output...
2015-12-29 22:29:48 - SonarrPostConversion - ERROR - Unable to refresh plex, check your settings.
Traceback (most recent call last):
File "/var/lib/sonarr/mp4_automator/autoprocess/plex.py", line 34, in refreshPlex
xml_sections = minidom.parse(urllib.urlopen(base_url))
AttributeError: module 'urllib' has no attribute 'urlopen'

Looks like another python2 to 3 bug with urllib. Just pushed an update that should resolve the issue. Give it a try.

I just tested it and everything look to be working correctly.
Thank you for all your help
On Wed, Dec 30, 2015 at 7:29 AM, Michael Higgins<EMAIL_ADDRESS>wrote:

Looks like another python2 to 3 bug with urllib. Just pushed an update
that should resolve the issue. Give it a try.
—
Reply to this email directly or view it on GitHub
https://github.com/mdhiggins/sickbeard_mp4_automator/issues/386#issuecomment-168011216
.


1.1.5 NoPorts  Design Nits

Is your feature request related to a problem? Please describe.
Here are some suggestions, if time allows!
Describe the solution you'd like
Dashboard

[x] Can we move the settings icon to match the padding to the left (where the logo is)

Settings

[x] Can we get make the hover of the toggle black (not purple) #25252525

[x] Can we get the checkbox to be on the left side, in order to match
the one above "Overide all profiles...."

[ ] Replace the advanced/ simple screens with ones that match the In-app UI (@Dariamarg)

Edit/ Create a Profile

[ ] Can the submit button be suppressed until all fields are entered?

[x] For custom relays can we add some text in the box along the lines of 'Custom' and add a radio dial to show that it is a selection (for example)

[ ] Can we also have the fields be fixed closer to each other (for example in fullscreen 'Device atSign' and 'Device Name' have a large gap); just have them be side by side

Here are the advanced and simple view screenshots
NoPorts Marketing Material (Dashboard).zip

@Dariamarg is this what you have in mind?

@Dariamarg Can the submit button be suppressed until all fields are entered? Currently if the use click submit it will trigger a validation error on all the fields that have errors.

@Dariamarg this is how the UI looks on a large screen now, It looks the same on smaller screen, I don't like this UI on a large screen, what do you recommend?

Her is another view with it centered. I'll have to refactor the app bar so I aligns in the centre. I'll do so if centering the content on a large screen is in keeping with your recommendation.

I think centering is a great idea! Maybe have it at full width so that theres no scroll (like in the relay section)

@Dariamarg is this what you have in mind?

@Dariamarg Your thoughts on this.


Prisma panic on invalid int

Bug description
Prisma will panic and become unusable if I query for an Int value that is too large for int.
{
    "where": {
        "id":<PHONE_NUMBER>47645765674576
    }
}

How to reproduce
Create the simplest schema possible with an int field and perform any query (e.g. count) with a where clause that includes an int that is larger than Number.maxSafeInt().
Expected behavior
Should not render Prisma client unusable.
Prisma information
model User {
  id          Int      @id @default(autoincrement()) @db.Int
  name        String?  @db.VarChar(255)
}

Environment & setup
Reproducible on Ubuntu and Centos
MariaDB
Node 14.17.5
Prisma Version
3.11.1

3.15 will ship a change that returns a structured error instead of a panic.


where to execute code?

I'm following the example to get account by user name but I don't know where to put this code. Any help would be greatly appreciated.
$instagram = new \InstagramScraper\Instagram(new \GuzzleHttp\Client());
$account = $instagram->getAccount('myspacetom');

If you are in a deploy/testing environment, put in index.php and check your codes!
BUT if you are in a production environment, It could be better to use Queues for your requests and process in the background.


Improve unmarshalling performance

Related to highsource/jsonix-support#6
@bartvde Here's an issue to address performance problems.

thanks @highsource do you think it's the stack based approach that the ol3 parser uses or something else?

"stack based approach that the ol3 parser uses" - not sure what this is, would you happen to have a specific pointer for me?
I've profiled around a bit, but, unfortunatelly don't see a single specific problem. However there are few things which may altogether contribute to inferior performance:

Support for QNames - Jsonix effectively has to parse all the attributes looking for namespace declarations. This can be made configurable so that you could disable QNames support if you need.
Support for xsi:type - also means all the attributes must be read. Can be disabled.
Detecting whitespace - surprizingly seems to take much time. Can be optimized.

I see there are good chances to improve performance, but it'll take effort. No quick wins here, unfortunately.
A few years ago I had a similar case (competing vs. OL2 back then). With a few targeted tweaks I could match/overcome OL2 performance. As far as I remember, attributes were quite an issue back then as well as detecting blank strings.

there is some explanation on the stack based approach here: https://github.com/openlayers/ol3/tree/master/src/ol/format

No, don't think this would help much. I'll run a few experiments later on. There's definitely very good potential.


Why?

Why would I use this instead of vanilla js?

it's better

it's better

it's better

Can't argue with that!


Questions about dataset

Is there any tutorial to load the dataset and rebuild the dynamics of the non-agent car, and I can't find any training code in this repo?

Hi @AliBaheri  You might misunderstood my question. Actually I want to restore the given imitation learning data in Carla and make a replay on it.


[BUG] every request reinitializes LoopResources in HttpClientPluginConfiguration#httpClient

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
every request reinitializes LoopResources in HttpClientPluginConfiguration#httpClient
Expected Behavior
LoopResources just init once.
Steps To Reproduce

run ShenyuAdminBootstrap;
run ShenyuTestHttpApplication;
run ShenyuBootstrapApplication;
send http request:

POST http://localhost:9195/http/shenyu/client/hello
Accept: application/json
Content-Type: application/json

Environment
ShenYu version(s): current

Debug logs
nop
Anything else?
nop

@dragon-zhang


demo project 405 (Method Not Allowed)

Hello, I'm trying to setup influxdb as a db for my traces.
While the traces are correctly pushed by the collector, Jaeger ui gives an error.
I believe there is something wrong in the jaeger-influxdb address, but I'm not sure.
I'm using influx db cloud InfluxDB Cloud powered by TSMStorage Engine Version 2 (167c4c6)

The following is the docker-compose that I'm using.
services:
  jaeger-query:
    image: jaegertracing/jaeger-query:1.49
    stop_grace_period: 10s
    ports:
      - "16686:16686" # web UI
    depends_on:
      - jaeger-influxdb
    environment:
      LOG_LEVEL: debug
      SPAN_STORAGE_TYPE: grpc-plugin
      GRPC_STORAGE_SERVER: jaeger-influxdb:17271
      GRPC_STORAGE_CONNECTION_TIMEOUT: 30s
      # QUERY_HTTP_SERVER_HOST_PORT: :16686
      # ADMIN_HTTP_HOST_PORT: :16687
      QUERY_UI_CONFIG: /jaeger-ui-config.json
    volumes:
      - ./jaeger-ui-config.json:/jaeger-ui-config.json:ro

  jaeger-influxdb:
    image: jacobmarble/jaeger-influxdb:0.5.9
    stop_grace_period: 10s
    environment:
      LOG_LEVEL: debug
      LISTEN_ADDR: :17271
      INFLUXDB_TIMEOUT: 30s
      INFLUXDB_ADDR: "us-central1-1.gcp.cloud2.influxdata.com"
      INFLUXDB_BUCKET: "beta-opentelemetry"
      INFLUXDB_BUCKET_ARCHIVE: "beta-opentelemetry-archive"
      INFLUXDB_TOKEN: [REDACTED]

  hotrod:
    image: jaegertracing/example-hotrod:1.41
    stop_grace_period: 1s
    ports:
      - "8080:8080" # web UI
    depends_on:
      - otelcol-influxdb
    environment:
      JAEGER_AGENT_HOST: otelcol-influxdb
      JAEGER_AGENT_PORT: 6831

  otelcol-influxdb:
    image: otel/opentelemetry-collector-contrib:0.87.0
    command: ["--config", "/config.yml"]
    stop_grace_period: 10s
    volumes:
      - ./otelcol-config.yml:/config.yml:ro

Logs of the containers with DEBUG level.
jaeger-influxdb-1   | {"level":"debug","ts":1709289390.728273,"caller":"internal/influxdb.go:218","msg":"executing query","query":"SELECT \"service.name\" FROM 'calls__sum' GROUP BY \"service.name\""}
jaeger-influxdb-1   | {"level":"debug","ts":1709289390.7285042,"caller":"internal/influxdb.go:218","msg":"executing query","query":"SELECT \"span.name\", \"span.kind\" FROM 'calls__sum' WHERE \"service.name\" = 'sawmill-be' GROUP BY \"span.name\", \"span.kind\""}
jaeger-influxdb-1   | {"level":"error","ts":1709289391.6135364,"caller":"jaeger-influxdb/main.go:110","msg":"gRPC interceptor","error":"Unknown: SqlState: \u0000\u0000\u0000\u0000\u0000, msg: [FlightSQL] unexpected HTTP status code received from server: 405 (Method Not Allowed); transport: received unexpected content-type \"text/html\" (Unknown; ExecuteQuery)","stacktrace":"main.run.func1\n\t/project/jaeger-influxdb/cmd/jaeger-influxdb/main.go:110\ngithub.com/jaegertracing/jaeger/proto-gen/storage_v1._SpanReaderPlugin_GetServices_Handler\n\t/go/pkg/mod/github.com/jaegertracing/jaeger@v1.49.0/proto-gen/storage_v1/storage.pb.go:1487\ngoogle.golang.org/grpc.(*Server).processUnaryRPC\n\t/go/pkg/mod/google.golang.org/grpc@v1.57.0/server.go:1360\ngoogle.golang.org/grpc.(*Server).handleStream\n\t/go/pkg/mod/google.golang.org/grpc@v1.57.0/server.go:1737\ngoogle.golang.org/grpc.(*Server).serveStreams.func1.1\n\t/go/pkg/mod/google.golang.org/grpc@v1.57.0/server.go:982"}

jaeger-query-1      | {"level":"error","ts":1709289391.6137898,"caller":"app/http_handler.go:493","msg":"HTTP handler, Internal Server Error","error":"plugin error: rpc error: code = Unknown desc = Unknown: SqlState: \u0000\u0000\u0000\u0000\u0000, msg: [FlightSQL] unexpected HTTP status code received from server: 405 (Method Not Allowed); transport: received unexpected content-type \"text/html\" (Unknown; ExecuteQuery)","stacktrace":"github.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).handleError\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:493\ngithub.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).getServices\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:164\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\ngithub.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).handleFunc.traceResponseHandler.func2\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:537\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\ngithub.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).handleFunc.WithRouteTag.func3\n\tgo.opentelemetry.io/contrib/instrumentation/net/http/otelhttp@v0.43.0/handler.go:273\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\ngo.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*middleware).serveHTTP\n\tgo.opentelemetry.io/contrib/instrumentation/net/http/otelhttp@v0.43.0/handler.go:217\ngo.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.NewMiddleware.func1.1\n\tgo.opentelemetry.io/contrib/instrumentation/net/http/otelhttp@v0.43.0/handler.go:81\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\ngithub.com/gorilla/mux.(*Router).ServeHTTP\n\tgithub.com/gorilla/mux@v1.8.0/mux.go:210\ngithub.com/jaegertracing/jaeger/cmd/query/app.createHTTPServer.additionalHeadersHandler.func4\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/additional_headers_handler.go:28\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\ngithub.com/jaegertracing/jaeger/cmd/query/app.createHTTPServer.CompressHandler.CompressHandlerLevel.func6\n\tgithub.com/gorilla/handlers@v1.5.1/compress.go:141\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2136\ngithub.com/gorilla/handlers.recoveryHandler.ServeHTTP\n\tgithub.com/gorilla/handlers@v1.5.1/recovery.go:78\nnet/http.serverHandler.ServeHTTP\n\tnet/http/server.go:2938\nnet/http.(*conn).serve\n\tnet/http/server.go:2009"}

jaeger-influxdb-1   | {"level":"error","ts":1709289391.6210887,"caller":"jaeger-influxdb/main.go:110","msg":"gRPC interceptor","error":"Unknown: SqlState: \u0000\u0000\u0000\u0000\u0000, msg: [FlightSQL] unexpected HTTP status code received from server: 405 (Method Not Allowed); transport: received unexpected content-type \"text/html\" (Unknown; ExecuteQuery)","stacktrace":"main.run.func1\n\t/project/jaeger-influxdb/cmd/jaeger-influxdb/main.go:110\ngithub.com/jaegertracing/jaeger/proto-gen/storage_v1._SpanReaderPlugin_GetOperations_Handler\n\t/go/pkg/mod/github.com/jaegertracing/jaeger@v1.49.0/proto-gen/storage_v1/storage.pb.go:1505\ngoogle.golang.org/grpc.(*Server).processUnaryRPC\n\t/go/pkg/mod/google.golang.org/grpc@v1.57.0/server.go:1360\ngoogle.golang.org/grpc.(*Server).handleStream\n\t/go/pkg/mod/google.golang.org/grpc@v1.57.0/server.go:1737\ngoogle.golang.org/grpc.(*Server).serveStreams.func1.1\n\t/go/pkg/mod/google.golang.org/grpc@v1.57.0/server.go:982"}

It looks like Storage Engine Version 2 doesn't support FlightSQL and I believe it's the root of the issue, I cannot find a way to create a storage engine version 3, I guess it's more a billing/account/organization problem or something rather than an issue with the plugin.

I'm closing the issue since I've been confirmed that InfluDB with storage engine V3 is currently supported only in AWS.


Support unique indexes when rebalancing ranks

The goal of this PR is to improve support for apps having a unique index on their ranked columns.
The problem we were seeing in our app is that rebalancing would fail if it attempted to move a record to a position that is already in use (see the example in specs).
I went with the simplest approach possible: see if the target value already exists and if it does, decrement the value until we find an available one. I believe it's reasonable to expect there is always breathing room available for us to work with. E.g. given a million records, the gap used for rebalancing is 4295.
Eager to hear your thoughts!

Hi @janklimo :) Thanks for putting this together.
That's an interesting problem. I wonder if an alternative would be to wipe the rank values before rebalancing? We're going to throw them away anyway, and the operation should be safe if done within a transaction. What do you think?

@brendon that's a good idea, should allow for a cleaner code. I'll look into it.

Thanks @janklimo, I have a flappy test that sometimes fails on Postgres (as you can see in the travis run). I've spent quite a while trying to debug it but I can never reproduce it locally.
There seems to be another failing combination with mysql. It looks like a config error. What do you think? :)

@brendon I figured out the problem with the failing jruby spec is that that early version of activerecord-jdbcmysql-adapter needs a corresponding version of jdbc-mysql otherwise it will fail with a later one. I'm looking into why the other spec is failing.

Thanks for your hard work @janklimo, Just a quick note, you need to edit the Appraisals file then run appraisal install (or update?) which will update the gem files automatically.
Did you make any headway on that flappy test? Let me know when this is ready to merge :)

Thanks for the update. Can you confirm you ran appraisal install? I assume you did and the file generated matched the one you modified manually? If not, commit that new file and I'll merge this for you :)

Hey @brendon yes I did and the output matched the changed I made previously.
Hard to say what made that one spec flaky. I cleaned it up a bit so I can debug what's wrong and it started to pass again so I never got to see where the problem was 😅
Just rebased the commits. Over to you 👍

So it failed again. I'll look into it. Wish I could assign myself/reassign you to make the responsibility clearer.

Haha, yea it's perplexing. Let me know if there's anything I can do to help RE travis :)

Yes that's as far as I got. The items are slightly out of order:
869, 867, 870, 1001 expected
869, 870, 867, 1001 was
(had to use a differ to spot the difference! :D)

That seems to have fixed it? I'm rerunning the test suit to be sure :)

Hi @brendon no, that's just to prove the error is caused by two records having the same row value before rebalancing. Which, now that I think about it, should never happen:

It leads to non-deterministic behavior when rebalancing - order will or will not be maintained. Either way, ranks will not remain equal after being rebalanced.
The gem has no API for setting the same value for two records - you need to resort to bypassing callbacks with update_all.

I remember acts_as_list has a section on data integrity. Maybe we should acknowledge the same best practice and have a similar recommendation?

Thanks @janklimo, that could be an idea. I did add a paragraph a while back around this concept:

IMPORTANT: Note that you MUST append _position to the column name when setting a new position on an instance. This is a fake column that can take relative as well as absolute index-based values for position.

It is a bit obscure and to be fair, when I started using this gem, I also got confused and tried to set the column directly.
At least your test clears up that flappy test. Postgres must be more sensitive to this issue for some reason. Are you happy to tidy up that test and back out the debug stuff and I can merge this for you :)

This looks great :) Thanks @janklimo :) I really appreciate your hard work on this.

I've released 0.4.6 for you.

You're the best, thank you for your help with this @brendon 💯

reset_ranks! attempts to set the rank column to nil. This fails when the rank column is set as NOT NULL.

Sigh! :D restrictions are fun eh! :D I guess the question is, do you need a NOT NULL restraint on a column completely managed by this gem? As far as the developer is concerned, the gem should be able to use this column as it requires as long as you have a stable outcome (ordering is correct and maintained). We nullify the scope in a transaction, so if it fails it'll roll back to its original state.
If you can find a good way around this, please let me know and I'd be happy to look at a PR :)

@brendon I could switch back to our previous strategy of not resetting ranks. Instead, we'd check if a rank is taken, then progressively lower the target rank by 1 until a unique value is available.

@janklimo, I'd be reluctant to do this as I think the nullifying solution works well and is cleaner. @abhitrivedi, is it possible to relax your restriction so that it only resolves after the transaction commits?


src: add APyFixedArray.to_bits()

PR Summary

PR Checklist

N/A "closes #0000" is in the body of the PR description to link the related issue
[x] new and changed code is tested
[x] relevant changes added to CHANGELOG.md
[x] new functionality is documented

@Theodor-Lindberg, here is the to_bits() implementation for APyFixedArray. Feel free to borrow material for a similar method to APyFloatArray.


feat: add fetchBlockList method to fetch who's blocking who

Overview
Adds the fetchBlockList method which can fetch a list of who's blocking who. A list of blocked users can be pulled by providing a blocker_fid or a list of users who blocked someone by providing the blocked_fid
Example
# index.js
const result = await client.fetchBlockList({blockerFid: 419871})

// Stringify and log the response
console.log(JSON.stringify(result));

$ node index.js | jq
{
  "blocks": [
    {
      "object": "block",
      "blocked": {
        "object": "user",
        "fid": 499008,
        "custody_address": "0x9744fda1e8cd653b7c15f4f2f1da541ca91f3699",
        "username": "ronnie-coleman",
        "display_name": "WieheißtduHastdugegessenttkkwwp",
        "pfp_url": "https://imagedelivery.net/BXluQx4ige9GuW0Ia56BHw/4594bb85-4ff1-4840-387b-495b6f8dbf00/rectcrop3",
        "profile": {
          "bio": {
            "text": "Eng"
          }
        },
        "follower_count": 4823,
        "following_count": 4061,
        "verifications": [
          "0x90b53660bd66d2b88587b0785bb43d032d917ccf",
          "0x3dd1dfb2f0533ed4c54d17915889d18e8bf871bf"
        ],
        "verified_addresses": {
          "eth_addresses": [
            "0x90b53660bd66d2b88587b0785bb43d032d917ccf",
            "0x3dd1dfb2f0533ed4c54d17915889d18e8bf871bf"
          ],
          "sol_addresses": []
        },
        "active_status": "inactive",
        "power_badge": false
      },
      "blocked_at": "2024-09-04T14:41:20.000Z"
    }
  ],
  "next": {
    "cursor": null
  }
}

@valerierose looks good. let's bump the SDK version


IS-138 Provider name is given for copyright as config setting

Closes #138
@oestrogen Another PR with frontend fixes needed. The /api/ui response may now contain a providerName field holding the application provider name to be displayed in the copyright statement.
If no providerName is received, or if no name is set for the current language, we don't display a copyright statement.
Also fixed developer.yml and sandbox.yml with missing metadata config. @felix-hellman Could you deploy to sandbox after this PR is merged?

I will update Sandbox when this is merged.


grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with status = StatusCode.INVALID_ARGUMENT

I am trying to test this library for a project and running into this issue where create_assessment call fails.
Environment details

OS type and version:
MAC OSX 10.15.7

Python version: python --version
python 3.8.6 (can be reproduced with python3.9.2)

pip version: pip --version
pip 21.0.1

google-cloud-recpatchaenterprise version: pip show google-cloud-recpatchaenterprise
Name: google-cloud-recaptcha-enterprise
Version: 0.3.0
Summary: UNKNOWN
Home-page: https://github.com/googleapis/python-recaptcha-enterprise
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages
Requires: google-api-core, proto-plus
Required-by:

Steps to reproduce

pipenv shell
pipenv install google-cloud-recaptcha-enterprise
export GOOGLE_APPLICATION_CREDENTIALS=<path-to-json-file>
Copy the example from here, replace relevant variables and run it.

Stack trace
Traceback (most recent call last):
  File "/Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages/google/api_core/grpc_helpers.py", line 73, in error_remapped_callable
    return callable_(*args, **kwargs)
  File "/Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages/grpc/_channel.py", line 923, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "/Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages/grpc/_channel.py", line 826, in _end_unary_response_blocking
    raise _InactiveRpcError(state)
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
	status = StatusCode.INVALID_ARGUMENT
	details = "Request contains an invalid argument."
	debug_error_string = "{"created":"@1617270971.707825000","description":"Error received from peer ipv4:<IP_ADDRESS>:443","file":"src/core/lib/surface/call.cc","file_line":1068,"grpc_message":"Request contains an invalid argument.","grpc_status":3}"
>

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "test_recaptcha_enterprise.py", line 24, in <module>
    response = client.create_assessment(request)
  File "/Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages/google/cloud/recaptchaenterprise_v1/services/recaptcha_enterprise_service/client.py", line 442, in create_assessment
    response = rpc(request, retry=retry, timeout=timeout, metadata=metadata,)
  File "/Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages/google/api_core/gapic_v1/method.py", line 145, in __call__
    return wrapped_func(*args, **kwargs)
  File "/Users/dev/.local/share/virtualenvs/test-google-recaptcha-enterprise-yBATfFin/lib/python3.8/site-packages/google/api_core/grpc_helpers.py", line 75, in error_remapped_callable
    six.raise_from(exceptions.from_grpc_error(exc), exc)
  File "<string>", line 3, in raise_from
google.api_core.exceptions.InvalidArgument: 400 Request contains an invalid argument.

The error can be reproduced with and without VPN.

Hi @hafeezibbad,
Could you try using the REST API instructions as well to see if you can reproduce the error? The python client library unfortunately makes it hard to access error details right now (see https://github.com/googleapis/python-api-core/issues/99). If you send a request directly to the REST API it may have additional details about which field is invalid.

Could you try using the REST API instructions as well to see if you can reproduce the error?

Hi, I can not reproduce the same error with API key.

I have noticed one more thing that if projects/ is not project_name, we get the same error.

I have noticed one more thing that if projects/ is not project_name, we get the same error.

I don't understand - were you not following the pattern projects/your_project_name when passing the project name?

Just had the same, your project_id might just be wrong for this call. Unfortunately the error message is so misleading that you have to try it in the REST API to get the full list of errors.

I don't understand - were you not following the pattern projects/your_project_name when passing the project name?

I created new service account, added recaptcha-related roles to the service account and that worked. While testing the new account, I found the issue that projects/ is necessary. This has not been communicated in the guide here.

Unfortunately the error message is so misleading that you have to try it in the REST API to get the full list of errors.

Yes, this is an issue. It makes debugging so difficult. I have not run into this issue with REST API at all.
One more thing to mention here, the data model for RestAPI is different from the data model in library, for example, Rest API does not include risk_analysis unlike python-recaptcha-enterprise library. So, there no intuitive direct mapping between Rest API and library's functionality.

I'm going to close this issue as obsolete as details is now populated with more helpful information showing "Invalid Parent resource name <PROJECT_ID>" instead of "Request contains an invalid argument".
In addition, the docstring of CreateAssessmentRequest mentions that the format of the parent field should be projects/{project}.
Traceback (most recent call last):
  File "/usr/local/google/home/partheniou/.pyenv/versions/py39/lib/python3.9/site-packages/google/api_core/grpc_helpers.py", line 72, in error_remapped_callable
    return callable_(*args, **kwargs)
  File "/usr/local/google/home/partheniou/.pyenv/versions/py39/lib/python3.9/site-packages/grpc/_channel.py", line 946, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "/usr/local/google/home/partheniou/.pyenv/versions/py39/lib/python3.9/site-packages/grpc/_channel.py", line 849, in _end_unary_response_blocking
    raise _InactiveRpcError(state)
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
        status = StatusCode.INVALID_ARGUMENT
        details = "Invalid Parent resource name 'xxxxx'"
        debug_error_string = "UNKNOWN:Error received from peer ipv4:<IP_ADDRESS>:443 {created_time:"2023-04-15T11:59:24.659107248+00:00", grpc_status:3, grpc_message:"Invalid Parent resource name \'xxxx\'"}"
>


SCB-346 Add third party licenses

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.1%) to 95.374% when pulling 69ac811eaa40a0091f8cce458f7d4ed3c7ea7a0c on yangbor:master into 86cee600dd8560b2ac6b17424c130d1f5abb5739 on apache:master.


boto3 execute_statementfor redshift

Kindly assist. After running below code as written in the boto3 documentation. I got the "AttributeError: 'Redshift' object has no attribute 'execute_statement'" error. What am I doing wrong?
import boto3
client = boto3.client('redshift-data')
response = client.execute_statement(
ClusterIdentifier='myCluster',
Database='dev',
DbUser='awsuser',
Sql='create table scorecard (name varchar20(),dept varchar30())',
StatementName='My Table',
WithEvent=False
)
/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/botocore/client.py in getattr(self, item)
573         raise AttributeError(
574             "'%s' object has no attribute '%s'" % (
--> 575                 self.class.name, item)
576         )
577
AttributeError: 'Redshift' object has no attribute 'execute_statement'

You may need to update your version of boto3 to have access to the new data-api methods.

Thanks @IanMeyers .I was able to upgrade and got it working.


What stands behind bufsize == 0 enforcement in subprocess pipes?

I see the issue https://github.com/python/asyncio/issues/119 but see no explanation. In practice I have annoying buffering (4kbytes or so, Linux, py3.4) which disallows me to get stdout lines as they actually appear. Which is very strange if you recall that bufsize forced to be 0 (behaviour is more like bufsize = -1).

Are you sure the buffering is in asyncio and not in the subprocess whose
stdout you are trying to read?
On Sunday, September 27, 2015, Vitalik Verhovodov<EMAIL_ADDRESS>wrote:

I see the issue #119 https://github.com/python/asyncio/issues/119 but
see no explanation. In practice I have annoying buffering (4kbytes or so,
Linux, py3.4) which disallows me to get stdout lines as they actually
appear.
—
Reply to this email directly or view it on GitHub
https://github.com/python/asyncio/issues/272.

--
--Guido van Rossum (on iPad)

Oh, yes, didn't expect usual print() will buffer being connected to pipe. Calling sys.stdout.flush() solved the problem.
from time import sleep
import sys

class Unbuffered(object):
    def __init__(self, stream):
        self.stream = stream

    def write(self, data):
        self.stream.write(data)
        self.stream.flush()

    def __getattr__(self, attr):
        return getattr(self.stream, attr)

sys.stdout = Unbuffered(sys.stdout)

for i in range(100):
    print(i)
    sleep(1)

It's standard Unix behavior: output is buffered by default unless connected to a tty device (which a pipe is not). In Python 3 you can also use print(blah, flush=True).


Don't try to redraw the game board while calculating next move

removed a call to refresh that was made right after requesting next move. Because of the asynchronous nature, it would render the board in the middle of searching for the next move.


Install grub

With KVM on the horizon, start installing grub to MBR

Turns out this is a bunch of work for not much gain, gonna pass here


[BUG] TypeError: (intermediate value).default is undefined

Bug Report Quick Checklist

[x] I am on the latest version of Snowpack & all plugins.
[x] I use package manager yarn (Fill in: npm, yarn, pnpm, etc).
[x] I run Snowpack on OS macOS 11.2.3 (Fill in: Windows, Mac, Linux, etc).
[x] I run Snowpack on Node.js v12+

Describe the bug
I had to skip Snowpack 3.2.0 because it had some rimraf issue but 3.3.0 seemed to build correctly. However, when I run it locally I see the following:
Unhandled Runtime Error

TypeError: (intermediate value).default is undefined
Source

http://localhost:8080/_snowpack/pkg/intl-messageformat.v2.2.0.js [:429:47]

core<@http://localhost:8080/_snowpack/pkg/intl-messageformat.v2.2.0.js:429:47
createCommonjsModule@http://localhost:8080/_snowpack/pkg/intl-messageformat.v2.2.0.js:10:5
@http://localhost:8080/_snowpack/pkg/intl-messageformat.v2.2.0.js:295:32

It is not something I'm directly depending on but something that react-aria seems to depend on. Haven't seen this issue before so I have to revert back to 3.0.13 for now.
To Reproduce

npx create-snowpack-app react-snowpack --template @snowpack/app-template-minimal
npm install react-aria
Edit the index.js to include react-aria, something like

import * as aria from "react-aria";
console.log(aria);

npm run start
See error!

Expected behavior
That the app would run correctly like it did with 3.0.13.

I hit this as well, looks like intl-messageformat has a known-ish (but closed) issue about one of their dependencies not supporting ESM yet:
https://github.com/formatjs/formatjs/issues/1913
Which people on that issue have been working around by pulling it from jspm.dev:
// In es-dev-server.config.js
module.exports = {
  middlewares: [
    function rewriteFastMemoize(context, next) {
      if (context.url === '/node_modules/fast-memoize/src/index.js') {
        context.status = 302;
        context.redirect('https://jspm.dev/fast-memoize');
        return;
      }
      return next();
    },
  ],
};

Not sure how to do that in snowpack.

FWIW the .default is undefined happens on this line:
es5.defineProperty(MessageFormat, '__parse', {value: intl$messageformat$parser$$["default"].parse});

Where the ...$parser$$... import is:
import intl$messageformat$parser$$ from '/_snowpack/pkg/intl-messageformat-parser.v1.4.0.js';

So for some reason, that import is returning undefined now, and sounds like it did not before, in snowpack v3.0.13.

So one module, intl-messageoformat-v2.2.0 is trying to look up ...other-modules...[["default]":
es5.defineProperty(MessageFormat, '__parse', {value: intl$messageformat$parser$$["default"].parse});

And when I look in intl-messageformat-parser-v1.4.0 source code (within dev tools), I see parse there:
var parser = (function() {
  ...lots of code...
  return {
    SyntaxError: peg$SyntaxError,
    parse:       peg$parse
  };
})();

export default parser;

So it looks like there should be a default export, but when I look at that in the debugger, it's just parse and SyntaxError directly, and no default key:

So, it looks like snowpack is getting confused about whether there's a default export on this module or not.
It looks like there should be (the source uses export default parser) and imported code thinks it should be (it's doing module["default"].parser) ... but the module that actually exists at runtime doesn't have it.
Where / how would this miscommunication be happening?

FWIW I've got the same codebase running on vitejs (I'm doing a shoot-out of the two systems). Not sure if that helps or not.


Determine Preferred Language when locale is coming from server

@PascalPrecht
HI Pascal

I Want to use browser locale when user is not login, in this case page would serve index.html. For this i have used $translateProvider.determinePreferredLanguage() that would translate according to browser locale, it's working very well.

Once user login, then home.html page would serve and there want to use user's locale.

So there is any way to determine preferred language on the basis of user's locale that's coming from server.
(function() {
'use strict'
var mainApp = angular.module("mainApp", [ "ngResource", "ngCookies", "pascalprecht.translate" ]);
mainApp.config(function( $translateProvider) {
$translateProvider.useStaticFilesLoader({
prefix : '/lang/',
suffix : '.json'
});
$translateProvider.determinePreferredLanguage(function () {
// here I want to call rest api that would return user locale for example              {"status":200,"locale":"de_DE"} and then switch locale according to rest response
             });
	$translateProvider.useCookieStorage();
});

})();

You just have to write your own logic on processing the REST response and then call and set the stack correctly with
``
$translate.use(....)

Oh.!! Thanks for quick reply.
@tspaeth  I am not able call resource in $translateProvider.determinePreferredLanguage function.
how can i inject $resource or $http services in that function.?

If you take a look at the documentation at https://docs.angularjs.org/guide/module, you'll see that basically, it is (better : should not be) not possible to use services in the configuration / bootstrapping phase. There are modules like https://github.com/philippd/angular-deferred-bootstrap or https://github.com/rajeshsegu/angular-lazy-bootstrap that might help here. But that is out of scope for our module.

Well. The $translate.use() must be implemented in your run() or any other controller/service. You should define a static (unless you expose this one directly in the html) language id, in order the app can show you text until your custom language negotiation has been resolved. Actually a classic "language stored in server side session" scenario.


[rust] check route equality and keep existing route in case of failure

…o query route from remote

Which Issue(s) This PR Fixes

Fixes #issue_id
Brief Description

In current implementation, the route is updated each time after querying remote queues. This will cause the index to be reset to zero that some queues won't be selected for polling or sending messages if the route is updated frequently. And in case of failure route requests, the route would be removed instead of being kept for availability.
The commit fixed these two issues above.
How Did You Test This Change?

add 2 test cases.

Codecov Report
Attention: Patch coverage is 93.58974% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 47.85%. Comparing base (430ed16) to head (50509fe).
Report is 1 commits behind head on master.

Files
Patch %
Lines

rust/src/client.rs
93.58%
1 Missing and 4 partials :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #710      +/-   ##
============================================
- Coverage     56.05%   47.85%   -8.21%
  Complexity      664      664
============================================
  Files           259      207      -52
  Lines         15856    12434    -3422
  Branches       4089     3786     -303
============================================
- Hits           8888     5950    -2938
+ Misses         6679     6191     -488
- Partials        289      293       +4

Flag
Coverage Δ

nodejs
?

rust
79.82% <93.58%> (+0.86%)
:arrow_up:

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

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


cannot parse proper ttl due to old owlapi version

The turtle parser included in owlapi version3 cannot parse curied identifiers of the form RRID:IMSR_JAX:000644. Owlapi version 4 and greater switched to use the Rio turtle parser which correctly handles these. The most expedient fix for this is probably https://github.com/SciGraph/SciGraph/issues/186.

See also https://github.com/SciCrunch/NIF-Ontology/commit/3f04cff65bfabe44239527c5336c71c850be76bd#diff-9b3526f6575cc729a142749e7ba60eff.


Increment build number, if desired, before compiling rather than after

Currently if options.incrementBuildNumber is set, the build task will increment the build number field in the pdxinfo file, but will only do so after compiling with pdc. This causes the resulting build to embed metadata with an obsolete build number, and another rebuild is needed for the updated build number to be included. The game metadata can be retrieved via playdate.metadata in Lua (which is how I found this).
This small change moves the increment build number step so that it happens in the task runner before the pdc compile step, and that way the resulting build will contain updated metadata from the pdxinfo file.

This will be published shortly in version 1.14.8. Sorry for the delay!


Periodic situation

I want to know if this net satisfy the Periodic situation?
When I did 222 expansion of CH2 in the example, the energy is not 8 times that before expansion.

I'm not the author, but I think it is because of the use of FAVOR (attention part of the paper).
In FAVOR part, attentions are calculated for real atoms but not for periodic images of atoms. it doesn't take into account minimum image conventions (I'm not check the source code but the equation in the paper shows that).
This means if you have 5 atoms in unit cell the possible number of attention is 5 * 5 = 25. when you expand the unit cell by 2 x 2 x 2, the possible number of attentions increases to (85) * (85) = 1600.
So that reason, it seems spooky-net isn't invariant for selection of unit cell.


Hi @naveed-iqbal

          Hi @naveed-iqbal

I was mistaken, this seems to be an issue with codesign. Can you check you've automatic signing enabled and also check your Xcode preferences and, make sure your account has all the certificates
please how do I find the "automatic signing"

Closing this issue, since no valid information has been provided.


lists should just be numbered, not bulleted and numbered

Don't use both bullets and numbers in https://github.com/axisgroup/evaluation-toolkit/blob/master/4.Conduct-the-test/Research-Methods/ExpertReview/README.md
Just use a numbered (ordered) list starting at zero using html, like this:

Item 0
Item 1
Item 2

can you check now?

Much better 👍
Andrew Nelson
andyhub.com
832-677-7180
(sent from mobile)
On Nov 30, 2017 18:15, "Manasvi Lalwani"<EMAIL_ADDRESS>wrote:

can you check now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/axisgroup/evaluation-toolkit/issues/65#issuecomment-348351866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuouupBi5-v1N9_YxNCPbUbYCUX69xgks5s7zb-gaJpZM4QxMTW
.


Create chocolatey package for svcat installation on Windows

It would be awesome if we had a chocolatey package to install svcat on Windows
(could have sworn I saw this as an issue in the past but cannot find it now).

/remove-lifecycle stale


feat: supported tokenize and truncate_prompt endpoints

Resolves #25

/deploy_review

/deploy-review

/deploy-review

/deploy-review


Multi task mzr

FAILED tests/test_evaluation/test_metrics/test_multi_task_metrics.py::MultiTaskMetric::test_evaluate - TypeError: list indices must be integers or slices, not str

Codecov Report

Merging #7 (70be3ae) into multi-task (404d15d) will decrease coverage by 0.04%.
The diff coverage is 89.15%.

@@              Coverage Diff               @@
##           multi-task       #7      +/-   ##
==============================================
- Coverage       89.04%   89.00%   -0.05%
==============================================
  Files             151      151
  Lines           11537    11455      -82
  Branches         1844     1836       -8
==============================================
- Hits            10273    10195      -78
+ Misses            997      995       -2
+ Partials          267      265       -2

Flag
Coverage Δ

unittests
89.00% <89.15%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

mmcls/structures/multi_task_data_sample.py
75.00% <50.00%> (-15.15%)
:arrow_down:

mmcls/models/utils/data_preprocessor.py
94.80% <60.00%> (-2.50%)
:arrow_down:

mmcls/models/classifiers/image.py
96.15% <75.00%> (-3.85%)
:arrow_down:

mmcls/datasets/transforms/formatting.py
90.08% <91.66%> (-0.99%)
:arrow_down:

mmcls/models/heads/multi_task_head.py
96.96% <94.59%> (+0.86%)
:arrow_up:

mmcls/datasets/transforms/__init__.py
100.00% <100.00%> (ø)

mmcls/evaluation/metrics/multi_task.py
94.59% <100.00%> (+3.21%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


The copyright logo is safe

Otherwise the & on the © gets turned into & which means
the footer reads as "© Crown copyright"

Is there a reason why we’re using & and not just © in the source?


fix(builder): if exit code from build pod is not 0 it must return an error

This avoid the creation of new releases that will end in a CrashLoopBackoff status

@aledbf testing this now

@arschles to test this you use something like this:
mkdir slugfailure
cd slugfailure
git init
echo "web: true" > Procfile
git add Procfile && git commit -m "Import 1"
deis create slugfailure && git push deis master

@aledbf the test you described works fine, as well as a similar test using https://github.com/deis/example-go


App gets killed

After a backgroundMode.enable();

I go to another app, check notifications, and the notification that the app is in background is indeed there. I can see the app is in background and if I come back to the app works as expected.
I turn off the screen, wait for a while
Come back and notifications is gone and the app has been killed

Any ideas?

I have the same problem.
In my case, the app is killed if i opened another apps.
I tried open the Facebook app, while my background app is running,
After a while, the background app is killed.
But, it is fine as long as I did not open another app.
Any ideas ?

I believe there is no way to prevent the app from being killed. The Operating system kills it when needed to free up resources. The best thing you can do it save your UI state into a temporary storage.


Context Service: routerContext always needs to be true to update the page title

Describe the bug
The routerContext boolean of the Context Service always needs to be set with the value true to update the page title.
This should not be needed, because this creates an extra listener on routechange.
To Reproduce
Steps to reproduce the behavior:

Add the ContextModule to your appModule

import { CookieconsentModule } from '@acpaas-ui/ngx-components/layout';

ContextModule.forRoot({
   defaults: {
      title: 'Context title',
   },
   routerContext: true,
}),

Expected behavior
routerContext shouldn't be needed.
OS (please complete the following information):

Device: MacBook Pro (Retina, 13-inch, Late 2013)
OS: macOS High Sierra (Version 10.13.6)

Picked up by ACPaaS frontend team

Solved in #123


Milestone 2:- Add submit & refresh features

I have added support for submitting & refresh features.
These changes allow the user to submit & refresh scores from the API.
This includes JS modules, functions, event listeners, and fetching & storing data functions from the API.

For WebHint linter error, please refer:-


fix wrong display after cocos2d-x-3.17.2

since cocos2d-x-3.17.2, the engine change some logic about premultipliedAlpha, if set false, the result of this image is 'hasAlphaPremultipledAlpha() == true'
here are some relative codes:
/** @def CC_ENABLE_PREMULTIPLIED_ALPHA

If enabled, all textures will be preprocessed to multiply its rgb components
by its alpha component.
*/
#ifndef CC_ENABLE_PREMULTIPLIED_ALPHA

define CC_ENABLE_PREMULTIPLIED_ALPHA 1
#endif
// premultiplied alpha for RGBA8888
if (color_type == PNG_COLOR_TYPE_RGB_ALPHA)
{
if (PNG_PREMULTIPLIED_ALPHA_ENABLED)
{
premultiplyAlpha();
}
else
{
#if CC_ENABLE_PREMULTIPLIED_ALPHA != 0
_hasPremultipliedAlpha = true;
#endif
}
}

I'm using 3.17.2, but I didn't find any problem?

behavior changed after 3.17.2 release, but version code on GitHub is also 3.17.2.


CVE in iris library

Iris which is used by multiple ODIM components is vulnerable to a few CVEs
github.com/kataras/iris/v12 v12.1.8 is vulnerable to:
CVE-2021-42576 [mitre]
CVE-2021-23772 [mitre]
CVE-2021-29272 [mitre]

This issue is resolved in ODIM services


Don't accept Linklocal address as IP .

Link Local addresses should not be assignable as IPs .
Tested By :
1. Configure Linklocal addres using the command
curl -g -k -H "X-Auth-Token: $bmc_token" -X PATCH -D patch.txt -d '{"IPv6StaticAddresses":[{"Address":"fe80::a94:efff:fe82","PrefixLength":64}]}' https://${bmc}/redfish/v1/Managers/bmc/EthernetInterfaces/eth0
2.Seeing that the link local address is getting configured in the GET command
curl -k -H "X-Auth-Token: $bmc_token" -X GET https://${bmc}/redfish/v1/Managers/bmc/EthernetInterfaces/eth0

Defect link:
https://jazz07.rchland.ibm.com:13443/jazz/web/projects/CSSD#action=com.ibm.team.workitem.viewWorkItem&id=546470

Can one of the admins verify this patch?


Task - Edit TaxonName - Move synonyms section not visible

The section to move synonyms (which show up after synonymy of a higher rank taxon) does not show up any more.

Probably another extend issue, looking


add something to install ycm reqs in virtualenv for devs

six & waitress, bottle, jedi

This would be a new Makefile target, similar to install-deps


Difference between regular and bundled file

I was looking all over website and here but wasnt able to find out: what's the difference? Would be nice to have a breakdown of what bundle includes. Website example just show including bundle script

The bundle is the core + the default display plugin. It's what 99% of the users need.


Keyboard shortcuts in user guide

Hey, I would really appreciate some keyboard shortcuts in the user guide.
For example 't' for the table of content and '?' for a overview of all shortcuts.
I don't want to always have to click on 'Table of contents'.
If someone is able to implement this with prototype, I'm not, I would really like it.

I've kept this as a "call for contributions" kind of issue, but after 3 years it's obvious that nobody is interested in submitting a patch, so closing.


Animation support

As of now, this library doesn't support the animation performed between switching of the view states.
Are you planning to include this feature in the future?
Shall I work on it & raise a PR?

Yes, I welcome to raise a PR and contribution code.👏
We can work together to complete the functions you propose.

Sounds fun :)
Thank you, will start working on it :)
💃


[New architecture] Not able to delete a draft emoji in composer

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

This issue is related to the ongoing new architecture adoption. It was reported on the build with new architecture enabled on the relevant PR.

Version Number: 1.4.42-1
Reproducible in staging?: no
Reproducible in production?: no
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @ishpaul777 here
Slack conversation:
Action Performed:
Break down in numbered steps
Expected Result:
Be able to delete draft emoji
Actual Result:
Deleting not possible
Workaround:
Should be fixed by https://github.com/Expensify/App/commit/2c02bb51fbbf809161e88fbb3d7a235f8aca9e81
Platforms:

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

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

Screenshots/Videos
Available in the linked comment.

View all open jobs on GitHub

Will be tested once the PR is deployed.


In case of mvn /npm cmd failure show stdOut in extension to user

Currently in case of any failure in resolving dependencies for mvn and npmextension just shows Command failed ... but doesn't show the exact error.
It would be useful if we parse STDOUT and show as error to the user in output channel of VSCode

closing as fixed


Install Vim plugins example

Hi there,
I installed codequery and it works fine as a standalone. The displayed 'vim-codequery' demo looks very powerful. I followed the instructions on how to install the Vim plugins. Per the installation example I used the 'Plug' plugin manager:

~/.vim/autoload$ curl -fLo ~/.vim/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim
~/.vim$ mkdir plugged
~/.vim/plugged$ git clone https://github.com/devjoe/vim-codequery.git
vim .vimrc and added (I missed adding the 'unity' plugin for the initial test)
call plug#begin('~/.vim/plugged')
Plug 'vim-codequery'
call plug#end()
invoke gvim (version 8.0.685) raised the following error:

"[vim-plug] Invalid argument: vim-codequery (implicit `vim-scripts' expansion is deprecated)"
Can both 'vim-codequery' and 'unity' plugins be installed without a plugin manager? It would be great if instructions were available somewhere.
If a plugin manager is required, can the installation example show a working case?
Thank you,
4zo4

Hi @4zo4 ,
Just change vim-codequery to devjoe/vim-codequery in step 4.
FYI, you can find the cause of this exception here.
Vim8 comes with a primitive plugin manager. Let me take some time to try.
I will rewrite installation part because of this issue and #18.
Thank you!

Oh, it's quite easy to use Vim8's package manager.
Installation guide was done in #18. So I closed this issue.
Still feel free to ask me any question.


Task runner and common dev tasks

Pick one, it doesn't matter: Grunt or Gulp? I vote Grunt, but will use either
Tasks:

JSHint
JSCS
Test Runner

+1 to Grunt

Cool. I can get started on this

That would be fantastic. I don't have much experience building scalable test benches so it'd be great to learn more about Grunt.

Working through this and I'm a little overwhelmed. These changes necessary to make these new quality control tools happy are massive.

There are a lot of bugs that will be fixed in this process, so care should be taken to test the patch thoroughly.


Add tests

While looking into how to run your project locally, I found your article
https://towardsdatascience.com/how-to-deploy-a-ruby-serverless-function-to-vercel-da8c3581f7c2

readme should probably have some instructions for running serverless locally,
readme update incoming.


active-directory-saas-teamphoria-tutorial.md classic azure portal that no longer available

Creating an issue
I guess note should be modified, because classic portal no longer available
">[!Note]:

This embedded guide is brand new in the new Azure portal, and we’d love to hear your thoughts. Use the Feedback ? button at the top of the portal to provide feedback. The older guide for using the Azure classic portal to configure this application can be found here.-->"

https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/active-directory/active-directory-saas-teamphoria-tutorial.md

Hey @SychevIgor - Thanks for your feedback! Could you provide this feedback by revisiting the original document page (on http://docs.microsoft.com) and using the "Give documentation feedback" feature in bottom of the page? That way, your valuable feedback is funneled through our appropriate internal processes where the content authors and subject matter experts will get notified, and issues will get resolved in timely manner.


still errore on multiple commands

Describe the bug
I have an automation that First change the storage default mode to "solar only" (or Charge only etc) and After set the site limit to 1, or 5000 deoending on the automation. The script Works fine, but often Just One setting Is changed,  because the script terminate with this error:
Choose: Option 1 executed
Stopped because an error was encountered at 17 agosto 2023 alle ore 11:03:39 (runtime: 0.01 seconds)
Connection to inverter ID 1 failed.
I already tried to inserti a delay of 1 minute between the command, but nothinf changes.
If i run again the automation by hand also the second setting Is applied.
Expected behavior
All commands must be processed
Screenshots
No response
Diagnostic File
Debug logs
No response
Home Assistant Version
2023.8.2
solaredge-modbus-multi Version
2.4.2 pre 2
Installation Type
HAOS
Read the Instructions

[X] I have read https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki
[X] My configuration follows https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Configuration

Additional Context
No response

Try Release v2.4.2-pre.3 and setting Inverter Command Delay between 3-10 seconds.

Seems to be solved with pre.3 and 20ms delay + 30sec pause during the automation... Up and working since 3 days, thanks!


Dots in cookie name being replaced to underscores

Laravel Version: 5.3.1
PHP Version: 7.0.8
Database Driver & Version:  not related

Description:
Dots in cookie names are replaced by underscores, so using same name in creating and getting cookie is not possible in that case.
Steps To Reproduce:
Assume following code:
\Cookie::queue('adm::system.update', 'test', 10)
Then on next request:
\Cookie::get('ers::system.update' == null, but
\Cookie::get('ers::system_update' == 'test'

I could make a fix for that

I don't think it is Laravel issue...
Laravel in this case use Symfony Cookie component. BUT it is also not the Symfony issue.
As it mention on php.net PHP will automatically replace any dots in incoming variable names with underscores.
I think it is also related to how PHP itself is working with cookies.

It's a limitation of the cookie spec (RFC6265) and HTTP/1.1 (RFC2616). This applies to most header fields, and doesn't appear to have changed with HTTP/2.
Essentially, you can't use control characters in the cookie-name component, of which the . character is one.

Looks like an incorrect usage then. Perhaps we should add some validation inside to throw an exception on invalid usage?

Thanks @michaeldyrynda. :)

Either that, or maybe just add a note to docs to highlight valid characters.

Note to self: do this. :P

Assuming that Laravel is developer friendly, exception will be nice if name is incorrect. ☺ shorter way while debugging... It is not clear enough. I was thinking that Laravel is converting any name to be valid underneath.

Maybe this is something symfony is doing under the hood with bad input? I've not checked. Perhapse their implementation is safest, and this is one for the docs.

As @KryDos pointed out above, it's PHP handling the conversion, so it's silently happening at the language level. I wouldn't change the handling as it's going to break any code using it as-is currently, which you'd have to add to the changelog. If you're documenting the behaviour, i'd just document the conversion, pointing to the PHP docs if necessary.

PHP's $_COOKIE array contains underscores as well, but in Chrome's dev tools you can see that the dots have been set in the name.

@michaeldyrynda The RFC does not prevent using . in cookie names, what it does prevent is any control characters (escape, backspace, etc) and these characters : (, ), <, >, @, ,, ;, :, \, <, >, /, [, ], ?, =, {, }, space and horizontal tab.
The conversion of dots (and spaces) to underscores is for sure done by PHP, similar to how it does that with $_GET and $_POST. Adding this as a note to docs.


Strong Password Standards Enforcements during registration

Implementing a python snippet to ensure strong password standards enforcement during registration.

Informing users of strong password standards guidelines as recommended by international cybersecurity experts

Random password generator facility available based on user input

If custom password set, checking whether password meets recommended security standards

I have created it sir @Pranjal360Agarwal

assign it to me

@Bharadwajshivam28 I have already closed an issue created by you related to this only

I was not assigned

@Bharadwajshivam28 I have already closed an issue created by you related to this only

I am making a issue assign me and give iwoc tag with difficult tag also

I can do it. Kindly assign it to me.

Thank you. Will start working on this issue.
On Sun, 21 May, 2023, 11:23 pm Pranjal Agarwal, @.***>
wrote:

Assigned #46
https://github.com/Pranjal360Agarwal/ApnaBharat-Bus-Booking-Reservation-System/issues/46
to @yaminimasand https://github.com/yaminimasand.
—
Reply to this email directly, view it on GitHub
https://github.com/Pranjal360Agarwal/ApnaBharat-Bus-Booking-Reservation-System/issues/46#event-9296357461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A35PSGKVZOIFWT7Y3R3M46DXHJJCVANCNFSM6AAAAAAUMRVXT4
.
You are receiving this because you were assigned.Message ID:
<Pranjal360Agarwal/ApnaBharat-Bus-Booking-Reservation-System/issue/46/issue_event/9296357461
@github.com>


ci: update golangci-lint to latest release

This PR updates the  CI tool to the latest release i.e 1.43.0 and fixes all failures which are found in older tools. New tools checks are marked for TODO.
updates #2595
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>@ceph/ceph-csi-maintainers @ceph/contributors requesting a review on priority as rebasing this kind of PR becomes difficult.

/retest ci/centos/mini-e2e/k8s-1.22

/retest ci/centos/mini-e2e/k8s-1.22

Failed to deploy snapshot pieces:
Error from server: error when creating "https://raw.githubusercontent.com/kubernetes-csi/external-snapshotter/v4.0.0/client/config/crd/snapshot.storage.k8s.io_volumesnapshotcontents.yaml": etcdserver: request timed out

logs

@mergifyio rebase

@mergifyio refresh


Upload .well-known files using default ignore override

This removes all standard filters and replaces the faulty regex hack with a new default override that is used on top of any specified excludes.
Also this is getting more complex than it should be. Can we just use .gitignore or .ignore files on top of include/exclude and get rid of default "removed items" like node_modules and hidden files? Maybe just add a warning or add an option to preview a publish to see what would be uploaded?

@ObsidianMinor, thanks for fixing this.  Re .gitignore, docs for the ignore crate claim it respects that by default. I haven't tested to make sure that's the cause.
Adding a --dry-run option to publish is a great idea. I'll make an issue.


Ensure every language uses the Request and Response structure objects

This will allow us to iterate the contents of the structure without iterating the content of the SDK wrappers. For endpoints that return status, throw an appropriate exception instead of returning a failure status.
Languages:

[x] .NET
[x] Go
[x] Java
[x] Node
[x] Python
[x] Ruby
[x] Swift
[x] Web

https://github.com/trinsic-id/sdk/pull/506
https://github.com/trinsic-id/sdk/pull/505
^ Once these are merged, this ticket is done.


How can you turn off Watch mode?

This a bit annoying I see this in console: "Ran all test suites related to changed files." everytime I make a change to my file it starts running tests.... I'm running puppeteer / UI tests, sometimes in a loop and it starts bombarding the tests.
How can watch mode be turned off, this is really bad.

was an issue with Orta jest-vscode plugin that ran jest when starting vscode i believe and it would run non stop during all changes.


Feature/shape families

Description

Adds a ShapeFamily class for encapsulating the generation of shapes by a fixed set of rules. Numerous specific instances are included, including data from Beth and Daphne's packing paper and Pablo's assembly paper as well as more standard classes like regular polygons. A complete infrastructure is set up to simplify the addition of new shape classes by just defining the call operator for a class within this infrastructure for analytically constructible shapes, or by adding a JSON file for tabulated shapes.
Motivation and Context

Resolves #12.
Types of Changes

[ ] Documentation update
[ ] Bug fix
[ ] New feature
[ ] Breaking change1

1The change breaks (or has the potential to break) existing functionality.
Checklist:

[ ] I am familiar with the Contributing Guidelines.
[ ] I agree with the terms of the Contributor Agreement.
[ ] The changes introduced by this pull request are covered by existing or newly introduced tests.
[ ] I have updated the changelog and the credits.

I've added a number of you as reviewers since you've expressed interest in discussions at some point. Feel free to remove yourselves if you would prefer not to review this PR.


ImportError: cannot import name 'SubstrateRequestException' from 'substrateinterface'

Hello. I am trying to run a program and I get an ImportError: cannot import name 'SubstrateRequestException' from 'substrateinterface' error. Please tell me what could be the reason? Are there any additional requirements for the environment? I would be grateful for your answer.

Which version of python are you using ?

fixed by commit 5b84bc3ced53f046d6971c1e5266c863bc7b0d4e


Installation instructions missing 'influxd' instruction for OSX users

Can someone please add either of the below (which ever is correct) to the OS X installation instructions?
$ influxd

or
$ sudo /etc/init.d/influxdb start

It's there for the linux distro instructions, but missing for the OS X instructions. I actually uninstalled influxdb while evaluating it because the current instructions lead to the below error, arising because I hadn't actually started the daemon:
$ influx
Failed to connect to http://localhost:8086
InfluxDB shell 0.9.2

whereas this worked:
$ influxd

#  then in a separate shell:

$ influx
Connected to http://localhost:8086 version 0.9.2
InfluxDB shell 0.9.2

This might save other people evaluating your tool a bit of time and frustration.
Hope it helps,
Jonathan

@yoniholmes are you talking about the post-install instructions in the package or the instructions on the installation docs page?

Hi Sean,
Thanks for your response. I'm specifically talking about the Installation page on the web docs. If you take a look, both 'Ubuntu & Debian' and 'RedHat & CentOS' have the step 'Then start the daemon by running:'.  OS X doesn't. Because I was ignoring the first two and only focusing on OSX, I didn't realise that I need to start the daemon.
Subsequently, on the Getting started page there's no mention of needing to have the daemon started. Under 'Logging in and creating your first database' it should also probably read:

If you’ve installed InfluxDB locally, the influx command should be available via the command line. Once you have started the InfluxDB daemon (see Installation instructions), executing influx will start the CLI and automatically connect to the local InfluxDB instance. If influx is not on your path, try /opt/influxdb/influx. The output should look like this:

Thanks,
Jonathan

Thanks, @yoniholmes. I'm closing in favor of new issue on docs repo: https://github.com/influxdb/influxdb.com/issues/251

thanks

+1


Drag and drop of file on Upload button does not seem to work reliably

Seems drop target fails on current master.

Seems to be working now.


feat: Add nodeSelector and tolerantions option to promitor-agent-scraper

Hello,
i've added these options to allow pod assignments to specific nodes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Sure, I will add to second one too.

PS: Feel free to revert the version Rev for now

I'm not sure what do you mean, change the revision to Rev?

Sure, I will add to second one too.

PS: Feel free to revert the version Rev for now

I'm not sure what do you mean, change the revision to Rev?

I was referring to the chart version, sorry for not being clear!

I was referring to the chart version, sorry for not being clear!

@tomkerkhove added also to promitor-agent-resource-discovery chart and version reverted.

@tomkerkhove added also to promitor-agent-resource-discovery chart and version reverted.

Could you update the Readme of the charts please?

Sure, done :)

Thank you for the contribution @blafry!


Comparison during snapshot update should use compare function.

This line:
if encoded_expected_value is None or encoded_value != encoded_expected_value:

is using != rather than relying on the internal compare function, which means that this code cannot be adapted easily to types that do not support regular Python comparison, especially numpy arrays.
(Aside, it'd be especially nice if the Snapshot class allowed compare/encode/decode to be customized more easily; I'm currently experimenting with array comparison using numpy.allclose() by patching the internal _get_compare_encode_decode function.)

Agree with the comment, at least compare function should be a parameter of assert_match and assert_match_dir, to allow better assert on binary snapshot (pandas.DataFrame, pyarrow.parquet, ...).
The default encode and decode functions are flexible enough, since we can encode to bytes prior the assert, but the compare function should be customisable to improve assertion error message like below:
E       AssertionError: value does not match the expected value in snapshot test_linear/linear.parquet
E         (run pytest with --snapshot-update to update snapshots)
E       assert b'PAR1\x15\x0...3\x00\x00PAR1' == b'PAR1\x15\x0...3\x00\x00PAR1'
E
E         At index 1135002 diff: b'\xc0' != b'\xbc'

It should be straightforward to add an optional compare parameter to the assert functions.


Support adding tracing to clients already lifted to Kleisli

Depends on instances from https://github.com/trace4cats/trace4cats/pull/608 - these are temporarily provided here in a package object

Closing this. Feel free to reopen it if this change is still needed.


Incorrect positioning when using append-to-body and scrolling parent element relative to the body

I have a ui-select element inside a container element that has a scroll bar. When the select is open and I start to scroll, the select element remains on the same position and doesn't re-position itself. This happens cause it is appended to the body and we scroll the container element that is positioned relative to the body. Here is a plunkr with the issue.
A suggestion for an easy fix would be to lock the scroll outside the ui-select when it is open. That's how the old ui-select2 works.

+1

+1

+1

For those looking for an immediate work-around, I wrote a complimentary directive which implements the easy fix @PesanskiGoran references in the original post. I tested this with IE 11 and Chrome 51, both on Windows 7. (It's written in TypeScript, but native JavaScript developers should get the gist.)
/**
 * A complimentary directive for the angular-ui-select directive. When placed on
 * the root <ui-select> element, this directive listens to all mouse wheel
 * activity within the viewport. Whenever mouse wheel activity occurs when (1)
 * the associated ui-select drop down is open and (2) the mouse is outside of
 * said drop down, then said mouse wheel events are cancelled.
 *
 * Example usage:
 *
 * <ui-select disable-outer-scrolling
 *            ...>
 *    ...
 * </ui-select>
 */

class DisableOuterScrollingDirective {
    public static DisableOuterScrolling($document: ng.IAugmentedJQuery) {
        return {
            restrict: 'A',
            require: 'uiSelect',
            link: (
                scope: ng.IScope,
                element: ng.IAugmentedJQuery,
                attrs: ng.IAttributes,
                $select
            ) => {
                var isMouseOverElement: boolean;

                // Add an event listener and handler to any mouse wheel events
                // that occur within the viewport.
                $document.on('wheel', (event: JQueryEventObject): void => {
                    // If the drop down is open AND the mouse is hovering over
                    // the drop down.
                    if ($select.open && !isMouseOverElement) {
                        event.preventDefault();
                    }
                });

                element.on('mouseover', (): void => {
                    isMouseOverElement = true;
                });

                element.on('mouseout', (): void => {
                    isMouseOverElement = false;
                });

                // Event listeners must be cleared when the element is removed
                // from the DOM. Without this code, a memory leak will occur and
                // the disable scrolling behavior will persist indefinitely.
                element.on('$destroy', (): void => {
                    $document.off('wheel');
                    element.off('mouseover');
                    element.off('mouseout');
                });
            }
        };
    }
}

export = DisableOuterScrollingDirective;

+1

I believe this is because the whole ui-select container is append to body on open,where as the jQuery select 2 implementation appends the dropdown menu ,i.e the ui-select-choices container to the body and also closes the menu on the parent container scroll

+1

+1

+1
Issue affects both vertical and horizontal scrolling.

Please do not "+1" issues, use github 👍 reactions if you really must. Comments should be reserved for constructive discussion/issue resolution. +1s merely add unnecessary noise and scrolling between other, more helpful, replies.
Other ways you can actually help include:
Submitting a pull request that resolves the issue/adds the requested feature
Researching possible implementations or causes of bugs
Providing reproduction plunkrs if one isn't already available
Thanks! 😄

@PesanskiGoran Do you want to allow the parent container to scroll when the select is open?
I fixed/bypassed the floating issue within a scrollable parent by setting a variable on scope with the uis-open-close directive and then conditionally applying a .scrollable or '.locked' class on the parent using ng-class based on that value.
e.g.
<div ng-class="{ scrollable: !lockScrolling, locked: lockScrolling }">
    <ui-select uis-open-close="lockScrolling = isOpen"
                 append-to-body="true">
    ...
    </ui-select>
</div>

Hope this helps.

You can fix it by using functions of uiSelect in your controller, just catch the scroll event and in call back use  this code
var uiSelectController = angular.element('.ui-grid-filter-select.open').controller('uiSelect');
                if (uiSelectController && uiSelectController.open) {
                    uiSelectController.close(true);
                    uiSelectController.clickTriggeredSelect = false;
                }