Reindex job stuck at "Queued"

Describe the bug
After submitting a reindex job, the job remains queued indefinitely with no progress. The issue persists after deleting and resubmitting the job. Similar to #2200 ?
FHIR Version?
R4
Data provider?
CosmosDB
To Reproduce
Steps to reproduce the behavior:

Create search parameter, test on a single resource (success)
Submit reindex job for new parameter (success)
Query reindex job ID - status is stuck at "Queued"

Expected behavior
Reindex job should run
Actual behavior
Reindex job remains "Queued"

Thanks for reporting the issue @elgib - Are you observing this issue on OSS FHIR service or are you using managed Azure FHIR service (Azure API for FHIR / Azure Health Data Services)?

@EXPEkesheth we're using the OSS FHIR service

@elgib , have added in our queue for investigation. Can you please share with us search parameter json used for creating custom search parameter. Will inform once we have more details/ questions.
#114359

@EXPEkesheth here's an example of one of our custom search parameters.
{     "resourceType": "SearchParameter",     "id": "e46bd3c4-f278-4039-841b-892e931596fe",     "meta": {         "versionId": "1",         "lastUpdated": "2023-11-22T12:54:08.333+00:00"     },     "url": "http://1beat.care/fhir/search-parameters#patient-care-unit",     "name": "patient-care-unit",     "status": "draft",     "description": "Reference to Organization resource that represents the care unit currently responsible for the patient.",     "code": "care-unit",     "base": [         "Patient"     ],     "type": "reference",     "expression": "Patient.extension.where(url = 'http://1beat.care/fhir/extensions#patient-care-unit').value",     "target": [         "Organization"     ] }
Thanks for looking into this. We would appreciate any updates as this is blocking key areas of work for our team. Any short-term recommendations would also be helpful -- for example, should we try rolling back to an earlier version?

@EXPEkesheth are there any updates on this issue, or timeframes for a fix?

@elgib - As you are using the OSS FHIR service , you would need to explicitly enable Reindex in deployment template (https://github.com/microsoft/fhir-server/blob/main/samples/templates/default-azuredeploy-docker.json#L272). Please ensure this setting is enabled in your instance.
Have you used reindex capability in FHIR server OSS before?

@EXPEkesheth we have enabled reindex operations and run several reindex jobs successfully in the past. The last successful reindex was late November 2023. We have not made any changes to our setup since then.

@elgib Thanks for the information. We will look into the issue and get back incase of any questions.

@elgib - we recently made improvements in reindex operation. We hope it helps address your issue, can you please execute reindex and let us know the outcome?


Request for polygon merging.

Can we have a button to merge multiple polygons into one polygon?

Hi @yuanyuangoo
Polygon can be only grouped within one frame.
In exported annotations grouped objects have the same group_id identifier. Please, refer to documentation: https://opencv.github.io/cvat/docs/manual/advanced/shape-grouping/
Also there is a feature under active development #4543, like Paint & Brush tools. It uses binary masks instead of vector shapes and it does not have such restrictions like polygons.
I hope it will help, I will close the issue.

@yuanyuangoo , could you please give us an idea what you mean? How do you want to merge them?

#774

@nmanovic Hi, like if I only want to merge two polygons that partly overlapped, it would delete two polygons and create a new polygon that would be the union of the two old polygons. Grouping just makes them very messy, and I cannot send them to the background or foreground together.
It is useful when I label a big object, and I want to add more area to the labeled object. Which point is the starting point would not be an issue for me.

I am +1 for this issue, being able to merge objects in 1 frame into a common object would be helpful. Eg some toolits only allow instance-segmentation (differentiate between instances, eg apple1, apple2), and not semantic segmentation (you only differentiate the class/type of objects - apple/orange). So by merging objects manually in CVAT dataset we can implement "semantic segmentation" even using only instance-segmentation toolkits.

@yuanyuangoo, @breznak, how do you think, would it be useful for annotators to be able to work with grouped objects like we can in vector image editors (CorelDraw, Adobe Illustrator, Inkscape) or other tools (like Google Presentations, Word, or PowerPoint)? I mean, to work with a group just like a single object (select by 1 click, move all elements to bg/fg, drag by mouse, resize etc.), while still being able to work with separate objects by a ctrl+click?

Hi, this idea is helpful for semantic segmentation, which would work for me. But it would work for instance segmentation if I want to add a part to a labeled object when labeling a big thing.

Hi, this idea is helpful for semantic segmentation, which would work for me. But it would work for instance segmentation if I want to add a part to a labelled object when labelling a big thing.

From: Maxim Zhiltsov @.>
Sent: 07 October 2022 23:15
To: opencv/cvat @.>
Cc: Alan Shangguan @.>; Mention @.>
Subject: Re: [opencv/cvat] Request for polygon merging. (Issue #5006)
@yuanyuangoohttps://github.com/yuanyuangoo, @breznakhttps://github.com/breznak, how do you think, would it be useful for annotators to be able to work with grouped objects like we can in vector image editors (CorelDraw, Adobe Illustrator, Inkscape) or other tools (like Google Presentations, Word, or PowerPoint)? I mean, to work with a group just like a single object (select by 1 click, move all elements to bg/fg, drag by mouse, resize etc.), while still being able to work with separate objects by a ctrl+click?
—
Reply to this email directly, view it on GitHubhttps://github.com/opencv/cvat/issues/5006#issuecomment-1271402735, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC5SJ53GAM4667GXGFW3KQ3WB7Z3RANCNFSM6AAAAAAQXDX43Y.
You are receiving this because you were mentioned.Message ID: @.***>

I'd also find this function incredibly useful (I used it extensively in with other tools, e.g. V7, and I think it's a great functionality). And having it in the polygon tool rather than in a brush-like tool would be great.

@Fraccalo , we have added a brush tool. Probably it will help a little bit with your use case: https://opencv.github.io/cvat/docs/manual/advanced/annotation-with-brush-tool/


Add new event handler to allow tracking of progress of extraction progress for individual entry

Add new event handler to allow tracking of progress of extraction progress for individual entry. This allows for showing or logging progress of the extraction process, especially useful for large files that might take a long time to extract.
This allows us to assign an event to the reader as such:
reader.EntryExtractionProgress += Reader_EntryExtractionProgress;
And then for example use it as such:
`
private int lastPercentage = -1;
//---//
private void Reader_EntryExtractionProgress(object sender, ReaderExtractionEventArgs e)
{
var progress = (SharpCompress.Readers.ReaderProgress)e.ParamList[0];
if (progress.PercentageRead % 10 == 0 && lastPercentage != progress.PercentageRead)
{
// Do something for every 10 percent of the extraction
}
}
`

Why did you close the other PR to recreate it?  My review comments are still the same: https://github.com/adamhathcock/sharpcompress/pull/225

Hi. The code is completely rewritten so I thought it made sense to close the existing one and create a new instead. Sorry if it was confusing, was not my intention. The review comments in #225 is not really valid anymore since the code is now rewritten using an event handler instead.

It's the same code on the same branch.  Did you mean to make this pull request from a different branch other than gardebring:master ?

Sorry but you lost me now.
The old code used an action as a parameter passed along. The new code use an event handler, so it is not the same code.
However, the PR do contain the old commits that were reverted.
So, first commit is old code, second is revert, third is the new code.
I can create a completely new PR with only the final change instead if you want. It will not have a functional different, but the  commit history will look nicer.

So the code is correct as you only have your master branch.  Pull Requests work per branch so it was updated when you pushed the latest changes.
My review from the other Pull Request is the same as this Pull Request as it's the same branch with the same code.
Ill just move over my comments.

Hi again and thanks for feedback. For some strange reason I could not see your code comments previously, which probably explains the confusion. I will look into them and get back to you.

Alright, so I believe the requested changes now has been implemented. Let me know if there is anything else :)


PACKAGES: mod-chat fails to connect

FYI: @winwisely112

might be CI / k8 related as was working a day ago.
Or maybe the recent chat Protobuf changes.

Context:

mac ( but same error on mobile and web )

REPRO:

click chat icon and nothing else.
then see logs below..

MAC:
flutter: creating IonModule!!
flutter: -- ChatModule INITIALIZED
flutter: Chat tapped
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Failed to receive messages from the server: gRPC Error (14, Error connecting: SocketException: OS Error: Connection refused, errno = 61, address = <IP_ADDRESS>, port = 64023)
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Another exception was thrown: 'package:flutter/src/gestures/mouse_tracking.dart': Failed assertion: line 222 pos 12: '(value is PointerAddedEvent) == (lastEvent is PointerRemovedEvent)': is not true.
flutter: Failed to receive messages from the server: gRPC Error (14, Error connecting: SocketException: OS Error: Connection refused, errno = 61, address = <IP_ADDRESS>, port = 64030)

WEB:
-- ChatModule INITIALIZED
Chat tapped
Failed to receive messages from the server: gRPC Error (14,
XhrConnection status 404)
Failed to receive messages from the server: gRPC Error (14,
XhrConnection status 404)

need a url and a port. Web is using localhost.

Who from ?
Flutter side ?

Web is using relative path

https://chromium.googlesource.com/external/github.com/flutter/flutter/+/refs/tags/v1.10.4/packages/flutter_tools/lib/src/mdns_discovery.dart
https://github.com/Horus125/dart-pkg/blob/master/multicast_dns/lib/multicast_dns.dart


Make rubocop require trailing comma in multiline expression

Description
This makes the linting rules in ruby consistent with eslint. We decided to change the equivalent rule there because it makes diffs more accurate and reduces annoying syntax errors.
Tests
rubocop -a -R

Are you sure you ran this rule (using the right config)? This does not fail?
There seems like there are plenty of places where this is done yet.

Sorry you got here too soon! I just committed the fix.


Paid work

We need someone who can help us set up an exchange, using this code.
Email me if you can do the job<EMAIL_ADDRESS>

reply mail

@ThusoMathebe scammer alert  = TheDaBest

unfair dispute?

you can see image demo at here: https://github.com/peatio/peatio/issues/819
contact me facebook you can see video or anydesk to see real: https://www.facebook.com/sidav2

TheDaBest  = scammer
scammed be twice from four different accounts. BE ALERT

TheDaBest = scammer
scammed be twice from four different accounts. BE ALERT

hmm four different accounts haha
kiddo

@Thedabest  can you share your skype and email? I talk with u thanks

@Thedabest can you share your skype and email? I talk with u thanks

t.me/no_public_cc
i see you on telegram

Thanks for the warning


Make sure intermediate build operations are emitted

This commit simplifies the JUnit Platform test execution listener, so that
it also generates test events for "intermediate" tests. The previous implementation
was mostly copied from the legacy JUnit listener, which didn't account for
nested test structures.
There is one drawback of the approach used in this PR, which is that
because there wasn't such a concept before, we had to make a decision on
how to expose the nested tests. We're using "test classes", because it
matches closely the concept of a "test suite" which is already in use.
But it means that for those intermediate operations, the method name isn't
properly available: one has to rely on the display name, which will
actually resolve to the method name by default.
See #5975

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

OK, I've already triggered ReadyForNightly build for you.

@bot-gradle test RFN

OK, I've already triggered ReadyForNightly build for you.

@bot-gradle test RFN

@bot-gradle test RFN

OK, I've already triggered ReadyForNightly build for you.

OK, I've already triggered ReadyForNightly build for you.


add silent option

Currently the verbose option disables all the logging, including the more relevant information at start up (ie port, config file etc)
I'd suggest to rename the verbose option to silent and use the verbose flag to only disable individual message logging.

You can now specify a log level in the config file, e.g.:
{
    "logging": "debug" // or: none, fatal, error, warning, info (default)
}

and override it on the commandline using the -l (or --loglevel) option.
Released in 0.6.0.


fix(rails empty state): collapse Rails EmptyState title when no title provided, matches React

BUILD-619
Description
Matches react implementation:

Removes empty SageEmptyState title tag when no title defined
html_safes text description

Screenshots

before
after

Test notes
Ensure title is collapsing
Related
BUILD-619

@joedakroub Ah, good call, adding 🙏


Verified Users

Hack together a user-centric way of tracking verified user's accounts

0e51b72 removed the main verification code for this issue


Error: Module parse failed: 'import' and 'export' may appear only with 'sourceType: module'

in 1.0.1 fixed


[release/v1.7] Add changelog for v1.7.3

What this PR does / why we need it:
Add changelog for v1.7.3 (manual cherry-pick of #3061)
What type of PR is this?
/kind documentation
Does this PR introduce a user-facing change? Then add your Release Note here:
NONE

Documentation:
NONE

/assign @kron4eg @embik
/hold
waiting for #3060 to get merged first

/approve

/hold cancel


openjdk-systemtest failing with stf.pl not found in jvmtest in ci.adopt

We have builds failing in systemtest in ci.adopt due to stf.pl not found under jvmtest folder in the build. e.g, https://ci.adoptopenjdk.net/view/System tests/job/openjdk10_hs_systemtest_x86-64_linux/lastSuccessfulBuild/artifact/openjdk-tests/TestConfig/test_output_15186193597840/TestTargetResult.tap/view/
This is due to the fact that, with the new change in TestKitGen being used from openj9, there is no JAVA_VERSION in the path. We need to update the systemtest build script and remove JAVA_VERSION.

duplicated by: https://github.com/AdoptOpenJDK/openjdk-tests/issues/257

https://github.com/AdoptOpenJDK/openjdk-tests/pull/258

The issue is resolved in the sysmtetest builds.


Can't use the AdminLTE after installing it

Morning, first of all sorry for my bad inglesh.
I'm trying to use this template for a very first time, so i'm facing a problem whem i try to use it. I have followed all the instructions on de ReadMe file but it doesn't work for me.
I'm using Laravel 5.5. Would like to know how you implemented it in your projects, if u extracted the files to project or used other way...
Thamks

I figured out what i missed: I created a new project using laravel new projectname and by now the lateste version of laravel is 5.6 not 5.5  (and AdminLTE need to be updated to work on it). So i just created a new one specifying the version.
Thanks for all and sorry for my destriction


rename variables for append blob access condition

closes https://github.com/Azure/azure-sdk-for-cpp/issues/1026

/azp run cpp - storage


Automatic change of the theme (light / dark) according to the system

Hi,
it would be nice if the standard theme would be automatically change from dark to light or light to dark according to the system settings of windows 11.

/question
This feature already exists. Please follow steps from https://code.visualstudio.com/docs/getstarted/themes#_auto-switch-based-on-os-color-scheme

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Translate non-usfm files

Add feature to translate files in other formats as well.

docx/docs/odt
plain text
json
xml

Being implemented in MetaphrazeMT https://github.com/binu-alexander/MetaphrazeMT/


3 testsuite issues with Python3 / pandas 0.25

Hi,
I've tried to build the Debian package from latest HEAD with Python 3.7 and Pandas 0.25.  There are three remaining errors in the test suite:
======================================================================
ERROR: test_munging_invalid_type_to_self_type (skbio.sequence.tests.test_sequence.TestDistance)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/sequence/tests/test_sequence.py", line 2369, in test_munging_invalid_type_to_self_type
    Sequence("ACGT").distance(42)
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/sequence/_sequence.py", line 1539, in distance
    other = self._munge_to_self_type(other, 'distance')
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/sequence/_sequence.py", line 2161, in _munge_to_self_type
    return self.__class__(other)
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/sequence/_sequence.py", line 622, in __init__
    s = np.frombuffer(sequence, dtype=np.uint8)
TypeError: a bytes-like object is required, not 'int'

======================================================================
ERROR: test_init_invalid_sequence (skbio.sequence.tests.test_sequence.TestSequence)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/sequence/tests/test_sequence.py", line 465, in test_init_invalid_sequence
    Sequence(('a', 'b', 'c'))
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/sequence/_sequence.py", line 622, in __init__
    s = np.frombuffer(sequence, dtype=np.uint8)
TypeError: a bytes-like object is required, not 'tuple'

======================================================================
FAIL: test_no_variation_pearson (skbio.stats.distance.tests.test_mantel.MantelTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/python-skbio-0.5.5a/.pybuild/cpython3_3.7_skbio/build/skbio/stats/distance/tests/test_mantel.py", line 247, in test_no_variation_pearson
    npt.assert_equal(obs, (np.nan, np.nan, 3))
  File "/usr/lib/python3/dist-packages/numpy/testing/_private/utils.py", line 355, in assert_equal
    assert_equal(actual[k], desired[k], 'item=%r\n%s' % (k, err_msg), verbose)
  File "/usr/lib/python3/dist-packages/numpy/testing/_private/utils.py", line 427, in assert_equal
    raise AssertionError(msg)
AssertionError:
Items are not equal:
item=0

 ACTUAL: 0.0
 DESIRED: nan

----------------------------------------------------------------------
Ran 2352 tests in 44.011s

FAILED (SKIP=30, errors=2, failures=1)

Are you able to reproduce this? Any idea how to fix these?
Kind regards, Andreas.

@tillea thanks for reporting this. From this Travis build Python 3.7 and Pandas 0.25 seem to be working fine. What version of NumPy were you using?

Thanks for the quick reply.
On Thu, Nov 14, 2019 at 09:24:50AM -0800, Yoshiki Vázquez Baeza wrote:

@tillea thanks for reporting this. From this Travis build Python 3.7 and Pandas 0.25 seem to be working fine. What version of NumPy were you using?
I'm using the Debian packaged numpy 1.17.4.

@tillea thanks for reporting this, after building a brand new environment with the latest dependencies (conda create -n test --file ci/conda_requirements.txt && pip install -r ci/pip_requirements.txt && pip install --no-deps .) I was also able to reproduce this error.
@ebolyen any chance you can have a look at this?

  Invalid dtype: %s" %
                    sequence.dtype)
        elif isinstance(sequence, Sequence):
            # Sequence casting is acceptable between direct
            # decendants/ancestors
            sequence._assert_can_cast_to(type(self))

            if metadata is None and sequence.has_metadata():
                metadata = sequence.metadata
            if (positional_metadata is None and
                    sequence.has_positional_metadata()):
                positional_metadata = sequence.positional_metadata
            if (interval_metadata is None and
                    sequence.has_interval_metadata()):
                interval_metadata = sequence.interval_metadata
            sequence = sequence._bytes
            self._owns_bytes = False
            self._set_bytes(sequence)
        else:
            # Encode as ascii to raise UnicodeEncodeError if necessary.
            if isinstance(sequence, str):
                sequence = sequence.encode("ascii")
>           s = np.frombuffer(sequence, dtype=np.uint8)
E           TypeError: a bytes-like object is required, not 'int'

../../../miniconda3/envs/latest-skbio/lib/python3.7/site-packages/skbio/sequence/_sequence.py:622: TypeError

@ElDeveloper, looks like it is just a change from AttributeError to TypeError from numpy, which makes a lot more sense for the tests involved. I'll have a PR for this shortly.

Sorry, spoke too soon, not 100% sure what is happening with FAIL: test_no_variation_pearson (skbio.stats.distance.tests.test_mantel.MantelTests), but the sequence ones are straight-forward.

I am unable to reproduce the last item with:
§ conda list pandas
# packages in environment at /home/evan/.conda/envs/test:
#
# Name                    Version                   Build  Channel
pandas                    0.25.3           py37he6710b0_0
(test) 13:32:29 ~/workspace/biocore/scikit-bio@bug-cigar
§ conda list numpy
# packages in environment at /home/evan/.conda/envs/test:
#
# Name                    Version                   Build  Channel
numpy                     1.17.4                   pypi_0    pypi

(numpy 1.17.4 isn't on conda yet, but they do note that it needs a new Cython which is perhaps why its being slow to get integrated)

On Mon, Nov 18, 2019 at 12:34:50PM -0800, Evan Bolyen wrote:

I am unable to reproduce the last item with:
...
(numpy 1.17.4 isn't on conda yet, but they do note that it needs a new Cython which is perhaps why its being slow to get integrated)
Do you see any chance to create a virtual machine with Debian unstable?

From looking into this, it seems like an issue associated with the version of SciPy being used. When there's no variance pearson can return 0 or np.nan depending on the version. 1.3.1 seems to work fine, @tillea what version of scipy are you running?

@ElDeveloper Currently we are at SciPy 1.2.2

We are transitioning to SciPy 1.3.1 as being tracked at https://release.debian.org/transitions/html/auto-python-scipy.html

Thanks for the information. Then it sounds like the solution to this issue is to update the test suite with the new exception classes (as noted by @ebolyen).


App keeps crashing in background

[Device Brand]: google
[Device Model]: Pixel 7
[Android Version]: 14
[Android API Level]: 34
[Process ID]: 2012
[CPU ABI]: arm64-v8a
[Package Name]: com.capyreader.app
[Version Name]: 2024.07.1009
[Version Code]: 1009
[Target SDK]: 34
[Min SDK]: 30
[Error Type]: JVM
[Crash Time]: 2024-07-16T16:26:43.915
[Stack Trace]:
java.lang.RuntimeException: android.os.TransactionTooLargeException: data parcel size 2248124 bytes
Bundle stats:
  androidx.lifecycle.BundlableSavedStateRegistry.key [size=2247256]
    m:-1 [size=2246652]
      -2pkxns [size=2245340]
      lpeo4b [size=1256]
PersistableBundle stats:
  [null]
	at android.app.servertransaction.PendingTransactionActions$StopInfo.run(PendingTransactionActions.java:146)
	at android.os.Handler.handleCallback(Handler.java:959)
	at android.os.Handler.dispatchMessage(Handler.java:100)
	at android.os.Looper.loopOnce(Looper.java:232)
	at android.os.Looper.loop(Looper.java:317)
	at android.app.ActivityThread.main(ActivityThread.java:8592)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:580)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:878)
Caused by: android.os.TransactionTooLargeException: data parcel size 2248124 bytes
	at android.os.BinderProxy.transactNative(Native Method)
	at android.os.BinderProxy.transact(BinderProxy.java:586)
	at android.app.IActivityClientController$Stub$Proxy.activityStopped(IActivityClientController.java:1527)
	at android.app.ActivityClient.activityStopped(ActivityClient.java:101)
	at android.app.servertransaction.PendingTransactionActions$StopInfo.run(PendingTransactionActions.java:135)
	... 8 more

Just to get some more details, have you used the "mark as read" or background refresh features?
This may also be related to feed size. How many feeds do you think you have? An OPML file would be helpful but not necessary.

Thanks for this. I've spent the past few hours digging into the the TransactionTooLargeException. I was able to reproduce by

Loading an article with full content
Clicking a link which sends you to either an in-app browser or your web browser
Hitting "back"

When you press back, the TransactionTooLargeException happens. The fix is to rework some of the article data. I have a potential fix (https://github.com/jocmp/capyreader/pull/183) that I'll test again tomorrow (local time).

I have a fix out for this in the latest release, 1010. Please reopen this ticket if you continue to see issues.
Per your other question, I've opened another ticket to investigate ways to cut down on feed refreshes with https://github.com/jocmp/capyreader/issues/185. Thanks again for the thorough bug report!


Adapt Quantum Espresso test to PE 18.07

Status: module not available.

The latest release of QuantumESPRESSO is 6.3: we have provided an EasyBuild recipe with the latest Cray PE 18.08, but unfortunately the sanity check of the ReFrame check fails due to a different value of the total energy returned by the code. We can easily adapt the sanity check to match the new value, however we should probably inform users about this mismatch.

Some constants have changed since QuantumESPRESSO-6.2.0 therefore we need to adapt the value of the total energy used for the sanity check. Adapting this value, the test passes on Dom`:
dom101:/scratch/snx1600tds/lucamar/quantumespresso$ reframe -Rc $HOME/reframe/cscs-checks/apps -n quantum_espresso_cpu_check --map-module="QuantumESPRESSO: $MY/easybuild/modules/all/QuantumESPRESSO/6.3-CrayIntel-18.08" -r
Command line: /apps/common/UES/jenkins/SLES12/easybuild/software/reframe/2.14/bin/reframe -Rc /users/lucamar/reframe/cscs-checks/apps -n quantum_espresso_cpu_check --map-module=QuantumESPRESSO: /apps/dom/UES/sandbox/lucamar/easybuild/modules/all/QuantumESPRESSO/6.3-CrayIntel-18.08 -r
Reframe version: 2.14
Launched by user: lucamar
Launched on host: dom101
Reframe paths
=============
    Check prefix      :
(R) Check search path : '/users/lucamar/reframe/cscs-checks/apps'
    Stage dir prefix     : /scratch/snx1600tds/lucamar/quantumespresso/stage/
    Output dir prefix    : /scratch/snx1600tds/lucamar/quantumespresso/output/
    Perf. logging prefix : /scratch/snx1600tds/lucamar/quantumespresso/perflogs
[==========] Running 1 check(s)
[==========] Started on Tue Oct  9 15:14:31 2018

[----------] started processing quantum_espresso_cpu_check (Quantum Espresso check (cpu))
[ RUN      ] quantum_espresso_cpu_check on dom:mc using PrgEnv-intel
[       OK ] quantum_espresso_cpu_check on dom:mc using PrgEnv-intel
[----------] finished processing quantum_espresso_cpu_check (Quantum Espresso check (cpu))

[  PASSED  ] Ran 1 test case(s) from 1 check(s) (0 failure(s))
[==========] Finished on Tue Oct  9 15:22:39 2018

I have opened the pull request https://github.com/eth-cscs/reframe/pull/501


Crashes track error ios

Hey 😄 finally we have trackError for iOS as well 🚀
I had to update the AppCenter/* and AppCenterCapacitorShared pod versions in all three packages so the example would build
@capacitor-community/appcenter-crashes

Implemented Crashes.trackError method for iOS
Added trackError base methods for iOS and Android (they're not used, but could be used)
Updated AppCenter/Crashes pod to 4.3.0
Updated AppCenterCapacitorShared pod to 0.3.2
Bumped version number from 0.6.0 to 0.7.0

@capacitor-community/appcenter-analytics

Updated AppCenter/Analytics pod to 4.3.0
Updated AppCenterCapacitorShared pod to 0.3.2
Bumped version number from 0.3.0 to 0.3.1
Change enable method name to setEnabled

@capacitor-community/appcenter

Updated AppCenter/Core pod to 4.3.0
Updated AppCenterCapacitorShared pod to 0.3.2
Bumped version number from 0.7.0 to 0.7.1

By the way, I'm not used to writing Swift, so please let me know if the code could be improved 😇

Is there something else I need to fix?

Is there something else I need to fix?

Yeah a couple things. While trying to run the example project I found these. Builds work but example app crashes.
// AppCenterAnalyticsPlugin.m
CAP_PLUGIN_METHOD(setEnabled, CAPPluginReturnNone); // setEnabled not enable

// AppCenterAnalyticsPlugin.swift
@objc func setEnabled(_ call: CAPPluginCall) {
    implementation.enable(call.getBool("enable") ?? false)
    call.resolve()
}

Is there something else I need to fix?

Yeah a couple things. While trying to run the example project I found these. Builds work but example app crashes.
// AppCenterAnalyticsPlugin.m
CAP_PLUGIN_METHOD(setEnabled, CAPPluginReturnNone); // setEnabled not enable

// AppCenterAnalyticsPlugin.swift
@objc func setEnabled(_ call: CAPPluginCall) {
    implementation.enable(call.getBool("enable") ?? false)
    call.resolve()
}

All fixed now ✨

@hrafnkellbaldurs Thanks for helping on this.


rules.abaplint.org refactoring

notes:
https://www.kirupa.com/html5/filtering_items_in_a_list.htm
https://material.io/components/chips
https://codepen.io/hansmaad/pen/QGYpey   with counter
https://codepen.io/debabrata100/pen/pOLyPW   css only


Incomplete AST but no parse error on invalid declaration in pattern assignment

If I run the latest formatter on:
[a, final d] = y;

It fails with:
The formatter produced unexpected output. Input was:
[a, final d] = y;
Which formatted to:
[a, d] = y;

This code isn't correct, since you can't have a variable declaration inside a pattern assignment. The parser doesn't report an error, but analyzer also doesn't produce a complete AST which the formatter can use. The parser/analyzer should do one or the other. I know it's a weird gray area as to which errors are actually "syntactic" enough to prevent formatting. I generally prefer the formatter to be permissive, so if it's easy to have this wrong code produce a complete AST, that would be my preference. But it's also fine if this is just a parse error and the formatter can tell the user to fix their code before they format it.

I don't think it's valid to have an assignment at the top level of a compilation unit. When I copied the code into an editor it produced several errors, many of them parser errors, so I'm not sure why the formatter isn't seeing them.

I don't think it's valid to have an assignment at the top level of a compilation unit.

Sorry, I should have been clearer. Yes, I was formatting that line of code as a statement inside a function. If I run the formatter on:
main() {
  [a, final d] = y;
}

It prints:
Hit a bug in the formatter when formatting /Users/rnystrom/Documents/temp.dart.
The formatter produced unexpected output. Input was:
main() {
  [a, final d] = y;
}

Which formatted to:
main() {
  [a, d] = y;
}

Please report at github.com/dart-lang/dart_style/issues.

I checked internally and I don't see any errors of any kind coming from the parser. Is it possible that the formatter is running the parser in a mode/phase that completes before it gets to the point where those errors are generated?

I'm not aware of any way for the parser to complete without producing diagnostics, but I'm also at a bit of a loss to explain what you're seeing. The only thing I can suggest is that it's possible that the reporting of the error hasn't been implemented yet. I was told earlier today that there's still outstanding parser work related to recovery. This might be a result of such unfinished work.

I think this must just be a bug where the proper diagnostic isn't being reported at all. I'm triaging failures with the flag flip CL, and I noticed that co19/LanguageFeatures/Patterns/pattern_assignment_A04_t04 is failing too because compile errors are missing. Here's a minimal repro:
main() {
  [final d] = [2];
}

When I run this with records and patterns enabled, I get:
  error • temp.dart:23:10 • Undefined name 'd'. Try correcting the name to one that is defined, or defining the name. •
          undefined_identifier

Note that that's a resolution error. It's trying to assign to an existing variable named d. There's no error at all about having an invalid variable declaration inside a pattern assignment. The formatter doesn't resolve, so it won't get see any resolution errors, so it will dutifully try to format this and then fail because the AST doesn't have final.


Please invite me to the GitHub Community Organization

Name
Ivan Pesenti
Discord Username (if applicable)
https://discordapp.com/users/750370897105518644
Additional Context
Invited by email and curious to be part of this community.

You have received an email to join, or press this link - https://github.com/JaidevStudio - , and you'll see a banner, for you to join the organisation! 🎉
You can also show that you are part of this org, by making yourself public. This will show our profile org on your profile:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

🎉🎊Welcome to Jaidev Studio🎉🎊!!

Why not contribute to...
Unique-Github-Profiles!

If you need anything- help or support - please reopen this issue. Thank you.


Extension not detected while trying to log-in

Dev Home version
0.601.297.0
Windows build number
10.0.22621.2506
Other software
OS Build Version: 10.0.22621.2506.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.23
Steps to reproduce the bug
Try to log in.
Expected result
It should log in because I have an extension installed.
Actual result
It said this " No extension detected that wants your developer account while trying to login " ( Or something like that I don't remember ).
Included System Information
Physical Memory: 11.89GB (2.06GB free)
Processor Architecture: x64

Duplicate of https://github.com/microsoft/devhome/issues/1729


configure: error: Cannot find OpenSSL's

[jose@localhost ~]$ sudo pecl install mongodb
[sudo] password for jose:
downloading mongodb-1.1.2.tgz ...
Starting to download mongodb-1.1.2.tgz (642,582 bytes)
.....................done: 642,582 bytes
349 source files, building
running: phpize
Configuring for:
PHP Api Version:         20100412
Zend Module Api No:      20100525
Zend Extension Api No:   220100525
building in /var/tmp/pear-build-root6aljUg/mongodb-1.1.2
running: /var/tmp/mongodb/configure
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for a sed that does not truncate output... /usr/bin/sed
checking for cc... cc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether cc accepts -g... yes
checking for cc option to accept ISO C89... none needed
checking how to run the C preprocessor... cc -E
checking for icc... no
checking for suncc... no
checking whether cc understands -c and -o together... yes
checking for system library directory... lib
checking if compiler supports -R... no
checking if compiler supports -Wl,-rpath,... yes
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking target system type... x86_64-unknown-linux-gnu
checking for PHP prefix... /usr
checking for PHP includes... -I/usr/include/php -I/usr/include/php/main -I/usr/include/php/TSRM -I/usr/include/php/Zend -I/usr/include/php/ext -I/usr/include/php/ext/date/lib
checking for PHP extension directory... /usr/lib64/php/modules
checking for PHP installed headers prefix... /usr/include/php
checking if debug is enabled... no
checking if zts is enabled... no
checking for re2c... no
configure: WARNING: You will need re2c 0.13.4 or later if you want to regenerate PHP parsers.
checking for gawk... gawk
checking whether to enable mongodb support... yes, shared
checking OpenSSL dir for mongodb... yes
checking PHP version... 50416
checking whether to enable developer build flags... no
checking whether to enable code coverage... no
checking whether to use system libbson... no
checking configuring libmongoc... ...
checking whether to use system libmongoc... no
checking for pkg-config... /usr/bin/pkg-config
configure: error: Cannot find OpenSSL's <evp.h>
ERROR: /var/tmp/mongodb/configure' failed

Hey guys i need help with installing the php libraries. when i ran the pecl comand the terminal brought out this:

configure: error:  Cannot find OpenSSL's <evp.h>
ERROR: /var/tmp/mongodb/configure' failed

when i opened /var/tmp/ i couldn't even find the mongodb folder. what should i do.
if the formationg is wrong sorry mate im new here.

What Linux distro are you running?
It seems like you are missing the OpenSSL or OpenSSL-dev packages.

I'm running CentOS 7 is there a yum package for it?

You should be able to install it using:
yum install -y openssl-devel

However my usual distro is Debian based so I am used to apt :-) you may have to troubleshoot syntax of that command.
Try installing after that and it should hopefully find the required OpenSSL source

Sweet it installed! just one last thing mate.
Build process completed successfully Installing '/usr/lib64/php/modules/mongodb.so' install ok: channel://pecl.php.net/mongodb-1.1.2 configuration option "php_ini" is not set to php.ini location You should add "extension=mongodb.so" to php.ini
i can't find the php.ini file and people say to create a <?php phpinfo(); ?> but i cant open php files on my browser.
lastly i get permision denied from:

[jose@localhost ~]$ sudo echo "extension=mongodb.so" >> php --ini | grep "Loaded Configuration" | sed -e "s|.*:\s*||"
bash: /etc/php.ini: Permission denied

sorry for all the trouble im giving you.

What web server have you installed to serve your website/project? Apache, Nginx or something else?

oh dam. uh i thought i could use mongodb as a web server. sorry im completely new to website building im trying to this project: https://github.com/vichan-devel/vichan/blob/master/README.md what webserver should i install.


MVP

[ ] Add missing/any documentation
(Note that the example project is not very explanatory)
[ ] Provide single entry-point (mod.ts file, with named exports)
[ ] Clean-up, unify and format
[x] Get rid of jsx/preact (I see no use for introducing a dependecy for something than can be achieved in our case with less code using string-concatenation)

It turns out that it is not possible (within reasonable limits) to create the webworker dynamically in this way, so it still requires a separate GHActWorker class to be used in a webworker file.
(Sidenote: I have removed the "Service" from the name as it is not a ServiceWorker)
New proposed example usage:
(main.ts):
import { GHAct, type Config, type Job } from "."
const config: Config = { ... };
// worker must be in separate file, use GHActWorker there
const worker = new Worker(import.meta.resolve("./worker.ts"), { type: "module" });
const server = new GHActServer(worker, config);
await server.serve();

(worker.ts):
/// <reference lib="webworker" />
import { GHActWorker, type Job } from ".";
const config: Config = { ... };
new GHActWorker(self, config, (job: Job, log) => {
  log(`Proudly executing ${JSON.stringify(job, undefined, 2)}`);
});


[resources] Use cached value synchronously on web.

The change speeds resources web rendering up by the reading a cached value instantly by request (it was being dispatched to the end of the UI queue in LaunchedEffect)
Release Notes
Highlights - Resources

Speed resources web rendering up by the reading a cached value instantly

Highlights - Resources

Better just "Features - Resources"


incorrect reference to another section of the paper??

In the Dataflows architecture section we have the following sentence:

When data is transferred from the gateway to the cloud service, it is encrypted. See the Data in Processing section above.

should we not be pointing to the data in transit section instead?  The data in processing section does not talk to data transfer between gateway and cloud service.

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

ID: 6b0a5167-9181-bcb4-091c-9e603d2cf296
Version Independent ID: a8cbde60-5bb8-875f-3b8c-75dac8a2525f
Content: Power BI security white paper - Power BI
Content Source: powerbi-docs/guidance/whitepaper-powerbi-security.md
Service: powerbi
Sub-service: powerbi-resource
GitHub Login: @paulinbar
Microsoft Alias: painbar

Thanks for your comment, @rosscouldrey. This issue has been assigned to the doc author for review.

Hi @rosscouldrey, sorry for the VERY delayed response. Fixed as per your suggestion. Should be online a day or so. Thanks!


Read project version from version.hpp

Modifies CMakeLists.txt to determine the version from the version.hpp file to avoid duplication.
Some relevant discussion in #166

Why not creating version.hpp by PEGTL version in CMakeLists.txt?
Reading a file and extracting the version by regex looks much complex than using the recommended cmake flow (cmake tutorial).

Coverage remained the same at 100.0% when pulling 1ca58d5134877edb628cc23f655e2e45a83c9fa4 on pleroux0:master into 51dbee85e8100b70ef15db6ce6ba896c9aa32d9c on taocpp:master.

That is also an option, but someone still has to check it into the repository. In a pure CMake project, it can reside in the build directory like generated files usually do (just as in the tutorial).
It makes sense to me having the version.hpp as the source of truth so that it can be used as pure header only library (just copy the sources).

I like this patch as it seems "simple" and straight-forward to me.

version.hpp is the single source of truth.
If someone uses CMake, CMakeLists.txt takes the version from the above file.
If someone does not use CMake, it still works out-of-the-box.
No extra step (manual/automated) to sync two files.

Am I missing any downside?

Why not creating version.hpp by PEGTL version in CMakeLists.txt?
Reading a file and extracting the version by regex looks much complex than using the recommended cmake flow (cmake tutorial).

The only problem with the CMake flow is that it assumes CMake is used. The PEGTL simply doesn't enforce users (even Colin and me) to use CMake. Support for CMake is nice, being forced to use it is... not so nice. Maybe I'm old-school, but as long as there is no downside to Paul's approach, I tend to merge the PR soon. :)

oh, I see. I have to agree, your approach sounds better since this project doesn't require cmake.
Thank you @d-frey @pleroux0


No error log when uploading a malformed helm chart to Harbor.

Build: Version v1.6.0-c5a79fb5
Go to Helm Charts tab click upload and select a .yaml file as a helm chart, there's an error as expected.

But when I check the log in ui.log,  I don't see any error.
There should be explicit error at least in the log file telling user the chart is malformed.

There should be logical formatted error message returned by the API:
{"error": "error message"}

{error: "chart metadata (Chart.yaml) missing"}
"chart metadata (Chart.yaml) missing"

assigned to @ninjadq to add more detailed error message in frontend layer

fixed by pr #5537


ApolloClient.Builder.okHttpClient() returns null instead of this

Version
main
Summary
This code:
new ApolloClient.Builder().okHttpClient(okHttpClient()).build()

throws a NullPointerException because the okHttpClient setter method on ApolloClient.Builder returns null rather than this.
The javadoc for that method says this:
@return The {@link Builder} object to be used for chaining method calls

which makes it seem like a typo.
Steps to reproduce the behavior
Run new ApolloClient.Builder().okHttpClient(new OkHttpClient()).build() and observe that a NullPointerException is thrown.
Logs
No response

PR: https://github.com/apollographql/apollo-kotlin/pull/5778

Fixed in 3.8.4


Add real-calc extension

Description
The Real Calc extension allows you to update financial values based on various Brazilian price indices.
Screenshot

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Wrote you on slack 🙂

@pernielsentikaer It's ready.

Wrote you again 🙂

@pernielsentikaer It's ready again.


Web - Main Account Email Missing in "Share Somewhere" Section During Task Assignment

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

Version Number: v1.4.51-3
Reproducible in staging?: y
Reproducible in production?: y
Issue reported by: Applause - Internal Team
Action Performed:

Create a new Expensify account.
Go to FAB > Assign Task.
Add a title.
Navigate to the "Share Somewhere" section.

Expected Result:
The main account email should be automatically present in the "Share Somewhere" section by default.
Actual Result:
The main account email should be automatically present in the "Share Somewhere" section by default.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/380f0922-b572-4653-892a-c033d63e8e3f
View all open jobs on GitHub

@jliexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think that this bug might be related to #vip-vsb
CC @quinthar

Same, cannot repro on Staging v1.4.52-5.


Fix [Error] [ The named parameter 'isShowFullScreen' isn't defined ]

This parameter is thrown an exception

This is already normal before

I'm sorry, there was an error message during code execution: 'isShowFullScreen not defined.'
That's why I didn't notice the parameter 'isShowFullLockedScreen' on GitHub.


Remove button not showing on image control on first image load.

Issue description:
Image control issue: If there is no default image assigned, the first time you load an image with the control the "remove image" button does not appear (it's set to "hidden).

Version used:
Latest dev
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
	Kirki::add_field( 'my_child', array(
		'type'        => 'image',
		'settings'    => 'gm_jumbotron_image_upload',
		'label'           => esc_attr__( 'Upload an image for single image background', 'mytheme' ),
		'description' => esc_attr__('Recommended min width 1900 for full, 1200 for boxed layout', 'mytheme'),
		'section'     => 'gm_jumbotron_style',
		'default'     => '',
		'active_callback' => array(
			array(
				'setting'  => 'gm_jumbotron_select',
				'operator' => 'in',
				'value'    => 'image',
			),
		),
	) );

This is weird....
I can't replicate it using this: https://github.com/aristath/kirki/blob/e21b42e022cd34973eae9ab90e519c361cfa11d0/example.php#L488-L493
Can you please test this by adding in your wp-config.php this line:
define( 'KIRKI_TEST', true );

and then test using the develop branch?
It seems to be working fine on my end 😕

Is there something else that could control the remove button hiding? I commented out all the removeButton.hide() and the conditionals on removeButton.show() but still, the button doesn't show until a page refresh.
I've circled in red how it's fine after the page refresh.
And yea it is doing it on the test version as well:

Unfortunately I've never used Underscore (JS) templates but if I remove this: <# if ( '' === url ) { #> hidden <# } #> from line 75 in class-kirki-control-image.php the button works properly.

Just an update, I still can't replicate this issue in the latest dev version

Unable to replicate in develop or master. OPs implementation may be causing this as a side effect of something else.


Darkcoin <IP_ADDRESS> -> Dash <IP_ADDRESS>

Now, Darkcoin changed his name for Dash.

Well, we already have that shell called "dash", so nix-env might replace one with the other. Hmm, I forgot: it would perhaps even attempt to "upgrade" from one to the other. We should better change the name, I think.
Any ideas for a better name? Other distros use "dash" for the shell, so we should keep that one for it.

Do you say something about changng the binary names? Or the package name - to something as "altcoins.dashpay"?
As I list the directory, this is the result:
$ ls fakeroot/nixpkgs/outputs/altcoins.dash/bin                                                                                ~
dash-cli  dashd  dash-qt  dash-tx  test_dash  test_dash-qt
No clashing with "dash" shell.

No, the binaries don't clash. The package names do, e.g.:
$ nix-env -qa dash
dash-<IP_ADDRESS>
dash-0.5.8

I suppose "dashpay" is the best alternative?

"Dashpay" is good for me :+1:

Pushed the rename.


Added phase_range function to Lune API

Added phase_range function, which is similar to phase_hunt but instead returns every instance of a  particular moon phase between two Dates. (Please see the README and tests for info.) This is useful for determining when, for example, a new moon occurs during a given month, or which months have blue moons, etc. [enhancement]
Cleaned up the code in the tests a little bit, because I'm pedantic like that. [style]

This PR fixes #16. (Pending approval, of course :) )
This PR would be a minor version bump, since it's a pure superset of previous functionality. (I propose 0.4.0.)

LGTM other than the couple comments I left, thanks for this contribution :)

Awesome! I believe everything should be squared away now. Thanks :)

WOO!

Strange... when I run the tests locally I get this:
❯ npm test

><EMAIL_ADDRESS>test /Users/ryanseys/lune
> standard && mocha --reporter min

/Users/ryanseys/lune/test/index.js:3
const assert = require('chai').assert
^^^^^
SyntaxError: Use of const in strict mode.
    at exports.runInThisContext (vm.js:73:16)
    at Module._compile (module.js:443:25)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at /Users/ryanseys/lune/node_modules/mocha/lib/mocha.js:219:27
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/ryanseys/lune/node_modules/mocha/lib/mocha.js:216:14)
    at Mocha.run (/Users/ryanseys/lune/node_modules/mocha/lib/mocha.js:468:10)
    at Object.<anonymous> (/Users/ryanseys/lune/node_modules/mocha/bin/_mocha:403:18)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3
npm ERR! Test failed.  See above for more details.

What does node --version say? That error looks like it's from an old version.

Ah yeah, my node is super out of date.

It might be a good idea to add "engines" : { "node" : ">=4.0.0" } or something like it to the package.json, now that we're making use of ES6 features. :)

Added! 0.4.0 released!

:heart:


Is the snackbar duration configurable?

At the moment it looks like the snackbar goes away after a set time or when the bound bool value is set false.  It would be nice to configure that behaviour, for example to only disappear on the bound bool set too false.  Perhaps this is possible, and I'm just not seeing the appropriate properties.

Now it is. I implemented a Timeout property for MatSnackbar and requested @SamProf to pull. The issue can be closed as soon as it is merged.

@rshillington ,
This should be good to close out?

yes, I implemented this moths ago @rshillington please close!


Figures sv

Hello @Farnazmdi! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file lineage/LineageTree.py:

Line 61:13: E122 continuation line missing indentation or outdented
Line 62:13: E122 continuation line missing indentation or outdented
Line 63:13: E122 continuation line missing indentation or outdented
Line 63:80: E501 line too long (87 > 79 characters)
Line 70:13: E122 continuation line missing indentation or outdented
Line 71:13: E122 continuation line missing indentation or outdented
Line 72:13: E122 continuation line missing indentation or outdented
Line 72:80: E501 line too long (96 > 79 characters)
Line 73:80: E501 line too long (85 > 79 characters)
Line 74:80: E501 line too long (85 > 79 characters)
Line 108:1: W293 blank line contains whitespace
Line 109:38: W291 trailing whitespace
Line 110:25: E701 multiple statements on one line (colon)
Line 111:13: E701 multiple statements on one line (colon)

In the file lineage/figures/figure6.py:

Line 22:80: E501 line too long (263 > 79 characters)
Line 23:80: E501 line too long (252 > 79 characters)
Line 31:80: E501 line too long (126 > 79 characters)
Line 55:1: W293 blank line contains whitespace
Line 58:1: W293 blank line contains whitespace
Line 73:80: E501 line too long (104 > 79 characters)
Line 79:1: W293 blank line contains whitespace
Line 80:80: E501 line too long (104 > 79 characters)
Line 85:1: W293 blank line contains whitespace
Line 89:1: W293 blank line contains whitespace
Line 119:80: E501 line too long (241 > 79 characters)
Line 122:80: E501 line too long (253 > 79 characters)
Line 129:80: E501 line too long (98 > 79 characters)
Line 139:80: E501 line too long (97 > 79 characters)
Line 140:80: E501 line too long (95 > 79 characters)
Line 151:80: E501 line too long (97 > 79 characters)
Line 152:80: E501 line too long (95 > 79 characters)
Line 163:80: E501 line too long (97 > 79 characters)
Line 164:80: E501 line too long (95 > 79 characters)
Line 175:80: E501 line too long (94 > 79 characters)
Line 185:80: E501 line too long (93 > 79 characters)
Line 186:80: E501 line too long (91 > 79 characters)
Line 197:80: E501 line too long (93 > 79 characters)
Line 198:80: E501 line too long (91 > 79 characters)
Line 209:80: E501 line too long (93 > 79 characters)
Line 210:80: E501 line too long (91 > 79 characters)

In the file lineage/figures/figure7.py:

Line 22:80: E501 line too long (266 > 79 characters)
Line 23:80: E501 line too long (252 > 79 characters)
Line 30:80: E501 line too long (126 > 79 characters)
Line 54:1: W293 blank line contains whitespace
Line 56:32: E231 missing whitespace after ','
Line 57:1: W293 blank line contains whitespace
Line 60:1: W293 blank line contains whitespace
Line 66:80: E501 line too long (92 > 79 characters)
Line 68:80: E501 line too long (96 > 79 characters)
Line 71:1: W293 blank line contains whitespace
Line 72:80: E501 line too long (108 > 79 characters)
Line 78:1: W293 blank line contains whitespace
Line 94:1: W293 blank line contains whitespace
Line 99:1: W293 blank line contains whitespace
Line 103:1: W293 blank line contains whitespace
Line 105:80: E501 line too long (102 > 79 characters)
Line 106:80: E501 line too long (96 > 79 characters)
Line 107:1: W293 blank line contains whitespace
Line 111:1: W293 blank line contains whitespace
Line 113:53: E231 missing whitespace after ','
Line 113:80: E501 line too long (113 > 79 characters)
Line 113:84: E231 missing whitespace after ','
Line 114:53: E231 missing whitespace after ','
Line 114:80: E501 line too long (111 > 79 characters)
Line 114:84: E231 missing whitespace after ','
Line 115:1: W293 blank line contains whitespace
Line 143:80: E501 line too long (241 > 79 characters)
Line 146:80: E501 line too long (253 > 79 characters)
Line 153:80: E501 line too long (98 > 79 characters)
Line 163:80: E501 line too long (97 > 79 characters)
Line 164:80: E501 line too long (95 > 79 characters)
Line 166:80: E501 line too long (98 > 79 characters)
Line 167:80: E501 line too long (98 > 79 characters)
Line 175:80: E501 line too long (97 > 79 characters)
Line 176:80: E501 line too long (95 > 79 characters)
Line 178:80: E501 line too long (99 > 79 characters)
Line 179:80: E501 line too long (99 > 79 characters)
Line 187:80: E501 line too long (97 > 79 characters)
Line 188:80: E501 line too long (95 > 79 characters)
Line 190:80: E501 line too long (103 > 79 characters)
Line 191:80: E501 line too long (103 > 79 characters)
Line 199:80: E501 line too long (94 > 79 characters)
Line 209:80: E501 line too long (93 > 79 characters)
Line 210:80: E501 line too long (91 > 79 characters)
Line 212:80: E501 line too long (98 > 79 characters)
Line 213:80: E501 line too long (98 > 79 characters)
Line 221:80: E501 line too long (93 > 79 characters)
Line 222:80: E501 line too long (91 > 79 characters)
Line 224:80: E501 line too long (99 > 79 characters)
Line 225:80: E501 line too long (99 > 79 characters)
Line 233:80: E501 line too long (93 > 79 characters)
Line 234:80: E501 line too long (91 > 79 characters)
Line 236:80: E501 line too long (103 > 79 characters)
Line 237:80: E501 line too long (103 > 79 characters)

I already have this branch open on a PR @Farnazmdi


remove repeated lines

TBOX-217
In tamr-toolbox/run_backup_restore.py, lines 14-16 were repeated in lines 17-19. Lines 17-19 have been removed.

↪ Pull Request

✔ PR Todo

[ ] Add documentation
[ ] Add examples
[ ] Ensure you are following the practices in the contributor guide and coding_standards

Use consistent and descriptive name conventions
Uses google-style docstrings
Uses versioned API/client methods only
Avoid global variables
Avoid giant try/catch
Don't use mutable default variables
Use a main() function in your scripts

[ ] Added/updated testing for this change (ensure you are testing any changes/new code!)
[ ] Update title to link to TBOX issue (i.e. start pull request title with TBOX-xx)
[ ] Approval from first code-reviewer
[ ] Approval from second code-reviewer (only required if a second reviewer has left comments)
[ ] Pass all checks

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

Can you edit the title of this PR to start with the ticket: "TBOX-217"?


Doesn't work

apt-get install slowhttptest -y
While trying the copy pasted examples from the readme:
Option - requires an argument.
Try 'slowhttptest -h' for more information
It never works. It doesn't care that everything is in accordance with the manual.

what is your linux?
Something simple like slowhttptest -u http://whatever works?

# apt-get install slowhttptest -y
# slowhttptest -u http://km.mk
Option - requires an argument.
Try 'slowhttptest -h' for more information
# uname -a
Linux x 4.1.6-249 #1 SMP Mon Aug 31 12:34:47 UTC 2015 armv7l GNU/Linux
# lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 8.2 (jessie)
Release:	8.2
Codename:	jessie

Try to https://github.com/shekyan/slowhttptest/issues/18#ref-commit-af5dbb5

I have no control how people distribute it over apt-get. Somewhere it is using wrong getopt

Seems to work on amd64 debian-8.8.0. Something is wrong on the arm systems.
I'll compile it and see what happens.


Wine update

Motivation for this change
Update wine to current 5.22.
Includes commits from #103233
Tested it on couple games -- have some regressions, but most stuff works.
/cc @pstn @dasJ (please review cert-path.patch with care)
Things done

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

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

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

https://bugs.winehq.org/show_bug.cgi?id=50163 probably a regression which I hit when try one of my games.

@pst @SFrijters  I think about  move one more commit on staging, and patch from ticket mentioned above (otherwise too much stuff become broken). Any objections?

Hm, is this regression so much worse than might occur otherwise in Wine unstable releases that we need to patch it ourselves? I'm okay with it, but if there's more problems in 5.22 there's probably a new and hopefully better release in 2 weeks (we were lagging behind 5.21 for a while a well).

Patch seems legititmate

@SFrijters it why I ask for advice.  Is not a custom patches, just cherry-picks form master  of wine and wine-staging respectively. Both fix  msvcrt.dll.so (stack overflow/segfault around win's GetLocale, and cosh in math library). If it a good idea -- I push additional commit, if not -- let wait @dasJ review and merge.

@avnik Ok, let's do that then.
A note on cert-path.patch: I tried to do a nixpkgs-review on the current state of the PR and the patch no longer applies correctly to the stable wine version, so we'll need two variants of that one for the time being I think.

@SFrijters  done! I  also split patches = [] so stable and unstable can have isolated set of patches.
Will run overnight rebuild of staging to ensure that all good

works for me, but borgs eval  still pending


Added child container 'scoping' to the ServiceProvider

Looks goofy, but this was a technique we used within FubuMVC for automated testing to allow a single integration test to override services in the framework's container without allowing those overrides to bleed into subsequent tests.

What is the difference between this and the StructureMapServiceScopeFactory?

So the scope factory deals with creating and tracking nested containers per request. This change using the child containers (not the same thing, and folks very frequently get them mixed in) allows us to temporarily change the root, application container that's used to "calve" the per request nested containers.


Remove the device info and device class fields from BluetoothDevice

According to https://github.com/WebBluetoothCG/web-bluetooth/pull/229,  device info and device class fields from BluetoothDevice will be removed from now on. Let's remove them from our "Device Info" sample.

R=@jeffposnick

Quick check: along with the removals, there's an addition of log('> Connected:        ' + device.gatt.connected); to the sample. Just wanted to make sure that was intentional. If so, 👍 and feel free to merge.

It is intentional. Thanks for checking this out though!


Support full round-trip for LightCurve <-> Data

Before this PR, the glue translator for LightCurve only translated a few columns from the underlying astropy TimeSeries object to Data Components, and vice versa.
This PR supports translation of all columns in a Kepler light curve (and hopefully other missions too). To test the feature, I've introduced a FITS file with a segment of a Kepler light curve that contains only 10 rows/times, but all columns in a real Kepler light curve retrieved from MAST (40 kb).
I still rely on lightkurve to parse the FITS file, rather than passing the path to lcviz.load_data. This is a workaround to avoid downloading a FITS file with lightkurve, since https://github.com/lightkurve/lightkurve/pull/1299 has not yet been released (as first noted in https://github.com/spacetelescope/lcviz/pull/10).
This translator may need updates in the future if we add Components that should not be translated into columns in the TimeSeries.

this seems to result in the flux-vs-time viewer showing time vs dt instead of... flux vs time (although now in main I'm seeing uncertainty vs flux which is also wrong, so maybe its not entirely related to this PR?) 🤔

maybe its not entirely related to this PR?

I think that's unrelated – I'm seeing the correct axes on this branch.

ok, I am seeing the correct behavior with latest on jdaviz main, I think the display units branch is causing issues, but we'll track that there.  Sorry for the noise!