Example configuration doesn't seem to run through

In reviewing this for JOSS I attempted to run through the example configuration, but think something may have gone wrong (possibly user error). I started by following the installation instructions which went smoothly. Then, I ran the example configuration to download the data:
pyveg_run_pipeline --config pyveg/configs/config_all.py

Which produced a directory with all of the downloaded data (aztestoutput__2020-07-27_10-40-15). I then tried to use this data to run the pyveg_gee_analysis script via:
pyveg_gee_analysis --input_dir aztestoutput__2020-07-27_10-40-15/gee_27.94_11.58_combine/

Which produced the following output (and error):
-----------------------------------
Running analyse_gee_data.py
-----------------------------------
Reading results from "/home/bzq/workspace/monitoring-ecosystem-resilience/aztestoutput__2020-07-27_10-40-15/gee_27.94_11.58_combine/results_summary.json"...

Preprocessing data...
---------------------
- Dropping vegetation outliers...
- Fill gaps in sub-image time series...
Traceback (most recent call last):
  File "/home/bzq/.conda/envs/veg/bin/pyveg_gee_analysis", line 8, in <module>
    sys.exit(main())
  File "/home/bzq/.conda/envs/veg/lib/python3.7/site-packages/pyveg/scripts/analyse_gee_data.py", line 284, in main
    analyse_gee_data(input_dir, spatial)
  File "/home/bzq/.conda/envs/veg/lib/python3.7/site-packages/pyveg/scripts/analyse_gee_data.py", line 204, in analyse_gee_data
    ts_dirname, dfs = preprocess_data(input_dir, n_smooth=4, resample=False, period='MS')
  File "/home/bzq/.conda/envs/veg/lib/python3.7/site-packages/pyveg/src/analysis_preprocessing.py", line 806, in preprocess_data
    dfs = fill_veg_gaps(dfs, missing)
  File "/home/bzq/.conda/envs/veg/lib/python3.7/site-packages/pyveg/src/analysis_preprocessing.py", line 562, in fill_veg_gaps
    df_ = df_.append(missing_rows, ignore_index=True).sort_values(by='date')
  File "/home/bzq/.local/lib/python3.7/site-packages/pandas/core/frame.py", line 6199, in append
    elif isinstance(other, list) and not isinstance(other[0], DataFrame):
IndexError: list index out of range

I believe this is related to #350

This issue has been fixed and merged into master.


Fix AddOrUpdateAnnotationAttribute for values of type Class

What's changed?

Fixes AddOrUpdateAnnotationAttribute to work with values of type Class
What's your motivation?

Fix a bug.
Anything in particular you'd like reviewers to focus on?

No.
Anyone you would like to review specifically?

No.
Have you considered any alternatives or workarounds?

No.
Any additional context

No.
Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've read and applied the recipe conventions and best practices
[x] I've used the IntelliJ IDEA auto-formatter on affected files

Thanks a lot for this @froque ! Perhaps good to add an example where the class argument needs an import, as opposed to the java.lang imports seen now, just to be sure that's handled correctly.

@timtebeek I am having trouble with that. Can I delegate that to you ?


Add Oppo

I have only done minimal research as of yet, but Oppo seems to be quite notorious for not allowing any services to run in the background when the screen is turned off, and I believe it would be a worthy addition to this site.
The only situation that I can personally point to is with the Oppo F1S, I encountered a user for which my app's background services were killed (including accessibility services, which needed re-enabling) every time they turned the screen off. So far, a workaround for this is:

pinning the app to the recent apps screen
enabling it in the app list inside the security app's "startup manager" and "floating app list" (com.coloros.safecenter / com.coloros.safecenter.permission.Permission)
turn off battery optimizations
give the service a persistent notification to remain in the foreground

All four of those needed to be done before the app would function. Here are links to some other resources verifying that some of the above steps work on other Oppo devices:
https://forum.xda-developers.com/android/general/coloros-5-0-how-to-allow-apps-running-t3847738
https://forum.xda-developers.com/find-X/help/killing-apps-screen-off-arghh-t3818105
https://oppo-au.custhelp.com/app/answers/detail/a_id/1313/~/how-to-lock-applications-in-the-background%3F
https://www.quora.com/How-do-you-add-apps-into-Whitelist-in-OPPO-F1s-phone
Hopefully I will have the time to contribute to this project more - I hope to start organizing my own records of background service-related problems on various devices sometime soon, and will gladly merge the information I find with this project if it is of use.

Hello, I can confirm we have a lot of OPPO issues reported from our apps. But as there are not so many devices out there we have minimum feadback from users on how to overcome the issues.
So many thanks for posting the details. I guess we can start with that as a base and build on that later on..

@fennifith , As of Color OS 6 the "Auto Start" setting has moved to the App Info Page, Startup Manager app is not there on COS 6.

You can also click on the power saving option under each app and allow the app to run in background. If something like smart restriction is enabled, if the app is not cleared from the tasks, it should be running background processes but it the app is closed, the background services resume immediately when the app is opened again.

Hi,
my app doesn't work on Oppo A31 with Android 9 and ColorOS 6.1. App gets location and sends it to server. If I screen off, app works only 5 minutes and then stops. I use Handler and AlarmManager and Foreground Service with Notification to handle background work. App works correctly on Huawei, Samsung, Sony and Xiaomi phones.
I turned off battery optimization by permission in app.
I have in Settings -> Battery:

Power save mode off,
Intelligent protection for use energy off,
Not standard energy save -> my app -> Run in backgroound,
Energy save options -> Optimisation while sleep off.

In Settings -> App management -> my app -> Auttostart I have on.
I tried to lock and pin app. Still the same problem.
Do you have any ideas to resolve my problem?


transport reference counting problem

There is a problem with commit 9d853baa73703c4f5db9cb9917f33426f737dd21 in that if a SCO device disconnects then on re-connect all SCO transports are rejected.
I've traced this to reference counting of the transport object - its reference count never seems to drop below 3 so the cleanup code in  ba_transport_unref() is never invoked. As a result, the transports hash table retains  the entry into the next connection, which is therefore rejected.
If this is intended behavior for the transport ref_count then perhaps that commit should be reverted until an alternative method for detecting multiple SCO transports can be devised.

If this is intended behavior for the transport ref_count

If I understand you correctly, no, that's not the intended behavior. The count should drop to 0. When the device disconnects the lifetime of the transport object should be over. If the refcount is not 0 by then something is definitely not right with transport lifetime management...
I can try to debug this as well. What is your test case? You are connecting sink (?) BT device to bluealsa with all (?) profiles (A2DP and e.g. HFP), then you are disconnecting BT device entirely or only HFP profile?

Testing with bluealsa<->bluealsa
On HF machine (this machine is set to trust AG in bluetoothctl):
bluealsa -p hfp-hf

On AG machine:
bluealsa -p hfp-ag

Then on AG machine:
$ bluetoothctl connect B8:27:EB:2B:8D:C3
Attempting to connect to B8:27:EB:2B:8D:C3
[CHG] Device B8:27:EB:2B:8D:C3 Connected: yes
Connection successful

$ bluealsa-cli list-pcms
/org/bluealsa/hci0/dev_B8_27_EB_2B_8D_C3/hspag/sink
/org/bluealsa/hci0/dev_B8_27_EB_2B_8D_C3/hspag/source

$ bluetoothctl disconnect B8:27:EB:2B:8D:C3
Attempting to disconnect from B8:27:EB:2B:8D:C3
[CHG] Device B8:27:EB:2B:8D:C3 ServicesResolved: no
Successful disconnected

$ bluealsa-cli list-pcms

$ bluetoothctl connect B8:27:EB:2B:8D:C3
Attempting to connect to B8:27:EB:2B:8D:C3
[CHG] Device B8:27:EB:2B:8D:C3 Connected: yes
Failed to connect: org.bluez.Error.Failed br-connection-refused

$ bluealsa-cli list-pcms

$

Hmm - hold off on this one - the HF machine was not running master branch after all. Looks like a bug in one of my own experiments :(

Yes, sorry, this is not an issue with master branch.


Demo

This PR is based on #1
It implements a simple demo site allowing to select a sample and to play the corresponding animation.
It is implemented with svelte as a temporary framework while waiting for ivy to be ready.

Note that this PR now also includes #2 (in addition to including #1)


PEOPLE-60 Filtering people for projects

There were some inconsistencies concerning the definition of "current project" in various places in the app. This should take care of them. Done as part of a ticket which deals with this functionality at some point.
JIRA: https://netguru.atlassian.net/browse/PEOPLE-60

:+1: looks good!


Improve Notification Bar

After
Before

There a slight changes to the background. I'm not an CSS expert. For comparison  with a closed notification:

After
Before

Fixes #5

@jasonblais you might want to take a look. Note that the colour didn't change. We can change that to be more appropriate when we actually use the notification bar for something

@hanzei @jwilander It's not immediately clear what the improvement is? (I couldn't decipher it from the diff file).

@jasonblais I updated the screenshots to better represent the changes. Please take a look.

@hanzei Thank you! This helps a lot.
Can you help make the whitespace left and right of the text equal? Currently it's greater on the left hand side.

A designer suggested 32px on both sides, but I'm not sure if that's the appropriate value.

I changed it so the text has 32px spacing to the left edge and the close icon right. Is this what you meant?

I looks unproportional to me because the text leans to the left side. I would rather have a equal spacing to the left and right edge.

@hanzei Hmm, that feels wrong, I agree.
I hate to do this, but can you help revert back to what you had and we'll merge it instead? I'm looking at the notification bar we use for the Mattermost server and the spacing matches with what you had before.

I really really apologize for making you go back and forth, I'll do a more careful job next time before suggesting a change :)

Sure thing, no problem.


How can we use busboy to upload a file in object storage without using os.tmpdir(), since by doing this we are creating a temporary storage for it. I don't want that i directly want to call object storage

I don't know what you're asking. busboy does not touch any kind of storage, it only provides streams. It's up to you to direct where the data goes. Maybe you're confusing busboy with some other module?

If I have to upload data directly to object storage, what I am
comprehending is that first, file event will be triggered and in that, I
would have to call os.tempdir to write the uploded file  (and save it in
temporary storage) and when the streaming is finished, I can call the
finish event of busboy and pass this temporary file to the object storage.
Is that correct?
If it is so, what I don’t want is, not saving in temporary storage (via
os.tempdir) and directly pass it to object storage.
On Wed, 3 Aug 2022 at 8:28 PM, mscdex @.***> wrote:

I don't know what you're asking. busboy does not touch any kind of
storage, it only provides streams. It's up to you to direct where the data
goes. Maybe you're confusing busboy with some other module?
—
Reply to this email directly, view it on GitHub
https://github.com/mscdex/busboy/issues/313#issuecomment-1204062295, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKD6XHUQC3UAWUFPWXOPZ3LVXKCH5ANCNFSM55PFETNQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

That's beyond the scope of this project. If the library or service you're using to store the upload does not support streaming directly to the cloud/object storage, that is their problem. There is nothing busboy can do about that, as it only provides streams which never touch the local disk.

Okay, thanks
I just wanted to confirm one thing, why is os.tempdir() used when we have
to upload in firebase. I saw it in the code.
On Wed, 3 Aug 2022 at 10:06 PM, mscdex @.***> wrote:

That's beyond the scope of this project. If the library or service you're
using to store the upload does not support streaming directly to the
cloud/object storage, that is their problem. There is nothing busboy can
do about that, as it only provides streams which never touch the local disk.
—
Reply to this email directly, view it on GitHub
https://github.com/mscdex/busboy/issues/313#issuecomment-1204205868, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKD6XHRAZWEHXHAW5Q2Z4TDVXKN2NANCNFSM55PFETNQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

busboy does not use temporary directories. You will need to ask that question on the issue tracker of the module that handles uploads to firebase, as I presume they are the ones using a temporary directory.
If you're referring to the use of a temporary directory in the example in the readme, then that is because it is simply that -- an example. It's not something you have to do.


TC380 - Can deploy in with 2 masters, so can achieve redundancy for Kubernetes deployment (Terraform)

Testnet,
Grid v0.1.32
With 2 masters configured, getting this error disregarding to which node the Kubernetes is provisioned
│ Error: couldn't invalidate broken attributes: couldn't get node 5 contract 4988: contract not found: object not found
│
│   with grid_network.net1,
│   on main.tf line 25, in resource "grid_network" "net1":
│   25: resource "grid_network" "net1" {
│
╵

As noted before, we don't support multimaster clusters.

Not supported for now


[Feature] Option to change receiver code pattern

Would be nice to have it for changing its pattern to many simple ones, like currently, it's three random words, aren't they? So an option for making it two words, one word, random numbers with certain number of digits etc
Uhm, this thing https://github.com/schollz/croc#custom-code-phrase

Perhaps it would be best for me to allow the user to specify their own.

Perhaps it would be best for me to allow the user to specify their own.

I guess so. Thanks


Bump normalize-url from 4.5.0 to 4.5.1

Bumps normalize-url from 4.5.0 to 4.5.1.

Release notes
Commits

updated-dependencies:

dependency-name: normalize-url
dependency-type: indirect
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>

last change

done

done


Use static typed language instead of JavaScript

Actually, I guess the advantages of using statically typed languages are obvious. I believe it is time to migrate to one of them like TypeScript.

thanks amazing, but how can we have both js and ts elegant ? can we continue on a branch or mirror repo thing ?
Thanks


Implement "Add Token" modal on Tokens page with stubs and OAuth flow using localStorage

Part 2 of #885 (part 1 was #899). Adds components that will be reused for #892.
Adds the AddEditTokenModal and AddEditTokenForm to src/app/common/components so they can be used in multiple pages. For now they are only used on the Tokens page. Formik is set up for the fields and validation, and the modal is connected to Redux, but no actions are wired up.
Fields are in place based on this mockup:

I also added a proof of concept for the OAuth login flow using a self-closing new tab for the login page. It works very well, and has the benefit of preserving all of the original tab's state without any difficult-to-maintain state persistence code.
To test the redirect/storage approach, I stood up a fake cluster login page at https://cam-oauth-testing-login.surge.sh/. That page will take a URL parameter called state and when you click the login button it will redirect you to http://localhost:9000/oauth-landing?state=${state}&code=mock-code-12345 (with a random code each time). So it is useful for making sure the landing page stores that mock authorization code in the correct place and the Add token modal reacts properly to that event. I use the state param to uniquely identify each login attempt so the storage event listener is only looking for that specific response (in the unlikely event that the user is trying to do this process in more than one window at the same time, it will still work).

That new tab closes itself immediately after reaching my new /oauth-landing page, but if for some reason it fails to close itself (possible user-specified browser restriction? not sure if that's a thing) this is what that landing page looks like:

Rebasing now to resolve conflicts, I'll push again in a few.


Add the corrections/suggestions given by linters

Clear all console logs warning related to linting and errors for cross dependencies for JQuery for the project.

@abhu-A-J Can you assign this to me?

@abhu-A-J Can you assign this to me?

Go for it! Since it's very basic I'm giving you 12 hrs ! After that the issue will be re assigned!

@abhu-A-J  I sent a PR. I was unable to link it to this issue.
Kindly check the PR.
Sorry for any inconvenience caused from my side. 😅


Cell.init() got an unexpected keyword argument 'value_text'

Packages
Scylla version: 2024.1.10-20240918.24a78645ca8b with build-id ee974e7eb45c7a3007c99f53b7ccd7ce54dcd23c
Kernel Version: 5.15.0-1070-aws
Issue description

[ ] This issue is a regression.
[ ] It is unknown if this issue is a regression.

The test failed with error:
2024-09-24 13:58:24.992: (TestFrameworkEvent Severity.ERROR) period_type=one-time event_id=827149b1-2213-4229-b349-ab0177eae252, source=PerformanceRegressionPredefinedStepsTest.test_mixed_gradual_increase_load (performance_regression_gradual_grow_throughput.PerformanceRegressionPredefinedStepsTest)() message=Traceback (most recent call last):
File "/home/ubuntu/scylla-cluster-tests/performance_regression_gradual_grow_throughput.py", line 36, in test_mixed_gradual_increase_load
self._base_test_workflow(cs_cmd_tmpl=self.params.get('stress_cmd_m'),
File "/home/ubuntu/scylla-cluster-tests/performance_regression_gradual_grow_throughput.py", line 73, in _base_test_workflow
self.run_gradual_increase_load(stress_cmd_templ=cs_cmd_tmpl,
File "/home/ubuntu/scylla-cluster-tests/performance_regression_gradual_grow_throughput.py", line 163, in run_gradual_increase_load
results = run_step(stress_cmds=stress_cmd_templ, current_throttle=current_throttle,
File "/home/ubuntu/scylla-cluster-tests/sdcm/utils/decorators.py", line 271, in wrapped
send_result_to_argus(
File "/home/ubuntu/scylla-cluster-tests/sdcm/argus_results.py", line 135, in send_result_to_argus
argus_client.submit_results(result_table)
File "/usr/local/lib/python3.10/site-packages/argus/client/base.py", line 224, in submit_results
self.check_response(response)
File "/usr/local/lib/python3.10/site-packages/argus/client/base.py", line 68, in check_response
raise ArgusClientError(
argus.client.base.ArgusClientError: ('API Error encountered using endpoint: POST /api/v1/client/testrun/scylla-cluster-tests/253d4698-6509-4b08-906c-d6815bb4a04f/submit_results', "Cell.__init__() got an unexpected keyword argument 'value_text'")

The failure happened from @latency_calculator_decorator
Impact
Describe the impact this issue causes to the user.
How frequently does it reproduce?
Describe the frequency with how this issue can be reproduced.
Installation details
Cluster size: 3 nodes (i4i.4xlarge)
Scylla Nodes used in this run:

perf-regression-predefined-steps-ub-db-node-253d4698-3 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
perf-regression-predefined-steps-ub-db-node-253d4698-2 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
perf-regression-predefined-steps-ub-db-node-253d4698-1 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)

OS / Image: ami-083b92d3782b9a373 (aws: undefined_region)
Test: scylla-enterprise-perf-regression-scylla-predefined-steps-vnodes.jenkinsfile
Test id: 253d4698-6509-4b08-906c-d6815bb4a04f
Test name: scylla-staging/yulia/performance/scylla-enterprise-perf-regression-scylla-predefined-steps-vnodes.jenkinsfile
Test method: performance_regression_gradual_grow_throughput.PerformanceRegressionPredefinedStepsTest.test_mixed_gradual_increase_load
Test config file(s):

perf-regression-predefined_steps.yaml

Logs and commands

Restore Monitor Stack command: $ hydra investigate show-monitor 253d4698-6509-4b08-906c-d6815bb4a04f
Restore monitor on AWS instance using Jenkins job
Show all stored logs command: $ hydra investigate show-logs 253d4698-6509-4b08-906c-d6815bb4a04f

Logs:

db-cluster-253d4698.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/253d4698-6509-4b08-906c-d6815bb4a04f/20240924_140104/db-cluster-253d4698.tar.gz
sct-runner-events-253d4698.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/253d4698-6509-4b08-906c-d6815bb4a04f/20240924_140104/sct-runner-events-253d4698.tar.gz
sct-253d4698.log.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/253d4698-6509-4b08-906c-d6815bb4a04f/20240924_140104/sct-253d4698.log.tar.gz
loader-set-253d4698.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/253d4698-6509-4b08-906c-d6815bb4a04f/20240924_140104/loader-set-253d4698.tar.gz
monitor-set-253d4698.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/253d4698-6509-4b08-906c-d6815bb4a04f/20240924_140104/monitor-set-253d4698.tar.gz

Jenkins job URL
Argus

Should be fixed in https://github.com/scylladb/argus/pull/460


Update SMSegment.swift

At the moment 'titleOffSelectionFont' is ignored as it is never used inside of the code. The proposed change will update the font of the segment's label in the setSelected method to ensure that correct font is being used.

I realised that supporting onSelection font is non-trivial due to layout issues. Will try to resolve this now and submit a new request


How to use .fetch("subreddit", {after: ...., before: ....}) ????

I dont know how to use after: and before in the fetch function. I would have thought it would fetch ALL the posts (except itself) before / after the given post id?

Or do i use the .name?

or how do i use Listing:FetchMore()? This could be a solution for me too.


Hide progress bar when mixer is not running

Closes #51.
Hides mixer status bar when the mixer is not activated

Padding added to give cleaner look


Validation failure on schema with "oneOf" or "anyOf"

Steps to reproduce:

copy the schema from https://bhch.github.io/react-json-form/playground/#20-oneof (or from the anyOf example)
paste the schema into https://bhch.github.io/react-json-form/playground/#18-validation
click Submit

Current result:
Unsupported type "undefined" for object properties (keys).

Expected result:
The object should either be correctly validated against the schema or not give an unexpected error.

Awesome! ❤
When do you think this will be released?
Also, when do you this will be merged with django-jsonform?

I'll try to get it out today.


Change icon color white when  selected. (IN PROGRESS , LET ME REISSUE THIS PR)

Overview of the Pull Request:
Describe your changes here...
Change made to travelMethod Icon colour  when selected it will be same as the text colour which is white.
This will now only focus on the icon colour to be white when active and selected.
(https://trello.com/c/z50v3GrA/297-dev-travel-method-selection-page)
How Has This Been Tested?

Test locally as I select the travelMethod button the icons colour have to be the same as the font colour.
Confirm the colour of icon change when selected align with text color.
Confirm when on hover it has to be light colour.
Confirm after deselect the button will show light blue which is on hover.
Confirm the on active button will not change colour when the mouse hover.

Pull Request Checklist:
Make sure the following checkboxes[ ] are checked with x before sending your PR -- thank you!

[ x] Have you included a link Trello card / Github issue and written sufficient description to help others review your work?
[ x] Is your pull request up-to-date with main branch?
[ x] Have you checked That code is well formatted? Did npx prettier --check . return no warnings/errors?
[ x] Have you written instructions on how to test that your changes work as expected?
[x ] Have you tested your work thoroughly on your local machine to ensure you are not breaking anything?

The SVG file original colour is fill with "none" so I can control inline style.  This is how we can change the SVG colour.

The SVG file original colour is fill with "none" so I can control inline style. This is how we can change the SVG colour.
Hi @Amutha37 ,

Sorry, I should have been clearer about the benefits of using fill="currentColor.
Using fill="currentColor" will give the icons a default color if none was given, and allow us to change the icon color using Chakra's color prop.
The added benefit of having a default color applied to icons is that they can be visually inspected by opening the svg file (rather than showing nothing with fill="none")
Also, using fill="currentColor" is the suggested approach in the Chakra docs
With fill="currentColor":

With fill="none":


Fix passing tracked options

It fixes https://github.com/lifeart/ember-ref-bucket/issues/52

@wozny1989 thank you for report and fix! 5.0.2 published


chore: add node:16 to the test matrix

Which problem is this PR solving?
Adds current node version(16) to the test matrix.

https://github.com/open-telemetry/opentelemetry-js/pull/2244
https://github.com/open-telemetry/opentelemetry-js-api/pull/85

That's interesting. Does anyone have experience with the error in CI?
Since we're not caching anything yet in the node:16 runs, I assume it has to do with the underlying node container.
npm ERR! code EACCES
npm ERR! syscall mkdir
npm ERR! path /github/home/.npm/_cacache/content-v2/sha1/46/c3
npm ERR! errno -13
npm ERR!
npm ERR! Your cache folder contains root-owned files, due to a bug in
npm ERR! previous versions of npm which has since been addressed.
npm ERR!
npm ERR! To permanently fix this problem, please run:
npm ERR!   sudo chown -R 1001:121 "/github/home/.npm"

npm ERR! A complete log of this run can be found in:
npm ERR!     /github/home/.npm/_logs/2021-05-31T19_13_29_158Z-debug.log

@anuraaga, I reverted karma-webpack version to 4 for packages, you've contributed. Is that alright?


Use markdown parser for literate pyspec

About
This PR suggests a refactoring of how we handle parsing the pyspec from the literate markdown files.
~This PR should serve as basis for discussion for now and should not be merged until this comment is updated otherwise.~
Discussion
We currently have a hand-rolled parser that scans markdown files in the /specs tree and pulls out python code to build the pyspec. I ran into at least two situations in #2329 where I needed further flexibility in handling how the parser works so wrote this PR to refactor the implementation along with introducing a new feature to fix one of the aforementioned situations.
First, I'll motivate w/ these situations:

If you look at the code before this diff https://github.com/ethereum/eth2.0-specs/pull/2329/commits/4606920e25bc55be4d2a37243a5c757f649dff72, you will see the definition of the constant SYNC_SUBCOMMITTEE_SIZE in p2p-interface.md on line 89. The current parser only grabs functions or class definitions from "```python" blocks so I had to inline the constant into the function body. The existing way to handle this would require the creation of a "constants" table in the front matter of this document and that seems a little heavy just to define this one constant that exists only to facilitate computation, not something that is important to the spec or eth2 itself.

And the next issue arose when I made p2p-interface.md executable as it attempts to parse constants from any table it encounters. If you refer to that file currently, we have tables for tabular data but they do not hold spec constants so the spec build fails: https://app.circleci.com/pipelines/github/ethereum/eth2.0-specs/5565/workflows/87c8cc23-5cb0-4f3e-aad6-6a14025577c0/jobs/34587  There are a number of ways we could think to resolve this but a general one is simply an inline method to skip parsing of the next markdown element.

To address these problems, I've used a markdown parser to handle breaking up the spec files into textual elements and started using the python ast module to handle the parsing of python code. By using the markdown parser, I hope to make it easier to work over Markdown elements -- for example, parsing comments (and then using some convention to handling "skipping" tables, e.g. resolving (2)). And by beginning to use ast, hope to make it easier to deal with python code moving forward. Unfortunately, there is not a straightforward way to "dump" an AST object back to source code so stopped at a solution for (1) but this should lay the groundwork if we need it for some more pressing issue in the future.
~If we get general agreement on this PR/approach, then I can add a few more edits to deprecate the existing get_spec and clean up some of the attending code in setup.py.~

update: looks like i'll have to put the "spec building" code from setup.py into a separate module as using a third-party lib breaks setup.py on install -- will wait to get a clearer idea we want to proceed before moving ahead to fix that

#!/bin/bash

mkdir spec-experiment
cd spec-experiment

python3 -m venv venv
. venv/bin/activate

pip install "git+https://github.com/ralexstokes/eth2.0-specs@use-markdown-parser-for-literate-pyspec"

echo "
from eth2spec.altair import spec
print(spec.BeaconState)
" >> experiment_code.py

python3 experiment_code.py

Confirmed that this works, but it's silent about the marko install (it's there in the venv though)


Scroll after switching tabs does not register properly

Current behavior
This is an issue that, as far as i can see, is only occurring on Android.
In the situation where you have 2 (or more) tabs that have content which is scrollable, and the tab-view has swipeEnabled={true}. When the user switches from 1 tab to the other (see video), the first scroll interaction is not picked up by the scrollview inside the tab, but is captured by react-native-tab-view.
The strange thing is, that in the expo snack that i've linked, the behavior isn't present. However, when creating a react-native project from scratch (using react-native cli), and using the following code, the issue does happen.
Package.json:
"dependencies": {
    "react": "16.13.1",
    "react-native": "0.63.0",
    "react-native-pager-view": "^5.4.1",
    "react-native-tab-view": "^3.1.1"
  }

App.js:
import * as React from 'react';
import { ScrollView, Text, StyleSheet } from 'react-native';
import { TabView, SceneMap } from 'react-native-tab-view';

const FirstRoute = () => (
  <ScrollView style={{ flex: 1, backgroundColor: '#ff4081' }}>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
  </ScrollView>
);

const SecondRoute = () => (
  <ScrollView style={{ flex: 1, backgroundColor: '#673ab7' }}>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
    <Text style={styles.text}>This is text</Text>
  </ScrollView>
);

const renderScene = SceneMap({
  first: FirstRoute,
  second: SecondRoute,
});

export default function TabViewExample() {

  const [index, setIndex] = React.useState(0);
  const [routes] = React.useState([
    { key: 'first', title: 'First' },
    { key: 'second', title: 'Second' },
  ]);

  return (
    <TabView
      style={{marginTop: 32}}
      navigationState={{ index, routes }}
      renderScene={renderScene}
      onIndexChange={setIndex}
      swipeEnabled={true}
    />
  );
}

const styles = StyleSheet.create({
  text: {fontSize: 50, marginBottom: 64}
})

Expected behavior
I expect that a vertical scroll (and keeping in mind that users don't scroll 100% vertical) is picked up by the component inside of the tab (such as the scrollview in my example), so that the user can access all the content that is available.
Reproduction
https://snack.expo.dev/@ted-q42/tabbar-scrollview
Platform

[X] Android
[ ] iOS
[ ] Web
[ ] Windows
[ ] MacOS

Environment

package
version

react-native-tab-view
3.1.1

react-native-pager-view
5.4.1

react-native
0.65.1

expo
(not using expo)

node
v14.16.0

npm or yarn
6.14.11

Expo isn't used in my project

I tested this in my team's app as wel. We are still using v2 of react-native-tab-view. But as soon as we upgrade to v3, that uses the pager-view package, this happens. The scroll/swipe down is registered as a vertical swipe. This is a major UX-problem and prevents us from upgrading.

I have same issue :(

same issue

same issue

I have the same issue on Android

same issue @satya164 please resolve this sir


Does incomplete labeling affect SAM2 training?

I have some images with a lot of instances (>100) for each. I am wondering if I labeled only some of the instances, will the training be affected. I noticed that bounding boxes can be used as prompts in the training. Does this mean only information in a bounding box
is used when predict a single mask?  If so, other part of image will not affect the training?

It will most likely negatively impact the training, as features are extracted from the entire image. If some of the object you are interested in segmenting were not labeled, the model will learn to ignore them.

@ywangwxd The prompts are expected to refer to a specific instance. So if there are other instances in the image that are not annotated during training, it is not a problem.


Remove static properties for ie8 compatibility

By removing usages of class-properties which are still experimental, ie8 is fully supported. We also protect against it happening in the future with a blacklist of that feature in babel.

Please either bump .babelrc to stage 1 or blacklist the class properties transform so future changes can't break this.

How do I blacklist a plugin for babel? Googling proved messy. I think if that's doable, it'd be best since it's the smallest surface area change.

{
  "stage": 0,
  "loose": "all",
  "blacklist": [
    "es7.classProperties"
  ],
  "plugins": [
    "dev-expression"
  ]
}

off the top of my head, but you should double check that this works (i.e. that Babel throws a fit at you if you then try to use a class property)

Boom. it worked! Well done. Some features are hard to find, didn't know that was a thing!

Can you squash those commits?

done

You need to update the tests as well, I think.

all the tests pass for me locally, are you saying there is a test I should add for this?

Not with the Babel changes. See e.g. https://github.com/rackt/react-router/blob/v1.0.0/modules/tests/Router-test.js#L258-L260.
Might make more sense to de-blacklist that transform for tests, though.

Babel doesn't transform statics to normal object properties?

Nope, it uses Object.defineProperty.

Hey guys,
Sorry I'm having issues figuring out how to best make the tests not use the
default babelrc but instead use a different one. The babel loader is always
a mystery to me in how much it respects the default babel parameters... Any
help would be greatly appreciated.
On Fri, Nov 13, 2015 at 12:05 PM, Jimmy Jia<EMAIL_ADDRESS>wrote:

Nope, it uses Object.defineProperty.
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/react-router/pull/2540#issuecomment-156489408.

Just clean up the tests to not use class properties then. Probably easiest way to do it.

Hey guys, I've made the appropriate changes, can someone confirm that this (https://github.com/rackt/react-router/pull/2540/files#diff-82b958d0880900cdd7c05f94eb64e281R271) looks okay to them?

@timdorr Did this catch all the code nits you saw? I think dropping static properties is worth it if that's all it takes for us to support IE8.

LGTM.

Woot! Thanks for everyone's help!

@taion Did you want to merge this or should I?

I don't want @mjackson to yell at me again :see_no_evil: :laughing:
I'll merge though :D

Oops, slipped up - https://github.com/rackt/react-router/pull/2540/files#diff-82b958d0880900cdd7c05f94eb64e281R259 is wrong. I'm just going to fix in a separate PR.

@ryanflorence What do you think of cutting a v1.0.1 for this? Thus far, as far as I can tell, nothing else has come up that we can resolve before v1.1.0.

I would love to see that too. It's been two weeks, so a minor update isn't a big deal :smile:


API verification fails

Hi (me again, I know),
API verification fails. See error message:
$ boldigger-cline api_verification trial.xlsx COI_cluster_reps_curated_no_contam.fa
14:55:20: Starting API verification.
14:55:20: Collection OTUs without species level identification and high similarity.
Traceback (most recent call last):
File "c:\users\nauras\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
return run_code(code, main_globals, None,
File "c:\users\nauras\programs\python\python39\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\Nauras\Programs\Python\Python39\Scripts\boldigger-cline.exe_main.py", line 7, in
File "c:\users\nauras\programs\python\python39\lib\site-packages\boldigger_cline_main.py", line 73, in main
api_verification.main(args.xlsx_path, args.fasta_path)
File "c:\users\nauras\programs\python\python39\lib\site-packages\boldigger_cline\api_verification.py", line 17, in main
raw_data, data_to_check, seq_dict = extract_data(xlsx_path, fasta_path)
File "c:\users\nauras\programs\python\python39\lib\site-packages\boldigger\api_verification.py", line 31, in extract_data
raw_data = pd.read_excel(xlsx_path, sheet_name = 'BOLDigger hit')
File "c:\users\nauras\programs\python\python39\lib\site-packages\pandas\util_decorators.py", line 296, in wrapper
return func(*args, **kwargs)
File "c:\users\nauras\programs\python\python39\lib\site-packages\pandas\io\excel_base.py", line 304, in read_excel
io = ExcelFile(io, engine=engine)
File "c:\users\nauras\programs\python\python39\lib\site-packages\pandas\io\excel_base.py", line 867, in init
self._reader = self.enginesengine
File "c:\users\nauras\programs\python\python39\lib\site-packages\pandas\io\excel_xlrd.py", line 22, in init
super().init(filepath_or_buffer)
File "c:\users\nauras\programs\python\python39\lib\site-packages\pandas\io\excel_base.py", line 353, in init
self.book = self.load_workbook(filepath_or_buffer)
File "c:\users\nauras\programs\python\python39\lib\site-packages\pandas\io\excel_xlrd.py", line 37, in load_workbook
return open_workbook(filepath_or_buffer)
File "c:\users\nauras\programs\python\python39\lib\site-packages\xlrd_init.py", line 170, in open_workbook
raise XLRDError(FILE_FORMAT_DESCRIPTIONS[file_format]+'; not supported')
xlrd.biffh.XLRDError: Excel xlsx file; not supported
Seems like a problem with reading Excel files?? BOLDiger results file with BOLDigger top hits attached.
trial.xlsx
I have upgraded to the most recent version of boldigger-cline, but for some reason it keeps saying version 1.0.0 is installed.
Cheers
Nauras

Will be fixed by using the correct upgrade. See my comment on the other issue.

Thanks, appreciate it.


Fix Build and Deploy Tests

This patch fixes the package-lock.json to work with node version
v14.18.1 which is used in the GitHub Actions workflows.

The partially removed web-vitals now also break the build.

Already fixed as part of #518


api: Add JSDoc links to API calls

Adds links to all API calls that have a corresponding online documentation page.

Thanks @borisyankov!
This isn't actually all of them, right? At a quick look at the left-sidebar nav on https://zulipchat.com/api/ and spot-checking the ones that look likely: there are several in messages/ and several in subscriptions/ that this doesn't cover. Possibly others; I haven't checked comprehensively.
I'll go merge this but with the commit message edited to not say it's all of them.


Ensure Compatibility of All Commands with PostgreSQL

We have identified the need to test all commands in Namingo for compatibility with PostgreSQL. This is a crucial step in ensuring that our application functions seamlessly across different database systems, particularly when transitioning from MariaDB to PostgreSQL.
A key area of focus should be the "update" column in the domain table. We suspect that there might be nuances in how PostgreSQL handles data in this column compared to MariaDB. Thorough testing and validation are required to confirm its behavior and ensure data integrity.
Additionally, specific attention must be paid to date calculations. These operations are known to have subtle differences between database systems, and it's essential to ascertain that our date-related functionalities perform consistently across both MariaDB and PostgreSQL.
We are calling for contributors to assist in this testing process. Here’s how you can help:
1. Test Execution: Run existing commands and observe their behavior in a PostgreSQL environment. Pay special attention to data manipulation commands and those involving date calculations.
2. Issue Reporting: If you encounter any discrepancies or errors, please document them with detailed descriptions and steps to reproduce. This information will be invaluable in pinpointing and resolving compatibility issues.
3. Suggestions and Fixes: If you have expertise in PostgreSQL and have suggestions for modifications or fixes to improve compatibility, your input would be highly appreciated. Code snippets, pull requests, or detailed explanations of potential solutions are all welcome.
Your contributions to this effort will play a significant role in enhancing the robustness and flexibility of our system. Thank you in advance for your support and involvement!

Proposal is to update update column everywhere to lastUpdate

Update: most parts should work with PostgreSQL. Only some date calculaitons can fail, we are gradually replacing them with compatible alternatives.

    if ($db_type == 'pgsql') {
        $pdo->exec("SET search_path TO registry, registryTransaction, registryAudit, public");
    }


Automated cherry pick of #4757: fix: brand of compute quota key should be fixed

Cherry pick of #4757 on release/3.0.
#4757: fix: brand of compute quota key should be fixed

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.0@0adccb5). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             release/3.0   #4760   +/-   ##
=============================================
  Coverage               ?   6.79%
=============================================
  Files                  ?     524
  Lines                  ?   89738
  Branches               ?       0
=============================================
  Hits                   ?    6095
  Misses                 ?   83136
  Partials               ?     507

Flag
Coverage Δ

#aFlag
6.79% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/guestdrivers/esxi.go
3.49% <0%> (ø)

pkg/compute/guestdrivers/kvm.go
3.37% <0%> (ø)

pkg/compute/models/guest_actions.go
0% <0%> (ø)

pkg/compute/guestdrivers/huawei.go
8.92% <0%> (ø)

pkg/compute/guestdrivers/google.go
26.31% <0%> (ø)

pkg/compute/guestdrivers/aws.go
4.58% <0%> (ø)

pkg/compute/guestdrivers/aliyun.go
5% <0%> (ø)

pkg/compute/guestdrivers/zstack.go
6.66% <0%> (ø)

pkg/compute/guestdrivers/qcloud.go
3.64% <0%> (ø)

pkg/compute/guestdrivers/openstack.go
2.87% <0%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


problem with tabrisJsCard plugin on tabris 2.0.1 beta

Hello i have a problem with this plugin
https://github.com/jokorivai/cordova-plugin-tabrisjs-card
i followed the syntax for a manualy import but i have this error

this is my code i have no problem with the require()
let initializeCard = require('../modules/tabrisjscard.js').createCard;
let card1 = initializeCard('Card1 Content', 'Card1 Title').appendTo(tabLesson);

Looks like the plugin isn't properly handling a missing options parameter.  Try passing {} as the third parameter or use one of the other examples.
CC @jokorivai

it's work thank you you save my night

Thanks @cookieguru. Closing this one.

@ishigo1987 this issue has been fixed and available on npm. Just update it.


[WIP] Add level1 and level2 metadata

Currently this is just the script, but once the spreadsheet is finished and we're happy with the data, we can supply it to this script to update all of the guides.
Usage: import-nav-reorg.rb [options]
    -s, --source [STRING]            Path to local TDC Source
    -c, --csv [STRING]               Path to CSV file with changes
    -t, --types [STRING]             CSV list of types of content to update
    -d                               Dry run

Updated to run the script and tag all existing guides


IAM support for Bookshelf on S3

If not, I think this would be a great addition so as not to have to create users with permanent access keys on AWS instances.

Having just tried not specifying the access key in chef-server.rb, I can say that that does not work

The original reporter is correct that bookshelf doesn't currently support IAM.  I've modified the title to make this into an enhancement request.

@stevendanna do you know if any work towards this is planned?  I looked into scraping up time for a PR, but found out that this is in Erlang, which will take me forever to figure out.  I was just setting up a new cluster, and got hit by this issue.  I'd love to not have to use a user + key for this; it complicates setup (having to store the key outside of git), and is less secure.

+1 for support on this ticket!
This would be a great improvement.

+1
This would be a very useful addition

+1
This would be extremely helpful, one less credential to keep secret, rotate and track...

Hey @pghalliday, just for my understanding, Are you
A) storing cookbooks in amazon s3 -
In this case bookshelf is not being used. Erchef talks directly to s3.
Here we would want erchef to use IAM credentials to authenticate user to s3.
B) storing cookbooks on the chef-server -
In this case erchef does not talk to s3, used bookshelf instead to store cookbooks.
Here we would want erchef to use IAM credentials to authenticate user to bookshelf and bookshelf be able to accept the IAM credentials as authentication.
We think you want case A, storing cookbooks in amazon s3.
That would help us define the scope of the request better. Thank you!

https://github.com/chef/chef-server/issues/2217 is a dependency on us being able to support IAM data in the future


Date.parse accepts illegal ISO8601 format

Date.parse("2018-10-21Z"); // lacks time, it should be 2018-10-21T00:00Z

Expected: NaN
Actual:<PHONE_NUMBER>000
Date.parse("2018-1-7T00:00Z"); // no padded zero, should be 2018-01-07T00:00Z

Expected: NaN
Actual:<PHONE_NUMBER>000

After a bit of research, I think this behavior is actually for the best. The spec says "If the String does not conform to that format the function may fall back to any implementation-specific heuristics or implementation-specific date formats." Currently V8 accepts 2018-10-21Z and SpiderMonkey accepts 2018-1-7T00:00Z, so the best possible behavior for compat is to accept both.


Upload dialog is not localized

frontend/src/components/molecules/messages/components/AskUploadButton.tsx is missing localization.

Looking forward for PR with missing localisation! ❤


dermatoplasm, gillied

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,g]

/trunk merge


When I run npx create-react-app my-app, git bash says : "Creating a new React app in {path}" but it stucks here. I'm waiting but noting is happening.

Is this a bug report?
(write your answer here)

Did you try recovering your dependencies?

(Write your answer here.)
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

(paste the output of the command here)
Steps to Reproduce

(Write your steps here:)

Expected Behavior

(Write what you thought would happen.)
Actual Behavior

(Write what happened. Please add screenshots!)
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Ok just be patient. It takes a long time to install.

Sounds like this is resolved; it does take a while.


Issues and suggestions for hitrate command

First the suggestions for the hitrate command:

A header showing the clan names would be very helpful. At the moment you don't know instantly which is opponent and which is home clan.
What about TH 11 vs TH 11 3 stars?

Bug reports

10 vs 11 seems to be wrong. It shows 1/2 attacks for opponent although there was no attack from a TH 10 vs 11. But maybe I'm wrong and the second table is referring to the defenses and not opponent attacks?

Home team should always be on the left. I can add 11v11. The person who requests hitrate in the first place didn't ask for it so I stuck with what he asked for the begin with.

Will be fixed in the next release. Is updated on the develop branch to include all TH levels 3 star.


Setting up web.config for IIS to connect to service process.

I've ran into a brick wall; the documentation is clearly lacking but I think there's more than that as well.
We have this already very large web application written in .NET 5; and it currently hosts itself directly on a service process and really should operate that way. It's not designed to recycle worker processes; but rather gets its best performance if its process runs forever. It was always the plan however to run it from a subtree under IIS; that is accessed by https://hostname/path rather than http://hostname:highport/; and we're looking to update our own setup instructions to set this up. For cloud installs we handle this in the fan-out application gateway on azure; but some customers host themselves.
What I can't find is documetation for the <aspNetCore node; particuarly I believe it should be able to connect to http://localhost::highport rather than launch a new process but I can't figure out how. I have been unable to find the reference document on the aspNetCore node so I don't know what the true extent of its capabilities are.

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

ID: f99cd387-53b0-8b37-4f2c-fc4edd1ed080
Version Independent ID: a1fcfbef-c5f0-c0f4-4246-8aee6ac72aea
Content: Out-of-process hosting with IIS and ASP.NET Core
Content Source: aspnetcore/host-and-deploy/iis/out-of-process-hosting.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @jhudsoncedaron ... The guidance that the product unit reviewed and improved recently is here ...
https://docs.microsoft.com/aspnet/core/host-and-deploy/iis/web-config?view=aspnetcore-5.0#attributes-of-the-aspnetcore-element
WRT your scenario: We aren't going to be able to support you from the docs side. You can ask the product unit for advice by opening an issue on their repo at ...
https://github.com/dotnet/aspnetcore/issues
... but be forewarned 💀 that they're awfully busy and can only offer limited product support for edge case scenarios. They may comment and help you with your scenario, or they may not be able to help and close your issue as a support request that's outside of ASP.NET Core. Closing an issue usually happens when they feel that the question is too far afield of the ASP.NET Core Module's interaction with IIS and the issue is more of a pure IIS hosting question.
As a last resort, you know that you can reach out to pros on public support channels who have worked with similar scenarios. We recommend the usual channels ...

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

... but, of course, there are others, including IIS-specific channels.


Blocking/deadlock/response not received exceptions when creating expectations from response callbacks

First off, thanks for quickly fixing issue #570! Unfortunately it looks like the fix for that issue introduced another - when creating expectations from within client-side response callbacks I'm now seeing requests time out and/or netty blocking exceptions being thrown.  Here's an example of the blocking exception, which seems to be the most common:
14:15:14.620 ERROR o.m.c.n.w.WebSocketClientHandler - web socket client caught exception
org.mockserver.client.netty.websocket.WebSocketException: Exception while receiving web socket message
	at org.mockserver.client.netty.websocket.WebSocketClient.receivedTextWebSocketFrame(WebSocketClient.java:105)
	at org.mockserver.client.netty.websocket.WebSocketClientHandler.channelRead0(WebSocketClientHandler.java:79)
	at io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
	at io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:323)
	at io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:297)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
	at io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1434)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
	at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:965)
	at io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:163)
	at io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:656)
	at io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:591)
	at io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:508)
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:470)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:909)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Thread.java:844)
Caused by: org.mockserver.client.netty.websocket.WebSocketException: Exception while starting web socket client
	at org.mockserver.client.netty.websocket.WebSocketClient.<init>(WebSocketClient.java:77)
	at org.mockserver.client.ForwardChainExpectation.initWebSocketClient(ForwardChainExpectation.java:180)
	at org.mockserver.client.ForwardChainExpectation.respond(ForwardChainExpectation.java:81)
	at com.test.robu.MockServerIssue.createCallbackExpectation(MockServerIssue.java:20)
	at com.test.robu.MockServerIssue.lambda$testExpectations$1(MockServerIssue.java:35)
	at org.mockserver.client.netty.websocket.WebSocketClient.receivedTextWebSocketFrame(WebSocketClient.java:92)
	... 22 common frames omitted
Caused by: io.netty.util.concurrent.BlockingOperationException: DefaultChannelPromise@7c08a754(incomplete)
	at io.netty.util.concurrent.DefaultPromise.checkDeadLock(DefaultPromise.java:395)
	at io.netty.channel.DefaultChannelPromise.checkDeadLock(DefaultChannelPromise.java:159)
	at io.netty.util.concurrent.DefaultPromise.await(DefaultPromise.java:225)
	at io.netty.channel.DefaultChannelPromise.await(DefaultChannelPromise.java:131)
	at io.netty.channel.DefaultChannelPromise.await(DefaultChannelPromise.java:30)
	at io.netty.util.concurrent.DefaultPromise.sync(DefaultPromise.java:337)
	at io.netty.channel.DefaultChannelPromise.sync(DefaultChannelPromise.java:119)
	at io.netty.channel.DefaultChannelPromise.sync(DefaultChannelPromise.java:30)
	at org.mockserver.client.netty.websocket.WebSocketClient.<init>(WebSocketClient.java:47)

Sometimes requests simply time out though, with an error logged like so:
org.mockserver.client.netty.SocketCommunicationException: Response was not received from MockServer after 20000 milliseconds, to make the proxy wait longer please use "mockserver.maxSocketTimeout" system property or ConfigurationProperties.maxSocketTimeout(long milliseconds)
These requests take only milliseconds so I'm sure it's simply the same blocking/deadlock issue manifesting slightly differently.
These issues are only present in version 5.5.1.  Everything works as expected on 5.5.0.
Here is an example class that reproduces the issue: https://gist.github.com/ininrobu/c14d2dd53879f0e81e505c836e81caa7

I'm not sure this is an issue I can solve without adding considerable overhead.  The crux of the issue is that you are re-using the same MockServerClient instance inside a web socket call back handled by the MockServerClient.  The problem is that these share EventLoopGroup but also have Futures, etc to ensure the right things occur in the correct order.  This means that is a Future is blocked waiting on a task on the same EventLoop you have a deadlock because that task won't get scheduled because of the blocking call to the Future.  I may be able to unpick this but initial basic attempts don't look promising unless I can remove all futures and replace them with callbacks.  That is particularly hard because inside the MockServer everything is asynchronous but the MockServerClient API isn't.  I'll try but in the mean time the simple solution is as follows:
import org.mockserver.client.MockServerClient;
import org.mockserver.integration.ClientAndServer;
import org.mockserver.mock.action.ExpectationResponseCallback;
import org.mockserver.model.HttpRequest;
import org.mockserver.model.HttpResponse;

import java.net.HttpURLConnection;
import java.net.URL;
import java.util.UUID;

import static org.mockserver.model.HttpRequest.request;
import static org.mockserver.model.HttpResponse.response;

public class MockServerIssue {

    public static void main(String[] args) throws Exception {
        new MockServerIssue().testExpectations();
    }

    private final MockServerClient mockServerClient = new ClientAndServer(8080);

    private void createCallbackExpectation(final String path,
                                           final String method) {
        new MockServerClient("localhost", 8080)
//        mockServerClient
                .when(request().withMethod(method).withPath(path))
                .respond(new ExpectationResponseCallback() {
                    @Override
                    public HttpResponse handle(HttpRequest request) {
                        // Actual logic omitted, typically some sort of serde/bookkeeping
                        return response().withBody(String.format("Contrived payload from %s %s", method, path));
                    }
                });
    }

    /**
     * Call this to reproduce the issue
     */
    private void testExpectations() throws Exception {
        mockServerClient
                .when(request().withMethod("GET").withPath("/reproduce"))
                .respond(new ExpectationResponseCallback() {
                    @Override
                    public HttpResponse handle(HttpRequest request) {
                        final String id = UUID.randomUUID().toString();
                        final String path = String.format("/resources/%s", id);
                        createCallbackExpectation(path, "GET");
                        createCallbackExpectation(path, "PUT");
                        createCallbackExpectation(path, "PATCH");
                        createCallbackExpectation(path, "DELETE");
                        return response().withBody(id);
                    }
                });

        final URL url = new URL("http://localhost:8080/reproduce");
        final HttpURLConnection connection = (HttpURLConnection) url.openConnection();
        connection.setRequestMethod("GET");
        System.out.println("Got response code: " + connection.getResponseCode());
    }
}

It won't be possible to improve this without adding too much overhead by creating too many thread pools (one per object callback).  Therefore I've added a meaningful error about not reusing the MockServerClient.  That way it is up to the user to use multiple MockServerClient instances (each one has a thread pool) as necessary.  I'm going to close this ticket as this is the best possible without other negative impacts.


issue: Error in HEX values on the palette

Did you check docs and existing issues?

[X] I have read all the flow.nvim docs
[X] I have searched the existing issues of flow.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.1 - Build Type: Release - LuaJIT 2.1.17
Operating system/version
Debian 12 bookworm on WSL2 W10
Describe the bug
I'm working on bringing your colorscheme to Terminal on Windows for those who are using Neovim through WSL. While reading the HEX values in your flow-palette.png I noticed that the values for light blue and cyan in the dim row are the same. After perusing the repo, I found the HSL values to get the correct HEX:

light_blue = {
    dim = "#0F578A"
}

cyan = {
    dim = "#14B8AA"
}

Steps To Reproduce
flow-palette.png -> dim row -> light blue and cyan
Expected Behavior
I do not know how to create that kind of picture to represent the palette, but, if I find a way I'll submit the corrected .png.

Hey @Someone-Who-Writes-Code thanks for opening this issue! I created the image with Figma but it is not shared the file. I will update it in the following days :) thanks again!

Fixed!

Thank you! I’ve been out for a while due to medical issues, but I wrote a Python program that outputs a similar template in PNG format and totally forgot to put it here. In a couple days I'll send you the colorschemes to windows terminal.


wont work for webcamxp

it says every time "get data from API failed"

Hello, I've added debug mode to this application. Please update the software with pip, and run this:
sfw search webcamXP --debug=True
And please send the bug you've encountered here, thanks!


Add additional test cases for logSanitizer

Two changes outside of your PR I'm going to recommend. I've messaged you the details on teams due to their sensitivity.


Bump FastExpressionCompiler to version 3.3.4

Please bump version of FastExpressionCompiler to version 3.3.4

@tcadm v3.3.4 was rolled back due to a bug in it.


to-do

Need to clean up Yosemite script and check for benchmark updates.
Need to complete ElCapitain

New artifacts section
make some statements more audit/remediate instead of assuming original setting
go through all sections even if our environment doesn't use them

Cleaning Yosemite script today.

closing as this has long since been done.


[Feature Request] Add "open destination folder" button to the web version

Requested Feature
Add "open destination folder" button to the web version
Possible implementation
No response

No stale


Temperature from Homekit sensors?

Luke! Thanks a lot for your plugin!
May be it's time to use temperature data from homekit sensors?

Yesss It would be!!
Most of users have at least one of homekit compatible sensor

@Partizan7676 @rapsody2000 I created a solution to this when @lprhodes included an external source for the temp settings in 314. This only works for Hue motion sensors (which include temperature)

you can use node.red to read the value and to be published on MQTT telemetry... than read that telemetry with the plugin.

you can use node.red to read the value and to publish it as a MQTT telemetry... than can be read with the plugin.

what plugins are you using to do this with node-red?

Unfortunately this is a limitation in Homekit.


Attempting to get terminal with results in odd NTERNAL_GHCID.putStrLn errors

Hey,
Attempting to use System.Console.ANSI getCursorPosition, setCursorPosition (to do the dance to find out the width of the terminal) but when run in ghcid the following comes out:
<interactive>:24:1-22: error:
    Not in scope: ‘NTERNAL_GHCID.putStrLn’
    Perhaps you meant one of these:
      ‘INTERNAL_GHCID.putStrLn’ (imported from System.IO),
      ‘INTERNAL_GHCID.hPutStrLn’ (imported from System.IO),
      ‘INTERNAL_GHCID.putStr’ (imported from System.IO)
    No module named ‘NTERNAL_GHCID’ is imported.

at the end of the program run, which is just weird on all kinds of levels. (It works when built as a binary and run independently in a shell.) There was also a ^[[41;1R that appeared on the console a few lines earlier.
Not sure that this is your problem in any way whatsoever, but I thought I should mention it.
AfC
P.S. Do you know a way to get terminal width?

The code I'm running is
getConsoleWidth :: IO (Int)
getConsoleWidth = do
    pos <- getCursorPosition
    case pos of
        Just (row, col) -> do
            setCursorPosition 999 999
            pos' <- getCursorPosition  -- bottom right
            setCursorPosition row 0
            case pos' of
                Just (height, width) -> do
                    return width
                Nothing -> return 80
        Nothing         -> return 99

In ghcid I get 99 back when I run that right after startup. But as a binary I get the correct current terminal width.
(this is the trick used by the resize command, apparently.

If you are moving the cursor while ghcid is sending commands to it I imagine you are messing up the output and ghcid is getting bust as a result?
Ghcid already gets the size of the terminal using https://hackage.haskell.org/package/terminal-size-<IP_ADDRESS>/docs/System-Console-Terminal-Size.html - I suggest just using that.

Interesting. Didn't know about that one.
fwiw, calling its size inside ghcid results in Nothing. Weird. Oh well. It does work as advertised when built as a binary. Thanks for the suggestion. I'll use it in preference to the ANSI hack (and accept that it'll be a bit unavailable in ghcid sessions).


Fix MTIAsyncVideoCompositionRequestHandler crash

Crash when sourceImages will return no images for track, or when sourceImages is empty
This fixes #255

Thanks @casper6479


Try to make macro names terser without losing clarity

The blends can probably skip the word "immediate", in fact the probably all can. Or at least say "imm" or something?
The shifts can probably use shl and shr

#53 closes this enough. We might have a double check before 0.4 goes out but it's at least "closed enough" for now.


Update HAOS 2025

Hello,
I have this message from HAOS in my log, can you change the value ?
POWER_KILO_WATT was used from hass_stokercloud, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfPower.KILO_WATT instead, please report it to the author of the 'hass_stokercloud' custom integration
TEMP_CELSIUS was used from hass_stokercloud, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please report it to the author of the 'hass_stokercloud' custom integration
MASS_KILOGRAMS was used from hass_stokercloud, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfMass.KILOGRAMS instead, please report it to the author of the 'hass_stokercloud' custom integration
STATE_UNLOCKED was used from nodered, this is a deprecated constant which will be removed in HA Core 2025.10. Use LockState.UNLOCKED instead, please report it to the author of the 'nodered' custom integration
Thank you

Hi,
So looking at earlier messaged, the original author is no longer active on this repo.
A new updated one can be found here.


Imagery.json API / schema documentation incorrect

The API docs state that custom imagery definitions should follow the structure defined by editor-layer-index however, it doesn't. data/imagery.json uses the template, scaleExtent and polygon properties which are not defined in that schema. It would be good to have updated docs about what the schema actually is.

I'm updating the API docs today to include a note that iD's imagery structure doesn't match editor-layer-index.  I'll leave this issue open, because I agree we should document it better, or possibly change our structure to match editor-layer-index more closely.

I'll leave this issue open, because I agree we should document it better,

nevermind what I said here, I just updated the docs and closed the issue..


Unexpected initial token 'Boolean' when populating object. Expected JSON object or array

I am migrating to the newest elastic/nest and found a problem. For example I have this code
public class MyDocument : IDocument
{
    [JsonProperty("$type")]
    public virtual string Type => type;
    private static readonly string type = $"{typeof(MyDocument).FullName}, {typeof(MyDocument).Assembly.GetName().Name}";

    public DateTime SomeDate { get; set; }

    public string SomeText { get; set; }

}

public interface IDocument
{
    // at least two classes implements it. we need this to query and return documents by interface
}

public void InitializeIndex()
{

   EnsureIndex(elasticClient, "my-index",
                              d => d.Settings(s => AnalysisSettings.TextIndex(s.NumberOfShards(2)
                                                                               .NumberOfReplicas(10)))
                                    .IncludeTypeName());

    EnsureMapping<MyDocument>(elasticClient, "my-index",
                                            Types.Type<MyDocument>(),
                                            x => x.AutoMap()
                                                  .DateDetection()
                                                  .NumericDetection()
                                                  .IncludeTypeName()
                                                  .Index("my-index"));
}

public static void EnsureIndex(IElasticClient client, string indexName, Func<CreateIndexDescriptor, ICreateIndexRequest> createFunc = null)
{
    var existsResponse = client.IndexExists(Indices.Index(indexName));
    if (!existsResponse.Exists)
        client.CreateIndex(indexName, createFunc);
}

public static void EnsureMapping<T>(IElasticClient client,
                                    Indices index, Types type,
                                    Func<PutMappingDescriptor<T>, PutMappingDescriptor<T>> customize = null) where T : class
{
    var readOnlyDictionary = client.GetMapping<T>(m => m.Index(index).AllTypes()).Indices;
    if (!readOnlyDictionary.ContainsKey(type.ToString()))
        client.Map<T>(m =>
                      {
                          var defaultSettings = m.AutoMap().IncludeTypeName().Index(index);
                          return customize != null ? customize(defaultSettings) : defaultSettings;
                      });
}

This code on the first run  InitializeIndex creates index with such mappings
"mappings": {
	"mydocument": {
		"numeric_detection": true,
		"date_detection": true,
		"properties": {
			"someText": {
				"type": "text",
				"fields": {
					"keyword": {
						"ignore_above": 256,
						"type": "keyword"
					}
				}
			},
			"someDate": {
				"type": "date"
			},
			"$type": {
				"type": "text",
				"fields": {
					"keyword": {
						"ignore_above": 256,
						"type": "keyword"
					}
				}
			}
		}
	}
}

On a second launch I got this exception lasticsearch.Net.UnexpectedElasticsearchClientException: Unexpected initial token 'Boolean' when populating object. Expected JSON object or array. Path 'my-index.mappings.date_detection', line 1, position 46 when var readOnlyDictionary = client.GetMapping<T>(m => m.Index(index).AllTypes()).Indices was executed.
All of this code works fine with our current elastic 6.6.1 and NEST 6.4.0
Serializer settings JsonSerializerSettings { TypeNameHandling = TypeNameHandling.Auto }
NEST/Elasticsearch.Net version: 6.8.0
Elasticsearch version: 7.2.0

NEST/Elasticsearch.Net version: 6.8.0
Elasticsearch version: 7.2.0

You'll need to use a NEST 7.x client with Elasticsearch 7.2.0. Would you be able to try with the latest 7.x on Nuget (7.0.0-beta1 at this time)?

7.0 no longer returns types in the middle, you might get better results specifying include_type_name=true temporarily.
However as @russcam mentioned its not supported to run 6.x against 7.x.
https://www.elastic.co/guide/en/elasticsearch/reference/current/removal-of-types.html

Thanks! I guess I'll wait for a major 7.0 release and try with it.


"snapper create" error message "10-sdbootutil.snapper' failed."

localhost:~ # snapper create
Server-side plugin '/usr/lib/snapper/plugins/10-sdbootutil.snapper' failed.

The snapshot is created apparently. But what does the error message mean? System is an up to date openSUSE Tumbleweed x86_64

fixed in #34


Add support for propery source origin introduced in boot 2

This would be helpful in debugging configuration problems.
@dsyer @twicksell

Boot's env endpoint changed for a simple key -> value, to key -> {value, origin}. Config server would need to be backward compatible. Two options, either a new prefix (v2) or a content type (which is what boot uses). I think I lean towards the content type.
Boot's content type application/vnd.spring-boot.actuator.v2+json, so I would propose application/vnd.spring-cloud.config-server.v2+json.
Example output:
//<PHONE_NUMBER>3612
// http://localhost:8888/foo/default

{
  "name": "foo",
  "profiles": [
    "default"
  ],
  "label": null,
  "version": "a611374438e75aa1b9808908c57833480944e1a8",
  "state": null,
  "propertySources": [
    {
      "name": "https://github.com/spring-cloud-samples/config-repo/foo.properties",
      "source": {
        "foo": {
          "value": "from foo props",
          "origin": "URL [file:/tmp/config-repo-7662911556124581061/foo.properties]:1:6"
        },
        "democonfigclient.message": {
          "value": "hello spring io",
          "origin": "URL [file:/tmp/config-repo-7662911556124581061/foo.properties]:2:27"
        }
      }
    },
    {
      "name": "https://github.com/spring-cloud-samples/config-repo/application.yml",
      "source": {
        "info.description": {
          "value": "Spring Cloud Samples",
          "origin": "URL [file:/tmp/config-repo-7662911556124581061/application.yml]:2:16"
        },
        "info.url": {
          "value": "https://github.com/spring-cloud-samples",
          "origin": "URL [file:/tmp/config-repo-7662911556124581061/application.yml]:3:8"
        },
        "eureka.client.serviceUrl.defaultZone": {
          "value": "http://localhost:8761/eureka/",
          "origin": "URL [file:/tmp/config-repo-7662911556124581061/application.yml]:7:20"
        },
        "foo": {
          "value": "baz",
          "origin": "URL [file:/tmp/config-repo-7662911556124581061/application.yml]:9:6"
        }
      }
    }
  ]
}

We could enhance the origin depending on the environment. For example, git blame information could be added, etc...

From /actuator/env
{
  "activeProfiles": [],
  "propertySources": [
    {
      "name": "configService:https://github.com/spring-cloud-samples/config-repo/foo.properties",
      "properties": {
        "foo": {
          "value": "from foo props",
          "origin": "Remote file foo.properties:1:6"
        },
        "democonfigclient.message": {
          "value": "hello spring io",
          "origin": "Remote file foo.properties:2:27"
        }
      }
    },

@spencergibb this is sort of related: https://stackoverflow.com/q/57149736/526664


Refactor cleanup - Telemetry Notice

Refactor discovering whether azd is in Cloud Shell to use the new "runcontext" package.
To release: ensure that a manual Cloud Shell test pass has no regressions.

@danieljurek assuming its more of internal refactoring, moving it to backlog. Pull it in when appropriate.


How to store and access the session object (e.g. with user) in pinia store

Ask your question
how can I access the logged-in user in my pinia store? For example in a store action?
//pinia store
const { data  } = useSession()
export const useMyStore = defineStore({
  id: 'my-store',
  ...
  actions: {
      async customActionFunction() {
        console.log(data.user.name) //--> not available
      },
  }
}

Thanks a lot for your help
Additional information
No response

@BracketJohn thx a lot! your answer gave me the right hints.
I ended up doing something like this:
export const useAuthStore = defineStore({
    id: 'auth-store',
    state: () => {
        return {
            ...useSession(),
        }
    },
})


Setup new domain (near.org)

Overview
Setup new urls for wallet and near-contract-helper using near.org domain. We will keep the nearprotocol.com subdomains live as well, for the time being.
TODOs

[ ] Wallet

[x] wallet.testnet.near.org
[ ] wallet.near.org redirects to-> wallet.testnet.near.org
[x] wallet.betanet.near.org
[x] wallet.devnet.near.org

[x] Helper

[x] helper.testnet.near.org
[x] helper.betanet.near.org
[x] helper.devnet.near.org

@vgrichina Who should handle this?

Basically done at this point besides confusion around wallet.near.org redirects to-> wallet.testnet.near.org


Unable to get the config value of a proxy target in the frontend

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce

Step 1
Step 2
...

Context

Your Environment

NodeJS Version (v12):
Operating System and Version (e.g. Ubuntu 14.04):
Browser Information:

That's working as intended imo, the proxy endpoint should be used rather than using the target directly.

@Rugvip

That's working as intended imo, the proxy endpoint should be used rather than using the target directly.

Sorry, I've  submitted this issue while writing it. I've updated with some context.

Using useApi(discoveryApiRef).getBaseUrl('proxy') you can find the base URL of your proxy, which would be something like https://backstage.example.net/api/proxy.
On top of that you probably have some hard coded, well known prefix for the proxied route, let's say /artifactory. You know this prefix because you choose it yourself, so you can freely hard code it in the frontend.
Now if you get an URL from artifactory, you should be able to just slap that path on top of the previous ones, so you get https://backstage.example.net/api/proxy/artifactory/<path goes here>.
Does that match what you want to do?

Thanks @freben, using URL to get only the pathname/search from the original url is rather smart and accomplish what  I want to do.
The only thing is discovery.getBaseUrl returns a Promise, but I can work around that.
There is still an issue regarding the config visibility but since there is an alternative solution, I close this issue.


make sure inputs live on CPU for ctc decoder

addressing #2274

thanks @carolineechen and @mthrok for the thorough review of my first PR!


ENYO-6299: Call stop() when clicking not on but out of paging controls

Checklist

[X] I have read and understand the contribution guide
[X] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[X] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
If clicking on a paging control several times, the Scroller including the paging control did not scroll smoothly.
Resolution
Historically to stop scrolling when pressing a mouse down on items while flicking, stop() in Scrollable was called in https://github.com/enactjs/enact/pull/1502.
But the DOM element having that mouse down event handler was changed from the container having items to the container having not only items but also paging controls in https://github.com/enactjs/enact/pull/2468.
Then whenever clicking on the paging controls, the stop() was called and the Scroller did not scroll smoothly.
So if clicking on paging controls, the stop() is now not called.
Links
ENYO-6299
Comments
I tried to make any unit test for this issue and spent several time. But it failed. I hope that we could use TC instead of unit test case for this issue.

Hello @webOS101
With forwardWithPrevent, the code became much elegant. I'm happy to know how to use forwardWithPrevent brightly.

:Awesome:


Unexpected object caching

For testing purpose, I regularly want to reset the database content with mongorestore (parse-server is shut down while running that command). After the reset I run some code which creates a new class. Then, when I repeat the process -- reset the db, run the code again -- something weird happens: Parse seems to remember that class's instances from the previous run and recreates them. So with every run, I get more instances.
Again in short:

reset mongodb
create new class, add instance A
reset mongodb again
create new class, add instance B
now Parse contains instances A and B!

Can this be explained, or is it a bug?

That is highly unlikely as we don't store anything on Parse-server itself. That has to be related to your restoration process.

Any hint how I could evaluate whether it's related to the mongodb restoration?
The "ghost objects" for sure don't come from the backup data itself. I created the dump once and did not touch it afterwards; by looking at the objects' createdAt dates I can see that they have been created after  the dump was exported.
Btw -- I even could reproduce this with parse-dashboard: after the db reset (step 3 in my above list) I tried to manually add a class of the same name from inside the dashboard (without creating any instances); as soon as the class appeared in the data browser, the old instances popped up again.

I think this is happening because of your usage of drop:

Before restoring the collections from the dumped backup, drops the collections from the target database. --drop does not drop collections that are not in the backup.

So in your backup there documents for _Schema but no documents for the Data collection. When you restore, the _Schema will be dropped and recreated. Data will be unaffected. This will result in the dashboard not knowing about Data until you recreate it.
If you took your backup after creating the Data class but not adding data then you would get the same result without having to recreate the schema.
You can also verify by repeating your steps up to and including 6, then enter the mongo shell:
db.Data.count() should verify the Data documents were not removed.
db.Data.drop() will actually remove the documents and the collection

Great, thanks for your help with this! I'll change my test process to just copy a backup mongo data dir over instead of using mongorestore. This makes the process less easy to automate because starting/stopping mongodb from script is not as straightforeward, but at least I no longer have those objects reappearing.


Password characters

I am see this error when I try to create accounts using a Create Account Link and use a complex password (it seems):

When I installed my test installation, I used much simpler passwords and did not have this problem.
When I set up my production install accounts, I was able to work around this and successfully create accounts by removing most of the special characters from the password.
I guess there's no harm (and good diagnostic info) in revealing that one of the passwords that would not work was: s9m/4Tu:sL@_nFf-. It was generated with a password manager. There was another that failed but unfortunately I don't have that any more.
A couple of weird things I noticed:

the admin account I set up initially has a few of what I assume are similar problem characters
as Admin, I can see the "failed" accounts in the list of users, even though they can't log in (I tried that too)

I think you should be able to reproduce this easily. Just ask if you want more info :)

Heh, I just noticed a typo in that popup dialogue too: "you administrator" → "your administrator".

Hmm, the fix should be quick. I am wrapping up a feature in the next couple days, and then I'll address this. Thanks for catching this!

This should be fixed in the latest container. When you have time, could you verify the fix?
With the 'failed' accounts, the password was saved incorrectly. Those accounts can be accessed from the admin panel with the 'lock' icon, and then the password can be reset (correctly this time), or they can just be deleted.
Thanks for your help with this project.


Revert "Better runtime failure messages (not yet enabled by default)"

Reverts apple/swift#25978. I believe this commit may have broken a few tests; running CI to see if reverting it helps.

@swift-ci please smoke test

Merging to unblock the bots.


[Bug] The White screen is displayed on the opened app after collapsing the app by double tapping on the device Back button

Description: The White screen is displayed on the opened app after collapsing the app by double-tapping on the device Back button
Steps to reproduce:

Launch the App
Tap twice on the Device Back button
Tap on the App icon
Pay attention to the white screen

Expected result: The screen that was displayed before collapsing should be displayed after reopening the app
Actual result: The White screen is displayed on the opened app after collapsing the app by double-tapping on the device Back button
Environment:  Android App (Staging/DEV)1.25.1-20, (PROD) 1.27.1
Devices list:
Xiaomi Mi 8 // Android 9
Attachment:
https://www.screencast.com/t/LZlDXC4j
Logs: https://app.zenhub.com/files/166818382/18f561f6-89b4-49be-972b-5e925978cb72/download

@serdiukov-o-nordwhale
Here is new logs of the issue:
Steps:

Launch the App
Tap on the "Skip and create wallet" button
Tap twice on the Device Back button
Tap on the App icon
Pay attention to the white screen

https://app.zenhub.com/files/166818382/e85d4279-5e0c-4bd5-89bb-982e7394711a/download
Also, there is another way when the app just crashes? but it probably related to the same issue:
Steps:

Launch the App
Tap on the Device Back button
Tap on the App icon
Tap on the "Skip and create wallet" button
Pay attention to the crash

Logs of the crash: https://app.zenhub.com/files/166818382/5e349b16-918b-4148-bd4c-97a8d64c80e4/download

@serdiukov-o-nordwhale @sirpy  The issue is not reproduced. The screen that was displayed before collapsing is displayed after reopening the app. The app is not crashed with the second way of the steps.
Checked on the Android App [Staging] 1.28.1-7 (1654)
Checked on the Xiaomi Mi8 // Android 9
Attachment:
https://www.screencast.com/t/eOX4VeUP

@serdiukov-o-nordwhale @sirpy
The issue is not reproduced. The screen that was displayed before collapsing is displayed after reopening the app. The app is not crashed with the second way of the steps.
Checked on the Android App [Staging] 1.29.1 (1690)
Checked on the Samsung A80 // Android 10
Attachment:
https://www.screencast.com/t/nrqyip9ZOg


Invalid stats report request causes 500 return code

Fixes #3229
@scottbommarito @xavierdecoster @skofman1 @shishirx34

:shipit:

:shipit:

Looks good! :shipit:

In what situation are we getting an invalid stats report?

See #3229 :)

lgtm


ManualValidation should have the option onTimeout: "cancel"

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Feature
Enter Task Name: ManualValidation@0
Issue Description
Actually I'm looking for a way to manually start pipeline parts (like I would do with gitlab-ci manual steps).
The ManualValidation@0 comes closest to want I need:
steps:
- task: ManualValidation@0
  timeoutInMinutes: 1440
  inputs:
    instructions:
      onTimeout: "reject"

But very often I just let the time-out period pass since no follow-up action is needed. The task will be rejected as expected, but I need something that is more neutral, like onTimeout: "cancel" and the pipeline does not have an error but a cancel status:

Is it possible to have an update on the feature? It's pretty annoying to see all our job in error when they are canceled in reality.
I am also interrested to implement it by myself if it's possible. Could you please give me some information on how to proceed?

Any updates on this?
BTW is the code for ManualValidation open source, so we can contribute to it and add this functionality?

I need this feature!!!

Please reactivate this

Similar request from the dev community: https://developercommunity.visualstudio.com/t/Cancel-should-cancel-all-stages/1534464


Headings on the edition index are unclear

https://emylonas.github.io/xml-minimal/edition/
The index on the left of the page reads:
EDITION (all caps, bold and with more white space below)
Libro Azzurro
then the pages for the introduction and the plants
is it possible to replace "EDITION" with "Libro Azzurro"? or just eliminate EDITION?
Ultimately Sabrina would like to have the books lists (Rosso, Giallo, etc.) and you can either click an arrow to list the plants, Or don't have the submenus on this page, and just reveal all the plants on the next level down. This is not for now. Just noting it so it's here. Probably should move to new issue when the first part of this one is complete. But taking this into account may affect how the current issue is handled.

Heading is now Table of Contents


forbid-prop-types / Allow PropTypes.any in one case

Hi,
I would like to avoid defining proptypes of props that are not directly used by a component but are just passed through. Example :
// Parent.js
// In this case p2 is just passed through
const Parent = ({ p1, p2 }) => (

<Child p2={p2} /> {p1}

);

I have already posted an issue in prop-types project that you shoud read to understand :
https://github.com/facebook/prop-types/issues/305
The result of the previous issue is that the feature i need should be at lint time not run time. This is why i'm here ;-)

As Parent does not need to know the type of p2, i would like to be able to define it as PropTypes.any().
But to avoid to have to disable the rule each time and to avoid later issues (for example if the prop is finally used in a later update) it would be great if the rule would have this behaviour :
PropTypes.any() is allowed while the prop is only passed to a child component.
I don't know if it is feasible.

I don’t think it is - my opinion there is the same as here, that the child component should be properly exposing its propTypes, or that a code comment suffices (including an eslint override).

@ljharb What would you recommend as a general rule ? disabling eslint rule on the line ? import child's proptypes (what if you can't) ?

If you can, do; if you can’t, then personally I’d make my own version of what their propTypes should be, and import/use that; if you do in fact need an “any” then an eslint override comment.

Thanks. My goal is to avoid passing too much time defining (and re-re-re-defining) types. Our discussion points me out that what i need is maybe to avoid types more generally (proptypes and linting types). I am not a "type"script fan. Will think about that.
I keep this open for now. Let me know if any other idea that could simplify this comes to head :)

I'm not a TS fan either, but propTypes are very critical (and more powerful in most ways than a type system). I'd suggest trying very hard to ensure everything has rigorous propTypes defined.

Could you develop or point me out an article that explains why they are very critical and powerful ?

No, I'm not aware of any. Power, because they can, for example, target "integer", which TS can not - see https://npmjs.com/airbnb-prop-types for many examples.

Why component props should have types but function arguments not ? Is it only for "documentation" ?

That's not a thing that's possible in JS from outside a function, so it's a tangential issue.

I was just trying to understand why component props should have their types verified, while in JS in general types are not verified.

All function arguments should be verified in JS; just with runtime type checks. PropTypes are a DSL for those type checks, but in a way that easily strips them out in production.

I mean what the difference between these :
// 1. Component
const ComponentLogger = (props) => {
    console.log(props.message);
    return null;
};

<ComponentLogger message="hello" />

// 2. Function
const functionLogger = (message) => {
    console.log(message);
    return null;
};

functionLogger('hello');

Are you saying that both should use PropTypes ?

The first absolutely should. The second, since it's not a React component, should be something like this:
const functionLogger = (message) => {
    if (typeof message !== 'string') {
      throw new TypeError('message must be a string');
    }
    console.log(message);
    return null;
};


Is it safe to self host nancy and open to the internet?

Is it safe to put a nancy self hosted server into production and internet facing (Behind cloud flare of course)?
Also, whats the effect of slowlorris or similar attacks on nancy? i read the maximum connection count was determined as half the logical core count, which seems like it could be flooded easily with slowlorris connections.

You haven't stated how you plan to self-host (i.e. HTTP.sys or Kestrel), but in either case, nancy is really not the relevant factor here - it's the HTTP server. HTTP.sys has long been considered production-ready, and Kestrel is also considered production-ready now.
DISCLAIMER: Not a Nancy maintainer, just a Nancy user.

You haven't stated how you plan to self-host (i.e. HTTP.sys or Kestrel), but in either case, nancy is really not the relevant factor here - it's the HTTP server. HTTP.sys has long been considered production-ready, and Kestrel is also considered production-ready now.
DISCLAIMER: Not a Nancy maintainer, just a Nancy user.

Is the default in nancy http.sys?

It really depends what precisely you are doing, and what framework you are targeting.


RFE: please make it possible to core dump on program failures by default

I would like to have "crash" as the default value of GOTRACEBACK. I know I can export the environment variable in system profile but it is not so uncommon that applications run in clear environment (env -i). I am asking because, I want to automatically report program crashes and the easiest way how be notified about them is to register own /proc/sys/kernel/core_pattern.
Thank you for considering my request.

We're not going to make that the default. But doesn't https://golang.org/pkg/runtime/debug/#SetTraceback work for you?

I am working on ABRT - Automatic Bug Reporting Tool - that is enabled on Fedora. We detect core dumps, python exception, ... and would like to detect failures of go programs. Currently, we must export GOTRACEBACK=crash and pollute global environment to achieve this goal
SetTraceback would work if I needed to set it for a single program, but I want to be able to detected failures of all go programs.
I don't want you to make "crash" the default. I would be happy with a build-time configuration option that would make that default for Fedora.

We don't really do build-time configuration options.
About the closest we have is the CGO_ENABLED and GO386/GOARM environment variables.
Can you just add the 1-line patch to the Fedora repo? I assume you likely already carry a configuration patch or two in your tree?

Making coredump the default affects too many users but
I don't think the benefit outweigth the cost and confusion
it causes -- not every one has such automatic bug reporting
tool (and I also think if Go has such a report builtin, we
will just get a lot of irrelevant bug reports, besides it's
a violation of user's privacy.)

@minux, it sounds like he only wants to enable it by default on Fedora.
@jfilak, if you modify Fedora's version of Go, please do a (fast) check at start-up to verify you're running on Fedora (and ABRT is available) before setting the default GOTRACEBACK value.
It would be kinda unfortunate if a Fedora user distributed binaries to non-Fedora users and they were surprised by core dumps they didn't ask for and didn't expect.

Oh, I see. If we change something in https://github.com/golang/go/blob/master/src/runtime/runtime1.go, the change will be built into programs. That's something different than I hoped for and I'm not sure we want to do that. Perhaps we can find a way how to let official Go programs do core dump files while leaving user Go programs untouched. Seems that it won't be a 1-line patch.
Just a question, could it be an option of go build - e.g. --traceback=crash? It would be easier for us to pass this argument on command line when building official packages, then detecting that a programs is not running on Fedora.

What's an "official Go program"? One that Fedora builds and includes in the package manager?
You could use go build --tags=use_abrt (some arbitrary tag you make up), and then add a file to Fedora's copy of the runtime package, like:
// +build use_abrt

func init() {
    SetTraceback("crash")
}

So if that tag is specified when you build official programs, that file with +build use_abrt is included, and it then sets "crash" as the default for that process.

What's an "official Go program"? One that Fedora builds and includes in the package manager?

Yes, exactly.

You could use go build --tags=use_abrt ...

Sounds great! I'll give it a try.
Thank you very much indeed.