Illegal GO id used in 'abolished double-stranded DNA bridging'

http://purl.obolibrary.org/obo/GO_0106260
Dont fix it now.. :P give me a day to sort this  ODK stuff out.

Harrumph. Sync issue. That is the correct ID, from this GO ticket. That was closed almost a week ago, and the term is visible in QuickGO. How long does it take to reach everything else?

One week after GO release I would say! Thats great then, the issue will sort itself. If you wand you can close it.

the term finally made it into a GO release (phew)


Expired figure in How To Do part.

It seems that figure bed tva1.sinaimg.cn used in howtodo.md is now 403 forbidden, making it necessary to recover the expired figure.
Just make it into an issue.

any alternative?

https://sm.ms/ might be OK.

Solved in commit ce0290571937431c9a427d2d891ca6b806380966


assume content-type 'application/json' as default

lot of tile api endpoint needs header content-type to be set. Most of them only get json so assume the content is json (it should fail if it's not well formatted)
cc @rochoa

HTTP no longer a Windshaft responsibility.
This should be addressed in https://github.com/CartoDB/Windshaft-cartodb/issues/99


Improve readability for equipment totals

changed the font size to 14px and made it bold type

https://github.com/dragonjet/KC3Kai/issues/275

@bamboo3250 thanks for suggestion xD

no problem :+1:


SubsamplingScaleImageView is not a constructor

If the demo apps cannot help and there is no issue for your problem, tell us about it
I run 'tns plugin add nativescript-image-zoom' and everything installs fine.
The error occurs when I load a page with the ui:ImageZoom/ control present.
I do include the xlmns on the page element, but the application crashes with this error:
'TypeError: com.davemorrissey.labs.subscaleview.SubsamplingScaleImageView is not a constructor
File: image-zoom.js, line: 13, column: 15'
Which platform(s) does your issue occur on?
-Android
Please, provide the following version numbers that your issue occurs with:

CLI: 4.0.0
Cross-platform modules: 4.0.0
Runtime(s): Android -> 4.0.1
Plugin(s):
"dependencies": {
"font-awesome": "^4.7.0",
"nativescript-accordion": "^5.0.3",
"nativescript-image-zoom": "^1.1.1",
"nativescript-localstorage": "^1.1.5",
"nativescript-theme-core": "~1.0.4",
"nativescript-ui-sidedrawer": "^4.0.0",
"tns-core-modules": "^4.0.0"
},
"devDependencies": {
"babel-traverse": "6.26.0",
"babel-types": "6.26.0",
"babylon": "6.18.0",
"lazy": "1.0.11",
"nativescript-dev-typescript": "^0.7.1",
"typescript": "^2.8.3"
}

Please, tell us how to recreate the issue in as much detail as possible.
I just installed it, and used it in xml as specified, with: src="~/res/img1.jpg" and no other attributes.
Is there any code involved?

<ActionBar title="Test">
    <GridLayout class="action-bar" columns="*, 7*">
        <Label col="0" class="fa" text="" tap="goBack"></Label>
    </GridLayout>
</ActionBar>

<StackLayout>
    <ui:ImageZoom src="~/res/img1.jpg"/>
</StackLayout>

Thee plugin needs to be updated to support 4.x

Update to the latest version


nixos/test-driver: use ASCII single quotes everywhere

Description of changes
A few places used Unicode U+2018/U+2019 left/right single quotes (but
not always correctly balanced). Let's just use plain ASCII single quotes
everywhere.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Amended to fix typo s/qoutes/quotes/.


typo fix some mistake in Doris & Spark Column Type Mapping

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[x] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

I will update when new spark-doris-connector release.

run buildall

run buildall

run buildall


Detected layernorm nodes in FP16.

When i convert llama model with fp16 precision
python convert_checkpoint.py --model_dir /llama_dir --dtype float16 --output_dir /llama_dir_trt
trtllm-build --checkpoint_dir /llama_dir_trt  --output_dir  /llama_dir_trt_build --max_batch_size 64 --max_input_len 7168 --max_output_len 1024 --max_num_tokens 32768 --gpt_attention_plugin float16 --gemm_plugin float16 --context_fmha enable --paged_kv_cache enable --remove_input_padding enable

I see warnings in logs
[05/05/2024-13:14:38] [TRT] [W] Detected layernorm nodes in FP16.
[05/05/2024-13:14:38] [TRT] [W] Running layernorm after self-attention in FP16 may cause overflow. Exporting the model to the latest available ONNX opset (later than opset 17) to use the INormalizationLayer, or forcing layernorm layers to run in FP32 precision can help with preserving accuracy.

Does it accumulate in fp16 or in fp32?
My version of trtllm is https://github.com/NVIDIA/TensorRT-LLM/tree/06c0e9b1ec38f981d023a223b212b312cfb62417

Same problem when runnning qwen-14B, using float16, and my sentence output is different from pure pytorch. Using trtllm-0.8.0 @byshiue

Same problem when runnning qwen-14B, using float16, and my sentence output is different from pure pytorch. Using trtllm-0.8.0 @byshiue

If the results are different from pytorch but the quality is good or the score is similar, it is expected.

Please reopen this  ticket if there's further discussion.

tensorrt_llm-0.11.0.dev2024060400
[06/09/2024-16:23:35] [TRT] [W] Detected layernorm nodes in FP16.

@nv-guomingz reopen please

@akhoroshev Could we measure the trtllm engines' output quality via mmlu script?

@nv-guomingz  I have no problems with the quality of generation, but this warning is very annoying. Previously @byshiue  said that it should run FP32

@nv-guomingz I have no problems with the quality of generation, but this warning is very annoying. Previously @byshiue said that it should run FP32

Thanks for confirming, let me check with TRT guys for this warning and send u feedback later.

This message seems to go away when using bfloat16 fwiw

Maybe It only detect FP16?

The issue is fixed on TensorRT 10.3, which is release last week. Close this bug.

[09/24/2024-14:02:00] [TRT] [W] Detected layernorm nodes in FP16.
0.14.0.dev2024091000
@byshiue reopen please

I am seeing this warning log in TensorRT 10.5. Can we please re-open this issue?


refactor: use get_prev for Index lookup

Searching for a range allocates memory, lookup_prev doesn't.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#541  👈
#540
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Nov 4, 5:49 AM EST: A user started a stack merge that includes this pull request via Graphite.


Typo in chart archive

helm/k8dash/k8dash-0.0.1.tgz/k8dash-0.0.1.tar/k8dash/templates/service.yaml contains a typo:
  ports:
  - name: k8dash
    port: 80
    targetport: 4654

Should be
  ports:
  - name: k8dash
    port: 80
    targetPort: 4654

This is already the case in helm/k8dash/templates/service.yaml so I just re-packaged the chart.
.helmignore had to be added so that k8dash-0.0.1.tgz didn't wind up in the new archive

Thanks for the fix!


Change DefaultObjectMapper to NOT overwrite final fields unless explicitly asked to

https://groups.google.com/d/msg/jackson-user/1EFZzvhu_j0/NnDNahOZFAAJ
Turns out Jackson will very happily auto-detect final fields and overwrite them. This was causing a few bugs in how things were named to go unnoticed.
This is labeled as Discuss because it breaks backwards compatibility, and is intended to show a few locations where naming errors exist. If it is deemed we should fix this behavior I can add backwards compatibility.
If accepted there is a risk that some POJOs which do not have proper serde tests may break.
Another option is to leave the behavior as is, and simply have it generally be known that this behavior exists in Jackson.

BrokerServerViewTest.testSingleServerAddedRemovedSegment:113->setupZNodeForServer:384 » NodeExists

From the doc, realtime ingestion is using maxRowsInMemory and batch ingestion is using rowFlushBoundary, though in the code they share the same field, is it intentional or simply because we want backward compatibility?

@guobingkun both processes should use the same config and the docs shoudl be changed to reflect this. 0.9.0 is a good chance to make this change.

@fjy :+1:

Failed java 7 : TaskLifecycleTest.testRealtimeIndexTask:833 null

I added some fields in a separate commit that permit backwards compatibility for the known problem classes.
This is intended to make the upgrade process go smoothly.
It is intended that some point in the future the second commit can simply be reverted to remove the backwards compatibility once the upgrade has gone smoothly. So maybe 0.9.1?

@guobingkun / @fjy any other comments?

:+1:

Tentatively adding a Release Notes tag here, though I'm not quite certain what to say about it other than "version may require more testing than normal to ensure your configs don't get weird"


 Favicon.ico

A new favicon would be good instead of a React icon

A new favicon would be good instead of a React icon

yeah I tried making one, I'll consider making it again


fix ocp4_latest cluster with OCP47 argocd is stuck on old version operator

The verify tasks are failing because openshift-gitops operator version 1.2 can't be install.
From some debugging the operator is fixed on 1.0.1 and it doesn't upgrade when a new subscription in added

@seansund please give me access to the Cluster in IBM Cloud so I can login and see if I can fix it or we just get a new cluster

ping @seansund can you take a look or give me access

I have acces to the OCP cluster now, going to take a look at fixing argo for github actions


[ProgressIndicator] current step isn't using the fill: blue

What package is this for?

[x] React
[ ] Angular

Describe the bug
ProgressIndicator's current step svg should be blue, but its showing as black. I think this is a regression from the latest fill: currentColor change in the carbon update.
Because of this, we should go through and check all fill: currentColor areas to make sure they're working correctly
To Reproduce
Steps to reproduce the behavior:

Go to https://carbon-addons-iot-react.com/?path=/story/watson-iot-progressindicator--stateful
Click on Step 5
See black fill

Expected behavior
Should be blue according to https://react.carbondesignsystem.com/?path=/story/progressindicator--default
Environment/versions:

OS: MacOS
Browser: chrome
carbon-addons-iot-react version: v2.114.0

Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
No specific

@sstone2423 I think this have been fixed. Can you confirm so that we can close?

Fixed


CFY-7562. Secret create upsert

In this PR, the create secret command is updated to handle the upsert flag that is now supported by the REST API.
Requires: cloudify-cosmo/cloudify-rest-client#206

Why are we calling this option --upsert and not something more obvious like --update? (UPSERT is only likely to be familiar if you've encountered it in the context of RDBMSs... I hadn't heard of it before today despite having used databases for ~ 15 years...)

@funkyHat I think 'upsert' is reasonable as an arg name here, as long as our helptext is accurate. Our current helptext suggests that it really is just update, not upsert.

@funkyHat @geokala Thanks for your review. --upsert has been renamed to --update-if-exists which indeed should be clearer.


METRON-2153: ParserIntegrationTest should print failed messages

Contributor Comments
This PR fixes a minor bug in the ParserBolt that calls the incorrect method when processing errors from the ParserRunner.  The StormParserDriver.handleError method overrides the ParserBolt.handleError method but is never called because StormErrorUtils.handleError is called directly.
Testing
This fix should give more useful information on a failed test.  Add text to the first line of https://github.com/apache/metron/blob/master/metron-platform/metron-integration-test/src/main/sample/data/asa/raw/asa_raw and run the StormParserIntegrationTest.  You should get detailed log data about which messages passed and which messages failed along with various stack traces.  Before this PR only a single stack trace was printed.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

[ ] Have you ensured that any documentation diagrams have been updated, along with their source files, using draw.io? See Metron Development Guidelines for instructions.

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

+1 by inspection

+1 by inspection, thanks @merrimanr!


Markdown syntax support

Hi @blocknotes
It would be nice to have an option to add markdown syntax support with quilljs-markdown.
Thanks for making this plugin :)

It would be nice to have an option to add markdown syntax support with quilljs-markdown. Thanks for making this plugin :)

Hey @7nohe.
Sorry for replying so late.
Thank you for the PR. But I think that adding any plugin should be handled on the app side, not on the activeadmin_quill_editor component side.
A lot of people probably use the standard rich editor, so having unnecessary plugins included by default in not optimal.
I would prefer a usage example for it instead - in the examples folder.


sc.pp.normalize_per_cell(adata)

hi!
thanks for developing the wonderful algorithm!!
when I run the tutorial sc.pp.normalize_per_cell(adata),
the error occurUFuncTypeError: Cannot cast ufunc 'divide' output from dtype('float64') to dtype('int64') with casting rule 'same_kind',
when I change the code tosc.pp.normalize_total(adata),comes another errorAttributeError: 'DataFrame' object has no attribute 'n_counts'.
could you give some advice?Thanks!!!

Hi!
I apologize for not encountering the error you mentioned earlier. My hunch is that it might be related to the version of the scanpy package you are using. Would you mind updating your package to the latest version and giving it another try?

Hi!
Thank you for the quick reply!!I create a new environment and it works. Could you please tell me how does enhancement
 of an existing clustering result work? Could you give an example? Thanks again!
------------------ 原始邮件 ------------------
发件人:                                                                                "sldyns/scAce"                                                                                @.>;
发送时间: 2023年10月8日(星期天) 下午3:25
@.>;
<EMAIL_ADDRESS>主题: Re: [sldyns/scAce] sc.pp.normalize_per_cell(adata) (Issue #2)
Hi!
I apologize for not encountering the error you mentioned earlier. My hunch is that it might be related to the version of the scanpy package you are using. Would you mind updating your package to the latest version and giving it another try?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hi,
That's a great new!
For clustering enhancement, you can supply pre-clustering results as y_pred_init when using the run_scace function by setting init_cluster=y_pred_init.
For specific examples, please refer to our reproducibility codes


Clean messages after delay

A toggle that makes Parrot clean after itself. For each message Parrot sends, after the specified delay it cleans both response and request messages. It should only be callable by guild members with the Manage Messages permission.
Usage

!autoclean defaults the delay to something like 5 seconds
!autoclean <delay in seconds>

In retrospect, I don't personally see the need for a command like this and I'm a bit reticent how Parrot would handle editing potentially deleted messages (I suppose we should keep this in mind regardless).
Also, would it make sense deleting queue messages given that they live update? I think this change would add unneeded complexity to the code and I'm in favor of closing this issue if you're as well, @joao-conde & @afonsojramos.

We have and most servers will have a specific command for bots. So I would say we can close this.

In the case of big servers the original message can get lost quite quickly, so I still think this would be a welcome feature, even if we don't develop it right away.
As for the commands for bots, I'm not so sure that servers have message cleaning specific for bots. Maybe for whole channels, but I haven't seen it for bots in specific.

Thank you both for your inputs. I think @joao-conde meant servers usually have a dedicated text channel for music bots?
IDK, this is a music bot, I don't see the need for cleaning - there are plenty of bots out there that already do this.
I'm closing it as not planned for the time being.


Add CODE_OF_CONDUCT.md

Fixes #18820

(minor comment) None of the files in the top-level of the repo use _ in the filename, and several of the .md files use - to separate words in their filenames. However, since the _ is used instead of spaces, this could well be reasonable (in the other .md filenames, the -s are representing punctuation. As I said, just a comment to make sure this is intended.

(minor comment) None of the files in the top-level of the repo use _ in the filename, and several of the .md files use - to separate words in their filenames. However, since the _ is used instead of spaces, this could well be reasonable (in the other .md filenames, the -s are representing punctuation). As I said, this just a comment to make sure this is intended.

Good point. I chose the underscore because it was the filename recommended by GitHub's help. Personally, I would prefer underscores for all 'capital' markdown files in the root directory, but that would have to be decided and fixed independently. So for the sake of consistency, I'll rename the file from CODE_OF_CONDUCT.md to CODE-OF-CONDUCT.md for the moment.
I checked in the Web Interface that GitHub recognizes that filename as well, as the screenshot shows:

The name was OK with underscores too but this one is fine as well.

@t8m agreed. It' just the obvious inconsistency with the names of the other top-level markdown files which is disturbing.
@openssl/otc: what is your opinion about replacing dashes with underscores (in a followup pull request), as suggested by @tom-cosgrove-arm ?
Dashes
ACKNOWLEDGEMENTS.md
AUTHORS.md
CHANGES.md
CODE-OF-CONDUCT.md
CONTRIBUTING.md
FAQ.md
HACKING.md
INSTALL.md
NEWS.md
NOTES-ANDROID.md
NOTES-DJGPP.md
NOTES-NONSTOP.md
NOTES-PERL.md
NOTES-UNIX.md
NOTES-VALGRIND.md
NOTES-VMS.md
NOTES-WINDOWS.md
README-ENGINES.md
README-FIPS.md
README.md
README-PROVIDERS.md
SUPPORT.md

Underscores
ACKNOWLEDGEMENTS.md
AUTHORS.md
CHANGES.md
CODE_OF_CONDUCT.md
CONTRIBUTING.md
FAQ.md
HACKING.md
INSTALL.md
NEWS.md
NOTES_ANDROID.md
NOTES_DJGPP.md
NOTES_NONSTOP.md
NOTES_PERL.md
NOTES_UNIX.md
NOTES_VALGRIND.md
NOTES_VMS.md
NOTES_WINDOWS.md
README_ENGINES.md
README_FIPS.md
README.md
README_PROVIDERS.md
SUPPORT.md

what is your opinion about replacing dashes with underscores (in a followup pull request)

I would see that as unnecessary churn. So no, please.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master in 63df86b041, thanks!


Start on the flyby step up regulator design

("Dark power part 2"). So we can produce higher voltages.
I think @SeanKetring  was heading this up?

@SeanKetring successfully created a working breadboard prototype with this functionality. This project is shelved until after SEcon, when Sean or another dev can work on creating a PCB layout for it.


fix(datepicker): fix popup to work with ng-model-options:updateOn='default'

When using datepicker with popup and ng-model-options="{ updateOn: 'default'}", value can not be set via popup.
Plnkr with the problem(and hacky solution):
http://plnkr.co/edit/4LLbxCkWBzoPwJuOAbye?p=preview
This PR fixes this problem and selection(update model) via datepicker popup is possible again.
Use case for this scenario
Often we define ng-model-options={ updateOn: 'blur'} for multiple wrapped elements(e.g form).
For datepicker that is inside that element(form) usually doesn't make sense to update model value on blur, so we need to add ng-model-options to the datepicker element.
Technical information
AngularJS directive ng-model-options parses the given expression and object that is set to ngModel.$options is different then the object in expression. Here is ngModelOptions directives from angularJS code that changes this:
var ngModelOptionsDirective = function() {
  return {
    restrict: 'A',
    controller: ['$scope', '$attrs', function($scope, $attrs) {
      var that = this;
      this.$options = copy($scope.$eval($attrs.ngModelOptions));
      // Allow adding/overriding bound events
      if (isDefined(this.$options.updateOn)) {
        this.$options.updateOnDefault = false;
        // extract "default" pseudo-event from list of events that can trigger a model update
        this.$options.updateOn = trim(this.$options.updateOn.replace(DEFAULT_REGEXP, function() {
          that.$options.updateOnDefault = true;
          return ' ';
        }));
      } else {
        this.$options.updateOnDefault = true;
      }
    }]
  };

So when we use ngModelOptions.$options value to compile uib-datepicker-popup-wrap , we lose updateOn property and ngModel can not be updated using the pop up.

LGTM - thanks for this subtle catch...don't like how Angular seems to handle this.


Can't pay invoice - FAILURE_REASON_NO_ROUTE (to node with I have channel)

Background
My friend opened a channel from his new node, so he can test and see how LN is working. Channel has been opened (https://amboss.space/edge/780007842453520384), I´ve created him a 2 invoices (500sats each) and he was able to pay it. But when he created invoice with 500 sats I am always getting error "FAILURE_REASON_NO_ROUTE" even I set outgoing channel to his.

Actual behaviour
lncli output:
Payment hash: c249883e0a7f320804dd7d296aa0601f33cfce1d25e923622dfe8524fb1d0c0a
Description: pruba
Amount (in satoshis): 500
Fee limit (in satoshis): 500
Destination: 03e14450218824b1d7d3f7d2cb454377fcd7b02f21374058f56c89d8150db0782a
Confirm payment (yes/no): yes
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
| HTLC_STATE | ATTEMPT_TIME | RESOLVE_TIME | RECEIVER_AMT | FEE | TIMELOCK | CHAN_OUT | ROUTE |
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
Amount + fee:   0 + 0 sat
Payment hash:   c249883e0a7f320804dd7d296aa0601f33cfce1d25e923622dfe8524fb1d0c0a
Payment status: FAILED, reason: FAILURE_REASON_NO_ROUTE
[lncli] FAILED

Logs from LND:
2021-11-14 14:07:09.692 [WRN] CRTR: Failed to find route for payment c249883e0a7f320804dd7d296aa0601f33cfce1d25e923622dfe8524fb1d0c0a: unable to find a path to destination
2021-11-14 14:07:09.696 [ERR] CRTR: Payment c249883e0a7f320804dd7d296aa0601f33cfce1d25e923622dfe8524fb1d0c0a failed: no_route

When I am trying it with  --outgoing_chan_id getting following:
Payment hash: c249883e0a7f320804dd7d296aa0601f33cfce1d25e923622dfe8524fb1d0c0a
Description: pruba
Amount (in satoshis): 500
Fee limit (in satoshis): 500
Destination: 03e14450218824b1d7d3f7d2cb454377fcd7b02f21374058f56c89d8150db0782a
Confirm payment (yes/no): yes
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
| HTLC_STATE | ATTEMPT_TIME | RESOLVE_TIME | RECEIVER_AMT | FEE | TIMELOCK | CHAN_OUT | ROUTE |
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
Amount + fee:   0 + 0 sat
Payment hash:   c249883e0a7f320804dd7d296aa0601f33cfce1d25e923622dfe8524fb1d0c0a
Payment status: FAILED, reason: FAILURE_REASON_INSUFFICIENT_BALANCE
[lncli] FAILED

When we try with lower amount (450sats):
Payment hash: cb3f6270c9e2c5fec4596ed9757327d1d25b56723488073ca5f66033337c4f1b
Description: test
Amount (in satoshis): 450
Fee limit (in satoshis): 450
Destination: 03e14450218824b1d7d3f7d2cb454377fcd7b02f21374058f56c89d8150db0782a
Confirm payment (yes/no): yes
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
| HTLC_STATE | ATTEMPT_TIME | RESOLVE_TIME | RECEIVER_AMT | FEE | TIMELOCK | CHAN_OUT | ROUTE |
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
+------------+--------------+--------------+--------------+-----+----------+----------+-------+
Amount + fee:   0 + 0 sat
Payment hash:   cb3f6270c9e2c5fec4596ed9757327d1d25b56723488073ca5f66033337c4f1b
Payment status: FAILED, reason: FAILURE_REASON_INSUFFICIENT_BALANCE
[lncli] FAILED

getchaininfo:
{
    "channel_id": "780007842453520384",
    "chan_point": "b1baf4303575640dcbd6b986d39613f705aa6897f398f2c8a68ac08f7513dde7:0",
    "last_update":<PHONE_NUMBER>,
    "node1_pub": "028d829e9254bf0a816983fb79c61d1aefcdc210a3c6f37177d62db98e3ea3a8f7",
    "node2_pub": "03e14450218824b1d7d3f7d2cb454377fcd7b02f21374058f56c89d8150db0782a",
    "capacity": "20000",
    "node1_policy": {
        "time_lock_delta": 40,
        "min_htlc": "1000",
        "fee_base_msat": "1000",
        "fee_rate_milli_msat": "1",
        "disabled": false,
        "max_htlc_msat": "19800000",
        "last_update":<PHONE_NUMBER>
    },
    "node2_policy": {
        "time_lock_delta": 40,
        "min_htlc": "1000",
        "fee_base_msat": "1000",
        "fee_rate_milli_msat": "1",
        "disabled": false,
        "max_htlc_msat": "19800000",
        "last_update":<PHONE_NUMBER>
    }
}

Every channel has a chan_reserve_sat that is reserved for paying the closing fees. Either side isn't allowed to go below that channel reserve. So since you started out with a balance of 0 on that channel, you'll only be able to pay back through that channel once your balance is above the reserve. That is quite in-intuitive, I agree. And I think the reserve can be set to a minimum now with ANCHOR channels since the fees are paid through CPFP.

Closing as it's a reserve issue, in the future you can search the tracker first to find other instances like this.


fscache: simple fixup for busy mountpoint

When snapshotter exits (either normally or abnormally), it will not have a
chance to umount erofs mountpoint, so if snapshotter resumes running and mount
again (by a new request to create container), it will need to ignore the mount
error device or resource busy.
Signed-off-by: Yan Song<EMAIL_ADDRESS>

Codecov Report

Merging #83 (dc3e940) into main (f780759) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   28.78%   28.78%
=======================================
  Files          17       17
  Lines        1501     1501
=======================================
  Hits          432      432
  Misses       1005     1005
  Partials       64       64

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 f780759...dc3e940. Read the comment docs.


first pull request -

LS_DS_111_A_First_Look_at_Data.ipynb in module1-afirstlookatdata/

Assignment complete, minus stretch goals.

https://colab.research.google.com/drive/1VcvBCm-TN4tntYUUL_a3_iwre8rshlbc

WARNINGS:

When pushing from colab notebook, the first in the list of alpha-sorted repos is selected by default. MOST LIKELY NOT THE RIGHT ONE
Sometimes, but not always, the filename gets '-' renamed to '_' --resulting in a different target (and different file) in github, so NOT over-writing the previous version. Now there are two files in my push history!


-CertStoreLocation parameter not recognized in Powershell

I am trying to use the New-SelfSignedCertificate cmdlet with the -CertStoreLocation parameter. I am using the value of "Cert:\CurrentUser\My". the error message I get back is "A parameter cannot be found that matches parameter name 'CertStoreLocation'"
[Enter feedback here]

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

ID: a2cc6356-761f-c845-6ccb-8fd12b1dc7ba
Version Independent ID: 08045f21-38f3-4c0a-bbeb-c51cdb6615e8
Content: New-SelfSignedCertificate (pki)
Content Source: docset/winserver2022-ps/pki/New-SelfSignedCertificate.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

This repository is for PowerShell core documentation. You seem to be looking for support, which we can't provide here.
I suggest you try posting your issue with context in one of the available community support forums.


How do I set a custom message for missing user input?

Hi,
Whenever a user skips a question the message shown on the user side is "Missing input ...".
Is there any way to customize that message?
I've tried the following without effect:

cf-user-response-missing attribute to the input tag
dictionaryData object passed at init with "user-response-missing" key

Thanks!

@AfaanBilal you can set this globally for CF by overriding the dictionary. See this example from the docs. You want to change "user-reponse-missing" (notice the prop name is misspelled...)

Thank you!


word bomb mini rework

Hopefully its finally done, might be missing a few commits from master i forgor

internally reworks word bomb mini to try and make it more generic and have support for future games and making word bomb mini easier to develop
reworks remarks to make them easier to edit and add upon
slightly changes leaderboard functionality for future use
changes dictionaries to support sets (in most cases) to hopefully make that section a bit faster
other various qol stuff

sendReplyWithID
isPlural
save state for word bomb mini
partially types db.ts

i think generatePrompt (in dictionary.ts) is where the option to change the minimum amount of solutions should be rather than enforcing it when generating the prompt frequency map


SCC PowerShell Cmdlets - Considerations for App-Only Auth

Similar to what was posted in https://github.com/MicrosoftDocs/office-docs-powershell/issues/9759.
It has come up that the limitations of Cert-Based Auth are due to User-Context cmdlets. Is there any way to provide a comprehensive list of which cmdlets are user-driven, so that they can be added as considerations? The reasoning behind this is due to the deprecation of Basic Auth, many customers will be turning to Cert-Based Auth as a means to provide support for scripting. If there is not 1:1 parity, customers cannot feasibly change to cert-based auth.

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

ID: d2cfafdf-cbe9-2ffe-eabe-becc66d51728
Version Independent ID: 4a46c8a8-dc70-d877-271e-6679c465a6d5
Content: App-only authentication
Content Source: exchange/docs-conceptual/app-only-auth-powershell-v2.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

Additionally, this may impact further cmdlets within the following documentation - https://docs.microsoft.com/en-us/powershell/module/exchange/?view=exchange-ps

@SeanTGeoghegan, I'm sorry, I need further explanation of what User-Context or user-driven cmdlets are.
The last time I checked in that other issue you mentioned, the cmdlets that are called out are the only ones known not to work with CBA.

@SeanTGeoghegan Hope this comment is helpful for you, If you see a documentation update is required, please feel free to reopen this issue and answer the query. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


How (not) to lie with CARTO

@oriolbx can you give me the favor of reviewing this new workshop? :-)

Added just a comment as a review :)


default nva sku in terraform script does not work

Have to change it to "fortigate_vwan_nva" to make it work.
the default vaule is "'fortinet-sdwan-and-ngfw' or 'fortinet-ngfw'.   but it does not work.
fortinet-azure-solutions/FortiGate/AzureVirtualWAN/terraform/varible.tf
variable "fgt_sku" {
default = "fortigate_vwan_nva"
}

Thanks for reporting this. We will check this and update the template where needed.

Updated modules have been posted on the below link which contains other FortiGate Terraform modules as well:
https://github.com/40net-cloud/terraform-azure-fortigate


Feat: Adding Animation

Describe the feature

Your about us page already looks fantastic, but incorporating animations on the images used could make it even more captivating. With my experience in this area, I'm eager to contribute to this enhancement.

Add ScreenShots
FOR EXAMPLE
https://github.com/VanshKing30/FoodiesWeb/assets/115486960/554862ff-fed5-4908-89da-72c477b57182
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

This will make the website more buggy , alredy there exisit a lot of animations !!


⚠ GitHub has degraded performance

In 81c3a8f, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 148 ms

Resolved: GitHub performance has improved in 32d9204.


Adding google analytics on Wiki

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Can you please add a little tutorial on how add uptime kuma to google analytics ?
🐻 Uptime-Kuma Version
1.21.0-beta.0
💻 Operating System and Arch
Ubuntu 22.04 LTS
🌐 Browser
Chrome: 110.0.5481.178
🐋 Docker Version
23.0.1
🟩 NodeJS Version
No response

Can you please add a little tutorial on how add uptime kuma to google analytics ?

Can you describe what you want to achieve?

I want to add google analytics to my uptime kuma and know how to have the id of this https://img.uruk.dev/ .


Add files via upload

@erezoz1 First stage of verification passed, official domain is sites.google.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://sites.google.com/edu-darom.org.il/yuvalim2023/בית/ה1

@erezoz1 Could you please clarify what is the organization that owns and uses this email domain? Is it a single school?
Please provide us with:

the school official website URL
the school street address, including city and country.

@erezoz1 You might miss my previous comment. Please react.

Hi I don't want to proceed with my application I will just use the
community version thank you very much
בתאריך יום ו׳, 1 במרץ 2024, 6:38, מאת Philip Torchinsky <
@.***>:

@erezoz1 https://github.com/erezoz1 You might miss my previous comment.
Please react.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/20111#issuecomment-1972493409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSAZHH5LINTI73CSFKYMN3YWAA57AVCNFSM6AAAAABDIM7OAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZSGQ4TGNBQHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


onAuthenticate

Beyond file selection, the oauth token is useful, for example for file download.
Currently, there is no way to get the token, as react-google-picker keeps it as a private var. The only workaround I found was to override createPicker. However, this is not the right solution, as it forces you to rewrite the whole picker creation when you only wanted a piece of information.
With onAuthenticate, you get a simple and convenient way to get the token.

@phbernard Thanks a lot for your improvement.
Could you please remove dist from PR? After that I'll be able to merge your code

Damn! I didn't realize you were going to get this last commit. It was intended to be more or less "private".
Alright, dist gone again, and React.PropTypes vs PropTypes merged.

@phbernard it happens when you'r doing git push 😄


Update Modern CLI Calculator

I created and updated my Modern CLI Calculator, Hope you like it,, 🙂

Great job!! Keep it up!! 👍🏻


Added and documented resampler abc.

Addresses #74.

Thanks! I've also deprecated ClusteringResampler, since that was an unmaintained experiment. It already was undocumented and not imported by default, so I think this is a fairly low-impact deprecation.

Yah that makes sense. I still would like to work on the clustering you
mentioned when you were here in Waterloo. It may have to wait until after I
finish writing my thesis though.
On Sep 22, 2016 9:48 PM, "Chris Granade"<EMAIL_ADDRESS>wrote:
Thanks! I've also deprecated ClusteringResampler, since that was an
unmaintained experiment. It already was undocumented and not imported by
default, so I think this is a fairly low-impact deprecation.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/QInfer/python-qinfer/pull/96#issuecomment-249080000,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABe1ttJOb03fjmCmY2Xe44aMMLxVVvNoks5qszAHgaJpZM4KEh-q
.

Agreed, I'd very much like to work on it as well; in fact, that's part of my motivation for deprecating this feature. I don't think that the current design of ClusteringResampler is the right way to do that, even though the ideas are still interesting.

Coverage increased (+0.06%) to 48.127% when pulling<PHONE_NUMBER>60bda94a7973e4e35279813736eec5 on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Coverage increased (+0.06%) to 48.127% when pulling<PHONE_NUMBER>60bda94a7973e4e35279813736eec5 on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

I'm not sure why the CI builds are broken. Again. Investigating now.

Coverage increased (+0.06%) to 48.127% when pulling 38c5b3581532105f0f4dc0080e586da929738a3e on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Coverage increased (+0.06%) to 48.127% when pulling f50b074948ecd1ace837051e607dec37338daebb on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Coverage increased (+0.06%) to 48.127% when pulling 45ce958b193cae6ffe36e75b1f86da1246a824bf on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Coverage increased (+0.06%) to 48.127% when pulling c7b057b402da8af9414dd10c234773725040ef22 on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Coverage increased (+0.06%) to 48.127% when pulling 0ce19c468aec9391a7e3d956cde35b85385e150e on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Coverage increased (+0.06%) to 48.127% when pulling 0ce19c468aec9391a7e3d956cde35b85385e150e on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

I think changing the backend for Matplotlib did it, there seems to be some sort of a conflict with the recent updates to Qt. Strangely, installing older versions didn't help, but this is more correct anyway, in that we shouldn't have our tests contingent on Qt at all.

Coverage increased (+13.4%) to 61.504% when pulling a04c692e495934b86b86f3788cc3e7cc539d7c8e on feature-resampler-abc into d46aca0a84c8574addd6601fb70fa59ae845c378 on master.

Welp, that was annoying. Merging now.


Fix the bug of compile saved model file to saved model

Signed-off-by: zhangkaili<EMAIL_ADDRESS>

bors r+

bors r+

bors r+


Seemingly High Ram usage

I just deployed UpSnap stock (sqlite) docker image and I like it. When I looked at the ram usage, The upsnap_app is using  561MB of used memory and 101MB of cache. This is with just one device added to it. I find it hard to believe that a simple web interface and a scheduler need that much ram. The Reddis instance is only taking about 12MB (Usage + cache).
I'm not saying the code is bad. I'm saying that probably the internal DJANGO service needs to be tweaked for more realistic RAM usage.

I'm using the amd64 docker-compose from this github repo.

Hi there. Django in combination with celery and redis is known to use a lot of ram. I've read something about reducing workers, but likely I won't change anything to v2 as I'm currently working on v3 which is written in go. (If you find some quick and easy improvements, let me know.)
v3 currently uses around 13mb (although not all features are inplementet yet).
I hope it will only take a few more days until I can get a first release candidate out.

Yea. the redis container isn't the problem. It is running in a very reasonable about of ram. I think the biggest thing is the number of workers. I think we could reduce down to 2 and still be find. I'll do some looking into it.

I dropped the works from 4 to 2 in run.sh. That dropped the RAM usage from about 561MB to about 422mb usage and 26MB cache. Not great but better.

Api is now available in the v3 beta release. I will add some documentation later on in the wiki.
https://github.com/seriousm4x/UpSnap/releases/tag/v3.0.0-beta.1


Creative Commons license

May I suggest the Creative Commons Zero v1.0 Universal (CC0-1.0) or the Creative Commons Attribution 4.0 International (CC-BY-4.0)?

I moved this from prql-brand.
@vanillajonathan opened it at the right place, but here we can track it & plan to discuss

If you need legal advice you can consult the Software Freedom Law Center (SFLC).

I don't have a reliable advice to give here.
But you do have my approval to apply any commonly used license.

I also think the "book" should be licensed under the Creative Commons as that would make it easier to incorporate materials from the "book" into blogs, wikis, white papers, tutorials and books.

They are for different things; open source license cover computer software and things such as binaries and source code availability which does not apply to content while open content licenses are for works of art and documentation.

Do you have any sources for requiring a different license for docs? The first source I found was https://opensource.stackexchange.com/questions/6110/is-there-any-particular-reason-to-license-docs-with-cc-by-4-0-rather-than-mit, which suggests there's no need to have different licenses

No, I don't.
But I guess Creative Commons is easier to understand for the layman, while the Apache License which is quite lengthy can be difficult to understand as it deals with binaries, source code, trademarks and patents.
Wikipedia and Wikibooks require content to licensed under CC or GFDL.

By publishing changes, you agree to the Terms of Use, and you irrevocably agree to release your contribution under the CC BY-SA 3.0 License and the GFDL.

OK, if we can get a reliable source that answers "why do you have two licenses?" we can reopen. Otherwise I'd vote to keep this simple and avoid multiple licenses.

https://github.com/dotnet/brand/blob/main/LICENSE

Yes, we should do it for the brand, that's a good idea — contribution to that repo welcome!
Without a good reason, I don't think we should do it for the docs, which are already open source

@vanillajonathan do you want to PR the CC license into the prql-brand repo?

I think we want something as close as possible to https://foundation.rust-lang.org/policies/logo-policy-and-media-guide/, rather than CC-Zero


test: Add SDK test for all mint operations

Closes: #283

@beeman the tests pass in local env. but I am not sure how to add and enable the mint in the environment that we use for CI. Other than that, it should fine for review.

@beeman the tests pass in local env. but I am not sure how to add and enable the mint in the environment that we use for CI. Other than that, it should fine for review. Do you have any suggestions?

That makes sense. I think it's something that would be addressed by #285 - I'm going to pull this branch in and test if that makes it work and implement that.

@RamirezAlex make sure to pull in the branch before working on it. I added a way to run the sdk-e2e against the local API (without spinning up its own) which makes debugging easier.
yarn nx run sdk-e2e:e2e-local --verbose --watch


Difficult to use with unit tests

Thanks for open sourcing this project!
I am using the latest release (v1.8), and I use methods such as await Await.UnitySyncContext(); and await Await.BackgroundSyncContext(); in the production code.
However, when this code is under testing (NUnit tests run with the Unity test runner in Edit mode), these calls fail because AsyncManager.Initialize() has not been called. This is because [RuntimeInitializeOnLoadMethod(RuntimeInitializeLoadType.BeforeSceneLoad)] does not do anything when running the tests.
It would be nice to have a fix for this. I suggest creating a mechanism to be able to do this initialization from code (so that it can be called in the tests setup method). I will create a pull request with my suggestion for a fix.

@muckSponge I have created pull request #8 now. It would be great if you could review this. :-)


Add typing notifications constants to POST /register response.

As noted in this PR comment and in this conversation in the development community, we should add the three typing notification constants set on the server to the POST /register response.

TYPING_STARTED_WAIT_PERIOD
TYPING_STOPPED_WAIT_PERIOD
TYPING_STARTED_EXPIRY_PERIOD

These constants are referenced in the API documentation for POST /typing. Therefore that documentation would also need to be updated as part of these changes.

@zulipbot claim

@zulipbot claim

@zulipbot abandon

I'm working on this with Samuel Adeyemi, Wilhelmina & Oluwaseyifunmi

@zulipbot claim

Still working on the issue - we were reviewing the implementation of the "presence" feature to understand how to add the typing constants.

@zulipbot claim

Dear Zulip team,
Thanks for reaching out.
We made a PR to the issue - here's the link:
https://github.com/zulip/zulip/pull/26447
*Regards, *
Samuel Kabuya
On Thu, Aug 10, 2023 at 12:53 PM zulipbot @.***> wrote:

@KabuyaSamuel https://github.com/KabuyaSamuel We noticed that you have
not made any updates to this issue or linked PRs for 10 days. Please
comment here if you are still actively working on it. Otherwise, we'd
appreciate a quick @zulipbot abandon comment so that someone else can
claim this issue and continue from where you left off.
If we don't hear back, you will be automatically unassigned in 4 days.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/25656#issuecomment-1672915646, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A3JVSBPSW2DUX5QLUDZDXL3XUSVTLANCNFSM6AAAAAAYGNVCHY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@zulipbot - here's the link to our PR as per the email request to comment on the issue.
https://github.com/zulip/zulip/pull/26447

@zulipbot claim

@zulipbot claim

This was fixed in #26477.


Strange behaviour within bit.dev

Environment
System:

OS: Windows 10 10.0.18362
CPU: (8) x64 Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
Memory: 1.75 GB / 7.97 GB

Binaries:

Node: 12.13.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.19.2 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.12.1 - C:\Program Files\nodejs\npm.CMD

npmPackages:

styled-components: ^4.4.1 => 4.4.1

Steps to reproduce
Im using CRA + SC to configure theme and also I have bit components(each components has SC installed as dependency)
Expected Behavior
I expect my bit components working together on local development without errors and each one having SC installed as dependency.
When I use without bit everything works like a charm.
Actual Behavior
./src/components/typography/dist/node_modules/styled-components/dist/styled-components.browser.esm.js
  Line 143:64:   'SC_DISABLE_SPEEDY' is not defined                                     no-undef
  Line 1374:5:   Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 1382:7:   Expected an assignment or function call and instead saw an expression  no-unused-expressions
  Line 1536:3:   Definition for rule 'flowtype-errors/show-errors' was not found        flowtype-errors/show-errors
  Line 1820:27:  'stream' is not defined                                                no-undef

Search for the keywords to learn more about each error.

I’m not sure why bit.dev is but these are purely linter warnings based on imports that are safe to use with Webpack, for example this one where we’re checking whether a global is defined: https://github.com/styled-components/styled-components/blob/921e14f7af063ea7518145860bac8135d79040d5/packages/styled-components/src/constants.js#L15
If you have a longer active on node_modules it’s probably advisable to disable it for all dependencies

@kitten probably I need to import components without dist, I need to use tsx files instead of compiled files.

Going to close as this is definitely a third-party issue and not something wrong with the library itself.


Create a TaskLauncher that calls SCDF to launch Tasks

It should be a poll based consumer (it should entertain a cutoff on maximum concurrently running tasks)

Determine the credential handshake between the TaskLauncher and the SCDF endpoint (it should somehow handled internally; most likely an improvement in the deployers)

child of spring-cloud/spring-cloud-dataflow-samples#66
required for spring-cloud-stream-app-starters/app-starters-release#141

This issue was moved to spring-cloud-stream-app-starters/tasklauncher-data-flow#2


Use time from options

This PR allow to specify a time for the Cesium rendering.
@gberaudo and @fredj review please ?

The time is used on each render call, so I'm curious why you want to provide a static time. The time provided with Cesium should be increasing with each frame.
This is going to conflict with my PR (#411), which is using the time from requestAnimationFrame to determine when to render frames. I could pass that frame time to Cesium in place of the old Cesium.JulianDate.now() code.

Because I want the sun to be always at the same place. But I'm open to suggestions to achieve that.

Ahh, I didn't poke into what the time provided to Cesium was controlling. Now I'm on the same page.
I refactored the render loop code a bit so we're going to have a merge conflict between our PR's, but the changes are still compatible.

Updated with a function instead of a constant

Rebased.

@oterral, would you please modify the main example to use this feature?
I imagine we could have a checkbox to select between real time and constant time.
It is fine to have shadows always enabled.

Yes sure

example added

Thanks


Callback to function should be passed data instead of needing to read stdout

Right now, as per the docs:
    // Outputs nodes with bridge tags to stdout
require('osmlint').bridgeOnNode([-122.1, 36.9, -121.9, 37.0], 15, './osm.mbtiles', function() {
    console.log('done');
});

I would expect the callback function() there to receive something like data with the invalid features rather than needing to read stdout.

None of the validators aggregate features right now (example) but rather write feature by feature to stdout to save memory. This makes sure they scale.
We could make aggregation optional like so:
require('osmlint').bridgeOnNode(
    [-122.1, 36.9, -121.9, 37.0],
    15,  true /* means aggregate! */,
    './osm.mbtiles', function(geojson) {
    console.log(geojson);
});

@batpad @lxbarth :
I've implemented the requirement on this issue : https://github.com/osmlab/osmlint/commit/4a082eef2ee58698aa7d7d5691d6af411a8bd785
to add the options  like here: https://github.com/osmlab/osmlint/blob/dev/test/test.aggregate.js#L12-L16,  the test.aggregate.js works fine.
For cli, https://github.com/osmlab/osmlint/commit/41c9bdf480c7e014a7523bf095301f77cf0edd0b
Next action

[ ]  @batpad  can you do a quick revision?
[ ] @rub21 merge.

cc. @geohacker

@Rub21 - thank you for taking a stab. Though, this seems slightly inefficient to me. We are writing the feature to stdout as well as returning the feature back to the main script. The behavior I'm imagining is:

the map script always return a Feature to the main script.
main script always return a FeatureCollection
cli.js sends a callback, and writes the FeatureCollection to stdout.

@batpad thoughts?

I think the explanation of why the validators just write to stdout and don't return makes total sense. I'm not sure adding the option to return is worth the over-head.
Writing to stdout and returning values is definitely going to be inefficient and we should avoid. We could add an option to either write to stdout OR return, but I would really punt on that.
In my mind, the ideal option would be something like being able to pass a Writeable Stream object to the validator that gets written to. This could be stdout or any writeable stream like object, I would punt on this for now though. @geohacker do you want to just close this?

Sounds good. We can revisit.


Feature request: please add two functions for turning page, forward/backward.

environment
This is a feature request:
In standard vim tab or other editor, such as notepad++, when the tab is full, they have 2 arrow icon for turning page forward/backward, etc,

Please add two functions for turning page, forward/backward. It's useful when the tab bar is full. User can quickly find file name by turning page.
Thanks.

There is a mapping for that function available see my comment here https://github.com/vim-airline/vim-airline/issues/1823#issuecomment-438563714
It is not possible to display that in the tabbar however.

Am 23.11.2018 um 09:54 schrieb 9527KGG<EMAIL_ADDRESS>environment
This is a feature request:
In standard vim tab or other editor, such as notepad++, when the tab is full, they have 2 arrow icon for turning page forward/backward, etc,
Please add two functions for turning page, forward/backward. It's useful when the tab bar is full. User can quickly find file name by turning page.
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

oh ~ Thanks for letting me know.


Subscribe to updates for a particular article or topic area

Subscribe to updates for a particular article or topic area. Receive notification whenever an article or set of articles changes.

⚠ Idea migrated from UserVoice
Created By: Tom Resing
Created On: 2017/11/09 00:04:13 +0000
Votes at Migration: 6
Supporters at Migration: 6

Thanks for your suggestion. I've created a internal work item to track your request.

#log-suggestion

🚀 ATTENTION: Internal request logged.

Hello - reviewing and cleaning up some older feedback/issues. Thanks!


How to set a different note density?

Probably a newcomer's question, but how do I set a different note density? I probably have to set a new argument for performance_controls, but I'm stuck... (alternatively: is there a group I can adress those kind of questions to?)

Which model are you referring to? PerformanceRNN? If so, @iansimon should be able to help you.

First, make sure you're using the density_conditioned_performance_with_dynamics (or multiconditioned_performance_with_dynamics) config, and the corresponding checkpoint available here.
Then, in the call to performance_rnn_generate, set the --notes_per_second flag to the desired note density.  It's quantized to powers of 2, so in essence only responds to doublings or halvings of note density.

Thank you. I was referring to performance_sequence_generator and the colab-notebook. I wanted to set "notes_per_second" in generator_options and failed... it takes the value from control_signals, which I don't know how to adress


KICS not flagging SSO policy

I'm running KICS on my infrastructure, but  inline SSO Policy below is not being flagged as “SSO Policies with Full Privileges” or as an “IAM Policy with Full Privileges”. Is there any reason for this?
module "SSO_permissionset" {
  source = "../../..//modules/permissionset"

  permission_set_name        = "SSO-permissionset1"
  inline_policy_to_attach = local.sso_permissionset_policy
}

# define the inline policy for SSO permission set
locals {
  sso_permissionset_policy = <<EOT
{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Sid": "VisualEditor0",
      "Effect": "Allow",
      "Action": "*",
      "Resource": "*"
    }

permissionset.tf

resource "aws_ssoadmin_permission_set" "permission_set" {
  name             = var.permission_set_name
  description      = var.permission_set_description
  instance_arn     = var.awssso_arn
}

resource "aws_ssoadmin_permission_set_inline_policy" "inline_policy" {
  inline_policy      = var.inline_policy_to_attach
  instance_arn       = var.awssso_instance_arn
  permission_set_arn = aws_ssoadmin_permission_set.permission_set.arn
}

Already in https://github.com/Checkmarx/kics/issues/6457


Update LICENSE

Thank you for fixing that! BTW do you have any experience whether change of formatting in the copyright line will make GitHub unable to recognize the license file? This file was automatically generated by the GitHub

Happy to help :)
I have no experience about it but I can change it back to be sure nothing breaks.


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: dujkbofvrl3he
Region: us-east-1
Step: Domain activation
Status: Failed
Issue/question
I have a particular subdomain for this project that always tries to redirect from https://subdomain.domain.com to https://www.subdomain.domain.com/ — even though i've not set it up that way

Hi @nagey , could you please take a look on your rewrites and redirects section? Anything here?

yep. good to know that the console is putting it in there i suppose. would be neat to have a link from where it says "redirects to" to the redirects panel. I had assumed it was handled within something internally (like the cloudfront distro, for example)


Build error

I am trying to build the project in Android Studio 4.0.1 on a Linux host, following the instructions from BUILD.md, but running into a few issues.
After building BoringSSL and NanoPB, and setting their locations in build.gradle, I got a first error, which I was able to fix with this patch (I'll make a full PR when the build succeeds):
--- a/exposurenotification/build.gradle
+++ b/exposurenotification/build.gradle
@@ -93,7 +93,7 @@ protobuf {
         artifact = 'com.google.protobuf:protoc:3.11.4'
     }
     dependencies {
-        protobuf files("${depsRoot}/nanopb-<IP_ADDRESS>-macosx-x86/generator/proto/")
+        protobuf files("${nanopbRoot}/generator/proto/")
     }
     generateProtoTasks {
         all().each { task ->

Now I am getting the following error:
/home/cgrigis/C4DT/Projects/exposure-notifications-internals/exposurenotification/src/main/cpp/CMakeLists.txt : C/C++ debug|arm64-v8a : CMake Error at /home/cgrigis/C4DT/Projects/exposure-notifications-internals/exposurenotification/src/main/cpp/CMakeLists.txt:62 (add_library):
  Cannot find source file:

    gen/exposure_key_export.pb.c

  Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm .hpp
  .hxx .in .txx

It seems the execute_process() in exposurenotification/src/main/cpp/CMakeLists.txt is not triggered... any ideas?

Oops! Good catch!
As for the build failing, I never had the time to really dig into why cmake sometimes failed, but when I had a build fail, I'd go to the root of the project and run:
find . -name ".cxx" -type d -exec rm -rf {} \;

Then I'd choose Build > Rebuild project. -- It's a pretty big hammer, but it seems to have worked for me? Let me know?

@nic0lette Thank you for your reply!
Unfortunately, the hammer did not make it work for me. :-(
I finally managed to track it down to the following changes:

./gen did not exist, so I force a creation
the protobuf executable is just protoc on non-WIN32 (macOS too I believe)

--- a/exposurenotification/src/main/cpp/CMakeLists.txt
+++ b/exposurenotification/src/main/cpp/CMakeLists.txt
@@ -36,10 +36,14 @@ include_directories(matching ${BORING_SSL_ROOT}/include)
 if(WIN32)
     set(PROTOC "protoc.exe")
 else()
-    set(PROTOC "protoc.exe")
+    set(PROTOC "protoc")
 endif()

 # Generate the NanoPB C source and headers
+execute_process(
+        COMMAND mkdir -p ./gen
+        WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+        )
 execute_process(
         COMMAND ${NANOPB_ROOT}/generator-bin/${PROTOC} -I ${NANOPB_ROOT} -I ${NANOPB_ROOT}/generator/proto/ -I ../proto --nanopb_out=./gen ../proto/exposure_key_export.proto
         WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}

I combined all my changes in #15.

Hey, I wonder if you could take a peek at the latest code pushed up that uses Docker for building BoringSSL (and includes a script for pulling down NanoPB)?

Tried it, and all works well. Thanks. :)

Fixed in da29e8f8ec2a3e970e1a435d5e40013b1a0e6172


[feat] Quest: Vote Quest

Create a quest to check if a user has voted for a new color to add to the color palette. This will be a daily quest, so have the dayIdx be taken as a constructor argument and store to use in the is_claimable function.
Create corresponding tests.
References

Example Quest
Vote metadata

i want to

Can I hop on this? @b-j-roberts

Definitely @princeibs, good luck! Let me know if you have questions.

@princeibs Have you started working on this? If so, could you create a draft PR of your work so far?
Otherwise, I'd like to assign this to someone available to work on it.

Just to confirm, will I include tests for this feature?
@b-j-roberts

@princeibs Can you please submit a PR of what you have done so far, even if its not done? I'd like to get this moving.

I will push my progress on the pr ASAP

If anyone else is willing to help implement this, feel free to submit PRs / work here.

Quest implementation completed, tests is in progress. Kindly check out the draft pr @b-j-roberts

test is completed. pr ready for review


Landed state flight mode (Do not merge)

This pull request is made to allow easier debug to the problem described in the issue https://github.com/mavlink/MAVSDK/issues/885
Please do not merge this pull request.
It might require several taking off with landed state to see the problem (block into infinite while loop)
On the other hand, flight mode will never block in the while loop.
Even if you receive landed state or flight mode asynchronously it will be the same problem
To use the landed sate:
./offboard udp://:14540 landed_state
To use the flight mode
./offboard udp://:14540 flight_mode

Ok, Would you like to keep both of them? the landed state and the flight mode as a use case for the users?

 Sorry!

@julianoes, However, we can replace the sleep by using the flight mode to wait for the takeoff process to be completed.
What do you think of the last commit, I think this can be merged

But this is then just a workaround for the fact that PX4 is not picking up the landed state correctly. However, that will be fixed with https://github.com/PX4/Firmware/pull/13075/commits/f913cc24aec47fab3c55aad35841d41e30a4199d.

Ok, So I will replace it with the flight mode with landed state, then you can merge when it is fixed at PX4 level.


Manage and Get FOXy buttons broke on FOX page

Overview
The "Manage" and "Get FOXy" buttons on the FOX Opportunities page are broke. The clicking on them does nothing.
References and additional details

go to app.shapeshift.com
log in with your preferred wallet
go to the FOX Opportunities page
Click on the FOX tab and click the Manage button
Notice the click does nothing
Then, click on the FOXy tab and click on the Get FOXy button.
Notice the click does nothing

Acceptance Criteria
When clicking on the manage button on the FOX tab and the Get FOXy button on the FOXy tab, a new window should pop up that allows you to manage your FOX stake

.
Need By Date
No response
Screenshots/Mockups

Estimated effort
No response

Will tackle this one


Feedback-Slide (red one) shows when navigating back to index

Always shows up as first slide if you go back to index instead of moving to the next milestone

This slide is actually going away for the I/O, but going to be used after in new codelabs

OO

closing for now


Search only returns lower case results.

Go to the search box. Type "plug". Or type "Plug".  Click on "plugin" and it will take you to topic "plugin (25)", which represents only the lowercase topic name -- there are 53 other items under topic "Plugin".

Thanks @Marxsal I can confirm the issue – the JavaScript that is handling the search doesn't know how to slugify the topics correctly, and is linking to the unslugified version of the title.

This was fixed by the switch to a TW5 UI.


MongoDb Support

Can I use this for MongoDb backend? I use ObjectId instead of Long but am stuck here
        services.AddMultiTenancy<ApplicationTenant, ObjectId>()
            .AddRequestParsers(parsers =>
            {
                parsers.AddChildPathParser("/tenants/");
            });
        //.AddEntityFrameworkStore<IApplicationUserDbContext, ApplicationTenant, ObjectId>();
        services.ConfigureMongoDbIdentity<ApplicationUser, ApplicationRole, ObjectId>(mongoDbIdentityConfiguration);

I’m not familiar with MongoDb myself. You would need to implement the IQueryableTenantStore to read/write tenants from/to MongoDb unless you used some other implementation to store your tenants.
Other than that it should work I would assume. Are you able to fork one of the sample projects and convert it to MongoDb with the minimum number of changes and then let me see what you have?

I have added you as a contributor to the project you can review the projects what am doing wrong I appreciate the help.
Thanks

https://github.com/hakeemoriola/MultiTenancyServer.MongoDb

Try changing your startup code to this:
            // Add Multi-Tenancy services.
            services.AddMultiTenancy<ApplicationTenant, ObjectId>()
                .AddRequestParsers(parsers =>
                {
                    // To test a domain parser locally, add a similar line
                    // to your hosts file for each tenant you want to test
                    // For Windows: C:\Windows\System32\drivers\etc\hosts
                    // <IP_ADDRESS>    tenant1.tenants.local

                    parsers.AddSubdomainParser(".tenants.local");

                    //parsers.AddChildPathParser("/tenants/");
                })
                .AddInMemoryStore(new ApplicationTenant[]
                {
                    new ApplicationTenant()
                    {
                        Id = ObjectId.GenerateNewId(),
                        CanonicalName = "tenant1",
                        NormalizedCanonicalName = "TENANT1"
                    }
                });

Then add <IP_ADDRESS>    tenant1.tenants.local to your C:\Windows\System32\drivers\etc\hosts file.
Then test using http://tenant1.tenants.local/
Let me know if that works and if not what your issues are. You need to give more details.

System.ArgumentNullException: "Value cannot be null.\nParameter name: implementationFactory"
at Microsoft.Extensions.DependencyInjection.ServiceCollectionServiceExtensions.AddScoped[TService,TImplementation](IServiceCollection services, Func2 implementationFactory)\n   at MultiTenancyServer.Configuration.DependencyInjection.TenancyBuilder2.AddTenantStore[TStore](Func2 storeFactory) in C:\\projects\\multitenancyserver\\src\\Core\\Configuration\\TenancyBuilder.cs:line 72\n   at Microsoft.Extensions.DependencyInjection.TenancyBuilderExtensions.AddInMemoryStore[TTenant,TKey](TenancyBuilder2 builder, IEnumerable`1 tenants) in C:\projects\multitenancyserver\src\Stores\TenancyBuilderExtensions.cs:line 29\n   at ApplicationHost.Startup.ConfigureServices(IServiceCollection services) in /Users/hakeemoriola/Documents/GitHub/infusync/infusync.identity/Startup.cs:line 155\n--- End of stack trace from previous location where exception was thrown ---\n   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.ConfigureServices(IServiceCollection services)\n   at Microsoft.AspNetCore.Hosting.Internal.WebHost.EnsureApplicationServices()\n   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Initialize()\n   at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()\n   at ApplicationHost.Program.Main(String[] args) in /Users/hakeemoriola/Documents/GitHub/infusync/infusync.identity/Program.cs:33

Can you get latest and try again? It appears the factory parameter in the service registration is no longer optional after the upgrade from .NET Core 2.0 to 2.2. I have fixed this based on the error you reported but didn't have time to test it.

I have upgraded the library just as you suggested I could say it works based on the Log attached. and I made a few modifications on the code to make it work with MongoDB. the modification I will like to share with you if you don't mind.
if you can help review the log attached and point me if am in the right direction.
right now I can't log in to Identitityserver
Logs.txt

You should use the domain parser as its much simpler to use.
parsers.AddSubdomainParser(".tenants.local")
Then you need to make sure you go to the URL that matches a tenant.

I noticed it's not persisting the Tenants I seeded. Added IEntityBase just for it to be compatible with the repository pattern I am using.
public class ApplicationTenant : TenancyTenant, IEntityBase {
public string DisplayName { get; set; }
public DateTime CreateDate { get; set; }
public DateTime? UpdateDate { get; set; }
public string CreatedBy { get; set; }
}

Yes that is because we are using the in-memory store. If you want persistence then you need to implement the ITenantStore yourself. The point of the in-memory store is for simple testing so you could ensure the tenancy context is set and working in your requests. If that’s working fine now, then you can implement ITenantStore with an MongoDB implementation.


Stop retrying after 24 hours

Trello: https://trello.com/c/Dgf0fbal/703-stop-retrying-for-temporary-failures-after-24-hours
Note: I'm putting this up at the end of Friday as I'm on leave Monday and didn't know if we were in a rush to get this out or not. It may need some additional thought alongside the review if there's any edge cases I've missed.
This sets up a new state on DeliveryAttempt for when they have had 24 hours of temporary_failure statuses from Notify and then ceases to retry. It is a final state.
This isn't a particularly good solution to the problem though as it leaves us with the slightly thorny situation where we can have DeliveryAttempts that have returned as temporary_failure and are marked as different states in the db. This basically loads DeliveryAttempt with accumulative state which is far better served on Email.
I've done some work implementing this into state into email but it will require a couple of deploys so I've put this together as a faster approach to getting the situation looked at. Storing state on Email will resolve a few other complexities with archiving too.

I'm going to close this in favour of the alternative approach.


added responsiveness for college reviews iframes

In index.html, I have updated the code for college reviews youtube iframes and added media queries to make them responsive to different screen sizes.

hey @a-arham-x
Can you provide the link to GitHub pages deployed to your branch.
I would be able to check the changes in a better way.

hey @a-arham-x
Can you provide the link to GitHub pages deployed to your branch.
I would be able to check the changes in a better way.

hey @a-arham-x Can you provide the link to GitHub pages deployed to your branch. I would be able to check the changes in a better way.

Sure. Here is the link to the deployed pages. https://a-arham-x.github.io/Hacktoberfest-Web/

Hey @a-arham-x,
I guess your branch is not deployed correctly,
It's showing 404 error.\
Consider deploying the branch again.
All the best

Hey @a-arham-x, I guess your branch is not deployed correctly, It's showing 404 error.\
Consider deploying the branch again. All the best

I am sorry, For some reason my page is not opening if I comment the link here and gives the 404 error. If you copy and open the url https://a-arham-x.github.io/Hacktoberfest-Web/ in another tab, then it works.


PowerToys Run

Microsoft PowerToys version
58.0
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
2022-05-23.txt
✔ Expected Behavior
to use the alt+space shortcut to search my PC for a document
❌ Actual Behavior
PowerToys Run crashed and generated a prompt for me to file a bug report here
Other Software
No response

System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
   Source: WindowsBase
   TargetAssembly: WindowsBase, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35
   TargetModule: WindowsBase.dll
   TargetSite: IntPtr WndProcHook(IntPtr, Int32, IntPtr, IntPtr, Boolean ByRef)
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

## Environment
* Command Line: "C:\Program Files\PowerToys\modules\launcher\PowerToys.PowerLauncher.dll" -powerToysPid 6784 --started-from-runner
* Timestamp: 05/23/2022 11:45:30
* Wox version: <IP_ADDRESS>
* OS Version: Microsoft Windows NT 10.0.22621.0
* IntPtr Length: 8
* x64: True
* CLR Version: 6.0.4
* Installed .NET Framework:
   * v4 Client 4.8.09032
   * v4 Full 4.8.09032
   * v4.0 Client <IP_ADDRESS>

/needinfo

/dup #18166

For any users that are able to replicate this consistently, can you please update to 0.68.1 and disable thumbnail generation to verify if it fixes the issue? Please let us know on #18166!


Update owlapi-distribution to 4.5.20

Updates net.sourceforge.owlapi:owlapi-distribution from 4.5.15 to 4.5.20.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "net.sourceforge.owlapi", artifactId = "owlapi-distribution" } ]

labels: library-update, semver-patch

Superseded by #115.


Add support for authorizing query context params

Description
The query context is a way that the user gives a hint to the Druid query engine, so that they enforce a certain behavior or at least let the query engine prefer a certain plan during query planning. Today, there are 3 types of query context params as below.

Default context params. They are set via druid.query.default.context in runtime properties. Any user context params can be default params.
User context params. They are set in the user query request. See https://druid.apache.org/docs/latest/querying/query-context.html for parameters.
System context params. They are set by the Druid query engine during query processing. These params override other context params.

Today, any context params are allowed to users. This can cause 1) a bad UX if the context param is not matured yet or 2) even query failure or system fault in the worst case if a sensitive param is abused, ex) maxSubqueryRows.
This PR adds an ability to limit context params per user role. That means, a query will fail if you have a context param set in the query that is not allowed to you. To do that, this PR adds a new built-in resource type, QUERY_CONTEXT. The resource to authorize has a name of the context param (such as maxSubqueryRows) and the type of QUERY_CONTEXT. To allow a certain context param for a user, the user should be granted WRITE permission on the context param resource. Here is an example of the permission.
{
  "resourceAction" : {
    "resource" : {
      "name" : "maxSubqueryRows",
      "type" : "QUERY_CONTEXT"
    },
    "action" : "WRITE"
  },
  "resourceNamePattern" : "maxSubqueryRows"
}

Each role can have multiple permissions for context params. Each permission should be set for different context params.
When a query is issued with a query context X, the query will fail if the user who issued the query does not have WRITE permission on the query context X. In this case,

HTTP endpoints will return 403 response code.
JDBC will throw ForbiddenException.

Note: there is a context param called brokerService that is used only by the router. This param is used to pin your query to run it in a specific broker. Because the authorization is done not in the router, but in the broker, if you have brokerService set in your query without a proper permission, your query will fail in the broker after routing is done. Technically, this is not right because the authorization is checked after the context param takes effect. However, this should not cause any user-facing issue and thus should be OK. The query will still fail if the user doesn’t have permission for brokerService.
The context param authorization can be enabled using druid.auth.authorizeQueryContextParams. This is disabled by default to avoid any hassle when someone upgrades his cluster blindly without reading release notes.

Key changed/added classes in this PR

QueryContext tracks user params and separates them from others.
QueryHolder has a state indicating whether the context in the native query is valid.
QueryLifecycle retrieves context params from a valid source.

This PR has:

[x] been self-reviewed.
[ ] added documentation for new or modified features or behaviors.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[x] added integration tests.

tagging Incompatible because this adds a new method without a default implementation to the Query interface (which is an @ExtensionPoint). Still trying to decide what exactly to do about it.

@clintropolis maybe we can add a default implementation for getQueryContext() that returns null. Callers should check the return value and use getContext() instead if null is returned. When context param authorization is enabled but getQueryContext() returns null, since we cannot distinguish user context params and others, the authorizer authorizes all context params returned by getContext(). What do you think?

@clintropolis maybe we can add a default implementation for getQueryContext() that returns null. Callers should check the return value and use getContext() instead if null is returned. When context param authorization is enabled but getQueryContext() returns null, since we cannot distinguish user context params and others, the authorizer authorizes all context params returned by getContext(). What do you think?

I did something somewhat similar in #12564, except instead of null I returned a specific object and a default implementation method to check for it. I considered null, but ultimately decided to do this instead so that I didn't have to mark getQueryContext nullable, because it didn't seem like legitimate implementations should ever return null. I'm not very attached to the way I did it though, so if you think null is better its more or less the same thing I think.


ctb-ios-changes

See most recent Readme for description: https://github.com/CTBConsulting/Card.io-for-Capital-One-vertical-Cards/blob/dev/README.md

I've added a few extremely minor comments. Otherwise, it looks like this huge PR mostly comprises a couple of large libraries, on which I really can't comment. I hope that they don't increase the shipped size of card.io terribly much...
At first, you might want to branch card.io, with a new branch that incorporates these and future related changes, and make both versions of card.io available to clients.


give reasonable error message when given torch array as imput

model.eval only takes as input numpy or dask arrays. we could allow torch arrays or at least print an understandable error message

implemented


Fix opn and clipboardy on Linux

Two issues:

opn has a dynamically-loaded binary and needs to always be excluded (also added clipboardy)
copy-webpack-plugin doesn't preserve the executable bit when copying node_modules, switch to filemanager-webpack-plugin

@StephenWeatherford Nice.  This will fix the regression in App Service? https://github.com/Microsoft/vscode-azureappservice/issues/857

@StephenWeatherford Nice. This will fix the regression in App Service? Microsoft/vscode-azureappservice#857

Yes.


getting "Can't read from server. It may not have the appropriate access-control-origin settings."

Hi!
I did set up a pretty straightforward example app. I am getting:
"Can't read from server. It may not have the appropriate access-control-origin settings."
from ui all the time. That is somehow realated to CORS which i dont know anything about.
Spring-mvc-app running on tomcat.
Anyone can give me directions?
Regards!

Hi,
I'm using springfox-swagger2 and springfox-swagger-ui version:2.5.0 with spring boot 1.3.6.RELEASE
when i set this property:spring.jackson.serialization.wrap-root-value=true ,i get "Can't read from server. It may not have the appropriate access-control-origin settings." too.

@AbsoluteTiger the issue you're seeing is a duplicate of #597


use expect so clippy does not complain

Currently, when using #[br(count = x)] above a field in a struct, clippy will complain that unwrap is being used (within the macro). No variation of #[allow()] seemed to silence the problem.
This happens because #![warn(clippy::unwrap_used)] is being used throughout the crate I'm working on - it's annoying but helps catch some (potential) issues, so I figure a fix here will likely be the best solution (and hopefully prevent this issue happening to others, too).

Thank you for the patch and sorry for the delay! Since panicking is a very unpleasant thing to do, I’ve implemented an alternative approach that eliminates any panicking unwrap entirely by returning an error for out-of-range count values instead, in #198. Give it a try and let me know how it works!


WIP - Development of LMEst 6 DoFs code, using the standards set by ROSS

Addition of a new class for 6 degrees of freedom FEM elements, adding this capability to ROSS. This will be implemented as per the standards defined by Friswell, following his coordinate system and the tapered element used by ROSS.
No pre nor post-processing or accessory codes will be modified. Only additional classes for the 6 DoF elements (shaft, discs, etc.) will be added inside the appropriated Python codes.

Codecov Report

Merging #447 into master will decrease coverage by 0.21%.
The diff coverage is 56.09%.

@@            Coverage Diff             @@
##           master     #447      +/-   ##
==========================================
- Coverage   76.31%   76.09%   -0.22%
==========================================
  Files          15       15
  Lines        3985     4025      +40
==========================================
+ Hits         3041     3063      +22
- Misses        944      962      +18

Impacted Files
Coverage Δ

ross/shaft_element.py
89.64% <56.09%> (-5.16%)
:arrow_down:

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 49b90fe...3792691. Read the comment docs.

According to what has been agreed upon, the code for the 6 DoF will follow the coordinate system and conventions adopted by Friswell. Being so, the element-level FEM matrices will be defined considering the following local coordinate vector:
:math:`[u_0, v_0, w_0, \theta_0, \psi_0, \phi_0, u_0, v_0, w_0, \theta_0, \psi_0, \phi_0]^T`

Being the following their ordering for an element:

    :math:`x_0,u_0`  - horizontal translation;
    :math:`y_0,v_0`  - vertical translation;
    :math:`z_0,w_0`  - axial translation;
    :math:`\theta_0` - rotation around horizontal, bending on the yz plane;
    :math:`\psi_0`   - rotation around vertical, bending on the xz plane;
    :math:`\phi_0`   - torsion around axial, z direction.

Still to do:

fix the matrices coordinate system and addition of Timoshenko shear into Kc matrix as an additional matrix, then, enter the Kc  matrix;
enter the mass matrices;
enter the gyroscopic matrices;
enter the other stiffness matrices;
create the additional defs for extra matrices from Lalanne's formulation.

Updating To-Do list:
 - Manually test the shaft testing function;
 - Debug and test the shaft_element for the 6DoF
 - Move on to the disks and bearings sub-routines

Next steps...

Manually testing the shaft 6DoF model
Disk testing function implementation for the 6DoF model
Updating the bearing models for 6DoF

if the shaft model tests go well, the rest should be considerably easier to finish.

Coming back from 6 DoF implementation using Lalanne's coordinate systems for LMEst Petro EDGE project.


Define mutation permissions in Meta class

Now that PR #3891 is merged we can think of further improvements to our mutations code. One thing that I noticed while doing the refactoring is that almost every mutation defines user_is_allowed method and in most cases, they perform the same logic of checking if a user has a particular permission. What do you think about refactoring it so that we could define permissions at the Meta class level, e.g.:
class CollectionRemoveProducts(BaseMutation):
    class Meta:
        description = 'Remove products from a collection.'
        permissions = ['product.manage_products', ]

At the same time, we could keep the user_is_allowed as an internal method used in BaseMutations. It could be overridden if you needed to perform some additional checks e.g.:
@classmethod
def user_is_allowed(cls, user, input):
    return user.is_authenticated

(example taken from LoggedUserUpdate mutation).
The default behavior of user_is_allowed would simply do user.has_perms([permissions]) where permissions would come from the Meta class.
Such changes would allow us to reduce the number of lines of code per mutation even more.
What do you think @akjanik  @korycins @NyanKiyoshi?

I like this suggestion. We could have a meta field login_is_required = False (default) for the cases where permissions = [] (any) and the user is required to be authenticated (or not).

At the meta class level, I would only add the permissions field as this is the most common case. login_is_required as a meta class field limited to a single case; if we needed to also check is_staff, we would have to add additional meta class field staff_is_required. All of that could be achieved by overriding user_is_allowed and doing both checks manually.

Alright, sounds good to me :+1:

Good to me.
What do you think about changing the name user_is_allowed to something like user_has_perm or check_permissions?
I think that this will be more meaningful and more explicitly pointing to permissions object.

check_permissions sounds good 👍

We will need to have #3942 merged first so we don't end up in a big mess!


Implement missing functions in Date

The two NSUnimplemented functions in Date relate to a class property
timeIntervalSinceReferenceDate which matches an instance property
of the same name, and methods relating to the unkeyed coder.
Implement the timeIntervalSinceReferenceDate by instantiating a
new instance and delegating to the instance property, and change
the unkeyed coder implementations to a fatalError indicating that
they are unsupported.

@swift-ci please test


Avoid to return nil failure message

What this PR does / why we need it:
Avoid return nil failure message like following:
lastProbeTime: <nil>
lastTransitionTime: 2022-03-10T03:19:06Z
message: Failure attempting to delete VMI: <nil>
reason: FailedDelete
status: True
type: Failure

Logs in virt-controller:
{"component":"virt-controller","kind":"","level":"info","msg":"Stopping VM with VMI in phase Running due to runStrategy: Halted","name":"test-stop","namespace":"default","pos":"vm.go:725","timestamp":"2022-03-10T03:19:06.757895Z","uid":"598f466b-0932-4998-b316-e93710635251"}
{"component":"virt-controller","kind":"","level":"info","msg":"Dispatching delete event for vmi default/test-stop with phase Running","name":"test-stop","namespace":"default","pos":"vm.go:991","timestamp":"2022-03-10T03:19:06.773812Z","uid":"598f466b-0932-4998-b316-e93710635251"}
{"component":"virt-controller","kind":"","level":"error","msg":"Reconciling the VirtualMachine failed.","name":"test-stop","namespace":"default","pos":"vm.go:306","reason":"Failure attempting to delete VMI: \u003cnil\u003e","timestamp":"2022-03-10T03:19:06.773870Z","uid":"598f466b-0932-4998-b316-e93710635251"}
{"component":"virt-controller","level":"info","msg":"re-enqueuing VirtualMachine default/test-stop","pos":"vm.go:199","reason":"Failure attempting to delete VMI: \u003cnil\u003e","timestamp":"2022-03-10T03:19:06.803926Z"}
{"component":"virt-controller","kind":"","level":"info","msg":"Stopping VM with VMI in phase Running due to runStrategy: Halted","name":"test-stop","namespace":"default","pos":"vm.go:725","timestamp":"2022-03-10T03:19:06.803985Z","uid":"598f466b-0932-4998-b316-e93710635251"}
{"component":"virt-controller","kind":"","level":"error","msg":"Reconciling the VirtualMachine failed.","name":"test-stop","namespace":"default","pos":"vm.go:306","reason":"Failure attempting to delete VMI: \u003cnil\u003e","timestamp":"2022-03-10T03:19:06.804002Z","uid":"598f466b-0932-4998-b316-e93710635251"}

Which issue(s) this PR fixes:
Fixes #7339
Special notes for your reviewer:
Reproduce steps:

Create a VM with runStrategy: RunStrategyAlways.
Stop the VM.

Release note:

NONE

/ok-to-test

/lgtm
/approve

/cherrypick release-0.49


listed_on_gfonts_api always warn

This check is always popping, even when the font is already published on GF.
⚠ WARN Family not found via Google Fonts API. [code: not-found]

I didn't see this check for a while but appeared in this PR: https://github.com/google/fonts/pull/4614#issuecomment-1170184428
I don't understand this check tbh, can the rational be more explicit or can we just disable it?
cc @m4rc1e @vv-monsalve

Please let's just disable the check, it warns for any new font and it doesn't make sense.

thanks, I'll try to address it this week


Geth nitpicks

Version numbers usually start with a lowercase v
I'm unsure if it's a good idea to list 32 bit downloads. A 32 bit binary by definition can only access 4GB of RAM (or 2, depending on OS), which is generally fairly little for mainnet. As such, my suggestion would be to never even offer the user the possibility to install 32 bits on a 64 bit platform (and to warn them on 32 bit ones that it's not suitable for mainnet).

This is a harder nut to crack: If I switch the network from mainnet to Rinkeby, the UI leaves the default datadir in place (~/.ethereum), which is wrong (it should be ~/.ethereum/rinkeby (same for the keystore)).
My suggestion would be to not set these two folders by default, unless the user overrides them explicitly. That way Geth can remain smart and use its default locations.

thanks for testing and reporting. we'll clean those up shortly.

@evertonfraga if we can keep dataDir and keystoreDir fields in settings but have it default be empty and not pass the flag in if empty, that would be a good solution

captured here: https://github.com/ethereum/grid/issues/538