Charts are visibly refreshing when updated

On version 3.3 this is not happening.
From version 3.4 it almost looks like that updated charts disappear and reappear in a blink of an eye, but it is visible.

I got the exact same problem.
I'm using Highstock with react-jsx-highcharts.
Navigator is enabled (set to 1mn).
LineSeries work well.
But ColumnSeries refresh the whole chart.

Apologies for the late reply, my son was born 5 weeks ago, and I haven't had time to look after my open source stuff. Looking into this now.

@pinkynrg can you please give more information on how to reproduce what you are seeing. Ideally a CodeSandbox/Stackblitz reproducing the problem, or even information on which series types/browser you are using.
@pinkynrg and @kMeillet by any chance, is the issue you're seeing on Firefox or Safari? Just wondering if it's related to this bug https://github.com/highcharts/highcharts/issues/9245

Right now I don't have a lot of time but I can tell you right now that the
issue is happening on Chrome too.
On Sun, Jun 2, 2019, 3:24 PM Will Hawker<EMAIL_ADDRESS>wrote:

@pinkynrg https://github.com/pinkynrg can you please give more
information on how to reproduce what you are seeing. Ideally a
CodeSandbox/Stackblitz reproducing the problem, or even information on
which series types/browser you are using.
@pinkynrg https://github.com/pinkynrg and @kMeillet
https://github.com/kMeillet by any chance, is the issue you're seeing
on Firefox or Safari? Just wondering if it's related to this bug
highcharts/highcharts#9245
https://github.com/highcharts/highcharts/issues/9245
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/whawker/react-jsx-highcharts/issues/209?email_source=notifications&email_token=AALH4TZ7PFVLOB3HULDFWH3PYQSNJA5CNFSM4HMLDDL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWX5BSQ#issuecomment-498061514,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALH4TZ7SISVIYH5G4HVFADPYQSNJANCNFSM4HMLDDLQ
.

How many data points do the series on your charts general have? Are you using Highcharts boost module?

Generally not more than 300 points per chart so no, I'm not using Highcharts boost, since I don't think the data load is high enough to justify its use.

Would either of you be able to post an example of your code? It's difficult to debug in the dark here.

Here's a reproduction: https://stackblitz.com/edit/base-chart-example-h2gjjk

Thanks @jhartling, I have managed to reproduce on vanilla Highcharts, so I have raised the follow issue https://github.com/highcharts/highcharts/issues/10883


[Geeks For Geeks] Lucky alive person in a circle.

Aim

will be adding solution for lucky alive person in a circle. in GFG folder.

Details

Level of the Question - Medium.
Topic - Bit Manipulation

Problem Statement :-

Given N people standing in a circle where 1st is having a sword, find the luckiest person in the circle, if, from 1st soldier who is having a sword each has to kill the next soldier and handover the sword to next soldier, in turn, the soldier will kill the adjacent soldier and handover the sword to next soldier such that one soldier remains in this war who is not killed by anyone.

Please assign me this issue, LMG SoC'21 Participant.

Programming language

[ ] C
[ ] C++
[X] Java
[ ] Python

@Saurabh4626 @ravikr126 @Ayush7614 Please Assign me this issue. I will solve it in java.
LGM-SOC'21
Thank You

please assign me the above problem statement in python

please assign me this issue in C++. Thank you

I would like to solve this issue in python

@Saurabh4626 @ravikr126 @Ayush7614 Please Assign me this issue. I will solve it in C.
LGM-SOC'21


ci: restore slack notifications with new token

What ❔
Restores proper sending of Slack notifications about new releases after ITS-392 fix.

Why ❔
To notify the other teams about new zksolc releases.

Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.

Sorry, I’m not qualified to comment.


Add gcc 9.2 for C too

Note: untested.

Whoops! Thanks for this, I forgot :blush:

You're welcome! I wonder if we should either have a test-step to about forgetting or generate the file (parts of it) automatically.


Trying the Demo - It is asking for openAIApiKey but that's not in the instructions

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

azd auth login
azd up
environment name
Subscription
Location
then it asks for
Enter a value for the 'openAIApiKey' infrastructure parameter:
but nothing in the documetation for this
Any log messages given by the failure

The Help option doesn't tell me anything other than
Hint: OpenAI API Key
Expected/desired behavior

Move on to the next step of the demo
OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)
Windows 11 Pro

Versions

23H2
Mention any other details that might be useful
Please help me!
I'm so excited to learn AI!
My company really REALLY needs me to learn all this.
Thanks!

Thanks! We'll be in touch soon.

dupe of #318
@KristinBethea  please close this issue :)


[Monitor] Documentation updates

Add a section about log retrieval to ingestion README
docs.microsoft.com -> learn.microsoft.com

Ref: https://github.com/Azure/azure-sdk-for-python/issues/26099

API change check
API changes are not detected in this pull request.


Add/Update CODEOWNERS

Add line to CODEOWNES, or add file if it doesn't exist. Issuse https://github.com/cyberark/community/issues/70

Why do we need all these approvals from a specific person?
Can't the team approve it? they are the owners of the code and its quality

The team can approve almost all changes themselves. The 3 files that have roles assigned to them have that approval for helping ensure consistency across all repos.  For the .trivyignore, it's also to make sure that Shahar or I understand why a CVE is being ignored and that there's an appropriate comment because one or the other of us is likely going to have to explain why we still have the 3rd party vuln we're ignoring at some point in the future long after anyone who works on it will remember why we're not vulnerable.

Failing test doesn't look related


As an author I should be able to browse the public posts of everyone

4

5

4/10

5/10

5/10

Average 4.6


Retain shipping option on refresh

This refactors the shipping and shipping options views to retain choices made by the user in the event of a purchase view refresh, and handles situations where the old selections are no longer valid.
This solves a number of edge cases where the user could end up with nothing selected or an invalid selection if they did something unexpected while changing their addresses in Settings.
Closes #1375

@rmisio this is ready for review again, everything should be in working order.


Fix billing query

Fix appcat:metering query for the cronjobs
Increasesed quotas for objects according to https://kb.vshn.ch/appuio-cloud/how-to/day2ops/override-default-quota.html
Decreased the amount of failed and successful jobs to be retained.

Checklist

[x] The PR has a meaningful title. It will be used to auto generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[ ] Link this PR to related issues or PRs.

By default was it 59 hours with one hour step?

By default was it 59 hours with one hour step?

minutes, everything should be in minutes, and yes it's 59 minutes due to some issues with promql when setting 60m

Once we verified that the metrics are correct, we need to backfill the weekend as well.


Completed Timer/Histogram Utility

As mentioned in https://github.com/gwAdvNet2015/gw-kv-store/issues/34.
I will be making an edit shortly to fix some formatting issues with certain files.

@Chryel -- I'm assuming you guys will also PR this into the gw-kv repo.

@twood02 this was added in https://github.com/gwAdvNet2015/gw-kv-store/commit/8907b9e8bea98d413d7b8e4f3db104d8d1295d1e

@Chryel - Does this version have any fixes that had to be made for the gw-kv version?  Is it ready for me to merge this here?

The version for adv-net-samples is up to date. I just migrated the files to the gw-kv repo from here.

ok, I've merged with adv-net-samples


Rework nav menu and homepage registry data to not be part of global template context

Proposed Changes
Either:

Remove the nautobot.extras.registry.registry from the global Nautobot template rendering context altogether and provide templatetags that can instead be used to reference it as needed in specific contexts
Omit the nav_menu and homepage_layout registry data from the template rendering context at least.

Justification
The global view template rendering context currently includes the entire nautobot.extras.registry.registry dictionary, which currently includes all data under the nav_menu and homepage_layout keys. When developing/debugging view templates using the django-debug-toolbar, this results in literal pages of context data, making for a subpar developer experience. The nav_menu data should only be needed when rendering inc/nav_menu.html and the homepage_layout data should only be needed when rendering the actual Nautobot home page; in all other templates these are just noise.

Closed in #2017


Be more responsive-friendly?

Currently we have to pass the width of the table into the component on initial rendering, and the resulting table has a fixed width which does not response to window resizing etc. Is it possible to make the component more responsive without compromising the performance?

You can do what I did: make the table resize to the containing div: https://github.com/ofersadgat/fixed-data-table-simple/

@ofersadgat That's a lot of boilerplate code…
I feel like being responsive-friendly should be built in, especially since the examples adapt to the container size.

+100

To be fair, most of it isnt actually boilerplate code. You can use fitted table directly in your code without modification.

Here's an example just using a window.resize handler, it's essentially what the example pages do.
https://gist.github.com/parties/a67c75c2ccc5ee6fe48a

That example doesnt really work because it wont tell you if only a component resizes but the outermost doesnt. The way that the example handles this case is by manually refreshing and relying on React to not rerender if nothing changed.

Thanks for bringing this up. Making width an optional field would be a great addition to the table. Otherwise, I would recommend using a window.resize handler as mentioned by @parties to explicitly update the width of your table.

I had this same problem and couldn't find any existing module to solve it, so I had a go at writing one: https://github.com/digidem/react-dimensions
It's aim is to get the dimensions of the containing element and pass them to fixed-data-table, and the dimensions when the window is resized.
Not that the containing element must have a height, or nothing will be rendered. The README isn't great, sorry about that.
You can use it as a higher order component or an ES7 decorator.
Let me know how it works and whether it is useful. My first go at a React component, so apologies for any silly mistakes.

This would really be a great enhancement. Modern websites are responsive. I spent hours trying to make my fixed-data-table responsive in many different ways, and still it is a hundred times slower than the examples page, and I cannot figure out why I can make it work for width but not for height. It is a lot of code that could be common to all fixed-data-table users.

Check https://github.com/digidem/smart-data-table/blob/master/src/data-table.jsx for an example about how to do this with https://github.com/digidem/react-dimensions
I really need to write better docs / example, but the short answer is that react-dimensions is a Higher-Order component that can be used like a mixin to add this.props.containerWidth and this.props.containerHeight to the component. Something like this:
import { Table } from 'fixed-data-table'
import GetContainerDimensions from 'react-dimensions'

class DataTable extends React.Component {
  render () {
    return (

        {columnNodes}

    )
  }
}

export default GetContainerDimensions()(DataTable)

I tried this, but it does not get rendered, although I did set a height for the parent component. What am I doing wrong ? My render function:
var Dimensions = require('react-dimensions');
...

    var tab = Dimensions()(

        {cols}

);

    return

        {tab}

;

And this says: "Illegal export declaration":
var SamplesTable = React.createClass({
...
    return

        {cols}

);

;

export default Dimensions()(SamplesTable)

There are a lot of things I don't understand in your package.

I mean, it looks like I have to transform all of my code to make it fit in I have no idea how with an extended Component or something, while it should just be available as simple as a 'table-responsive' bootstrap class.

Hi @muraveill, my apologies that the package is confusing. Your second attempt is close, but it's not working because I included some ES6 code in the example, my apologies. Try changing this line:
export default Dimensions()(SamplesTable)

to
module.exports = Dimensions()(SamplesTable)

I don't get why we need to export it. Couldn't Dimensions() just return a wrapped react component instead of an obscure function ? I already have a module.exports = SomeOtherComponent, and when I put them both in an object for the export, it does not seem to work.

Hi @muraveill, Dimensions()(SomeOtherComponent) returns a wrapped react component. The API might have been simpler as just Dimensions(SomeOtherComponent) but I wanted to be able to define some options Dimensions(options)(SomeOtherComponent). You should not export twice, just export the wrapped component. This might help: https://medium.com/@dan_abramov/mixins-are-dead-long-live-higher-order-components-94a0d2f9e750

Using the Dimensions workaround is just that. It shouldnt be brought up here as a solution to the obvious flaw in this Table component that only uses fixed width and sets it with inline styles no less!
It should be possible to render the table so it spans the width of its container.
+100 for this to be added.

I don't really understand why these dimension settings are set here anyway. Look and feel should be handled in your own CSS files. The way it is now, the inline styles can't even be overwritten.

+1 for reasons pointed out by @muraveill. I don't want to take a performant table, only to butcher it with my own rendering requirements.

Note that the react-dimensions package works fine now, this is what I am using in the end. Still, it should be a straightforward feature of fixed-data-table.

+1 for react-dimensions as the solution. Using this as well.

+1 Responsive friendly should be built in.
Using window resize handler is a poor man's approach. There will be cases that the table size need to be adjusted due to the dimension changes of neighboring elements (such as shown/hidden). In these scenarios, window resize handler won't work, and it's messy to use custom inter component communication.

+1

+1 for react-dimensions  but yes as everyone is saying, it should be built in.

+100

+1

I don't see why it is so hard to implement a reactive container for your table component? I was able to accomplish this in a really trivial way. Simply add/remove event listeners using the react lifecycle methods and update a state property that is tied to your table's width. The team could build them in, yes, but for now everyone could just try using a little bit of vanilla JS to accomplish fluid tables with minimal effort.

@ryanhca

Nobody said it was hard.
Should every developer really rewrite again his own version of the exact same functionality?
Would you trust every one of your employees on implementing it correctly (what is debounce again)?
Minimal effort !== no effort, and it accumulates with thousands of users.
There is no discussion about it, it is just what a library is made for. react-dimensions is one, and it works. The only question is could it be part of the core code instead of a separate dependency.

One thing I love about React is the composibility of its components and the power of higher order components.  I think this is a great case for using that since the existing FDT API already supports the necessary functionality for resizing.
@jdelafon your points regarding rewriting functionality and the effort it takes to find and learn a solution are valid.  I think a good way to address them is to create an example that demonstrates using a HOC like react-dimensions and add it to our FDT2 fork.  I'll attempt to do this over the next week..
I don't think there's value in integrating the solution directly into FDT / FDT2 since FDT isn't currently maintained, and FDT2 only has a few part time maintainers.  That won't address the issues around getting the HOC to work with an existing code base or handle specific edge cases, but maybe we can discuss and address those as they arise.
Do folks feel this is a good direction for the repo?

@jdelafon I just don't see why the solution needs to be a global one. I do trust all of the employees I work with. And by seeing all the complaints here about it not being in the core of the package I figured everyone was having a hard time with it. Sorry if my comment came at you in a rough way this morning. As for your comment ('it is what the library is made for)  I have to respectfully disagree. The library was made for displaying massive amounts of data in a table which they have accomplished at FB and now by the team maintaining the library. Everything else they've managed to accomplish has been sugar on top. You've asked for solutions to your problems and you've received them, why do you need someone else to code them for you?

We are using fixed-data-table extensively and managing it's width using higher order components. I think it is not a problem for many other users as well.
Lot's of people complain that is not maintained, but it already does what it was designed to do. And does it well.

@tkirda I agree this library does an excellent job of handing large amounts of day, but there are a lot of small bugs that are clearly broken.

An example for doing responsive resizing with react-dimensions is now available here:
http://schrodinger.github.io/fixed-data-table-2/example-responsive.html

I think theres a bug in the ^example when you scroll down the page the whole table scrolls above the navigation.

Ah thanks for pointing that out.  That's an issue with the left navigation and example pages layout for FDT.  The example should be fine to use though.  I'll get the left nav issue fixed shortly.

Should be fixed now.  Thanks again for pointing that out!

Is anyone has a typescript definition for react-dimensions? I'm trying to create one but still having some issues in the code. My table needs to be responsive when both side panels are expanded. I have toggleable panels in both sides of the table and I'm having hard time to make the table responsive. I'm using react fixed-data-table in typescript ES6.


Verify prefix for secret name

Problem:
If secretInfo has a colon in it, the split does not work properly
Solution:
A colon is a valid value so check if the secret is prefixed with the namespace where the secret lives
https://github.com/rancher/rancher/issues/19202

LGTM


feat: add pre-commit hook

Add a .pre-commit-hooks.yaml file so that regal, more specifically regal lint can be utilised as a pre-commit hook via the popular pre-commit project.

Hey Adam! And thanks for this 👍 I'll take a closer look tomorrow
language: golang

name: policy file lint
description: policy file linting with regal from Styra
entry: regal lint
files: (.rego)$

Sorry, not sure why this got closed as I submitted the comments. That was not my intention.

Once this is merged, I have a follow-up PR with a regal-download hook which fetches the binary from Github.


changes in  back to top icon

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Here I have changed the back-to-top icon (the icon you wanted)

Hello mam, @neha3423, Please review this code and approve it

This is not a helpful change.

@spicycode


Update Documentation : Update User Guide

Codecov Report

Merging #292 (05e9758) into master (704b7bb) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 05e9758 differs from pull request most recent head 32ba3b2. Consider uploading reports for the commit 32ba3b2 to get more accurate results

@@            Coverage Diff            @@
##             master     #292   +/-   ##
=========================================
  Coverage     63.29%   63.29%
  Complexity      957      957
=========================================
  Files           169      169
  Lines          3152     3152
  Branches        325      325
=========================================
  Hits           1995     1995
  Misses         1037     1037
  Partials        120      120

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 d9cedcc...32ba3b2. Read the comment docs.


install-orchestrator script should wait until user authorize js9 docker as a zerotier member instead of exit with error

install-orchestrator should wait until user authorize js9 docker as a zerotier member instead of exit with error

Please try the install from master-orchestrator-ztinstall branch to confirm it fix the issue

version
orchestrator: 6b78e46


chore: release main

:robot: I have created a release *beep* *boop*
@opentelemetry/instrumentation-nestjs-core: 0.28.1
Bug Fixes

opentelemetry-instrumentation-nestjs-core: copy metadata to wrapped handler (#796) (2c4a834)

This PR was generated with Release Please. See documentation.

@open-telemetry/javascript-approvers I need an approval to merge this

:robot: Release for @opentelemetry/instrumentation-nestjs-core is at https://github.com/open-telemetry/opentelemetry-js-contrib/releases/tag/instrumentation-nestjs-core-v0.28.1 :sunflower:


Interactable not responding to menu, grip take 2. Also add TouchHandler

Overview
Fixes: #4492 by changing Interactable to listen to OnInputDown instead of OnPointerDown. I did this because pointer events are designed to only be sent for certain buttons, in most cases the 'select' button. So pointers do not send events when menu button, grip button, any other button is pressed. Interactables however want to listen to all possible buttons, so they need to listen to OnInputDown.
I needed to do some potentially confusing logic to ensure that the poke pointer does not cause "select" action to trigger interactable when you are near a button and air tap. This happens because when an input does an input down like air tap, that input gets dispatched to every object focused by every pointer attached to that input, even if the PokePointer shouldn't really be dispatching air taps.
I also made a few other improvements:

Add a basic TouchHandler class to expose touch events as unity events
Expose SetInputDown() and SetInputUp() events for Interactable to let more complex logic set the states of the Interactable.
Add more details to the TouchEvent data structure
Fixes: #4611 by listening for touch events from children
Update Interactable prefabs and examples examples to work with touch events out of the box (fix up the NearInteractionTouchables).

Verification

Tested that Interactables respond to grip, menu buttons in Windows MR with motion controllers.
Test in editor that handinteractionexamples scene works
PlayMode tests pass.
Testing on HoloLens currently

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/4673/head:name_of_local_branch
git checkout name_of_local_branch

I think this change could also fix #4611 because the change is updating interactable to listen to events from children.

/azp run mrtk_pr

/azp run mrtk_docs

/azp run mrtk_pr


CU-86drpnc78 - Refactor test_nep17.py and test_simple_nep17.py to use BoaConstructor

Summary or solution description
Refactored test_nep17.py and test_simple_nep17.py files to use BoaTestCase in place of BoaTest for unit testing.

Task linked: CU-86drpnc78 Refactor example_tests/test_nep17.py and test_simple_nep17.py to use BoaConstructor

coverage: 92.111%. remained the same
when pulling a3d0749a53a48eb8a1ff4089e08dbf8dedec7db3 on CU-86drpnc78
into fa4b67bd7bbed68acd10b282220fb61cc20b1ca6 on development.


when  run " bash checkpoints/download.sh"

when i run " bash checkpoints/download.sh"there is a error:
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to www.dropbox.com port 443: Connection refused
Archive:  pascal_ResFCN.zip
End-of-central-directory signature not found.  Either this file is not
a zipfile, or it constitutes one disk of a multi-part archive.  In the
latter case the central directory and zipfile comment will be found on
the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of pascal_ResFCN.zip or
pascal_ResFCN.zip.zip, and cannot find pascal_ResFCN.zip.ZIP, period.
mv: cannot stat 'pascal_ResFCN/State_Dicts/best_model.pth': No such file or directory
mv: cannot stat 'pascal_ResFCN/history.json': No such file or directory

do you fix this problem?


Signing in Error : Application with identifier 'abcdd' was not found in the directory

While signing in, application says that, the registered app is not found in the active directory web apps, even though app is already present.
The application is already registered using https://apps.dev.microsoft.com and it's available in the listing

Able to fix this issue. hence closing.


Unhandled exception: Premature end of stream

crit: IdentityServer4.Hosting.IdentityServerMiddleware[0]
      Unhandled exception: Premature end of stream
PeterO.Cbor.CBORException: Premature end of stream
   at PeterO.Cbor.CBORReader.ReadByteData(Stream stream, Int64 uadditional, Stream outputStream)
   at PeterO.Cbor.CBORReader.ReadStringArrayMap(Int32 type, Int64 uadditional)
   at PeterO.Cbor.CBORReader.ReadInternal()
   at PeterO.Cbor.CBORReader.ReadStringArrayMap(Int32 type, Int64 uadditional)
   at PeterO.Cbor.CBORReader.ReadInternal()
   at PeterO.Cbor.CBORReader.Read()
   at PeterO.Cbor.CBORObject.Read(Stream stream, CBOREncodeOptions options)
   at PeterO.Cbor.CBORObject.DecodeFromBytes(Byte[] data, CBOREncodeOptions options)
   at PeterO.Cbor.CBORObject.DecodeFromBytes(Byte[] data)
   at Fido2NetLib.Objects.CredentialPublicKey..ctor(Byte[] cpk)
   at Fido2NetLib.AuthenticatorAssertionResponse.VerifyAsync(AssertionOptions options, String expectedOrigin, Byte[] storedPublicKey, UInt32 storedSignatureCounter, IsUserHandleOwnerOfCredentialIdAsync isUserHandleOwnerOfCredId, Byte[] requestTokenBindingId)
   at Fido2NetLib.Fido2.MakeAssertionAsync(AuthenticatorAssertionRawResponse assertionResponse, AssertionOptions originalOptions, Byte[] storedPublicKey, UInt32 storedSignatureCounter, IsUserHandleOwnerOfCredentialIdAsync isUserHandleOwnerOfCredentialIdCallback, Byte[] requestTokenBindingId)

fail: Microsoft.AspNetCore.Server.Kestrel[13]
      Connection id "0HM844518RFR3", Request id "0HM844518RFR3:00000001": An unhandled exception was thrown by the application.
PeterO.Cbor.CBORException: Premature end of stream
   at PeterO.Cbor.CBORReader.ReadByteData(Stream stream, Int64 uadditional, Stream outputStream)
   at PeterO.Cbor.CBORReader.ReadStringArrayMap(Int32 type, Int64 uadditional)
   at PeterO.Cbor.CBORReader.ReadInternal()
   at PeterO.Cbor.CBORReader.ReadStringArrayMap(Int32 type, Int64 uadditional)
   at PeterO.Cbor.CBORReader.ReadInternal()
   at PeterO.Cbor.CBORReader.Read()
   at PeterO.Cbor.CBORObject.Read(Stream stream, CBOREncodeOptions options)
   at PeterO.Cbor.CBORObject.DecodeFromBytes(Byte[] data, CBOREncodeOptions options)
   at PeterO.Cbor.CBORObject.DecodeFromBytes(Byte[] data)
   at Fido2NetLib.Objects.CredentialPublicKey..ctor(Byte[] cpk)
   at Fido2NetLib.AuthenticatorAssertionResponse.VerifyAsync(AssertionOptions options, String expectedOrigin, Byte[] storedPublicKey, UInt32 storedSignatureCounter, IsUserHandleOwnerOfCredentialIdAsync isUserHandleOwnerOfCredId, Byte[] requestTokenBindingId)
   at Fido2NetLib.Fido2.MakeAssertionAsync(AuthenticatorAssertionRawResponse assertionResponse, AssertionOptions originalOptions, Byte[] storedPublicKey, UInt32 storedSignatureCounter, IsUserHandleOwnerOfCredentialIdAsync isUserHandleOwnerOfCredentialIdCallback, Byte[] requestTokenBindingId)

I am getting this error only when I authenticate using Platform with Windows Hello PIN.
When I use Platform with Android browser works without any problem and in Cross-platform with Yubikey also works, only the Platform with Windows Hello PIN give me that error.
Platform with Windows Hello PIN:
Assertion Options:
{
    "challenge": "0Suv6zwHC9f_ZPsp2yH1-Q",
    "timeout": 60000,
    "rpId": "exemple.com",
    "allowCredentials": [
        {
            "type": "public-key",
            "id": "joXfhGtqNMdEYt_C9vuqlZ1xXpujTTw3sY2w3-SucBE",
            "transports": [
                "internal"
            ]
        },
        {
            "type": "public-key",
            "id": "d6cUunNV6WWOw_2ruO7uTBLxnbt9Ek-x1OwFL1fkjYH9ecMfuhxmDT75uSrZqUZizDmfE7iofv_1ZeVNg4IgbQ",
            "transports": [
                "ble",
                "nfc",
                "usb"
            ]
        }
    ],
    "userVerification": "discouraged",
    "status": "ok",
    "errorMessage": ""
}

Authenticator Attestation:
{
    "id":"joXfhGtqNMdEYt_C9vuqlZ1xXpujTTw3sY2w3-SucBE",
    "rawId":"joXfhGtqNMdEYt_C9vuqlZ1xXpujTTw3sY2w3-SucBE",
    "type":"public-key",
    "response":{
            "clientDataJSON":      "eyJ0eXBlIjoid2ViYXV0aG4uZ2V0IiwiY2hhbGxlbmdlIjoiMFN1djZ6d0hDOWZfWlBzcDJ5SDEtUSIsIm9yaWdpbiI6Imh0dHBzOi8vZWx0b25wYXN0aWxoYS5jb20iLCJjcm9zc09yaWdpbiI6ZmFsc2UsIm90aGVyX2tleXNfY2FuX2JlX2FkZGVkX2hlcmUiOiJkbyBub3QgY29tcGFyZSBjbGllbnREYXRhSlNPTiBhZ2FpbnN0IGEgdGVtcGxhdGUuIFNlZSBodHRwczovL2dvby5nbC95YWJQZXgifQ",
            "authenticatorData": "JODq-RrrOA6E2bL8N35CWgjO8ZWu54VrEh_dsgXH_uQFAAAAEA",
            "signature": "WsyTo_vKVe1G5lGDYtZlLA1vF0cr6Ir6STX8XyQNl_NeJZ6-ILCWGvzaWT__mNO3X_CssWtkqjvLvV3NxK6T_U_WsIRYk-SAD-igROXqGe9Yv8LlUuOoH9N2MoKZGnFa5oPpN0IWoWdbKmyLZPCIfS8Sqhxw6U8Z4NeBILBNAPJCUvDo6hCb87Xdi7MgN3Bm_h0Lhe8qZhj-m2nJQ4u0HdVHvp2kkrGZgTAdPPXBaiobs5NDF_7CbafUYyGoy0naPPjwL9i7pjOm931V8rjcIrWpiz2iz5LsD8mfp7pQWMbY5zks5Yfxm1TctF_opHbLRPTFfD7BCEuI0qVWVlf0Eg",
            "userHandle": "QfY7MV5-ek-k46zHAVZjvw"
    },
    "extensions":{}
}

Does anyone know what is happening? From what I test maybe the clientDataJSON is giving problems.

The thing you have labeled "Authenticator Attestation" is actually an assertion, and it looks pretty reasonable.  I am pretty the issue is going to be the stored public key, it is checked and loaded here.  It must be corrupted to the point where it is no longer valid CBOR.  How are you storing the public keys?  In the attestation verifier, the exact same check is done on the public key before it is handed back to the caller for storage, so it had to get stomped on after that.

@aseigler Thx, it was the length of public-key on my database was too small.
I have another problem, is about the origin that comes from the Android, here is error:
Unhandled exception: Origin android:apk-key-hash:LBpmHljqwJJLdXVNpjfdAz2k2MNojcSFZZIuRj1B4wI not equal to original origin android:apk-key-hash:LBpmHljqwJJLdXVNpjfdAz2k2MNojcSFZZIuRj1B4wI
Fido2NetLib.Fido2VerificationException: Origin android:apk-key-hash:LBpmHljqwJJLdXVNpjfdAz2k2MNojcSFZZIuRj1B4wI not equal to original origin android:apk-key-hash:LBpmHljqwJJLdXVNpjfdAz2k2MNojcSFZZIuRj1B4wI
  at Fido2NetLib.AuthenticatorResponse.BaseVerify(String expectedOrigin, Byte[] originalChallenge, Byte[] requestTokenBindingId)
  at Fido2NetLib.AuthenticatorAssertionResponse.VerifyAsync(AssertionOptions options, String expectedOrigin, Byte[] storedPublicKey, UInt32 storedSignatureCounter, IsUserHandleOwnerOfCredentialIdAsync isUserHandleOwnerOfCredId, Byte[] requestTokenBindingId)
  at Fido2NetLib.Fido2.MakeAssertionAsync(AuthenticatorAssertionRawResponse assertionResponse, AssertionOptions originalOptions, Byte[] storedPublicKey, UInt32 storedSignatureCounter, IsUserHandleOwnerOfCredentialIdAsync isUserHandleOwnerOfCredentialIdCallback, Byte[] requestTokenBindingId)

This happens because android is sending this origin "android:apk-key-hash:LBpmHljqwJJLdXVNpjfdAz2k2MNojcSFZZIuRj1B4wI" that is correct but in AuthenticatorResponse you guys have in BaseVerify on line 74 this:
            if (!string.Equals(FullyQualifiedOrigin(this.Origin), expectedOrigin, StringComparison.OrdinalIgnoreCase))
                throw new Fido2VerificationException($"Origin {Origin} not equal to original origin {expectedOrigin}");

The function "FullyQualifiedOrigin" cause the origin "android:apk-key-hash:LBpmHljqwJJLdXVNpjfdAz2k2MNojcSFZZIuRj1B4wI"   to become this "android://", there is any temporary solution?

Yes, I fixed that today via #217.  Expect a v2.0.1 release soon with that fix.

2.0.1 is out! https://www.nuget.org/packages/Fido2


Moving a compiled binary to a clean OS would throw cannot install provided ErrorHook

I'm compiling binaries from this project, and it's working fine on the dev machine. However, when I move the binaries to a clean linux os, running the bins returns this error:
$ cast
Error: cannot install provided ErrorHook, a hook has already been installed

Location:
cli/src/handler.rs:70:13

and the line in question is:
eyre::set_hook(Box::new(move |_| Box::new(Handler)))?;

There's also a related issue where people have ran into the same problem on Win 10.
What is causing this?

Polite reminder @yaahc

Thank you, I'm sorry for the long delay, I'll make sure to resolve this asap (probably after the new year).

I've been watching this thread as I recently ran into the same error message after introducing a seemingly unrelated change on a project and finally figured out what was going on. I'm documenting it here in case it can be useful to others.
When the auto-install feature is enabled (which it is by default), if an error handler is needed but one isn't configured yet, eyre will install a default one: https://github.com/yaahc/eyre/blob/c32a8d0b6799ef4c862df1034202f9d6e5a5f400/src/lib.rs#L600-L603
What happened in my case was that I introduced some code that triggered and logged an eyre::Report before the code that setup the error handler, which meant that by the time it was executed the default handler was already in place. This led to a lot of confusion on my part as I wasn't aware of the default handler behaviour and assumed I was somehow trying to set up a handler twice.
This is quite similar to your description @xphoniex, as in my case the new code I introduced was actually checking for the presence of a file that exists on some systems but not others.
Hope this helps.

I was able to solve this exact issue by ensuring that git and libusb are installed on the new machine where the compiled binaries are moved to. Hopefully this helps for your issue.

Sorry, a bit confusing, is this a typo, or does commit https://github.com/eyre-rs/eyre/pull/110/commits/615d16ca0dd6a1c2e34e668607b83bc3f5c49c32 actually resolve this issue @dbanty ?

Hmm, looks like pulling color-eyre into the workspace also pulled in the commit history, which means commits that referenced closing issues for color-eyre now retroactively closed issues for eyre 😱. What a fun problem 😅

As stated in the issue, it is the auto install being a footgun, which imho is an anti feature.
Tracing does not do anything by default as this easily leads to unexpected behavior.
We've run into a similar issue with the compile probes, and want to give all this a stir up and remove the probes and instead opting for cargo cfg flags to enable nightly behavior as  they are prone to silently breaking or misleading behavior.


My slide

git checkout -b my-slide origin/my-slide not working and giving "fatal: A branch named 'my-slide' already exists."


MES causes left clicks through spec bar

Type
Incorrect behavior
Operating System
Windows
OS Version
No response
Bug description
Using menu entry swapper to alter the left click option of an object causes left clicks to register that object through the special attack orb (see video). In this instance I changed left click to quick enter a room in raids 3, and then clicking my special attack orb when it was positioned over that quick start object causing me to run towards that object rather than speccing which resulted in my death. This incident was easily reproduced by a Runelite contributor (Enriath) from discord so she referred me here to report the issue.
Screenshots or videos

Video 👇
https://twitter.com/mandruers/status/1575200734345207808?s=46&t=piJwIYHxBmm0v4cb3yqLnA
RuneLite version
No response
Logs
No response

I thought I'd mention that a band-aid fix would be to add custom entry:
Use,Special Attack,*
This will prioritise the spec orb in all cases, I do agree that this should be default behaviour though


QueryInterestRatePerBlock

returns the current effective interest rate for both Stable and Variable interest rate types, expressed as a Rational

@Benjmhart Could you elaborate on what needs to be done in this endpoint or should I dig into a finance book?
We have a ReserveInterest type in each Reserve for each Coin in the LendingPool, unsure of the requirement here.

let's call this one blocked on specification issues for now.


New Resource: IAM Role

We eventually want to fully support IAM resources, but to get started we need to be able to CRUD a role and attach/detach an existing policy to it.
Instances need to be able to be assigned/unassigned existing and chef-managed IAM roles.

@tyler-ball is it being worked on?

@Fodoj Not yet

@tyler-ball I'd like to work on it then. Should I do it with aws dk v1 or v2? What is master branch targeted to?

You should use the V2 SDK.  We're going to convert all resources to use that eventually, so we may as well have new resources use that.  aws_route_table and aws_vpc_peering_connection are examples of resources/providers that use the V2 SDK.

@tyler-ball I've created a PR, currently still work in progress, but roles and instance profiles are there: https://github.com/chef/chef-provisioning-aws/pull/343


chessai upload perms on hackage

@scotty-web/maintainers Can someone please give me maintainer permissions for scotty on hackage?

I've added you as a maintainer.

Thanks!


Scheduler.none implementation

So right now we have the lack of a passed scheduler determining that various behaviors should not be scheduled. This leads to checks like if(scheduler) { /* schedule things */ } else { /* call things here */ }, which is totally fine. However, it does mean that it's hard to go back to the unscheduled behavior for operators (ours or otherwise) that are created with default scheduling.
For example, if (psuedo-code) someCustomOperator(...args, scheduler=nextTick) were to default to nextTick scheduling, we don't really have a compelling way to switch it back to not being scheduled at all.
The idea is pretty straight forward, implement a scheduler that doesn't really schedule anything. Then in those if blocks I mentioned above, look for it: if(scheduler && scheduler !== Scheduler.none). Pretty basic.
The name can be bike-shedded, of course, but make it snappy. :p

@blesh I would question why we would ever default to something other than undefined (outside the time-related operators) before factoring the forking out of the creation operators.
Here's a pseudo-code example of a synchronous iterative scheduler. I don't know if it's worth doing (introduces more method calls at runtime), but we could (probably) eliminate forking in the creation operators:
Scheduler.none = new IterativeRecursiveScheduler();

class IterativeRecursiveScheduler implements Scheduler {
  public actions: Action[] = [];
  public scheduled: boolean = false;
  public active: boolean = false;
  now() { return Date.now(); }
  flush() {}
  schedule<T>(work: (state?: any) => Subscription<T>|void, delay?: number, state?: any): Subscription<T> {
    return new RecursiveAction(work).schedule(state);
  }
}

class RecursiveAction extends Subscription<T> implements Action {
  private running: boolean = false;
  private scheduled:boolean = false;
  public state?: any;
  public delay?: number;
  public scheduler: Scheduler;
  public work: (state?: any) => void|Subscription<any>;
  constructor(work) {
    this.work = work;
  }
  schedule(state?: any, delay?: number) {
    if (this.running) { return; }
    this.state = state;
    this.running = true;
    this.scheduled = true;
    do {
      this.scheduled = false;
      this.execute();
    } while (this.scheduled);
    this.running = false;
  }
  execute() {
    if (this.isUnsubscribed) { throw new Error("Can't execute a canceled Action."); }
    this.work(this.state);
  }
}

class RangeObservable extends Observable {
  private start: number;
  private end: number;
  private scheduler: Scheduler;
  constructor(start: number, end: number, scheduler: Scheduler = Scheduler.none) {
    super();
    this.start = start;
    this.end = end;
    this.scheduler = scheduler;
  }
  static dispatch(state) {
    const { start, index, end, subscriber } = state;
    if (index >= end) {
      subscriber.complete();
      return;
    }
    subscriber.next(start);
    if (subscriber.isUnsubscribed) { return; }
    state.index = index + 1;
    state.start = start + 1;
    (<any> this).schedule(state);
  }
  _subscribe(subscriber) {
    subscriber.add(scheduler.schedule(RangeObservable.dispatch, 0, {
        index, end, start, subscriber
      }));
  }
}

I would question why we would ever default to something other than undefined

why we would, but anyone can make a custom operator or have some operator that defaults to a specific scheduler. Animation observable libraries defaulting things to animationFrame scheduler, for example. In those cases, if someone wants to not schedule events, they'd need some sort of scheduler that could be passed.

FWIW, I think for our creation operators, we should still just check for no scheduling, and then handle it appropriately.
if (scheduler && scheduler !== Scheduler.none) {
  // schedule stuff
} else {
  // do it right now.
}

mostly for short call stacks and better performance.

I would question why we would ever default to something other than undefined
We wouldn't probably, but anyone can make a custom operator or have some operator that defaults to a specific scheduler. Animation observable libraries defaulting things to animationFrame scheduler, for example. In those cases, if someone wants to not schedule events, they'd need some sort of scheduler that could be passed.

Yes, this isn't possible today unless the animation library falls back to iterative dispatch in the case that no scheduler is supplied (just like range, etc.).
If we want a generic recursive scheduler (like RxJS 4's ImmediateScheduler) that doesn't suffer from stack overflows when synchronously rescheduling many times in a row (for example, Observable.range(0, 10000, Scheduler.immediate) errors from a stack overflow in RxJS 4), we'll have to include something like what I implemented above.

Closing as this has died out.

Actually it looks like this still has value for #854 and #855

Rebooting discussion here #1196


Must have hardware

Hello —
what is the must have hardware for a self driving project that doesn’t use the Apollo self driving software?
IPC is a must have as well as the camera, the processor, gpu etc. A monitor as well.
what else?
I will appreciate it if you reply.

Pls ref to to
What hardware is needed for Apollo?


Redundant create_padding_mask(inp) masks

Hello,
in your tutorial about Transormer I think there are redundant create_padding_mask(inp) masks. The same mask is created for Encoder and Decoder too. Please look here:
    # Encoder padding mask
    enc_padding_mask = create_padding_mask(inp)

    # Used in the 2nd attention block in the decoder.
    # This padding mask is used to mask the encoder outputs.
    dec_padding_mask = create_padding_mask(inp)

It's the same padding mask created from inp. It's memory inefficient.
Link: https://www.tensorflow.org/text/tutorials/transformer#create_the_transformer
Thanks.
Have a nice day.

Hi @markub3327 ! Could you please post this text repository?

Yes, I can create PR in tensorflow/text.
Thanks.

https://github.com/tensorflow/text/issues/863

Hi @gadagashwini ! Could you look at this issue?

@markub3327,
dec_padding_mask is expected. This padding mask is used to mask the encoder outputs.
dec_padding_mask is for input and target language
dec_padding_mask = self.create_masks(inp, tar)

@gadagashwini
Now the snippet look like here:
def create_masks(self, inp, tar):
    # Encoder padding mask
    enc_padding_mask = create_padding_mask(inp)

    # Used in the 2nd attention block in the decoder.
    # This padding mask is used to mask the encoder outputs.
    dec_padding_mask = create_padding_mask(inp)

    # Used in the 1st attention block in the decoder.
    # It is used to pad and mask future tokens in the input received by
    # the decoder.
    look_ahead_mask = create_look_ahead_mask(tf.shape(tar)[1])
    dec_target_padding_mask = create_padding_mask(tar)
    look_ahead_mask = tf.maximum(dec_target_padding_mask, look_ahead_mask)

    return enc_padding_mask, look_ahead_mask, dec_padding_mask

The same thing with lower memory consumption and better way is:
def call(self, inputs, training):
    # Keras models prefer if you pass all your inputs in the first argument
    inp, tar = inputs

    padding_mask, look_ahead_mask = self.create_masks(inp, tar)

    enc_output = self.encoder(inp, training, padding_mask)  # (batch_size, inp_seq_len, d_model)

    # dec_output.shape == (batch_size, tar_seq_len, d_model)
    dec_output, attention_weights = self.decoder(
        tar, enc_output, training, look_ahead_mask, padding_mask)

    final_output = self.final_layer(dec_output)  # (batch_size, tar_seq_len, target_vocab_size)

    return final_output, attention_weights

def create_masks(self, inp, tar):
    # Padding mask
    padding_mask = create_padding_mask(inp)

    # Used in the 1st attention block in the decoder.
    # It is used to pad and mask future tokens in the input received by
    # the decoder.
    look_ahead_mask = create_look_ahead_mask(tf.shape(tar)[1])
    dec_target_padding_mask = create_padding_mask(tar)
    look_ahead_mask = tf.maximum(dec_target_padding_mask, look_ahead_mask)

    return padding_mask, look_ahead_mask

@markub3327,
Would you like to contribute to this issue?
Or any suggestion or workaround to fix this issue.Thanks!

@gadagashwini
Yes I can create PR.

@gadagashwini
I create PR at https://github.com/tensorflow/text/pull/869, is it OK?
Thanks.

@markub3327, Thanks for the PR.


Fix error in ease out quart easing function

Discovered the that the RBBEasingFunctionEaseOutQuart was actually outputting values between 2 - 1.
t = 0.0, POW (0 - 1, 4) + 1 = 2
t = 0.5, POW (0.5 - 1, 4) + 1 = 1.0625
t = 1.0, POW (1 - 1, 4) + 1 = 1

t = 0.0, 1 - POW (0 - 1, 4) = 0
t = 0.5, 1 - POW (0.5 - 1, 4) + 1 = 0.9375
t = 1.0, 1 - POW (1 - 1, 4) + 1 = 1

Good catch, thanks :sparkling_heart:


Noter js (hactoberfest)

Hi , I  have worked on the issue of adding error popup when submitted blank notes . I have modified the alert popup and now it looks better. [before it was just normal JS alert popup ]

It's creating a bug. The error is being generated but the empty note still gets displayed. Please fix it @SxxAq


backTitleVisible not work

Seems to this option still not work.
Should use hook again - headerBackTitle: '',?

the problem is still present
are there any updates?

Can reproduce the issue as well


fix(docsearch): update custom css for the logo

the 'powered by' logo has been changed to the new one
cc @lukyvj who originally made this file.
fixes #1977
b116f23dfb247ff39b371512673cb0bd29e97bb7 didn't fully fix this because it's using the logo, not the "powered by" logo

before
after

Thx 👌


a3-megagpu-8g pricing missing in some regions

Hi – thanks for a this project!
I noticed a3-megagpu-8g was added to pricing.yml in 6df5fae0da1379b1fd937e7c9d7ba854112a3a8c but multiple regions were removed in d45989c8b7aa31cd0b3b415c21bfbc2805f8889a (for instance us-central1)
Just wanted to file an issue first in case it's something obvious. But let me know and I can try to go deeper and see if I can fix the issue.

Only instances with all required SKUs are calculated and displayed. The a3-megagpu-8g is missing the price for the GPU in the region us-central1:
https://cloud.google.com/skus/?hl=en&filter=Nvidia H100 80GB Mega GPU running&currency=USD

Got it. The prices are showing up on this though: https://cloud.google.com/spot-vms/pricing#pricing-components
It's also showing up in the cloud console when I try to launch an instance (and I'm able to launch it).
Is it possible those prices fall under the last items in that list you linked to? I.e.

"Nvidia H100 80GB Plus GPU attached to Spot Preemptible VMs running in Americas"
"Nvidia H100 80GB Plus GPU running in Americas"

It's the exact same price:

I can't answer that with certainty. It's just strange that the name is different. This would be the first SKU where the name is different. Nvidia H100 80GB Mega GPU vs Nvidia H100 80GB Plus GPU.
Please take a look at your billing to see which SKUs you are being charged for. If you are launching such large instances, you certainly have a good contact with Google (TAM). Maybe you can ask Google and you can share the answer here.


Wipe Secure Cell key copy on deallocation

Secure Cell currently makes a copy of the master key it was given. Since Objective-C is fairly low level and exposes the key property, it makes sense to wipe the sensitive key from memory when it can no longer be used by Secure Cell.
Do so by changing the underlying property type to NSMutableData and calling the wiping code in dealloc of TSCell.
While we're here, improve API docs of the basic Secure Cell class that actually stores the key.
Note that in Objective-C the key property returns NSData which respects retain-release mechanics so the users get a reference to the same data object as used by Secure Cell. However, in Swift NSData is bridged into Data type which has value semantics and effectively copies our copy of the key. We obviously cannot and should not wipe the copies we are not aware of, so this is a best effort approach.
Checklist

[X] Change is covered by automated tests
[X] Benchmark results are attached (somewhat interesting, but hopefully irrelevant)
[X] The coding guidelines are followed
[X] Public API has proper documentation

Question: in new passphrase interface, passphrase is stored in key property, so we don’t need to take any other steps to wipe it, right?

Correct. It can be accessed as key and it will be wiped too.


Noobie question about spotify connect

Hey,
I searched both the docs and the questions here on the project but I couldn't find a clear answer. I
Is it possible to use this library as an interface to control a spotify connect device (Pc) from a python script on another device(Rpi2)? (e.g play specific playlists etc)
Thank you for you time folks,
Cheers!

It is possible! It might be a bit tricky to call the rust code from python though.
I'm working on a HTTP API around this library so that it can easily be used from any language, although I have very little time to work on it.  I pushed an initial commit tonight, at the very least it should give you an idea of how it is done: https://github.com/Spotifyd/spotifyd-http.

Like @SimonPersson said, this library can be used both to play music and control devices, although the latter hasn't been the main focus so far.
You may want to look at spotcontrol which is a re-implementation of librespot in Go, focused on the control part.
In any case, calling from python may be tricky, and the easiest solution is to wrap librespot or spotcontrol in an HTTP service (like @SimonPersson has done) and call it from your python script.

spotifyd-http seems pretty cool, if only I could contribute :P


--log-level=debug should log REST API calls

We already log client-side REST API calls (https://github.com/weaveworks/weave/blob/f08148cabbd103c1f8112bd0b426c75924fea683/api/weaveapi.go#L25). However, there is no logging of the calls at the server-side.


Add React Hot Loader 3

This commit wires up react-hot-loader 3 to
be used in the application.  There are numerous
benefits to the latest release, all of which
can be seen at https://github.com/gaearon/react-hot-loader
Note that the specific implementation around
wrapping in a Root component is part of how
react-hot-loader 3 needs to be configured.
Note that the package is brought in as a
dependency, not a dev dependency because of
how it is switched at runtime or not.
More information on the migration can be
viewed at:
https://github.com/gaearon/react-hot-loader/tree/master/docs

Related: #216

Changes Unknown when pulling 3865ef70880958bb1dc1906eb8f094d53b3edf06 on oshalygin:feature/add-hmr-3 into ** on coryhouse:master**.

Changes Unknown when pulling 9203ee33231e5741b47b4c6ff75f3d4d4db3e133 on oshalygin:feature/add-hmr-3 into ** on coryhouse:master**.

Changes Unknown when pulling 9203ee33231e5741b47b4c6ff75f3d4d4db3e133 on oshalygin:feature/add-hmr-3 into ** on coryhouse:master**.

Excellent! Thanks for the PR! This looks great to me but I'm going to let it sit for a couple days so hopefully others can chime in. Thanks again!

Sounds good thanks @coryhouse !

I put in my comment on the version number, other than that all appears to be accurate.

@kwelch watcha mean by version number?  I didn't see your comment :)

In the package.json instead of having a specific version number mentioned it just says "next".
It is an oddity with yarn, I put in a ticket yarnpkg/yarn/#1306, but it appears to have resurfaced. Additionally, yarn does not use the .npmrc so it does not save exact and as the point that this was last discussed in #363, they had not set on a solution.

I'll update it, thanks for the tip!

Changes Unknown when pulling b1ab8ae5f16507805bc4b222679f8bc9eccd813b on oshalygin:feature/add-hmr-3 into ** on coryhouse:master**.

thanks @kwelch

Thanks, we appreciate the PR. Although, I have used v3 while it was in beta and had no issues. I am with Cory on leaving this open for a little while to ensure it gets proper eyes on it.

Is there anything holding this PR up?

Ah, forgot to merge this after it was updated. Merged! Thanks @oshalygin! 👍

Thanks gents, love this starter btw :)


Cancelling mdPickers from  inside an Angular-material modal closes the modal and not the picker

If I  use mdPickers inside an angular-material modal and trigger the date picker to open, then close the picker it will close the modal underneath first. I must click 'Cancel' a second time in order to close the picker.
This is not very usable if I'm entering a lot of dates inside the modal because each time I need to cancel a date-picker it will close the underlying modal which I will then have to open again.
I suggest it is a scope issue because mdPickers is essentially a modal which  is being called inside of a modal.

Hi, take a look at this codepen and tell me if it helpshttp://codepen.io/alenaksu/pen/NNRQrE

@DomClaxton any news about the issue?

I close the issue. Feel free to reopen if you need support


Simplify check_json_dict

This is taking too many weird inputs right now.

I will attempt to start the simplification through classes.

I feel that this is mostly done for the moment. More is mostly cosmetical.


Windows default directory path for DOTNET_PACKAGES is wrong

I think the default Windows path for DOTNET_PACKAGES is wrong. As far as I know it should be %userprofile%\.nuget\packages.

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

ID: 0ba6c9b5-64cb-1dc6-21d0-ad8f0a1185fe
Version Independent ID: 0f6c658c-b477-2d2c-2a9a-d74f6ae38d3f
Content: dotnet command - .NET Core CLI
Content Source: docs/core/tools/dotnet.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

Documentation needs this update. Still pointing to wrong path.


elasticsearch: optimize default setup

reduce memory footprint of about ~1G
disable threshold that switches indices to read-only mode if hit
the default of 10% free disc space is ridiculous on modern developer machines
eg. 50GB free space on modern ssds (512+ GB) is more than enough to run elasticsearch

Thanks again!


Getting an error message "Response is nill"  when a request times out (cute v 0.1.20)

After updating to the cute v0.1.20 getting an error message in Allure [HTTP] Response is nil, when a request times out.
There is no detailed description of the error. It is difficult to determine the cause of the error from the description in Allure and Goland.

fix https://github.com/ozontech/cute/pull/74

released v0.1.21


Update gulp-sass to eliminate sass error when running gulp.

The newest version of gulp-sass fixes the sass error when running gulp. Fixes #10

Thanks for pull req.


Ch11: 11.5 Minibatch-sgd & 11.6 Momentum

Job d2l-zh/PR-948/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-948/

Forum links checked;
cite, numref checked.


Add tutorial 13_trotterQRTE.ipynb

Summary
Qiskit Algorithms tutorial for Trotter QRTE.
Details and comments
The original discussion can be found in the original Qiskit PR.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 90.038%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6444

Relevant Lines:
7157

💛  - Coveralls

@ialsina LGTM - thanks for work and contributing this tutorial.


Limited chain sync benchmark

this is a benchmark that runs for a limited time (e.g. one hour) and syncs blocks from a local byron-proxy. Alternatively, these blocks could also be downloaded from a remote proxy.
the reason for limiting the time is that the block chain grows every day, and thus the times of different runs cannot be compared directly. Using the obtained result we can at least estimate how much time it may need to sync a chain of certain size (i.e. 3.5*10^6 blocks).

bors r+


Basic tree-sitter question

Hi,
I have a very basic tree-sitter question...
I am using tree-sitter generate within the cloned directory on a macos.
The output is
tree-sitter generate
Replacing nan dependency with node-addon-api in package.json
Adding node-gyp-build dependency to package.json
Adding prebuildify devDependency to package.json
Adding an install script to package.json
Adding a prebuildify script to package.json
Adding peerDependencies to package.json
Adding types to package.json
Adding files to package.json
Updated build.rs with the /utf-8 flag for Windows compilation
Replacing index.js with new binding API
Replacing binding.cc with new binding API
Replacing binding.gyp with new binding API

May I ask, how to procede here? Maybe you have a hint!?
Actually I am using emacs and not nvim, but I assume that I can somehow load these package.json files.

Hi, all the replacing happens because I haven't updated to the new bindings API yet; kinda no time on hand and I'd like to support the old way a little bit more.
It's not clear what you want to do with the parser, but if all you want to do is to use the highlighting in Emacs, it's probably good to add it to https://github.com/emacs-tree-sitter/tree-sitter-langs
as you install it with https://emacs-tree-sitter.github.io/installation/; I'm just wing it here, have no actual experience with Emacs.
If you wanna do other stuff with it
the generate command compiles the parser into a WASM file; so you can really use it almost everywhere.
I kept switching between the native and wasm (web assembly) implementations for the LSP server for performance considerations at the time (and the hope of supporting vscode)
you can look at the relevant diffs here: https://sourcegraph.com/search?q=context:global+type:diff+repo:^github\.com/FoamScience/foam-language-server%24+foam.wasm&patternType=keyword&sm=0
those files will provide an example JavaScript usage.
I also can point you to a Rust project https://github.com/foamscience/foam-highlighter using the parser.
There are also golang bindings; but I'm not sure if it's feasible to provide parser bindings for CommonLisp

Hi,
thank you for your quick reply!
Yes, for now I would just use it for syntax highlighting. I will look into the tree-sitter-emacs more.
... and I need to look more into it to understand the other things I could do with it.
Thank you very much!


Corrected the link to add-azurekeyvaultkey page

Because the existing link (https://docs.microsoft.com/en-us/powershell/module/azurerm.keyvault/add-azurermkeyvaultkey) resulted a 404.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@marcduiker : Thanks for your contribution to the Azure documentation! The author, @cabailey, has been notified to review your proposed change.

#sign-off


Fix note dealing impossible trip times

Because of NA removal, this note is returned:
Impossible trip times reported, trip times set to .

Put the NA back in there somehow.

Closed in #154.


updated qlstephen (1.4.3)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Thank you for the contribution. It needed some fixes, so they were made in 4ef5ff6bf263c5d6cc5a3a3a1c091f5d6e97c618. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Query Parameters

This is an initial rev for #7 . Adding functionality to add include the following query parameters on every request:

format
shallow

On a side note - I'm not too sure if it's worth implementing the silent print query parameter or if we should just default every request to use print=silent since Update, Push, and Set don't return the object response.

I will make a separate PR for adding Ordering Data and Querying Data

LGTM :+1:


implement gettext translations

Currently, i18n is handled with PHP arrays. However, this is all done manually, and is not very sustainable moving forward. It's not very useful to load into memory a huge array with all the languages, just to retrieve one language.
It would be much better to use gettext, which is built into PHP. And po / mo files can be hooked into a translation server such as Weblate, in order to better handle translations.

Keeping in mind that we are also working with Latin, I wonder how this would work out with Latin in the picture. Linux has no "Latin" locale that can be installed with say locale-gen la.
In order for gettext to translate to the correct locale, I believe the locale needs to be set globally, for example:
setlocale(LC_ALL, 'de_DE.UTF-8');

I don't believe this can be done with Latin? Perhaps we need a special handling for Latin?

Can confirm that setting the locale does not work for Latin, because it's not installable as a locale in the server. This isn't a problem for the names of the Liturgical Events, which are handled as JSON data sources. It is only a "problem" for the Messages that are output along with the Liturgical Events, which is not so much of a problem. Who really wants to read those messages in Latin anyways?
Directory structure has been created, pot file has been created with xgettext (using only LitCalAPI.php as source):
xgettext --from-code=UTF-8 -o i18n/litcal.pot includes/LitCalAPI.php

Translation component has been created on Weblate.

added includes/LitCalMessages.php to the xgettext command (which I have added to a Github action so I don't have to run it manually whenever the PHP source files are changed). Also added the translators: tag to --add-comments='translators:', and it seems to be working nicely!
Most translation strings have now been ported to gettext. Even Liturgical ranking strings have been ported. Next step, port the Commons and all the parts that go to make up the Commons...

Have started porting the liturgical grade, liturgical colors, and Commons to gettext.
However for these we DO need a correct Latin translation. Translating in weblate won't do for Latin, because it's simply not picked up as a valid locale by gettext. Latin translations for these strings will have to be hardcoded into the relative enums, and when invoking the i18n method that has been created for these enums, we'll have to detect the current locale: if Latin, use the hardcoded Latin string, otherwise use gettext

Pretty much complete in PR #69


Add link to Colab for Ensembling HOWTO

I would like to include a colab tutorial on training a flax CNN model(using linen) that demonstrates training using colab TPU's effectively using pmap.Would you like to have this example in your library?

Thanks for your issue! What dataset and architecture you are planning on using?

I am working on others examples as well.This is the colab notebook I wish to include in the flax examples(can be included in the community examples),it would be great if you could have a look at this.
https://colab.research.google.com/drive/1FHhOAqBpcT4IZyDhgiZJWfdxcOPwOo-h?usp=sharing

MNIST dataset and a simple CNN model.This is basically to demonstrate the simple training pipeline on multiple devices on TPU's.Hope my notebook helps.

Moving forward I would like to include a few more examples relating to transformer based models.Looking forward to these as well.

Is there are difference between your Colab and our HOWTO on ensembling? https://flax.readthedocs.io/en/latest/howtos/ensembling.html

Its the same,I just thought of gathering things together so that,it would be easier to run it as a single colab notebook.

I just thought to include this in the community example section,so that everyone can easily run it on the fly.

Please do suggest on what other things I could work on to contribute to the community examples section ,I would love to implement things and contribute there.

Hi @RohanVKashyap, I think it may indeed be useful to add a link to the Colab in the Ensembling HOWTO. Could you create a PR adding this? I've updated this issue title and assigned it to you.

@RohanVKashyap any  updates on this?

Hi,sorry for the delay.I was caught up in some work.I will raise a PR with the colab link for the CNN model in a couple of days and get that done.

Great! Just one question: I took a look at your Colab and the code is formatted with very little whitespace. We probably would want the layout of the Colab to match the layout in our HOWTO, so could you please make sure they match?

Yes sure

@RohanVKashyap any updates on this?

Closing this due to inactivity.


Thumbnails for Downloaded Files

Thumbnails do not render even for files and folders that are marked "Always keep on device."  Only when you open the file does the thumbnail appear.  This makes it difficult to work with large amounts of files when you need to see the thumbnails

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

ID: 8f0d9527-a78b-d18b-d218-881fac11423d
Version Independent ID: 3ad8fc7b-4869-dbdc-802b-a804f65d0e39
Content: Configure syncing with the new OneDrive sync client for Sharepoint 2019
Content Source: SharePoint/SharePointServer/install/new-onedrive-sync-client.md
Product: sharepoint-server-itpro
GitHub Login: @kaarins
Microsoft Alias: kaarins

Yep, I see the same behavior only against SharePoint 2019. ODfB Online/OneDrive personal appear to work like you want them to.

Sorry I should have included the details of our setup.  This is with O365/SharePoint along with Windows 10 (16299.785) and OneDrive (18.212.1021.0007)

@mzavada I let the PG know in December of this issue -- they're aware of it. I have no ETA on a fix or what the priority of the fix is (or it may have been fixed already).
@scanum please close.


Add Elixir Make as a compiler for the NIF

This allows the NIF to be cross compiled using standard
C compiler environment variables such as CROSSCOMPILE and
CC. This is especially important with Nerves.
You can still use Zig compiler by setting CC=$zig cc

Thank you for jumping at that! elixir_make integration was next on my list so this just solves it :)


add 32 founds (2024-03-27)

Been a while - found a few!

Looks like automation picked up the changes indirectly - closing.

@roycewilliams No, the automation cannot pick up the nonstandard founds. Multiple with capital first letter and one where the . after the * is missing.

Ooh! The project awakens! I'll take a look soon

Works for me - thanks!


HA Log error radios==0.1.1 package not able to be installed

Hi, I found a few problem with the latest installation script.
After the install finished. I checked the homeassistant logs system->log
there are multiple errors in the log related to async wait functions inside the python code.
and a message that pip is not able to install "radios==0.1.1," needed for the radio integration.
FIX:
add "opkg install python3-pycares" this dependency is missing.
Other problem, install script stops at pip install of zeroconfig package
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
aioesphomeapi 13.0.2 requires noiseprotocol<1.0,>=0.3.1, which is not installed.
aioesphomeapi 13.0.2 requires protobuf>=3.19.0, which is not installed.
aioesphomeapi 13.0.2 requires async-timeout>=4.0, but you have async-timeout 3.0.1 which is incompatible.

I'm not sure if this is a real issue or not, looking at the script it seems these pip dependencies can be ignore. to work around the install from stopping I removed the line from the top of the script:
set -e
A better alternative would be to install the missing dependencies.
noiseprotocol & protobuf can be installed using pip without issues it seems.
Fix for: async-timeout>=4.0
To fix this I compiled a new version of the async-timeout version. openwrt build system
PKG_VERSION:=4.0.2
PKG_HASH:=2163e1640ddb52b7a8c80d0a67a08587e5d245cc9c553a74a847056bc2976b15

Please note that I'm using different hardware (ZBT - WG1608 router 32MB flash 512MB ram MT7621 compiled custom kernel to support Float emulation needed for numpy) but I believe this is not related to the problems above.
aliexpress WG1608
lastly: Many many thanks for this installation script, It helped me so much. I learned so much from this and I went from total linux noob to quite capable user :D. I studied this script in detail line by line, added comments to everything trying to figure out how you managed to get this working, its amazing thank you for your work.

Hi @littleboot
Thanks for the bug report, I tried to implement your comments it in the last commit.

Fix for: async-timeout>=4.0

I think it is not needed and I downgraded the version in the METADATA of the package. I believe, it will help to use the prebuilt version from OpenWrt.
Glad to hear that the script helped you to improve your skills!
If you have an opportunity to check the script working with async_timeout-3.0.1, it would be awesome.

Hi @devbis,
Thank you for the update, I will perform a clean install of my router firmware and try it out.
I had another  problem on the latest stable release of openwrt-22.03, I post the fix here it might be useful for someone else:
The docutils (V0.14) package could not be installed.
This problem was solved on the master branch of openwrt by a version update of  docutils V0.19. I'm not sure if this problem is platform dependent or not.
To fix this without recompiling. The package can be installed from the snapshot package repo using the download url. Note that opkg does not support https:// change the link to http:// and it should work.
example: opkg install http://downloads.openwrt.org/snapshots/trunk/ar71xx/packages/hiawatha_7.7-2_ar71xx.ipk

Building docutils is broken on 22.03 since October. You can download and install package from archives:
https://archive.openwrt.org/releases/22.03.2/packages/ , just choose your architecture.
Example for arm_cortex-a7_neon-vfpv4:
wget --no-check-certificate https://archive.openwrt.org/releases/22.03.2/packages/arm_cortex-a7_neon-vfpv4/packages/python3-docutils_0.14-2_arm_cortex-a7_neon-vfpv4.ipk -O /tmp/t.ipk && opkg install /tmp/t.ipk


Element Nesting too Deep

How to ask for help
I am trying to implement a discretised form of a continuous robot which has n_bodies=100. For this purpose, I used a composite element (rope) which I further saved into an .xml using mujoco's function. Afterwards, I have added actuation within the last 10 links.
When trying to load the .xml back, I get an error of Error=XML_ELEMENT_DEPTH_EXCEEDED ErrorID=18 (0x12) Line number=494: Element nesting is too deep..
This happens when I am just exporting and importing the model back as well.
How should I approach this? Thank you!

Oh wow. This looks like an issue at the tinyxml parser level. Let us investigate and get back to you...

Ugh, yes it is. See here.
We'll see if we can fix this upstream or find some other workaround, but this is not something we can easily fix in our own code. I'll keep this open and report back when we have more info.
In the meantime, maybe you can reduce the number of bodies in the chain?

Thank you very much @yuvaltassa!
That helped a lot! I managed to get away with 95 elements.
I was thinking maybe it will be a good feature to be able to attach sections of composite elements. For example one could make a section of 10 elements followed by another and so on...
This will allow defining different properties (i.e. stiffness) to different blocks, thus making the block act like a stiffer or more flexible block. This property might help in the modeling of continuum robots.

The maximum depth in tinyxml has now been increased to 500.


how to use it ??

This repo is left public to serve as an educational resource/reference. No support is provided.
If you want to quickly bootstrap a Gatsby + Contentful + Netlify site, check out this repo: https://github.com/ryanwiemer/gatsby-starter-gcn


Add Laravel 8 support

Add Laravel 8 support

@WilburJZHAO thank you for the contribution! Could you add changes to travisci.yml as well? Here is example how I added PHP8 support to another one package: https://github.com/cybercog/laravel-love/commit/0a8820e732e6aefaefc0b970c1ea7c82b7ad4f37

@WilburJZHAO sorry, I was too tired when I wrote that comment and gave a link on a wrong commit. There is no need to add PHP8 support to add Laravel 8 support. My bad.

@antonkomarev All good, I just took out the PHP 8 support.

I'm away from my computer, so cannot make a release. Will do it on the next week.

@WilburJZHAO sorry for long wait time, but adding new Laravel versions required more than just adding them in composer.json: #23
Laravel Ownership v5.3.0 has been released with Laravel 6.x, 7.x, 8.x support and PHP8 support.

@antonkomarev Thank you!


hapi-fhir-cli v6.2.4 can't execute 'run-server' command

When I'm trying to run server command, it's failing with (see full output in the hapi.log):
>  hapi-fhir-cli run-server -v r4
------------------------------------------------------------
🔥  HAPI FHIR 6.2.4 - Command Line Tool
------------------------------------------------------------
Process ID                      : 29867
Max configured JVM memory (Xmx) : 4.0GB
Detected Java version           : 19.0.1
------------------------------------------------------------
2023-01-20 10:30:30.121 [main] INFO  c.u.f.c.LogbackUtil Logging configuration set from file /logback-cli-on.xml
2023-01-20 10:30:30.134 [main] INFO  c.u.f.c.RunServerCommand Preparing HAPI FHIR JPA server on port 8080
2023-01-20 10:30:30.414 [main] INFO  c.u.f.c.RunServerCommand Starting HAPI FHIR JPA server in R4 mode
2023-01-20 10:30:45.276 [main] WARN  o.s.w.c.s.AnnotationConfigWebApplicationContext Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'myResourceDaosR4' defined in ca.uhn.fhir.jpa.config.GeneratedDaoAndResourceProviderConfigR4: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [java.util.List]: Factory method 'resourceDaosR4' threw exception; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'myAccountDaoR4': Unsatisfied dependency expressed through field 'myTerminologySvc'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'terminologyService': Unsatisfied dependency expressed through field 'myDeferredStorageSvc'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'termDeferredStorageSvc': Unsatisfied dependency expressed through field 'myJobCoordinator'; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying b
ean of type 'ca.uhn.fhir.batch2.api.IJobCoordinator' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {@org.springframework.beans.factory.annotation.Autowired(required=true)}
2023-01-20 10:30:45.291 [main] ERROR o.s.w.c.ContextLoader Context initialization failed
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'myResourceDaosR4' defined in ca.uhn.fhir.jpa.config.GeneratedDaoAndResourceProviderConfigR4: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [java.util.List]: Factory method 'resourceDaosR4' threw exception; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'myAccountDaoR4': Unsatisfied dependency expressed through field 'myTerminologySvc'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'terminologyService': Unsatisfied dependency expressed through field 'myDeferredStorageSvc'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'termDeferredStorageSvc': Unsatisfied dependency expressed through field 'myJobCoordinator'; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'ca.uhn.fhir.batch2.api.IJobCoordinator' available: expected at least 1 bean whic
h qualifies as autowire candidate. Dependency annotations: {@org.springframework.beans.factory.annotation.Autowired(required=true)}
	... 91 common frames omitted
2023-01-20 10:30:45.307 [main] INFO  c.u.f.c.RunServerCommand Server started on port 8080
2023-01-20 10:30:45.307 [main] INFO  c.u.f.c.RunServerCommand Web Testing UI : http://localhost:8080/
2023-01-20 10:30:45.307 [main] INFO  c.u.f.c.RunServerCommand Server Base URL: http://localhost:8080/baseR4/
^C2023-01-20 10:31:59.79 [Thread-0] INFO  ca.uhn.fhir.cli.App HAPI FHIR is shutting down...

Could you please suggest how this can be fixed?

6.8.5 seems to have removed run-server entirely:
% hapi-fhir-cli run-server -v r4
Unrecognized command: run-server

------------------------------------------------------------
🔥  HAPI FHIR 6.8.5 - Command Line Tool
------------------------------------------------------------


Fix wrong ENV for APP_CERT

APP_CERTIFICATE causes fatal error on launch as docker env is APP_CERT.

updated the env var name

updated the env var name


Sort Items in post feed card

Great work @nil1511, it's working well for me. Do you have concern @p0o about the implementation? Maybe we could have the logic on a separated js file?

Something i notice: when there is a title then a preview text, the space between these 2 element will be much bigger than usual.
Before:

After:

I think there should not have 2 times the PostFeedCard__cell--body element in this case.

@adcpm Updated the styles
@p0o Thank you for the suggestion. I have updated the code. let me know if more improvement is needed.


fix: add pypi extras through pixi add --pypi package[extra]

Fixes #1697
We broke pixi add --pypi pytest[dev] as it didn't add the [dev] anymore. Now that is fixed.

@baszalmstra I wasn't happy, and found out pixi add pytorch::pytorch was also broken. I now also fixed that in this pr and just reuse the requirement and matchspec as is.


lnd refuses inbound connections

lnd thinks that a node is connected even though the tcp socket is down. gossip-ping timeout doesn't help either.
also a mutex is hung somewhere because updatechanpolicy doesn't work, but most other commands work.
it doesn't seem to forward/send/receive any payments though.
2018-06-06 15:31:09.794 [WRN] SRVR: Received inbound connection from peer 03685a27a2d29f3f14d02dde544537317b624e39e540b6389fc6404dc3d452cb1d, but already connected, dropping conn
2018-06-06 15:36:10.048 [WRN] SRVR: Received inbound connection from peer 03685a27a2d29f3f14d02dde544537317b624e39e540b6389fc6404dc3d452cb1d, but already connected, dropping conn
2018-06-06 15:41:10.379 [WRN] SRVR: Received inbound connection from peer 03685a27a2d29f3f14d02dde544537317b624e39e540b6389fc6404dc3d452cb1d, but already connected, dropping conn
2018-06-06 15:46:10.664 [WRN] SRVR: Received inbound connection from peer 03685a27a2d29f3f14d02dde544537317b624e39e540b6389fc6404dc3d452cb1d, but already connected, dropping conn

$ sudo netstat -nap | grep <IP_ADDRESS>
$ lncli connect 03685a27a2d29f3f14d02dde544537317b624e39e540b6389fc6404dc3d452cb1d@<IP_ADDRESS>:9735
[lncli] rpc error: code = Unknown desc = already connected to peer: <IP_ADDRESS>:58642
$ sudo netstat -nap | grep <IP_ADDRESS>

@robtex @wpaulino I think this might be related to #1008

@robtex do you see this log entry:
https://github.com/lightningnetwork/lnd/blob/master/server.go#L1369
anywhere before the occurrence? Would have printed Peer <IP_ADDRESS>:58642 has been disconnected.
Your comments on not being able to send/receive make me think the switch is deadlocked somehow. Best guess atm is that the switch is blocked while tearing down the peer, even though the connection itself has been closed.
Since we don't fully remove the peer from the server's map until this process happens, we continue to deny the incoming connection requests.
This may have been partially addressed by #1339, which fixes a potential deadlock w/in the switch. If you have more logs from this we might be able to glean if this was indeed the culprit

We often experience connection issues with Bitrefill and Rompert, usually when we update and restart Our eclair nodes: it sometimes becomes impossible to reopen existing channels, and we enter a connect/disconnect loop.
We could confirm (by looking at the logs) that in at least one case it was the issue that @robtex described above: lnd thought it still had an existing connection and refused the new one.
We could reproduce this problem without eclair: connect lnd A to lnd B, kill -9 lnd A, it won't connect to lnd B anymore (I guess it would again after a very long timeout).
The problem is that LN nodes multiplex traffic over a single connection per peer (see https://github.com/lightningnetwork/lightning-rfc/blob/master/01-messaging.md#connection-handling-and-multiplexing): multiple connections per peer are not allowed, yet it is very hard to detect that connections are gone.
Since making sure that there is only one connection per peer is handled at the application level, wouldn't it be better, when a new tcp connection is created or accepted, and LN handshake has been processed properly, to keep it and replace the existing one ?
This is what we do on eclair, and we believe that it makes handling "unreliable" peers such as mobile phones easier. WDYT ?

We could reproduce this problem without eclair: connect lnd A to lnd B, kill -9 lnd A, it won't connect to lnd B anymore (I guess it would again after a very long timeout).

Do A and B have a channel with each other? We don't reconnect unless we have an active channel with  the node. By connect, do you mean it can't connect at all, or just not auto reconnect?

yet it is very hard to detect that connections are gone.

It should be the case that we get an EOF error from the OS (in the golang runtime) when the connection is severed. If it's the case that for w/e reason, killing the other process doesn't actually hang up the file descriptor, then would expect the connection to die after a time out.
Are you able to repro the above on the latest master?

wouldn't it be better, when a new tcp connection is created or accepted, and LN handshake has been processed properly, to keep it and replace the existing one ?

That's what we do: there's a tie breaker w.r.t which connection we keep, and which one we kill.

We just keep the new connection (handshake was successful so it should be good)

They key is that both node make the same decision though. How do you handle concurrent connection attempts? (there is no old conn)

The issue w/ just keeping the new connection is that then you may never converge to a stable state. Either side can always replace the connection, rather than the state machine always terminating, and then ignoring any "uncompliant" connections after a decision has been made.

@sstone I wrote up a description of the problem/solution here: https://github.com/lightningnetwork/lnd/pull/1351
In the long term, I think we should move to pubkey comparison as a tie-breaker, since it's the only stable procedure. I've implemented a patch for lnd that should play nicely with currently deployed strategies in lnd and eclair :)


RFC: message reply button

Inspired by gmail app, I moved the reply button next to the sender, to where the "flag" star was before. But compared to the reply button it doesn't deserve a highly prominent position like that, maybe we can place it somewhere else?

Looks nice. But won't it be annoying having to scroll up once a message has been read from top to bottom to be able to reply?

Inspired by gmail app, I moved the reply button next to the sender, to where the "flag" star was before. But compared to the reply button it doesn't deserve a highly prominent position like that, maybe we can place it somewhere else?

Do it like the Gmail app? Always display the subject below the action bar. Put the star to the right of the subject?

But won't it be annoying having to scroll up once a message has been read from top to bottom to be able to reply?

The "send" menu is still in overflow. Gmail covers this with a bottom bar below the message.

Do it like the Gmail app? Always display the subject below the action bar. Put the star to the right of the subject?

Can do. It's a bit more work so I wanted to hear feedback first (and split up the PRs)

how bout this?

Wow, looks very nice, I can't wait to build it and try it out live!

Nice.
I'm not sure about the position of the encryption indicator. But that's easily something we can figure out later.

I moved the encryption indicator so the icons form a triangle rather than a jagged pattern. There's probably still room for improvement, but overall it's certainly better than before :)

And having them bigger with descriptions, too. I heard more than one person say "I can never remember which one of those buttons is for replying" on the old bottom bar. Personally I thought the arrow is the single obvious candidate, but people are reluctant to click on stuff without confirmation it's the right thing. So having big buttons with descriptions is definitely a good idea. That can be another PR, though :)


Healpy on conda is out of date.

The latest version of healpy (1.10.X)  is not accessible via conda.  Only 1.9.1 is available.
I opened a ticket with OpenAstronomy:
https://github.com/OpenAstronomy/conda-channel/issues/34#issuecomment-303777596
They say that the problem is with conda-forge.  Can we please get conda-forge to see the latest version of healpy?

I tried submitting a Pull Request to conda forge, https://github.com/conda-forge/healpy-feedstock/pull/3, let's see if it builds correctly


Fix race condition around SessionAttemptControl.archiveTasks

WorkflowExecutor.propagateSessionArchive lists up finished root tasks
using one transaction. Then, use another transaction to archive the
attempts of the root tasks. This is intentional to avoid long lock
because archiving tasks is expected to take long time especially if the
attempt has many tasks.
However, because they use two transactions, another thread may archive
an attempt between the two transactions. Therefore, the later
transaction needs to re-check state of the attempt before actually
archiving it.
Without this fix, a thread may try to archive an empty attempt because the other thread archives (=deletes) all tasks. And because task_archives table has unique index on attempt id, the transaction fails with duplicate key value violates unique constraint "task_archives_pkey" message. So, this problem doesn't cause inconsistent data.

@komamitsu PTAL

👍

Adding an unit test for this issue would be nice when you get a chance.

Thank you for your review!
Please go ahead your work that is blocked by this issue. I'm going to add tests later.


RationalNumber.FindRational(4.41m, MathS.Settings.FloatToRationalIterCount) fails to find a rational

Maybe we should set default FloatToRationalIterCount to 15 (from the single-argument overload of FindRational) instead of 5.

@WhiteBlackGoose Something as simple as (21/10)^2 cannot appear as a fraction. This is very unexpected.

@WhiteBlackGoose Will it be okay to increase this number?

I don't think that's an issue, but you can increase it.

I mean by default.

I know


SPIKE - Investigate Future-Dated Test Results from IL

We want to determine if there is a time zone issue causing future dated tests results to be sent to state health departments. Ideally there is no timezone issue and our fix here: https://app.zenhub.com/workspaces/simplereport-better-data-team-605b43d51ea9f700119a48ee/issues/cdcgov/prime-simplereport/3091 alleviates the issue

I think we are good to close this but lets discuss in refinement today as a team

Findings: after 2 sessions with RS IL thinks they are receiving test results in CST but they are actually getting it as UTC. SR will write up a story to send UTC + offset value in the future, RS will correct this issue with IL in the short term


Laravel 10 + Monolog 3

Support for Laravel 10/Monolog 3. This is a breaking change and would require a new major version.

On a side note, I will probably be submitting more pull requests, with other potentially breaking changes. So I would suggest waiting a week or two before releasing v2.0

I included one additional commit, which introduces a breaking change in the way configuration is performed (see README).

@MattApril Thanks for contributing the PR. Since this PR has some breaking changes on the ENV for existing users and a few additional refactoring which are not required as of now. So, we are closing this. And we have added a new version 2.x from our end which supports Laravel 10 or higher.
We very much appreciate your effort and thank you for your work as it minimized our efforts on the new version.


some startup script stuff

doesnt affect normal users helps me alot :3

Fixed those things you asked for


Service call failed! in odd number runs

System Configuration
Please complete Your Configuration detail below.

Version
Best Known
Your Configuration

Operating System
Ubuntu 14.04.4 LTS
Ubuntu 14.04.4 LTS

Kernel
4.4.0-040400-generic
3.10.96+

Backend
video4linux
video4linux

ROS
indigo
indigo

ROS RealSense
Latest Release
1.0.3

librealsense
0.9.1
0.9.1

R200 Firmware
<IP_ADDRESS>
<IP_ADDRESS>

Expected Behavior
I want to run realsense_camera_nodelet on odroid xu4, Samsung Exynos5422 ARM® Cortex™-A15 Quad 2.0GHz/Cortex™-A7 Quad 1.4GHz. I am using SR300 camera. I used the package for sr300 from here:
https://github.com/nlyubova/realsense
and librealsense library for ARM from here:
https://github.com/Maghoumi/librealsense
I reduce fps to 30.
Actual Behavior
Every time that I run the node for the first time it crashed, and I received Error below, but second time it run without error, and alternatively once time it crashes and the other time it runs successfully :
[FATAL] [1356999608.372189726]: Service call failed!
[SR300_nodelet_manager-2] process has died [pid 10925, exit code -11, cmd /opt/ros/indigo/lib/nodelet/nodelet manager __name:=SR300_nodelet_manager __log:=/root/.ros/log/fa7186aa-53a8-11e2-bf31-7cdd908e0595/SR300_nodelet_manager-2.log].
log file: /root/.ros/log/fa7186aa-53a8-11e2-bf31-7cdd908e0595/SR300_nodelet_manager-2*.log
[camera_nodelet-3] process has died [pid 10931, exit code 255, cmd /opt/ros/indigo/lib/nodelet/nodelet load realsense_camera/RealsenseNodelet SR300_nodelet_manager rgb/image_raw:=app/camera/rgb/image_raw camera/depth/image_raw:=depth/image_raw camera/color/image_raw:=rgb/image_raw camera/infrared1/image_raw:=ir/image_raw camera/infrared2/image_raw:=ir2/image_raw camera/depth/points:=depth/points camera/depth/camera_info:=depth/camera_info camera/color/camera_info:=rgb/camera_info camera/infrared1/camera_info:=ir/camera_info camera/infrared2/camera_info:=ir2/camera_info __name:=camera_nodelet __log:=/root/.ros/log/fa7186aa-53a8-11e2-bf31-7cdd908e0595/camera_nodelet-3.log].
log file: /root/.ros/log/fa7186aa-53a8-11e2-bf31-7cdd908e0595/camera_nodelet-3*.log

Do you have any idea? I think some where we should add waitforSerivce line, but I don't know where.
Steps to Reproduce
I can run the roslaunch successfully using line below:
timeout 10 roslaunch realsense_camera realsense_sr300.launch
roslaunch realsense_camera realsense_sr300.launch

We are not yet support SR300 in the the mainline code. Support is in the queue and is prioritized, but it doesn't currently existing on the official release.
I'd suggest upgrading your kernel to a 4.4.x release. The librealsense direction discourage the use of the 3.19 kernel with patches and you are at 3.10.


Implement Bookmark feature

When?

User is reading an article.

What?

User should be able to bookmark (Save for later) the article.
User should be able to view the bookmarked articles by clicking on the user avatar > bookmarks

@aadeshkulkarni may I work on this feature as well?

Sure.
Please share your approach to implement this feature before starting dev

As the ticket mentioned will be creating a page to see "Bookmarked Pages" in the user avatar dropdown.

In every article I will add an option to "bookmark" maybe adding a star icon by the title

Then in the list you should be able to see which articles you have bookmarked with a star icon

On the backend

would create a UserBookmarkToPost table which would have a mapping of which user by id bookmarked which post by id

Alright. Sounds good.
2 Suggestions:

Naming the table Bookmark would suffice, instead of UserBookmarkToPost.
Use a Bookmark Icon instead of Star Icon (Similar to how Medium does it) [You can use https://heroicons.com/ since it's actively supported by the Tailwind community]

@jyo142 - Good work! The feature is now live.
I've tested it and adding few minor bugs as new issues.

@aadeshkulkarni got it! will take that bug


gulpfile: Seems like watchBundles do not work

Any change saved to the JavaScript files do not trigger buildBundle.
On another project, I fellback watching all .js and run buildBundles everytime.


browserWindow.webContents != getCurrentWebContents()

Why require('remote').getCurrentWindow().webContents == require('remote').getCurrentWebContents() returns false?
It's like every time you call browserWindow.webContents is different object. I tried overriding a function in WebContents but the second the I accessed it through browserWindow.webContents it wasn't changed. However, the same approach using require('remote').getCurrentWebContents() works correctly. Is this something that needs to be fixed?

One addition:
// in main process returns true
browserWindow.webContents == browserWindow.webContents

// in renderer process returns false
browserWindow.webContents == browserWindow.webContents

It seems like its possible to return the same instance. This is seen in getCurrentWebContents() so I suggest normalizing this behavior and making browserWindow.webContents also consistent always returning the same instance.


batchsize() documentation

The docs are now clearer, the original docs a little confuse.

Codecov Report

Merging #163 (20ed4fe) into main (03f8fb1) will increase coverage by 0.15%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #163      +/-   ##
==========================================
+ Coverage   88.50%   88.65%   +0.15%
==========================================
  Files          15       13       -2
  Lines         600      564      -36
==========================================
- Hits          531      500      -31
+ Misses         69       64       -5

Impacted Files
Coverage Δ

src/batchview.jl
79.54% <ø> (-1.31%)
:arrow_down:

... and 11 files with indirect coverage changes