Response to Comment email

~MO fails to email User when second User posts a Comment in response to (first user's) Comment.~
Users do not receive "response to your comment" emails; emails created when a second User posts a Comment in response to (first user's) Comment. (These are different from emails created when another user posts a Comment on your Observation.)

This may have functioned properly as of 2024-06-05 (which should help isolate when the bug occured).
See attached.
Gmail - Re_ A new bug_.pdf

These emails are being created, but not sent. Example, from local log/email-debug.log:
...
2024-07-29 06:18:24 -0700 SAVE QueuedEmail::CommentAdd from=OMS to=BrianLok comment=200662
2024-07-29 06:18:24 -0700 MAIL comment from=5485 to=176018 object=observation-552668
2024-07-29 06:23:32 -0700 SAVE QueuedEmail::CommentAdd from=OMS to=Joe Cohen comment=200662
2024-07-29 06:23:32 -0700 MAIL comment from=5485 to=4468 object=observation-552668

@pellaea, @nimmolo, @mo-nathan:
Any ideas about how to debug this, or even figure out when things broke?
(I can't figure out how to test whether a particular commit was working, as I think mail is never actually sent in development mode. Am I wrong?)

Maybe try responding to your own comments with a second account, or ping me and i'll respond.
I suspect the problem is in email, e.g. a spam filter external to MO.

@nimmolo: Thanks for the suggestion. I have tried that, but will try it again with addresses at different email providers.

Tried again. Left comment and two responses on your Obs https://mushroomobserver.org/122027.
Original comment from me.
1st response from user Oregon Mycological Society. I did not get a response to this. (I did check spam & trash.0
2nd response from me. Oregon Mycological Society (with non-Gmail email provider) didn't get a response to the above.

nimmo
Jul 29, 2024, 10:28 PM (8 hours ago)
Joe -
This may possibly have to do with our standing re: suspected spammer lists. There's a way to check if we're on a blocklist, and a way to get cleared, but it takes time.

@nimmolo: Re standing on suspected spammer lists:

If you can point me in the right direction(s), I'll try to get some more info.
I checked a recent DMARC report from Google. Perhaps it indicates problems with DKIM. See https://us.dmarcian.com/dmarc-xml/details/LzAByfGq0o2St1rH/. I don't know what that means.
Is there an issue with the test server?
I also checked Debouncer to see if mushroomobserver.org was on a blacklist. I didn't see any problems.

It sounds like you know more than I remember about it.
I don’t imagine there’s an issue with the test server, but it’s possible.
According to the DMARC report, the problems are reported by this Google IP address
mushroomobserver.org	<IP_ADDRESS>	mail-sor-f41.google.com
The IP address associated with test.mushroomobserver.org  is :
<IP_ADDRESS>

On Jul 30, 2024, at 6:54 AM, Joseph D. Cohen @.***> wrote:
@nimmolo https://github.com/nimmolo: Re standing on suspected spammer lists:
If you can point me in the right direction(s), I'll try to get some more info.
I checked a recent DMARC report from Google. Perhaps it indicates problems with DKIM. See https://us.dmarcian.com/dmarc-xml/details/LzAByfGq0o2St1rH/ https://us.dmarcian.com/dmarc-xml/details/LzAByfGq0o2St1rH/. I don't know what that means.
Is there an issue with the test server?
I also checked Debouncer to see if mushroomobserver.org was on a blacklist. I didn't see any problems.
—
Reply to this email directly, view it on GitHub https://github.com/MushroomObserver/mushroom-observer/issues/2263#issuecomment-2258412174, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAO3TP4ZACZSXC5H6UT6JNTZO6LKHAVCNFSM6AAAAABLUFQM3CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJYGQYTEMJXGQ.
You are receiving this because you were mentioned.


Properties: Replace "combined files" icon

Closes #1213.

I increased the size to 72, because in 70 it looked a bit blurred
Before:

After:

@XPoppyX It might make sense to create a combined font with all the new icons you are adding.


Added chronological-format

Q
A

Bug fix?
no

New sample?
yes

What's in this Pull Request?
This sample formats your SharePoint list view to look like a chronological order.
This view is responsive and colors depend on the theme you're using.
Thanks for your contribution! Sharing is caring.

@fcatinot thanks for the great sample!
I've made a few adjustments:

Removed debugMode
Changed dot border color to use theme
Changed from using direction to use flex-direction to prevent text issues (specifically punctuation like ! moving to the start of a sentence)

Please let me know if you have any questions or concerns. Hope to see more samples from you in the future, thanks!!


Issue/infra tickets 198 quickstart lsm rc fix

allow specifying 6.5.0rc rather than full 6.5.0rc20231208092517
reworked version constraint logic to be compatible with dev, RC and stable

closes #198

Note to self: should probably also be applied to OSS quickstart.


Wording of review redlines in PR

To make more user friendly to average person, update:
"Redline to current draft" to "Redline to most recent edition"
and
"Redline of PR"  to "Redline to Published"

Also: note that the redline is useless (and perhaps shouldn't be shown) on a PR for the initial draft of a document (I committed the template document to main before creating a branch (and PR) for the initial draft of the document)


Does it support utf8?

Load utf8 string, display incorrectly

Could you provide more information?

var L = luaL_newstate();
luaL_openlibs(L);
{
// luaL_dostring(L, "config = {'您好'}"); // Direct dostring is correct
luaL_dofile(L, "D:\main.lua"); // but dofile is not correct, file encoding is UTF-8, file content is "config = {'您好'}"
lua_getglobal(L, "config");
lua_pushinteger(L, 1);
lua_gettable(L, -2);
Console.WriteLine(lua_tostring(L, -1));
}
lua_close(L);

This is weird behavior because that directly calls a native function within the dll library. What version of Lua were you using?

Lua53

luaL_dostring(L, File.ReadAllText("D:\main.lua")); // this is also correct
It looks like the dofile is incorrect.

Seems like in lua53 luaL_dofile will call luaL_loadfile which calls luaL_loadfilex (just like lua) with mode null. Yada yada if it fails it leaves an error on the stack. Could you tostring the error, perhaps that will give us more information.

There is no error in luaL_dofile.
I've given you all the code to try out.

I got around to trying it out, and have confirmed the issue. Investigating.

..................
Can't fix without modifying source......

In c to C#,  const char *buff should correspond to byte[], not string, using string will result in the wrong size being passed.
public static int _luaL_dofile(lua_State L, string fn) { 	string script = File.ReadAllText(fn); 	int status = luaL_loadbufferx(L, script, (ulong) script.Length, $"@{fn}", "bt"); // script.Length is wrong 	if (status > 0) 		return status; 	return lua_pcall(L, 0, LUA_MULTRET, 0); }
public static extern int luaL_loadbufferx(lua_State L, string buff, size_t sz, string name, string? mode);
LUALIB_API int luaL_loadbufferx (lua_State *L, const char *buff, size_t size,                                  const char *name, const char *mode) {   LoadS ls;   ls.s = buff;   ls.size = size;   return lua_load(L, getS, &ls, name, mode); }

That might depend on the mode "bt", actually, but I am certainly not walking the code to find out what specifically is going on. Lua traditionally has a very bad source.

If you could give it a try with both versions and see if it actually matters that would be great.


some problem about training.

Hi ,I find there is no logger.py in utils,but  "  from utils.logger import *  " in the train.py.Could you give me this file? Thank you.

Sorry, the logger.py was accidentally ignored by my .git ignore file. I have added it now.


Obfuscate sensitive data in hornet's log output

Is your feature request related to a problem? Please describe.
Users are likely to share their hornet logs with the community to get help on issues they encounter.
I think most users are not aware that the output can contain sensitive data, for example:
"identityprivatekey": "*****"

Having this obfuscated minimizes the risk of it being exposed.
The other values to consider obfuscating are the salt and password hash, although those are less likely to get hacked, it is probably good to obfuscate them anyway.
Of course, we should remind the community not to share their config.json fully with anyone as that could potentially expose their node's private key if it is set there.
Describe the solution you'd like

Obfuscate sensitive config params in log output
Add a warning in docs for users not to share the config.json fully, only the parts required if they need assistance

Seems as we have missed some config options to be masked in stdout. This is actually something which is already in place, for example for the dashboard password salt/hash.

Fixed with #1100


Add support for hermetic nixos configurations

This change adds a Terraform variable hermetic and supporting logic to the Nix expression evaluated by nix-instantiate to better support users that need to evaluate configurations that supply their own pinned nixpkgs or wrap eval-modules to customize module special args.
e.g. at Awake we pin nixpkgs and build our NixOS configuration closures completely sealed from anything on the NIX_PATH. This change simply allows one to say "I'm supplying my own expression that results in a nixos system, so please use that".
A minor additional change was to further purify the Nix expression by using the supplied pkgs.lib from the derived NixOS system instead of effectfully re-importing .

@zimbatm does this meet your review requirements?

thanks!


Allow Markdown Preview editor to be windowable.

I expect that this is reliant upon https://github.com/microsoft/vscode/issues/204285#issue-2117104618 first.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

https://github.com/microsoft/vscode/issues/204286#event-11711796486

@mjbvz, what's it a duplicate of? Couldn't you at least add a URI to the correct issue?

#195715

https://github.com/microsoft/vscode/issues/204286#issuecomment-1928625480

Thank you, @mjbvz.


Added missing SetActiveDOF in VectorField.

This fixes an ongoing issue with VectorFieldPlanner immediately failing
in certain cases (which turned out to be cases where the Active DOFs
set on the robot did not match the Active Manipulator).
I also removed some unused imports.

This is not a bug. Active DOFs and the active manipulator are independent:

active DOFs represent the DOFs that may be modified by the planner
the active manipulator defines the end-effector frame that is used in the objective function.

It's perfectly reasonable to set the active DOFs to a subset of the active manipulator's GetArmIndices() if you only want to plan for those DOFs. It's also valid, although not useful, to set the active DOFs to joints that do not affect the end effector.
If the planner does not work when these are not equal, which is very possible, then that is a bug. We should not work around the bug by setting the active DOFs to those of the active manipulator.

@psigen What was our conclusion on this?

I need to refactor this to do a check instead of setting the DOFs.

This PR is being used for reference, but should not be merged.


my program doesn't accept  wildcards to match filename  in PowerShell

my program doesn't accept  wildcards *.* to match filename  in PowerShell,... in CMD works fine
this is what I used:  py -m nuitka  Fixor.py  --onefile

No issue template, no fun.


Add dependencies for yacc, autoconf and subversion

I needed to install some older Rubies (e.g. 1.8.7-p375) and discovered that there were some additional dependencies required.
This pull request adds these packages to the dependency classes.

I don't think I'm going to include this simply because 1.8.7 is so old and no longer maintained. Not to mention the puppet community has moved on to only supporting 2.1 now. I feel if there is a special case like the one you are trying to solve, it's probably best that you handle those extra package dependencies yourself outside of the module.
Thanks for the PR though. Please let me know if you have any further thoughts or questions!


Morning session peer-to-peer code review

All things look nice and great design implementation. but there are some minor things to work on.
It's a best practice to avoid using fixed height because It can affect the responsiveness of your page
https://github.com/stephenkati/Global-Canvas-Art-Exhibition-Concert-/blob/edc19ce3603f9db4cda9683a1bc7e8fa9e4fce91/css/style.css#L112
-Although the current layout is great and responsive it would be great to use 2 columns just above 768px(screen size) as per the template design by Cindy Shin (https://www.behance.net/adagio07).
https://github.com/stephenkati/Global-Canvas-Art-Exhibition-Concert-/blob/edc19ce3603f9db4cda9683a1bc7e8fa9e4fce91/css/style.css#L736-L745

Glad you let me know. Issue fixed


[SQL][Minor] make StringComparison extends ExpectsInputTypes

make StringComparison extends ExpectsInputTypes and added expectedChildTypes, so do not need override expectedChildTypes in each subclass

Merged build triggered.

Merged build started.

Test build #31850 has started for   PR 5905 at commit b374ddf.

Test build #31850 has finished for   PR 5905 at commit b374ddf.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait StringComparison extends ExpectsInputTypes

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31850/
Test PASSed.

Thanks. Merging.


⚠ Bofrost.fr has degraded performance

In 742b5d8, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1160 ms

Resolved: Bofrost.fr performance has improved in 8e605ce after 6 minutes.


Resizable AppShell slots

Describe the feature in detail (code, mocks, or screenshots encouraged)
I think it would be useful to have the AppShell slots be resizable by dragging the corner.
What type of pull request would this be?
None
Provide relevant links or additional information.
No response

@Rimantovas the slots don't have an inherit size, they take the size of the element within. This means you could drop your own div elements in and use the resize styling:

https://tailwindcss.com/docs/resize
https://developer.mozilla.org/en-US/docs/Web/CSS/resize


Fix early dropping of Option parameters

See issue #257.
Rust inserts a drop() because the Option<Object> is moved into the pattern match. This caused the Object (Ref<Image> in the case of issue #257) to be NULL once Godot tried to use the value.
Adding a borrow causes the value to live as long as the function, by which point we have transferred ownership of the Object to Godot.

Thanks for the PR! You were a bit faster than me :)
I think #259 is a bit more comprehensive, since it also covers methods with varargs, and also adds manual drops to prevent further instances from passing compilation. If you agree, we could close this PR and add you as a co-author in #259.
Alternatively, you can incorporate the changes in #259 in this PR, and I'll close the other one.
Thanks again for finding the bug and contributing!

Yeah, no problem - I will close this one, it's easier.


build fails if you import keyframes directly

The problem

if you import { keyframe } from "@stylex/stylex" the build will fail because its apparently looking for the reference of stylex.keyframes()
How to reproduce

Steps to reproduce:

import { keyframe } from "@stylex/stylex"
create an animation
try to run next build

Expected behavior

vscode in particular autocompletes to import { stylex } from "@stylexjs/stylex" which will fail because we cannot call that var directly, and you will receive an unrelated error about css variables. So if you simply autocomplete props or create or keyframes etc you can benefit from the autocomplete with an import such as import { create, props, keyframes } from "@stylex/stylex".  However the build will fail with the the keyframes import unless it is prefixed with the base path.  props and create however work fine.
Environment (include versions). Did this work in previous versions?

can't reproduce it. could you share more details? specifically, what kind of error you get and what version of NextJS or stylex do you use? if you tried this in the example app, it could be that it was due to the eslint rule, but now that should be fixed.

@nedjulius Although that PR is merged, it's not in a release yet. So the ESLint error will still exist.

Fix released in v0.5.0.


Undefined method admin_url in SessionController

I believe I have run into the bug described here: https://github.com/plataformatec/devise/issues/1350
In short, I have a devise resource called 'Admin', but I don't have 'resource :admin' or 'resources :admins' defined in my routes.
Upon login, I get this error from the SessionController: "Undefined method admin_url in SessionController".
Stack trace:
actionpack (4.0.0.rc1) lib/action_dispatch/routing/polymorphic_routes.rb:129:in `polymorphic_url'
actionpack (4.0.0.rc1) lib/action_dispatch/routing/url_for.rb:159:in `url_for'
actionpack (4.0.0.rc1) lib/action_controller/metal/redirecting.rb:100:in `_compute_redirect_to_location'
turbolinks (1.2.0) lib/turbolinks.rb:15:in `_compute_redirect_to_location_with_xhr_referer'
actionpack (4.0.0.rc1) lib/action_controller/metal/redirecting.rb:70:in `redirect_to'
actionpack (4.0.0.rc1) lib/action_controller/metal/flash.rb:40:in `redirect_to'
actionpack (4.0.0.rc1) lib/action_controller/metal/instrumentation.rb:61:in `block in redirect_to'
activesupport (4.0.0.rc1) lib/active_support/notifications.rb:159:in `block in instrument'
activesupport (4.0.0.rc1) lib/active_support/notifications/instrumenter.rb:20:in `instrument'
activesupport (4.0.0.rc1) lib/active_support/notifications.rb:159:in `instrument'
actionpack (4.0.0.rc1) lib/action_controller/metal/instrumentation.rb:60:in `redirect_to'
actionpack (4.0.0.rc1) lib/action_controller/metal/responder.rb:136:in `redirect_to'
actionpack (4.0.0.rc1) lib/action_controller/metal/responder.rb:194:in `navigation_behavior'
actionpack (4.0.0.rc1) lib/action_controller/metal/responder.rb:163:in `rescue in to_html'
actionpack (4.0.0.rc1) lib/action_controller/metal/responder.rb:161:in `to_html'
actionpack (4.0.0.rc1) lib/action_controller/metal/responder.rb:154:in `respond'
actionpack (4.0.0.rc1) lib/action_controller/metal/responder.rb:147:in `call'
actionpack (4.0.0.rc1) lib/action_controller/metal/mime_responds.rb:330:in `respond_with'
devise (3.0.0.rc) app/controllers/devise/sessions_controller.rb:18:in `create'
actionpack (4.0.0.rc1) lib/action_controller/metal/implicit_render.rb:4:in `send_action'
actionpack (4.0.0.rc1) lib/abstract_controller/base.rb:189:in `process_action'
actionpack (4.0.0.rc1) lib/action_controller/metal/rendering.rb:10:in `process_action'
actionpack (4.0.0.rc1) lib/abstract_controller/callbacks.rb:18:in `block in process_action'
activesupport (4.0.0.rc1) lib/active_support/callbacks.rb:462:in `_run__60510980372902786__process_action__callbacks'
activesupport (4.0.0.rc1) lib/active_support/callbacks.rb:80:in `run_callbacks'
actionpack (4.0.0.rc1) lib/abstract_controller/callbacks.rb:17:in `process_action'
actionpack (4.0.0.rc1) lib/action_controller/metal/rescue.rb:29:in `process_action'
actionpack (4.0.0.rc1) lib/action_controller/metal/instrumentation.rb:31:in `block in process_action'
activesupport (4.0.0.rc1) lib/active_support/notifications.rb:159:in `block in instrument'
activesupport (4.0.0.rc1) lib/active_support/notifications/instrumenter.rb:20:in `instrument'
activesupport (4.0.0.rc1) lib/active_support/notifications.rb:159:in `instrument'
actionpack (4.0.0.rc1) lib/action_controller/metal/instrumentation.rb:30:in `process_action'
actionpack (4.0.0.rc1) lib/action_controller/metal/params_wrapper.rb:245:in `process_action'
activerecord (4.0.0.rc1) lib/active_record/railties/controller_runtime.rb:18:in `process_action'
actionpack (4.0.0.rc1) lib/abstract_controller/base.rb:136:in `process'
actionpack (4.0.0.rc1) lib/abstract_controller/rendering.rb:44:in `process'
actionpack (4.0.0.rc1) lib/action_controller/metal.rb:195:in `dispatch'
actionpack (4.0.0.rc1) lib/action_controller/metal/rack_delegation.rb:13:in `dispatch'
actionpack (4.0.0.rc1) lib/action_controller/metal.rb:231:in `block in action'
actionpack (4.0.0.rc1) lib/action_dispatch/routing/route_set.rb:80:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/routing/route_set.rb:80:in `dispatch'
actionpack (4.0.0.rc1) lib/action_dispatch/routing/route_set.rb:48:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/routing/mapper.rb:44:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/journey/router.rb:71:in `block in call'
actionpack (4.0.0.rc1) lib/action_dispatch/journey/router.rb:59:in `each'
actionpack (4.0.0.rc1) lib/action_dispatch/journey/router.rb:59:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/routing/route_set.rb:654:in `call'
warden (1.2.1) lib/warden/manager.rb:35:in `block in call'
warden (1.2.1) lib/warden/manager.rb:34:in `catch'
warden (1.2.1) lib/warden/manager.rb:34:in `call'
rack (1.5.2) lib/rack/etag.rb:23:in `call'
rack (1.5.2) lib/rack/conditionalget.rb:35:in `call'
rack (1.5.2) lib/rack/head.rb:11:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/params_parser.rb:27:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/flash.rb:241:in `call'
rack (1.5.2) lib/rack/session/abstract/id.rb:225:in `context'
rack (1.5.2) lib/rack/session/abstract/id.rb:220:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/cookies.rb:486:in `call'
activerecord (4.0.0.rc1) lib/active_record/query_cache.rb:36:in `call'
activerecord (4.0.0.rc1) lib/active_record/connection_adapters/abstract/connection_pool.rb:626:in `call'
activerecord (4.0.0.rc1) lib/active_record/migration.rb:366:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/callbacks.rb:29:in `block in call'
activesupport (4.0.0.rc1) lib/active_support/callbacks.rb:392:in `_run__3988114632256429821__call__callbacks'
activesupport (4.0.0.rc1) lib/active_support/callbacks.rb:80:in `run_callbacks'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/callbacks.rb:27:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/reloader.rb:64:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/remote_ip.rb:76:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/debug_exceptions.rb:17:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
railties (4.0.0.rc1) lib/rails/rack/logger.rb:38:in `call_app'
railties (4.0.0.rc1) lib/rails/rack/logger.rb:21:in `block in call'
activesupport (4.0.0.rc1) lib/active_support/tagged_logging.rb:67:in `block in tagged'
activesupport (4.0.0.rc1) lib/active_support/tagged_logging.rb:25:in `tagged'
activesupport (4.0.0.rc1) lib/active_support/tagged_logging.rb:67:in `tagged'
railties (4.0.0.rc1) lib/rails/rack/logger.rb:21:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/request_id.rb:21:in `call'
rack (1.5.2) lib/rack/methodoverride.rb:21:in `call'
rack (1.5.2) lib/rack/runtime.rb:17:in `call'
activesupport (4.0.0.rc1) lib/active_support/cache/strategy/local_cache.rb:83:in `call'
rack (1.5.2) lib/rack/lock.rb:17:in `call'
actionpack (4.0.0.rc1) lib/action_dispatch/middleware/static.rb:64:in `call'
railties (4.0.0.rc1) lib/rails/engine.rb:511:in `call'
railties (4.0.0.rc1) lib/rails/application.rb:96:in `call'
rack (1.5.2) lib/rack/lock.rb:17:in `call'
rack (1.5.2) lib/rack/content_length.rb:14:in `call'
rack (1.5.2) lib/rack/handler/webrick.rb:60:in `service'

This only seems to happen when I set after_sign_in_path_for in my ApplicationController:
def after_sign_in_path_for(resource)
    stored_path = stored_location_for(resource)

    return stored_path if stored_path

    if resource == :admin
      root_path
    end
  end

  def after_sign_out_path_for(resource)
    if resource == :admin
      new_admin_session_path
    end
  end

Project that reproduces the issue can be found here: https://github.com/elsurudo/devise-login-undefined-method
Go to: http://localhost:3000/admins/sign_in (NOT just http://localhost:3000) and sign in as<EMAIL_ADDRESS>Any help would be appreciated!

Another way you can solve it is, I hope it helps.
 def after_sign_in_path_for(resource)       if resource.class == Admin         admins_dashboard_path       end     end


Support of encoding type 23 and 24 needed

I did my research and the new prefix should be "tony".
And I always have the following error running the program. I don't know what's wrong.
Traceback (most recent call last):
  File "kakaodecrypt.py", line 166, in <module>
    KakaoDbDecrypt.run(db_file, enc_table, dec_table, do_print)
  File "kakaodecrypt.py", line 135, in run
    contents = KakaoDecrypt.decrypt(user_id, enc_type, contents)
  File "kakaodecrypt.py", line 88, in decrypt
    key = KakaoDecrypt.deriveKey(key, salt, 2, 32)
  File "kakaodecrypt.py", line 72, in deriveKey
    KakaoDecrypt.pkcs16adjust(I, j * v, B)
  File "kakaodecrypt.py", line 24, in pkcs16adjust
    x = (b[len(b) - 1] & 0xff) + (a[aOff + len(b) - 1] & 0xff) + 1
TypeError: unsupported operand type(s) for &: 'str' and 'int'

Thank you John.
For testing purposes, could you provide me with a complete row encrypted with "enc":23, like in #1?

@jiru I found that the latest database uses enc 24:
1|1714824122888697856|1|199681262821716|185911128|RrODdJ7LDIcozGqjIHvEZ2x+U8hunYSaFCULVnY/A8YZhE3njV19nT1JIEzn/NUS|{}|1525360165|0|646846704|1714818457667575808|0|None|{"notDecoded":false,"origin":"MCHATLOGS","c":"05-06 21:58:19","isSingleDefaultEmoticon":false,"defaultEmoticonsCount":0,"isMine":false,"enc":24}
2|1714824624108150785|1|199681262821716|195847548|GvH+4xyB2hzOpbATZlNMJQ==|{}|1525360225|0|646907080|1714824122888697856|0|None|{"notDecoded":false,"origin":"MCHATLOGS","c":"05-06 21:58:19","isSingleDefaultEmoticon":false,"defaultEmoticonsCount":0,"isMine":true,"enc":24}
3|1714824645146779648|1|199681262821716|195847548|1pPepn0lJSCRLxK9P5kaNQ==|{}|1525360227|0|646909580|1714824624108150785|0|None|{"notDecoded":false,"origin":"MCHATLOGS","c":"05-06 21:58:19","isSingleDefaultEmoticon":false,"defaultEmoticonsCount":0,"isMine":true,"enc":24}
4|1714824678197895168|1|199681262821716|195847548|IICZJO/83CXZWZhNmiWmHg==|{}|1525360231|0|646913580|1714824645146779648|0|None|{"notDecoded":false,"origin":"MCHATLOGS","c":"05-06 21:58:19","isSingleDefaultEmoticon":false,"defaultEmoticonsCount":0,"isMine":true,"enc":24}

I see. I don’t think that messages that have been encoded with enc:23 in the database are converted to enc:24 after updating the app, so you should still be able to find some using a command like:
./kakaodecrypt.py -p KakaoTalk.db | grep '"enc":23'

But some of them are using enc 24 I think. I'm seeking for support for enc 24 as well.

Yes, I will add support for enc 24 since you provided me with data encrypted with enc 24. But I would like to add support for enc 23 too, and I can't do that unless you provide me with data encrypted with enc 23. You may find some relevant enc 23 data using the command I mentioned.

I've got only 278 lines, all of them end with "enc":24.


[META] UX improvements and bug fixes

The issue is intended to track progress on improving the quality of user experience provided by OpenSearch Security Analytics by aligning to the OUI components and the design system guidance and by improving interactions with the pages.
The improvements are organized by page/experience so they can be implemented separately.
Correlation engine:

#826
#828
#829
#833
#834

Findings:

#837

Detection rules:

#843

Catch All Triage - 1 2 3 4 5 thanks for opening this


add routes by addRoutes  will not show in $router.options.routes

Version
3.0.1
when l add routes by addRoutes  ,but $router.options.routes is not update

That is normal, options is the object passed to the vuerouter constructor. It's not modified afterwards.

After ‘addRoutes’, the routers added can accessable, but can not be found in this.$router object. So how can I get added routers? Or save a copy instead?

It's ok that options.routes are those passed during initialization, but then we need a method to get also routes added with addRoutes. What about exposing an array with all routes such as this.$router.routes?

@posva it's reasonable that $router.options.routes not modified afterwards, but is there any method available to retrieve the current routes? for me, after addRoutes succeeded there should be a way to get the current routes which are available. Thanks!

@posva Thank you for the reply, sure, please take your time.

Hi @posva, as suggested, I am currently storing an array with the routes added dynamically, so that it can later be used by the rest of my code. However, since all the routes (including dynamically added ones) should be already stored somewhere to get the routing working, why not just exposing the variable where they are actually stored? Could you kindly explain what is critical in exposing it?

@posva any updates on requests in this thread asked by the community?

Maybe rename it to vue-static-router

See https://github.com/vuejs/rfcs/pull/122 getRoutes


global variables in the conflicts field should not be substituted

See https://github.com/ocaml/opam-repository/pull/23402/files#r1116960175

@kit-ty-kate could we please get more detail on this issue so that we can work on it? It's not clear to me from the example what's broken and what the outcome should be. Thank you :)

capnp-rpc-unix specified an old lower-bound for re. This particular re requires jbuilder, which spurs out deprecation warnings (switch to Dune 2! switch to Dune 2!) when used. The warnings were messing up the output of expect tests, in this case I think it was failing the diff of mdx tests. The solution, instead of bumping the lower-bound of re to one version supporting Dune, was to make the package conflict with jbuilder, so that the solver would also select a newer version of re. As the problem only appeared during tests, we added to the opam package
conflicts: [
  "jbuilder" { with-test }
]

Kate remarks however that global variables (with-test) shouldn't be expanded in the conflicts field: ocaml-ci is too zealous and this behaviour is buggy. It shouldn't have passed on this opam file. The with-test was removed.
We should:

change ocaml-ci so that global variables are not substituted in some fields (where's the doc about that?);
probably amend the opam-lint step to reject such opam files.


Communicate about the new release

Communicate about the new release

[x] Roche
[x] testers

Posted on our Slack admiral_testing_01 channel and the Roche internal admiral testers Google chat.


Auto-detect AttachRequired

Resolves #2

/assign @msau42

@davidz627 Sure, would that just go in pkg/connection/connection_test.go?

@davidz627 Also, is there a canonical / preferred method for running unit tests locally for this project?

make test

lgtm, please squash

Squashed most of the patched, left the first one on its own since it's just minor stuff.

/lgtm


sendTokensToAddress RPC

What kind of PR is this?:
/kind feature
What this PR does / why we need it:
Add sendTokensToAddress RPC
Which issue(s) does this PR fixes?:
Fixes part of #48
Additional comments?

Behavior
Test fails when source address is provided but passes when an empty object is passed.
Fails at
  it('should create a transaction with source address provided', async () => {
    const to = await client.wallet.getNewAddress()
    const from = await client.wallet.getNewAddress()
    const transactionHex = await client.account.sendTokensToAddress({ [from]: '2@DFI' }, { [to]: '10@DBTC' })

    expect(typeof transactionHex).toStrictEqual('string')
  })

Error

Take note of conflicts too.

Something wrong with your last commit 734aa24
You reverted the changes you made in 02fd385 which was correct

Something wrong with your last commit 734aa24
You reverted the changes you made in 02fd385 which was correct

@jingyi2811 , what exactly is wrong? I have to make the changes to implement the additional test
https://github.com/DeFiCh/jellyfish/pull/323#discussion_r644674877

@canonbrother , referring to this comment, forward selection mode is not behaving as outlined. It is changes for each test you run making it hard to assets.

@canonbrother , referring to this comment, forward selection mode is not behaving as outlined. It is changes for each test you run making it hard to assets.

i guess the first acc created is forward, can you double check this??

one more test for sending ANT to CAT

@canonbrother , i think sendTokensToAddress only support sending the same currency type.

one more test for sending ANT to CAT

@canonbrother , i think sendTokensToAddress only support sending the same currency type.

yes i expect the error throw

i'm good overall

refactor your test


CR: Need instance to GameController class

Objection:
Viewport positioning requires calls of functions:
Vec2 cocos2d::Lawer::getPosition()  & cocos2d::Lawer::setPosition(Vec2)
Goal:
Achieve access to tsg::game::GameController within tsg::map::MapController.
This could be done with:

Call Of Constructor of MapController with pointer to GameController Instance;
Call of onInit this pointer to instance of GameController.

please use "CR" prefix in title to easy filtering CR's from other issues

Hehe.. didn't mention this. ok.

yeah, just figured it out  either :)


Add support for LotusScript

I propose simple solution to highlight syntax in LotusScript lss files. It can be treated as Visual Basic.

Please see the guidelines to add support for a new extension to Linguist. In particular, notice the requirement on in-the-wild usage.

Do you mean I need to add this: github_search
LotusScript is very similar to ViasualBasic wikipedia
However there are some differencies: ibm
Actually, according to these differencies, it should be then a separate definition. The reason I committed initial request is that I'm going to upload some framework libraries to share with the community and I would like to see it highlighted and to keep it simple if possible. One of the options is just to rename files to .bas, but this could confuse people. I would like to focus on the project I want to share and not on the tools like highlighting.

Echoing @pchaigno's feedback, I only see 62 repositories and 38 users for this. It unfortunately doesn't meet our minimum requirements for in-the-wild adoption.
From the Contributor's Guide:

We try only to add languages once they have some usage on GitHub. In most cases we prefer that each new file extension be in use in hundreds of repositories before supporting them in Linguist.

For your specific use case, you should use .gitattributes to override linguist behavior.


SASS: Sort @extend separately from @include

This relates to issue #143, which I also believe is a good idea.
I feel that @extend should be sorted separately from @include as they are often used for different purposes.
When writing modular SASS, @extend is used to extend modules, and personally I'd like to see what modules a selector is extending first in my SASS, because it affects so much.
@include mixins are used more frequently as shorthand for multiple css rules.
Preferably, I'd like to be able to sort my @include mixins along with related css properties, as per issue #143.
Thanks

I believe, that it's also a good idea to keep @include just right after @extend, cause this rules can be overridden too.

Guys, how are you going to override @extend?
It adds class of selector, in which used @extend, to extended selector, not vice versa
Example:
.extendMe {
  border: 0;
}

.class {
  @extend .extendMe;
  border: 1px solid #000;
}

result:
.extendMe,
.class {
  border: 0;
}

.class {
  border: 1px solid #000;
}

It's not important where exactly @extend has been placed — in the beginning of properties or in the end, and thus it doesn't affect overriding of properties in same selector.
It's more important in terms of code readability where it should be placed, I'd say.

Made this in my fork, only file changed is: https://github.com/csscomb/csscomb.js/blob/master/lib/options/sort-order.js
"$include mixinName" works for mixins with "@mixin mixinName", fallback is always to "$include" if the mixin name is not found.

Fixed in dev, send @hudochenkov some love for this feature :hearts:

Is there a timeframe for when this will get merged to master?

+1 is this going to be merged into master?

+1


Update security PGP key to use new wagtail.org domain

This should be updated in https://github.com/wagtail/wagtail/blob/main/docs/contributing/security.md too.

Thanks for this, Karl! I've merged this change, along with updating the other references, into my main branch with the rest of the changes https://github.com/wagtail/wagtail/pull/7845. This can probably be closed now.

Closing on that basis


METAMODEL-236: Fixed

Fixes the serializability of OperatorType (and FunctionType which I also found had the same issue)

+1


Project structure: the routes folder

Could you please explain what to put under routes folder?
Do I have to put the mapping Component <=> Url or the components using react-router?
Many thanks in advance for your help.
Mauro

@mauro-ni , From the description and the example code, you would place the react-router Routes and the components they should render.
src/routes is responsible for handle root level routes, use this to navigate to your features, like /discussions/* renders the <DiscussionsRoutes /> component
src/features/discussions/routes is responsible for handling any routes after that.
Meaning that the root level routes will navigate to your feature, but then your feature will handle any routing with in it.
I hope that helps.
-G

@CodeNameGrant many thanks for your reply.
If you give a look at Discussions feature you see that the folder also contains Discussion.tsx and Discussion.tsx. I prefer leaving these components in the feature compontents directory.
What do you think?
Many thanks,
Mauro


Reference jwt server

Notary is not very useful in production without a JWT server for authn/z. The JWT implementation expected does not seem particularly compatible with non-Docker Enterprise / Docker Hub implementations (judging by GH issues of people trying to use auth0 and others).
Would it be possible to get a simple reference implementation of a JWT server that works with notary?

This works for me:

setup client on Keycloak with enabled feature docker-v2 auth
grab endpoint url from tab Installation / Registry Config File in Keycloak UI
grab public certificate from tab Installation / Docker Compose Yaml in Keycloak UI
ensure to have the public Keycloak cert file (i.e. keycloak.pem) in the notary-server container (e.g. mount secret)
add the auth section in server-config.json:

 "auth": {
  "type": "token",
  "options": {
        "realm": "https://<my-keycloak-server>/auth/realms/<my-realm>/protocol/docker-v2/auth",
        "rootcertbundle": "/go/src/github.com/theupdateframework/notary/fixtures/keycloak.pem",
        "service": "docker",
        "issuer": "https://<my-keycloak-server>/auth/realms/<my-realm>"
      }
    },

In Keycloak needs to create the same user/credentials like the registry where you want to push. The next remote sign/push will ask for username/password, which is checked in Keycloak, becomes the JWT for Notary and under the same credentials to push to registry.
So far, so good. Unfortunatelly you are still able to delete other user trust if you have a valid JWT. There is no relationship between your credentials and the trust store in Notary.
References:

https://www.keycloak.org/docs/latest/securing_apps/#docker-registry-configuration
https://github.com/theupdateframework/notary/blob/master/docs/reference/server-config.md#auth-section-optional


Inconsistent dictionary for released wikitext-103 models

Hello
I am basically want to evaluate the perplexity of the pre-trained language models. Therefore I replicate the example from here.
My command line is like this
fairseq-eval-lm data-bin/wikitext-103 \     --path data-bin/adaptive_lm_wiki103/model.pt \     --max-sentences 2 \     --tokens-per-sample 512 \     --context-window 2000
This works few days ago even though the perplexity was pretty high ( it was around 197..).
And after commit 6209d7d, when
I run the same command line, I have runtime error like below:
 File "/data/s3475743/venv/bin/fairseq-eval-lm", line 11, in <module>     load_entry_point('fairseq', 'console_scripts', 'fairseq-eval-lm')()   File "/data/s3475743/download_LM/fairseq/fairseq_cli/eval_lm.py", line 252, in cli_main     distributed_utils.call_main(args, main)   File "/data/s3475743/download_LM/fairseq/fairseq/distributed_utils.py", line 173, in call_main     main(args, **kwargs)   File "/data/s3475743/download_LM/fairseq/fairseq_cli/eval_lm.py", line 80, in main     suffix=getattr(parsed_args, "checkpoint_suffix", ""),   File "/data/s3475743/download_LM/fairseq/fairseq/checkpoint_utils.py", line 190, in load_model_ensemble     filenames, arg_overrides, task, strict, suffix,   File "/data/s3475743/download_LM/fairseq/fairseq/checkpoint_utils.py", line 211, in load_model_ensemble_and_task     model.load_state_dict(state["model"], strict=strict, args=args)   File "/data/s3475743/download_LM/fairseq/fairseq/models/fairseq_model.py", line 93, in load_state_dict     return super().load_state_dict(new_state_dict, strict)   File "/data/s3475743/venv/lib/python3.6/site-packages/torch/nn/modules/module.py", line 830, in load_state_dict     self.__class__.__name__, "\n\t".join(error_msgs))) RuntimeError: Error(s) in loading state_dict for TransformerLanguageModel: 	Missing key(s) in state_dict: "decoder.output_projection.weight".
Also, I did not train the model, so I basically download it and evaluate the same wiki-103 dataset.
Thank you in advance

Thanks, this is an issue with adaptive softmax + recent changes that broke backward compatibility. I'll submit a fix shortly.
After the fix, this is what I get:
$ fairseq-eval-lm data-bin/wikitext-103 --path adaptive_lm_wiki103/model.pt --max-sentences 2 --tokens-per-sample 512 --context-window 2000
# (...)
# 2020-05-13 06:59:43 | INFO | fairseq_cli.eval_lm | Loss (base 2): 4.2274, Perplexity: 18.73

Thank you so much for your reply!!
I just pulled git and tried out what you suggested.
However, I still has the same error.
fairseq-eval-lm data-bin/wikitext-103 --path data-bin/adaptive_lm_wiki103/model.pt --max-sentences 2 --tokens-per-sample 512 --context-window 2000
Traceback (most recent call last):   File "/data/s3475743/venv/bin/fairseq-eval-lm", line 11, in <module>     load_entry_point('fairseq', 'console_scripts', 'fairseq-eval-lm')()   File "/data/s3475743/download_LM/fairseq/fairseq_cli/eval_lm.py", line 252, in cli_main     distributed_utils.call_main(args, main)   File "/data/s3475743/download_LM/fairseq/fairseq/distributed_utils.py", line 173, in call_main     main(args, **kwargs)   File "/data/s3475743/download_LM/fairseq/fairseq_cli/eval_lm.py", line 80, in main     suffix=getattr(parsed_args, "checkpoint_suffix", ""),   File "/data/s3475743/download_LM/fairseq/fairseq/checkpoint_utils.py", line 190, in load_model_ensemble     filenames, arg_overrides, task, strict, suffix,   File "/data/s3475743/download_LM/fairseq/fairseq/checkpoint_utils.py", line 211, in load_model_ensemble_and_task     model.load_state_dict(state["model"], strict=strict, args=args)   File "/data/s3475743/download_LM/fairseq/fairseq/models/fairseq_model.py", line 94, in load_state_dict     return super().load_state_dict(new_state_dict, strict)   File "/data/s3475743/venv/lib/python3.6/site-packages/torch/nn/modules/module.py", line 830, in load_state_dict     self.__class__.__name__, "\n\t".join(error_msgs))) RuntimeError: Error(s) in loading state_dict for TransformerLanguageModel: 	Missing key(s) in state_dict: "decoder.output_projection.weight".
Do you think I still miss out something?

Thank you so much for your reply!
I tried with your fix and now I can get perplexity.
However, the perplexity score is pretty higher than yours. I am not so sure why mine is way higher.
This is what I got  :
fairseq-eval-lm data-bin/wikitext-103 --path data-bin/adaptive_lm_wiki103/model.pt --max-sentences 2 --tokens-per-sample 512 --context-window 2000
And this is the results:
`
2020-05-14 11:41:34 | INFO | fairseq.tasks.language_modeling | dictionary: 267744 types
2020-05-14 11:41:34 | INFO | fairseq_cli.eval_lm | loading model(s) from data-bin/adaptive_lm_wiki103/model.pt
2020-05-14 11:41:43 | INFO | fairseq.tasks.language_modeling | dictionary: 267744 types
2020-05-14 11:41:43 | INFO | fairseq.data.data_utils | loaded 4358 examples from: data-bin/wikitext-103/test
2020-05-14 11:41:44 | INFO | fairseq_cli.eval_lm | data-bin/wikitext-103 test 230 examples
2020-05-14 11:41:52 | INFO | fairseq_cli.eval_lm | num. model params: 246933504
2020-05-14 11:42:38 | INFO | fairseq.logging.progress_bar | :    101 / 115 wps=4638
2020-05-14 11:42:45 | INFO | fairseq_cli.eval_lm | Evaluated 245569 tokens in 52.2s (4699.92 tokens/s)
2020-05-14 11:42:45 | INFO | fairseq_cli.eval_lm | Loss (base 2): 7.6252, Perplexity: 197.43
`
Again, thanks for fix!

Ah, shoot, I figured it out. The wikitext-103 dictionary used for that model isn't the same as the one originally used (dict.txt had a slightly different order, since it was generated before this fix: https://github.com/pytorch/fairseq/commit/10ad74953f671168c474d0da5700d91661e2b5a2).
So you'll need to preprocess wikitext-103 using that dictionary.
I'm going to update the docs with info about this.

Just to ask, does this bug also effect the WMT-19 English language models? Or was it generated after the dictionary order was made deterministic?

Just to ask, does this bug also effect the WMT-19 English language models? Or was it generated after the dictionary order was made deterministic?

No, the issue here is if you download the wikitext-103 dataset and preprocess it from scratch, it generates a dictionary that's not compatible with the released wikitext-103 model checkpoint. The workaround is to preprocess the data using the dictionary provided in the model archive.
For the WMT'19 language models, since we don't release instructions for preparing the data from scratch, this isn't really an issue -- you're always going to use the provided dictionary file since that's the only option available.

I've retrained the model to use the new dictionary (the one produced on master), available here: https://dl.fbaipublicfiles.com/fairseq/models/lm/adaptive_lm_wiki103.v2.tar.bz2


Document - running Meltano in Production + RBAC

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/528
Originally created by @valexieva on 2019-03-25 16:42:50

Problem to solve
Currently, it's unclear how a user can run Meltano in production. We need to write a tutorial clarifying the required steps to spin up an instance.
Target audience
Admin + regular users
Further details
In order for the tool to be professionally used in a real life setting, a user needs to be able to run it in production
Tasks to Complete

[x]  Document how to run a Flask application using gunicorn
[x]  Document the Flask configuration meltano/api/config.py
[x]  @bencodezen Document steps to spinning up production instance - this is a priority
[x]  @bencodezen Document steps to using RBAC
[x]  @bencodezen explain how the feature flagging works for users who want to turn it on

What does success look like, and how can we measure that?
Find a simple way to express the value of RBAC and how it works to users
(cc: @dmor )

View 30 previous comments from the original issue on GitLab


onPositionOutsideLocation being called even inside the location

Hello,
I have followed the step by step instruction to set up the indoor location.
While I am outside the location, it calls the onPositionOutsideLocation correctly,
but when I am inside the location, it calls both onPositionUpdate and onPositionOutsideLocation
repeatedly. Is this normal behavior or an experimenting environment problem?

Hello @wjs9715 !
This is not the correct behaviour. May I ask you for further details on this case? Is it always happening, or only when you re-visit your place? Also, may I ask you for phone details - model, OS version.
Cheers!

@pawelDylag Thanks for the response,
I have tested with 2 different devices and the results were same
Sony F3113, Lenovo TB3-730F
Both running 6.0, compileSdkVersion 25
Also, it is always happening. Could this be problem with the beacon setup?
I am testing with 4 UWB beacons deployed to each corner of a 4m X 5m room.

Hey @wjs9715
This is awesome, thanks for the details! The bug was definitely in our SDK. I know what was the problem and the fix is scheduled for next release. Can't promise we will release new version today, but it's definitely a matter of few days
Thanks a lot for reporting this 👍  - will notify you when new version is live.

Hello, @pawelDylag any updates on when the new release will be ready?

Hey @wjs9715 - the new SDK version 2.0.0 is live. See the CHANGELOG to see the details. Your case shouldn't be a problem anymore :)

@pawelDylag
Hey! Thanks for the new release!
Can you check if the repo is pushed to the maven or jcenter?
It shows
Error:(30, 13) Failed to resolve: com.estimote:indoorsdk:2.0.0

@wjs9715  - gosh, thanks for pointing this out. It seems that it's still pending in JCenter. Will notify you when it will be online :)

Hey @wjs9715 - version 2.0.0 should be now accessible :)

@pawelDylag
Hey, I just tested but seems still has issues.
When I start positioning, onPositionOutsideLocation gets called once then nothing happens from there.
I compiled and tested with the example app also, but could not get it to work with both of device I mentioned above. It does not detects nor update the location.
Can you please check?

Hey @wjs9715
We are still working on underlying ScanningSDK (new in Indoor 2.0.0) to handle as many phone models as possible. Couple of necessary questions to debug your problem (check on both phones):

Does your app have Location permissions enabled?
Does it have Bluetooth turned on?
What is your Beacon setup - is Estimote Location packet enabled? is iBeacon packet enabled?

Also, when launching positioning, could you check your log messages (no filters)? Android bluetooth stack should log any info about its status. Something like ble: registered callback or so. Could you send me those?
Although we are trying to test our new scanning mechanism on as many phones as possible, yours were not on the current priority list, so this is why we need to rely on your feedback for now :)
But don't worry, we will manage that! 💪

@pawelDylag
Thx for the response

Yes, both of them has location permission.

Yes, both of them has Bluetooth turned on.

Yes, the beacons have location packet and iBeacon enabled.

10-24 14:24:56.166 9592-9592/com.example.airport D/BluetoothAdapter: getLeState() returning 12 10-24 14:24:56.222 9592-9592/com.example.airport D/BluetoothAdapter: getLeState() returning 12 10-24 14:24:56.222 9592-9592/com.example.airport D/BluetoothAdapter: STATE_ON 10-24 14:24:56.225 9592-9592/com.example.airport D/BluetoothAdapter: getLeState() returning 12 10-24 14:24:56.230 9592-9605/com.example.airport D/BluetoothLeScanner: onClientRegistered() - status=0 clientIf=5 10-24 14:24:56.236 9592-9592/com.example.airport I/BluetoothLeScanner: startRegisteration: mLeScanClients={com.estimote.scanning_sdk.packet_provider.scanner.PostLollipopEstimoteScanner$scan$1$callback$1@6df36c8=android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper@66b7361} 10-24 14:24:56.246 9592-9592/com.example.airport D/BluetoothAdapter: getLeState() returning 12 10-24 14:24:56.247 9592-9592/com.example.airport D/BluetoothAdapter: STATE_ON 10-24 14:24:56.248 9592-9592/com.example.airport D/BluetoothAdapter: getLeState() returning 12 10-24 14:24:56.256 9592-9604/com.example.airport D/BluetoothLeScanner: onClientRegistered() - status=0 clientIf=6 10-24 14:24:56.263 9592-9592/com.example.airport I/BluetoothLeScanner: startRegisteration: mLeScanClients={com.estimote.scanning_sdk.packet_provider.scanner.PostLollipopEstimoteScanner$scan$1$callback$1@6edde86=android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper@eb8c347, com.estimote.scanning_sdk.packet_provider.scanner.PostLollipopEstimoteScanner$scan$1$callback$1@6df36c8=android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper@66b7361} 10-24 14:24:57.161 9592-9592/com.example.airport I/System.out: onPositionOutsideLocation called
This is Bluetooth stack log
onPositionOutsideLocation gets called once and nothing happens from there.

@pawelDylag
We found a device that works and its finally working!
Thanks for the help.

@wjs9715
i face the same problem, can you tell me how did you solved it ?

Bump

@rusmichal
is it must be indoor beacon (Location
UWB Beacon) or it can work with ble beacon(Location
BLE Beacon) ?


support FromIterator<(T, T)>

It would be nice to be able to use an iterator over dependency pairs to create a TopologicalSort.

Hrm, this would conflict with the existing FromIterator implementation. IMO, this one is more useful. Thoughts?

Maybe use an intermediate structure to differentiate between the two?


Fatal error: Uncaught Error: Call to undefined method Slim\Slim::pipe()

Cors Middleware is normally installed with composer.
My index.php of API is:

<?php

$loader = require 'vendor/autoload.php';

//Consults Routes
$app = new \Slim\Slim(array(
    'templates.path' => 'templates'
));

use Tuupola\Middleware\CorsMiddleware;

#$app->pipe(ImplicitOptionsMiddleware::class);
$app->pipe(CorsMiddleware::class);

$app->add(new Tuupola\Middleware\CorsMiddleware([
    "origin" => ["*"],
    "methods" => ["GET", "POST", "PUT", "PATCH", "DELETE"],
    "headers.allow" => [],
    "headers.expose" => [],
    "credentials" => false,
    "cache" => 0,
]));

$app->get('/consults/', function() use ($app){
	(new \controllers\Consults($app))->list();
});

$app->get('/consults/:id', function($id) use ($app){
	(new \controllers\Consults($app))->get($id);
});

$app->post('/consults/', function() use ($app){
	(new \controllers\Consults($app))->new();
});

$app->put('/consults/:id', function($id) use ($app){
	(new \controllers\Consults($app))->edit($id);
});

$app->delete('/consults/:id', function($id) use ($app){
	(new \controllers\Consults($app))->delete($id);
});

//Vehicles Routes
$app->get('/vehicles/', function() use ($app){
	(new \controllers\Vehicles($app))->list();
});

$app->get('/vehicles/:id', function($id) use ($app){
	(new \controllers\Vehicles($app))->get($id);
});

$app->get('/vehicles-top/', function() use ($app){
	(new \controllers\Vehicles($app))->listTop();
});

$app->post('/vehicles/', function() use ($app){
	(new \controllers\Vehicles($app))->new();
});

$app->put('/vehicles/:id', function($id) use ($app){
	(new \controllers\Vehicles($app))->edit($id);
});

$app->delete('/vehicles/:id', function($id) use ($app){
	(new \controllers\Vehicles($app))->delete($id);
});

$app->put('/vehicles/:id', function($id) use ($app){
	(new \controllers\Vehicles($app))->incrementConsult($id);
});

$app->run();

I have a error with the line
$app->pipe(CorsMiddleware::class);

Fatal error: Uncaught Error: Call to undefined method Slim\Slim::pipe() in C:\wamp64\www\tabela-fipe\backend\tabelafipe-backend\api\index.php on line 15
Is slim the problem here?

Slim does not have pipe() method. Instead it uses add().
$app->add(new Tuupola\Middleware\CorsMiddleware);

Thanks, @tuupola
Now, I have this error with the line above:
$app->add(new Tuupola\Middleware\CorsMiddleware);

Error: Fatal error: Uncaught Error: Class 'Tuupola\Middleware\CorsMiddleware' not found
If, I change the class to Cors, I have other error:
$app->add(new Tuupola\Middleware\Cors);

Error: Fatal error: Uncaught TypeError: Argument 1 passed to Slim\Slim::add() must be an instance of Slim\Middleware, instance of Tuupola\Middleware\Cors given...

You have installed old version 0.5.2 or older. Try again with the latest. For this you need PHP 7.1 or higher.

Looking at the error message:
Error: Fatal error: Uncaught TypeError: Argument 1 passed to Slim\Slim::add() must be an instance of Slim\Middleware, instance of Tuupola\Middleware\Cors given...

Are you running Slim 2? It is not supported. This middleware can be used with Slim 3 or higher.

@tuupola
I checked here and my PHP version is 7.2.18. I'm using WAMP server locally.
My Slim version is 3.12.2

Ok. I would try installing the latest version of middleware. Judging from the classname you have  0.5.2 or older.


Tabris application requires permissions that aren't used

Problem description
Simple applications require permissions that they should not.
Expected behavior
Simple apps that don't use location,  internet, or camera, should not require those permissions. These permissions can scare off privacy minded and security minded costumers, because  with the simplicity of some apps, these permissions seem nefarious.
Environment

Tabris.js version: 3.4.0
Device: N/A
OS: Android and IOS

Code snippet
Fork the Tabris.js Example App
Build the application using the online build service
Downloaded the .apk
run aapt dumb permissions yourapp.apk and notice:
package: yourapp.bundleID
uses-permission: name='android.permission.ACCESS_NETWORK_STATE'
uses-permission: name='android.permission.INTERNET'
uses-permission: name='android.permission.CAMERA'
uses-permission: name='android.permission.ACCESS_COARSE_LOCATION'
uses-permission: name='android.permission.ACCESS_FINE_LOCATION'

As mentioned in https://github.com/eclipsesource/tabris-js/issues/2043#issuecomment-621197668 this has been fixed already in the nightly builds.  As it is not possible to re-publish code under the same version this is only available in the nightly builds.  It could be available in 3.5.1 if a patch release is made in the future.  In the meantime, you'll want to build your app against that aforementioned version or later

It appears I misunderstood, I thought the fix was that permission strings wouldn’t be required anymore, but the permissions still would be shown. If the permissions were removed then this is closed.

Just tested again in 3.5.0-dev.20200430
./aapt dump permissions app-release-unsigned.apk
package: com.packagenname
uses-permission: name='android.permission.ACCESS_NETWORK_STATE'
uses-permission: name='android.permission.INTERNET'
uses-permission: name='android.permission.CAMERA'
uses-permission: name='android.permission.ACCESS_COARSE_LOCATION'
uses-permission: name='android.permission.ACCESS_FINE_LOCATION'

Regarding iOS:
When you submit an app to Apple, they check which iOS SDKs are referenced in it. That is why your app was rejected and you received an email about missing usage description strings.
An automatic code review cannot infer if that code will be executed or not, it can only check if references exist or not.
That is why even when you build our hello world app it ~is~ was rejected, even though it did not use your location, camera, etc. It was rejected because the Tabris.js framework binary contains references to those APIs. We have camera support in it and so on.
The thing which should keep you calm is that iOS does not allow using any of those APIs without asking the user for permission first. Tabris.js guards you against it, but if you would try to use the camera, location, or any other privacy concerning API (in a pure native app) without asking for permission, the operating system will kill your app immediately.

What does it mean for Tabris.js developer?

You should not be worried about your application users' privacy because Tabris.js executes only your code. We do not track or log anything in any way.

Even if anyone, anyhow would like to "spy" through your application and take pictures or track location, access pictures library, your users are protected by the operating system on their phones.

If you plan to use camera or access users' location, please update usage description strings as described in https://github.com/eclipsesource/tabris-js/issues/2043#issuecomment-621197668.

We have to provide generic descriptions in the platform to make Apple's "automatic code review" happy. Otherwise, every Tabris.js app would need to have them defined in cordova/config.xml or we would have to remove all APIs that are based on privacy-sensitive APIs from Tabris.js. Which no one wants, right? :)

Ah okay. My bad I understand now. And regarding android, to anyone worried, even though the permissions are technically there in the apk, they are not shown when installing the apk. Thank you for clearing up my misunderstanding.

@Joshua-Beatty If you are hyper-concerned you can also add a build hook to remove them completely.  edit-config doesn't support remove but you can add a hook to scrub them:
config.xml:
<hook type="before_compile" src="remove_permission.js" />

remove_permission.js:
#!/usr/bin/env node
const fs = require('fs');

if(fs.existsSync('platforms/android')) {
	const MANIFEST = 'platforms/android/AndroidManifest.xml';
	let lines = fs.readFileSync(MANIFEST, 'utf8');

	lines = lines.replace(/\s+<uses-permission android:name="android\.permission\.ACCESS_(?:FINE|COARSE)_LOCATION" \/>/g, '');

	fs.writeFileSync(MANIFEST, lines);
}


SQLite 3.40.0 change order or rows when traversing a tree in a CTE

It's not particularly important that default order of row should be preserved. But this test was checking if we collect base to derive classes. It seems pre 3.40 we were getting Geom3d property before Element which was wrong. So, 3.40 fixes the ordering issue in which CTE traverses the tree.

Is this supposed to target imodel02 instead of master? If not, what exactly does this PR do?

Is this supposed to target imodel02 instead of master? If not, what exactly does this PR do?

I think you're right Paul, technically this commit should've merged into imodel02 at the same time Affan's PR to update SQLite in imodel-native. This will fix the failing test in my open PR https://github.com/iTwin/itwinjs-core/pull/4823 to merge into master with 3.6.0 addon.

This will fix the failing test in my open PR https://github.com/iTwin/itwinjs-core/pull/4823 to merge into master with 3.6.0 addon.

Why not just make this change in your PR then?

Included in #4823.


Feature request: support manual r? without automatic reviewer assignment

In Miri we generally don't use automatic reviewer assignment, instead one of us just grabs a PR when something still needs review. It would still be nice to be able to use r? though. So it would be nice if triagebot could let us use r? without automatically assigning someone to every new PR.

@RalfJung ok so basically disable automatic assignment when a new PR with A-miri label is created, correct? There are also a number of labels in the same area (A-mir*). Should they be ignored as well?

This is for PRs in https://github.com/rust-lang/miri/, so no there's no label involved. I am just looking for a way to say in triagebot.toml that we want to support r? but not have anyone automatically assigned.


Test Libreswan cable-driver (will be the default in future release of Subctl)

This is labeled as part of the 0.5.0 project.  Has it been completed or should it be moved to 0.6.0?

Moving to 0.6.0, as LibreSwan does not yet work with non default ports:
https://github.com/submariner-io/submariner/issues/683

@skitt @nyechiel
I will keep this issue open until we change the the default cable driver from "StrongSwan" to "LibreSwan".
This would happen only when @mangelajo libreswan fix is merged to LibreSwan official release.
For now, in order to use LibreSwan on On-Prem cluster (with NAT), we have to explicitly add --version libreswan-git.

@manosnoam just to be clear, this isn’t Miguel’s fix, we’re testing a pre-release of Libreswan which fixes the NAT port issue but the fix is provided by upstream.

bump

Ready for testing now that https://github.com/submariner-io/submariner/pull/904 is merged

Libreswan cable driver is working great in subctl version v0.7.0-53-g7c6a09f:
https://qe-jenkins-csb-skynet.cloud.paas.psi.redhat.com/job/Submariner-OSP-AWS-Globalnet/4/Test-Report/


Native support for .env files

Is your feature request related to a problem? Please describe.
@oclif/core should natively support .env files using process.loadEnvFile
Describe the solution you'd like
Call process.loadEnvFile at the beginning of run using path set in settings
The call should be gated by a check for the node version to avoid any runtime errors.

Is there any workaround for supporting .env files in oclif?

You can you use dotenv in CLI's bin/run.js to load a .env file

You can you use dotenv in CLI's bin/run.js to load a .env file

@mdonnalley I see! I've already done this actually. 😄  But how can I make that work for the bin/run.cmd bin file for Windows?

bin/run.cmd just runs bin/run.js so whatever you do in there should also work for windows


Issue 168 vulnerability checks

New tasks to automate checking for vulnerabilities in images.

@nigelgbanks Do you have instructions on testing this?

Not really, its kinda low level infrastructure consumed by isle-buildkit. it's actively being used by the pull request  https://github.com/Islandora-Devops/isle-buildkit/pull/183
You can see the generate grype reports in github actions on the pull request as proof that they are being generated.

Since you've been looking a pull requests, bump @dannylamb.


Variable selection starting with dollar sign

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-08T23:31:51.320Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

php-debug
felixfbecker
1.10.0

php-intellisense
felixfbecker
1.1.1

Steps to Reproduce:

In source code editor doubleclick on variable name starting with $ sign like $test (commonly used by php and javascript and jQuery).
Only variable name test without $ is selected. Whole variable $test should be selected.

Solution here: https://github.com/Microsoft/vscode/issues/2036#issuecomment-193873124

@alexandrudima and @rebornix, I see our custom wordPattern for javascript used when single clicking on a word, but not when double clicking. It looks like it is falling back to the standard word regexp in that case. Is this the expected behavior?

Setting editor.wordSeparators is what I exactly was looking for. I'm closing this issue.


anvil_reset not resetting the blocknumber

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[x] Foundryup

What version of Foundry are you on?
forge 0.2.0 (cb0c5af 2022-05-13T23:03:11.896722251Z)
What command(s) is the bug in?
anvil --fork-url http://localhost:8545 --fork-block-number 14000007 --port 8544
Operating System
Linux
Describe the bug
In: curl -X POST localhost:8544 --data '{"jsonrpc":"2.0","method":"anvil_reset","params":[{"forking":{"jsonRpcUrl": "http://localhost:8545", "blockNumber": 14000000}}],"id":1}'  --header 'Content-Type: application/json'
In :  curl -X POST localhost:8544 --data '{"jsonrpc":"2.0","method":"eth_blockNumber","params":[],"id":1}'  --header 'Content-Type: application/json'
Out : {"jsonrpc":"2.0","id":1,"result":"0xd59f87"} # not the right block number

Previous attempt to fix this issue : https://github.com/foundry-rs/foundry/pull/1614


⚠ AGEG website (prod) has degraded performance

In 730fd85, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2026 ms

Resolved: AGEG website (prod) performance has improved in 80ca72c.


Forge script (deploy) ouputs wrong goerli etherscan url api

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (08d4315 2023-01-22T00:04:38.303742334Z)
What command(s) is the bug in?
forge script script/WBRL.s.sol:WBRLScript --broadcast --verify --rpc-url ${RPC_URL} --sender $FOUNDRY_SENDER
Operating System
Linux
Describe the bug
1- Action::  We submit a script to GOERLI:
forge script script/WBRL.s.sol:WBRLScript --broadcast --verify --rpc-url ${RPC_URL} --sender $SENDER_ADDRESS
results obtained:
Submitting verification for [src/WBRL.sol:WBRL] "0x91566F97219Ae9b7a989006f8Ee0A5AcCAdA38e8".
Submitted contract for verification:
Response: OK
GUID: pwmd9jjbdgefuv8wthpkk6imp7a33iivu7yjcki2ns2czfc9ej
URL:
https://api-goerli.etherscan.io/apiaddress/0x91566f97219ae9b7a989006f8ee0a5accada38e8
...
Contract successfully verified
All (1) contracts were verified!
2 - The Problem:
Forge script outputs this wrong etherscan api url:
// https://api-goerli.etherscan.io/apiaddress/0x91566f97219ae9b7a989006f8ee0a5accada38e8
// Correct url is:
// https://goerli.etherscan.io/address/0x91566f97219ae9b7a989006f8ee0a5accada38e8
The use of the wrong goerli etherscan url causes subsequent errors when using: cast storage (all)

thanks for flagging this, see linked PRs for fixes.


Enhance documentation with detailed versioning guidelines

This PR updates the documentation to align the versioning guidelines with the terminology used by the /ocabot merge command, which recognizes the keywords major, minor, and patch.

I improved the part that was commented on by @SirAionTech

ping

Can someone merge this? Thanks!
Looking at https://oca.github.io/repo-maintainer-conf/odoo.html#the-odoo-community-association-website, someone from @OCA/board is needed.


added CsrfHeaderMiddleware

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

#65
Added new CsrfHeaderMiddleware
Added new method CsrfMiddleware::withSafeMethods()

Thanks for a pull request. I've left some comments. Also, tests are currently failing. Needs fixing.

Please check the readme with some links to the used yii packages.

Thank you!


"Learn More" link for "Create new LUIS resource" is missing

Describe the bug
On the screen for "Create New LUIS resources", there is text that says "Learn more".
This text:

Is missing a "." at the end.
Has no Hyperlink associated with it.

Version
Version:  1.4.0-nightly.236875.b9e46c2
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[X] Electron distribution

OS

[X] Windows

To Reproduce
Steps to reproduce the behavior:
Add a new LUIS key via the Window:

Select a subscription.
Select "Create new LUIS resources"
Read the text.
Expected behavior
A hyperlink that lets me learn more, as the text indicates.

Make hyperlink:
https://aka.ms/composer-luis-learnmore


multiple react content packages

Currently it is not possible to install multiple content packages that include react components in a single AEM instance - at least with the build process in the archetype. The reason is that the basic react functionality from aem-react-js is bundled with the react components.
TODOs

chunks
Separate base functionality into common chunk.
The bundled javascript must be separated into separate chunks. One chunk contains the common functionality and then there will be a chunk for each content package.
versions
Additionally we need to take into account different versions of aem-react-js. Each content package is compiled against a specific version, while the script engine can currently only use a single. It is possible to support different versions of aem-react-js inside the script engine but then the contained react components cannot be rendered in the same react component tree.
deployment
Each content package could deploy a react script engine client. All clients are managed by the single script engine. The script engine consult the client which contains the react component before rendering. If there are multiple clients with the same aem-react-js dependency then the they shoudl be rendered in a single javascript engine. Hence there will be javascript engines specific to aem-react-js version. The ReactParsys must also include child components dependeing on their compatibilty with the current javascript engine.

Is there any update on splitting current single big bundle into smaller chunks (e.g. per component)?

No, this is still unresolved.


UnicodeDecodeError: 'gbk' codec can't decode byte 0xa4 in position 7599: illegal multibyte sequence

pip install -e .
Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple
Obtaining file:///E:/2023_Code_WEB/segment-anything-2
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... error
  error: subprocess-exited-with-error

  × Getting requirements to build editable did not run successfully.
  │ exit code: 1
  ╰─> [17 lines of output]
      Traceback (most recent call last):
        File "D:\anaconda3\envs\segment-anything-2\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
          main()
        File "D:\anaconda3\envs\segment-anything-2\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "D:\anaconda3\envs\segment-anything-2\lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 132, in get_requires_for_build_editable
          return hook(config_settings)
        File "C:\Users\mm\AppData\Local\Temp\pip-build-env-rf9w8u_t\overlay\Lib\site-packages\setuptools\build_meta.py", line 458, in get_requires_for_build_editable
          return self.get_requires_for_build_wheel(config_settings)
        File "C:\Users\mm\AppData\Local\Temp\pip-build-env-rf9w8u_t\overlay\Lib\site-packages\setuptools\build_meta.py", line 327, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=[])
        File "C:\Users\mm\AppData\Local\Temp\pip-build-env-rf9w8u_t\overlay\Lib\site-packages\setuptools\build_meta.py", line 297, in _get_build_requires
          self.run_setup()
        File "C:\Users\mm\AppData\Local\Temp\pip-build-env-rf9w8u_t\overlay\Lib\site-packages\setuptools\build_meta.py", line 313, in run_setup
          exec(code, locals())
        File "<string>", line 21, in <module>
      UnicodeDecodeError: 'gbk' codec can't decode byte 0xa4 in position 7599: illegal multibyte sequence
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build editable did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

setup.py
Read the contents of README file
with open("README.md", encoding='utf-8') as f:
LONG_DESCRIPTION = f.read()

Hi @monkeycc, it seems that the Hugging Face emoji in README.md is causing trouble with gbk codec. It should be fixed in #218.
For now, you can manually replace the line (https://github.com/facebookresearch/segment-anything-2/blob/0db838b11726893f151fa5826ecfa744e1a7760f/setup.py#L20) with
with open("README.md", "r", encoding="utf-8") as f:


JIRA: Bugfix - Bumping up jira library to 3.2.0 to have pycontribs/jira#1145 fixed.

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[ ] Testing of all the changes has been performed (for bug fixes / features)
[ ] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[X] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[X] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[ ] Verify all checks are passing.
[ ] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[X] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[ ] New App
[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the readme.html.
[ ] If this is a new connector or you are adding new actions

[ ] Please document in the readme.html all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the readme.html.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[ ] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Due to a bug in JIRA python library prior to 3.2.0 release, SOAR JIRA app can assign a newly created ticket to improper user -- the old code searches for all possible matching users, and only takes the very first one. However, the results are sorted alphabetically, so the very first match often isn't the best/exact match. That leads to tickets that are assigned to unexpected users.

What is the new behavior? (OPTIONAL)

As a part of pycontribs/jira#1145, this behavior was changed to search for the best match if multiple users are returned in the username search.

Other information (OPTIONAL)

None

Pay close attention to (OPTIONAL)

The only thing that's needed (hopefully) is the update of requirements.txt and the follow-up static packages linked to the app. No idea if the static packages are somehow collected automatically or I need to add them manually to this PR.

Screenshots (if relevant)

Thanks for contributing!

Hello @valorcz, A PR with the Jira library version update to 3.5.0 is merged in next, hence closing the PR.
cc: @ishans-crest


chore: audit x/upgrade module

Closes https://github.com/celestiaorg/celestia-app/issues/1961

Codecov Report

Merging #2072 (cd5f1cc) into main (b77ac63) will decrease coverage by 0.11%.
The diff coverage is 40.00%.

:exclamation: Current head cd5f1cc differs from pull request most recent head 4a31d75. Consider uploading reports for the commit 4a31d75 to get more accurate results

@@            Coverage Diff             @@
##             main    #2072      +/-   ##
==========================================
- Coverage   24.28%   24.17%   -0.11%
==========================================
  Files         125      125
  Lines       14248    14248
==========================================
- Hits         3460     3445      -15
- Misses      10432    10446      +14
- Partials      356      357       +1

Impacted Files
Coverage Δ

x/blob/types/payforblob.go
69.54% <0.00%> (-4.02%)
:arrow_down:

x/mint/types/constants.go
100.00% <ø> (ø)

test/txsim/blob.go
80.00% <100.00%> (-0.65%)
:arrow_down:

x/blob/ante/ante.go
85.00% <100.00%> (-3.47%)
:arrow_down:

x/blob/keeper/keeper.go
80.00% <100.00%> (-2.76%)
:arrow_down:

... and 1 file with indirect coverage changes

but we do still need some functionality so that IBC continue to compile/work

It's not immediately clear what functionality is preserved in the forked x/upgrade module that is needed for IBC to work.

Does our fork intentionally not register the CLI command? I can't do
$ ./build/celestia-appd query upgrade
Error: unknown command "upgrade" for "query"

Ref: https://docs.cosmos.network/v0.46/modules/upgrade/04_client.html#cli

It's not immediately clear what functionality is preserved in the forked x/upgrade module that is needed for IBC to work. Maybe we clarify that in a follow-up.

ref https://github.com/celestiaorg/celestia-app/pull/1629#discussion_r1170264709

re: the cli commands
since we are unable to submit the software upgrade proposals, imo we should also remove the cli commands that create them since there is no reason to create them


Can not access msk with latest version (2.1.0)

Hi friends,
My version of MSK cluster is 3.5.1, and my client is an EC2 instance with IAM role attach exactly policy
I can't work the SASL mechanism = AWS_MSK_IAM with version 2.1.0 (It can not load credentials)
Here is the error logs:
ERROR org.apache.kafka.common.errors.SaslAuthenticationException: An error: (java.security.PrivilegedActionException: javax.security.sasl.SaslException: Failed to find AWS IAM Credentials [Caused by software.amazon.awssdk.core.exception.SdkClientException: Unable to load credentials from any of the providers in the chain AwsCredentialsProviderChain(credentialsProviders=[AwsCredentialsProviderChain(credentialsProviders=[EnvironmentVariableCredentialsProvider(), SystemPropertyCredentialsProvider(), WebIdentityTokenCredentialsProvider(), ProfileCredentialsProvider(profileName=default, profileFile=ProfileFile(sections=[])), ContainerCredentialsProvider()])]) ...
But i can work with version 1.1.1
Why? Maybe SDKv2 not working? Help me explain this issue.
Thank you!

Hi @hoalt14 , indeed, I miss something during migration, the fix is in PR: https://github.com/aws/aws-msk-iam-auth/pull/169

Hi @hoalt14 , indeed, I miss something during migration, the fix is in PR: #169

still not working 2.1.0
but, It works in 2.0.3
My environment is as follows

MSK Version (Apache Kafka) : 3.5.1
Kafka Client : https://downloads.apache.org/kafka/3.7.0/kafka_2.12-3.7.0.tgz

@jogilsang The fix is not deployed inside the 2.1.0, so it is "normal" to have it, a maintainer should initiate a release

Will this fix be released soon?

We're hitting the same. Can the release be expedited please?

Hi there,
Facing the same issue, here

Fix has been released.


Fix the default version from 0.2.0 to 0.1.1

What type of PR is this?
/kind chore
What does this PR do / why we need it:
See the errors below
Error response from daemon: manifest for quay.io/argoprojlabs/argocd-operator:v0.2.0 not found: manifest unknown: manifest unknown

Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

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

The default version is v0.2.0 and got released.


How can i see logcat

I cannot found any log from terminal? How can i see the log print from GetServer

Start on debug mode, in release all log are suppress for security reasons

Oh thanks. It works fine on VsCode, but it doesn't work on AndroidStudio

Maybe bug from IDE
https://github.com/flutter/flutter/issues/47913

Maybe bug from IDE
https://github.com/flutter/flutter/issues/47913


ARROW-8401: [C++] Add byte-stream-split AVX2/AVX512 implementation

For decode, similar implementation with the SSE version for both AVX512 and AVX2.
For encode, float path implemented for AVX2/AVX512 also double for AVX512.
AVX2 double fall back to SSE as currently no epi16 permute support.
BM_ByteStreamSplit result reach up to 3x for AVX512, 2x for AVX2.
Signed-off-by: Frank Du<EMAIL_ADDRESS>

Below is the size 1024 result,
SSE:
BM_ByteStreamSplitDecode_Float_SIMD/1024            132 ns          132 ns      5296552 bytes_per_second=28.891G/s
BM_ByteStreamSplitDecode_Double_SIMD/1024           387 ns          386 ns      1818101 bytes_per_second=19.7603G/s
BM_ByteStreamSplitEncode_Float_SIMD/1024            263 ns          263 ns      2663415 bytes_per_second=14.5051G/s
AVX2:
BM_ByteStreamSplitDecode_Float_SIMD/1024            104 ns          104 ns      6727450 bytes_per_second=36.686G/s
BM_ByteStreamSplitDecode_Double_SIMD/1024           265 ns          265 ns      2641345 bytes_per_second=28.7856G/s
BM_ByteStreamSplitEncode_Float_SIMD/1024            169 ns          169 ns      4147448 bytes_per_second=22.6001G/s
AVX512:
BM_ByteStreamSplitDecode_Float_SIMD/1024           70.3 ns         70.2 ns      9315691 bytes_per_second=54.356G/s
BM_ByteStreamSplitDecode_Double_SIMD/1024           173 ns          173 ns      4038812 bytes_per_second=44.0386G/s
BM_ByteStreamSplitEncode_Float_SIMD/1024           88.3 ns         88.2 ns      7936455 bytes_per_second=43.2519G/s

You also probably know this, but the BYTE_STREAM_SPLIT encoding is brand new to the project and so not too many people will be able to reap the benefits of these optimizations. Wanted to mention anyway

Can you add unit tests that test the different AVX implementation directly, its not clear that there is good unit test coverage at the moment.

More tests added for different block and with suffix, and also fix for other comments. Pls help to review it again. Thanks.

You also probably know this, but the BYTE_STREAM_SPLIT encoding is brand new to the project and so not too many people will be able to reap the benefits of these optimizations. Wanted to mention anyway

Thanks, actually I don't know as I am very new to arrow:) And it will be very great if you can input us some priority lists of encoding which are more popular to end user, or where I can find the info. We can check the candidates though not each encoding may suitable to SIMD.

Cool, we're happy to have a SIMD expert involved in the project! Maybe we can have a mailing list discussion about the SIMD optimization topic since many people may have ideas about areas where there may be high value. One idea is SIMD-accelerated small size hashes, for values under say 128 bytes I wonder if we can do better than xxhash3

For naming the style guide dictates acronyms should be camel cased.

Ah, fair enough.

Thank you for being persistent @jianxind :-)

For the record, here are the results on an AMD Ryzen with clang 9.0:
-------------------------------------------------------------------------------------------------------
Benchmark                                             Time             CPU   Iterations UserCounters...

Post an interesting finding: gcc is able to auto vectorize the naive scalar implementation and generate much faster code than clang, approaching handwritten sse4.2 performance. With gcc, the benchmark result is kind of weird, sse4 is faster than avx2, maybe longer test data will fix the gap.
See https://godbolt.org/z/joQ4iy.

Post an interesting finding: gcc is able to auto vectorize the naive scalar implementation and generate much faster code than clang, approaching handwritten sse4.2 performance(Encode_Float test). With gcc, the benchmark result is kind of weird, sse4 is faster than avx2, maybe longer test data will fix the gap.
See https://godbolt.org/z/joQ4iy.

Ahh, which path for bad AVX2 result, decode or encode, float or double, and size? The data I post was measured on a server which support AVX512, and I also has a i9 client which capable of AVX2, all result shows AVX2 better than SSE. Maybe the behavior is different per HW, but generally say we can achieve more benefit from the advanced  SIMD instructions.

Ahh, which path for bad AVX2 result, decode or encode, float or double, and size? The data I post was measured on a server which support AVX512, and I also has a i9 client which capable of AVX2, all result shows AVX2 better than SSE. Maybe the behavior is different per HW, but generally say we can achieve more benefit from the advanced SIMD instructions.

Sorry I forgot to say my test env, which is essential to make sense benchmark results.
I tested on AMD EYPC 7251 (znver1), build with gcc-7.4 and clang-9.
Benchmark result is in this link: https://pastebin.com/g1b9C6Tp
With gcc, sse4 code is faster than avx2 on that platform.
I tried some Intel platforms(i7, e5) just now, they work as expected, avx2 > sse4 >> scalar.
This is just some random findings, doesn't mean anything concrete. For performance critical code, I think hand written optimization code has much more consistent and predictable behaviour than compiler generated ones. Some projects (e.g, intel isa-l) go extreme to write everything with assembly to bypass compiler optimizer completely.

AMD "Zen" CPUs apparently have 128-bit SIMD units: 256-bit AVX instructions are split in two instructions for execution.
AMD "Zen 2" CPUs, however, have 256-bit SIMD units, and they benefit from AVX code: see the numbers I posted above (I have a AMD Ryzen 9 3900X).

Make sense. Thank you.


pkg_resources.DistributionNotFound: The 'raven==5.27.0' distribution was not found and is required by gitfs

I`ve tried to launch gitfs on mac os 10.12.2 but got an issue:
mvadim-MacBook-Pro:~ mvadim$ gitfs Traceback (most recent call last):   File "/usr/local/bin/gitfs", line 6, in <module>     from pkg_resources import load_entry_point   File "/usr/local/Cellar/gitfs/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3019, in <module>     @_call_aside   File "/usr/local/Cellar/gitfs/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3003, in _call_aside     f(*args, **kwargs)   File "/usr/local/Cellar/gitfs/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3032, in _initialize_master_working_set     working_set = WorkingSet._build_master()   File "/usr/local/Cellar/gitfs/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/pkg_resources/__init__.py", line 655, in _build_master     ws.require(__requires__)   File "/usr/local/Cellar/gitfs/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/pkg_resources/__init__.py", line 963, in require     needed = self.resolve(parse_requirements(requirements))   File "/usr/local/Cellar/gitfs/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/pkg_resources/__init__.py", line 849, in resolve     raise DistributionNotFound(req, requirers) pkg_resources.DistributionNotFound: The 'raven==5.27.0' distribution was not found and is required by gitfs
But raven with necessary version is installed:
mvadim-MacBook-Pro:~ mvadim$ raven --v 5.27.0

Issue was related to incorrect path for installed by pip modules.


Google Sign In OAuth2 Security Flaw

Hey @himanshusharma89
I was going through the app and came across the following issue:

I pressed sign in with google on the sign-up page.
Didn't select an account and pressed back.
The app redirected me to the Dashboard with null values with though I wasn't authenticated.

I am attaching a screenshot of the app being logged in without auth.

I would like to work on this issue as well along with the Github auth implementation during Hacktoberfest.

Go ahead @rish07. Assigning this issue to you.

Any update @rish07?

Hey @himanshusharma89,
I actually have an exam tomorrow so I'll fix it by tomorrow evening.

Okay, @rish07. Good luck.

Also, the null value is still there after successful sign-up, but it doesn't occur if we restart the app. I checked the mapping methods, it is fine but still, the issue is there. Have a look into this also.

Sure

@himanshusharma89, I would like to solve this issue if @rish07 is facing any problems.

@arbazdiwan, @rish07 will work on it and update us soon

ok Himanshu
On Sat, 3 Oct 2020, 10:22 pm HIMANSHU SHARMA<EMAIL_ADDRESS>wrote:

@arbazdiwan https://github.com/arbazdiwan, @rish07
https://github.com/rish07 will work on it and update us soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/smaranjitghose/girlscript_app/issues/275#issuecomment-703132135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5PZSHJGI36AVJ5PO27VYDSI5JENANCNFSM4R46CGVA
.


#2879 – Incorrect display of attachment points for some Functional Groups

How the feature works? / How did you fix the issue?
There are no actual changes in application logic. It just changes the display for some specific functional groups
closes #2879
Check list

[ ] unit-tests written
[ ] e2e-tests written
[ ] documentation updated
[x] branch name doesn't contain '#'
[x] PR name follows the pattern #1234 – issue name
[x] PR is linked with the issue
[x] base branch (master or release/xx) is correct

Previewed. There are no comments.