Power BI Visuals not honoring format strings in 1500 CM model with Calculation Groups?

Hi guys, not sure yet if this is a Power BI, AAS or a Tabular Editor issue but we have found that our new 1500 cube with calculation groups is not behaving correctly in Power BI
Power BI Matrix/Table and Cards work correctly but all of the visuals (line/bar/pie etc) do not honour the format string
e.g.  left is a table, right is a bar chart for the same Axis/measure

Other things I've noticed

In Power BI we see every measure duplicated - the dupe holds the value for the format string and is always hidden

Excel pivot tables with live connection to the AAS work correctly

This model was built and deployed with Tabular Editor. Not sure if deploying with the MS tool might make any difference but that is one thing that I would like to try.
Any other ideas on what might be causing this? Seems a pretty major issue that I would have thought Microsoft would have found during their dev/test...

I suggest you to open a ticket with Microsoft support as soon as possible.
For the first issue: it should depend on the visuals, for sure any custom visual should not work with dynamic format string, whereas the native Card visual should work correctly. I'm not sure about other visuals, but it is possible that there are visuals not working. I already raised the issue months ago with Microsoft, but as usual only tickets from customer can help to escalate the issue.
The second one (duplicated measures) is weird and I didn't notice it before. However, it's something that Microsoft has to fix in Power BI.
So my suggestion is to open two different tickets for the two issues, because they involve different teams within Microsoft and you will streamline the process this way. Please forward the ticket ID here when you have them - thanks!

Thanks Marco, thats a real shame and will prevent us from being able to release Calculation Groups at all. Surprising that it affects all our existing normal measures, but I guess they have had to change the whole format string system without any backward compatibility.
Card, Multirow card, Table and Matrix honour the format string. All the other standard visuals do not.
I guess custom visuals will be upgraded over time, or is the ability not yet provided to the custom visuals?
The first issue ticket number is<PHONE_NUMBER>01023
The second is less of an issue, but will post when we have that one

@o-o00o-o I would recommend holding off on deploying Calculation Groups/1500 models until at least CU2.  We are tracking a pretty nasty issue right now with tons of BLANK()s coming seemingly from out of nowhere, and MS is tracking CU2 for our fix in particular.  Thank you for filing your bugs; hopefully these issues shake out soon.  Calculation Groups are incredible.

Closing this as neither of these issues can be resolved within Tabular Editor (it's simply just using the TOM to apply the changes - you would get result by manually creating a calc group through TOM or through the use of a TMSL script). Please raise a ticket with Microsoft as suggested by Marco.

Just for visibility, the Power BI support team are now agreeing this is a known limitation and that it isn't planned to be developed until summer this year with no delivery ETA.
This means that we (and I'm sure most existing cubes with Power BI reports against them) will be prevented from using calculation groups until them
I've created a community post with the details here.
https://social.msdn.microsoft.com/Forums/en-US/3c71faf0-6613-4ed2-b34b-48961732f514/adding-a-single-calculation-group-to-aas-breaks-most-of-the-existing-company-power-bi-reports?forum=AzureAnalysisServices
@marcosqlbi I'm not sure if you or Alberto are able to help with your MS contacts to make sure this is put in front of the product owners - feels like a big oversight to me and I'm willing to be whoever decided on the priority of this wasn't aware that this would break existing reports that don't even use calculation groups.

Thanks for letting us know. Just out of curiosity, have you specified FormatStringExpressions for any of the Calc Items in your Calc Group? What happens if you specify SELECTEDMEASUREFORMATSTRING() as the FormatStringExpression across all your calc items?

yeah we tried all of that when we initially saw the problem. It makes no difference, Power BI just isn't looking in the right place for these non-table visuals
In the basic example with a Date Calculations table with a "YOY %" then yes, you have to provide a format string to have this present the number as a % instead of the selectedmeasure's formatstring.
As soon as you add a calculation group to a model, Analysis Services dramatically changes the model in how it provides the queries with the format string to apply.
Suddenly every measure now has 2 measures with the same name (the measure itself and also the measure containing the format string) - you see it in both power bi and dax studio

So now the query tools need to write queries that return the format string for this specific context, rather than just being able to take the format string from the measure.
It is a shame they didn't implement Format Strings in the same way they did for Data Types as this is provided as metadata with the resultset - although now you have to check per cell of results, again instead of being able to just get the datatype from the column definition.

The issue is bigger than I realized initially. I discovered this because custom visuals were not supporting dynamic format strings. But the fact that this breaks standard format string... well this is huge. I tried to raise awareness of the issue in MS pointing to the blog post in the Microsoft forum, let's see what happens in 1-2 days otherwise we'll escalate making it more visible in a blog post.

I guess the reason it’s not implemented as metadata is because you can have different format strings in the same column for different rows, whereas the data type is always the same across all rows for any given column. So they probably had to come up with a different implementation in which the format string is returned in a separate column.

@marcosqlbi Thanks for your support on this, Yes it is a massive issue as it breaks static format strings for normal measures. Appreciate anything that you can do to raise awareness about this. Considering most people using AAS/Cube are now using Power BI, it means that this great feature from the AAS team cannot be used by anyone due to this bug in Power BI. The problem was it was hidden as we did all our testing on our Calculation Groups using Matrix and Table visuals. Only when we released, immediately users were complaining that their existing charts with % measures were now showing as unintuitive decimal numbers without any format string applied.
@otykier Well, interestingly this is now the same for datatypes, As soon as you add a Calculation Group to a model, all measures have their datatype changed to variant and now the datatype can be changed for each cell rather than by each column for a similar reason as now the output of a Calculation Item being applied to a measure can change its data type. If you check the output of a DAX query (like using Invoke-AsCmd.ps1) it now indicates the output datatype per cell instead of for each column in the header area (as it used to). So I'm not sure why they didn't use this same method for Format Strings tbh.. perhaps there is a reason, or maybe two different approaches to the same problem.

The MS forum suggested I create a Idea so I did that here https://ideas.powerbi.com/forums/265200-power-bi-ideas/suggestions/39750307-format-strings-don-t-work-on-a-ssas-model-with-cal

The issue with measures changing their data types to variant is familiar. It happens when you have calculation items that do not follow certain patterns, such as CALCULATE(SELECTEDMEASURE(), ...) where the data type is guaranteed to not change. These patterns are unfortunately not documented, as far as I know. If you have any calc items that doesn’t follow this pattern, they have to decay all measure data types to variant.
As to the issue with format strings, here is some speculation on what I think is going on. Maybe @marcosqlbi knows the internals in more details.
Invoke-AsCmd actually uses XMLA to send the command to the model, meaning the returned result is a cellset, which explains why the data type is specified for each cell. On the other hand, when querying the model with an OLEDB connector, we get a flat dataset as the result, where its not possible to have differing data types per cell. I’m guessing Power BI uses the latter or another similar protocol that returns a flat dataset. You probably don’t see this issue with the format strings in Excel. That’s because Excel also uses XMLA, but since it uses XMLA with MDX it’s also able to get the format string per cell.
As to the fix: Microsoft either needs to make sure that all visuals query the hidden format string measures for every measure that is being used in the visual, as well as ensure that the visual actually applies that format string when rendering the value. Alternatively, they should query the model using XMLA and consume the returned cellset, but this probably requires much more effort.

The issue with the visuals is that are different types of visual, and depending on that, they manage variant or not. It seems this is an unfinished work and this is the behavior they probably have also when they connect to Multidimensional. The only thing we can do is to give more visibility and hope other big customers raise the issue with Microsoft.

@otykier well, the XMLA thing definately changed either due to switching to CM 1500 or by adding a Calculation Group to the Model as we have to modify our Automated Test tools. Previously we could use a datatype indicator in the header of the XMLA result but then we had to switch to cope with the datatype in each cell as the header just said variant now. Perhaps it was there before but we didn't need it as it was good enough to refer to the header one so we could tell numeric types for tolerance in our test tools.
@marcosqlbi the support team did mention something about it being a known issue for MD.
So what you guys might be suggesting is that if I can stop the measures all changing to variant (perhaps with identifying that pattern) then perhaps we can work around this issue.
I will definitely give that a go.. although it may not be easy or possible in our case for it to really do what we want..

Hi folks,
I have the same issue here, and I have discovered something that might help in the struggle.
The thing is, in my case, I find the following:

When Calc groups are added to the model, yes, the measures become variant. But this is not an issue by itself!...
If all FormatStringExpression of all Calculation Items are empty, then the behaviour of measures is unchanged (at least in my testings) - the usual format strings of normal measures are properly shown in graphs, business as usual, all happy
If afterwards, I add just one FormatStringExpression to ANY of the Calculation Items, (including SELECTEDMEASUREFORMATSTRING() or any other), then the bug arises and graphs in PowerBI do not respect their source formatting (matrix is OK as you have reported)

So in my case, it looks that the problem does not come from just 1500, or from adding calculation groups, but from adding a single FormatStringExpression  to the model.
Therefore, in my case so far I am able to add Calculation Groups and Items without and issue to an existing model (phew!), but not use the custom format feature.
I hope this helps shed some light, and yes, Microsoft should solve this!
Best regards, Martin

Another breaking change I just found is that I have some format strings set
to the words "Short Date" in my model in TE.
That's all broken now; instead of giving me what I expect, my dates are
displaying as "S12orA DaAe" -- evidently all format strings need to be
explicit!
On Sun, Feb 23, 2020 at 3:13 PM zenzeinet<EMAIL_ADDRESS>wrote:

Hi folks,
I have the same issue here, and I have discovered something that might
help in the struggle.
The thing is, in my case, I find the following:

When Calc groups are added to the model, yes, the measures become
variant. But this is not an issue by itself!...
If all FormatStringExpression of all Calculation Items are
empty, then the behaviour of measures is unchanged (at least in my
testings) - the usual format strings of normal measures are properly shown
in graphs, business as usual, all happy
If afterwards, I add just one FormatStringExpression to ANY of the
Calculation Items, (including SELECTEDMEASUREFORMATSTRING() or any other),
then the bug arises and graphs in PowerBI do not respect their source
formatting (matrix is OK as you have reported)

So in my case, it looks that the problem does not come from just 1500, or
from adding calculation groups, but from adding a single
FormatStringExpression to the model.
Therefore, in my case so far I am able to add Calculation Groups and Items
without and issue to an existing model (phew!), but not use the custom
format feature.
I hope this helps shed some light, and yes, Microsoft should solve this!
Best regards, Martin
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/otykier/TabularEditor/issues/380?email_source=notifications&email_token=AA5N5TL5Z3HHWUJ4YVYPMHDRELKHDA5CNFSM4KEWBB42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMWGA2Q#issuecomment-590110826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5N5TJFX4RLMLLFN3ZWEDDRELKHDANCNFSM4KEWBB4Q
.

@zenzeinet thank you for sharing this!
Actually this does provide an ok workaround so at least we can deploy the Calculation Groups to live and have our tests continue to work.
It just means that for the few calculation groups that require % format strings, these will still show as 0.01 etc. I can't see a good workaround for this, although I may want to try creating a new base measure that just presents as a % and see if this can work as without a format string it seems to automatically pick up the base measure format string.

@o-o00o-o
Yes, that is what we are currently doing. It is too bad, as the custom formating would be very nice in quite a few situations (and I cannot even add it selectively - it is all or nothing). Currency exchange, % measures, ratios, showing +/- simbols, custom dynamic units (eg showing Kb, Mb, Gb) etc.
I'm hoping microsoft addresses this bug, and for us the wait is at least a few months, as we're running PBIRS which updates 3 times per year. Hopefully it should be done next release, but what a bummer they did not find this in testing. I personally found the bug too and was initially blaming myself for not finding the proper setting in a line graph, thinking "damn, I'm quite sure this was showing %s properly before..."
Best regards

Hi,
It seems Microsoft has started the work:
https://ideas.powerbi.com/forums/265200-power-bi-ideas/suggestions/39750307-format-strings-don-t-work-on-a-ssas-model-with-cal
Unfortunately it seems it will take them several months... if Marco, Daniel or anyone reading this could push them a little it would be great.
Best regards

We have no power to push it - it's just good it's now in their roadmap.

Strange, the web page at
https://ideas.powerbi.com/forums/265200-power-bi-ideas/suggestions/39750307-format-strings-don-t-work-on-a-ssas-model-with-cal
has now dissapeared. Any of you know if this has been fixed in recent updates? I cannot test it until a new PBIRS version is released...
Thanks

They are working to solve the issue but I have no idea why they removed the item!

Update: A fix was just published to the July 2020 version of Power BI Desktop, where the dynamic formatting issue has been solved. Should be fixed in both the Windows Store version and the MSI.

Update: A fix was just published to the July 2020 version of Power BI Desktop, where the dynamic formatting issue has been solved. Should be fixed in both the Windows Store version and the MSI.

It doesn't look like it is fixed universally. The line chart works but column chart is still broken

I have the August 2020 version of the desktop and the issue still hasn't resolved it seems. All my measures are coming out as "Variant"

The measures have to be variant always when there are calculation groups.
That is not the issue.
The issue are visualizations not honoring the default formatting and/or the
calculation group format string.
PowerBI Matrix are OK. So are excel ones. The issue is with the remaining
visuals, eg line charts.
El lun., 12 oct. 2020 14:35, Nishant Jain<EMAIL_ADDRESS>escribió:

I have the August 2020 version of the desktop and the issue still hasn't
resolved it seems. All my measures are coming out as "Variant"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/otykier/TabularEditor/issues/380#issuecomment-707093110,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGTQLM6UEOBKWLWG6PI43ODSKLZY7ANCNFSM4KEWBB4Q
.

this issue has just tripped me up, and will have to revert to a model with no calc groups to honour existing reports! Noticed in Smart narrative visual (although not GA) can no longer render Text based measures because data type is variant.
any ideas on a fix from Microsoft?

@vibesniper which version of PBI Desktop are you using? Does this issue only affect the smart narrative visual?

I have to say I'm really frustrated with Microsoft's response to this and their commitment to the calculation item feature in general.
I have had a support ticket open with them for over a year. They have told me it was solved for all visuals except for Guage), but it isn't.
It might work now if you exclusively use Calculation Items and never use a measure on its own (but seriously.. who does that). Calculation Items are really quite limited on their own as often business users want to mix different combinations of calculation items for different measures - which ultimately still requires discrete measures today anyway.
As soon as you set a single format string on a calculation group, all format strings defined on measures are ignored (except on table and matrixes)
You shouldn't need to remove your calc groups however as they could still be useful for code centralisation. You just can't set Format Strings on any Calculation Items and it should works OK. Depending on your use, most of the time your calculation items are probably returning the same format string as the base measure, however if you are trying to do something like % variance from absolute figures (YOY % etc.) then you run into difficult territory. Right now we just multiply that number by 100 in our calculation item and then usually provide a % measure with the correct format string that calls the Calculation Item (has to divide by 100 again) and expresses the sensibly formatted %.
I actually sent an update to my ticket earlier this week. I will continue to chase them to help them feel our pain.
Really feels like they have put this into either the "its too hard" or the "something else shiny over here" bucket

@vibesniper which version of PBI Desktop are you using? Does this issue only affect the smart narrative visual?

March version (MS Store installation) - have just tested other visuals as text measure, Card, Multi-card and OKviz custom card states all work. So I may have been hasty in my issue, as I guess Smart Narrative is still in preview?
So maybe this more of a decision of using calc groups in model vs using visuals in preview. Just quite a shame after using calc groups/format strings to enhance the native visuals alongside smart narratives to get the most out of a report.

@o-o00o-o good points, I had a blog post about this topic and I wrongly thought Microsoft would have fixed the issue when they fixed the support in the first few visual, so I didn't complete it. I am putting it again in my todo list.

Thanks @marcosqlbi, I'm sure increasing the public visibility of this issue will help Microsoft prioritise this to unlock all the features of Calculation Groups

@vibesniper now try a column chart with a normal measure that has a format string (e.g. 0.00%) and notice that it now doesn't work

Having just tried it with the latest versions of Power BI Desktop April 2021 and Card with States 1.6.8:
As soon as you set a Format String Expression e.g. "0.0%;-0.0%;0.0%" on any calculation item,
measures with their own Format String set to e.g. 0.0%;-0.0%;0.0% all loose their percentage formatting.
Interestingly that now works fine in the standard KPI visual (I remember it wasn't the case several months ago). But still doesn't work in Card with States (which seems practically the only decent option for KPI targets/thresholds and small multiples).
It does seem like a major limitation. Or am I missing something?

The API for custom visuals still doesn't provide access to the feature. We have to wait Microsoft to release an API for custom visuals to support dynamic format strings, then wait until we implement it, then wait until the component is certified and published by Microsoft. However, we're still waiting for step 1.

That's too bad, such a great feature as calc groups + calc format strings
still not totally matured (for pbi charts, although it works great on
excel). Hope they finish that soon, they ain't in a hurry it seems!
On Sat, Apr 17, 2021, 15:40 Marco Russo @.***> wrote:

The API for custom visuals still doesn't provide access to the feature. We
have to wait for Microsoft to release an API for custom visuals to support
dynamic format strings, then wait until we implement it, then wait until
the component is certified and published by Microsoft. However, we're still
waiting for step 1.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/otykier/TabularEditor/issues/380#issuecomment-821824303,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGTQLM3K4TBXEPEKSIGVNKDTJGFUXANCNFSM4KEWBB4Q
.

@o-o00o-o In my real model I thought most native visuals were behaving, and Card with States by OKviz.  In my experience it's just the smart narrative visual not being able to render text measures, which is more disappointing since it's still not working now GA in May update. Dummy file below: The calc groups that replace the original measure (in the smart narrative) with an integer work fine though.
Dummy pbix

This issue is so stupid.. And dangerous because initial testing with matrices/tables does not reveal it.
Also, look at this: Bar chart without field in "axis" ignores formatstring. But if I put in a field, it is correctly formatted..

@marcosqlbi , this article should contain a warning in the intro, in my opinion.
https://www.sqlbi.com/articles/controlling-format-strings-in-calculation-groups/
Or do you consider this issue solved?

Just adding to the chat that I have encountered this today and it is still NOT fixed.


💡 [ADD]: ANIMATED UPLOAD BUTTON

Details

This issue is about adding a new animated upload button
The animation will start to play once the user clicks the button
GSAP Is used to animate the button

Screenshots
https://github.com/Spyware007/Animating-Buttons/assets/92252895/afce6a67-db8c-42ef-b8db-979b9632288a
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Sorry for the inconvinience, but we are closing all the issues! Please open a new issue if you want to contribute anything. Also now you can create a new button from the website itself! You will get points for those too! I will add the instructions on how to create a new button soon and create an empty pr for the same to get points!


Stuck in contract arguments, Please help, thank you so much!

I couldn't verify the contract, i have replace the arguments info, but still not working, please help!! thank you so much!

Dear team, after a long struggle, i have finally fix the error, then another obstacle comes up @@!! My API key invalid can not verify!! As a dummy for blockchain world, I have no clue what to do....
After a long....freaking hours research, eventually is because the BSC chain needs they specify API key to work.
Thank you so much for your guy explore me to a new era, specially the tutorial is amazing!!


VAD for e2e diarization

Thank you for maintaining and working on this library.
Hi, I am looking for documentation or perhaps pointers/suggestions for performing end-to-end diarization.
The current AMI diarization recipe utilizes labeled data to perform utterance splits. (as I understand it)
How would one approach an unlabeled AMI dataset to perform diarization?

Good question!
I have some of my students working on VAD. After having a reliable VAD we
can provide a recipe for AMI without using the provided segmentation.
As for E2E diarization, this is an extremely interesting research topic.
Over the last months, we tried some approaches, but the performance wasn't
good at all. This is an exellent topic for a collaboration with any
contributors interested in that!
Best,
Mirco
On Mon, 19 Apr 2021 at 09:52, Daulet Nurmanbetov @.***>
wrote:

Thank you for maintaining and working on this library.
Hi, I am looking for documentation or perhaps pointers/suggestions for
performing end-to-end diarization.
The current AMI diarization recipe utilizes labeled data to perform
utterance splits.
How would one approach an unlabeled AMI dataset to perform diarization?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/speechbrain/speechbrain/issues/657, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA2ZVU2H4IXPKVM3HOMPWLTJQYRTANCNFSM43F4DMOQ
.

Thank you for your input @mravanelli,
Can you point me to an approach that you think is worthwhile to try to further e2e diarization?
I am thinking a simple approach is segmenting on silences, but there maybe be a better approach to this which I'm unaware of which would address overlapping speech and sudden speaker change.

There was a growing interest in the literature on this topic in the last year but I'm not aware of a solution really working much better than standard non end2end with a reasonable amount of data (but I might be wrong here). If someone from the community would like to suggest something, this is really and interesting topic for a collaboration. The E2E neural speaker diarization is something that looks pretty reasonable to me: https://arxiv.org/pdf/1909.06247.pdf


add minimumInputLength & maximumInputLength

@iStefo any thoughts on this? I would need this option but would also like to avoid using forks :)

Sorry for the long delay and thank your for your addition! I would be happy to have the maximumInputLength parameter supported. Even though #61 was filled only today, I went ahead and merged that one because it provided a test case for this feature.
I will just implement a test case for maximumInputLength similar to the one provided for minimumInputLength.

:+1:


Add "Override Virtual Method" to "Quick Fix"

Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
Version:   2.1.401
Commit:    91b1c13032
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Users\nemos\scoop\apps\dotnet-sdk\current\sdk\2.1.401\
Host (useful for support):
Version: 2.1.3
Commit:  124038c13e
VS Code version: 1.27.1
C# Extension version: 1.16.1
Steps to reproduce
public class SuperClass
{
	virtual public void VirtualMethod() { /* Some code */ }
}

public class ChildClass : SuperClass
{
	override VirtualMethod // Call "Quick Fix" here
}

Expected  behavior
The whole method signature and (possibly) base.VirtualMethod(); should be added.
Actual behavior
No code actions available

Instead of implementing this as a code action, we would implement this as a special completion list that comes up after you type override (which is a behavior Visual Studio has had for a long time). https://github.com/OmniSharp/omnisharp-roslyn/pull/884 is tracking that work cc @DustinCampbell

Oh, that fits me well... I'll follow the linked issue, thanks!


Exception using .get_ipc_handle(...) on array from `as_cuda_array(...)

Reporting a bug

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

Am trying to access .get_ipc_handle(...) on a Numba DeviceNDArray created through as_cuda_array(...). Here's some code using CuPy that demonstrates this use case:
import cupy
import numba.cuda

a = cupy.arange(5)
a2 = numba.cuda.as_cuda_array(a)

a2.get_ipc_handle()

However am getting the following exception when calling .get_ipc_handle(...):
Traceback (most recent call last):
  File "script.py", line 7, in <module>
    a2.get_ipc_handle()
  File "/datasets/jkirkham/miniconda/envs/rapids13dev/lib/python3.7/site-packages/numba/cuda/cudadrv/devicearray.py", line 298, in get_ipc_handle
    ipch = devices.get_context().get_ipc_handle(self.gpu_data)
  File "/datasets/jkirkham/miniconda/envs/rapids13dev/lib/python3.7/site-packages/numba/cuda/cudadrv/driver.py", line 847, in get_ipc_handle
    memory.owner.handle,
AttributeError: 'cupy.core.core.ndarray' object has no attribute 'handle'

It appears this code that is trying to compute the offset causes the issue
https://github.com/numba/numba/blob/cc7e7c7cfa6389b54d3b5c2c95751c97eb531a96/numba/cuda/cudadrv/driver.py#L850
cc @Iroy30 @kkraus14

@gmarkall any chance you could take a look at this please? Thanks.

Already looking into it :-)

Already looking into it :-)

Brilliant, thanks!

The use of the owner of a MemoryPointer is a bit overloaded:

It's used to keep the owning object alive by holding a reference to it - as is set up by as_cuda_array.
It's also expected to be a MemoryPointer which points to the base of the allocation - this made sense when Numba was "isolated" and didn't communicate with other libraries through the CUDA array interface, but doesn't make as much sense now.

I think the second use can be replaced with a call to cuMemGetAddressRange instead, to disentangle these two uses, and also allow getting the base of the allocation for pointers that come in through the CUDA array interface - I'm experimenting with a fix now.

Fix proposed in #5437.

@jakirkham Thanks for creating the Issue. It is related to the error described in https://github.com/rapidsai/cugraph/issues/790. The proposed fix solves it.


IsInRange does not include bounds

I would expected IsInRange(0,100) to treat 0 and 100 as valid values. It doesn't.
Why do i consider the bounds to be valid? Let's say i have valid values ranging from int.MinValue to 0.
(Also notice my use of the word ranging. Seams natural to include bounds, doesn't it?!)
With IsInRange i would not need to do something like IsInRange(int.MinValue-1,0). Now, of course, i cannot subtract 1 from int.MinValue (at least not without converting to a datatype with a larger range first).

I was wrong. It includes bounds.


Text classifier Tasks

In the Text Classification Python Example, text is already imported, yet while defining classifier python.text was used.
Before
options = text.TextClassifierOptions(base_options=base_options)
classifier = python.text.TextClassifier.create_from_options(options)

After
options = text.TextClassifierOptions(base_options=base_options)
classifier = text.TextClassifier.create_from_options(options)

Hi Tarun. Thanks for the update! I Just pushed the commit through on my side since your PR also includes other PRs within it.


Add pending changes window

the difference to "Show Changes" is that it's a window that you can dock with other VS windows, and you can click on a file and it will open it in VS.
you could also revert, commit selected files from there, but what i (and many of my colleagues) miss the most is just being able to easily open the files that i changed without searching for them.
I would expect it to be in a TSVN->Windows menu.

Adding a Tool Window
HOWTO: Create a toolwindow with a ToolWindowPane class in a Visual Studio package
Requested by: Consuela Stancu Mara

Added with v2.6

This plugin depends on which version of .net

This plugins uses .net 4.6.1

thanks


Missing-hints for Multi images in an embedded single tweet.

https://dev.twitter.com/web/embedded-tweets

It has an overlay covers these images.


can't find variable  TopBarTextExample

Expected behaviour
Actual behaviour
I've been trying to get the example of the repo working and I am getting this message can't find variable  TopBarTextExample
Code sample, screenshots
What have you tried
The code I have is similar as the example here
/* @flow */

import React, { PureComponent } from 'react';
import { StyleSheet } from 'react-native';
import { TabViewAnimated, TabBar } from 'react-native-tab-view';
import SimplePage from './SimplePage';

import type { NavigationState } from 'react-native-tab-view/types';

type Route = {
  key: string,
  title: string,
};

type State = NavigationState<Route>;

export default class TopBarTextExample extends PureComponent<void, *, State> {
  static title = 'Scrollable top bar';
  static appbarElevation = 0;
  console.log(title)

  state: State = {
    index: 1,
    routes: [
      { key: '1', title: 'First' },
      { key: '2', title: 'Second' },
      { key: '3', title: 'Third' },
      { key: '4', title: 'Fourth' },
    ],
  };

  _handleChangeTab = index => {
    this.setState({
      index,
    });
  };

  _renderHeader = props => {
    return (
      <TabBar
        {...props}
        scrollEnabled
        indicatorStyle={styles.indicator}
        style={styles.tabbar}
        tabStyle={styles.tab}
        labelStyle={styles.label}
      />
    );
  };

  _renderScene = ({ route }) => {
    switch (route.key) {
      case '1':
        return (
          <SimplePage
            state={this.state}
            style={{ backgroundColor: '#ff4081' }}
          />
        );
      case '2':
        return (
          <SimplePage
            state={this.state}
            style={{ backgroundColor: '#673ab7' }}
          />
        );
      case '3':
        return (
          <SimplePage
            state={this.state}
            style={{ backgroundColor: '#4caf50' }}
          />
        );
      case '4':
        return (
          <SimplePage
            state={this.state}
            style={{ backgroundColor: '#2196f3' }}
          />
        );
      default:
        return null;
    }
  };

  render() {
    return (
      <TabViewAnimated
        style={[styles.container, this.props.style]}
        navigationState={this.state}
        renderScene={this._renderScene}
        renderHeader={this._renderHeader}
        onRequestChangeTab={this._handleChangeTab}
      />
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
  },
  tabbar: {
    backgroundColor: '#222',
  },
  tab: {
    width: 120,
  },
  indicator: {
    backgroundColor: '#ffeb3b',
  },
  label: {
    color: '#fff',
    fontWeight: '400',
  },
});

How are you running the example?

I have set up a project locally and yes I have included the code from the demo just to test but I am getting this undefined variable
M

On Jul 18, 2017, at 5:26 PM, Satyajit Sahoo<EMAIL_ADDRESS>wrote:
How are you running the example?

Hello Satyajit, sorry it’s actually my mistake. I wasn’t including the right component name.
M

On Jul 18, 2017, at 5:26 PM, Satyajit Sahoo<EMAIL_ADDRESS>wrote:
How are you running the example?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/react-native-community/react-native-tab-view/issues/305#issuecomment-316202920, or mute the thread https://github.com/notifications/unsubscribe-auth/AEJ3ooI0tBQ1KpAY0CQJmDoOaXb3s9wjks5sPSMKgaJpZM4Ob581.


rplidar_ros: 2.0.0-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository rplidar_ros to 2.0.0-1:

upstream repository: https://github.com/allenh1/rplidar_ros
release repository: https://github.com/allenh1/rplidar_ros-release.git
distro file: foxy/distribution.yaml
bloom version: 0.9.7
previous version for package: null

rplidar_ros
* Update SDK to Version 0.12.0 (#14 <https://github.com/allenh1/rplidar_ros/issues/14>)
  * Register the rclcpp component
  * Update RPLIDAR SDK to version 1.12.0
* Update ROS 2 parameters and use node's clock instance (#9 <https://github.com/allenh1/rplidar_ros/issues/9>)
  * Update ROS 2 parameters and use node's clock instance
  * Fix scan_mode listing output
  * Stop motors and exit when set_scan_mode() call fails
* Fix compilation with eloquent (#6 <https://github.com/allenh1/rplidar_ros/issues/6>)
* Use Composition node with launch files (#4 <https://github.com/allenh1/rplidar_ros/issues/4>)
* Composable nodes (#3 <https://github.com/allenh1/rplidar_ros/issues/3>)
  * Begin implementation of composable rplidar_ros::rplidar_node
  * Declare composition node library in CMake, as well as continue the port
  * Get to a compiling state
  * Add start/stop motor callbacks + more driver setup
  * Add publish loop for scans
  * Add composition node
  * Lint
* Port rviz and launch files to ROS2 (#2 <https://github.com/allenh1/rplidar_ros/issues/2>)
  * Port non-rviz launch files to ROS2
  * Compatibility with rviz2
  * revert whitespace changes
  * Port the remaining launch files to ROS2
  * Revert more whitespace changes
  * Fix luanch and rviz install path indent level
* Ros2 port (#1 <https://github.com/allenh1/rplidar_ros/issues/1>)
  ROS 2 port
  * Port CMakeLists.txt
  * Port package.xml
  * Port client.cpp
  * Port node.cpp
  Fix compilation
* Support TCP
* upgrade sdk 1.10.0
* upgrade sdk 1.9.0
  [new feature] support baudrate 57600 and 1382400, support HQ scan response
  [bugfix] TCP channel doesn't work
  [improvement] Print warning messages when deprecated APIs are called; imporve angular accuracy for ultra capsuled scan points
* [bugfix]modify scan_mode at test_rplidar.launch and test_rplidar_a3.launch
* Contributors: Dan Rose, Hunter L. Allen, WubinXia, kint

Note, this is a release of a fork of the original repository: https://github.com/Slamtec/rplidar_ros
Considering that it's been ~2 years trying to get this port upstream with not much response (https://github.com/Slamtec/rplidar_ros/issues/4), we can track releases from the fork. We can switch to an upstream release eventually, if a port/release is made.
cc/ @WubinXia @kintzhao @tony-slamtec

@jacobperron sounds perfectly reasonable to me. :)


example best practice

PR Type
Documentation

Description

Improved formatting and readability in the GitHub installation guide.
Reorganized and clarified the list of additional tools and features in the PR-Agent Pro overview.
Simplified the description of best practices and added a link to an example file.
Added a new file with recommended Python best practices, covering various topics such as imports, packages, iterators, lambda functions, default argument values, true/false evaluations, lexical scoping, and threading.

Changes walkthrough 📝
Relevant filesDocumentation

github.mdMinor formatting improvements in GitHub installation guide

docs/docs/installation/github.md
Removed unnecessary blank lines.  Reformatted instructions for pinning action to a specific release.

+4/-1

pr_agent_pro.mdReorganized and clarified PR-Agent Pro overview

docs/docs/overview/pr_agent_pro.md
Added blank lines for better readability.  Reorganized and clarified the list of additional tools and features.

+23/-16

improve.mdSimplified best practices description and added example link

docs/docs/tools/improve.md
Simplified the description of best practices.  Added a link to an example best practices file.

+9/-13

EXAMPLE_BEST_PRACTICE.mdAdded example best practices for Python

docs/docs/usage-guide/EXAMPLE_BEST_PRACTICE.md
Added a new file with recommended Python best practices.  Included sections on imports, packages, iterators, lambda functions, default argument values, true/false evaluations, lexical scoping, and threading.

+188/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/review auto_approve


Using @Retrofit.Body with a method marked w/ @PUT doesn't emit body JSON

Also tried using @Retrofit.PUT instead of @PUT but it makes no difference.
Swapping @PUT w/ @POST makes the body JSON get written.
Method declaration looks like this:
    @Retrofit.PUT("/api/site/{siteId}/campaign/{campaignId}")
    public abstract BaseResponse updateCampaign(@Header(Endpoints.HTTPHEADER_TOKEN) String authToken,
            @Retrofit.Path("siteId") String siteId,
            @Retrofit.Path("campaignId") long campaignId,
            @Retrofit.Body Campaign campaign);

@PUT is used for updating an object usually.  @POST is for creating new objects.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28133067-using-retrofit-body-with-a-method-marked-w-put-doesn-t-emit-body-json?utm_campaign=plugin&utm_content=tracker%2F8327951&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F8327951&utm_medium=issues&utm_source=github).

Sorry, @PUT is not supportted with @Body, instead of use @Part.
@PUT("/user/photo")
User updateUser(@Part("photo") RequestBody photo, @Part("description") RequestBody description);

@POST with @Body:
@POST("/users/new")
User createUser(@Body User user);

@Part doesn't emit stuff in JSON format...looks like this instead:
PUT /api/site/4/campaign/41 HTTP/1.1
SDK: 1.0
X-API-TOKEN: xJK8a75PKM6mcVNu5b8AvDQQs
Content-Type: multipart/form-data; boundary=163ecb4b-c498-4a92-baee-9ab9cdcea7e5
Content-Length: 283
Host: admin.raizlabs-sandbox.com
Connection: Keep-Alive
Accept-Encoding: gzip
User-Agent: okhttp/2.5.0

--163ecb4b-c498-4a92-baee-9ab9cdcea7e5
Content-Disposition: form-data; name="Status"
Content-Length: 14

active-running
--163ecb4b-c498-4a92-baee-9ab9cdcea7e5
Content-Disposition: form-data; name="Name"
Content-Length: 8

updated1
--163ecb4b-c498-4a92-baee-9ab9cdcea7e5--

While the server expects a JSON encoded body like you send up w/ a POST.
I tried leaving @Multipart off the method and it still use the MIME multipart encoding.

Sorry, NotRetrofit is not supported PUT JSON body, supported PUT multipart only.

Had a quick look at NotRetrofit's source code...looks like RetrofitProcessor.java::buildBody() only executes if it's a POST but making it also accept a PUT should work too unless you mix in @Part parameters which wouldn't make sense.
Is there another technical reason it doesn't work?
Sorry for the questions...just trying to understand why because I think it's valid REST.

Nothing. Because I'm not sure whether it works fine, just disable it temporary. Maybe you could test it.

Happy to...any beginner's guide on how to fork/test the library?
Simplest thing is probably to enable it temporarily in the next official release build :-)

You could test it by the following build.gradle to use latest package of commit via jitpack:
repositories {
    jcenter()
    maven { url "https://jitpack.io" }
}

dependencies {
    compile 'com.github.yongjhih.NotRetrofit:retrofit-android:eb262c5264'
    apt 'com.github.yongjhih.NotRetrofit:retrofit-processor:eb262c5264'
}

It's generating correct code for translating @Body now, but it also adds another line that blanks out the body:
requestBuilder.put(RequestBody.create(MediaType.parse(typedOutput.mimeType()), bodyString));
requestBuilder.put(RequestBody.create(MediaType.parse(_MIME_APP_JSON_CHARSET_UTF8), ""));
It's adding the last line in the .vm around line 368.  I think you need a boolean flag that tracks whether you emitted a body already and if not, it adds that second line.

Fix duplicated Body:
    compile 'com.github.yongjhih.NotRetrofit:retrofit:1cc4afe5e1'
    apt 'com.github.yongjhih.NotRetrofit:retrofit-processor:1cc4afe5e1'

Thanks.  I even tried this which uses only the @Body as I expected it to do :-)
@PUT("/api/site/{siteId}/campaign/{campaignId}")
    public abstract Campaign updateCampaign(@Header(Endpoints.HTTPHEADER_TOKEN) String authToken,
            @Retrofit.Path("siteId") String siteId,
            @Retrofit.Path("campaignId") long campaignId,
            @Retrofit.Body Campaign campaign, @Retrofit.Part("Part") String part, @Retrofit.Field("field") String field

This is as expected because the Body should override everything else :-)
It's good for release..thanks for the quick fix.  :-)

You're welcome.
2015年11月11日 上午5:32 於 "Ken Yee"<EMAIL_ADDRESS>寫道：

Thanks. I even tried this which uses only the @Body
https://github.com/Body as I expected it to do :-)
@PUT("/api/site/{siteId}/campaign/{campaignId}")
public abstract Campaign updateCampaign(@Header(Endpoints.HTTPHEADER_TOKEN) String authToken,
@Retrofit.Path("siteId") String siteId,
@Retrofit.Path("campaignId") long campaignId,
@Retrofit.Body Campaign campaign, @Retrofit.Part("Part") String part, @Retrofit.Field("field") String field
This is as expected because the Body should override everything else :-)
It's good for release..thanks for the quick fix. :-)
—
Reply to this email directly or view it on GitHub
https://github.com/yongjhih/NotRetrofit/issues/5#issuecomment-155573563.


Introducing iframe elements on v2

Since I'm planning to switch to v2 soon, I'm preparing a major update of our apps and I tried v2.0.0-beta2: suddenly a wild iframe appeared in the DOM (causing issues with AdBlock, BTW).
Is v2 really introducing an iframe element when drawing graphs?

@heruan v2 is adding an iframe. The reason an iframe is added is to detect when the canvas container resizes. There is a window resize event, which was used by v1, but it only detects when the entire window resized and not a specific element. This led to the problems of charts not displaying correctly when their container was initially display: none; and then displayed (See #762 ).
I investigated a number of solutions for implementing better resize behaviour. Almost all of the available solutions use some combination of a timer / requestAnimationFrame. Unfortunately, this is bad for mobile battery life since we have to poll the size of the container continuously and check for a change.
The iframe solution works because it is set to be the same size as the canvas container node. Since the iframe is a window, the window resize event can be listened to and then passed up out of the iframe to the chart to tell it when the container node resized.
There is some other discussion on this topic in #2024

Thank you @etimberg for the clear explanation, it doesn't sound that bad for this purpose but unfortunately I'm having resize issues when using v2 in Single Page Applications based on Angular 2 and Aurelia. May it be because they build the DOM dynamically, injecting/removing elements while the user navigates the app? I'll build up a sample project and open separate issues!

@herun if you could build up a sample project, that'd be great! I'm wondering if we can do the dom addition in a callback. The users could provide an overridden implementation that works with their framework (angular, react, etc)

Issue reference: https://github.com/nnnick/Chart.js/issues/2212

Just to add my $0.02 on this: The hidden iframe causes some issues for me when integrating Chart.js v2 into my Echo 3-based client-side application. Specifically, the insertion of the hidden iframe just before my canvas node prevents the canvas element from rendering properly (its height and width are left unset). My workaround is to disable resize listening:
Chart.defaults.global.responsive = false;

Continuing with what @sknick did I took it one step further and decided to use the global flag as the kill switch for the iframe. I read the comment in the src about binding so the responsiveness could be changed dynamically but in my case I cared less about that and more about the performance gain* I would receive by not having the iframe.
var origResizeListener = Chart.helpers.addResizeListener;
Chart.helpers.addResizeListener = function(node, callback){

    if(Chart.defaults.global.responsive){
        origResizeListener(node,callback);
    }

};
Chart.defaults.global.responsive = false;

*Under "normal usage" I never saw a performance problem but I have an edge case scenario. We have 100+ little doughnut charts on the page and if someone were to change the filter on the data source it was taking much longer to reload than I was expecting. I knew it wouldn't be pretty I was surprised at the actual slowness. I noticed the favicon constantly flickering and started digging in. That's when I saw the iframe and tried to turn it off which lead me here.

The hidden iframe introduces an accessibility issue as it raises the warning AX_FOCUS_01 "These elements are focusable but either invisible or obscured by another element".  I think the warning may be mitigated by adding aria-hidden="true"to the iframe element.

@metatribal where can I see that accessibility warning?

I'm using the Google Accessibility Tool, here:
https://chrome.google.com/webstore/detail/accessibility-developer-t/fpkknkljclfencbdbgkenhalefipecmb?hl=en
On Sep 4, 2016 11:00 AM, "Simon Brunel"<EMAIL_ADDRESS>wrote:

@metatribal https://github.com/metatribal where can I see that
accessibility warning?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chartjs/Chart.js/issues/2210#issuecomment-244619067,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFAjfMlnAMBIaD6rxfLx7u1BLxzoY4aDks5qmwdBgaJpZM4H-k2J
.

Thanks :)

I can't reproduce the accessibility issue, maybe it needs some specific context:

I'm traveling right now; but will look at my laptop next time i get a
chance.  I ran across this while putting together an accessibility report.
There may have been specific context or tooling involved.
On Sep 4, 2016 11:42 AM, "Simon Brunel"<EMAIL_ADDRESS>wrote:

I can't reproduce the accessibility issue, maybe it needs some specific
context:
[image: image]
https://cloud.githubusercontent.com/assets/3874900/18233169/013d46e0-72e0-11e6-96f5-7f409afb57c9.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chartjs/Chart.js/issues/2210#issuecomment-244621313,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFAjfOhMSogtM-r6m-N2eSfe2blmrq3Uks5qmxEVgaJpZM4H-k2J
.

I'm reopening this since the use of an iframe to get responsiveness is semantically wrong and it's resulting in more than one side effect. I understand @etimberg reasons, but in my opinion a library well-written like chart.js should respect standards and accessibility and have tricks to enable some extra functionality available as an opt-in (e.g. enabled by an option).
By default, there should be no wild iframes nor listeners since the developer have many ways to keep track of the canvas size and decide what to do and when.

Agree with you @heruan, but the default behavior can't be changed because it would break too many projects (at least for version 2.x).
However, we are working on 2 main changes about chart responsiveness: at very short term (certainly v2.3.1), the iframe will not be injected anymore if responsive: false (not the default config, but will solve your issue). We are also investigating a method to replace the troublesome iframe by a group of div to detected size changes (probably v2.4), so most of issues related to performance, accessibility, standards, ad blockers, etc., because of the iframe, should disappear.

Fixed by #3364 (no wild iframe if responsive: false - not the default) and available in v2.4.0

Might need to reopen this.  I have a chart in a scrollable div, and when the iframe is included in the DOM the canvas scrolls but nothing else does. When I manually remove the iframe from the DOM everything scrolls with the div correctly. I have a fixed height on a parent element, with 100% width. Unfortunately, setting the responsive: false option prevents the width from adapting to 100% of the view so I can't just set that to false.  This was not a problem with version 1.x.
Before scrolling with iframe:

After scrolling with iframe (bad):

Scrolling without iframe (good):

@johnmanko can you provide a live example of this issue?

After additional testing, it looks like this may be a problem with a specific version of Chrome.
Version 53.0.2785.143 Built on Ubuntu , running on Ubuntu 16.04 (64-bit)
The following version of Chrome everything works as it should:
Version 54.0.2840.99 m
It works on Firefox.


Configuring cache from Faraday adapter

I thought it would be possible to pass the cache parameter to Typheous adapter in Faraday, but it doesn't seem so.
Faraday.new do |c|
  c.adapter :typhoeus, cache: cache
end

I get this error instead.
Ethon::Errors::InvalidOption: The option: cache is invalid.

Would it be possible to add that functionality? I hardly ever use Typhoeus directly, and it's hugely important to be able to cache the requests on an individual basis.

Actually, this seems to be a bug. I cannot configure it directly from Typhoeus either:
[3] pry(main)> Typhoeus.get('example.com', cache: Cache.new)
Ethon::Errors::InvalidOption: The option: cache is invalid.

I am also getting this behavior. My workaround for now is some dirty code in my request class:
def with_cache(cache)
  original_cache = Typhoeus::Config.cache
  Typhoeus::Config.cache = cache

  yield.tap { Typhoeus::Config.cache = original_cache }
end

Thats a bug in Typhoeus.

This happening :(


CFY-6966 Trigger deployment env recreation

Requires:
https://github.com/cloudify-cosmo/cloudify-manager/pull/889
https://github.com/cloudify-cosmo/cloudify-rest-client/pull/179
This approach was taken because:

Trying to fake the context into acting as a different tenant is now considerably harder and every attempt simply resulted in deployment env recreation being run as the user who was running the snapshot.
Trying to make this workflow run as a sub-workflow of the restore workflow failed because system wide workflows cannot be run when there are other workflows running (this error was given even when both workflows were specified not to be system wide).

It's likely that the best solution of this would be to make the create+restore snapshot work only on the current tenant, and have the CLI do the heavy lifting for taking snapshots for multiple tenants. This could have other benefits aside from making reasoning about and implementing security measures simpler.

Not needed, approach changed.


fix: further ensure Vue doesn't swallow errors

@phonzammi I've another little minor refactoring idea; if it's ok with you I'll create a seperate PR for it after this one is merged. (@AurelienLourot the converstion about ReturnType<typeof renderToNodeStream> is unrelated to this PR and you can ignore it.)

@brillout , yes please


Change default listen directives

Bacula defaults to listen on any IPv4 using standard bacula ports, these changes add support to listen on specific IPv4 or IPv6 addresses and custom port numbers.
This is necessary in IPv6-only environments or when IPv6 is simply preferred over IPv4.

This has been unanswered for a very long time. closing.


IndexError: list index out of range on Fedora 38

What went wrong?
playing songs using the play cmd produces a IndexError: list index out of range on Fedora 38 amd64 using a venv
Download started: https://youtu.be/L5uV3gmOH9g
Download complete: https://youtu.be/L5uV3gmOH9g
WARNING: Could not grab data from i18n key on_player_play-onChannel_playingMention.
Task exception was never retrieved
future: <Task finished name='Task-61' coro=<MusicBot.on_player_play() done, defined at /home/ozzy/Desktop/MusicBot/musicbot/bot.py:560> exception=IndexError('list index out of range')>
Traceback (most recent call last):
  File "/home/ozzy/Desktop/MusicBot/musicbot/bot.py", line 646, in on_player_play
    videoID = url.split("watch?v=")[1].split("&")[0]
              ~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

How do we reproduce this?
be running Fedora 38 on amd64
be running a venv virtual envirerment
python run.py
also this is happening with the dev branch
Which version of the bot are you using?
None
Which operating system are you using?
Other
Just checking...

[X] I have confirmed that my issue has not already been reported.
[X] I am using Python 3.8 or higher.
[X] I have updated my Python dependencies.

This would happen on the review branch as well, since it's caused by my implementation of embedding the now playing messages.

I've done both

I just tried doing it with links doesn't work but doing names does

Huh, weird behavior I'm not really sure what the underlying issue is. Basically all this code does is rewrite the URL to be fetchable to create an embed. Is there any other errors?

the problem is with using urls in the play command but the weird thing is I haven't noticed this issue on other OSs I've only been having this issue on Fedora 38 I tried it on Arch Linux and Windows and it worked fine

I can't remember off the top of my head by if but I think when trying to do the np cmd if a link was used to play the np cmd breaks in the same way

I was testing this last night so my memory of the np cmd issue is not great

I'm at work rn, but I'll look into this tonight when I get home. It shouldn't have any issues with a link at all, but if it is that's very odd. Thank you for telling me

np

again I see this is issue on Fedora 38 but not other OSs tho I've only tested Windows and Arch Linux

I just pushed a change to address this, please lmk if it solves it.

yes that change fixed it thanks


How to use the colored version of icons?

I take advantage of the last activities on this repo to ask for this question : how to use the colored version of icons ?
Is this now possible ?
This prevents me to use PlantUML as it full potential ...
Originally posted by @ChrisProlls in https://github.com/plantuml-stdlib/Azure-PlantUML/issues/20#issuecomment-1006379981

Hi Just an upvote here as i was planning to also use colored icons.
However i'm not able to figure out how to exactly use them in plantuml.
I'm breaking my head on it as it is unclear.
If it is possible to use the Color versions instead of the Mono than can we add examples to the repository readme ?
Thanks for creating the issue.

I don't believe this is currently possible. According to the PlantUML documentation, sprites are monochrome. When invoking PlantUML to create the sprite, it'll be converted to greyscale automatically unless another color is specified. The end result will always be monochrome. This can be confirmed by creating a sprite using a full color icon:
java -Djava.awt.headless=true -jar scripts/plantuml.jar -encodesprite 16z dist/Compute/AzureDisk.png
The resulting sprite is greyscale:
@startuml color-demo

sprite $AzureDisk [70x70/16z] {
xTO73jmm30LH5CEVf07e_hUDe1HbLlOIOdgP0pmNrVQ__vqljRtKEszJh-MP928har8IbHN96RCKcTc3iON8xCOvh9SWOEo6iHo18HiTgw9C8J8YygIePvBf
7d5FNqKUronyLTmzG_nwBOYi2f5HN4ihuZ4LlniV-SN2gBXJYOaWcKV45JqtYSYgnCQi9GB5CsG6X4fu4Q4IleNOGAXu34SPCfLXGEb5qCb_RG21qXtB4VDX
t2iJaeZCI0o43FDX0Pa_0mgPIcsjWdmxu-sXab67slji30k7aqEdxE0_C4s-h1KVcGrWH8YTczKDLaNOKKPO0Iagwjb8Kiey6CPKKMpjlQAdLyaLuxnV4ntf
J9JivWC1oDSAc8r4nQe85LC5eJAg65Is7HaGCDAMiIg0KW8AcPmI121WO253GC1irzRg1giZh4YiLOcrkf1RxTVqlmy
}

Alice -> Bob : Testing <$AzureDisk>

@enduml

Although the sprites themselves are monochrome, there are various ways in which color can be used:

Using the color tag: <color:orange><$sprite>
Using the sprite's color attribute: <$sprite,color=orange>
Setting the background color of the container a sprite is in: rectangle "<$sprite>" #orange

Sadly, using a gradient on the container does not work well with the sprite's background (see example below).

PlantUML Diagram Source
@startuml
!$AzurePuml="https://raw.githubusercontent.com/plantuml-stdlib/Azure-PlantUML/release/2-1/dist"

!include $AzurePuml/AzureCommon.puml
!include $AzurePuml/Storage/AzureDiskStorage.puml

rectangle "default Sprite" as a {
    rectangle "<$AzureDiskStorage>" as a1
}

rectangle "<color:...>" as b {
    rectangle "<color:orange><$AzureDiskStorage>" as b1
    rectangle "<color:blue><$AzureDiskStorage>" as b2
}

rectangle "<$sprite,color=...>" as c {
    rectangle "<$AzureDiskStorage,color=orange>" as c1
    rectangle "<$AzureDiskStorage,color=blue>" as c2
}

rectangle "Container Color" as d {
    rectangle "<$AzureDiskStorage>" as d1 #orange
    rectangle "<$AzureDiskStorage>" as d2 #blue
}

rectangle "Sprite Color and Container Color" as e {
    rectangle "<$AzureDiskStorage,color=orange>" as e1 #blue
    rectangle "<$AzureDiskStorage,color=blue>" as e2 #orange
}

rectangle "Container Gradient" as f {
    rectangle "<$AzureDiskStorage,color=blue>" as f1 #orange|blue
    rectangle "<$AzureDiskStorage,color=blue>" as f2 #orange|blue
    rectangle "<$AzureDiskStorage,color=blue>" as f3 #orange|blue
    rectangle "<$AzureDiskStorage,color=blue>" as f4 #orange|blue
}

a -[hidden]-> b
b -[hidden]> c
b -[hidden]-> d
d -[hidden]> e
d -[hidden]-> f
@enduml


KeyError when no connections are available using HashClient

When the HashClient runs out connections it will start raising KeyError (even with ignore_exc=True). Reproduce with this:
>>> from pymemcache.client.hash import HashClient
>>> client = HashClient([('localhost', 11211)], use_pooling=True, ignore_exc=True,
...                     timeout=2, connect_timeout=2)
>>> client.set('foo', 'bar')
True
>>> client.get('foo')
'bar'
>>> # Stop Memcached server
...
>>> client.get('foo')
>>> client.get('foo')
>>> client.get('foo')
>>> client.get('foo')
>>> client.get('foo')
>>> client.get('foo')
>>> client.get('foo')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/Peter/.vim/venv/lib/python2.7/site-packages/pymemcache/client/hash.py", line 226, in get
    return self._run_cmd('get', key, None, *args, **kwargs)
  File "/Users/Peter/.vim/venv/lib/python2.7/site-packages/pymemcache/client/hash.py", line 214, in _run_cmd
    client = self._get_client(key)
  File "/Users/Peter/.vim/venv/lib/python2.7/site-packages/pymemcache/client/hash.py", line 134, in _get_client
    client = self.clients[server]
KeyError: None

@sontek
I took a look at the code. This is caused by the _get_client method in HashClient returning None when there are no live nodes. Fixing that is going to require a little bit of code restructuring, since there isn't a good way to return a correct value from the place the error occurs. I'll have time for this next week, adding John Anderson in case he has time sooner.

Thanks for poking me, I didn't notice this error report. I'll fix it up tonight

@cgordon @renstrom Sorry I got busy the other night but #81 will make it graceful when ignore exceptions are set.

Will close this when it is deployed to pypi (this weekend).


Change preview showing the name instead of the preferredUsername

I think the preview should show the name instead of the preferredUsername next to the users icon.
I also added myself to the contributors list this time.

@Menrath can you please add your change to the readme.txt and changelog.md!?!

Done! Thought I might skip it, because the feature that this PR targets is not released yet.


fixing missing packages and version conflicts

Installing dependencies from requirements.txt was broken for me.
Steps to reproduce
virtualenv ve
source ve/bin/activate
pip install -r requirements.txt

Here are the results -
Processing /private/var/folders/sy/f16zz6x50xz3113nwtb9bvq00000gp/T/abs_477u68wvzm/croot/certifi_1671487773341/work/certifi
ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/private/var/folders/sy/f16zz6x50xz3113nwtb9bvq00000gp/T/abs_477u68wvzm/croot/certifi_1671487773341/work/certifi'

....

ERROR: Cannot install -r requirements.txt (line 23) and starlette==0.25.0 because these package versions have conflicting dependencies.
The conflict is caused by:
    The user requested starlette==0.25.0
    fastapi 0.89.1 depends on starlette==0.22.0

Feel free to close the PR if the specified package versions were intentional.
Also added a test requirements file to automate adding test-only dependencies.

@diptanu the requirements.txt file is pretty outdated for the package, and it lists more dependencies than are actually needed for the repo. I've been installing via setuptools. One fix for the requirements file could be to just add
 #
 # installs dependencies from ./setup.py, and the package itself,
 # in editable mode
 -e .

instead of the list of packages. Lmk what you think

Closing this PR since I didn't see the requirements specified in the setup.py. Might want to consolidate where requirements are specified to avoid any confusion.


Text encoder training support FLux

i saw that on here https://github.com/huggingface/diffusers/blob/main/examples/dreambooth/README_flux.md
you can train the text encoder.
Is it possible to add it to sd-scripts

+1 On this. When will this be reflected in Kohya?


Write initial Multisend.sol educational challenge & tests

Description
Create Multisend challenge as per issue #3.
TODO

[x] Create initial Multisend.sol
[x] Create initial Multisend.t.sol tests (pseudo-code)
[x] Debug tests and contract to ensure challenge works
[x] Add Event tests and emissions
[x] Write README sections

Additional Information
Some assumptions were made when starting this PR:

It's okay that address array params have repeating addresses (perhaps the use case was someone pasting in addresses into the params and they didn't bother to combine repeat addresses with corresponding amounts in the amounts array)
sendTokens() to only carry out the transference of one ERC20 token at a time, aka not to do a variety of different ERC20 transfers within the single function call.

Related Issues
Closes #3
Your ENS/address: steve0xp.eth

Current status:

Bug occurring on the transference of tokens and ETH. So there probably is something wrong with the actual implementation in Multisend.sol. Sadly don't have time to debug today, but can get to it in the coming days.

@escottalexander @swellander thanks for your PR reviews. I've made the additional changes you've outlined. :)


[multistage] [bugfix] Temp fix for grpc receiving buffer full issue

Make grpc receiving buffer unbounded so we don't drop data block.
Add a debug logging so we can know the maximum buffer size we have reached.

lgtm overall. this should be a good first step. could we add a simple test?

we are not able to test this because

integration test doesn't allow us to specify # of blocks
grpc mailbox test is not working since we have thread leak which I will fix later and test all of the cases
this PR is urgent because we are not able to return correct result when data load is large

Codecov Report

Merging #10099 (4ec719f) into master (04d09e5) will increase coverage by 40.79%.
The diff coverage is 93.75%.

@@              Coverage Diff              @@
##             master   #10099       +/-   ##
=============================================
+ Coverage     27.87%   68.67%   +40.79%
- Complexity       53     5736     +5683
=============================================
  Files          1979     1992       +13
  Lines        107281   107993      +712
  Branches      16323    16415       +92
=============================================
+ Hits          29909    74162    +44253
+ Misses        74417    28569    -45848
- Partials       2955     5262     +2307

Flag
Coverage Δ

integration1
24.72% <0.00%> (+0.08%)
:arrow_up:

integration2
?

unittests1
68.00% <93.75%> (?)

unittests2
13.69% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../mailbox/channel/MailboxContentStreamObserver.java
67.14% <93.75%> (+67.14%)
:arrow_up:

...pinot/core/data/manager/realtime/TimerService.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/core/plan/StreamingInstanceResponsePlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/operator/streaming/StreamingResponseUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ager/realtime/PeerSchemeSplitSegmentCommitter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/utils/grpc/GrpcRequestBuilder.java
0.00% <0.00%> (-90.91%)
:arrow_down:

...ator/streaming/StreamingSelectionOnlyOperator.java
0.00% <0.00%> (-90.00%)
:arrow_down:

...he/pinot/core/plan/StreamingSelectionPlanNode.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...lugin/stream/kafka20/KafkaStreamLevelConsumer.java
0.00% <0.00%> (-88.34%)
:arrow_down:

...che/pinot/common/auth/StaticTokenAuthProvider.java
0.00% <0.00%> (-87.50%)
:arrow_down:

... and 1466 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

thanks for the explanation @61yao please follow up with the query option as @ankitsultana mentioned this will cause potentially worse memory leak issues


Authentication-Results header

Hi there!
I have an issue similar to #61.
I don't know why in 'Authentication-Results' header DMARC=fail. I've already checked /etc/opendmarc/IgnoreDomains. My domain is listed there.
Can you suggest anything to fix?

I think it's a bug in OpenDMARC, which despite "IgnoreMailFrom" and "TrustedAuthservIDs" tries to apply DMARC policy to outgoing e-mails. But this behavior has no effect on the legitimacy of outgoing mails.
I hope this bug will be fixed in the next version of OpenDMARC with Debian Stretch.

Fixed in version 1.1.
Now using the Rspamd DMARC module.


The Subatomic Config Server Build Config fails due to permissions on the ONBUILD directive

As part of implementing https://github.com/absa-subatomic/laboratory/issues/14, the ONBUILD directive is used to support adding additional root CA certificates.
However, this change of user in the ONBUILD is not allowed in a OpenShift build:

What's strange is the Jenkins build does not error on this:

Even though that part of the Dockerfile is essentially identical.


Error: domains - http://deefauph.com,https://theeptoah.com, action - /event, error- Failed to fetch

I got this error message on my web page, how can I fix it?
Unhandled Runtime Error Error: domains - http://deefauph.com,https://theeptoah.com, action - /event, error- Failed to fetch

It does not seem like the issue you're seeing is related to this NPM package. May I suggest searching your codebase for the domains you listed and where they might be called from? Also try disabling all your browser plugins or using a different browser to see if the issues persist


Support blank lines, ignore existing ENV variables, handle missing newline

Added tests and implemented support for the following:

Ignore existing ENV variables like other dotenv implementations npm python ruby etc etc
Support blank lines in the .env file
Handle missing newline at the end, because it happens

Thanks man

np, take care


DS-2551: JSPUI show thumbnails only if user has read permission

#928 is a cherry-pick of this commit on dspace-5_x.


bingo get does not update {package}.mod go version

bingo version output:
v0.5.2
go version output:
go version go1.18 darwin/arm64
What happened:
I'm running with bingo get<EMAIL_ADDRESS>which looks like:
module _ // Auto generated by https://github.com/bwplotka/bingo. DO NOT EDIT

go 1.15

require github.com/plexsystems/promdoc v0.7.0

Then I did the update with: bingo get<EMAIL_ADDRESS>and I got:
module _ // Auto generated by https://github.com/bwplotka/bingo. DO NOT EDIT

go 1.15

require github.com/plexsystems/promdoc v0.8.0

What you expected to happen:
Getting:
module _ // Auto generated by https://github.com/bwplotka/bingo. DO NOT EDIT

go 1.18

require github.com/plexsystems/promdoc v0.8.0

How to reproduce it (as minimally and precisely as possible):
already described
Logs (use bingo get -v <thing you do> for verbose output):
Anything else we need to know:
I fixed this by removing {package}.mod (in this case promdoc.mod and then run the get on v.0.8.0

Hey, thanks for reporting.
I assume you mean it does not work because the go statement is still 1.15?
Yea, we don't change go statement version for compatibility. In practice this is never needed, unless you want to use potentially new commands in .mod file which bingo does not sue. Why you want that go version to be improved?

Well I just noticed it because I first checked the package on github and saw that they changed their mod file. I think it's "weird" or if anything, a little bit confusing to not exactly use the 'upstream' mod file. If you have a clean setup and use the bingo get we do get the exact version. So that does not make it consistent.
I personally don't have a strong opinion about how it should work.

Just to be strict, exeat version of go? Because that is simply version of Go module format nothing else (:


Not able to lauch istio-operator and ingressgateway with private image repo

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
We have launched an istio cluster via helm charts and flux. Images were being pulled from docker. Now want to change the hub from docker.io to Azure ACR so that image from private repo is being pulled.
In the istio control plane yaml, i have changed the hub to private one and given the image pull secret in values.global.imagepullsecret.
By giving this config, i am able to pull the images for istiod pods. But this config is not working for istio-operator pod and ingressgateway pods.
I tried to provide support in chart of gateway & operator for incorporating image pull secret but that is not working
Is the support available for pulling the image for these 2 depoyments?
Request you to please provide a way around on this.
Version
$kubectl version --short
Client Version: v1.23.0
$helm version --short
v3.8.0+gd141386
Istio chart version
1.15.3
1.11.1

Additional Information
Already added in description
Affected product area

[ ] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

Have you tried .Values.imagePullSecrets for those two?

Yes. It is not working. Also i am not able to see any support of imagepullsecret for operator and gateway deployment in template/deployment file.

Can you check if the service account is correctly configured with imagePullSecrets?

They are set in the service account rather than the deployment


Add existing user IDs to FreeBMD2 (mongodb) so that they are not taken by new transcribers

ready to be tested on test2

ready to be tested on test2

@DeniseColbert to take a look.

@DeniseColbert to take a look.

Tested and working:

Tested and working:

Code deployed to test3. So closing it


Feature request: Set working directory to source file

Description of your problem
In RStudio you have a quick option to set the working directory to source file which is very handy. Be great to have an option in the working directory dropdown: set to source file. Currently you have to open the file and then follow the exact same steps to set the working file directory which is double the work for a lazy person like myself :)
As an aside, in Ubuntu 16.04, the bookmarks are also not shown in the file open dialogue.

Spyder Version: 3.1.3
Python Version: 2.7.12
PyQt Version: 5.5.1+dfsg-3ubuntu4
Operating system: Ubuntu 16.04

Dependencies
jedi =0.9.0      :  0.9.0 (OK)
matplotlib >=1.0 :  2.0.0 (OK)
nbconvert >=4.0  :  5.1.1 (OK)
numpy >=1.7      :  1.12.1 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
pep8 >=0.6       :  1.7.0 (OK)
psutil >=0.3     :  5.2.1 (OK)
pyflakes >=0.5.0 :  1.5.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.6.5 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
rope >=0.9.4     :  0.10.5 (OK)
sphinx >=0.6.6   :  1.5.5 (OK)
sympy >=0.7.3    :  None (NOK)

Also saw this, don't see that in Spyder3 https://github.com/spyder-ide/spyder/issues/1844

Could you post a screenshot of RStudio's feature? Thanks.

hello


Simulation of a four lbar linkage mechanism

Good Day everybody, I've been watching the coding train for some days and since I'm studying mechanical engineering I've decided to try and make a simulation of a four bar linkage mechanism, however I've been facing some issues with the code, my main issue right now is that the link four according to the code never changes lenght (and it shouldn't) however the line drawn does, if someone could lend me a hand on this I'd be glad
I've comment the code both in english and spanish
Mecanismos_2.zip

Hey Vako!
This repository manages the Coding Train website, it isn't the right place for asking questions.
The best things you can do is join our discord.
P.S. Sorry for the late reply, we didn't use the issue section of this repository for a long while.
Hope to see you there!


"Revert to Live" interaction with Link plugin

Under some circumstances, Revert to Live interacts with the Link plugin in such a way that it loses the link text in the editor, and shows a missing image:

To replicate this:
Insert a Link plugin; the standard plugin icon will be replaced with the link text:

Save those changes and publish them.
Select History > Revert to Live.
Open the text plugin again for editing. The link text will be lost.
pip list output:
argparse (1.2.1)
dj-database-url (0.3.0)
Django (1.6.5)
django-classy-tags (0.5.1)
django-cms (3.0.3)
django-mptt (0.6.1)
django-reversion (1.8.2)
django-sekizai (0.7)
djangocms-admin-style (0.2.2)
djangocms-column (1.3)
djangocms-file (0.0.1)
djangocms-flash (0.0.2)
djangocms-googlemap (0.0.5)
djangocms-inherit (0.0.1)
djangocms-installer (0.5.3)
djangocms-link (1.3.5)
djangocms-picture (0.0.2)
djangocms-style (1.3)
djangocms-teaser (0.0.1)
djangocms-text-ckeditor (2.1.6)
djangocms-video (0.0.1)
html5lib (0.999)
Pillow (2.5.1)
pip (1.5.6)
pytz (2014.4)
setuptools (3.6)
six (1.7.3)
South (1.0)
wsgiref (0.1.2)

Ok, closing this


Add par_split(Fn) and par_split_whitespace for strings

Both par_split and par_split_terminator can now accept Fn(char) -> bool for the separator, in addition to a simple char.  This is somewhat modeled after the standard library Pattern, but that's not stable so we can't use it directly.  From this, par_split_whitespace follows easily.
This also adds a "private" method to ParallelString, as suggested by @bluss, so we can add methods without worrying about breaking third-party implementations.
Fixes #133.

This looks really nice 👍


[MAS2.4.3] Focus is not landing on the interactive data on the page after activating the 'Show Bing Sports' using Adaptive cards button.

Actual Result:
Focus is not landing on the interactive data on the page after activating the 'Show Bing Sports' using Adaptive cards button.
Expected Result:
Focus should land on the all interactive data which appear on the page after activating the 'Show Bing Sports' using Adaptive cards button.
Repro Steps:

Open URL https://microsoft.github.io/BotFramework-WebChat/full-bundlein Edge browser.
Notice 'Upload File' and 'Send' button icons present in bottom of screen.
Type 'help' in the input control at the bottom of the screen and hit enter (or click on the send button).
Content will upload, Navigate to any "Show Bing Sports using Adaptive cards button " button by using Tab key and press Enter on it.
New data will upload, navigate to the data by using Tab key.
Observe that focus is not landing on the all interactive data.

User Impact:
If keyboard focus is not not logical and not lands on the related content after perform the action then the user who is dependent on keyboard will get confuse.
MAS Reference:
MAS2.4.3 - Focus Order
Test Environment:
OS: Windows 10
OS Build: 17134.345
OS Version: 1803
Browser: Edge
Attachment:
MAS2.4.3_Focs order.zip

This is by-design.
The card has 3 hot spots to click on:

The whole container
The left column (the panther and the word "CAR")
The panther image

So the TAB is going thru correct tab order.

#HCL;#Accessibility;#HCL_BotFramework_WebChat;A11yMAS;WCAG2.4.3


Write Config Only Option

Add write_config function option to only write the config.  Sometimes syncing firewall is not necessary or desirable.  ex: changing log display options.

Do you have any situation where you need this?

It would be nice in some situations such as just making a temporary change to status log display (formatted/raw, forward/reverse, quantity) not to initiate a sync or restart any services, etc.


Add migration docs note that .control-label is gone in v4

Use .form-control-label instead?
Refs #17777.

Fixed by #17808.


Infinite loop on buy-crypto route

Overview
When loading the buy-crypto route we end up in an infinite loop, causing the icons to never load and tens of thousands of console logs.
References and additional details

Acceptance Criteria
Find and fix the cause of this loop.
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Started investigating:
It's not an infinite loop, but we reload market data for around 1500 assets when visiting the page, making things super slow


Update dev dependencies with recent style additions?

DEVELOPMENT.md was recently updated with dev dependencies for styling. It says to:
pip install black[jupyter] flake8 flake8-isort isort

Should we add this to the dev dependencies in setup.py?
https://github.com/asreview/asreview/blob/9b35dfbf42cc34e37dd4256f07d388daa2c80d3d/setup.py#L44-L52
(probably install black without the jupyter stuff )

Sounds good!

PR was merged, closing


Fix docker repository problem when travis builds foulkon

Codecov Report

Merging #101 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
- Coverage   93.31%   93.28%   -0.04%
==========================================
  Files          30       29       -1
  Lines        3411     3394      -17
==========================================
- Hits         3183     3166      -17
  Misses        171      171
  Partials       57       57


Stress Testing

As the system administrator
I want to check how many people can access the landing page and submit an application simultaneously
So that the team can work with the contingencies as needed.
Actions

[ ] Check with Platform Services what the procedure is to run a "Stress Test."
[ ] Rate limiting.
[ ] Technical solution: Implement a slowdown - prevents a crash of the application.

@MarsRomer Load testing should be done before stress testing


PS4 Controller (D-Pad Not working in inventory and R1 and R2 with JEI)

Previously I had Controllable and JEI configured great in Minecraft 16.5 (Forge). The D pad would allow me to navigate around the inventory and I could use R1 and R2 to switch pages for items in the JEI bar.
However, since installing Minecraft 18.2 (Forge) and updating Controllable and JEI to the correct versions the D-Pad does not navigate in the inventory and I cannot use R1 and R2 (Bindings) for JEI.
These are the versions:
Controllable: 0.16.0-1.18.2
JEI 18.2-<IP_ADDRESS>
I am unsure where the problem is from? I don't understand want I cannot use the control pad in the inventory to move items around.

I have the same issue. I am running 1.18.1 and we are running the Better MC 1.18.1 modpack. Same issue on server and single player. I can use D-PAD in the menu but not for crafting for some reason. Any help appreciated!

I am having the same issue with my xbox one controller, running the same versions of both Controllable and JEI.
Both are working fine together in my 1.19 world, but not in my 1.18.2 world.

Same issue with xbox controller, D-pad in inventory doesn't work with JEI

Due to how Controllable integrates with JEI, I have to depend on the base JEI mod, which is prone to code breaking changes. JEI simply made some internal changes which broke the code in this mod. Nothing I can do to protect against this but I'll have a patch out soon to address the issue.

Fixed in https://github.com/MrCrayfish/Controllable/commit/43ca4f8851c33444beef0f38aa37de1942aebe35


EVM target compiler for RISC-V

Project Abstract
This proposal is about to support RISC-V as a target of zink-lang/zink, as parity recently introduced revive, zink is about to solve the same problem for developers but in a different approach, the goal of this proposal is for supporting developers compiling zink contracts on both PolkaVM and EVM
Grant level

[ ] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $30,000, 3 approvals
[x] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] Payment details have been provided (Polkadot AssetHub (USDC & DOT) address in the application and bank details via email, if applicable).
[x] I understand that an agreed upon percentage of each milestone will be paid in vested DOT, to the Polkadot address listed in the application.
[x] I am aware that, in order to receive a grant, I (and the entity I represent) have to successfully complete a KYC/KYB check.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[x] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @clearloop:matrix.org

Left a comment in the private chat.

closed as discussed in private.


Feature/error reviver

Resolves #64 .
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines, including the relevant style guides.
[x] Read and understand the Code of Conduct.
[x] Read and understood the licensing terms.
[x] Searched for existing issues and pull requests before submitting this pull request.
[x] Filed an issue (or an issue already existed) prior to submitting this pull request.
[x] Rebased onto latest develop.
[x] Submitted against develop branch.

Description

What is the purpose of this pull request?

This pull request:

contains a straight-forward port of utils-error-reviver

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #
fixes #

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

@stdlib-js/reviewers

Thanks for the comment! Fixed the issues.


⚠ ClamAV has degraded performance

In d42478a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 55 ms

Resolved: ClamAV performance has improved in bfc87e3.


Hope can add the switch to cusntom API_KEY on frontend

It can turn out to be a public service rather than self use.

Duplicate to #31


Function worker can't start when setting up a geo-replicated cluster

Describe the bug
When setting up two geo-replicated clusters with functions-worker enabled, one cluster will fail to start functions worker because the cluster is not in the cluster list of public/functions.
To Reproduce

Setup cluster A
Setup cluster B. B shares the same configuration store with A.

Expected behavior
Cluster should start function worker as expected.

Closed as stale. Please open a new issue if it's still relevant to the maintained versions.


How to specify an overlay for nixpkgs

Hi!
I am using multiple overlays over the standard pkgs attribute to access unstable & master channels:
nixpkgs.overlays = [
  (final: prev: {
    unstable = builtins.import inputs.unstable {
      inherit (final) config;
      inherit system;
    };
  })
  (final: prev: {
    master = builtins.import inputs.master {
      inherit (final) config;
      inherit system;
    };
  })
]

As I am using the master branch of nixvim, it expects pkgs to be the unstable channel. Mine targets nixos-23.05, I would like to know if there was a way to pass to nixvim.nixosModules.nixvim a specific pkgs attribute. For example:
modules = [
  nixvim.nixosModules.nixvim
  {
    nixvim.pkgs = pkgs.unstable;
  }
];

Thanks for your time!

Oops I realized there was a discussion page, sorry for the noise...

https://github.com/nix-community/nixvim/discussions/529


Glfw Error 65550: Failed to detect any supported platform

[info] [fragment.cpp:588] Loading extensions from configs...
[info] [gxf_executor.cpp:262] Creating context
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'receivers:0'
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'render_buffer_input'
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'camera_up_input'
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'camera_look_at_input'
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'camera_eye_input'
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'input_specs'
[info] [gxf_executor.cpp:1764] creating input IOSpec named 'receivers'
[info] [gxf_executor.cpp:2171] Activating Graph...
[info] [gxf_executor.cpp:2201] Running Graph...
[info] [gxf_executor.cpp:2203] Waiting for completion...
[info] [greedy_scheduler.cpp:191] Scheduling 2 entities
Authorization required, but no authorization protocol specified
Glfw Error 65550: Failed to detect any supported platform
[error] [gxf_wrapper.cpp:59] Exception occurred when starting operator: 'visualizer' - Failed to initialize glfw
[warning] [entity_executor.cpp:539] Failed to start entity [visualizer]
[warning] [greedy_scheduler.cpp:243] Error while executing entity 12 named 'visualizer': GXF_FAILURE
[error] [entity_executor.cpp:630] Entity [visualizer] must be in Started, Tick Pending, Ticking or Idle stage before stopping. Current state is StartPending
[info] [greedy_scheduler.cpp:401] Scheduler finished.
[error] [program.cpp:580] wait failed. Deactivating...
[error] [runtime.cpp:1649] Graph wait failed with error: GXF_FAILURE
[warning] [gxf_executor.cpp:2204] GXF call GxfGraphWait(context) in line 2204 of file /workspace/holoscan-sdk/src/core/executors/gxf/gxf_executor.cpp failed with 'GXF_FAILURE' (1)
[info] [gxf_executor.cpp:2214] Graph execution finished.
[error] [gxf_executor.cpp:2222] Graph execution error: GXF_FAILURE
Traceback (most recent call last):
File "/opt/nvidia/holoscan/examples/v4l2_camera/python/v4l2_camera.py", line 95, in
main(config_file=config_file)
File "/opt/nvidia/holoscan/examples/v4l2_camera/python/v4l2_camera.py", line 89, in main
app.run()
RuntimeError: Failed to initialize glfw
[info] [gxf_executor.cpp:291] Destroying context

USING rtx 4090 for the work
but i am facing problems related to GXF and video replayer related program.
but Holoscan is running well

You'll likely need to enable X11 or Wayland forwarding.
 To leverage a display, add the flags below:
    X11 :
        run xhost +local:docker as a prerequisite so that X11 is configured to allow commands from docker
        add -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY to the docker run command
   Wayland: add -e ${XDG_RUNTIME_DIR} -v ${XDG_RUNTIME_DIR}::${XDG_RUNTIME_DIR} -e XDG_SESSION_TYPE -e WAYLAND_DISPLAY to the docker run command

Source: https://catalog.ngc.nvidia.com/orgs/nvidia/teams/clara-holoscan/containers/holoscan
Just after the docker run it explains some options you can set.

It works now thank you very much

Thank you for the inquiry @suveerudayashankara and for the support @genevanmeter !

./dev_container launch --ssh_x11
so it gave all the permissions for it to work

@genevanmeter are you saying it worked once you passed --ssh_x11 ? Or that you expected permissions to work since you used it and it did not?


How to set default EarlyStopping patience?

Is it possible to set the default EarlyStopping patience without creating a custom early stopping callback?
Instead of writing:
trainer = pl.Trainer(early_stop_callback=EarlyStopping(patience=XXX))

I'd like to overwrite the default patience directly and then use EvalResult(early_stop_on=...).

No. Any problem with using a custom callback??

No, just wondering. I just would like to keep the early stopping things in one place, EvalResult, instead of jumping elsewhere to change settings.
Thanks. Closing.