feat: add hmmm log level to onyx

Details
Added hmmm logger to Onyx.
Fixed Issues
$ https://github.com/Expensify/react-native-onyx/pull/485#discussion_r1541351332
PROPOSAL: N/A
Tests
N/A

[ ] Verify that no errors appear in the JS console

Offline tests
N/A
QA Steps
N/A

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@kirillzyusko thanks for your PR
TSC checks are failing

@lakchote it's kind of expected, because this PR relies on Expensify/react-native-onyx#524
I think the correct action would be to merge PR in Onyx, then publish a new version, and then use a new version here and merge this PR. Am I right?

Exactly, that'd be the way to go 👍

@lakchote I bumped, but it looks like latest version of onyx contains a lot of breaking changes in TS (so typescript check stage will fail anyway).
@marcaaron Is it a known issue? 👀

Hmmm... not sure about the TS checks to be honest. Maybe we can ask in the Slack channel.
There was a problem with Onyx not automatically deploying a new package version. I believe it was resolved, but maybe that is related. @roryabraham I think we just need to bump to the latest, but use the commit from here:
https://github.com/Expensify/react-native-onyx/pull/522#issuecomment-2023258465

Seems like after we merge: https://github.com/Expensify/App/pull/38114
Then you can remove the Onyx bump.
cc @mountiny - we are bumping this coming from this PR too I think. So, lots of Onyx changes in one release. Don't lose your hat !!! 🤠

Thank you for the bump! The process now is to do a full regression testing on web and some native platform by QA on any new onyx bump. Given we havent been able to bump in a while there is a backlog and we are adding 23 version here
Regression completed there and we need to resolve/ confirm if 3 issues are related to the onyx changes or not. Its looking promising but lets wait for that PR to make it to production in case we would need to make any revert/ hotfixes in Onyx.
Also noting that the TS errors been fixed on that PR, jest units still failing I think

What's the latest here? This looks ready to merge to me apart from the TS checks.

What's the latest here? This looks ready to merge to me apart from the TS checks.

I'd like the TS checks to pass, you'd like to proceed without it?

@marcaaron @lakchote I fixed TS errors 👍 Would love to see it being merged 😊

Thank you both @marcaaron @kirillzyusko for your involvement in this!

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

I'd like the TS checks to pass, you'd like to proceed without it?

LOL, no, but I can see the confusion there. Just asking for the status update 😄
And now we are ready for merge 😂

🚀 Deployed to production by https://github.com/marcaaron in version: 1.4.70-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[DOC] Anomaly detection getting started notebook

Describe the issue linked to the documentation
Aeon is still missing the anomaly detection getting started notebook (linked from the home page): https://www.aeon-toolkit.org/en/stable/examples/anomaly_detection/anomaly_detection.html
PR #1932 introduced some general documentation in the API documentation that needs to be moved to this notebook as well.
Suggest a potential alternative/fix
No response

Hi! I'd like to work on this.

You are welcome to do so. If you require any further assistance, please let us know here or in Slack.

I already created an example notebook for one of our tutorials. It can be used as a starting point: https://github.com/aeon-toolkit/aeon-tutorials/blob/main/ECML-2024/Notebooks/part6_anomaly_detection.ipynb


Listen Category missing from Settings > Categories list when turned off

Description:
There are 2 flows for turning off listening mode:

Settings > 'Listening Mode' setting; OR
Settings > 'Categories and Phrases' > edit the 'Listen' category.

When using flow 2 the category remains in the list and behaves as any other category would when hidden. However, when using flow 1 the category is removed from the list of categories. The 'Listen' category should remain in the list of categories even when turned off from within the Settings > 'Listening Mode' setting.
Steps to Reproduce:

From the main screen, navigate to Settings > 'Listening Mode'.
Tap on the 'Listening Mode' toggle button to turn the feature OFF.
Return to Settings
Tap 'Categories and Phrases'
Note that the 'Listen' category is not in the list.

Expected behavior:
After turning off Listen Mode from within Settings > 'Listen Mode', a user should still be able to re-enable listening mode from Settings > Categories and Phrases > edit 'Listen' category.
Actual behavior:
After turning off Listen Mode from within Settings > 'Listen Mode' the 'Listen' category is removed from the list of categories.

Device Information:
Latest build on develop as of 05.10.2022

@SashankPatel
Tagging you hear to help make you aware that I created this bug ticket.

@SashankPatel
I just realized that the 'Listen' category would also be removed from the list once the user exists the 'Categories' screen. Now I understand this to be expected behavior, but wonder if it is still a more pleasant user experience to keep the 'Listen' category listed. It can be a jarring experience to turn it off from the Categories screen one minute, then it go "missing" the next time you look for it, since you're "conditioned" to edit categories from the list screen and only tapping "Remove Category" does the same behavior for the others. 🤔 🤔

Yeah so when you toggle it off, the category should be removed from the main screen, but I think it should still be visible in Settings so you can turn it back on lol. It should basically act like the Show Category toggle. Its kinda weird that the setting is in two places. Maybe we remove the category settings options from the edit view?

I agree that it should act like the 'Show Category' toggle. As far as two places, I don't mind that actually. But since it's faster for head tracking to go into Settings > Listening Mode it makes sense to me to remove 'Listen' from the list of categories altogether. So I am down for reducing to one flow; being just Settings > Listening Mode.

This has been tested and verified. It works like a charm 🌟  🤩  🌟
Tested on iPhone 8 Plus (iOS 14.8.1) and an iPhone X (iOS 14.2)


Simplify usage outside build script

Currently cc heavily relies on environment variables to determine default host, target and other compiler params. This makes it hard to use outside build script as part of normal runtime code, especially because Cargo provides such env variables exclusively to build scripts and not even to env! macro in regular code.
In fact, the only reasonable workaround I found is to use a build script to generate a Rust file containing all needed env vars as constants and then to include that back in a file calling cc::Builder::new().
This works, but a bit cumbersome. So, I wondered if cc would be open to including same workaround inside its own build script so that all defaults would "just work" in any callsite, not just in other build scripts.

I'm going to close this since it's not a goal for this crate but PRs are of course always welcome if someone has suggestions for how to support this!


Set up Radio buttons.

Fixes #...
Changed the type of input to "radio".
Deleted the related javascript and styling files.
Changed the name of the filter from "BooleanButtonSelect," to "BooleanFilterSelect".
Please check the following:

[ ] Do the tests still pass?[^1]
[ ] Does the code comply with the style guide?

[ ] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

@AbhinavOhri please can you read through the CI failures and fix up any issues. Also please update the description of the PR to include a reference to the issue and some screenshots of what's changed - before and after ideally.
Then you can mark  the PR as ready and add a comment that it's ready.
Additionally, in future, please read through the issue and be considerate of others that have already started on it. Maybe it's better to help those that had questions and work together.
We have a helpful guide here as an issue.
https://github.com/wagtail/wagtail/issues/10070

@lb- I opened this PR to get some reviews about my work as I was not sure whether I had done it in the right manner.  Also, I started the work on this issue before the other person's comment. I will take care next time for sure.

Thanks @AbhinavOhri I'll close this now as the other PR has now been merged.
https://github.com/wagtail/wagtail/pull/10123


oracle jdk url is no longer valid

The remote server returned an error: (302) Moved Temporarily.
URL http://download.oracle.com/otn-pub/java/jdk/8u171-b11/512cd62ec5174c3487ac17c61aaa89e8/jdk-8u171-windows-x64.exe#/dl.7z is not valid

I have no issues installing Oracle 8 JDK:
C:> scoop install oraclejdk8
Installing 'oraclejdk8' (8u171-b11) [64bit]
jdk-8u171-windows-x64.exe (207,3 MB) [========================================================================] 100%
Checking hash of jdk-8u171-windows-x64.exe... ok.
Extracting... done.
Running pre-install script...
Linking ~\scoop\apps\oraclejdk8\current => ~\scoop\apps\oraclejdk8\8u171-b11
'oraclejdk8' (8u171-b11) was installed successfully!

Can you run a scoop status and a scoop checkup and provide the output?

PS C:\Users\Ihar_Hancharenka> scoop status
Scoop is up to date.
Everything is ok!
PS C:\Users\Ihar_Hancharenka> scoop checkup
No problems identified!

Seems to be related to issue 913 in scoop-extras. I would ask on Gitter, or create an issue in scoop and then reference this issue. The manifest itself works (see above), but for some reason scoop is unable to follow the redirects in some cases.

Frankly speaking, few weeks ago I was able to install Oracle JDK at Win 8 (or 8.1). But after the mandatory upgrade to Win 10 - that does not work anymore.

I decided to stay with zulu8 for now

@iharh @raikesy You can create an issue in scoop, mentioning this issue and the one in scoop-extras.

With the new aria2 download support introduced in PR #2312 you should be able to download and install oraclejdk. Just make sure you have aria2 installed.


Split shooter spoolup into Accelerate and Stabilize states

Split up the spoolup state per this strategy:

Not yet fully tuned with balls, but seems to be working well. Closing for now.


MoZ card: detail modal link to full story

link to full story from the detail modal. add a class to the link, which will point to the interstitial card.

@TylerFisher I have the text in place that I want to link to the interstitial card. When I try to add an anchor tag to the modal, it breaks the modal, so I just have < b > tags in there for now designating the link.


[BUG] addition of the pxd breaks pure mode

Describe the bug
Valid pure python mode code stops compiling if pxd file with declarations from the code is added.
To Reproduce
py file:
import cython

@cython.cfunc
def foo() -> cython.void:
    pass

pxd file:
cdef void foo()

output:
Error compiling Cython file:
------------------------------------------------------------
...
import cython

@cython.cfunc
^
------------------------------------------------------------

ex.py:3:0: Function signature does not match previous declaration
Traceback (most recent call last):
    ...
    raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: ex.py

but if we will remove type declarations (specifically cython.cfunc, but let's remove all of them) then everything will compile:
py:
def foo():
    pass

(same pxd)
I would say that if we can compile that without type declarations then we should be able to do the same with them.
And clearly, the problem is not in the type declarations, that do not match.
Environment:

OS: Windows
Python version 3.9.0
Cython version 3.0.0a8

I can see the advantage of mixing annotations and pxd files because the annotations are useful to someone reading the code, and the pxd file exports it to other Cython modules. So those should work.
I don't really see the point of mixing pxd files and @cython.cfunc decorators - since you've already provided the information in the pxd the decorator is kind of pointless. Personally I'm not too worried if that doesn't end up working.

I don't really see the point of mixing pxd files and @cython.cfunc decorators - since you've already provided the information in the pxd the decorator is kind of pointless.

The same logic can be applied to pyx and pxd: "Why should we repeat ourselves in them if there is no point in doing the same for py?"

I think it actually works now. But I am not sure that any tests capture this behavior and it can be relied upon.

I've added a test in #5974 - does look to work to me too.


Fix for Turbo Asset UI: Tooltip pops up and does not hide anymore

Pull Request
Hi,
Thanks for helping us improve the Predix UI platform by submitting a Pull Request.
To help us merge your Pull Request as fast as possible, please fill out the following sections:

A description of the changes proposed in the pull request:

The change proposed is at the line 58 changed the delay '1000' to '100'
<px-tooltip
orientation="top"
smart-orientation
delay="100"
tooltip-message="{{cellValue}}">

A reference to a related issue (if applicable):

We have a data table and column cell is clickable and has restricted length which will enable the tooptip. When the cell is clicked before 1 sec (1000 sec delay imposed on the tootltip to display), the tooltip is not hidden even the control goes to next page.

@mentions of the person or team responsible for reviewing proposed changes:

Team Turbo Asset

working tests:

Since it is simple configuration change, no tests were effected/written.

Hi,
Thank you for your pull request.
For us to accept this change this would have to be configurable either at the column level or at the table component level and not a hardcoded change.

I'm going to close this based on the notes above and the conversation we had. If you can make it configurable we'd gladly accept this PR. Thanks.


Is staking automatic?

I've seen some setup instructions where Staking needs additional configuration.  Is it automatic now?  Would be great to have some guidance on this in README.md, as well as some logging or other verification output that says that it is enabled correctly.
This article has a magi.conf configuration that seems fairly recent.  Can we just cut and past the value for the configuration into ~/.magi/magi.conf?
nano ~/.magi/magi.conf
And put in there the following:

# You must set rpcuser and rpcpassword to secure the JSON-RPC api
rpcuser=magirpc
rpcpassword=7dhf8SRjR8R5P9icfqjjbqAZqysYXeQSuX539JhXseLz
# Listen for RPC connections on this TCP port:
rpcport=8232
# server=1 to accept JSON-RPC commands
server=1
# listen=1 to accept connections from outside
listen=1
# RPC connection from localhost allowed
rpcallowip=<IP_ADDRESS>
# Add nodes to connect to specific peers
addnode=<IP_ADDRESS>
addnode=<IP_ADDRESS>
# posii=1 to enable PoS staking; default = 0
posii=1
# Transaction under stake with a value greater than the threshold is being splitted
stakesplitthreshold=500
# Transactions with values less than the threshold will combine into one
stakecombinethreshold=500

Just read something else that says that all we need to do is set posii=1 in magi.conf?

yes. you can copy paste that conf file.
You need 50xmg minimum before the wallet will stake.

Awesome!  Thanks - Any idea on when the pools are going to be back up?


Support custom exit codes for major/minor changes

Or should it simply always return 0 on identical; 1 on minor change; 2 on major change?
So no flags at all?

Let me ponder that. I think we'd very often want it to not give an error code for minor changes, as they're rarely a problem...

@jskeet please take another look - the second commit is a different potential way of controlling exit-code behaviour.


update to work with reflectable 0.4.0 and use bestEffortReflectedType

Fixes https://github.com/dart-lang/polymer-dart/issues/651, cc @eernstg

lgtm

I can see that PolymerMixin and PolymerBase are non-generic, so reflectedType is safe for both pre- and post-transformed code. I don't know enough about the context to tell whether there are any dark corners in the usage of bestEffortReflectedType in the remaining locations. Do you not need to distinguish between the situation where pre-transformed code delivers a reflectedType which embeds the type arguments (because bestEffort.. uses reflectedType, which is by the way the only one that works for, say, the type of a variable in pre-transformed code), and post-transformed code delivers a dynamicReflectedType (which is the only one that works for the type of a variable in post-transformed code if its type annotation contains a type variable)? You certainly can't expect equality tests to succeed in both cases (but, e.g., something like if (myReflectedType == List || myRelfectedType == const TypeValue<List<int>>().type) .. might handle the situation for List).

Yes, I have some code which can handle both List and List<int> etc.

OK, cool!


Update Matrix

Please approve. I've updated the website url on token details.
First PR #16947
Updated PR #19150
Thank You.

Approve This
Thank You.

merged

Thank You


so i died error

04 18:07:36 raspberrypi python[765]: ERROR:root:Unknown answer from Bitcoin Core. Reason: Internal Server Error
Apr 04 18:07:37 raspberrypi dialog[439487]: *** err
Apr 04 18:07:37 raspberrypi dialog[439487]: /dev/tty1: Permission denied
Apr 04 18:07:37 raspberrypi dialog[439487]: *** err
Apr 04 18:07:37 raspberrypi dialog[439487]: Oh, oh, it's an error! possibly I die!
whats this error?

thank you


[FEATURE REQ] CognitiveServices should provide a better error message when trying to use AAD Token against a region endpoint.

Cognitive Services only allows AAD Tokens on endpoints that have custom domains.  Right now, if you try to use the TokenCredential client overload against a region endpoint you get this error:
{"error":{"code":"BadRequest","message": "Please provide an authentication key."}}
It would be great if we could improve that error message to indicate that the user is trying to use AAD against a region endpoint and that they should use a custom subdomain instead.
azsdke2e

@ctstone is this something the service can detect and change?

@maririos, it's not something the FR team controls, but I'll try to find the right team to address this.

Thank you!!

@ctstone - Any update on this?  Do we know who owns it?

@jongio, Cognitive Services Platform Team has opened an internal issue to track this.

@ctstone do you know if they've set a priority/estimated date for it's resolution?

ping @ctstone

hi @maririos, sorry for the late reply. This change is complete. If it's not rolled out to all regions, it should be soon.
The new response body is:
{
  "error": {
    "code": "BadRequest",
    "message": "Please provide a custom subdomain for token authentication, otherwise API key is required. (Please see https://aka.ms/cogsvc-authenticatewithtoken)"
  }
}

Very cool!  Thanks @ctstone for getting this done and thanks @maririos for all the follow ups!


Improve TTS voice

In the Raspberry.org AIY forum we have been discussing the TTS voice and how to improve it
The thread is https://www.raspberrypi.org/forums/viewtopic.php?f=114&t=188515
A significant improvement is changing tts.py from
try:
subprocess.call(['pico2wave', '--lang', lang, '-w', raw_wav, words])
subprocess.call(['play', raw_wav, '--no-show-progress', '--guard'])
finally:
os.unlink(raw_wav)
to
try:
words = '<volume level='60'><pitch level='133'>' + words + ''
subprocess.call(['pico2wave', '--lang', lang, '-w', raw_wav, words])
subprocess.call(['play', raw_wav, '--no-show-progress', '--guard'])
finally:
os.unlink(raw_wav)

You guys are so amazing! I tried to reduce the volume with pico2wave but couldn't figure it out.
I'll try this out on a few Voice Kits I have here, but it looks like a good approach. Would you like to create a pull request for this, or should I?

Hi, one of the contributors suggests that new line should be better expressed as
words = '<volume level='60'><pitch level='133'>' + words + ''
I haven't got a clue about pull requests so grateful if you would. Thank You

If you want to insert code into a GitHub comment you can use the button "<>" above the text field, or surround it with ``` like this:
```
words = '<volume level=\'60\'><pitch level=\'133\'>' + words + '</pitch></volume>'
```

I know this is a closed issue, but wouldn't it be nice to expose the volume and pitch like we do with the language as optional arguments. This way we would have some more control in our own code. Since I don't use the voicehat kit, but my own speakers, amplifier and usb soundcard the volume and pitch are still a bit low. It also could give a bit more "emotion" to the voice when responding.
Let me know if this is of interest or not, so if it is worth to making a pull request.

Yes – I would also like this. Currently I amend _tts.py as follows
words = '%s' % words
rather than the default pitch of 130.
I really do my best to avoid altering any of your modules and much prefer it when you expose arguments that we can set in our own code such as the optional facility to set the language in aiy.audio.say('some words to say', lang='en-GB')
Sent from Mail for Windows 10
From: Maurice van Kruchten
Sent: 18 November 2017 10:18
To: google/aiyprojects-raspbian
Cc: sheridat; Author
Subject: Re: [google/aiyprojects-raspbian] Improve TTS voice (#101)
I know this is a closed issue, but wouldn't it be nice to expose the volume and pitch like we do with the language as optional arguments. This way we would have some more control in our own code. Since I don't use the voicehat kit, but my own speakers, amplifier and usb soundcard the volume and pitch are still a bit low. It also could give a bit more "emotion" to the voice when responding.
Let me know if this is of interest or not, so if it is worth to making a pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Enable ironic cleaning

Enable ironic to clean disk metadata only.

I wonder if we just need to add a new state?
// Change a node to "available" state
func (workflow *provisionStateWorkflow) toAvailable() (done bool, err error) {
        switch state := workflow.d.Get("provision_state").(string); state {
        case "available":
                // We're done!
                return true, nil
        case "cleaning":
                // Not done, no error - Ironic is working
                log.Printf("[DEBUG] Node %s is '%s', waiting for Ironic to finish.", workflow.d.Id(), state)
                return false, nil
        case "manageable":
                // From manageable, we can go to provide
                log.Printf("[DEBUG] Node %s is '%s', going to change to 'available'", workflow.d.Id(), state)
                return workflow.changeProvisionState(nodes.TargetProvide)
        default:
                // Otherwise we have to get into manageable state first
                log.Printf("[DEBUG] Node %s is '%s', going to change to 'manageable'.", workflow.d.Id(), state)
                _, err := workflow.toManageable()
                if err != nil {
                        return true, err
                }
                return false, nil
        }

        return false, nil
}

@stbenjam Would you happen to know? ^

Yes, that's all it needs.

https://github.com/openshift-metalkube/terraform-provider-ironic/pull/16

The installer needs an update to pull in the new terraform provider - that's here, https://github.com/openshift-metalkube/kni-installer/pull/45. You can test this updated container with that, it should work now.
I also filed https://github.com/openshift-metalkube/terraform-provider-ironic/issues/17 for me to ensure I handle all the possible state transitions, there's a few more that aren't covered, mostly xyz failed states.

With the terraform change to add support for clean-wait this should fe fine.

I’d not merge this until the installer change is in

@bfournie What else is holding this up?

https://github.com/openshift-metalkube/kni-installer/pull/45 has been merged, this should be good to go.


Package request: scikit-learn

Project description
Python Library
Metadata

homepage URL: https://pypi.org/project/scikit-learn
source URL: https://pypi.org/project/scikit-learn/
license: bsd
platforms: unix, linux, darwin

Add a :+1: reaction to issues you find important.

https://search.nixos.org/packages?query=scikit-learn


difficulties when visiting in China

upload this page into a server located in mainland China

Solved.
Website address is THIS.


Consolidate Validation lab and Encore disease name

Two datasources both studying colon cancer cell lines, however one is annotated with colorectal cancer, the other is annotated with colorectal carcinoma. We should pick the most granular one.

The disease annotation for validation lab is now changed from colorectal cancer - EFO_0005842 to colorectal carcinoma - EFO_1001951. Metadata for PPP datasources (containing disease annotation) is stored in a separate repository: https://github.com/opentargets/PPP-evidence-configuration/pull/2


Add missing flags to README

'--rtl' and '--preserve-variables' flags were documented on the source file but missing from the README

Thanks!


Import/Export Config Command

Would be cool to have this available rather than hunting through directories.

What's the desired output here? Should it dump the json to stdout? Prompt for a file location to write to?

I think prompting for file input/output makes sense.

The more I think about this, the more I think we might want to go with the stdout approach. It would let the user do things like marco export | xclip to allow for easy sharing. You could also do marco export > ~/marco.backup.json (assuming bash).


May you please update monaco-editor to version 0.34.0

I have opened this pull request to do this update

I have opened this pull request to do this update


Could I help you guys to translate the API`s doc into chinese?

I`d love to do these .  And I used to use 'webdriver.io'

Nice, we'd love help with that! But since the API and docs are still in a state of flux, I'm wondering if we should wait a bit? Eventually we plan to have a proper documentation page/site for 1.0.

@Aaronphy We discussed this with @pavelfeldman and @paulirish. We won't be able to maintain chinese translation as a part of this repository because english docs change rapidly and translation would be constantly outdated.
However, if you host the translation on your own website and will take care of it, we'll be happy to refer to it from our docs!

@ebidel @aslushnikov  I will have a try to take care of it .

Thanks @Aaronphy. Reach back to us when you're ready so that we can refer to your translation.


[BUG] Multiple errors with fresh created app with Remix and Mui

Describe the bug
After creating app with Remix and Mui console displays multiple errors
react-dom.development.js:86

       Warning: Expected server HTML to contain a matching <meta> in <head>.
    at meta
    at head
    at html
    at http://localhost:3000/build/root-QGHZKJ44.js:3781:51
    at App
    at RemixRoute (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:6093:3)
    at RenderedRoute (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:4491:5)
    at RenderErrorBoundary (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:4867:9)
    at Routes (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:4740:5)
    at Router (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:4687:15)
    at RouterProvider (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:4638:5)
    at RemixErrorBoundary (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:5721:5)
    at RemixBrowser (http://localhost:3000/build/_shared/chunk-BT37IK5M.js:7106:55)
printWarning @ react-dom.development.js:86
react-dom.development.js:12507

       Uncaught Error: Hydration failed because the initial UI does not match what was rendered on the server.
    at throwOnHydrationMismatch (react-dom.development.js:12507:9)
    at tryToClaimNextHydratableInstance (react-dom.development.js:12535:7)
    at updateHostComponent (react-dom.development.js:19902:5)
    at beginWork (react-dom.development.js:21618:14)
    at HTMLUnknownElement.callCallback2 (react-dom.development.js:4164:14)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:4213:16)
    at invokeGuardedCallback (react-dom.development.js:4277:31)
    at beginWork$1 (react-dom.development.js:27451:7)
    at performUnitOfWork (react-dom.development.js:26557:12)
    at workLoopSync (react-dom.development.js:26466:5)
2errorBoundaries.js:70

       Error: Hydration failed because the initial UI does not match what was rendered on the server.
    at throwOnHydrationMismatch (react-dom.development.js:12507:9)
    at tryToClaimNextHydratableInstance (react-dom.development.js:12535:7)
    at updateHostComponent (react-dom.development.js:19902:5)
    at beginWork (react-dom.development.js:21618:14)
    at beginWork$1 (react-dom.development.js:27426:14)
    at performUnitOfWork (react-dom.development.js:26557:12)
    at workLoopSync (react-dom.development.js:26466:5)
    at renderRootSync (react-dom.development.js:26434:7)
    at performConcurrentWorkOnRoot (react-dom.development.js:25738:74)
    at workLoop (scheduler.development.js:266:34)
RemixRootDefaultErrorBoundary @ errorBoundaries.js:70
react-dom.development.js:86

       Warning: An error occurred during hydration. The server HTML was replaced with client content in <#document>.
printWarning @ react-dom.development.js:86
8react-dom.development.js:12507

       Uncaught Error: Hydration failed because the initial UI does not match what was rendered on the server.
    at throwOnHydrationMismatch (react-dom.development.js:12507:9)
    at tryToClaimNextHydratableInstance (react-dom.development.js:12535:7)
    at updateHostComponent (react-dom.development.js:19902:5)
    at beginWork (react-dom.development.js:21618:14)
    at beginWork$1 (react-dom.development.js:27426:14)
    at performUnitOfWork (react-dom.development.js:26557:12)
    at workLoopSync (react-dom.development.js:26466:5)
    at renderRootSync (react-dom.development.js:26434:7)
    at performConcurrentWorkOnRoot (react-dom.development.js:25738:74)
    at workLoop (scheduler.development.js:266:34)
2react-dom.development.js:26912

       Uncaught Error: Hydration failed because the initial UI does not match what was rendered on the server.
    at throwOnHydrationMismatch (react-dom.development.js:12507:9)
    at tryToClaimNextHydratableInstance (react-dom.development.js:12535:7)
    at updateHostComponent (react-dom.development.js:19902:5)
    at beginWork (react-dom.development.js:21618:14)
    at beginWork$1 (react-dom.development.js:27426:14)
    at performUnitOfWork (react-dom.development.js:26557:12)
    at workLoopSync (react-dom.development.js:26466:5)
    at renderRootSync (react-dom.development.js:26434:7)
    at performConcurrentWorkOnRoot (react-dom.development.js:25738:74)
    at workLoop (scheduler.development.js:266:34)
react-dom.development.js:12507

       Uncaught Error: Hydration failed because the initial UI does not match what was rendered on the server.
    at throwOnHydrationMismatch (react-dom.development.js:12507:9)
    at tryToClaimNextHydratableInstance (react-dom.development.js:12535:7)
    at updateHostComponent (react-dom.development.js:19902:5)
    at beginWork (react-dom.development.js:21618:14)
    at beginWork$1 (react-dom.development.js:27426:14)
    at performUnitOfWork (react-dom.development.js:26557:12)
    at workLoopSync (react-dom.development.js:26466:5)
    at renderRootSync (react-dom.development.js:26434:7)
    at performConcurrentWorkOnRoot (react-dom.development.js:25738:74)
    at workLoop (scheduler.development.js:266:34)
react-dom.development.js:19849

       Uncaught Error: There was an error while hydrating. Because the error happened outside of a Suspense boundary, the entire root will switch to client rendering.
    at updateHostRoot (react-dom.development.js:19849:57)
    at beginWork (react-dom.development.js:21615:14)
    at beginWork$1 (react-dom.development.js:27426:14)
    at performUnitOfWork (react-dom.development.js:26557:12)
    at workLoopSync (react-dom.development.js:26466:5)
    at renderRootSync (react-dom.development.js:26434:7)
    at recoverFromConcurrentError (react-dom.development.js:25850:20)
    at performConcurrentWorkOnRoot (react-dom.development.js:25750:22)
    at workLoop (scheduler.development.js:266:34)
    at flushWork (scheduler.development.js:239:14)
query.ts:462

       Query data cannot be undefined. Please make sure to return a value other than undefined from your query function.  I think it's due to an incompatibility between MUI and Remix. Refine can't apply the fix you specified, but I'm sure it will be useful to someone in the future. Thanks a lot! ⚡


update-packages --force-upgrade generates conflicting isolate versions.

Steps to reproduce ./bin/flutter update-packages --force-upgrade.
Expected result: versions of isolate in flutter packages don't have conflict. bin/flutter analyze --flutter-repo succeeds.
Actual result: They do conflict, bin/flutter analyze --flutter-repo does not succeed.
$ find . -name pubspec.yaml -exec grep isolate {} \; -print
  isolate: 2.0.0
./examples/stocks/pubspec.yaml
  isolate: 2.0.0 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade"
./dev/benchmarks/microbenchmarks/pubspec.yaml

$ find . -name .packages -exec grep isolate {} \; -print
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./examples/hello_world/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./examples/layers/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./examples/platform_channel_swift/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./examples/catalog/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-2.0.0/lib/
./examples/stocks/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./examples/platform_channel/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./examples/flutter_gallery/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./packages/flutter_goldens/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./packages/flutter/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./packages/flutter_localizations/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./packages/fuchsia_remote_debug_protocol/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./packages/flutter_driver/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./packages/flutter_test/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/bots/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/tools/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/tools/vitool/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/integration_tests/ui/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/integration_tests/external_ui/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/integration_tests/flavors/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/integration_tests/platform_interaction/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/integration_tests/channels/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/manual_tests/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/devicelab/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/benchmarks/complex_layout/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-2.0.0/lib/
./dev/benchmarks/microbenchmarks/.packages
isolate:file:///Users/srawlins/.pub-cache/hosted/pub.dartlang.org/isolate-1.1.0/lib/
./dev/automated_tests/.packages

Apparently a few tools want isolate 2.0.0, but the dependency on test puts a dependency on analyzer, the latest of which depends on isolate >=0.2.2 <2.0.0.
Blocking #17442. See cirrus analyze.

Thanks @devoncarew !


Shapefiles in Ferret

Reported by karl smith on 27 Jun 2011 17:46 UTC

(Steve) Our need to write into shapefiles is much greater than our need to read from them.   Essentially a way to supply GIS-friendly outputs from LAS (and Ferret).  So this is probably the place to focus initially.
Outputting a curvilinear grid as a collections of parallelograms is one use case: -- where you are given separate IxJ arrays of lon, lat and V (possibly V(z,t) )
(Karl) There is the PolygonM shape, which contains X,Y, and M (measure), and PolygonZ, which contains X, Y, Z, and M.  So I see those as solutions to V and V(z).  For the V(z,t) case, we could write out separate shapefiles for each time step.

(Steve) In doing this we're giving a shape to the Z axis.  My untested purely-hunch guess is that

if the Z axis is in meters, then it is better to handle the Z axis in the same manner that you discuss for T -- deliver a collection of 2D shapes
if the Z axis is sigma or density dependent or some other more complex spatial structure, (and assuming that we want to preserve the native coordinates) then an XYZ polygon would be appropriate

In the course of this maybe you can psyche out the answer to if/how a Shapefile can hold a point time series.

But seems like lots of duplication of points.  There is a "Triangle Strip" MultiPatch (surface patches - used to create surfaces) where triangles are made from a new point and the last two points in the shape.  This would be more compact, but would make triangles instead of parallelograms.

If that seems better, then definitely consider it.  "Tesselations" are an everyday part of GIS.  And your approach would much better preserve the connectivity structure Just that we don't want to get into too much code development over the algorithm to turn divide the quadrilaterals into triangle pairs.  It sounds trivial, but the usual anxiety about devils hiding in the details.  For example, we want to associate a value with each grid cell, which raises all of the questions of edge-centered, face-centered, versus vertex values, etc.

Or better (if we don't need the shape) there is the option of just using the Point, PointM, or PointZ shapes if we just want to specify values at points.  In this case we could use the record (database entry) associated with each shape (a point in this case) to specify all V values at that point.  I am thinking this would better follow the traditional use of a shapefile since V are values at the indicated longitudes and latitudes rather that over some polygon area.

The most pressing use case is Shapefiles in GIS, where they are usually used to render maps, rather than to convey data.  I'm skeptical that encoding the grids as a collection of points will make a continuous color map.

All of this looks (from 50,000 ft up) to be quite simple using PyShp once we decide which way to go.

For outputting rectilinear grids -- especially is they are regularly and equally spaced in both X and Y -- can a Shapefile hold a raster like this without decomposing it into square polygons?  Dunnno.  Something to learn about.  (Our current "arcGrid ASCII output is pretty primitive ....)
The Point shapes may be all we need here.
With respect to reading the Shapefiles, if you encounter a stand-alone utility that can dump a Shapefile into some kind of ASCII representation(s), that might be enough to meet the needs of our users.  No need to put specific time into this.  Just something to keep your eyes open for.
I certainly could quickly whip something out using PyShp if we can come up with what the ASCII file should look like.  But maybe the intention here was more about seeing if there is already some well-known ASCII format used by other programs.

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1864

Modified by karl.smith on 27 Jun 2011 17:48 UTC

Modified by karl.smith on 6 Jul 2011 20:46 UTC

Comment by karl.smith on 26 Aug 2011 16:28 UTC
Created the following PyEFs for what is assumed to be the most common use cases.  These only read/write one value per shape but if there is a need, functions reading/writing multiple values (eg, SST, AIRT, and SLP for each quadrilateral) could easily be created.
SHAPEFILE_READXY(SHAPEFILE,MAXPTS)
Returns X,Y coordinates of shapes from shapefile.  Missing value separates shapes.
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
MAXPTS: Max. number of points to return (-1 for all, but reads shapefile twice)
SHAPEFILE_READXYVAL(SHAPEFILE,VALNAME,MAXPTS)
Returns X, Y, and a value from shapes in a shapefile.  Missing value separates shapes.
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
VALNAME: Name of value to retrieve (STRING)
MAXPTS: Max. number of points to return (-1 for all, but reads shapefile twice)
SHAPEFILE_READXYZ(SHAPEFILE,MAXPTS)
Returns X,Y,Z coordinates of shapes from shapefile.  Missing value separates shapes.
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
MAXPTS: Max. number of points to return (-1 for all, but reads shapefile twice)
SHAPEFILE_READXYZVAL(SHAPEFILE,VALNAME,MAXPTS)
Returns X, Y, Z, and a value from shapes in a shapefile.  Missing value separates shapes.
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
VALNAME: Name of value to retrieve (STRING)
MAXPTS: Max. number of points to return (-1 for all, but reads shapefile twice)
SHAPEFILE_WRITEVAL(SHAPEFILE,VALUE,VALNAME,MAPPRJ)
Writes a shapefile of XY quadrilaterals associated with given values
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
VALUE: Shape values; X and Y axes required, Z axis optional, T axis undefined or singleton
VALNAME: Name for the shape value (STRING)
MAPPRJ: Common name or WKT description of map projection; if blank, WGS 84 is used (STRING)
SHAPEFILE_WRITEXYVAL(SHAPEFILE,GRIDX,GRIDY,VALUE,VALNAME,MAPPRJ)
Writes a shapefile of XY quadrilaterals from the curvilinear data arrays.
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
GRIDX: X values (longitudes) for the quad. grid; must be 2D on X and Y axes
GRIDY: Y values (latitudes) for the quad. grid; must be 2D on X and Y axes
VALUE: Shape values; must be 2D on X and Y axes
VALNAME: Name for the shape value (STRING)
MAPPRJ: Common name or WKT description of map projection; if blank, WGS 84 is used (STRING)
SHAPEFILE_WRITEXYZVAL(SHAPEFILE,GRIDX,GRIDY,GRIDZ,VALUE,VALNAME,MAPPRJ)
Writes a shapefile of XY quadrilaterals with Z coordinates from the curvilinear data arrays.
SHAPEFILE: Shapefile name (any extension given is ignored) (STRING)
GRIDX: X values (longitudes) for the quad. grid; must be 2D on X and Y axes
GRIDY: Y values (latitudes) for the quad. grid; must be 2D on X and Y axes
GRIDZ: Z values (levels) for the quad. grid; must be 2D on X and Y axes
VALUE: Shape values; must be 2D on X and Y axes
VALNAME: Name for the shape value (STRING)
MAPPRJ: Common name or WKT description of map projection; if blank, WGS 84 is used (STRING)


Show 3 cities in one row in horizontal mode

Expected behaviour
In horizontal mode, change number of cities to 3 (instead of 2)

Hi may I take this task?

Updates @thilinatnt ?

@Swati4star Oh sorry, actually I thought this is already taken as I din't get the confirmation message

I think, we shouldn't go ahead with this issue. Flipping of cards will also look weird if there are 3 cities in a row 👍
and thanks for such a detailed investigation on this issue!


Feature Suggestion - Existing subtitle text accuracy enhancement

Hi there.
Amazing project, it looks like -almost- exactly what I'm looking for and I think this feature might be worth implementing.
Right now I'm trying to improve the subtitles available on a series. This series has subtitles, but they're very ...poor. Timing is only OK, but word accuracy is dog poo. One example within seconds of the opening in the first episode:
Subtitle:
- I question that proposal.
- What does it matter?

Actual scene:
- Considering the circumstances, I question that proposal at this time.
- What does it matter? We're not-

The problem is that the series is scifi, so planet/race/person names and technobabble come up frequently, which often means transcription models have a tendency to become too...creative.
EDIT: Forgot to mention here -- this is why I'm trying to use the original subtitle text as an initial_prompt below -- I believe this will help with matching said difficulties to text, and in my experience this has been the case.
My current (very WIP) effort uses the .srt to split the audio segments for each subtitle out of a media file, then processes each individual audio clip with Whisper -- using the original subtitle text as initial_prompt, patience=2 -- and the text I'm getting from Whisper is a great accuracy match but I'm fighting with timing stuff.
Is this worth implementing here?
Is there a better way?
Can this project already accomplish my goal, but somehow I've missed it?
Thanks for your time.

Hi, @codefaux , glad to know the transcription you are getting has great accuracy via prompting. Currently this project transcribes the whole audio without segmenting it. The enclosing method could be modified to take in the original subtitle cues, aligning with your idea, and ofc. it means the original time codes need to be reasonably accurate. Your approach sounds to me a promising way to improve the subtitle quality and it would also be interesting to know how well Whisper works on very short audio segments without surrounding context.

Hey there. Thanks for your attention. I think there's strong merit here, but after a few days of poking at it, I can say -- there are a few issues and I'm at my limit on improving the situation.
Using a DVD source with what I would judge to be very accurate timecodes on their less accurate wording, I've found a few key notes;

Significant improvement in word accuracy versus original subtitles, with typically flawless word accuracy versus audio, except....
Issues with things like names and technobabble when the supplied original subtitle line does not include the word(s), suggesting that a transcription AI which allowed a supplied list of names/etc would be much better suited for this sort of thing -- if/when such an AI model exists. -- This is a big one for scifi content especially, and I don't see a way around it for now.
Frequent issues breaking subtitles between speaking voices, when one subtitle segment contains two voices speaking separate lines sequentially, even when input text properly differentiates them and they are clearly different voices (for example switch between male and female voices of diverse vocal quality) -- obviously not a huge deal, time and model improvements can help with this, perhaps some parameter tuning.
Quite significant issues with hallucination when word fragments are present in the clips, which is kind of what you were likely alluding to regarding reasonably accurate time codes. -- Might be able to work around this with intelligent silence detection around splits and combining nearby subtitle groups and so on but realistically the effort to build that framework would be pretty monumental, and to questionable gains.

I've reached the end of the road regarding my own ability to improve this, I'm not deeply versed in either python or AI model manupilation/etc, it's been a somewhat brute force approach from the start and I don't know how to begin implementing the changes required either here or on my own project.
If you wish for help testing implementations of the above, I'd gladly provide it, but as of now I'm only worth as much as any other Ideas Guy, lol.

No worries at all. It has been quite common so far for people with less insight into the code base to use issues to throw ideas. Do you have a loyalty-free pair of video and subtitle files for me to test this? If you want to draw a PR draft that's also welcome.

I do not have properly 'free' clips to share, unfortunately -- my sole test subject has been a DVD boxed set processed by MakeMKV.
I also can't provide a PR; my work so far is literally a pair of python scripts acting on ffmpeg directly, completely separate from implementation in this project in any way.

Hi, @codefaux, just checked in a naive implementation on taking in prompts during transcription. Feel free to give it a go with your clips and let me know if this improves the outcome. Examples of usage:
pip install -e .
# Use subtitle content as the prompts when transcribing each audio segment
subaligner -m transcribe -v video.mp4 -s subtitle.srt -ml eng -mr whisper -mf turbo -o subtitle_aligned.srt
# Use a global prompt when transcribing the entire audio
subaligner -m transcribe -v video.mp4 -ml eng -mr whisper -mf turbo -ip "your initial prompt" -o subtitle_aligned.srt

During my test I did spot hallucinations, some of which were mitigated by the internal filtering logic but not all. Configuring patient is not available yet and if this turns out to be promising, future improvement will also include speeding up the process by introducing parallelism.


Add support for CentOS

Signed-off-by: aneessahib<EMAIL_ADDRESS>Adding CentOS support to GSC.
To test - follow similar procedure as followed with Ubuntu (test/README).

This change is

@woju - pls check if all looks good.

@dimakuv - all done. took me a while to notice that you hadn't used reviewable for the latest set of comments.


Clean LogicManagerTest #327

Part of #327

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/6] LogicManagerTest: Remove tests
[v1 2/6] LogicManagerTest: Remove unused methods following tests removal
[v1 3/6] LogicManagerTest: Remove Storage component
[v1 4/6] LogicManagerTest: Simplify the use of generics
[v1 5/6] LogicManagerTest: Remove unnecessary test
[v1 6/6] LogicManagerTest: Add test

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/495/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

Coverage decreased cos LogicManagerTest is not testing the Storage component anymore

v2
@Zhiyuan-Amos submitted v2 for review.

[v2 1/6] LogicManagerTest: Remove repeated tests
[v2 2/6] LogicManagerTest: Remove unused methods
[v2 3/6] LogicManagerTest: Remove Storage component
[v2 4/6] LogicManagerTest: Remove generic type name
[v2 5/6] LogicManagerTest: Remove unnecessary test
[v2 6/6] LogicManagerTest: Add test for CommandException thrown

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/495/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

Coverage decreased cos LogicManagerTest is not testing the Storage component anymore

Coveralls mentions that the drop seems to come from FindCommandParser.java and AddCommandParser.java though. We still have some tests covering the Storage component.

ah. #494 and #431 will resolve that

@limmlingg fyr(for your review)

v3
@Zhiyuan-Amos submitted v3 for review.

[v3 1/6] LogicManagerTest: Remove repeated tests
[v3 2/6] LogicManagerTest: Remove unused methods
[v3 3/6] LogicManagerTest: Remove Storage component
[v3 4/6] LogicManagerTest: Remove generic type name
[v3 5/6] LogicManagerTest: Remove unnecessary test
[v3 6/6] LogicManagerTest: Add test for CommandException thrown

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v2 and v3)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/495/3/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

@yamgent for you to ask external reviewers for review :)

v4
@Zhiyuan-Amos submitted v4 for review.

[v4 1/8] LogicManagerTest: Remove repeated tests
[v4 2/8] LogicManagerTest: Remove unused methods
[v4 3/8] LogicManagerTest: Remove Storage component
[v4 4/8] LogicManagerTest: Remove generic type name
[v4 5/8] LogicManagerTest: Remove unnecessary test
[v4 6/8] LogicManagerTest: Add test for CommandException thrown
[v4 7/8] LogicManagerTest: Add successful test
[v4 8/8] LogicManagerTest: Merge LogicManagerTest#execute_verifyHistory_success

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v3 and v4)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/495/4/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v5
@Zhiyuan-Amos submitted v5 for review.

[v5 1/11] LogicManagerTest: Remove repeated tests
[v5 2/11] LogicManagerTest: Remove unused methods
[v5 3/11] LogicManagerTest: Remove Storage component
[v5 4/11] LogicManagerTest: Remove generic type name
[v5 5/11] LogicManagerTest: Remove unnecessary test
[v5 6/11] LogicManagerTest: Place private methods after test methods
[v5 7/11] LogicManagerTest: Rename test method
[v5 8/11] LogicManagerTest: Add test for CommandException thrown
[v5 9/11] LogicManagerTest: Add successful test
[v5 10/11] LogicManagerTest: Merge LogicManagerTest#execute_verifyHistory_success()
[v5 11/11] LogicManagerTest: Initialise object variables during declaration

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v4 and v5)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/495/5/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v6
@Zhiyuan-Amos submitted v6 for review.

[v6 1/11] LogicManagerTest: Remove repeated tests
[v6 2/11] LogicManagerTest: Remove unused methods
[v6 3/11] LogicManagerTest: Remove Storage component
[v6 4/11] LogicManagerTest: Remove generic type name
[v6 5/11] LogicManagerTest: Remove unnecessary test
[v6 6/11] LogicManagerTest: Place private methods after test methods
[v6 7/11] LogicManagerTest: Rename test method
[v6 8/11] LogicManagerTest: Add test for CommandException thrown
[v6 9/11] LogicManagerTest: Add successful test
[v6 10/11] LogicManagerTest: Merge LogicManagerTest#execute_verifyHistory_success()
[v6 11/11] LogicManagerTest: Initialise object variables during declaration

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v5 and v6)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/495/6/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

Made the changes. Will put this on hold until the rest of the tests are merged in.


Question:- Carousel with an Enlarged active Item and two complete side previews slightly hidden by it.

Is this a bug report, a feature request, or a question?
Question

Hi, I'm already using this amazing library for my Project and it worked well for my Carousel requirement (Except looping issue in Android).
But I have a different Carousel design requirement for a different screen which require an enlarged active item and 2 previews on the left and right side which are slightly hidden by the active item, and all item are circle.
here is the screenshot.

I tried to create this carousel with this library and also searched on the internet for a solution or alternative but haven't succeeded yet.
Is a Carousel like this possible with this library?
If not then is there any alternative you can suggest?
Thank you

I was able to implement this using this example by some modifications.
but it is not efficient cause I have to use ScrollView.
If I use Flatlist iOS is somewhat close to the design but showing one preview item behind and one on top of the active item and in android preview items are clipped which is messing the whole design.
I tried some workarounds but didn't work.
here is my implementation.

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2


[Bug] debug_trace_call and debug_trace_call_many invalid parameter

Component
provider, pubsub
What version of Alloy are you on?
https://github.com/alloy-rs/alloy?rev=188c4f8#188c4f8f
Operating System
None
Describe the bug
In alloy-provider crate I believe debug_trace_call and debug_trace_call_many functions should have  GethDebugTracingCallOptions instead of  GethDebugTracingOptions
https://github.com/alloy-rs/alloy/blob/6ce3c05062fe77a449d761984ec89d47e7d0253e/crates/provider/src/debug.rs#L74
https://github.com/alloy-rs/alloy/blob/6ce3c05062fe77a449d761984ec89d47e7d0253e/crates/provider/src/debug.rs#L88
Thank you for your attention.

yeah these should indeed be GethDebugTracingCallOptions


Tax function code consolidation

As I work through PR #629, I've been thinking about how we compute tax rates from the tax function parameters in OG-USA.  There is some duplicate code in the model as txfunc.get_tax_rates(), tax.MTR_income, and tax.ETR_income all use similar lines to compute tax rates from the tax function parameters.
This not only duplicates lines of code, but makes updates to the model (e.g., adding or changing tax functions) more difficult because these functions need to be changed in multiple places.
One difficulty in consolidating these functions is that (at least in the case of the DEP functional form) the parameters are in a different order in txfunc.get_tax_rates() than in the ogusa.tax functions.  E.g., for a DEP function, txfunc.get_tax_rates expects parameters to be in the order A, B, C, D, max_x, max_y, share, min_x, min_y, shift, shift_x, shift_y.  Whereas in the ogusa.tax functions, the model expects the order A, B, C, D, max_x, min_x, max_y, min_y, shift_x, shift_y, shift, share.
To fix this, I'd like to keep parameters (in the case of the DEP form) in the order A, B, C, D, max_x, max_y, share, min_x, min_y, shift, shift_x, shift_y throughout OG-USA.  This order is helpful since the parameters that are not estimated through the tax function estimation, min_x, min_y, shift, shift_x, shift_y, are appended to those that are.  This ordering helps to reduce lines of code and reordering or parameters in the tax function estimation.
If we do this reordering, we can have on get_tax_rates() function that is called in ogusa.txfunc and ogusa.tax, reducing duplicate code and making updates easier.
One issue with changing this, is we break backwards compatibility with tax function parameters that have been estimated previously.
@rickecon Are you ok with breaking compatibility?  Any suggestions on better approaches to consolidating the code?

@jdebacker. I think this consolidation is a great idea, and I am fine with breaking backwards compatibility (we are still pre-version-1.0).

This issue was resolved with PR #629.


/Pass has to be first command

Looks like we need to prohibit the effectuation of /nick and /user if the /pass was not sent before. i.e. if the user is not confirming they have the correct access rights to the server.

/pass needs to be the first now.


ci: Use PostgreSQL 16 and fix database issues in local/sandboxes

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

Project coverage is 96.08%. Comparing base (a545c50) to head (a24c6e1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #513   +/-   ##
=======================================
  Coverage   96.08%   96.08%
=======================================
  Files         153      153
  Lines        3293     3293
=======================================
  Hits         3164     3164
  Misses        129      129

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


[E2E] add udf function e2e test

Purpose of the pull request
#7312

Codecov Report

Merging #8250 (1c827f9) into dev (303845b) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8250      +/-   ##
============================================
+ Coverage     44.93%   44.94%   +0.01%
- Complexity     3949     3950       +1
============================================
  Files           672      672
  Lines         26182    26182
  Branches       2825     2825
============================================
+ Hits          11765    11768       +3
+ Misses        13306    13304       -2
+ Partials       1111     1110       -1

Impacted Files
Coverage Δ

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
53.52% <0.00%> (+2.81%)
:arrow_up:

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 303845b...1c827f9. Read the comment docs.


Re-enable TestValidatorToRememberValidation

Describe the bug
TestValidatorToRememberValidation is skipped, and there is no explanation or issue attached. File this issue to keep track.
cc: @mandarjog @douglas-reid

Kuat can you try enabling this again and seeing if it is still flaky?

@ozevren This issue is of stateful validation, referential integrity.
If a yaml file submits "A, B and C" subdocuments where C depends on B and B depends on A. In that case the validator should remember what it has recently validated because the next validation call based on previous can come in before the previous doc is fully committed to the API server.
We decided that this was not needed at the time, so disabled test.
Are we supporting referential integrity ? I am closing this issue. @ozevren please re-open it if this needs to be there.

No, there is no support for referential integrity. Should we keep the bug open to remove the test then?


fzn-or-tools "Cannot call Constains() on selected_inputs"

I'm running:
./fzn-or-tools model.fzn
which fails with:
Cannot call Constains() on selected_inputs
*** Check failure stack trace: ***
    @        0x109545bba  google::LogMessage::Fail()
    @        0x10954304a  google::LogMessage::SendToLog()
    @        0x1095448ff  google::LogMessage::Flush()
    @        0x10954d6b8  google::LogMessageFatal::~LogMessageFatal()
    @        0x109546165  google::LogMessageFatal::~LogMessageFatal()
    @        0x108943987  operations_research::fz::Argument::Contains()
    @        0x108959ab5  operations_research::fz::Presolver::PresolveSetInReif()
    @        0x108957f4b  operations_research::fz::Presolver::ApplyRule()
    @        0x10896a1f0  operations_research::fz::Presolver::PresolveOneConstraint()
    @        0x108970c01  operations_research::fz::Presolver::Run()
    @        0x1088f7b1e  operations_research::fz::ParseFlatzincModel()
    @        0x1088f8896  main
Abort trap: 6

model.fzn can be found here: https://gist.github.com/anonymous/bdf9debb10ae00f71f3550ee57be6711

FWIW I'm using v6.3.4431 on OSX; and the file runs fine with flatzinc

I believe you are using set variables, and they are not supported.
Sorry.
--Laurent
PS: BTW, using --use_cp_sat is a much better solver.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-08-06 15:49 GMT-07:00 Ryan Havar<EMAIL_ADDRESS>
I'm running:
./fzn-or-tools model.fzn
which fails with:
Cannot call Constains() on selected_inputs
*** Check failure stack trace: ***
@        0x109545bba  google::LogMessage::Fail()
@        0x10954304a  google::LogMessage::SendToLog()
@        0x1095448ff  google::LogMessage::Flush()
@        0x10954d6b8  google::LogMessageFatal::~LogMessageFatal()
@        0x109546165  google::LogMessageFatal::~LogMessageFatal()
@        0x108943987  operations_research::fz::Argument::Contains()
@        0x108959ab5  operations_research::fz::Presolver::PresolveSetInReif()
@        0x108957f4b  operations_research::fz::Presolver::ApplyRule()
@        0x10896a1f0  operations_research::fz::Presolver::PresolveOneConstraint()
@        0x108970c01  operations_research::fz::Presolver::Run()
@        0x1088f7b1e  operations_research::fz::ParseFlatzincModel()
@        0x1088f8896  main
Abort trap: 6
model.fzn can be found here: https://gist.github.com/anonymous/
bdf9debb10ae00f71f3550ee57be6711
FWIW I'm using v6.3.4431 on OSX
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/447, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17erheyRSeBHuWHdM5K4I-G1YUavOks5sVkLqgaJpZM4Ou3Wy
.

Yeah, it does use a set. Is this something that's going to be supported in the future?  If not, I guess returning an error message instead of a stack would be the expected behavior =)

Not in the near future, and yes about the error message :-)
Now, with the sat solver backend, the path may be simpler than with the CP
solver.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-08-07 9:33 GMT-07:00 Ryan Havar<EMAIL_ADDRESS>
Yeah, it does use a set. Is this something that's going to be supported in
the future? If not, I guess returning an error message instead of a stack
would be the expected behavior =)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/447#issuecomment-320713635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17Wtusio36WCgt3w_Dioa0I_WJiZiks5sVzxMgaJpZM4Ou3Wy
.

We have added support for sets using the nosets,mzn redefinition file.
I will close this issue.


Support human in the loop

I’m using CVAT to do an object detection labeling, my method is to label 1000 images and download to train my model , then upload predictions to CVAT and fix, do it again and again , is there any way to do all these steps in one platform?

@SaihholdChiu , it is something which we want to implement in the nearest future.

Hi @nmanovic, do we have any news on this? Thank you!

By the way, Label Studio has a workaround for this, which is integrating a model to the system so that we can train/inference it during the labeling process. Thanks to the prediction score, we sort the samples to annotate the ones with low confidence first.
Can we do something similar?

any update on this ?
I am also highly interested. As far as I'm aware SAM (and SAM2) were trained with a similar way.
There are some tutorials about finetuning SAM2 on custom data, that could be used for this.
e.g.: https://towardsdatascience.com/train-fine-tune-segment-anything-2-sam-2-in-60-lines-of-code-928dd29a63b3

@nikste , @doantientai , we are working on a general approach for that. I hope to release something this year.


Reverse proxy requests to api.parse.com allowing for early client migration?

This is an exploratory question in case someone here has already done this.
Is it possible to run nginx and rewrite api.mydomain.com to api.parse.com and push out new app versions with that?
This would become a Step 0 to the excellent migration guide, allowing for me to move my mobile clients to a domain I control before performing the Mongo migration and using this package to host my own backend?
Will api.parse.com and the (latest) clients SDKs be happy with that?

I'm not sure about the SDKs but I would recommend setting up an API gateway for more flexibility.

I have done it on my app and works fine

@JcMinarro could you provide some details of your implementation?

:+1: would be super helpful to us and others if you can provide the config or some other info

I have use nginx. Config is the next:
server {
    listen 80;
    listen [::]:80;

    root /var/www/[YOUR_DOMAIN_HERE]/html;
    index index.html index.htm;

    server_name [YOUR_DOMAIN_HERE] www.[YOUR_DOMAIN_HERE];

    access_log /var/log/nginx/[YOUR_DOMAIN_HERE].access.log cache;
    error_log /var/log/nginx/[YOUR_DOMAIN_HERE].error.log;

    location / {
        proxy_pass	https://api.parse.com/1/;
    }
}

Thanks for the info @JcMinarro! This sounds like a great way to get a new version of you app out early so you can be ready to switch while you verify that your app still works with parse-server.

@JcMinarro thanks for the info. You might to set proxy headers such as originating IP and such as well.

Sorry to be late to this party, but if you want your proxy new API URL to reflect your future parse-server URL for example https://api.example.com/parse you will need a config something more like this if using Nginx:
server {
  listen 443 ssl;
  ssl_certificate yourcert.crt;
  ssl_certificate_key yourkey.key;
  server_name api.example.com;

  location /parse {
    proxy_pass https://api.parse.com/1;
  }
}

I highly recommend using secure HTTPS communication. If you need a free cert, you can use Let's Encrypt.


Fix linting for errors for Elm 0.18

The PR that fixed linting for Elm 0.19 broke 0.18 linting, primarily
because Elm 0.19 logs to stderr, but Elm 0.18 logged to stdout for
errors. I've fixed this by checking the Elm version and then parsing the
errors differently, reinstating the old behaviour for 0.18 but leaving
the new behaviour for 0.19.

Thanks!

@7sharp9 thank you! Would you consider shipping a new release shortly with this change on? This will considerably improve my experience until we can upgrade our app at work!

@jackfranklin Unfortunately I don’t have the credentials to update the addin.  Cc @Krzysztof-Cieslak


This spark connector is no longer supported

azure-sqldb-spark has reached end of life in 2020.  Can someone please update this page with the new connector sql-spark-connector

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

ID: 493e6bd2-ce9d-c473-a623-910b428d47b7
Version Independent ID: a45fa45f-9f56-5162-f62a-f3d86d088487
Content: Use the Spark connector with Microsoft Azure SQL and SQL Server - Azure SQL Database
Content Source: azure-sql/database/spark-connector.md
Service: sql-db-mi
Sub-service: development
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

Hi @hiumingw Thanks for reaching out, We will Investigate and update you shortly.

Hi @hiumingw! Thanks for taking time to open this GitHub issue. This doc was updated in Sept 2020 to reflect this. The NOTE at the top of the page recommends the Apache Spark Connector for SQL Server and Azure SQL instead.

#please-close


prune seed

remove production_mnist as it the same as  research_mnist just using a deprecated dictionary on the __step_end
some next steps/PRs

add testing
generating docs

pretty much adding CI/docs tools as we have in Sandbox

Hello @Borda! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file setup.py:

Line 10:80: E501 line too long (127 > 79 characters)

before #9 it was almost duplicated, then #9 updated just research part and left production_mnist outdated
I was asking @williamFalcon and he agreed with this pruning... :]


'round': ambiguous call to overloaded function

Visual studio 2022, C++ 20, Warning level 4, using Clipper 2 via vcpkg. Clipper2 1.3.0
    template <typename T2>
    inline void Init(const T2 x_ = 0, const T2 y_ = 0)
    {
      if constexpr (std::numeric_limits<T>::is_integer &&
        !std::numeric_limits<T2>::is_integer)
      {
        x = static_cast<T>(std::round(x_));  // <<<<<<<< C2668 'round': ambiguous call to overloaded function
        y = static_cast<T>(std::round(y_));
      }
      else
      {
        x = static_cast<T>(x_);
        y = static_cast<T>(y_);
      }
    }

I can remedy this simply by replacing !std::numeric_limits<T2>::is_integer with std::is_floating_point_v<T2>, alternative would be to specify an template explictly or to cast, I would rather use the is_floating_point primary type trait because those are simple, compiler optimized and mutually exclusive (each type is always and only in one primary type trait category).
I am not sure why numeric_limts was used this way (not for limits but type classing), I would recommend that primary type traits be used throughout instead including using is_integeral for integer types.
https://en.cppreference.com/w/cpp/types/is_floating_point
https://en.cppreference.com/w/cpp/types/is_integral
Here is my proposed solution that would help if structured types are used that need to be rounded, but are not necessarily is_floating_point or is_integral_type because the primary types would match custom or extended types from boost most likely as is_class  because they are wrappers.

// can we call std::round on the type? default false
template <typename T, typename = void>
struct is_round_invocable : std::false_type {};

template <typename T>
struct is_round_invocable<T, std::void_t<decltype(std::round(std::declval<T>()))>> : std::true_type {};

...

    if constexpr (is_round_invocable<T2>::value && !std::is_integral_v<T2>)
    {
        x = static_cast<T>(std::round(x_));
        y = static_cast<T>(std::round(y_));
    }
    else
    {
        x = static_cast<T>(x_);
        y = static_cast<T>(y_);
    }

Thank you for the very helpful feedback, I've made the changes you've suggested and will upload them shortly.

I am not sure why numeric_limts was used this way

Just showing my inexperience with C++ 😱.

Thank you for the very helpful feedback, I've made the changes you've suggested and will upload them shortly.

I am not sure why numeric_limts was used this way

Just showing my inexperience with C++ 😱.

There is actually very good reason to use std::numeric_limits<T>::is_integer in cases where a user defined type is used. Developers may define a template specialization on std::numeric_limits<T>::is_integer, because they are not allowed* to specialize std::is_integral - or the other primary type traits. The problem is as soon as you create a user type it will detected as std::is_class, and you are not allowed* to change that behavior, but the common solution is to fall back on std::numeric_limits<T>::is_integer.
None of what I typed above changes my proposed solution. It basically just rounds if round is supported and the type is not an internal integral type.
P.S. I've been using Clipper in Delphi for a very long time. I appreciate all your work and sharing that with the world
* It leads to undefined behavior.


Support for lightning addresses on subdomains

Paying to lightning addresses hosted on subdomains is currently not possible using lnbits because the validation rules prevent it.
Steps to reproduce

Open lnbits, click on "paste request"
Enter a lightning address hosted on a subdomain<EMAIL_ADDRESS>See error "invalid lnurl"

Analysis
lnbits/core/views/api.py:
if len(name_domain) == 2 and len(name_domain[1].split(".")) == 2:
The second part of this validation could probably be removed to allow lightning addresses on subdomains, too.
Happy to prepare a PR if you agree with this change.

Fixed by #914


Exception occurs when trying to play from queue

Please tick all applicable boxes.

[X] I am using Python 3.5 or higher (run python --version on the command line)
[X] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (1.9.8)
[ ] The latest review version

What type of issue are you creating?

[X] Bug
[ ] Feature request
[ ] Question

Description of issue
The bot fails to play the next song in the queue instead it outputs an exception however remains funcional.
After the error occurs issuing another play command will cause the song to play like normal.
The error occurs 100% of the time
Steps to reproduce

Start the music bot
queue 2 songs

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
musicbot.log

I think this is fixed in review?


Fixed modal and question show cosmetics

Changes
Context
Testing
Pre-merge checklist

[x] Build succeeds locally and in CI
[x] Proper documentation
[x] Tests
[x] Git history clean

Coverage remained the same at 27.34% when pulling c0f3e03bd372543c973b80d13cb0171cdc5465c1 on cosmetic-fixes into d55c0f2e50354ec6393ce18460d782cc19f43925 on master.


Feature Request: To deploy on Kubernetes

Please provide us with the following information:

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

Minimal steps to reproduce

Able to run on Kubernetes

Any log messages given by the failure

Expected/desired behavior

To have a dockerfile and deploy it on Kubernetes.
OS and Version?

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

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Not exactly Kubernetes, but we have a branch here that deploys the sample to Azure Container Apps: https://github.com/Azure-Samples/azure-search-openai-demo/tree/aca
We presented about that at Python Day, so you can rewatch our talk in the stream.
https://www.youtube.com/live/9xxpn-bJes0?si=t42tW34eEzXmAwRF

The repo now includes a Dockerfile and optional deployment to Azure Container Apps:
https://github.com/Azure-Samples/azure-search-openai-demo/releases/tag/2024-09-19
I dont envision also supporting deployment to Kubernetes, as we don't need that complexity yet, but hopefully the Dockerfile helps for those who are deploying to AKS.