RangeWeekly and RangeMonthly

In my case I have weekly and Monthly reports;
I really like the way luigi uses:
luigi --module all_reports RangeDaily --of AllReports --start 2015-01-01,
but how can I use the same pattern for tasks with larger frequencies?
I probably can create a custom class based on RangeDaily and just change timedelta(days=i) with timedelta(days=31), but then I won't be able to use luigi cli interface, right? shall I pr those two classes?
Another corresponding question - how can I marry those with luigi.date_interval.Month parameter classes? or shall I in this case generate them from date parameter within the task?

Hey there,
I had the same problem and I just build my own classes and placed them in a utility module. You really only have to override the finite_datetimes method when inheriting from RangeDaily.
I used Pandas date_range method and designed the Range classes to NOT include any incomplete week or month if the stop date is not end of week or month.
`
from luigi.tools.range import RangeDailyBase
import pandas as pd
class RangeWeekly(RangeDailyBase):
"""
Run tasks for all weeks that are contained in the date range.
Always take the last day (Sunday) and will leave out incomplete weeks.
"""
def finite_datetimes(self, finite_start, finite_stop):
    """
    Simply returns the points in time that correspond to turn of week.
    """
    rng_idx = pd.date_range(finite_start, finite_stop, freq="W")
    rng = [x.to_pydatetime() for x in rng_idx.tolist()]
    return rng

class RangeMonthly(RangeDailyBase):
"""
Run tasks for all months that are contained in the date range.
Always take the last day of the month and leave out incomplete months.
"""
def finite_datetimes(self, finite_start, finite_stop):
    """
    Simply returns the points in time that correspond to turn of month.
    """
    rng_idx = pd.date_range(finite_start, finite_stop, freq="M")
    rng = [x.to_pydatetime() for x in rng_idx.tolist()]
    return rng

`
Hope this helps :)

Hello @robin-weiss-sm
Could you show how you run tasks with RangeWeekly in the cli?

@robin-weiss-sm thanks,
actually I have the same question as @Marabou

To be honest I don't ever run these range tasks from the CLI. It might be a problem because you can only specify one "--module" path if your tasks are split up into subfolders (as is my case).
What I end up doing is writing runner classes that run all my weekly aggregations. You could also just wrap your single use case in them I suppose.
import datetime
import luigi
from luigi import date_interval

from tasks.utils import RangeWeekly
from tasks.visits.unique_visitors import UniqueVisitors

class RunWeeklyVisitsTasks(luigi.WrapperTask):
    # Default start date if not defined
    start_date = datetime.date(2017, 1, 1)
    # Range does not include last date, so range ends with but includes yesterday
    end_date = datetime.date.today()
    date_range = luigi.DateIntervalParameter(default=date_interval.Custom(start_date, end_date))

    def requires(self):
        range_params = {
            'param_name': 'date',
            'start': self.date_range.date_a,
            'stop': self.date_range.date_b,
            'days_back': 365,
            'task_limit': 365
        }

        yield RangeWeekly(of=UniqueVisitors, of_params={"taskParamA": "A", "taskParamB": "B"}, **range_params)
        # ... put any other weekly aggregations here

With "of_params" you can pass any params through to your actual tasks.
If you HAVE to use the CLI, I guess you could put the RangeWeekly task definition in the same python module as your 'runnee' task. If you name that file "tasks.py" , then it would end up something like
python -m luigi --module tasks RangeWeekly --start '2018-01-01' --stop '2018-11-12' --of MyTask  --param-name 'date' --task-limit 100 --days-back 100 --local-scheduler

Remember that if your "date" input parameter is named 'date' you'll have to change the "param-name" value in the CLI call.
Hope that helps.


Update defender-endpoint-trial-user-guide.md

I'm part of the MDE Care Engineering Team, The reason for adding the note is that we've been receiving numerous ICM's requests for offboarding scripts after customer licenses expire. Previously, we had a pipeline in place to handle this on an exceptional basis, but it has since been decommissioned, and we no longer have that capability. By including this note, we aim to proactively inform customers and make the information publicly available, encouraging them to offboard devices before their trial period ends.

Thank you @vydixit. Expect these updates to be live at around 11 AM Pacific Time.


Added factorial problem by Lucas Trogo

Description
Write here what you have changed
Issue No.- #

Type issue number after hash(#)

Checklist

[ ] Yes, this task assigned to me.
[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] My Filename is in camelCase format
[x] I have added Problem statement, Problem link in file.

Missing and wrong issue number, Marking it as sapm


crash

crash on Android 8.1.0

same error on Android 7.1.1, Moto G4 plus

Same error Sony G8142 Android 9, API28
2019-03-02 09:50:50.744 18572-18572/com.example.android.camera2raw E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.example.android.camera2raw, PID: 18572
    java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.Object android.hardware.camera2.CameraCharacteristics.get(android.hardware.camera2.CameraCharacteristics$Key)' on a null object reference
        at com.example.android.camera2raw.Camera2RawFragment.configureTransform(Camera2RawFragment.java:1046)
        at com.example.android.camera2raw.Camera2RawFragment.access$000(Camera2RawFragment.java:121)
        at com.example.android.camera2raw.Camera2RawFragment$1.onSurfaceTextureAvailable(Camera2RawFragment.java:212)
...

In fact, I think this problem results from the device does not support RAW photography. I suggest downloading an app called "OpenCamera" and check device info to find out if your device supports RAW shooting.
lgyjg's fork addresses this crash.

This sample has been migrated to a new location (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


doc-link parser / links for non-kit sections

Documentation links for Svelte and Advanced Svelte sections.

Going to close this as the main Svelte docs are undergoing an overhaul that would break all these links in any case — we'll have to take another run at it some time when we can properly think through how to structure the data in a maintainable fashion


fix: prevent website not found errors after firebase deploy completes

Progress towards #64
This change is expected to restore frontend UI loading after firebase deploy completes. It does not address the gap between terraform apply and firebase deploy.

I updated the code to correctly validate that the URL should error (the assertResponseContains helper presumes a healthy URL, so that was the test failure). My go isn't the strongest so it could be improved. If you're okay with my change, mark this as "ready for review" and I'll approve and merge.

I refactored the code to have an assertErrorResponseContains function.


report: extract guilty file

A simple prototype of extracting the guilty file from a report.
Need to run this on bots and see in which cases it extracts the file incorrectly.

All done, PTAL

LGTM


Forgot password support

Is Forgot password supported ?

For that you will need to use the this:  BNetworkManager.sharedManager().getNetworkAdapter().sendPasswordResetMail());
I use it on the login page so people send a request to their email then they reset it using the automatically generated email and link

@ganeshub Have you gotten forgot password working? I am closing this issue since the question was answered.

@SimplisticApps Thanks
@kykrueger It works


Add a link to PICO-8 color palette for web

Added

Color Palette - Hex and RGB colors codes for web.

Notes
Please note dark blue color in PICO-8 v0.1.6 differs from original one.
Latest (v0.1.6)

Original

Thanks :smiley: :+1:

@felipebueno thank you! :)


test_tpat error

I have build this project with required gcc=7.3.0 and llvm 9.0.1.
my onnxruntime=1.9.0, onnx=1.10.0
when I run the test_tpat.py. I got following error:
Traceback(most recent call last)
file "test_tpad.py" line 3908, in
test_abs()
.............
File "../python/onnx_to_plugin.py", line 98, in onnx2plugin
input_model_path, nodes, plugin_name_dict
file "../python/onnx_to_plugin.py", line 43, in generate_plugin_library
cuda_kernel.run()
file "../python/cuda_kernel.py" in line 69, in run
mod, params, self._target, include_simple_tasks = True, opt_level = op_level
TypError:autoscheduler_get_tunning_tasks() got an unexpected keyword argument 'opt_level'
can anyone help?
thankyou!

it seems that autoscheduler_get_tunning_tasks do not have the default parameter "opt_level".
I browsed all versions of tvm on GitHub, and did not find any version of the corresponding interface "autoscheduler_get_tunning_tasks" to receive the "opt_level" parameter. so what is the official tvm version and what should I do?

on

We have made some changes to the TVM source code. So you should try to use BlazerML-tvm(https://github.com/Tencent/TPAT/tree/main/3rdparty).
We recommend using mirroring directly. you can found the Dockerfile at https://github.com/Tencent/TPAT. The Dockerfile is based on the mirrors of NVIDIA:
FROM nvcr.io/nvidia/tensorflow:20.06-tf1-py3
you can choose the base mirrors in https://catalog.ngc.nvidia.com/containers
Tips: the mirrors must be installed tensorflow and tensorrt.

it seems that autoscheduler_get_tunning_tasks do not have the default parameter "opt_level". I browsed all versions of tvm on GitHub, and did not find any version of the corresponding interface "autoscheduler_get_tunning_tasks" to receive the "opt_level" parameter. so what is the official tvm version and what should I do?

We have made some changes to the TVM source code. So you should try to use BlazerML-tvm(https://github.com/Tencent/TPAT/tree/main/3rdparty).
We recommend using mirroring directly. you can found the Dockerfile at https://github.com/Tencent/TPAT. The Dockerfile is based on the mirrors of NVIDIA:
FROM nvcr.io/nvidia/tensorflow:20.06-tf1-py3
you can choose the base mirrors in https://catalog.ngc.nvidia.com/containers
Tips: the mirrors must be installed tensorflow and tensorrt.

it seems that autoscheduler_get_tunning_tasks do not have the default parameter "opt_level". I browsed all versions of tvm on GitHub, and did not find any version of the corresponding interface "autoscheduler_get_tunning_tasks" to receive the "opt_level" parameter. so what is the official tvm version and what should I do?

We have made some changes to the TVM source code. So you should try to use BlazerML-tvm(https://github.com/Tencent/TPAT/tree/main/3rdparty).
We recommend using Dockerfile directly. you can found the Dockerfile at https://github.com/Tencent/TPAT. The Dockerfile is based on the mirrors of NVIDIA:
FROM nvcr.io/nvidia/tensorflow:20.06-tf1-py3
you can choose the base mirrors in https://catalog.ngc.nvidia.com/containers
Tips: the mirrors must be installed tensorflow and tensorrt.

Thanks for your suggestion, I re-experimented as you suggested, but still got some errors. as follows.
WARNING:tensorflow:From test_tpat.py:33: The name tf.set_random_seed is deprecated. Please use tf.compat.vl.set_random_seed inste ad.
WARNING:tensorflow:From test_tpat.py:36: The name tf.ConfigProto is deprecated. Please use tf.compat.vl.ConfigProto instead.
begin to process
2022-08-11 16:33:11.870908627 [W:onnxruntime:Default, onnxruntime_pybind_state.cc:5O9 CreateExecutionProviderlnstance) Failed to create TensorrtExecutionProvider. Please reference https://onnxruntime.ai/docs/execution-providers/TensorRT-ExecutionProvider.htm l#requirements to ensure all dependencies are met.
2022-08-11 16:33:11.870959029 [W:onnxruntime:Default, onnxruntime_pybind_state.cc:535 CreateExecutionProviderlnstance) Failed to create CUDAExecutionProvider. Please reference https://onnxruntime.ai/docs/reference/execution-providers/CUDA-ExecutionProvider.h tml#requirements to ensure all dependencies are met.
Couldn't find reusable plugin for node abs_O
Start auto-tuning!
2022-08-11 16:33:11.955133673 [W:onnxruntime:Default, onnxruntime_pybind_state.cc:535 CreateExecutionProviderlnstance) Failed to create CUDAExecutionProvider. Please reference https://onnxruntime.ai/docs/reference/execution-providers/CUDA-ExecutionProvider.h tml#requirements to ensure all dependencies are met.
URLError(gaierror(-2, 'Name or service not known'))
Download attempt 0/3 failed, retrying.
URLError(gaierror(-2, 'Name or service not known'))
Download attempt 1/3 failed, retrying.
WARNING:root:Failed to download tophub package for cuda: <urlopen error [Errno -2) Name or service not known>
Compile...
WARNING:auto_scheduler:/tmp/tuning.log does not exist!
WARNING:download:URLError(gaierror(-2, 'Name or service not known'))
Download attempt 0/3 failed, retrying.
WARNING:download:URLError(gaierror(-2, 'Name or service not known'))
Download attempt 1/3 failed, retrying.
WARNING:root:Failed to download tophub package for cuda: <urlopen error [Errno -2) Name or service not known>
Running...
2022-08-11 16:33:13.053279847 LW:onnxruntijiie:DefauLt. onnxruntimejybind_state.cc:535 CreateExecutionProviderlnstancel Failed to create CI.OAExecutionProvider. Please reference https ://onnxruntie.ai/docs/reference/execution-providers/OLA-ExecutionProvider.h tmlsrequirements to ensure all, dependencies are met.
2022-08-11 16:33:13.348876189 [W:onnxruntuie:Default. onnxrunti.e_pybind_state.cc:535 CreateExecutionProviderlnstancel Failed to create CtLAExecutionProvider. Please reference https ://onnxrunti.ne.ai/docs/reference/execution-providers/OJ)A-ExecutionProvider.h tmlsrequirenients to ensure aU dependencies are met.
rm •rf ./lib/tpat_abs_O.so ./obj/*
if ( ! -d ./obj 1: then rkdir -p ./obj; fi
/opt/Lib/cuda-10.2//bin/nvcc -w -std—c++11 .M -MT tpat_abs_0.o -I. .I/opt/lib/cuda10.2//s.ples/common/inc -I/opt/lib/cuda-10.2/ /include -I/opt/l.ib/cuda-11.1/include -I/home/bitbrain/pzhu/third_package/tensorrt/TensorRT-<IP_ADDRESS>/include -I/usr/include -o tp at_abs_0. d s rc/tpat_abs_0 . cu
/opt/1.ib/cuda-10.2//bin/nvcc •w -std=c++11 -I. -I/opt/lib/cuda-10.2//samples/coon/inc -I/opt/Ub/cuda-10.2//include -1/opt/lib? cuda-li .1/include -I/ho.e/bitbrain/pzhu/thirdjackage/tensorrt/TensorRT-7.0.0. 11/include -I/usr/include .Xco.piler -fPIC -arch=sm _75 o tpat_abs_0.o -c src/tpat_abs_0.cu
a /opt/lib/cuda-10.2//bin/nvcc •w -std.c++11 -I. -I/opt/lib/cuda-10.2//samples/coonhinc .I/opt/tib/cuda.10.2//incl.ude -I/opt/li b/cuda-11.1/include -I/home/bitbrain/pzhu/thirdj,ackage/tensorrt/TensorRT-<IP_ADDRESS>/include -I/usr/include -Xca.piler -fPIC -arch= sm_75 -G -lineinfo -o tpat_abs_0.o -c src/tpat_abs_O.cu
g÷÷ -w -std=c÷+fl -shared -o tpat_abs_0.so tpat_abs_0.o -L/opt/lib/cuda-10.2//1ib64 -L/opt/lib/cuda-10.2//1ib64 -L/hooie/bitbrain/ pzhu/thirdjackage/tensorrt/TensorRT-<IP_ADDRESS>/lib -lnvinfer -lcudart -icuda -WL-rpath—/opt/lib/cuda-10.2//1ib64 -Wi. -rpath./opt /lib/cuda10.2//lib64 -WI. .rpath/ho.e/bitbrain/pzhu/thirdpackage/tensorrtfTensorRT-<IP_ADDRESS>/lib
if ( .d ./lib ); then mkdir p ./lib; fi
my *.o ./obj/
my *.d ./obj/
my .so ./lib/
0nnx_ne_mapping_trt_plugin: { ‘abs_O’: tpat_abs_0’)
(TensorRTi EOR: 3: getPluginCreator could not find plugin: tpat_abs_0 version: 1
In node 0 (importFaflbackPluginlmporter): UISIJPCRTED_N00E: Assertion failed: creator && Plugin not found, are the plugin ne. version. and namespace Correct?”
test_tpat.py:278: DeprecationWarning: Use build_serialized_network instead.
engine = builder.buiid_engine(network, builder_config)
[TensorRT] EFOR: 4: [network.cpp::validate::2411J Error Code 4: Internal. Error (Network must have at least one output) LERJ engine is None
PyOLA ER: The context stack was not enpty upon module cleanup.
A context was still active when the context stack was being cleaned up. At this point in our execution. GDA may already have been deinitialized. so there is no way we can finish cleanly. The progral will be aborted now.
Use Context.pop() to avoid this problem.
@buptqq


Adds sleep before running 'odo unlink' to avoid race condition

What type of PR is this?
/kind failing-test
What does this PR do / why we need it:
$subject
We need this since it's become an annoyance in getting CI to pass for PRs.
Which issue(s) this PR fixes:
Fixes #4301
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:
CI should be green. Especially CI shouldn't fail due to #4301.

[ssh:Fedora 32] • Failure [301.546 seconds]
[ssh:Fedora 32] odo devfile supported tests
[ssh:Fedora 32] /home/fedora/amqp_ci_rcv_odo-pr-build_PR_4428/repo/tests/e2escenarios/e2e_devfile_test.go:23
[ssh:Fedora 32]   odo debug support for devfile components
[ssh:Fedora 32]   /home/fedora/amqp_ci_rcv_odo-pr-build_PR_4428/repo/tests/e2escenarios/e2e_devfile_test.go:65
[ssh:Fedora 32]     Verify output debug information for java-openliberty debug works [It]
[ssh:Fedora 32]
[ssh:Fedora 32] [Fail] odo devfile supported tests odo debug support for devfile components [It] Verify output debug information for java-openliberty debug works
[ssh:Fedora 32] /home/fedora/amqp_ci_rcv_odo-pr-build_PR_4428/repo/tests/helper/helper_run.go:34
[ssh:Fedora 32]
[ssh:Fedora 32] Ran 30 of 30 Specs in 836.482 seconds
[ssh:Fedora 32] FAIL! -- 29 Passed | 1 Failed | 0 Pending | 0 Skipped

/retest

[ssh:Fedora 32] • Failure [301.783 seconds]
[ssh:Fedora 32] odo devfile supported tests
[ssh:Fedora 32] /home/fedora/amqp_ci_rcv_odo-pr-build_PR_4428/repo/tests/e2escenarios/e2e_devfile_test.go:23
[ssh:Fedora 32]   odo debug support for devfile components
[ssh:Fedora 32]   /home/fedora/amqp_ci_rcv_odo-pr-build_PR_4428/repo/tests/e2escenarios/e2e_devfile_test.go:65
[ssh:Fedora 32] [Fail] odo devfile supported tests odo debug support for devfile components [It] Verify output debug information for java-openliberty debug works
[ssh:Fedora 32] /home/fedora/amqp_ci_rcv_odo-pr-build_PR_4428/repo/tests/helper/helper_run.go:34
[ssh:Fedora 32]
[ssh:Fedora 32] Ran 30 of 30 Specs in 918.405 seconds
[ssh:Fedora 32] FAIL! -- 29 Passed | 1 Failed | 0 Pending | 0 Skipped

Same error as previous CI run
/retest

The irony. Below failure is the exact thing this PR is trying to fix (https://github.com/openshift/odo/pull/4428#discussion_r578989948):
 [odo]
 ✗  Waiting for component to start [5m] [WARNING x7: Failed]
[odo]  ✗  Failed to start component with name eyvodd. Error: Failed to create the component: error while waiting for deployment rollout: timeout while waiting for eyvodd deployment roll out\nFor more information to help determine the cause of the error, re-run with '-v'.
[odo] See below for a list of failed events that occured more than 5 times during deployment:
[odo]
[odo]  NAME                                      COUNT  REASON  MESSAGE
[odo]
[odo]  eyvodd-7cbd48b669-7zpxw.16651d420a71b7c8  7      Failed  Error: secret
[odo]                                                           "eyvodd-etcdcluster-example"
[odo]                                                           not found
[odo]
[odo]
Deleting project: peswyzkths
Running odo with args [odo project delete peswyzkths -f]

/lgtm

/lgtm cancel

@dharmit: The following test failed, say /retest to rerun all failed tests:
Test name	Commit	Details	Rerun command
ci/prow/v4.6-e2e-4x-psi	416bc6f link /test v4.6-e2e-4x-psi
Full PR test history. Your PR dashboard.

It looks like the problem is still present :-(
I0219 13:45:59.806280 2675524 events.go:52] Warning Event: Count: 6, Reason: Failed, Message: Error: secret "jxqesj-etcdcluster-example" not found
[ssh:Fedora 32] [odo]
I0219 13:46:13.595252 2675524 events.go:52] Warning Event: Count: 7, Reason: Failed, Message: Error: secret "jxqesj-etcdcluster-example" not found
[ssh:Fedora 32] [odo]  ✗  Waiting for component to start [5m] [WARNING x7: Failed]
[ssh:Fedora 32] [odo]  ✗  Failed to start component with name jxqesj. Error: Failed to create the component: error while waiting for deployment rollout: timeout while waiting for jxqesj deployment roll out\nFor more information to help determine the cause of the error, re-run with '-v'.
[ssh:Fedora 32] [odo] See below for a list of failed events that occured more than 5 times during deployment:
[ssh:Fedora 32] [odo]
[ssh:Fedora 32] [odo]  NAME                                     COUNT  REASON  MESSAGE
[ssh:Fedora 32] [odo]
[ssh:Fedora 32] [odo]  jxqesj-c8df4fd78-42k4r.16652a06959602c9  7      Failed  Error: secret
[ssh:Fedora 32] [odo]                                                          "jxqesj-etcdcluster-example"
[ssh:Fedora 32] [odo]                                                          not found
[ssh:Fedora 32] [odo]
[ssh:Fedora 32] [odo]
[ssh:Fedora 32] Deleting project: kmsirtuuwn
[ssh:Fedora 32] Running odo with args [odo project delete kmsirtuuwn -f]

/refresh

/lgtm


Add IP rate limit for activation service

closing because this can be fixed with: https://github.com/threefoldtech/tf_operations/issues/1562#issuecomment-1505041996


Remove cint

Removed CINT declarations

Can one of the admins verify this patch?

The introduced changes remove obsolete CINT declarations, everything seems to work fine. Merging the PR.


Profile - Corresponding pronoun in Spanish is not displayed when switching language

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

Action Performed:

Launch the app and login
Click settings > profile
Change preferred pronouns to Call me by name and save
Click back arrow
In preferences change language to spanish
Click profile ( perfil)

Expected Result:
Corresponding pronouns should be selected / displayed in Spanish
Actual Result:
Displayed is English and pronouns category changed
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.0.94-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

https://user-images.githubusercontent.com/43996225/132258834-bbce4181-bf4c-462a-93f1-d3acce0b6443.mp4
Expensify/Expensify Issue URL:
View all open jobs on GitHub

Triggered auto assignment to @HorusGoul (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

The pronouns field allows you to enter anything, but we offer a few of them so the user can select them more easily instead of having to type them. To know whether the user has selected one of the predefined options, we compare what we have stored in the field and each predefined option. If no one matches, we assume it's custom, and that's because we stored the "Call me by my name" using the English locale, and it doesn't match when we switch to Spanish because it's comparing it with "Llamame por mi nombre".
We could solve this by storing something like __predefined_callMeByMyName and then checking if the value has the prefix __predefined_. If it has it, grab the rest of the string and use it to get its corresponding translated string like this:
const PREDEFINED_PREFIX = '__predefined_';
const isPredefined = pronouns.startsWith(PREDEFINED_PREFIX);

if (isPredefined) {
  const localeKey = pronouns.replace(PREDEFINED_PREFIX, '');
  pronouns = this.props.translate(`pronouns.${localeKey}`);
}

...

What do you think?

Issue reproducible during KI retests.

@Expensify/engineering Could someone take a look at my proposed solution? 😄

Not sure I get what you are proposing, where would that code be?
I think the main problem is that we are saving the copy in the personal details instead of saving a value that represents it. Like if tomorrow we decided to change the copy in english for that option, it would also break for all existing users. So, can we make each option have a value that does not change with the copy nor with the language and save that in the personal details?

This code would be in places where the pronouns are displayed, such as ProfilePage.js.
Yeah, that's what I was referring to, but we also allow arbitrary values if the user chooses the Self-select option. The idea I shared solves that by making the predefined options (example: He/him) have the value of  __predefined_heHimHis, but display to the user He/him or Él depending on the locale selected.
Here's what we have in the en.js file:
https://github.com/Expensify/App/blob/e33fcf7c049b92f3aac2b7c8d91e54b9bed584b8/src/languages/en.js#L418-L425
If we store __predefined_heHimHis and remove the __predefined_ part, we can use the rest (heHimHis) like this to obtain the translated pronoun. On the other hand, if the user has introduced a different pronoun, we'll display it.
To implement this change, we'll need to update the way we create the items to display in the selector, right now it's a simple string list, but the picker also supports this format: { label: "He/him", value: "__predefined_heHimHis"  }. At the moment, we're creating a list this way:
https://github.com/Expensify/App/blob/28ae2f16669d64b1a98ce806b05b0648d4cac106/src/pages/settings/Profile/ProfilePage.js#L77
We would have to change it to be something like:
const pronounsList2 = Object.entries(this.props.translate('pronouns'))
            .map(([key, value]) => ({
                label: value,
                value: `__predefined_${key}`,
            }));

And then adapt the rest of the code to it. We'll always store that value unless __predefined__selfSelect is selected, indicating that we must save the value from the input instead.

Given you are adding label and value, I am not sure why we need the predefined string manipulation. Maybe I don't quite get this because I am not sure what we are saving in the DB exactly in each of the cases, do you know?

The bug consists of the following: We set He/him in English, which will store He/him in the database. Then we change to Spanish, and it will show  Self-select  along with the input field. If this worked correctly, the input would be hidden, and the selector should have the Él option selected.
That happens because when checking to see if the currently selected option is one of the predefined pronouns, we make this comparison: "He/him" === "Èl". That won't return true, and it will assume it's a Self-select one manually introduced by the user.

ok, so yeah, I think the solution of having label (which translates) and value (which remains fixed) is the best course of action. In the DB we would save always the value, which would not change with language nor copy changes.

Issue reproducible during KI retests.

Created a PR for this, will finish it after N6.

Issue reproducible during KI retests.

Issue not reproducible during KI retests.

@HorusGoul Hello! Can you check this thread https://expensify.slack.com/archives/C01GTK53T8Q/p1633613313469600 and verify if the PR fix will also tackle this really similar issue?

Pronouns are not being translated in user details page

Issue reproducible during KI retests.


[chess game] section styling

Describe the Feature you want to end
I would like to enhance section design

Expected behavior
it helps in increasing the user interaction to the site

Screenshots (optional)

Additional context (optional )

@MohitGupta121  Kindly, assign this to me.

No need

No need

I don't understand what do you mean by "no need". I would like to enhance the section using my skills


Migrate the pallet macros of some tests and benches

Signed-off-by: koushiro<EMAIL_ADDRESS>Polkadot address: 15XuanNimo5951s3RjFTPX1AvYVnCwfr3SDhb3AV4fQF3LpK

/tip small

^ @koushiro

@shawntabrizi updated

/tip small

bot merge

/tip small


Allowing injection in UILogicTests via dedicated environment variable.

Add support for UI development in UILogicTests.

Some explanation what are UILogicTests:

If the custom TEST_HOST used to load XCTestBundle, and this custom TEST_HOST is a normal iOS/macOS/tvOS app,
then XCTestCase can instantiate NSWindow / UIWindow, make it key and visible. Such NSWindow / UIWindow works without differences to one created during normal app start. Developer can set UIView / NSView or UIViewController / NSViewController as a contents of this window.

Unless we "pause" XCTest execution, the created custom NSWindow / UIWindow will immediately disappear and deallocate. To "pause" XCTest execution we can use standard XCTestExpectation. If we will set timeout for such expectation, say, 1h, then we can interact with custom NSWindow / UIWindow for the 1h.

As result, XCTest can be used for the "per-screen UI development" purpose without need to launch full app.

When XCTestCase configured to avoid any "pauses" (on CI server), then custom NSWindow / UIWindow used to capture UIView / NSView screenshot in a form of PNG-image. This screenshot used in an image comparison tool which generates report with UI differences between releases/branches/etc.

UILogicTests are very helpful. To make them even more helpful we would like to add injection support (as it was before 131235c89fe16709db169c324a53e30f6ddb71c4) via dedicated environment variable.
Thank you in advance!

This short video demonstrates how injection works inside XCTestCase.
https://github.com/user-attachments/assets/65d47dc5-6ce9-4c33-a504-ea6588757376

Thanks Vlad and for tidying up the code, I'll update the release candidate.


[Feature Request] enable merged cells in heading and subheadings

It is currently not possible to have multiple Rows in headings, like so:

It is possible using code block notation, see #42


Allow to set multiple keys per user

With this PR, a list of public keys can be set per user account. This should help when e.g. rotating SSH keys for a user that automation uses or when a user has multiple machines with different SSH keys. The recommendation is still that a user generally has one SSH key.

Is this ansible role published somewhere? Do we have a version number? If so, this is a breaking change and it should be a major version bump..

It's not published somewhere, so we can go ahead and merge this.


Fix a missing entry for intenseTextStyle in the Profile schema

Fix a missing entry in the JSON schema for intenseTextStyle.
The JSON schema was missing an entry in the Profile section for
intenseTextStyle. I have added it as it appears in AppearanceConfig.
Note that this is currently duplicated in the schema -- however, this is
the pattern used already in Profile as AppearanceConfig entries have
alternate descriptions (and I have updated the description in that
section to make it clear it applies to unfocused terminals).
Longer-term, it likely makes sense to consolidate all entires into
ApperanceConfig and rely on the description for the unfocusedApperance
object/the name of the object to make the limited scope of those keys
clear, so that Profile can simply extend ApperanceConfig and the
duplication in the schema can be reduced.
Validation Steps Performed
Validation with schema verifying tools including VS Code.
Partially addresses #13387

(I changed the wording from resolves to addresses, as the former would cause GitHub to automatically close the issue.)
Thanks for doing this!


Crash when MinIO bucket does not have versioning enabled #545

Fixes #545
Notes for the reviewers:

I assumed the ICreate#configure() method is to proceed to some checks before installing the document-related endpoint, based on this comment, is it OK for you?
As an exception in the configure method would not stop the server, I rather called process.exit(). However I feel like just throwing an exception (and thus disallowing using the /docs endpoint) can be also reasonable.
It may also need some tests. I will write them when the main idea of the implementation is stable (especially when my questions above and this comment are answered :) ).

Thanks in advance for your help :pray:

I'm surprised that throwing an exception in configure wouldn't propagate to mergedServerMain.main and so prevent any server that includes docs endpoints from starting. I would prefer an exception if possible.

I made a change here where the main server is called:
https://github.com/gristlabs/grist-core/pull/546/commits/76df23c7e2088a990746fd601ff8503f3aa550a0
Whether I start using yarn start or yarn start:prod, it seems that the stubs/app/server/server.ts file is executed

Ah, sounds like node is waiting for something to finish up. Maybe mergedServerMain.main needs a catch clause that calls server.close() to shut down anything that is already started, if there is an exception.

Ah, sounds like node is waiting for something to finish up. Maybe mergedServerMain.main needs a catch clause that calls server.close() to shut down anything that is already started, if there is an exception.

I attempted the below patch, with no luck, the process does not exit despite the server being closed:
https://github.com/incubateur-territoires/grist-core/commit/f46bbe4714f22816dfdc3b728b9f3f104d5e6df2

@fflorent let me know if you'd like this landed as is, or if you have ambitions to write a test for it (your call, not essential in this case in my opinion).

OK then, I am OK merging without tests.


[Bug]: The native MySQL POST statement restricts the number of rows affected by SQL (estimated) and cannot take effect

ODC version
odc432
OB version

What happened?

What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?
Set limit on the number of rows affected by SQL (estimated), add data in native MySQL
Anything else we need to know?
No response

pass


Wachter implementation

Hey everyone,
in the paper is mentioned that DiCE is compared to SimpleCF proposed by Wachter, is there the code of generating those Counterfactuals in this repo?
I am afraid I cannot find it in the files
Thanks so much for the help!
Cheers

The method works only for differentiable models like neural networks. It is not included separately, but if you set the diversity loss coefficient to be zero for the tf or pytorch explainers, then you will obtain a method same as theirs.
If you notice any difficulties, let us know.


inject port and env from start script

rebased to latest master

can we use parameter instead of bash argument?
something like npm start --port=3000 --env=development, by default it will be port 3000 and dev environment?

so it must be npm start -- --port=3000 --env=development. does it look weird?

Can we make it like npm start NODE_ENV=development?

possible!

@tungv we will need to change the start script to
if [[ -z "${NODE_ENV}" ]]; then
  export NODE_ENV=${1:-development}
fi

if [[ -z "${PORT}" ]]; then
  export PORT=${2:-3000}
fi

node --harmony build/server.js

so that we can use NODE_ENV=production npm start, then we are good to go.

rebased

lgtm

:+1:


Any chance of using without expo?

Hi there,
is there any chance that this project could be made compatible with non-expo react-native projects?

Hi,
You can use it in a bare react-native project if you have expo modules installed (https://docs.expo.dev/bare/installing-expo-modules/), but I suppose you're saying without expo at all.
Sadly for now it is not planned to make this project compatible outside of expo modules

I suppose as long as I don't have to change other things in the app and can continue working on it like before, using expo like this will be fine. You think it might make sense to refer to this guide in readme for projects using react-native directly?

Normally yes, it shouldn't impact your project to much.
And it is already in the readme, here: https://github.com/pchmn/expo-material3-theme?tab=readme-ov-file#installation-in-bare-react-native-projects 🙂

Oh, I must have missed that :sweat_smile:
But yea, will follow that guide, thanks!


VCPM audit

Are you going to allow for providing less data about what packages we install over to VCPM when we run npm audit and cache certain of the data to return audit results quickly?

Not sure what you're asking here -- like limit the data being sent back to VCPM when you audit?


Multiple fields creation in single request hit

We have some requirement where we need to create list dynamically. We tried to use createFieldAsXml for multiple fields creation with xml schema.
`var fieldXML = "<  Field Type='Text' Required='FALSE' StaticName='myTechName' Name='myTechName' DisplayName='My name to displayed' />";
$pnp.sp.web.lists.getByTitle('My List').fields.createFieldAsXml(fieldXML).then(function(result) {
console.log(result);
});`
This work with single field, but what about multiple fields in single request hit. I tried it with multiple fields taking reference of element file but facing following issue.
{"odata.error":{"code":"-1, System.Xml.XmlException","message":{"lang":"en-US","value":"Unexpected end of file has occurred. The following elements are not closed: Fields. Line 1, position 232."}}}
Please suggest the correct way to add multiple fields while list creation or later.
Note : We have different type of fields to add.

What you've done on creating multiple fields, such as the reproduce steps or script/code?

Thank you Seiyasu for your suggestion, The baching worked for me.
But I think we can enhance operation and add option to create a list with fields.


Adding remaining Virtual Guest CRUD operations

In reference to the work we've been doing to complete the Terraform SoftLayer Provider work that @eirslett started. We've been working on getting all CRUD operations for Virtual Guests implemented as referenced here https://github.com/TheWeatherCompany/softlayer-go/issues/4 and here https://github.com/TheWeatherCompany/terraform/issues/5. This PR includes quite a few additions that fully round out Virtual Guest abilities.

Looks like an issue with github.com/tools/godep

Nice. Can you please update so Travis passes? It might be you need to rebase since I've made some changes yesterday. Once you do and resubmit I will review ASAP. Thanks,
Max

OK seems that github,com/tools/godep was down or something so that caused the failure. I restarted job and all good.
Let me review. Thanks.

Hi, all,
Thanks for your contributions, however, I am not able to merge it because you have no tests!
As you can see we have unit tests for every public methods for any class we have. Additionally, I would feel more comfortable if you also added a new integration test that exercises the new methods. If does not have to be thorough but should at least exercise the main new features.
I am closing this PR and hope you can make the additions I suggest and resubmit. I will then review again.
Thanks and look forward to your updated PR.
Best,
dr.max

@igoonich let's get those tests written and resubmit this ASAP.
Thanks!

@maximilien thanks for the feedback, you have a great project here and I'm glad we can contribute!


cargo rm unexpectedly removes patches

Problem
When I run cargo rm dependency, I expect Cargo to remove the crate dependency from the current crate's [dependencies], and other related necessary changes, like removing dependency from the right-hand side of features that previously enabled an optional dependency on "dependency" or one of its features "dependency/…".
Cargo does these things, but it also wipes out my patch.crates-io section from the workspace's manifest. I could not figure out how this seems related in any way to removing a single dependency, so I think this is a bug.
Steps
# Cargo.toml

[workspace]
members = ["serde", "serde_derive"]

[patch.crates-io]
serde = { path = "serde" }

# serde/Cargo.toml

[package]
name = "serde"
version = "1.0.0"

[dependencies]
serde_derive = { path = "../serde_derive" }

# serde_derive/Cargo.toml

[package]
name = "serde_derive"
version = "1.0.0"

+serde/src/lib.rs and serde_derive/src/lib.rs which can be empty files.
In the serde directory, run:
$ cargo check  # sanity check--the manifests are valid

$ cargo rm serde_derive

Expected diff:
diff --git a/serde/Cargo.toml b/serde/Cargo.toml
index 759de5d..10027d7 100644
--- a/serde/Cargo.toml
+++ b/serde/Cargo.toml
@@ -3,6 +3,3 @@
 [package]
 name = "serde"
 version = "1.0.0"
-
-[dependencies]
-serde_derive = { path = "../serde_derive" }

Actual diff produced by Cargo:
diff --git a/Cargo.toml b/Cargo.toml
index 583dd2f..60ffc4f 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -2,6 +2,3 @@

 [workspace]
 members = ["serde", "serde_derive"]
-
-[patch.crates-io]
-serde = { path = "serde" }
diff --git a/serde/Cargo.toml b/serde/Cargo.toml
index 759de5d..10027d7 100644
--- a/serde/Cargo.toml
+++ b/serde/Cargo.toml
@@ -3,6 +3,3 @@
 [package]
 name = "serde"
 version = "1.0.0"
-
-[dependencies]
-serde_derive = { path = "../serde_derive" }

Possible Solution(s)
No response
Notes
No response
Version
cargo 1.73.0-nightly (7ac9416d8 2023-07-24)
release: 1.73.0-nightly
commit-hash: 7ac9416d82cd4fc5e707c9ec3574d22dff6466e5
commit-date: 2023-07-24
host: x86_64-unknown-linux-gnu
libgit2: 1.6.4 (sys:0.17.2 vendored)
libcurl: 8.1.2-DEV (sys:0.4.63+curl-8.1.2 vendored ssl:OpenSSL/1.1.1u)
ssl: OpenSSL 1.1.1u  30 May 2023
os: Ubuntu 22.04 (jammy) [64-bit]

If I'm reading the reproduction steps correctly, the serde patch is unused.  This is from #11194.

Note the GC process was expanded to clean up not just references to the dependencies just removed, but also references of all dependencies. This was because there's not an easy way to determine which dependencies correspond to the given TOML keys, without either 1) figuring that out before the removal (therefore having to predict the behavior), or 2) returning that information from the remove function (somewhat unorthodox for an op).

Note that this is referencing some of the structure of cargo.  This was implemented as remove and gc_workspace functions which adds some burden for connecting the pieces (but not terribly).
This seemed ok because the chance of this seemed low and people could selecting add/revert the relevant parts.

Updated repro:
# serde/Cargo.toml

[package]
name = "serde"
version = "1.0.0"

[dependencies]
serde_derive = { path = "../serde_derive" }

[dev-dependencies]
serde_json = "=1.0.0"

cargo rm serde_derive still incorrectly removes the patch despite the patch being used.

Thanks for clarifying the reproduction case!
Looks like the bug is that we only check workspace member dependencies for a patch applying.  That is correct for workspace inheritance but not for patches.

I wonder if a simple way of resolving this is to generate the lockfile (resolve_ws) before we run gc_workspace and then we can look to see if the patched result is present in what was resolved.

@epage hey, I'm a first time contributor here. I was wondering if I could claim this issue? Also, if it's ok, are there any glaring gotchas that you see and I should be on the look out for here?

I'd start with a test that demonstrates the bug, making this its own commit. This way when the commit with the fix will show the change in behavior, making it easier for a reviewer to see that this has the intended affect.
https://doc.rust-lang.org/nightly/nightly-rustc/cargo/ops/fn.resolve_ws.html
The start of the code in question is here.
You are also welcome to come to office hours to talk it over.

@epage awesome! thank you for pointing me in the right direction. I'll try working through it.
@rustbot claim

@epage Looks like the source is that the gc_workspace function removes patches where no workspace members have an explicit dependency for the patched crate. This doesn't appear to be the way it should work since the patch can technically override a crate that is a nested dependency of an explicit dependency. Seems like we can't reliably determine if a patch is used until after the workspace has been resolved.
I'm thinking I could break out gc'ing patches to a separate function and run it after resolve_ws using its unused_patches result to determine what to remove. If something is removed, then run resolve_ws again. What do you think?
Additionally, I think there's a bug in unused_patches for path dependencies
https://github.com/rust-lang/cargo/blob/b644a40a1f8406a08bc64305a6a8fd6bb5a34071/src/cargo/core/resolver/resolve.rs#L142
This marks the package unused only if the graph doesn't contain it, but since the patch is a member it'll always be in the graph.

I'm not quite following what the problem is.

Commenting out the gc patch code, Resolve's unused_patches field from resolve_ws isn't being populated with the patch in the failing example even though it's unused. It seems like there a bug in the code which adds the patch to the unused field

If I'm understanding correctly, your proposed fix would mean we never GC patches for workspace members?  I can live with that, at least for now.  This is a minority case (workspace member that is never depended on in the workspace) and people might want to keep that around.
I'd simplify the test case to the minimum number of variable which is just a transitive dependency that is being patched.

I'm proposing the following:

Fix the issue where unused patches are not being added to the resolve properly for path dependencies
Move patch gc logic to a separate function to be executed after resolve_ws
Use Resolve result's unused_patches in gc logic function to automatically unused patches
Re-execute resolve_ws if something was removed in previous step

This'll automatically remove unused patches but leave ones that have an actual nested dependency. In its current form, patches are being removed if it isn't overriding a direct dependency.
I do wonder if we should be automatically removing unused patches without an explicit flag from the user. Maybe a flag could be added that'll enable this? --gc-unused-patches?

Fix the issue where unused patches are not being added to the resolve properly for path dependencies

If I'm understanding correctly, this is unrelated to the current issue and should be handled separately.  Feel free to create an issue and investigate it further (I'd want input from others more involved with the resolver to even decide whether this is a bug or not) but I don't think this should be part of the discussion fro this issue, in the same PR, or investigation of it blocking progress on this issue.

I do wonder if we should be automatically removing unused patches without an explicit flag from the user. Maybe a flag could be added that'll enable this? --gc-unused-patches?

That is a separate discussion as well

Ok, fair enough. I was thinking they should be separate discussions as well. My proposal then is:

Move patch gc logic to a separate function to be executed after resolve_ws
Walk resolve_ws's graph to determine if a patch is used and remove the patch if it isn't
Re-execute resolve_ws if something was removed in previous step


no-default-export: Convert to function

PR checklist

[ ] Addresses an existing issue: #0000
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

Overview of change:
Just a refactor.

There's already an open PR for that rule
Am 06.05.2017 19:35 schrieb "Andy Hanson"<EMAIL_ADDRESS>
PR checklist

Addresses an existing issue: #0000
New feature, bugfix, or enhancement

Includes tests

Documentation update

Overview of change:
Just a refactor.
You can view, comment on, or merge this pull request online at:
https://github.com/palantir/tslint/pull/2698
Commit Summary

no-default-export: Convert to function

File Changes

M src/language/utils.ts
https://github.com/palantir/tslint/pull/2698/files#diff-0 (5)
M src/rules/noDefaultExportRule.ts
https://github.com/palantir/tslint/pull/2698/files#diff-1 (40)

Patch Links:

https://github.com/palantir/tslint/pull/2698.patch
https://github.com/palantir/tslint/pull/2698.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/palantir/tslint/pull/2698, or mute the thread
https://github.com/notifications/unsubscribe-auth/ALaeKC6jEhDoA0JqGFHR_UkDYH_M5WN-ks5r3K9jgaJpZM4NS2BG
.

OK, #2630 looks good.


convert intermediate graph qc to plotly

After further investigation I've come to the following conclusion:
Matplotlib is the better tool for the registration and tensor map figures than plotly, as plotly's favour is when we're showing lots of data which we wish to interact with rather than figures.


solution invisible

The solution of the exercise https://de.serlo.org/209112 with the ID https://de.serlo.org/210136 isn't shown in the old design in the folder https://de.serlo.org/121941

If I click on edit, the whole (very long) solution is visible

ping @kulla

The solution is no visible (also in the old design). I guess it was just a caching issue...


[zuul] Add in base job for testing with zuul

This is a draft/experimenting branch. It will likely not be merged, but will work as a scratchpad for other changes.
Depends-On: https://github.com/infrawatch/smart-gateway-operator/pull/143
For index-deploy scenario:
Depends-On: https://github.com/infrawatch/service-telemetry-operator/pull/470
For local build scenario:
Depends-On: https://github.com/infrawatch/service-telemetry-operator/pull/478

check-rdo

check-rdo
Rechecking to see if failure is intermittent

recheck Zuul was hitting retry_limit on all builds previously

recheck

test

I want to compare the jenkins and zuul jobs to see if the zuul failure is the same reason
The latest zuul build ran smoketests, which failed.
Job output indicates that sg-core didn't build correctly

test

test

test

test

test

test

test

test

recheck

recheck

recheck

retry

recheck

recheck

recheck

This is a big change. Some of the changes within can be separated:

ansible-lint

operator-sdk

logging improvements

use absolute paths instead of relative

This will remain a draft until these changes (and potentially others)  have been merged

recheck

recheck

I'm closing this, since most things I added have been merged. The local_build-index_deploy scenario is the last remaining piece, and that is under review at: https://github.com/infrawatch/service-telemetry-operator/pull/495


How long does it take to run the code?

How long does it take to train on TCFC dataset? Could you tell the experiment setting (GPU)?
I wasn't able to find Appendix file for the paper (https://ojs.aaai.org/index.php/AAAI/article/view/17711).

We use 4 Quadro RTX 6000 GPU with 24G memory.
The training tasks about 10 hours if I remember correctly.

I have uploaded a camera ready version of our paper and appendix in this repo.

Thank you for your prompt response:)


bug: fix makefile to correctly indicate correct solution for missing gcc package

Generated from a discord thread:
Problem

FYI: The suggestion here doesn't actually work:

gcc not installed for ledger support, please install or set LEDGER_ENABLED=false.  Stop.

This is for building celestia-app. @MicahZoltu found that they were "able to install GCC and get around it, but the error message was a lie when it said I could do LEDGER_ENABLED=false to avoid needing GCC."

Matt cleaned up this section of the Makefile on main but it hasn't been backported to v1.x. I verified by checking that "gcc not installed" does not appear on main so in order to close this issue, we likely want to backport https://github.com/celestiaorg/celestia-app/pull/2449

Closed by https://github.com/celestiaorg/celestia-app/pull/2612


Double check light illuminated for transmission

https://github.com/pex-gl/pex-shaders/blob/1f2fbe666ba1f63fb43518e5387425acdad0de1b/shaders/chunks/light-directional.glsl.js#L28-L30
Is it needed, and if yes, for all lights?

Removed with new transmission implementation.


Add error handling

Below is the list of modules that need a output['error_msg'] in case of an exception:

[x] anime
[ ] book
[x] currency
[x] dictionary Line#20
[ ] fact
[ ] joke
[ ] movie
[ ] music
[ ] quote
[ ] time
[ ] wiki

I can take this

sure @jorgeavaldez, let me know if you need any help! :smile:

@jorgeavaldez Are you still working on this, or would it be okay if I worked on a couple modules?

Added pull request for currency (#65) and anime (#66).

@swapagarwal ...I like to work on this..!!

@nikhilgeo This is being done in #74
Feel free to take up any of the other issues. :smile:

ok...Cool..!!


Fixed incorrect object destructuring

The code as it is implemented now (considering the stage-3 Rest/Spread Properties proposal) would type o as {o: {z: boolean} }
I think it's just a typo and it should be just var {x, y, o} in the destructuring declaration. Otherwise the typing + the following explanation should be fixed.

After the docs revamp, I don't know where this page is anymore. cc @samwgoldman who might write new destructuring docs.


Issue #1468 Do not create config for existing cachemanager

Issue #1468 Do not create config for existing cachemanager

Looks good in principle - just visibility and code style remarks

Good to go once rebased


Maintenance: cSpell cleanup

Is your feature request related to a problem? Please describe.
Currently, the code base is scattered with cSpell ignores.
Describe the solution you'd like
Remove cSpell ignores from source files.
Clean up cSpell.json and make sure it's the only place where ignores are added.
Consider enabling cspell for CI builds.

Still need to add it to CI builds


Use different version in pom.xml for main/quarkus-main/camel-main branchs to avoid the conflicts/mess from local maven repo

current we have three (main/quarkus-main/camel-main) branches which use the same version in pom.xml. This can cause conflicts if we need to switch between these branches, because they are not compatible. Ideally, we should use different versions for different branches to avoid such conflict from local maven repo

Is there some documentation about what these three branches are and how they relate?  I haven't found any.

Hi @djencks ,
I can't find the doc also, and here is my understanding.
main is the lastest development branch for camel-quarkus(with released camel and quarkus versions); camel-main uses the latest development main branch of camel(SNAPSHOT came dependency); quarkus-main uses the latest development main branch of quarkus(SNAPSHOT quarkus dependency). camel-main and quarkus-main can help us find issues early from camel and quarkus main branch seperately, hence camel-quarkus can move to next camel or quarkus release more smoothly.
Freeman

Thank you! What a great idea!  I agree on the need for different version names.  These branches are never released, so they are always snapshots, right?  Would appending something to the version keep maven from confusing things, e.g. 2.7.0.camel-SNAPSHOT and 2.7.0.quarkus-SNAPSHOT?

On Dec 21, 2021, at 5:05 AM, Freeman(Yue) Fang @.***> wrote:
Hi @djencks https://github.com/djencks ,
I can't find the doc also, and here is my understanding.
main is the lastest development branch for camel-quarkus(with released camel and quarkus versions); camel-main uses the latest development main branch of camel(SNAPSHOT came dependency); quarkus-main uses the latest development main branch of quarkus(SNAPSHOT quarkus dependency). camel-main and quarkus-main can help us find issues early from camel and quarkus main branch seperately, hence camel-quarkus can move to next camel or quarkus release more smoothly.
Freeman
—
Reply to this email directly, view it on GitHub https://github.com/apache/camel-quarkus/issues/3401#issuecomment-998762453, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAELDXR33X623ACGNTUBCQTUSB3RDANCNFSM5KOVOLPA.
You are receiving this because you were mentioned.

Yes, camel-main and quarkus-main  are kinda experimental branches. And +1  for version like 2.7.0.camel-SNAPSHOT and 2.7.0.quarkus-SNAPSHOT

This would add steps in rebasing main to camel-main and quarkus-main, and would add steps when we want to push camel-main to main, and push quarkus-main to main
I don't see why this causes issues? They both represent the main SNAPSHOT code and are rebased every day from main. The objective is to test the main branch against future versions of Camel and Quarkus. And maintaining those branches, is easy when the nightly rebase from main to those works automatically.
If we change their SNAPSHOT name: Every time we add a new extension, or a new test project, we need to manually rebase to those 2 versions because the version of SNAPSHOTS would not be compatible and couldn't be done automatically
I'd say -1 for today. Maybe as Maven/Github CI experts, you can convince me otherwise

@djencks About what are those branches, it is explained on the CI guide   : https://camel.apache.org/camel-quarkus/next/contributor-guide/ci.html

Thanks @zbendhiba for the clarification!
My original issue is that when switching between main/camel-main/quarkus-main branch, I need to rebuild camel-quarkus project locally to fill my local maven repo, with "-Dquickly" option I can get it done within 5mins now, so this isn't a big deal for me anymore.
Feel free to close this ticket.
Freeman

@ffang I see now what your issue is.

with "-Dquickly" option I can get it done within 5mins now

You may want to try mvnd (mvnd clean install -Dquickly takes under 1 min on my machine) https://github.com/apache/maven-mvnd#how-to-install-mvnd

Thanks @ppalaga !


Fix nil output  name attribute conversion

The attribute described(link below) was being converted incorrectly for certain pages I tried.
https://www.w3schools.com/tags/att_a_name.asp
I created this gist based on the page I was trying to convert and before fix I was getting -
'  ' # => [ ]
'87471742' #> [87471742]

I admit this is really badly written non-html5 doc, but I really need to convert this for a project in production. After this fix, these conversions are fine.
Kindly, merge this fix. Thanks!

Hi @soundasleep, I think we shouldn't convert 'a' tags to markdown like syntax when they have name attributes?
Example
# Expected Behavior
87471742 #> 87471742
# Actual Behavior
87471742 #> [87471742]

The actual behavior works well when the 'a' tag has an href attribute but for name attributes, it shouldn't add the square brackets. What do you think?

I think your proposal is totally fine. If an  tag does not have any href, then it shouldn't display as a link. Can you write this into your commit and add a test case (or two) showing the difference, and get the test to pass on travis-ci? Thank you!


Remove misguided sfadmin organization edit route

In a previous pull request ( #4 ) I added an edit route to for Organizations. I thought that the edit page (/sfbrigade/organizations/<id>/edit) would be where we'd implement this organization edit UI.
After talking with Michael and Deborah, it turns out that the page in the mockup should be at the show route (/sfbrigade/organizations/<id>) for organizations, and that forms should be rendered inline.
This PR removes the edit route. A following PR will start to add inline forms into the show route.

Sweet, thanks for that, when you have an inline form example setup, I'll work on the async part.

LGTM (I don't know Ruby, I agree with the engineering logic explained here)


Fixed #6: Reviewed the explorer.md file.

Codecov Report

Merging #24 (4cc4c6e) into main (eb6e9da) will increase coverage by 0.27%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   95.43%   95.70%   +0.27%
==========================================
  Files          19       19
  Lines         701      699       -2
==========================================
  Hits          669      669
+ Misses         32       30       -2

Impacted Files
Coverage Δ

squids/actions.py
100.00% <100.00%> (+4.16%)
:arrow_up:

squids/tfrecords/explorer.py
96.51% <100.00%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


EditText skips fast key presses

If you try to type too quickly, apparently masked edittext fails to process input in time, becomes inactive and skips input completely, resulting in laggy, slow performance.
Logcat messages when it happens:
W/IInputConnectionWrapper: endBatchEdit on inactive InputConnection

@ildar2 have you resolve this problem?

Я решил в своем вопросе проблему. Нужен был ввод номера телефона и я указал inputtype = number. Поставил на Phone и все стало быстро восприниматься.

Thanks for the report! Yes, there's such a problem! If you want to make a pull-request, I can provilde you all the info, how to set up the project, review your code, and so on. Unfortunately, I have quite many issues to process right now, so if I do it myself it an take a way longer time.


have only api server and move prompt to a cli

Is your feature request related to a problem? Please describe.
Today when the dfs is run as a prompt it cannot be run as a server with API.
Describe the solution you'd like
What we want to do id to seperate bot of them

dfs server , should start the server and expose the API and should be the main program
dfs promt, should connnect to a server and get stuff done through API (eat you own dog food)

Describe alternatives you've considered

None

Additional context

None

Fixed in https://github.com/fairDataSociety/fairOS-dfs/pull/20


[mod_openssl] Allow to selectively disable TLS 1.0, 1.1 and 1.2 versions

Up until now, lighttpd allowed to disable SSLv2 and SSLv3 through the ssl.use-sslv2 and ssl.use-sslv2 lighttpd.conf options, however the same was not offered to selectively disable TLS versions.
On the lighttpd forum, only enabling the TLS 1.2 ciphers is suggested as a mitigation to disable TLS 1.0 (https://redmine.lighttpd.net/boards/2/topics/5797), however it disables TLS 1.1 as well.
This patch adds the option to selectively disable the TLS versions (using OpenSSL's SSL_OP_NO_TLSv1, SSL_OP_NO_TLSv1_1, SSL_OP_NO_TLSv1_2 parameters), without having to change the cipher suite configuration, by adding the ssl.use-tlsv10, ssl.use-tlsv11 and ssl.use-tlsv12 lighttpd.conf options.

Thanks for the patch.  However, I'm looking to combine these directives into a single directive rather than having an ever-increasing number of directives to manage "allowed SSL/TLS versions".  I might have some time to prototype something in Sept, but you're more than welcome to have a go at it.

Hi Glenn,
Thanks for your response. I assume that you won't accept this patch then, is that correct?
Could you elaborate a bit on what you envision for that single directive thingy? I can try to give it a look in the meantime, if it is of any help to you.

Hi Glenn,
I haven't got a response from you to my previous message.
I may have some time to contribute a definitive fix for this issue, if you're not going to accept the one I'm pulling here, but would like to hear more from you on what you expect it to be.

Not sure what @gstrauss has in mind, but maybe something based on SSL_CONF_* (see https://wiki.openssl.org/index.php/Manual:SSL_CONF_cmd(3)).

@lima-j sorry I have not had a chance to get to this yet.  Yes, @stbuehler is on the mark.  Rather than a growing list of directives to address the multitude of options, I'd like to provide some interface to something like https://wiki.openssl.org/index.php/Manual:SSL_CONF_cmd(3)

@lima-j  please have a look at https://github.com/gstrauss/lighttpd1.4/commit/c09acbeb8a030942d9825b3d0dd01c84e0a0b919 for new, experimental ssl.openssl.ssl-conf-cmd directive

@gstrauss it looks good, thanks for letting me know!!


Bypass requested from Google Forms
Response ID: 22
App to request
Al-Bank
App Store link
https://apps.apple.com/us/app/al-bank/id421357586
App version
7.7.0
Jailbreak and iOS version
Unc0ver 14.3
Bypass tried

A-Bypass
Liberty Lite
Hestia
Shadow
HideJB
KernBypass
vnodebypass
Disabling tweak injection with Choicy or libhooker-configurator

Which worked
No one did work
Checklist submission

The app is missing from the table or bypasses are not working anymore
I have checked through all open issues and none matched my app
I have tried at least one bypass before submission.

AL-Bank is already on the list. Closing this issue

Maybe a new update broke the working bypass?

AL-Bank was listed for quite a while with no known bypass

AL-Bank is already on the list. Closing this issue

Maybe a new update broke the working bypass?

AL-Bank was listed for quite a while with no known bypass

Oh :skull:


[Bug] Safari 10.3.3 - Cannot redefine property: flat

Browser: Safari 10.3.3
Os: iOS
Error: Cannot redefine property:flat

Error as described in NewRelic
Cannot redefine property:flat at Function.defineProperty at CreateMethodProperty in https://cdn.polyfill.io/v3/polyfill.min.js Line 3 : Column 880 at anonymous in https://cdn.polyfill.io/v3/polyfill.min.js Line 3 : Column 6263 at anonymous in https://cdn.polyfill.io/v3/polyfill.min.js Line 3 : Column 30143
Our polyfill.io url
https://cdn.polyfill.io/v3/polyfill.min.js?features=default,es2015,es2016,es2017,fetch,Promise.prototype.finally,NodeList.prototype.forEach,NodeList.prototype.%40%40iterator,Array.prototype.flat,IntersectionObserver

Hi @FreshRob, could you also post the full user-agent string please? That helps me verify our user-agent parsing is working correctly, we recently changed it for iOS applications such as SnapChat and Facebook

Working on it, the tool we used to find the error abstracts away the full raw user agent


Multi-Tasks/Quick-Tasks

Sometimes, the items in your Todo are very small, e.g. a shopping list
* bread
* butter
* more bread

In it's current form I'd have to type tb -t @shopping bread, tb -t @shopping butter and tb -t @shopping bread. How about being able to write tb -t @shopping bread|butter|more bread, which could be translated to
* bread
* butter
* more bread

Thanks

I will take a look at getting a PR for this


Plugin fails without CSharpier.MSBuild reference

Environments

IDE Version: 2024.2.1
Extension Version: 1.8.0
CSharpier Version:  0.29.1
Operating System: MacOS
.csharpierrc Settings: None
.editorconfig Settings: None

Log Output
Steps to reproduce

Have csharpier installed globally
Have the csharpier plugin installed in rider
Create a new C# solution
Enable 'Run on Save' in CSharpier plugin in rider
Make changes to the generated Program file

Expected behavior
Changes to be formatted
Actual behavior
No formatting applied, the with the following errors in the log:
2024-08-26 16:08:20,175 [ 294030]   FINE - #c.i.c.CSharpierLogger - Running /usr/local/share/dotnet/dotnet tool list  in /Users/robertrudman/RemuxPruner
2024-08-26 16:08:20,241 [ 294096]   FINE - #c.i.c.CSharpierLogger - Unrecognized command or argument ''

2024-08-26 16:08:20,241 [ 294096]   FINE - #c.i.c.CSharpierLogger - Running 'dotnet tool list' to look for version
2024-08-26 16:08:20,241 [ 294096]   FINE - #c.i.c.CSharpierLogger - Output was:
 null
2024-08-26 16:08:20,242 [ 294097]   FINE - #c.i.c.CSharpierLogger - Ensure there is a csharpier process for /Users/robertrudman/RemuxPruner
2024-08-26 16:08:20,242 [ 294097]   FINE - #c.i.c.CSharpierLogger - Looking for csproj in or above /Users/robertrudman/RemuxPruner that references CSharpier.MsBuild
2024-08-26 16:08:20,242 [ 294097]   FINE - #c.i.c.CSharpierLogger - Looking for /Users/robertrudman/RemuxPruner/*.csproj
2024-08-26 16:08:20,243 [ 294098]   FINE - #c.i.c.CSharpierLogger - Looking for /Users/robertrudman/*.csproj
2024-08-26 16:08:20,243 [ 294098]   FINE - #c.i.c.CSharpierLogger - Looking for /Users/*.csproj
2024-08-26 16:08:20,243 [ 294098]   FINE - #c.i.c.CSharpierLogger - Looking for //*.csproj
2024-08-26 16:08:20,243 [ 294098]   FINE - #c.i.c.CSharpierLogger - Running /usr/local/share/dotnet/dotnet tool list  in /Users/robertrudman/RemuxPruner
2024-08-26 16:08:20,295 [ 294150]   FINE - #c.i.c.CSharpierLogger - Unrecognized command or argument ''

2024-08-26 16:08:20,295 [ 294150]   FINE - #c.i.c.CSharpierLogger - Running 'dotnet tool list' to look for version
2024-08-26 16:08:20,295 [ 294150]   FINE - #c.i.c.CSharpierLogger - Output was:
 null
2024-08-26 16:08:20,299 [ 294154] SEVERE - #c.i.o.e.i.DocumentImpl - CompositeException (2 nested):
------------------------------
[1]: Cannot invoke "String.split(String)" because "output" is null
[2]: Cannot invoke "String.split(String)" because "output" is null
------------------------------

CompositeException (2 nested):
------------------------------
[1]: java.lang.NullPointerException: Cannot invoke "String.split(String)" because "output" is null
	at com.intellij.csharpier.CSharpierProcessProvider.findCSharpierVersionInToolOutput(CSharpierProcessProvider.java:172)
	at com.intellij.csharpier.CSharpierProcessProvider.lambda$getCSharpierVersion$1(CSharpierProcessProvider.java:139)

Adding a reference in the project to CSharpier.MSBuild fixes this. However, this requirement is not mentioned anywhere in the plugin setup: https://plugins.jetbrains.com/plugin/18243-csharpier
This happens both with both locally and globally installed csharpier

I couldn't reproduce this in windows/ubuntu but I'm pretty sure I know what code was causing the problem you are seeing. I submitted 1.8.1 to jetbrains, it may take a day or two for them to approve it.

Just following up to confirm that the change made did in fact fix the issue (tested on 1.8.2). Thanks!


Remove DisplayVersion from Amazon.AWSCLI version 2.18.12

For AWSCLI, the DisplayVersion always differs by .0 at the end from the semantic PackageVersion.
.0 at the end plays no part in package matching and CLI treats both versions the same.
This PR removes DisplayVersion to make it easy for PR authors to author manifest for this package
and to avoid blowing up publishing pipelines in case an incorrect DisplayVersion goes through.

Microsoft Reviewers: Open in CodeFlow

URL: https://awscli.amazonaws.com/AWSCLIV2-2.18.12.msi
OK
(Automated message - build 923)


beta-3 with angular 1.5.6 and higher doesn't bind component resolves

I decided to update angular from 1.5.5 to 1.5.8, but noticed that promises from resolves don't bind the promise result anymore. However, i noticed they are actually present in the $scope.$parent.$resolve if i check the $scope on my component.$onInit.
After i did a rollback to angular 1.5.5 the bindings worked again.
State definition:
$stateProvider.state('dashboard', {
    url: '/dashboard',
    component: 'dashboard',
    resolve: {
       /* Makes sure the widgets for the current user are loaded */
       widgets: ['authenticationService', 'dashboardService', function (authenticationService, dashboardService) {
          return dashboardService.loadWidgetsAsync(authenticationService.user.id);
       }],
       /* Makes sure the app definitions are loaded */
       apps: ['appService', function (appService) {
         return appService.getAppDefinitionsAsync();
       }]
    }
});

Component definition:
angular.module('myApp')
  .component('dashboard', {
    bindings: {
      widgets: '<', // binding for widgets
      apps: '<' // binding for app definitions
    },
    controller: ['$scope', ' DashboardController],
    templateUrl: '/app/views/dashboard/dashboard.html'
});

in angular 1.5 components, you should avoid using $scope.   The resolves should be bound on the this object.  For example:
function DashboardController() {
  console.log(this.widgets)
}

heres a plunker
http://plnkr.co/edit/AeBySYQSfoCCIe6dzVI8?p=preview

Closing because I think the question is answered, and no response.


Compiling on ARM

Hi,
Really interested in your Dovecot search with Elastic!
I am trying to build it on ARMv7 along with dovecot-2.2.18 source code.
./configure --with-dovecot=../dovecot-2.2.18
The configure command above returns the following and I am not sure what does the "dovecot-config" file refers to.
checking for dovecot-config in "/root/dovecot/dovecot-2.2.18"... not found
configure: Use --with-dovecot=DIR to provide the path to the dovecot-config file.
configure: error: dovecot-config not found
Thanks a lot for your help,
Louis

Have you completed dovecot from source before starting the fts compile?
Have you installed dovecot?
---- On Sun, 13 Sep 2015 05:36:04 +1000<EMAIL_ADDRESS>wrote ---- Hi,
Really interested in your Dovecot search with Elastic!
I am trying to build it on ARMv7 along with dovecot-2.2.18 source code.
./configure --with-dovecot=../dovecot-2.2.18
The configure command above returns the following and I am not sure what does the "dovecot-config" file refers to.
checking for dovecot-config in "/root/dovecot/dovecot-2.2.18"... not found
configure: Use --with-dovecot=DIR to provide the path to the dovecot-config file.
configure: error: dovecot-config not found
Thanks a lot for your help,
Louis
—
Reply to this email directly or view it on GitHub.

I had a 2.1 Installed. Just compiled 2.2.18 and managed to ./configure fts-elasticsearch, thanks!
The make did not go through though:
In file included from fts-elasticsearch-plugin.c:8:0:
fts-elasticsearch-plugin.h:5:29: fatal error: fts-api-private.h: No such file or directory
Strange thing is this file is found:
../dovecot-2.2.18/src/lib/module-context.h`` but this one is not: ../dovecot-2.2.18/src/plugins/fts/fts-api-private.h`
Any clue?
Thanks

Can you confirm the -I path's on the gcc calls? The first one should be:
libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I/tmp/dovecot/dovecot-2.2.18//src/plugins/fts
fts-api-private.h exists in src/plugins/fts for Dovecot.

Yes, here are the first calls:
libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../dovecot-2.2.18/src/plugins/fts -I/root/dovecot/dovecot-2.2.18 -I/root/dovecot/dovecot-2.2.18/src/lib (...)
The file ../dovecot-2.2.18/src/plugins/fts/fts-api-private.h exists in the dovecot source directory.

Interesting--that's definitely a bug. Can you do a make clean and then re-run configure with the full path to dovecot-2.2.18?
./configure --with-dovecot=/root/dovecot/dovecot-2.2.18
I'll push a fix for this tomorrow.

make clean and full absolute path made it :+1: Thanks!

ping bug ist still unresolved... but using full path works for me too.

Sorry for that! A fix is up now.


Added support for CMUpdater 2.0 from CM10.

Also, fixed nullptr while calculating deltas.

 Do you need help with implementation of incremental updates?

Thank for the patch, but this is totally anti-pattern and not integrated with the Framework at all. Does somebody really use CM10 still today? There's no more commits by Cyanogen, and it' basically deprecated.
I don't think this would be really useful to many people...
Regarding xdelta3, yes, If you have a better approach that works without any special requirements to create delta ZIPs that would be awesome. Although I read that it's not so easy to be implemented. Therefore, if you are considering doing it, please re-fork this project and start freshly from the last commit. Thank you! :)


[2274] sync courses from un-enriched providers

Context
We originally believed we wouldn't be syncing courses for providers who hadn't added enrichments. We now are.
Changes proposed in this pull request
Don't filter out providers that aren't "publishable" when generating list for bulk sync.
Guidance to review
This is customer-affecting bug, about 270 courses are currently affected by this issue.
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Bulk sync courses even if providers aren't enriched

Interesting that this breaks no tests?

Ya. :-/ I'll have a look at that, but things like this do slip between the cracks at times.

We need to test this locally to see exactly what changes in the bulk sync before we deploy this.

11056 courses are syncable with this:
[3] pry(main)> courses = RecruitmentCycle.syncable_courses
[4] pry(main)> courses.count
=> 11056

From the current list of 299 missing courses, 273 are now in the list of syncable courses that weren't before, leaving 26 courses still not going to be synced.
courses = RecruitmentCycle.syncable_courses
# work/rollover-oct/missing-courses-20191003.tab is a list of all the missing courses from the Google sheet
missing_course_codes = File.read('work/rollover-oct/missing-courses-20191003.tab')
missing_courses = missing_course_codes.split("\n").map { |c| Course.get_by_codes('2020', *(c.split)) }
(missing_courses & syncable_courses).count
=> 273
(missing_courses - syncable_courses).count
=> 26

I think all this means that this fix is good ...


xrdp debian9

systemctl status xrdp
`.service - xrdp daemon
Loaded: loaded (/lib/systemd/system/xrdp.service; enabled; vendor preset: enabled)
Active: active (running) since Fri 2019-11-15 09:53:11 MSK; 14min ago
Docs: man:xrdp(8)
man:xrdp.ini(5)
Process: 999 ExecStopPost=/bin/rm -rf /var/run/xrdp/sockdir (code=exited, status=0/SUCCESS)
Process: 996 ExecStop=/usr/sbin/xrdp $XRDP_OPTIONS --kill (code=exited, status=0/SUCCESS)
Process: 1026 ExecStart=/usr/sbin/xrdp $XRDP_OPTIONS (code=exited, status=0/SUCCESS)
Process: 1016 ExecStartPre=/bin/sh /usr/share/xrdp/socksetup (code=exited, status=0/SUCCESS)
Main PID: 1027 (xrdp)
Tasks: 1 (limit: 4915)
CGroup: /system.slice/xrdp.service
└─1027 /usr/sbin/xrdp
Nov 15 10:07:32 server.domain.com xrdp[1423]: (1423)(140643874998016)[DEBUG] Closed socket 12 (AF_INET6 ::ffff:serverip port 3389)
Nov 15 10:07:32 server.domain.com xrdp[1423]: (1423)(140643874998016)[ERROR] Listening socket is in wrong state, terminating listener
Nov 15 10:07:43 server.domain.com xrdp[1027]: (1027)(140643874998016)[INFO ] A connection received from: ::ffff:myip port 64469
Nov 15 10:07:43 server.domain.com xrdp[1027]: (1027)(140643874998016)[DEBUG] Closed socket 12 (AF_INET6 ::ffff:serverip port 3389)
Nov 15 10:07:43 server.domain.com xrdp[1424]: (1424)(140643874998016)[DEBUG] Closed socket 11 (AF_INET6 :: port 3389)
Nov 15 10:07:43 server.domain.com xrdp[1424]: (1424)(140643874998016)[INFO ] Using default X.509 certificate: /etc/xrdp/cert.pem
Nov 15 10:07:43 server.domain.com xrdp[1424]: (1424)(140643874998016)[INFO ] Using default X.509 key file: /etc/xrdp/key.pem
Nov 15 10:07:43 server.domain.com xrdp[1424]: (1424)(140643874998016)[DEBUG] Security layer: requested 3, selected 0
Nov 15 10:07:43 server.domain.com xrdp[1424]: (1424)(140643874998016)[DEBUG] Closed socket 12 (AF_INET6 ::ffff:serverip port 3389)
Nov 15 10:07:43 server.domain.com xrdp[1424]: (1424)(140643874998016)[ERROR] Listening socket is in wrong state, terminating listener`

I could not get you. You just pasted the log. Speak in words of yours.


Design initial report output

Decide on what to include (perhaps as a list of goals for output).
Sketch what the output should look like.
Keep PDF pagination in mind (compatibility with PDF and notebook-based output).

Precursor to #2

For inspiration: https://github.com/WayScience/Durbin_lab_data/blob/main/2.processing_data/2.single_cell_qc.ipynb

Inspired from Jenna's example, drafted the following (trying to be flexible to one or many features within a threshold set):

After clicking to "unhighlight" non-outliers, one can see the outliers as patterned traces:

I think we'd need one chart per threshold set to make this reproducible, but still thinking through this.

And also, log of the y-axis:


[leafy-health] the health-check class should have an abstract class

the abstract class would provide the unhealthy and healthy methods
class AppCheck < Leafy::Health::AbstractCheck
  def call
    if app.crashed
      unhealthy( 'application crashed' )
    else
      healthy( 'application ok' )
    end
  end
end

I would add the additional criteria in that AbstractCheck define call such that it always responds unhealthy. Not implementing a call method means your service isn't healthy until it's implemented :)

+1


refactor: component SendAddressInput

Acceptance Criteria

Change the component class to component function
Extend validateAddress utility function to receive the network as
second argument
Get the network from redux state

https://github.com/HathorNetwork/hathor-wallet-mobile/assets/5992210/cbcc9e2a-861f-4301-b971-fd61c4d079e3
Security Checklist

[ ] Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

This is actually a fix, not a refactor, right?


[parser] Matrix fields do not adhere to the expected format

You can see here that what's supposed to be each one field of its own, it's interpreted as a single field with a name of the list of variables as it appears in source.
So Matrix should be changed.

@kurinoku I'm aware of that case and it's an exception. No plans to change it.


Need to update yaml files for examples

Examples use old yaml format:
zeebe:
client:
gateway.address: "<IP_ADDRESS>:26500"
security.plaintext: true
Where it appears the newer versions add 'broker' and change gateway.address to gateway-address
zeebe.client.broker.gateway-address=<IP_ADDRESS>:26500
zeebe.client.security.plaintext=true

Thanks for reporting! I adjusted that according to the docs (and also simplified it a little bit - as I found some things confusing).
I was even thinking if we should combine starter/worker into one app - as this is a common misunderstanding that you need one app per worker - so it might be better so simply have one Spring Boot App doing everything. WDYT @superbeagle?


Safety for poisson random calls

Hi everyone,
Base Julia can't convert large floats into Int type using floor or round etc. This creates a problem for sampling from Poisson's with a large mean because  this is used in fast polyalgo for Poisson sampling.
e.g.
using Distributions
log_μ = 48.0 #Plausible value in a log-scale random walk with a Poisson link
d = Poisson(exp(log_μ))
 rand(d)

ERROR: InexactError: trunc(Int64, 7.016735912097631e20)
Stacktrace:
[1] trunc
@ ./float.jl:905 [inlined]
[2] floor
@ ./float.jl:383 [inlined]
[3] PoissonADSampler
@ ~/.julia/packages/Distributions/oGEEM/src/samplers/poisson.jl:54 [inlined]
[4] PoissonADSampler
@ ~/.julia/packages/Distributions/oGEEM/src/samplers/poisson.jl:49 [inlined]
[5] sampler
@ ~/.julia/packages/Distributions/oGEEM/src/univariate/discrete/poisson.jl:144 [inlined]
[6] rand(rng::Random.TaskLocalRNG, d::Poisson{Float64})
@ Distributions ~/.julia/packages/Distributions/oGEEM/src/univariate/discrete/poisson.jl:148
[7] rand(::Poisson{Float64})
@ Distributions ~/.julia/packages/Distributions/oGEEM/src/genericrand.jl:22
[8] top-level scope
@ REPL[18]:1

Now this would not be problematic, because the logpdf calls are not affected but for some reason a rand call comes into using Turing at inference time (not sure why...).
Is there any chance of a safer version of the Poisson/Negative Binomial that can detect if round(BigInt, ... should be used?

Hi @SamuelBrand1, thanks for the issue!
Could you provide a MWE that shows what you're trying to do with Turing that causes this crash? My initial impression is that this seems like an upstream issue with Distributions.

Hey @penelopeysm
Its actually quite hard to construct a MWE here from a clean script, for example,
using Turing, Distributions

@model function rw_model(rw₀, n)
    ϵ_t ~ filldist(Normal(), n)
    rw = rw₀ .+ cumsum(ϵ_t)
    return rw
end

@model function data_model(ys, rw)
    for i in eachindex(rw)
        ys[i] ~ Poisson(exp(rw[i]))
    end
    return ys
end

@model function pois_rw(ys, rw₀)
    n = length(ys)
    @submodel rw = rw_model(rw₀, n)
    @submodel gen_ys = data_model(ys, rw)
    return rw, gen_ys
end

generative_mdl = pois_rw(fill(missing,10), 3.)
ys = generative_mdl()[2] .|> Int
inference_mdl = pois_rw(ys, 48.) #deliberately bad choice for rw_0

chn = sample(inference_mdl, NUTS(), 1000)

Works ok with package versions:

[31c24e10] Distributions v0.25.110
[fce5fe82] Turing v0.33.3

And this is roughly analogous to what @seabbs and myself are doing atm.

This seems to suggest that I should move down towards Distributions, as well as investigating why the large mean Poisson problem is hitting our modelling but not in this minimal script.

Agreeing this does seem to be a Distributions issue so flagging it there makes sense. F2F @SamuelBrand1 and I have discussed that the SciMl ecosystem has had to develop their own faster Poisson (presumably due to issues changing it in Distributions). As this seems like an issue that would be common for users Turing users weighing in on the Distributions issue might be useful (as the rand call being buried in the inference call is quite confusing/hard to resolve).

F2F @SamuelBrand1 and I have discussed that the SciMl ecosystem has had to develop their own faster Poisson (presumably due to issues changing it in Distributions).

That's basically a myth, and I'm not really sure where it comes from. The sampler in SciML and Distributions are actually basically identical and already quite a few years ago I noticed that SciML is not faster: https://github.com/SciML/PoissonRandom.jl/issues/6

Issue in Distributions for the problem here: https://github.com/JuliaStats/Distributions.jl/issues/821
We should hopefully be able to fix it when overhauling rand etc. more generally. But my gut feeling is that it's somewhat suspicious if such large values show up in your model anyway - I guess it's an indication that you might want to rescale your quantities?

Closing in favour of upstream issue

Issue in Distributions for the problem here: JuliaStats/Distributions.jl#821
We should hopefully be able to fix it when overhauling rand etc. more generally. But my gut feeling is that it's somewhat suspicious if such large values show up in your model anyway - I guess it's an indication that you might want to rescale your quantities?

Thanks for the link.
The problem is so much "a model". @seabbs and I are working on a meta-modelling pkg for users who want to build epi models.
There is definitely a big role for educating users on Bayesian methods, e.g. if sampling is bad this usually indicates issues with the model and can be addressed via reparameterisation etc.
However, if your modelling throw Exceptions because of prior choices thats pretty problematic because the mental model here isn't a single power user with a background in probabilistic modelling who is comfortable with reading Exception messages. Its easier to get "something" back and then look at output and adjust priors/reparam. That kind of robustness is a real feature of using stan for example.


add calculator to find perpendicular distance of line form origin

Proposal
add calculator to find perpendicular distance of line form origin
(A clear and concise description of what the proposal is.)

i like to work on it


fix(Dockerfile): add bash required by helper.RunShell

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind tests

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area commands

/area pkg

/area events

What this PR does / why we need it:
To fix the following error when run in a container:
ERRO action error                                  action=helper.RunShell as=httpd error="exec: \"bash\": executable file not found in $PATH"

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

fix(Dockerfile): add bash required by helper.RunShell

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leogr
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 0a563c5122396e165c5f73c12650f9ab16dd0c81


WFLY-6190 additional permissions for SingletonReentrantTestCase

https://issues.jboss.org/browse/WFLY-6190

Seems OK


Create additional info blocks

You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.

I've done lots of things for my hobby to find one I'd stick with, involving several music instruments: drums, bass guitar. I learned English and Chinese language, and I've always loved sports.


Fix some more reviews related with tests, simultaneous sidebars/popups, and popup owners

https://codereview.chromium.org/1168383002/#msg11

Work in progress at https://github.com/ltilve/chromium/compare/41bb8dccfcb9f031...sidebar-action Still some changes to clarify both sidebars and popups at the same time.

All the changes for patchset #8 had been submitted, and replied to the comments at https://codereview.chromium.org/1168383002/#msg13


show ⌘ on Mac in both search button and search input field

closes #1447
@kecnry can you test with this branch?  I don't have a mac

Thanks!  Confirmed that it is now replacing Ctrl with ⌘ in both the search button and overlay input field.


Create Prime Triplets

Given an array A of n integers , find number of triplets (i,j,k) where i<j<k such that there is atleast one prime digit which is common to each number.
Input Format
The first line contains N integers followed by the number of elements
Constraints
N<=10^5
Ai<=10^18.
Output Format
Output the number of triplets
Sample Input
5
21
22
23
24
25
Sample Output
10

Can you assign me this one?

Done

could I work on this?

yes


Update coroutines and AGP

Updates AGP to 4.2.0-alpha15 that resolves the AGP inlining bug and to coroutines 1.4.0 that brings performance improvements.

bors r+


saved_change_to_tag_names? not working…

isn't this creating a tag_names attribute?

There's a bit of explanation around this…
Early versions of Gutentag supported the changed state methods for tag_names, but Rails 4.2 changed the underlying logic behind this - and at the time, it seemed it was only useful for database columns. tag_names is obviously not a database column, so I had to drop support for Rails 4.2 onwards.
Your question prompted me to revisit and see if it was possible again in more recent versions of Rails. As it turns out, it is, and I've just spent a few hours getting it working - including Rails 4.2, and so these features are available in Gutentag for every version of Rails it supports (3.2-5.2).
The commits are in the master branch - if you'd like to give it a shot, that'd be great:
gem "gutentag", "~> 2.1",
  :git    => "https://github.com/pat/gutentag.git",
  :branch => "master"

I'll aim to get a gem release with these changes (likely v2.2.0) out in the next week or so.

v2.2.0 has just been released :)


Extended Clouds

Mod name
Extended Clouds
Curseforge link
https://www.curseforge.com/minecraft/mc-mods/extended-clouds
Other links
https://modrinth.com/mod/extended-clouds
What it does
Makes the cloud distance match your rendering distance instead of hardcored 4 chunks
Why should it be in the modpack
Very vanilla-like, an alternative to custom skies for those who like vanilla aesthetics #72, should not cause conflicts, Bedrock parity maybe? (haven't checked)
Why shouldn't it be in the modpack
Performance needs testers on low-end computers
Could be considered opinionated, though at the same time it feels like something Mojang would do
No toggle/slider/options to customize it
Categories
Graphics optimization, New feature
Additional details
CF API enabled

An alternative to custom skies for those who like vanilla aesthetics

Yes, but still, it doesn't support custom skies.
(GitHub link btw.)

As custom skies/clouds are now supported, I don't think there is a demand for this, especially due to lack of options and compatibility right now.


Improvement proposal to mitigate address poisoning attacks

Describe the bug
Not a bug but I was victim of an address poisoning / dusting attack and I think this could have been mitigated by a simple UI fix
Here is what happened :

sent a test amount of sol to my ledger, transaction went through
went to send a good amount of SOL to my ledger, copied the address from the previous transaction, checked the address, start and end of address were OK.
didn't receive the SOL to my ledger, checked the address again it was the wrong address...
Turns out a SOL account with the same start and end as my ledger address was created and transactions to this wallet were injected in my history.

To Reproduce
Do exactly what I did while talking on the phone so you only have 20% of your brain available for transferring assets.
Expected behavior
My suggestion would be to change the green check icon on the left of each Transfer to an "incoming" or "outgoing" icon.
Users would be more suspicious of an incoming transaction and wouldnt copy and address from it.
This is a very low hanging fruit that could save tons of tokens of SOLs.
Donations welcome to make back my SOLs 🤣 : 25Tdv6UamFdVA7shbuGdCTCzAev1nsPVCKK4coYDVLTB
Smartphone (please complete the following information):
Happened on latest version of iOS wallet

Can't edit the label but definitely not a bug


re!i5

Please use the following bug reporting template to help produce actionable and reproducible issues. Please try to ensure that the reproduction is minimal so that the team can go through more bugs!

A brief description

Expected results

Actual results (with terminal output if applicable)

Your Windows build number

Steps / All commands required to reproduce the error from a brand new installation

Strace of the failing command

Required packages and commands to install

See our contributing instructions for assistance.

Can you please fill out the template when posting?

This looks like an accidental post, closing this out.