[WIP] replace Virtus with dry-types

As we know Virtus isn't maintained anymore, so it makes sense to use something different for coercing. There is a dry-types lib which was created as a replacement for Virtus (at least a part of it). Although, it is only about coercion.
This change gets rid of Virtus and adds dry-types to the stack. Almost, all tests pass. However, there are a few tests which were modified, actually, they didn't make any sense. Yeah, Virtus was providing
some "magic" to make them green, but I think they were unreal.
The structure inside didn't change match. I tried to integrate dry-types without huge refactoring.
If people want to use Virtus for custom types, it is possible after implementing a parse method which initializes a model.
class User
  include Virtus.model

  attribute :id, Integer
  attribute :name, String

  def self.parse(attrs)
    new(attrs)
  end
end

This PR is kind of "work in progress", I would like to gather feedback then proceed. If somebody can add this change to their project and launch tests, it will be awesome, the goal is to avoid breaking changes.
Missing parts:

[ ] changelog
[ ] upgrade instruction (at least about Virtus models as custom type)
[ ] solve an issue with Hashie::Mash

It closes #1347.

Following this, shout if you get stuck or need some help

👀

Dry-types requires Ruby 2.4 or greater :disappointed:

Dry-types requires Ruby 2.4 or greater 😞

Deprecate Ruby 2.3 with a note, changelog, readme change, etc.

Hi @dblock I made recommended changes, please, have a look again. :relaxed: Thanks for your patience! :smile:

@dblock I've made changes to README and UPGRADING. Please, have a look.

I'm good with this! @myxoh want to confirm and merge?

Let's get this build to green? There's some legit looking failure with rack-edge.

@dblock that commit broke Rack :disappointed:

I merged this, thank you so much for doing this work @dnesteryuk, it's a big one. I suggest you email the mailing list describing this change and asking people to give the master branch a try, especially if they do custom type checking.

@dblock I've just tried to run a test suite on one of my projects against your master branch, and all green :grin: There's no custom type checking though.

@dblock where is the mailing list? :grimacing:

@dblock where is the mailing list? 😬

http://groups.google.com/group/ruby-grape

Any reason why we tied to version 1.1.x of dry-types? @dnesteryuk
I tried Grape from the master branch with Ruby 2.7 and see a lot of deprecation messages like:
/.../ruby-2.7.0/gems/dry-types-1.1.1/lib/dry/types/any.rb:18: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call

Even if it's already fixed or would be fixed in the future versions of Dry-types we can't update to it due to version restrictions :(

Any reason why we tied to version 1.1.x of dry-types? @dnesteryuk I know it changes even with patch version changes, but ... I don't think it's a good solution btw

We should relax it, please PR.


Nullable types in golang

Reason/Context
Please try answering few of those questions

Why we need this improvement?
Nullable strings are vital in many languages, also in go! In our current project we have put a lot of effort to support nullability between frontend and bff (graphql) - if it cannot be kept between services (asyncapi), it's a dealbreaker!
Currently type: ["integer", "null"] is translated to interface{} which is not typesafe. Other generators, e.g. https://github.com/99designs/gqlgen solves the problem by converting optional primitives to pointers, which are nullable. E.g. a type string becomes type *string.

https://github.com/asyncapi/modelina/commit/53edd32fcbc2dde8f9f36a0d344e2abc45b1fec2
This change addresses nullable types in typescript - could it be expanded to go?
Description
Please try answering few of those questions

Will this be a breaking change?
Yes

@wiegell thanks for opening the issue!
As I am not that well versed in Go, hope you don't mind answering some questions 😅 Is it ALL types that just need *type for it to be nullable?
v2 actually can handle this, we just need to alter a few lines of code for the type generation 🙂

Yes all pointers (*) are nullable, but some value types can also be nil.
See this proposal: https://github.com/golang/go/issues/30177
Motivation -> 2nd bullet explains using a pointer to simulate nullable.
Proposal -> first bullet list is a list of types that cannot be nil.
So it would be needed to add * to the golang type corresponding to a nullable version of the following asyncapi types:
string
number
integer
boolean
object
I presume asyncapi array type is translated to golang slice (which is nullable in contrast to a golang array of fixed length), so in that case a value type without * will suffice.
I'm not with my computer the next couple of days, but i would like to check if my reasoning here checks out with the graphql generator mentioned above, and i'll get back to you!

As a really good example you can check this project for struct generation: https://github.com/deepmap/oapi-codegen
But we also need to take care of non required with the omitempty struct tag:  https://pkg.go.dev/encoding/json#Marshal

So i just checked with openapi - they use string pointers for non-required fields as well and then value types when the string is required.

I started using AsyncAPI generation for Go and found this current limitation, which is a deal breaker for me as well.

Thanks @jonaslagoni for pointing out the file. I'll take a look at that some time this week.

Just an update: I ended up writing my own template and made a quick fix for the nullable types I needed, which is not worth sharing since it's more a hack than a proper solution. So I won't spend time with this ticket, unfortunately.


Could not read config from meta tag with name "ghost/config/environment"

Issue Summary
Running on Ubuntu 14.04 after upgrading from 0.8.0 to 0.10.0. When attempting to load the admin panel, I just get a blank screen and this in the console:
ghost.min.js?v=a3e352156b:25 Uncaught Error: Could not read config from meta tag with name "ghost/config/environment".

Steps to Reproduce

Start with 0.8.0
Remove core, package.json, npm-shrinkwrap.json, index.js
Move 0.10.0 equivalents into ghost directory
Remove node_modules
Run npm install --production
Run npm start --production

All pages will load fine, but when I navigate to mywebsite.com/ghost I just get a blank page.
Technical details:

Ghost Version: Now 0.10.0
Node Version: v4.5.0
Browser/OS: Linux 14.04 on Chrome 52
Database: Sqlite

Hey @dalanmiller When i follow http://support.ghost.org/how-to-upgrade/, i don't get this error.
Can you please swing by our slack channel and ask for help there? We can provide better support there than in the Github issue tracker, and if this turns out to be an actual issue with Ghost this issue can be reopened.

Not sure if you were able to fix this issue @dalanmiller but I got it working after I killed the node instance.
> sudo netstat -lp
tcp  0 0 *:2368 *:* LISTEN 11111/node

> sudo kill -9 11111

Then restarting got everything working.

the same as me... from 0.7.9 to 0.11.0

FYI, For anyone finding this - I ran into this issue when migrating to a new server 0.79 to 0.11.7. I hadn't installed the SSL certificate on the new server, but was using CloudFlare in front of Ghost. Switching CloudFlare from Full to Flexible SSL mode enabled me to login to the admin panel.


Python function execution consistency

Issue
I'm having a strange consistency issue when I try running one of my functions. When I make small, seemingly insignificant, modification to my python script, the likelihood of the function running decreases. When it doesn't run, I get following log output:
2019-04-13T14:24:53 Welcome, you are now connected to log-streaming service.
2019-04-13T14:24:56.773 [Information] Executing 'Functions.finn-old-homes' (Reason='This function was programmatically called via the host APIs.', Id=830c1f21-f356-4408-996d-14405f629e11)
2019-04-13T14:26:53 No new trace in the past 1 min(s).
However, If I modify my python script in a way which only affects the execution time (reducing the number of items in my loops dramatically), the function is able to execute successfully. This shouldn't be the case considering I opted for an App service plan.
Is this a bug or am I doing something wrong?
Environment info
Pricing tier: (B1: 1)
Functions Version: 2.0
Worker: Python
azure CLI version: 2.0.61
Resource Group: finn-resourcegroup
Functions app: finn
App Service plans: finnserviceplan

Are you making edits to your app directly via portal?

As far as I know, that's not possible with Python as it's in preview mode. I'm making the changes locally and deploying with the Functions Core Tools.

Can you describe your scenario in more detail? I'm seeing that the function in question is timer triggered once every minute. That might be related to why reducing execution time leads to the function working correctly more often.

I think I figured out the issue. If you take a look at Operation ID "uS6GQyRpdbg=" for function finn-old-homes you will see it failed due to the timeout value being exceeded. The script executes somewhat faster on my local machine than what I had originally allocated as "functionTimeout" in host.json, so I was a bit surprised the timeout value had been reached. I'm uncertain why there are performance differences. Could it be related to writing to Blob storage or http requests from my script?
I just doubled "functionTimeout" to 10 hours and ran the job. Hopefully, 10 hours should be more than sufficient for getting the task done. I will get back to you with results.

Could it be related to writing to Blob storage or http requests from my script?

This could potentially be the case. The App Service Plan has a 30 minute timeout by default in v2.x (https://docs.microsoft.com/en-us/azure/azure-functions/functions-host-json#functiontimeout), but there is no upper limit. It's hard to say why this might be happening without knowing the specifics of what your function is doing.

Hi @WalidMustapha, is this issue resolved with the timeout change?

Hi @maiqbal11,
Sorry I didn't get back to you sooner. I ended up deploying using Apache Airflow which accomplishes what I was looking for from Azure functions by using DAG file definition. I never managed to get the function working with the timeout change, but I didn't have time to debug.

@WalidMustapha, sorry to hear about that. I'll close the issue for now. Feel free to re-open if you need more assistance on this.


fix(trait): trait error "camel-catalog already exists" during upgrade

Fix #3822

Release Note
NONE

@squakez Sure. I'll do so shortly.


Per Division Paired Unit Template

Describe the feature
Our paired unit template at the moment is TROJAN{incremental} which is a paired callsign for the firearms division. However because its a global paired template it means other units that are not firearms are also marked as TROJAN{incremental}.
The requested feature would allow us to set per division paired unit templates so that the unit merging will use a callsign format from their own division and not one they are not in.
Eg. Roads - RAPTOR{incremental}, Firearms TROJAN{incremental} etc.
Additional Context
No response

closes by #722


[e2e] use ci job id to name stacks

What does this PR do?

Use job id instead of pipeline id to name pulumi stacks on the CI
Motivation

Limit collisions on stacks across jobs running on the same pipeline. We currently ask to ensure each concurrent job has one stack with one unique name, this change would allow to run the same test suite with different flavours on the same pipeline
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Active mouseover too active?

Currently as you mouseover features, the table view updates. It can be hard to get it to stay on the feature you're interested. You can hold down the control key to disable the table updates.
It might be preferable to reverse this. Ie, by default, mousing over a feature does NOT update the table view, and you have to hold down the control key to make it update.
Alternatively, you have to click on a feature to make it update? (No control key involved.)?

You now have to click on a feature to update the FeatureDetails section.


Making sure ordering of tests does not matter for BlobStorage tests

Codecov Report

Merging #773 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #773      +/-   ##
============================================
- Coverage     65.68%   65.63%   -0.06%
+ Complexity     3914     3912       -2
============================================
  Files           342      342
  Lines         25947    25947
  Branches       3244     3244
============================================
- Hits          17043    17030      -13
- Misses         8002     8011       +9
- Partials        902      906       +4

Impacted Files
Coverage Δ
Complexity Δ

...github.ambry.rest/AsyncRequestResponseHandler.java
88.52% <0%> (-2.3%)
23% <0%> (ø)

...in/java/com.github.ambry.store/BlobStoreStats.java
83.73% <0%> (-1.44%)
90% <0%> (-2%)

...java/com.github.ambry.router/GetBlobOperation.java
88.32% <0%> (-0.27%)
30% <0%> (-1%)

.../main/java/com.github.ambry.store/HardDeleter.java
81.93% <0%> (ø)
42% <0%> (ø)
:arrow_down:

.../src/main/java/com.github.ambry.store/Journal.java
91.37% <0%> (+1.72%)
20% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Any way to convert the model to tensorflow?

Hello:
I have changed the pose-ae-train to detect keypoints no-human related. This results in different number of outputs (not 17) and different weights in layers. Anybody knows how to convert the pytorch model to tensorflow (so I can infere with pose-ae-demo)?
I tried pytorch2keras, onnx and mmdnn but none of them works.
Thanks in advance.

have you solve this problem?


Dependencies Status not Rendering in Readme

I can fix this , please assign this ro me

Hello,
I tried to search for the site https://david-dm.org/ and it seems like the site is down or so. This is the reason the dependency status is broken.
If there are any alternatives I will be happy to help.


Task Count for Specific Time Frames in Dashboard

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
I would like to request a feature that enables users to create a dashboard displaying the number of tasks created on specific dates, such as daily, weekly, monthly, quarterly, and yearly. While it's possible to include the created date in tasks, there isn't a straightforward method to count the tasks for each time frame and represent this count as a 'task count' in the dashboard.
✔ Solution
Implementing a feature that simply displays the number of tasks resulting from a query would provide valuable insights, such as:

Comparing tasks added vs. tasks completed within a given timeframe
Being more mindful of the tasks added to avoid overwhelming oneself by continuously adding tasks without completing them

This feature would help users better manage their tasks and maintain productivity. Any input or guidance on this feature request would be greatly appreciated. Thank you in advance!
❓ Alternatives
I have considered learning dataview and trying to apply it so this could be the result, but then I would have to make two queries and have a seperate note where all my task queries reside in. It gets messy.
📝 Additional Context
This feature would be helpful for users who want to analyze their task management and productivity trends over time. It would provide an overview of task distribution, making it easier to plan and adjust workflows accordingly.

Ohhhh just the count. Well, I could imagine a 'hide tasks' option but not sure that many ppl would wish to hide their tasks collected with Tasks plugin.
But there is also a pretty straightforward solution with dataview: there should be an option to get the length of dv.taskList(tasks, groupByFile) which would bo the trick. Check https://blacksmithgu.github.io/obsidian-dataview/api/code-reference/ for details.

I would start with an dataviewjs inline like
`$= dv.pages().file.tasks.where(t => !t.completed).where(t => t.text.includes("#task")).length`

Let's not continue this conversation here as it goes out of the Tasks topic. I suggest asking for help in dataview GitHub or Obsidian Discord. If you need some help feel free to send me a message. You can find me in Obsidian Discord with the same name.

Thanks for getting back to me. I opened a discussion thread #1863 with claremacrae and he told me

@lightningRalf No, she told you...

There is already a request for Task queries to only show the number of matching tasks, and not the matching tasks:

#1626

I am therefore closing this as a duplicate.
If you would like to split any of the other aspects of this out in to individual topics, feel free.


Samples fail upload if no coverage

Could add a try/except so samples can be uploaded and failed in the database.
Probably similar issue to #21

fixed in #50


ros2_controllers: 4.13.0-1 in 'jazzy/distribution.yaml' [bloom]

Increasing version of package(s) in repository ros2_controllers to 4.13.0-1:

upstream repository: https://github.com/ros-controls/ros2_controllers.git
release repository: https://github.com/ros2-gbp/ros2_controllers-release.git
distro file: jazzy/distribution.yaml
bloom version: 0.12.0
previous version for package: 4.12.1-1

ackermann_steering_controller
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

admittance_controller
* Fix segfault at reconfigure of AdmittanceController (#1248 <https://github.com/ros-controls/ros2_controllers/issues/1248>)
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Lennart Nachtigall, Sai Kishor Kothakota

bicycle_steering_controller
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

diff_drive_controller
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

effort_controllers

No changes

force_torque_sensor_broadcaster
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

forward_command_controller

No changes

gripper_controllers

No changes

imu_sensor_broadcaster
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

joint_state_broadcaster
* [Joint State Broadcaster] Publish the joint_states of joints present in the URDF (#1233 <https://github.com/ros-controls/ros2_controllers/issues/1233>)
* Contributors: Sai Kishor Kothakota

joint_trajectory_controller

No changes

parallel_gripper_controller
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

pid_controller

No changes

position_controllers

No changes

range_sensor_broadcaster
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

ros2_controllers

No changes

ros2_controllers_test_nodes

No changes

rqt_joint_trajectory_controller

No changes

steering_controllers_library

No changes

tricycle_controller
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

tricycle_steering_controller
* Fixes tests to work with use_global_arguments NodeOptions parameter  (#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>)
* Contributors: Sai Kishor Kothakota

velocity_controllers

No changes

Holding for Jazzy sync


not caching resource if used across multiple components

I am using this on a component that is used many times, although the caching does not seem to be working. If I look at <head> after loading 10 components I see 10 of the same scripts there.
I see here that you are checking if the resource is in requestedUrls but you are never adding to requestedUrls. Could that be part of the problem?

update: actually I think it is not caching the resource at all. It is just that I am only rendering these components once.


Document the Entity Framework Identity model

Documenst the Entity Framework Core model used by ASP.NET Core Identity and includes guidance on customizing the model.

@Rick-Anderson @HaoK Here is the first draft of the long-awaited EF model customization doc.
(I haven't attempted to put it in the table of contents and I'm sure some of the doc tags are wrong. Hopefully @Rick-Anderson can take care of such things when merging.)

@HaoK  Internal review URL

@divega FYI

@scottaddie hopefully that won't mess up https://github.com/aspnet/Docs/pull/7301

@ajcvickers  @divega  @HaoK  give me today to do an edit pass then it's all yours.

@divega @HaoK  I'm finished with my edit commits, go away with your review.

@scottaddie  we can fix the UID when we figure out where it should go. Probably under security/authentication

@ajcvickers  can I merge then fix the TOC?

@Rick-Anderson Go ahead and do whatever it is you need to do. I'll look into the samples, but I won't get to it today.

@ajcvickers  I won't have time to integrate the samples for at least a week so no hurry.


deprecated http outlier detection

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: GregHanson
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: ayj, yutongz
Assign the PR to them by writing /assign @ayj @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS
pilot/OWNERS [GregHanson]
tests/OWNERS

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

@GregHanson: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c100494a0f25e1bfa103143599ebf6e599bcc6dd
link
/test istio-unit-tests

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

@GregHanson: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
5f4a45a84d7ec8451f3e69d3e779ac021a6f6f56
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

@GregHanson: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
5f4a45a84d7ec8451f3e69d3e779ac021a6f6f56
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
5f4a45a84d7ec8451f3e69d3e779ac021a6f6f56
link
/test istio-pilot-e2e

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

@GregHanson: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
5f4a45a84d7ec8451f3e69d3e779ac021a6f6f56
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
5f4a45a84d7ec8451f3e69d3e779ac021a6f6f56
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
5f4a45a84d7ec8451f3e69d3e779ac021a6f6f56
link
/test e2e-bookInfo

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


FGA doesn't scroll and take 2 drop screenshots

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
NA
FGA build number
1583
Describe the bug
FGA doesn't scroll and take 2 drop screenshots when there are more than 20 drops or the scroll bar is scrollable
Video
This run doesn't have more than 20 drops but it should be representative. Let me know if you need a video with 20+ drops.
https://youtu.be/Efz27oCThBE
Device model
oneplus 5t
Android version
10
Screen size
2160x1080
RAM
8GB

Looks like the whole drops area has moved up with the widescreen update.
The scrollbar area we check to see if needed to scroll needs to be adjusted for wide-screens.


Tracking: CVE-2020-8558

Tracking for CVE-2020-8558: Node setting allows for neighboring hosts to bypass localhost boundary

kubernetes/kubernetes#92315

Note: This is a backport request to gravity LTS releases where upstream kubernetes did not receive this particular patch on the particular version.


Option to set log level

Is your feature request related to a problem? Please describe.
Currently spelling issues are always console warnings.
Describe the solution you'd like
I'd like the ability to set them to errors, in order to use mkdocs-spellcheck within GitHub Actions workflows.
I basically want mkdocs build to fail on spelling errors, but print out all the spelling errors to console before failing.
Describe alternatives you've considered
N/A
Additional context
N/A

I believe that's exactly what you get with MkDocs' strict mode: mkdocs build -s.

Yep I must've missed that. Thanks!


TST: stats: fixed misnamed tests

Reference issue

part of https://github.com/scipy/scipy/issues/17652
What does this implement/fix?

Fixes incorrectly named tests which were previous not ran
Additional information

There is also https://github.com/scipy/scipy/blob/509beb488ae5eabe6f0806af8ded688ac33e31c1/scipy/stats/tests/test_continuous_basic.py#L638-L644 which is not currently called as of #15250. Can it just be removed?

Re: https://github.com/scipy/scipy/pull/17770#issuecomment-1379533068
No, it looks like there was a misunderstanding in gh-15250. I wrote that the distributions could be removed from fail_loc_scale, which meant that:
        if loc_scale_ok:
            check_loc_scale(distfn, arg, m, v, distname)

could become:
        check_loc_scale(distfn, arg, m, v, distname)

so that it could be run for all distributions. Instead, it was removed entirely, and I didn't notice the mistake. Please add it back.

Currently one failure
_________________________________________________________________________________ test_moments[truncpareto-arg102-True-True-False] __________________________________________________________________________________
[gw4] linux -- Python 3.10.8 /home/jakeb/miniconda3/envs/scipy-dev/bin/python
scipy/stats/tests/test_continuous_basic.py:324: in test_moments
    check_loc_scale(distfn, arg, m, v, distname)
        arg        = (1.8, 5.3)
        distfn     = <scipy.stats._continuous_distns.truncpareto_gen object at 0x7fec57902920>
        distname   = 'truncpareto'
        higher_ok  = True
        is_xfailing = False
        k          = 3.221085799636361
        m          = 1.744069443799767
        normalization_ok = True
        s          = 1.833550308871743
        sup        = <numpy.testing._private.utils.suppress_warnings object at 0x7fec53cf86d0>
        v          = 0.7076717914498358
scipy/stats/tests/test_continuous_basic.py:643: in check_loc_scale
    mt, vt = distfn.stats(loc=loc, scale=scale, *arg)
        arg        = (1.8, 5.3)
        distfn     = <scipy.stats._continuous_distns.truncpareto_gen object at 0x7fec57902920>
        loc        = array([10., 20.])
        m          = 1.744069443799767
        msg        = 'truncpareto'
        scale      = array([10., 20.])
        v          = 0.7076717914498358
scipy/stats/_distn_infrastructure.py:1109: in stats
    mu = self._munp(1, *goodargs)
        args       = (array(1.8), array(5.3))
        cond       = array([ True,  True])
        default    = array([nan, nan])
        g1         = None
        g2         = None
        goodargs   = [array([1.8, 1.8]), array([5.3, 5.3])]
        kwds       = {'loc': array([10., 20.]), 'scale': array([10., 20.])}
        loc        = array([10., 20.])
        moments    = 'mv'
        mu         = None
        mu2        = None
        output     = []
        scale      = array([10., 20.])
        self       = <scipy.stats._continuous_distns.truncpareto_gen object at 0x7fec57902920>
scipy/stats/_continuous_distns.py:8902: in _munp
    if n == b:
E   ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
        b          = array([1.8, 1.8])
        c          = array([5.3, 5.3])
        n          = 1
        self       = <scipy.stats._continuous_distns.truncpareto_gen object at 0x7fec57902920>
============================================================================================== short test summary info ==============================================================================================
FAILED scipy/stats/tests/test_continuous_basic.py::test_moments[truncpareto-arg102-True-True-False] - ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

CI is now green so this is ready for review

Thanks @j-bowhay! Good to have found that bug.


Azure Offline Sync Updated At pulls whole data during the Incremental Sync

I have a table controller, which has one way sync (Only get since there is no any update from mobile). So the very first time when I do the Pull Async it has pulled whole data since my Update At filed has the same timestamp for all the records. Now Second time I just wanted to do the incremental Pull (Note I do pass a name value to Pull Aync to enable incremental sync).
But the problem is, it retrieves whole data again since there is no updated time stamp in Updated At (Because there is no update method in the Table Controller since it is one-way sync-Azure to mobile). How can I omit pulling data again if there is no update while all the Update At values are same time stamp? The issue is same as mentioned in below link. Do we still need to do one record update in order to fix this?
https://social.msdn.microsoft.com/Forums/office/en-US/ff002e85-1313-449f-89eb-cd45c1a4846c/mobile-services-sync-pullasync-and-updatedat?forum=azuremobile
Thanks in Advance

Hi @thabotharshan
Thanks for getting in touch!
As mentioned in the link, this is by design.
Incremental sync feature store latest updatedAt value and then add the filter to select only entities with updatedAt value greater or equal than you already have on the device.
So, to make it correctly you can:

Avoid using the same updatedAt values as mentioned in the doc:

To use incremental sync, your server must return meaningful updatedAt values

Add some custom filter to query after the initial sync. Of course, if you can identify the hallmark of initial entries.

Best,
Ivan

Did anyone manage to resolve the issue?


use AssertJ for more explicit assertions

LGTM, just need to rebase to the latest changes on the branch.


Combined upload

Based on designs shared in #1128, this code introduces a new UploadActivity that aims to replace the old Share / Multi-Share activities.  As mentioned in #1091, this is a work in progress but should help get things started.

Codecov Report

Merging #1503 into upload-overhaul will decrease coverage by 0.18%.
The diff coverage is 0%.

@@                Coverage Diff                 @@
##           upload-overhaul   #1503      +/-   ##
==================================================
- Coverage             3.14%   2.95%   -0.19%
==================================================
  Files                  136     147      +11
  Lines                 7381    7840     +459
  Branches               714     754      +40
==================================================
  Hits                   232     232
- Misses                7134    7593     +459
  Partials                15      15

Impacted Files
Coverage Δ

.../fr/free/nrw/commons/di/ActivityBuilderModule.java
0% <ø> (ø)
:arrow_up:

...n/java/fr/free/nrw/commons/theme/BaseActivity.java
56.25% <ø> (ø)
:arrow_up:

...nrw/commons/category/CategoriesAdapterFactory.java
0% <0%> (ø)
:arrow_up:

.../free/nrw/commons/di/CommonsApplicationModule.java
32.43% <0%> (-22.12%)
:arrow_down:

.../nrw/commons/category/CategoryRendererAdapter.java
0% <0%> (ø)

...e/nrw/commons/category/CategorizationFragment.java
0% <0%> (ø)
:arrow_up:

.../fr/free/nrw/commons/category/CategoriesModel.java
0% <0%> (ø)

...e/nrw/commons/upload/UploadCategoriesRenderer.java
0% <0%> (ø)

...ava/fr/free/nrw/commons/upload/UploadActivity.java
0% <0%> (ø)

...fr/free/nrw/commons/utils/AbstractTextWatcher.java
0% <0%> (ø)

... and 20 more

Continue to review full report at Codecov.

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

@psh Thanks for submitting this! :) I loved the mockups on #1128 and the code structure in UploadActivity looks far better than its predecessor's IMO.
Just to clarify, do the basic functions of UploadActivity work for you currently? I understand from what you mentioned at #1091 that all the additional stuff (e.g. quality checks, GPS extraction etc) has not yet been implemented. However, I haven't managed to get any of the basic functionality to work personally. I can see that the gallery/camera icons in Contributions haven't been hooked up, but even using Gallery > Share > Commons app (which looks like it SHOULD work) crashes me every time (error log below). I'm not very familiar with Jacoco and not very sure why a code coverage library keeps throwing fatal exceptions, or if the exceptions are even related to this PR (this is not our only jacoco-related crash, see #1483 ).
Am I missing something, or is the basic functionality not intended to be tested yet?
05-10 03:10:47.207 17259-17259/fr.free.nrw.commons.debug E/AndroidRuntime: FATAL EXCEPTION: main
    Process: fr.free.nrw.commons.debug, PID: 17259
    java.lang.IllegalAccessError: Final field 'fr.free.nrw.commons.upload.UploadView.$jacocoData' cannot be written to by method 'boolean[] fr.free.nrw.commons.upload.UploadView$$CC.$jacocoInit$$STATIC$$()' (declaration of 'fr.free.nrw.commons.upload.UploadView$$CC' appears in /data/app/fr.free.nrw.commons.debug-2/base.apk:classes2.dex)
        at fr.free.nrw.commons.upload.UploadView$$CC.$jacocoInit$$STATIC$$(Unknown Source)
        at fr.free.nrw.commons.upload.UploadView.<clinit>(UploadView.java)
        at fr.free.nrw.commons.upload.UploadPresenter.<init>(UploadPresenter.java:26)
        at fr.free.nrw.commons.upload.UploadPresenter_Factory.get(UploadPresenter_Factory.java:21)
        at fr.free.nrw.commons.upload.UploadPresenter_Factory.get(UploadPresenter_Factory.java:7)
        at dagger.internal.DoubleCheck.get(DoubleCheck.java:47)
        at fr.free.nrw.commons.di.DaggerCommonsApplicationComponent$UploadActivitySubcomponentImpl.injectUploadActivity(DaggerCommonsApplicationComponent.java:1513)
        at fr.free.nrw.commons.di.DaggerCommonsApplicationComponent$UploadActivitySubcomponentImpl.inject(DaggerCommonsApplicationComponent.java:1493)
        at fr.free.nrw.commons.di.DaggerCommonsApplicationComponent$UploadActivitySubcomponentImpl.inject(DaggerCommonsApplicationComponent.java:1474)
        at dagger.android.DispatchingAndroidInjector.maybeInject(DispatchingAndroidInjector.java:81)
        at dagger.android.DispatchingAndroidInjector.inject(DispatchingAndroidInjector.java:102)
        at fr.free.nrw.commons.di.CommonsDaggerAppCompatActivity.inject(CommonsDaggerAppCompatActivity.java:40)
        at fr.free.nrw.commons.di.CommonsDaggerAppCompatActivity.onCreate(CommonsDaggerAppCompatActivity.java:22)
        at fr.free.nrw.commons.theme.BaseActivity.onCreate(BaseActivity.java:23)
        at fr.free.nrw.commons.auth.AuthenticatedActivity.onCreate(AuthenticatedActivity.java:32)
        at fr.free.nrw.commons.upload.UploadActivity.onCreate(UploadActivity.java:106)
        at android.app.Activity.performCreate(Activity.java:6912)
        at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1126)
        at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2877)
        at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2985)
        at android.app.ActivityThread.-wrap14(ActivityThread.java)
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1635)
        at android.os.Handler.dispatchMessage(Handler.java:102)
        at android.os.Looper.loop(Looper.java:154)
        at android.app.ActivityThread.main(ActivityThread.java:6692)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1468)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1358)

Two things that could have contributed -

I was only testing by sending pictures from the photos app on the phone rather than buttons within the app - specifically driven by wanting to test the incoming intents in the manifest.
I rebased and rebuilt the code against master, but didnt run it before submitting.

I have been playing with the newest versions of Android Studio - they might have updated the gradle version / gradle wrapper properties - the .gradle and gradle directories might be suspect and could be flushed + copied from the master version.

Thanks for the PR @psh
Although we appreciate the efforts put into making these changes, I am not in favor of merging it as:

the mentioned PR is still a WIP. It's sometimes difficult to build upon code that has changed 1000s of lines and might break/undo a few functionalities. Analysing and fixing all those issues would take a lot of time as our testing process is still manual.
the PR in itself tries to address two issues: refactoring, adding combined uploads. A PR should ideally address just one issue unless the changes can't be segregated
It would add a substantial amount of workload on the team if the combined upload is picked up along with the refactoring at this moment. Originally the plan was to pick up just the refactoring for now.

If it's not useful right now, and conflates multiple issues, that's fine.


Say "no tests" instead of "ignored" when file is run with no tests

Currently, if a file has no tests, it says Testing foo ... ignored, but it should instead say Testing README.md ... no tests. Or maybe it should fail? This is similar to #105 in that there's a general question around what should happen to "invalid" tests.

Allowing files without tests is very intentional.  For example, when running trycmd across clap examples, the README doesn't have any code blocks which is fine.  Unlike #105 where there were workarounds, the only workaround here is to explicitly skip these files which then has to be kept up to date without seeming benefit.
At the moment, I lean towards the more consistent output of "ignored" over changing it to "no tests".  Besides my preference, we'd have to consider the value worth making the change and ensuring all the right information is in the right places.

For example, when running trycmd across clap examples, the README doesn't have any code blocks which is fine.

That makes sense to me.

I lean towards the more consistent output of "ignored" over changing it to "no tests".

This is good if you're expecting the file to be ignored, but if you don't know why it's being ignored that's not super helpful. Maybe the output should actually look more like cargo? Here's what they do:
     Running tests/api.rs (target/debug/deps/api-208fdccdd2171b92)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 3 filtered out; finished in 0.00s

     Running tests/generator.rs (target/debug/deps/generator-a9e45428f33384fb)

running 1 test
test foo ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 284 filtered out; finished in 0.18s

Specifically, they say how many tests are running. So instead of ignored, you could say 0 tests run. And when the command doesn't exist that can go into the ignored tests buckets.

If we were providing a full test harness, I'd be more open to that direction.  As we are a test function within an existing harness, we'll really muddle the output if we mix things like that.
If/when we switch to being a test harness*, I also would prefer to not summarize results per file as that would get pretty noisy.
* What we really need is not for every lib making its own test harness but a proper implementation of pytest for Rust which would allow us to plug everything into the test harness.

Hmmm, yeah those are good points.

At the moment, I lean towards the more consistent output of "ignored" over changing it to "no tests".

Could you then share an example of why the output being consistent is important? If that's really the preference then this issue is kind of moot and may as well be closed. My main argument in favor of not keeping things consistent is so you can specify why the test was ignored.
PS: a pluggable test harness would be amazing! That's a very cool future direction I wasn't aware of.

I have not fully articulated my thoughts on it and I do not have time to articulate my thoughts on it.  Of any of the potential usability issues, this is one of the lowest on my list across my projects.

:shrug: Sounds good. If this issue sits around long enough maybe the test harness stuff will become a thing and this can be closed through that.

I'm closing this within the scope of trycmd as it is today and instead say this should be handled as part of my custom test harness project (TBD).


Fix flipped sense in TerminalDispatch::CursorPosition

Fixes #4107.

(thanks @j4james)
I chose to flip the sense here instead of rewriting the function. Options include...
bool success = SUCCEEDED(x) && SUCCEEDED(y);

if (success)
{
    success = SUCCEEDED(a) && ...
}

if (success)

and
THROW_IF_FAILED();
THROW_IF_FAILED();
THROW_IF_FAILED();

(this works because we have a CATCH_RETURN_FALSE)

I was going to say you actually shouldn't even need those SizeTToShort checks, because a VT parameter can never be greater than a short (it should be clamped by the state machine), but I see now that PR #3956 changed that behaviour. So instead of being pointless, these checks are now also wrong.
If a parameter is greater than a short, it should be clamped rather failing, i.e. you should be able to use a sequence like \e[1;999999H to move to the top right corner of the screen. That used to work, but doesn't anymore.
And note that CUP won't be the only operation broken by that change (some may even crash the conhost). Either they all need to add their own parameter clamping code, or we should add the clamping back in the state machine (which would be my preference). That said, it's probably best to get this particular fix merged ASAP and worry about the clamping in a followup issue.

Ah dammit I was about to submit a PR myself after debugging this issue today. 😣
(In fact PowerShell was broken entirely due to this change. It returns cursor position sequences on each input!)
I agree with @j4james that values should be capped because such a lenient behavior is probably what you'd want anyways. From a cursory glance SetCursorPosition seems to be robust against negative values too. For instance:
bool TerminalDispatch::CursorPosition(const size_t line,
                                      const size_t column) noexcept
{
    const auto clamp = [](const size_t val) noexcept -> short {
        constexpr size_t min = 1;
        constexpr size_t max = std::numeric_limits<short>::max();
        return static_cast<short>(std::clamp(val, min, max) - min);
    };

    return _terminalApi.SetCursorPosition(clamp(line), clamp(column));
}

(It's certainly not beautiful code, so please feel free to use it. 😄 A wrapper class with saturating arithmetic would surely be helpful here in the future.)

@lhecker Note that the CursorPosition method is just one example of the clamping problem. We're going to have the same issue in many other operations.

A wrapper class with saturating arithmetic would surely be helpful here in the future.)

Yeah, that's being discussed in #4013. In the sort term, though, adding the clamping back in the state machine would at least solve a lot of the problems.

@j4james @lhecker those are both good suggestions, though I think I'd probably want to get this fix merged in ASAP, since the Terminal's pretty horribly broken without it. I'd probably just file a follow up with those suggestions personally, though I'd be curious to hear @DHowett-MSFT's thoughts on the matter.

We'll talk about prioritizing #4013 for this week since it's becoming a problem and will support other work.

We'll likely end up doing 4013 this week.

This is an unblocker.


An illegal reflective access operation has occurred

I‘m getting the following error:
$ mvn --version
Apache Maven 3.3.9
Maven home: /usr/share/maven
Java version: 11.0.5, vendor: Ubuntu
Java home: /usr/lib/jvm/java-11-openjdk-amd64
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "2.6.32-042stab140.1", arch: "amd64", family: "unix"

$ java --version
openjdk 11.0.5 2019-10-15
OpenJDK Runtime Environment (build 11.0.5+10-post-Ubuntu-2ubuntu116.04)
OpenJDK 64-Bit Server VM (build 11.0.5+10-post-Ubuntu-2ubuntu116.04, mixed mode, sharing)

~/slp-indexer/slp-indexer-service$ mvn clean install
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.inject.internal.cglib.core.$ReflectUtils$1 (file:/usr/share/maven/lib/guice.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of com.google.inject.internal.cglib.core.$ReflectUtils$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
[INFO] Scanning for projects...
Downloading: https://repo.maven.apache.org/maven2/org/springframework/boot/spring-boot-starter-parent/2.1.2.RELEASE/spring-boot-starter-parent-2.1.2.RELEASE.pom
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for com.bitcoin.indexer:slp-indexer-service:0.0.2: Could not transfer artifact org.springframework.boot:spring-boot-starter-parent:pom:2.1.2.RELEASE from/to central (https://repo.maven.apache.org/maven2): Unexpected error: java.security.InvalidAlgorithmParameterException: the trustAnchors parameter must be non-empty and 'parent.relativePath' points at no local POM @ line 14, column 10
 @
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project com.bitcoin.indexer:slp-indexer-service:0.0.2 (/home/becash/slp-indexer/slp-indexer-service/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for com.bitcoin.indexer:slp-indexer-service:0.0.2: Could not transfer artifact org.springframework.boot:spring-boot-starter-parent:pom:2.1.2.RELEASE from/to central (https://repo.maven.apache.org/maven2): Unexpected error: java.security.InvalidAlgorithmParameterException: the trustAnchors parameter must be non-empty and 'parent.relativePath' points at no local POM @ line 14, column 10 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException

I think this has something to do with my server configuration. I switched providers and now it compiles fine. Will close.


Why is parse JSON transaction obsolate?

In Wasabi we're implementing transaction building and broadcasting. I thought it'd be fun to let the user to mess with the JSON and broadcast that instead.
I don't feel strongly about this feature, but I wanted to ask why JSON tx parse is obsolate?

Ops, that's a PSBT json in the screenshots, nevermind you get the idea.

Because I did not see any reason to use this. But this seems a good reason.
I think though it is way better that you make a JSON parser for PSBT, as it is likely people will want to mess around with it.

Thanks! I'll think about it. We have a feature freeze right now, because Wasabi is not very user friendly anymore, but in the future it'd would be fun to add an IDE for this and we can rename it to Wasabi Studio :)


Update ROS 2 package maintainers.

Precisely what the title says, for ros2 support.

At some point #164 has to be addressed.


How to test Sticky using enzyme / chai / mocha combo?

Hi
I only had a very brief experience with Sticky and the only way of testing what Sticky renders is to do something like this:
// component
(...)
<Sticky>
{ () => { return (
    <div><Component1 some={prop} /><Component2 onClick={() => doSomething()} />
); } }
</Sticky>
(...)

-------------

// test

//not working
const component1 = parent.find(Component1);
//or
const component1 = parent.find(Sticky).find(Component1);

// working, but ugly
const sticky = parent.find(Sticky);
const stickysChildren = sticky.props().children().props.children;
const component1 = stickysChildren[0];
const component2 = stickysChildren[1];

Is there an elegant solution to testing Sticky? Updating parent component (via update() does not help).

@smolak Yeah I'm having the same issue...
I wonder if this would be solved if the Sticky component was designed to accept a React Component or a function?  Technically it's not a Higher Order Component (at least with respect to React) unless you can put a React component into it as a child, no?

Are you using functional components by any chance? This version relies on using a components ref, and functional components do not use those.
If not, sounds like a legit bug

I was able to get around this by just mocking Sticky using jest like so:
jest.mock('react-sticky', () => ({
    Sticky: (props) => props.children({ isSticky: false, style: {}, ...props }),
}));

Hm @keith-gould-zocdoc so does this seem like a bug with how Sticky renders in Enzyme? I'm not super familiar with the quirks of rendering with Enzyme, but it does seem like it should call the child function.


Mac build failed

Hello, I'm having trouble with installing recdec. I have been following the prerequisites and looked at #291, but none of it work. Here is my error output:
[ 71%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/loader/segment.cpp.o
CMake Error at /Users/iinusen/Documents/retdec/external/src/yaracpp-project-stamp/yaracpp-project-build-Release.cmake:16 (message):
Command failed: 2
'/Library/Developer/CommandLineTools/usr/bin/make'
See also
/Users/iinusen/Documents/retdec/external/src/yaracpp-project-stamp/yaracpp-project-build-*.log

make[2]: *** [external/src/yaracpp-project-stamp/yaracpp-project-build] Error 1
make[1]: *** [deps/yaracpp/CMakeFiles/yaracpp-project.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 71%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/loader/intel_hex/intel_hex_image.cpp.o
[ 71%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/loader/macho/macho_image.cpp.o
[ 71%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/loader/raw_data/raw_data_image.cpp.o
[ 72%] Linking CXX executable retdec-llvmir2hll
[ 72%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/loader/elf/elf_image.cpp.o
[ 72%] Built target retdec-llvmir2hlltool
[ 72%] Linking CXX static library libretdec-loader.a
[ 72%] Built target retdec-loader
make: *** [all] Error 2
log.txt
I also attached the full log file.

Hi. Could you please attach the following two log files?
/Users/iinusen/Documents/retdec/external/src/yaracpp-project-stamp/yaracpp-project-build-err.log
/Users/iinusen/Documents/retdec/external/src/yaracpp-project-stamp/yaracpp-project-build-out.log

They contain the reason why the yaracpp build failed.

yaracpp-project-build-out.log
yaracpp-project-build-err.log

Thank you. From yaracpp-project-build-err.log, it seems to be the same error as in #425. Could you please try this approach to see whether it fixes the problem for you?

Thank you, it worked!

Great :+1:


Logo URI Path

The path "ms-appx:///Assets/MicrosoftEdgeSquare150x150.png" is not working on Surface Hub S2. It causes the tile to not initialize and event log ID 17 in Microsoft-Windows-ShellCommon-StartLayoutPopulation%4Operational with parameter is incorrect.

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

ID: 69882e91-6bef-d524-7059-20ba67129f4e
Version Independent ID: ee0f2eb3-97da-4306-5cd1-2b6ae1858ed9
Content: Configure Surface Hub Start menu
Content Source: devices/surface-hub/surface-hub-start-menu.md
Product: surface-hub
Technology: windows
GitHub Login: @robmazz
Microsoft Alias: robmazz

Hmm, if you intended to close this issue, what is the reason for closing it (just out of curiosity)?

I screwed up… I was using ms-appdata:/// instead of ms-appx:///

On Sep 24, 2019, at 4:45 PM, Trond B. Krokli<EMAIL_ADDRESS>wrote:
Hmm, if you intended to close this issue, what is the reason for closing it (just out of curiosity)?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/windows-itpro-docs/issues/5027?email_source=notifications&email_token=AHJKQB2HBOFF4UDKU2CXGB3QLJ365A5CNFSM4I2DVXYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7PX4BI#issuecomment-534740485, or mute the thread https://github.com/notifications/unsubscribe-auth/AHJKQBYF2E4U7AEUXCE2T3LQLJ365ANCNFSM4I2DVXYA.


Fix bug when user can update any displayed field

At this pull request I've added optional protection from updating displayed fields [which $form->display()] by optional ignoring that kind of fields. Now it's made with backwards compatibility.
But I think that better be safe, than sorry and make this behavior by default.
How to use:
$form->ignoreDispayFields(bool);
If security for you is in prior, than you can use it with Form::init() method @ app/Admin/bootstrap.php (https://laravel-admin.org/docs/#/en/model-form-init?id=form-initialization-settings): just add
$form->ignoreDispayFields(); there.
I'm using unset() without isset() because of this https://stackoverflow.com/a/35822050/4941870 and that It's easier to ask forgiveness than it is to get permission.
P.S.: sure, you can just add all display fields to ignore([]) but I found it very uncomfortable.

Hello.
Was it fixed?


207 - Integration get deaccessioned dataset

What this PR does / why we need it:
This PR adds the integration with js-dataverse module to get the metadata information of deaccessioned datasets in order to display it in the Dataset View Mode.
Which issue(s) this PR closes:

Closes #207

Special notes for your reviewer:
While I was doing the integration I realised that in order to display the Dataset page for a deaccessioned data we also need to integrate the get dataset citation of deaccessioned datasets. I opened an issue to address this change, so in this PR you won't be able to open the Dataset page for a deaccessioned dataset.

#223

Suggestions on how to test this:
Step 1: Run the development environment

Run npm I
cd packages/design-system && npm run build
cd ../../
Check that you have a  .env file such as the .env.example, with the VITE_DATAVERSE_BACKEND_URL=http://localhost:8000 variable
cd dev-env
./run-env.sh unstable
To check the environment go to http://localhost:8000 and check your local dataverse installation

Step 2: Test Dataset View mode with the implemented changes for the getFilesTotalDownloadSize

Go to http://localhost:8000
Login as admin using username: dataverseAdmin and password: admin1
Create a new Dataset
From the dataset view mode copy the search parameters from the url. Ex.: ?persistentId=doi:10.5072/FK2/LHGRHP&version=DRAFT
Go to http://localhost:8000/spa/datasets and paste the search parameters. Ex.: http://localhost:8000/spa/datasets?persistentId=doi:10.5072/FK2/LHGRHP&version=DRAFT
You are seeing now the same Dataset in the SPA, it should work as usually. As I described above this PR cannot display a deaccessioned dataset on it's own, in order to do that we need to finish this other issue #223

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
No
Additional documentation:

coverage: 98.312%. remained the same
when pulling ef941b8c34ecaa12c6d7c969db8774dbc7a45a69 on feature/207-integration-get-deaccessioned-dataset
into 563f95d52464952d0c19ce054839700282d0f681 on develop.


Can this code be used for gray images?

I have a depth map. Can I use this code?

You should be able to, if you have issues with the number of channels you could duplicate them along the 3rd-axis.


USD values

When downloading transaction data as CSV from etherscan it includes "TxnFee(USD)" and "Historical $Price/Eth". Is there any way to get these values from the APIs?

Closing this as it's not an issue. I would highly recommend going through the documentation of the API as this package offers whatever the official Etherscan.io API offers in an 1:1 fashion.


VinVL - NoCaps pretrained model, empty folder

Hi, thanks for releasing OSCAR+.
I try to download pretrained OSCAR+ model for the task of NoCaps as instructed in VinVL_DOWNLOAD.md, using
path/to/azcopy copy 'https://biglmdiag.blob.core.windows.net/vinvl/model_ckpts/nocaps' <target folder> --resursive
However, the download cannot be done with INFO
INFO: Any empty folders will not be processed, because source and/or destination  doesn't have full folder support
Is this folder empty? If it's a valid folder, is the pretrained model in it VIVO?
Thanks

I got all the models by running
path/to/azcopy copy https://biglmdiag.blob.core.windows.net/vinvl/model_ckpts/* <target folder> --resursive
nocaps models are located in model_ckpts/image_captioning/
but it is surely not a good way to download things

I got all the models by running:
path/to/azcopy copy https://biglmdiag.blob.core.windows.net/vinvl/model_ckpts/* <target folder> --resursive
nocaps models are located in model_ckpts/image_captioning/
but it is surely not a good way to download things

Thanks~ I guess the nocaps models in model_ckpts/image_captioning are supposed to what are mentioned in VinVL_MODEL_ZOO, namely nocaps_base_xe.zip and nocaps_base_scst.zip?    It isn't the VIVO model, is it?

It's probably VinVL, not VIVO

hi~ have you success to inference nocaps? how to do it ?


Add MXNet Ops for fast multihead attention

Description
Add new optimized Ops for Multihead attention
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
[X] Code is well-documented:
For new C++ functions in header files, their functionalities and arguments are documented.
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

Add 4 Ops: (matmul(K,Q) and matmul(attention_weights, V). For both self-attention and encoder-decoder
Add unit test for those Ops

Comments

https://github.com/Caenorst/gluon-nlp/tree/fast_mha shows a example of integration in BERT, it will not be PRed to gluon-nlp as it's breaking it (out of BERT usage).
Those Ops require to have a different layout (sequence, batch, encoding) except for the masked softmax / dropout
Those Ops change the ordering of projection weights, which means a pretrained BERT without those Ops need to have the weights processed as in: https://github.com/Caenorst/incubator-mxnet/commit/e98761456ba0343664ba550e056f00db31516ac7#diff-4758fb9329d438de2836db2634a8f5f7R2505-R2519 in order to use those Ops.
The argument bwd_ignore_zero_init allow to further speedup and reduce the memory consumption but is only giving good results with MXNET_EXEC_ENABLE_ADDTO set to 1, it's also a dirty trick as it's actually not "adding to" but initializing (it rely on the fact that the two ops inputs are actually complete (not overlapping and using the whole tensor), despite using the same tensor.

I had to remove some tests on my Ops, the behaviour for bwd_ignore_zero_init cannot be tested as we can't enable the kAddTo yet.

@aaronmarkham is the website preview functionality still working after the website upgrade? I cannot see the preview of this PR: https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-16408/12/index.html

Actually I personally don't like this kind of super fused operator which doesn't have explicit semantics and also lacks the flexibility as a general operator. I'm not sure if we should encourage our users to call these strange and half-baked operators in their models and how to do that if they already implemented their models? Besides, it also increase the burden for the community to maintain the complicated logic in the code.
I totally agree that fusions can bring performance improvement and make our framework outperforms other competing products. But I still think it would be better if fusions can be done in following ways:

Graph fusion pass: we have subgraph in NNVM and more features in the coming Relay IR;
Accelerating library support: integrate APIs and primitives from library and make them an operator is more acceptable. Just like what we did for fused RNN layers?
Maintain your customized fused operators in a separated repository and teach users how to integrate them into MXNet if they really want. I think the dynamic custom operator feature may help in this situation?
Just my two cents.

@TaoLv I agree that the dynamic custom operators feature would be a great fit here, but since it is not yet ready contrib is the next best place for such operator, also I did not modified any core component of MXNet or any existing model. Of course having an automatic fusion is better, but you first need the Op before having the fusion.
@eric-haibin-lin We intend to provide as a follow up a wrapper op that could be used for this and cudnn multihead attention that would unify this, but we don't want to wait for the cudnn multihead attention to cover all the usecases we needed right now.

I can no longer find MXNET_EXEC_ENABLE_ADDTO in mxnet. Is this flag required?

@eric-haibin-lin Only with bwd_ignore_zero_init=True, otherwise it works find regardless. I let this flag hoping that MXNET_EXEC_ENABLE_ADDTO get back soon

@marcoabreu, @samskalicky Are your comments resolved?

I am still not comfortable with the bwd_ignore_zero_init  flag. The MXNET_ENABLE_EXEC_ADDTO  env var does not exist nor take any effect in the current version of mxnet. I'd suggest we remove this flag and related code for this PR. After MXNET_ENABLE_EXEC_ADDTO  is properly introduced by other PRs, we can add relevant optimization around it.

I do see there are some useful discussions about the design and flags used in this PR. Also seems you already have considerations about future improvement with MHA from cuDNN, dynamic customized operator or with subgraph and fusion pass. So I would hope you can prepare a design proposal to cleanly describe the design, the limitations, user suggestions, and your further plans. Please also cross post the proposal to dev@ so it can get fully discussed and I also think the community can learn much from that. If possible, some guys can help to fill the missing part on CPU side.
BTW, what does that mean to MXNet Gluon if the CPU implementation is missing for an operator? @eric-haibin-lin

I agree a design proposal is helpful, but it is not required for a contrib op. The purpose for contrib ops is that ppl who are interested in this op can try and use the op in gluonnlp Block as @Caenorst suggested, and developers can gather timely feedbacks.
The work on official multi-head attention block API / operator API, would definitely require a proposal and discussion on dev@ since it involves developers from different hardwares including CPU and GPU.
As far as gluon-nlp is concerned, we will not integrate this op (without CPU support) to the official API for now. However, we will redirect users who are interested to the example usage provided by @Caenorst if they only care about using GPUs.

Following @eric-haibin-lin comment I removed the flag bwd_ignore_zero_init and will put it back once AddTo functionality is supported

Please @TaoLv as @eric-haibin-lin indicated there is no impact on Gluon, I removed the flag, so there is no reason to block this PR.

Thank you @eric-haibin-lin for clarifying the impact on MXNet Gluon. My change request is not only for that but also for an elaborative proposal about these new operators. Although they will live under contrib op category, I didn't see a clear scope definition about contrib op in MXNet.
To make it clear, I hope below items can be described in the proposal:

What's the current design? APIs, features, formulas, limitations, performance, ...
Why cuDNN MHT, dynamic operator, or subgraph doesn't help here?
Do we need call for contribution for CPU implementation?
How long will these operators exist? If I understand correctly, once they're in, less likely they will be removed before next major release. So contrib op is not a trivial thing.
Will they be replaced with other implementations? with the same operator interfaces? If not, how about backward compatibility?
Do we encourage users and downstream projects to use them?
Known issues or further plan to improve?
I hope the proposal and more discussions can help other developers as well as users to understand these operators better and also make MXNet development more open.
I don't want to be a hostile here. So it's not necessary for you to finish the proposal before merging this PR.

@aaronmarkham is the website preview functionality still working after the website upgrade? I cannot see the preview of this PR: https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-16408/11/index.html

No, a variety of features of the new site didn't work on S3.
To preview, you need to follow the directions on the wiki, or use the devmenu features I added in this PR: https://github.com/apache/incubator-mxnet/pull/16514

@TaoLV:

The current design are two separated Ops which represent each matrix multiplication part of the multihead attention on both case of self-attention and encoder-decoder the performance depend on the model implementation / hardware. As an example on our internal contains we have about 15% improvement on BERT training. The Ops force to modify the layout to (sequence_length, batch), and interleave the weights for input linear projection.

cuDNN MHA Ops are not as fast as those Ops yet. Subgraph API can be doable to automatically fuse to those ops, but the implementation is not obvious as the two Ops are inter-dependent. Those ops require to modify the layout which we can't do on the whole model without user involvement yet.

I don't think a CPU implementation would be important for now. Those ops have been designed to have specific performance improvements on GPUs, as there are no plans to have them on Gluon-NLP those Ops are here to give the option to the users to have a non-negligible speedup on GPU if they need to. We do provide an example of implementation on https://github.com/Caenorst/gluon-nlp/tree/fast_mha, but don't intend to present it as a "standard" implementation of multihead attention.
We do plan to send a design proposal to dev@ for the CuDNN fused multihead attention integration we're prototyping. For that efforts, we are looking for contributions from the CPU side to make the feature better.

We don't know, it depend on future performance / ergonomic improvement and users feedbacks.

We don't know, it depend on future performance / ergonomic improvement and users feedbacks.

NVIDIA would encourage and help users that are looking for a very efficient implementation of multihead attention on GPU. We wouldn't encourage the usage for somebody who's looking for "A BERT implementation" for both CPU and GPU.

Unfortunately because of lack of kAddTo full functionality there is a cudaMemset that could be avoided, we are looking forward completing this functionality. We have several options for improvement but we want to release some tests internally before making any plans. We would like to have a single op for the whole MHA but it constraint to force the _masked_softmax + dropout inside the Op, and that would remove some of the flexbility of the current set of Ops, hence our current decision to scatter it in two ops.

Thanks for your response @Caenorst. Looking forward to your general proposal for cuDNN MHA integration. Now I'm withdrawing the change request.

Thanks Tao.  Looking foward to working with you and others on MXNet's MHA definition.

BTW, cublasGemmStridedBatchedEx used in this PR is broken in Cuda 10.1 which will cause crashes on p2 instances. Seems fixed in Cuda 10.2 (as per release notes).

So it's related to cuda and not the arch?

No. My comment was wrong


MSA dataset

Hi!
I am currently following your great work MSA Transformer, and I would like to reproduce it myself. I wonder if it is possible to download the 4.3T msa dataset you used in this paper? Thank you so much!
Best,
Yinjun

It is currently not publicly available. We are looking into releasing it.

Is it possible to use the trained model for inference on single protein sequences?


#21 fail runner when executed script fails

This change seems to fix the problem described in #21.
yarn devcmd test && echo This should not be shown if the command fails works correctly when executed in bash
(but not when executed in cmd.exe, which seems to be an issue with yarn.cmd)

TEST

manual: ✔
integration: ✔

Danke @schierla für deinen PR!


Warn about deprecated code paths

We've tagged some functions/classes as @Deprecated. This only helps warn when trying to develop new features against that code path, however. Users of Heroic get no indication if they're using a config that loads a deprecated code path, and aren't likely to migrate unless that code breaks completely. We can improve this by logging a warning message.

It might also be worth sending analytics events about deprecated feature usage, although we need to be careful that its both anonymized and not sending a ton of data.


Feature/dmps endpoint fixes

-Resolves minor issue where control proc name comparison fails due to "-" in processor name (DMPS3-300-C)
-Adds property to DMPS chassis to define if it is a 4K type DMPS
-Fixes creation for DM-RX on a DMPS is the IP ID is required, such as on a DMPS3-300-C
-Fixes creation for all DM-TX types on DMPS, both with IPID and without IPID

Resolves #824

Proposed solution for #826. There is a loop that holds the thread on InitializeSystem() until devices are registered, but it will time out after 60s just in case something hangs.


Please invite me to the Recode-Hive GitHub Community Organization

Name
Akash Kumar
Discord Username (if applicable)
akashkumar241203
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


[CI/CD]Checking the dependency licenses

Track the root issue #1460
Apache has strict requirements for license, so ci needs to verify license, you can refer to skywalking or other apache dependence license check flow.
skywalking link: https://github.com/apache/skywalking/blob/master/tools/dependencies/check-LICENSE.sh
if you want to contribute, you may reply "i want to implement it"

who want to contribute this feature？ please reply，thanks.

I'm the author of the script https://github.com/apache/skywalking/blob/master/tools/dependencies/check-LICENSE.sh, need my help to port the SkyWalking script to this project?

@kezhenxu94 yes,  thanks for your help~~


profiling tests fail without valgrind

They should be disabled if valgrind is not available. It would also be good to use the valgrind executable found in CMake (https://github.com/eProsima/Fast-RTPS/blob/a742bdbc17946e4ff712d395d3365a122df06476/CTestJenkins.cmake#L40) rather then the hard coded name.

Last minute solved for 1.2.0. Thanks for the heads up!

For the record the change is in aa47b4621791f54b7c69447f2b442c2455a9c151.


Web version

hello could you tell me the version used for apache, mysql and php thank you.

you need php 5 and not 7. Any apache and mysql


Logged error message when setting 'none' as scale color

When using a scale for displaying custom link states, a PHP error is recorded on each execution if a color is set to 'none':
(please note that some line numbers might be off by 4-8 lines since I already had my fix in place but commented it to recreate the error code)

CMDPHP PHP ERROR WARNING
Backtrace: (/poller.php[754]:api_plugin_hook(), /lib/plugins.php[74]:api_plugin_run_plugin_hook(), /lib/plugins.php[177]:weathermap_poller_bottom(), /plugins/weathermap/setup.php[787]:weathermap_run_maps(), /plugins/weathermap/lib/poller-common.php[211]:WeatherMap->DrawMap(), /plugins/weathermap/lib/Weathermap.class.php[3255]:WeatherMap->AllocateScaleColours(), /plugins/weathermap/lib/Weathermap.class.php[3163]:myimagecolorallocate(), /plugins/weathermap/lib/WeatherMap.functions.php[283]:imagecolorallocate(), CactiErrorHandler())

and

ERROR PHP WARNING in Plugin 'weathermap': imagecolorallocate(): Red component is out of range in file: /usr/share/cacti/site/plugins/weathermap/lib/WeatherMap.functions.php on line: 283

My scale definition is:
SCALE updown 0 0 none 0.png
SCALE updown 2 2 none 2.png
SCALE updown 1 1 none 1.png

As far as I could track down the error, it seems that the massive Regex in Weathermap.class.php (around line 3163) sets $red, $green and $blue to -1 if 'none' is set instead of a color.
This leads to a PHP error when executing either imagecolorallocate() or imagecolorexact() in function myimagecolorallocate() in WeatherMap.functions.php in the lines after 267.
My fix was to allocate a transparent color if -1 is set for either of R,G,B, so my fixed myimagecolorallocate() is:
// wrapper around imagecolorallocate to try and re-use palette slots where possible
function myimagecolorallocate($image, $red, $green, $blue)
{
        // it's possible that we're being called early - just return straight away, in that case
        if(!isset($image)) return(-1);

        // fix "component out of range" issue for 'none' colors, where R,G,B are set to -1
        // creates a 'none' color by allocating black with full transparency
        // maybe this issue can even better be avoided at Weathermap.class.php[3163] ??
        if($red<0 || $green<0 || $blue<0)
                return (imagecolorallocatealpha($image, 0, 0, 0, 127));

        $existing=imagecolorexact($image, $red, $green, $blue);

        if ($existing > -1)
                return $existing;

        return (imagecolorallocate($image, $red, $green, $blue));
}

I think it might be better to catch a 'none'-color already at the regex, but I did not want to break anything else, so this was the less invasive fix for me.

@lclbo, you should re-evaluate this bug once the following pull request is merged: https://github.com/Cacti/plugin_weathermap/pull/39

@lclbo , if you could do the pull request, that would be awesome.  Let me know you can do it.

Okay, this is fixed now


[Feat] Editing project to be published on npm

User-Facing Changes
N/A
Description
Editing project info on package.json to be published on NPM.

Please check my questions, but if you think it is ok as it is, it is approved.
Also, please copy and adapt the rulesets we have on our main project to here.

Changes done. Regarding the rulesets it will be done afterwards.


Week 7

What is your question?
In the Achievements PNG File, it shows tankman. IDK if that is supposed to be there, like if Psych Engine is supposed to have week 7? If not, oh well. Just I would like to know.

They probably haven`t added it because week 7 is still newgrounds exclusive.

They probably haven`t added it because week 7 is still newgrounds exclusive.

no its not
your allowed to have week 7 in your mods now

@ShadowMario, sorry for pinging you or whatever. But do you have an Idea on when week 7 version will release for psych?


AppSwitch does not support boolean values

I've got a CoreUI Pro (Clever) project I'm working on, I'm trying to use an AppSwitch component in a form that updates values from a mongoDB document that has a boolean value in the given field. This would appear to be the primary use case for a Switch, but the "value" prop only supports string values, so I get the error message both when the form is rendered and when the switch is flipped on and off:
Warning: Failed prop type: Invalid prop value of type boolean supplied to AppSwitch, expected string
There does not appear to be any documentation in the docs area about what props do what for AppSwitch.
Please advise the best way to bind a boolean form value to this component.

hi @alex-woodall-premierfoods
The value attribute specifies the value of an <input> element - this is the value that is sent and has meaning when submitting a form only. (name="value").
To flip the switch on and off, use checked property instead.

Many thanks, that worked, I'm a big fan of CoreUI, but as a customer, if I'm honest, the documentation needs some work, I appreciate that fixing bugs usually takes precedence over documentation but it does make it tricky when as a developer you can't tell what does what.

@alex-woodall-premierfoods thanks for the follow-up
we are working on the next version and improved docs


Make colors configurable

Here's how the extension looks like on GitHub with the Stylish theme:

The colors don't look nice. Assuming there's no way to use some colors from GitHub's CSS (which Stylish overrides), can we make the colors configurable at least?

Yep, great idea.

Stylish is really interfering with a lot of my UI. The Code Review coloring isn't working at all. I can fix the Jenkins failure text/background easy by using built-in github text classes, though.


Add image volume support.

This carries #241 with additional fixes to the test suite from me. From @Random-Liu's original:
For image volume, currently we only:

Create host directory and mount into the image;
Change the ownership of the host directory to be the same with mount point inside the image.

What we are going to do in the future:

Consider whether we need to copy existing content in the image into the volume. #186.
Consider whether we should add the helper into containerd containerd/containerd#1482.
Consider how to properly manage these volumes in kubelet kubernetes/kubernetes#52032.

Signed-off-by: Lantao Liu<EMAIL_ADDRESS>Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

@ijc Thanks!


YARN-5727. Support visibility semantic in MapReduce.

953

 Quality gate: Failed
41 Issues

 1 Vulnerabilities
 40 Code issues

Details

 Quality gate: Failed
39 Issues

 1 Vulnerabilities
 38 Code issues

Details


add optimized string

Pull Request for issue
Description

add str package
faster then pure string and bytes.Buffer

Testing Instructions

Additional Comments

yes , i test strings.Builder vs Str, strings.Builder is faster

yeah :)


New feature: HTML popup box

Spilt to new plugin
https://github.com/YU000jp/logseq-plugin-weekday-notice


Tweaked draft reference and added more workflow info

@stain can you take a look at this from a workflow perspective?


The visualizer doesn't hide even when I close the media player

Hello,
the visualizer does not hide even if I close itunes. I have the auto hide option enabled.
Is this my fault?

Hello!
It's probably not your fault ;)
I would assume that iTunes is somehow reporting the player state wrong and this will break the Auto-Hide option. As we currently have a lot of issues regarding iTunes as some recent changes broke something, we don't have a fix for now.
As soon as we found the cause of the issues, we will keep everyone up-to-date who is subscribed to this issue (You can subscribe by clicking on the button on the right)

The 4.2 beta fixes a lot of the remaining stale data issues. Let me know if you still have any issues after updating.

Now the autohide feature works. But sometimes the visualizer won't reappear

And with the Beta, the Visualizer starts to lag.

Visualizer lag is likely unrelated given the changes we made. I will look into the itunes open issue though, probably will get a chance tomorrow.

All these issues should be resolved in Rainmeter 4.2 Beta Release - r3004. The only minor issue is that Rainmeter can lockup for a second or so when you close iTunes but that is Apple's fault.


[Enhancement] [Suggestion] New layout style like official games

I would really like to have a new paste layout like in the original games.
Here's my take:

I don't know Go so here's my html+css test:

* {
    box-sizing: border-box;
}
body {
    display: grid;
    grid-template-columns: repeat(2, minmax(auto, 550px) );
    overflow: hidden;
    grid-gap: 15px;
    color: white;
    background-color: black;
}

.entry {
    display: grid;
    grid-template-columns: 350px 5px 300px;
    grid-template-rows: auto;
    padding: 15px;
}
.section1 {
    display:grid;
    grid-column: 1/2;
    grid-row: 1/2;
    grid-template-rows: 1em 1em 1em 1em 1em;
    grid-template-columns: auto auto;
    z-index: 1;
}

img, .img, .divider, .info {
    user-select: none;
    -moz-user-select: none;
    -webkit-user-select: none;
    -ms-user-select: none;
    z-index: 0;
}
.img {
    grid-column: 1/2;
    grid-row: 1/2;
    justify-self: right;
    display: grid;
    grid-template-columns: 1fr;
    grid-template-rows: 1fr;
}
.pokemon-img {
    width: 140px;
    height: 140px;
    opacity: .8;

    grid-row: 1/2;
    grid-column: 1/2;
    justify-content: center;
    align-items: center;
}
.item-img {
    width: 40px;
    height: 40px;
    grid-row: 1/2;
    grid-column: 1/2;
    justify-self: end;
    align-self: end;
}
.teratype {
    width: 30px;
    height: 30px;
    z-index: 1;
    opacity: 1;
    grid-row: 1/2;
    grid-column: 1/2;
    justify-self: end;
    align-self: start;
}
.divider {
    grid-column: 2/3;
    grid-row: 1/2;

    margin:1px;
    padding:1px;
    width: .1px;
    height: auto;
    background-color: gray;
    border-radius: 2px;
}

pre {
    white-space: pre-wrap;
    margin: 0px;
    padding-left: 0px;
    font-size: 15px;
}
.info {
    font-size: 10px;
    color: yellow;
}

.name {
    grid-area: 1/1;
}
.item, .clearText {
    color: white;
}
.ability {
    grid-row: 2;
    grid-column: 1/3;
}
.ttype {
    grid-row: 3;
    grid-column: 1/3;
}
.evs {
    grid-row: 4;
    grid-column: 1/3;
}
.ivs {
    grid-row: 5;
    grid-column: 1/3;
}
.nature {
    grid-row: 6;
    grid-column: 1/3;
}
.moves {
    grid-column: 3/4;
    grid-row: 1/2;
    white-space: pre-line;
    margin: 0px 10px 10px 10px;
    justify-self: start;
    align-self: start;
}

.genderM {
    color: blue;
}
.genderF {
    color: red;
}
.attr {
    color: #A0A0A0;
}
.fire{
    color: #F08030;
}
.flying {
    color: #A890F0;
}
.normal {
    color: #A8A878;
}
.ice {
    color: #98D8D8;
}
.grass {
    color: #78C850;
}
.earth {
    color: #E0C068;
}

Thanks for the suggestion!
I hope you don't mind that I won't use this directly (or even if I tweaked it a bit) , but I do really appreciate you taking the time to make a prototype - it's very good inspiration and you have several good ideas here.
P.S. you have a few syntax issues here (Atk 0 rather than 0 Atk, and I think Showdown has decided on Tera Type:, not Teratype: - maybe it accepts both, idk).

Thanks a lot for appreciating the concept and taking it into account! 😊
I don't mind any tweaks at all, feel free to adjust it however you like (if it will be implemented).
I slightly changed class names and I rely a lot on CSS-grids but page-structure remains similar to yours and text-selection works correctly.
Concerning IVs/Tera you are right, I miss-typed it.


CVXOPT has difficulty solving problems where feasible set is a single point

Can someone explain to me why this returns UNKNOWN?
Am I missing something?
from gpkit.shortcuts import *

x0 = Var('x0')
x = Var('x')
y = Var('y')
M = 100

m = Model(x0,
         [x + M <= x0,
          0.1*x**-1 <= 1,
          x + y <= 1,
          2**(-0.5)*x**(-0.25)*y**(-0.25) <= 1])

m.solve()

# Output
# RuntimeWarning: final status of solver 'mosek' was 'UNKNOWN', not 'optimal'.

I believe x = y = 0.5 should be a valid solution.

![plot]
(~/Desktop/plotissuepng)

hmm, that's indeed surprising. Here's an equivalent version of the problem that I find easier to intuit:
from gpkit.shortcuts import *

x0 = Var('x0')
x = Var('x')
y = Var('y')
M = 100

m = Model(x0,
         [x + M <= x0,
          0.1 <= x,
          x + y <= 1,
          .25 <= x*y])

m.solve()

cvxopt also says unknown.

Still fails even removing the 0.1 <= x:
from gpkit.shortcuts import *

x0 = Var('x0')
x = Var('x')
y = Var('y')
M = 100

m = Model(x0,
         [x + M <= x0,
          x + y <= 1,
          .25 <= x*y])

m.solve()

mosek solves all of these fine for me. But cvxopt fails on all of them.

Mosek solves the first one for you?

that's correct, mosek solves all three for me.

@pgkirsch, it fails for you?? do the unit tests run for you?

All 3 fail for me. Wait what?? 123 unit tests are failing and I just pulled... what on earth?

is your mosek license expired?

Oh yeah it expired yesterday

Hmph!! Mosek and cvxopt both fail for me on this even simpler MWE:
from gpkit.shortcuts import *

x = Var('x')
y = Var('y')

m = Model(x,
         [x + y <= 1,
          .25 <= x*y])

m.solve()

Taking minimalistic to the extreme, cvxopt and mosek both return unknown on this:
from gpkit.shortcuts import *

x = Var('x')

m = Model(x,
         [x + .25/x <= 1])

m.solve("cvxopt")

Is x = 0.5 the only valid solution to that?

yeah, I think the feasible set is a single point, that might be part of the problem (though I'm a little vexed that neither solver would find that point).

and cvxopt solves the original if you slack the x + y <= 1 constraint.

This is just a cvxopt issue, right?

If so, the todo would seem to be raising the issue with them...

cvxopt and mosek both return unknown on this:
from gpkit.shortcuts import *
x = Var('x')
m = Model(x, [x + .25/x <= 1])
m.solve("cvxopt")

We need to decide whether we think that's acceptable behavior. It's infinitesimally close to being infeasible; so perhaps not worth worrying about.

Before complaining to the solver-writers, we should ask ourselves what we would want the dual solution to be...

Sensitivity of >1 to the second posynomial, implying an x variable sensitivity of...1...oh dear.

I'm pretty sure that if the second constraint is formulated as x + .25/x <= s, where s is a specified constant, then the sensitivity to that posynomial approaches infinity in the limit as s approaches 1 from above. You can see this by running a simple experiment setting s to 1.01, 1.001, 1.0001, etc, and looking at that sensitivity.
I'm not surprised that the solvers have problems when s==1; I support closing this unless anyone objects.

Sounds good to me.

@pgkirsch, this was originally opened by you; could you read it over and close if you agree?

Ok sounds good. closing.

Related to #428, in the sense of confusing solver output for modelers.


Create base fetch hook

Create a useBaseFetch hook which will not be exported to consumers, but can be used by both useFetch and useLazyFetch - this will centralize our request state management to a single place (vs. copying it across both hooks).

👍🏼👍🏼

Closed with #16


Prevent click on the step

Can I prevent click on the step somehow? I don't want to allow moving to the next step (unless user let's say filled form)

Hi, I'm rewriting this component and the new version will allow the consumer much more control.

@adi518 oh nice! is there any estimates?

temporary CSS workaround here:
     /* prevent click events on all steppers */
div.v-stepper > div.v-stepper-root > div.v-step {
    pointer-events: none;
}

Sorry to disappoint, my plans were slightly hindered by real life. I don't like to leave things uncompleted and I do want to keep this little project maintained, so I will take care of it once things settle down a little.

No worries! Life and work happens.

Any update on this? Also, do you have the WIP in another branch? Maybe you could uploaded and get some help from the community.

Any update on this? Also, do you have the WIP in another branch? Maybe you could uploaded and get some help from the community.

We ended up writing our own stepper and have abandoned this one. Good luck @justinbeatz

@theGrower Did you open-source it? I'm looking to get back to this and modernize it for Vue 2/3, unless there's a better one out there.

@adi518 No, we did not open-source it. I'm sorry. I'm also no longer at that company to be able to access the code base and discuss making it open source.

That was a quick reply, thanks. I looked at awesome-vue and couldn't find new alternatives. This means it's probably worth reviving this one.

@adi518 Honestly, it wasn't that much work to build our own with the exact functionality we wanted. I think the other dev, granted she's pretty darn good, had it build in an afternoon. Most of the framework we needed was there in HTML5, ECMAScript 6, and Vue.js. A little custom CSS and, BOOM, it was up and running.
I'd say it's worth giving it a go.
If I remember correctly, it was essentially just re-styling tabs maybe? That was a long time ago.

Which developer are you talking about? btw, it is generally not wise to build a component from scratch. It's better to fork and add the functionality you want unless it deviates significantly from the original functionality. That way you can also contribute back to the original project, be it an individual or software company. This component has sophisticated logic for customizing it to a large extent, something many other components do not offer. I can also say that most of the issues that people start are related to insufficient knowledge with Vue.js and using its scoped slot feature.

Sorry, the Dev that built out our own stepper at my last employer.
I agree with you; however, to fork this project, fix what we needed, and maintain it would have been a lot more work than we had the ability to take on. By all means, I support you in this effort. Link me the project, I'd like to follow its progress.

Oh, also, I'm just realize which topic this is. I was thinking of a different topic... hell, it may have been a different stepper we tried. I'm sorry. You are the maintainer as well.
My bad. I do remember the CSS I wrote above and it worked, and then we wrote our own. I'm completely out of context in my thoughts.


async bug

https://github.com/twitter/typeahead.js/blob/master/src/typeahead/dataset.js#L273
line 272-273
rendered += suggestions.length;
that._append(query, suggestions.slice(0, that.limit - rendered));
This is flawed.  I tried the demo site: https://twitter.github.io/typeahead.js/
Type "aasdfaf" as the query and watch what goes over the wire.  4 results come in but only 1 is displayed.  The limit is 5.
The following works:
suggestions = suggestions.slice(0, that.limit - rendered);
that._append(query, suggestions);
rendered += suggestions.length;

this is an issue i also ran into

:+1:

I think we have like 5 pull requests and at least 10 issue tickets raised about this. Not likely to be fixed as the project seems close to abandoned.

:+1:  is there a fork with this fix applied?  Pretty surprising this isn't getting more visibility as it's a fairly severe bug.

@scottnewman Thanks, saved me a ton of time.

Maybe you could rename this ticket so more people who are having this problem can find the solution. :)


VLANs missing source tag

VLAN imports from vCenter are only tagged with "NetBox-synced", but not with a source tag like other objects. Why?

Goood question.

Hi @Azmodeszer,
I fixed this in the latest development  branch. Could you please check it out and test if it's working now?

Works.


Some locations in the map for India is not locateable

Hi Concerned,
I am using the openlayer openstreet view API for my website but for some latitude & longitude the map does not show the name of the local near by areas.
Can you please help me in order to do so.
For example : https://www.indiamart.com/kaushalenterprises/enquiry.html

This is not an OpenLayers issue, but an issue with the data you are using.


Feature/project implementation

Dear reviewer, it is a pleasure to request a review for my PR before merging into main from dev.
Below were the PRs with commits made to meet the project requirements:

Merge pull request https://github.com/flemton/budget_app/pull/1 from flemton/feature/setup
Merge pull request https://github.com/flemton/budget_app/pull/2 from flemton/feature/models_and_auth
Merge pull request https://github.com/flemton/budget_app/pull/3 from flemton/feature/tests
Merge pull request https://github.com/flemton/budget_app/pull/4 from flemton/feature/controllers_and_routes
Merge pull request https://github.com/flemton/budget_app/pull/5 from flemton/feature/linters_setup
Merge pull request https://github.com/flemton/budget_app/pull/6 from flemton/feature/styling
Merge pull request https://github.com/flemton/budget_app/pull/7 from flemton/feature/tests
Merge pull request https://github.com/flemton/budget_app/pull/8 from flemton/feature/readme

Thanks sir @zieeco  for the awesome review, however, there is not enough time to implement all the changes and get them approved before the deadline so I’ll just relax and repeat the week, thanks again

Hello, Mr @mikethreels, thank you for the review, I made the push(commit and push) but didn't see it on GitHub in the PR for some reason so I tried making other changes to see if it was the commit or something was wrong with my repo. Once again, thanks.


Cannot Connect to Maven Central

I am receiving "Unable to connect to Maven Central" when I use dependency-check-gradle plugin latest version.  Seems it is trying to reach Maven central.
Its failing even when I set proxy in gradle.properties or Gradle command line

gradle  dependencyCheckAnalyze

I am using a downloaded NVD file and setting data directory and using it.
12:03:09.428 [DEBUG] [sun.net.www.protocol.http.HttpURLConnection] sun.net.www.MessageHeader@67cf49485 pairs: {null: HTTP/1.1 400 Bad Request}{Date: Tue, 05 Dec 2017 20:03:09 GMT}{Server: nginx/1.12.1}{Content-Length: 0}{Connection: keep-alive}
12:03:09.429 [DEBUG] [org.owasp.dependencycheck.analyzer.CentralAnalyzer] Could not connect to Central search (tries left: 4): Could not connect to MavenCentral (400): Bad Request
12:03:09.429 [DEBUG] [sun.net.www.protocol.http.HttpURLConnection] sun.net.www.MessageHeader@6e895b925 pairs: {null: HTTP/1.1 400 Bad Request}{Date: Tue, 05 Dec 2017 20:03:09 GMT}{Server: nginx/1.12.1}{Content-Length: 0}{Connection: keep-alive}
12:03:09.429 [DEBUG] [org.owasp.dependencycheck.analyzer.CentralAnalyzer] Could not connect to Central search (tries left: 4): Could not connect to MavenCentral (400): Bad Request
12:03:09.429 [DEBUG] [sun.net.www.protocol.http.HttpURLConnection] sun.net.www.MessageHeader@272afc4e5 pairs: {null: HTTP/1.1 400 Bad Request}{Date: Tue, 05 Dec 2017 20:03:09 GMT}{Server: nginx/1.12.1}{Content-Length: 0}{Connection: keep-alive}
12:03:09.430 [DEBUG] [org.owasp.dependencycheck.analyzer.CentralAnalyzer] Could not connect to Central search (tries left: 4): Could not connect to MavenCentral (400): Bad Request
12:03:09.431 [DEBUG] [sun.net.www.protocol.http.HttpURLConnection] sun.net.www.MessageHeader@718e60f75 pairs: {null: HTTP/1.1 400 Bad Request}{Date: Tue, 05 Dec 2017 20:03:09 GMT}{Server: nginx/1.12.1}{Content-Length: 0}{Connection: keep-alive}
12:03:09.431 [DEBUG] [org.owasp.dependencycheck.analyzer.CentralAnalyzer] Could not connect to Central search (tries left: 4): Could not connect to MavenCentral (400): Bad Request
12:03:09.432 [DEBUG] [sun.net.www.protocol.http.HttpURLConnection] sun.net.www.MessageHeader@266e146b5 pairs: {null: HTTP/1.1 400 Bad Request}{Date: Tue, 05 Dec 2017 20:03:09 GMT}{Server: nginx/1.12.1}{Content-Length: 0}{Connection: keep-alive}
12:03:09.432 [DEBUG] [org.owasp.dependencycheck.analyzer.CentralAnalyzer] Could not connect to Central search (tries left: 4): Could not connect to MavenCentral (400): Bad Request
12:03:09.432 [DEBUG] [sun.net.www.protocol.http.HttpURLConnection] sun.net.www.MessageHeader@7070c4c5 pairs: {null: HTTP/1.1 400 Bad Request}{Date: Tue, 05 Dec 2017 20:03:09 GMT}{Server: nginx/1.12.1}{Content-Length: 0}{Connection: keep-alive}

I would need the full debug log (at least the dependency-check portion) to assist with this further.  But it looks like the connection is getting blocked at your proxy.