Input improvements

I haven't seen @SamProf on here in at least 2 weeks. I hope all is well.
I will just keep adding my commits to this pull request until there is more merge activity on this repo.

MatIconButton - Added OnClickStopPropagation
MatButton - Added OnClickStopPropagation
MatDatePicker - Do not display the time in the text field by default. Respect the Format and EnableTime parameters.
MatDatePicker - Fixed the labeling of the DisableMobile Demo
MatDatePicker - Fixed Minimum and Maximum implementation by comparing only dates if EnableTime is false, and comparing dates and times if EnableTime is true
MatNumericUpDownField - Added FieldType parameter with Numeric, Currency, and Percent types. Percent was needed as a special implementation because the .NET percent format string multiplies the value by 100. This means 0.45 displays as 45%. So I had to hook into the parsing so that when you entered "45", it was converted to the value 0.45. Updated the demo to reflect the new FieldType
MatBlazorSwitchT - Merged PR #449 [Add MatBlazorSwitchT implementations for nullable primitive types Resolves #365]
package.json - Fixed invalid structure and updated some packages to address some of the vulnerabilities identified in audit

I just @fire-birdie comment that "I ran into this problem while trying to bind MatNumericUpDownField to an int? field. Looking at the 2.0.0 release notes makes it seem like the decision to omit support for non-decimal nullable types was intentional, but I can't figure out why. I'll prepare a PR to add support for the other primitive types, I just hope it doesn't break anything..."
I needed int? support as well, so I merged that PR. If it causes problems, can we just keep int? support without breaking other things?

Thanks for contributing Ron!!

I'm trying to get to production quality on inputs and tables. More to come!

I just made the last update to this PR. I was going to make separate PRs, but then I realized that the updates built on each other, so it was easier to just update this branch. Feel free to review and merge.

I am ill, in a 1-3 days I will return =)

Thank you for your big contribution!
I need time to check this PR. I think tomorrow I will check and will make new release with this changes

Thank you for this great repo! It has been very useful to me, so the least I can do is contribute back.

@RonPeters Please fill out a comment on this issue: https://github.com/SamProf/MatBlazor/issues/474   I think you might be a contributor.

Thank you. Visually it's good job.
can you fix merge conflicts here?
'
src/MatBlazor.Demo/Demo/DemoMatTextField.razor
src/MatBlazor.Demo/Demo/DemoNumericUpDownField.razor
'

@SamProf Thanks, I was just about to merge the conflicts.


don't automatically add favorites in SimpleFS operations

This would affect keybase fs too, but as John suggested we can just add keybase fs favorites add/ignore.

Code looks fine, but should we wait to merge it until we have the explicit add/remove calls as well and are calling them in the right places in the GUI?

Sure; I opened the PR partly as a continuous discussion. Now that it sounds good to you folks, I'm gonna add the two commands to keybase fs.

when people get a new folder notification and click into it from the GUI, it won't show up in their favorites list and they won't see it in Finder.

Good point; I didn't realize new folders don't actually get added into the list until user goes into it. It sounds like this is actually one of those situations where we should actually automatically add to favorites. WDYT?

It sounds like this is actually one of those situations where we should actually automatically add to favorites. WDYT?

What situation are you talking about?  The whole reason favorites are added on first access to begin with is because we don't want them auto-added on creation, because then people could just spam your favorites list by making folders with you.  I think we should preserve that behavior in the GUI -- if we can't do it the way we've been doing that for technical reasons related to async operations, we should do it with explicit calls.
I'm not exactly convinced that there's not a better way to handle this, by say serializing requests coming in through the GUI to make sure that folder accesses and ignores happen strictly one after another.  But I have no clue how that code works, and that might not be possible, so I defer to your judgement on the best way to do that.

Good point; I didn't realize new folders don't actually get added into the list until user goes into it. It sounds like this is actually one of those situations where we should actually automatically add to favorites. WDYT?

Have we considered just handling this in the UI? I.e. if you've navigated into a TLF that you haven't favorited, have a banner at the top that asks you to favorite it.

Have we considered just handling this in the UI? I.e. if you've navigated into a TLF that you haven't favorited, have a banner at the top that asks you to favorite it.

I don't remember discussions about that, but I wasn't in on the design meetings.  But in general it seems like a bad policy to make the UI more complicated, in order to compensate for buggy/non-deterministic code.

I don't agree that not favoriting is buggy/non-deterministic.
We currently favorite automatically upon navigation into a TLF because the filesystem doesn't have UX capabilities to allow the user to choose.
Now that we have a UI where we have full freedom in making things opt-in or opt-out, we should at least step back to reevaluate what is the right default behavior for the user.

Oh I follow your logic now, strib. You were saying that we shouldn't be looking at a UI-based solution to address the current buggy/non-deterministic ignore/favorite race.
Fair enough, but I think these two issues are in the grand scheme of things orthogonal. Yes, we could fix the buggy behavior to behave just like the FS does. But maybe this is a good opportunity to evaluate the right experience for the user.

Yep, we're on the same page now.  I'm just channeling my inner-malgorithms and venturing that the design team will favor fewer options for the user in almost all cases.  I think that's why we never had a keybase fs add command in the first place.

What situation are you talking about?

I'm talking about the flow where user sees the badged folder, and goes into the folder from GUI, where we should probably automatically add it (or better, prompt user to add it by clicking a button, but need design).

The whole reason favorites are added on first access to begin with is because we don't want them auto-added on creation, because then people could just spam your favorites list by making folders with you.

If spamming is a concern, badging is kind of the same right? I guess user can click "Ignore", but with the above proposal, I think it works the same way.

I'm not exactly convinced that there's not a better way to handle this, by say serializing requests coming in through the GUI to make sure that folder accesses and ignores happen strictly one after another.

I think there are things we can do, either in the KBFS process or in the frontend. This is not because there isn't another way to solve it, but that with GUI work, there are now ways to add/remove favorites other than through the FUSE interface, so I took it as a chance to make a change to hopefully improve the UX.
For now, it probably won't make much of a difference, but once we support opening arbitrary path in GUI, I feel like a more predictable behavior about adding stuff to favorites would be desirable.

If spamming is a concern, badging is kind of the same right? I guess user can click "Ignore", but with the above proposal, I think it works the same way.

Yeah, but as you say it's at least in a place where ignores can happen, and it feels better to have the spammy stuff separated from the regular stuff with a label/badge, which isn't possible in the favorites list in the CLI/finder.

I took it as a chance to make a change to hopefully improve the UX.

I disagree that it would improve the UI.  It seems better to just do the expected thing under the hood, without bothering the user to make a decision about it. Just personal opinion though.

For now, it probably won't make much of a difference, but once we support opening arbitrary path in GUI, I feel like a more predictable behavior about adding stuff to favorites would be desirable.

I don't understand this part about opening arbitrary paths.

Just to be clear, the proposal is to:

not add to favorites in any of simpleFS operations;
if a folder is badged (newly created), add it to favorites automatically when user goes into that folder in GUI, or access it from keybase fs. This is what I meant by the situation where we automatically add favorites.

With above two points, it's actually completely the same for GUI now, since at this time the only way for user to enter or interact with a TLF is through a TLF list, which for now we include the favorited TLFs, and badged new TLFs.  So By combining above two points, we are not changing any behavior. The difference comes when we support opening arbitrary KBFS path in GUI.
For example, we can start to make stuff like /keybase/private/strib,songgao/wifi-password into a link, and when user clicks it, we show that path inside GUI's File's tab. This would make it possible to navigate into a TLF without going through a TLF list, so it's possible to (in GUI) interact with a TLF that's neither favorited nor badged. With the above 2 points, this wouldn't result in favoriting the TLF. But with what we have in master now (i.e. before this PR), it would favorite the TLF.

Ah and I just saw the discussion above you had @strib @jzila . Somehow GitHub didn't show me or bring them to my intention earlier. Sorry for missing them.
Re whether to have a banner/button to ask user to add a non-favorited TLF to favorites: I don't think either way it'd make a big difference now because the only way user goes into a TLF for now is through the TLF list that we show, which has favorited TLFs and badged new ones. So perhaps we can probably bring it up with the design team and see how they think?

Yeah basically, but not entirely the same. We'd only favorite new badged TLFs, at the time when user goes into it. But if user does stuff with SimpleFS (or an old Keybase window somehow) in an ignored (or rmdired) TLF, this wouldn't favorite them again. For normal flows, yes the behavior would be the same.

For example, we can start to make stuff like /keybase/private/strib,songgao/wifi-password into a link, and when user clicks it, we show that path inside GUI's File's tab. This would make it possible to navigate into a TLF without going through a TLF list, so it's possible to (in GUI) interact with a TLF that's neither favorited nor badged. With the above 2 points, this wouldn't result in favoriting the TLF. But with what we have in master now (i.e. before this PR), it would favorite the TLF.

Seems to me like this should favorite the TLF.  If the user interacted in a useful way with the folder, we should always favorite it so they can find it later.  That's just my personal opinion though, it should be a question for design.

Closing for archival. Not sure if this PR is relevant any more.


ImportChangeType Enum Members

Method: GetMembersAffectedByImport
API Endpoint: GET /#account_id/members/imports/#import_id/members
Change Type. Documented there should only be two: a ('add') and u ('add and update'). However test cases have run into an 'n' status, of which the meaning is unknown.

Thanks @MikeSmithDev.  Now if we can only have better docs and more updates from their team, that would be great.


Naming Conventions

Summary
With this release the app template will have the ability to define multiple conventions at the global level (apply to all Teams) and at the template level.
The global level will work in the same way that the out of the box Azure AD group naming policy works.
The template level will allow you to define a specific naming convention per teams template that you create. You can also create templates using the 'standard' template with no additional content, this way you can effectively define 'types' of teams and apply a unique naming convention.
Intended Outcome
To be able to define multiple naming conventions for Teams created through the Request-a-team app template.
How will it work
You will be able to define a single global naming convention for all created Teams e.g. GRP_Purchasing_Contoso. Where 'GRP' is the Prefix, 'Purchasing' is the team name and 'Contoso' is the company AND a template level convention.
There will be the ability to use the following user attributes for the Prefix and/or Suffix:

Department
Company
Office
StateOrProvince
CountryOrRegion
JobTitle

Text can also be used in combination with these attributes to achieve a naming convention similar to the above.
Global naming conventions will be defined in the 'Team Request Settings' list.
Template naming conventions will be defined in the 'Teams Templates' list.

I see support for naming conventions as a high value feature for customers, this is regularly requested by customers that I have demonstrated this solution to.

@alexc-MSFT , I see that you moved this into "Completed".
Where is the documentation on how to implement/leverage this feature?  I cannot find "prefix" anywhere in the Deployment Guide?
Is it possible to customize these prefixes to leverage our own SharePoint lists as opposed to Active Directory?
Thank you in advance.


Always copy the mesh when a Mover is instantiated

Helps to avoid weird bugs where the original mesh is changed.

Closed by completion of #48.


[DO NOT MERGE] feat: add amendments list page

High Level Overview of Change

The Explorer will roll out new features to include voting data (amendments + fee) soon. This page will be one of the new feature added.
Add a amendments page to display all amendments in voting and enabled.
Note that the links to specific amendment summary page (name, id) is not yet enabled (still white text). It will be updated once the amendment summary page has been created.
Please do not merge this PR until the validator-history-service has been updated with voting data.
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Translation Updates
[ ] Release

Before / After

Desktop View

Mobile View

Add Amendment Summary page for each individual amendment

What's the difference between "voters" and "consensus"?

What's the difference between "voters" and "consensus"?

Voters is the number of UNL voters who voted 'Yea', while consensus is voters/number of UNL validators. You can tell the consensus by voters and vice versa, I just follow the design here

Can we move the Name column a bit closer to the Voters column? Right now it looks very close to ID and there's a lot of space in between Name and Voters

Can we move the Name column a bit closer to the Voters column? Right now it looks very close to ID and there's a lot of space in between Name and Voters. We can also increase the content area for the ID column as well by a few characters to balance it out

@bugsbunnies I have updated the distance between ID and Name (see the updated screenshot). We do not want to show a lot of ID column since it is insignificant and can be distracting (similar to what xrpscan does)

Not really a fan of the navigation change.  I think the "Network" navigation item should not be collapsed with "Explorer".
Instead i would like to see:

Explorer
Network

Nodes
Validators
Upgrade Status
Amendments

xrpl.org
Github

Not really a fan of the navigation change. I think the "Network" navigation item should not be collapsed with "Explorer".
Instead i would like to see:

Explorer

Network

Nodes
Validators
Upgrade Status
Amendments

xrpl.org

Github

@ckniffen the navigation has been updated as in the screenshot.


chore: make $refreshTtl optional and re-order arguments for list push and dictionary increment

Made $refreshTtl optional and default to true.
re-ordered arguments for list push and dictionary increment operations so that $trunncateFrontToSie, $truncateBackToSize, and amount are before $refreshTtl.
updated the tests to reflect the above changes.

This PR needs to get merged before working on #54.
Closes #49
Closes #50

@cprice404
As I implemented CollectionTtl, I wasn't entirely sure when to use withRefreshTtlOnUpdates.
If users want to refresh TTL on updates, it can be achieved by CollectionTtl::of(10) since $refreshTtl = true by default.
Maybe I'm missing something here.

@cprice404 As I implemented CollectionTtl, I wasn't entirely sure when to use withRefreshTtlOnUpdates. If users want to refresh TTL on updates, it can be achieved by CollectionTtl::of(10) since $refreshTtl = true by default. Maybe I'm missing something here.

Good question.  It's just there for API symmetry, in case people want to write very explicit code.


JS SDK 2.0 Low Level Design

This ticket captures the effort of producing a low level design document for the development of JS SDK 2.0 based on the research findings produced.

https://github.com/casper-ecosystem/casper-js-sdk/discussions/332#discussioncomment-8445291


UserWarning: Unknown distribution option: 'test_requires'

# pip3 install vaping
Collecting vaping
  Using cached vaping-0.3.0.tar.gz
	Complete output from command python setup.py egg_info:
	/usr/lib64/python3.4/distutils/dist.py:260: UserWarning: Unknown distribution option: 'test_requires'
	  warnings.warn(msg)
	error in vaping setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected ',' or end-of-list in whichcraft==0.4.0 ; python_version<'3.3' at  ; python_version<'3.3'

	----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-v0pduzzd/vaping/

'install_requires' must be a string or list of strings containing valid project/version requirement specifiers

sounds like pip / setuptools needs to be updated
pip install pip -U
pip install setuptools -U
then try again

You were right, as soon as i ran the second command it started working, thanks!


menu.html is not reloaded when loading newly generated appcache file

When deploying for instance ArcticWeb from branch Release-2.5 and then later replacing it with ArcticWeb from master branch, then the partials/common/menu.html seems to be not reloaded unless I actively disable caching using the Chrome developer tools.

http://stackoverflow.com/questions/24549377/how-to-configure-etag-on-nginx

Information that may be valuable to solve problem by configuring Wildfly:
https://gist.github.com/remibantos/5e86829e1ba6ad64eea1
http://undertow.io/undertow-docs/undertow-docs-1.2.0/predicates-attributes-handlers.html
https://developer.jboss.org/thread/249279?tstart=0

I've had a look into Wildfly source code. It seems, that it be default does NOT support ETag response header for static files. ResourceHandler only adds ETag response header if ETag exists for static resources (see usage of Resource reference):
https://github.com/undertow-io/undertow/blob/07d0ccb1eefb04a3b712af4e2c8f3a303081b0b1/core/src/main/java/io/undertow/server/handlers/resource/ResourceHandler.java
And PathResource always returns null as ETag value for static files:
https://github.com/undertow-io/undertow/blob/07d0ccb1eefb04a3b712af4e2c8f3a303081b0b1/core/src/main/java/io/undertow/server/handlers/resource/PathResource.java
Don't know if ETags can be added by configuration. Have asked on StackOverflow:
http://stackoverflow.com/questions/36573152/how-to-configure-wildfly-8-2-to-include-etag-response-header-for-static-files-h


renamed annotation project

Current coverage is 95.09%

Merging #1106 into master will decrease coverage by -0.02% as of 02edcd9

@@            master   #1106   diff @@
======================================
  Files           77      77
  Stmts         9310    9310
  Branches      1728    1728
  Methods          0       0
======================================
- Hit           8855    8853     -2
- Partial         90      91     +1
- Missed         365     366     +1

Review entire Coverage Diff as of 02edcd9

Powered by Codecov. Updated on successful CI builds.


Add inverted normals when flipped model scale

Solves #29317
When the user scales a 3d model by -1, the model lighting gets messed, thanks to @lyuma I could solve this problem!
In the image examples below, its a multimeshinstance with half side of the kart flipped to another!
PS: Must be cherrypicked for 3.x
Before:

After:

@Calinou I was thinking, instead of checking add it as a option in parameter called: "Invert Normals"
@clayjohn Will test and return

@Ansraer you're right, @lyuma  and me was trying to find a solution for that, I've tried tweaking a lot of parameters:
NORMAL = -NORMAL;
BINORMAL = -BINORMAL;
TANGENT = -TANGENT

but none of the two last ones changed anything sadly, this is just a partial solution to the problem!

There were some unfinished review comments. Can this be updated? It's great!


Overlapping patches

Hi
Thanks for the wonderful paper and very clear code.
I have a question. Do you think that your model will still work if we give it overlapping patches? (I need to remove healthy patches from a bag of overlapping patches of WSIs on TCGA)

Hi @hussein-jafarinia
Have you checked the image patches shared by Bin? It seems to me that the patches were already extracted with overlap as I saw the same patch repeated. I am confused about the patch extraction step so really appreciate your feedback.
Best regards.

Hi @hussein-jafarinia
Have you checked the image patches shared by Bin? It seems to me that the patches were already extracted with overlap as I saw the same patch repeated. I am confused about the patch extraction step so really appreciate your feedback.
Best regards.

By overlap I don't mean same patches repeating. I mean for example a patch having 50% overlap with other patches in it's horizontal and vertical axes (which is something that some methods on histopathology do at least for their training process)
I haven't checked the details of their tiling/patching code for training(deepzoom_tiler.py). But the result of test_crop_single.py isn't overlapping (I've run it) and based on what the paper says and shows in it's figures it seems like their patches aren't overlapping.

@hussein-jafarinia
Seems you are talking about extracting patches for only one magnification. I am interested in the multiple magnifications. If you check some of the files in the provided patches for TCGA, you will see the higher magnification patches are indeed repeated within the lower magnification folders.

it may be not work well in overlapping patches,  the same eara will influence the distance between the different patches.


Contact us page logo is not linked with home page.

Description
Hey @vivekvardhan2810 please assign me this task so that I can start working on this.
https://github.com/user-attachments/assets/84dca000-38dd-464f-9f1d-a2718ed3153d
As every page logo is linked with home page but as mention in screen recording contact us page is not linked with home page
Steps to Reproduce
When you open contact us page and wish to go to home page by clicking on website logo , it will not redirect you you to home page .
Expected Behavior
In contact us page website logo should navigate end user to home page.
Actual Behavior
In contact us page website logo is not  navigating end user to home page.
Screenshots
https://github.com/user-attachments/assets/a65505ce-cb57-4e95-8fd0-dc672d03c56a
Additional Information
No response
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@sneha056 the screenshot or the video that you have sent is of courses page not contact Page
Please check it and Update it.

Sorry for inconvenience, here's the correct video
https://github.com/user-attachments/assets/bf86f5ac-4ded-4844-9e33-a51f3248e5e8

@sneha056 Assigning this issue to you.
Make Sure to complete it within 2 days.


The exact parameter has no effect, as if it does not exist

I use the following system for my project：
 "axios": "^0.19.2",
    "immutable": "^4.0.0-rc.12",
    "react": "^16.13.1",
    "react-dom": "^16.13.1",
    "react-redux": "^7.2.0",
    "react-router-dom": "^5.1.0",
    "react-scripts": "3.4.1",
    "react-transition-group": "^4.3.0",
    "redux": "^4.0.5",
    "redux-immutable": "^4.0.0",
    "redux-thunk": "^2.3.0",
    "styled-components": "^5.0.1"

The routing configuration is as follows：
      <Provider store={store}> {/*Provider会自动把storer提供在他内部的组件*/}
        <BrowserRouter>

          <Switch>
            {/**<Route exact path="/"  render={()=>
home
}></Route>*/}
            <Route exact path="/" component={Home}></Route>
            <Route exact Path="/detail"   component={Detail}></Route>
          </Switch>
        </BrowserRouter>
      </Provider>

But now, when I visit any of the following addresses in the browser, I will enter the detail page. It seems as if the exact parameter does not exist
http://localhost:3000/detail/2
http://localhost:3000/detail
http://localhost:3000/det
http://localhost:3000/detdsafasdasdf
http://localhost:3000/detail/2aaa

Why is this? Thank you.

The same is true for the react-router-dom version changed to 5.1.2.

You have a capital P on your path prop. They must be lowercase.


Add focus variant details to the genome browser page sidebar and drawer

Description

Added types for expected variation api response based on these two PRs in the variation data model: https://github.com/Ensembl/ensembl-vdm-docs/pull/1 and https://github.com/Ensembl/ensembl-vdm-docs/pull/2.
Added variant summary information in the track panel, and in the drawer. The information in the drawer is based on the mapping between the data and the UI provided in this table in Confluence

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-1867
Deployment URL(s)
http://focus-variant-details.review.ensembl.org/genome-browser/grch38?focus=variant:rs779377781&location=13:32357725-32357764
NOTE: the Copy button won't work on the review deployment; because the browser api that it uses only works when the site is accessed over https.

Could you please rework on the spacing?

Why is source db link not linked to rs699?
Shouldn't it be https://www.ncbi.nlm.nih.gov/snp/rs699 instead of https://www.ncbi.nlm.nih.gov/snp/

Unfortunately Copy button didn't copy content to clipboard :-(

Why is source db link not linked to rs699?

Because of the data that Variation provided. I asked them last week if they want to change this; they probably will.

Could you please rework on the spacing as per the XD?

I've added space between elements of a VCF string. As to the space between the VCF string and the copy button, the space in the XD is incorrect, because it does not take into account that when you click on the Copy button, it changes into a wider lozenge:

Andrea wanted the space between the VCF string and the lozenge to be 12 px; which means that between the VCF string and the "Copy" element, the space will be wider.
Here's what I've got after the change:

Minor, missed space here.


helm: add hubble UI support for GKE dataplane v2

To support GKE dataplane v2 the TLS_RELAY_SERVER_NAME backend environment variable needs to match the configure google managed DNS names (*.kube-system.svc.cluster.local)
This change retains the default behaviour but allows the environment variable to be configurable
Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: https://github.com/cilium/cilium/issues/28708

@gandro - Commits squashed - should be good but let me know if not.

/test

looks like the gateway tests are failing because the branch is missing examples/kubernetes/gateway/grpc-route.yaml which got added recently. we probably need to rebase 👀


Truncating of text is too aggressive

Even single lines are truncated when it should not.
https://issue-136-casino-page--caesars--hlxsites.hlx.page/caesars-palace/drafts/swbarman/casino

@arumsey In the future we'll have to either find another way for the config, or make it more robust. It's hard to get the hang of that config thing...


Feature to have tabs full width of header

When using Custom Header, the scroll "field" for the view gets a lot more narrow than usual (without Custom Header)
Latest and greatest versions of everything (HA, Custom Header and Android app)
First picture is standard HA app layout.

Second picture is with Custom Header enabled:

HA version: 0.106.5
Android App version: 1.8.0-144
Custom Header: 1.3.2
'''custom_header:
exceptions:
- conditions:
user: 232d473cf7c942019bd10f482e93e7e4
config:
disable_sidebar: true
hide_config: true
hide_help: true
hide_raw: true
hide_unused: true
menu_hide: true
options_hide: true
show_tabs: ellas
voice_hide: true
- conditions:
user: 145eac3770944160af5a689aa2f37c3e
config:
disable_sidebar: true
hide_config: true
hide_help: true
hide_raw: true
hide_unused: true
menu_hide: true
options_hide: true
show_tabs: siris
voice_hide: true
shadow: false
resources:

type: module
url: /community_plugin/custom-header/custom-header.js

2nd part of this post:
https://community.home-assistant.io/t/custom-header/155399/473
TLDR: This is a feature, but may at some point be configurable

OK, I get it.
Thanks for the info.
Can live with it for now.
I explicitly disabled "Display Tab Chevrons" at least, to get a bit more space.
But I have heard about people stopped using Custom Headers, just because of this.
Great plugin though!
Keep up the good work!

No need to close this one, going to change it to a feature request.

Added in latest version by default
https://community.home-assistant.io/t/custom-header/155399/636


How to send json and process on server

Hi, All
I need to send information in json format and render on server and return another json. I would like to leave some functions configured on the server to process this json. This is possible, I'm starting to use Thruway and I still have many doubts.

@msouzarunner - When you return data from your RPCs or publish events, that data is fed through json_encode, so you have complete control over the serialization if desired through the JsonSerializable interface.

Hi, mbonneau.
 Would you have some examples of how this works on the server side and on the client side for me to understand better?

Closing issue for inactivity


Import mnemonic uses BIP32 to create account

Codecov Report

Merging #29 into master will increase coverage by 0.27%.
The diff coverage is 25%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   48.94%   49.22%   +0.27%
==========================================
  Files          30       29       -1
  Lines         333      323      -10
  Branches       62       58       -4
==========================================
- Hits          163      159       -4
+ Misses        142      138       -4
+ Partials       28       26       -2

Impacted Files
Coverage Δ

src/vault/providers/memory.js
85.36% <ø> (ø)
:arrow_up:

src/vault/types.js
0% <ø> (ø)
:arrow_up:

src/vault/providers/rpc.js
3.44% <0%> (+3.44%)
:arrow_up:

src/vault/vault.js
65.51% <33.33%> (-6.9%)
:arrow_down:

Continue to review full report at Codecov.

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


Add source_code_uri to gemspec

What are you trying to accomplish?

Since #1472, we cannot confirm source code url from rubygems. So, it added metadata["source_code_uri"] in gemspec.
What approach did you choose and why?

Not in particular.
Anything you want to highlight for special attention from reviewers?
Not in particular.

Good idea! Could you please add a changelog entry?


Create a Unique Identity for Clear My Record CfKC/UMKC

#Introduction
As we create a unique service to deliver assistance with expungement and ancillary services,there will be a need to establish an identity to differentiate our service from other similar services, including the national "Clear My Record" service being designed by Code for America.  The unique identity should include a unique name and a logo.

Some notes on possible trademarks:
Respect Yourself - Staple Singers
Reset * reset.legal
Renew renew.legal
Restart - restart.legal available
Rethink rethink.legal not available
heart.legal
clear
clar
klar
claro
clair
sync
synch
sync.me
exp.express
exp.legal
rise.legal
Reset - set again or differently
Renew - give fresh life or strength to
Restart - start or begin again
Rethink - think again about (something such as a policy or course of action), especially in order to make changes to it
Respect
Reside - Trash Day
Vote
Reset - Clear My Record
Revive - Free the Lots
Resist
reason.legal
express.legal
reconnect.legal
Reframe
Refresh
Retain
Retake
Rethink
Retie
Retighten
Express
express.legal
Reason
Become
True
Align
Season s,@has
Not for the list
Retrain


PEFile utilities have buffering issues

In PEFile::GetPdbSignature there is a 'break' instruction which skips freeing the buffer cache
The current cache size is 1 buffer, but this method uses 2 at the same time. You may want to consider increasing your cache size to 2 to cover the common case.

https://github.com/Microsoft/clrmd/blob/master/src/Microsoft.Diagnostics.Runtime/Utilities/pefile.cs#L118

Hello!  This issue is being closed due to its age.  The codebase has moved significantly since this was filed.  Please feel free to open a new issue for ClrMD about this issue if it still reproduces on ClrMD 1.0.  (Also, I apologize for having not looked at or resolved this earlier and for resolving it now.  In order for me to be able to make some progress on the growing issue count in ClrMD I need to clear the backlog down to a manageable set of issues, but do feel free to file a new one if this is still a problem.)


Consider change repo license

This repository does not hold source code of the OpenJDK project. To ease contributions to flows and bug reports, we are considering change the license to MIT.

I approve!


clashleaders.clash.api.ApiException in GET /search.json

Error in ClashLeaders
clashleaders.clash.api.ApiException in GET /search.json
API returned non-200 status code: 503
View on Bugsnag
Stacktrace
clashleaders/clash/api.py:70 - find_clan_by_tag
clashleaders/views/clan.py:27 - search

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in GET /clan/shwe-mandalay-qv89v009.xlsx

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in GET /clan/qiang-jie-xiao-ao-jiao-lyvl8qlq.xlsx

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in GET /clan/shbb-wsby-q0rgjjlj.xlsx


Error when building in a workspace

I am trying to use this inside a workspace to compile one member which will then be called from another member, however it is failing to deserialize the Cargo.toml.
The project structure looks like:
Cargo.toml
host
 - Cargo.toml
 - src/main.rs
udf
 - Cargo.toml
 - src/lib.rs

The workspace is defined in the top level Cargo.toml, and then I try to call cargo wasi build from the udf folder. This errors with the following:
< [/code/some_project/udf] cargo wasi build
> Finished dev [unoptimized + debuginfo] target(s) in 0.04s
> error: failed to deserialize as TOML: /code/some_project/Cargo.toml
> Caused by:
>    missing field `package` at line 3 column 1

If I add [package] to the worspace's Cargo.toml I get the following error:
> Caused by:
>   no targets specified in the manifest
>   either src/lib.rs, src/main.rs, a [lib] section, or [[bin]] section must be present

Oops, should be fixed now, but thanks for the report!


Missing section number in front of the section names (html and pdf)

Ok, my index.rst uses toctree :numbered:
The generated html has a numbered toc after the cover page, but the  entries in the html have not. I attached the pdf from the demo as example. The toc is numbered, the sections in the pdf are not.
I am using Sphinx 4.5.0. Maybe this is the issue.
Sphinx-SimplePDF-DEMO.pdf
Originally posted by @kreuzberger in https://github.com/useblocks/sphinx-simplepdf/issues/19#issuecomment-1249155694

I currently have no idea there it comes from.

The normal singlehtml build works fine
commenting out the section in the simplepdf.py builder (__init__) shows the "normal" theme without section names too

       if self.app.config.html_theme != "simplepdf_theme":
            print("Setting theme to sphinx_simplepdf")

copying the "overwritten" parts in the simplepdf build from Singlehtml build (the ones included by the super()) has also no effects

@kreuzberger - Off topic, I was curious to know your configuration for the "page numbers" for top-left-content and "section titles" for top-right-content?

@sachin-suresh-rapyuta That's the default configuration.
Not used anymore for the DEMO PDF, to test certain things.
But each other PDF should start with this config.
The default config can be found in this file:
https://github.com/useblocks/sphinx-simplepdf/blob/main/sphinx_simplepdf/themes/simplepdf_theme/static/styles/sources/_variables.scss
$top-left-content: config('top-left-content', 'counter(page)') !default;
$top-center-content: config('top-center-content', '') !default;
$top-right-content: config('top-right-content', 'string(heading)') !default;

So its counter(page) for top-left-content and string(heading) for top-right-content.
These magic strings are coming from the weasyprint library.
Unfortunately I haven't found a nice overview for these magic strings yet.

Oh, that was silly to ask, 'bottom-right-content' already had this. Thanks, @danwos!

Seems the error with the section numbers comes from sphinx, where concrete name checks are used to identfy the build to add section and figure numbers. Refer to the Sphinx issue provided here

Due to the answers in https://github.com/sphinx-doc/sphinx/issues/11640 the desired way to be is to have a custom simplepdf writer (could extend Html5Translator) to get the two functions (sectionnumber and fignumber) implemented properly

Interesting and thanks for diving deeper into the root cause.
Do you have some time to create such a customized writer?
I'm currently totally overloaded with other projects, but happy to test stuff if needed.


Calling fetch to refresh a model has issues (MySQL only?)

I've just started testing my codebase against MySQL instead of sqlite3, and I'm seeing a bit of weird behaviour where calling:  instance.fetch() with an already existing instance returns a promise which gets fulfilled with null.
This is not the same as what was happening with my sqlite3 tests so I suspect there may be a bug in the MySQL implementation?  Any thoughts?
v0.7.7, btw.

The project leadership of Bookshelf recently changed. In an effort to advance the project we close all issues older than one year.
If you think this issue needs to be re-evaluated please post a comment on why this is still important and we will re-open it.
We also started an open discussion about the future of Bookshelf.js here https://github.com/bookshelf/bookshelf/issues/1600. Feel free to drop by and give us your opinion.
Let's make Bookshelf great again


Feature: Save API JSON (briefly) if requested by client

I'm using this library for archival purposes (eg. saving my favourites to SQLite) and I find it useful to keep the raw API response around after the API call since it often contains extensions, fields, etc., that aren't necessarily parsed out and returned in library objects.
e.g. from my Akkoma instance, status responses have information about quote-boosting which doesn't exist in the mastodon.Status struct.  Also various pleroma and akkoma extensions for the status text and instance information.
Since almost no-one is going to need this, it's gated behind Client.SaveJSON being true (and limited to 100MB of JSON to be safe.)  By default, the library will behave exactly the same as before.
(I admit it's not a great solution but I don't think there is one for a situation like this without extending every API method to optionally return the JSON or similar.)

I prefer to do with TeeReader.
type Client struct {
    ...
    JsonWriter io.Writer
}

if c.JsonWriter != nil {
    return json.NewDecoder(io.TeeReader(resp.Body, c.JsonWriter)).Decode(&res)
} else {
    return json.NewDecoder(resp.Body).Decode(&res)
}

client := mastodon.NewClient()
var buf bytes.Buffer
client.JsonWriter = &buf
// something to do with a client

Neat solution!  It does require the caller to remember to issue a buf.Reset() before every API call they want the JSON from though -- a definite footgun (I know because I just encountered it) and a bit untidy.  Can't do the Reset() from the library though because it's not available on io.Writer.
A localised WriteResetter interface fixes things and keeps the client code tidy.
type WriteResetter interface {
    io.Writer
    Reset()
}

type Client struct {
    ...
    JSONWriter WriteResetter
}

    if c.JSONWriter != nil {
        c.JSONWriter.Reset()
        return json.NewDecoder(io.TeeReader(resp.Body, c.JSONWriter)).Decode(&res)
    } else {
        return json.NewDecoder(resp.Body).Decode(&res)
    }

Thoughts?

Reverted to simple io.Writer with client-initiated resetting.  Think making JSONWriter a simple *bytes.Buffer and moving the reset to the library makes more sense (when would someone supply a non-bytes.Buffer as somewhere to save the JSON?) but could live with this solution.

It can check before call Write()
if resetter, ok := c.JSONWriter.(WriterResetter); ok {
  resetter.Reset()
}

Ah, cool, that was one of the interim solutions I came up with but I wasn't
sure if people liked that kind of type-checking.
I will do it that way, thanks!
On Thu, May 25, 2023 at 10:09 AM mattn @.***> wrote:

It can check before call Write()
if resetter, ok := c.JSONWriter.(WriterResetter); ok {
resetter.Reset()
}
—
Reply to this email directly, view it on GitHub
https://github.com/mattn/go-mastodon/pull/183#issuecomment-1562555301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACUKMLDHMTPXUR6TMAKWDXH4OU3ANCNFSM6AAAAAAYLPUEUI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ah, I think I got almost to that solution twice with the WriteResetter and then checking if an interface{} could call Reset().  Never checked a type assertion on an interface that's bigger than a type before.  Thanks.

Sorry my delays. Thank you.


Add profile info

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

__
Summary: 👌 This code looks good


type error on cp2k_input

Hi! I was trying to simply run mof_phase_transition.py example, however I get an error:
Traceback (most recent call last):
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/dataflow/dflow.py", line 884, in _unwrap_futures
    new_inputs.extend([dep.result()])
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/concurrent/futures/_base.py", line 451, in result
    return self.__get_result()
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/dataflow/dflow.py", line 301, in handle_exec_update
    res = self._unwrap_remote_exception_wrapper(future)
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/dataflow/dflow.py", line 573, in _unwrap_remote_exception_wrapper
    result.reraise()
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/app/errors.py", line 122, in reraise
    reraise(t, v, v.__traceback__)
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/six.py", line 719, in reraise
    raise value
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/app/errors.py", line 145, in wrapper
    return func(*args, **kwargs)
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/app/bash.py", line 47, in remote_side_bash_executor
    raise e
  File "/home/bamaz/software/micromamba/envs/psiflow_env/lib/python3.10/site-packages/parsl/app/bash.py", line 33, in remote_side_bash_executor
    executable = func(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/psiflow/reference/_cp2k.py", line 201, in cp2k_singlepoint_pre
  File "/opt/conda/lib/python3.10/site-packages/typeguard/__init__.py", line 1032, in wrapper
  File "/opt/conda/lib/python3.10/site-packages/typeguard/__init__.py", line 875, in check_argument_types
TypeError: type of argument "cp2k_input" must be str; got dict instead

I've tried to get to where the source of this problem is, but despite the beautiful documentation, I've failed... ☹
Is the fault on the psiflow side or is it a cp2k issue? Any idea how to fix it?

Hi Bartosz, which version of psiflow did you install in your local micromamba environment, and which container are you using? We had to update the CP2K interface at some point because Pymatgen's parser couldn't deal with fluorine atoms (F -> FALSE), and I'm thinking that there was a small fix for an error like this right after we released 3.0.4.
Can you try installing psiflow directly from the main branch ( pip install git+https://github.com/molmod/psiflow) while using the 3.0.4 container (at oras://ghcr.io/molmod/psiflow:3.0.4_python3.10_cuda)?
That being said, we are planning a major overhaul of psiflow in the next few weeks, which will introduce a large number of new features but more importantly streamline the API and reduce possibility of errors like this one. We're currently finalizing this on the ipi  branch. While the API will be very similar, it should support many more use cases (path integral MD, replica exchange, thermodynamic integration, ... ). See here for some examples.

Hi Sander, thanks for the quick response! I was using the latest version of psiflow, while I had to copy the link to the old version of the container (3.0.0). I have corrected this and it appears that everything is now working correctly!
Happy to hear that psiflow is developing so fast, it looks like a very promising tool. I will update if I encounter any problems.


Move update, collision and logic to a class character

This change allow us to reuse the collision and update code

Coverage decreased (-0.04%) to 5.742% when pulling d03703b6a9ef754d2c53c9a3f4383e03032c846c on Hispar:master into 6e92e528f2758abf77031b92f31e0ae9a35af54e on traverseda:master.


How to add a header row to excel

Is that possible to add a header column (with e.g. a title) upfront exported data?
We can assume, that we talk about excel file.

You can modify the tablib dataset in the after_export() method.
More info here.

According to my research it's not what I'm looking for.
Upper hint allows to modify a data content.
What I need to do is:

Add a row before headers (center it) -> e.g. with constant value "Hello world"
Increase column width, but according to this ticket it's not possible
Modify exported file name

Do you see any solution for my needs?

Add a row before headers (center it) -> e.g. with constant value "Hello world"

You can only do this by modifying the spreadsheet directly using openpyxl.  You should be able to override hooks to modify data as described here.

Increase column width, but according to ticket it's not possible

It would be possible with openpyxl.

Modify exported file name

You can do this by overriding get_export_filename()

Thank you, it's clear now :)


Namespace specifications

On stand we discussed how apps might reuse / share information.
Right now, several apps look into in the wavi namespace to pull public or basic profile information such as name, profile pic, etc.
This should really be renamed to something more generic such as profile or public.
### Action items
- [ ] Migration code for apps using the .wavi namespace to move data to .profile (or whatever it ends up being called)
- [ ] Specification document for generic namespaces - .wavi/.profile, .contacts, etc.
- [ ] Open source the specification and make it readily available for contributions - could be part of at_protocol, could be it's own repo

I thought of a cool way to do this, the atSign that owns the namespace could publish a spec (of some yet to be determined format) to the .namespace namespace:
For example the wavi namespace could be found at:
public:spec.namespace@wavi
By anyone who wishes to use it.


[1.10.2] /profile c Not profiling chunks, tries to do an entity count instead

Hi there, we're using the latest 1.10.2 version of TickProfiler on our servers (jenkins59), but one thing we seem to be missing is the ability to profile single chunks. From the doccumentation it looks like /profile C used to do this, however it does not work in the latest version. Was the ability to profile an individual chunk intentially removed?
I see an open issue for the 1.12 version for the same thing but nothing yet for the 1.10 version.

It's implemented again for 1.12 now in this commit: 17952a525012938fb4cc716bb
I need to backport that commit to 1.10.2.


var.id in monsoon_wang diagnostics needs to be set, also, obs results saved

@lee1043 Will try to have this done and rerun early Weds AM.

@gleckler1 no problem. I got a way to grep the var.id as random number, so no rush on this. Necessity of OBS is actually higher priority, but I can mimic one of model as OBS just for code developmental purpose. Thanks!

@lee1043 @gleckler1 can i help on this?

@doutriaux1 @lee1043 Thanks CD, I am working on this one now.   You and I need to talk about how to host data to run "examples" ...

@gleckler1 does pcmdiweb.llnl.gov have a webserver open on the green?

@doutriaux1 yep, that machine is hosting https://pcmdi.llnl.gov

@doutriaux1 and is where we will host plots from the PMP and supporting data (PMPobs, data to run examples, etc).

@lee1043 the beginnings of our new obs is at:  /p/user_pub/pmp/pmp_obs/PMPObs/pmpobs1-5-1/atmos/mon/pr/GPCP-2-3/gn/v20180706
.... however, what you need is the annual cycle climatology of this data.  You can use as a place holder /work/gleckler1/processed_data/obs/atm/mo/pr/GPCP/ac/pr_GPCP_000001-000012_ac.nc
I will start working on the annual cycle climatolgies of the new database.

Hi Jiwoo,
Some updated results for monsoon_wang are in:
/p/user_pub/pmp/pmp_results/tree_v0.3/pmp_v1.1.2/diagnostic_results/monsoon/monsoon_wang/cmip5/historical/092618
And
/p/user_pub/pmp/pmp_results/tree_v0.3/pmp_v1.1.2/metrics_results/monsoon/monsoon_wang/cmip5/historical/092618
This should include observations in both, and an MPI_area_average in the metrics.
Let me know if/what additional problems you find.
Thanks,
P
From: Jiwoo Lee<EMAIL_ADDRESS>Reply-To: PCMDI/pcmdi_metrics<EMAIL_ADDRESS>Date: Tuesday, September 25, 2018 at 11:56 PM
To: PCMDI/pcmdi_metrics<EMAIL_ADDRESS>Cc: gleckler1<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [PCMDI/pcmdi_metrics] var.id in monsoon_wang diagnostics needs to be set, also, obs results saved (#577)
@gleckler1https://github.com/gleckler1 no problem. I got a way to grep the var.id as random number, so no rush on this. Necessity of OBS is actually higher priority, but I can mimic one of model as OBS just for code developmental purpose. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PCMDI/pcmdi_metrics/issues/577#issuecomment-424606269, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEV6rUNPo5RbI4BHtytiPPzsJnsPG1YTks5ueyUngaJpZM4W5qXx.


mysql flexible creation using Bicep {"code":"InvalidParameterValue","message":"Invalid value given for parameter administratorName. Specify a valid parameter value."}

Hi Guys,
Im getting this issue for mysql server creation with bicep latest api. I'm giving the administrator name as user assigned managed Identity name {"code":"InvalidParameterValue","message":"Invalid value given for parameter administratorName. Specify a valid parameter value."}. Pls help.

@Kosaln
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

@Kosaln
This GitHub case is similar to a previous issue that you raised. Our content team is currently working on it, but it will take some time to update the documentation. Now I am going to close this issue as Duplicate of https://github.com/MicrosoftDocs/azure-docs/issues/124887


drop lexical-core dependency

Closes #49
Let's see how this affects benchmarks...

Well, this works but it does come at a performance regression. Potentially one solution is to use minimal-lexical, but that doesn't seem any better than using any of the rest of the (seemingly unmaintained) lexical family.
Another option is to add our own implementation of float parsing and not use either std or lexical.

I think it looks like serde has extracted the bits of lexical that it needs into their own package.
I think we should either:

use a third party package
see if serde makes the methods we would need public, and use them?
copy and paste the bits of lexical we need into here - this might mean we could reuse the part of the float before the . which we've already parsed

Just adding a direct cross-reference to related PR https://github.com/pydantic/jiter/pull/57.

Yes, let's close this for now, can always revisit later.


DicePoker

Thanks for all the great work!
I like the pull-request format, I will try it for my contributions as well.


Fixed issues with the Fedora build

Fedora builds were failing whenever there were package updates via yum (due to waiting for user input), and also due to LLVM not being compiled with RTTI enabled. With these changes, builds have been successful this morning.

This looks good.

Ok after talking with @spoonincode and @pacificcode etc, I think we need to document the need for an environment variable for LLVM_DIR as part of the fedora build that links together our web assembly compiler and the libraries we link to for JIT/ABI generation
BIG NOTE
This link is something we want to remove in the future.   We should be able to upgrade the LLVM version for our WebAssembly toolchain and the LLVM version that backs our runtime JIT/tools independently.


feat(imagehotspots): add selecting and creating hotspots

Closes #1770
Summary
The ImageHotspots is enhanced to support selecting hotspots for editing and adding new ones by clicking on the image. It is still possible to drag the image when in isEditable mode which makes sense for precision positioning of new hotspots.
Change List (commits, features, bugs, etc)

Added isEditable prop to ImageHotspots.jsx inorder to enable selection and adding hotshpots
Added callback for when a hotspot is clicked (selected) in the ImageHotspots.jsx
Added callback for clicking the image to create a new hotspot ImageHotspots.jsx
Changed the drag code to start drag on mouse move instead of mouse down in ImageHotspots.jsx
Added isSelected to Hotspot.jsx
Refactored the StyledComponents into external css for Hotspot.jsx
Added a story
Added unit tests

Acceptance Test (how to verify the PR)

Go to https://deploy-preview-1813--carbon-addons-iot-react.netlify.app?path=/story/watson-iot-imagehotspots--editable
Try selecting exiting hotspots. Make sure they don't move.
Try adding new hotspots by clicking on the map.
Zoom and make sure you can stil drag

Note!
Clicking on hotspots doesn't always show the tooltip. This is an existing bug for which I filed a issue #1808
Apparently the same issue also prevents the selection from happening sometimes here. It works in dev mode so I assume it is a timing issue. I will look into that.

:tada: This PR is included in version 2.141.0-next.24 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.141.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG] libcamera-vid adding "-t ##" causes tcp server to disconnect

Server: Running on a RPiZero2W:
 date; camera-bug-report -c "libcamera-vid  -t 30 -n --inline --listen -o tcp://<IP_ADDRESS>:5000"

Client: Running on a Dell Inspiron (over wifie) running Gentoo:
 date; ffplay tcp://<IP_ADDRESS>:5000  -vf "setpts=N/30" -fflags nobuffer -flags low_delay -framedrop -loglevel debug ; date;

I launch server and it waits for a connection.
I launch client and it connections and then connection is terminated with:
Invalid data found when processing input

Attached Server log: 1495 lines in bug-report_server_20220210202326.txt
bug-report_server_20220210202326.txt
Attached Client log: 28 lines in client_taurus_ffmpeg_debug.txt
client_taurus_ffmpeg_debug.txt
If I omit the "-t 30", the client successfully connects and the stream successfully works until I terminate either client or server.  Note: i tried to get ffplay to work for a limited duration, but for some reason that was ignored and I have to Ctrl-C to terminate this client process.  As far as I can tell, I'm using the most verbose output for ffplay.  Please let me know if there is anything I can add to the command line.  This seems like a signal is being sent from one and not interpreted by the other.

update: 9:04 PM 2/10/2022 PST
I then tried to run the server with no time limit and client with no time limit and now the server closes down after a client connection.
Here's my server command line:
  clear;date; camera-bug-report;libcamera-vid   -n --inline --listen -o tcp://<IP_ADDRESS>:5000;date

and bug report as file bug-report_server_no_wrap_202202102102.txt
Client ffmepg log as file: client_taurus_ffmpeg_202202102102.txt
I'm wondering if someone at a low level, perhaps disclosed in demesg is afoot.
client_taurus_ffmpeg_202202102102.txt
bug-report_server_no_wrap_202202102102.txt
server_run_console_log_202202102102.txt

-t 30

The units for this parameter are in milliseconds.  This is why the libcamera-vid application shut down near instantaneously after the command was run.  If you are looking to run for 30s, you need -t 30000000.

Hi, thanks for the suggestion. I think we'd be happy for it to output something like that when libcamera-vid exits, Please be sure to run the style checker (in utils/checkstyle.py) before submitting a pull request. Thanks!

If I do so, it will be in the next two weekends.  Daytime job really drains me.  Thank you.


How to use these forms in Sitefinity

Hi Tim,
How do get access to these forms through the Sitefinity CMS?  I understand how to add the project to my solution and to build it, but what do I have to do in order to get access to SuperForms in Formsbuilder?
Thanks,
Adam

Actually, I realized that I have to install the module.  That's done and now I have access to the superform controls in the formbuilder.  However, when I run the form, the conditional logic doesn't seem to take effect.  I have a textbox field that I've setup to only show on a certain value in a different dropdown field.  Unfortunately, the textfield is always visible (although it is disabled) on the form and it never gets hidden.  I think there's probably something I'm not doing correctly, but I can't figure it out.  Do I have to include a JavaScript file in the page?

Glad to hear you got it installed. To enable the additional functionality, you'll need to use the Forms Widget that is installed in the Pages toolbox. Also, please select if it is a conditional, or progressive, form from the dropdown on the second tab.
I've been dragging my feet on writing documentation but will add it to my to do list.


Cache the bookmarks locally and load those first

This is the first step in making the app more efficient. It still means we will be asking for the whole bookmarks JSON object on every load, but at least it won't be blocking the UI from rendering.

I think I did this one


Add setChatStickerSet and deleteChatStickerSet methods

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (97b9a0b) to head (41bb713).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              master       #63   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity      1235      1247   +12
===========================================
  Files            259       261    +2
  Lines           4448      4479   +31
===========================================
+ Hits            4448      4479   +31

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Mat/Pat - Bug - length of leave should not be 0

We should not allow maternity & parental leave length to be 0.
Decimal values i.e 16.5 should not be allowed
negative values should not be allowed
AC

[ ] Maternity leave length should be a whole number from 1 to 17
[ ] Parental standard leave should be a whole number from 1 to 35 if taking full 17 week mat leave or 37
[ ] Parental extended leave should be a whole number from 1 to 63 if taking full 61 week mat leave or 63

Tested all three points; confirmed working!
Moving to PO!


new side navigation elements added

Looks like a merge conflict because I merged some earlier pull requests :S do you have time to fix it?

Hey, no worries 😊
I'll look into the merge conflict.
I'm pretty new to working on open source stuff, but I'll try and get it sorted today :)

From: Benny Carlsson<EMAIL_ADDRESS>Sent: 11 October 2018 09:34
To: lingonsaft/hacktoberfest
Cc: beckyw5; Author
Subject: Re: [lingonsaft/hacktoberfest] new side navigation elements added (#1482)
Looks like a merge conflict because I merged some earlier pull requests :S do you have time to fix it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/lingonsaft/hacktoberfest/pull/1482#issuecomment-428868867, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AihiHMzQSkBEVH8qwNx0i4SWKuL-XiZ_ks5ujwKpgaJpZM4XWaLS.

Great :) we also have a video on the website on how to solve merge conflicts ;)


Add typehints in favour of PHPDocs for TransformsRequest middleware

As a minor improvement (and since the project is only targeting >= PHP 7.1) I've updated the TransformsRequest middleware to use the type hints where possible and removing the (redundant) @params and @return.
This PR doesn't touch the cleanValue method as it is overloaded and contains support for mixed types.
Feel free to reply if this PR needs rework (Since this is my 1st PR on this project)

The framework doesn't use return typehints at this time. Doc blocks are preferred. While this may change in the future, this file should be consistent with the rest of the framework.

Heya, thanks for sending in your first PR. Please see the comment above. At the moment we don't implement these type-hints in the framework.

Thnx for the feedback. However Im trying to wrap my head around why not to add these basic typehints (Especially the object type hints have been there since PHP 5, July 2004). Also reading https://github.com/laravel/ideas/issues/1409 where the majority seems to be on the pro side. But there seems to be little progress on a feature which I consider very positive for strengthening the framework and assist the programmer. As currently the responsibility lies with the programmer using the framework. I would expect the framework to help.
Btw there seems to be some typehints in the framework: https://github.com/laravel/framework/blob/5.8/src/Illuminate/Foundation/Testing/Concerns/MakesHttpRequests.php#L55

why not

I think these two things are relevant:

As the answer given in here ( https://github.com/laravel/framework/pull/28373#issuecomment-487919950 )

this file should be consistent with the rest of the framework

approval by Taylor Otwell

Basically Taylor so far hasn't gone forward merging such PRs which shows his stance is (not yet?) "pro" them.


fix convert value

fix convert value

Codecov Report

Merging #283 into master will decrease coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #283      +/-   ##
==========================================
- Coverage   83.54%   83.42%   -0.13%
==========================================
  Files           8        8
  Lines        1939     1937       -2
==========================================
- Hits         1620     1616       -4
- Misses        240      241       +1
- Partials       79       80       +1

Impacted Files
Coverage Δ

vm/vmConvertToX.go
85.98% <100%> (-1.64%)
:arrow_down:

vm/vmLetExpr.go
84.74% <100%> (-0.34%)
:arrow_down:

Continue to review full report at Codecov.

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

Thank you very much for finding this issue and coming up with a fix.
There was a few changes in the fix so instead of going over all the changes just made a new PR.
https://github.com/mattn/anko/pull/284

Think this PR can now be closed


fix: support mapped tasks

This issue is to add support for mapped tasks to the Launch Form UX. While mapped tasks are most often one-dimensional (eg, k:v) flyte supports nested mapping (n-dimensional) as well. To support both use-cases we will provide UX that will renders a standard input field for 1D mappings and a text area (eg, where users can paste JSON text) for n-D mappings.
https://user-images.githubusercontent.com/105876962/175947368-c69d07a8-c4b4-410f-b0e8-6291e32afcd0.mov
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
Map type was unsupported and there were no inputHelpers for MAP. I added those helpers and enabled the Map type to be supported. The UI is like the above video
Tracking Issue
fixes https://github.com/flyteorg/flyteconsole/issues/483

@jsonporter @anrusina Please review this again. I've added multiple keys ability for mapped type. The only issue is that I'm using onBlur on the keyInput and this rerenders the component that prevents onFocus for the valueInput first time.

Updated Video. It doesn't enable/disable the Launch Plan button but it submits the valid value (removing duplicates/empty key/values)
https://user-images.githubusercontent.com/105876962/176738220-fbc804ec-cac4-421e-8542-9c164922c8ec.mov

Failing test is not related to the PR. It is "fixed" separately in master: https://github.com/flyteorg/flyteconsole/pull/533


pynndescent blows up without interpretable error message

Hi, first of all, we've been using UMAP 0.3.x in production for a while and it's been working great! However, we've been trying for a while but can't seem to get 0.4.x working reliably with pynndescent. Here is a simple example of it blowing up (umap=0.4.4, pynndescent=0.4.7):
import numpy as np
import umap

rng = np.random.default_rng()
vecs = rng.choice([-1, 0, 1], p=[0.1, 0.8, 0.1], size=(5000, 100))
print(vecs)

umap.UMAP(verbose=True).fit_transform(vecs)

this fails as soon as the dataset is big enough for pynndescent to kick in, and independent of the metric used etc (tried dice, jaccard e.g.). Uninstalling pynndescent does the trick, but it would be nice to get the speed improvements working ;) The output is:
[[ 0  0  0 ...  0  0  0]
 [ 0 -1  1 ...  0  0  0]
 [ 0  0  0 ...  1  1 -1]
 ...
 [ 0  0  0 ...  0  0  0]
 [ 0  0  0 ...  0  0  0]
 [ 0  1  0 ...  0 -1 -1]]
UMAP(a=None, angular_rp_forest=False, b=None,
     force_approximation_algorithm=False, init='spectral', learning_rate=1.0,
     local_connectivity=1.0, low_memory=False, metric='euclidean',
     metric_kwds=None, min_dist=0.1, n_components=2, n_epochs=None,
     n_neighbors=15, negative_sample_rate=5, output_metric='euclidean',
     output_metric_kwds=None, random_state=None, repulsion_strength=1.0,
     set_op_mix_ratio=1.0, spread=1.0, target_metric='categorical',
     target_metric_kwds=None, target_n_neighbors=-1, target_weight=0.5,
     transform_queue_size=4.0, transform_seed=42, unique=False, verbose=True)
Construct fuzzy simplicial set
Thu Jun 18 17:29:41 2020 Finding Nearest Neighbors
Thu Jun 18 17:29:41 2020 Building RP forest with 10 trees
Thu Jun 18 17:29:41 2020 NN descent for 13 iterations

---------------------------------------------------------------------------
TypingError                               Traceback (most recent call last)
<ipython-input-15-7240c5648635> in <module>
      6 print(vecs)
      7
----> 8 umap.UMAP(verbose=True).fit_transform(vecs)

~/anaconda/envs/grapy/lib/python3.7/site-packages/umap/umap_.py in fit_transform(self, X, y)
   2010             Embedding of the training data in low-dimensional space.
   2011         """
-> 2012         self.fit(X, y)
   2013         return self.embedding_
   2014

~/anaconda/envs/grapy/lib/python3.7/site-packages/umap/umap_.py in fit(self, X, y)
   1799                 self.low_memory,
   1800                 use_pynndescent=True,
-> 1801                 verbose=self.verbose,
   1802             )
   1803

~/anaconda/envs/grapy/lib/python3.7/site-packages/umap/umap_.py in nearest_neighbors(X, n_neighbors, metric, metric_kwds, angular, random_state, low_memory, use_pynndescent, verbose)
    298                 max_candidates=60,
    299                 low_memory=low_memory,
--> 300                 verbose=verbose,
    301             )
    302             knn_indices, knn_dists = nnd.neighbor_graph

~/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py in __init__(self, data, metric, metric_kwds, n_neighbors, n_trees, leaf_size, pruning_degree_multiplier, diversify_epsilon, n_search_trees, tree_init, random_state, algorithm, low_memory, max_candidates, n_iters, delta, n_jobs, compressed, seed_per_row, verbose)
    977                     leaf_array=leaf_array,
    978                     verbose=verbose,
--> 979                     seed_per_row=seed_per_row,
    980                 )
    981         else:

~/anaconda/envs/grapy/lib/python3.7/site-packages/numba/dispatcher.py in _compile_for_args(self, *args, **kws)
    399                 e.patch_message(msg)
    400
--> 401             error_rewrite(e, 'typing')
    402         except errors.UnsupportedError as e:
    403             # Something unsupported is present in the user code, add help info

~/anaconda/envs/grapy/lib/python3.7/site-packages/numba/dispatcher.py in error_rewrite(e, issue_type)
    342                 raise e
    343             else:
--> 344                 reraise(type(e), e, None)
    345
    346         argtypes = []

~/anaconda/envs/grapy/lib/python3.7/site-packages/numba/six.py in reraise(tp, value, tb)
    666             value = tp()
    667         if value.__traceback__ is not tb:
--> 668             raise value.with_traceback(tb)
    669         raise value
    670

TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Failed in nopython mode pipeline (step: nopython frontend)
Internal error at <numba.typeinfer.CallConstraint object at 0x177afc210>.
Failed in nopython mode pipeline (step: nopython mode backend)
scalar type Tuple() given for non scalar argument #5

File "../../anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py", line 232:
def generate_leaf_updates(leaf_block, dist_thresholds, data, dist, dist_args):
    <source elided>

    for n in numba.prange(leaf_block.shape[0]):
    ^

[1] During: lowering "id=16[LoopNest(index_variable = parfor_index.3050, range = (0, $38binary_subscr.5, 1))]{186: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (241)>, 136: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (238)>, 138: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (238)>, 88: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (232)>, 1134: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (245)>, 174: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (238)>, 1136: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (245)>, 208: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (243)>, 210: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (244)>, 112: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (233)>, 212: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (245)>, 114: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (236)>, 214: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (238)>, 216: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (245)>, 90: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (233)>, 219: <ir.Block at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (245)>}Var(parfor_index.3050, pynndescent_.py:232)" at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (232)
[2] During: resolving callee type: type(CPUDispatcher(<function generate_leaf_updates at 0x12d7fd7a0>))
[3] During: typing of call at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (265)

Enable logging at debug level for details.

File "../../anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py", line 265:
def init_rp_tree(data, dist, dist_args, current_graph, leaf_array):
    <source elided>
        updates = generate_leaf_updates(
            leaf_block, dist_thresholds, data, dist, dist_args
            ^

[1] During: resolving callee type: type(CPUDispatcher(<function init_rp_tree at 0x12d7fd8c0>))
[2] During: typing of call at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (464)

[3] During: resolving callee type: type(CPUDispatcher(<function init_rp_tree at 0x12d7fd8c0>))
[4] During: typing of call at /Users/thomas/anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py (464)

File "../../anaconda/envs/grapy/lib/python3.7/site-packages/pynndescent/pynndescent_.py", line 464:
def nn_descent(
    <source elided>
    if rp_tree_init:
        init_rp_tree(data, dist, dist_args, current_graph, leaf_array)

Would you say that pynndescent is still at an experimental stage and not recommend its use in production?

pynndescent is mostly stable and should work; this appears to be an odd error and I would suggest the first thing to do would be to update your numba (and llvmlite) installation to the most recent available.

Hi, thanks for suggesting looking into package versions, since that's in fact been the problem. However, it was not so much related to having the latest version of numba and llvmlite, but rather the latest of pynndescent. The issue is that the latest pynndescent feedstock in fact enforces and older version of Numba and llvmlite (==0.46 and ==0.30 respectively), and since I already had newer versions installed, conda's only solution was to install and older pynndescent that didn't have this restriction (0.4.3). In short, as currently configured in the feedstock, one can either have the latest pynndescent with slightly older versions of numba and llvmlite, or an older pynndescent with the newest numba and llvmlite, but only the former combination seem to work (with regard to this particular issue).

Sorry; there was a brief period where newer versions of numba (particularly 0.47) contained an issue that broke pynndescent. More recent versions of numba fixed that issue. I'll see if I can update the feedstock.

No problem! I don't really need the newer versions in particular, so simply requiring the specific, latest version of pynndescent works for me. But probably a good idea anyway to exclude numba 0.47 while allowing newer versions of it.


New ben-wait executable

Coverage increased (+0.1%) to 85.47% when pulling 1cc1e1875d892fc16bb7ae510755928e4c45ac20 on feat/ben-wait into f60993c00e4421dc0b64437e9765fc9bdcb2868d on master.


Add Hindi & Punjabi

Two-for-one: Punjabi and Hindi translations!
Randhawa.eth

@prabhchintan Great thanks! Question: Would it make sense for us to use the flag of India for Hindi, and the flag of Pakistan for Punjabi? I'm aware that Punjabi is spoken in both Pakistan and India, and that Urdu is the official language of Pakistan. So even while it's not perfect, I'm wondering if you think this is acceptable?

India : 🇮🇳 : Hindi :: Pakistan : 🇵🇰 : Urdu

@encryptedDegen let's use the flag of India for both Hindi and Punjabi (IN) (be sure to include the (IN))


Object "Vacuum state" missing state (cleaning rooms)

Hi there..
i have a new roborock s5 max..
my vis shows me the status of the vacuum, which worked fine with the old model.
unfortunately the vacuum state is not being shown if i use room cleaning.
i noticed that the value of the object displayed is different than the others...
normally you have something like "status(number)"
if i select room cleaning the value is just a number... in that specific case it is the number 18.
with the widget "basic - view in widget" it shows every status i set, except for the 18...
is it yet not well implemented?

I have model 1S and did some enhancements for this model, but not yet commited to git.
The state for room cleaning is number 18.
Due to not knowing which other vacuum model supports this, my enhancement checks the model type and at the moment only whether it is a "roborock.vacuum.m1s". Also the "fan_power" has 4 new different values for my model.
Maybe I have to ask @MeisterTR or some other involved dev. how to merge the new model specific stuff correct for keeping the adapter generic. Then I can make a pull request in near future.

Hi diginix,
thanks for your feedback.
i finally got it working
i edited the object raw data and added a new line with "18": "Room Cleaning"
i dont know why it did not work instantly... but now it works.

I have model 1S and did some enhancements for this model, but not yet commited to git.
The state for room cleaning is number 18.
Due to not knowing which other vacuum model supports this, my enhancement checks the model type and at the moment only whether it is a "roborock.vacuum.m1s". Also the "fan_power" has 4 new different values for my model.
Maybe I have to ask @MeisterTR or some other involved dev. how to merge the new model specific stuff correct for keeping the adapter generic. Then I can make a pull request in near future.

Can you commit your changes? I cannot get my mi robot 1S to run and your other changes also sound really useful

It's already committed to my fork. But it will not help if the old adapter won't run with your 1S. I was able to use my 1S fully with adapter version 1.1.6 without my changes.

ok.. i have installed the adapter and everything looks fine so far..
though i cant tell if it is because i did the changes manually or if the come from the update now..
guess someone else who hasnt done any manual changes would be the apropriate candidate to test.
to me the issues is solved
so feel free to close this issues.


How parse the tags using AEXML

Hi Team,
Good Day!
Am familiar in XML Parsing using NSXMLParser in Objective-C. Now, am working in Swift I can use the NSXMLParser to parse the XML Response but, I have got suggestions to use third party framework (Library) to parse the xml content easily. I googled and found "AEXML" framework for SWIFT programming. I got confused to access the tags from "AEXMLElement".
I have attached the sample response and my sample code for your reference.
I want to parse the main tag "item" and its sub tags "title", "link", "description", "author", "pubDate". How can I handle this?
Can you please help me to parse the xml files like this?
Looking forward your help. Thank you.
Warm Regards,
Yuvaraj M

Hello,
in your case each item is AEXMLElement which contains other elements which you can access like item["title"].value or item["link"].value.


error: Not in scope: data constructor ‘N.SockOpt’

when compiling on debian10 armhf chroot it exists with:
[4 of 7] Compiling Types

/wstunnel/src/Types.hs:39:11: error:
    Not in scope: data constructor ‘N.SockOpt’
    Neither ‘Network.Socket’,
            ‘Network.Socket.ByteString’ nor ‘Data.Streaming.Network’ exports ‘SockOpt’.
   |
39 | sO_MARK = N.SockOpt 1 36 -- https://elixir.bootlin.com/linux/latest/source/arch/alpha/include/uapi/asm/socket.h#L64
   |           ^^^^^^^^^

--  While building package wstunnel-<IP_ADDRESS> using:
      /root/.stack/setup-exe-cache/arm-linux/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.4.4 --builddir=.stack-work/dist/arm-linux/Cabal-<IP_ADDRESS> build lib:wstunnel exe:wstunnel --ghc-options " -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1

fixed after reverting 34645524c3221a596fb59e8dbad4381f10f93933 for /wstunnel/src/Types.hs


Remove jQuery and add tests to theme-a11y.js

This is 🔥 🔥
There's one problem I encountered recently, we need to find a way to only select the visible focusable elements.
Given this code :

  <input id="textInput1" type="text" />
  <input id="textInput2" type="text" />
  <button id="button">Some button</button>

    hidden link

Our test focuses the first tabable element after tabbing the last tabable element in a container will fail, because if you tab 4 times, the test will receive #link and not #textInput1.
This can happen a lot when dealing with mega nav where you have second or third level navigation hidden. Demo store example here.
I had this fixed in the jQuery version by using .filter(':visible') - maybe we can adapt it for our needs ? Code here

@huguestennier @tauthomas01 👍 ?

LGTM 👍


core: setup process and toolchain for performance monitoring

Proposed to have one test for detailed level with the same input file which is run under some profiler. manually. Need to find the best tools which can help to find some performance problems.

valgrind + kcachegrind fit good


cleanup code: remove unused imports in several java source files

Fix: remove the imports.

any questions ?

Hi again, @holiday-sunrise. Thanks for reporting this and all your recent contributions! @mlischetti and I reviewed this yesterday. Since the change involves so many files and the import configuration can be specific to each user, we will tackle this ourselves once all your pending contributions are merged and close your matching PR in the meantime.
This way, we can avoid any disruptions to the PRs in progress and make sure all imports are organized with the contribution guidelines order. We apologize in advance for not noticing the imports did not match the proposed guideline.
Cheers!

they do it :-)


Make sure ICESat and ICESat-2 are used consistently

also IceBridge

This was reviewed and updated by our Tech Writer and USO teams prior to advertisement - I see consistent mentions of both missions in https://github.com/nsidc/NSIDC-Data-Tutorials/blob/main/notebooks/iceflow/0_introduction.ipynb.


Add middleware to track statistics

The original prototype contained a statistic middleware that pushed various infos into an influx DB.
This doesn't make sense when using a cloud based CDN as backend (which has its own analytics engine), but since file based caching is also supported, it may make sense to gather these stats.

Rejected in favor to prevent users privacy


chore(release): 29.2.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.181%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4763

Relevant Lines:
4886

💛  - Coveralls


Does rd_kafka_consumer_poll block for any reason?

Hey
I am calling Consume(TimeSpan=100ms) method that doesn't contain a while loop, and I am expecting this method should just return either null (no message) or an instance of consumeResult without blocking.
And I put some debug logging before and after this method, and strangely it only made 3 calls then it hangs at the 4th call, however, if I change my Kafka server setting to a random invalid host that doesn't exist, it will happily calling it again and again with null as the return value which is what I wanted.
Can you help me understand is there any blocking condition inside the c lib that does blocking?
Regards
Thanks

Consume should never block if you provide a timeout. can you provide a code snippet + set the Debug config property to "all" and post the output here (over a period of time where you're experiencing the issue)?

Consume should never block if you provide a timeout. can you provide a code snippet + set the Debug config property to "all" and post the output here (over a period of time where you're experiencing the issue)?

Hey @mhowlett , thanks for reply, how do I enable debug logging for confluent-kafka, can you be more specific please?

Don't worry. I think I found how to enable debug


Prompted to enter SSH RSA key password twice when using ssh-agent plugin

Hello everyone. Thank you all for maintaining oh-my-zsh. It has made me more productive for over half a decade.
Commit 2a603856 started checking for SSH keys that had already been loaded by checking if a key with their signature had already been added to the ssh-agent. See this line for the relevant line in the commit.
This has had the unintended consequence that now, if your SSH key is encrypted, you are prompted twice for your SSH key decryption passphrase.  The first time is to decode your passphrase to get the signature, and the second is to decrypt your key to add it the the ssh-agent.  Here is a debug log of the ssh-agent plugin being loaded.
This was very confusing for me when it started happening. I was even concerned that something malicious was going on to phish my password. I get the reason why this was added, but I wonder if there's a way to get around being prompted twice. Perhaps we could skip checking the signature if the key's filename already matches a filename loaded into the ssh-agent.

Perhaps we could skip checking the signature if the key's filename already matches a filename loaded into the ssh-agent.

That's what the current code does: https://github.com/robbyrussell/oh-my-zsh/blob/86ea319536a8012b9e8f508d3d257029014bdafe/plugins/ssh-agent/ssh-agent.plugin.zsh#L42-L43
Try upgrading OMZ. Still, when the key is not loaded the double prompt will still happen... I don't really know how to solve that. Maybe there's a better way to get the signature, I'll have to read the ssh-keygen docs.
I also see that there's a bug in how we parse the output of ssh-add -l, that should be fixed:
+_add_identities:22> ssh-add -l
+_add_identities:22> line=The agent has no identities.
+_add_identities:23> loaded_sigs+=agent
+_add_identities:24> loaded_ids+=has

Returning to this, how did you create the id_rsa key, so I can reproduce it in my machine?

I followed GitHub's instructions for generating an SSH key. Namely in the terminal I ran:
ssh-keygen -t rsa -b 4096 -C<EMAIL_ADDRESS>
When prompted I:

Saved the key to the default location, ~/.ssh/id_rsa
Specified an encryption password

Sadly I can't reproduce this, it just prints the fingerprint for me. What version of openssl do you have installed?

I'm running Ubuntu Xenial and my openssl version is OpenSSL 1.0.2g.

idk if this is relevant to what you're doing at all, but I started debugging this zsh thing only to realize I was getting prompted for my password twice because I was going through a jumpbox.

I also see that there's a bug in how we parse the output of ssh-add -l, that should be fixed:
+_add_identities:22> ssh-add -l
+_add_identities:22> line=The agent has no identities.
+_add_identities:23> loaded_sigs+=agent
+_add_identities:24> loaded_ids+=has

This was fixed in 101ea872.
I can't say anything new about the main issue though.

@motevets can you if the following command still prompts for a password and what it outputs?
ssh-keygen -lf /home/motevets/.ssh/id_rsa < /dev/null


Possibility to call controller without Tenant

Is your feature request related to a problem? Please describe.
I'm in a scenario that I want to call a controller and fetch data from the database without checking the tenant
Describe the solution you'd like
Giving the possibility to ignore tenant, for exemple getting all vehicles from vehicle table whatever the tenant is.
Is that already possible in this project ?
Thank you

On your Infrastructure/Persistence/Configuation, do not put builder.IsMultitenant();

You can use Entity Framework Core's "IgnoreQueryFilters" which will bypass any QueryFilters defined (mutlti tenancy is implemented on ef core using query filters):
https://learn.microsoft.com/en-us/ef/core/querying/filters#disabling-filters

If you take a look at the Finbuckle docs, there is a section explaining how the EF filters work, I believe you should be able to temporarily disable the tenant filter from being applied by EF.
The other option is to use dapper with raw SQL, but I can understand why that would be annoying.
The Finbuckle docs are quite thorough.


Impossible to login - UVO

Hi!
Just tried using this integration. I seem impossible to login into UVO via the integration.
Reset my password just in case. Tried creating a different account, no luck.
Trying to login out of the Netherlands. Account is already migrated. Tried creating a new account to enter that but that is not working aswell.
I'm sure the problem is at my side but i'm unable to figure it out

you are having the same issues here: https://github.com/fuatakgun/kia_uvo/issues/79
let's continue there


[LightGBM] [Fatal] Socket recv error, code: 104 when training with binary objective.

Describe the bug
I've met a socket error when training with binary objective.
[LightGBM] [Fatal] Socket recv error, code: 104
19/11/07 14:54:06 WARN LightGBMClassifier: LightGBM reached early termination on one worker, stopping training on worker. This message should rarely occur
I've tried to train a model during 500 iterations. However, when training 318 iteration, the error has been occurred and stop my training.
It is a similar issue with #569. However objective parameter value is different. In my experiment, the value is "binary".
To Reproduce
Steps to reproduce the behavior, code snippets encouraged
Expected behavior
A clear and concise description of what you expected to happen.
Info (please complete the following information):

MMLSpark Version: [e.g. v0.17]
1.0.0-rc1
Spark Version [e.g. 2.4.3]:
2.3.0
Spark Platform [e.g. Databricks]
AWS EMR

** Stacktrace**
Please post the stacktrace here if applicable

If the bug pertains to a specific feature please tag the appropriate CODEOWNER for better visibility
Additional context
19/11/07 14:41:25 INFO LightGBMClassifier: LightGBM worker calling LGBM_BoosterUpdateOneIter
[LightGBM] [Fatal] Socket recv error, code: 104
19/11/07 14:54:06 WARN LightGBMClassifier: LightGBM reached early termination on one worker, stopping training on worker. This message should rarely occur

@ocworld I believe https://github.com/Azure/mmlspark/issues/569 is already fixed, and this is a different issue.  I think this might either be due to faulty network connection or possibly out of memory, but I can't be sure.  I would have to try and repro the problem.  I have worked on adding saving on each k iterations to try and mitigate this problem, but haven't sent out a PR yet for this functionality.  Will leave this issue here for now in case we repro this.  Do you see this error consistently?

New features that you work on is enjoyed to me. In my experiment, I implemented these functions myself with a hook object as a delegate. After your works about them, can I create pull request to you?
It is hard to reproduce this issue because of AWS EMR cost. I additionally implemented
with retry feature in my app. After that, I have to plan check this error consistently.

hi @ocworld
"can I create pull request to you" sure, feel free to create a pull request to mmlspark repository with any new features you might have
"It is hard to reproduce this issue because of AWS EMR cost" no worries, I will just keep this issue open for now, hopefully I will get a reliable reproduction or encounter this as well at some point

@imatiach-msft I've tried to reproduce the error several times, it was not happened after this issue. I assume that it is a very rare case

It's not rare, in fact very reproducible (even on a single machine). Not restricted to classifiers (regressors are affected too). Not even in SynapseML, because lightgbm_ray.RayLGBMRegressor is affected too. Seems to be not related to training time, but to:

early stopping AND
a particular (useful in practice!) combination of the number of estimators and learning rate:

learning_rate=0.01

# n_estimators=100 # OK
# n_estimators=200 # OK
# n_estimators=400 # OK
# n_estimators=800 # # [LightGBM] [Fatal] Socket recv error, code: 104
n_estimators=1600 # [LightGBM] [Fatal] Socket recv error, code: 104

early_stopping_rounds = 10