New public map

New dashboard style applied.
Share buttons included.

Twitter share button would add owner twitter name if it is present.

Fullscreen map button is hidden by default (it is already present in the map info).
Mobile layout ready.

If device is a mobile, map will take up almost the device height screen. Only a small space showing the first sentence of the title (if it has more than one line).

Not possible to edit Disqus styles :(.

@Kartones , @rafatower , take a look at this (do a CR, but keep in mind it will probably change soon).
@xavijam I've added related_visualizations to (new) Carto::Visualization model, and the following methods in it name, title, description and likes were already there):

geometry_types
mapviews
For likes_count you just have to call likes.count. I think this is all you need. I don't know what "More user maps (0..3)" should return, but it's likely to be just a query.

Fixes #3120.
Fixes #3222.

What to test
The backend side of this changes the way visualization and table is loaded at Admin::VisualizationsController, and more specifically it's used in show, public_table, public_map, show_organization_public_map, show_organization_embed_map, show_protected_public_map, show_protected_embed_map and embed_map. Names are quite self-explanatory, but in case you want the specific routes, you can check routes.rb file, from lines 233 to 257:

get '(/user/:user_domain)(/u/:user_domain)/tables/:id/'
get '(/user/:user_domain)(/u/:user_domain)/tables/:id/map'
get '(/user/:user_domain)(/u/:user_domain)/tables/:id/table'
get '(/user/:user_domain)(/u/:user_domain)/tables/:id/public'
get '(/user/:user_domain)(/u/:user_domain)/tables/:id/public/table'
get '(/user/:user_domain)(/u/:user_domain)/tables/:id/public/map'
get '(/user/:user_domain)(/u/:user_domain)/tables/:id/embed_map'
get '(/user/:user_domain)(/u/:user_domain)/viz'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/map'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/table'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/public'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/embed_map'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/public_map'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/protected_embed_map'
get '(/user/:user_domain)(/u/:user_domain)/viz/:id/protected_public_map'

Frontend tests were OK :+1: (details)

:+1:

Frontend tests were OK :+1: (details)

FYI (no action required): right now page loads but throws a MAP: error creating layerCartoDB cartodb-leaflet needs at least a CartoDB table name, user_name and tile_style JS error, I think I'm not loading all layer information ok.

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Apart from the UUID check... :+1:  Great job!

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

@rafatower @Kartones maybe you should CR this now, because I won't be available for some days and its not likely to be deployed today. As you see in the description it affects many endpoints. In case you want to reduce the scope you just need to move public_map action (both url and method) to an new controller and undo the changes at admin/visualizations_controller.rb. As you see the changes are minimal.

ping @iriberri, because we will need to pass smoke tests, right @juanignaciosl?

Yep, it's really important (see the list of current affected endpoints). There's something wrong with organizations, I think, so if you can begin testing, @iriberri , begin with organizations and password-protected visualizations. And stop with any error. cc @rafatower @Kartones

I'm preparing some slides atm. Will start checking in a couple of hours, is it ok? I understand this is in production under a flag right?

Frontend tests were OK :+1: (details)

Nope, AFAIK there isn't any flag for it.

It is not in production now either ;-)

@javierarce, could you check html, css and js please?
Basically I have reused existing code, so not a big deal.

Frontend tests were OK :+1: (details)

Checking…

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Forgot to mention that I have reused code from new public dashboard as well :)
cc @javierarce

:+1: backend looks good

Ok, frontend looks good :cactus:

@iriberri, when you can start with tests, just ping @juanignaciosl or me if you have questions.

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

One detail, which is related and unrelated at the same time.

Do we expect that users add the @ by themselves? :)
Here it works fine, because there's not need for an @:

We could check if the twitter_username starts with a @ character, and if so, don't add it. No problem!

Public map is not being shown: https://ded01testorg.cartodb-staging.com/u/ded01orguser2/viz/ded01orguser2.2ce790e2-d86e-11e4-89b4-040120e0c101/public_map?redirected=true

When changing the privacy of the maps, even though you make them private, they're cached and still appear in "more from this user"

Not an issue, but this is kind of "duplicated" in the public profile, and in the button of the map you dont share the author's twitter :)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Embed isues fixed: Was a code flow going through Carto::User but calling the unimplmented in_database method. As we were going to need it anyway, I've researched and half-implemented its logic using AR.

It's still missing the pooling stuff (AR has an internal per-class pooler but I have to see how would fit with our "custom ids" for connection), but else I can just change the ID used to hash connections at our pooler to distinguish between Sequel and AR and store both for the time being.
I've also had severe issues with the new Adapters and Presenters due to using Carto::Api::LayerVizJSONAdapter instead of:
module Carto
module Api
class LayerVizJSONAdapter

Literally, the first method has some Rails require issues generating the vizjson call, but changing to 2nd method syntax everything is fine, so I'll change all code to use "module-based" namespacing.
Apart from that... it's a matter of testing more, and I'm not sure if we should also change redis cache key to differentiate new and old enpoints to avoid caching wrong or invalid content.

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

@rafatower CR please (brace yourself)

Frontend tests were OK :+1: (details)

If we're ok with the db connections, then ok to merge from my side

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


Fix visibility of testing module

The testing module has to be public (apparently) for consumers to resolve it. This might just be an issue with the IDE, but as a quick workaround we'll make the module public.

Codecov Report

Merging #56 (1313901) into main (bf5ba56) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #56   +/-   ##
=======================================
  Coverage   95.34%   95.34%
=======================================
  Files          31       31
  Lines         666      666
=======================================
  Hits          635      635
  Misses         31       31

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 bf5ba56...1313901. Read the comment docs.


updated the footer with adding email,phone and chat option here

#47 issue
Description
add the email,chat and phone option in the footer.

kindly assined it a label for any level with gssoc-ext label also.
hope for a merge .


Cannot download Image

Hello,
I am using Xcode 8 and Swift and I am trying to use Kingfisher in my project. However, I havent managed to load a image  from firebase to a UIImageView. I want to download some images and use them in a Array so that I can use them in a scroll view.
Now although the code compiles, I get: fatal error unexpectedly found nil while unwrapping Optional value. If I use a placeholder image with setImage, the placeholder image is displayed but not the one in the URL and the app doesnt crash. So the setImage is working but not setting the downloaded image, but the placeholder. Can anybody help?

import UIKit
import Kingfisher
class Backprogramme: UIViewController {

override var prefersStatusBarHidden: Bool {
    return true
}

@IBOutlet var BackPScroll: UIScrollView!

var imageArray = [UIImage]()
var folie1Image = UIImageView()
var folie2Image = UIImageView()
var anView = UIImageView()

override func viewDidLoad() {
    super.viewDidLoad()

   let folie1URL = "https://firebasestorage.googleapis.com/v0/b/backmaster-cdb60.appspot.com/o/Folie1.PNG?alt=media&token=efcb8e93-b817-41f3-a96f-946fd47cf468"
   let resource1 = ImageResource(downloadURL: URL(string: folie1URL)!, cacheKey: folie1URL)
   folie1Image.kf.setImage(with: resource1)

    print(folie1Image.frame)
    if (folie1Image.image == nil) {
        print("X")

   }

  imageArray = [folie1Image.image!]

    for i in 0..<imageArray.count{

        let imageView = UIImageView()
        imageView.image = imageArray[i]
        let xPosition = self.view.frame.width * CGFloat(i)
        imageView.frame = CGRect(x: xPosition, y: 0, width: self.BackPScroll.frame.width, height: self.BackPScroll.frame.height)

        BackPScroll.contentSize.width = BackPScroll.frame.width * CGFloat(i+1)
        BackPScroll.addSubview(imageView)

    }

    // Do any additional setup after loading the view.
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}

/*
// MARK: - Navigation

// In a storyboard-based application, you will often want to do a little preparation before navigation
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    // Get the new view controller using segue.destinationViewController.
    // Pass the selected object to the new view controller.
}
*/

}

The setImage method of Kingfisher is an async operation. The image property of your folie1Image is still nil when you trying to force unwrap it.
It is not a Kingfisher issue but the way you use Swift. I suggest to ask in Stackoverflow instead of opening an issue here. You would get more chance to be helped there.


Update README

Primarily changes dynamic datasets to growing datasets, since we do not actually support dynamic datasets yet.

@robinholzi if you could check this out and see if you notice anything else we should update that'd be great thank you!


Fixes

Slide Assignment Poll: Only show keypad keys for votecollector polling types.
Single votes view: Only show keypad serial numbers for votecollector polling types.
Delegate board: Change short name to "JoSm" for "John Smith".
Removed console debug logs.
POLLTYPES: Refactored 'votecollector_anonym' to'votecollector_anonymous'.
Bug fix validating candidates.

Successfully tested.
@jwinzer I pushed a commit which improves the delegate board layout (100% width etc.). Please test again. Then you can rebase to one commit.


Switch to the 2.0.0-alpha.0 steamlocate release

The fully-switch-from-steamy-vdf branch got released under 2.0.0-alpha.0

Sweet! Thank you.


Stopped Application memory

Hi @frodenas,
It could be nice to add the applications stated somehow, because now stopped application are counted as consuming the quota which I think it's not correct.
Most of my org are over quota because their have 20 or 100 stopped apps.
Thanks

@shinji62 The cf_application_info metric has already a state label, is this what you are looking for? or do you need any additional info?

Sorry Ferdy, I think my question was not enough clear, in fact the dashboard like cf_organization_memory_quotas.json include stopped application in the calculation of quota usage.
Maybe this is expected, but I was just wondering, it's difficult to check when you org is running of quota when all of them are +100% and so on.

Can you test this dashboard from my fork @shinji62 please (https://github.com/mkuratczyk/prometheus-boshrelease/blob/master/src/cloudfoundry_dashboards/cf_organization_memory_quotas.json) and verify it works for you? Of course I'll submit a PR if it does.

@shinji62 I've merged @mkuratczyk PR, please let me know if this fixes your issue.

Thanks PR resolve my issue.
Thanks @mkuratczyk


Branch 132376998

See test timeout at: https://ci.tensorflow.org/job/tensorflow-pull-requests-cpu-python3/1375/consoleFull
Trying again.
@tensorflow-jenkins test this please.

See test timeout at: https://ci.tensorflow.org/job/tensorflow-pull-requests-cpu-python3/1375/consoleFull
Trying again.
@tensorflow-jenkins test this please.

@tensorflow-jenkins test this please.


[SDPV-474,490,423] Response Set Curation

This addresses the 508 issues, adds response sets to the curation page, and deals with the broken links on the page.
Make sure you include the related JIRA issue in the title e.g. '[SDP-007] Fixed navbar issue'
Make sure you have checked off the following before you issue your Pull Request:

[x] Added unit tests for new functionality
[x] Passed all unit tests using rails test with 90%+ coverage
[x] Added cucumber tests for any new functionality
[x] Passed all cucumber tests using bundle exec cucumber
[x] Passed overcommit hooks, including running all code through Rubocop
[x] If any database changes were made, run rake generate_erd to update the README.md
[x] If any changes were made to config/routes.rb run rake jsroutes:generate
[x] If any HTML was added or modified check to make sure it was still 508 compliant with WAVE tool / carried over any attributes from similar sections

I think the PR is fine. I have two conflicting thoughts on Section.potential_duplicates method:

This method will make a lot of calls to elasticsearch. It may make sense to batch them up with https://www.rubydoc.info/gems/elasticsearch-api/Elasticsearch/API/Actions#msearch-instance_method
It's unfortunate to turn off ABC metrics for the code. potential_duplicates functionality could be pushed into methods in Question and ResponseSet. However, this would make batching up the queries harder.

@eedrummer As for reducing the number of elasticsearch calls I added a story SDPV-495  to the backlog to reduce load time of the curation wizard on large surveys. This would include switching over to multisearch.
https://publichealthsurveillance.atlassian.net/secure/RapidBoard.jspa?rapidView=6&projectKey=SDPV&view=planning&selectedIssue=SDPV-495
For now I got rid of the ABC Metrics disable and extracted logic to the Question and RS level


Update readme

Tried to fix some issues i faced while installing the package. The readme didnt seem to provide sufficient information for inexperienced users like me on installing the package. Created a patch for the readme file including the steps i had to perform to get the package running on my system.
Fixes issues #30 and #55

thanks for the update !


Low level API UpdateAsync does not perform increment on version property

If there is a class with a property with DynamoDbVersionAttribute, when a low level UpdateAsync attempt is made to increment the version, it performs SET instead of increment.
The same operation, performed from CLI works as expected
Background:
public class Data
{
     public string Data {get; set;}
     // many other properties
     [DynamoDBVersion]
     public int? Version {get; set;}
}

If the above class is saved with dbContext.SaveAsync, version numbers are handled property as per documentation.
But if a low level API IAmazonDynamoDB) is used to increment the version, the version is set to the increment value. If the update request is for some other attribute, increment happens as is should.
The same request, done from the CLI works:
aws dynamodb update-item --table-name MyTable --key '{"Id":{"S":"myid"}}' --update-expression "ADD #V :incr" --expression-attribute-names '{"#V":"Version"}' --expression-attribute-values '{":incr": {"N": "5"}}'

Expected Behavior
Low level API to behave as CLI.
Current Behavior
Assuming the Version attribute in the database is set to "5", when UpdateAsync is used with:
var request = new UpdateItemRequest
            {
                TableName = "MyTable",
                Key = new Dictionary<string, AttributeValue>
                {
                    {"Id", new AttributeValue {S = "myid"}}
                },
                ExpressionAttributeNames = new Dictionary<string, string>
                {
                    {"#V", "Version"}
                },
                ExpressionAttributeValues = new Dictionary<string, AttributeValue>
                {
                    {":incr", new AttributeValue {N = "1"}}
                },
                UpdateExpression = "ADD #V :incr"
            };

The value of the Version attribute in dynamo is set to "1", instead of "6".
Using "SET #V = #V + :incr", as well as hard-coding the attribute name in the UpdateExpression instead of using #V, produces the same result.
If another attribute is used, let's say "Counter", the update performs as expected, and Counter is incremented each time.
Context
Partial updates (ajax calls) to update only certain properties, while bumping the version so stream consumers can keep in sync with the updates.
Your Environment

AWSSDK.Core version used:  <IP_ADDRESS>
Targeted .NET platform: .Net Core 2.1

Hi @sloncho, I wasn't able to reproduce this issue.  Are you sure the object that you are updating already has an existing "Version" attribute at the time of the update?  If it didn't, that would explain why you are seeing the value set to the increment.


Remove generated file

Remove a generated file with paths specific to the user who originally added it to the repo.

*/dist is already in .gitignore. Not sure how this one slipped in.


readd ACLs for currently logged in user to nvidia devices (gh issue #52)

use udev's uaccess tag for open driver
re-add static udev magic for proprietary driver (ignores uaccess tag for some reason)

See https://github.com/openSUSE/nvidia-driver-G06/issues/52 for more details.

@scaronni-nvidia JFYI ...

Changes tested now. Fixes ACLs for logged-in user when using open and proprietary driver.


Initial make install instructions.

Document how to install on Ubuntu and CentOS (other distributions will
follow). I want folks to review the structure and then we can see add
the remaining platforms.
This is an initial draft, I expect a lot of discussion. Note that the instructions are long
and not that easy to follow. Any advice on how to make them simpler is appreciated.
The dockerfiles allow us to validate the instructions in a semi-automated way,
eventually we could run them as part of the CI builds.

This change is

PTAL.

PTAL.

@frankyn PTAL.

@vadimkiyanko this PR also includes changes to cmake/IncludeGrpc.cmake, and cmake/IncludeProtobuf.cmake and a few other files.
Did you also apply those changes?  They are necessary for the Dockerfile to work.  I just merged the PR into master.  Please let us know if that does not work.

@coryan Sorry, missed this changes. Yes, it works fine! Thanks.


Vanilla cacti spawn in desert villages

Recently have been working on removing the vanilla trees spawned with villages (DynamicTreesTeam/DynamicTrees#518) and noticed that vanilla cacti also appear to spawn in desert villages. These should receive the same treatment.

Fixed by #16, update will come alongside next DT beta.


Add Lufia 2 YAML

Weighted yaml for filler games for Lufia II. Focuses on longer games beatable by new players.

From MKI:
Iris treasure seems like it could be quite a bit too low, especially if it rolls less than 10/255, (YAY battling  3% chances on the fiirst iris treasure and then suffering through much lower odds for the rest of them, unless that's changed odds do get worst I'd recommend a minimum of 40 to air on a slightly easier side compared to the masochism that is 1/9 * 3/255 or 3/2295 that each normal chest turns into the desired last iris treasure, by the end of that world.
party start level should probably also be a range (1-10), it could be way too easy of random-low rolled level 50+
Blue chest:     random-high: 1 it has the issue of being too low on a chance (1/255), for reference I recall an earthbound LP that took a month for a 3/255 chance for the player too have RNG roll to get a Rare item.  Or the case where it can take years for a shiny Pokemon to appear for some people.
Other than those two extremes I think it's fine.

Made updates based on feedback.


Modify pos instead of *pp when getting mark character position

Fix #10148

I noticed this PR only after creating the patch.  I just did a bit more to have only one call to buf_byteidx_to_charidx().


Robot Name - Iteration 2

Nice iteration.  Take a look at the comments, refactor and resubmit when ready.  Let me know if you have any questions.


tests does not work on php7.1

There was 1 error:

Funct\Tests\Collection\CollectionSizeTest::testCollectionSize with data set #3 (null, true, 0)
count(): Parameter must be an array or an object that implements Countable

/Users/mokevnin/mokevnin/funct/src/Collection.php:508
/Users/mokevnin/mokevnin/funct/src/Tests/Collection/CollectionSizeTest.php:34
--
There were 2 failures:

Funct\Tests\Strings\StringsIsLowerTest::testStringIsLower with data set #16 ('ąčęėįšųūž', false, true)
Failed asserting that false matches expected true.

/Users/mokevnin/mokevnin/funct/src/Tests/Strings/StringsIsLowerTest.php:52

Funct\Tests\Strings\StringsIsLowerTest::testStringIsLower with data set #18 ('ĄČĘĖĮŠŲŪŽ', false, true)
Failed asserting that false matches expected true.

/Users/mokevnin/mokevnin/funct/src/Tests/Strings/StringsIsLowerTest.php:52

Thank you for rising issue and PR 👍


Various functions are not working correctly when logged in as a non-sysadmin

There is a need to review all functionality when operating the system as a non-sysadmin user. Various operations are failing, which work fine when logged in as a sysadmin.

Access control is working correctly.


Simplify metadata construction with builders

This patch series makes it a bit easier to construct metadata by wrapping the process with builders.
Additionally, to make it easier to work with generic metadata, it both moves the version and expires into the Metadata trait, and also rewrites the SignedMetadata to wrap a generic Metadata object. This allows us to more easily access the signed metadata state.

Right now it draws inspiration for default metadata values from go-tuf, where it defaults to the version to be 1 and expires at:

root - 365 days from now
snapshot - 7 days from now
targets - 3 days from now
timestamp - 1 day from now

Are these the right values?

It depends on your use case, but I would imagine for typical scenarios, it should be:

root - 365 days from now (offline keys, this is reasonable)
targets - 1 month from now (offline keys, this seems reasonable)
timestamp + snapshot (1 day, typically online keys, this is reasonable)

What do you guys think? @JustinCappos

Oops, I misread go-tuf, it expires targets metadata after 3 months. I've updated the PR to use that.

That is extremely reasonable!


Design and implement agent management server-side library

Once we know how the agent management should work and we have the protocol specification we want to implement a Go library which makes it easy for the backends to adopt the agent management solution. The library can be used by any vendor who has a Go-based backend and wants to manage the Collector remotely.
This task requires an API design for the library first, and then an implementation can be done.

[x] https://github.com/open-telemetry/opamp-go/issues/26
[x] https://github.com/open-telemetry/opamp-go/issues/32

Closing, initial work is done.


microsoft debugger for multiple languages

Is your feature request related to a problem? Please describe.

Describe the solution you'd like:

Describe alternatives you've considered:

Additional context:

Closing as incomplete. Please feel free to add more information if you have any and we can re-open the issue.


Added the TimeOfArrivalExample and the RangeISAMExample_plaza2

I added two Jupyter notebook files: one featuring an example on GTSAM using time of arrival signals, and a second one illustrating how GTSAM ISAM2 can be run to solve a 2D SLAM problem based on noisy range measurements from the ranging dataset "Plaza 2".

Will merge and add to the ToC afterwards


change the view/tags folder's name

I think that the name for the tags view folder in the branch 2.0 should be View/Tags, not View/tags

Fixed and committed in the 2.0 branch.


Added priority tag to LCP above the fold

Add priority loading to hero
Summary
Tells next that this is priority. It was flagging as LCP
Related to #34
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project and has no linting errors
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation, including any applicable ADRs
[x] My changes generate no new warnings
[ ] I have added tests that fail without these changes
[x] New and existing tests (unit, integration, accessibility) pass locally
[ ] Documentation updated
[ ] If there are security concerns they are addressed or ticketed after being discussed

@braxex any idea how to re-run these checks that are stalled??

Bypassing merge checks since ode climate stalled


Search box should have a max height

Origin page

on mobile with .catalog-search-box-0 #search-input.catalog-search-box selector having max-height set to 100px works well. Check it out and I can push changes if that works

Didn't work for me. The issue is mobile search, you can scroll the results which is a little strange

i see . :(

Closing as too low-pri. Functional but not ideal on mobile.


Ensure that we link the right set of compatibility libraries on Apple OS's

Update the OS runtime mapping for Apple OS versions and ensure that we don't link
against compatibility libraries we don't need.
Fixes rdar://84065193

@swift-ci please test

macOS Toolchain
Download Toolchain
Git Sha - e21296b0917c87e2b0edd260aa6d6fef203d16e4
Install command
tar -zxf  swift-PR-39738-1177-osx.tar.gz --directory ~/


[Select] Overflow Styling In Body Removed After Select Component Is Closed

[x ] The issue is present in the latest release.
[ x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Any overflow style applied to the body element is removed after the Select component is closed.
... becomes ... when the Select component is opened (with disableScrolLock: false).
Then, when the Select component is closed, it becomes ..., removing the overflow-x styling.
Expected Behavior 🤔
When the Select component is closed, the overflow-x (and any other overflow styling) should be brought back.
If body initially has style="overflow-x: hidden;", and after the Select component is opened it's style="overflow: hidden;", then when the Select component is closed, the style should be back to its original form: style="overflow-x: hidden;".
Steps to Reproduce 🕹
Codesandbox Example
Steps:

Go to public/index.html in your React project and add an attribute to body: style="overflow-x: hidden;".
Go to src/index.jsx and render a Select component with a few items.
Load the page, note that the body has overflow-x: hidden in its style.
Open and then close the Select component, and see that body no longer has the overflow-x: hidden styling.

Context 🔦
When I want to disable horizontal scrolling, or otherwise modify the overflow of the page, that is applied in the body element, the Select component reverts it.
Your Environment 🌎

`npx @material-ui/envinfo`
  System: OS: Windows 10 10.0.19042
  Binaries:
    Node: 14.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 7.19.1 - ~\Desktop\flyingBox\ui\node_modules\.bin\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.55)
  npmPackages:
    @material-ui/core: ^4.12.1 => 4.12.1
    @material-ui/icons: ^4.11.2 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.60 => 4.0.0-alpha.60
    @material-ui/pickers: ^3.3.10 => 3.3.10
    @material-ui/styles: ^4.11.3 => 4.11.4
    @material-ui/system: 4.12.1
    @material-ui/types: 5.1.0
    @material-ui/utils: 4.11.2
    @types/react: 17.0.5
    react: ^17.0.1 => 17.0.2
    react-dom: ^17.0.1 => 17.0.2

  Browser Used: Chrome

The relevant code is located at ModalManager:118, I assume. The issue is that CSS automatically overrides any specific overflow definitions, such as overflow-x when the more generic overflow is set. A solution to this problem would be to also store overflow-x and overflow-y:
    restoreStyle.push({
      value: scrollContainer.style.overflow,
      property: 'overflow',
      el: scrollContainer,
    },
    {
      value: scrollContainer.style.overflowX,
      property: 'overflow-x',
      el: scrollContainer,
    },
    {
      value: scrollContainer.style.overflowY,
      property: 'overflow-y',
      el: scrollContainer,
    });

Thanks for the report.
Frozen repro with v5: https://codesandbox.io/s/modal-overflowx-restore-1noky?file=/src/index.tsx:0-402


Negative Level Actor Resource

Made a Negative Level Actor Resource, where the value is the number of negative levels you have. Minor issue is that there are currently no modifiers to current HP/SP. If this isn't the approach we wanna take with -ve Levels that's fine, just wanted to put the idea out there.

That's precisely what I had in mind for the actor resources when I built the system! :)


#21 restrict booleanliteral

Depends on #38
Closes #21

Looks good, it's only missing some tests now.
I also need to add a test that covers the case where a level is defined but no ArithmeticOperator is used.
Let's also postpone the merging until we decide on #40, as it uses a slightly different approach.


Add subpanel expansion options

Right now, when a Parameterized object with a Parameter obj that is also a Parameterized is displayed in a Param pane, Panel will create an extra widget ... that when clicked will open a separate set of widgets to edit that subobject's parameters.  This is useful behavior for property sheets, which need to handle any arbitrary collections of nested objects, but when designing a dashboard with a fixed layout it's difficult to handle the dynamic resizing involved in this approach.  I propose that we add two Boolean flags to control whether we want the user to be able to control the subobject's widgets visibility, and whether we want them visible at first:

expandable=False, expand_at_first=True:  Start out expanded; provide no widget to unexpand
expandable=False, expand_at_first=False: Start out unexpanded; provide no widget to expand
expandable=True,  expand_at_first=True:  Start out expanded, but allow unexpanding
expandable=True,  expand_at_first=False: Start out unexpanded, but allow expanding (current default)

This way people could have a fixed layout if they wish, either with or without the ability to edit subobject parameters.

The names for these options are now:
expand_by_default and toggleable_subobjects
If we want better names we should decide now.

Those are indeed truly terrible names. :-)   How about subobject_expanded and subobject_widget (or subobject_button)?

@philippjfr , Jean-Luc and I just discussed it, and I now propose simply expand (for expand_by_default) and expand_widget (for toggleable_subobjects).  These parameters would control not just subobjects, but also things like param.DataFrame that would have bulky representations best not expanded unless actually needed.  Thus it's appropriate for them not to contain the word "subobject".
Could we also maybe have expand_widget be None by default?  None would mean that if expand is True, then expand_widget=False (as when it's expanded initially most people won't need or want a way to unexpand it, particularly as long as there is no responsive layout resizing), and if expand is False, then expand_widget=True (to give people a way to get access to the subobject parameters).  This way most people would only need to mess with one parameter, namely expand.
Also, should subobject_layout=pn.Column by default, since the widgets themselves are in a column by default? It's hard to say what default is best, but in the examples I've looked at I generally want pn.Column so that I don't overwrite the items to the right of the widgets.  subobject_layout might be best renamed to expand_layout to match expand...

@philippjfr, can you please weigh in on these names?  I'd like to stop using the old ones in our docs if at all possible...

Forgot about this one, I'm happy with most of your suggestions, going for a walk and then making a PR for these. I'll have to think about subobject_layout in the meantime.

Decided to adopt all the names except for expand_widget which I've instead named expand_button.


fix: specify full filepath to importMeta.d.ts (fixes #4125)

Description
Fixes #4125
The docs don't specify that a full filepath with extension is required, but it does seem to be.
Additional context
These triple-slash directives seem to be pretty fiddly.  There has been https://github.com/vitejs/vite/pull/4031 and https://github.com/vitejs/vite/pull/4003 trying to get this working.

What is the purpose of this pull request?

[X] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[] Ideally, include relevant tests that fail without this PR but pass with it.

Thanks @jceipek and @IanVS!
@gunters63, @userquin would you confirm that this PR fixes your issues?

Thanks @jceipek and @IanVS!
@gunters63, @userquin would you confirm that this PR fixes your issues?

Yes, works for me. Thanks!


Livewire V3: Events not captured

We are not seeing events captured when using this with Livewire V3.
Will try to add a minimal repro case later in case you cannot reproduce easily.

@kassah yeah, some example would be appreciated.

I also am experiencing this issue, any event dispatched from a livewire component is not shown in the event view

@kassah @jcordingley should be fixed in v1.1.1


AMCBLDC – Validate the thermal model further and link it to the control logic

Following up on #14, we're required to:

Make more observations in a wider range of temperatures for validation purposes.
Close the loop between the output of the thermal model and the control logic.

Regarding the control logic, we only need to fiddle with the supervisor_rx, as the newly created flag enable_thermal_protection is already handled by the outer controller.

If we want to observe a wider spectrum of temperatures, I think we need to apply some kind of load to the wrist, or the motor on the Lego setup, otherwise we might never reach more than 35C, like in the previous test:

Also, we might benefit for some cooling tool like a fan, so that we can reach a zero temperature delta more quickly.

@lawproto Helped me mount on the wrist the shaft that Dev used in the past, upon suggestion of @fiorisi :https://github.com/icub-tech-iit/ergocub-design-lowerarm/issues/106
So now we can properly load it.

At the moment the wrist setup is exhibiting a strange behaviour after mounting the weights' shaft.
There is a constant current $Iq = 45mA$ on the AMCBLDC 2 (out of 3) when the wrist is idle:

Even though the shaft weighs more than the 4-fingered hand, the joints do not tilt down because of gravity, so I expect the current to be 0 mA.

A bit puzzling.
We should understand the source of this readout in IDLE.
At any rate, in the meanwhile, we can start the tests disregarding this phenomenon.

It seems like the problem is on the AMCBLDC 2, because the issue disappears if I switch to another board. Additionally, a noticeable ticking noise can be heard when the motor spins.
Nonetheless I agreed with @pattacini F2F that we can go on by monitoring another motor, eg the third one.

I managed to start one decent trial, but the performance seems very poor on the model side. At 300 sec I added 500 gr weight, and even though the motor temperature increases noticeably, the model estimation shoots up.
A comment on the data:

It needs temporal alignment, the dip in temperature estimation is caused by putting it in idle to load the weight
The initial temperature that I removed as offset was 26C for the model, and 30C for the camera.

The only thing added was the $I_d^2$, correct? I need to think about why It could influence the estimation so much.

That's what I thought too, we could log I'd through CAN as well and see how it behaves.

The direct current $I_d$ is very noisy and it could have an effect on the overall evaluation of the temperature. It can reach high peaks of hundreds of milliamps, but it is quickly controlled to zero.

Here is a detail of the Id when the motor is moving. There are significant peaks, but they are not long lasting.

The thermal model is implemented through a LP filter with a very low pass-band; hence it should take the DC value (~ 0 A) and filters out all the high-frequency ripples.
At any rate, we may consider removing $I_d$ (temporarily[^1]) from the model to check the outcome.
[^1]: The direct current provides a contribution we cannot ignore.

Reflecting on the parameters, the thermal resistance $R_{th2}=49 K/W$ seems worryingly high.
In steady state, applying constant $I_{q} = 200mA$, we get $\Delta T = 0.2^2 \cdot 25.9 \cdot (7 + 49) = 58C$, which is a temperature differential that was not observed so far, even with 25C ambient temperature.
It would be useful to know where 49 K/W comes from, because if we take

the rated current $I_N = 0.36 A$ at 22C ambient temperature
the phase-phase resistance $R=25.9 \Omega$
the maximum temperature allowed for the windings $T_{max}=125C$

We can find a total thermal resistance
$$
\frac{\Delta T_{max}}{P_{max}} = \frac{125 - 22}{0.36^2 \cdot 25.9} = 30.7 K/W
$$
which is significantly lower than (7 + 49) K/W
If possible, I'd keep in touch with Faulhaber to gather more info, while we could fit the thermal model from the camera observations.

Just for sake of completeness, here is the snippet that I use to call the model:
    static uint8_t counter_Ts_10ms = 1;

    if(counter_Ts_10ms == 10)
    {
        AMC_BLDC_step_Time_10ms();
        counter_Ts_10ms = 1;
    }
    counter_Ts_10ms++;

Which at the moment seems ok to me and does not result in too many calls that would ruin the estimation.

else counter_Ts_10ms++;

Otherwise the sample time will be 9 ms.

Reflecting on the parameters, the thermal resistance $R_{th2}=49 K/W$ seems worryingly high. It could mean that a power draw of 2W results in 198C.

$2$ $W$ yields 98 $^oC$ over $R_{th2}=49 K/W$, not 198 $^oC$; was it a typo?
Our motor is of $6$ $W$ power and to reach $2$ $W$, we would thus need $i=\sqrt{2/25.9}=277$ $mA$, which is not implausible, after all.
Why are you relating $T_{max}$ to $I_N$?
$I_N$ doesn't cause the motor to reach $T_{max}$, in my understanding.

Something we kind of neglected so far:
Although $E[I_d]=0$, it comes out that $E[I^2_d]>0$ and thus it impacts the temperature estimation, as it should actually be.

I made a couple of other tests during the past few days, both without and with added weights to the wrist.
No weight
This is the resulting plot without weight. Sadly, in the first case the thermal camera software crashed and the logging was lost, but I can guarantee that the measured temperature was only off by 1C. (From now on I'll also note manually the temperature every minute")

500 gr weight
Here I kept the offset for transparency, but we see that the measured temperature with the thermal camera reaches very similar values to the above ones.

What are the movements of the wrist in both cases, i.e., with and without load?
The patterns seem different.

What are the movements of the wrist in both cases, i.e., with and without load?

The pattern is mostly the same, a repeated yaw motion, +-40 deg and then rest at 0.
Continuing with the tests to make sure that the temperature significantly increases with the current, I applied 1Kg payload to the wrist. In the first plot, I repeated an alternate yaw motion, while in the second plot I kept the wrist still.
The results still seem to be poor,  but we can notice that the temperature estimation increases constantly due to the constant current (220mA), while the measured one settles at 41.5C

The thermal model flashed in the firmware, in its current form behaves just like the continuous one. Of course we can fit a model with the data, but maybe it's just a matter of reworking the transfer function.

Reflecting together with @pattacini and @maggia80 we decided to:

Read the temperature with a thermocouple, to avoid any doubts regarding the thermal camera setting:
Consider only the one of the two thermal resistances in the model

Using only the 7 K/W resistance at the moment seems to yield a small temperature difference, 1.7C after 15 minutes, compared to the 6.3 measured with the thermocouple

A second test with 1Kg payload and 7 K/W resistance, gives the following results (in temperature delta) in steady state:

Thermocouple 10.5 degC
Thermal camera: 11.7 degC
Thermal Model:  4 degC

There's a discrepancy also in this case, which suggest the necessity of estimating the thermal model.

A side note:
Since in the model $\Delta T_0 = 0$ and $T_{environment} = 25 C$ ( for example) we will have to be careful when power cycling the boards. We do not store the $\Delta T$ before turning off the boards, and therefore at the next boot the model will be reinitialized with the above values. However, the motor might still be warm enough to cause issues.

Reflecting together with @pattacini and @maggia80 we decided to:
* Read the temperature with a thermocouple, to avoid any doubts regarding the thermal camera setting:

* Consider only the one of the two thermal resistances in the model

In the scenario of no payload applied and using only the 7 K/W resistance at the moment, seems to yield a small temperature difference, 1.7C after 15 minutes, compared to the 6.3 measured with the thermocouple.

Thinking about it, it would not make sense to use the 7K/W resistance, because if we used that we would have to take the temperature difference between windings  and housing, and we cannot measure the first. I think we should stick with the 49 K/W (or its fitted equivalent), because that's only where we are able to probe the temperature.

49K/W seems still too high given the measurements. As @maggia80 pointed out, the motor is not tested in the same conditions as ours, especially without gearbox. That could change the $R_{th2}$ estimate.

Let's proceed with identifying the parameters $R_t$ and $\tau$ robustly by covering a wide range of operating conditions with currents and temperatures (it'll certainly take a while to do so).
As usual, the validation will need to be performed on motors/setup/operating-conditions not seen during fitting.


Grafana Dashboard PolicyReports panel columns need updating

I'm using the Grafana dashboard named PolicyReports provided in the Helm chart.  To be consistent with other dashboards, the panel columns should be updated as follows:
Failing PolicyRules

Add category and severity columns in front of namespace

Failing ClusterPolicyRules

Filter out Pass and Skip statuses since this is supposed to be for failing policies only
Reorder columns to be inline with other dashboards (category, severity, kind, name, policy, rule, status)
Remove container column

Hey thanks for your suggestions, I will have a look on it.


Update pact-consumer-swift version

pact-consumer-swift is apparently now implemeting Pact specification v2, and does not state anything about beta anymore! :-)

Thanks for the contribution :)


[Feature tags] What does “required” mean?

The descriptions of numerous features use the word “required”. Sometimes it appears to have just the conversational meaning, close to “needed”. At other times it appears to mean an actual specification requirement, e.g., in the “script/language sensitivity” fields. But then it doesn’t state whether it’s a requirement on fonts or on shaping engines or on applications or what else.
The text should be clarified to identify which “required” actually constitutes requirements and on whom.

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

ID: 844c49c1-bbee-b298-f6da-ace893d4e52b
Version Independent ID: c8880433-22b5-a615-586f-fee121d096fd
Content: Feature tags - Typography
Content Source: typographydocs/opentype/spec/featuretags.md
Product: typography

When used in the name of a feature, 'required' means it has a default state of on, without the ability to be disabled by the user. If we were to make default state a standard data point for all features, that would clarify the meaning of a major use of the term required, as well as clarifying the status of required features—in the same sense—that do not have the word as part of the name. We could then check to see if there are other uses of the term in the spec that do not have the same meaning, and revise the wording of those to clarify the particular meanining.

Use of "required" in the "Script/language sensitivity" field is certainly ambiguous, and it would be good if that were clarified. We can at least start with that.
I doubt there is any feature for which it could be said that its use in a font is required, even for support of a particular script. I wouldn't want to try to create a Bangla font without using the 'blwf' feature, but I could conceive of implementing a Bangla font (albeit inefficiently) using only the 'ccmp' feature.
So "required" statements in the script/lang field should really pertain to software implementations.
But how best to word things? Two options:

State the application requirement in the script/lang field:

"Script/language sensitivity: Used in a number of Indic scripts. Support of this feature is required in applications that support these scripts."

Don't mention in the script/lang field, but leave that to the "application interface" field:

Application interface: In recommended usage, this feature triggers substitutions required for correct display of certain scripts. It must be applied in the appropriate contexts, as determined by script-specific processing requirements.
...
Script/language sensitivity: Used for a number of Indic scripts.

Thoughts?

Perhaps “required” can be replaced by a more verbose description of the three states any given features can have:

Should be always applied (i.e. without the ability to not apply it, but this concerns application UIs mostly, low level APIs usually allow any feature to be disabled).
Should be applied by default.
Should not be applied by default.

Or a less verbose record of one or other of the three default states possible for each feature:
1,0
1,1
0,1
Where the first binary number indicates whether the feature is active by default, and the second indicates whether it can be toggled on and off at the application/user level. That is the notation system I used here, and it seems to be have been easily understood by developers looking for a clear indication of default state.

If we're thinking that the eventual direction should be a field/fields to indicate equivalent to John's 1,0 / 1,1 / 0,1, then I think what would make sense for now would be my option #2 above. The application interface fields have already been revised for #601 and have the kind of wording illustrated above, so I'd only be changing the script/lang fields to remove the ambiguous "required".

Good to know that it’s a requirement on implementations, not on fonts.
It’s a bit more complicated than a simple 1 or 0 though. A number of these statements refer to specific scripts or script groups, and looking at the shaping engine documentation, not all such features are applied in all shaping engines. So the real answer is that “Shaping engines must enable or disable this feature as described in their respective documentation” (which assumes that such documentation and implementations agree, which is unfortunately not always the case).
“Support” is too unclear in this context. As John and Khaled said, it’s really about whether the feature is applied by default.
Some of the descriptions seem quite bizarre, BTW. For example in cjct: “Required in Indic scripts that show similarity to Devanagari”. How is “similarity” defined here? I’m using the feature in a Balinese font, knowing that the USE applies it – but Balinese is not even an Indic script.

I've revised the script/lang field in many feature descriptions. See attached for current draft revisions.
features_uz.pdf
features_ae.pdf
features_fj.pdf
features_ko.pdf
features_pt.pdf

So the real answer is that “Shaping engines must enable or disable this feature as described in their respective documentation”

Or (for 1,1 features), ‘Shaping engines SHOULD enable and apply this feature by default, and MUST if called for in respective script shaping documentation’. Again, this presumes accurate documentation. In the not-inconceivable event that USE is not the be-all and end-all shaping engine, and that other script-specific engines are developed, we want to encourage them to be inclusive in application of features.

Some of the descriptions seem quite bizarre, BTW. For example in cjct: “Required in Indic scripts that show similarity to Devanagari”. How is “similarity” defined here?

Yes, that's bizarre unless one knows the history of the cjct feature and what it addresses. It should be something like: ‘The 'cjct' feature is applied before final orthographic unit reordering, and is required for Indic and related scripts that optionally shape some conjuncts ligatures with explicit virama as pat of the input, rather than first shaping half form sequences, and in which subsequent reordering of reph or vowel signs is affected by continued presence of explicit virama after processing of cjct.’
I would add, ‘Font makers may use the cjct feature for any ligation that they want to take place prior to final reordering of the glyph run.’
A lot of the existing feature descriptions presume script-specific layout engines, in which the features are presumed to relate to specific aspects of certain scripts or categories of script. USE really alters the way in which font makers look at features: they become, on the one hand, simply organisational holders for lookups and, on the other hand, tools for managing glyph ordering, and hence output and input for subsequent lookups, in orbit around the reordering moment.


OAuth cards from OAuthPrompt do not render within teams app

Versions
What package version of the SDK are you using.: ~4.14.0
What nodejs version are you using: v14.15.1
What browser version are you using: Chrome
What os are you using: MacOS Catalina
Describe the bug
OAuth cards from OAuthPrompt do not render within teams app. It works fine within the Bot Framework Emulator and within 'Test in Web Chat' option in the Azure bot page in Azure portal. It does not however show the login card within the teams web app or within teams desktop app.
To Reproduce
Steps to reproduce the behavior:

Clone the BotBuilder Samples repository.
Navigate into the teams auth example number 46 folder.
Install the dependencies.
Create an Azure bot on Azure portal and configure an OAuth connection for 'Azure AD V2'
Set up the environment variables in the `.env' file in the sample number 46.
Run the bot and run ngrok for a https url.
Set up the bot in Azure portal to use the ngrok URL + /api/messages as the messaging endpoint.
Test the sample using the following methods:

Inside of Bot Framework emulator: works fine
Inside of 'Test in Web Chat' option: works fine
Update the manifest.json file in the teamsAppManifest with the correct ids and zip the contents of the folder and upload as a new app in teams. Test the behavior in teams web and teams desktop.

Expected behavior
The login prompt card shows up just like it did on the emulator and the web test chat feature.
Actual behavior
The prompt never shows up. If you already login from a different client, you will get auto logged in. But if you logout with the 'logout' command and try to login again, the prompt does not show up.
Screenshots
Works in Bot Emulator

Works in Test in Web Chat

Does not work in teams

Additional context
Looking into it a bit more, I see the prompt card that is sent just sends an activity with the content below. Without using OAuthPrompt, if you simply use sendActivity to send this object from your bot, it behaves the same way (works everywhere except in teams).
{
  "type": "message",
  "inputHint": "acceptingInput",
  "attachments": [
    {
      "contentType": "application/vnd.microsoft.card.oauth",
      "content": {
        "buttons": [
          {
            "type": "signin",
            "title": "Sign In",
            "value": "https://token.botframework.com/api/oauth/signin?signin=<another_id>"
          }
        ],
        "connectionName": "<your-connection-name>",
        "tokenExchangeResource": {
          "id": "<some_id>",
          "uri": "api://botid-<app_id>",
          "providerId": "<some_other_id>"
        },
        "text": "Please sign in"
      }
    }
  ]
}

I had a call with someone from the Microsoft support team and got this issue resolved. Apparently I did have problems in my configuration. I am closing this issue, but I think it helps to have some error message displayed somewhere that will help us debug when there are issues in our OAuth configuration instead of it having it not show any errors and also not work.

Linking to answers.microsoft.com where I posted what worked for me, so that people can find it if they face the same issue as I did.
https://answers.microsoft.com/en-us/msteams/forum/msteams_tfb-msteams_topother/oauthprompt-card-does-not-render-inside-of-teams/96ad96cc-4f8b-4cd5-9274-a487bb25a74e

Seems this issue pop-up again. Please do needful.
https://github.com/microsoft/BotBuilder-Samples/issues/4013
https://github.com/OfficeDev/microsoft-teams-library-js/issues/2510


Issues connecting to the Database using MySqli.

I am using PHP-Rbac framework for the first time... I have tried setting it up using Mysql it then displays that the try was successful but doesn't populate the db.... when I try connecting using MySqli it displays the following errors:
Notice: Unable to prepare statement: INSERT INTO phprbac_rolepermissions (RoleID,PermissionID,AssignmentDate) VALUES (?,?,?), reason: Table 'php_rbac.phprbac_rolepermissions' doesn't exist in E:\xampp\htdocs\sandbox\php_rbac\src\PhpRbac\core\lib\Jf.php on line 149
Warning: call_user_func_array() expects parameter 1 to be a valid callback, first array member is not a valid class name or object in E:\xampp\htdocs\sandbox\php_rbac\src\PhpRbac\core\lib\Jf.php on line 159
Fatal error: Call to a member function execute() on boolean in E:\xampp\htdocs\sandbox\php_rbac\src\PhpRbac\core\lib\Jf.php on line 160
what could be the reason?

I think it has created some of the database, so it thinks its there, but it actually isn’t complete.
Try giving it root credentials so that it can create a new db and populate it.

On Aug 11, 2015, at 9:14 AM, Kunwark<EMAIL_ADDRESS>wrote:
I am using PHP-Rbac framework for the first time... I have tried setting it up using Mysql it then displays that the try was successful but doesn't populate the db.... when I try connecting using MySqli it displays the following errors:
Notice: Unable to prepare statement: INSERT INTO phprbac_rolepermissions (RoleID,PermissionID,AssignmentDate) VALUES (?,?,?), reason: Table 'php_rbac.phprbac_rolepermissions' doesn't exist in E:\xampp\htdocs\sandbox\php_rbac\src\PhpRbac\core\lib\Jf.php on line 149
Warning: call_user_func_array() expects parameter 1 to be a valid callback, first array member is not a valid class name or object in E:\xampp\htdocs\sandbox\php_rbac\src\PhpRbac\core\lib\Jf.php on line 159
Fatal error: Call to a member function execute() on boolean in E:\xampp\htdocs\sandbox\php_rbac\src\PhpRbac\core\lib\Jf.php on line 160
what could be the reason?
—
Reply to this email directly or view it on GitHub https://github.com/OWASP/rbac/issues/73.

I'm getting the same results with a database user that has full permits.  The databases aren't getting created.

Same issue here. The database tables are not created during the installation process.

Same problem here. This project looks very promising but is lacking some detail to get it up and running (at least for me).

Can you guys update me on what exactly you did to reproduce this issue?
-A

On Oct 20, 2015, at 9:28 PM, 9SL9<EMAIL_ADDRESS>wrote:
Same problem here. This project looks very promising but is lacking some detail to get it up and running (at least for me).
—
Reply to this email directly or view it on GitHub https://github.com/OWASP/rbac/issues/73#issuecomment-149752740.


python: fix datafusion.functions access

Rationale for this change
The example in our python user doc is not working with the latest binding implementation. Found this while validating https://github.com/apache/arrow-datafusion/pull/1253
What changes are included in this PR?

Fixed datafusion.functions access
Added tests
Fixed python docs
Implemented __str__ for PyExpr

Are there any user-facing changes?
NO

is there any downside in making f = datafusion.functions work on top of import datafusion.functions as f?

Not really, it just feels odd for me. If we want to expose the functions as a module then I would stick with import datafusion.functions. Otherwise we could also expose all of the functions as staticmethods on a class e.g. FunctionRegistry (which would make the rust code simpler perhaps) or we could also expose it as a python object instead of an actual module.
If we want to remain backward compatible then we can re-export the functions submodule, column as col and literal as lit symbols.
I don't have a strong opinion, so feel free to merge this PR - we can discuss it further after the release.

@kszucs I pushed an update to keep the behavior consistent with pyarrow and updated the user doc to use the new api. Can you take another look?

A bit late, but LGTM. Thanks @houqp!

Thank you @kszucs for the epic refactor. Time to release it to pypi :)


Change the type of input to password in register page

Is your feature request related to a problem? Please describe.
In the register page, the input type of password should be password instead of tel.
Describe the solution you'd like
Add input as password so that stars appear when password is typed.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
I would like to work on this under LGMSOC

Hey! I had previously made a commit to this repo, but now when I want to make PR for this issue, why is that previous commit showing which I don't want?


Add JIIT, Noida, India

@shermix I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host jiit.education not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


[lint] Various fixes

This takes care of most lint warnings/errors that currently show up.

I'm holding off merging until @GregAC and @eunchan could give this a look also.

LGTM :)

On Apr 15, 2022, at 1:15 PM, Michael Schaffner @.***> wrote:
I'm holding off merging until @GregAC https://github.com/GregAC and @eunchan https://github.com/eunchan could give this a look also.
—
Reply to this email directly, view it on GitHub https://github.com/lowRISC/opentitan/pull/12143#issuecomment-1100369351, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJDG3TVQZ4UF7SYQIMILQTVFHE7LANCNFSM5TPI4GOA.
You are receiving this because you were mentioned.

Thanks @GregAC! How would such a fix look like? Insert a flop on the idle_o output path?

I have added a mubi sender for the idle_o signal, which also contains a flop (this is actually good for CDC also, since the idle_o signal was driven combinationally before).

Thanks @GregAC! How would such a fix look like? Insert a flop on the idle_o output path?

RTL wise yes, but there may be a DV environment impact too (as it models the timing down to the cycle).

Ok, let's take care of flopping the IRQ and idle_o signals here: #12215. I have removed these patches from this PR.


suppress all unintended CDK messaging in amplify info

Environment information
System:
  OS: macOS 14.3.1
  CPU: (10) arm64 Apple M1 Pro
  Memory: 83.23 MB / 32.00 GB
  Shell: /opt/homebrew/bin/fish
Binaries:
  Node: 20.11.1 - ~/Library/Caches/fnm_multishells/19127_1710431357384/bin/node
  Yarn: undefined - undefined
  npm: 10.2.4 - ~/Library/Caches/fnm_multishells/19127_1710431357384/bin/npm
  pnpm: 8.15.4 - ~/Library/Caches/fnm_multishells/19127_1710431357384/bin/pnpm
NPM Packages:
  @aws-amplify/backend: 0.13.0-beta.6
  @aws-amplify/backend-cli: 0.12.0-beta.6
  aws-amplify: 6.0.20
  aws-cdk: 2.132.1
  aws-cdk-lib: 2.132.1
  typescript: 5.4.2
AWS environment variables:
  AWS_PROFILE = josef-gen2
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

NOTICES         (What's this? https://github.com/aws/aws-cdk/wiki/CLI-Notices)

29483(cli): Upgrading to v2.132.0 or v2.132.1 breaks cdk diff functionality

Overview: cdk diff functionality used to rely on assuming lookup-role.
With a recent change present in v2.132.0 and v2.132.1, it is
now trying to assume deploy-role with the lookup-role. This
leads to an authorization error if permissions were not
defined to assume deploy-role.

Affected versions: cli: >=2.132.0 <=2.132.1

More information at: https://github.com/aws/aws-cdk/issues/29483

If you dont want to see a notice anymore, use "cdk acknowledge <id>". For example, "cdk acknowledge 29483".

Description
see info above:
NOTICES         (What's this? https://github.com/aws/aws-cdk/wiki/CLI-Notices)

29483(cli): Upgrading to v2.132.0 or v2.132.1 breaks cdk diff functionality

Overview: cdk diff functionality used to rely on assuming lookup-role.
With a recent change present in v2.132.0 and v2.132.1, it is
now trying to assume deploy-role with the lookup-role. This
leads to an authorization error if permissions were not
defined to assume deploy-role.

Affected versions: cli: >=2.132.0 <=2.132.1

More information at: https://github.com/aws/aws-cdk/issues/29483

If you dont want to see a notice anymore, use "cdk acknowledge <id>". For example, "cdk acknowledge 29483".

while testing noticed this also occurs when running sandbox


i want to add a program on Longest Common Prefix in c++

please assign me this project under hacktober fest 2023.

Hey i'm sure i can do this so please assign me this

please assign me

Hey !! Please assign me this i am sure i can do it in a most presentable manner.

Please assign me this issue

I wish to contribute. Kindly assign me this issue.

I have raised pr for #2276. Can you please merge it and review @shuklaabhinavv

Hello,
I am actively participating in Hacktoberfest 2024. I came across your repository, and I am very interested in contributing to it. I would love to work on this repo or any other task where you feel I can provide value. So Please assign me this repo.


[Feature]sort-flink support configurable deserialization.

Description
sort-flink support configurable deserialization.
Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The issue has supported by org.apache.inlong.sort.protocol.source.SourceInfo.deserializationInfo

Theorg.apache.inlong.sort.protocol.source.SourceInfo.deserializationInfo.


Support NextJS edge runtime

This PR is working with NextJS edge run-time (tested with Chat LlamaIndex (using pnpm build && pnpm start - pnpm dev doesn't work yet))
Changes:

Make core edge compatible
Removed non-edge compatible deps

Next steps:
Discuss how to deal with non-edge compatible deps - see comments

But no matter how, this would need more effort because we need to change many path import

For example, some files would indirectly import vectorStore.js
index.edge-light.js -> ... -> verctorStore.js

seems not working

working, but some deps seem not correct


Could you please add the 'Hacktoberfest' topic to this repository?

Hey this repository is not officially part of Hacktoberfest 2023, this is for first-time contributors at Karunya University to practice hands-on. Sorry for the inconvenience.
Next year we are officially planning to be a part of Hacktoberfest see you then!
Thank you.


_TrivialClosureConvert mis-specifies dynamic attributes

I believe this should say static=False: https://github.com/patrick-kidger/equinox/blob/5a5bf287f73032201c34ff2c73e73573f8232739/equinox/_ad.py#L560

Nope! This is intentional. Given some pytree-of-arrays, then we calleval_shape to transform it into a pytree-of-shapedtypestructs which we compare against later, to be sure we are later called with compatible arguments:
https://github.com/patrick-kidger/equinox/blob/5a5bf287f73032201c34ff2c73e73573f8232739/equinox/_ad.py#L546-L549
In this case we actually store the pytree-of-shapedtypestructs as a static (tuple of shapedtypestructs, treedef). This allows the user to pass this object around safely -- across JIT boundaries, eval_shape boundaries etc. -- without those shapedtypestructs erroneously getting turned back into dynamic objects.
I appreciate the heads-up though -- I can see how the 'dynamic' in the name suggests otherwise!

Ah, my mistake, I see what you mean. I think I misdiagnosed this, but there's some real issue related to tracer hashing related to ClosureConvert. Here's a minimal repro via diffrax:
import jax
import equinox
import diffrax

print(f"{jax.__version__=}\n{equinox.__version__=}\n{diffrax.__version__=}")
# jax.__version__='0.4.30'
# equinox.__version__='0.11.4'
# diffrax.__version__='0.5.1'

# Accelerate the deprecation of tracer hashability, so it raises a TypeError rather than a warning:
from jax._src import deprecations
deprecations.accelerate('tracer-hash')
jax.jit(hash)(1)
# TypeError: unhashable type: <class 'jax._src.interpreters.partial_eval.DynamicJaxprTracer'>

from diffrax import diffeqsolve, Dopri5, ODETerm, SaveAt, PIDController

vector_field = lambda t, y, args: -y
term = ODETerm(vector_field)
solver = Dopri5()
saveat = SaveAt(ts=[0., 1., 2., 3.])
stepsize_controller = PIDController(rtol=1e-5, atol=1e-5)

sol = diffeqsolve(term, solver, t0=0, t1=3, dt0=0.1, y0=1, saveat=saveat,
                  stepsize_controller=stepsize_controller)
# TypeError: unhashable type: <class 'jax._src.interpreters.partial_eval.DynamicJaxprTracer'>

Something in this call stack ends up computing the hash of a tracer... I thought it may have been in _TrivialClosureConvert, but it looks like it's something else (but closely related). Any thoughts on this?

Ah, I see! Haha, looks like we were hashing tracers all along and never even noticed. This should be fixed in #769.

For the record -- i.e. future me, if I ever need this -- the problem was that we generate a _ClosureConvert on this line:
https://github.com/patrick-kidger/equinox/blob/5a5bf287f73032201c34ff2c73e73573f8232739/equinox/internal/_loop/checkpointed.py#L245
which is a callable that includes some tracers in its pytree structure. This ultimately hits a lax.while_loop(cond_fun, ...). Now lax.while_loop hashes its function arguments to try and record their jaxprs, as in https://github.com/google/jax/issues/13554. This previously worked because tracers were hashable and we never even noticed, but when tracers become unhashable then it explodes.
The fix is just to add a dummy lambda val: cond_fun_(val) wrapper.


Feat: : Implement a 404 Not Found Page

Describe the feature
Feature Request: Implement a 404 Not Found Page
Title: Implement a 404 Not Found Page
Description:
Create a custom 404 Not Found page to improve user experience when visitors encounter broken links or try to access nonexistent pages on the website. The 404 page should be informative, user-friendly, and consistent with the site's desig
Add ScreenShots

Record

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

Hello @HarshadaGirase !
Someone is already assigned this issue, if they fail to do so you will be pinged and assigned the issue

Okay but kindly assign asap

The issue is already addressed.


[bugfix] Generate run reports per test in CI generate

Fixes #1894.
@sabo I am opening this PR from your fork. This is what I also thought as a possible solution. I will expand it a bit and merge it.

Hello @vkarak, Thank you for submitting the Pull Request!

In the file reframe/frontend/ci.py:

Line 30:80: E501 line too long (86 > 79 characters)

In the file reframe/frontend/cli.py:

Line 606:9: E303 too many blank lines (2)

Do see the ReFrame Coding Style Guide

Replaced by #1972.


Update citation and checkpoint descriptions

I replaced the arXiv citation with the ICLR one (BibTeX copied from OpenReview), as well as adding a CITATION file for GitHub's new citation feature.
I also wrote some more descriptions for the T0 Single Prompt and T0 Original Task Only checkpoints.

top thanks Albert!


[Incompatibility] Condensed Creative mod - Unable to scroll smoothly

It works with the condensed creative mod but I'm unable to scroll correctly
https://mclo.gs/gBYRZSB

Can you try it with the discrete scroll option from the base game enabled? This seemed to help on my computer. I will need to fix the trackpad with discrete scrolling probably, but can you just check that you can actually scroll with that option enabled.

yeah discrete scrolling works

Todo: make trackpad scrolling work correctly with discrete scrolling enabled.

By correctly do you mean where scrolling doesn't zoom through the inventory?

(I added this with an edit :) )
It seems the code still exists and is used, just not in 'interfaces', so I'll fix it by making it be used in interfaces also when discrete scroll is enabled.
So yeah basically.

ah ok. And by interfaces you mean screens like the inventory screen, and/or anything that has scrolling?

Yes, anything that's not scrolling on the hotbar basically, for it to be fixed without this workaround would require the other mod to accept fractional scrolling.

Fixed in 1.5


BLD: pyfolio can use newer empyrical

Also don't need special install for empyrical on travis
We don't have any pyfolio C extensions, but when we do, pip installing pyfolio should include that.

:+1:


hope can add the slowMo param to the connect function

Hi luqihao, I actually tried this already however it breaks the tests. Please feel free to try as it looks like you’re doing, but also run the tests. I think electron or something is not expecting this, but I didn’t get far into debugging why it broke.


Android Basics: Dogglers app

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-project-dogglers-app?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-project-dogglers-app#2
In which task and step of the codelab can this issue be found?
3. Get started]
Describe the problem
If you click the Grid Button, it doesn't look correct because the ViewType passed to onCreateViewHolder is 0 so depending on how you coded onCreateViewHolder you will most likely default to the wrong layout. For example I had this
    val layoutResourceId = when(viewType) {
        Layout.GRID -> R.layout.grid_list_item
        else -> R.layout.vertical_horizontal_list_item
    }

based on reading the codelab. Originally I had this version below since normally I don't like leaving things to chance and looking back  I wish I had kept it since it would have saved me some debugging time :)
    val layoutResourceId = when(viewType) {
        Layout.HORIZONTAL, Layout.VERTICAL -> R.layout.vertical_horizontal_list_item
        Layout.GRID -> R.layout.grid_list_item
        else -> throw IllegalArgumentException("Unknown viewType $viewType")
    }

The fix was very easy I just added to DogCardAdapter:
override fun getItemViewType(position: Int): Int {
    return layout;
}

Then everything was good again :)
Steps to reproduce?

Run the app
Click Grid button
See that the layout looks bad. In my case I was still using R.layout.vertical_horizontal_list_item instead of R.layout.grid_list_item.

Versions
Android Studio Chipmunk | 2021.2.1 Patch 1
Build #AI-212.5712.43.2112.8609683, built on May 18, 2022
Runtime version: 11.0.12+0-b1504.28-7817840 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 10.15.7
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 4
Registry: external.system.auto.import.disabled=true
Non-Bundled Plugins: org.intellij.plugins.markdown (212.5457.16), Dart (212.5744), io.flutter (69.0.2)
Additional information
Include screenshots if they would be useful in clarifying the problem.

Thanks. I was running into this issue too. Another solution is to use the "layout" variable instead of "viewType" as the condition for the when() statement.


Client cannot ping lighthouse, docker

I am not able to get a node in one docker container (<IP_ADDRESS>/24) to ping a lighthouse node in another container (<IP_ADDRESS>), so:
root@5d2b42dfdf67:/data# ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    (snip)
2: nebula: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1300 qdisc fq_codel state UNKNOWN group default qlen 500
    link/none
    inet <IP_ADDRESS>/24 scope global nebula
       valid_lft forever preferred_lft forever
2101: eth0@if2102: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default
    link/ether 02:42:0a:c0:0f:42 brd ff:ff:ff:ff:ff:ff link-netnsid 0
    inet <IP_ADDRESS>/26 brd <IP_ADDRESS> scope global eth0
       valid_lft forever preferred_lft forever
root@5d2b42dfdf67:/data# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
(hangs)

or the other way around
root@mfsmaster:/data# ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    (snip)
2: nebula: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1300 qdisc fq_codel state UNKNOWN group default qlen 500
    link/none
    inet <IP_ADDRESS>/24 scope global nebula
       valid_lft forever preferred_lft forever
2103: eth0@if2104: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default
    link/ether 02:42:0a:c0:0f:02 brd ff:ff:ff:ff:ff:ff link-netnsid 0
    inet <IP_ADDRESS>/26 brd <IP_ADDRESS> scope global eth0
       valid_lft forever preferred_lft forever
root@mfsmaster:/data# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
(hangs)

I have spent the last 5 hours troubleshooting, I know I've probably missed something obvious but I can't figure out what it is. From other github issues I've gathered that this setup should work fine. If someone here could have a look at the config files I'd be very thankful. If you need more information please let me know

Nebula network does not overlap with docker network or other internal network (docker is configured to use /26 pools of <IP_ADDRESS>/11)
Lighthouse port is open in router/firewall
Port forwarding the client and adding it to static_host_map as well did not solve the issue

docker-compose
version: '2.4'

services:

  lighthouse:
    image: derkades/nebula:testing
    ports: ['26486:26486/udp']
    cap_add: ['net_admin']
    devices: ['/dev/net/tun:/dev/net/tun']
    volumes:
      - type: bind
        source: ./nebula_lighthouse
        target: /etc/nebula
    networks: [lighthouse]

  client:
    image: derkades/nebula:testing
    cap_add: ['net_admin']
    devices: ['/dev/net/tun:/dev/net/tun']
    volumes:
      - type: bind
        source: ./nebula_client
        target: /etc/nebula
    depends_on: [lighthouse]
    networks: [client]

networks:
  lighthouse:
  client:

Dockerfile (nothing special)
FROM alpine AS downloader

RUN wget https://github.com/slackhq/nebula/releases/download/v1.3.0/nebula-linux-amd64.tar.gz && \
    tar -xzf nebula-linux-amd64.tar.gz || true # For some reason tar command exits with exit code 1 but it works fine

# Cannot use alpine, nebula needs GNU libc
FROM ubuntu:20.04

RUN mkdir /data
WORKDIR /data

COPY --from=downloader /nebula /bin/nebula
COPY --from=downloader /nebula-cert /bin/nebula-cert

CMD ["nebula", "-config", "/etc/nebula/config.yaml"]

client config.yaml
pki:
  ca: /etc/nebula/ca.crt
  cert: /etc/nebula/client.crt
  key: /etc/nebula/client.key

static_host_map:
  "<IP_ADDRESS>": ["mydomain:26486"]

listen:
  host: <IP_ADDRESS>
  port: 0

punchy:
  punch: true

lighthouse:
  am_lighthouse: false
  interval: 30
  hosts:
    - '<IP_ADDRESS>'

tun:
  disabled: false
  dev: nebula
  drop_local_broadcast: false
  drop_multicast: false
  tx_queue: 500
  mtu: 1300

firewall:
  conntrack:
    tcp_timeout: 12m
    udp_timeout: 3m
    default_timeout: 10m
    max_connections: 100000

  outbound:
    - port: any
      proto: any
      host: any

  inbound:
    - port: any
      proto: any
      host: any

lighthouse config.yaml
pki:
  ca: /etc/nebula/ca.crt
  cert: /etc/nebula/lighthouse.crt
  key: /etc/nebula/lighthouse.key

static_host_map:
  "<IP_ADDRESS>": ["mydomain:26486"]

lighthouse:
  am_lighthouse: true
  interval: 30

  hosts: []

listen:
  host: <IP_ADDRESS>
  port: 26486

punchy:
  punch: true

tun:
  disabled: false
  dev: nebula
  drop_local_broadcast: false
  drop_multicast: false
  tx_queue: 500
  mtu: 1300
  unsafe_routes:

firewall:
  conntrack:
    tcp_timeout: 12m
    udp_timeout: 3m
    default_timeout: 10m
    max_connections: 100000

  outbound:
    - port: any
      proto: any
      host: any

  inbound:
    - port: any
      proto: any
      host: any

client log
It only shows "message sent" not "message received", is this normal?
time="2020-11-16T16:02:22Z" level=info msg="Firewall rule added" firewallRule="map[caName: caSha: direction:outgoing endPort:0 groups:[] host:any ip: proto:0 startPort:0]"
time="2020-11-16T16:02:22Z" level=info msg="Firewall rule added" firewallRule="map[caName: caSha: direction:incoming endPort:0 groups:[] host:any ip: proto:0 startPort:0]"
time="2020-11-16T16:02:22Z" level=info msg="Firewall started" firewallHash=21716b47a7a140e448077fe66c31b4b42f232e996818d7dd1c6c4991e066dbdb
time="2020-11-16T16:02:22Z" level=info msg="Main HostMap created" network=<IP_ADDRESS>/24 preferredRanges="[]"
time="2020-11-16T16:02:22Z" level=info msg="UDP hole punching enabled"
time="2020-11-16T16:02:22Z" level=info msg="Nebula interface is active" build=1.3.0 interface=nebula network=<IP_ADDRESS>/24 udpAddr="<IP_ADDRESS>:26487"
time="2020-11-16T16:02:22Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:23Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:23Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:24Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:24Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:25Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:26Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:27Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:02:28Z" level=info msg="Handshake message sent" handshake="map[stage:1 style:ix_psk0]" initiatorIndex=1902636086 remoteIndex=0 udpAddr="my.ip:26486" vpnIp=<IP_ADDRESS>

lighthouse log
<IP_ADDRESS> is the default gateway
time="2020-11-16T16:04:31Z" level=info msg="Firewall rule added" firewallRule="map[caName: caSha: direction:outgoing endPort:0 groups:[] host:any ip: proto:0 startPort:0]"
time="2020-11-16T16:04:31Z" level=info msg="Firewall rule added" firewallRule="map[caName: caSha: direction:incoming endPort:0 groups:[] host:any ip: proto:0 startPort:0]"
time="2020-11-16T16:04:31Z" level=info msg="Firewall started" firewallHash=21716b47a7a140e448077fe66c31b4b42f232e996818d7dd1c6c4991e066dbdb
time="2020-11-16T16:04:31Z" level=info msg="Main HostMap created" network=<IP_ADDRESS>/24 preferredRanges="[]"
time="2020-11-16T16:04:31Z" level=info msg="UDP hole punching enabled"
time="2020-11-16T16:04:31Z" level=info msg="Nebula interface is active" build=1.3.0 interface=nebula network=<IP_ADDRESS>/24 udpAddr="<IP_ADDRESS>:26486"
time="2020-11-16T16:04:31Z" level=info msg="Handshake message received" certName=chunk_robin_1 fingerprint=a8b72155a0cd06ecde07bab31d8cbfdcc27c9231c3f9d09562af1f22707eceae handshake="map[stage:1 style:ix_psk0]" initiatorIndex=3477944642 remoteIndex=0 responderIndex=0 udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:31Z" level=info msg="Handshake message sent" certName=chunk_robin_1 fingerprint=a8b72155a0cd06ecde07bab31d8cbfdcc27c9231c3f9d09562af1f22707eceae handshake="map[stage:2 style:ix_psk0]" initiatorIndex=3477944642 remoteIndex=0 responderIndex=1230715052 udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:33Z" level=info msg="Handshake message sent" cached=true handshake="map[stage:2 style:ix_psk0]" udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:35Z" level=info msg="Handshake message received" certName=chunk_robin_1 fingerprint=a8b72155a0cd06ecde07bab31d8cbfdcc27c9231c3f9d09562af1f22707eceae handshake="map[stage:1 style:ix_psk0]" initiatorIndex=3477944642 remoteIndex=0 responderIndex=0 udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:35Z" level=info msg="Handshake message sent" certName=chunk_robin_1 fingerprint=a8b72155a0cd06ecde07bab31d8cbfdcc27c9231c3f9d09562af1f22707eceae handshake="map[stage:2 style:ix_psk0]" initiatorIndex=3477944642 remoteIndex=0 responderIndex=2671008165 udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:37Z" level=info msg="Handshake message sent" cached=true handshake="map[stage:2 style:ix_psk0]" udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:39Z" level=info msg="Handshake message received" certName=chunk_robin_1 fingerprint=a8b72155a0cd06ecde07bab31d8cbfdcc27c9231c3f9d09562af1f22707eceae handshake="map[stage:1 style:ix_psk0]" initiatorIndex=3477944642 remoteIndex=0 responderIndex=0 udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:39Z" level=info msg="Handshake message sent" certName=chunk_robin_1 fingerprint=a8b72155a0cd06ecde07bab31d8cbfdcc27c9231c3f9d09562af1f22707eceae handshake="map[stage:2 style:ix_psk0]" initiatorIndex=3477944642 remoteIndex=0 responderIndex=2914711756 udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>
time="2020-11-16T16:04:41Z" level=info msg="Handshake message sent" cached=true handshake="map[stage:2 style:ix_psk0]" udpAddr="<IP_ADDRESS>:11650" vpnIp=<IP_ADDRESS>

Your nebula configs look correct.
Can those containers ping each other on <IP_ADDRESS> net? This appears to be a docker networking issue, and not a Nebula problem.
I'm familiar enough with Docker, but just out of curiosity, what is the purpose of downloading the binaries with alpine and then flipping over to Ubuntu?
Side thought: have you tried running with "sudo docker run --privileged  just to test?

So I set this up to test because I've been meaning to do it anyway, are you doing some manual steps here after you run your docker-compose?
Because as far as I can tell, the actual client/server config files aren't getting pushed into either container.
It also looks like your compose file is creating two networks. This is most likely what you've been kicking rocks over.

I have given up on this so I'll close this issue to keep your issue list clean.
Regarding the two networks, that should work because the client<->lighthouse communication should go externally (the lighthouse port is exposed and port forwarded). I intentionally put them in two separate networks to test this without having to put it on two different machines. Maybe it can't hole-puch properly with NAT loopback?


Checkbox issues

Checkbox doc says to use form field as FormField, but it's actually Formfield. This caused an issue here
Edit: Copy+pasting in the code snippets results in checkbox labels bottom aligned. Not sure why.
Edit: Also if you click the label instead of the checkbox, the pink animation does not go away until you click again. This happens on the demo pages.

This is unrelated, but it seems like the buttons do not have a default margin. Is that expected behavior?

a.) Yes the Formfield should be fixed
b.) Pink animation bug is open with mdc-web
c.) We don't really have any call on css mdc-web owns it completely
c.) Can you explain Layout Grid with some screenshot?

I would think it is supposed to change vertical alignment right? It doesn't seem to do anything in the examples. Thanks!

Update: this bug is due to the fact we were not giving height to the row. will update the demo.


UDP scanning requires cap_net_raw priv

Raw sockets generally require root or CAP_NET_RAW capability.  There was a similar q on scapy here -
https://stackoverflow.com/questions/22421290/scapy-operation-not-permitted-when-sending-packets
Perhaps a note in the README that says you need elevation... at the least you should put an exception in, or you get something like:
./piescan.py -sU -t <IP_ADDRESS> -p 623

	  piescan v1.1 -- https://www.twitter.com/@_g0dmode
	---------------------------------------------------
        ip_and_port = q.get()
	    A simple, fast, lightweight TCP/UDP scanner

[ 10:32:29 - 18/06/2019 ] Scan started - Host: <IP_ADDRESS>

Exception in thread Thread-1:
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 801, in __bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 754, in run
    self.__target(*self.__args, **self.__kwargs)
  File "./piescan.py", line 273, in sniffer_thread
    sniffer = socket.socket(socket.AF_INET, socket.SOCK_RAW, socket.IPPROTO_ICMP)
  File<EMAIL_ADDRESS>line 191, in __init__
    _sock = _realsocket(family, type, proto)
error: [Errno 1] Operation not permitted

Yeah thats right, happy to take a pull request dude.

done.


In API v1.0, Security Considerations is optional

The Security Considerations section is optional. You can take it out if you aren't going to use it. If at some point you want to apply for an IANA registration, they'll look for that section. I can point you to examples.

done ... see PRs #73, #74, #78, #80


MacOS "Traffic Lights" Appearance is Incorrect

Current:

What they should be:

Here's what is needed in codepen as a solution for electron:
https://codepen.io/atdrago/pen/yezrBR

+1
We discussed dropping in a library for this at some point.


[Relay][Frontend][BugFix] Fixed relay grouped convolution selection

I was experiencing an error, where grouped convolution didn't work for groups!=num_input_channels.
It turns out that in python/tvm/relay/op/nn/_nn.py, if groups!=1, then depthwise convolution is used.  This is incorrect, since a depthwise convolution is groups==input_channels.
This pull request attempts to fix the issue.
However, note that the attrs object does not have a property for input_channels, only output_channels.  The latter is what the fix uses. This does not affect correctness in the current implementation, since depthwise convolution always has input_channels==output_channels.
However, if depthwise convolution has a depth_multiplier parameter, where N filters are applied to each input channel, then the current bug fix would not be correct.  As it stands, depth multiplied depthwise convolutions do not seem to be implemented right now.
This Jupyter notebook demonstrates the problem that the PR fixes.

@Wheest ,

Just my small hint regarding input_channel to see how it's done elsewhere, derived from tinfo[0]:     https://github.com/dmlc/tvm/blob/2bf666601d3ebefef7f55e39a1e7fd99f2c4a910/topi/python/topi/x86/conv2d.py#L330-L331

Have tried using the OpenCL backend with grouped convolutions, using the example in the notebook gist.
However, this backend only works with groups==1 and groups==in_channels.  All others fail with
  File "/home/wheest/tools/tvm/src/op/tensorize.cc", line 336
TVMError: Check failed: Equal(lhs, rhs): Failed to match the compute with TensorIntrin tensor_intrin's declaration  provided= reduce(combiner=comm_reducer(result=[(x + y)], lhs=[x], rhs=[y], identity_element=[0]), source=[(int32(x(rc))*int32(y(rc)))], axis=[iter_var(rc, Range(min=0, extent=4))], where=(uint1)1, value_index=0), intrin=  reduce(combiner=comm_reducer(result=[(x + y)], lhs=[x], rhs=[y], identity_element=[0]), source=[(int32(x(rc))*int32(y(rc)))], axis=[iter_var(rc, Range(min=0, extent=4))], where=(uint1)1, value_index=0)
During handling of the above exception, another exception occurred:

TVMError: Check failed: Equal(lhs, rhs): Failed to match the compute with TensorIntrin tensor_intrin's declaration  provided= reduce(combiner=comm_reducer(result=[(x + y)], lhs=[x], rhs=[y], identity_element=[0]), source=[(int32(x(rc))*int32(y(rc)))], axis=[iter_var(rc, Range(min=0, extent=4))], where=(uint1)1, value_index=0), intrin=  reduce(combiner=comm_reducer(result=[(x + y)], lhs=[x], rhs=[y], identity_element=[0]), source=[(int32(x(rc))*int32(y(rc)))], axis=[iter_var(rc, Range(min=0, extent=4))], where=(uint1)1, value_index=0)
Error during compile function
-----------------------------
v0.0.1
%1 = fn (%p0: Tensor[(1, 512, 28, 28), float32], %p1: Tensor[(512, 256, 3, 3), float32], __dict__=meta[StrMap][0]) -> Tensor[(1, 512, 26, 26), float32] {
  %0 = nn.conv2d(%p0, %p1, groups=2, channels=512, kernel_size=[3, 3]) // ty=Tensor[(1, 512, 26, 26), float32]
  %0
}

This happens after the Relay schedule phase, though I'm still tracing what is going wrong.  Pointers from codebase aficionado appreciated.

@Wheest The NCHWc template that uses tensorization is only for CUDA. I believe this caused the error.
Please also check if PR #3070 fixed the issue of group conv here

Yes @vinx13, the PR #3070 fixed the issue, and is a superset of all  this PR set out to achieve, so am closing it for now.  Thanks for the guidance all!


Update 1.0.2 does not install in pimatic

Updating node-red in pymatic gives the error
error [pimatic]: Error running "./node_modules/pimatic/ppm.js install<EMAIL_ADDRESS>--unsafe-perm"
Removing the pimatic-node-red folder from the node_modules folder  and restarting pimatic does not fix the problem.

I need a bit more information.

Do you have a log file?
From which version were you trying to update?
I assume you are using the update function using the pimatic web UI?

I need a bit more information.

Do you have a log file?
From which version were you trying to update?
I assume you are using the update function using the pimatic web UI?

Hi Jos
Yes I'm using the update function from pimatic.
I added the log file (renamed to txt because of upload ;-)
npm-debug.log.txt

Solved in 1.0.3. See also: https://forum.pimatic.org/topic/2085/node-red-for-pimatic-available/17

Hi Jos
Update workes OK now.
Thanks


only fetch pubkey in case we want to build on remote

This is a bit faster and also helps to mitigated cases where additionally options that are not respected by ssh-keyscan (i.e. JumpHost) are used

@mergify queue


travis builds (only) produce SEGFAULTS

But that means, binaries in releases crash.

was due to clang ppa for ubuntu bionic (travis).
Resolved by using docker for the build


[MM-38350] - clean-up: folder structure rework

Summary
this PR aims to improve the folder structure of the global header implementation.
The idea behind the structure I am proposing is to separate the global header in 3 sub-components:

LeftControls
CenterControls
RightControls

Each of them has their own unique set of items inside, depending on the app that is currently active.
Ticket Link
MM-38350
Release Note
NONE

Sorry @hmhealey there were changes sowing that came from a bad merge ... I reset the branch and merged again and this time it did work out correctly (I don't know what happened the first time). So could you please re-review? Thanks! 🙇🏼♂

Thanks @michelengelen - Global Header looks good - as before.
One issue, it's not hidden on mobile webview

.

Sorry @jgilliam17 ... that was a fix that did not make it into this branch as of now ... I have added it from #8490 ... It should work now! 🙇🏼♂


[Question]: Is Process Hacker a focus-stealing detection program?

It says here https://en.wikipedia.org/wiki/Focus_stealing#Focus-stealing detection programs:

Focus-stealing detection programs
Programs have been written to identify what is stealing focus.

Process Explorer

I cannot find that feature in Process Explorer.
Do you remember/know where is it?
Does PH have that too?
If so, where?

Does PH have that too? If so, where?

The processes tab and/or windows tab?

I cannot find that feature in Process Explorer.

The person who wrote it probably used Process Explorer to find a program stealing focus therefore Process Explorer is a tool for focus-stealing detection.


bug: fix #15

Fixes #15 by adding empty cells to the word Doc for empty table rows in the HTML.

Clsoing, merge #23 instead as it has two fixes.


Design Ideas

Just saying, there's quite a few visions for how to dramatically improve the visuals of Trello.
https://dribbble.com/tags/trello

Doesn't appear to be in consideration. I'm going to close this as I hate to leave issues open.


Google Tag Manager layout addition

Introduce Google Tag Manager support into technical documentation services by adding a new configuration option in the config/tech-docs.yml file.
gtm_id: GTM-xxxxx
The documentation changes are within https://github.com/alphagov/tdt-documentation/pull/126

Hey @lfdebrux,
We're really keen to use GTM on our docs-as-code sites. This PR seems to do just that (thanks @grahampaulcook :pray: ).
I think the only conflict with the latest, greatest code is a single line in version.rb. If we could get this resolved would you consider this for inclusion?
Many thanks,
Edd


Force download all images on click event

Are you reporting a bug or requesting a feature?
Requesting a feature
Is there a way to force download all images on click event?
Thanks!

Hi @umesh-velir,
thank you for this request. Could you clearify two of your sentences?

I could not use aspect ratio approach

Are you referring to the tip&trick I wrote in the tips and tricks section, about occupying vertical space? If yes, can I ask you why can't you use that?
The "load all images" feature shouldn't be hard to implement in the script, it's just a loop and a function call, but I fail to imagine a case where this could be useful to anybody. In other words... you need to convince me :)

Hi @verlok! What if you're trying to force download all images in a carousel.

Hi @gyoungbe,
LazyLoad shouldn’t be used in carousels,  since carousels don’t use scroll events but they move the elements in their own way.
 Would there by any way to do that programmatically @verlok ?

Yes, you can just select all lazy loaded images via jQuery or document.querySelectorAll, cycle them in a loop and copy the attribute from the data-src to the src, and to the same with the srcset. Do you know how to do that or you need a hint?

Of course I can develop a public method on LazyLoad which immediately loads all the images. It would be simpler to you, wouldn’t it.

That would be great! Massive thanks!

@umesh-velir, you're served.

You now have a method called loadAll which you can use to force download all the images.
Please find the code and the demo in the demos/load_all.html demo.

Thanks @verlok!! 👏

You got them!