build_runner ^1.5.2 is incompatible with less_dart

I'm trying to get less_dart working with the latest angular package, 5.3.1 (migrating from Angular 2 to 4 and now to 5), but no matter how much i move versions around (it's like playing pot-a-mole), less_dart seems to be depending on an old version of build_runner which is not compatible with the latest Angular package.
name: _____
version: 1.0.0
description: _______

environment:
  sdk: ">=2.0.0 <3.0.0"

dependencies:
  angular: ^5.3.1
  angular_router: ^2.0.0-alpha+22
  angular_forms: ^2.1.2
  event_bus: any
  http: any
  js: ^0.6.0
  less_dart: any
  logging: "^0.11.3+1"
  html: any
  html_unescape: "^1.0.1+2"

dev_dependencies:
  angular_test: 2.3.0
  build_runner: ^1.5.2
  build_test: ^0.10.2
  build_web_compilers: ^1.2.0
  test: ^1.0.0

entry_points: ['*.less', '*.html']

The current Dart SDK version is 2.3.1.

Because no versions of build_runner match >1.5.2 <2.0.0 and build_runner 1.5.2 depends on build_config >=0.4.0 <0.4.1, build_runner ^1.5.2 requires build_config >=0.4.0 <0.4.1.
And because less_dart >=2.0.0 depends on build_config ^0.3.1 and less_dart <2.0.0 requires SDK version <2.0.0, build_runner ^1.5.2 is incompatible with less_dart.
So, because _________ depends on both less_dart any and build_runner ^1.5.2, version solving failed.

+1

Pull request: https://github.com/AdalbertoLacruz/less_dart/pull/22

Version 2.3.0 is now in pub. Hope works for you.

it does, thanks Adalberto!


Typographical Error in the README File

The README file contains a typographical error in the "Getting Started" section. Specifically, the word "Th" should be "The" in the sentence: "Th website is a Next.js project using Tailwind for styling and design."

if this issue is open can u assign it to me @Manan21st

Hey @Manan21st I would like to work in this issue. Can you please assign this issue to me?

I would like to work on this issue. Can You please assign this issue to me.


ENH: spectral Epochs / averages objects

We should expand the spectral API to power maps / single trial power etc.
psd_epochs = compute_epochs_psd(epochs, fmin, fmax)
cc @agramfort

closing. Reopen if you want


Update db server host name

Syncing the restored DB that is now powering go.ifrc.org.

It's nice and working.


Fix broken links and remove unneeded whitespace

Fix count.ly and IBM natural language understanding broken links, both links was moved.
Remove unneeded whitespace.
Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Thank you


Remove Implicitly Unwrapped Optionals

This removes all possible Implicitly Unwrapped Optional  types to adopt Swift evolution

@nethraravindran Can you resolve the conflicts please?


White text in Search Bar input, changed width for search and email list

The black text in the search bar input seems out of place, white suits it better in my opinion.
Also, I've changed the margin/width styling for the emails in the inbox list, so they sit in the center of the screen, like Inbox.

Nice! I like the white text.
The centered emails looks good when the sidebar is closed, but when opened it shifts the emails to the right. In google inbox opening the sidebar doesn't shift the emails. Maybe we can replicate that somehow?


fix: missing types for /utils path in package.json

As per the title

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unabled customizing zinnia blog

Actual behavior
When I copy both  zinnia folder from git, my application producing following error:
Invalid block tag: 'zinnia_loop_template', expected 'empty' or 'endfor'
Expected behavior

Blog page should contain my header, footer and added blog post's.

Steps to reproduce the issue

Install django-blog-zinnia==0.15.2 using 'pip' command.
Copy zinnia/base.html from git to existing django project's templates directory.
Extend zinnia/base.html by your project's base.html

Specifications

Zinnia version: 0.15.2
Django version: 1.7.1
Python version: 2.7.6
Operating system: Ubuntu-14.04

Disclaimer
Please read these questions carefully and answer honestly with an x
into all the boxes [ ]:

[X ] I have read the guidelines for contributing linked to above.
[ X] I have checked for duplicate issues.
[ X] This is not a support request.

Hello,
you cannot copy the templates from the git repository, because is too advanced compared to your version v0.15.2.
The zinnia_loop_template templatetag was introduced in v0.16, so yo have at least to adapt the copied templates, or copy the templates directly from your installed version.
Regards


webpack Module parse failed

Version
3.0.0-beta.7 - work fine
but "ol-mapbox-style": "^3.0.0-beta.9" - generate errors

webpack-dev-server --inline --progress --config build/webpack.dev.conf.js

ERROR  Failed to compile with 7 errors13:15:45
error  in<EMAIL_ADDRESS>Module parse failed
: Unexpected token (17:5)
You may need an appropriate loader to handle this file type.
|  */
| class Color {
|     r: number;
|     g: number;
|     b: number;
@ ./node_modules/ol-mapbox-style/index.js 23:0-60
@ ./node_modules/babel-loader/lib!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./src/components/pages/OlMap/OlMap.vue
@ ./src/components/pages/OlMap/OlMap.vue
@ ./src/router/index.js
@ ./src/main.js
@ multi (webpack)-dev-server/client?https://localhost:8080 webpack/hot/dev-server ./src/main.js
and etc. see atachment
rr.txt

This is not an issue with ol-mapbox-style. You'll need to modify your babel and/or webpack configuration to support ES2015 features like class. In addition, the config also needs to strip flow types (which are used by some dependencies).
For inspiration, take a look at https://github.com/boundlessgeo/ol-mapbox-style/blob/bd7581c797ed97165aa0d55a6d5e1aa7e9062b85/.babelrc and https://github.com/boundlessgeo/ol-mapbox-style/blob/b2df90a8d9f03ef0e03c1ea098b4e1beac47cbc4/webpack.config.js.

I got same problem.

Correct. The new expression parser was introduced with v3.0.0-beta.9. Make sure you use e.g. the babel-loader or buble-loader plugin for ol-mapbox-style's dependencies.

how to solve it. if babel version is different.

it's work fine with this webpack config ...

thank you @ahocevar


Updated formatting on Readme

Includes the addition of a settings table which can be expanded upon in the future.  Restructure of install steps and moved the contributions badge higher up to improve visibility.
Can be quickly previewed here.

I agree with your changes, thank you @JohnAkerman


Invasive + Introduced Plants

Consider how invasive and introduced plants may play into shaping the ecoregion floral communities?
More disturbed ecoregions vs less disturbed? Does this effect our generalist vs specialist bee pollinator diversity?

Possible resource: USDA invasive plant list.
Caveat: Questionable if this can address naturalized plants. Also...what truly defines as invasive? Especially when considering large spatial scale analyses?


add test title when import scan result

Hello Team,
could you allow to enter the title of the test when importing a scan result (how you can do when creating a test) ?
best regards,
Sal

Could you try to make a PR for that?


Add finalized datasets to dataset selector panel

[x] Abortion Clinics in TX
[ ] HCSDs in TX
[ ] HCSDs in FL
[ ] HCSDs in MS
[ ] All Endocrinologists in QHPs in FL

All FL datasets pending completion of #241
finalized datasets are based on narratives discussed in this doc: https://docs.google.com/document/d/1JtPVLzul9rSJK1cesKIhLtE0lF3dqoC5YsE_saRx_UU/edit#

all datasets here: https://drive.google.com/drive/u/0/folders/1q3Z_pybHTFzk0v8CBZEEct0MGw0PQfIU

Mechanism is put in place to make this easier, but they still needed to be added.


build: add ability to interactively run guix builds, use container that has unprivileged user

Additional Notes

Not everything can be retained on volumes, /gnu/store and /var/guix are pre-populated when extracting the Guix archive, which is done during the image building process.

Guix also highly cautions against modifying the contents of the store by any means outside manipulating it through the Guix daemon (see https://guix.gnu.org/en/manual/en/guix.html#The-Store)
This is enforced by means of bind-mounting /gnu/store as read-only for anything that isn't the daemon

This container also has the Xcode SDK archive hardcoded in (see guix-start) and will need to be updated if the SDK version is updated
The container runs as privileged process and all operations are being done as the root user (no unprivileged user exists within the container due to its barebones nature) No longer true, as we're not inheriting the Alpine container. An unprivileged user is used for git-specific operations with a configurable UID/GID pair which should be matched with the host user.
This container implicitly trusts https://bordeaux.guix.gnu.org and https://ci.guix.gnu.org (see entrypoint) as it fetches substitutions from it to avoid re-building unpatched/unmodified packages by fetching them from a remote server.

This can be disabled at the expense of significantly longer build times (to note, a lot of build outputs are discarded in between instances of this container as not everything is or can be cached, so this is recommended against)

Motivation
@PastaPastaPasta was interested in the possibility of being able to run Guix builds as it is run by CI interactively with the fewest commands possible. Since my personal devenv is entirely defined using Docker, I had some experience with interactive containers.
We initially tried using the Alpine container (and earlier versions of this PR built upon this container) already available in develop but two problems arose:

build-push-action@v2 is not particularly cooperative with edrevo/dockerfile-plus (which is how we get the INCLUDE+ macro).

All Docker builds are context-dependent (when building a container, you need to define a "base" context from which all relative paths are interpreted), dockerfile-plus doubly so. Your context directory is loaded into the builder and therefore, it is imperative to keep the context as minimal as possible, at the risk of increased resource consumption and longer build times.
build-push-action would not cooperate despite efforts accepting the tradeoff and passing the repository root as the context

Simultaneously, we realized permissions issues were starting to arise with https://github.com/dashpay/dash/pull/5426, after backporting https://github.com/bitcoin/bitcoin/pull/25099, which is an important pre-requisite for subsequent backports that introduce Apple Silicon target support.

The Alpine container proved too barebones and introducing support for it would require changes to it that would future backports. Hence, it was decided to leave it as a vestigial entry while we use our own Docker container.
The container here intentionally does not inherit anything from our base containers to avoid using dockerfile-plus

The context problem still persisted after moving to the Ubuntu-based container (which matches the rest of our CI) but Docker implements something called "additional contexts" that allow you to do things like COPY operations with a different directory as the base.

I rebased https://github.com/dashpay/dash/pull/5426 on top of this PR and it resolved the safe.directory problem

Makes me sad this had to get moved over to ubuntu, instead of alpine, but probably fine. Started my build, let's make sure it works, seems good so far

arm-linux-gnueabihf dash-qt is non-deterministic...
CI had
b94461d685321ea0d6add4f2b07ae3e9da434f69ef4f5be66fecce819cb32d86  guix-build-01c022e6e244/distsrc-01c022e6e244-arm-linux-gnueabihf/src/qt/dash-qt

I have
ca34664ebdcaec0915dc26900131eb2a1581f3e7fef7e116e7e9cc22bdd40b9c  guix-build-01c022e6e244/distsrc-01c022e6e244-arm-linux-gnueabihf/src/qt/dash-qt

previously I had
0078b2550977e6272463409925a02ed852370308641ffca07e055923f3583672 guix-build-d159340395ce/distsrc-d159340395ce-arm-linux-gnueabihf/src/qt/dash-qt

I got 2 binaries: b94461d685321ea0d6add4f2b07ae3e9da434f69ef4f5be66fecce819cb32d86 from my build and ca34664ebdcaec0915dc26900131eb2a1581f3e7fef7e116e7e9cc22bdd40b9c from pasta.
There's difference in disassembled code of function double_conversion::FastFixedDtoa(double, int, double_conversion::Vector<char>, int*, int*)
@kittywhiskers need to backport this one: https://github.com/bitcoin/bitcoin/pull/25643/files
Otherwise builds are non-deterministic now. @PastaPastaPasta be informed

We can probably move forward with this PR then and backport that in the next set of guix backports

I still have couple extra comments that I accidentally put to wrong PR.

can you avoid code duplication between contrib/containers/guix/scripts/guix-check and  .github/workflows/guix-build.yml?
instead hard-coded 32 jobs better to get environment variable:             -e ADDITIONAL_GUIX_COMMON_FLAGS='--max-jobs=32' \ by one of the ways: you can use $(nproc --all)" or "$(grep -c ^processor /proc/cpuinfo)" or "$(getconf _NPROCESSORS_ONLN)"

The reason why we cannot de-duplicate is because both guix-check and guix-build are "run-anywhere" and so they cd to the workspace mount in the script, if not rely on absolute paths altogether. I feel like for an interactive instance which is supposed to be as-few-steps-as-possible (it's why I made those scripts and had a pseudo-motd printed out), we have to account for operator error and think one step ahead.
We could de-duplicate it but that would require the scripts to be modified to be relative, which defeats the point of embedding them into PATH since they won't "run-anywhere", which makes the interactive prompt harder to use.


Can't trigger dag with passed in conf values

I'm trying to trigger a dag and pass in json data in the conf parameter.  The plugin creates the correct cmd to run airflow: airflow trigger_dag --conf '{"test_key": "test_value"}' test_dag2 and if I copy and paste that into the terminal it works.  But it appears  there's a problem with escaping the single quotes in the json, based on the airflow_cmd_split  array produced: ['airflow', 'trigger_dag', '--conf', '\'{"test_key":', '"test_value"}\'', 'test_dag2']
so airflow_cmd_split[3] == \'{"test_key":
and airflow_cmd_split[4] == "test_value"}\'
but I think they should be the same element, possibly even combined with --conf, I'm not sure as I'm not familar with subprocess.Popen().
[2017-06-07 17:18:14,674] [26435] {rest_api_plugin.py:668} INFO - airflow_cmd array: ['airflow', 'trigger_dag', '--conf', '\'{"test_key":', '"test_value"}\'', 'test_dag2']
[2017-06-07 17:18:14,674] [26435] {rest_api_plugin.py:669} INFO - airflow_cmd: airflow trigger_dag --conf '{"test_key": "test_value"}' test_dag2
[2017-06-07 17:18:14,674] [26435] {rest_api_plugin.py:789} INFO - Executing CLI Command: ['airflow', 'trigger_dag', '--conf', '\'{"test_key":', '"test_value"}\'', 'test_dag2']
[2017-06-07 17:18:16,154] [26435] {rest_api_plugin.py:819} INFO - RestAPI Output: {'stderr': "b'usage: airflow [-h]\\n'b'               {list_dags,task_state,render,webserver,clear,task_failed_deps,run,trigger_dag,flower,unpause,worker,version,backfill,variables,kerberos,pool,resetdb,serve_logs,upgradedb,initdb,connections,scheduler,list_tasks,test,dag_state,pause}\\n'b'               ...\\n'b'airflow: error: unrecognized arguments: test_dag2\\n'", 'stdin': '', 'stdout': "b'[2017-06-07 17:18:15,316] {__init__.py:57} INFO - Using executor LocalExecutor\\n'"}

Yeah you're right. In your case above airflow_cmd_split[3] and airflow_cmd_split[4] should be combine so that Popen uses them as a single argument. Its the space in the json string thats causing them to be split into separate elements. For now you can get rid of the space and that will fix the problem temporarily, but thinking about it this would be a problem if you wanted to include a space in one of the values.
I'll see about finding a better solution to address this.

@StephenWetzel I have implemented fix for this issue. Let me know if you test it out and it does not work for you. Also when you test, use it as {"test_key": "test_value"} without any single quotes around it.


Added and exposed ability to unregister grid

When can we expect this to be merged?
I have a problem #187 and need to clear the grids.


name changed

Name will be changed on forum as well.
If we want the URL to change I think we have to update the repo name, so that's been left alone for now.

looks good to me.


To gather memory and detailed timing information

These arguments:

Allow us to gather memory information
Detailed timing from different modules

They are specially useful in the MOM6 coupled model context.
No changes in answers (for uncoupled and coupled models).

This is definitely a @wmputman thing to approve. I wonder if FMS profiling has a cost at 30000+ cores like Bill runs at? Not sure. Maybe @aoloso can do a test at scale to make sure nothing goes wrong?

This is definitely a @wmputman thing to approve. I wonder if FMS profiling has a cost at 30000+ cores like Bill runs at? Not sure. Maybe @aoloso can do a test at scale to make sure nothing goes wrong?

I had an okay from @wmputman (on teams); agree with you that its best he does take a look it here ...
Would be curios to know of how things change from @aoloso's testing. 🙏

Doing a quick c1440 L181 run with
&fms_nml   print_memory_usage=.true.   domains_stack_size = 24000000   clock_grain = 'MODULE'   clock_flags='DETAILED' /

Per @aoloso:

Enabling those FMS profiling features causes a 21,600 core run to crash with what appears to be a stack corruption error (probably at a collective call MPI_Gather not 100% sure yet though)

Thanks @aoloso, @mathomp4. I suppose we could further investigate the cause ourselves and also:

check with someone at the FMS end of things ... @underwoo maybe he would know or someone else there?
also for same reasons, FYI: @marshallward

Until we have resolved this, I will have this PR as a draft.

@sanAkel Note, maybe this isn't an issue in newer FMS? We are still behind the current FMS at the moment, but if someone at FMS can say "Oh yeah, we fixed some of that code in 2021.03", then we might just need to wait until I can move us to FMS-in-Baselibs which is currently at 2021.04 (plus the needed GEOS changes not yet in FMS).

Let's wait for any update from the FMS folks ...

I should mention the crash reported earlier wasn't because of FMS. There was a system problem at that time.

I should mention the crash reported earlier wasn't because of FMS. There was a system problem at that time.

@aoloso would you please add ⬆  to the FMS issue
As for:

There appears to be no slowdown in performance. However, I couldn't see any output where memory use or timings from FMS are recorded. Perhaps I am missing something?

Do we have a call to: mpp_exit() like in our plug for MOM6. It was @tclune's suggestion that we put it in there, so I guess it works for "coupled" only??

@aoloso Thanks for ⏫ updates. So are we all good with these changes??

✅ No crash, high-res runs?
✅ Got diagnostics?

@mathomp4, also please confirm?
Thanks again!

I just ran it in a C48 AMIP run and got a single line extra line out like @aoloso did. So seems good

Thanks! @mathomp4 when @aoloso updates- hope all okay, I will un-draft it.

Reg: https://github.com/GEOS-ESM/GEOSgcm_App/pull/289#issuecomment-1044765374
I assume all okay from @aoloso. Hence un-drafting this PR.


resourceGroupLocation

where  I can find resourceGroupLocation if am not westus ?
I can't find my resourceGroupLocation

There's a list of Azure Regions taht can be found here:
https://azure.microsoft.com/en-us/global-infrastructure/regions/
If you are using the Azure CLI, you can get a list with a dynamically populated list of locations and API Versions supported by each Azure resource provider by running:
az provider list

@waytai does this solve your problem ?

Closing this out, because I believe it's been answered. :)


【3.0】Refactor reference config

What is the purpose of the change

Fix typo
Clarify the relationship between methodmodel, ReferenceConfig, and MethodConfig
Increase the readability of service references

Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8455 (46629cf) into 3.0 (0a07eae) will decrease coverage by 0.18%.
The diff coverage is 61.80%.

@@             Coverage Diff              @@
##                3.0    #8455      +/-   ##
============================================
- Coverage     63.26%   63.08%   -0.19%
+ Complexity      312      309       -3
============================================
  Files          1104     1104
  Lines         46258    46263       +5
  Branches       6977     6957      -20
============================================
- Hits          29267    29184      -83
- Misses        13723    13794      +71
- Partials       3268     3285      +17

Impacted Files
Coverage Δ

...g/apache/dubbo/config/AbstractInterfaceConfig.java
86.75% <0.00%> (ø)

...e/dubbo/registry/integration/RegistryProtocol.java
59.27% <ø> (ø)

...n/java/org/apache/dubbo/config/AbstractConfig.java
81.65% <31.57%> (+1.02%)
:arrow_up:

.../java/org/apache/dubbo/config/ReferenceConfig.java
75.11% <63.73%> (+0.11%)
:arrow_up:

...ain/java/org/apache/dubbo/config/MethodConfig.java
84.31% <65.21%> (-3.57%)
:arrow_down:

...a/org/apache/dubbo/config/ReferenceConfigBase.java
69.49% <100.00%> (ø)

...java/org/apache/dubbo/rpc/model/ConsumerModel.java
55.38% <100.00%> (+5.38%)
:arrow_up:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-20.69%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-15.22%)
:arrow_down:

... and 33 more

Continue to review full report at Codecov.

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


Add logging functionality

If you are familiar with logging libraries we can make use of them in case of errors, some info that would be useful to log is:
Status code  and error message of the api response
Cf-Ray response header(unique identifier of the api request)

Logging done.


Implement power manager overrides per default if chroot is stored on a removable device

Would it be possible to implement the power manager overrides defined in https://github.com/dnschneid/crouton/wiki/Power-manager-overrides per default if the chroot is stored on a removable device by adding the overrides to the enter-chroot shell script?
As far as I know everyone who is using a removable device encounters problems if they dont use the overrides.

It seems to be highly platform-specific; for some, removable devices work fine.  The overrides listed there really break power management as a workaround and can't be added as defaults.


Authorization without login window?

I know it's insecure and everything, but we need it.
I changed the "link" function in Com0x82DropboxSessionProxy.m at the end (last 2 lines) in this way and it seems to work (if I supply the token and the secret, it does silent authorization, if not, it just does what it did before)
Could you please check it and if possible introduce into the main code?
id accesskey = [args objectForKey:@"accesskey"];
id accesssecret = [args objectForKey:@"accesssecret"];
id userid = [args objectForKey:@"userid"];
if (accesskey == nil || accesssecret == nil)
{
    UIViewController *currentVC = [TiApp controller];
    [[DBSession sharedSession] linkFromController:currentVC];
}
else
{
    session = [[DBSession alloc] initWithAppKey:key appSecret:secret root:root];
    session.delegate = self;
    [DBSession setSharedSession:session];
    [session updateAccessToken:accesskey accessTokenSecret:accesssecret forUserId:userid];
}

Since this is basically undocumented functionality, and as you said, is a completely wrong security practice anyway, I'm not really wiling to add the code to this project.


Fixed typo on method in DefaultJSONParser

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


Il distance overlaps

This PR includes logic to calculate a dataset of SL overlaps based on a distance threshold + changes to the former overlap logic that looked at the SL credible sets:

TBC

Codecov Report

Merging #125 (655f072) into main (850026f) will increase coverage by 0.17%.
The diff coverage is 90.32%.

@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   82.27%   82.44%   +0.17%
==========================================
  Files          46       46
  Lines        1286     1310      +24
==========================================
+ Hits         1058     1080      +22
- Misses        228      230       +2

Files Changed
Coverage Δ

src/otg/method/colocalisation.py
100.00% <ø> (ø)

src/otg/overlaps.py
73.33% <50.00%> (ø)

src/otg/dataset/study_locus_overlap.py
84.00% <81.81%> (-2.67%)
:arrow_down:

src/otg/common/schemas.py
100.00% <100.00%> (ø)

src/otg/dataset/study_locus.py
96.60% <100.00%> (+0.21%)
:arrow_up:

This branch is old. I prefer to close it and commit the changes to a newer branch of the project.


[Update Extension] Orion - add ability to search tabs

Description
Adds the ability to search your Orion tabs, activate them, close them, etc.
Since Orion only exposes the tab URL and title (via automation scripting), there are some quirks if you have duplicate tabs open - only the first one will show up, as there's no way to differentiate them. Maybe this will change in the future.
Thanks to Safari extension - I adapted a bunch of code from there XD
Closes #4770
Screencast

Checklist

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

Hi @pernielsentikaer, thanks for testing this.
I'm not sure what's happening there. I did a search for the error, and it seems to relate to devtools/webinspector. Is there anything about your setup that involves that? Maybe remote inspection enabled or something? Any other information you can provide?
I'm struggling to think of how this would relate to AppleScript/JXA.

I checked once more, and all commands work, EXCEPT Search Tabs, which still gives me the same error as above. I have a standard Orion version 😞

Actually, yes, I do @Plonq 😕

Actually, yes, I do @Plonq 😕

Interesting. Can you check that Raycast has appropriate automation permissions? System Settings > Privacy and Security > Automation.

Hi @Plonq 👋
Confirmed that both Safari and Arc is on the list 😢

Yes, they're both on the Raycast list, sorry for the confusion 😊

Yes, they're both on the Raycast list, sorry for the confusion 😊

@pernielsentikaer Sorry I think there's still confusion here. Raycast needs to be on the Privacy & Security > Automation list, in MacOS System Settings. I.e., you need to give Raycast permission to use automation features of the OS, which includes AppleScript/JSA.
The browsers don't need to be on any list.

I found out it's related to language, after switching the system language to English again, it works as expected - do you have a clue what we could change?

I found out it's related to language, after switching the system language to English again, it works as expected - do you have a clue what we could change?

Hmm. This is the first time I've used AppleScript/JXA, so I don't know much about it. But the only thing I can think of that would be affected by language is the name of the app - "Orion", due to this code:
const orion = Application("Orion");

But I am assuming that the names of apps aren't translated (right?).

Let's deal with that in another PR/issue if people is having the issue 😊


Add woodcutting requirement for ironman

Add the 56 woodcutting requirement (splitting artic logs) for ironman players.

Thanks, added it in with an additional IronmanRequirement type I've just created so it won't incorrectly display non-ironman as not having the requirements.


Getting error message when trying to setup Gradle with a project

Expected Behavior
We expect it to launch, however it throws this error.
Current Behavior
Presently there is this error shown.
Context

This issue is preventing the application from running as expected, with the intended features.
Steps to Reproduce

Your Environment

Build scan URL:

This looks like a change made in the Android plugin in AGP 3.5.  Here's a similar report: https://github.com/mannodermaus/android-junit5/issues/167
Running with --stacktrace might reveal more information about where that error is coming from, but it's likely that you'll need to downgrade AGP or upgrade some other plugin to that's compatible with AGP 3.5+


Handle rejected promise so that the submit form does not freeze on error

The current implementation await until submitContext.submit is resolved. However, it is sometimes rejected and the following code to enabled the submit form is never executed in this case and thus the submit form freezes forever.
For example, when a reaction that is already added to the latest message is sent by ASS (by submitting +:something: twice), the slack server returns an error and the submit form will never work thereafter.

I think it is simpler to use catch method, e.g. https://github.com/mazun/ASlack-Stream/blob/master/src/components/slack/list/slacklist.component.ts#L398


Fail fast for log error

Which issues of this PR fixes:

This PR try to fix #321
Change log:

Fail fast for log error
Some refactoring

@lzydmxy please review.

LGTM


Fix typo 'SHFFULE' to 'SHUFFLE'

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Typo

run starrocks_fe_unittest

[FE PR Coverage check]
:heart_eyes: pass : 7 / 8 (87.50%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/planner/DataPartition.java
0
1
00.00%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
7
7
100.00%


429 (Too Many Requests) when trying to get meta data for YouTube videos

Hello.
Firstly, thanks for the great tool!
I'm currently having an issue when trying to receive Youtube meta data to use a link previews on a site I'm working on. For this I use a Node Package named 'link-preview-js', which requires a CORS proxy server.
CORS-anywhere was working just fine initially, prepending the video URLs with the CORS-anywhere demo URL. Then I started to get this 'too many requests' error and so thought perhaps I'd made too many calls (even though I have not made many) and so I cloned the repo and hosted my own CORS-anywhere on Heroku. Strangely enough, I am still getting the error.
It only seems to happen on YouTube videos, as DailyMotion videos give me no error and return the data I need.
Any idea what may be causing this issue?
Many thanks,
Dave

Look at the exact response body of the response. That will tell you whether the issue is with the proxy or the actual response from the server (i.e. YouTube).


Chrome v59 no longer supports embedded credentials in url.

Chrome v59 no longer supports embedded credentials in url (https://www.chromestatus.com/feature/5669008342777856). for example we earlier used to give url with username/password like<EMAIL_ADDRESS>but now v59 onwards we can no longer use url like this in codecept.json under protractor-->url.
so now how to handle windows authentication pop-up for username/password in codceptJS.

Provide console output if related. Use --verbose mode for more details.

# paste output here

Provide test source code if related

// paste test

Details

CodeceptJS version: v0.6.3
NodeJS Version:v6.9.5
Operating System: macOS Sierra V 10.12.5
Protractor || WebDriverIO || Nightmare version (if related) Protractor Version 5.1.2
Configuration file:

# paste suite config here

Hi @manjeetsingh1986
Temporarily I passed following configuration to webdriverio helper and it helped me
 "desiredCapabilities": {
    "chromeOptions": {
      "args": [
        "disable-blink-features=BlockCredentialedSubresources"
      ]
    }
  }

But I don't know how to pass chrome options to protractor through codeceptjs.

Hi @BorisOsipov
Thank you for you suggestion. actually it worked with below configuration 👎
"capabilities": {
"chromeOptions": {
"args": ["disable-blink-features=BlockCredentialedSubresources"]
Thanks a lot for your help.


FIX for #49: Ensuring unassociated  elements are not reloaded unnecessarily.

This is a fix to address issue #49. Helps with ensuring unassociated  elements are not reloaded even though they may already be visible and/or are not loaded lazily.

Can we get this merged in.. this is a fairly important fix for unnecessary video reload issue


Don't fail pipeline init if github token secret exists

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

now, when secret exists:
~/workspace/aws-reinvent-2018-trivia-game(master)$ archer pipeline init
? Would you like to add an environment to your pipeline? Yes
? Which environment would you like to add to your pipeline? david
? Would you like to add an environment to your pipeline? No
? What is your application's GitHub repository? https://github.com/SoManyHs/aws-reinvent-2019-trivia-game
? Please enter your GitHub Personal Access Token for your repository: https://github.com/SoManyHs/aws-reinvent-2019-trivia-game [? for help] ****
✔ Success! Secret already exists for https://github.com/SoManyHs/aws-reinvent-2019-trivia-game! Do nothing.
✔ Success! Wrote the pipeline manifest for https://github.com/SoManyHs/aws-reinvent-2019-trivia-game at 'ecs-project/pipeline.yml'
✔ Success! Wrote the buildspec for the pipeline's build stage at 'ecs-project/buildspec.yml'
The manifest contains configurations for your CodePipeline resources, such as your pipeline stages and build steps.
The buildspec contains the commands to build and push your container images to your ECR repositories.

Recommended follow-up actions:
- Update the build phase of your buildspec to unit test your applications before pushing the images.
- Update your pipeline manifest to add additional stages.
- Run `archer pipeline update` to deploy your pipeline for the repository.

~/workspace/aws-reinvent-2018-trivia-game(master)$ cat ecs-project/pipeline.yml
# This YAML file defines the relationship and deployment ordering of your environments.

# The name of the pipeline
name: pipeline-badgoose-aws-reinvent-2019-trivia-game

# The version of the schema used in this template
version: 1

# This section defines the source artifacts.
source:
  # The name of the provider that is used to store the source artifacts.
  provider: GitHub
  # Additional properties that further specifies the exact location
  # the artifacts should be sourced from. For example, the GitHub provider
  # has the following properties: repository, branch.
  properties:
    access_token_secret: github-token-badgoose-aws-reinvent-2019-trivia-game
    branch: master
    repository: https://github.com/SoManyHs/aws-reinvent-2019-trivia-game

# The deployment section defines the order the pipeline will deploy
# to your environments.
stages:
    -
      # The name of the environment to deploy to.
      name: david

Could there be a scenario where the secret name might already exist, but the value needs to be updated?


implement mat_sort

Need to implement sorting on a specific column. This is specifically for my kdtree implementation but could be useful in other random cases.
At the moment I only care about sorting on a single column, not going to bother with multi-column sorting (at least, not yet).

implemented in c3a241c4b0e1f1fbbdee8f4b0ea46974c01d740f


[wip] inspector improvements

fixes
wip
wip
wip
wip
rebase

Inspection should be collapsed by default.

Also please add an issue detailing the unit tests needed.

@ry PTAL

@ry ptal


Bintrie

Opening for comparison.

@ChosunOne is this ready for review or are you still working on this?

Still working on it.  Note the conspicuous lack of passing tests

No pressure or anything, just don't want to look at it until you give the go-ahead. Take your time with it.

@chosunone are you too busy to work on this; shall I assign it to someone else?

@jamesray1 I'm working on the same thing with go-ethereum.  Let me know if you're still interested in this.

Yes, @tcsiwula has started work on this. I'm working on sharding p2p with gossipsub.

Forgot to mention this, but my go implementation of a binary trie might be of interest.  While proofs ended up being about half the size of radix 16 proofs, this is still far from optimal as mentioned here.  I'm interested in optimizing this further and would appreciate feedback.

Hi Eli,
Thanks for your message. At the moment I'm working on gossipsub, but once that's done I will work on other things like the bintrie and implementing shasper once the spec is finalized. Perhaps @paulhauner may have time to work on a bintrie.


Add label support to ssh key creation

HetznerCloudAPI#getSSHKeyByName(String) is currently broken as well as HetznerCloudAPI#getSSHKeyByFingerprint(String). Will release a fix before merging this as the test will break otherwise.

Thank you for your contribution! Feature is now available with 2.15.1.


TEMPLATE END ERROR: BUNDLE INSTALL FAILS

When i run the following command mentioned below it ends with an unexpected error. If any one find solution to this or having some clue about this error please let me know.
Thanks for suggestions and solutions in advance.
#bundle install
ERROR:
Fetching gem metadata from https://rubygems.org/.............
--- ERROR REPORT TEMPLATE -------------------------------------------------------
Error Report
Questions
Please fill out answers to these questions, it'll help us figure out
why things are going wrong.

What did you do?
I ran the command /usr/local/bin/bundle install

What did you expect to happen?
I expected Bundler to...

What happened instead?
Instead, what happened was...

Have you tried any solutions posted on similar issues in our issue tracker, stack overflow, or google?
I tried...

Have you read our issues document, https://github.com/bundler/bundler/blob/master/doc/contributing/ISSUES.md?
...

Backtrace
Errno::ENOENT: No such file or directory - git
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:166:in ``'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:166:in `block (2 levels) in git'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:244:in `block in capture_and_filter_stderr'
  /usr/share/ruby/tempfile.rb:324:in `open'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:242:in `capture_and_filter_stderr'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:166:in `block in git'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/shared_helpers.rb:83:in `call'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/shared_helpers.rb:83:in `with_clean_git_env'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:165:in `git'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:176:in `block in has_revision_cached?'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/shared_helpers.rb:65:in `chdir'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/shared_helpers.rb:65:in `block in chdir'
  /usr/share/ruby/monitor.rb:211:in `mon_synchronize'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/shared_helpers.rb:64:in `chdir'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:225:in `in_path'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:176:in `has_revision_cached?'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git/git_proxy.rb:94:in `checkout'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git.rb:302:in `fetch'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/source/git.rb:160:in `specs'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/lazy_specification.rb:76:in `__materialize__'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/spec_set.rb:85:in `block in materialize'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/spec_set.rb:82:in `map!'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/spec_set.rb:82:in `materialize'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/definition.rb:170:in `specs'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/definition.rb:158:in `resolve_remotely!'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/installer.rb:279:in `resolve_if_needed'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/installer.rb:83:in `block in run'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/process_lock.rb:12:in `block in lock'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/process_lock.rb:9:in `open'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/process_lock.rb:9:in `lock'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/installer.rb:72:in `run'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/installer.rb:25:in `install'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/cli/install.rb:65:in `run'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/cli.rb:224:in `block in install'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/settings.rb:136:in `temporary'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/cli.rb:223:in `install'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/vendor/thor/lib/thor.rb:387:in `dispatch'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/cli.rb:27:in `dispatch'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/vendor/thor/lib/thor/base.rb:466:in `start'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/cli.rb:18:in `start'
  /usr/local/share/gems/gems/bundler-1.16.0/exe/bundle:30:in `block in <top (required)>'
  /usr/local/share/gems/gems/bundler-1.16.0/lib/bundler/friendly_errors.rb:122:in `with_friendly_errors'
  /usr/local/share/gems/gems/bundler-1.16.0/exe/bundle:22:in `<top (required)>'
  /usr/local/bin/bundle:23:in `load'
  /usr/local/bin/bundle:23:in `<main>'

Environment
Bundler       1.16.0
  Platforms   ruby, x86_64-linux
Ruby          2.0.0p648 (2015-12-16 revision 0) [x86_64-linux]
  Full Path   /usr/bin/ruby
  Config Dir  /etc
RubyGems      <IP_ADDRESS>
  Gem Home    /home/test/.gem/ruby
  Gem Path    /home/test/.gem/ruby:/usr/share/gems:/usr/local/share/gems
  User Path   /home/test/.gem/ruby
  Bin Dir     /home/test/bin
OpenSSL
  Compiled    OpenSSL 1.0.1e 11 Feb 2013
  Loaded      OpenSSL 1.0.1e-fips 11 Feb 2013
  Cert File   /etc/pki/tls/cert.pem
  Cert Dir    /etc/pki/tls/certs
Tools
  Git         not installed
  RVM         not installed
  rbenv       not installed
  chruby      not installed

Bundler Build Metadata
Built At          2017-10-31
Git SHA           10f20fa33
Released Version  true

Gemfile
Gemfile
source 'https://rubygems.org'

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '4.2.4'
# Use SCSS for stylesheets
gem 'sass-rails', '~> 5.0'
# Use Uglifier as compressor for JavaScript assets
gem 'uglifier', '>= 1.3.0'
# Use CoffeeScript for .coffee assets and views
gem 'coffee-rails', '~> 4.1.0'
# See https://github.com/rails/execjs#readme for more supported runtimes
gem 'therubyracer', platforms: :ruby

# Use jquery as the JavaScript library
gem 'jquery-rails'
# Turbolinks makes following links in your web application faster. Read more: https://github.com/rails/turbolinks
gem 'turbolinks'
# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.0'
# bundle exec rake doc:rails generates the API under doc/api.
gem 'sdoc', '~> 0.4.0', group: :doc

gem 'pg'
gem 'devise'
gem 'activeadmin', git: 'https://github.com/activeadmin/activeadmin.git'
gem 'rollbar', '~> 2.14.0'
gem 'gcm'
gem 'newrelic_rpm'

# Use ActiveModel has_secure_password
# gem 'bcrypt', '~> 3.1.7'

# Use Unicorn as the app server
# gem 'unicorn'

# Use Capistrano for deployment
# gem 'capistrano-rails', group: :development

group :development, :test do
  gem 'byebug'
  gem 'rspec-rails'
  gem 'factory_girl_rails'
  gem 'shoulda-matchers'
end

group :test do
  gem "simplecov"
  gem "codeclimate-test-reporter", "~> 1.0.0"
end

group :development do
  # Access an IRB console on exception pages or by using <%= console %> in views
  gem 'web-console', '~> 2.0'

  # Spring speeds up development by keeping your application running in the background.

You have a git gem but don't appear to have git installed or available in the ruby environment.

I'm closing this for now. If you're still experiencing your original issue don't be afraid to re-open this ticket.


Darkmode fixed

It bugs in small screens. You should put another background color for the dark mode, not make it invisible.


Improvement for page /id/check

cannot be installed

Hi @Glamoraaa, can you please explain which browser you are trying to update or install, and which platform (i.e. Android, Windows, etc.) you are using?


Add custom view settings

I'd like to start implenting custom view settings.  A JSON column should be added to user_preferences that contains view settings handled by the app.

One option to test with would be changing the order of notes.


”Gara Docăneasa”, ”Gara Tălășman”, ”Obârșeni”, ”Valea Lungă” are listed in the drop-down menu

Description:
When user writes ”Gara Docăneasa”, ”Gara Tălășman”, ”Obârșeni”, ”Valea Lungă” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Gara Docăneasa"

Expected results
The location appears in the drop-down menu as "Sat Gara Docăneasa, VASLUI (Vinderei)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Gara Tălășman"

Expected results
The location appears in the drop-down menu as "Sat Gara Tălășman, VASLUI (Vinderei)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Obârșeni"

Expected results
The location appears in the drop-down menu as "Sat Obârșeni, VASLUI (Vinderei)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Valea Lungă"

Expected results
The location appears in the drop-down menu as "Sat Valea Lungă, VASLUI (Vinderei)".


Update windows-server-2016-improvements.md

added warning about STS

Cancelling this PR. Product Group doesn't like the update.

Cancelling this PR. Product Group doesn't like the update.

Hi @ryanriesMSFT, I think my team is the product group. We like the update. Why is the PR closed? We really do want the link in the UtilizeSSLTimeData section. What is the problem?


Documentation: Where to create a new ember app in a Rails app

@rwz  Where do you suggest a newbie (in this case, me) can create an ember app (with ember-cli) for a Rails project?
That info would be useful for newbies, and could perfectly fit inside the project's README file.
As soon as I get an answer I can send you a PR with an updated README :)

Well, what do you mean "where"? Do you mean where in your Rails app the EmberCLI app should be? The suggested path would be app/frontend given your app is called "frontend".


Update documo-tutorial.md

remove just in time provisioning support, since we don't support it.

@djllap : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add tip for console apps missing logs

Could we consider adding a note under https://docs.microsoft.com/en-us/azure/azure-monitor/app/asp-net-troubleshoot-no-data#no-data-from-my-server regarding console apps? We had a heck of a time trying to figure why App Insights worked from visual studio but not when run on a server. The Flush must be built into visual studio or something. Definitly a gotcha that we spent a lot of time chasing and adding it here could help others
https://docs.microsoft.com/en-us/azure/azure-monitor/app/worker-service
                // Explicitly call Flush() followed by sleep is required in Console Apps.
                // This is to ensure that even if application terminates, telemetry is sent to the back-end.
                telemetryClient.Flush();
                Task.Delay(5000).Wait();

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

ID: 59b9aed0-b6a7-d3e9-d616-85e865966444
Version Independent ID: df2e545f-7f65-82b7-c793-2d04d91d7d9e
Content: Troubleshooting no data - Application Insights for .NET - Azure Monitor
Content Source: articles/azure-monitor/app/asp-net-troubleshoot-no-data.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@harre096 Our SME will review your feedback, thanks for reaching out on this issue!


Issue with evaluate_generator() and datagen_flow()

Hello all,
I am having a problem when using the evaluate_generator() method or when using datagen.flow().
Example:
for X_batch, Y_batch in datagen.flow(X_train, Y_train,batch_size=128):                   model.train_on_batch(X_batch, Y_batch)
model(dataAugmentation.flow(X_val,Y_val,batch_size=128),len(X_val))
The error I am getting is:
UnboundLocalError: local variable 'index_array' referenced before assignment.
Any ideas?
Thank you.
EM

Do you call datagen.fit(X_train) before datagen.flow? Could you provide more details on your error message (e.g. file, line number)?

If you add the line     model.evaluate_generator(datagen.flow(X_test, Y_test,                         batch_size=batch_size),val_samples=len(X_test)) to the cifar10_cnn.py example, the error is generated.

@EnriqueSMarquez My Keras is in version '1.0.0' with TensorFlow backend. Adding that line to the cifar10_cnn.py doesn't produce any error in my case. Maybe you can check your version and update to the latest one?

I have updated it. I am running with theano as backend. My Keras version is 1.0.1 and my then version is 0.9.0.
Thank you.


Add validating max nested document/array depth

Description
Closes #1639.

Add validating max nested document/array depth to align it with MongoDB requirement.
Readiness checklist

[x] I added/updated unit tests.
[x] I added/updated integration/compatibility tests.
[x] I added/updated comments and checked rendering.
[x] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[x] (for maintainers only) I set Reviewers (@FerretDB/core), Labels, Project and project's Sprint fields.
[x] I marked all done items in this checklist.

Hi,
Sorry for not responding - I made  the reaction to the last comment to let you know I do remember.
I could not continue, because I got quite busy at work recently. I wanted to go on with it around now.
I saw some updates on this PR + original issue from @noisersup .
Please let me know if there is anything what I can make here or to take another issue :blush:

@rumyantseva please resolve your conversations

@noisersup that PR requires your approval 😁


Config for use with a Mixer type MIDI controller such as the KORG nanoKONTROL Studio

Hi,
I'm to stumble on your project, especially seeing the owner being a fellow piano player :).
I want to say right from the start, I'm not a programmer, so I won't be able to do any development type of work needed for this. The  readme still feels lacking to someone with my knowledge, so I decided to try my luck with a request. I hope you like my idea and that it's a reasonable ask for you to create it:).
The reason I was looking to control my lights with my nanokontrol is, because I happen to have one anyway, and it would be so awesome to control my room lights with the faders and buttons.
In the past, with the help of a youtube video, I was able to run a python script using phue, with the press of a key, but combining that with MIDI capabilities I feel is beyond my level for now.

@kutibotond Hi Botond, thanks for reaching out. Were you able to get the program running with a keyboard? I don't have a nanokontrol so it will be difficult for me to develop the feature you're requesting.


Access application with a map and sidebar list

As an unregistered user I want to access the application, so that I can see a map with places on the right side and a list with places on the left side of the window.
Mockups:

Acceptance Criteria:

WHEN Unregistered user access the site, THEN system should navigate him/her to the main page of website with:
a. Map zoomed out to area of Lviv
b. List of places (Names)

Test places are on the map

System allows to zoom in/out a map.

Sidebar list of cafes include names of test cafes and their address.

List of places should be sorted by name.

Test cafes, which are in the list of sidebar should be visible on the map in parallel.

TBD:

Research best map to integrate with a website??

As an unregistered user I want to access the application, so that I can see a map with cafes (TBD including maps integration)
Mockups:
Acceptance Criteria:
1.
Assumptions and Constraints:
TBD:

Not relevant.


Add test cases for the logging.go file

Description
Add test cases for the Logging Initialisation code which parses the logging configuration in core.yaml
Motivation and Context
There were previously no tests for parsing the config in core.yaml. In the process of looking at issue #802 I decided it would be easier to fiddle with if there were tests in place.
How Has This Been Tested?
This pull request consists of tests.
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by: Julian Carrivick<EMAIL_ADDRESS>

For me, it seems to be OK. Travis/Jenkins errors seem to be totally unrelated. (protoc not found, peer_basic.feature:1267 fails)

reverify jenkins

Travis is rerunning now. Will investigate if the failure shows up again

@juliancarrivick-ibm Shall I go ahead and merge this PR or do you plan to make changes based on @christo4ferris's comments? Seems those can be made in a second PR if that works better for you.

I could live with follow-on PR.

@srderson Go ahead and merge it!
Depending on how I go with issue #802 I might create another pull request for those extra tests.

LGTM


can't build in macos

========================================
CreateMSBuildFolder
========================================
Copying Mono MSBuild runtime...
Copying MSBuild libraries...
Copying MSBuild SDK resolver...
An error occurred when executing task 'CreateMSBuildFolder'.

----------------------------------------
Teardown
----------------------------------------
Shutting down MSBuild server...
Shutting down VB/C# compiler server...
MSBuild server shut down successfully.
VB/C# compiler server shut down successfully.
Error: One or more errors occurred. (/Users/wellbye/repos/lib/omnisharp-roslyn/tools/Microsoft.DotNet.MSBuildSdkResolver/lib/netstandard2.0/Microsoft.DotNet.MSBuildSdkResolver.dll does not exist)
        /Users/wellbye/repos/lib/omnisharp-roslyn/tools/Microsoft.DotNet.MSBuildSdkResolver/lib/netstandard2.0/Microsoft.DotNet.MSBuildSdkResolver.dll does not exist

Microsoft.DotNet.MSBuildSdkResolver doesn't exist in tools on macos, but it does on windows.

can you please provide the full build log? also, how did you invoke the build script? please try ./build.sh --configuration Release --verbosity Verbose --target All --publish-all --archive and please ensure you have Mono 5.18+ installed.
each PR and each release is built on macOS too so it definitely works.

I'm sorry it's my problem.
maybe  a accidently wrong change to the tools/packages.config:

I don't know which util modifies it(looks like a auto upgrade)
after resetting it , the build goes correctly.

thanks I will close it then 👍


word double-click selection in 1.54.0a

After upgrade to 1.54.0 double clicking on a code (Javascript) word do not select the whole word but stops at first symbol ($)

I'm experiencing the same issue with PHP and double clicking $variable only select the word variable.

I'm experiencing the same issue with PHP and double clicking $variable only select the word variable.

Thanks for reporting this,
I can also reproduce this on atom 1.54 on Mac OS

Thanks for reporting this,
I can also reproduce this on atom 1.54 on Mac OS

We should be able to configure the word boundaries (e.g. I want _ to be a non-word for python snake_case)

We should be able to configure the word boundaries (e.g. I want _ to be a non-word for python snake_case)

On second thought, why not make it a per-language option?

On second thought, why not make it a per-language option?

@ThatXliner, agree... it would be nice to have more control over copying. Right now, it sucks being in PHP and not getting the $, however, there are some times when I just want the text.

@ThatXliner, agree... it would be nice to have more control over copying. Right now, it sucks being in PHP and not getting the $, however, there are some times when I just want the text.

I think we would need to some how dynamically change the regex here
https://github.com/atom/atom/blob/2f6f41b96e8dc4a1b4eac4647e67de5b85b89bcf/src/cursor.js#L731
depending on the grammar

Any movement on this? I've had to freeze my Atom version at 1.53.0.


Phantom Python Introduction

When I click on pandas https://datascienceinpractice.github.io/assignments/D2_Pandas.html  there is no "Introduction to Python" in Assignments in the table of contents on the left.
When I click on Data Analysis https://datascienceinpractice.github.io/assignments/D5_DataAnalysis.html  it suddenly appears.
This issue was raised as part of the JOSE review https://github.com/openjournals/jose-reviews/issues/121

hey @krother - thanks for the note on this - I was able to replicate that on the previous deployment, there was some kind of quirk with the table of contents for the assignments, and in particular that Python Introduction would often disappear.
I have updated Jupyter-book, and redeployed, and no longer see this issue. It seems to be fixed with the re-deployment, so I'm going to close this issue - but please re-open if it doesn't seem to be fixed for you!


Update StateFlow blog

Why are these changes needed?
Update the StateFlow blog to reflect the changes to the paper.
Minor: minor correction to autodefense blog.

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.08%. Comparing base (6c3d779) to head (9156626).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2262      +/-   ##
==========================================
- Coverage   37.94%   36.08%   -1.87%
==========================================
  Files          77       77
  Lines        7784     7784
  Branches     1667     1667
==========================================
- Hits         2954     2809     -145
- Misses       4580     4735     +155
+ Partials      250      240      -10

Flag
Coverage Δ

unittests
36.08% <ø> (-1.85%)
:arrow_down:

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

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


fix retain cycle

@yo1995 I'd like to get this merged even though this code will be changed in a future PR. Because I'd like to have the bug fixed in case we have to revert the other PR for some reason. This doesn't fix all known issues with the code that we discovered, but it fixes the memory leak, which I think would be good to do.


Script is no longer working

We get the "No add blockers allowed" message for a moment and then a blank screen.

Same thing for me using Opera.

We get the "No add blockers allowed" message for a moment and then a blank screen.

I'm using the addon successfully on Chrome right now.
The only issue I've seen in the last few days is that it doesn't auto-skip the panel with the "skip ad" button on it. And all that means is waiting 5 seconds for the timeout to occur.
Ads are still being skipped/blocked.
Are you positive that you don't have anything else blocking the ad?
What you're describing is exactly what happened to me when I first installed RemoveAdblockThing but still had uBlock enabled on YouTube.
uBlock blocking the ad and causing the black screen made it seem like RemoveAdblockThing wasn't working.

The only issue I've seen in the last few days is that it doesn't auto-skip the panel with the "skip ad" button on it. And all that means is waiting 5 seconds for the timeout to occur.

If you're seeing the "skip ad" button then this script isn't working. What's the point if ads are still playing?

We get the "No add blockers allowed" message for a moment and then a blank screen.

I use Firefox + uBlock Origin. When I get this issue, I just go to uBlock dashboard, then to 'filter lists', purge all caches, wait for it to clear then restart my browser. It's working again.

Yes, going to the uBlock dashboard, and then to 'filter lists' and purging all the caches worked for me.

Bump, this is still happening for https://getadblock.com on a Chrome browser.

Same thing for me using Opera.

With Opera, you will have to disable or whitelist youtube with the ad blocker built into opera. My laptop gave me issues and disabling opera's adblock fixed it. my desktop worked without that for some reason, which is odd, but win 11 (desktop) and win 10 (laptop) could be to blame


Will Tenant access be added?

Hi ,
Thank you for the great plugin.  Will Tenant functionality be supported on the roadmap?

Hi @gil0109 ,
it's not immediately on the roadmap. But I've already thought about it.
Maybe a simple functionality using a selected small number of groups to identify the tenant is sufficient for now. It is clear that multi-tenancy would be a clear advantage in larger distributed scenarios. Also to manage access rights e.g. in the cockpit much more effectively.
I can't promise you a timeline, but I will think about a possibility to realize it.

Would be great to have top-level groups as tenants.

@VonDerBeck Any updates on this? I'd be happy to contribute a PR towards this but probably need some guidance on what your thought process is...

@VonDerBeck Any updates on this?

Is there a way perhaps we could plugin our own implementation for tenants? I think this might be better than trying to build a generic solution...

Hi @aminmc,
you're free to fork the repository at any time. What you have to do is to implement the tenant parts of the IdentityProvider (you'll find this when you look at e.g. https://github.com/camunda/camunda-bpm-identity-keycloak/blob/master/extension/src/main/java/org/camunda/bpm/extension/keycloak/KeycloakUserQuery.java - there is a UnsupportedOperationException for the tenant specific query). You can then implement this in any specific way you think is appropriate for your use case. Just try it out and go for it.

Hi @VonDerBeck , you mentioned you have a plan in mind. If it's ok, can you please share what you've thought of ? We just integrated this library in one of our projects and we were planning to implement our own experimental tenant support. Just want to make sure we don't digress too much from your approach so that we can upgrade easily. We were thinking of using keycloak realms to represent tenants. Our thought process was that we would ideally want to keep individual tenants as separate as possible and a one-to-one mapping between keycloak realms and camunda tenants feels like a good way of achieving this. Could you perhaps share your thoughts on this ?

Hi @juja0,
using keycloak realms to represent tenants might seem intuitiv from the Keycloak side. A few questions:

how would you handle scenarios where users choose to connect Keycloak to their LDAP / Active Directory and tenants are represented by specific group memberships?
using different Keycloak realms: does that mean you plan to configure a camunda-identity-service client within each realm?
how do you plan to handle tenant independent queries?

Connecting to several realms would likely require changing queries per tenant and different tokens per realm. Currently I don't think that it is the way to go for the Keycloak Identity Provider plugin, because it will blow up things quite a lot. Please let me know, if you think that I'm wrong with that opinion.
My simple and basic approach for tenants would be to define a e.g. a group prefix (or a list of predifened tenant names), treat all groups starting with that prefix as tenant and filter them out within group queries. As there a typically only a handful of tenants this is only very light overhead when querying groups.

how would you handle scenarios where users choose to connect Keycloak to their LDAP / Active Directory and tenants are represented by specific group memberships?

To be honest I never considered this scenario. The way I understood the terminology of 'tenant' is that each tenant would be a completely isolated entity (different organizations for example) and if users are using LDAP/AD, I assumed they would have separate LDAP/AD setup for each tenant. I am not sure how accurate my assumption is or if it reflects the majority of usecases so it's worth waiting for others to chime in with their usecases and thoughts on this. The group based tenant representation may be ok for most users I imagine, but for companies with regulatory requirements to keep tenants in separate realms, this might be a no-go. This is because if a user has access to add someone to a group within a realm, they probably have access to add them to any group in the same realm. So a malicious user can add themselves to any group and have access to any data in camunda. I might be overthinking this a bit but I can definitely see this becoming a hard requirement in our company (not sure about other similar companies).

using different Keycloak realms: does that mean you plan to configure a camunda-identity-service client within each realm?

That was not my original thought but now that you've mentioned it, I am beginning to think that it might be more secure and cleaner but as you highlighted, it might bloat the implementation on the plugin side. The approach that I had in mind was to use something similar to keycloak-admin-client to make the queries to keycloak (the client is just an example. we could just make the same REST APIs that the client makes using the existing KeycloakRestTemplate in the plugin). The advantage of using this is that we don't have to retrieve separate tokens for each realm. Keycloak installation usually comes with a default admin-cli in the master realm which can perform actions on any realm, so I imagine we can easily create a more restrictive version of it which only has read access to all groups/users across realms. But to be honest I haven't fully thought through this yet so I might be missing something here.

how do you plan to handle tenant independent queries?

Tenant independent queries can behave the same way they do now thus making them backward compatible. The realm parameter that is specified in the configuration currently can serve as a sort of default/fallback realm for non-tenant based queries. I don't imagine users will expect to be able query users across any non-default realm using a non-tenant based query. And since camunda queries have the tenant field in them, we can just direct our REST APIs to the realm(tenant) mentioned in the query and if not present, use the default realm.
That being said, I do agree that whatever approach we take, it will increase the scope of this plugin so I'm open to thoughts.

we have any update for this feature?

@rsilvareberth

we have any update for this feature?

Hi Réberth, sorry, but currently there are no updates on this. This is due to the fact that multi-tenancy is a thing which is very specifically handled in each organisation. So it's still open - and a longterm feature.


Is it possible to add any channel right to the .evn?

Hi,
thank you for your script and work (:
is it possible to add any give teamspeak channel permission to the .env?
(like setting the voice quality and channel icon)
with kind regards

Hi,
functionality does not exist yet. What permissions do you need?
Regarding setting a channel icon, you would need to know the icon ID, which you can only get via server query. If this is not an issue for you I can implement it as well.
Best,
Dominik

Hi @DSDV,
I published version v1.2.0. You are now able to provide channel codec and coded quailty via environment variable.
Best,
Dominik

Hi,
and thanks for the work /sorry for my late reply.
(we had a server migration and got our sinus bot up and running again).
i personally don't like the musicbot and that it needs a full team speak installation.
But the scripting add-ons are just so feature rich.
What we needed looked like that:

Hi,
got it. Currently you can set everything you can set in channel creation except, icon, encryption and file permissions.
Anyway thanks for the feedback and impressions to improve.
Best Dominik


refactor: 单元测试中避免使用 assert

顺带修复 #556 中的 import 问题

Codecov Report

Merging #559 into master will not change coverage by %.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #559   +/-   ##
=======================================
  Coverage   71.77%   71.77%
=======================================
  Files          80       80
  Lines        3164     3164
=======================================
  Hits         2271     2271
  Misses        893      893

Flag
Coverage Δ

#unittests
71.77% <100.00%> (ø)

Impacted Files
Coverage Δ

wechatpy/pay/api/jsapi.py
50.00% <100.00%> (ø)

wechatpy/schemes.py
96.49% <100.00%> (ø)

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 44c359f...282460c. Read the comment docs.


When will there be the first official version?I'm looking forward to it

Not sure why it matters, the extension is working fine whether it is 0.2 or 1.0.


[BlobStorage] Reuse the http client configured in the provided HttpPipeline during BlobBatch construction.

Currently the BlobBatch constructor rebuilds the http pipeline based on the provided pipeline but it uses the default HttpClient, that seems incorrect to me as the clients might use a custom http client.

Thanks for the review! I've just included the change log notes.

Excellent! We really appreciate your willingness to contribute!


Twitter Agent has tools to control the wallet from LLM calls

Examples
https://app.near.ai/agents/zavodil.near/near-agent/latest/run
https://app.near.ai/agents/zavodil.near/botfather/latest/run


Add tinymce as dependency?

Dependabot opened a PR (HEPData/hepdata#742) to bump invenio-assets from 3.0.0 to 3.0.1, but the CI failed with error messages like:
ERROR in unable to locate '/opt/hostedtoolcache/Python/3.9.18/x64/var/hepdata-instance/assets/node_modules/tinymce/skins/content/default/content.css' glob
Error: unable to locate '/opt/hostedtoolcache/Python/3.9.18/x64/var/hepdata-instance/assets/node_modules/tinymce/skins/content/default/content.css' glob
    at CopyPlugin.runPattern (/opt/hostedtoolcache/Python/3.9.18/x64/var/hepdata-instance/assets/node_modules/copy-webpack-plugin/dist/index.js:407:28)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /opt/hostedtoolcache/Python/3.9.18/x64/var/hepdata-instance/assets/node_modules/copy-webpack-plugin/dist/index.js:747:28

For now, I fixed the problem on the HEPData side by adding tinymce as a dependency (https://github.com/HEPData/hepdata/pull/742/commits/00cadae792677a024d951fb72bfa35c28381f538).  But shouldn't tinymce be added as a dependency in invenio_assets/assets/package.json if it is needed to build the assets following the changes made in PR #161?

Closed by commit 9b15f2d98fe07dfa3484421c1bd5e331c7832936 made by @kpsherva included in v3.0.2 release.  Thanks!


Cover Functionality with Unit Test

This is a technical task that includes the creation of the Unit Tests for the critical functionality of the Guardian. The tests will include coverage of the following components:

UI Service
Guardian Service
Message Broker Service

@Stepan-Kirjakov , is currently working on this and will be working on this the W/O 11/22.

https://github.com/hashgraph/guardian/blob/develop/README.md


API with transaction and write as parameter

I think transaction.GetDocumentSnapshotAsync and all similar api is not so convenient
Instead, could we write it as
firestoreDB.Collection("players").Document(follow.Key).SnapshotAsync(transaction);

It could be just an extension method
public static Task<DocumentSnapshot> SnapshotAsync(this DocumentReference doc,Transaction transaction,CancellationToken cancellationToken = default(CancellationToken))
{
    return transaction.GetDocumentSnapshotAsync(doc,cancellationToken);
}

// same for query / create / set / update / delete

Also I wish this could be possible
firestoreDB.Value.Collection("players").StreamAsync(transaction); // no equivalence in transaction

I'm not sure it makes much sense to stream results in a transaction. I'd expect streaming to be a relatively long-running aspect, whereas I believe a transaction is intended to be relatively short.
As for taking the transaction in an overload, personally I prefer to make it really clear that you're working with the transaction, right from the start. Of course, you could always add your own extension method if you want to - that doesn't have to be in the library.
@schmidt-sebastian Any comments? Have you seen similar requests in other languages?

I do see what you mean about moving code into a transaction, but I'm still a little uncomfortable with it.
We'll see what Sebastian says, but I personally wouldn't want to put those extension methods in. But the beauty of extension methods is that you can do that yourself - you don't need the library to support you at all for that to work.

Also I have seen internally we have
https://github.com/GoogleCloudPlatform/google-cloud-dotnet/blob/2be9e34b54bd4c99ad717a23be61ed399b021e62/apis/Google.Cloud.Firestore/Google.Cloud.Firestore/Query.cs#L550-L557
So stream in transaction should be possible
Also now I remember that I was familiar with transaction as parameter because low level api is like this
https://github.com/GoogleCloudPlatform/google-cloud-dotnet/blob/fdeff43831550f65783fab0a38244182e048824b/apis/Google.Cloud.Firestore/Google.Cloud.Firestore/Transaction.cs#L79-L81

Sorry, I just saw this.
This is the first time we have gotten this kind of feedback. Generally, we recommend reducing both the number and the scope of transactions in your code. Transactions acquire locks on the documents returned from the backend and can prevent other clients from making progress.
The current API encourages a small scope for these locks and makes transaction boundaries very visible.  Furthermore, we are also actively working on API features that replace some of the functionality for which transactions are commonly used and hope that this can further alleviate your concerns.
If you want to mix & match transactions with regular code, you should take a look at our WriteBatch support. The WriteBatch API is very similar to the Transaction API, with the notable admission of document reads.

@schmidt-sebastian If the obvious of transaction is your concerned I was propose this pattern
var docTransaction = await transaction.Collection("users").Document(key).SnapshotAsync();
docTransaction.Set(obj); // internally is transaction.Set(doc)

In anyway transaction was needed for safety to write value so locking is needed
I don't have a mix & match transactions with regular code. What I said above is, I have regular code written and work well. But then I would like to convert all of it into transaction. But the transaction API make it look somewhat difference from normal api. So I try to propose a way that make it look more familiar and could be convert more easily
And I don't propose this api to make it a large scope. Even in small scope or small block or actually any size of block, having convenient transaction api that could easily move in or out of transaction block is better

Have you look at the Precondition API?
You can use Precondition.LastUpdated(timestamp) to lock during an update() call. You would pass the updateTime from a DocumentSnapshot (https://github.com/GoogleCloudPlatform/google-cloud-dotnet/blob/master/apis/Google.Cloud.Firestore/Google.Cloud.Firestore/DocumentSnapshot.cs#L59) and then use this Precondition in a normal call to update(): https://github.com/GoogleCloudPlatform/google-cloud-dotnet/blob/master/apis/Google.Cloud.Firestore/Google.Cloud.Firestore/DocumentReference.cs#L162
For single document updates, this achieves the same effect as a transaction (if we disregard optimistic versus pessimistic locking). You can even do this via the WriteBatch API and combine multiple updates with individual Preconditions.

@schmidt-sebastian I use transaction as it should be. Unless you said transaction will be removed from firestore eventually I would not stop using it. You try to make me use precondition is relegate the management that transaction already did to me. I don't think it convenient
I use writebatch sometimes when read and write is far apart between many await but when it just straightforward set, I prefer transaction because it more convenient and in the scope
I know what I am doing. But I would like to have all possible option I could choose to solve problem. Please don't encourage me to go back using low level API

Transactions are definitely not going away! There is some overlap with other APIs that might help you and others in cases where running everything within the context of a single function is not the best option.
That being said, we value user feedback tremendously and are still very much evolving our API. We haven't gotten much feedback about this specific part of the API, and as such, value every opinion we can get on this matter. I will discuss your proposal with the rest of our team and keep you updated (likely by the end of next week).

Thanks very much for the feedback!
As I understand it, the proposal here is to change:
transaction.SnapshotAsync(docRef)

into
docRef.SnapshotAsync(transaction)

(and likewise for all the other transaction methods: Set, Create, etc.)
This would be another reasonable way to design the API, but we chose to do the former since:

Transaction.Set() is synchronous and returns Void while DocumentReference.SetAsync() is asynchronous and returns Task and so they're not compatible.
WriteBatch and Transaction both follow the same pattern, and so for Set(), Create(), etc. we'd need overloads for accepting a Transaction, a WriteBatch, or neither. Having extra overloads starts to make it more difficult to use the API and expand it over time.
By hanging operations off of the Transaction and WriteBatch objects, it's very clear and discoverable to users (via statement completion) what operations you can do within a transaction or write batch.  Otherwise you'd need to search through all of the DocumentReference methods for the ones that accept a Transaction or WriteBatch in one of their overloads.

Alternatively, we could accept both ways of doing things (as I think you are suggesting with the extension method) but we generally try to avoid having multiple ways of doing the same thing unless there's a meaningful benefit, since it creates ambiguity when generating documentation, samples, etc. and increases the API surface area that we need to maintain (if we add an argument in one place we need to add it in the others as well).
In this case, since you can create these extension methods yourself, I think we'll probably hold off on incorporating them directly into the SDK. We're open to additional feedback and will certainly reconsider if there turns out to be substantial demand or compelling use cases we haven't considered.
Thanks again for the thoughtful feedback! We appreciate it.

@mikelehen By your side suggestion, in my case I was already write extension method for tuple (transaction,document)
And so I was writing code like this
firestoreDB.RunTransactionAsync(async(transaction) => {
    var doc = transaction.Collection("users").Document(uid); // Tuple<Transaction,DocumentReference>
    var user = await doc.SnapShotAsync();
    doc.Set("counter",user.Get("counter",0) + 1);
});

All the update API was reduced to sync version. I also made the same for writebatch. But I think that this api should be extended from the same source
I mean you could have something like IReadAsync,IWrite,IWriteAsync that implement the logic and used to compose into DB,Transaction and WriteBatch
DB : IReadAsync,IWriteAsync
Transaction : IReadAsync,IWrite
WriteBatch : IWrite
And all three sharing underlying code
What I see is Transaction and always hold DataBase so I think we don't necessary need to reference it again in transaction codeblock. It more clean that way

That's a reasonable suggestion, thanks!
We'll keep it in mind for the future and watch out for other people trying to write common code that works across Transaction and WriteBatch objects. If it's a common pattern it may be worth introducing the interface.

I'm going to close this now as we have no plans to change the API design at the moment. The feedback is valuable though, and we'll bear it in mind for future options.


problem with clean_taxa

Somehow clean_taxa() is removing most taxa in my last update, so instead of thousands of fish taxa in the final database I end up with a couple of hundreds... to be fixed asap

the classification of fish was changed... again to the following classes:

Actinopteri is not a class but a superclass now, so Teloestei, Holostei, and Chondrostei need to be added to filter fish taxa


Default location for compiled MIBs should use $XDG_CACHE_HOME

The XDG specs define a more appropriate location for compiled MIBs than the current default of ~/.pysnmp/mibs/:

$XDG_CACHE_HOME defines the base directory relative to which user specific non-essential data files should be stored. If $XDG_CACHE_HOME is either not set or empty, a default equal to $HOME/.cache should be used.

On Linux this usually just means that
~/.pysnmp/mibs/

becomes
~/.cache/pysnmp/mibs/

Many other tools like pip, pre-commit or black already use that location.
For Python the appdirs library handles the default and Mac OS X/Windows compatibility.
>>> from appdirs import user_cache_dir
>>> user_cache_dir("pysnmp")
'/home/user/.cache/pysnmp'

I'm willing to try my hand at a PR if you tell me whether this is worthwhile.

Just noticed that you intended to send a pull request.
If you send to https://github.com/lextudio/pysnmp, my team is willing to review and accept it.


[ADD] <Languages/C++ Programming>

Category Name
Languages
Subcategory Name
C++ Programming
Additional Context
I will be creating a C++ programming subcategory and will also be adding some resources under it
Please assign me this issue under GSSOC'23
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I have checked that this category does not exist
[X] I have checked that this subcategory does not exist
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@Swapnil-Singh-99 assigned to you, you may start working on it now


Does elastalert can mention room on HipChat ?

I`m using Elastalert with sending alert on HipChat, but I'm interesting does can mention alert on HipChat ?

This does not work currently.  Using an @mention requires that the notification be sent via message_format: 'text'.  They default to html which does not allow @mentions.  I have a PR I will be submitting shortly to fix this.  So hopefully they will accept it and it will be resolved soon.

That's great. Thank you very much. I will wait to fix this.

@ryan-dyer-sp
Do you have some info for this issue ?
Thanks.

@ryan-dyer-sp is there any update on this?
How can I add @mention to the hipchat alert?


CheckEmail

I think CheckEmail should be check really email not username even tough default username is email.

I'll review asap!
But my first impressions is: Not always a username will be the e-mail. At registration flow the user could have a username that differ from e-mail.
When users make a Federated Login at first time, then username and e-mail is the same.
Anyway, I'll check asap.
These days of Covid I'm moving from my city, so it's really hard to get some time to work here!

Hi, I've reviewed the register flow.
Email and username isn't always the same. Actually we're enforcing that username and e-mail to be unique, in this way uesrs can do login by both.
So accepting this pr will cause some bugs!


workbench: Adds actions for moveEditorToFirstGroup, moveEditorToSecondGroup, moveEditorToThirdGroup

Adds the following actions for workbench tabs -

moveEditorToFirstGroup -  ^ ⌘ 1
moveEditorToSecondGroup - ^ ⌘ 2
moveEditorToThirdGroup - ^ ⌘ 3

Fixes: #40525

@bpasero Are there any test to extend upon?

@bpasero Any idea why Travis is failing?

LGTM thanks 👍

Hi,
Why couldn't I find these shortcuts in my keyboard references now? I think I used to be able to press shift+Alt+2 and shift+Alt+3 to move the editors (it will create it if the group does not exist)
Thank you

The VSCode version im using

@hardianlawi they got removed. Read this and this.


WIP: Initial support for Gradle

Ported the maven projects to Gradle
Changes that might break the pom-files:

Moved .java files so that the can be picked up by convention

Nice to have:

Initial support for artifact publishing
Added jacoco for code coverage

This is a spike to see how the project can be built with Gradle instead of maven.
I've left the iml:s and the pom:s to be able to run the project side by side.
How to build
./gradlew build # Builds the project
./gradlew test # Runs the tests
./gradlew publishToMavenLocal # Publish the artifacts to a local repo

It's possible to import a gradle project in intellij, see:
https://www.jetbrains.com/help/idea/2017.1/gradle.html#d309423e253
Things that needs to be fixed if you choose to jump ship to Gradle.

[ ] Publishing artifacts to bintray?
[ ] Adjust jvm version, not sure which one you'll target

I'm moving this to another branch


Feedback API https://api.thenewboston.com/feedback always return HTTP/1.1 500 Internal Server Error

Hi Team,
Bug Description
When I click the submit via https://thenewboston.com/faq/all "Send Us Your Feedback" then the API Feed is always returning HTTP/1.1 500 Internal Server Error
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://thenewboston.com/faq/all
Scroll down to "Send Us Your Feedback"
Input required details and Click on 'submit'
See error the respons: Sorry, there was an error with your submission. Please try again later.

When you use burpsuite you can see the response is HTTP/1.1 500 Internal Server Error as you can see on my screenshot below
Expected behavior
This should be return 200 OK success
Actual behavior
Its returning 500 Internal Server Error
Screenshots/Recordings

OS and Browser

OS:MacOS Big Sur v11.4
Browser Lastest Chrome, Firefox

Additional context
N/A
Account Number
eebcfd246d98cbfbaa131a4459234d1e7b32b48370a97a2fcec90feae77a5ac9
Kind Regards,
Rey

Here is the PoC Video: https://user-images.githubusercontent.com/17446899/125958423-67ef0bf4-787a-4d8d-b78a-4c215384bed2.mov

Hi @jamessspanggg, Thank you for the response.
I retest the bug, and it seems that the response now is 200 OK.

Hope you can confirm if this is already fixed ?.
Thank you!
Kind Regards,
Rey

Yep its all good now, I think yesterday with had an issue with our BE. Thanks for filing the issue!

Okay thats Cool, Thanks for the response @jamessspanggg


Handle architecture in createami

Ensure instance type and base AMI passed in createami args imply compatible architectures
Verify OS used for createami supports architecture implied by base AMI
Pass architecture to script used to build AMI
Implement tests accordingly

NOTE: this PR adds a dependency on the recordclass package in our integration testing framework. This was added in order to use its eponymous recordclass to create what are essentially mutable namedtuple objects (in this case to be used as simulated argparse.Namespace objects).
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1829 into wip/arm-support-feature-branch will increase coverage by 0.46%.
The diff coverage is 97.14%.

@@                        Coverage Diff                         @@
##           wip/arm-support-feature-branch    #1829      +/-   ##
==================================================================
+ Coverage                           54.67%   55.14%   +0.46%
==================================================================
  Files                                  28       28
  Lines                                4499     4530      +31
==================================================================
+ Hits                                 2460     2498      +38
+ Misses                               2039     2032       -7

Impacted Files
Coverage Δ

cli/pcluster/configure/easyconfig.py
80.00% <ø> (ø)

cli/pcluster/commands.py
22.24% <94.73%> (+2.76%)
:arrow_up:

cli/pcluster/config/validators.py
85.36% <100.00%> (ø)

cli/pcluster/utils.py
48.65% <100.00%> (+3.93%)
: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 408a649...8a24398. Read the comment docs.

addressing travis failure now. it's specific to mocker assertion methods that were deprecated for 3.5 and then re-introduced at 3.6?


TST: stats: xfail ttest_ind_randperm_alternative2 on 32 bit

Reference issue
Closes gh-13526
What does this implement/fix?
Xfails ttest_ind_randperm_alternative2 on 32 bit because it uses too much memory.

thanks @mdhaber

I suppose you could add a gh-13527 to the xfail message

OK, I can do that when I improve the exact part of the test.


"undeclared" issues when handling template classes in .h files

clangd-vscode may encounter "undeclared" issues when handling template classes in .h files if the header files are not included.

System information
Output of clangd --version:18.1.3
Editor/LSP plugin: clangd-vscode
Operating system: ubuntu 20.04

Duplicate of https://github.com/clangd/clangd/issues/45.


Add parser changes to allow annotations for each member in a tuple

Purpose

Allow annotations to be added to each member in a tuple

Fixes #36390
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@ushirask, please fix the PR title to be similar to the commit messages. See https://github.com/ballerina-platform/ballerina-lang/pull/30227#issuecomment-829996231.

We also need to add a path for the optional annotation in tuple members in the BallerinaParserErrorhandler

Sorry I missed this before.

We also need to change,
https://github.com/ballerina-platform/ballerina-lang/blob/56e03dd7b2938ef06557f46baf457c965f83ed13/compiler/ballerina-parser/src/main/java/io/ballerina/compiler/internal/parser/BallerinaParserErrorHandler.java#L5037-L5038 to

if (isInTypeDescContext()) {
    startContext(ParserRuleContext.TUPLE_MEMBERS);
    return ParserRuleContext.TUPLE_MEMBER;
}

Because we added ANNOTATIONS under TUPLE_MEMEBERS ctx we need to add an entry in getNextRuleForAnnotationEnd() for that ctx.

Shall we also add a recovery test for tuple annotations?

@lochana-chathura the alterenatives for TUPLE_MEMBER should be
private static final ParserRuleContext[] TUPLE_MEMBER =
            { ParserRuleContext.ANNOTATIONS, ParserRuleContext.TYPE_DESCRIPTOR };

right? not ANNOTATIONS & TYPE_DESC_IN_TUPLE

@lochana-chathura the alterenatives for TUPLE_MEMBER should be
private static final ParserRuleContext[] TUPLE_MEMBER =
            { ParserRuleContext.ANNOTATIONS, ParserRuleContext.TYPE_DESCRIPTOR };

right? not ANNOTATIONS & TYPE_DESC_IN_TUPLE

Nope. it has to be TYPE_DESC_IN_TUPLE. will explain you offline.


sometimes not mounting /Users

https://github.com/TheNewNormal/coreos-osx/pull/19

The fix was release with the latest v1.1.5