[Feature] Adding an SIP calculator (Mutual Funds)

Description
Adding an SIP calculator for mutual funds.

Users will be able to calculate their returns by giving several parameters as input , such as

SIP amount.
Monthly or quarterly option.
Expected % returns.
Total Duration (In years).

The formula used by the SIP calculator is:
FV = P × ({[1 + i]n – 1} / i) × (1 + i)
FV = Future value.
P = SIP amount.
i = Expected rate of return per frequency period.
n = Number of instalments.
Using this calculator, users will be able to strategize their personal mutual funds investements.
Screenshots
Proposed design /feature.

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Closing this issue myself as it is already implemented.


Show more info for unbound breakpoints

In my nodejs project I have moved my source files into the /src directory. I updated tsconfig.json sourceRoot property to reflect the changes but now all the break points are unbound. They don't work.
From what I've read, the reason they don't work is the file paths are not found. Everyone in the discussions ask to post the log that shows where the file paths are.
This feature request is to show more information if breakpoints are unbound. In the tooltip show more data! Where is vscode looking for the file, what option do I need to change to update that? Where can I get more info!?!
Please add more details and solutions to fix unbound breakpoint.

Do you see this message when you hover the breakpoint?

If not you can also run the command "Diagnose breakpoint problems" manually


[Dev] Apply formatters and linters to styled-components code

So you use https://github.com/morishitter/stylefmt to format at runtime, then https://stylelint.io to lint at CI time.
Dev Time
Looks like we can use: https://www.npmjs.com/package/styled-components-stylefmt to automate the formatting
This may need editor support so we can run it when you press save, and it can be added to the husky pre-commit/pre-push formatters,
Editor support could come from https://github.com/mrmlnc/vscode-stylefmt - probably can make some changes to support using styled-components-stylefmt instead.
CI Time
We can use https://stylelint.io to lint that everything is 👍

#681 gets us to the point where our CSS looks and feels like web CSS. It adds inline reports etc.
I think to make it auto-fix, we'd need to build

TypeScript support to https://www.npmjs.com/package/styled-components-stylefmt
Extend either vscode-styled-components or vscode-stylefmt to handle using styled-components-stylefmt


Is there anyway to change tab programmatically

Hello, your project is awesome, but i have a question, is there anyway to change tab programmatically. Code should be like this
let newIndex = 2 this.tab_ref.changeTo(newIndex);

There's no API via a ref. You can use the Bottom Navigation as controlled component to change the active tab programatically: https://github.com/timomeh/react-native-material-bottom-navigation/blob/master/docs/Usage.md#controlled-component
Executing methods on a component via a ref is a bit of an antipattern in React.


setOption() ignored in some cases on EntityDatatables

I am using building a table with the following code:
        $datatable = $this->builder->createDatatableFromEntity(QuantityRegister::class);
        $criteria = Criteria::create()
            ->andWhere(Criteria::expr()->eq('workstation', $workstation))
        ;
        $datatable->setCriteria($criteria);
        $datatable
            ->addColumn(new DateColumn('timestamp', format: AbstractBaseEntity::DATETIME_STRING_FORMAT, displayName: $this->ts->trans('Date')))
        ;
        $datatable->handleRequest($request);
        if ( $datatable->isSubmitted()) {
            return  $datatable->getResponse();
        }
        $datatable->setOptions([
            'serverSide' => true,
            'processing' => true,
            'pageLength' => 16,
            'ajax' => $this->rs->generate('custom_route', [
                'id' => $workstation->getId(),
            ]),
            // We can order by field specifying which column is
            'order' => [0, 'desc'],
        ]);

Where the builder is an instance of the DatatableBuilderInterface.
If setOptions() is called just before addColum(), for example, this options are overwritten when handleRequest is called. I have to explicitly call setOptions as in the example.
Is this correct or is it a bug?

Hi,
lets make your example easy :
        $datatable = $this->builder->createDatatableFromEntity(QuantityRegister::class);
        $datatable
            ->addColumn(new DateColumn('timestamp', format: AbstractBaseEntity::DATETIME_STRING_FORMAT, displayName: $this->ts->trans('Date')))
        ;
        $datatable->handleRequest($request);
        $datatable->setOptions([
            'serverSide' => true,
            'processing' => true,
            'pageLength' => 16,
            'order' => [0, 'desc'],
        ]);
        if ( $datatable->isSubmitted()) {
            return  $datatable->getResponse();
        }

In this is example it's will work fine in all situations ,setOptions before or after handleRequest will works correct !
but when you're using 'ajax' option here you need to call setOptions before handleRequest !because we use 'ajax' option inside this is function.
but if you want it after ,can you use setPath place of set 'ajax' option like that :
$datatable = $this->builder->createDatatableFromEntity(QuantityRegister::class);
        $criteria = Criteria::create()
            ->andWhere(Criteria::expr()->eq('workstation', $workstation))
        ;
        $datatable->setCriteria($criteria);
        $datatable
            ->addColumn(new DateColumn('timestamp', format: AbstractBaseEntity::DATETIME_STRING_FORMAT, displayName: $this->ts->trans('Date')))
        ;
        $datatable->setOptions([
            'serverSide' => true,
            'processing' => true,
            'pageLength' => 16,
            'order' => [0, 'desc'],
        ]);
        $datatable->handleRequest($request);
        if ( $datatable->isSubmitted()) {
            return  $datatable->getResponse();
        }
        $datatable->setPath($this->rs->generate('custom_route', [
           'id' => $workstation->getId(),
        ]));

But , your example still doesn't make sense!
If you want to use a custom route like :
'ajax' => $this->rs->generate('custom_route', [
    'id' => $workstation->getId(),
]),

You don't need to add:
$datatable->handleRequest($request);
if ( $datatable->isSubmitted()) {
    return  $datatable->getResponse();
}

Because getReponse() called by datatable to generate the data.

Totally true, I didn't need this code
$datatable->handleRequest($request);
if ( $datatable->isSubmitted()) {
    return  $datatable->getResponse();
}

as it is already placed on my custom route.
Thanks for your quick and clear answer!

But there is a problem. If I don't call
$datatable->handleRequest($request);

then the following error appears;
Error:
Typed property Aziz403\UX\Datatable\Model\EntityDatatable::$path must not be accessed before initialization
  at vendor/aziz403/ux-datatable/src/Model/EntityDatatable.php:297
  at Aziz403\UX\Datatable\Model\EntityDatatable->createView() (vendor/aziz403/ux-datatable/src/Twig/DatatableExtension.php:58)
  at Aziz403\UX\Datatable\Twig\DatatableExtension->renderDatatable()

Yep, I think you're right!
I will fix it in the next few days.
Thanks


Clicking article button by default opens the first article

It feels like the article view/hide button should probably open to the article picker or there should be explicit way to identify a default article for it to open, ideally with collection record information included (Though this would be Smithsonian specific functionality...)

For an example see the Wiz boots here:
https://3d.si.edu/object/3d/costume-boots-wizard-wiz-broadway-worn-carl-hall:d8c64114-4ebc-11ea-b77f-2e728ce88125

Added in v0.20.0


ticklabels of log scale axis should be 'log' by default?

I feel it might be more natural to use 'log' for ticklabels when log scale is used.
Steps to reproduce
import proplot as pplt

fig = pplt.figure()
ax = fig.subplot(xlabel='x axis', ylabel='y axis')
ax.format(yscale='log', ylim=(1e-5, 1e5))

Expected behavior: [What you expected to happen]

Actual behavior: [What actually happened]

Steps for expected behavior
I can fix it by hand with yticklabels='log'
import proplot as pplt

fig = pplt.figure()
ax = fig.subplot(xlabel='x axis', ylabel='y axis')
ax.format(yscale='log', ylim=(1e-5, 1e5), yticklabels='log')

Proplot version
Paste the results of import matplotlib; print(matplotlib.__version__); import proplot; print(proplot.version)here.
3.4.3
0.9.5

Or is it possible to configure this behavior to default via rc files? I didn't see how can I do it so far.

Similar problem for symlog scale:
pplt.subplot(xscale='symlog', xlim=(-1e4, 1e4))

This behavior is intentional, just because I liked the consistency of one formatter for everything and prefer plain numbers for published figures, but it's definitely not ideal for axes spanning larger orders of magnitude.
There is currently no rc setting to change this but I like this idea. I think I'll add something like rc['formatter.log'] that can be set to False to disable the applicaton of AutoFormatter for log/symlog axes. Alternatively I could try something like rc['formatter.logthresh'] that auto-disables the application of AutoFormatter when the axis spans more than N orders of magnitude.... but the implementation of this might be too tricky.
I'll also add a note about the rc setting to this section of the docs, along with an example that shows the different formatters for each axis scale. Note in your example ax.format(yformatter='log') or ax.format(xformatter='symlog') would work.

...I might also consider setting the default value rc['formatter.log'] to False if my preference is in the minority. Maybe I'll start a poll like we did for import proplot as pplt.

Yes, it would be great to have something like rc['formatter.log'].
I wonder it would be quite common to meet axes spanning several orders of magnitude, that is basically why we need log scale :)

Hah, fair point.


Template validation error

During creation of stack I tried to change UCPNodesInstanceType size to m4.large but get error.
: Template error: Unable to get mapping for AWSInstanceType2Arch::m4.large::Arch

cc @bchav

@erikaulin looks like we're missing the Type2Arch mapping for that instance type. you can simply add the following under  "AWSInstanceType2Arch" to address it
"m4.large": {
                "Arch": "HVM64"

we'll fix it in next release.

OK thanks learning more and more about AWS and cloudformation templates :)  I project forked and added all the m4 options.

fixed with #44


Unable to open previously saved .xml file

**OS: Mac
**PyQt version: 5.15.1
Python 3.8.6 Homebrew

Python crashes and this error appears when i "Open Dir > .xml file"
Traceback (most recent call last):
File "labelimg.py", line 1367, in openFile
self.loadFile(filename)
File "labelimg.py", line 1065, in loadFile
self.lineColor = QColor(*self.labelFile.lineColor)
AttributeError: 'LabelFile' object has no attribute 'lineColor'
zsh: abort      python labelimg.py

Also having this issue when opening an annotation directly. #341 and #210 did not work for me either.
I opened up a debugger and the initialization call to labelFile does not do anything with the filename, so it's obvious why there's no lineColor attribute or imageData. I'm guessing this workflow hasn't been exercised recently, and most just open a directory?
Opening a directory doesn't seem to work in my case. Note that I am just trying to open existing pascal labels, and the labels are in a different directory from the images (but referred to correctly via pascal VOC path attribute)

see the script to change the file names in the XML files.link

Band-aid solution:
self.line_color = QColor(*self.label_file.lineColor) if hasattr(self.label_file, "lineColor") else DEFAULT_LINE_COLOR
self.fill_color = QColor(*self.label_file.fillColor) if hasattr(self.label_file, "fillColor") else DEFAULT_FILL_COLOR

…but yeah [still TODO:] need to cycle through the different colours and map them properly I think.


add ability to not dismiss pop label on touch

Awesome pod, thanks! I added a delegate to give more control over when the pop label is dismissed.

closed in favor of #14 on separate branch


json_scanf2: the max number of nested paths and the max key length

I propose to set up the above to max numbers:

the max number of nested path :   8
the max number of key length: 128

This function is designed to de-serialize data from rest API.  I expect most JSON data from rest API will not exceed the two limits.  But the two limits will simplify the memory management and reduce the number of malloc/free.
Let me what do you think?
I can make a change.

Yeah I find that pretty reasonable.

Yeah I find that pretty reasonable.

it is changed in 448df78.  we can always increase the values if it's necessary.

it is changed in 448df78.  we can always increase the values if it's necessary.


Docs: Fix Nexmo integration guide

In Nexmo Gem version 5.0.0, some API breaking changes appeared. Leading to this kind of error:
NoMethodError: undefined method `send_message' for #<Nexmo::Client:0x007f848744dcd0>

In this commit, I suggest locking the major version of Nexmo Gem to 4. In the future, it would be better to fix the integration.
Useful links:

https://github.com/Nexmo/nexmo-ruby/blob/master/CHANGES.md
https://github.com/visualitypl/textris/blob/master/lib/textris/delivery/nexmo.rb

Ping?

Pong. Sorry to keep you waiting! Thanks for your input, sounds reasonable :)

Hehe ! Thanks. 😊


Upgrade swagger-ui to 2.1.4

This adds support to display a curl command that is equivalent to a
request entered through the web UI. Interestingly enough, 2.1.4 is a
higher version number than 2.1.8-M1.

This depends on ELR-2232

[Current coverage][cc-pull] is 50.19%

Merging [#169][cc-pull] into [master][cc-base-branch] will increase coverage by 0.01%

@@             master       #169   diff @@
==========================================
  Files           703        704     +1
  Lines         38556      38576    +20
  Methods           0          0
  Messages          0          0
  Branches       6204       6206     +2
==========================================
+ Hits          19350      19365    +15
- Misses        19182      19187     +5
  Partials         24         24

Powered by Codecov. Last updated by [dae8a98...45e100d][cc-compare]
[cc-base-branch]: https://codecov.io/gh/linkedin/pinot/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/linkedin/pinot/compare/dae8a982eef957159d125a67aff84e68965ffa67...45e100d0f83232490cadc7191a83e0eaba77b97e
[cc-pull]: https://codecov.io/gh/linkedin/pinot/pull/169?src=pr

Awesome, displaying curl call will be nice. LGTM


Support new language: Julia

As mentioned in #527 by @marcusps, it would be nice to support the Julia language. Here it is!
I've checked the "Language contributor checklist" and confirmed that this passed the unit test.
But before completing the job, let me ask some questions.

To avoid misclassification with Matlab, I slightly modified the relevance of comment style of it. Is this permissible?
I want to escape expressions surrounded by '$(' and ')' in a string literal like Ruby or CoffeeScript, but the way of Ruby and CoffeeScript is not satisfactory to me because the escaped expressions are highlighted awkwardly. Is there any way to completely escape these expressions as if they are in normal code?

When there is any inappropriate manner in this pull request, please let me know.
Thanks.

Finally merged, thanks!

oh, I completely forgot the pull request, sorry.
Anyway, thank you for merging!


Fix precedence in parsePyTestModuleCollectionResult

For ...

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[X] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[X] ~Has sufficient logging.~
[X] ~Has telemetry for enhancements.~
[X] ~Unit tests & system/integration tests are added/updated~
[x] ~Test plan is updated as appropriate~
[X] ~package-lock.json has been regenerated by running npm install (if dependencies have changed)~

When discovering tests with pytest, the Python Test Log ended with:
Test Discovery failed:
TypeError: Cannot read property 'item' of undefined

The developer console gives me:
[Extension Host] Python Extension: displayDiscoverStatus TypeError: Cannot read property 'item' of undefined
	at module.exports.l.parsePyTestModuleCollectionResult.t.forEach.t (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77591:45)
	at Array.forEach (<anonymous>)
	at l.parsePyTestModuleCollectionResult (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77527:11)
	at module.exports.l.getTestFiles.e.split.forEach (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77463:23)
	at Array.forEach (<anonymous>)
	at l.getTestFiles (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77458:29)
	at l.parse (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77445:24)
	at h.<anonymous> (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77387:31)
	at Generator.next (<anonymous>)
	at s (/Users/tammoippen/.vscode/extensions/ms-python.python-2019.1.0/out/client/extension.js:77303:21)```

Which pointed me to `parsePyTestModuleCollectionResult` and `parentNode` not being correctly guarded (`undefined` in my case).

 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.:x: tammoippen sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

After tempering with my tests, I think the issue occurs with newer pytests and tests with multiline docstrings and using verbose output. See here aswell: https://github.com/pytest-dev/pytest/pull/4582. I think this breaks the parser?


Detect multiple Literal members in a union

Fix #110

Typeshed PR here: https://github.com/python/typeshed/pull/6984

Python 3.7 and 3.8 fail the CI because they don't have ast.Constant. Actually they have it, but Python just doesn't use it in the AST for whatever reason.
I added print(ast.dump(literal)) just under for literal in literals_in_union:, and here's how the AST looks:
akuli@akuli-desktop:~/flake8-pyi$ cat a.pyi
from typing import Union
from typing_extensions import Literal

x: Union[str, Literal['foo'], Literal['bar']]
akuli@akuli-desktop:~/flake8-pyi$ env37/bin/flake8 a.pyi
Index(value=Str(s='foo'))
Index(value=Str(s='bar'))
a.pyi:4:10: Y030 Multiple Literal members in a union. Combine them into one, e.g. "Literal[]".

akuli@akuli-desktop:~/flake8-pyi$ env310/bin/flake8 a.pyi
Constant(value='foo')
Constant(value='bar')
a.pyi:4:10: Y030 Multiple Literal members in a union. Combine them into one, e.g. "Literal['foo', 'bar']".

We should be prepared to handle Literals containing "literal ints, byte and unicode strings, bools, Enum values and None", according to PEP 586.

Ugh bytes and enums. Great.

Thinking about it a bit more, I think the logic can be quite simple: if it's a Tuple, append the elts, and if it's anything else, append the object as is, regardless of whether it's ast.Constant or ast.Str.

Thinking about it a bit more, I think the logic can be quite simple: if it's a Tuple, append the elts, and if it's anything else, append the object as is, regardless of whether it's ast.Constant or ast.Str.

Yeah, mypy can reliably complain if a user puts in an incompatible value, we just need to handle the formatting

I added one more test case, because I wasn't sure whether it would work in different versions. It seems to be fine though.

I added one more test case, because I wasn't sure whether it would work in different Python versions. It seems to be fine though.

Thanks! I was having my supper :)

Do you want me to hold off on merging until #113 is in?

I think I'll merge this first and fix the conflicts in my PR then.

Thanks for the quick implementation here @AlexWaygood!

Thanks for the quick implementation here @AlexWaygood!

Pleasure 😀


serialization/ deserialization is not robust enough

The current method relies on textBefore and textAfter properties and fails if they change. For example, if I create a highlight on any of the comments on the page https://news.ycombinator.com/item?id=28267975, the textBefore string would contain 'x minutes/hours ago' where the value of x will change later, making the entire process unstable.
I think there needs to be a more robust approach to storing the highlight location.
Maybe some inspiration can be taken from - https://github.com/WorldBrain/Memex/blob/develop/src/highlighting/ui/highlight-dom-range.ts?

I did my research, there are basically two ways to do range serialization / deserialization:

prefix, text, suffix
offset in page

Mainstream annotation software such as diigo and weava all use (1.), while Memex supports both, but now defaults to 2.
Generally speaking I think 1. is the more robust way to go, because it is resistant to content changes.
With 2. , when there is any change to the content (user updating a post, "1 minute ago" becoming "10 minute ago"), there will be drifts to the highlight.
Unless you are annotating static content that will never change (PDFs), or you have a snapshot mechanism built into the software, it's not recommended, the drifts may be huge after some time.
For 2. Memex is using https://github.com/tilgovi/dom-anchor-text-position
For 1., Memex is using https://github.com/tilgovi/dom-anchor-text-quote, a cool library that I never heard of before.
After digging into the algorithm of dom-anchor-text-quote, I did find some room for improvement:
a. If prefix + text + suffix match fails, we can try to use text + suffix match, prefix + text, or even just text match
b. https://github.com/google/diff-match-patch can be used to optimize text matching performance
I think a. is going to be enough to solve the case for hacker news (the user wouldn't highlight 'x minutes before', right?), I will implement it.

I tried to fix this problem in https://github.com/notelix/web-marker/commit/88f71f29eddda95a81e9fe4391aa2bffd1a09439
Please try<EMAIL_ADDRESS>to see if it is good

Thank you so much for doing the research and writing about it in detail.
I tried the new version and it seems fine for the case I described. However, I still think there's more to do about it in future versions. Unfortunately it seems to me that there will always be some issues (due to the complexity of the modern web, it is not really a limitation on annotation libraries) with the intricate design of some pages and in that case maybe  a more verbose failure result might be useful - so that the users can be informed about what has changed and what can be done about the rotten highlight.

Yes, there is no perfect algorithm for annotation deserialization, because the web is always changing.
In order to show a more verbose failure result and ask users what to do with the rotten highlight, you can actually catch the deserialization failure exception, and show a modal dialog.
Since everyone is using a different UI framework, what web-marker library can do is to throw an exception, and let the upstream system decide what to do with the exception. (I should probably use a custom exception class though, so that the upstream system can decode the exception with instanceof)

Here is some demo code taken from https://github.com/notelix/notelix
It triggers deserialization on URL change, and retries failed deserialization with a backoff algorithm up to 10 times. It should be able to handle most of the cases.
whenUrlChanges(() => {
    setTimeout(() => {
        loadAllAnnotationsData();
    });
});

let lastUrl;
function whenUrlChanges(callback) {
    setInterval(() => {
        const currentUrl = getNormalizedUrl();
        if (lastUrl !== currentUrl) {
            callback();
        }
        lastUrl = currentUrl;
    }, 500);
}

function getNormalizedUrl() {
    let href = window.location.href;
    let indexOfHash = href.indexOf("#");
    let indexOfQuestion = href.indexOf("?");
    if (indexOfHash > 0 && indexOfQuestion > 0 && indexOfHash < indexOfQuestion) {
        return href.substr(0, indexOfHash) + href.substr(indexOfQuestion);
    } else if (indexOfHash > 0) {
        return href.substr(0, indexOfHash);
    } else {
        return href;
    }
}

export function loadAllAnnotationsData() {
    let failureCount = 0;
    return queryAnnotationsByUrl(getNormalizedUrl()).then((list) => {
        list.map((item) => {
            state.annotations[item.uid] = item;

            try {
                marker.paint(state.annotations[item.uid]);
            } catch (e) {
                failureCount++;
                retryPaintMarker(0, item.uid);
            }
        });
    });
}

function retryPaintMarker(retryCount, uid) {
    setTimeout(() => {
        try {
            marker.paint(state.annotations[uid]);
        } catch (e) {
            if (retryCount < 10) {
                retryPaintMarker(retryCount + 1, uid);
            } else {
                // give up
                delete state.annotations[uid];
                // You can show a modal to inform users of the rotten highlight here
            }
        }
    }, (retryCount * 0.5 + 1) * (1 + Math.random()) * 1000);
}


为何没有看到网易云音乐和Evernote笔记？

网易云音乐，这些个云，都感觉像是个  网页浏览器....😀
加上也无妨

在线听一些歌还是蛮方便，还有一些歌单直接听:)

Evernote is in the README. Closing this.
(As best I understand, this is taken care of. I ran the Chinese through Google Translate so I kind of understand what you guys said.)


[IPv6] Change openflow pipeline for L2 Pod networking

Add a new table named IPv6 to handle IPv6 ND Solicitation,
ND advertisement and IPv6 Multicast traffic.

Add flows in openflow tables (spoofGuardTable, IPv6,
conntrackTable, conntrackStateTable, conntrackCommitTable,
L2ForwardingOutTable) for handling IPv6 L2 Pod networking.

/test-all

/test-windows-conformance

/test-all

/test-all

/test-windows-conformance

/test-windows-networkpolicy


Fix AggregateException in OAuth2Api+d__20.MoveNext()

Version: 0.6 beta (7) | es.um.tellop
Stacktrace
0 TellOP.Api OAuth2Api+d__20 MoveNext null 0
Reason
android.runtime.JavaProxyThrowable: System.AggregateException: A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread. ---> TellOP.Api.UnsuccessfulApiCallException: The API call did not complete successfully (status code: InternalServerError)
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/407823/crash_reasons/157274957

Fix rev #843. Publish in 0.7beta


[go_router_builder] update the version of analyzer being used

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Go to pubspec.yaml and add riverpod_lint : ^2.0.0 and go_router_builder :  ^2.3.0 as dependencies
Run flutter pub get

Expected results
Should be able to resolve dependencies and get packages.
Actual results
Because go_router_builder >=1.0.13 depends on analyzer >=4.4.0 <6.0.0 and riverpod_lint 2.0.0 depends on analyzer ^6.0.0, go_router_builder >=1.0.13 is incompatible with riverpod_lint 2.0.0.
And because no versions of riverpod_lint match >2.0.0 <3.0.0, go_router_builder >=1.0.13 is incompatible with riverpod_lint ^2.0.0.
So, because general_app_structure depends on both riverpod_lint ^2.0.0 and go_router_builder ^2.3.0, version solving failed.
Code sample
Add this to pubscpec.yaml
dev_dependencies:
  flutter_test:
    sdk: flutter
  riverpod_lint: ^2.0.0
  go_router_builder: ^2.3.0

Screenshots or Video

Screenshot

Logs
Logs
Because go_router_builder >=1.0.13 depends on analyzer >=4.4.0 <6.0.0 and riverpod_lint 2.0.0 depends on analyzer ^6.0.0, go_router_builder >=1.0.13 is incompatible with riverpod_lint 2.0.0.
And because no versions of riverpod_lint match >2.0.0 <3.0.0, go_router_builder >=1.0.13 is incompatible with riverpod_lint ^2.0.0.
So, because general_app_structure depends on both riverpod_lint ^2.0.0 and go_router_builder ^2.3.0, version solving failed.

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.10.6, on Microsoft Windows [Version 10.0.22621.2134], locale en-US)
    • Flutter version 3.10.6 on channel stable at C:\Users\gogom\Documents\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (5 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\Users\gogom\AppData\Local\Android\sdk
    • Platform android-33, build-tools 32.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2022.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] VS Code (version 1.81.1)
    • VS Code at C:\Users\gogom\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.70.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.2134]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 115.0.5790.171
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 115.0.1901.203

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Reproducible using the dependencies provided above.

flutter doctor -v
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.4.1 22F770820d darwin-arm64, locale en-GB)
    • Flutter version 3.10.6 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (5 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/223.8836.35.2231.10406996/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode-14.3.0.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.10121639/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] Android Studio (version 2022.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/223.8836.35.2231.10406996/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2)
    • IntelliJ at /Users/nexus/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 75.1.4
    • Dart plugin version 232.8660.129

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1.4)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.9225.16/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/232.8660.185/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (4 available)
    • Pixel 7 (mobile) • 28291FDH2001SA            • android-arm64  • Android 13 (API 33)
    • Nexus (mobile)   • 00008020-001875E83A38002E • ios            • iOS 16.6 20G75
    • macOS (desktop)  • macos                     • darwin-arm64   • macOS 13.4.1 22F770820d darwin-arm64
    • Chrome (web)     • chrome                    • web-javascript • Google Chrome 115.0.5790.170

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.14.0-5.0.pre.26, on macOS 13.4.1 22F770820d darwin-arm64, locale en-GB)
    • Flutter version 3.14.0-5.0.pre.26 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 03aa95be67 (2 hours ago), 2023-08-16 01:53:05 -0400
    • Engine revision 7cc6a5832a
    • Dart version 3.2.0 (build 3.2.0-74.0.dev)
    • DevTools version 2.26.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/223.8836.35.2231.10406996/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode-14.3.0.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.10121639/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] Android Studio (version 2022.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/223.8836.35.2231.10406996/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1.4)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.9225.16/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/232.8660.185/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (4 available)
    • Pixel 7 (mobile) • 28291FDH2001SA            • android-arm64  • Android 13 (API 33)
    • Nexus (mobile)   • 00008020-001875E83A38002E • ios            • iOS 16.6 20G75
    • macOS (desktop)  • macos                     • darwin-arm64   • macOS 13.4.1 22F770820d darwin-arm64
    • Chrome (web)     • chrome                    • web-javascript • Google Chrome 115.0.5790.170

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

This issue also occurs when go_router_builder 2.3.0 is used with freezed 2.4.2
Because go_router_builder >=1.0.7 <1.0.8 depends on analyzer >=2.7.0 <5.0.0 and go_router_builder <1.0.7 depends on analyzer >=2.7.0 <4.0.0, go_router_builder <1.0.8
  requires analyzer >=2.7.0 <5.0.0.
And because go_router_builder >=1.0.8 <1.0.13 depends on analyzer ^4.4.0, go_router_builder <1.0.13 requires analyzer >=2.7.0 <5.0.0.
And because go_router_builder >=1.0.13 depends on analyzer >=4.4.0 <6.0.0 and freezed >=2.4.2 depends on analyzer ^6.0.0, freezed >=2.4.2 is incompatible with
  go_router_builder.
So, because retail_plus depends on both freezed ^2.4.2 and go_router_builder any, version solving failed.

Linking https://github.com/flutter/flutter/issues/132115 for reference.

I believe this was done in https://github.com/flutter/packages/pull/4921 and can now be closed.


Integrate ptxas compiler to iree-compile

Currently iree-compile generates PTX and puts into vmfb file. The PTX is compiled on the runtime using cuModuleLoadDataEx. This is insufficient in three ways:

Cannot try different ptxas compiler version.
ptxas might behave differently than cuModuleLoadDataEx.
Keeping PTX increases vmfb file size.

The PR implements followings:

Introduces two flags in the iree-compile ---iree-hal-use-ptxas-from=<path> ---iree-hal-use-ptxas-params=<params>
Compiles the iree generated PTX and keeps it as cubin in the vmfb file. This reduces the file size significantly.

Flags can use them like below :
iree-compile code.mlir --iree-hal-use-ptxas-from=/usr/local/cuda-11.8/bin/ptxas -o code.vmfb
NOTE: Compiling the generated PTX code
 $ /usr/local/cuda-11.8/bin/ptxas -arch sm_80  /tmp/iree-cuda-ptx-src-1a146b -o /tmp/iree-cuda-ptx-src-1a146b.cubin 2> /tmp/iree-cuda-ptx-log-0de384

One can pass -v to see extra information such as register spilling, static shared and local memory usage and etc.
iree-compile code.mlir --iree-hal-use-ptxas-from=/usr/local/cuda-11.8/bin/ptxas --iree-hal-use-ptxas-params=-v -o code.vmfb

NOTE: Compiling the generated PTX code
 $ /usr/local/cuda-11.8/bin/ptxas -arch sm_80 -v /tmp/iree-cuda-ptx-src-ceb4df -o /tmp/iree-cuda-ptx-src-ceb4df.cubin 2> /tmp/iree-cuda-ptx-log-dec4da
ptxas info    : 0 bytes gmem
ptxas info    : Compiling entry function 'matmul_dispatch_0_matmul_1024x1024x1024' for 'sm_80'
ptxas info    : Function properties for matmul_dispatch_0_matmul_1024x1024x1024
    0 bytes stack frame, 0 bytes spill stores, 0 bytes spill loads
ptxas info    : Used 188 registers, 376 bytes cmem[0]

Does this also work on windows ?

Does this also work on windows ?

I don't have windows here to try it out. I used llvm's api for file management and invoking the ptxas. It might just work

Thanks for this analysis and adding the PR. Focusing on the below runs which has the same cuda/driver version of 11.8. Please confirm that test-driver.vmfb  and test-cutlass-driver.vmfb are using520.61.05 driver installed as sudo sh cuda_11.8.0_520.61.05_linux.run --driver (see link)
Based on the above:
test-driver.vmfb            ->  9396807
test-cutlass-driver.vmfb    ->  8565386
test-cuda118.vmfb           ->  9385476
test-cutlass-cuda118.vmfb   ->  8520628

Normalizing it with the fastest run (Numbers show slowdown w.r.t. ptx compiled using ptxas from cuda 11.8 toolkit ):
test-driver.vmfb            ->  1.10
test-cutlass-driver.vmfb    ->  1.005
test-cuda118.vmfb           ->  1.10
test-cutlass-cuda118.vmfb   ->  1.00

Both driver's JIT compiled ptx-to-sass and toolkit's ptxas compilation to sass reaches the fastest performance with --iree-flow-add-cutlass=true. Is this a correct assertion?

Thanks for this analysis and adding the PR. Focusing on the below runs which has the same cuda/driver version of 11.8. Please confirm that test-driver.vmfb and test-cutlass-driver.vmfb are using520.61.05 driver installed as sudo sh cuda_11.8.0_520.61.05_linux.run --driver (see link)
Based on the above:
test-driver.vmfb            ->  9396807
test-cutlass-driver.vmfb    ->  8565386
test-cuda118.vmfb           ->  9385476
test-cutlass-cuda118.vmfb   ->  8520628

Normalizing it with the fastest run (Numbers show slowdown w.r.t. ptx compiled using ptxas from cuda 11.8 toolkit ):
test-driver.vmfb            ->  1.10
test-cutlass-driver.vmfb    ->  1.005
test-cuda118.vmfb           ->  1.10
test-cutlass-cuda118.vmfb   ->  1.00

Both driver's JIT compiled ptx-to-sass and toolkit's ptxas compilation to sass reaches the fastest performance with --iree-flow-add-cutlass=true. Is this a correct assertion?

There are cases where the driver's JIT compiles to ptx-to-sass and the toolkit's ptxas build produces different SASS even if the versions are the same. Also, I can't say that a version of ptx compiler is fast for any program. This depends on your existing PTX. This was my past experience as well.
That's why having chance to choose different versions of driver and ptxas compilers is useful, so we can see where do we loose/gain performance.


Improve XDMF implementation

Add time de-duplication. ParaView chokes on duplicate time entriees
Add error when absolute path is provided. Paraview doesn't like
absolute paths for XDMF data sources. At least on Windows.

Added some suggestions.


Empty URL args make GTM fail to load.

When using just GTM ID to initialize the plugin, the empty tags included in the URL cause it to fail to load.
Example:
https://www.googletagmanager.com/gtm.js?id=GTM-PBBB4LF&l=&gtm_auth=&gtm_preview=&gtm_cookies_win=x
vs.
https://www.googletagmanager.com/gtm.js?id=GTM-PBBB4LF

Looks like it's just the "x" at the end that breaks it. Can it be removed?

I'm afraid the issue on hand is not related to this plugin, because other implementations do the same. Why is it happening on my end then?

It's actually the empty &l= tag added when dataLayerName is not specified.

PR #3

@yasiupl This one is inactive for a while. I've started a webpack5 fork with dependencies updates and merged your PR as it was not approved here.


fix(ArrowFunction mutator): don't mutate () => undefined

Fixes #2306

It is so big if statement I would rather use normal "if/else" instead of ternary. It really reduces readability

And could you also check if "() => void" is mutated?

[...] ternary. It really reduces readability

I think there are 2 major causes:

using extra-large ---print-width setting in Prettier
indenting issue that I just reported in https://github.com/prettier/prettier/issues/8741

This is really unfortunate, since I find ternaries to be much more too the point in general.

Yahoo! Thanks, @nicojs !

Sure thing @rodoabad ! Just remember that this will be part of Stryker 4.0. I'm planning to release a new version of the beta later this week and we'll be working to a release this month or next month.


Documentation for all classes and functions.

We will be lightly following google code style for commenting code.
https://google.github.io/styleguide/pyguide.html#Comments
I'll try to find some automated checking for this to add to travis because automation is fun.

It looks like pydocstyle does this. I created a new branch to implement the autochecking

Done


Backend : Add support for policy evaluation API

On Add role page we are allowing users to select multiple policies and on UI we want to show the evaluation of selected policies like this.

To achieve this UI will need API support where UI will pass the list of policies and in response backend will be sending evaluation for passed policies.

@sureshms are we taking this up for this release?

@harshach ack. Will get it done for the release 0.13.


Issue found when attempting to run

import tidylib
tidylib.tidy_fragment('asdf', dict(input_xml=True))[0]
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/tidylib/tidy.py", line 227, in tidy_fragment
return get_module_tidy().tidy_fragment(text, options)
File "/usr/local/lib/python2.7/dist-packages/tidylib/tidy.py", line 233, in get_module_tidy
_tidy = Tidy()
File "/usr/local/lib/python2.7/dist-packages/tidylib/tidy.py", line 95, in init
if self._tidy is None:
AttributeError: 'Tidy' object has no attribute '_tidy'

Found after attempting to pip install pytidylib and running it through a python prompt. Evidently some steps are missing in the installation process, or the global declaration isn't proper. I am running Python 2.7.6.

Missed an install step. :(


⚠ Bofrost.at has degraded performance

In b2b89a8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1558 ms

Resolved: Bofrost.at performance has improved in d0a2628.


Added is_chrome_only (#244)

Implements #244

Thank you so much for doing this! Most of the changes that I requested are pretty minor things, but would just help the code flow a bit better.

Thanks for the feedback! I made most of the changes, and left a couple of comments where I had issues! :smile:

@teovoinea I think I responded to your questions there, let me know if I missed any.

@ohanar Thanks! Everything made sense :)

Thanks!

Does this fix #244/is it open for a reason?

@DJMcNab No, Github didn't autodetect it. I just closed it.


SVGs are rendered as images

Using
{{plantuml(svg)
  Bob -> Alice : hello
}}

results in redmine printing the SVG as image. Therefore, e.g. links in the SVG can not be clicked.
Is there a possibility to incorporate this behaviour?

Is there a possibility to incorporate this behaviour?

Not at this time.


Update references to measurementlab throughout

This change updates all remaining references to "measurement kit", "measurement_kit" or "MEASUREMENT_KIT" to variations of "measurement lab". This change also updates the "single_include" version of libndt7.hpp and what I hope will be a final update to the README.md taking all recent changes into account.
Part of:

https://github.com/m-lab/ndt7-client-cc/issues/2

This change is

Thank you!


ModelHawkes.init must be called last

in init to set default dtype as hawkes models are not templated

I don't think ModelHawkes.init was even called there hence the comment - this is order of operations - but sure can add "must be called last for xyz" .

Yes, maybe that's a bit different. It's just as this order is not obviously mandatory by simply looking at the code, it worths mentioning that it is done on purpose.


Using the sleep x days buttons makes the status timers for combatHud disapear

Using the sleep x days buttons makes the status timers for combatHud dissapear, can't get them back without restarting the game

I found out that it's because this mod uses the showHud checkbox as a template, can be fixed by using the other buttons as templates instead. I can send a updated dll with the fix.

I saw #2, have you verified this does not have any adverse effects? Seems like it would cause camera issues instead.
I will have a look when I have some time whether there are other listeners I forgot to remove as that would be the proper fix.


Fix a miscompile in the swift function merging pass.

In rare corner cases the pass merged two functions which contain incompatible call instructions.
See source comment in the change for details.
rdar://problem/43051718

@swift-ci smoke test and merge


add_child adds a child even if the child fails validation

class Post
  validate do |post|
     if post.parent && post.cannot_be_a_child
        post.errors[:base] << "A post cannot be a child if cannot_be_a_child is set to true"
     end
  end
  acts_as_tree
end

parent_post = Post.create!
child_post = Post.create!(cannot_be_a_child:true)
child_post.parent
#=> nil

parent_post.add_child child_post
#=> returns new child with parent set

child_post.errors
#=> error from validation...

child_post.parent
#=> points to parent_post

parent_post.children.include? child_post
#=> true

Expected: add_child should not add the child if validation fails on child.save.
Even better: there should be an add_child! that bubbles up the validation error to the user

if you reload the model the child will not be there.
:+1:  in adding add_child!, can you submit a PR ?

Reloading fixed it. I investigated the pattern a bit more, and add_child! doesn't make sense, since even the ActiveRecord collections seem to behave like add_child (and do not bubble up the error). Closing.


The [Required] attribute is not respected for non-nullable data type

I am currently working with a Asp.NetCore WebAPI with Swashbuckle.
I noticed that when producing the swagger.json, the [Required] attribute cannot be recognised for any non-nullable type (e.g. int, DateTime).
I also create a pure Asp.NetCore WebAPI project for testing, and this issue is still coming.
Now, I have to use the following SchemaFilter to make it work:
if (propertyType.IsValueType
    && !(propertyType.IsConstructedGenericType && (propertyType.GetGenericTypeDefinition() == typeof(Nullable<>))))
{
    if (property.CustomAttributes.Any(attr => attr.AttributeType == typeof(RequiredAttribute) ))
    {
        if (model.Required == null)
        {
            model.Required = new List<string>();
        }
        model.Required.Add(schemaPropertyName);
    }
}

Is this a bug or I missing some configurations?

All good, I found out this is the problem about user-posting, so it is not the issue for swagger since even it is labelled as [Required] the ModelState still cannot catch the missing for a non-nullabe property.


On windows "/rest/v1/products/" gives a wrong path

Hi,
I need the mock server to run on Windows. So far it works (though a lot of the directory concatenation has the wrong path separator ¯_(ツ)_/¯).
Unfortunately I have a problem:
http://localhost:3001/rest/v1/products/ - calculates a wrong folder name
http://localhost:3001/rest/v1/products - if I remove the slash it works
Best
Sebastian

Hi @sebastiandeutsch,
therefor you should change the foldername eg. with "#" in end or without.
Usually an API definition has just one variant. Removing the slashes isn't my preferred way to fix this.

I guess you're right - an API should have just one definition.
But the path generation on windows is really messed up if I add the trailing slash. The generated path tries to read a file at C:\GET\mock/response.txt. Wouldn't it be lovely to sanity check if the paths are leaving __dirname?


Franz does not load any services anymore and keeps "playing the trumpet" or "saddling the unicorn"

Describe the bug
Franz does not load any services anymore and keeps "playing the trumpet" or "saddling the unicorn"
after a while (a real long while) it throws an error that it couldnt load services
To Reproduce
Steps to reproduce the behavior:

start franz (under linux)

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

Manjaro with Gnome

Duplicate: https://github.com/meetfranz/franz/issues/1689

Yes Same Issue here

What to do ?

have some hope... (pun intended) ;)

Same issue here

Same issue here:) looks like Franz global error:/ which gonna makes my day a nightmare:)

Issue seems to be fixed

works again

It's OK for me !

Working Properly Now

Still not working for me; Samma issue on Gnome/Manjaro, Franz v. 5.4.1-1 (AUR). Downgrade to 5.3.3-1 starts fine but cannot login (signing in spins forever). Nothing wrong with my account either; works fine on MacOS.

Same issue on MacOS (Mojave 10.14) spins forever and then stops responding or just crashes


Jinja2 3.1.0 breaks Elyra

Describe the issue
Jinja2 released a new version, which causes Elyra to fail. See, for example,
AttributeError: module 'jinja2' has no attribute 'Markup'

in https://github.com/elyra-ai/elyra/runs/5679498953?check_suite_focus=true
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots or log output
If applicable, add screenshots or log output to help explain your problem.
Log Output

Paste the log output here.

Expected behavior
A clear and concise description of what you expected to happen.
Deployment information
Describe what you've deployed and how:

Elyra version: [e.g. 1.5.3]
Installation source: [e.g. PyPI, conda, from source, official container image, custom container image]
Deployment type: [e.g. local installation, Docker, Kubernetes, Kubeflow [notebook server] , Open Data Hub]
Operating system: [e.g. macos, linux]

For reference:

Breaking changes introduced in https://jinja.palletsprojects.com/en/3.1.x/changes/#version-3-1-0
Known upstream dependencies that fail as a result of ^^

nbconvert (no cap)

This will mean patch releases for all of our 2.x and 3.x branches as well

See nbconvert https://github.com/jupyter/nbconvert/pull/1737/files and subsequent release for proper fix


Error alf homework

Hello,  I have an error on several tests (ex:" Unexpected token + in JSON at position 6") and I don't know exactly what it is and where I should go, in the code, to solve it. A little help, please :D

Most probably you are using the JSON.parse function, but give it a parameter that is not a valid json string.


我希望后期Hop项目GUI页面可以全部汉化而不是使用者去翻译

我希望后期Hop项目GUI页面可以全部汉化而不是使用者去翻译

Feel free to contribute, we would need a couple of people who are strong in both English and Chinese to make this work.
The UI can be translated on https://translate.project-hop.org/


Class to support using HTTPS repos

This changes allows the use of repos served over HTTPS by just adding
include ::apt::transport_https

The class is not automatically included when a HTTPS source is detected so that it doesn't cause conflicts for people already installing the necessary packages.

Afaik modern day Debian and Ubuntu distributions include both by default. From what I can see unmodified Jessie and upwards and Trusty and upwards already do.
Though I understand the usefulness of it, since current releases of Ubuntu and Debian already come preinstalled with these packages and it's really a super-tiny wrapper I'm hesitant to accept this.
@mhaskel @DavidS: thoughts please?

If it only covers a small use case i tend to agree with @daenney

@daenney agree.

Mm, I am wrong about Debian Jessie, that one doesn't come with secure-transport. I'm guessing Ubuntu does because the PPA's use it.

The problem we're running into is that multiple modules need to have apt-transport-https. This allows those modules to avoid the duplicate resource problem. I don't really care how this is implemented, I just need a solution to that.
I just confirmed that neither wheezy nor jessie have apt-transport-https installed by default.
I chose a separate class because it keeps ::apt simpler. This is functionally completely separate from the ::apt class. If it weren't for backward compatibility, I would have just installed these packages unconditionally.
I'll wait until there's a decision on the preferred approach before fixing the problems above. (Thanks for the review!)

If it weren't for backward compatibility, I would have just installed these packages unconditionally.

I fail to see how backwards compatibility has anything to do with it. What do you mean?

If I just add the packages to ::apt, anybody who upgrades and was already installing the packages will get duplicate resource errors. That's particularly problematic if a 3rd party module is installing the packages (e.g. nodejs).

That's a very good point, thanks for clearing that up.
I'm inclined to say we should accept this on account of:

Debian being a bit unhelpful by not including this by default
Though repository and package integrity is guaranteed in other ways we should enable people to easily use repositories accessible over HTTPS

I'd still like to hear the thoughts of the rest of the module team based on the recent comments.

Also, if y'all have more comments on the code or tests, I would love to hear them. (The existing ones are great and all make sense.) I don't get much chance to work on modules for public consumption, but it would be great for me to be able to contribute more.

Added a comment for more robustness. Overall, I feel like this satisfies enough of a usecase (with the newfound knowledge that Debian does not package by default) to be merged. I like that it is encapsulated via a wrapper class, so that it can be invoked as-needed.
Overall, I'd be ok merging once the recommended changes are made. Thanks @danielparks !

@danielparks So we're good to go on this. Once the comments in the review have been addressed we'll be happy to merge this.

This is indeed an important missing piece in the 'apt' module. I have run into duplicate errors for 'apt-transport-https'(included by third party modules) on multiple occasions. Please merge and include this in the next release.

I might actually have a minute to get to this tomorrow. Wish me luck.

\o/ good luck @danielparks

Updated! Apparently I did need that luck. Or is it still tomorrow?
But… I haven't had a chance to actually run this, since my vagrant environment blew itself up. Plus, see my inline comment.

I've removed the before; looks like Travis is green again.


No stroke properties available when using a color ramp

If you apply a color ramp to a polygon dataset, the stroke properties are disabled and you get the always funny looking gaps in between each polygon (which can be accounted for by adjusting polygon-gamma):

The only way to add an outline is to go to the CartoCSS and add it there same with opacity.
I have defaults to add when ready!

Hey @makella I can configure the stroke with no problems...

can we do it as default cartocss? there are a couple of properties to set that are outside of the options available in the form... or only form?

We should either use only form ooptions or extend the form options if we
think is the best
2016-07-06 23:13 GMT+02:00 Mamata Akella<EMAIL_ADDRESS>
can we do it as default cartocss? there are a couple of properties to set
that are outside of the options available in the form... or only form?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/8595#issuecomment-230908633,
or mute the thread
https://github.com/notifications/unsubscribe/AAIENUQk5y0zjLhXjLoGuNSRuTIWG3gfks5qTBpxgaJpZM4JGgPX
.

--
Sergio Alvarez Leiva

ok then for now, let's use this
 polygon-opacity: 0.9;
  line-color: #fff;
  line-width: 0.5;
  line-opacity: 0.8;

that is all form ready.

which color?
2016-07-06 23:18 GMT+02:00 Mamata Akella<EMAIL_ADDRESS>
ok then for now, let's use this
polygon-opacity: 0.9;
line-color: #fff;
line-width: 0.5;
line-opacity: 0.8;
that is all form ready.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/8595#issuecomment-230910030,
or mute the thread
https://github.com/notifications/unsubscribe/AAIENTlqfd3Wahe1WfBSij1Fcjx-02iKks5qTBuvgaJpZM4JGgPX
.

--
Sergio Alvarez Leiva

oh sorry... so this is when you symbolize by numeric value on polygons (choropleth) the line options don't exist. So we can put in the defaults above for color value by polygon. The fill color will come from the ramp. Also, if we could add the 0.9 polygon opacity, that would be awesome.

They exist! Or am I getting this wrong?
On Jul 7, 2016 8:19 AM, "Mamata Akella"<EMAIL_ADDRESS>wrote:

oh sorry... so this is when you symbolize by numeric value on polygons
(choropleth) the line options don't exist. So we can put in the defaults
above for color value by polygon. The fill color will come from the ramp.
Also, if we could add the 0.9 polygon opacity, that would be awesome.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/8595#issuecomment-230988799,
or mute the thread
https://github.com/notifications/unsubscribe/AAIENa4lYas7itR8PGLeHKfzxByqV1wMks5qTJqJgaJpZM4JGgPX
.

weird. this is what i was seeing. let me try a new map or something.

@saleiva ok, its there now. but clearly looks horrible


Missing support for OpenSRS and Namecheap

Current release is missing the following domain registrars :
OpenSRS
dotTk
Namecheap

DotTk was removed from BoxBilling official support. https://github.com/boxbilling/boxbilling/issues/255#issuecomment-73515154
BoxBilling development team is solely responsible for developing core and main functionality. Since BoxBilling is open source modular application, you can add features of your choice and extend BoxBilling. It can be done by adding new BoxBilling extension. If you are not familiar with how to do that, please have a look at BoxBilling documentation, we have prepared Example module to get you started. If you need any help developing extension please use BoxBilling forum.
Source code can be found at GitHub's BoxBilling repository.
If you have problems creating new module or implementing new functionality, please use our contact form and request a quote. Try to detail your problem as much as possible so that we could understand what exactly needs to be done.

Thank you for your reply.
I found I had the dottk module in the wrong location, however I am looking into the false negative issue and will attempt to correct it.  If i get this module fully functional, I will submit a pull request for your review.
Regarding the other methods, I have submitted a ticket at the link provided (thank you for that), and included a link to the API documentation for Namecheap's SSL services.
I will be submitting a pull request for the latest OpenSRS php library although I do notice you have this in your dev build.

https://github.com/boxbilling/boxbilling/pull/295 DotTK support is now functional with this pull request.

As @vaidas-lungis said we have decided to drop support for DotTK registrar and moved it to our extensions repository. Please open pull request there.

This issue is marked as closed but I can't find OpenSRS support anywhere?

@ralphonz - I'd suggest you to create an issue related to this in our fork named FOSSBilling. As the majority of the latest team, we had to move to a new organization due to the problems arising from the fact that the BoxBilling project is dependent on a single person. We now have more flexibility and an active development environment in the new organization.
We also have a Discord community. If you want to join, here is the link. The codebase is the same and we're planning to make a new release soon. Until then, preview builds are automatically being generated and made available on our website.


Reference to flash elements

I really liked this module, but I'd like to see the following feature that is present in this other library:
https://github.com/wmluke/angular-flash
Following that module idea, you can specify some id to the flash-message directive, so then when you call Flash.create service you could set target id too.
ie:

Flash.create('success', message, 'custom-class', 'login-flash');
//Or better you could make last parameter as a config object
Flash.create('success', message, {
class: 'custom-class',
target: 'login-flash'
});

Hey @edriang,
thank you for your feedback. That would actually be quite neat so I'll have a look at that soon.


Config comment typo

Config comment telling you to use the wrong name
Fixed in #159

Fixed, merged


for blogspot

how to use this code in google blogspot?

This is a very broad question about programming in the general sense and out of scope to answer here.
Optimize-plugin is a plugin written for webpack. Webpack is most commonly used to prepare assets for single page applications written with web technologies. So to be able to use this plugin you need to have control over your build pipeline, use webpack and have access to the webpack configuration.


Istio-proxy custom format access_log doesn't generate until second request.

Bug description
We are running Istio (version:  1.5.0 ) under ACK cluster, applications running are using envoy filter for the custom access log. We are using ambassador service for forwarding the inbound request to the backend. There is a strange thing noticed, that custom access log generated for any incoming request does not appear immediately but appears merged with next request, however it captures the timestamp of first request. I know it sounds odd.
Here's the log:
First request log: Notice the path (/istio-auth/first-request) and timestamp (2020-03-19T14:44:44.899Z) this is default access log not the custom one.
{"startTIME": "2020-03-19T14:15:10.161Z", "method": "GET", "Authorization": "-" ,"path": "/istio-auth/"}\\n{"bytes_sent":"103","upstream_cluster":"inbound|5050|http-app|uat-app-hello-istio-auth.technology-newsroom-system.svc.cluster.local","downstream_remote_address":"xxx.xxx.xxx.xxx:xxxx","authority":"uat-api.scmp.work","path":"/istio-auth/first-request","protocol":"HTTP/1.1","upstream_service_time":"2","upstream_local_address":"-","duration":"4","downstream_local_address":"xxx.xxx.xxx.xxx:xxxx","upstream_transport_failure_reason":"-","route_name":"default","response_code":"200","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.132 Safari/537.36","response_flags":"-","start_time":"2020-03-19T14:44:44.899Z","method":"GET","request_id":"0893ed8c-0d98-9474-879d-9a155973bfc7","upstream_host":"xxx.xxx.xxx.xxx:xxxx","x_forwarded_for":"xxx.xxx.xxx.xxx","requested_server_name":"-","bytes_received":"0","istio_policy_status":"-"}
On Second request:  Custom access log for the first request gets logged along with default log for second request.  Path: (/istio-auth/second-request), Notice the timestamp for custom access log for first request 2020-03-19T14:44:44.899Z, but it appeared only when second request was triggered.
{"startTIME": "2020-03-19T14:44:44.899Z", "method": "GET", "Authorization": "-" ,"path": "/istio-auth/first-request"}\\n{"bytes_sent":"104","upstream_cluster":"inbound|5050|http-app|uat-app-hello-istio-auth.technology-newsroom-system.svc.cluster.local","downstream_remote_address":"xxx.xxx.xxx.xxx:xxxx","authority":"uat-api.scmp.work","path":"/istio-auth/second-request","protocol":"HTTP/1.1","upstream_service_time":"3","upstream_local_address":"-","duration":"4","downstream_local_address":"xxx.xxx.xxx.xxx:xxxx","upstream_transport_failure_reason":"-","route_name":"default","response_code":"200","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.132 Safari/537.36","response_flags":"-","start_time":"2020-03-19T14:46:51.377Z","method":"GET","request_id":"ea190826-c287-986d-86ae-5bf36cfde331","upstream_host":"xxx.xxx.xxx.xxx:xxxx","x_forwarded_for":"xxx.xxx.xxx.xxx:xxxx","requested_server_name":"-","bytes_received":"0","istio_policy_status":"-"}
I also tried to add a newline character to seperate the custom logs from default, but does not seems to work.
Kindly help us troubleshoot this issue of delay logging.
[ X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure
Expected behavior
Custom format access log must be logged in realtime, along with default access log.
Steps to reproduce the bug
Unknown
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
Istio version:
client version: 1.4.0 control plane version: 1.4.2 data plane version: 1.4.2 (13 proxies), 1.5.0 (3 proxies)
kubernetes version:
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.9", GitCommit:"3e4f6a92de5f259ef313ad876bb008897f6a98f0", GitTreeState:"clean", BuildDate:"2019-08-05T09:22:00Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"darwin/amd64"} Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.8-aliyun.1", GitCommit:"51888f5", GitTreeState:"", BuildDate:"2019-10-16T08:29:13Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
HELM version:
Client: &version.Version{SemVer:"v2.14.2", GitCommit:"a8b13cc5ab6a7dbef0a58f5061bcc7c0c61598e7", GitTreeState:"clean"} Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
How was Istio installed?
Using HELM client
Environment where bug was observed (cloud vendor, OS, etc)
Cloud Provider: Alicloud
Service: ACK (Alicloud cluster for Kubernetes)
Envoyfilter yaml:
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  labels:
    env: uat
    project: hello-istio-auth
spec:
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: ANY
      listener:
        filterChain:
          filter:
            name: envoy.http_connection_manager
    patch:
      operation: MERGE
      value:
        typed_config:
          '@type': type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
          access_log:
          - config:
              format: '{"startTIME": "%START_TIME%", "method": "%REQ(:METHOD)%", "Authorization":
                "%REQ(AUTHORIZATION)%" ,"path": "%REQ(X-ENVOY-ORIGINAL-PATH?:PATH)%"}\\n'
              path: /dev/stdout
            name: envoy.file_access_log

I think envoy does some batching/buffering?

@howardjohn Thanks,  is batching/buffering enabled by default?  I am not sure where to look for configuration. Kindly help check.

Strangely it's now working after updating the envoyfilter manifest to change the access log format to JSON and also tried adding a new custom header field.
Updated envoyfilter.yaml
kind: EnvoyFilter
metadata:
  labels:
    env: uat
    project: hello-istio-auth
spec:
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: ANY
      listener:
        filterChain:
          filter:
            name: envoy.http_connection_manager
    patch:
      operation: MERGE
      value:
        typed_config:
          '@type': type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
          access_log:
          - config:
              json_format:
                Authorization: '%REQ(AUTHORIZATION)%'
                custom-header: '%REQ(custom-header)%'
                method: '%REQ(:METHOD)%'
                path: '%REQ(X-ENVOY-ORIGINAL-PATH?:PATH)%'
                startTIME: '%START_TIME%'
              path: /dev/stdout
            name: envoy.file_access_log

Log evidence -
{"downstream_remote_address":"xxx.xxx.xxx.xxx:xxxx""authority":"uat-api.scmp.work","path":"/istio-auth/eric-latest","protocol":"HTTP/1.1","upstream_service_time":"2","upstream_local_address":"xxx.xxx.xxx.xxx:xxxx","duration":"4","upstream_transport_failure_reason":"-","route_name":"default","downstream_local_address":"xxx.xxx.xxx.xxx:xxxx","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.132 Safari/537.36","response_code":"200","response_flags":"-","start_time":"2020-03-23T02:34:12.734Z","method":"GET","request_id":"b476bace-7506-9006-bc75-7178b5a6b2b5","upstream_host":"<IP_ADDRESS>:5050","x_forwarded_for":"xxx.xxx.xxx.xxx:xxxx","requested_server_name":"-","bytes_received":"0","istio_policy_status":"-","bytes_sent":"103","upstream_cluster":"inbound|5050|http-app|uat-app-hello-istio-auth.technology-newsroom-system.svc.cluster.local"}
{"path":"/istio-auth/eric-latest","startTIME":"2020-03-23T02:34:12.734Z","method":"GET","custom-header":"-","Authorization":"-"}
{"bytes_received":"0","istio_policy_status":"-","bytes_sent":"86","upstream_cluster":"inbound|5050|http-app|uat-app-hello-istio-auth.technology-newsroom-system.svc.cluster.local","downstream_remote_address":"xxx.xxx.xxx.xxx:xxxx","authority":"uat-api.scmp.work","path":"/istio-auth/eric-with-header","protocol":"HTTP/1.1","upstream_service_time":"16","upstream_local_address":"xxx.xxx.xxx.xxx:xxxx","duration":"17","upstream_transport_failure_reason":"-","route_name":"default","downstream_local_address":"xxx.xxx.xxx.xxx:xxxx","user_agent":"curl/7.64.1","response_code":"200","response_flags":"-","start_time":"2020-03-23T02:36:11.925Z","method":"GET","request_id":"955465a0-b288-918b-9c64-6025d5fdb05a","upstream_host":"xxx.xxx.xxx.xxx:xxxx","x_forwarded_for":"<IP_ADDRESS>","requested_server_name":"-"}
{"path":"/istio-auth/eric-with-header","startTIME":"2020-03-23T02:36:11.925Z","method":"GET","custom-header":"testing","Authorization":"-"}
Can you put some light on how is the issue magically resolved, and not there is no delay in logging anymore , is it some kind of bug ?

I have the issue too...


bigloo: explicitly depend on unistring on darwin

Motivation for this change
Build is currently broken on darwin.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 140521 run on x86_64-darwin 1

1 package built:

bigloo

https://hydra.nixos.org/build/155226001

hydra.nixos.org/build/155226001

Whats the issue with that? builds fine.


Update cc_grpc_library.bzl

This pr resolves issue #20675 . To ensure that each unique directory is added only once to the --proto_path arguments in the protoc command within the cc_grpc_library.

@gnossen in case you are interested here

@Anurag-Modak are you still working on this?


Add classifications for TagHelper element/attribute completion tooltips in VS

Summary of the changes

Adds classifications for Razor TagHelper element/attribute completion tooltips.
For the most part, I based the "expected" behavior off the old non-LSP Razor editor. I also looked at C#'s tooltip output to ensure we have semi-consistent behavior with them.
One part I was unable to figure out from the present infrastructure is how to tell a user type from a non-user type, so for now everything is just grouped under "Types". C# in particular makes a distinction between these two scenarios.

Before:

After:

Fixes:
https://github.com/dotnet/aspnetcore/issues/30226

Side note, does anyone know how to write our own descriptions for TagHelpers that will show up in the IDE? I verified the behavior in this PR using the default TagHelpers + unit tests, but for further testing I think it would be helpful if I could figure out how to write my own descriptions.

So excited to see this all come together @allisonchou!! Everything is going to feel so much more polished!

What does C# do if you have a <code> bit in a doc comment?

Ahh they convert it to stylized text - just figured out how to do that on the Razor end too, so I matched their behavior.
Sidenote, while testing things out yesterday/today, I noticed there's a lots of various xml comment syntax that this PR still doesn't handle correctly. remarks, list, exception, etc. I'm working addressing them all now but seems like there's additional work I didn't anticipate, so this may not merge in as fast as expected. 😦

Filed https://github.com/dotnet/aspnetcore/issues/32286 to track further improvements to this feature such as handling more XML comment tags and increasing parity with C#. Taylor mentioned the existing implementation should hopefully cover the vast majority of Blazor component tooltips, so just pushing off the rest of this til later. 😄
If no one has any objections, I will merge this PR once all the tests pass.

No objections at all. Pumped to get this in!


select event is only fired when the selection spans multiple nodes

It looks like the 'select' event is only fired when a selection spans multiple nodes, while the documentation states that the select event should be fired when "[t]he user selected some text." Based on the documentation, I would expect that we would get a select event whenever the selection changes, regardless of the boundaries of the selection.
I believe these events are getting filtered out in Editor.js, line ~344-346:
    if ( anchor !== focus ) {
        this.fireEvent( 'select' );
    }

Agreed. Sorry it took a while for me to look at this.


Clarify labeling of volume field on Prepare Shotgun Libraries page

User said, "I'm not sure why Volume is here", and couldn't figure out what value should go in field.
Raised in shotgun walkthrough 07/31/2018

I'll confirm with the wet-lab and the source what the intention of the field was supposed to be.

In brief, labcontrol/gui/templates/library_prep_shotgun.html is the page that contains this volume field, and unlike its 16s counterpart, there are no comments on what it's used for.
The value 'volume-input' gets assigned to the 'volume' key before going out on the POST call to endpoint /process/libary_prep_shogun.
The POST request is handled by the class LibraryPrepShotgunProcessHandler, where volume is passed along with the other parameters to LibraryPrepShotgunProcess.create().
Each parameter in create() has a one-line description, and here is the description for Volume. Note that most peoples' intuition is that volume is supposed to represent the final volume of the well, but this appears to be the initial value:
...
volume : float
The initial volume in the wells
(see class LibraryPrepShotgunProcess(Process): in process.py)


Resume Not working?!

Whenever I try to pause my music its pauses correctly, but when I try to resume it, it emits the event but doesn't resume playing the music.

Code?
Lavalink Server console?


Concurrent Request, pulling from multiple APIS

I am not sure how to do this, but I want to have Concurrent Requests to for example api1, api2 with each having its own data, loading and error state. How do I do that? What should I pass into the run function? The URL? Very new to this, so I would love some help!

Maybe you should separate requests?
const firstRequest = useReqeust(service, {manual: true});
const secondRequest = useReqeust(service, {manual: true});

// firstRequest.run()
// secondRequest.run()

// firstRequest.loading
// firstRequest.data

// secondRequest.loading
// secondRequest.data


Higher CPU/GPU usage when viewing GIF preview thumbnails

The Issue
When viewing the explore page which currently has the @3dprinting channel (or when going to lbry://@3dprinting), GPU and CPU usage is noticeably higher and stays as such file they are in focus.
The same happens when I visit https://giphy.com, so I'm not sure if there's any solution to this at the moment...besides potentially not allowing GIFs, or limiting their size for thumbnails.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

It's sad but probably all GIF animation on tiles needs to go :(

Will replace with a new ticket to only loop GIFs on mouseover.


JOHNZON-35

Mapper fails on short and byte array
Changed the use of Integer.class.cast to Number.class.cast

Merged


Azure Powershell Task v.4.168.0 affect Enable-ContextAutoSave

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Question
Enter Task Name: Azure Powershell
Environment

Server - Azure Pipelines

Agent - Hosted

Issue Description
In my script I have an Enable-AzContextAutoSave which always work fine. But recently all my new releases and redeployed releases (which succeeded before) were failed. My pipeline configurations were all the same, the script hasn't been touched, I just trigger the pipeline.
I did some investigation and found out that Get-AzContext in my script returned null, including every command that is using the context (e.g. Get-AzSubscription return error message said I have to run Connect-AzAccount).
After a while I realized that the tasks version have just changed from 4.167.0 to 4.168.0 and then the one that failed all my release is the Enable-AzContextAutoSave command.
So I had a look in your repo and found this:
https://github.com/microsoft/azure-pipelines-tasks/commit/e4127e0977a9fd22caf09a53b627bb1263765ba7
Should there be any notice to the users before rolling out this change?? I understand that this might be a minor change, but it could affect users. I have to spent some times to find the cause of this out-of-nowhere issue and it affected my deployment timeline.

we will do internal investigation why it got regressed, could you please confirm if you have changed any PS version in task

I have not changed any configuration in my pipeline task. The task version changed from 4.167.0 to 4.168.0 by itself.

@KorKiak The change was made due to security reasons.
Earlier what was happening : if you have a pipeline with azurepowershell task on a agent and in the same pipeline you add another agent job with same agent and task powershell, it was saving az context as it was not scoped properly. So if you run some az command in powershell task, it was giving correct results which should not be the required behavior.
Due to this security reason, we are scoping the context only for the current process.

Thanks for your explaination! But can you please explain why the Enable-AzContextAutoSave command then ended up not saving anything to the context?
Will Enable-AzContextAutoSave -Scope Process work?


Unable to scroll behind the sheet, but able to click

Even without a Backdrop the content behind the sheet is not scrollable, but it is clickable.
It is likely that the suggestion in pull request #138 can add functionality to resolve this.
This is due to the scroll-locking behaviour by default.

PR #138 has been merged and disableScrollLocking is now released in v2.1.0.


Those aren't floats...

Am I misunderstanding something here? I fail to see floats.

Can one of the admins verify this patch?

Signed CLA

I believe these are actually floats. Sure, floats that each carry only one bit of information, but partly for historical reasons, partly for convenience, they are floats.


notifications: Add # to stream name in desktop notifications.

Follows the general pattern of always using a stream marker.

Merged, thanks @alya!


V0.12.0.x remove nLastPaid from CMasternodeBroadcast

We should not update nLastPaid from broadcasted messages

thanks!


Add continuous subscription to subscribe method at a specific frequency

This PR solves https://github.com/eclipse/kuksa.val/issues/652
How can be tested?

Use this test.json file as the VSS input for the broker. It contains the necessary overlays modified to test it with the ChangeType values Continuous and OnChange to verify the expected behavior
test.json

You can subscribe with databroker-clie to the signals Vehicle.Driver.HeartRate (Continuous update) and Vehicle.Body.Trunk.Front.IsLightOn (OnChange update).

Kuksa-client seems to have some bugs that need testing for this new feature.

What to expect?

The value for Vehicle.Driver.HeartRate should be set, as well as for Vehicle.Body.Trunk.Front.IsLightOn.
Subscribe to both paths, and the terminal should display the Continuous values every second and OnChange values only when updated.

Kuksa client to test it
https://github.com/eclipse-kuksa/kuksa-python-sdk/pull/21

Are you happy with the content of this PR now @argerus ? I.e. is it ready for merge?

Probably not. As is currently stands, this PR sets a hard coded update frequency of 1 second for everything, with no way to set it to anything else.
This combined with the fact that we don't include a VSS db with the change types configured, and every signal now defaults to Continuous is probably not the behaviour we want. I do have a version of the VSS db with all change types configured, so perhaps that will make the out of the box behaviour better (if we merge that first).
I still think this PR should include the ability to change the update frequency through the API, otherwise I'm not sure much is gained by merging it.

Will then add the frequency parameter to the subscribe request on this PR, was thinking to split it into different PRs but makes no sense.

we need to make sure that the changes are reflected in the sdk's. Python one https://github.com/eclipse-kuksa/kuksa-python-sdk/pull/21 is taken care of already. Do we want to have it in GO fixed as well? Rust SDK is working I guess?

After community call was agreed to avoid changes in VSS and to add the "ChangeType" directly to the subscription request implicitly with the optional frequency parameter.

Reopening all PRs closed by accident by boschglobal maintenance

Databroker has been migrated to https://github.com/eclipse-kuksa/kuksa-databroker.
Please open a new pull request in that repo.


[Powertoys update] Update requires to disable the Microsoft Text Input Application

Windows build number: 10.0.19041.264
PowerToys version: v0.18.1
PowerToy module for which you are reporting the bug: auto update

The update program requires to disable the Microsoft Store, Microsoft Text Input Application, Mail which are run at background. I can manually kill the process for Microsoft Store and Mail but Microsoft Text Input Application. However, I can just ignore this to continue. Just unsure does this effect anything.

if it only was this

@enricogior we need to look at this in 0.21

@crutkas
I have the feeling that this is not something we can fix it, we don't uninstalled shared dll, it looks like Windows gets confused and thinks that those apps are using our dlls.

We need a way to reproduce this, I hit this once extremely early on but not anymore.

without repro, we can't debug

Understood. I have not had this happen again since I posted this.

we still need a repro for this.  we've seen it happen but not recently

This is a random Windows Installer bug, choosing Ignore is good enough and won't prompt to restart Windows.

As one of the first ones to have reported this issue, I can tell you that I have not seen this issue since I reported. The update now is smooth and quick.

@Smiley0227
thanks for the follow up!


StopPlaybackAndRemoveNotification option is not worked 100% times

Describe the Bug
The android does not respect AppKilledPlaybackBehavior.StopPlaybackAndRemoveNotification option
The reproduction: 20/100.
We have experienced the issue across different models of devices and different android system versions.
Our options:
await TrackPlayer.updateOptions({
    android: {
      appKilledPlaybackBehavior: AppKilledPlaybackBehavior.StopPlaybackAndRemoveNotification,
    },
    capabilities: [Capability.Play, Capability.Pause],
    notificationCapabilities: [Capability.Play, Capability.Pause],
    compactCapabilities: [Capability.Play, Capability.Pause],
    progressUpdateEventInterval: PROGRESS_UPDATE_EVENT_INTERVAL,
  });

Steps To Reproduce

Open the app
Leave the app opened for ~20 minutes
Start playing track
Minimize the app
Close the app
The track is still playing and the notification is presented in notification center

**Environment Info:**
react-native-track-player versions: "4.0.1", "4.1.1".
Real devices (developing, production)
OS: Android
System:
OS: macOS 14.5
CPU: (10) arm64 Apple M1 Pro
Memory: 93.53 MB / 16.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 18.19.0
path: ~/.nvm/versions/node/v18.19.0/bin/node
Yarn:
version: 1.22.22
path: ~/.nvm/versions/node/v18.19.0/bin/yarn
npm:
version: 10.2.3
path: ~/.nvm/versions/node/v18.19.0/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.14.3
path: /Users/ianiskondakovmercury/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 23.5
- iOS 17.5
- macOS 14.5
- tvOS 17.5
- visionOS 1.2
- watchOS 10.5
Android SDK:
API Levels:
- "34"
Build Tools:
- 33.0.1
- 34.0.0
System Images:
- android-34 | Google APIs ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2023.3 AI-233.14808.21.2331.11926650
Xcode:
version: 15.4/15F31d
path: /usr/bin/xcodebuild
Languages:
Java:
version: 17.0.11
path: /usr/bin/javac
Ruby:
version: 3.1.2
path: /Users/ianiskondakovmercury/.rbenv/shims/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.73.8
wanted: 0.73.8
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: false
iOS:
hermesEnabled: true
newArchEnabled: false

I found the conversation in Discord with the same issue. https://discord.com/channels/567636850513018880/1188151539764822036
But i can not find any solution.

this is due to the foregroundservice (MusicService) being stopped AND killed at that time and RNTP cannot restore the service and reattach properly as it should (eg. audio_service can). this disconnects the notificationManager and the service and/or the activity, causing turning off activity wont properly terminating the service.
u might get somewhere by fiddling with the service termination logic, or alternatively set stopForegroundGracePeriod to some astronomically large number to force keeping your service, at the expense of ~0.2% battery usage per hour.

@lovegaoshi thank you for suggested workaround. Sorry for late response, we have tested the workaround.
I added stopForegroundGracePeriod: 99999999, and unfortunately it did not not help, and the reproduction now is 50/100.
Can you explain a bit more about this suggestion?

u might get somewhere by fiddling with the service termination logic

https://github.com/doublesymmetry/react-native-track-player/blob/9ff32f0f5a7b4f96900bf422810d587f5d2e3789/android/src/main/java/com/doublesymmetry/trackplayer/service/MusicService.kt#L774
u should also consult how audio_service archives this logic
On Thu, Jun 20, 2024, 1:01 AM NinjaDevRN @.***> wrote:

@lovegaoshi https://github.com/lovegaoshi thank you for suggested
workaround. Sorry for late response, we have tested the workaround.
I added stopForegroundGracePeriod: 99999999, and unfortunately it did not
not help, and the reproduction now is 50/100.
Can you explain a bit more about this suggestion?
u might get somewhere by fiddling with the service termination logic
—
Reply to this email directly, view it on GitHub
https://github.com/doublesymmetry/react-native-track-player/issues/2328#issuecomment-2180057792,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZMOVVUE2HT5VZRRSH7DUZLZIKD45AVCNFSM6AAAAABJOQANAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBQGA2TONZZGI
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Thank you!


No such image: error

Trying to run the demo. When I run
lewis2020 = BART(model="bart.large.cnn")
summary2 = lewis2020.predict(document)

I Get
Not Found ("no such image: danieldeutsch/lewis2020:1.1: No such image: danieldeutsch/lewis2020:1.1")
and for dou
404 Client Error for http+docker://localhost/v1.41/images/danieldeutsch/dou2021:1.0/json: Not Found ("no such image: danieldeutsch/dou2021:1.0: No such image: danieldeutsch/dou2021:1.0")
But this model works
liu2019 = BertSumExtAbs(device=0)

Very strange! Both images do exist on Docker Hub (Lewis, Dou). If the Docker image does not exist on your machine, it should be automatically downloaded.
Could you try running docker pull danieldeutsch/lewis2020:1.1 and docker pull danieldeutsch/dou2021:1.0, then retrying the demo code? This should not be necessary, but it will help identify where the problem is.

It looks like the dou2021 image may be stale. My DockerHub account uses the free tier, and it seems like they may delete images after they haven't been used for some time. If this is the case, I will investigate upgrading my membership.
The lewis2020 image should not be affected by this, though. The 1.1 tag was pushed 2 months ago.

Ah ok. it's working after I ran docker pull :).

This error should not happen because the Docker python library automatically downloads the images if they don't exist, so perhaps the downloads failed for some reason? 🤷♂ Anyway, glad it's fixed

Thanks :)  I just tried again on a fresh EC2 instance and worked without docker pull.


Feature suggestion : arcade physics engine body.setMaxSpeed(speed)

Currently arcade physics engine has body.setMaxVelocity(x, y) to limit velocity  in x and y part separated.
For example, body.setMaxVelocity(200)

If acceleration is (200, 0) every tick, the velocity will be (200, 0) -> max speed is 200.
If acceleration  is (200, 200) every tick, the velocity will be (200, 200) -> max speed is 282.8427....

How about adding body.setMaxSpeed(speed) to limit the speed (vector length of velocity) in a steady value?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Yeah, worth adding at some point I think.

Set MaxVelocity will also limit the final angle of moving direction when speed-up continuously.
For example  -- asteroids movement, moving direction of ship will become 45/135/225/315 degrees (maxVelocity is (200,200) here) if initial angle of ship is not 0.

Is someone working on this feature? If not, I'd like to try to do this.


App processes running old release after a successful update of environment variable

Experienced this issue tonight:

Updated one environment variable in an app running on our Flynn cluster
Flynn created a new release (I didn't check to see if the app was running the release, though)
Updated a second environment variable in the same app
We noticed that the change didn't have the effect we were expecting
Tried restarting processes by scaling down then up
Effect of the previous step was that instead of 4 processes (2 web, 2 workers), we now had 8 (4 web, 4 workers)
Thinking it was an issue with the version, I pushed the latest from git repo to the cluster (i.e. > git push $remote $branch:master)
Checked the output of flynn -a $appname ps and noticed the processes were not running the release that i just pushed, but one from 2 hours prior

I have debug logs to share; ping me on IRC.

@tjboring please email the logs to me<EMAIL_ADDRESS>(feel free to encrypt them with my PGP key).

@tjboring I have been unable to identify the issue here.
I have updated the debug logs so that they contain the scheduler leader's in-memory state (#2998), but you will need to be running on a cluster which has that change (it will most likely be in nightly tonight, but not in stable just yet).
In the mean time, please collect more debug logs as this continues to happen (preferably just after a deployment, and from each host in the cluster) as they may provide more insight.

@lmars thanks, lewis! i just upgraded to the latest stable today, but i'll keep an eye and grab debug logs as necessary.

So, this happened about in the last 24 hours. Here's what I know:

~16h ago, we had a deploy from CircleCI, which was successful
In the last hour, I had a dev ask why his code wasn't running in the environment
checking the output of flynn -a $app ps i could see workers that were running a release from ~19h before

To resolve this issue, I did the following:

`flynn -a $app scale -r $release_id web=0 clock=0 worker=0
verified no processes running
`flynn -a $app scale web=N clock=N worker=N

I've collected the controller logs and emailed them as attachments.

Looking at the logs from the latest incident, here is what happened:

A deploy began at 2016-08-10T18:29:24 which determined that the old release scale was worker:2 clock:1 web:2 but actual jobs running was worker:1 clock:1 web:2 (i.e. a missing worker job)
The deploy scaled the new release up to worker:2 clock:1 web:2 but then failed when waiting for the down events for the old release as it in fact got two worker down events rather than one (the deploy shouldn't actually fail in this case, bug tracked in #3054)
The app's release remained as the old (now scaled down) release (bug tracked in #3285) and a subsequent deploy realised the app was scaled down so did a default scale of web=1

We should investigate why the worker thought only one worker was running when there were in fact two, but fixing both #3054 and #3285 will prevent this from being an issue.