Batch unwatch test issue fixed

Changes made:

Created a ReactiveNode interface to specify the structure of nodes being processed;
Added a type guard function 'isReactiveNode' to check if a given object matches 'ReactiveNode' interface;
Refactored the loop to use 'useReactiveNode' and validate properties ('wrapper' and liveConsumerNode) before processing.

Yes, the wrapper.ts had some alterations for passing this specific test, but they also ensure that the core functionality works correctly

Does this change do the same as #26? Should it be landed?

@ds300 Just to clarify, I've tested these changes thoroughly, and they don't introduce any new issues. These sanity checks are not meant to hide real errors but to prevent the feature from breaking when it encounters unusual situations.

@Roeck continuing here to avoid clogging up the other PR with discussion about at this one.
Your PR makes it so that legitimate (if very unlikely) errors would be silently ignored. But that's not what we want. If any of those sub conditions in your if condition are false then it's an error in the core signals library and we should throw. You could change them to assertions that throw, and that would be ok to merge and keep during this prototyping phase, but ultimately those are things that can and should be tested by unit tests rather than sanity checks at run time.

Landed https://github.com/proposal-signals/signal-polyfill/pull/26 instead


fix: upgrade to version 3.0.663

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

Needs approval from an approver in each of these files:

OWNERS

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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
3ec705aff4fb4da863ff6250f3cb47bb558df256
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/test kube


Operators in test function don't have highlighting

Using sqaure brackets shows highlighting for operators, but using the test function removes this.

Example code

Treesitter Playground output

Adjusting the test_command grammar rule would do the job:
test_command: $ => choice(
    < bracket syntax rule >,
    seq(
        'test',
        optional($._test_expression),
     ),
),

I have my doubts whether it should be done, [ ] syntax for test is a special case just like in case of bash grammar. I was kinda bored and added rules for the test command arguments.
Anyway, making an exception in this case should be all right. I'll add the rule and test cases this week if no-one does it first :]

@ram02z I added the change you requested: https://github.com/krnik/tree-sitter-fish/commit/1c25205cd7a68434bd1e576972193b3994404307
Now the input: test 0 -eq 1 will be represented as:
(test_command
  (binary_expression
    (integer)
    (test_option)
    (integer)))

Let me know if it works for you ; ] And feel free to close the issue if it does.

@krnik Thanks for the change. I had an issue updating it through TSUpdate. It seemed to not want to get the latest commit and was stuck on 3e31952b0ef7454b2c52af8e60e4db9b6f43c82f. So, I manually compiled it and got the following error:
The 'gyp==0.1' distribution was not found and is required by the application.
This is probably an issue on my machine becuase once I installed npm-gyp, it compiled fine. Also, the compiled file size went up from 86.2k to 228k.
As for the change, the highlight now appears. The example you gave, test 0 != 0, should probably not be parsed the same way since the operators = and != are meant to be used on strings, but they do work on integers. Though, it may be best practice to invalidate it.

nvim-treesitter has hardcoded commit hash of all grammar repos. It's automatically updated from time to time.
In order to test the newest build of some grammar it's necessary to compile it manually.
In the end, I will hold on with parsing = and != as a test_option. It's fine the way it is for now :]


ssh-askpass-fullscreen pegs CPU when used

Describe the bug
When executing ssh-askpass-fullscreen either via the SSH_ASKPASS environment variable, or executing it directly results in 100% cpu usage by ssh-askpass-fullscreen. This happens in x11 and wayland (using kde plasma 6)
Steps To Reproduce
Steps to reproduce the behavior:

echo hi | ssh-askpass-fullscreen
Run top in another terminal.
Watch CPU fly.

Expected behavior
I expect to see the full screen popup and not hear the fans whir
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Running from the command line you can see tons of:
(ssh-askpass-fullscreen:15096): GdkPixbuf-CRITICAL **: 12:59:13.960: gdk_pixbuf_composite_color: assertion 'GDK_IS_PIXBUF (src)' failed

printed out.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.30, NixOS, 24.05 (Uakari), 24.05.20240506.f9ae6eb`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.21.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/store/28gbsahwg0xy03rap0c1b8k7ay22yymk-source`

Add a :+1: reaction to issues you find important.

My nixpkgs is at f9ae6eb42758fcf324018b025527937c1d553789

issue reproduced on Hyprland as well.
running git push when not logged into git causes my machine to require a hard restart due to this issue.


RDKUI-469,RDKUI-470 - code changes in controller UI to support frontp…

…anel and securityagent plugins.
signed off by<EMAIL_ADDRESS>

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

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/ThunderUI/92/rdkcentral/ThunderUI

Commit: 179ed1c6751920f3cdb07a16901818af59b4c977

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Not a problem

Commit: 179ed1c6751920f3cdb07a16901818af59b4c977


Configure OTLP metrics exporter with environment variables

https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk_exporters/otlp.md
OTLP exporter spec defines that the OTLP exporter should be automatically configured with a PeriodicMetricReader when using OTEL_METRICS_EXPORTER. Their aggregation and temporality preferences should also be configured with OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE and OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION.

I saw you added this to the metrics GA milestone. Do you believe this is blocking the metrics SDK?

No, I believe it's not blocking but rather good to have.

Just went through the code and saw that the following env-vars from this spec are also not implemented right now:

OTEL_EXPORTER_OTLP_METRICS_COMPRESSION (currently the trace setting is used instead)
OTEL_EXPORTER_OTLP_METRICS_INSECURE (currently the trace setting is used instead)
OTEL_EXPORTER_OTLP_METRICS_CERTIFICATE (currently the trace setting is used instead)
OTEL_EXPORTER_OTLP_METRICS_CLIENT_KEY (currently the trace setting is used instead)
OTEL_EXPORTER_OTLP_METRICS_CLIENT_CERTIFICATE (currently the trace setting is used instead)

Not stale as we'd need to implement OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION after #3506 lands. :slightly_smiling_face:
(we also need to address the comment I made earlier)

When this is done, we'll also need to update the spec compliance matrix with the newly supported env vars.

Just went through the code and saw that the following env-vars from this spec are also not implemented right now:
* `OTEL_EXPORTER_OTLP_METRICS_COMPRESSION` (currently the trace setting is used instead)

* `OTEL_EXPORTER_OTLP_METRICS_INSECURE` (currently the trace setting is used instead)

* `OTEL_EXPORTER_OTLP_METRICS_CERTIFICATE` (currently the trace setting is used instead)

* `OTEL_EXPORTER_OTLP_METRICS_CLIENT_KEY` (currently the trace setting is used instead)

* `OTEL_EXPORTER_OTLP_METRICS_CLIENT_CERTIFICATE` (currently the trace setting is used instead)

This should work now as of #4971 🙂
I think we're only missing OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION now.


add .button.outline.error

The outline error button are very useful for a non-primary action that could trigger a dangerous action.
For example, this Github Pull Request form:

We already have a danger button. Is there a need for an outline button specifically?

Yes sorry about that,
I'm a big fan of the minimalism approach of this CSS Framework so I completely understand your reluctance.
I previously tried to use
<button class="button danger outline">cancel</button>

But this gave me :

Note that other combo (dark.outline, primary.outline, secondary.outline), was working.
So I had to create the .button.danger.outline rule, so I could get:

Also this unify the button usage (any color can have an outline variance)
Finally, I used the outline variance because in think:

"error + primary" buttons can be used if both choice have the same weight (ex: OK/Cancel)
"error.outline + primary" button can be used if the dangerous is secondary (ex: Delete/Save) == The delete button is not supposed tu be used as often as the "Save" button.

Yes, look here.
The only outline variants supported are default, primary, secondary and dark.

Any reason to support default, primary, secondary and dark but not error ?

yeah, since error and success are states whereas other's are just levels of content

I just added more utilities for text, background and border which can be used to achieve what you are looking for!
https://jenil.github.io/chota/#utilities

Much appreciated
using button.text-error.bd-error I get a grey background

using button.text-error.outline the background is good again

I also tried button.text-error.outline but I get a double red+grey border

I see. That's because the button uses an inset box-shadow.
I'll update the buttons to use a border in that case.

Published v0.6.1


Thanks. I think the issue has been fixed in my Ubuntu after recreate the environment.

          Thanks. I think the issue has been fixed in my Ubuntu after recreate the environment.

Originally posted by @ChenYuYeh in https://github.com/intel/intel-npu-acceleration-library/issues/108#issuecomment-2254193505

could you post the details of your fix? I have the same issue. Only matmult works, the rest of the examples gave the error:
ImportError: cannot import name 'CompilerConfig' from 'intel_npu_acceleration_library.compiler'
I followed the instruction to install intel-npu-acceration-lirbary using pip.
thanks

I did a pip install from repo_root :+1:
pip install "intel-npu-acceleration-library @ git+https://github.com/intel/intel-npu-acceleration-library.git"
and still the same error

I'm having the same problem...

Same here


SSL Certificate for bfnightly.bracketproductions.com expired

Was going through the tutorials last night when the certificate expiration hit 1/14/2020.

Oops! It turns out that Let's Encrypt update scripts work better when you actually remember to add them to cron. Fixing that up now - thanks!

This should be fixed now. It turns out it was in cron, but the cert system was trying to auto-upgrade, and the security settings on that box didn't allow it.

Ok, I've received confirmation that it works outside of my admin network. Thanks again!


sasl authentication

How do I authenticate using SASL with the library? We have zook servers that are on SASL and need jaas conf supplied at the moment....

@samuel Even i am trying to figure out how to authenticate using SASL. Does the library support that?

I dont think it does ... I ended up building a scala client to zook and exposing a akka tcp endpoint to wrap around the problem

Repo owners - Please respond to issues open so atleast we get a quick idea on where the project stands. I am closing this issue as there has been no update for 3 years. I have since moved on and was able to complete the project differently.


Create 0000-01-02-Amal5G.md

created a branch, created a file and made a commit, and opened a pull request

Close the first course comfortable


Board is not selected

I am looking to use CLion instead of the Arduino IDE to run code on my ESP32 (It is a custom one using the ESP32-S3 chip). I added the boards.txt file that is used in the Arduino IDE to get the correct board (ESP32-S3 Dev Module). However, whoever I try to make a new sketch, the create button is disabled and I see "Board is not selected" at the bottom (see screenshot). What am I supposed to do? I was able to create a new sketch previously before I downloaded the Legacy Arduino IDE for the Arduino SDK. I am on an M1 Pro mac if that helps.
FYI I need to use Arduino not PlatformIO because the club uses it and I don't want to try anything different.

@vsch Sorry for the ping but I needed urgent help and I've been pulling my hair over this error for a few hours now.

I have this issue as well.

I have also this issue


[mobile]Initialize booleans in Apple proxy resolvers

Commit Message: Initialize booleans in Apple proxy resolvers
Additional Description: This fixes issue where the resolver thinks it's already started and system proxy setting is not read.
Risk Level: Low
Testing: existing tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/assign @RyanTheOptimist


Encoding issue

GitLens Version:9.2.3
VSCode Version:1.30.1
OS Version:Windows 10

My files is big5 encoding (Not UTF-8), when I using gitlens view history, the encoding will change to UTF-8.
I have to change it by myself, but previous version is auto guess my file encoding.
I have setting vscode setting.json
"files.autoGuessEncoding": true

Steps to Reproduce:

In file history have this issue

but it is normal in vscode git viewer

it's normal in version 8.5.6

I have comment at this issue but no one see it
https://github.com/eamodio/vscode-gitlens/issues/40#issuecomment-447181307

Should I provide any information?

@jjasoncool sorry I haven't had a chance to look into it yet

after version 9.8.1 this issue has been fixed


Broken build?

Hi,
sorry, I'm having problems today trying to use your libary:
➜  ~  npm install react-mentions
<EMAIL_ADDRESS>node_modules/react-mentions
➜  ~  node
> require('react-mentions')
Error: Cannot find module 'react-mentions'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at repl:1:2
    at REPLServer.self.eval (repl.js:110:21)
    at Interface.<anonymous> (repl.js:239:12)
    at Interface.emit (events.js:95:17)
    at Interface._onLine (readline.js:202:10)
    at Interface._line (readline.js:531:8)

Have you broken the build maybe? I couldn't use it in two different hosts.

It works for me:
➜  ~  npm install react-mentions
<EMAIL_ADDRESS>node_modules/react-mentions
➜  ~  node
> require('react-mentions')
{ MentionsInput: ...
  Mention: ... }

Did you just upgrade to version 0.1.4? In that case, you could try if version 0.1.3 still works for you: npm install<EMAIL_ADDRESS>Anyways, I would assume some misconfiguration on your end.

Yes, that worked when I executed npm cache clean and then install.
Did you try with npm install git://github.com/effektif/react-mentions#custom-regex and having the cache clean? That seems to be broken, also with master.

Ah alright, you were already using the branch.... Thanks for the hint, I just fixed the module's main in the package.json.

Thanks @jfschwarz, now it works ;-).


移除方法名中的Cat

那次只改了V2中的代码，现发现NotificationController.java也有同样的代码。

Coverage increased (+0.05%) to 50.224% when pulling 6715b77e1bc280a261bd0584514142e2083f1b24 on powerYao:master into f6d4da92ab11a2d8ba01b2cbf1f1649166d71b86 on ctripcorp:master.

Codecov Report

Merging #689 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #689      +/-   ##
============================================
+ Coverage      46.7%   46.75%   +0.04%
  Complexity     1481     1481
============================================
  Files           341      341
  Lines          9392     9392
  Branches        928      928
============================================
+ Hits           4387     4391       +4
+ Misses         4680     4675       -5
- Partials        325      326       +1

Impacted Files
Coverage Δ
Complexity Δ

...nfigservice/controller/NotificationController.java
89.09% <100%> (ø)
12 <0> (ø)
:arrow_down:

...ervice/service/ReleaseMessageServiceWithCache.java
85.71% <0%> (+1.19%)
24% <0%> (ø)
:arrow_down:

...work/apollo/biz/message/DatabaseMessageSender.java
51.06% <0%> (+6.38%)
6% <0%> (ø)
: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 f6d4da9...6715b77. Read the comment docs.

Coverage increased (+0.05%) to 50.224% when pulling 609eae4ec349fc4053afd064eed44fdead7f2884 on powerYao:master into f6d4da92ab11a2d8ba01b2cbf1f1649166d71b86 on ctripcorp:master.

the logWatchedKeysToCat method name is fixed in a previous pr - #688
as to the null check for namespace, I don't think it will be null, can you give an exact case?

NotificationController.java 还没有改，null检查如果一定不为空，可以不用处理，谢谢

NotificationController.java昨天在 #688 中已经改了~
config controller中的namespace参数一定是不为空的，因为是从path variable中映射出来的~
Thanks for your pull request anyway!

好的，哈哈


How do I build a base class and extend or overwrite attributes?

Hi there,
Thanks for this gem, it makes such a difference to my codebases :)
I have a problem I'm not sure how to solve. I have two very similar Contact classes... one is a sender, and the other is a receiver.
About 80% of their attributes are identical, except for the following differences:

Some attrs are required on the sender, and not on the receiver, and vice-versa
The receiver has 1 extra attr the sender doesn't have
The sender has 2 extra attrs the receiver doesn't have

I attempted to make a BaseContact class that defines the common attrs and add fairly lax validators to it, I then attempted to inherit from the BaseContact, and then redefine the changed attrs using a new for_attributes block.
Unfortunately, it looks like doing that completely blows away the previous definition in the base class.
I also tried making calls to ValueSemantics::Attribute.define but that didn't work either...
I really don't mind repeating myself here, but this was a headscratcher for me, and I was wondering if you had a solution or some guidance?

Hi Gabriel,
It's a little bit tricky to get this gem to work with inheritance because it defines the #initialize method, which would have to call super with a hash of attributes, but most of the time super on the parent class takes zero arguments so it would need to manually check the inheritance hierarchy first to decide what to do. There are a few other difficulties too. It's a whole mess.
However, the gem was designed with this use case in mind, although there is no nice DSL for it yet. There are a couple of ways you could do it.
Probably the easiest way is to instance_eval a common block. It's not particularly pretty, but I think it should work. It would look something like this:
BASE_CONTACT_ATTRIBUTES = Proc.new do
  name String
  address Address
end

class SenderContact
  include ValueSemantics.for_attributes {
    instance_eval(&BASE_CONTACT_ATTRIBUTES)
    postage_cost Money
  }
end

The other option is to manually build ValueSemantics::Recipes for the classes, then use ValueSemantics.bake_module to convert them into a Module to include. That would look something like this:
# ValueSemantics::DSL.run returns a ValueSemantics::Recipe
BASE_CONTACT_RECIPE = ValueSemantics::DSL.run do
  name String
  address Address
end

class SenderContact
  include ValueSemantics.bake_module(
    ValueSemantics::Recipe.new(
      attributes: BASE_CONTACT_RECIPE.attributes + [
        ValueSemantics::Attribute.define(:postage_cost, Money),
      ]
    )
  )
end

That has a lot of ugly boilerplate.
To implement it as proper feature of the gem, all that boilerplate just needs to be shoved under the hood, and we could have something like:
BASE_CONTACT_ATTRS = ValueSemantics.shared_attributes do
  name String
  address Address
end

class SenderContact
  include ValueSemantics.for_attributes(extending: BASE_CONTACT_ATTRS) {
    postage_cost Money
  }
end

# Or maybe...

class SenderContact
  include ValueSemantics.for_attributes {
    include_attributes! BASE_CONTACT_ATTRS

    postage_cost Money
  }
end

I might look at adding this feature next week, when I have some free time.


Handle atom proxy authentication

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
There is a lot of pain associated with proxy configuration and proxy authentication in atom. So the request is to do something like this: https://github.com/Microsoft/vscode/pull/22369 to handle proxy auth in atom.
Additional Information
Atom and vscode both are built on electron and js so it is possible to provide global proxy authentication for atom as well. It will reduce time and frustration associated with it.

Thanks for the feedback, we'll pass this along to the maintainers team - we'll go ahead and close this since we have more specific issues open related to proxies as mentioned in the Teletype issue (https://github.com/atom/teletype/issues/236#issuecomment-364118183) and because we treat our issues list as the Atom team's backlog.


Inactive airlines have wrong colour until vimrc sourced or terminal window resized

Inactive airline bars (but not vertical window separators) blend into the background when creating new windows/splits. The colours are corrected by resizing the terminal window and by :source ~/.vimrc
Before sourcing vimrc:

After sourcing vimrc:

looks like some colors are not correctly setup. How to reproduce?

ping

closing as duplicate of #58


Contracts that use libraries are not sized accurately

Hi @ItsNickBarry! Love this plugin
When sizing a contract whose bytecode includes library address placeholders, e.g. __$....$__, it seems like the size measurement is incorrect - it only returns size up to the first placeholder entry.
A repro based on how length is measured here is:
const a = "aaaa__$aaaa$__aaaa";
const b = "bbbbbbbbbbbbbbbbbb";

a_bufferLength = Buffer.from(a, 'hex').length;  // Issue w/ non-hex chars?
b_bufferLength = Buffer.from(b, 'hex').length;

a_stringLength = a.length;
b_stringLength = b.length;

console.log(`a_bufferLength: ${a_bufferLength}`);
console.log(`b_bufferLength: ${b_bufferLength}`);
console.log(`a_stringLength: ${a_stringLength}`);
console.log(`b_stringLength: ${b_stringLength}`);

> a_bufferLength: 2  // Short
> b_bufferLength: 9  // OK
> a_stringLength: 18
> b_stringLength: 18

Does #2 correctly resolve this?  I'm not especially familiar with libraries.

Yeah! Looks good. I swapped that change into my local node_modules copy and got what I was expecting.

Fixed in version 2.0.3.


webui: add links to external resources

Link to ATT&CK framework definitions:

Link to MBC catalog entries:

Link to capa rule website entries (final URL pending, insert placeholder value for now):

(and remove "source" column)
Link SHA256 to VirusTotal:

also #2204

Link to ATT&CK framework definitions

@williballenthin MITRE ATT@CK TTPs alreay link to the MITRE project?

ATT&CK, yes (my apologies), MBC no.
Lets add an underline on hover (or other indicator) so that it's obvious that these are links. I didn't realize this until I tried to click the data.

Lets add an underline on hover (or other indicator) so that it's obvious that these are links. I didn't realize this until I tried to click the data.

There is already a cursor pointer on hover, I will try to maybe highlight the links with blue color to indicate they are clickable.

I'm not sure if the links from the rule title to the repo are the most intuitive/user-friendly. I'd prefer a separate icon (as implemented currently) to show the full rule content and (another one) to link to the repo.
I'd expect that clicking the rule name in this view would expand the details (like clicking on the > now).

I agree with @mr-tz here, I think having the rules titles clickable will cause a lot of misclicks when trying to expand a tree node resulting in accidental redirects to capa-rules which is not a good UX.

I'd expect that clicking the rule name in this view would expand the details (like clicking on the > now).

I thought of this as well. It is not built-in, and needs to be manually implemented.

@williballenthin

Link SHA256 to VirusTotal

I am thinking of adding a small VT icon next to the hash, maybe also include hybrid-analysis, OPSWAT Metadefender? Once clicked they will respectively pivot to:
www.virustotal.com/gui/file/<SHA256>
www.hybrid-analysis.com/sample/<SHA256>
https://metadefender.opswat.com/results/file/<sha256>/hash/multiscan

@williballenthin

Link SHA256 to VirusTotal

I am thinking of adding a small VT icon next to the hash, maybe also include hybrid-analysis, OPSWAT Metadefender? Once clicked they will respectively pivot to:

www.virustotal.com/gui/file/<SHA256>

www.hybrid-analysis.com/sample/<SHA256>

https://metadefender.opswat.com/results/file/<sha256>/hash/multiscan

Let's not link to other 3rd party sites for now.

MBC, ATT@CK and SHA256 now redirect to external resources.
Toggling the node when the title is clicked will be added to the tracker as P2.
Closed as completed.


Always record request having Pinpoint-Force-Sample request header

If the http request have Pinpoint-Force-Sample request header and the request path is not excluded, always record the request regardless of sampling rate.
This feature is off by default for the security reason & backward compatibility, can be enabled by profiler.sampling.force.header=true.  We could turn this feature on in development environment(dev profile) and off in production environment(release profile).

Codecov Report

Merging #6823 into master will increase coverage by 0.01%.
The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master    #6823      +/-   ##
==========================================
+ Coverage   39.13%   39.14%   +0.01%
==========================================
  Files        3159     3161       +2
  Lines       94168    94185      +17
  Branches    11810    11811       +1
==========================================
+ Hits        36854    36873      +19
+ Misses      54346    54341       -5
- Partials     2968     2971       +3

Impacted Files
Coverage Δ

...m/navercorp/pinpoint/bootstrap/context/Header.java
83.33% <ø> (ø)

...t/bootstrap/plugin/request/RequestTraceReader.java
76.27% <0.00%> (-2.68%)
:arrow_down:

...oint/profiler/sampler/HintSamplingRateSampler.java
75.00% <75.00%> (ø)

...rcorp/pinpoint/bootstrap/sampler/SamplingHint.java
85.71% <85.71%> (ø)

...corp/pinpoint/profiler/sampler/SamplerFactory.java
100.00% <100.00%> (ø)

...p/pinpoint/rpc/client/WriteFailFutureListener.java
56.25% <0.00%> (-18.75%)
:arrow_down:

...pc/server/LoggingServerMessageListenerFactory.java
90.00% <0.00%> (-10.00%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70.00% <0.00%> (-3.34%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.07% <0.00%> (-0.98%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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

@yjqg6666
Hi, I agree with your opinion.
I think pinpoint needs a debugging header for the development environment.
However, it is difficult to change Sampler in the current API.
First of all, I plan to add an API for sampling policy.

I'm thinking of two ways
Method 1
Add manual API to TraceContext.
  -TraceContext::newForceTraceObject () Trace
  -TraceContext::newDisableTraceObject () Trace
Method 2
Add a new Sampler API to understand RequestContext.
public interface ReqeustContextSampler {
       boolean isSampling (RequestAdaptor <T>, T);
}

public class DebuggingContextSampler implements ReqeustContextSampler {
       boolean isSampling (RequestAdaptor <T> ra, T v) {
               if (ra.getHeader ("Pinpoint-Debugging-Sample ", v) == "true" ) {
                       return true;
               }
       }
}

public class TraceContext {
     ~~~
      // Add new API
     Trace newTraceObject(RequestAdaptor<T> ra, T v);

     // Trace newTraceObject(RequestAdaptor<T> ra, T v, Options extendedOption);
}

I think it's good to add a new Sampler API, but I didn't have enough time to try.
Could you implement it?

@emeroad Actually our usage for this feature would be that if the controller have some specific annotations, the request calling the controller would be always recorded regardless of environment.
We would scan the annotations before servicing and put them into a list,  when the request come, pinpoint intercept the request and check for tracing, if true then force tracing.
Recent i may not have time for that，i have to complete the rest.

If using the method 1, the problem would be we have to add newForceAsyncTraceObject, newForceTraceObject, newDisableTraceObject. Agree with you.
https://github.com/naver/pinpoint/blob/ce0c257122a621f64d578b6258627c7d75c91333/bootstrap-core/src/main/java/com/navercorp/pinpoint/bootstrap/plugin/request/RequestTraceReader.java#L132-L137
Prefer method 2, we can decide whether sampling or not based on request, ex. request header, request path, request host, request cookie.

Actually our usage for this feature would be that if the controller have some specific annotations, the request calling the controller would be always recorded regardless of environment.

I think this is a great feature

@emeroad @RoySRose I make another PR #6834 for this.  We may refactor the internal implementation, now it's just working as expected.


Update MFAorNPS function so changes are not made untill pressing ENTER

The script should not make changes to the registry prior to the prompt that describes what it is doing. If the user presses Ctrl+C to cancel the script, changes would have already been made to the registry which are not obvious. This contradicts the "Press ENTER to continue"... message.

@syst3m4 , thanks for your reply.
There is an ongoing new script version to reflect provide logic about registry keys.
#please-close


ERROR A stack overflow in task eventTask has been detected.

Hi Rob, I get a stack overflow but not always.
What is the best way to proceed?
As a reminder, I use ESP32

Hi Alex: searching the code for eventTask, that would seem to be the UART callback task:
https://github.com/u-blox/ubxlib/blob/ffa9636be40b22cd5fca48902b6c4320787bc00e/port/platform/esp-idf/src/u_port_uart.c#L514-L519
The usual way to check this would be to hack the code above to add a fixed amount, say 2048 bytes, to the stack requested for tat task in the function call (i.e. stackSizeBytes + 2048) and then monitor the "low watermark" of that stack by calling uPortUartEventStackMinFree(), periodically from your application.  If you see it go below 2048 bytes then you have found the problem case.  It is quite likely to be some particular combination of events that causes a sudden jump in stack usage, rather than a slow reduction; at least that would be my guess.
Once you have the problem case, you can decide whether it is something that can be fixed somehow or whether it is just expected operation, in which case the stack size would need to be increased somewhat at the point that the UART event task is created:
https://github.com/u-blox/ubxlib/blob/ffa9636be40b22cd5fca48902b6c4320787bc00e/common/at_client/src/u_at_client.c#L2839-L2843
This you could do by overriding U_AT_CLIENT_URC_TASK_STACK_SIZE_BYTES at build time.

uPortUartEventStackMinFree expects a handle.
Where can I get this?

Ah, yes, sorry, you can get it with something like the following (this code not compiled by me, just to point you in the right direction):
// Assuming that your device handle is in devHandle

uAtClientHandle_t atHandle = NULL;
uAtClientStreamHandle_t stream = U_AT_CLIENT_STREAM_HANDLE_DEFAULTS;

if (uCellAtClientHandleGet(devHandle, &atHandle)) == 0) {
    uAtClientStreamGetExt(atHandle, &stream);
    if (stream.type == U_AT_CLIENT_STREAM_TYPE_UART) {
        // The UART handle of the AT client for the cellular device is now in stream.handle.int32
        printf("Calling uPortUartEventStackMinFree() on UART handle %d gives %d bytes.\n",
               stream.handle.int32,
               uPortUartEventStackMinFree(stream.handle.int32));
    } else {
        printf("AT client handle %p has stream handle type %d (expected %d)!\n",
               atHandle, stream.type, U_AT_CLIENT_STREAM_TYPE_UART);
    }
} else {
    printf("Unable to get AT client handle for device %p!\n", devHandle);
}

several times at the beginning:
Calling uPortUartEventStackMinFree() on UART handle 1 gives 2424 bytes.
and finally several times:
Calling uPortUartEventStackMinFree() on UART handle 1 gives 2184 bytes.

So getting a bit low but always above 2048, so you haven't reached a stack overflow case yet I think.

The problem does not always occur. I try to restart the process to possibly catch the overflow.
So far I have also seen the following value:
Calling uPortUartEventStackMinFree() on UART handle 1 gives 2088 bytes.
How does it even come about that the value is different?

How does it even come about that the value is different?

Timing and different paths through the code will generally cause stack usage to differ between runs.  But like I say, I suspect your issue is not some much a grey-scale thing as something that suddenly goes off the rails, a bug of form.
So you've never seen the stack of the UART event queue task go below 2048 bytes minimum free with the 2048 bytes hacked-in?

One thing you could potentially try is setting CONFIG_COMPILER_STACK_CHECK_MODE to something other than CONFIG_COMPILER_STACK_CHECK_MODE_NONE (the default).  This will tell the compiler to add additional stack guards on a per-function basis to check for stack breaches that will be checked on entry and exit.
The issue with this is that it will change the run-time behaviour, 'cos of all the checking going on, and that may have other effects, but if you haven't managed to trap the problem yet and you think it should have occurred by now then it may be worth a try.

Hi @alexmaron81: how did this go in the end?

Hi Rob, I still couldn't find the error.


Explaining Engine API access

More explanation around the Engine API.

Looks great. It might be useful to mention that we have weekly open meeting to review proposals.

I added the RFC process as an alternative to mailing the maintainers, that should cover the meeting review as well.


OS Error access denied , errno = 5

windows10
Oops; flutter has exited unexpectedly: "FileSystemException: Cannot create link, path =
'E:\app\pp\flutter-folio\windows\flutter\ephemeral.plugin_symlinks\shared_preferences_windows' (OS Error: 拒绝访问。
, errno = 5)".
A crash report has been written to E:\app\pp\flutter-folio\flutter_05.log.

Seems like some local build cache might be corrupt? Try flutter clean? Also, please post your flutter doctor --version

@esDotDev
PS E:\app\pp\flutter-folio> flutter clean
Deleting build...                                                    6ms
Deleting .dart_tool...                                               2ms
Deleting .packages...                                                1ms
Deleting Generated.xcconfig...                                       0ms
Deleting flutter_export_environment.sh...                            0ms
Deleting ephemeral...                                                5ms
Deleting .flutter-plugins-dependencies...                            1ms
Deleting .flutter-plugins...                                         0ms
PS E:\app\pp\flutter-folio> flutter doctor -v
[√] Flutter (Channel dev, 2.1.0-10.0.pre, on Microsoft Windows [Version 10.0.17763.1577], locale zh-CN)
• Flutter version 2.1.0-10.0.pre at D:\work\flutter
• Framework revision cc9b78fc5c (11 days ago), 2021-02-25 13:26:03 -0800
• Engine revision a252ec09b7
• Dart version 2.13.0 (build 2.13.0-77.0.dev)
[X] Android toolchain - develop for Android devices
X Unable to locate Android SDK.
Install Android Studio from: https://developer.android.com/studio/index.html
On first launch it will assist you in installing the Android SDK components.
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
If the Android SDK has been installed to a custom location, please use
flutter config --android-sdk to update to that location.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.7.6)
• Visual Studio at D:\Program Files (x86)\Microsoft Visual Studio\2019\Community
• Visual Studio Community 2019 version 16.7.30611.23
• Windows 10 SDK version 10.0.18362.0
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
[√] VS Code (version 1.53.1)
• VS Code at C:\Users\GF\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.17.0
[√] Connected device (2 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.17763.1577]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 88.0.4324.190
! Doctor found issues in 2 categories.
PS E:\app\pp\flutter-folio> flutter build windows
Warning: You are using these overridden dependencies:
! http 0.12.2 (0.13.0 available)
! http_parser 3.1.4 (4.0.0 available)
Running "flutter pub get" in flutter-folio...                      26.0s
Oops; flutter has exited unexpectedly: "FileSystemException: Cannot create link, path =
'E:\app\pp\flutter-folio\windows\flutter\ephemeral.plugin_symlinks\bitsdojo_window_windows' (OS Error: 拒绝访问。
, errno = 5)".
A crash report has been written to E:\app\pp\flutter-folio\flutter_06.log.
This crash may already be reported. Check GitHub for similar crashes.
https://github.com/flutter/flutter/issues?q=is%3Aissue+FileSystemException%3A+Cannot+create+link%2C+path+%3D+'E%3A\app\pp\flutter-folio\windows\flutter\ephemeral\.plugin_symlinks\bitsdojo_window_windows'+(OS+Error%3A+拒绝访问。
%2C+errno+%3D+5)
To report your crash to the Flutter team, first read the guide to filing a bug.
https://flutter.dev/docs/resources/bug-reports
Create a new GitHub issue by pasting this link into your browser and completing the issue template. Thank you!
https://git.io/JqLwd


pass correct apko binary runfiles path to apko_run launcher

In https://github.com/sourcegraph/sourcegraph, we've defined our own apko toolchain to integrate a patch to go-apk for the apko binary. Due to this, we pass register = False to apko_register_toolchains.
This surfaced a correctness issue when trying to run bazel run @rules_apko//apko, with the following error:
INFO: Running command line: bazel-bin/external/rules_apko/apko/_apko_run.sh
Workdir: /home/noah/Sourcegraph/sourcegraph
/home/noah/.cache/bazel/_bazel_noah/8fd1d20666a46767e7f29541678514a0/execroot/__main__/bazel-out/k8-fastbuild/bin/external/rules_apko/apko/_apko_run.sh: line 11: /home/noah/.cache/bazel/_bazel_noah/8fd1d20666a46767e7f29541678514a0/execroot/__main__/bazel-out/k8-fastbuild/bin/external/rules_apko/apko/_apko_run.sh.runfiles/__main__/bazel-out/k8-opt-exec-ST-13d3ddad9198/bin/external/dev_chainguard_apko/apko_/apko: No such file or directory

From the bazel docs on runfiles location, this particular quote highlights the correct way to build the path to the apko binary's runfile path: "The path to a File under the runfiles directory corresponds to File.short_path."
The change presented works both for our case, and also bazel run //apko:apko in the rules_apko repo :slightly_smiling_face:

can you sign your commits?

can you sign your commits?

done 👍

ping


Support HTTP/2 on ember

It would be great if an ember server could support HTTP/2

@ChristopherDavenport is actively working on this.

Done in https://github.com/http4s/http4s/pull/5657.


Additional Host Tags using DD_TAGS

Description
Using the DD_TAGS environment variable to configure the agent as part of the agent daemonset installation doesn't seem to add tags as described in the documentation. Separating tags with a comma as described seems to result in the creation of a single host tag, with the commas replaced by underscores. Using datadog/agent:latest.
Additional environment details (Operating System, Cloud provider, etc)
Running in GKE, within the Google Cloud Platform.
Steps to reproduce the issue
Modify the K8s manifest for the agent daemon install to use the DD_TAGS environment variable to add additional host tags:
- name: DD_TAGS
  value: env:staging,support:johnc

Describe the results you received
Nodes have the single additional host tag env:staging_support:johnc
Describe the results you expected
Node to have two additional tags env:staging and support:johnc

Thanks for reaching out @TheMagoo73
It looks like our documentation is inconsistent here. The tag list should be space-separated as explained in the docker part and not comma separated as explained in the heroku part

Thanks for confirming, that works like a charm. Awesome job.


The first property is not focused after expanding the 'Properties' window

Storage Explorer Version: 1.4.0
Platform/OS Version: Win10 /MacOS High Sierra /Linux Ubuntu 16.04
Architecture: ia32
Build Number: 20180730.1
Commit: 17eb92a4
Regression From: Not a Regression
Steps to Reproduce:

Select one storage account.
Click the group glyph to collapse the 'Properties' window.
Click the group glyph again to expand the 'Properties' window.

Expected Experience:
After Step2: The group glyph doesn't keep being focused.
After Step3: The first property is focused.
Actual Experience:
After Step2: Keep focusing the group glyph until after clicking elsewhere.
After Step3: The first property is not focused.

This is expected behavior. Why would you expect focus to jump to the first property?

Thanks for your update. Please get the reason from More Info.
More Info:
The issue doesn't reproduce when expanding the 'Activities' windows. Please get the Repro
Steps from below.
  Repro Steps:
  1. In 'Activities' window, make sure the first item 'Clear completed' link is not focused.
  2. Click the group glyph to collapse the 'Activities' window.
  3. Click the group glyph again to expand the 'Activities' window.

  Result:
  The first item - 'Clear completed' link is focused.

Not really sure why we have that behavior in the Activities panel. Either way, I don't consider this appropriate behavior. @jinglouMSFT can correct me if I'm wrong.

The behavior with the property grid is what I had expected. The Activity's seems to be a bit odd. Did we modify the behavior when fixing the accessibility bugs (for the Activity window)?

My vague memory is in the past we reasoned it was "unproductive" for a user to expand the panel then tab to the next control, because how often would a user expand the panel but want to stay on the button? So we automatically transfer focus to the next control in the panel. But after our accessibility push, this behavior can actually be more confusing for users.


/app/.heroku/python/bin/pip: No such file or directory

I am trying to deploy a Python app on heroku and I get the following output:
Counting objects: 10, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (10/10), 936 bytes | 0 bytes/s, done.
Total 10 (delta 0), reused 0 (delta 0)
remote: Compressing source files... done.
remote: Building source:
remote:
remote: -----> Python app detected
remote: -----> Installing python-3.6.0
remote:      $ pip install -r requirements.txt
remote:        /app/tmp/buildpacks/779a8bbfbbe7e1b715476c0b23fc63a2103b3e4131eda558669aba8fb5e6e05682419376144189b29beb5dee6d7626b4d3385edb0954bffea6c67d8cf622fd51/bin/steps/pip-install: line 20: /app/.heroku/python/bin/pip: No such file or directory
remote:  !     Push rejected, failed to compile Python app.
remote:
remote:  !     Push failed
remote: Verifying deploy...
remote:
remote: !	Push rejected to ***.
remote:
To https://git.heroku.com/***.git
 ! [remote rejected] master -> master (pre-receive hook declined)

Looks like, it cannot find pip executable. Is it a bug or am I doing something wrong?

I have changed runtime.txt from python-3.6.0 to python-3.5.2 and it worked!
This is really strange, because official documentation says that python-3.6.0 is supported and python-3.5.2 is not.

this is now fixed


ES6 Features Support List

Add a public table with support status for ES6 features.
Start with the:

WeakMap
WeakRef
Map
typed arrays

This is a good idea for when we migrate to out-of-band versions of JavaScriptCore, but since our current version is the one that shipped with Safari 8, this should do.

Since we will be migrating to new JSC version quite soon we'd better postpone this for the 1.2.0 milestone.

Here are the JavaScriptCore features for 1.6: https://github.com/NativeScript/webkit/blob/ios/Source/JavaScriptCore/features.json


Image loading is in the UI thread

Dear developers,
For image loading, Google says this:
https://developer.android.com/topic/performance/graphics/index.html
" Loading bitmaps on the UI thread can degrade your app's performance, causing slow responsiveness or even ANR messages."
I noticed that in the following code, image loading is in the UI thread:
https://github.com/xbmc/Kore/blob/master/app/src/main/java/org/xbmc/kore/ui/sections/remote/RemoteFragment.java#L271
org.xbmc.kore.ui.RemoteFragment.java  --->adjustRemoteButtons()   (line number: 271)
(bitmap is loaded with the invocation of decodeResource ())
the image resource is : https://github.com/xbmc/Kore/blob/master/app/src/main/res/drawable-xxhdpi/remote_background_square_black_alpha.png
adjustRemoteButtons() is called by: org.xbmc.kore.ui.RemoteFragment.java    onCreateView()
which means image loading is in the UI thread.
Do you think perform image loading in the UI thread will affect Kore's performance? If so, perhaps moving the images loading to a work thread can help make app more smooth in user interaction.
Looking forward to your response.
Thanks.

I haven't tested Kore on every possible device, but even on the virtual devices (which are terribly slow compared to real devices) I don't notice an unacceptable delay when inflating the remote. So, I'm not sure if this really is an issue, although the background image is fairly large.
Do you experience a noticeable delay when Android inflates the remote UI?

And that code is applicable to Jelly bean only. That's about 30% of the market nowadays, but it tends to get down.

This use to be a big issue back in time but not that deadly nowadays. Still not a very good practice though.

this api can speed up process - http://square.github.io/picasso/

Closing this, as it isn't really worth it.


Identifies BTC balance, but funds not reflecting after transfer

Hello,
At the outset, my thanks to the developers for coming up with this solution.
The tool was able to correctly identify balance and displayed it as such - ready to transfer.

I entered the destination BCH address (although it displays BTC value, I think we should indeed put in a BCH address in the field).

I get the message that the amount was sent, however, it does not reflect in my BCH wallet (had created one on bcc-wallet.btc.com). It still shows up as "waiting". [note - the BCH balance on display is an amount I previously had; it doesn't pertain to the "recovery" instance that I am describing]

Please help.

Oh bloomin' heck - have I actually performed a BTC transfer to a BCH address??!?
Any way to retrieve them?

Kar thick,
Was about to warn u coz I did the same thing and all my btc funds are gone. Should not send your btc funds to your BCH wallet. I made the same mistake coz we did not switch to  bch/ live net. However u can still get your BCH. Switch it BCH/livenet

I wrote to support and they suggest I write to BCH wallet support. Till now I've not received a response. If u know how pls write as I want to know too. Very painful experience. I felt as if I've switch btc for BCH.

Thanks, @rickynkl - but no luck. Switching it to bch/livenet doesn't reflect any funds. Not heard back from support either. This has been a disaster...

Your BCH should still be in there. I managed to extract it out and transfer to a BCH wallet

Nope - the BCH balance reflecting in the wallet I transferred the BTC to was something that I already had. It has nothing to do with the "reclaimed from fork" stuff.

Yes at first I mistakenly search funds on btc/livenet mode and my btc funds appeared and I sent it to my BCH wallet thinking it was BCH ( I had btc before Aug 1). And it did not appear in my BCH wallet( of course it would not coz they are btc funds). Big mistakes! Lost all my btc funds. Then I realised I should still have my BCH to claim from my copay  btc wallet. So I went back to recovery tool and ran and searched again this time with BCH/ livenet with my recovery phrase. And all the BCH funds came out. Hope this helps u.

You mean: you transferred BTC to BCH too (having searched with btc/livenet), it did not appear in the BCH wallet, then re-visited the tool (this time searched with bch/livenet)?
Yes, that's fine - already did it and recovered - but what about the btc that was transferred to the bch wallet in the first place? That's gone for good?

Im in exact same situation as yours. Copay support says it's gone and can't be reversed and suggested I write to the receiving BCH wallet support. So far no response from BCH wallet support.

Holy moly... Yes, I wrote to bcc-wallet.btc.com's support too. Yet to hear from them. Even shared the address to which the BTC was inadvertently sent.
Collateral damage, damn!

After an insane amount of reading and research, I got it done. @rickynkl - which BCH wallet do you use? I will try my best to help you out.

Thanks. What a relief to hear u got them back. Mine is Freewallet BCH

Can you share a link to it? Is it a web or desktop or mobile app?

It's a mobile app. Android

Oh no! If its these guys, they won't share your private keys with you! Without which, its going to be hell! Is this the service?
https://freewallet.org/support/knowledgebase/where-are-my-private-keys/

Yes

I hope they do. It must be buried somewhere in there. I sent them my txid

According to their FAQ (check the link above), they do not share private keys with individuals.
You will need to tell THEM to retrieve it for you (importing BTC is possible from a BCH address; I just did it) and then transfer it to a BTC address that you specify.
If you don't mind, can you share the transaction hash or destination bch wallet address?

I forward to u later as my records is stored in another device. Thanks

Appreciate your help here

It's a relief to hear it's possible to retrieve btc from BCH address

It certainly is, I have just done it, I won't be joking around - this is big, hard earned money. But I'm afraid in your case, you really have to push FreeWallet because they own the private keys (without which, it is next to impossible). If you are active on social media, Tweet them. Or reddit. Or buzz them on Facebook and LinkedIn.

Yes it certain is. Very very hard earned. The community needs to help and learn from each other with this newly acquired knowledge

On Mon, 21 Aug 2017 at 10:19 AM, karthik-swaminathan <
<EMAIL_ADDRESS>wrote:

It certainly is, I have just done it, I won't be joking around - this is
big, hard earned money. But I'm afraid in your case, you really have to
push FreeWallet because they own the private keys (without which, it is
next to impossible). If you are active on social media, Tweet them. Or
reddit. Or buzz them on Facebook and LinkedIn.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bitpay/copay-recovery/issues/61#issuecomment-323631407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Adr_Rz13bdR1jCiKoz4vxCDXtR--qxQ7ks5saOlIgaJpZM4O8Of7
.

Freewallet have responded yesterday. They said they will refund. Will take
some time they said. Thanks for your help. Phew!

Delighted to know that, @rickynkl. Happened to see a review on the Play Store (BitcoinCash wallet by FreeWallet) dated 20th August and wondered if that was you. Similar complaint. Transferred BTC to BCH wallet and therefore stuck.

Wrote to them since last week within the in app Bitcoin cash too. Yes I
also wrote in on the review too. As suggested by bitpay to write in to
receiving bcc wallet. That's an indication that's it can be retrieved. I've
learnt from this. Thanks
On Tue, 22 Aug 2017 at 10:39 AM, karthik-swaminathan <
<EMAIL_ADDRESS>wrote:

Delighted to know that, @rickynkl https://github.com/rickynkl. Happened
to see a review on the Play Store (BitcoinCash wallet by FreeWallet) dated
20th August and wondered if that was you. Similar complaint. Transferred
BTC to BCH wallet and therefore stuck.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bitpay/copay-recovery/issues/61#issuecomment-323903133,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Adr_Rx5IORnTZPZUPpGxSNpL4SwyCjIjks5saj9IgaJpZM4O8Of7
.


Nether portals teleportning oddly

Yeah very basic title, sorry but wasn't sure what to title this. Anyway, the issue I'm having I'm pretty sure is just a configuration issue but when I have myself or anyone enter or exit a nether portal it seems to only lead to the same point. To get a bit more specific I can make a nether portal anywhere in my main world but when I enter it it always takes me to the same place in the nether, And vice versa. Like I said this is probably a configuration issue but I've looked through the wiki and have yet to find anything. So if anyone has any fixes or advice it would be greatly appreciated.

This sounds like a very common issue that is due to the portals in your overworld being too close (within 1024 blocks of each other). If so, this wouldn't have anything to do with Multiverse-NetherPortals or your configuration of it. See http://www.minecraftwiki.net/wiki/Nether_Portal#Linkage_between_Overworld_and_Nether

Yep that's what it turned out to be, teleported to coords much further away and tried and it seemed to work as intended. Thanks.


Notes from looking at SrXplanar

@sbillinge and @timoHMM are here, and here is what we understand about how the scientific content of SrXplanar works.
# crop
tth = gen_th_matrix(rotation, tilt, center, distance)
q = tth_to_q(tth, wavelength)
# polarization correction
# solid-angle correction

Masking:

manual ROI
accept some band of factors times the average value in each Q ("robustness" test)
binary erosion, dilation
randomly discard some fraction of the pixels in the image to undersample

Fit to find rotation, tilt, x, y:

Integrate.
Compare left and right halves, top and bottom halves. Minimize.

There is a known 1D curve and a computed 1D curve. Stretch distance to detector until they overlap maximally.

Masks used by @CJ-Wright

beamstop mask
trim margin
eliminate q bands with too few pixels (i.e., < 10)
robustness: eliminate pixels X standard deviations away from the average in that q band

I am also working on benchmarking mask 4.  I want to double check that this finds the pixels that are truly bad and doesn't remove the correct pixels.

High-level overview of Powder Diffraction Analysis: Inputs and Outputs at each stage
Notes from a conversation today with Soham and Max in the Billinge group:
Calibration
Inputs:

D-spacing (e.g., from pyFAI/calibrations/*.D or elsewhere, including scikit-xray)
Calibration image
Wavelength "provided by beamline scientist" (or, in some cases, this is a fit parameter, an output)

Outputs are Geometry:

center
tilt
rotation
distance to detector
wavelength (sometimes -- see above)

Usually (not always) the calibration image does not need to be preprocessed. We are never going to integrate it; we just need to be able to see the rings.
Algorithm: There are several. pyFAI implements three (according to @CJ-Wright) and uses them in competition with each other. srXplanar uses a symmetry-based algorithm. scikit-image has some new, relevant algorithms that are worth exploring.
Integration
Inputs:

images
background image
geometry (obtained above from calibration image)

Output:

I(q)

Algorithm:

Tilt images to "square" them in accordance with known geometry.
Mask:

manually (polygon or freeform drawing)
removing edge issues
applying percentile- or sigma-based threshold around the mean or median

Correct:

polarization correction
solid-angle correction

Azimuthal integration to obtain uncorrected I(q) for images and background image
Subtract I(q) for background from I(q) for images. This involves manually scaling, presently. It could be automated.

Transforms
Input:

I(q)

Output:

S(q), F(q), g(r)

Fitting
Inputs:

g(r)
models

Output:

fit result

@danielballan here are pyFAI's algorithms  https://github.com/kif/pyFAI/blob/master/pyFAI-src/geometryRefinement.py#L246

Some related issues:

calibration

#199
#197
#175
#119

image corrections

#41
#11

@danielballan Technically none of my code does the calibration of the detector geometry, although I did write some code for XPD to calibrate the beam energy based on a series of detector images.
Also after discussion with @tacaswell the method of working with masks may be to unravel everything and then remove the pixels which are masked before the final histogram/binned_statistic function.
Also also, PDFgetX3 requires hand tuning of parameters including:

The maximum and minimum Q (scatter vector) to be used in the transform
The instrumental maximum Q
The rpoly factor which deals with the ad hoc corrections for Compton scattering and other factors

Speaking personally (i.e., XPD-centrically) we have a solution for the
calibration in xPDFsuite which is installed for users on XPD, so you will
be solving a problem that we don't need solved if you work on this
calibration problem.  I would definitely prefer that this be low on the
priority list compared to, well, just about everything else.....
S
On Tue, Nov 24, 2015 at 8:15 PM, Christopher J. Wright <
<EMAIL_ADDRESS>wrote:

@danielballan https://github.com/danielballan Technically none of my
code does the calibration of the detector geometry, although I did write
some code for XPD to calibrate the beam energy based on a series of
detector images.
Also after discussion with @tacaswell https://github.com/tacaswell the
method of working with masks may be to unravel everything and then remove
the pixels which are masked before the final histogram/binned_statistic
function.
Also also, PDFgetX3 requires hand tuning of parameters including:

The maximum and minimum Q (scatter vector) to be used in the
transform
The instrumental maximum Q
The rpoly factor which deals with the ad hoc corrections for
Compton scattering and other factors

—
Reply to this email directly or view it on GitHub
https://github.com/scikit-xray/scikit-xray/issues/346#issuecomment-159455044
.

--
Prof. Simon Billinge
Applied Physics & Applied Mathematics
Columbia University
500 West 120th Street
Room 200 Mudd, MC 4701
New York, NY 10027
Tel:<PHONE_NUMBER> (o) 851-7428 (lab)
Condensed Matter Physics and Materials Science Dept.
Brookhaven National Laboratory
P.O. Box 5000
Upton, NY 11973-5000
(631)-344-5661
email: sb2896 at columbia dot edu
home: http:// http://nirt.pa.msu.edu/bgsite.apam.columbia.edu/


revert partially Enable/Optimize flan t5 xxl on deepspeed z3 #257

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Name allowed keys and/guess on missing map key

Replaces existing author-centered hint with some that help the consumer, instead.
Inspired by:

cc: @voor

Tests failing in CI, but working locally: I failed to rebase off develop; on it.


.Net: New CreateFromType and CreateMetadataFromType methods

Motivation and Context
Closes #8946
Description

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

I thing this PR or the Issue misses the Problem / Motivation and Context


Working_with_FASTQ / sort dictionary

poses = qual_pos.keys()
poses.sort()

will only work in Python 2.5
in Python 3:

poses = sorted(qual_pos.keys())

That is why the code is slightly different than that:
poses = list(qual_pos.keys())
The list makes sort work.
Question: I suppose you are following the book and not the notebooks? Because these errors might be in the book (I do not have the book here to check)

i'm not sure the source now, but probably the book yes, can be closed then


Allow unbound variables in all parts of the query.

TODO: Still needs warnings, cleanups, and tests.
Fixes #1575

Would it be possible to keep the current behavior behind some flag?  It saved me a lot of time already. So it would be a pity to loos this functionality.

@aindlq Thanks for the input and the (in my opinion very reasonable) request.
I hope that I can find a way to get both behaviors with a reasonable amount of implementation work.

@joka921 Thank you for the --throw-on-unbound-variables flag!


How to cancel or destroy a getPage request with disableAutoFetch set

Dear pdf.js contributors,
With disableAutoFetch set, is there a way to cancel fetching on getPage() ? The same way one destroy() getDocument promise
It looks like it is possible but I found only internal functions.
Best, A.

With disableAutoFetch set, is there a way to cancel fetching on getPage() ?

Huh, calling getPage is what causes data to be requested (there's no cancelling involved); it's quite frankly difficult to understand what you're trying to ask here.

it's quite frankly difficult to understand what you're trying to ask here.
Oups.  One can cancel the fetching by calling destroy() on the promise, it is going to stop the GET request.

With disableAutoFetch set, fetching occurs just after getPage, but there is no destroy() to cancel the GET 206 request in case one wants to.

[...] but there is no destroy() to cancel the GET 206 request in case one wants to.

There's no way of doing what you're asking, short of destroying the loadingTask itself (and thus closing the entire document).

For example, one requires to cancel some pages currently being fetched because the user moves to other pages before the previous pages were fetched.

First of all, note that there's a couple of different ways that data could be loaded (using Fetch, XMLHttpRequest, or a PDFDataRangeTransport implementation). Secondly, there's generally speaking nothing that says that different pages wouldn't need data from the same byte range (and aborting a request would thus break other getPage calls).
Hence what you're asking for isn't possible, nor will not be supported either unfortunately (as outlined above, and the use-case seems fairly specialized anyway).

Secondly, there's generally speaking nothing that says that different pages wouldn't need data from the same byte range (and aborting a request could thus break other getPage calls).

Ok. In my use case, I fetch say page [-1, current, 1] whenever the user moves to a page. If a user moves fast to another current page, I am going to cancelAllRequests().

https://github.com/mozilla/pdf.js/blob/c3a1c679500e1e4cfa13291f2bcd58ebbba2eb14/src/display/transport_stream.js#L131-L139

Wait until all the requests are cancelled, and fetch the new [-1, current, 1] pages.
My question is: does cancelAllRequests() the best option for such scenario ?

I am going to cancelAllRequests().

As explained in https://github.com/mozilla/pdf.js/issues/11453#issuecomment-569335735 that will easily lead to all kinds of breakage, and isn't something that you should be calling manually (it's being used from WorkerTransport.destroy).

I am currently using PDFDataRangeTransport implementation for range requests.

Please note that the default range request functionality in PDF.js isn't in any way connected with PDFDataRangeTransport, so unless you're using the API along the lines below then you're not actually using PDFDataRangeTransport.
const loadingTask = getDocument({
  range: /* custom PDFDataRangeTransport here */,
  //  more parameter here
});

OK. I am going to setTimeout(() => getPage(), 200);
Thank you

What is the purpose of PDFDataRangeTransport ?

It allows completely custom data delivery, that you thus can implement in what ever way you want/need in your case (it's being used in the PDF Viewer that's built-in to the Firefox browser).
While it does allows a great deal of flexibility, it's consequently a fair bit more complex than just providing a URL when calling getDocument :-)

I found no examples or use cases out there

There's the API unit-tests and also the default viewer usages here, here, here and finally here and here.

Closing as answered by the comments above.

I had a similar issue with very large non-linearized PDFs where getPage was taking a very long time for later pages because most if not the whole PDF had to be downloaded. Specifically if the user closed the document but not the app it would nonetheless keep downloading. For this PDFDocumentLoadingTask.destroy works well, thank you for the advice @Snuffleupagus!
I would quickly point out that it would be nice to be able to cancel an individual page load too though because, of course, page loading can sometimes take WAY longer than rendering. Hunting around the code it seems like during the while loop here  it'd be pretty straightforward to add a simple cancellation check on each iteration, using something akin to an optional CancellationToken that could be passed into getPage and getPageDict. Would that not do the trick and would you entertain a PR that did that?


Documenting createErrorMessage and allowing custom message templates?

I found assembling custom validator messages to be burdensome. I decided to have a look how it's done in default validators and discovered that a nice helper createErrorMessage is used.
I ended up doing this:
import Ember from 'ember';
import BaseValidator from 'ember-cp-validations/validators/base';

// Monkey-patching the `messages` object
import messages from 'bealang-frontend-ember/validators/messages';
messages.noWhitespaceAround = 'should not have whitespace at the beginning or end'

export default BaseValidator.extend({
  validate(value, options, model, attribute) {
    if (value.length === value.trim().length) {
      return true;
    }

    // Using the secret helepr
    return this.createErrorMessage('noWhitespaceAround', options, value);
  }
});

Suggestions:

createErrorMessage should be documented in the readmy and not hidden from the public. ^_^
Should be able to extend messages gracefully in a way that prevents from accidentally overwriting existing messages (but allows explicit overwriting).

createErrorMessage should be documented in the readmy and not hidden from the public. ^_^

:+1:

Should be able to extend messages gracefully in a way that prevents from accidentally overwriting existing messages (but allows explicit overwriting).

This should be solved once we move over to our i18n solution #9

Closing in favor of #40 and #9.


Support Vmess protocal

Could 大佬 support Vmess protocal?

No, vmess encryption and decryption consume a lot of CPU resources, which is not cost-effective on serverless


Fix CI status badge in README

This seems to just be showing the status of whatever the most recent build happened to be, rather than the status of the main branch. Need to investigate. Thanks for the report @simkimsia

Thanks! Your "quick fix" is perfect. I'm not a big fan of pre-commit personally.

Fixed in #63

Ah this still doesn't work 😆 CI doesn't run on main so it just says "no status". What we want is the status of whatever the last thing that was merged into main is.. which probably isn't possible?
I'm not actually sure what value this badge really adds... maybe we should just remove it.

I think a good way to do this would be to add
on:
  push:
    branches:
      - main

To the GitHub Actions config so that the tests do run against main after a merge. This means the badge represents "do the tests pass against current main, which I think is what we want.

sounds good. wanna try?

It works! 🙂

love the green color passing badge! 🙌


[AutoPR] restapi_auto_graphrbac/data-plane

Codecov Report

Merging #2032 into master will decrease coverage by 2.11%.
The diff coverage is 65.43%.

@@            Coverage Diff             @@
##           master    #2032      +/-   ##
==========================================
- Coverage   53.06%   50.94%   -2.12%
==========================================
  Files        4772     4772
  Lines      119472   126315    +6843
==========================================
+ Hits        63398    64357     +959
- Misses      56074    61958    +5884

Impacted Files
Coverage Δ

azure-graphrbac/azure/graphrbac/version.py
100% <100%> (ø)
:arrow_up:

...c/azure/graphrbac/operations/objects_operations.py
28.07% <50%> (+2.61%)
:arrow_up:

...graphrbac/azure/graphrbac/models/key_credential.py
30.76% <50%> (-2.57%)
:arrow_down:

...c/azure/graphrbac/operations/domains_operations.py
29.09% <50%> (+2.67%)
:arrow_up:

...re/graphrbac/operations/applications_operations.py
28.44% <59.09%> (+3.67%)
:arrow_up:

...phrbac/operations/service_principals_operations.py
31.25% <61.11%> (+3.38%)
:arrow_up:

...ac/azure/graphrbac/operations/groups_operations.py
50.25% <72.22%> (+2.43%)
:arrow_up:

...bac/azure/graphrbac/operations/users_operations.py
68.93% <83.33%> (+1.47%)
:arrow_up:

...mgmt/web/operations/deleted_web_apps_operations.py
29.41% <0%> (-11.77%)
:arrow_down:

...mt/web/models/reissue_certificate_order_request.py
44.44% <0%> (-5.56%)
:arrow_down:

... and 43 more

Continue to review full report at Codecov.

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

@AutorestCI rebase

Rebase done and pushed to the branch

@AutorestCI rebase

Rebase done and pushed to the branch

(message created by the CI based on PR content)

@AutorestCI rebuild https://github.com/Azure/azure-rest-api-specs/blob/master/specification/graphrbac/data-plane/readme.md

Working on generating https://github.com/azure/azure-rest-api-specs/blob/master/specification/graphrbac/data-plane/readme.md for you!!!

Build SDK finished and cleaned

@AutorestCI rebuild https://github.com/Azure/azure-rest-api-specs/blob/master/specification/graphrbac/data-plane/readme.md

Working on generating https://github.com/azure/azure-rest-api-specs/blob/master/specification/graphrbac/data-plane/readme.md for you!!!

Build SDK finished and cleaned


Doubled scenes loose all changed properties

In this code nodes get instanciated, but properties aren't set on the doubles:
https://github.com/AlexDarigan/WAT-GDScript/blob/f4ff6b9cfe29c5e11898e8dac6cdc04b4460e65f/addons/WAT/core/double/factory.gd#L25-L28
Example:
var director = direct.scene("res://some/scene.tscn")
var double = director.double()
# Prints the default value, even if it's changed in scene.tscn.
print(double.member)

For at the very least top-level scenes should have this sorted with this fix. https://github.com/AlexDarigan/WAT-GDScript/commit/1c6cde758d1f3a1af12cbc624fa11a831db7d8d9

We're resetting too many things. May have to add some default exclusions or a way to make sure the property isn't exported. (The current change is why scene director test is broken).

Seems 8199 is the bit number for user exported variables, so we'll just focus on those.

This has been solved for a while I think so I'm going close this until it pops up again.


Add new blueprints for overload protection

Description of change
Checklist

[ ] Tested in playground or other setup
[ ] Screenshot (Grafana) from playground added to PR for 15+ minute run
[ ] Documentation is changed or added
[ ] Tests and/or benchmarks are included
[ ] Breaking changes


[Feature Request] Publish Dashboard as static release files

Right now we're building and copy-paste Dashboard in git directly:

waha (+ waha-plus)
waha-docs

Instead, dashboard repo (https://github.com/devlikeapro/waha-hub) should publish it as "Release" or somehow public available artifacts, and build processes for waha and waha-docs must download it and place in the right direction

[x] Publish publicly available build from waha-hub
[ ] Use it in waha-docs build process (no opt out) in github actions - always latest
[ ] Use it in waha build process (optional opt-out) in Dockerfile - specify the version?

https://github.com/devlikeapro/dashboard/
also we have dashboard for "staging" for free
https://devlikeapro.github.io/dashboard/

waha-docs
https://github.com/devlikeapro/waha-docs/blob/main/.github/workflows/pages.yml#L48-L55

waha https://github.com/devlikeapro/waha/commit/c7b1a05ed3a4e518d67874ff816411623992133e


Kubernetes 1.9 Changes Tracking Issue (kraken-lib & kraken-tools)

Change the etcd version to match kubernetes 1.9 conformance testing.
As of Nov 17th, they've been testing with Etcd 3.1.10 in this link
The sig scale notes  talk about the etcd version for 1.9

Check for the kubernetes tested version of kube-dns to see if it changed for this release.
Tips for looking. https://k8s-testgrid.appspot.com/
Select 'sig-scalability' -> 'sig-scalability-gce' -> 'gce-1.9' ->About (see these results in Gubernater) -> choose a green build -> Artifacts -> Artifacts folder -> scroll down to nodes.yaml

going to start working on this now.

calico upgrade is going to be a bit of a bother.  we need to ensure the 2.6 version is at least 2.6.5 before moving to 3.0.1 (do not use 3.0.0).

rbac and a bunch of app types (daemonset in particular) have gone GA so there are a lot of template/file changes based on version.  so many more files :(

all etcd reads now require a successful quorum read.  this is a note to self.

first attempt with a 1.9 cluster.  it is predictably not working.  lets find out why!

second attempt is failing when attempting to install the network fabric.  neat!  progress!

branch:  https://github.com/coffeepac/kraken-lib/tree/kubernetes-1.9

project canal doesn't support canal 3.x on kubernetes yet, reverting to 2.6.2.

got a valid 1.9 cluster running.  setting sonobuoy loose to see what it comes up with.  if it passes I'll push a PR.

log output from e2e as run by sonobuoy.
e2e.log

now to make 1.7 and 1.8 still work.

one last open question on the PR, after that this is done.  Putting into review.


Add reconfig API to Pruner

it is not a public API for users, just for pruning scheduler now.
did not support in 'AutoCompressPruner' 'LotteryTicketPruner' 'AMCPruner', because they will refactor to PruningScheduler or too complex.

closed. the modifications will be incorporated in model compression 2.0


MQTT switching between online and offline constantly

There is some bug in the MQTT service

When I activate only the basic with MQTT connections and only the Home Assistant, it works normally.
If I enable any of the features below, the bug occurs.

MQTT Information Advertisement
MQTT Link Advertise
MQTT Camera Settings Advertise
MQTT Telemetry Advertise

The problem occurs in Firmware 0.2.2 of models r35gb, y211ga and y291ga (beta)

I forgot to mention that the Wi-Fi is at 100% on most cameras and I tested it on 3 different network systems (cable, regular Wi-Fi and Mesh Wi-Fi).
I also tested with snapshot off and swap on, getting the same result.
Without configuring, the countless times I formatted, changed the cards and reset the settings.

Check if you are using the same client id for both services:

All cameras are properly identified and in their separate topics.
I'm using the MQTT Explorer app which makes it easy to see all cameras.
Using the default settings, I can get the online status and binary motion sensor.
But if I enable any of the features below, the online and offline situation occurs.

MQTT Information Advertisement
MQTT Link Advertise
MQTT Camera Settings Advertise
MQTT Telemetry Advertise

There are 2 ids in the same cam.
One for the page http://IP:8080/?page=mqtt and another in the page http://IP:8080/?page=mqtt_adv
Are they different?

/?page=mqtt

/?page=mqtt_adv

Ok, change the id in mqtt_adv to "yi-850c-adv" and try if it works.

It does not solve...
?page=mqtt_adv

No matter what setting I apply, if I enable some of the services below, the problem occurs.

MQTT Information Advertisement
MQTT Link Advertise
MQTT Camera Settings Advertise
MQTT Telemetry Advertise

Try to overwrite mqtt_adv_* scripts with the content of this archive:
mqtt_adv.tar.gz
Please, make a backup copy of the files before overwriting.

I believe it solved, all services are active and with stability in status.

Since my last report, there have been no availability changes and all information is being presented correctly in Home Assistant.
I believe the problem has been resolved.

Yes, I will apply it in the next build.
The problem happens when you set hostname and mqtt client id at the same value.

I inform you that after updating to version 0.2.3, the problem remains

I know, I prepared 0.2.3 before these changes.
Please use the scripts I posted above until the next release.


Removed controlsHidden check for Replay button

The Replay's visibility now depends only on uiProps state, so it wouldn't disappear on tap anymore.

Can we write a test to fix this behavior?

I'll research this problem.


Turnjs/flipbook integration with pdf.js viewer

We've started looking at implementing pdf.js with turn.js. We've written a very quick article on this as well as an example viewer. Very crude example just to get things working but it works if someone with more time and effort can get this implemented in a more effective way then that would move things forward greatly! We've hijacked the presentation mode in the standard viewer to use for the turnjs book. Perhaps another "flipbook mode" could be used to host a viewer for the turnjs or any other flip book implementation.
Hopefully this can help move others forwards. The combination of PDF.js and turn.js could be pretty amazing.
http://www.link-infotec.co.uk/articles/pdfjs-and-turnjs-reader/

Cool application! Since there is no PDF.js issue here, I'm going to close this, but feel free to add your project to https://github.com/mozilla/pdf.js/wiki/Gallery-of-user-projects-and-modifications once it is finished. Thanks.

Hi, great work. I instantly found so many applications for this in my mind. I was looking on more info on how can I integrate it with angularjs as well. I think there are a couple of wrappers for turn.js in angularjs but I am not able to figure out the integration part that you have done here with the pdf.js to create embedded viewers. can you probably share a reference document or some info on how can I get it working with pdf.js and angular.js

can you probably share a reference document or some info on how can I get it working with pdf.js and angular.js

@iamdeadman As not angular.js experts, we rely on people that knows the subject to create a documentation. If you think that you are comfortable with angular.js, I'll be glad to chat with you on '#pdfjs' IRC channel on the subject. I guess you just do via including pdf.js script tag and providing workerSrc to the pdf.worker.js file as you would do with normal JS app.


Create new image carousel component for homepage.

Create new image carousel component for homepage, with clearer images and better design. It should have 3-4 images, linking to different parts of the website. Images cannot be copied from any other source, and have to be designed by you. First image has to be the Women in Tech logo and all images in the carousel should be of same size.

I can help you with carousel redesign, but I'll require some help  for designing images

The main job would be to create the images @logicalahara , this is a graphic design issue. If you can make the images on your own I will assign this to you. Please let me know.

4 images to design but using the current carousel on site. Is that right ?

I'm on it, please give me some light on key elements to include in design

4 images to design but using the current carousel on site. Is that right @shikha-16 ?

Yeah, kind of. We could get on a call to discuss this, please join the discord!

I want to work on this issue, Can you assign me?

Can someone assign me


Will you support web components instead of JSX?

=> JSX.Element;
I am surprised to find the requisition is to use REACT I would rather see that you export to standard HTMLElement like web components are.
Do you plan to support the standard or is react given? ...wc would allow us to use it in any given js framework not only react.

hi, thanks for getting in touch!
The Blocks app uses React, but it communicates with individual blocks using iframes and message passing, so there is no inherent reason that Blocks must be written with React. At the moment only React is supported in this template, but we're working right now on generalizing it to other frameworks or just plain JavaScript.


Create a set of Benchmarks for Kapacitor performance

In order to ensure stability and performance we need to measure it. We need to find a set of benchmarks that we feel represent Kapacitor's performance.

This is being done.


Add collapser metrics

Replaces #571

Addresses #14

Hystrix-pull-requests #248 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » Hystrix » Hystrix-pull-requests #9 SUCCESS
This pull request looks good


Allow Opendistro Security to be disabled temporarily via API

Right now in order to disable security, we follow: https://opendistro.github.io/for-elasticsearch-docs/docs/security-configuration/disable/ which is tedious as it has to be done on all of the nodes. Could you please come up with a way to disable it via an API call just so it helps cases where one might be need to temporarily disable just for testing?
Maybe something like POST _opendistro/_security/_enable or POST _opendistro/_security/_disable. Of course, you do need to authenticate against ODFE to disable it.

@bharadwajrembar According to the documentation
Adding opendistro_security.disabled: true in the elasticsearch.yml file and restart cluster&nodes will disable security plugin.

Changing security posture (enable/disable security) of the distributed system like Elasticsearch should not be done through APIs due to risk associated with it. To modify security posture entirely (like enable/disable), one should have access to physical machine itself.


Deprecate vendor branches with knife cookbook site install

So we can kill it for Chef 14. This adds a CLI option to not vendor and warns if you don't use it. I think I did the option correctly but I should probably make some kind of test for this.
💣 💥

@coderanger Care to rebase this one?

unit tests are actually failing 'cuz new deprecation warnings get thrown

@lamont @coderanger Any reason we don't just warn now and yank this in 15? Seems a lot simpler than requiring folks to put it in the configs to keep it until we ship the next major.

You probably wanted @lamont-granquist

Closing in favor of #7012


contrib: test: redirect stderr to stdout

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

[retest]

@mrunalp PTAL

LGTM


Signing a message to a custom network not working

When signing a message through Ethereum or Goerli it is working fine through :
launchUrl(_uri, mode: LaunchMode.externalApplication);                                                      await wcClient.request(topic: _sessionData!.topic, chainId: "eip155:$chainToConnect", request: SessionRequestParams(method: 'personal_sign', params: [message, EthereumAddress.fromHex(address.value).hex]));
But when it is a custom network like SKALE, with id "eip155:1517929550" it doesn't work, the pop-up of sign is not showing in metamask.
This is what i get back :
pairing sendResult, id: null topic: fa58510fd9f8ad6f89147ac3885231da19de18d1f325eedf1314cfdc6ae0a365, method: wc_sessionRequest, params: WcSessionRequestRequest(chainId: eip155:1517929550, request: SessionRequestParams(method: personal_sign, params: [0x535f0849b570b9dd9ea6cba61438fa7d90f8bf8b4e429e0d9a208bf58da6eca2, 0xb767bf40a58b6c20555973dd2cde3e5b5467b14e])), ttl: null

What wallet are you using?
Are you sure the wallet is on the SKALE network?

From the app, it opens Metamask, first i connect on the SKALE network, no issue on that part, next, when i try to sign through "personal_sign" no popup appears, when if i try on Goerli or Ethereum it is ok.
I added the SKALE network on Metamask as a custom network and on that side it is ok also.

I contact them and they don't have a "personal_sign" method on their network.
This answer close the issue.

I contact them and they don't have a "personal_sign" method on their network.
This answer close the issue.


Include UTC offset in time range

Hi!
Thank you for this library; it really helps us in formatting date/time ranges.
In our application, we both show ranges in local time (browser/OS dependent) and UTC time, and I would love to have an option to include the UTC offset (+-HH:MM) in the formatted string as well. As the offset can be different between the start and end of the range (when spanning over one of the daylight saving time shift), I think there is a reason to include this functionality in Twix rather than just suffixing it myself.
What do you think about this? :)

I assume you mean for the smart formatter. Yeah, I think that's a reasonable feature request. I'd take a PR that looks like this:

Add an additional "slot" to the smart formatter
Add a hideOffset option that defaults to true
Add an offsetFormat option that defaults to "Z"

Yes, I was talking about the smart formatter. I would gladly write a PR for this feature. I can't promise any time plan for it, however. I will do it when I get time. Feel free to close this issue in the meantime if it clutters the list. ^_^


OCM-6029 | feat: allow to edit component routes of ingress

OCM-6029

/retest

@gdbranco it seems like the ocm-sdk-go version on the master branch was updated to v0.1.409 and this PR needs a rebase.

/lgtm


Add icons to digital exhibition guides listing page

What
Update design of exhibition guides listing page (this page which displays cards for all current exhibition guides) to include icons as per designs
Screenshot as example, to be replaced with link to designs:

Icons logic:
AD = Audio highlight tour is available
TEXT = Always shows, as applicable to Exhibition text guide being available and/or transcripts as part of audio/video content
BSL = BSL video highlight tour is available
Why
To indicate what resources are available (audio description, BSL, ) as these vary depending now that we allow Exhibition text without a highlight tour for example.
Background
Part of Improve usability of exhibition guides
Done when

see design on Figma, Key Designs space

@kasiabbb this is now in prod behind the toggle. I also added the missing icons on this page https://wellcomecollection.org/guides/exhibitions/Zdcs4BEAACMA6abC (we originally said we'd wait but it's behind the toggle anyway so I went ahead with it).
Feel free to close ticket or add comments if anything is amiss!

Fix in prod  for @kasiabbb to review

@rcantin-w one final thing - the padding should be consinstent across the pages, at the moment it is consistent on desktop but not on mobile (screens attached). Could we please have it at 8x16x16 on both pages? that seems to be consistent across other pages

@kasiabbb ready for further review


Remove data from old contracts releases

What this PR does
Removed data about all old deployments before Alderaan.
Resolves #1393
Why I'm making this PR
Each deployment adds roughly 1MB of data to the package size, even though they most likely won't ever be used again.
What's tricky about this PR (if any)
It's big.

Any reviewer can check these:

[ ] If the PR is fixing a bug or adding a feature, add an entry to CHANGELOG.md.
[ ] If the PR changed a Solidity source, run make compile_contracts and add the resulting raiden_contracts/data/contracts.json in the PR.
[ ] If the PR is changing documentation only, add [skip ci] in the commit message so Travis does not waste time.

[ ] But, if the PR changes comments in a Solidity source, do not add [skip ci] and let Travis check the hash of the source.

[ ] In Python, use keyword arguments
[ ] Squash unnecessary commits
[ ] Comment commits
[ ] Follow naming conventions

solidityFunction
_solidity_argument
solidity_variable
python_variable
PYTHON_CONSTANT

[ ] Follow the Signature Convention in CONTRIBUTING.md
For each new contract

[ ] The deployment script deploys the new contract.
[ ] etherscan_verify.py runs on the new contract.

Bookkeep

[ ] The gas cost of new functions are stored in gas.json.

Solidity specific conventions

[ ] Document arguments of functions in natspec
[ ] Care reentrancy problems
if the PR adds or removes require() or assert()

[ ] add an entry in Changelog
[ ] open an issue in the client, light client, service repos so the change is reflected there
Just adding a message in require() doesn't require these steps.

[ ] When you catch a require() failure in Solidity, look for a specific error message like pytest.raises(TransactionFailed, match="error message"):

And before "merge" all checkboxes have to be checked.  If you find redundant points, remove them.

After a discussion with @konradkonrad I decided to keep the red eyes contract data for now, as it isn't deprecated yet.


Configuration is still incorrect. Do you want to edit it again?

I'm a rookie。I would like to ask you some questions.Why doesn't pycharm run after I downloaded your project file？？Error：Configuration is still incorrect. Do you want to edit it again?Do I have to configure other environments?If there are other environments to configure, can you tell me how?I thank you Ao

What should I do in this situation？？？

Please refer to https://www.cnblogs.com/TD1900/p/12934987.html and then configure your Python interpreter. Hope it could solve your problem.

LY
@.***
Thank you very much. I wish you good health and all the best!

------------------ 原始邮件 ------------------
发件人:                                                                                "Friedrich-M/Wheeled-robot-path-planning-algorithm"                                                                                @.>;
发送时间: 2023年6月16日(星期五) 上午9:57
@.>;
<EMAIL_ADDRESS>主题: Re: [Friedrich-M/Wheeled-robot-path-planning-algorithm] Configuration is still incorrect. Do you want to edit it again? (Issue #2)
Please refer to https://www.cnblogs.com/TD1900/p/12934987.html and then configure your Python interpreter. Hope it could solve your problem.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

LY
@.***
Here's what I got when I ran the code:: SyntaxWarning: assertion is always true, perhaps remove parentheses?
  assert(self.planning_path.shape[1]==2,"the shape of planning path should be '[x,2]', please check your algorithm.")
How can I solve this problem?Or do you have any good suggestions?

------------------ 原始邮件 ------------------
发件人:                                                                                "Friedrich-M/Wheeled-robot-path-planning-algorithm"                                                                                @.>;
发送时间: 2023年6月16日(星期五) 上午9:57
@.>;
<EMAIL_ADDRESS>主题: Re: [Friedrich-M/Wheeled-robot-path-planning-algorithm] Configuration is still incorrect. Do you want to edit it again? (Issue #2)
Please refer to https://www.cnblogs.com/TD1900/p/12934987.html and then configure your Python interpreter. Hope it could solve your problem.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Add a default invalid key for Bing maps

Bing maps is not supposed to be used without a key, and it even fails displaying a proper error message in that case.
This commit adds a default invalid key for quick tests allowing usage of the map in case a key is not provided as it should.

Codecov Report

Merging #1826 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1826      +/-   ##
==========================================
+ Coverage   80.65%   80.66%   +0.01%
==========================================
  Files         601      601
  Lines       33764    33764
==========================================
+ Hits        27232    27237       +5
+ Misses       6532     6527       -5

Impacted Files
Coverage Δ

src/aria/map/providers/Microsoft8MapProvider.js
100.00% <100.00%> (ø)

src/aria/core/ResMgr.js
86.53% <0.00%> (-0.65%)
:arrow_down:

src/aria/jsunit/Assert.js
87.50% <0.00%> (-0.47%)
:arrow_down:

src/aria/widgets/calendar/CalendarController.js
90.31% <0.00%> (+0.26%)
:arrow_up:

src/aria/widgets/Text.js
77.27% <0.00%> (+3.03%)
:arrow_up:

src/aria/templates/DomElementWrapper.js
89.28% <0.00%> (+3.57%)
:arrow_up:

Continue to review full report at Codecov.

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

@benouat Thank you for your review!


Weird behavior on a touchscreen (iPad)

When the demo page is opened on a iPad (Safari), the first dragging alsomakes the page scroll. Following drag and drops do not make the page canvas move and it works as expected.
Also, after this happens, it is impossible to scroll the page normally by dragging it by a non-draggable parts of the page, until I tap somewhere on the background once - then the page scrolls normally.

It is duplicate of https://github.com/kutlugsahin/vue-smooth-dnd/issues/134
Note related to iPad, happens on all touch screens


Fixing some bugs in LWM2M

Besides that prototype, everything lgtm.
Tested locally running samples and test-lwm2m as well.

Oh,
While your're at it, could you please remove  * - Bootstrap. from the list of unsupported features in sol-lwm2m.h?

:+1:
I would suggest you add fixed compiler warnings on commit msg next time.

merged