Refactor the linker interface

Library name(s)

MarkDoc.Linkers
MarkDoc.Linkers.Markdown
MarkDoc.Generators.Basic

Description
Refactor the linker interface to include a reference to the linker and a reference to the target

Total: 30m


(not a bug) question about bert create_pretraining_data.tokenize_lines()

Description
In the function scripts.pretraining.bert.create_pretraining_data.tokenize_lines()
The code snippet:
for line in lines:
        if not line:
            break
        line = line.strip()
        # Empty lines are used as document delimiters
        if not line:
            results.append([])
        else:
            #<OMITTED FOR BREVITY...>
    return results

Suggests that empty or null lines (e.g. "" or None) break the for-loop returning only the lines that have been processed so far whereas stripped-empty lines (e.g. "  ") are used as document delimiters.
Could someone shed light as to what the (empty line + break-from-loop) is meant to accomplish? Are empty/null lines used as  delimiters?

This requires a format in the text file where documents are separated by empty lines. The following example consists of 2 documents, where each line is considered as a sentence in a document.
We use wikipedia corpus.
Wikipedia is great.

We also use bookcorpus.
Bookcorpus is also helpful.

So when the tokenize_lines function reaches the end of the file, if not line is triggered and we break the loop. When the function reads the empty line between two documents, the second if not line is triggered after line.strip() on line = '\n'.


Initial implementation of hover to show inferred type in LSP

TODO:

[ ] add inferred_type field to top-level decls
[ ] get the current version of the code being analyzed which hasn't been saved yet

Codecov Report

Merging #409 (8c3197f) into main (44171b8) will decrease coverage by 1.21%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #409      +/-   ##
==========================================
- Coverage   91.59%   90.37%   -1.22%
==========================================
  Files          53       54       +1
  Lines       15043    15247     +204
==========================================
+ Hits        13779    13780       +1
- Misses       1264     1467     +203

Impacted Files
Coverage Δ

crates/crochet_ast/src/values/expr.rs
75.55% <ø> (ø)

crates/crochet_lsp/src/main.rs
0.57% <0.00%> (-0.39%)
:arrow_down:

crates/crochet_lsp/src/visitor.rs
0.00% <0.00%> (ø)

crates/crochet_dts/src/parse_dts.rs
88.99% <0.00%> (ø)

crates/crochet_parser/src/lib.rs
94.39% <0.00%> (+0.05%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codecov Report

Merging #409 (8c3197f) into main (44171b8) will decrease coverage by 1.21%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #409      +/-   ##
==========================================
- Coverage   91.59%   90.37%   -1.22%
==========================================
  Files          53       54       +1
  Lines       15043    15247     +204
==========================================
+ Hits        13779    13780       +1
- Misses       1264     1467     +203

Impacted Files
Coverage Δ

crates/crochet_ast/src/values/expr.rs
75.55% <ø> (ø)

crates/crochet_lsp/src/main.rs
0.57% <0.00%> (-0.39%)
:arrow_down:

crates/crochet_lsp/src/visitor.rs
0.00% <0.00%> (ø)

crates/crochet_dts/src/parse_dts.rs
88.99% <0.00%> (ø)

crates/crochet_parser/src/lib.rs
94.39% <0.00%> (+0.05%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Could you olease update triton support to 0.3.0, old version of that even do not have a doc.

I can't install triton==0.2.3, I can't know what version of LLVM does that need. They even do not have doc of old version.

Hi @PistonY
I have added some changes in this PR  that uses the new version of Triton. Can you please try it to see if it solves the LLVM issue?
Thanks,
Reza


Add cookies pop-up

We have to show a modal where we tell to users we store some minimal information from them, and add the options to confirm/deny it, do we track something with Google Analytics, Cloudflare or other? @Tito-Kati

@JesusValera We have server-side analytics with Cloudflare, so we don't need to put one of those annoying cookie modals since the information we track from our users is 100% anonymous. 👍🏻 🌈


Stack Identification Bug Fix

Fixes #100
There were two problems with my previous implementation: not traversing to the returnTarget of calls, and counting the rhs of an assignment as a stack value if it contained a stack reference inside a load index or store.

Looks good, thanks


Add support for multiple oracles per chain

Related with the following issue: https://github.com/gnosis/safe-config-service/issues/209

The config-service now returns a collection of oracles (instead of a single oracle) so the deserialisation on the CGW side was adjusted. The payload exposed to the clients was also adjusted to reflect this new collection.

Opened as a draft in order to add a test which would test the collection (multiple entries or no entries). Currently I just adjusted the current tests that we have.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

25 of 34 (73.53%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 84.146%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/models/converters/chains.rs
12
15
80.0%

src/models/tests/chains.rs
9
15
60.0%

Files with Coverage Reduction
New Missed Lines
%

src/models/converters/chains.rs
1
77.42%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
6916

Relevant Lines:
8219

💛  - Coveralls

Opened as a draft in order to add a test which would test the collection (multiple entries or no entries). Currently I just adjusted the current tests that we have.

Done ✔


[Feature] Add PropertyConditionals as utility class

This pull request adds a new utility class named PropertyConditional:

They work the same as PropertyExpressions in a way that the patterns are made for you
A new function is used named SelfRegistrable: these classes can register themselves.

They don't actually automatically register, but the register process is just the same. This way we don't have to add new methods like addPropertyCondition() for each type of syntax (obviously every syntax type can now be transformed into a SelfRegistrable class, but we keep those methods as utility.
Note that I explicitly did not check for casting errors in the PropertyConditional implementation of the register method, since developers hard-code there syntax anyway and should take care of the errors themselves. If wanted I'll change this.

I think this implementation is highly valuable since it reduces the amount of methods needed to implement to only one (!!) for basic implementations.

Furthermore, I fixed a small grammar mistake and refactored some files.

Yeah, but I think you can still approve, since it doesn’t let me merge with conflicts. I’ll fix them tomorrow.


Provide access to FlatpickrInstance from CommonJS

In a CommonJS envelopment (such as node or browserify) you only have access to the flatpickr constructor function not the actual class. It would be nice to be able to access the prototype from a CommonJS environment. This would be useful to, for example, use the formatDate function.

Hello,
Sorry for the inconvenience.
The static parseDate and formatDate methods will be available by next release.
Thank you.


Update maps hot reload setup

live-server is now in dev dependencies so we do not need to install it separately
the live-server path in the docs was broken and I fixed it
updated the docs

Essentially we can develop maps by running the GH server, npm run watch and npm run liveServer in a separate terminal. However, we still need to set the GH server port in options.js. I was hoping to get rid of this using browserify-swap (see my first commit), but this is not working reliably (at least on my machine). After a few updates the wrong options.js is pulled in and we have to restart the terminal with npm run watch. Without browserify-swap the UI is also sometimes in a broken state (probably because live-server loads main.js before it was finished building), but this can easily be fixed with a browser refresh.

Maybe these options could be useful for live-server:

--wait=MILLISECONDS - (default 100ms) wait for all changes, before reloading
--no-browser - suppress automatic web browser launching
--browser=BROWSER - specify browser to use instead of system default

and actually if we do not want the automatic browser reload we could use a simple server and hit refresh ourselves. The important part is just that we serve GH maps from the file system where main.js is updated.


[DOCS] - install packages

**What section of the docs could be improved? **
I went through the steps in the docs. But it seemed that the py_client was not installed when I ran bootstrap-local-cluster and then the dovtenv was not installed (or requirements.txt was not installed) at any point.
I may have gone through the steps in a wrong order, or made a mistake. But perhaps we can check this when we get a few user testers going through the flow
Is it correct that:

py_client is installed on bootstrap-local-cluster?
requirements.txt is installed when I setup virtual env?

Fixed - all dependencies needed are in requirements.txt


Security vulnerabilities in x/text: CVE-2020-28852 / 2020-28851 / 2020-14040

Description
In x/text in Go before v0.3.5, a "slice bounds out of range" panic occurs in language.ParseAcceptLanguage while processing a BCP 47 tag. (x/text/language is supposed to be able to parse an HTTP Accept-Language header.)
In x/text in Go 1.15.4, an "index out of range" panic occurs in language.ParseAcceptLanguage while parsing the -u- extension. (x/text/language is supposed to be able to parse an HTTP Accept-Language header.)
The x/text package before 0.3.3 for Go has a vulnerability in encoding/unicode that could lead to the UTF-16 decoder entering an infinite loop, causing the program to crash or run out of memory. An attacker could provide a single byte to a UTF16 decoder instantiated with UseBOM or ExpectBOM to trigger an infinite loop if the String function on the Decoder is
Suggested Solution

Upgrade to version golang-golang-x-text-dev - 0.3.6-1,0.3.6-1 or higher (latest version v0.10.0) - included in new Go versions

CVEs

2020-28852
2020-28851
CVE-2020-14040

Suggested Fix: https://github.com/mandelsoft/vfs/pull/6


css: Move alert.scss to common/alert.scss.

This PR is a follow up of #14411 .It moves "static/styles/alert.scss" to "static/styles/common/alert.scss" so that it can be used in portico pages as well.

I think you should now include /styles/common/alerts.scss" in common.js since that file is required by both app.js and portico.js.
That means you would have to change various portico files that has alert and other classes in alerts.scss to fit the new styling changes. Changes like showing the alert when required for example since the behavior of alert class used currently in portico is to be shown while that of alert.scss is to be hidden.
I would wait for @timabbott to confirm this is the direction we want to take before doing further work on this.

I think it's a useful direction to merge these definitions of alert, but I also agree with Vishnu that we need to do do those changes, likely delete the existing alert CSS on portico pages, and then actually test every single use of the alert classes on the portico pages.

@hackerkid I had updated the PR by renaming the alert classes in portico pages to portico-alert* like naming (I could not think of a better name). I had tested all the use of new alert classes in the portico to the best of my knowledge. Please review it.
The circleci test is failing due to an error that is reported in automated testing.

@majordwarf can you help review this?

@SiddharthVarshney can you help review this?


[Backend] Docker builds

Create docker builds collection which triggers somewhat like https://github.com/GoogleContainerTools/kaniko builds and if successfull it should trigger a new rollout with the new release.
There should also be a repositories collection which watches git repositories on a branch and path (with auth) and on push or on sync it should trigger a new build.

This won't get implemented due to the complexity. Also we wanna strictly separate the CI from the CD.
But what we will do instead to automate the tag rollout while watching on a registry.


Is Youtube now restricted to 500 kbps downloads - seems so to me?

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE
Over the past months I had this on and off, now permanent.
Has Youtube identified youtube-dl on all servers and now restrict download bandwidth?
As it was on and off, I could just abort download and try again when I saw 50 KiB/s and after one or a couple or retries it becomes normal and it download in MiB/s again.
Now I cannot get anything but around 50 KiB/s no matter what I retry. Since yesterday just about.
I have perfect and fast download on rumble and bitchute, so youtube related only. And on those it can start as low but then quickly gear up to MiB/s, but very rarely.
As retired programmer, I get the feeling YT is identifying application, now on every server they have and it has become permanent. But this is a guess, The HTTP User-Agent thingy is what I am thinking of.
It could be youtube restricted due to load or something, not sure. Or just annoyed for lost ad views or something.
I fixed a document that I load urls into, and a bat file to do either youtube, rumble or bitchute download in various resolutions.
I have 100/100 Mbps fiber and works fine on other hosts as I mentioned.
Running latest youtube-dl from june 2021.
It's been flawless for many years, first as part of AVC then last year as separate app I use separately.
I there a way to tell youtube-dl which user agent to use, many browsers has this kind of plugin swapping what you get identified by?
Just a heads up from me anyway.
Thanks for all the efforts from devs on this app.

i don't know if that is the issue that was already present a while ago, but a few hours ago i had a lot of issues with youtube (normal watching, not downloading).. so could be server issues on youtubes side. couldn't even load 420p for me. when i looked it up online, it looked like a ton of other people had the same issue.
but youtube seems to throttel from time to time and i don't know if it really got fixed finally.. but it seemed to happen less for me since the last bugfix for it. so i would say its probably server issues on youtubes side this time.

youtube-dlg is basically a graphical user interface for youtube-dl. it gives you buttons, text fields etc. for a lot of settings so you can use them easier without needing a lot of commands in the cmd.. that makes it easier to test arround if you have issues or errors in a lot of cases.


Array conv_23/Conv_1/BatchNorm/FusedBatchNorm, which is an input to the Conv operator producing the output array conv_3//0/Conv/Relu6, is lacking min/max data, which is necessary for quantization.

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):NO
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):win10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):conda install tensorflow-gpu==1.13
TensorFlow version (use command below):1.13
Python version: 3.6
Bazel version (if compiling from source):NO
GCC/Compiler version (if compiling from source):NO
CUDA/cuDNN version: CUDA9.0 cuDNN7.1
GPU model and memory: p40

Describe the current behavior
I quantized training mobilefacenet,   and want to conver pb file to tflite file but failed
toco --output_file=face_freeze.tflite --graph_def_file=face_freeze.pb --input_arrays=x_i
nput  --output_arrays=BatchNorm/Reshape_1 --output_format=TFLITE --inference_typ
e=QUANTIZED_UINT8 --mean_values=127  --std_dev_values=128
Code to reproduce the issue
`def model(is_training , x, y, is_quantize ):
with slim.arg_scope([slim.conv2d, slim.fully_connected],
                    activation_fn = tf.nn.relu6,
                    weights_initializer=slim.variance_scaling_initializer(factor=3.0, mode='FAN_IN', uniform=True),
                    biases_initializer = tf.constant_initializer(0.0),
                    normalizer_fn=slim.batch_norm,
                    normalizer_params={'is_training': is_training, 'epsilon':0.001, 'scale': True, 'decay':0.9, 'updates_collections': tf.GraphKeys.UPDATE_OPS}):

    print('x',x.get_shape())
    # conv_1
    conv_1 = slim.conv2d(x, 64, [3, 3], stride=2, padding='SAME',  scope='conv_1')
    print('conv_1',conv_1.get_shape())

    # conv_2
    conv_2 = Residual(conv_1, num_block=2, num_out=64, _kernel=3,  _stride=1, pad='SAME', num_group=64, name='conv_2')
    print('conv_2',conv_2.get_shape())

   # conv_23
    conv_23 = DResidual(conv_2, num_out=128, _kernel=3 , _stride=2, pad='SAME', num_group=128, name='conv_23')
    print('conv_23',conv_23.get_shape())

   # conv_3
    conv_3 = Residual(conv_23, num_block=6, num_out=128, _kernel=3, _stride=1, pad='SAME', num_group=128, name='conv_3')
    print('conv_3',conv_3.get_shape())
    .....
    .....

    #softmax loss
        cross_entropy__ = tf.nn.sparse_softmax_cross_entropy_with_logits(
            labels=y, logits=fc7, name='cross_entropy_per_example')
    cross_entropy = tf.reduce_mean(cross_entropy__, name='cross_entropy')

    all_variable_decay = _all_variables_decapy(weight_decay,1.0)
    total_loss = cross_entropy + all_variable_decay

    tf.contrib.quantize.create_training_graph()`

I use tf.contrib.quantize.create_eval_graph() to create eval graph pb file
Other info / logs
2019-04-18 19:21:14.747016: F tensorflow/lite/toco/tooling_util.cc:1702] Array c
onv_23/Conv_1/BatchNorm/FusedBatchNorm, which is an input to the Conv operator p
roducing the output array conv_3//0/Conv/Relu6, is lacking min/max data, which i
s necessary for quantization. If accuracy matters, either target a non-quantized
output format, or run quantized training with your model from a floating point
checkpoint to change the input graph to contain min/max information. If you don'
t care about accuracy, you can pass --default_ranges_min= and --default_ranges_m
ax= for easy experimentation.
Looking forward to receive your reply,  thanks very much

I am facing a similar issue with Cifrarnet as well. I have trained the cifarnet model with quantization awareness. The model even converts into frozen model. During the conversion with tflite_converter, I get the following error:
2019-05-05 08:56:39.657151: F tensorflow/lite/toco/tooling_util.cc:1702] Array CifarNet/norm1, which is an input to the Conv operator producing the output array CifarNet/conv2/Relu, is lacking min/max data, which is necessary for quantization.
If accuracy matters, either target a non-quantized output format, or run quantized training with your model from a floating point checkpoint to change the input graph to contain min/max information.
If you don't care about accuracy, you can pass --default_ranges_min= and --default_ranges_max= for easy experimentation.
Aborted (core dumped)

Are there certain ops for which fake quant node needs to be manually added? Any insight into this will be of great help.
Thanks.

@sxsxsx Is this still an issue?
We see that you are using an older version of TF(1.x) which is not actively supported.We recommend you to kindly upgrade to TF v2.4 or later and let us know the outcome? Please refer to this thread  and let us know if it helps?
Thanks!


envsubst: init at 1.1.0

The envsubst command in the gettext package is missing a feature: sometimes, you want it to fail if the document refers to an unset variable, to detect certain typos and incorrect invocations.  This tool is a go implementation of envsubst with that and a few other features.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

~[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)~
~[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"~
[x] Tested execution of all binary files (usually in ./result/bin/)
~[ ] Determined the impact on package closure size (by running nix path-info -S before and after)~
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build envsubst

@GrahamcOfBorg build envsubst

Success on aarch64-linux (full log)
Attempted: envsubst
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/hva0v0n9jjdimzaava0wc6p04qdh8xd3-envsubst-1.1.0-bin
shrinking /nix/store/hva0v0n9jjdimzaava0wc6p04qdh8xd3-envsubst-1.1.0-bin/bin/envsubst
cannot find section .dynamic
strip is /nix/store/428gs2z4b8f9byvghzlpbjwjb3a7jwxx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hva0v0n9jjdimzaava0wc6p04qdh8xd3-envsubst-1.1.0-bin/bin
patching script interpreter paths in /nix/store/hva0v0n9jjdimzaava0wc6p04qdh8xd3-envsubst-1.1.0-bin
checking for references to /build in /nix/store/hva0v0n9jjdimzaava0wc6p04qdh8xd3-envsubst-1.1.0-bin...
cannot find section .dynamic
strip is /nix/store/428gs2z4b8f9byvghzlpbjwjb3a7jwxx-binutils-2.30/bin/strip
/nix/store/hva0v0n9jjdimzaava0wc6p04qdh8xd3-envsubst-1.1.0-bin

Success on x86_64-darwin (full log)
Attempted: envsubst
Partial log (click to expand)
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
find: '/nix/store/j0qdj9zk6mxz7d2hz5d0dq6x7aygj5mr-envsubst-1.1.0': No such file or directory
/nix/store/j8qag8gq7aqb7h280y6imfks5jzbpw9i-envsubst-1.1.0-bin

Success on x86_64-linux (full log)
Attempted: envsubst
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/ymx2k3anxk8s46gbkgs4rk8r1fyx3iik-envsubst-1.1.0-bin
shrinking /nix/store/ymx2k3anxk8s46gbkgs4rk8r1fyx3iik-envsubst-1.1.0-bin/bin/envsubst
cannot find section .dynamic
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ymx2k3anxk8s46gbkgs4rk8r1fyx3iik-envsubst-1.1.0-bin/bin
patching script interpreter paths in /nix/store/ymx2k3anxk8s46gbkgs4rk8r1fyx3iik-envsubst-1.1.0-bin
checking for references to /build in /nix/store/ymx2k3anxk8s46gbkgs4rk8r1fyx3iik-envsubst-1.1.0-bin...
cannot find section .dynamic
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
/nix/store/ymx2k3anxk8s46gbkgs4rk8r1fyx3iik-envsubst-1.1.0-bin

Thanks


Feature/recipes model

This one was an overhaul to get Recipes as its own model. I tested the routes/controllers using postman. Everything is working on my end, but please let me know if there is more to do before merging with main. Thanks!

Get all recipes per authorized user
Get single recipe per authorized user
Create recipe per authorized user (saves to Recipe model first, then saves recipe to user's saved recipes)
Delete recipe (only within authorized user)
Recipe is its own model
Simplifies route names
Omits unnecessary Mongoose version number when response is sent to client

The '__v' is a field Mongoose auto generates called the versionKey when a
document is created through Mongoose (not MongoDB). I don't want to say
it's extraneous information, but putting the  '-' in front of the '__v'
when used with a 'select' query will not show that field or its values
(which is usually just a '0').
It just makes the response a tad prettier lol.

LGTM
In routes/recipes.js
https://github.com/newtritious/Newtritious/pull/65#discussion_r790191611
:

);

/* ------------------------------ SAVE A RECIPE ----------------------------- */
app.post(
'/',
passport.authenticate('jwt', { session: false }),
async function (req, res) {

 const spoonacularRecipeId = req.body.id;

 try {

   const user = await User.findOne({

     _id: req.user._id

   });

   let recipe = await Recipe.findOne({ id: spoonacularRecipeId }).select(

     '-__v'

what's the significance of the '-__v'?
—
Reply to this email directly, view it on GitHub
https://github.com/newtritious/Newtritious/pull/65#pullrequestreview-860247160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANHGG3DSTUQD7S6OHKUHUYLUXMWFNANCNFSM5MI4B4XQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


R.isEmpty(document.querySelectorAll('thereisnospoon')) returns false

document.querySelectorAll returns NodeList and while:
R.length(document.querySelectorAll('thereisnospoon'))
> 0

Ramda see it as not empty:
R.isEmpty(document.querySelectorAll('thereisnospoon'))
> false

And that's because:
 empty(document.querySelectorAll('thereisnospoon'))
> undefined

I reviewed empty.js and am definitely scared. So I will not try to make PR.
Thanks
p.s. Why undefined is "not empty"?

Hi @slavaGanzin thanks for bringing this up. I definitely think this is a bug that we aren't handling empty NodeLists. I think that it's an even bigger problem than just empty because R.equals also doesn't handle NodeLists in a satisfactory way. So even if we updated empty then isEmpty would fail when it tried to do the equality check.
I think equals is an even trickier function than empty but this is definitely something we need to discuss and fix.

p.s. Why undefined is "not empty"?

undefined comes up true for isNil, but due to it being a single value it isn't the sort of thing that can be empty (it's not fillable in some sense). The same thing applies to 0 and false. It just doesn't make sense for those to be empty in the same sense as an Array is empty.

Like many collections (Strings, Maps, Arrays, ...) NodeLists too conform to the iterable protocol. I.e. an iterator is obtained by var it = document.querySelectorAll("meh")[Symbol.iterator]().
Although incorporating iterables as Ramda collections has been discussed here extensively it has never materialized yet.
benji6/imlazy works on iterables and has an API like Ramda:
IL.isEmpty(document.querySelectorAll("meh")); // true
IL.isEmpty(document.querySelectorAll("body")); // false

I reviewed empty.js and am definitely scared. So I will not try to make PR.

:rofl: :rofl: Thanks for that sentence! Yes, that describes exactly what I have often felt with Ramda!
Without understanding empty.js, perhaps one could just isEmpty = it.next().done; instead of the ugly chain _isArray(x) ? [] : _isString(x) ? '' : _isObject(x)?

After #2593:
function empty(x) {
  const len = length(x)
  return Number.isNaN(x)
    ? x != null && equals(x, empty(x))
    : len === 0;
}


[HOLD] Error message on Add Phone number page is not changing to new selected language

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

Action Performed:

Go to Settings - Profile - Phone number
Input any wrong phone number and password
Press Send Verification button, Observe error messages for input fields
Change language in another session
Observe error messages on Add Phone number page (everything else changed to new selected language)

Expected Result:
Error message on Add Phone number page should be changed to new selected language
Actual Result:
Error message on Add Phone number page is not changing to new selected language
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] MacOS / Chrome / Safari

Version Number: v1.2.50-14
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/211359904-062a2533-0c25-42cc-8b87-1ead119b496c.mp4
https://user-images.githubusercontent.com/43996225/211359943-c272f50d-21a8-4519-9cb8-2a5b60697675.mov
Expensify/Expensify Issue URL:
Issue reported by: @varshamb
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1673017112612729
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0136e59b29ad333b3b
Upwork Job ID:<PHONE_NUMBER>970121728
Last Price Increase: 2023-01-10

Reviewing this now.

I'm unable to recreate this because I don't have the option to change the language in Echat.

@thesahindia or @MonilBhavsar , can you please let me know if you are able to reproduce this?

Proposal
AddSecondaryLoginPage will require refactoring to use the Form.
I can do the refactoring.

Proposal
Root Cause
When we get wrong data, we set the error string to be displayed on the validation page directly in the onyx - https://github.com/Expensify/App/blob/main/src/libs/actions/User.js#L152
But this translation only happens when the user presses submit. It won't happen at runtime similar to this bug.
Solution
Instead of setting the actual error string in Onyx, set the localize key in the onyx. And localize the error directly in the view.
I have added one sample for the error mentioned in the bug, this sample can be extended to other error strings in the phonenumber flow
diff --git a/src/libs/actions/User.js b/src/libs/actions/User.js
index 532e7bb04c..72946e3d41 100644
--- a/src/libs/actions/User.js
+++ b/src/libs/actions/User.js
@@ -149,7 +149,7 @@ function setSecondaryLoginAndNavigate(login, password) {
             error = 'This login already belongs to an existing Expensify account.';
         }
         if (error.includes('I couldn\'t validate the phone number')) {
-            error = Localize.translateLocal('common.error.phoneNumber');
+            error = 'common.error.phoneNumber';
         }

         Onyx.merge(ONYXKEYS.USER, {error});
diff --git a/src/pages/settings/AddSecondaryLoginPage.js b/src/pages/settings/AddSecondaryLoginPage.js
index 892a75153d..1522407305 100755
--- a/src/pages/settings/AddSecondaryLoginPage.js
+++ b/src/pages/settings/AddSecondaryLoginPage.js
@@ -140,7 +140,7 @@ class AddSecondaryLoginPage extends Component {
                     </View>
                     {!_.isEmpty(this.props.user.error) && (
                         <Text style={styles.formError}>
-                            {this.props.user.error}
+                            {this.props.translate(this.props.user.error)}
                         </Text>
                     )}
                 </ScrollView>

https://user-images.githubusercontent.com/46707890/211636598-16ff65eb-d60e-4ea1-8cfc-baa9c661fbba.mov

Proposal
RCA
The reason that the error message is not translated is because we are not using the standard Form.js for this component. Moreover, the error message isn't translated either.
Solution 1
We can refactor the component to use Form.js, which will trigger the validation method when the component is re rendered on localization change.
Solution 2
We can make use of the componentDidUpdate() to trigger the error message to be refreshed on localization change.
/**
diff --git a/src/libs/actions/User.js b/src/libs/actions/User.js
index 532e7bb04c..7d0ef74128 100644
--- a/src/libs/actions/User.js
+++ b/src/libs/actions/User.js
@@ -398,6 +398,11 @@ function clearUserErrorMessage() {
     Onyx.merge(ONYXKEYS.USER, {error: ''});
 }

+function setErrorMessage(key) {
+    const error = Localize.translateLocal(key);
+    Onyx.merge(ONYXKEYS.USER, {error});
+}
+
 /**
  * Clear the data about a screen share request from Onyx.
  */
@@ -471,4 +476,5 @@ export {
     deletePaypalMeAddress,
     addPaypalMeAddress,
     updateChatPriorityMode,
+    setErrorMessage,
 };
diff --git a/src/pages/settings/AddSecondaryLoginPage.js b/src/pages/settings/AddSecondaryLoginPage.js
index 892a75153d..3e5a4e980f 100755
--- a/src/pages/settings/AddSecondaryLoginPage.js
+++ b/src/pages/settings/AddSecondaryLoginPage.js
@@ -64,6 +65,16 @@ class AddSecondaryLoginPage extends Component {
         this.setState({login});
     }

+    componentDidUpdate(prevProps) {
+        if (prevProps.preferredLocale === this.props.preferredLocale) {
+            return;
+        }
+
+        User.setErrorMessage('common.error.phoneNumber');
+    }
+

https://user-images.githubusercontent.com/54077356/211638524-4dbb2318-a7a4-4e1d-9c45-a3b7c9dc1611.mp4

As mentioned here, I think we might want to refactor AddSecondaryLoginPage to use Form, since we're already validating the form here if the language changes in Form.

This issue is a duplicate and it is present at multiple places, some instances were mentioned at https://github.com/Expensify/App/issues/8714. We haven't decided how we wanna approach these issues (discussion) I think we will need to discuss it in slack first but as mentioned by @iwiznia, we are not trying to fully internationalize the app right now. So I think we should just close this and discuss this later.

@thesahindia Just in case you missed this

Is that validation happening on the frontend or the server? If the former, I think this is a bug (though quite an obscure one that needs 2 tabs), but if the latter, I think we should close.

Yeah, it's from the backend.

Hmm, this is workflow is still using Deprecated API and requires to be cleaned up. That will probably fix this issue. Going to make in Internal

not overdue.

Not overdue, on hold.

Still on hold

Still on hold

Still on hold

Still on hold - internal.

Still on hold

Not overdue, on hold!

Not overdue, on hold!

Not overdue.

Based on the latest comments, I'm closing this!


Add Geonear instructions to ReadMe. Closes #1878

Hey guys!
Here it is a PR with the geoNear instructions.
That closes #1878

Codecov Report
Base: 88.21% // Head: 88.18% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (d365697) compared to base (e5e9193).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2487      +/-   ##
============================================
- Coverage     88.21%   88.18%   -0.03%
- Complexity      696      697       +1
============================================
  Files            32       32
  Lines          1595     1701     +106
============================================
+ Hits           1407     1500      +93
- Misses          188      201      +13

Impacted Files
Coverage Δ

src/Eloquent/HybridRelations.php
87.64% <0.00%> (-7.10%)
:arrow_down:

src/Queue/Failed/MongoFailedJobProvider.php
21.05% <0.00%> (-1.17%)
:arrow_down:

src/Queue/MongoQueue.php
100.00% <0.00%> (ø)

src/MongodbServiceProvider.php
100.00% <0.00%> (ø)

src/Auth/DatabaseTokenRepository.php
0.00% <0.00%> (ø)

src/Concerns/ManagesTransactions.php
100.00% <0.00%> (ø)

src/Validation/ValidationServiceProvider.php
100.00% <0.00%> (ø)

src/Relations/EmbedsOneOrMany.php
98.14% <0.00%> (+0.01%)
:arrow_up:

src/Relations/EmbedsMany.php
97.24% <0.00%> (+0.16%)
:arrow_up:

src/Helpers/QueriesRelationships.php
91.37% <0.00%> (+0.30%)
:arrow_up:

... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add Spyable

I'm thrilled to propose my macro: Spyable - a Swift macro that simplifies and automates the process of creating spies for testing.
I would love to hear your feedback about this macro, the pros and cons you've found, and any suggestions for improvements you might have.

This is cool! looks very similar to what AutoMockable did in Sourcery, was it inspired by it by any chance?

I believe in some use cases this macro can become a replacement for AutoMockable. To ensure a smooth and seamless transition from Sourcery to the Spyable macro, the initial version of Spyable is designed to generate spies for protocols in the same way as the AutoMockable template. Thanks to that, most projects using Sourcery with the basic AutoMockable template can be switched in minutes.
I would love to introduce new features in the next versions, which were very hard to persuade with Sourcery.


chore: bzlmod

Fixes #79

Maybe this needs to wait until the node_modules isn't vendored?

@gregmagolan rebased on the removal of vendored jest, but I'm stumped on the test failure under bzlmod...

it was rules_js registering an old version of node :(

rules_js 1.24.1 resolves the bzlmod node toolchain issue


PSR-15 support

This repository have the "PSR-15" tag but, the SessionMiddleware class does not implement the MiddlewareInterface interface from the PSR-15

That's true. I think I added this tag, because it's planned add a PSR-15 middleware.

The next major version will support the PSR-15 middeware as new default middleware.


Address arch board feedback for Opinion Mining design

[ ] Remove [EditorBrowsable(EditorBrowsableState.Never)] from single method AnalyzeSentiment(string document, string language) and AnalyzeSentimentAsync(string document, string language)
[ ] Consider making IncludeOpinionMining an enum flag like https://apiview.dev/Assemblies/Review/a17b6175b93847db83a99c7fcfeca9ab#Azure.Storage.Blobs.Models.BlobStates.
[x] Consider renaming MinedOpinion to MinedOpinions -> that way you know you are going to get opinions (English)
[ ] Hide public static SentenceSentiment SentenceSentiment(TextSentiment sentiment, string text, double positiveScore, double neutralScore, double negativeScore); in TextAnalyticsModelFactory

Consider renaming MinedOpinion to MinedOpinions -> that way you know you are going to get opinions (English)

V-team talked about this and decided to leave as is. The reason, if renamed, it will look like:
SentenceSentiment:
  Text
  Sentiment
  ...
  List<MinedOpinions> MinedOpinions

Both object and property plural might confuse the user into what the difference is, and because of that we decided not to rename

Consider making IncludeOpinionMining an enum flag like

I took a look at how other libraries in .NET manage this, read the documentation, and asked the service if we could have multiple analysis executed in a single document. i.e. apply opinion mining, and emotion based analysis into this doc.
Answer is yes.
Here is a proposal for .NET API View
Before the change:
await client.AnalyzeSentimentAsync(document, options: new AnalyzeSentimentOptions() { IncludeOpinionMining = true });

After the proposed change:
await client.AnalyzeSentimentAsync(document, options: new AnalyzeSentimentOptions() { IncludeAnalysis = AnalyzeSentimentType.OpinionMining });

I do like this proposal as it will allow .NET users to do something like
IncludeAnalysis = AnalyzeSentimentType.OpinionMining | AnalyzeSentimentType.EmotionBased

which means apply those 2 types of analysis for the same document, in the same call


能看的网址给个谢谢欢迎来讨论

关于能看的网址给个谢谢欢迎来讨论
>>>>>>点/此/进/入/最/新/观/看/入/口<<<<<<
新娘的婚纱一般为它的光芒添上了几分淡淡的浮晕在悄无声息间这熙和的阳光默默的照射在大地上像是母亲的手轻轻抚摸熟水

你可以将能看的网址给个谢谢欢迎来讨论	9、Youcannotappreciatehappinessunlessyouhaveknownsadnesstoo.不知道什么是忧伤，就不会真正感激幸福。

可以看到url谢谢欢迎讨论吗
O five uncle ha ha laugh, laugh after he put the fish bow carry to my bowl, then patted me on the head and said, "boy, you don't see there are so many fishing boats in the erhai lake?
They are all in the fishing?
Fish, born to eat, the zha will be cruel?"

请参考下方链接：
https://github.com/chiquane/rknxcsn/issues/2
https://github.com/powercvt/efq-nxvlo/issues/1
https://github.com/dedesew/whgq
https://github.com/fowerv/4673234/blob/main/README.md
https://github.com/goodiders/twyoqs
https://github.com/makedete/wazqs
https://github.com/tooere/ecggvo/issues/2
https://github.com/aoyouzg/gzldx/issues/8
https://github.com/chiquane/ooyzfm/blob/main/README.md
https://github.com/dedesew/226481/issues/3
https://github.com/makedete/ihghc/issues/2
https://github.com/tryuits/oif-xakru
https://github.com/whathoog/zfentjc
https://github.com/linevs/szpwh/blob/main/README.md
https://github.com/downdesa/hqlfywf/issues/1
https://github.com/aoyouzg/lqqrkqc/issues/2
https://github.com/makedete/fivaexo
https://github.com/makedete/thg-thggl/blob/main/README.md
https://github.com/aoyouzg/oxrby/issues/7
https://github.com/tuderw/1072308/issues/2


Fix ammend webstart

Codecov Report

Merging #2481 into dev will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             dev   #2481   +/-   ##
=====================================
  Coverage   94.6%   94.6%
=====================================
  Files        160     160
  Lines      19452   19452
=====================================
  Hits       18403   18403
  Misses      1049    1049

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 1adcc75...16bfc2c. Read the comment docs.


Use JsonPath expressions only ( no need for specific expression lang )

What would you like to be added:
Currently we allow the use of expressions. We should really only use JsonPath expressions everywhere. This eliminates the need for specific expression lang impls -> improve portability.
Why is this needed:

PR https://github.com/serverlessworkflow/specification/pull/97

solved by https://github.com/serverlessworkflow/specification/commit/02d9b22d8a2cd16e821b575b158cbe6480009143


Variable-size ckcallbacks

Original author: Jonathan Lifflander
Original issue: https://charm.cs.illinois.edu/redmine/issues/79

Instead of byte-pupable ckcallbacks, change them to be variable-size.
Outstanding work:

Fix use in CkReductionMsg
Implement semi-analogous structure for callbacks in messages
Test more complex cases using Thread callbacks with ChaNGa
Merge into mainline

Original author: Jonathan Lifflander
Original date: 2013-05-24 00:31:38

This will be obviated by the objid in ckcallbacks. Hence, a non-issue

Original date: 2013-09-17 20:19:32

De-target since it was rejected.


Fix typo in docs

Found a spelling mistake -
pronounciation > pronunciation
Refer to http://english.stackexchange.com/questions/5732/why-is-the-spelling-of-pronounce-and-pronunciation-different

@pra85 I'd love to merge this, but would you mind signing the CLA first? If not, let me know and I can correct this in a different PR.

@technicalpickles Sorry missed the CLA. I have signed it now


crontab doesn't update after cap deploy

Hi my application users capristone for deployement. But when i do a deploy the crontab doesnt get updated. However if i do "whenever --update-crontab" it does get updated.
this is how my deploy.rb looks like
set :whenever_command, "bundle exec whenever"
require 'whenever/capistrano'
any idea why it is not working?

If you're using Capistrano 3 you put the require in your Capfile:
require 'whenever/capistrano'

https://github.com/javan/whenever#capistrano-v3-integration

what do you mean by capfile? is it same as deploy.rb?

You should have both, like so:
your_project
├── Capfile
└── config
    └── deploy.rb

I have the same problem.
I use capistrano v3.4.0 and whenever v0.9.4 and rbenv. And I have

require 'whenever/capistrano'

in my Capfile. But crontab isn't updated after cap production deploy. And I see no whenever commands in my deploy log.

Did you have
set :whenever_identifier, ->{ "#{fetch(:application)}_#{fetch(:stage)}" }

```
in you deploy.rb?

Maybe it would be easier to write to crontab the path with current symlink and not the real directory name?

This failed for us using capistrano v3.4.0. Rolling back to cap v3.3.5 fixed the issue for us.

Also having no luck with capistrano v3.4.0 and whenever 0.9.4

It took some time for me to set this up, but I found out set you need to set whenever_roles correctly depending on your environment:
set :whenever_roles, [:db, :mail, :tasks]

With this extra configuration, everything is working for me with capistrano v3.5.0 and whenever 0.9.4. For testing, you can also use
set :whenever_roles, :all

@lacco Before you explicitly set :whenever_roles, did you have any servers in the :db role, which is the default?


version:10.0.19041.1381, i  need help

10.0.19041.1381

Duplicate of https://github.com/stascorp/rdpwrap/issues/1630

10.0.19041.1381

Attach your RDP Screenshot and your termsrv.dll

Nice, but there's still no file (termsrv.dll)

Nice, but there's still no file (termsrv.dll)

i ran uninstall.bat,then i could't connect my computer through mstsc,myabe  something went wrong with the file(termsrv.dll)

something went wrong with the file(termsrv.dll)

Attach your termsrv.dll!

termsrv.zip

termsrv.zip

@sebaxakerhtc

There's no pdb file from M$ for this version
вт, 7 дек. 2021 г., 05:00 CrazySciurini @.***>:

termsrv.zip
https://github.com/stascorp/rdpwrap/files/7647897/termsrv.zip
@sebaxakerhtc https://github.com/sebaxakerhtc
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/1644#issuecomment-987522752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHZDR4T4WPTWXP6A3C2YOQTUPV2GLANCNFSM5JCJ6HUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

thank you all the same! @sebaxakerhtc

all stuff in green but only one user per session

And the user wen going to login resive a messaje that is another user in the session continue any way yes or not
I don want to change de termsrv.dll cos in have a clean instalation make today, and I don´t want to do any one no more
but I don want to lose my job. Help pleas how I could fix it?

I have this issue

and I have edit the police edit to 999999 and still have this message.

all stuff in green but only one user per session

You have checked "Single session per user", so it's normal behavior.

There's no pdb file from M$ for this version вт, 7 дек. 2021 г., 05:00 CrazySciurini @.***>:
…
termsrv.zip https://github.com/stascorp/rdpwrap/files/7647897/termsrv.zip @sebaxakerhtc https://github.com/sebaxakerhtc — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1644 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHZDR4T4WPTWXP6A3C2YOQTUPV2GLANCNFSM5JCJ6HUA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have the same problem. Where can I get  pdb file from M$ and what's the meaning of M$? Sorry for totally new here.

There's no pdb file from M$ for this version вт, 7 дек. 2021 г., 05:00 CrazySciurini @.***>:
…
termsrv.zip https://github.com/stascorp/rdpwrap/files/7647897/termsrv.zip @sebaxakerhtc https://github.com/sebaxakerhtc — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1644 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHZDR4T4WPTWXP6A3C2YOQTUPV2GLANCNFSM5JCJ6HUA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have the same problem. Where can I get pdb file from M$ and what's the meaning of M$? Sorry for totally new here.

Oh! It's working now!

your version is 10.0.19041.1381?

There's no pdb file from M$ for this version вт, 7 дек. 2021 г., 05:00 CrazySciurini @.***>:
…
termsrv.zip https://github.com/stascorp/rdpwrap/files/7647897/termsrv.zip @sebaxakerhtc https://github.com/sebaxakerhtc — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1644 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHZDR4T4WPTWXP6A3C2YOQTUPV2GLANCNFSM5JCJ6HUA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have the same problem. Where can I get pdb file from M$ and what's the meaning of M$? Sorry for totally new here.

Oh! It's working now!
your version is 10.0.19041.1381?

version is 10.0.19041.1381, I installed this version today, but it looks no result for this version, anybody can help? who can send me this version file link?

how can I change other version? sorry I am very new guy for this, looking for helping

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/6cef426ad2ec8a2590218f25880538550a7f3ab3

how can I change other version? sorry I am very new guy for this, looking for helping

Try:

Download RDP Wrapper 1.6.2 and extract this to any directory.
Place rdpwrap.ini in the same directory.
Open CMD as administrator
Execute this commands

cd C:\path\to\directory\
rdpwinst -u -k
rdpwinst -i

Enjoy!

Original message: https://t.me/rdpwrap/49131

rdpwrap.ini: https://raw.githubusercontent.com/sebaxakerhtc/rdpwrap.ini/master/rdpwrap.ini


Pressing certain buttons makes the controller rotate in-app

Hello!
First of all, sorry if this is the wrong place to post this issue, or if it has already been posted. I did search around, but was unable to find any similar issue.
After well over a month troubleshooting, I finally got to a state where I can play almost perfectly! It's been an amazing experience which I wouldn't have access to without your amazing work and effort. Thank you!
So, I remapped the buttons to play Superhot VR (grip on Select is awful when you have to be gripping all the time), but when I grabbed the gun, I could only make it face myself, no matter how I rotated.
After analyzing and testing, and remapping the buttons to check if that was the problem, I found out that, no matter what configuration it's in (or application), pressing and holding the gamepad buttons (X, square, circle, oddly not triangle) on a controller makes the respective virtual controller/hand rotate.
Holding X makes it rotate clockwise gradually, until it is turned 180 degrees from me, at which point, any movement only makes it turn limitedly, while still facing backwards (this happens with the other buttons too). Holding Square makes the hand rotate more slowly, also clockwise, and stop at 90 degrees. Holding Circle, makes it slightly tilt counter-clockwise, and holding Triangle does nothing.
I'm using a PSVR with Trinus PSVR, a ping-pong ball with red LED for headtracking, four cameras, one in each corner of the room, and two PS Move controllers, one a few years old and one brand new. Also, in case it's important, both behave the same when calibrating the magnetometer: they won't go past 53%.
Edit: forgot to mention, I can try to record the issue if it helps.
Thank you very much for your time!


Hamburger button unresponsive

Hamburger button visible on smaller screens is unresponsive in Chrome (both desktop & mobile; didn't try in different browsers).

Cool, it seems like this happens because of an error in some pages.
Uncaught TypeError: Cannot read property 'PropTypes' of undefined
    ...
Uncaught ReferenceError: initReact is not defined
    at (index):114

In the examples, for example (doh!), that icon works...

Will close here, since it seems like it's a duplicate of #1808 with less information


Add some tests

From @jyasskin on June 6, 2016 2:14
I'm especially looking for a sample of testing the website.
Copied from original issue: firebase/friendlypix#17

We're going to be deprecating the friendlypix-web repo so we're closing all open PRs and Issues.  We apologize for not addressing your issue, we should have done more to maintain this repository and we will aim to do better in the future.
For developers who want to learn about Firebase on the web, we recommend the Get to know Firebase for Web codelab.


fix NODE_AUTH_TOKEN secrets name

Sorry, I should probably have given you the proper secrets name...

Fixed in 652800ffd9d400ec760635fdc5841c5ed63d5fbf


Set run_exports max_pin to x.x

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Follow-up to https://github.com/conda-forge/azure-core-cpp-feedstock/pull/11. Please approve/disapprove whether we should continue rolling out the max_pin="x.x" run exports to all the Azure SDK for C++ recipes

@teo-tsirpanis what about azure-identity-cpp?

I consider it to be a high-level package because it provides some user-facing functionality. Users don't usually specifically want to install azure-core-cpp or azure-storage-common-cpp.

Ok, I'm fine with this intermediate approach. We'll apply the stricter pins to azure-core-cpp and azure-storage-common-cpp for now and monitor how the next few releases go


Contract Request list FE

NHS SIGNER admin must be able to see the list of contract request using admin portal and filter them using input parameters:

[ ] UI design according confluence
[ ] implementation  confluence and apiary
[ ] test

[ ] ID договору change to ID контракту
[ ]

@svetavedmed Ready for test

[ ] on pages 6,7 and 8 button for change pages ">" doesn't work from the first time

@svetavedmed Please, make other task for this bug


add loader

[x] find and design the loader
[x] add loader on website
[x] verify it

all requirements done


Add new dataset: GermanGovServiceRetrieval

GermanGovServiceRetrieval: LHM-Dienstleistungen-QA is a German question answering dataset for government services of the Munich city administration. It associates questions with a textual context containing the answer
Checklist for adding MMTEB dataset
Reason for dataset addition: Domain-specific retrieval dataset for German

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[x] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

Do you have time to review? @guenthermi @Muennighoff @PhilipMay

I think everything looks good. Feel free to add points. You might also consider using ndcg_5 instead of 10 since the dataset is quite small.

Good point. I changed the main metric.

Do you have time to review? @guenthermi @Muennighoff @PhilipMay

Hey Malte. I have almost no knowledge with MTEB. Never implemented anything here. Sorry.
Maybe @rasdani could have a look?


Application hangs on Jersey Client invocation when using Helidon Jersey Connector

Environment Details

Helidon Version: 2.2.2
Helidon SE or Helidon MP: MP
JDK version: 11
OS: macOS Catalina 10.15.7 (also happens on Linux)
Docker version (if applicable): N/A

Problem Description
When using HelidonConnetorProvider in the Jersey ClientConfig, the WebClient call hangs. We use MP version 2.2.2 along with Jersey 2.33. Here is the piece of code that creates a new Client. This happens consistently on the local machine for every request (though it seemed intermittent in an OKE cluster environment).
ClientConfig config = new ClientConfig()
  .property(ClientProperties.READ_TIMEOUT, 0)
  .connectorProvider(new HelidonConnectorProvider()).getConfiguration();
client = ClientBuilder.newClient(config);
[Note: READ_TIMEOUT is set just to simulate the earlier Jersey default, likely unrelated to the issue]
The original working code is as below.
ClientConfig config = new ClientConfig();
client = ClientBuilder.newClient(config);
Based on further analysis with Trace level logging, here is what I found.
Our application uses Jersey Client in 2 places - the first to get an IDCS access token and the second to invoke an API endpoint. I only updated the (second) API call to use HelidonConnectorProvider as per code segment above.
Interestingly (and possibly due to the presence of helidon.jersey.connector in pom.xml), even the first Client call (to get token) is using Helidon WebClient! Also, this call goes through fine. The ‘hang’ only happens in the second API call.
I give below log snippets around the first (successful) IDCS call and the second failing call (both using Jersey Client). One difference between the 2 calls that I can think of is that the first call returns pretty quicky, whereas the second failing call takes several seconds. Is it possible that the channel is closed before response is fully processed (in the failing case)? (Note: The network call dumps in the logs are probably not important!)
Log snippet for successful call working_idcs_call.log
Log snippet for subsequent failing API call failing_api_call.log
Steps to reproduce
Tried to reproduce it with small program, but was unsuccessful

@fullymad Do you have a sense as to whether the issue is related to the service resources that you're accessing? That is, is this easily reproducible using a simple local resource?

Unfortunately, I could not reproduce this using the Helidon MP sample. As you can see, looking at the first (IDCS) call, it is working fine even in our application.  Do you think that is a reasonable suspicion?

@fullymad It's hard for me to say based on the log, but introducing a delay seems like a reasonable thing to try. Where is the okhttp3 logging coming from BTW?

@fullymad If you can reproduce the hang, could you provide a thread dump of the process? We have another issue that could be related to this one, and a thread dump may help determine that.

Sure, I will try that soon.

@spericas As for your question about okhttp3 logging, I see the following 2 distinct sections in dependency tree. We don't explicitly include it.
[INFO] +- io.helidon.tracing:helidon-tracing-jaeger:jar:2.2.2:compile
[INFO] |  +- io.jaegertracing:jaeger-client:jar:1.1.0:compile
[INFO] |  |  +- io.jaegertracing:jaeger-thrift:jar:1.1.0:compile
[INFO] |  |  |  - org.apache.thrift:libthrift:jar:0.13.0:compile
[INFO] |  |  +- io.jaegertracing:jaeger-core:jar:1.1.0:compile
[INFO] |  |  |  - com.google.code.gson:gson:jar:2.8.6:compile
[INFO] |  |  - io.jaegertracing:jaeger-tracerresolver:jar:1.1.0:compile
[INFO] |  +- com.squareup.okhttp3:okhttp:jar:3.14.2:compile
[INFO] |  |  - com.squareup.okio:okio:jar:1.17.2:compile
[INFO] |     |     - io.fabric8:kubernetes-client:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-rbac:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-admissionregistration:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-apps:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-autoscaling:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-apiextensions:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-batch:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-certificates:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-coordination:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-discovery:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-events:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-extensions:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-networking:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-metrics:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-policy:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-scheduling:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-storageclass:jar:5.2.1:compile
[INFO] |     |        +- io.fabric8:kubernetes-model-node:jar:5.2.1:compile
[INFO] |     |        +- com.squareup.okhttp3:logging-interceptor:jar:3.12.12:compile

Here is some additional information not quite present in my original report of the issue.

Original issue in OKE environment
The issue was intermittent whereby a bunch of GETs would succeed and a subsequent POST (or even a GET) would result in the calling application hanging. It was quite random.
In this case, the endpoint was valid and the Bearer token was correct. Rolling back use of Helidon connector (and using Jersey default) resulted in successful response from the endpoint.

Issue reproduced in local laptop
When I tested it locally, I had mistakenly extended the endpoint URL with invalid stuff (instead of the correct :port/a/b/c, I used :port/a/b/c/x/y/z). The host and port were correct. In addition, the token was invalid for that endpoint, so it must have returned 401 (as proven through Postman call). Of course, I never got 401 back because the application hung every single time!

My attached log files are from the above run.
Note: The environment is currently unavailable, so I am not able to retry the local test (with correct endpoint URL and correct token). I can possibly try this next week (if that will help).

Attempts to reproduce with Helidon MP sample
I have not been able to reproduce the issue using the sample program. I tried the following, but it never failed.

Used the correct and incorrect endpoint URLs with invalid token (as in 2) above).
Used the endpoint URL of another instance of Helidon MP sample as target server (no token though).
Made the target server wait 20 seconds before responding to the calling sample server
Added the 2 okhttp3 dependencies (artificially) in the calling application.

@fullymad OK, it sounds that this is an issue in the Helidon connector, likely in WebClient. There have been some recent fixes in this area but without a thread dump of the hang, it is hard to say if the fixes are related.
I have not been able to reproduce the problem on my end. We can try to get that thread dump or, another option, would be for me to provide some SNAPSHOT bits for you to try on your end. WDYT?

@spericas Forgot about your request for thread dump. Will do that later and provide it to you.
Along with that test, I could try your SNAPSHOT too.

The issue that could be related is solved by this PR #3365. Building the SNAPSHOT version of Helidon from master should have that fix now. Let me know if you want to try that out.

Looks like the fix is in the latest version (2.3.3). Our released application is on 2.2.2.
In any case, I can first try the SNAPSHOT version in my environment to see if it fixes the issue.

@fullymad I don't think it is in 2.3.3, only in SNAPSHOT at this time.

@spericas I will try out the SNAPSHOT soon. In the meanwhile, here is some additional data.

Attached is the thread dump for the hang scenario where the endpoint URL was not even valid (inadvertently tested).

jstack.txt

After I corrected the endpoint URL to be correct, the behavior was different. The invocation always returns with error after 1 minute (looks like some sort of timeout, that is not related to the READ_TIMEOUT value I have set to 0). Attached is a segment of the log.

timeout.txt
You can see from the below couple of extracted log lines that the application invokes the Client call around 21:52:40 and returns from that around 21:53:40 (with error ‘Connection reset by peer’).
2021-09-14 21:52:40.564-0700 | GET | :PostmanRuntime/7.28.3:dev-madhun:FAB-efec7548-3ca7-4dbc-9051-7e4fcd15f4fd |            helidon-3 |  | DEBUG | com.oracle.cx.communications.dx4c.fabric.routing.impl.RoutingServiceImpl | invoking method=GET
…
…
...
2021-09-14 21:53:40.835-0700 | GET | :PostmanRuntime/7.28.3:dev-madhun:FAB-efec7548-3ca7-4dbc-9051-7e4fcd15f4fd |            helidon-3 |  | DEBUG | com.oracle.cx.communications.dx4c.fabric.routing.impl.RoutingServiceImpl | Closing Jersey client

@fullymad Thanks for the additional data. I will analyze in more detail and get back to you. Could also be related to issue #3377 (at least the first thread dump).

@spericas

I had set READ_TIMEOUT property to 0 to indicate unlimited timeout period (the reason being that the original Jersey Client call using default URLConnector adapter seemed to have NO timeout set, so decided to simulate the same with HelidonConnectorProvider too). Strangely, we have scenarios where it takes well over 10 seconds and we decided to retain the previous behavior. Note: I think you are suggesting setting of timeout to a few seconds just as an experiment, right?

The client is trying to connect to a service running outside our OKE cluster. I will need to work with the other team to collect logs. Will try and do that. Yes, the 1-minute span seems like some internal timeout that is outside the purview of the READ_TIMEOUT setting!

By the way, I am still working with another team to push the Helidon 2.4.0-SNAPSHOT into our artifactory, so I can test the potential fix (with both the above scenarios).

@fullymad Hold on a bit on the SNAPSHOT test as there is one more fix coming that may be relevant to your issue. I'll let you know shortly.

@spericas Sure, will wait.

@fullymad You can try it now after PR #3427 has been merged.

Thanks, @spericas. I will try it soon (after a few days).

@spericas I finally tested with the latest 2.4.0-SNAPSHOT version. Hope the 2 log files are more useful than the ones I provided with 2.2.2 testing.
I tested the same 2 scenarios described a few updates above where I attached jstack.txt and timeout.txt.

(Invalid endpoint URL) This time it did NOT hang, so there is an improvement. It timed out after 1 minute with the error ‘Connection reset by the host’.

wrong_url_240_snapshot.txt

(Valid endpoint URL) Not much change in behavior from Helidon 2.2.2 test. It timed out after 1 minute with the error ‘Connection reset by peer’. I don’t remember getting ‘Connection reset by peer’ last time with 2.2.2!

correct_url_240_snapshot.txt
In case 2, I could not see any debug statements on the server side! It is as if the request did not even reach the server. Maybe TRACE level will show that the request reached the server. When I get to test this again with the other team, I will take care to use TRACE level and see if anything shows up on their server log!
A couple of points to note ...
a) I can ping and reach the destination host entel.poc2.dxfabric.us-ashburn-1.ocs.oc-test.com from my local machine.
b) I ran a Helidon MP Quickstart application after adding the same route for GET operation. In my main application configuration, I changed the destination host to be localhost:8080 where the MP Quickstart was running. This scenario worked fine. I could get response from the MP application. (The one difference is that the remote server had security enabled whereas the sample MP server had it disabled).

@fullymad OK, I will wait for the new set of logs. Back to your earlier message, the behavior of wrong_url_240_snapshot.txt seems consistent with the new changes in 2.4.0-SNAPSHOT.

OK, sounds good. Thanks for all the help.


binlogstreamer.go:47: [error] close sync with err: EOF

Hi While running the tool I am getting the following issue I am setting it on local windows:-

what is the version of your MySQL?

It is 8x in our environment we can also use 5 x if required
On Sun, 16 May 2021 at 16:38, WangXiang @.***> wrote:

what is the version of your MySQL?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/WangXiangUSTC/go-mysql-mongodb/issues/17#issuecomment-841802192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWFJX7GZASN54ZCGKVRLPTTN6RSRANCNFSM44VDPOSQ
.

--
Warm Regards
*Dheeraj Singh | Software Engineer | Girnar Technologies |*207 | Adarsh
Nagar | Near S Bhag Singh Chowk | Jaipur| 302004 | India |
+91<PHONE_NUMBER> office | +91<PHONE_NUMBER> mobile
www.girnarSoft.com http://www.girnarsoft.com/ A venture of girnarSoft
Proud owners of www.CarDekho.com http://www.cardekho.com/, India's #1
auto portal!
Confidentiality Notice & Legal Disclaimer: This E-Mail Message (including
attachments) may contain Confidential and/or legally privileged Information
and is meant for the intended recipient(s) only. If you have received this
e-mail in error and are not the intended recipient/s, kindly notify the
sender and then delete this e-mail immediately from your system. You are
also hereby notified that any use, any form of reproduction, dissemination,
copying, disclosure, modification, distribution and/or publication of this
e-mail, its contents or its attachment/s other than by its intended
recipient/s is strictly prohibited and may be unlawful.
Internet Communications cannot be guaranteed to be secure or error-free as
information could be delayed, intercepted, corrupted, lost, or contain
viruses. Girnar does not accept any liability for any errors, omissions,
viruses or computer problems experienced by any recipient as a result of
this E-mail.

I have not tested the MySQL 8.X, you can try it in 5.X 😄

Hi, By any chance is there future possibility to test the tool with MySQL 8.X

Hi, By any chance is there future possibility to test the tool with MySQL 8.X

Yeah, I plan to do this in the near future


Fix issue-600: UI: doesn't allow table level tags to be added

Describe your changes :

Closes #600 UI: doesn't allow table level tags to be added.
Closes #3543 Tags dropdown is going out side of screen area for entity level tags

Type of change :

[x] Bug fix
[x] Improvement

Frontend Preview (Screenshots) :

#3543

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@harshach @vivekratnavel @darth-coder00 @Sachin-chaurasiya

@ShaileshParmar11 Can you resolve conflicts?


PV Admission breaks when external provider's CloudConfig diverges

cloud-provider-aws recently added a new NodeIPFamilies field to CloudConfig. This was not added to the in-tree implementation.
PV admission breaks on a cluster with the new CloudConfig field set along with these flags set on kube-controller-manager and kube-apiserver: --cloud-provider=external --cloud-config=/path/to/cloud.config
persistentvolumes "aws-" is forbidden: error querying AWS EBS volume aws://eu-west-1a/vol-0ee079da636e6e7d8: unable to read AWS cloud provider config file: warnings:     can't store data at section "global", variable "NodeIPFamilies"
This can be seen on the In-Tree Volumes test failures in kops' IPv6 tests: job output and pod definitions for kube-controller-manager, kube-apiserver, and aws-cloud-controller-manager. pod logs are also available here.
I wasn't sure where to open this issue but I'm wondering how this should be handled. Should we not set apiserver's --cloud-config? Or add an equivalent dummy field in-tree so that cloud.config can be parsed? Or create a different cloud.config file for in-tree components vs the external CCM? or can the PV admission be replaced or removed?
From searching CloudConfigFile in k/k, the PV admission seems to be the only use for apiserver's --cloud-config flag. https://github.com/kubernetes/cloud-provider/issues/4 discusses PV admission but not in the context of the external migration.

/cc @andrewsykim @nckturner

We performed some tests in https://github.com/kubernetes/kops/pull/12435, removing the apiserver's --cloud-config as well as disabling the PersistentVolumeLabel admission plugin both result in the e2e tests passing but I believe they no longer have the topology.kubernetes.io labels but I haven't been able to confirm that. (I can confirm the topology.ebs.csi.aws.com labels are being added by the CSI controller)
As Nick suggested, we could:

Use a separate cloud-config for in-tree components until the webhook is in place
Add a dummy field to in-tree's cloud-config


[ARCH QUESTION] Kylin as second engine

Hello, we have a question, we use Apache Kylin as BI engine, and now interested is it possible to add Kylin as second engine to Metatron Discovery ?
If in theory it is possible, can you tell what point of integration or modification in code base we should to conside.
Thank you.

@ocoka First of all thank you for your interest in "metatron discovery". :)
It is optimized for druid we customized. In this reason, I doubt if it will work because it is not made considering the second engine.
But, as a theoretical approach, it may be possible to change the data source creation part and the data source query part. Actions related to the druid engine were collected in the "app.metatron.discovery.domain.engin" package.
If I'm in your situation, I'll try to manually configure the data source generation and change the data query part such as search or candidate method. If so, I would expect to see a simple dashboard.
In this case, you should refer to the "app.metatron.discovery.domain.engine.QueryService" interface.
I do not know if your question was answered. Still, I strongly recommend that you use druid, but if you have to use kylin, It's another option, you can create a data connection extension for kylin and then load the datasource using db connection mode from kylin.

@kyungtaak Thank you, your answer is important for us, in couple of past days we also come to decision that it is better to use Kylin jdbc connection mode, and your words confirmed our assumption. And we are further convinced that Druid is best engine, but idea about to make Kylin as optional second engine still present in mind, so we will look into detail about classes you've mentioned.
Thank you again.


Sanity check failed when submitting TX.

Hi, I'm using a view-only wallet to create a transaction then sign it with an offline wallet after importing outputs. I'm intermittently getting this error when broadcasting: transaction was rejected by daemon when using the submitTxs method. When using submitTxHex()it says Sanity check failed. I'm not sure how to prevent this, I'm on the latest node/library version (<IP_ADDRESS>)

Error I'm seeing on node: ```2023-01-19 16:02:56.357 [RPC0] WARNING wallet.wallet2 src/wallet/wallet2.cpp:7043 }
2023-01-19 16:02:56.366 E daemon_send_resp.status != CORE_RPC_STATUS_OK. THROW EXCEPTION: error::tx_rejected
2023-01-19 16:02:56.366 [RPC0] ERROR wallet.wallet2 src/wallet/wallet2.cpp:6527 daemon_send_resp.status != CORE_RPC_STATUS_OK. THROW EXCEPTION: error::tx_rejected
2023-01-19 16:02:56.366 [RPC0] INFO stacktrace src/common/stack_trace.cpp:133 Exception: tools::error::tx_rejected```

Yeah I've seen this error from wallet2 and I'm not sure what causes it. Searching around, others have reported the same error. It's originating from monero-project, not monero-java, which is merely a passthrough, so if you can isolate the issue, it would be good to open an issue on that repo.

I think this was user error. I was creating and signing a transaction before broadcasting the previously signed tx. Just had to make sure no new transactions are created while we haven't broadcasted the last signed transaction.


Chip 8 and Super Chip 8 ROM Testing

:smile: Welcome to Hacktoberfest 2019!
Thank you for checking out my Super Chip 8 Python project! This repository contains an emulator for a computer called the Chip 8. Technically the Chip 8 isn't actually a computer, but rather a language specification that was meant to run on different computers from the '70s and '80s. Most famously, it's been used to run games on HP graphing calculators.
The emulator is complete and runs on Windows, Linux, Mac, and UNIX computers. However, I need help with testing. There are a number of Chip 8 and Super Chip 8 ROMs that exist in the public domain that anyone can freely download and use (just Google chip8 public domain roms). I need your help testing those ROMs on the emulator to see if there are any problems.
Here's how you can help with testing, and contribute a pull request to the repository for Hacktoberfest 2019!
Prerequisites
In order to run the emulator, you will need a copy of Python 2.7. See the README.md file for more information on how to install Python.
For Testing
Here are the steps you will need to test out the emulator.

Download the Emulator. You can do this by going to the Releases page for this project. Make sure you download the latest archive file for the project (currently v1.0.0).  Optionally, you can clone the repository to get the source files. To do this, you will need git installed. Type git clone git@github.com:craigthomas/Chip8Python.git to clone the project (see the README.md file for more information).
Indicate what ROM you are going to test. On this issue, make a comment about what ROM file you are going to test. Make sure you aren't testing a ROM file that someone else is already testing by checking that nobody else has already claimed that ROM in the comments, and that the ROM isn't already listed in the ROM Compatibility section of the README.md file.
Try out a ROM. Run the emulator by using the command-line options (see the README.md file for more details on how to run a ROM). Test out the ROM and make sure it works. Make sure each key works, and make special notes if there appear to be problems with the emulator.

If the ROM works...
If your testing is successful, that's great! It's time to document the README.md file with your success. Here is where you can make a pull request to count towards your Hacktoberfest 2019 progress! Here are the steps you will need to do this:

Fork this project. You can do this by pressing the Fork button at the top right-hand side of the repository. This will create a copy of the project in your own Github account that you can use to make the change.

Update the README.md file in your repository. This can be done easily by clicking on the README.md file in your repository, and then clicking the Edit this file button along the top navigation bar (it looks like a pencil).

Update the section called ROM Compatibility. In the README.md file, there is a section called ROM Compatibility. Add a new line to that section with the ROM name, and a green check-mark:
| (ROM name here) | :heavy_check_mark: | |

Commit your changes to your forked repository. At the bottom of the editor, there will be a section called Commit changes. Add a title for the commit, and a description of what you did.

Create a Pull Request. From your repository, click on the button labeled New pull request from the Code tab. By default, this should initiate a pull request from your repository back to the Chip8Python codebase. Make sure you provide a good working title and a good description of what you tested.

That's it! I'll look at your pull request and let you know if there is anything missing. If everything looks good, your change will be merged into the master file. Thank you for helping out!
If the ROM doesn't work...
If the ROM file doesn't work, that's okay! Your testing work is very valuable, and you can still create a pull request to count towards Hacktoberfest 2019! Follow the same procedure as listed above in the If the ROM works... section, but for step 3, change the line that you add to the README.md file that describes what isn't working:
| (ROM name here) | :x: | (Describe what is wrong) |

Follow the rest of the procedure to create a pull request so that the README.md file is updated with the results of your testing. Thank you for helping out!
If you think you know how to fix the bug...
If your testing revealed a bug, and you can fix the bug, then please feel free to do so! I welcome any code contributions you feel are helpful and reflect the spirit of the project. To make a code change, simply fork the project, make the changes in your own repository, and then submit a pull request back to the main project repository. Make sure to include a descriptive title and well-documented pull request body that explains the changes that you made and why. Ensure that the automated tests complete successfully, and try to have a unit test or two that covers the bug condition. Thank you for helping out!

Closing issue - Digital Ocean may view this type of activity as spam, even though the testing aspect is actually quite helpful.


UPnP Errors - Fresh Install - Lets encrypt + Duck DNS

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.41.0
Python release (python3 --version):
3.4.2
Component/platform:
UPnP
Description of problem:
Fresh AIO install on raspbian jessie lite, UPnP fails, errors logged
Lets encrypt + Duck DNS installed
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

upnp:

Traceback (if applicable):
17-03-30 22:28:31 ERROR (MainThread) [homeassistant.setup] Error during setup of component upnp
Traceback (most recent call last):

File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/setup.py",
line 192, in _async_setup_component
    None, component.setup, hass, processed_config)
  File "/usr/lib/python3.4/asyncio/futures.py",
line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py",
line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py",
line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py",
line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/upnp.py",
line 26, in setup
    import miniupnpc
ImportError: No module named 'miniupnpc'

Additional info:
followed upnp commands to the letter.

Looks like its complaining about miniupnpc not being installed
You can try installing it manually by doing the following:
$ source /srv/homeassistant/homeassistant_venv/bin/activate
$ pip3 install miniupnpc
and see if that resolves your issue

I followed the commands as per the Upnp page (https://home-assistant.io/components/upnp/)
I did a fresh install and ran the commands as per the UPnP page in the veritual_env this time ($ source /srv/homeassistant/homeassistant_venv/bin/activate), same issue occurred.
As per the above suggestion (sudo pip3 install miniupnpc), installation is successful, but the same errors are generated by HA.

What directory should you be in when running the command "git clone https://github.com/miniupnp/miniupnp.git" ? Using AIO installer (virtual environment)

Ok, so figured it out.
For the AIO installer, you have to first change the user to the hass user, then run the command to change to the virtual environment. After that run the install command
sudo  su -s /bin/bash homeassistant
source /srv/homeassistant/homeassistant_venv/bin/activate
pip3 install miniupnpc


VSCODE does not respect AWS Named Profiles

VS Code Version: 1.54.3
OS Version:Windows_NT x64 10.0.19042

Steps to Reproduce:
1.Create an AWS Named profile Not Default with AccessKey, Secret, and Session Token
2. In VS CODE set the profile to your new named profile
3. Run a python script to do anything with that profile
4. It will use the Default profile not the named one

Does this issue occur when all extensions are disabled?: Yes

How is this related to VS Code?

It sounds like the Python library you are using doesn't read the current profile. Please contact the authors of that library, thanks.


Predicted images are getting worse while the metrics are still improving

I'm trying to train a U-net with VGG16 as a backbone in order to recognize 4 classes: sky, rocks, trees and background in a dataset of about 10000 images. I'm using categorical crossentropy loss function and IOU / F1-score as metrics.
The weird thing is that if I train the model for 20 epochs, the loss function and metrics are improving smoothly with the corresponding validation values. The model reaches about 90% of accuracy (with loss function about 0.02) and it seems that the network doesn't overfit.
However, after the training, the predicted images are black.
On the other hand, when if I train the model for about 5-7 epochs where the loss function had a value of about 0.20 and IOU/F1-score about 0.40, the predicted images are pretty much better, achieving an IoU of about 0.80 - 0.90.
How would you interpret this ? Is the issue related with the unbalanced size of the classes in the image ? I mean that sky and background, cover much bigger area in the images than rocks and trees.
Any thoughts ?

Getting the same issue

Hello @nataliameira,
No, unfortunately I didn't solve the problem with segmentation_models. The problem that I had, solved when I developed the model on my own.


Fix wrong logger class for PathConfigMatcher

Fixes https://github.com/keycloak/keycloak-client/issues/113

@HerrDerb Thanks for the PR!


Removed outdated mesos files

These mesos files were checked in 7 months ago and are no longer relevant.

LGTM


Bump version (major)

Action to bump version (major) was triggered. Check changes and approve accordingly.

Major version feels too much.
Close?

@raguiar9080 @festinaj ?

I'm under the impression @jstasiak opened this hence by reluctant to close without feedback, but I wonder now if it was by mistake since there's been no interaction here.

I had nothing to do with this PR (I think)

Ok, just wanted to make sure. I'll close it 👍


var vm = this; doesn't work...have to use $scope..why is that?

So I was attempting to set up formly in my project using the examples in the documentation.  Everywhere you use 'vm' I did as well.  And nothing worked.  It's only when I used $scope.whatever rather than vm.whatever did it work.
Why is that?
No workie:
  var vm = this;

 vm.user = {
    email<EMAIL_ADDRESS>  };

vm.userFields = [
    {
      key: 'email',
      type: 'input',
      templateOptions: {
        type: 'email',
        label: 'Email address',
        placeholder: 'Enter email'
      }
    },
    {
      key: 'password',
      type: 'input',
      templateOptions: {
        type: 'password',
        label: 'Password',
        placeholder: 'Password'
      }
    },
    {
      key: 'file',
      type: 'file',
      templateOptions: {
        label: 'File input',
        description: 'Example block-level help text here',
        url: 'https://example.com/upload'
      }
    },
    {
      key: 'checked',
      type: 'checkbox',
      templateOptions: {
        label: 'Check me out'
      }
    }
  ];

<formly-form model="vm.user" fields="vm.userFields">
  <button type="submit" class="btn btn-default" ng-click="vm.submit(vm.user)">Submit</button>
</formly-form>

This works:
  $scope.user = {
    email<EMAIL_ADDRESS>  };

  $scope.userFields = [
    {
      key: 'email',
      type: 'input',
      templateOptions: {
        type: 'email',
        label: 'Email address',
        placeholder: 'Enter email'
      }
    },
    {
      key: 'password',
      type: 'input',
      templateOptions: {
        type: 'password',
        label: 'Password',
        placeholder: 'Password'
      }
    },
    {
      key: 'file',
      type: 'file',
      templateOptions: {
        label: 'File input',
        description: 'Example block-level help text here',
        url: 'https://example.com/upload'
      }
    },
    {
      key: 'checked',
      type: 'checkbox',
      templateOptions: {
        label: 'Check me out'
      }
    }
  ];

<formly-form model="user" fields="userFields">
  <button type="submit" class="btn btn-default" ng-click="submit(user)">Submit</button>
</formly-form>

Are you using controllerAs syntax?  If you are not that could cause the issue you are seeing.

No I'm not.  But everything is working as expected so is it an issue?  Am I meant to be using controllerAs syntax?  Or am I good?

You don't have to use controllerAs to work with formly.  The examples are just typically written that way.

Okay, just checking.  I'm new to Angular.  Thanks for your help.

I highly recommend you use controller as syntax. :-) good luck!

Thanks @kentcdodds!  Can you explain why controller as syntax is better or point me to a good article that does?  I'd appreciate it.

This article by @pascalprecht talks about bindToController but it references some benefits of controllerAs as well: http://blog.thoughtram.io/angularjs/2015/01/02/exploring-angular-1.3-bindToController.html

Thanks @kentcdodds.  Much appreciated.  Loving Formly too...great stuff.

:+1: glad to hear it!

@sharpmachine controllerAs is considered best practice in Angular. It solves basically your vm = this problem, since that has been always a work-around.
Thanks @kentcdodds for mentioning our article :)

I seem to be having the same problem, I have setup my controller as.  Anything else that would cause fields not to show up?  My controller is wired up fine.  I am not using routes (not sure if that is a required).

Could you provide an example of what you're doing? It's easier to help
identify what's going wrong when I can see what you're doing.
http://help.angular-formly.com

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On Jul 3, 2015 11:34 PM, "mweels"<EMAIL_ADDRESS>wrote:

I seem to be having the same problem, I have setup my controller as.
Anything else that would cause fields not to show up? My controller is
wired up fine. I am not using routes (not sure if that is a required).
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/333#issuecomment-118463082
.

This was huge for me.  Not sure why I should have to do this but I had to do the following in my app where I was using routes.
$routeProvider
	.when('/home', {
		templateUrl: 'views/main.html',
		controllerAs: 'ctrl',
		controller: 'homeCtrl'
	})

And then on my page I had to use the following:
<form ng-submit="ctrl.onSubmit()" novalidate>
    <formly-form model="ctrl.user" fields="ctrl.userFields" form="ctrl.userForm">
        <button class="btn btn-primary" type="submit" ng-disabled="ctrl.userForm.$invalid">Submit</button>
    </formly-form>
</form>

Notice the "ctrl" stuff I had to add to make my form show up.  Is this the only way to do this?  I tried to follow other examples where people used "vm.userFields" directly without the extra controllerAs and extra . syntax.  At least this thread helped me figure this out.  I guess the tutorial I was following was just outdated.

You might want to take a look at:

https://github.com/johnpapa/angular-styleguide#controllers
https://github.com/toddmotto/angularjs-styleguide#controllers