Notifications RPC

The clientApp should be able to connect to Yo, identified as a specific user.
Yo should support jsonrpc of user-relevant notifications to the clientApp (I'm unsure of implemented transports for Yo)
Requesting notifications -

List of notifications order in descending .created value
Accept requests that include before=
If before= is not supplied the newest notification is the first item in the list.
accept requests that include read=<true|false> - absence of this gives all
accept requests that include type= otherwise gives all
default list size ~20?

The clientApp should be able to post to Yo mark as read one or more notifications

the PUT is a list of notification ids.
after successfully marking items as read, Yo returns the list of notifications (I like full content responses, but that's not always practical)

Notification JSON
{
    id: "UID", //needed to track .read
    read: false, //a boolean value
    notificationType: VOTE, //vote, receivedSteem, tagged, resteemed, postComment, commentComment ////this many not be a fully inclusive list.
        //The current notifications has 'total', 'feed', 'reward', 'send', 'mention', 'follow', 'vote', 'comment_reply', 'post_reply', 'account_update', 'message', 'receive'
    created: 0, //Hoping for epoch seconds, but can be whatever people are using. Created should be the "most relevant time" for the notification
    author: "the author that performed the action",

    //recievdSteem
    data: {
        amount: 10000.2, //only valid for receivedSteem
    }

    //tagged, resteemed, postComment, commentComment
    data:{
        item: { //item is the item/location where the action (described by notificationType) was performed
            author: "the item author",
            category: "the item category",
            depth: 0,
            permlink: "the item permlink",
            parentSummary: "",  //a string to summarize the parent item. Title or Content (max 255 chars)
                                //exists only for N_COMMENT_REPLY, N_POST_REPLY,
            summary: "", //a string to summarize the item. Title or Content (max 255 chars)
        },
        rootItem: { //exists only if item.depth > 0, property definitions follow item.*
            author: "",
            category: "",
            permlink: "",
            summary: ""
        }
    }
}

Please check with @sneak on whether we can use HTTP PUT/POST etc, i'm happy to implement this but the standard policy is to use pure JSON-RPC.

This shouldn't be a question anymore.  All of our backend services run via jussi and are JSON-RPC only, without any exceptions whatsoever.  That means POST, 100% of the time.
The only things that don't conform to this are the backend service health checks, which have to be GET (we like /.well-known/healthcheck.json - but those are handled from the load balancer, never a client app.
This means that there is no server push; clients must make a JSON-RPC periodically to check for new information.

I suggest using iso8601 times: eg 2017-09-25T07:11:20Z

There we have it.
I'll implement some get_notifications methods that return unsent/unseen notifications etc.
One comment though: the whole point of Yo is to provide underlying transports (including push), so we will have server push - but it's configured by configuring the transport ahead of time.

Yo has two parts to it.
The first part is a block follower, which fires events, which execute various event handlers, which, based on user preferences in the database, send various types of notifications.  One of those types of notifications are "Condenser Notifications" which go into their own table, independent of everything else Yo does (SMS, Email, APNS, Android, whatever else).  This, of course, pushes all sorts of events.
The second part is the JSON-RPC API, accessed by condenser, for reviewing/marking/deleting those Condenser Notifications, and for getting/setting the user's preferences for their event handlers above.  It is this part that we are discussing here - the JSON-RPC calls that condenser will be making to Yo.

Note that we will want an admin api that can be accessed with the proper blockchain-based username authentication with a predefined username that allows us to inject arbitrary events into users' Condenser Notifications, for injecting custom notifications of "synthetic" (i.e. non-blockchain) events, such as achievements or sitewide alerts.

The date format used by steemd is UTC iso8601 minus the Z on the end if we want to standardize on that

That works for me.  Our parsers should accept it with or without the Z, to conform to spec.  Ideally steemd would return them with the Z.

Implementation note: Yo should periodically prune its Condenser Notifications table of anything more than 14 days old, assuming that all users' notifications go into a single table, to ensure that the table doesn't grow too large.  Notifications are ultimately ephemeral and do not need to live forever, even unread.

Got it, for admin interface that can be an entirely separate app that simply talks to the same DB and uses the API.

No, keep it in the same app - just add another RPC method.  We'll authenticate it the same way as the other calls from clients, but with a blockchain username that is passed in with an environment variable.

I meant for the UI, my thinking was to have another app that isn't integrated into the same daemon process but still talks to yo's API and manipulates the same backend DB.

I've updated the doc to include two additional calls mark as shown, and  hide with respective new properties on the object.
PUT was more about expected behavior. RPC ftw.

I imagine our UI for a lot of these admin/backend RPC actions is going to be curl for the foreseeable future. :)

a few additional calls, taking ids list as a param:

RPC method name yo.set_notifications_unread (set .read false)
RPC method name yo.set_noficiations_unshown (set .shown false)
RPC method name yo.set_notifications_unhidden (set .hide false) (not using this in the frontend yet but we probably will want to sooner or later!)

Sending:
{'id': 1,
 'jsonrpc': '2.0',
 'method': 'yo.get_notifications',
 'params': {'test': True, 'username': 'test_user'}}
Response headers:
{'Date': 'Tue, 10 Oct 2017 19:42:19 GMT', 'Content-Length': '1473', 'Content-Type': 'application/json; charset=utf-8', 'Server': 'Python/3.6 aiohttp/2.2.5'}
Response JSON:
{u'id': 1,
 u'jsonrpc': u'2.0',
 u'result': [{u'created': u'2017-10-10T21:41:26.504400',
              u'data': {u'amount': 6.66},
              u'notify_id': 899505,
              u'notify_type': u'power_down',
              u'read': False,
              u'seen': False,
              u'updated': u'2017-10-10T21:41:26.504400',
              u'username': u'test_user'},
             {u'created': u'2017-10-10T21:41:26.504414',
              u'data': {u'amount': 13.37},
              u'notify_id': 4943982,
              u'notify_type': u'power_up',
              u'read': False,
              u'seen': False,
              u'updated': u'2017-10-10T21:41:26.504414',
              u'username': u'test_user'},
             {u'created': u'2017-10-10T21:41:26.504421',
              u'data': {u'resteemed_item': {u'author': u'test_user',
                                            u'category': u'test',
                                            u'permlink': u'test-post',
                                            u'resteemed_by': u'some_user',
                                            u'summary': u'A test post'}},
              u'notify_id': 7188138,
              u'notify_type': u'resteem',
              u'read': False,
              u'seen': False,
              u'updated': u'2017-10-10T21:41:26.504421',
              u'username': u'test_user'},
             {u'created': u'2017-10-10T21:41:26.504426',
              u'data': {u'item': {u'author': u'some_user',
                                  u'category': u'test',
                                  u'permlink': u'another-test',
                                  u'summary': u'Stuff etc'}},
              u'notify_id': 2056410,
              u'notify_type': u'feed',
              u'read': False,
              u'seen': False,
              u'updated': u'2017-10-10T21:41:26.504426',
              u'username': u'test_user'},
             {u'created': u'2017-10-10T21:41:26.504431',
              u'data': {u'amount': {u'SBD': 6.66, u'SP': 13.37},
                        u'item': {u'author': u'test_user',
                                  u'category': u'test',
                                  u'permlink': u'test-post',
                                  u'summary': u'A test post'},
                        u'reward_type': u'curation'},
              u'notify_id': 7647424,
              u'notify_type': u'reward',
              u'read': False,
              u'seen': False,
              u'updated': u'2017-10-10T21:41:26.504431',
              u'username': u'test_user'}]}

two requests:

i'd like to request once and receive any notificiations that were created after or updated since a given timestamp -- this will be handy for polling so we can capture any new or modified notifications without having to run two types of requests, i.e. a special param created_or_updated_after

can the response include meta info about the requested cursor & the total number of results available? this would save requests, specifically in the case of the polling requests -- for example, we'll ideally set the limit on those requests to minimize the likliehood of having to do more than one request, and it'd be helpful to know that we do not have to request the next page.

@GarethNelson - the general structure looks good with the following caveats

the resteem_item attribute should just be item
username at the root should be author (I get your thought there, but reasons)
there are no examples of rootItem in the response.

The json spec in the main description is exactly what we're looking for in terms of attr names and structure. It's based off comment responses returned by the api. It'll be more familiar to devs coming in and out of different parts of the codebase.

1 - resteemed_item i named that way for a specific reason, because the fields are slightly different in plain old "item", might map better to objects
2 - username is the same in all cases, author doesn't make sense for things like money transfers
3 - this will come in next update, i'll let you know

Ah! So, we actually need the structure to be the same, all types of notification elements are built with a single view component that relies on them being identical (except for very special cases like receive steem). We actually have the UI built out and working on mock data
author is conceptually better in this case, since the user who performed the action that triggered the notification could be considered the "author" of the notification. This also holds for "item.author" and "rootItem.author". (those are the authors of item & rootItem respectively)
sweet.

1 - i'll reorganise this to be the same
2 - to make it clear: the API takes "username" as "notifications relevant to this user", whereas "author" would mean the author of a piece of content - doing otherwise and using "author" for everything would confuse matters greatly. In the case of item and root_item data of course, author makes total sense.

Watch this space :)

TODOs for this item

[ ] return modified notifs from mark_* requests
[ ] respect updated_after and other filter options for get_notifications requst
[ ] add settings api
[ ] figure out auth
[ ] return real data

another request: instead of notify_type param, accept notify_types as an array of types to filter

@relativityboy @GarethNelson @bnchdrff I think this is resolved (other than auth which has its own issue). Anybody disagree?

This was more of a toplevel issue to track implementing the API - i'd call it closed, but @relativityboy can comment on anything missing.


document --platform flag for okteto build

okteto build has --platform flag which allows specifying what platform to build the container images for. While documenting we can also mention a common use case for this where if you're on an ARM based machine but using a non ARM cluster then you will need to run okteto build --platform linux/amd64 to build the images correctly so that they run on your cluster.

I'd like to take this up. Please assign this to me!

I've started working on this issue and would like to know couple of things before pushing the changes:

Where do we actually want to implement these changes? Is it here?
By documenting the flag --platform, do we intend to write a description of the flag and explain its use case? If Yes, I find that there's no such description written for other flags such as --file, --no-cache


Name the container port metrics in vllm

This is to align with the common practice to set the container port name to metrics so that the port name can use metrics as the port without specifying a number.
Getting metrics from vllm also requires setting the entrypoint to vllm.entrypoints.openai.api_server  and this is trakced in https://github.com/kubernetes-sigs/wg-serving/issues/18.

cc: @jjk-g

lgtm @ahg-g

/approve

Hi @jjk-g it seems it requires /lgtm label before merging.


[Proposal] Swap Adopt OpenJDK for Coretto

Is your feature request related to a problem? Please describe.
Coming from https://github.com/opensearch-project/opensearch-build/issues/1682#issuecomment-1055923578, consider swapping AdoptOpenJDK for Coretto.
Describe the solution you'd like
Swap AdoptOpenJDK for Coretto.
Additional Context
I'd like to hear thoughts from the community on this.

cc: @spotaws

On the fence with this idea, I understand the pros of this for testing purposes, but I also understand the optics of it may not be the best idea in liu of the most popular open source Java distribution.

@dblock why not Temurin ?

@dblock why not Temurin ?

I don't have any good reason for/against one or the other, Temurin is what we ship with now. I only wanted to bring up the discussion so everybody can say why one distribution is preferred to them vs. another.

Temurin is vendor-neutral (comes from Eclipse foundation) and fully certified OpenJDK binary distribution. I have nothing against Amazon Correto / Microsoft / Azul Zulu / Bellsoft / SAP / RedHat / ... OpenJDK distributions, but my main argument - having vendor-neutral  bundled JDK for open source project makes sense.

@dblock based on above discussion, how should we proceed for this issue? Should we swap AdoptOpenJDK for new JDK(Temurin) ?

@minalsha we use Temurin for JDK-8/11/17, sadly JDK-14 is only available from AdoptOpenJDK (it is used for BWC tests)

Let'd close this. We've decided that OpenSearch will continue using a vendor neural JDK.


Add write method to Message

Should write the contents of the message to a byte stream.

Fixed by  #157.


Windows app fails to start after a day of usage.

User reports it happening after multiple reinstalls of the application.
More details in : 366857
Error stack:
System.Windows.Markup.XamlParseException:
The invocation of the constructor on type <TogglDesktop.MainWindow>
that matches the specified binding constraints threw an exception.
---> System.InvalidOperationException: TogglSettingsView expected=96 actual=88
at TogglDesktop.Toggl.StartUI(String version, IEnumerable`1 experimentIds)
at TogglDesktop.MainWindow.finalInitialisation()

Closing as issue is out of date and has not been reported recently.


Fix invalid expression syntax

The expression had an extra '$' character which made it always evaluate to true.
See https://github.com/boostsecurityio/poutine/blob/main/docs/content/en/rules/if_always_true.md.
CLA: trivial
:hammer_and_pick: with :heart: by Siemens

thank you!  And concur with cla:trival

This pull request is ready to merge

Merged to the master branch. Thank you for your contribution.


CI is failing on Development

CI is currently failing on Development due to the extra index url not being included in the environment file. There may be more underlying failures as well once that is fixed.

CI works now, hurray!


Merge PR doesn't find rug function

Merge button doesn't work.

https://files.slack.com/files-pri/T29E48P34-F56J8APA8/slack_-_marlette_funding.png

Fixed a dogy node in our cluster.


Custom TextInputControls should be able to perform TextInputActions

Use case
We are building a custom on-screen keyboard using TextInputControl and while typing works great, there seems to be no way to perform a TextInputAction, since it only seems to go through the MethodChannel in TextInput.
Thus, we currently can't implement a "Submit" button on our custom keyboard.
Proposal
Similarly to TextInput.updateEditingValue(editingState), a method like TextInput.performAction(textInputAction) could be exposed, that calls performAction on the currently connected TextInputClient.

Ah I see. Yeah it should be possible to create a virtual keyboard in Flutter, but I hadn't thought about "submit" or other action buttons.
I think we'll need to know if this is possible on native Android and iOS, and if so, what are the APIs that allow you to do this.


Add basic ARIA mark-up

I've added basic ARIA mark-up for links and buttons.
The only thing I'm doubtful about is how to organize threads and replies. I experimented a bit, but this functionality needs to be tested and might be reverted in future.
I also didn't mark up user name links that toggle user IDs: I might implement it in future, but it needs also a description, otherwise a blind user won't understand what the click on a user name does.

LGTM. @Menelion, have you finished with it? May I merge it?

@igoradamenko, Yes, please. However, I still don't get why the CI is failing :).

That's because we don't have CI for pull requests now. Somewhen we will — #14.
But now — thank you for PR, it's awesome 🐨


Graphics: Slice out Algae Images

On Page 88 of the Reef Check manual, we need to get images with clear backgrounds of the 9 algae species surveyed.

[ ] Giant Kelp
[ ] Feather Boa Kelp
[ ] Bull Kelp
[ ] Pterygophora californica
[ ] Laminaria setchelli
[ ] Laminaria farlowi
[ ] Sargassum horneri
[ ] Sargassum muticum
[ ] Southern Sea Palm

Common image formats are best.

I'll get right on those graphics! Is there a specific pixel dimension you wanted? Also, how do you you want the species to be rendered? I can make cartoonish versions of the algae species or I can render them to look more realistic.


MWF nexient patch 1

This contribution attempts to close #598 by

adding errorsCollected() to AbstractSoftAssertions
removing the need from the various *SoftAssertion classes from accessing their proxies field directly
Testing that custom collectors with non-package access can access the errors they've collected

Integrated, thanks


Increasing Capacity UK

Detailed Description
In the backtest we use continuous increase effective capacity (need to check this) and use this to normalise the PV Live values. This is used for training. Currently in live the effective capacity slowly drifts down and then jumps up when PVlive update it. This is different from real life and can lead to some errors
Live effect capacity
Context

we train our model using values between 0-1 for pv solar genertion. So we need a really accuray capacity in order to up scale these things.

Possible Implementation

Look at the last two updates of capacity, and extroplate linear from there to get the extra capacity right now.
Still use ffective capacity, as this takes into account degradation

Here's the date for the last 1.5 years

Asked on here - https://github.com/SheffieldSolar/PV_Live-API/issues/36

On second thoughts, PVLive probably uses this step values for the estimate, so we should use these value to best estimate the PVLive out turns

Yep, I think I agree with the last comment - we will be assessed versus PV_Live, but agree for modelling net demand the increasing demand would make sense.

as discussed with PVLive, ill close this for the moment


reuse lcg during pack

Ref. https://github.com/d3/d3-hierarchy/pull/190#issuecomment-1086121332 Supersedes #192.

A detail is, I don't get why we need to keep Math.random at all, since the API is now fully deterministic?

True, I can drop that. In an earlier iteration I allowed passing a random implementation into packEnclose and packSiblings, but then I thought it might break calls to array.map(packSiblings) etc. if we allow the function to take an extra argument. And, I didn’t really want to expose that low-level functionality—I just wanted to ensure that the methods are externally deterministic, while still using “as much randomness as possible” internally.


Add bracket/comments matching to vscode plugin

Should help towards fixing #805. Haven't tested it extensively, but it should be a solid start.

Thanks!


Build OpenLDAP with cyrus-sasl support

Enable cyrus-sasl support
Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x ] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary
Enable cyrus-sasl support in OpenLDAP
Change Log
Build openldap with cyrus-sasl support
Does this affect the toolchain?
NO
Test Methodology
local build

Submitted another pull request that is based off the main branch.


[feature] Unify Extra Trees and Random Forest preview classes

Description
This unifies Random Forest and Extra Trees to a single file from the same inheritance classes to match stock scikit-learn. This reduces overall code size, and prepares for movement out of preview.  This improves preview RF to meet some of the standards of sklearnex in terms of testing and verbosity.
Changes proposed in this pull request:

rm extra_trees.py
use extra_trees.py code as base for forest.py
set Random Forest Classification and Random Forest Regression as child classes of ForestClassifier and ForestRegressor

/intelci: run

Todos: Fix testing issue, make changes for v1.4 (monotonic_cst) that were only made to forest.py


execute seed do follow the order in catalog file

#20

Coverage increased (+8.36%) to 94.29% when pulling ae6e800ba112a2205e0690c163d71f03ede2d8a3 on feature/seed_follow_order_in_catalog into 36273f60dbc9373550247d72a2b622c12e47d22d on master.

Coverage increased (+8.36%) to 94.29% when pulling ae6e800ba112a2205e0690c163d71f03ede2d8a3 on feature/seed_follow_order_in_catalog into 36273f60dbc9373550247d72a2b622c12e47d22d on master.

:+1:


chore(deps): update dependency org.apache.portals.pluto:pluto-container to 3.0.1

This PR contains the following updates:

Package
Change

org.apache.portals.pluto:pluto-container
2.1.0-M3 -> 3.0.1

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

This PR has been generated by Renovate Bot.

The upgrade to Portlet spec 3.0 is currently not a priority.
This may be revisited at a later date.


Error when calling cancel: args.stream.abort is not a function

I'm using React and I want to be able to cancel the postscribe load if my component unmounts before the script is loaded.  Why?  Because React will complain, rightly so, if I try and do things with an unmounted component because in some cases, when the afterAsync callback is called, the component is gone.  I need to cancel it from triggering, naturally I'd want to cancel it within my componentWillUnmount hook.
Looking into the documentation, it looks like postscribe(...) returns an object that you can call .cancel() on.  So I did that, but the error I get is:
args.stream.abort is not a function
This is referring to the WritableStream class and looking into it, I couldn't see anywhere that an abort() method exists.
Here is how I'm using the code in a React component in componentDidMount I call this:
componentDidMount () {
    this.postscribeCancel = postscribe(this.el, '<script type="text/javascript" src="https://great.url.here"></script>', {
      afterAsync: () => {
          this.setState({
            show: true
          })
      }
    })
}

this.el is an element reference from the DOM.  After the script loads, I want to show the element so that's why I call setState.
Then, to properly cancel this loading, I presumed I could do this:
  componentWillUnmount () {
    if (this.postscribeCancel) {
      this.postscribeCancel.cancel()
    }
  }

But when that fires, I get this error in the JS console:
Uncaught TypeError: args.stream.abort is not a function
That cancel function from the postscribe code is here:
https://github.com/krux/postscribe/blob/master/src/postscribe.js#L172
and for easy reference looks like this:
    cancel: () => {
      if (args.stream) {
        args.stream.abort();
      } else {
        args[1] = doNothing;
      }
    }

So I guess my ultimate question is, if abort() doesn't really exist on the WritableStream object, is there any other suggestion on how I can cancel to stop the afterAsync from triggering when the component is unmounted before the postscribe script loads.

My expectation would also be that even if the postscribe script loads successfully, I should be able to call this.postscribeCancel.cancel() and have no error thrown, it would just be a no-op at that point because there is nothing left to cancel.

I'm having the same problem.

Pull requests welcome


adding localization text in control page in 6 different languages

[x] This contribution adheres to CONTRIBUTING.md.
TODO: Check the above box with an 'x' indicating you've read and followed CONTRIBUTING.md.
TODO: Include high-level description of the changes in this pull request.
As part of User Story (https://ni.visualstudio.com/DevCentral/_workitems/edit/2665528/) We are trying to include localization description for Chinese, German, Japanese, French, Korean languages for different VeriStand third party products.
TODO: Justify why this contribution should be part of the project.
Taken reference from VeriStand control page (\argo\ni\nipkg\pool\ni-v\ni-veristand-2024\24.0.0\<IP_ADDRESS>451-0+f299\ni-veristand-2024_<IP_ADDRESS>451-0+f299_windows_x64.nipkg.control) and edited niveristand-embedded-data-logger-custom-device-plugin control page accordingly.
TODO: Detail what testing has been done to ensure this submission meets requirements.
Testing is in progress

/AzurePipelines run


Webapp:az webapp up failed to deploy with errors

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
'NoneType' object has no attribute 'reserved'
Traceback (most recent call last):
Local\Temp\pip-unpacked-wheel-2huwp6jm\knack\cli.py, ln 215, in invoke
azure\cli\core\commands\__init__.py, ln 654, in execute
azure\cli\core\commands\__init__.py, ln 717, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 709, in _run_job
azure\cli\command_modules\appservice\commands.py, ln 37, in _ex_handler
azure\cli\core\commands\__init__.py, ln 688, in _run_job
azure\cli\core\commands\__init__.py, ln 325, in __call__
pip-unpacked-wheel-0ydpnxq9\azure\cli\core\__init__.py, ln 784, in default_command_handler
azure\cli\command_modules\appservice\custom.py, ln 3692, in webapp_up
AttributeError: 'NoneType' object has no attribute 'reserved'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az login
az webapp up --subscription {} --location {} --name {} --plan {} --resource-group {} --html --os-type {}

Expected Behavior
It should deploy the project to app service.
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.16.0

Additional Context

webapp

#14287

Is this a known issue? Is this really an issue or just happening on my machine?
It is blocking my deployment but I haven't gotten responses yet.

Is this a known issue? Is this really an issue or just happening on my machine?
It is blocking my deployment but I haven't gotten responses yet.

Thanks for reporting .Will take a look.

Please try with the latest CLI version & let us know if this still repros. Thanks!


BackHandler make App Stop | Hang

"react-native-router-flux": "^4.0.0-beta.24",
react-native v0.48.3
Steps to reproduce
-Screens is getting Double (Pop ) and app gets Hang as Well .

Could you please share a gif?

This works for me:
 shouldCloseApp = state => {
    if (state.index > 0) {
      return false;
    }

    if (state.routes) {
      return state.routes.every(this.shouldCloseApp);
    }

    return true;
  }

backAndroidHandler = {() => {
  let { state } = Actions;

  if (state.routes) {
    // handle drawer
    if (state.routes[0].index === 1 && state.routes[0].routeName === 'drawer') {
      Actions.drawerClose();
      return true;
    }
  }

  if (this.shouldCloseApp(state)) {
    return false;
  }

  Actions.pop();
  return true;
}}

credit: https://gist.github.com/iesteban/724227c2dc1364066473dee5bbfc304d


Test: Revert "Fix candidate set address state handling"

Reverts ZcashFoundation/zebra#1709

Obsoleted by #1803.


Listen button has werid focus border

Focus on the Listen button, the focus is weirdly rendered due to the chevron

thanks for reporting! we just release a new version of Serenade with a different UI for that button, so this should no longer be an issue!


Edit of NodeMCU Voucher Profice via API thru Mikrotik Terminal

Dear Sir Ivan,
I am using multi vendo by assigning each sub-vendo with a different Voucher Profile and Code Prefix. There are instances that I pre-flash back up NodeMCU for contingency measures or reassign sub vendo to different stores. It will take so much time to open Nodemcu admin portal especially remote access thru Remote Winbox. It is easier , faster and more convenient if edit of NodeMCU Voucher Profile and Code Prefix can be done using API Script thru Mikrotik Terminal just like the convenience of changing the rates.
Respectfully yours,
Elvin Sotto Dagohoy
Davao City Philippines

Answered in the group already


widgets: add 12h time format

Closes https://github.com/hyprwm/hyprlock/issues/493.

why would you want to use an inferior time format?

No idea, but seems that some users prefer it.

wiki mr neeeded

https://github.com/hyprwm/hyprland-wiki/pull/791


Consistency on names and words: Alby Hosted Balance -> Alby Shared Node Balance

I've noticed some minor confusion regarding the term "Alby Hosted Balance." It's not a serious issue, but users seem to get a bit mixed up. Is there a specific reason why it's not called "Alby Shared Node Balance," as it is on the getalby.com web account?
If not, could we update the terminology to use the same name everywhere? This would make it easier for users to associate the terms consistently, especially when we address support inquiries related to misunderstandings or mismanagement.
I propose we change "Alby Hosted Balance" to "Alby Shared Node Balance" for consistency.

Also:
"Alby shared wallet" to "Alby Shared Node Balance" for consistency.

Actually, I don't mind if the final name is "Alby Shared Node Balance" or "Alby Shared Wallet" or "Alby Hosted Balance". What matters here is that it is the same name everywhere, so that the users with difficulties understands with clarity we are referring to the same Alby legacy wallet, where their funds from their legacy wallet were.

CC @itstomekk

Great find!
I'm all for having one name for it, also don't mind which.
Getalby.com dashboard calls it 'shared wallet' as well.
One more option to consider is Alby legacy wallet?

@jpl-btc @itstomekk would either of you like to try a PR for this one?

I’ve carefully read through the guide and have only one suggestion:
Could you rename the section title? Instead of "Backup," consider using "Backup & Recover."

You mean here?

PS I think the rest is fine. We are susnetting shared node so doesn't matter that much anymore... but maybe it should be under "lightning"?


Routing file version 2 nested layout path does not match

What version of Remix are you using?
1.11.1
Steps to Reproduce
Test repository: https://github.com/akun2333/remix-netlify
Deploy demo: https://superlative-chimera-88b2c1.netlify.app/
Expected Behavior
Normal route matching
Actual Behavior
The access path post/about is fine in development mode, but not when deployed to netlify

not getting post/about in dev, but when using flat routes and folders, the route file must be index
app/
├── routes/
│   └── _post.post/about
│       └── index.md

@akun2333 Remember that once you start adding folders, flat routes will switch to flat-folders mode (it assumes you want colocation). The folder name itself is the route. The route file must be index.js or (.tsx, .md, etc.).
It's the same as the index file when doing imports from a folder. import About from '~/routes/_post.post/about'
Any other file, like component.tsx, styles.css, logo.png, etc. will be treated as a colocated file.
flat-file
routes/_post.post.about.md
flat-folder
routes/_post.post.about/index.md


Implement ciriAS

Will not be done, since we will focus on developing the nf-core/circrna pipeline


build: Update GH actions for merge queue

Closing as we don't seem to be ready to merge queue, and this would not provide a ton of value inside this repo anyway given the fairly low activity


Fixing hardcoded release in shared module in evidence datasource parsers

In the evidence parser repo, the shared evidence.py module has the following lines:
remote_trait_mappings_url = (
    'https://raw.githubusercontent.com/opentargets/curation/22.09.1/mappings/disease/manual_string.tsv'
)

Hardcoding the release in the URL poses a problem in the evidence generation, because to keep it up to date, it needs to be updated all the time. Ideally the curation release should be taken as a parameter and changed dynamically. Maybe we can assume a latest release of some sort.

@prashantuniyal02 Please include this for the release intentions


Receiving Microsoft.Azure.Devices.Client.Exceptions.UnauthorizedException after restarting OpcPublisher.

Hello,
I'm running the opcpublisher as a native windows app.
When starting it for the first time, everything works fine. After the app has been terminated, subsequent attempts to start it again always results in the following error:
2018-09-05 13:48:59.928 +02:00 [INF] IoTHub device cert store type is: X509Store
2018-09-05 13:48:59.931 +02:00 [INF] IoTHub device cert path is: My
2018-09-05 13:48:59.933 +02:00 [INF] Attempting to register ourselves with IoT Hub using owner connection string.
2018-09-05 13:49:00.851 +02:00 [INF] Device 'IoTDevice' found in IoTHub registry. Remove it.
2018-09-05 13:49:00.932 +02:00 [INF] Adding device 'IoTDevice' to IoTHub registry.
2018-09-05 13:49:01.142 +02:00 [INF] Generate device connection string.
2018-09-05 13:49:01.161 +02:00 [INF] Adding device connectionstring to secure store.
2018-09-05 13:49:01.722 +02:00 [INF] Attempting to read device connection string from cert store using subject name: IoTDevice
2018-09-05 13:49:01.815 +02:00 [INF] IoTCentral mode: False
2018-09-05 13:49:01.823 +02:00 [DBG] Open IoTHub communication
2018-09-05 13:49:02.872 +02:00 [ERR] Failure initializing hub communication processing.
Microsoft.Azure.Devices.Client.Exceptions.UnauthorizedException: CONNECT failed: RefusedNotAuthorized
at Microsoft.Azure.Devices.Client.InternalClient.<>c.b__62_2(Task t)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
at OpcPublisher.HubCommunication.InitHubCommunicationAsync(TransportType transportType) in C:\Projects\iot-edge-opc-publisher\opcpublisher\HubCommunication.cs:line 152
After some time (approximately an hour), it gets working again.
What could be causing this issue?
best

Check whether you PC is connected to Internet.

Thanks, but internet connection is always fine.
I've finally added the device connection string argument within the command line in order to prevent the device always getting recreated. Now it works fine.


Only the first need created is displayed when created in Crome

The rest of the needs -  if they are of the same type - cannot be loaded. No error messages, excpet for in the browser console, that the need cannot be loaded. On the won node, the need does get created, no obvious error messages on owner-server either.

It can be the server side problem - it can be that sometimes the owner receives the success message faster, then then need is really stored on the server and can be loaded from linked data.

fixed in f350bee


sudden spike in pruning time

Describe the bug
i noticed a sudden spike in pruning time, from ~15/30ms to 5minutes
Logs are spammed by messages
failed sending request to peer's session
and
request to peer's session failed

I've updated to latest 1.1.0-rc2 version, opening this as requested from tg group.
https://t.me/paradigm_reth/29466
Currently, pruning time is around 7seconds, while on beta2 it was around 15ms
No changes has been made to any .toml or docker-compose, only to switch reth version from beta2>1.0.0-rc2
reth.log
Steps to reproduce
normal execution since beta.2 release
Node logs
No response
Platform(s)
Linux (x86)
What version/commit are you on?
Reth v0.2.0-beta.2 28f3a2e
What database version are you on?
2
Which chain / network are you on?
eth
What type of node are you running?
Full via --full flag
What prune config do you use, if any?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

Logs are spammed by messages
failed sending request to peer's session
and
request to peer's session failed

Just noting that these logs are unrelated to the problem, which is an extremely long commit time. Could you also provide the commit time / tx open time graphs?

Also new logs from the latest version
reth.log

Closing the issue as the pruning time is back to nominal time by upgrading to reth 1.0.0


A question about the frequency threshold determination

I used the call_freqb as described and output the BED file, column 11 of the file is the frequency and likelihood of methylation sites, I would like to ask you if there is a recommended filter value (how much frequency should be greater) as a trusted methylation site.

Hi @Wyinghao , thanks for using our tool. In my opinion, there's no common thresholds for methylated or unmethylated sites. It depends on the problem and the data. Generally, I would take >=0.7, <=0.3 to get highly methylated and lowly methylated sites, respectively.
Best,
Peng

Hi，peng！Thank you very much for your reply，I will adopt this standard that you recommend！

Closing this issue as it is inactive. If further issues arise, feel free to reopen this issue.


A set of composer fixes

Closes #1173 - newlines no longer inserted when you press cmd+enter
Closes #1149 - selected text is no longer linkified when you paste a url
Closes #1169 - disables the adult content controls if there is no media on the post

All 3 changes work

not for me (yet): https://github.com/bluesky-social/social-app/issues/1149#issuecomment-1684004929


Error response with application/json body throws InvalidStateError

When I return a validation error message from my server as with content type application/json with status code 400 this throws  the following error:
Uncaught InvalidStateError: Failed to read the 'responseText' property from 'XMLHttpRequest': The value is only accessible if the object's 'responseType' is '' or 'text' (was 'json').

on this line: https://github.com/mzabriskie/axios/blob/f9c46c5c1da731c150febd347708f229e9bc74ab/lib/adapters/xhr.js#L75. Apparently it's not allowed to access the responseText property if the responseType does not match '' or 'text'.
You seem to be aware of this since you are checking exactly this a few lines earlier (https://github.com/mzabriskie/axios/blob/f9c46c5c1da731c150febd347708f229e9bc74ab/lib/adapters/xhr.js#L62).
I'd submit a pull request, but I'm not entirely sure what case is meant by the (request.responseText && xDomain) code...

I think this could be resolved by switching the order of the conditional check:
(xDomain && request.responseText)

This way we only reference request.responseText if it's valid.

Sorry I was doing something else, but had this fix commited on my fork already before I saw your comment. Not sure what you prefer, but you seemed to have the check already in place so I just thought I'd use it.

I think it would be safer to do as I suggested above. Thanks for the PR!

A few minutes ago, I encountered same issue. but, is was already fixed. very nice! :+1:

Hi, I'm getting same error

I just encountered the same problem.

@GerryFudd do you use redux-axios-middleware ? If yes just take a look at this issue

I encounter similar problem @0.16.2.
ajaxInject.js:1 Uncaught DOMException: Failed to read the 'responseText' property from 'XMLHttpRequest': The value is only accessible if the object's 'responseType' is '' or 'text' (was 'blob').
Thank you for your help~


WIP: Mapping function plugin system

Description:
Closes #128
See #128 for design and reasoning behind this PR.
Type of change:

[ ] Bug fix.
[x] New feature.
[ ] Documentation update.
[ ] Testing.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

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

Comparison is base (fdbbd06) 75.08% compared to head (85b5cae) 75.45%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
+ Coverage   75.08%   75.45%   +0.36%
==========================================
  Files          17       17
  Lines        1405     1426      +21
==========================================
+ Hits         1055     1076      +21
  Misses        350      350

Files
Coverage Δ

tripper/namespace.py
89.28% <100.00%> (+0.12%)
:arrow_up:

tripper/triplestore.py
82.65% <100.00%> (+1.06%)
:arrow_up:

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

@jesper-friis Merge at your own convenience


August 8th, 2022 Community Meeting

Please add a topic in this thread and add a link to the GitHub issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on GitHub before coming into the meeting
(optional) Paste the image of an animal 😸

Take a pass through some people in our OWNERS files.
Remove the following because of inactivity:

@alicerum
@gabemontero
@sbose78
@shahulsonhal

At those above: I'd prefer to get people back of course. So, if there was just temporary inactivity, I'd be happy to keep more people in.
Add the following:

@coreydaley
@ricardomaraschini

🐢

unfortunately I don't see me having cycles anytime soon with my new assignment to pick back up

Neither do I, sorry 😔

We need a new reminder with the Zoom Link in Slack.
OWNER file cleanup

Shoubhik is on PTO
Enrique: we take that offline with him
Rolfe: also makes sense, especially for the website
Matthias: let's also add Rolfe in other places as reviewer; we can manually assign others
Enrique: use https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md#emeritus to keep engineers listed
Matthias: should we sync owners files ?

Pro: you always have an up-to-date set
Con: same owners for website, cli, triggers might not make sense

Grooming on Wednesday

Date for next release will be back on agenda


FF9.0 select text is aligned to top

In pull from Jan 17 text inside select on firefox 9 / win7 is aligned to top... looks strange.
IE9/Chrome on Win7 is good.

@jkresner Double check your screenshot link, please.

I can see it now. Is this in 2.0-wip or master?

2.0-wip

bootstrap.css (line 538) lose the padding:0;
select {
background-color: #FFFFFF;
padding: 0;
vertical-align: middle;
width: 220px;
}
Maybe add the padding: 0; in the select[multiple]

Removed the padding from the select, but didn't add it back to the select[multiple] for now. Will check those out later in a more comprehensive form cross browser testing session.


Updates for API 0.3

This concludes the migration to SequentialTransform. Notice that in order to support combinations of different Transform with tables we had to relax the dispatch for ParallelTableTransform.

Codecov Report

Merging #139 (5652971) into master (29bd21c) will increase coverage by 0.07%.
The diff coverage is 76.92%.

@@            Coverage Diff             @@
##           master     #139      +/-   ##
==========================================
+ Coverage   94.70%   94.77%   +0.07%
==========================================
  Files          27       26       -1
  Lines         831      766      -65
==========================================
- Hits          787      726      -61
+ Misses         44       40       -4

Impacted Files
Coverage Δ

src/transforms.jl
100.00% <ø> (ø)

src/transforms/parallel.jl
94.87% <76.92%> (-0.19%)
:arrow_down:

src/distributions.jl
90.90% <0.00%> (-0.76%)
:arrow_down:

src/transforms/scale.jl
93.75% <0.00%> (-0.70%)
:arrow_down:

src/colspec.jl
92.10% <0.00%> (-0.58%)
:arrow_down:

src/transforms/coerce.jl
92.85% <0.00%> (-0.48%)
:arrow_down:

src/transforms/select.jl
93.50% <0.00%> (-0.17%)
:arrow_down:

src/transforms/filter.jl
98.41% <0.00%> (-0.08%)
:arrow_down:

src/transforms/onehot.jl
97.29% <0.00%> (-0.08%)
:arrow_down:

... and 5 more

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

@eliascarv I will go ahead and merge this one to finish the full migration downstream. I understand you are a bit busy today.


file is missing in the python directory

Re-tested not fixed

deployed the latest code and tested the file processing of diksha enrolment we are getting above issue , and updated defect to @padmanabha1234

Re-tested Diksha TPD and ETB enrolment file processing are working as expected


Add support for translator role

This is to match the addition of the role on PFE

For reference https://github.com/zooniverse/Panoptes-Front-End/pull/3885


tools: add 1password cli

Motivation: improve experience to work with secrets.
Source: https://1password.com/developers/.

https://github.com/kamilsk/dotfiles/commit/9d81bb6dc12913126614799e255696f350a205f9


Function kwargs not rendering on docs.ms.com

Boils down to during processing within the sphinx-docfx-yaml extension, we're seeing the following earning when we enable debug output:
Can't get argspec for <class 'function'>: azure.mgmt.storage.v2015_06_15.aio.operations.StorageAccountsOperations.begin_create
Can't get argspec for <class 'function'>: azure.mgmt.storage.v2015_06_15.aio.operations.StorageAccountsOperations.check_name_availability

I know it's firing from extension.py, but I don't have a root cause as to WHY this is yet.
Note that class kwargs aren't having this same problem.

After spending a bunch more time with this, I believe this issue is at least twofold. Context here is that Johan raised this as a general issue. It both IS and ISN'T a general issue. We definitely do document some function keyword arguments. The problem is we don't seem to be doing it consistently. There also isn't a great way to programatically pull this detail out of the rendered docs.
For the specific issue that Johan raised, like here
You'll notice that SOME keyword params are rendered. And NONE of the param types are. That's because this doc string is broken in general.
The sphinx itself not being able to parse the doc string for azure-storage-blob. See our own github.io.

I am going to try a couple local sphinx runs. I'm thinking the easy answer will be to replace the " with ', but I'm digging and will submit a PR that resolves this on the packages that I can find that do this.
As for the "only some" function keyword params, I do have another PR that addresses other issues we've found as part of this investigation.


no applicable method for 'family' applied to an object of class "NULL"

Hello,
When I run scdesign3, the following error occurs
Input Data Construction Start
Input Data Construction End
Start Marginal Fitting
Marginal Fitting End
Start Copula Fitting
Convert Residuals to Multivariate Gaussian
Error in UseMethod("family") :
  no applicable method for 'family' applied to an object of class "NULL"

How can I solve it, thank you!

Hi Minghong,
Without your reproducible code, it would be challenging to debug. Could you please show your code?
The first thing you may want to try is to use a subset of your original data; sometimes, the data can be too large for your RAM.
Best regards,
Dongyuan

Hi Dongyuan,
I have the same question now when I am following the tutorial for batch effect simulation:
simu_res <- scdesign3(sce = example_sce,
                      assay_use = "counts",
                      celltype = "cell_type",
                      pseudotime = NULL,
                      spatial = NULL,
                      other_covariates = c("batch"),
                      mu_formula = "cell_type + batch",
                      sigma_formula = "1",
                      family_use = "nb",
                      n_cores = 1,
                      usebam = FALSE,
                      corr_formula = "1",
                      copula = "gaussian",
                      DT = TRUE,
                      pseudo_obs = FALSE,
                      return_model = FALSE)

And the error message is

Input Data Construction Start
Input Data Construction End
Start Marginal Fitting
Marginal Fitting End
Start Copula Fitting
Convert Residuals to Multivariate Gaussian
Error in UseMethod("family") :
no applicable method for 'family' applied to an object of class "NULL"

With macOS m1, and changed n_cores=1 in the function.
Best,
Danyang

Hi Dongyuan,
I have similar problems. Some of my datasets can run, but some of them cannot run. Could you help me check it plz?
Thank you so much!
real_sce <- readRDS("PDAC.rds")
sce_simu <- scdesign3(
    sce = real_sce,
    assay_use = "counts",
    celltype = "cell_type",
    pseudotime = NULL,
    spatial = c("row", "col"), # spatial location
    other_covariates = NULL,
    mu_formula = "s(row, col, bs = 'gp', k= 100)",
    sigma_formula = "1",
    family_use = "nb",
    n_cores = 1,
    usebam = TRUE,
    corr_formula = "1",
    copula = "gaussian",
    DT = TRUE,
    pseudo_obs = FALSE,
    return_model = FALSE,
    nonzerovar = FALSE,
    parallelization = "mcmapply"
  )

scDesign3 Simulation Start Input Data Construction Start Input Data Construction End Start Marginal Fitting Marginal Fitting End Start Copula Fitting Convert Residuals to Multivariate Gaussian Error in UseMethod("family") :    no applicable method for 'family' applied to an object of class "NULL" In addition: There were 20 warnings (use warnings() to see them)
PDAC.rds.zip

@littlecabiria Hi Cabiria,
I checked your data and it seems like a bug with usebam = TRUE. Can you set usebam = FALSE? It works for me on my computer. It is probably because the spot numbers are relatively small (~400); thus, the fast version (bam) may not work well since it needs subsampling.
Best,
Dongyuan

Hi Dongyuan,
Thank you so much! It works now!


Falcon 2326 Falcon Pull Request Builds failing

Two clashing versions of AbstractFalconAspect were coming in: One from falcon-cli and another from falcon-prism. The fix is to exclude falcon-metrics from falcon-cli dependency.
While I was at it, I also upgraded aspectj libraries, maven plugin and the java version.

See PR 402


Sync UI english text and the tutorial

See comments of https://github.com/cofacts/rumors-site/pull/405

Fixed in #415, closing


LRAUVState fields population status

This ticket keeps track of which fields in LRAUVState are populated, to eventually populate all of them. The LRAUVState is meant to be a mirror of SimResultStruct in the MBARI code base, which contains all the information MBARI’s original simulation provides to the control loop.
LRAUVState is defined in lrauv/lrauv_ignition_plugins/proto/lrauv_state.proto.
Checked box indicates populated.
Empty box indicates needs work.
(*) indicates populated but need modification (see section below).

[x] ignition.msgs.Header header

[ ] int32 errorPad_

[ ] int32 utmZone_

[ ] int32 northernHemi_

[x] float propOmega_

[ ] float propThrust_

[ ] float propTorque_

[x] float rudderAngle_

[x] float elevatorAngle_

[x] float massPosition_

[x] float buoyancyPosition_

[x] float depth_

[x] ignition.msgs.Vector3d rph_  // roll_, pitch_, heading_ in SimResultStruct

[x] float speed_

[ ] double latitudeDeg_(#90)

[ ] double longitudeDeg_ (#90)

[ ] float netBuoy_

[ ] ignition.msgs.Vector3d force_  // forceX_, forceY_, forceZ_ in SimResultStruct

[x] ignition.msgs.Vector3d pos_  // posX_, posY_, posZ_ in SimResultStruct (#48)

[x] ignition.msgs.Vector3d posRPH_  // posRoll_, posPitch_, posHeading_ in SimResultStruct

[x] ignition.msgs.Vector3d posDot_  // posXDot_, posYDot_, posZDot_ in SimResultStruct (#48)

[ ] ignition.msgs.Vector3d rateUVW_  // rateU_, rateV_, rateW_ in SimResultStruct

[ ] ignition.msgs.Vector3d ratePQR_  // rateP_, rateQ_, rateR_ in SimResultStruct

[x] float northCurrent_ (https://github.com/osrf/lrauv/pull/77)

[x] float eastCurrent_ (https://github.com/osrf/lrauv/pull/77)

[x] float vertCurrent_ (https://github.com/osrf/lrauv/pull/77)

[ ] float magneticVariation_

[ ] float soundSpeed_

[x] float temperature_ (https://github.com/osrf/lrauv/pull/77)

[x] float salinity_ (https://github.com/osrf/lrauv/pull/77)

[ ] float density_

[ ] repeated float values_  // Size 4 (https://github.com/osrf/lrauv/pull/77)

Populated but need modification:
Note that the coordinate frame convention interpreted by the MBARI controller is FSK.

[x] depth_ and rph_ are currently in the ROS coordinate frame convention - x forward, z up, y left. Need to change to the FSK convention that MBARI's controller uses - x fore, y starboard (right), z keel (down). (#48)

@braanan @arjo129

#81 is documenting on the message file itself which fields are populated, their units, frames of reference, etc. I suggest we close this issue after that's merged and track future changes on the message itself.


the messages creator needs optimization !

like making a new pdf file for every chat
and sort the messages from the newest to the oldest just like what the app does ..
and i don't think the images viewing system works as instagram made a lot of updates

should be fixed

should be fixed
@t0xic-m can we get an image of it working ? I CAN'T check it this while ;(

of course, here are 2 screenshots:
https://t0xic-m.github.io/instagram_json_viewer/screenshot1.jpg
https://t0xic-m.github.io/instagram_json_viewer/screenshot2.jpg
and a youtube video:
https://www.youtube.com/watch?v=0CUbTkrK62M

@t0xic-m hi great and all but why it's not clear ?
For messages it should be something like this
https://github.com/igdmapps/igdm
And i think every profile should have a different html file that contains everything about them.
Or every profile have a folder with html files of story and messages..etc

The messages section is basically the same as in the repo you linked. Just with more fancy design!
And there's not need for different profiles because when you download your data from instagram you only get your own profile!
Don't forget that this is not an instagram client, this is just an html interface that shows the json content in a human readable way!
But feel free to extend the python code to your own needs!

@t0xic-m
"download your data from instagram you only get your own profile!"
I meant the people  i am messaging. ..
And isn't that all too much for a one html file  ? Won't the browser crash and also using find takes forever and the browser becomes freezed.

Of course you could download every profile from the people you are messaging, but that's not what my goal was with this repo. But it is like 2 lines of python code to add if you want to do that, however I am pretty sure that instagram will deny your requests if you have to many conversations!
That's why every chat is in a separate html file, otherwise yes, your browser would probably crash!

Of course you could download every profile from the people you are messaging,

Mattte i mean the chat thread of them not their profiles !
"That's why every chat is in a separate html file,"
Is it ? This not what  i saw in the vid !

Maybe try running it first before you get more confused just by screenshots and a video ;)


Add additional snippets

I would love to see a few more snippets added to this plugin to help spped dev up a little. 2 immediate ones would be snippets to add the following.
import { graphql } from 'gatsby'

import { graphql } from 'gatsby'

See this pull request https://github.com/nickytonline/vscode-gatsby-snippets/pull/5

adding the gatsby image fragments as snippets would make this extension extremely useful. Right now it's not much use to me since I'm not using markdown. Any chance of adding those? I could do a PR though I'm not exactly sure what the snippets should look like

@TylerBarnes, thanks for your interest in the extrusion. I'd say, throw up a PR of what you think they should be and we can adjust them if need be.


Show "Custom" section in "Get Started" screen

Demo
~https://user-images.githubusercontent.com/43496356/228010939-427af9d3-cd2a-41f5-ac87-cb77c107a006.mov~
https://user-images.githubusercontent.com/43496356/228312421-5275a067-87cb-48e4-bdf8-5480c8bc2ad7.mov
Part of #3373

@julieg18 let's put this message after the existing plots and make it similar to the regular plots section. We can say something "Some plots require experiments to be selected. Add an experiment (can be a button)". We can also show a second button "+ Add" (to add a custom plot).

As discussed in the planning meeting, we're going to drop checkpoints, removing trends checkpoint plot types eventually. I'll  go ahead and remove the extra logic/ui surrounding the message when there are no selected experiments in this pr!


vNext Third Party Plugin Architecture - WIP/POC

EDIT:
This blew up a little... take a deep breath.
Docs are more notes at this point, and could use some tidying up.
New vNext plugin architecture, highly experimental.
Tools, mixins and modules can now be registered to a cornerstoneTools instance via these commands:
const cTools = cornerstoneTools.init();

cTools.thirdParty.registerTool(ExampleTool, 'Example');
cTools.thirdParty.registerModule(exampleModule, 'exampleModule');
cTools.thirdParty.registerMixin(exampleMixin, 'exampleMixin);

Note: Mixins are very limited in scope for now, as will become evident from the docs.
Also if you have a bunch of tools, mixins and modules that are related/depend on each other, you can stick them all in a plugin with the following schema:
const samplePlugin = {
  //Mandatory
  name: 'samplePlugin',

  // Optional
  mixins: [
    {
      mixin, // Mixin Object
      name   // Name of the mixin
    },
    ...
  ],

  // Optional
  modules: [
    {
      module, // Module Object
      name  // Name of the module
    },
    ...
  ],

  // Optional
  tools: [
    {
      tool:  // The tool Class / Constructor.
      name: // Name of the Tool.
    }
  ]

};

This plugin can then be registered like so:
const cTools = cornerstoneTools.init();

cTools.thirdParty.registerPlugin(samplePlugin);

Mixins and modules are plugged straight into cornerstoneTools ready for use. Tools from plugins can be enabled once registered by:
cTools.addTool(cTools.thirdParty.tools.HelloWorldMouseTool);

Please read the 'Third Party Addons' docs included for more info.
Feel free to give any feedback, this is very much a first draft/POC.

Hey @pieper , I didn't get that far on Friday. I'm going to make a hello world plugin with one tool, mixin and module, and stick it in an example on Monday 👍. I'll post in slack when it's up.

@pieper I made a simple plugin that logs to the console on click. The plugin contains one mixin, module and tool, that all interact.

We are going to merge this for now, however this framework will surely develop with time. We will review the plugin framework at RC-1 to see if any major revisions are needed before release.


PointCloud classification

Hi Nicholas,
Thanks for open-sourcing your code. I am using this method for point-cloud classification problem.
I wanted to know what classification head did you use in SHREC'11 dataset? The current DiffusionNet outputs at "vertices", "edges", or "faces". What if I want to output a single class for the whole point-cloud? Do I simply do mean over all point outputs? Did you try out different classification heads?

Hi,
I have trained the model and I want to use it on my point cloud data. I am stuck with an error that says "segmentation fault" but looking at the logs I see it is happening at get_operators (which calls compute_operators and all the way down to some robust_laplacian_bindings file). Any guess where I might be wrong? Is it possible to share your point cloud inference code with me?

@aunagar, sorry for bothering you, but do you remember how you did it?

Thank you both!

The robust_laplacian bindings (from this repo https://github.com/nmwsharp/robust-laplacians-py) are used to build the Laplace matrices for point clouds.
A segfault indicates something very bad is happening inside of that library (leading to an illegal memory access). This should definitely not happen under normal circumstances. Even with not-so-great data.
A few things to check:

Are you using the latest version of the robust_laplacian package?
Are you doing anything funky with your environment setup which might lead to code compiled on one machine being used on a different machine? (The robust_laplacian package contains compiled code under the hood, and using code compiled for the wrong machine can cause segfaults).
Does your point cloud data have anything especially weird about it, like NaN/inf values, or may points exactly on top of each other?

Yes, I have the latest robust_laplacian package. I am running the code on our hpc server. I don't know the inner workings of the environment exactly. There aren't any nan/inf or duplicates.

this is my code, residing in src folder. Any insight would be appreciated
import torch
import numpy as np
import diffusion_net
C_in = 3
C_out = 30
model = diffusion_net.layers.DiffusionNet(
C_in=C_in,
C_out=C_out,
C_width=64,
last_activation=lambda x : torch.nn.functional.log_softmax(x,dim=-1),
outputs_at='vertices')
checkpoint = torch.load('pretrained/diffusionnet_25jun.pth')
model.load_state_dict(checkpoint['model_state_dict'])
model.eval()
g = np.loadtxt('data/data.csv', delimiter=',', skiprows=1, usecols=(0, 1, 2))  # this has x, y, z coordinates of an object
verts = torch.from_numpy(g)
faces = torch.tensor([])
verts = diffusion_net.geometry.normalize_positions(verts)
frames, mass, L, evals, evecs, gradX, gradY = diffusion_net.geometry.get_operators(verts, faces)
features = verts
outputs = model(features, mass, L=L, evals=evals, evecs=evecs, gradX=gradX, gradY=gradY, faces=faces)
print(outputs)


package-info should not be reported as dead code

deadcode4j reports that package-info as unused classes. Those should probably be excluded from analysis.

What is in those classes?
Is it documentation only? Put it into package.html instead (though I admit that this is a matter of taste).
Annotations? deadcode4j considers common annotations (e.g. Hibernate) already. You can configure deadcode4j to mark classes as live or, if they're commonly used, file a PR to consider those as well.
I wouldn't ignore them by default; maybe provide a configuration parameter to ignore them...

In my specific case, those are used to specify JSR305 annotations at package level. Reading the JLS section 7.4.1 leads me to think that there is no valid use case having an actual class in a package-info.java file (a class named package-info would not be valid). But I might be lacking imagination...

Ah, I see (using JSR305).
You're right in that deadcode4j may be misleading since it treats package-info.java files as classes (although I believe the compiler really creates a class for such files); the idea behind that is to determine if such files could be removed as well (no matter if they result in a class file or not). E.g. by applying a script that removes specific annotations (or similar automated file manipulations), one could end up with a leftover package-info.java file that no longer serves any purpose - and deadcode4j tries to identify such "unnecessary" files.

Ok, I see the point. I'm not sure what solution would make sense in that case... Using the JSR305 to tag them as live does not seem right, since "real" classes could also be tagged with JSR305 and be dead. I don't know enough about deadcode4j to propose a solution, I just know enough to see that there is a problem... sorry :)

No worries, this is simply a scenario I didn't run into yet, but valid by its own right.
One could easily add an option to ignore package-info classes; or adapt the classesToIgnore parameter to accept regular expressions...


Philips hue: warning: api version 1.20.0

I'm getting this warning.
Don't think it's causing issues but wanted to bring it to your attention.
Philips hue: warning: api version 1.20.0

Indeed, no changes that impact homebridge-hue.  Warning no longer appears in v0.5.20.


PostgreSQL Can't navigate with composite foreign key referencing unique constraint

System information:

Operating system (distribution) and version: macOS 11.2.3
DBeaver version: <IP_ADDRESS>106012023

Connection specification:

Database name and version: PostgreSQL 11.7
Driver name: PostgreSQL 42.2.22

Describe the problem you're observing:
Trying to navigate with foreign key to the record in another table fails with an error:
'Navigate association' has encountered a problem.

Entity [relation_test.parent] association [child_fk] columns differs from referenced constraint [parent_pk] (2<>1)

Table from which I'm trying to navigate contains a composite foreign key that references unique constraint on two columns, but it looks like the navigate association tries to use primary key instead. Error occurs when I'm trying to navigate from the first table into a second one using:

navigate link icon next to the value in one of the columns with foreign key
Navigate -> Navigate link from context menu
Navigate -> References -> %foreign key reference name%

Steps to reproduce, if exist:
Create a parent table with a primary key and a unique constraint on two other columns, insert a record into it.
Create a child table with a composite foreign key on the parent using two columns that reference parent's columns that are part of unique constraint, insert a record into it.
Try to navigate from child table into a parent one with a foreign key.
Tested with the following structure and data:

Root grandparent table, that will be referenced from two other tables:

CREATE TABLE grandparent (
	id int8 NOT NULL,
	"name" varchar NULL,
	CONSTRAINT grandparent_pk PRIMARY KEY (id)
);

INSERT INTO grandparent (id, "name") VALUES(1, 'Grandparent record');

parent table that contains a unique constraint on two columns and will be referenced via composite foreign key:

CREATE TABLE parent (
	id int8 NOT NULL,
	grandparent_id int8 NOT NULL,
	"name" varchar NULL,
	CONSTRAINT parent_pk PRIMARY KEY (id),
	CONSTRAINT uq_parent_id_grandparent_id UNIQUE (id, grandparent_id)
);

ALTER TABLE parent ADD CONSTRAINT parent_fk FOREIGN KEY (grandparent_id) REFERENCES grandparent(id);

INSERT INTO parent (id, grandparent_id, "name") VALUES(10, 1, 'Parent record');

child table that contains composite foreign key on parent:

CREATE TABLE child (
	id int8 NOT NULL,
	parent_id int8 NOT NULL,
	grandparent_id int8 NOT NULL,
	"name" varchar NULL,
	CONSTRAINT child_pk PRIMARY KEY (id)
);

ALTER TABLE child ADD CONSTRAINT child_fk FOREIGN KEY (parent_id,grandparent_id) REFERENCES parent(id,grandparent_id);

INSERT INTO child (id, parent_id, grandparent_id, "name") VALUES(100, 10, 1, 'Child record');

Include any warning/errors/backtraces from the logs
2021-06-20 17:48:47.133 - Error during job 'Navigate association' execution
org.jkiss.dbeaver.DBException: Entity [relation_test.parent] association [child_fk] columns differs from referenced constraint [parent_pk] (2<>1)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetViewer.navigateAssociation(ResultSetViewer.java:3109)
	at org.jkiss.dbeaver.ui.controls.resultset.spreadsheet.SpreadsheetPresentation$6.run(SpreadsheetPresentation.java:1188)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

thanks for the report

Fixed

Verified


Can't upload ZIP files

Hi Jamf developer team
I can't upload zip file with jamf sync. In Jamf Admin, I could upload zip file.
When I put zip file into network volume, it disappears ....
Thanks

I've noticed that .zip and .dmg files are no longer showing on the file list, even if they exist on the server. The file may have been uploaded but just being displayed.

What kind of zip files did you need to upload? Jamf Admin allowed uploading of older non-flat packages, and it would convert those to zip files. I believe the original extension was still in the file name and ".zip" was appended. Jamf Sync will only show zip files with ".pkg.zip" and ".mpkg.zip" extensions. Jamf Admin would also allow you to upload .app files and it would zip them, however, that was in order to upload macOS installers, which is no longer relevant after macOS 10.13 I believe.

examples : pkg as disrectory : jamf admin compress it in zip file. Trend AV need a file near pkg to install computer in managment app. Zip file in a rule are "hidden" to be use with a script
Thank-you

Jamf Sync will compress pkg's and mpkg's that are directories into a zip file with a .pkg.zip or .mpkg.zip extension. As long as zip files have one of those two extensions, they should show up in the list. I tested it with the latest version and it's working that way.

Closing this ticket since I wasn't able to reproduce the issue and I haven't gotten further information. The last comment mentioned a script...if you have a specific example, feel free to reopen it with additional information.


How to find deCONZ (on Raspi) in iobroker (on Synology docker)?

Hello!
I run deCONZ on a Raspberry Pi 2 with phoscon as GUI and all works well. Now I installed iobroker in a Synology docker, but do not know if/how to find deCONZ in iobroker. I understood that there should be a deCONZ adapter, but when I search, nothing is found on my local LAN (despite PC, Raspberry Pi 2, and Synology are on the same subnet). I even unlocked the deCONZ/phoscon gateway "to allow 3rd party apps to register themself to the gateway", but that did not help either.
Am I maybe misunderstanding something completely? Thank you so much for sharing your experience and help.

Please show me your adapter configuration.

Hello @7wells ,
you open the configuration-page in iobroker (adapterz -> deconz) and use the IP of your Raspberry Pi 2 and the port of the gateway. In parallel you activate the 3rd party app registration in your phoscon.
Regards

Hello @7wells ,
you open the configuration-page in iobroker (adapterz -> deconz) and use the IP of your Raspberry Pi 2 and the port of the gateway. In parallel you activate the 3rd party app registration in your phoscon.
Regards

Thanks, guys for replying. Please accept my apologies that I did not provide more information. It is because meanwhile I have given up, i.e. I just use it as before (without iobroker or docker).

Thanks, guys for replying. Please accept my apologies that I did not provide more information. It is because meanwhile I have given up, i.e. I just use it as before (without iobroker or docker).


Rename Meeting Template to prevent naming clash

Rename to Meeting Template so that it does not clash with One-on-One Template.
Current behavior makes the actual 1:1 Template unreachable when inserting a template into a page. Instead, I get the Meeting Template due to the name clash.
After this change, the 1:1 Template can be inserted correctly.

Thank you!


Release azure-cli-acs 0.1.1b1

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Release of 'azure-cli-acs' with version '0.1.1b1' successful. View at https://pypi.python.org/pypi/azure-cli-acs/0.1.1b1.

GitHub release created. https://github.com/Azure/azure-cli/releases.