Add i18next and language switcher

Description
This PR adds i18next and react-i18next libraries to the project to initialize the i18n feature.
It also adds a language switch component to both the office and room appbars.
Initially only basic english and portuguese translations have been created.
The language will be set automatically according to the browser setting and if the user changes the language, the choice will be stored in the localStorage.
resolves #341
How to test?

Login to the Matrix
Look for the language switch button in the appbar and open it
Click one of the languages.

Expected behavior
The language names will change to their versions in the selected language, as well as the tooltip of the language switcher menu activator.

Gonna fix those codeclimate issues later today.

These codeclimate issues are odd. It says invalid token <> for react fragments. I think that's the only reason the function is exceeding the limit (might be wrong though), because there are other components which have way more LOC.
Example:
https://github.com/ResultadosDigitais/matrix/blob/fe762bf22011001ecb4c8651c15fa5dbf5b4e1e8/frontend/src/components/RoomCard.js#L60-L118


Poll all messages since message $id

This is incredibly annoying to work around in all the clients; I am implementing this now as part of the web UI.

Noooo. Why did I pick 10 as the message ID length. Now it's indistinguishable from a unix timestamp, which is also 10 characters long

Done, will be released as part of v1.16.0


Config missing from Atom 1.25 docs

The entire Config class is missing from the Atom 1.25 documentation, whereas it was present in the Atom 1.24 documenation.
A quick search of the atom-api.json file for the beta indicates that it's missing there too.

The v1.26 release branch doesn't have the commit that fixed this on the v1.25 release branch (#16957). Config will be missing again with the 1.26 release unless that commit is cherry picked.
@daviwil

Thanks for catching that, @GlenCFL!  Cherry-picked that commit over to 1.26-releases.


SURF shows covers in wrong color (doesn't use game's palette)

While I don't know how to get around this issue without breaking the 16 color limitation, it seems somewhat annoying that cover images aren't displayed in their correct colors if they were made with a custom palette.
Let me show you what I mean:
This is how my cover normally looks:

And this is how it looks when shown in SURF:

While it isn't that big of an issue in this particular example, it would be a very huge problem if the game were to use a significantly changed palette.
As a side note, SURF always seems to use the palette of the last game that was loaded in TIC-80. (So if you start a game through SURF, then close the game and open SURF again you'll see SURF using the palette of that game.)

I noticed this too. I think it could be possible to load the cart's palette when scrolling the entries.
There's a major problem anyway, the same as in editors when using custom palette. Consider the case you're using a palette composed of 16 shades of gray in your game, all the GUI colors will be poorly readable. The same for the SURF interface.
It would be nice to have a way to switch the palette and draw the background, then the next frame change another palette and draw the GUI. You'll pay on framerate 30 vs 60. This is not possible right now, changing palette affects also colors already draw the previously.
Don't know if this limit is something wanted or that simulate an hardware thing.

Perhaps it could be possible to draw Surf's UI using standard TIC palette but in OVR thing?

yes, it's possible with OVR

Fixed at the moment.
Closing.


[BUG] Help output is printed slowly

Describe the bug
The output printed by help is printed quite slowly.
To Reproduce
Steps to reproduce the behavior:

Go to the prompt
Type 'help' and press the enter key
It takes like 1 or 2 seconds to print everything

Expected behavior (if applicable)
See the help text getting printed more or less instantly
Desktop (please complete the following information):

OS: Arch Linux
Version: Built from Source

Additional context
Minor issue.

Might be a regression of #35, I think it might be buffering due to not using a back buffer and the current Trendy Terminal newline settings instead of scrolling.  Last time IIRC it was related to a long delay due to VT100.Report_Cursor_Position procedure.

Using https://github.com/onox/goblin-terminal (it uses VTE like gnome-terminal). But output is printed instant in tmux! :open_mouth:

Yup, if you use strace to check the length of the system calls you might see an issue.  htop has an easy way of visualizing this.

For each line printed, I see:

read()
read() returning E_AGAIN
write()
4x poll()
write()
clock_nanosleep()
2x poll()
write()

With weston-terminal the output is printed instantly and I see:

epoll_wait()
read()
write()

Slow in gnome-terminal as well (not a surprise since it uses VTE)

Thanks!  I'll look when I can, but it might be a couple of days before I get to this.


Can't set mathcal font in lualatex

(fontspec 2.5c, luatex 0.95, xetex 3.14159265-2.6-0.99996)
The following example:
\documentclass{report}
\usepackage{unicode-math}
\setmathfont[range=cal,Contextuals=Swash]{MinionPro-It}
\begin{document}
$\mathcal{F}$
\end{document}

Gives this result in xelatex:

But this in lualatex:

I am using the following workaround to set the mathcal font in lualatex (replacing the \setmathfont line ):
\usepackage{mathtools}
\newfontface\swash[Contextuals=Swash]{MinionPro-It}
\renewcommand\mathcal[1]{\mathmbox{\textrm{\swash #1}}}

Which seems to work, making me think that I had done things correctly and that this is a possible bug in fontspec. If it is a bug it isn't new - it was present in texlive 2015 when I first started using lualatex.

Hi Chris,
Thanks for reporting this. I can’t reproduce this problem myself, perhaps because I don’t have the Minion fonts on hand.
I tried this instead; can you confirm whether it replicates the problem you’re having?
\documentclass{report}
\usepackage{unicode-math}
\setmathfont[range=cal,Style=Swash]{EBGaramond12-Italic.otf}
\begin{document}
fontspec: {\fontspec{EBGaramond12-Italic.otf} Q}

text swash: {\fontspec[Style=Swash]{EBGaramond12-Italic.otf} Q}

mathcal: $\mathcal{Q}$
\end{document}

If that also fails for you, could you take a look in the log file and look for an entry near the end like this:
.................................................
. fontspec info: "defining-font"
.
. Font family 'EBGaramond12-Italic.otf(2)' created for font
. 'EBGaramond12-Italic.otf' with options
. [Path=/usr/local/texlive/2015/texmf-dist/fonts/opentype/public/ebgaramond/,Sty
le=Swash].
.
.  This font family consists of the following NFSS series/shapes:
.
. - 'normal' (m/n) with NFSS spec.:
. <->"file:/usr/local/texlive/2015/texmf-dist/fonts/opentype/public/ebgaramond/E
BGaramond12-Italic.otf:mode=node;script=latn;language=DFLT;+swsh;"
. - 'small caps'  (m/sc) with NFSS spec.:
. <->"file:/usr/local/texlive/2015/texmf-dist/fonts/opentype/public/ebgaramond/E
BGaramond12-Italic.otf:mode=node;script=latn;language=DFLT;+swsh;+smcp;"
.................................................

and report what you see.

Ok, I think I've made some progress.
Your example renders correctly for me with EBGaramond:

But not correctly with MinionPro-It. This is the result in both lualatex and xelatex:

The issue appears to be something related to the difference between the Style and Contextuals keywords - you may have though I was being silly using Contextuals=Swash in my example, but that's because using Style=Swash doesn't work in either xelatex or lualatex. With your example using Minion Pro and Contextuals=Swash:
\documentclass{report}
\usepackage{unicode-math}
\setmathfont[range=cal,Contextuals=Swash]{MinionPro-It}
\begin{document}
fontspec: {\fontspec{MinionPro-It} Q}

text swash: {\fontspec[Contextuals=Swash]{MinionPro-It} Q}

mathcal: $\mathcal{Q}$
\end{document}

I get the following with xelatex:

And this with lualatex:

So the problem here has something to do with contextual chararcters being able to use Minion's swashed style, but not all characters when Style=Swashed. And then the reason I was seeing the problem with mathcal specifically is because \mathcal{} in lualatex doesn't consider the first character a contextual, whereas xelatex appears to do so (but not further characters - in both xelatex and lualatex only the first character within \mathcal{} is actually rendered in the 'cal' font, the rest are rendered in the regular maths font, which is fine. The only difference is xelatex also considers the first char a contextual one in addition to being in the 'cal' font.).
So the problem seems to be perhaps two things:

Style=Swashed doesn't work with Minion Pro, although Contextuals=Swashed does.
Lualatex doesn't consider the first character within \mathcal{} to be a contextual, whereas xelatex does.

Just quickly for the time being — I am thinking this is a LuaTeX rather than a fontspec issue; it's not even clear to me if the current behaviour is a bug or just a difference in how XeTeX behaves. I'll leave this open here for now to remind me to check to see if there's anything I can do to work around the problem you're having. Of course, the best approach of course would be if the font provided a non-fragile way to access the swash characters!


Announce insight 5.5.5

Same as changelog

Now staged at https://snoopycrimecop.github.io/www.openmicroscopy.org/2019/08/29/insight-5-5-5.html.


Missing device configuration: fibaro and fdg-212 and dimming don't work

Checklist

[X] It is not in the configuration DB

[X] It was not merged recently or has a pending PR

[X] The device was interviewed completely and has no missing or undefined IDs

Which device is missing?
fibaro  fdg212
Manufacturer ID
0x010f
Product Type
0x0102
Product ID
0x1000
Firmware Version
v3.3
Is the device listed on the Z-Wave Alliance website?
https://products.z-wavealliance.org/products/4507
Do you have a manual?
FGD-212-EN-T-v1.3.pdf

This device is in the DB. Does your light support dimming? On quick look default of parameter 32 looks like there is an autocalibration.  You could try "0" to force dimming if you are sure your light supports it.

I am migrating from domoticz to HA. The light always worked in domoticz so i don't think there is an electrical problem.
When i look under configuration i get this:

So i think somthing goes wrong with the indentification of the dimmer.

@Frank18000 please fill out https://github.com/zwave-js/node-zwave-js/discussions/new?category=request-support-investigate-issue and provide all the requested information, especially the interview log. We can investigate there.


[RLStats] Remove Solo Standard

I currently have no idea how I'm gonna do it, 3 competitive modes don't really fit well in current design :/

Fixed in 6bd5a83315d4d5b732f82caaf8d32377439e4745


chore(main): release 2.92.0

:robot: I have created a release beep boop
2.92.0 (2024-01-02)
Features

Add new url for role master api keys (#3215) (924149c)
prepopulate control value on segment overrides (#3208) (68a1c6c)
tasks-processor: Add recurring task to clean up old recurring task runs (#3151) (9f83f27)

Bug Fixes

env variable instructions on locally-api.md (#3223) (4f2fa90)
erroneous booleans in feature tooltip (#3219) (3758d33)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Flagsmith/flagsmith/releases/tag/v2.92.0 :sunflower:


Update lodash _.partial and _.partialRight

lodash/lodash.d.ts
to authors(@bczengel @chrootsu). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

@vvakame :+1:

thanks all!


Problem on country analysis part

Hi,  I might have some suggestion, there is no intuitive comparability of data put together in different dimensions and orders of magnitude. could you improve that?

I see the problem. Thank you! The data in the feature vector is very scattered and the small data are swallowed by the big data. I would use standardization to deal with this situation. Z-score standardization may be best suitable in this case. The data will be converted to the center by removing the mean of each feature and then be scaled via dividing the non-normal feature by its standard deviation. By doing so, there could exist more comparability of the data in different dimensions and orders of magnitude.


Error building Cordova app for Windows Phone (Universal)

After installing phonegap-plugin-barcodescanner on a Ionic project I get an error when building for Windows Phone Universal with Visual Studio 2015.
I tried to remove the plugin and after that I can build the project without any problem.
Anybody experienced the same problem? Here's a log of the error (it says "file not found", but the .appx file does exist):
C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\AppxPackage\Microsoft.AppXPackage.Targets(2519,5): error APPX1204: Failed to sign 'C:\Users\APIAZZA00\Documents\Visual Studio 2015\Projects\Ionic Blank App1\Ionic Blank App1\platforms\windows\AppPackages\CordovaApp.Windows10_<IP_ADDRESS>_x86_debug_Test\CordovaApp.Windows10_<IP_ADDRESS>_x86_debug.appx'. SignTool Error File not found: C:\Users\APIAZZA00\Documents\Visual Studio 2015\Projects\Ionic Blank App1\Ionic Blank App1\platforms\windows\AppPackages\CordovaApp.Windows10_<IP_ADDRESS>_x86_debug_Test\CordovaApp.Windows10_<IP_ADDRESS>_x86_debug.appx
Thank you

Hello @aristt, this is related to package signing, see http://stackoverflow.com/questions/15099859/signtool-error-while-packaging-windows-store-app for details.


Engagement/jessica/15073 louisiana validation

This PR addresses changes request by LA after their first test message
Test Steps:

Make sure the integration tests pass.
Check the email and make sure that the fields are added that they specified other than PID 11.7 which SR does not send over and OBX.7 which we don't send over for this kind of test.

MSH.05.03 : Receiving Application Universal ID type is blank.
This field is required and should be set to 'ISO'

– Data Analyzed: : ''

MSH.06.02 : Receiving Facility Universal ID is blank.
MSH.6.2 must be 'LADOH' when sending ELR data to LA

– Data Analyzed: : ''

MSH.06.03 : Receiving Facility Universal ID type is blank
should be set to 'ISO'

– Data Analyzed: : ''

PID.11.07 : REQUEST - Patient Address Type is blank. It should be populated from HL70190 table.
Are you able to provide this field?

– Data Analyzed: : ''

PID.11.09 : REQUEST - Patient County/Parish Code failed check.
Are you able to provide this field? Please refer to table HL7 FIPS6-4

– Data Analyzed: : 'Orleans'

OBX.07 : REQUIRED - No Reference Range provided. Feedback: Ex: 'Not Detected'
Qualitative Results Reference Ranges: Negative for tests resulting as Positive

– Data Analyzed: : ''
Not Detected for tests resulting as Detected Normal for tests resulting as Abnormal

Changes

Addressed LA's concerns with the first test message.
Added an integration test for them.

Checklist
Testing

[ ] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Recommend adding an element that removes the county entirely if patient state not "LA" otherwise if you get a report with ordering facility state of "LA" and patient state of someone else the county won't be translated to FIPs code

Decided to just do this right and create the fips lookup


CNV-29392: VM actions menu jumps when vms on list change state

📝 Description
When appending the dropdown to the outer page, every re-render on that page will cause the dropdown to re-render and change location.
When using the Dropdown component from PF we can use the menuAppentTo prop to append the dropdown to the parent, however this presents a scrolling issue on the firefox browser, as the dropdown keeps getting focus and the scorlling would seem stuck. this is a PF issue that needs to be resolved as the focus is coming from the dropdown component.
🎥 Demo
Before:
https://github.com/kubevirt-ui/kubevirt-plugin/assets/67270715/483f3437-e26c-4b95-ba1b-5ad3e59fe30c
After:
https://github.com/kubevirt-ui/kubevirt-plugin/assets/67270715/25afa42a-7c70-4758-841c-191d53a770cc

getContentScrollableElement was there to prevent the issue that Hilda is pointing out.

@upalatucci @hstastna thanks for reminding me that issue :)
changed the PR and description accordingly please review again

@upalatucci @hstastna @metalice
Since the behavior of the dropdown is happening only in firefox, and is caused by the dropdown that keeps focusing on the menu only in firefox, I've opened an issue for that in PF https://github.com/patternfly/patternfly-react/issues/9326

@upalatucci @hstastna @metalice Since the behavior of the dropdown is happening only in firefox, and is caused by the dropdown that keeps focusing on the menu only in firefox, I've opened an issue for that in PF patternfly/patternfly-react#9326

Sounds reasonable to me. But as fixing bug in PF usually takes very long time, what would be cost of simply not using PF component (but some other one, maybe implemented by us) in this specific scenario? WDYT, Aviv?

Sounds reasonable to me. But as fixing bug in PF usually takes very long time, what would be cost of simply not using PF component (but some other one, maybe implemented by us) in this specific scenario? WDYT, Aviv?

@hstastna I'd say it can take at least a day to make a new dropdown component (assuming I can make one based on menu component from PF, if not that can take more to copy all the css)
I also noticed this issue is depending on the window's zoom level (90% and below this bug is not happening for me) so maybe it could be just a known issue that is written in the release notes

Okay for me @hstastna

@hstastna I'd say it can take at least a day to make a new dropdown component (assuming I can make one based on menu component from PF, if not that can take more to copy all the css) I also noticed this issue is depending on the window's zoom level (90% and below this bug is not happening for me)...

Of course it depends on the zoom level. And unfortunately, on the size of the window, too. This PR improves the jumping issue a little bit, but still not fixed completely. I'd suggest to merge this PR as is, to contact PF folks to fix this issue in the component asap.

...so maybe it could be just a known issue that is written in the release notes

Yes, this one, too, please. However I am not sure how to achieve this. Hopefully you know :D

PF issue link: https://github.com/patternfly/patternfly-react/issues/9326

/retest


Documentation needs more info on modifying templates

I read the docs on the wiki about the template contents. What I can't seem to find is some sort of description saying where the templates can be found within the IDE.
A bit of a description showing where/how to access the templates to modify them would really help.

mark

check there.
https://github.com/setial/intellij-javadocs/issues/55

Fixed in 4.01 version


fix: add non-null assertion to iconName

fix #132
Added non-null assertion in iconName so that TypeScript can infer the type as string, not string | undefined.

Just came here to do the same.
@manniL  is there any way we can get this merged?


Fix: Thumbnails display fix

Fixing the Thumbnails display refered here : https://github.com/OHIF/Viewers/issues/2796
You can compare the display here :
Before =>

After =>

As you can see I've just set a minimum size for the thumbnail to show and added the ohif-scroll bar css to the StudyBrowser component.

@Celian-abd sounds good, as the series fetching from backend is slow i think we should add a "loading" message if thumbnails.lenght === 0
@swederik did you considered to add a spinner component into OHIF, there a lot of spinners in this library (https://www.davidhu.io/react-spinners/).
We can propose something with celian, unless you want keep the control of your component style in your library  ?

@salimkanoun great idea for spinners, we had some brief discussion with @dan-rukas about having some sort of spinner that involves OHIF logo as in the windows in the OHIF logo were spinning, but as you know better than everyone the funding is limited, and there are higher priorities

@Celian-abd Great fix. I run the deploy preview version manually (since it was cancelled for some reason), and noticed the following UI (double scrollbar) which is not in the design files. I think if you remove the ohif-scrollbar class and just keep this PR to min-width ... we can merge it

@salimkanoun great idea for spinners, we had some brief discussion with @dan-rukas about having some sort of spinner that involves OHIF logo as in the windows in the OHIF logo were spinning, but as you know better than everyone the funding is limited, and there are higher priorities

You already have a spinning logo ? (never seen it, i think your logo is easy to animate just by changing alternatively the background of the 4 square), if you have a logo and just needed to be integrated, we will make you a PR with Celian this automn,
Salim

No logo, just the thoughts of a spinner logo

No logo, just the thoughts of a spinner logo

Ok, do you have sort of a wish list ? I think the best is too keep all ideas/ needs somewhere so we can address items of the list with Célian


[WIP] Rewrite zoom. Add a QComboBox to choose some current zoom level

Instead of having 4 buttons, I rewrite the UI inspired by other PDF viewer.
With this PR, we have:

One combobox with default zoom levels
Fit Page / Fit Width are inside the combobox and buttons are removed
Zoom In and Zoom Out are based upon this combobox too. Scale factor is always 1.2 but from the combobox value

If you agree with this new UI/features. I will finish my PR by polish it and fix zoomIn/zoomOut when combobox value is Fit Width.

Sorry, I thought one of us got to this already.    If it's ready to go and you're happy with it let us know.

@q5sys Sorry, was too busy last months. IIRC there is a problem when you change the page. I'll take a look next days.

No problem, take your time.  I saw ther we didnt reply recently and I
didn't want you to think that we were intentionally ignoring you.
Get to it when you can get to it.  Real life comes first.
On Sat, Jan 4, 2020 at 2:35 AM lbartoletti<EMAIL_ADDRESS>wrote:

@q5sys https://github.com/q5sys Sorry, was too busy last months. IIRC
there is a problem when you change the page. I'll take a look next days.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lumina-desktop/lumina-pdf/pull/7?email_source=notifications&email_token=ABDQJJYLCLIJAGZ6GIPPM7LQ4A34RA5CNFSM4JGO6QZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEICSY7A#issuecomment-570764412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDQJJ45WYJ7HFNZRBGDBQDQ4A34RANCNFSM4JGO6QZQ
.

@q5sys mmm sorry, time was long... I didn't contribute to lumina anymore since nothing was happening, but I saw the new version and your tweet.
So, I can't reproduce the bug anymore, so it's should be OK to merge this PR.
BTW, can you release it as 1.6.1 ? (same for calculator and lumina, cf<EMAIL_ADDRESS>lumina)
Thanks

@lbartoletti No need to apologize.  The project was stagnant for a while so I can understand why you didn't working on this further.
I dont remember much of this, so let me give it a check over this weekend.
I'm going to be releasing a v1.6.2 hopefully in a few weeks. Would you rather me re-release this as 1.6.2 then, or go head and put out a 1.6.1 release of this (once this is merged) in the meantime?


fix: updated spid-cie-oidc-django version

fix: pinned python docker image to 3.11-slim
fix: pinned design-django-theme to 1.4.8

@rglauco did you have tested this upgrade with the current spid-cie-oidc-django release you mentioned?

Yes, i've tested the updated Federation Entity and by now i can register RP entity and access to the login page. After that, when i try to login with SPID i receive an error it.spid.cie.oidc.exception.OIDCException: Missing WellKnown configuration which is the same i obtain by using spid-cie-oidc-django v.0.6.2. I'll investigate further.


Bar chart does not display integer values on x axis

Discussed in https://github.com/apexcharts/apexcharts.js/discussions/2827

Originally posted by gitgkk December 15, 2021
I'm using bar chart from ApexCharts in vue but I the problem is basic and has to do with ApexCharts.
The horizontal x axis has counts of items. When the number goes too low like 1, 2, the axis shows intermediate decimal values which doesn't make sense. The x axis shows floating point values, which is not the desirable output. How to display only integer values only with ticks at integer intervals like in second image?
Thanks for helping resolve this issue.

https://stackoverflow.com/questions/70336097/apexcharts-how-to-force-an-axis-in-bar-chart-to-display-integer-values-only
<apexchart ref="ratingChart" width="450" height="250" :options="ratingData.options" :series="ratingData.series" ></apexchart>
data: function () { return { ratingData: { options: { chart: { id: "item-rating", type: "bar", }, title: { text: "Item by Rating", align: "center", margin: 10, offsetX: 0, offsetY: 0, floating: false, style: { fontSize: "14px", fontWeight: "normal", fontFamily: "Arial", color: "#595959", }, }, noData: { text: "Loading...", }, xaxis: { categories: ["Very High", "High", "Medium", "Low", "Eliminated"], type: "category", tickPlacement: "on", labels: { rotate: 0, rotateAlways: false, }, decimalsInFloat: 0, }, colors: ["#E53935", "#FFA726", "#FDD835", "#7CB342", "#29B6F6"], dataLabels: { enabled: true, }, plotOptions: { bar: { distributed: true, borderRadius: 0, horizontal: true, }, }, }, series: [ { name: "itemCount", data: [10, 4, 8, 9, 3], //data: [1, 2, 2, 1, 3], // for low counts }, ], }, } }

Please do not double post your questions as issues.

Please do not double post your questions as issues.

@brianlagunas I understand this aspect. Nobody responded to this question and I need a quick answer. Should I close it in the discussion and repost it?


More detailed permissions for data output.

For my server, I would like to make parts of the API public. Mainly because I recently created a player position modification for overviewer maps.
So it would be cool to have a more detailed permission system to restrict access to certain parts of data.
For example, I don't need the inventory, connection, achievements and most of the statistics
Example permissions for my use case:
player.name
player.location.*
player.armor
player.health
player.experience
player.gameMode
player.food
player.statistics.deaths

Also restricting the access to POST endpoints would be nice.

This has been partly implemented in 190f527b28d5c90bf023937055696cb05349e48e and is in releases starting with v3.2.0.
The restricting of actual data has not yet been implemented, this will take a while as it's more involved, I am however already working on it.

This is implemented in 284ae5268677d913d32071b8be5814cb07368db0, starting with version 3.3.0.
Documentation can be found here and more generally here

Looks awesome 👍


Create command browse for focusing the browser and focusing websites tabs

Some people have moved away from using native apps to using web apps when available in order to be able to use Rango. One small issue with this is that there is no way to directly focus one of those web apps. In knausj we have the command open {website} to open one of our custom websites defined in websites.csv. So, for example, I use open talon slack often. The problem is every time I use the command it opens a new tab. It would be nice to have a command that worked like open but that it would focus the open tab if there is already one with the same domain. If there is none it would open the page in a new tab (maybe with the option of making it pinned). This command needs to work globally and not just when the browser is focused.
So, if in my websites I have https://app.slack.com/client/T7FPSMV8F,talon slack and I use the command browse talon slack, one of two things can happen:

If there is already one tab open that matches the url, for example https://app.slack.com/client/T7FPSMV8F/C03DN7M6KG8/thread/C03DN7M6KG8-1682434151.155699 it will focus that tab.
If there is no page that matches that url it will open the page https://app.slack.com/client/T7FPSMV8F in a new tab.

There are a couple of issues with this implementation though:

It can't handle redirects to a different domain. So if you have in your websites.csv https://talonvoice.slack.com,talon slack it will always open a new tab because going to that address is going to redirect to https://app.slack.com, so the user needs to make sure it stores the redirected address.
It can't handle multiple pages with the same domain. A user might have the talon slack website and also a different slack workspace. Using this browse command is going to focus whichever tab was previously open no matter which workspace you specified.

One alternative to handle this second issue is to focus the tab with the same domain but also navigate to the website specified in websites.csv. Maybe this could be a different command, for example something like browse go talon slack. Another alternative is to match the entire address when using browse go, so it no tab matches the entire url it opens a new tab. Afterwards the browse command would prefer the tab that matches the url better.
Since this is going to be a global command we also need a way to know which is the default browser to be able to focus it. It could be stored the first time the user uses a Rango command, and also have a command like rango set default to override that.

Having this command be available globally is more difficult than expected. There is no easy way to switch to a given browser or even the default one that works for all OSes. Apart from that to make this work I would have to make the actions to send commands to the extension available globally, which I find a bit messy.
So I have decided to make this command available only when the browser is focused.


Add basic bench testing for analyzers

Add basic performance bench tests for analyzers. For https://github.com/istio/istio/issues/18008.
These come in two forms:

A simple bench test using our unit test case data (minimal input sizes). This is simply a benchmarked version of the required analyzer unit tests.
A bench test using artificially constructed test data, with N of every possible resource (and each resource being entirely blank).

This doesn't give anywhere near perfect coverage and neither scenario is really "realistic", but it is enough to highlight some analyzers as being outliers in terms of how long they take to run, and that's something we can take action on.
Example output (filtering out results with < 0.01 ns/op):
 crwilson@crwilson:~/go/src/istio.io/istio$ go test -bench=. -run=XXX ./galley/pkg/config/analysis/analyzers -v | grep -v "0\.00"
goos: linux
goarch: amd64
pkg: istio.io/istio/galley/pkg/config/analysis/analyzers
BenchmarkAnalyzers/virtualServiceDestinationHosts-bench-8<PHONE_NUMBER>	         0.0115 ns/op
BenchmarkAnalyzersArtificialBlankData100/gateway.IngressGatewayPortAnalyzer-bench-8<PHONE_NUMBER>	         0.0231 ns/op
BenchmarkAnalyzersArtificialBlankData200/gateway.IngressGatewayPortAnalyzer-bench-8<PHONE_NUMBER>	         0.181 ns/op
BenchmarkAnalyzersArtificialBlankData400/gateway.IngressGatewayPortAnalyzer-bench-8                         	       1<PHONE_NUMBER> ns/op
BenchmarkAnalyzersArtificialBlankData800/auth.MTLSAnalyzer-bench-8<PHONE_NUMBER>	         0.0216 ns/op
BenchmarkAnalyzersArtificialBlankData800/gateway.IngressGatewayPortAnalyzer-bench-8                         	       1<PHONE_NUMBER>4 ns/op
PASS
ok  	istio.io/istio/galley/pkg/config/analysis/analyzers	22.562s

Note that bench results can be further profiled, see https://medium.com/@felipedutratine/profile-your-benchmark-with-pprof-fb7070ee1a94 for one article about how.

See https://github.com/istio/istio/issues/20123 which was inspired by this, may be something possible to leverage by this long term. Its pretty nebulous future though

@sushicw: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-galley-k8s-tests_istio
4402eb3e5175134a74faeded14b0c1f321c4bc99
link
/test integ-galley-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sushicw: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-galley-k8s-tests_istio
4402eb3e5175134a74faeded14b0c1f321c4bc99
link
/test integ-galley-k8s-tests_istio

integ-istioio-k8s-tests_istio
4402eb3e5175134a74faeded14b0c1f321c4bc99
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


feature: Add PWA functionality

Is your feature request related to a problem? Please describe.
This is a convenience feature, wherein users can "install" the web app to act as a native app on their handheld devices. I have daily code open on my tablet most of the time, and thought that this would be a welcome feature to have.
Describe the solution you'd like
Create PWA functionality, wherein users can "Install App" via Android and "Save to Home Screen" on iOS.

I have implemented this.


feedrapp.info is down / 502 Bad Gateway

Opened the issue just to inform you about this.

Thanks for quick fix! :)


Pixi-Particle sample

Hi, a little late, here is the very first sample using pixi-particles lib.
I modified the builld.hxml in samples to include samples/particles/Main.hx
I added a samples/_output/particles.html and some assets in samples/_output/assets/particles
I successfully built and tested the project in my browser.
The code is quite straightforward but I added some side comments to explain what I did.
I added a custom JsonLoader to load json config file for the particles but maybe you have more info on better ways for loading jsons and using them in Dynamic vars?
You'll tell me if that's ok for you.
++

made a few changes and merged to master :+1:
http://adireddy.github.io/demos/haxe-pixi/particles.html

Great! By the way cool idea for loading the json!


password changed  unexpected while using api PUT /api/v1/security/users/{pk}

Bug description
my password changed  unexpected while i using this api and without param "password" : PUT /api/v1/security/users/{pk}
How to reproduce the bug

login to /swagger/v1
using API: PUT /api/v1/security/users/{pk}
my request body: {"roles": [1,3]}

but my password changed unexpected，and i cant login with my old password
before：pbkdf2:sha256:600000$dwUuXbhBFZl0RjM5$614e0b7d24cbadf0fe67885c7b0a4646391cb014b82f074914ed952dad12de27
after：pbkdf2:sha256:600000$ztVn83UZKx0VtPT5$823617b8adfb61732f4fac0c4480f93a6e5c54e828680a347d1929de72a8c2d0

Screenshots/recordings
No response
Superset version
3.1.1
Python version
3.10
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[ ] I have searched Superset docs and Slack and didn't find a solution to my problem.
[x] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

@dpgaspar你好，能帮忙看一下吗？非常感谢！

https://github.com/dpgaspar/Flask-AppBuilder/pull/2179
FAB  Issues


OneToMany support broken? (or just confusing)

I've setup two classes with what I believe is the same setup from the Queen/Ant example.
If I try to save the "Ant" first, I get an error about "no such table" for the Queen table.  If I try to save the Queen first, I get an error about "no such table" for the Ant table.
I was going to try wrapping this in a transaction, but the Transactions documentation simply refers to an object called "database", which I have no idea how to get.

You can solve your issue using this post https://github.com/Raizlabs/DBFlow/issues/32#issuecomment-71563557

Fixed in 2.2

Hi, I am now using version 2.2.1 of the lib and getting still some weird behavior.
If we talk in Queen --> Ants example, then:

I am creating Queen object from JSON (where it has ants as array)
I am saving whole Queen object to DB using TransactionManager
I am getting Queen object in DB and its Ant objects in DB as well, but queen_id key is empty for all the ants.

I have tried overriding save like in #32 so that it was like:
    @Override
    public void save() {
        super.save();
        for (Ant ant : ants) {
            ant.associateQueen(this);
            ant.save();
        }
    }

And then I am getting exception like:
    java.lang.RuntimeException: com.raizlabs.android.dbflow.structure.container.ForeignKeyContainer$InvalidMethodCallException: Cannot call save() on a foreign key container. Call load() instead


Take ownership of Emby, Eight Sleep, Hikvision

Description:
Update CODEOWNERS to take ownership of my projects for future reviews.

You can add [skipci] to your commit message to bypass CI.


Paper Spigot jar not downloading when set to Specific Version

Bug Report
System Information

Ubuntu 18.04.4 LTS
AMP version <IP_ADDRESS>
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

Created Minecraft Java server, set Server Type to Paper Spigot, Release Stream to Specific Version and Paper Version to 1.15.2. Hit Download/Update, but the jar does not download and an empty paperclip.jar is created.
Debug logging shows that the version number is being omitted from the download URL, ie:
https://papermc.io/api/v1/paper//latest/download
(Note the missing version number between the forward slashes after 'paper'.)
By comparison, when the Release Stream is set to Stable, the jar downloads properly, and debug logging shows the correct URL:
https://papermc.io/api/v1/paper/1.15.2/latest/download
Bizarrely, though, this error only seems to happen the first time after a download is attempted on Specific Version. If after using a different Release Stream such as Stable, the Release Stream is changed back to Specific Version, then the download works fine.
Reproduction

See above.

Further testing suggests the issue is because the version number is not being set in AMP on first install to match what appears automatically in the dropdown - ie, there is no default set, even though 1.15.2 shows in the dropdown. The download works when the Paper Version is set to a different version such as 1.15.1, or even when it is set to 1.15.1 and then changed back to 1.15.2 before the download is attempted. The action of changing the version sets it in AMP, when it has not been set before.
Presumably the behaviour should be that AMP sets the initial default to whatever appears automatically in the dropdown (ie the latest version). So if the download button is hit, that version is downloaded. This would match the user's expectation, namely that what they see displayed in the dropdown is what will be downloaded, and they should not have to manually select what is already shown.

Now confirmed that this behaviour is the same for other Server Types, eg Official. If the Server Version is not changed from the initial value that appears in the dropdown, and Specific Version is chosen, then the jar is not able to be downloaded.
This is probably the same behaviour that is the subject of issue #218.

This should be fixed as of the latest update.


open-shell-np: Fixed pre_uninstall script so it only works when the user types uninstall.

[x] I have read the Contributing Guide.

/verify

Thank you, @HUMORCE


Enabled support for arbitrary user model

This enables the use of custom models.
A user can define which model to use by setting the PUSH_NOTIFICATIONS_SETTINGS key USER_MODEL to a model on the format appname.Model. The default user model is settings.AUTH_USER_MODEL which is the current user model in the library.
The admin.py's search_fields only contains the user model's USERNAME_FIELD if the user model actually has that field.

Could you fixup the second commit into the first one please?

@jleclanche of course, this is now fixed.

lgtm but can you explain how this is necessary? why can't we just use the user model from the setting?

@jleclanche ok! There are cases when you have multiple user models.
A current case for me is that I've integrated with another system which holds the users I want to send push notfications to. The users (via their mobile clients) communicate with one aggregating server which communicates with (1) an external server holding user data and (2) the django server which complements the external server with extra user data and push notification functionality. In this case it therefore makes sense to send push notifications to another user model than AUTH_USER_MODEL.

Ok. I'm not too happy with the solution but I think it's fine to expose that field anyway.

Remove the try block and I'll land it.

@jackfengji just tell me how you'd be happy with the solution.

I meant more in a general sense that I don't think the correct way to solve your issue is to change that. But I am fine with exposing that as a setting anyway, so might as well.

Fixed @jackfengji

Landed in 40cfe2b

@jleclanche this is a nice edition, some people don't extend any form of the django user object but still wan't these relationships. Thx


Roll engine to 8450a003ee2c9f6488b796944088f1f520928cd0

This contains the following commits:
flutter/engine@8450a00 Roll Skia back to d8a189fca3cdfa064de75a280d5af34cc5026bbe
flutter/engine@3192223 Roll src/third_party/skia/ 6198f39ad..e7df0bb90 (1 commit)
flutter/engine@5f08fca Roll src/third_party/skia/ cea8e3d13..6198f39ad (1 commit)
flutter/engine@a0be877 Make thread names more descriptive.
flutter/engine@9110e89 Expose View:OfferServiceProvider to UI thread components in a thread safe manner.
flutter/engine@eb5bbda Specify a custom isolate snapshot per AOT shell launch.
flutter/engine@d97b6d8 Allow explicit specification of the isolate snapshot.

https://github.com/flutter/engine/commit/d97b6d8#r28902114 is causing tests to fail.  Closing this - we'll have to roll to a newer engine.


ink analyzer

https://github.com/ink-analyzer/

AC:

Run on contract
If it shows something, make follow on tickets for that


Server turning unresponsive

When i make a query to the firestore database the whole server turns unresponsive like is locked, i have a similar server runing locally and everything works fine, the production server is running CentOs and my dev server is running on Windows, both places using Apache 2.4, i dont't know if it's something with grpc or what.

Can you share the code which is causing the issue? Which version of PHP are you using?

$db = $gcloud->firestore();
$postsRef = $db->collection('posts')->orderBy('date', 'DESC')->limit(50);
$posts = $postsRef ->documents();

Are there any errors logged on the server? Does it truly lock up the server, or do you mean the request blocks indefinitely?

Since we haven't heard back for a while, I'm going to close this out. Please feel free to re-open if this is still an issue.


@slack/bolt should be in devDependencies

The usage of<EMAIL_ADDRESS>installs a version that doesn't support Node 16, it was dropped in 3.14.x - https://github.com/slackapi/bolt-js/releases/tag/%40slack%2Fbolt%403.14.0.
On Node 16, this will result in this error:
[2/5] 🔍  Resolving packages...
verbose 0.269790083 Performing "GET" request to "https://registry.yarnpkg.com/contentstack".
verbose 0.42522025 Request "https://registry.yarnpkg.com/contentstack" finished with status code 200.
verbose 0.427749333 Performing "GET" request to "https://registry.yarnpkg.com/@slack%2fbolt".
verbose 0.498153625 Request "https://registry.yarnpkg.com/@slack%2fbolt" finished with status code 200.
verbose 0.499742 Performing "GET" request to "https://registry.yarnpkg.com/@slack%2flogger".
verbose 0.500095042 Performing "GET" request to "https://registry.yarnpkg.com/@slack%2foauth".
verbose 0.500514167 Performing "GET" request to "https://registry.yarnpkg.com/@slack%2fsocket-mode".
verbose 0.501111708 Performing "GET" request to "https://registry.yarnpkg.com/@slack%2ftypes".
verbose 0.50189825 Performing "GET" request to "https://registry.yarnpkg.com/@slack%2fweb-api".
verbose 0.586010042 Request "https://registry.yarnpkg.com/@slack%2flogger" finished with status code 200.
verbose 0.621053042 Request "https://registry.yarnpkg.com/@slack%2foauth" finished with status code 200.
verbose 0.632662 Request "https://registry.yarnpkg.com/@slack%2ftypes" finished with status code 200.
verbose 0.635226125 Request "https://registry.yarnpkg.com/@slack%2fsocket-mode" finished with status code 200.
verbose 0.640045208 Request "https://registry.yarnpkg.com/@slack%2fweb-api" finished with status code 200.
[3/5] 🚚  Fetching packages...
info<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "darwin" is incompatible with this module.
info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">= 18". Got "16.20.2"
verbose 0.915547417 Error: Found incompatible module.
    at MessageError.ExtendableBuiltin (/opt/homebrew/lib/node_modules/yarn/lib/cli.js:721:66)
    at new MessageError (/opt/homebrew/lib/node_modules/yarn/lib/cli.js:750:123)
    at checkOne (/opt/homebrew/lib/node_modules/yarn/lib/cli.js:48058:11)
    at Object.check (/opt/homebrew/lib/node_modules/yarn/lib/cli.js:48077:5)
    at /opt/homebrew/lib/node_modules/yarn/lib/cli.js:7348:73
    at Generator.next (<anonymous>)
    at step (/opt/homebrew/lib/node_modules/yarn/lib/cli.js:310:30)
    at /opt/homebrew/lib/node_modules/yarn/lib/cli.js:321:13
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Please either add the information about dropping Node 16 to the CHANGELOG.md or revert (if you intend for Node 16 to work, not sure if it's a priority for Contentstack)
or maybe add an engines >= field as well

Hello guys, any progress on this one?

Hey @sethidden @bartoszherba , we will be releasing a new version with the fix very soon. I will update here once its released.

@sethidden The fix has been released, please do check.


UEFI Secure Boot support

Provide support for using UEFI Secure Boot inside a VM. I have an existing Windows 10 ARM64 VM and it's not letting me to upgrade to Windows 11 because of Secure Boot not being available.

It seems this could be achieved with the QEMU TPM emulator device. If I understand correctly, this requires a swtpm process to run separately. This might mean it's not going to work on iOS (?).

Is it possible to build setpm in with UTM? My motivation is on getting this working is for macOS.


AI wave attack

To fix the entire pathfinding of stronghold crusader, there is no way to do this by editing some hexdata in the exe. BUT maybe there is still some tweaks possible to improve a couple of things:

The AI always attacks in stupid waves... seems like there is a maximum of one attacker per wall at anytime. As much as i like those big endgame armies, as usless they are if you always just have to shoot down a group of 50 at the same time. If the ai counts one man per wall, would it be possible to buff this as something like 5 per wall? Sadly this would overpower our little candle infidel to some kind of industrial pyromanic INFIDEL!!!. So if by chance there really is a tweak for the number of waves it would be awesome to limit them to 2 or 3.

(2. The engineers should build there siege equipment just outside the usual range of a mounted ballista. It would be important to know IF and how the ai decides where to place the siege equipment. If it does it by evaluating the distance to the nearest wall this could be quite an easy task, else just forget about it...
PS: those tents should have some space aswell - two siege engines are better than one with a nice little tent to it side)
(oh and assassins actually USING there ability to climb rather to try scratching down a stone wall with a simple sword)
(double oh, just attacking the nearest opponent instead of walking through the enemies line, to finally attack with one lonely half dead ladderman. I know this somehow touches the caracteristic of the different ais, but to be honest to implent this caracteristic firefly should have handled the pathfinding issues in advance)
Gracias!

Today i've made some 1-1 AI fights. Saladin was using armies with 200+ men. But he attacked with about 7 fighters at once.  That is the biggest wave-splitting i've seen so far o.O


Invalid command line Expected -option_name or --option_name, got '-' using gget muscle

What happened?
When I try to execute gget muscle via command line or via python script I get the same error message:
Via command line:
C:\Users\mypath > gget muscle fastas_test.txt
Or via python script:
file_name = "fastas_test.txt" gget.muscle(file_name) print(type(muscle_alignment)) print(muscle_alignment)
I get the same error message:
"Fri Aug 18 15:52:58 2023 INFO MUSCLE compiled.
Fri Aug 18 15:52:58 2023 INFO MUSCLE aligning...
Invalid command line
Expected -option_name or --option_name, got '-'"
This is how my fasta file is formatted:

Sequence1
MKTITLECMAIKLLDHLSEKEEQVKQRAEGEVKPEALQEALEKALKETGGVTVTKH
VRWTDGLTCGQLIKTNHPDKPTRIYVCLRGTKAKIADGAILKEDVIEKSTLNKFDLRG
ELKIEEKLRHLHPENSQSGDYPEAQIADGSKEVTLRQTCPTGWYFLRLLVDKDDKSYIK
ALAEKAAAKKVGELFNRYERPDCTLAKLDYVRVQVNIDAGGVLAALDRAEAIYALGLKA
ETEDGVVKLAADDKAAAPKAASRDPKAAGRVSSDPGVDSDGRGAPGPDGVGDSDSEDIAD
SEDKK
Sequence2
MSTQQIYTQVKVLQAMRHNGKLTLIIRATGGVVKLAADDKAAAPKAASRDPKAAGRVSS
DPGVDSDGRGAPGPDGVGDSDSEDIADSEDKK

gget version
0.27.0
Operating System (OS)
Windows
User interface
Command-line
Are you using a computer with an Apple M1 chip?
Not M1
What is the exact command that was run?
gget muscle fastas_test.txt

Which output/error did you get?
Fri Aug 18 15:52:36 2023 INFO MUSCLE compiled.
Fri Aug 18 15:52:36 2023 INFO MUSCLE aligning...

Invalid command line
Expected -option_name or --option_name, got '-'

Hi Emily,
gget muscle expects files in fasta format (even if the file is a .txt file), so the sequence headers should start with ">". I suspect that this is causing the problem here.
I tried to reproduce your error (I am not getting the exact same error message which might be a Linux vs. Windows thing) and provide a fix in this Colab notebook:
https://colab.research.google.com/drive/1Hk0ot5NNNyeQXCajEMcQYSwPkdQGmXJn?usp=sharing

It works when I run it in google colab. But when I copy the following portions into a python script and execute it locally I get the same error message.
code:
`
import gget
title1 = ">Sequence1"
seq1 = "MKTITLECMAIKLLDHLSEKEEQVKQRAEGEVKPEALQEALEKALKETGGVTVTKHVRWTDGLTCGQLIKTNHPDKPTRIYVCLRGTKAKIADGAILKEDVIEKSTLNKFDLRGELKIEEKLRHLHPENSQSGDYPEAQIADGSKEVTLRQTCPTGWYFLRLLVDKDDKSYIKALAEKAAAKKVGELFNRYERPDCTLAKLDYVRVQVNIDAGGVLAALDRAEAIYALGLKAETEDGVVKLAADDKAAAPKAASRDPKAAGRVSSDPGVDSDGRGAPGPDGVGDSDSEDIADSEDKK"
title2 = ">Sequence2"
seq2 = "MSTQQIYTQVKVLQAMRHNGKLTLIIRATGGVVKLAADDKAAAPKAASRDPKAAGRVSSDPGVDSDGRGAPGPDGVGDSDSEDIADSEDKK"
with open("fastas_test_2.txt", "w") as fasta_file:
fasta_file.write(title1 + "\n" + seq1 + "\n" + title2 + "\n" + seq2)
gget.muscle("fastas_test_2.txt")
`
error:
`
mypath> py gget_muscle_debug.py
Mon Aug 21 11:17:25 2023 INFO MUSCLE compiled.
Mon Aug 21 11:17:25 2023 INFO MUSCLE aligning...
Invalid command line
Expected -option_name or --option_name, got '-'
`

Is it possible that there is a "-" or spaces somewhere in the absolute path to your fasta file or muscle binary (inside gget)?

I don't think so. The python script "gget_muscle_debug.py" and the fasta file generated by the script and then used in gget.muscle "fastas_test_2.txt" is all in the same folder.

Even if they are in the same folder, gget muscle will use the absolute path to these files from the root directory of your computer. For Linux and Darwin, we can just put the paths in quotation marks and then there are no problems with spaces etc, but for Windows, this does not work unfortunately (this all happens under the hood in gget muscle).

That was it, thanks!


feat: Support Boolean datatypes after klickhouse switch

This can be extended for other datatypes as well now (relatively easily).
Thinking of forking klickhouse for our purpose. See the comment under Date32 for details.

@scsmithr can you try this out with clickhouse cloud once?


Added Hebrew support 😉

As said in the title. also, added tests. (:

It appears that some bug occurred that causes the CI checks to fail. Can you please re-run the jobs?

Apologies for the delay! Since I don't speak Hebrew I may need your input in the future to make sure the feature keeps working correctly. Sometimes the Wikiquote articles structure can change.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1718 ms

Resolved: Bofrost.at performance has improved in f50041d after 36 minutes.


aiohttp.web

Version
4
Traceback (most recent call last):
File "main.py", line 77, in
async def messages(req: web.web_request) -> web.Response:
AttributeError: module 'aiohttp.web' has no attribute 'web_request'
[bug]

tried multiple aiohttp versions and the only one working is aiohttp==3.1

@cyberf4n
Aiohttp removed 'web.web_request' from their releases after version 3.5.0 which the Python Samples use in the messaging endpoint functions. If you want to use version 3.5.0 or greater of Aiohttp, you must change the type of the req parameter in the messages function form 'web.web_request' to 'web.BaseRequest' or remove it altogether. This is how your messages function should look.
async def messages(req: web.BaseRequest) -> web.Response:
    body = await req.json()
    activity = Activity().deserialize(body)
    auth_header = req.headers['Authorization'] if 'Authorization' in req.headers else ''
    try:
        return await ADAPTER.process_activity(activity, auth_header, bot.on_turn)
    except Exception as e:
        raise e

This is how I have my requirements.txt file configured.
<EMAIL_ADDRESS>botbuilder-core>=4.0.0.a6
aiohttp==3.5.3

Hope this helps!


removed vulnerabilities

Before creating a pull request make sure that:
JIRA link (if applicable)
Remove vulnerabilities
Change description
removed vulnerabilities from suppressions
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X ] No

merged already


Bump Version 0.1.0

Description

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

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

Comparison is base (34830a7) 80.33% compared to head (f6d756e) 80.41%.
Report is 1263 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1463      +/-   ##
==========================================
+ Coverage   80.33%   80.41%   +0.07%
==========================================
  Files          95      103       +8
  Lines        6602     7261     +659
==========================================
+ Hits         5304     5839     +535
- Misses       1298     1422     +124

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


Color conversion handled through command line

Bare with me, Gopher newbie here ;P So I added the ability to toggle between the CIE and Euclidean algorithm with the command line flag -c. Additionally I ported my algorithm from C to Go which amplifies the strongest color and in many cases produces a better result. I recommend only using a threshold value of 2 or 3 here.
Examples:
pngs2cel -o gates.cel -pal_path town.pal cutgate.png // Euclidean
pngs2cel -c -o gates.cel -pal_path town.pal cutgate.png // CIE 2000
pngs2cel -t 2 -o gates.cel -pal_path town.pal cutgate.png // Euclidean multiplier of 2

Alright that should do it, @mewmew how do things look now? I tidied it up and also fixed the typo colour ;)


"degree Celsius" is a unit that has a space in it

Talking to Kay, she said this was not intended and it should be a - inbetween.

Addressed in PR#149


Worker-Main died: java.lang.IllegalArgumentException: Name and ID cannot both be blank

Since putting our 1.17 Fabric SMP behind a Velocity proxy, running FabricProxy-Lite-1.1.5.jar we are seeing these errors in the log files..
We are also running voicechat mod, which bypasses the proxy (TCP connection on a different port), which could be a factor.
Sometimes after a user logs in (but not every time)
[12:05:43] [Server thread/INFO]: Tatsu_Del[/<IP_ADDRESS>:58728] logged in with entity id 431 at<PHONE_NUMBER>785099, 63.0,<PHONE_NUMBER>1192093)
[12:05:43] [Server thread/WARN]: Player Tatsu_Del can't receive direct packet, using nbt packet instead
[12:05:43] [Server thread/INFO]: Tatsu_Del joined the game
[12:05:48] [Worker-Main-22/WARN]: Worker-Main-22 died
java.lang.IllegalArgumentException: Name and ID cannot both be blank
        at com.mojang.authlib.GameProfile.<init>(GameProfile.java:26) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillGameProfile(YggdrasilMinecraftSessionService.java:199) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillProfileProperties(YggdrasilMinecraftSessionService.java:186) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37469(class_2631.java:144) ~[server-intermediary.jar:?]
        at net.minecraft.class_156.method_17974(class_156.java:448) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37466(class_2631.java:139) ~[server-intermediary.jar:?]
        at java.util.concurrent.ForkJoinTask$RunnableExecuteAction.exec(ForkJoinTask.java:1434) ~[?:?]
        at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:295) ~[?:?]
        at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1016) ~[?:?]
        at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1665) ~[?:?]
        at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1598) ~[?:?]
        at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183) [?:?]
[12:05:48] [Worker-Main-22/ERROR]: Caught exception in thread Thread[Worker-Main-22,5,main]
java.lang.IllegalArgumentException: Name and ID cannot both be blank
        at com.mojang.authlib.GameProfile.<init>(GameProfile.java:26) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillGameProfile(YggdrasilMinecraftSessionService.java:199) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillProfileProperties(YggdrasilMinecraftSessionService.java:186) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37469(class_2631.java:144) ~[server-intermediary.jar:?]
        at net.minecraft.class_156.method_17974(class_156.java:448) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37466(class_2631.java:139) ~[server-intermediary.jar:?]
        at java.util.concurrent.ForkJoinTask$RunnableExecuteAction.exec(ForkJoinTask.java:1434) ~[?:?]
        at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:295) ~[?:?]
        at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1016) ~[?:?]
        at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1665) ~[?:?]
        at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1598) ~[?:?]
        at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183) ~[?:?]

Sometimes just happens any time:
[12:29:33] [Worker-Main-24/WARN]: Worker-Main-24 died
java.lang.IllegalArgumentException: Name and ID cannot both be blank
        at com.mojang.authlib.GameProfile.<init>(GameProfile.java:26) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillGameProfile(YggdrasilMinecraftSessionService.java:199) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillProfileProperties(YggdrasilMinecraftSessionService.java:186) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37469(class_2631.java:144) ~[server-intermediary.jar:?]
        at net.minecraft.class_156.method_17974(class_156.java:448) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37466(class_2631.java:139) ~[server-intermediary.jar:?]
        at java.util.concurrent.ForkJoinTask$RunnableExecuteAction.exec(ForkJoinTask.java:1434) ~[?:?]
        at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:295) ~[?:?]
        at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1016) ~[?:?]
        at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1665) ~[?:?]
        at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1598) ~[?:?]
        at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183) [?:?]
[12:29:33] [Worker-Main-24/ERROR]: Caught exception in thread Thread[Worker-Main-24,5,main]
java.lang.IllegalArgumentException: Name and ID cannot both be blank
        at com.mojang.authlib.GameProfile.<init>(GameProfile.java:26) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillGameProfile(YggdrasilMinecraftSessionService.java:199) ~[server-intermediary.jar:?]
        at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillProfileProperties(YggdrasilMinecraftSessionService.java:186) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37469(class_2631.java:144) ~[server-intermediary.jar:?]
        at net.minecraft.class_156.method_17974(class_156.java:448) ~[server-intermediary.jar:?]
        at net.minecraft.class_2631.method_37466(class_2631.java:139) ~[server-intermediary.jar:?]
        at java.util.concurrent.ForkJoinTask$RunnableExecuteAction.exec(ForkJoinTask.java:1434) ~[?:?]
        at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:295) ~[?:?]
        at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1016) ~[?:?]
        at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1665) ~[?:?]
        at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1598) ~[?:?]
        at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:183) ~[?:?]

I don't think FabricProxy-Lite will cause this issue, please check if other mods are trying to get player head with blank name.

Ah, do you think it's related to player heads?  We do have player head drops datapack, and we also have Bedrock clients joining via Geyser on the proxy.  Maybe the Bedrock player heads are the problem...

I've done some further testing on the server and have narrowed the problem down to player heads from Bedrock clients which had been placed as blocks.  The error occurs when connecting to the server while in the area of the placed heads, or when moving into chunk-loading range of the heads.
So confirmed that it is not related to FabricProxy-Lite.
Thanks @james58899 for pointing me in the right direction.


Add repository to ViewModel issue

URL of codelab:
https://developer.android.com/codelabs/basic-android-kotlin-compose-add-repository?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-5-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-add-repository#5
Specify the language of the codelab if it is not English:
In which task and step of the codelab can this issue be found?
6.8 - Update the code to pass variable viewModel as an argument to the MarsPhotosApp composable.
Describe the problem
The error below is given when the viewModel is passed as an argument to the MarsPhotosApp composable.
Type mismatch.
Required:
Modifier
Found:
MarsViewModel
The solution code also doesn't pass the viewModel as an argument.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version: Eel
API version of the emulator: NA
Additional information
Include screenshots if they would be useful in clarifying the problem.

Duplicate of #29, #25, #20.


Fix NPM WARN message about repo in package.json

When you run npm install on this repo right now, you usually get CLI feedback from NPM that says:
npm WARN<EMAIL_ADDRESS>No repository field.

Just defining the repository in the package.json fixes that. Minor thing but why not?

I think that is intentional. You'd end up overwriting that field anyway (or at least you should) when you push your version of the site using this starter.

Yeah, gatsby new deletes the .git directory to split you off from this repo so it doesn't make much sense to say in the package.json the site is still associated with this repo.
Thanks for working with us to improve things!


Update Graph API Permission request

Added 'Application.Read.All' to the scope of the Graph API call.
For Azure AD tenants that have not used 'Microsoft Graph Powershell' before the Get-AppIds command within Set-APIPermissions function will fail with an 'Insufficient permissions' error, as the Graph is unable to read App IDs with just 'AppRoleAssignment.ReadWrite.All' alone

Thanks @Andrew-Bennett! I created #358 for tracking. I will add some content to the PR (documentation, and other potentially other error mgmt).


change start year UI?

When someone changes the start year, a small text notice is displayed:

I believe some users may not see this and are then surprised when the tables start from the default start year instead of the start year they wanted to set.
@talumbau and @feenberg , what do you think about presenting a pop up that takes over the screen instead? It would need to say something like: "Changing the default start year will clear any tax law inputs you have made."..."Ok, go for it" or  "No, I want to keep the current start year".

I don't think this would be difficult for @mistakevin. Assigning to him.

On Fri, 8 Jul 2016, Matt Jensen wrote:

When someone changes the start year, a small text notice is displayed:
image
I believe some users may not see this and are then surprised when the tables start from the default
start year instead of the start year they wanted to set.

OK, pretty subtle.
The popup-box should block further modifications to the main screen and
should indicate that the default tax law parameters will change to the
indicated year.
dan

@talumbau and @feenberg , what do you think about presenting a pop up that takes over the screen
instead? It would need to say something like: "Changing the default start year will clear any tax law
inputs you have made."..."Ok, go for it" or "No, I want to keep the current start year".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the
thread.[AHvQVWRnV4LOK3PnOux_S4sIKdpf0cJtks5qTmgYgaJpZM4JIHsq.gif]

Can do. Shouldn't be too hard.

This feature is not currently working on TaxBrain.
From @feenberg via email:

In taxbrain I have been trying again to start the analysis in 2013, but the application ignores my request. There used to be a prompt when you selected 2013 - as far as I can tell the prompt is gone (good) but the analysis still starts in 2016. See
http://www.ospc.org/taxbrain/edit/4870
Interestingly if you select "edit parameters" the URL changes to add year=2016 as a parameter. I tried changing the URL to 2013, didn't affect anything.
thanks
dan

I have confirmed.

@MattHJensen @talumbau  It looks like the current deployed taxbrain.js is not the newest version

Looks like it's still loading the old static assets. I will try to upload the new ones and update this issue.

There must be something I don't understand about collectstatic. For this Django app, if DEV_DEBUG is set to False, the settings.py is set to not use our static assets. In particular, one thing it does is that the command python manage.py collectstatic collects the static assets in static/ and copies them to STATIC_ROOT which is staticfiles/. However, when I do that and then diff the directories static/ and staticfiles/, they are quite different. So, that is bad because my assumption was always that I would get the complete set of files there.
The procedure would then be (assuming we have everything in staticfiles/) to execute the steps in our wiki entry on how to GZIP and deploy these static assets.
But that will not produce desired results if staticfiles/ is wrong. So, before doing some kind of manual fix, maybe @mistakevin or @brendancol could explain where I'm doing wrong in what should happen with collectstatic and why there are difference between static/ and staticfiles/ after running it. Any ideas?

this might be a reasonable answer:
http://stackoverflow.com/questions/17507843/django-storages-not-detecting-changed-static-files
if the timestamp if the newer file does not indicate that it is actually newer (which can happen through git pulls because I neglected to do the collecstatic operation in my branch) then it looks like I have to pass collecstatic --clear to make sure it actually updates the file in staticfiles/. I'll experiment with this on a test bucket in S3.

This functionality is now restored. @mistakevin it would seem useful if there was some kind of "hook" that would either trigger on demand or when a PR is merged to master that would do as follows:

with DEBUG=False, run python manage.py collectstatic --clear so that all static content goes in to staticfiles/
execute some commands to gzip the CSS and JS files as described in the wiki entry
upload the resulting directory to the appropriate S3 bucket.

If this happened each time, we would never have to remember to do it. Is that feasible and/or a common solution to this problem?

Thanks. I let Dan know.

@talumbau Yep, that is usually done by the CI technology, TravisCI or CircleCI. And Heroku can actually listen for Githooks as well, so a full deploy could happen with merge into Master.


Request to add DataGrid style

I just want to tell you that material.avalonia project is really great.
But I found that the style of DataGrid has not been defined yet.
https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Controls.DataGrid/Themes/Default.xaml

DataGrid style are still WIP (working in progress). It looks like this now:

DataGrid with other controls (buttons, checkbox etc.) is still not tested. So I don't know how it looks like, but we will know later.

DataGrid style are still WIP (working in progress). It looks like this now:

DataGrid with other controls (buttons, checkbox etc.) is still not tested. So I don't know how it looks like, but we will know later.

Seems like this is working.


Enhance control on links appearance - support dashed links

Is your feature request related to a problem? Please describe.
The link configuration allows to customize just its color, opacity and width. It would be cool to have more control on the appearance of the links, like being able to draw dashed or dotted links.
Describe the solution you'd like
It's just a matter of allowing the Link component to accept supplementary svg presentation attributes and pass them to the generated <path> element as already doing with the already supported ones.
Describe your use case
In my specific case i need to draw dashed links.
Additional context
Most of the SVG presentation attributes would make no sense for the link (like fill), but i think that supporting stroke-dasharray, stroke-dashoffset and stroke-linecap would make perfectly sense. stroke-linejoin, stroke-miterlimit and others would make sense if the edges could be more complex than a single line.

Thanks, that's a nice idea! I dropped a few comments in the PR. Once you address them we can move this forward. Let me know if you need any help.

@aSlug are you planning on making the changes to this PR? I'm looking for exactly this behavior. Thanks for the contribution!

unfortunately i'm not able to follow this issue in the immediate future, feel free to fork the current changes and carry on the pr.

Thanks for the heads up @aSlug @lironhl followed up with a PR for this one.

Closed with https://github.com/danielcaldas/react-d3-graph/pull/344


Update: Hints

Codecov Report

Merging #311 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #311   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         122    122
  Lines        1268   1268
=====================================
  Hits         1268   1268

Continue to review full report at Codecov.

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


Sync Fork from Upstream Repo

[Describe your pull request here. Please read the text below the line, and make sure you follow the checklist.]

Pull request checklist
Read the Contribution Guidelines for detailed information.

[ ]  Changes are described in the pull request, or an existing issue is referenced.
[ ]  The test suite compiles and runs without error.
[ ]  Code coverage is 100%. Test cases can be added by editing the test suite.
[ ]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for these kind of bugs). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

74 of 74 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 98.649%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
5552

Relevant Lines:
5552

💛  - Coveralls


Question: Property and Operation Delegation

I found are very interesting use case here: https://eclipse.dev/basyx/asset_integration/
Question: Is this already working? If yes, can one ellaborate a bit more on the answer here:

But how can this integration be implemented via property delegation with Eclipse BaSyx? The good news: quite simple - without a single line of code.

When I short-path to the respective property, I only get back @(delegatedTo ...) as the value not the value read from the given endpoint.
Would also be great to know how it is done for Operation Delegation.
Thx ;-)

After having corrected my property to:
  {
      "idShort": "live",
      "qualifiers": [
          {
              "type": "delegatedTo",
              "valueType": "xs:string",
              "value": "http://localhost:8080/aidrunner/properties/count"
          }
      ],
      "valueType": "xs:integer",
      "value": "",
      "modelType": "Property"
  }

I get back an empty value. Maybe connection issues - will check.

Connectivity tested via docker exec -it CONTAINERID timeout 3 /bin/bash -c "echo > /dev/tcp/IPADDRESS/8080 >& /dev/null" && echo "ok" || echo "nok" seems to be working - however, still no value. Any ideas?

This is not implemented for V2 as of now. However, we will implement it in the coming weeks.
I will keep this ticket open and update it whenever the feature is implemented.

Operation Delegation is now implemented and will be included in the next release (i.e., next week).
Property Delegation, however, we're not planning to implement as of now. The reason is that it does not integrate well with the other architecture concepts of BaSyx (e.g., eventing). Since the value is only retrieved on-demand, features like eventing will not work as expected. I would advise for these scenarios to use the BaSyx DataBridge.
Do you have any scenario in mind where the DataBridge would not work but Property Delegation does?

Thanks for the efforts and the update!
I think it is fine to use a gateway (e.g., DataBridge or node-wot) to take care of the data collection and update of the respective Submodels or Submodel Elements. Eventing is anyhow preferred over cyclic polling.
Once the feature is pushed to main, it would be great if you could also provide a sample AAS to see both Operation Delegation as well as Eventing in action.
Another approach towards data collection in the AAS was recently prototyped in a AASX Package Explorer branch via AID / AIMC Submodels. Maybe both approaches can be streamlined.

Once the feature is pushed to main, it would be great if you could also provide a sample AAS to see both Operation Delegation as well as Eventing in action.

We're planning on providing these examples. I will send an update on the basyx-dev mailing list whenever they are available.

Another approach towards data collection in the AAS was recently prototyped in a AASX Package Explorer branch via AID / AIMC Submodels. Maybe both approaches can be streamlined.

We would like to integrate AID / AIMC with the BaSyx DataBridge in a first step. Additionally, configurating operation delegation via these submodels is also a very interesting topic. However, for this there is no concrete ETA at the moment.


Unexpected behavior in Array functions when pflag.Parse() is called multiple times

Using the following program,
func main() {
	arrayVar := []string{}
	pflag.StringArrayVar(&arrayVar, "flag", []string{}, "")
	pflag.Parse()
	pflag.Parse()

	for _, item := range arrayVar {
		fmt.Println(item)
	}
}

With the following input,
./main --flag a --flag b --flag c
Will produce the output:
a
b
c
a
b
c

I would instead expect this is only write values to the array if the target array is empty.

I just ran into this same issue today. In my particular case, I'm intentionally re-parsing as new information comes in to check for certain args supplied on the command line that are dynamically determined from other parts of the application, so the only obvious workarounds are either  creating a new flagset before re-parsing, or storing the results of parsed arrays elsewhere (though presumably future parsing could still cause these variables to grow without bound).
Is this expected behavior?

:+1: I was doing the same thing

I've worked-around the issue by parsing it in a different flag set, then adding that flag to the original one:
flgs := flag.NewFlagSet("temp", flag.ContinueOnError)
flgs.ParseErrorsWhitelist.UnknownFlags = true
flgs.StringArray(key, nil, usage)

if err := flgs.Parse(ca.Args); err != nil {
	return err
}

f = flgs.Lookup(key)
mainFlags.AddFlag(f)

Looking at the implementation of StringArray, I see the issue: Set has no way of knowing it's being called during a different Parsing instance, so it always appends the new value. At first I thought the changed check were superfluous, but it's there to ensure the default value is overwritten by the first parsed value. One "fix" is to set it to false before parsing, but if the input args are different between attempts, the slice would need to be reset to the original default. Regardless, it would be a behavior change, so the library would need either a method or a FlagSet option to enable it.
Another potential workaround is to create a custom type and manually reset it before re-parsing. Alternatively, you could use the existing type and just retain a collection of the underlying *[]string and reset them yourself -- in either case, you wouldn't need to create a new FlagSet just to parse the one flag, and if parsing behavior is dependent on the context of other nearby flags (setting aside that this is the reason you're supposed to set up flags ahead of time), that context would be available. The same logic would be required for any array type flags you're using.


i wonder is there okhttp-logging-interceptor proguard rules?

i wonder is there okhttp-logging-interceptor proguard rules? thx

No.


how to InvokeService in k8s?

dapr install in namespace myns, appid=myapp, and http handle router /test
forward port on local:
kubectl port-forward svc/myapp 50001:50001 3500:3500 8080:8080 -n myns

invoke in go sdk:
 res, err := client.InvokeService(ctx, "myapp.myns", "test")

this result is :  error invoking service: rpc error: code = NotFound desc = Not Found
browser open or call it use curl: http://localhost:3500/v1.0/invoke/myapp.myns/method/test  it has correct result
ps: Is there any other way to call the service locally

deploy 2 app in k8s.

app1: /test
app2: /test2 and use sdk invoke app1 /test
res, err := client.InvokeService(ctx, "myapp.myns", "test")

forward app2 port on local:
 kubectl port-forward svc/myapp2 50001:50001 3500:3500 8080:8080 -n myns

call app2/test2 use curl:
ystyle@Archlinux  ~  curl -v <IP_ADDRESS>:8080/test2
*   Trying <IP_ADDRESS>:8080...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8080 (#0)
> GET /test2 HTTP/1.1
> Host: <IP_ADDRESS>:8080
> User-Agent: curl/7.73.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Content-Type: application/json; charset=utf-8
< Set-Cookie: go_session_id=MGU1OTEzYTEtNmEzOC00NzUwLWE0YmUtMTJmZjE4ODU1ZTU2.a8b2c31a9c5286837d560cb3ac3bd85fdc6371b4; Path=/; Expires=Wed, 16 Dec 2020 03:48:01 GMT; Max-Age=604800; HttpOnly
< Date: Wed, 09 Dec 2020 03:48:01 GMT
< Content-Length: 87
<
* Connection #0 to host <IP_ADDRESS> left intact
{"code":-1,"msg":"error invoking service: rpc error: code = NotFound desc = Not Found"}%                                                                          ystyle@Archlinux  ~ 

go-sdk can't set  HttpExtension
https://github.com/dapr/java-sdk/blob/master/examples/src/main/java/io/dapr/examples/invoke/http/InvokeClient.java#L35-L37

@ystyle can you elaborate more on the actual issue here? Mainly, are these 3 separate issues or all three are part of this one issue?

@ystyle can you elaborate more on the actual issue here? Mainly, are these 3 separate issues or all three are part of this one issue?

The problem is caused by the absence of httpExtension, and the existing restful interface cannot be rewritten
gosdk default http method is  POST, and can not set to other

Got it, so both InvokeServiceWithContent and InvokeService defaulting to POST does not allow you to set other verbs on the request. OK, I changed the title of this issue to "Parameterize request method verb on service invocation" and will classify it as a bug.
Message: &v1.InvokeRequest{
   Method: method,
   HttpExtension: &v1.HTTPExtension{
      Verb: v1.HTTPExtension_POST, // parameterize
   },
}

Got it, so both InvokeServiceWithContent and InvokeService defaulting to POST does not allow you to set other verbs on the request. OK, I changed the title of this issue to "Parameterize request method verb on service invocation" and will classify it as a bug.
Message: &v1.InvokeRequest{
   Method: method,
   HttpExtension: &v1.HTTPExtension{
      Verb: v1.HTTPExtension_POST, // parameterize
   },
}

I tested it and it is available after adding parameters

@ystyle PR under review, once lands you will have to import from HEAD to use it until we make a new release.


update the list of available extension

Description:
Updated the list of available extension

Probably we could remove or rewrite "This Scaffold-ETH 2 extension", since all of them are SE-2 extensions.
Fine for now! We can have someone to help.
I've also started thinking if we should add a section into the SE-2 docs at some point


Upgrade to .NET 6 RC2

Fix compilation errors:

Expose internals of DistributedResponseCachingMiddleware to DistributedResponseCachingMiddlewareTests. This fixes #5.
Consider new namespace of CachedResponse

Upgradef to NET 6 RC2:

Set target framework to net6.0
Updated nuget packages to 6.0 or latest
Set Version to 6.0.0-rc2

Bugfixes:

Register ResponseCachingPolicyProvider as IResponseCachingPolicyProvider when CacheAuthorizedRequest is not enabled, to fix error when resolving for IResponseCachingPolicyProvider
Fix configuration of ResponseCachingOptions by configuring it explicitly. By just registering the CustomResponseCachingOptions, the options of the base objects would get lost when resolving for IOptions.

This looks great, thank you!
Merged, will try to update the nuget package later today.
Would you be able to update the documentation (Readme) also?
Also, I'm not using this package; it would be better if it was maintained by someone who's using it.
Would you be interested in being added as maintainer?
I could even see about having you able to upload builds to nuget.

Thanks for the offer, I gladly would be maintainer of this project. This would make it much easier for us to make future changes if necessary.


Remote caching disabled since turbo 1.9.4

🐛 Bug Report
Remote caching disabled since turbo 1.9.4
To Reproduce
Upgrade to >1.9.3 version
Expected behavior
Get Remote caching enabled
Since upgrade turborepo to 1.9.4 no way to get remove caching enable, someone have a fix or can reproduce the problem? thanks

Are you getting any errors from the client or server?  You can run turbo in a more verbose mode to get it to log information about when it tries to cache.

It seems turborepo removed support for .turbo/config.json
I had to delete that file and set the needed properties per script in the root package.json e.g. turbo -v --team=team_example  --token=ExampleToken --api=https://example.remote.cache.com build

Oh interesting.  You can also use environment variables, that's what we do.  TURBO_API, TURBO_TEAMID, TURBO_TOKEN.

We updated our docs to reflect this change a while ago. You can find the new version on the website https://ducktors.github.io/turborepo-remote-cache/custom-remote-caching


KARAF-5496 - NPEs in SyncopeLoginModule if "version" is not specified

By the way, the build fails on EventLoggerTest. I will fix that. No need to do the rebase (I will test on another PR).


Correct the principle username detection for OAuth 2.0 Client Credentials grant

Expected Behavior

When the /api/offenders/{offenderNo} endpoint is called, an OFFENDER_BOOKINGS record should be created in persistent storage.
Current Behavior

The system fails to create the OFFENDER_BOOKINGS record (full stack trace)...

...

org.springframework.dao.InvalidDataAccessApiUsageException: The given id must not be null!; nested exception is java.lang.IllegalArgumentException: The given id must not be null!

...

Caused by: java.lang.IllegalArgumentException: The given id must not be null!
        at org.springframework.util.Assert.notNull(Assert.java:201)
        at org.springframework.data.jpa.repository.support.SimpleJpaRepository.findById(SimpleJpaRepository.java:297)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:78)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:567)
        at org.springframework.data.repository.core.support.RepositoryMethodInvoker$RepositoryFragmentMethodInvoker.lambda$new$0(RepositoryMethodInvoker.java:289)
        at org.springframework.data.repository.core.support.RepositoryMethodInvoker.doInvoke(RepositoryMethodInvoker.java:137)
        at org.springframework.data.repository.core.support.RepositoryMethodInvoker.invoke(RepositoryMethodInvoker.java:121)
        at org.springframework.data.repository.core.support.RepositoryComposition$RepositoryFragments.invoke(RepositoryComposition.java:529)
        at org.springframework.data.repository.core.support.RepositoryComposition.invoke(RepositoryComposition.java:285)
        at org.springframework.data.repository.core.support.RepositoryFactorySupport$ImplementationMethodExecutionInterceptor.invoke(RepositoryFactorySupport.java:599)
        at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
        at org.springframework.data.repository.core.support.QueryExecutorMethodInterceptor.doInvoke(QueryExecutorMethodInterceptor.java<===========--> 87% EXECUTING [5h 31m 35s]
        at org.springframework.data.repository.core.support.QueryExecutorMethodInterceptor.invoke(QueryExecutorMethodInterceptor.java:13
...

Context

I am an external developer of a non-DPS system. My application is specified to consume API endpoints exposed by this project. My application will be authorized by HMPPS Auth using the OAuth 2.0 Client Credentials grant.
While stepping through the debugger during investigation of a separate, unrelated issue, I had observed that Spring Security refers to the security principle as "AnonymousUser" during the OAuth 2.0 Client Credentials grant flow.
Assuming that an "AnonymousUser" would not have a user name, then the following two lines are reasonable suspects for the root cause of the reported "The given id must not be null" error...

(username = ((UserDetails) userPrincipal).getUsername();)
staffUserAccountRepository.findById(currentUsername)

Steps to Reproduce

Call the /api/offenders/{offenderNo} endpoint with a JWT access token issued by HMPPS Auth for an OAuth 2.0 Client Credentials grant.

Your Environment

Windows 10
Docker Desktop Version 3.3.3 (64133)
IntelliJ IDEA 2021.1.2 (Community Edition)

Build #IC-211.7442.40

This one and these other two...

_Fix o.s.d.DataIntegrityViolationException in /api/offenders/
"Fix the build/config so that running locally on Windows 'Just Works'

...Will be what I would like to pick your brains on @petergphillips @andymarke, whenever we get a chance to talk off line.

For endpoints that create data you need to provide an extra parameter when you create the client credentials of username=<valid nomis user>.

Thank you @petergphillips,

"...For endpoints that create data you need to provide an extra parameter when you create the client credentials of username=<valid nomis user>..."

I will give that a shot in a few.
I guess that's an example of what they call "Implicit Knowledge" (aka, "Tribal Knowledge"). Right? Because I haven't seen that documented anywhere. And I've read a ton of stuff so far about several different HMPPS/MOJ/DPS projects.
Please correct me if I'm wrong though. Please point me to the documentation of that, that I overlooked? TIA.

"...<valid nomis user>..."

I will proceed on the assumption that "ITAG_USER" is one such <valid nomis user>.
Or can you share any shortcuts to getting a hold of a known <valid nomis user> @petergphillips ? It might be valuable to share that here; on the off chance that other future non-DPS devs stumble across this same issue. TIA.

"...<valid nomis user>..."
I will proceed on the assumption that "ITAG_USER" is one such <valid nomis user>...

Dear "other future non-DPS devs",
I am pleased to report that "ITAG_USER" is, indeed, one such <valid nomis user>  :)
I confirmed that, @petergphillips, with two curl calls from the command line...

Appending username=<valid nomis user> to the token request to hmpps-auth
Calling the /api/offenders/{offenderId}/booking endpoint with that access token

Now that leaves only the Spring Security OAuth 2.0 client credentials configuration so that it does those two things on my behalf, under the covers. Right @petergphillips?
If you could share any shortcuts for (or even better, links to documentation of) that configuration, that'd be so super deluxe! TIA.

...Now that leaves only the Spring Security OAuth 2.0 client credentials configuration so that it does those two things on my behalf, under the covers. Right @petergphillips?...

The Customizing the Access Token Request section of the Spring Security OAuth 2.0 reference documentation offers some pretty decent guidance.
I'm reasonably certain as far as the "What" needs to be customized goes. My first stab at the specifics (i.e., the "How") still needs a little more work though.
In the meantime, any links to hints, tips or suggestions are welcome @petergphillips. TIA.

...
...My first stab at the specifics (i.e., the "How") still needs a little more work though...

I've worked out a now-working configuration @petergphillips. I found the missing piece of the puzzle in the Spring Security JavaDoc...

...
Use OAuth2AuthorizedClientArgumentResolver(OAuth2AuthorizedClientManager) instead. Create an instance of ClientCredentialsOAuth2AuthorizedClientProvider configured with a DefaultClientCredentialsTokenResponseClient (or a custom one) and than supply it to DefaultOAuth2AuthorizedClientManager
...

The Spring Security OAuth 2.0 configuration I now have in place, now programmatically does the needful "...provide an extra parameter when you create the client credentials of username=<valid nomis user>" step.
Thanks again for sharing that username=<valid nomis user> clue 👍

Sorry for the delay.  An example of where we configure the web client to add the username can be found in WebClientConfiguration and the code for the converter is CustomOAuth2ClientCredentialsGrantRequestEntityConverter