Bullet option in placeholder definition not working

I use version 3.0.1
Bullets are not working when used in a master slide for the definition of a placeholder
{ 'placeholder': {
  options: { bullet:true, name:'desc', x:0.47, y:1.8, w:3, h:.2, fontSize:8},
  text: ''
}

If used without a placeholder they are working.

I have the same issue. I analyzed the generated pptx file and for some reason txBody has this:
<a:pPr>
  <a:buNone/>
</a:pPr>

instead of for example this:
<a:pPr indent="-342900" marL="342900">
  <a:buSzPct val="100000"/>
  <a:buChar char="•"/>
</a:pPr>

The following PR fixes the issue, but not completely. Only the first item has a bullet.
https://github.com/gitbrent/PptxGenJS/pull/620
Before and after screenshots.


[Feature Request]: Performance optimization for full text search if there are many terms

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
No response
Describe the feature you'd like
Either BMW or BMM could not skip efficiently if the query contains too many clauses. Optimizations are required for these scenarios
Describe implementation you've considered
No response
Documentation, adoption, use case
No response
Additional information
No response

Fix by #1342


Revert "Remove withNavigationFallback.js HOC"

Details

Restore the withNavigationFallback to avoid app crashing. Straight revert of PR https://github.com/Expensify/App/pull/29449.
Fixed Issues

$ https://github.com/Expensify/App/issues/29839
PROPOSAL: https://github.com/Expensify/App/issues/29839#issuecomment-1767530246
Tests

Login in ND
Go to any chat
Send any message
Hold message >> Delete message
Verify that message is deleted and user can continue use app

[x] Verify that no errors appear in the JS console

Offline tests

NA
QA Steps

Login in ND
Go to any chat
Send any message
Hold message >> Delete message
Verify that message is deleted and user can continue use app

[x] 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 / Chrome
[x] iOS / native
[x] iOS / 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 approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[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 a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] 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)
[x] 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)
[x] All JSX used for rendering exists in the render method
[x] 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

[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(themeColors.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 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.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/113963320/9babdc06-e94e-4752-8c6b-04777545b7cb

Android: mWeb Chrome

https://github.com/Expensify/App/assets/113963320/9f8d3bc3-19c6-4fb9-a3ec-02c58665ff1b

iOS: Native

https://github.com/Expensify/App/assets/113963320/1dc3a188-fc48-47c5-88ad-cb634d94323f

iOS: mWeb Safari

https://github.com/Expensify/App/assets/113963320/c39cd12e-370a-4dbc-82f8-1ba394004760

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/113963320/6c056dca-5566-4ab5-9c8d-c5e4e079b42c

MacOS: Desktop

https://github.com/Expensify/App/assets/113963320/cb93bed0-1c28-4839-9bca-2824c1008325

Alright, thanks let me find someone for the C+ review 👍

@rushatgabhane since it's late for you @situchan will be completing the checklist

Reviewer Checklist

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

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] 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)
[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 checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] 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
[x] I verified proper code patterns were followed (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] 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 approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[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 verified that this PR follows the guidelines as stated in the Review Guidelines
[x] 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)
[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] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] 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)
[x] 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)
[x] All JSX used for rendering exists in the render method
[x] 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

[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(themeColors.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 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.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/108292595/d546be7f-b9e4-482a-bcd6-594b53189926

Mobile Web - Chrome
https://github.com/Expensify/App/assets/108292595/b4b89856-d12e-4f3c-ba26-2cfdf2d6260f

Mobile Web - Safari
https://github.com/Expensify/App/assets/108292595/4203b2b3-476f-45a1-bb47-02dda33ec562

Desktop
https://github.com/Expensify/App/assets/108292595/b2826058-16c7-4321-b0d7-59c0764abde7

iOS
https://github.com/Expensify/App/assets/108292595/32418b8e-3540-4e74-a68e-118287b59b43

Android
https://github.com/Expensify/App/assets/108292595/5ddf6772-adaa-4b80-951b-802768939232

@tienifr what do you think about this bug @situchan encountered here? also cc @kacper-mikolajczak I see you encountered this bug before here

It did not throw any error or crashed the app but the comment was not deleted until I switched to another report and came back

This seems like another deploy blocker with different root cause

Yes, I was able to reproduce the resizing issue. Having looked at the errors, I thought it's not related to this PR.

Yeah my guess as well, I think we have another PR causing this issue. So I'm gonna go ahead and merge this one, thanks everyone!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

sounds good @tienifr, also, this PR just got deployed yesterday, @kacper-mikolajczak ran into this issue while working on it, could it be what's causing that bug on staging?

Yeah I know. I think we also need to figure out the resizing issue.

Hmm so strange. I can't reproduce it (i.e. the delete by popover menu issue) anymore on the latest main after we merged this. Thanks for your patience @youssef-lr !

I suspect #29784 (merged 15 min ago) might have fixed this as it's the only one related to popover merged within 1 hr.
I was able to reproduce 1 hr ago on this branch merged with latest main locally.

🚀 Cherry-picked to staging by https://github.com/francoisl in version: 1.3.86-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.86-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@rushatgabhane I can delete message successfully on staging (both from context menu and from mini context menu)
https://github.com/Expensify/App/assets/108292595/be59743d-b893-433c-a5ac-a8aa07325a73
https://github.com/Expensify/App/assets/108292595/ef7059b3-8968-4ae4-be66-4242ddd093fa

@situchan @tienifr the resizing error has made it to production, can we confirm if this PR is the culprit? just a guess as Kacper encountered this error while working on the PR

OK I can confirm that reverting that PR fixes the issue for me locally

Looking into it now 🔧

Thanks @kacper-mikolajczak!

🚀 Deployed to staging by https://github.com/youssef-lr in version: 1.3.88-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 1.3.88-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Updated to OpenVR 1.0.17

Updated to using OpenVR 1.0.17 and did a little bit of cleanup. Also removed the examples for the RGBA8 stuff as this seems unlikely to be merged into the Godot Master at this point in time and just leads to too much confusion.
@beniwtv  any chance you can compile linux builds again and do a PR with the linux so?

Will do when I get home tonight.
This is only for master / Godot master right?

Yup, I'm leaving Godot 3.0 as is.


[2.3.3] Climate Card Issue

There's a problem with climate maps concerning presets in sub-buttons. These do not update when they change.
Illustration :
https://github.com/user-attachments/assets/f17704ec-8104-4b86-93eb-e3f884633294

Hi! I will take a look at this 👌

Hi again! This issue should be fixed in the next release 🙂


Blank content when previewing in mail attachment

Hi. I am having this issue. Everything works fine except when I attach the file to my mail.
Browser: Google Chrome 67.0.3396.99 (Official Build) (64-bit)
Mail service: Gmail
Steps to reproduce the issue:

Open the demo page: http://evidenceprime.github.io/html-docx-js/test/sample.html
Download the .docx from demo page.
Go to Gmail, compose a new mail, attach the .docx above, then send it to yourseft.
Go to inbox, click on the attachment to view the .docx in a preview page.

Result: The content is blank. If I click on the "Open with Google Docs", the content is still blank.
Note that if I download the file from attachment and open it using Microsoft Office then there isn't any problem.

Close because of duplicate
https://github.com/evidenceprime/html-docx-js/issues/76


Various Improvments

Various improvements around the Binder API, Actions and the @Installs annotation.
This started out as a branch to look into #155 but various other improvements were made instead.

CI was broken but passed locally.

CI was broken but passed locally.


refact(cstor-operator): add missing upgradetasks CRD

Signed-off-by: prateekpandey14<EMAIL_ADDRESS>

Can you add this to https://github.com/openebs/charts/blob/gh-pages/cstor-operator.yaml as well.


avoid deprecated "out-of-band" authentication flow in auth.py

Per https://github.com/fsspec/gcsfs/issues/332, this code is possibly unused, so feel free to ignore/close.
The "out-of-band" OAuth 2.0 dance has been deprecated. In Python, this refers to the run_console flow. Instead, the run_local_server flow should be used.
https://github.com/fsspec/gcsfs/blob/8dec37d9c46d3f37255e0104077bb01aaace42f3/gcsfs/credentials.py#L186
You might be interested in https://github.com/googleapis/google-auth-library-python-oauthlib/issues/184

Seems to work nicely, thanks.


[GOBBLIN-2009] set perm and acl correctly

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-2009

Description

[x] Here are some details about my PR, including screenshots (if applicable):
From my observation, setting an ACL may change the file permission if it is not the same as permission. But changing permission does not change the ACLs; it may update the mask value which changes the effective permission of the ACL entry though.
Thus in order to achieve same ACL and same permission in destination as source, we should first apply ACLs and then apply permissions.

hdfs dfs -ls /tmp/x3
-rw-rwxr--   3 griddata users          0 2024-02-29 20:15 /tmp/x3

hdfs dfs -getfacl hdfs://ltx1-holdemnn01/tmp/x3
# file: hdfs://ltx1-holdemnn01/tmp/x3
# owner: griddata
# group: users
user::rw-
user:abora:r-x
group::rwx
mask::rwx
other::r--

hdfs dfs -setfacl -m group::r-x /tmp/x3
hdfs dfs -ls /tmp/x3
-rw-r-xr--   3 griddata users          0 2024-02-29 20:15 /tmp/x3

[permissions changed by changing acl]
hdfs dfs -chmod 674 /tmp/x3
hdfs dfs -getfacl hdfs://ltx1-holdemnn01/tmp/x3
# file: hdfs://ltx1-holdemnn01/tmp/x3
# owner: griddata
# group: users
user::rw-
user:abora:r-x
group::r-x
mask::rwx
other::r--

[ACLs did not change, mask changes though (actually not in this example, but it does if needed)]
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
LocalFileSystem does not have ACL feature, so it cannot be tested.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

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

Project coverage is 41.54%. Comparing base (a78ee54) to head (4625dac).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3887      +/-   ##
============================================
- Coverage     46.67%   41.54%   -5.13%
+ Complexity    11154     2198    -8956
============================================
  Files          2219      472    -1747
  Lines         87657    20046   -67611
  Branches       9621     2327    -7294
============================================
- Hits          40911     8328   -32583
+ Misses        43055    10835   -32220
+ Partials       3691      883    -2808

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


Consider moving DefaultConfigurationService and AnnotationControllerConfiguration into core module

The initial thinking was that there were not needed in the core but not having them is actually causing more problems than having them there.

Actually was a little too optimistic about it. I had forgotten about the fact that AnnotationControllerConfiguration requires an annotation processor to run to work properly, which shouldn't be brought into the core module, so this wouldn't be as straightforward as initially thought. We would need a "pure" runtime annotation processing version of AnnotationControllerConfiguration but then the question becomes how to share the behavior between the modules.


Add doc-comments for configuration APIs

Coherence-Signed#75
I need some help on a couple of these so pls leave comments.
@Eilon @HaoK @pranavkm @davidfowl @loudej

Hi @DamianEdwards, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@divega

ping @Eilon @lodejard

Will resend this against release

Hang on. Will move all comments to the new PR.


Crashing upon placing Block

Describe the bug
Placing the Creative Resonance Extractor, or the regular resonance Extractor, gear, diorite...
To Reproduce [Optional]
Steps to reproduce the behavior:
1.Place resonance extractor
2. See error
**Expected behavior **
Not crashing
Additional context
I have all required mods+JEI (and Configured for the second test).
exit code of "-1"
Without "configured"
crash-2022-09-08_09.16.35-client.txt
With "configured"
crash-2022-09-08_09.19.31-client.txt
With regular extractor:
crash-2022-09-08_09.44.11-client.txt
Beam Reflector:
crash-2022-09-08_09.45.51-client.txt
Gear:
crash-2022-09-08_09.47.58-client.txt
Diorite:
crash-2022-09-08_10.22.08-client.txt

The issue is due to an incompatibility between your Forge version (41.1.0) and your Patchouli version (1.19-74).
See:

Closed because this is not a Crossroads bug.


Putting in the actual host instead of <IP_ADDRESS>

I found that putting in the actual host, in my case IP address instead of <IP_ADDRESS> get CORS working. <IP_ADDRESS> as host returns no results. An update on the docs might help

I don't understand your statement here. Can you clarify what you mean?

On your readme.md, you have this under Example:
var host = process.env.PORT ? '<IP_ADDRESS>' : '<IP_ADDRESS>';
I think it would help to state that people need to replace <IP_ADDRESS> with their domain/IP eg www.example.com

You should be using <IP_ADDRESS> if the server is public.
Concidentally, someone else has submitted a PR here at #60 (see also the discussion there).


fix: The enter syscall scopes hint is wrongly formatted.

Problem: The enter syscall scopes hint was wrongly formatted.
Solution: Fix to the expected state.
Issue Number: N/A
Type

[ ] feature
[x] bugfix
[ ] dev (no functional changes, no API changes)
[ ] fmt (formatting, renaming)
[ ] build
[ ] docs
[ ] testing

Description
Breaking changes?

[ ] yes
[x] no

@whichqua this is a regression to cairo-lang 0.13.2.
The problem wasnt the formatting, but that 'component_hashes': os_input.declared_class_hash_to_component_hashes, (a new input for the OS) was added
cc @shamsasari


Fixing dependencies listed in requirements.txt file

Lucas, I simplifed the requirements.txt. Can you go into one of your isolated systems (one with miniconda and a rust compiler), and run
pip install -r requirements.txt and pip install . to see if anisoap properly installs?

I'm going to close this PR because it's confusing.
Let's try to treat documentation updates like feature branches which get deleted after merged into main.


bad SQL in PredictTipSciKitPy

Msg 512, Level 16, State 1, Procedure dbo.PredictTipSciKitPy, Line 5 [Batch Start Line 0]
Subquery returned more than 1 value. This is not permitted when the subquery follows =, !=, <, <= , >, >= or when the subquery is used as an expression.
Msg 39004, Level 16, State 20, Line 2
A 'Python' script error occurred during execution of 'sp_execute_external_script' with HRESULT 0x80004004.
Msg 39019, Level 16, State 2, Line 2
An external script error occurred:
Error in execution.  Check the output for more information.
Traceback (most recent call last):
File "", line 5, in
File "D:\ProgramData\MSSQLDEVSERVER\Temp-PY\Appcontainer1\01A307C0-EF9E-4752-B979-B1480B68445F\sqlindb_0.py", line 43, in transform
mod = pickle.loads(lmodel2)
TypeError: a bytes-like object is required, not 'NoneType'
SqlSatelliteCall error: Error in execution.  Check the output for more information.
STDOUT message(s) from external script:
SqlSatelliteCall function failed. Please see the console output for more information.
Traceback (most recent call last):
File "D:\MSSQL15.MSSQLDEVSERVER\PYTHON_SERVICES\lib\site-packages\revoscalepy\computecontext\RxInSqlServer.py", line 605, in rx_sql_satellite_call
rx_native_call("SqlSatelliteCall", params)
File "D:\MSSQL15.MSSQLDEVSERVER\PYTHON_SERVICES\lib\site-packages\revoscalepy\RxSerializable.py", line 375, in rx_native_call
ret = px_call(functionname, params)
RuntimeError: revoscalepy function failed.

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

ID: 5a3289a4-d387-44cc-3e1e-4eadebf26472
Version Independent ID: 292e69a9-c3b0-833e-138b-4522b00430f2
Content: Python tutorial: Run predictions in SQL stored procedures - SQL machine learning
Content Source: docs/machine-learning/tutorials/python-taxi-classification-deploy-model.md
Product: sql
Technology: machine-learning
GitHub Login: @garyericson
Microsoft Alias: garye

@WilliamDAssafMSFT Would you be able to have a look at this? Thank you.

@wvaughn409 We will need to investigate with engineers. Please provide SQL Server version/edition information, OS, as well as the tools you are using to execute. The more information you can provide the better.

@wvaughn409 Can you follow up with environmental information?


Is the express .env exercise correct?

Description
Extract from the exercise:

The .env file is a hidden file that is used to pass environment variables to your application.
...

Link to the exercise.
We write  a .env file holding a VAR=something. We can console log this variable, by running "Run" in a repl.it file.
But this is not the standard Node.js behavior. npm start or node server.js won't load any variable unless we include the dotenv package. So If any user tries to emulate that project locally, it won't work the same.
I may be missing something simple, but just find it strange, and asking in repl.it didn't help much so far.
Add a Link to the page with the problem:
Because it's a hidden file the only way is to build it yourself. A way is to create a new repl.it and a .env file, then console logging the value.
Directory Tree Image

Can you assign this issue to me ?

@TAbhay We have yet to determine if this is a bug.  We will attach a help wanted label once we make the determination of what needs to done.

To address the issue:
I believe repl.it (like Glitch) parses the environment variables behind the scenes. It is true that local development (and possibly some hosting providers) will require the use of the dotenv package (or something similar). It is worth adding a note to this challenge explaining this?

@nhcarrigan This is the same issue that was present on the Advanced Node challenges (and probably many others).  Maybe we just need to update the boilerplate to already have the dotenv package in package.json and add the require('dotenv').config() line as the first line of the myApp.js file.

To address the issue:
I believe repl.it (like Glitch) parses the environment variables behind the scenes. It is true that local development (and possibly some hosting providers) will require the use of the dotenv package (or something similar). It is worth adding a note to this challenge explaining this?

I - as someone that is learning - believe it is. But I cant speak for everyone. Also, I asked on repl.it, and no one gave me a clear answer of what happens when you press "Run" (cause it should just run node start but something else it's going on there).
And if you open up the event run from "Run" it's just imposible to understand (for me at least).

Just do

npm install dotenv
Add require('dotenv').config() in myApp.js file
Add .env file with PORT = 3000
Now npm start
It will work

https://repl.it/join/knrubabi-tabhay1

Just do
1. **npm install dotenv**

2. Add **require('dotenv').config()** in myApp.js file

3. Add **.env** file with **PORT = 3000**

4. Now **npm start**
   It will work

If you read the issue, you'll find out I've done that, but that should be clear from the exercise. Plus that's not the main problem, the problem is that because something is loaded in the background, understanding how it works is made more difficult. For example, do you know why console logging a .env variable runs fine without the .dotenv package? You might do, I doubt someone that is learning will.

@RandellDawson I think that's probably the best approach. 🙂

That makes sense to me, too.  We'd like to be as platform agnostic as possible and dotenv helps with that.
It might be worth re-wording the challenge slightly, too:

This file is secret, no one but you can access it, and it can be used to store data that you want to keep private or hidden.

to

This file should be kept out of version control, then it can be used to store data that you want to keep private or hidden.

and

The environment variables are accessible from the app as

to

The dotenv package makes the contents of .env accessible in the app as

Sounds like we are set on:

Add dotenv to the package.json
Require it in the appropriate location
Use Oliver's wording (above), for clarity

Hey guys, linked a pr for adjusting the boilerplate code above, if you'd like to review.

I just finished the APIs and microservices cert. Many of those projects used a "sample.env", which didn't work at all. I needed to rename on my local and drop into the repl.it project. The require('dotenv') is there.

Many of those projects used a "sample.env", which didn't work at all. I needed to rename on my local and drop into the repl.it project.

This is standard behaviour - the sample.env file exists to show the required env values. You can copy or rename it to .env and fill in the secrets. 🙂

Do we want to go through and add the dotenv package to all of the boilerplate repositories that rely on environment variables?

afaik in repl.it they put all effort in making dotenv not work as expected. imho the best way is to explain the user how it should be done locally, and that this is already set in the background by repl.it.
So dotenv should be removed as a package, and .env will work the same.
I may be wrong.

dotenv feels like a hack. repl.it should have a built in way to add environment variables to containers -- or whatever they are using. Beyond control of this repo, I suppose.

Repl.it handles environment variables natively. However, dotenv is required for local development, which is where this discussion is going, I believe.
Testing this with a brand new repl.it instance, and I am able to log environment variables without dotenv.

Do we want to go through and add the dotenv package to all of the boilerplate repositories that rely on environment variables?

If we teach the user to user about using dotenv (I can not remember if we do), then the boilerplates for the projects should not come with it.  The projects should have the absolute minimum possible and the user adds the necessary libraries/code to implement the projects.

From a quick glance, we don't seem to mention the dotenv package currently.
Would it be a better approach to modify the "Use the .env File" challenge to include instructions on using dotenv for local development?

Would it be a better approach to modify the "Use the .env File" challenge to include instructions on using dotenv for local development?

I like this idea.

Okay, so we could add the following text to the challenge file at the end of the Instructions section:
If you are working on this challenge locally, you will need to use the `dotenv` package for node to properly read the `.env` file. Install the package with `npm install dotenv` in your terminal. Then, at the top of your `myApp.js` file, import the package with `const dotenv = require('dotenv') and load the environment variables with `dotenv.config()`.

@RandellDawson if you're happy with this verbiage/approach I think we could open this one up to first-timers?

you know better but saving a line here is justified requires('dotenv').config()
because dotenv is not needed for anything else afterwards

On 15 Dec 2020, at 23:26, nhcarrigan<EMAIL_ADDRESS>wrote:
Okay, so we could add the following text to the challenge file at the end of the Instructions section:
If you are working on this challenge locally, you will need to use the dotenv package for node to properly read the .env file. Install the package with npm install dotenv in your terminal. Then, at the top of your myApp.js file, import the package with const dotenv = require('dotenv') and load the environment variables with dotenv.config()`.
@RandellDawson if you're happy with this verbiage/approach I think we could open this one up to first-timers?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

While you are correct, require('dotenv').config() works just fine, I'd argue that the two line approach will be more clear and understandable for campers at this point in the curriculum.

@nhcarrigan I am leaning towards the one liner, since that is typically how I have seen it used.  I also think we can just state the user should install the dotenv package to properly read the .env file and take out the part about "if working on the challenge locally".  If the user installs the package on repl.it or Glitch, it will not hurt anything. How about changing the Instructions section to the following:

Let's add an environment variable as a configuration option.
Store the variable MESSAGE_STYLE=uppercase in the .env file. Then tell the GET /json route handler that you created in the last challenge to transform the response object’s message to uppercase if process.env.MESSAGE_STYLE equals uppercase. The response object should become {"message": "HELLO JSON"}.
You will need to install the dotenv package  using npm install dotenv in your terminal (if not using repl.it). Then, at the top of your myApp.js file, import and load the environment variables with require('dotenv').config().

That approach looks good to me. 🙂

@santimirandarp Any thoughts on my last suggestion above?

@santimirandarp Any thoughts on my last suggestion above?

@santimirandarp Any thoughts on my last suggestion above?

Looks good to me. But since repl.it is a bit unpredictable, I'd test there are no troubles if the user actually includes the package, I'll test it now.

@santimirandarp Any thoughts on my last suggestion above?

Looks good to me. But since repl.it is a bit unpredictable, I'd test there are no troubles if the user actually includes the package, I'll test it now.

Just to mention/remention it: On the forum, it comes up surprisingly often that campers get confused that a .env file needs to be created. So, could we change @RandellDawson suggestion to include this:

Let's add an environment variable as a configuration option.

Create a new .env file, and store the variable MESSAGE_STYLE=uppercase in it. Then tell the GET /json route handler that you created in the last challenge to transform the response object’s message to uppercase if process.env.MESSAGE_STYLE equals uppercase. The response object should become {"message": "HELLO JSON"}.

You will need to install the dotenv package using npm install dotenv in your terminal (if not using repl.it). Then, at the top of your myApp.js file, import and load the environment variables with require('dotenv').config().

Just to mention/remention it: On the forum, it comes up surprisingly often that campers get confused that a .env file needs to be created. So, could we change @RandellDawson suggestion to include this:

Let's add an environment variable as a configuration option.

Create a new .env file, and store the variable MESSAGE_STYLE=uppercase in it. Then tell the GET /json route handler that you created in the last challenge to transform the response object’s message to uppercase if process.env.MESSAGE_STYLE equals uppercase. The response object should become {"message": "HELLO JSON"}.

You will need to install the dotenv package using npm install dotenv in your terminal (if not using repl.it). Then, at the top of your myApp.js file, import and load the environment variables with require('dotenv').config().

@RandellDawson Was this issue discussed before repl.it removed the ability to add .env files?

@paulywill Sorry for butting in, but yes. We did try to clarify that with this line:

Note: If you are using Replit, you cannot create a .env file. Instead, use the built-in SECRETS tab to add the variable.

We can probably close this issue as stale, because there have been changes to the lesson, and it comes up slightly less frequently.

Thank you for reporting this issue.
This is a standard message notifying you that the issue has become stale, so we are going to close it.
If you think we're wrong in closing this issue, please request for it to be reopened. Thank you and happy coding.


Update README.md with another project link

https://github.com/sbarisic/nanovg_dotnet

Thanks for the link!
Another option for the API style could have been html canvas style:
NanoVGContext ctx = NVG.CreateContext(...);

ctx.BeginPath(...);
ctx.Fill(...);

Thanks for the tip, i created a new NanoVGContext struct and made all of the functions as extension methods. So both API styles are now available. 😄


Do not use OnBootSec in the .timer file for updates since OnCalendar is already defined

The next run for the OnCalendar timer is calculated when the timer is first installed and is unaffected by previous runs triggered by the OnBootSec timer.
With the current implementation, if I reboot my computer at 05:50, the update service will be executed at 05:55 triggered by by the OnBootSec timer. It will then be executed again at 06:00, triggered by the OnCalendar timer. The previous run that was triggered by bootup will not affect the next run that happens at 06:00.
If your intent was to introduce a delay before the service is started, a better approach would be to use ExecStartPre=/usr/bin/sleep 300 in the .service file.
If this is the intended behaviour, let me know and I will close the issue.

I've been carrying the flatpak ones at least for a while without any real thorough review. My service unit knowledge is out of date and could use some peer review with how we're doing it.
The intended behavior is for the flatpaks to update on a reasonable schedule without bothering the user, should we just keep the OnCalendar and not bother with the OnBootSec?

If you remove the OnBootSec instructions and keep the OnCalendar instruction, the service will trigger on hours that are multiple of 6 (0000, 0600, 1200, 1800). I think you should remove the OnBootSec instruction.

I agree with that. Both Flatpak and rpm-ostree should update at the same time cohesively. I think running updates sometime at night and again later if the updates failed would be the best approach
Having Flatpaks update 4 times a day isn't really necessary and the fact that rpm-ostree just outright fails without retrying isn't really acceptable
So, I think our aim here should be to adjust the current update policy for Flatpaks and rpm-ostree so that they run at 4 in the morning (keeping TZ in mind) and retry later on failure

@bsherman

Whatever we want, we should do it for rpm-ostree updates as well as flatpak so they behave in a similar manner.

If we do this consistently for all 3 units... then we can provide simple instructions for users how to change the behavior, either to disable or change schedule, etc.

I think a simpler solution is to use a single foo.timer which actives foo.service. foo.service would then start flatpak-system-update.service, flatpak-user-update.service, and rpm-ostreed-automatic.service; which can be accomplished by adding Wants=flatpak-system-update.service flatpak-user-update.service rpm-ostreed-automatic.servicedirective tofoo.serviceandAfter=foo.service` directive to the other three. So all three services start when a single timer is reached. [systemd.service]
The three services could also be run one after the other and only after the previous has executed to completion using the After= and Type=oneshot directives.

nightly updates at 4am (should respect system timezone setting)

if an update is missed due to power off or sleep, immediately run on boot or when next available

Both of these are the default behaviour for timer units so no change would be required. If the timer is reached when the computer is turned off, the timer activates upon the next boot.
@EyeCantCU

the fact that rpm-ostree just outright fails without retrying isn't really acceptable

I think adding a Restart=on-failure directive would address this. The service should be restarted any time rpm-ostree exits with a non-zero status.

I appreciate the feedback @un1c3 !
I very much like the suggestion of a single timer and using simple service dependencies to run the three services.
I feel a bit hesitant to modify rpm-ostreed-automatic.service or trigger it from a different timer than the default rpm-ostreed-automatic.timer`, but since the timer is not enabled by default, that could be fine.
I'm going to tinker with an implementation based on these suggestions, as I do think we need this cleaned up.

Well, I don't think the idea works for flatpak-user-update.service to be Wanted by a foo.service since the former is a --user service.
Additionally, we already have these 3 distinct timers so I'm inclined to keep them rather than removing the 2 flatpak timers, only to add a new service and timer, while leaving the Fedora default rpm-ostreed-automatic.timer as disabled.
It also adds complexity.
I'll have a PR shortly with a simple implementation of what @EyeCantCU and I had already discussed and incorporate the restart on failure suggestion.

In the end, I didn't change the  rpm-ostreed-automatic.service behavior by adding Restart=on-failure.  realistically, this is a daily update. if it fails, it seems fine to try the next day rather than retrying several times in rapid succession.
Overall the solution as presented in the linked PR strives to achieve the goal in the simplest possible manner.


Unable to load preload script @capacitor\electron\dist\electron-bridge.js

Describe the bug
Just migrated from electron 8.5.5 to 19.0.9 for an electron ionic desktop application project. When trying to open electron window with the command 'npx cap open electron', it throws an error: Unable to load preload script<EMAIL_ADDRESS>Implementation
mainWindow = new BrowserWindow({
    width: 400,
    height: 700,
    resizable: false,
    title: "Desktopia",
    show: false,
    icon: path.join(__dirname, 'favicon.ico'),
    webPreferences: {
      backgroundThrottling: false,
      nodeIntegration: true,
      preload: path.join(__dirname, 'node_modules', '@capacitor', 'electron', 'dist', 'electron-bridge.js')
    }
  });

Expected behavior
Electron window should open with the command 'npx cap open electron'
Screenshots

Desktop (please complete the following information):

OS: Windows
Version : 10

'npx cap open electron' is referencing discontinued electron platform, and not the capacitor-community platform where you are writing this issue. If you want to use this capacitor-community platform read the docu

npx cap copy @capacitor-community/electron is same error....


Configure systemd to send SIGQUIT on stop

(Thank you for contributing to Caddy! Please fill this out to help us make the most of your pull request.)
1. What does this change do, exactly?
This updates the systemd unit to configure the use of SIGQUIT to trigger a graceful shutdown when the service is stopped.
2. Please link to the relevant issues.
I didn't create an issue for this, but I did look at open issues matching "systemd", and none of them appeared to be relevant.
3. Which documentation changes (if any) need to be made because of this PR?
The exact behavior of systemctl start caddy, systemctl reload caddy, and systemctl stop caddy are not currently discussed in the README. This modification does not, in my opinion, violate the principle of least surprise for the behavior of systemctl stop caddy.
4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I am willing to help maintain this change if there are issues with it later

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

I haven't forgotten about this PR, @gmacon -- thanks for your contribution. I'll catch up on PRs later this week, or more likely next week as I won't be on the computer much for a bit.


Add support for Azure, OpenAI, Palm, Anthropic, Cohere, Replicate Models - using litellm

This PR adds support for models from all the above mentioned providers using https://github.com/BerriAI/litellm
TLDR: querybook gets:

more models out of the box -  Azure, OpenAI, Palm, Anthropic, Cohere, Replicate
instant integration of models by modifying the model param (Easy to add models in the future, just change the model param)
log I/O to Posthog, Sentry, Helicone, Supabase without making any code changes & get API Key management

Here's a sample of how litellm completion is used:
from litellm import completion

## set ENV variables
# ENV variables can be set in .env file, too. Example in .env.example
os.environ["OPENAI_API_KEY"] = "openai key"
os.environ["COHERE_API_KEY"] = "cohere key"

messages = [{ "content": "Hello, how are you?","role": "user"}]

# openai call
response = completion(model="gpt-3.5-turbo", messages=messages)

# cohere call
response = completion("command-nightly", messages)

# anthropic call
response = completion(model="claude-instant-1", messages=messages)

@czgu can you please take a look at this PR when you get the chance ? Happy to add docs/testing if this initial commit looks good to you😊

@ishaan-jaff  thanks for your PR! we'll have an update of the LLM feature soon, will revisit this PR after that.

@jczhong84 any update on this ?

@jczhong84 What was LiteLLM missing to be useful to you? Any feedback here would be helpful


23 collect sample input files

At present our client's focus is working on 2 different files data i.e. ORCA and Gaussian. We have the software running for ORCA file data but it doesn't work on the Gaussian files data. We received the latest Gaussian files from Dr. McCulla and the latest ORCA files from Dr. McCulla's student Peter Maness. Dr. McCulla is fine with the files being uploaded in the open source since they are already published data. For this issue, the inputs directory has been created to contain the working and broken subdirectories. The broken subdirectory, since the software doesn't work for it, contains the 12 Gaussian files data that the client provided. The working subdirectory contains the new 8 ORCA files data.

Looks good to me. All the files are segregated into working and broken sub-directories.


Feature: Lair Actions Pop up

On turn change, check current init and previous init to decide if its the Lair's turn.
Scour current combatant list (bleh) for lair action items and populate a dialog with item.roll() callbacks.

I can work on this, I would suggest instead hook on createCombatant (or whatever its called) and check for lair actions and add to a map stored globally, or on the combat itself.

Flagging info on the combat itself sounds like the right approach. It would
allow us to manage multiple combats easier :stress:
On Wed, Mar 10, 2021 at 2:49 PM kandashi @.***> wrote:

I can work on this, I would suggest instead hook on createCombatant (or
whatever its called) and check for lair actions and add to a map stored
globally, or on the combat itself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/trioderegion/dnd5e-helpers/issues/74#issuecomment-795987183,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADRQOM3CMVJ2KGW56FNC3NDTC7EMTANCNFSM4YYOYNJQ
.

Will be avaliable in 1.11.0

Adding Legendary Action management to this too.


Attempting to perform an infinite precision BigDecimal exponentiation

The readme mentions that division is the only operation which cannot be performed with infinite precision, but the exponentiation feature uses division internally, which therefore makes exponentiation susceptible to the same problems as division.
Is this a documentation issue or a bug?

Hi @JoonasC, good catch, this is a documentation issue, it should be mentioned that exponentiation with negative exponents has the same limitation as division. The reason for the limitation is that results of some divisions can be infinitely long, i.e. 1/3 = 0.333333... and the library needs to have way to stop instead of calculating to infinity (or more realistically until it runs out of memory)

Could you also implement an exponentation method which accepts a decimal mode?

Sure, I'll look into that as well!


ENH: add leaves_color_list for cluster.dendrogram dictionary.

Reference issue
fix #12190
What does this implement/fix?
I added a new key leaves_color_list in the dendrogram dictionary informing about color leaves.
This PR enables for users to retrieve leaves colors from it.
Additional information
This is a sample usage:
import matplotlib.pyplot as plt
import numpy as np

from scipy.cluster.hierarchy import linkage, dendrogram

x = np.array([[5, 3],
              [10, 15],
              [15, 12],
              [24, 10],
              [30, 30],
              [85, 70],
              [71, 80]])
z = linkage(x, 'single')
d = dendrogram(z)
points = d['leaves']
colors = d['leaves_color_list']
for point, color in zip(points, colors):
    plt.plot(x[point, 0], x[point, 1], 'o', color=color)
plt.show()

Looks like this works as advertised. I'm still slightly worried about that counting logic so would like to test on some real-world data. Let's ping @alejandro-ariza who requested this feature. @alejandro-ariza can you please test this implementation?

There's also some bio data at https://github.com/scipy/scipy/issues/5238#issuecomment-249499573 that's a nice test, with some more colors.

There's also some bio data at #5238 (comment) that's a nice test, with some more colors.

I tried this nice test.
This is the code:
import matplotlib.pyplot as plt
import numpy as np
import pandas as pd
from scipy.cluster.hierarchy import linkage, dendrogram

gasch_data = 'experiment_data.txt'
dists = pd.io.parsers.read_table(gasch_data, index_col=0)
dists = np.asarray(dists)
Z = linkage(dists, 'ward')
plt.figure()
dn = dendrogram(Z, truncate_mode='lastp',
                p=40, show_contracted=True,
                orientation='right')
labels = np.array(dn['ivl'])
colors = np.array(dn['leaves_color_list'])

for color in np.unique(colors):
    print(color, ":", labels[np.where(colors == color)])

This is the dendrogram figure

This is the output of the code:
C1 : ['(39)' '(112)' '(97)' '(92)' '(92)' '(41)']
C2 : ['(104)' '(58)' '(11)' '1356' '(22)' '(19)' '(44)']
C3 : ['(37)' '(17)' '(38)' '(233)' '(102)' '(90)' '(80)' '(291)' '(14)' '(146)'
 '(146)' '(145)' '(121)' '(100)' '(130)' '(262)' '(244)' '(16)' '(280)'
 '(220)' '(178)' '(141)' '(72)' '(72)' '(317)' '(460)' '(83)']

The code looks work correctly.

Hi.  I'm trying to use leaves_color_list to color the labels in my dendrogram plot.
Code
import numpy as np
import matplotlib.pyplot as plt
from scipy.cluster.hierarchy import linkage, dendrogram

# Data
x = np.array([[1, 0, 0],
              [0, 0, 1],
              [0, 2, 0],
              [0, 0, 1],
              [0, 1, 0],
              [0, 1, 0]] )

# DENDROGRAM
plt.figure( figsize=( 15, 3 ) )
plt.box( False )
z = linkage( x, "single" )
d = dendrogram( z )

for label, leaf_color in zip( plt.gca().get_xmajorticklabels(), d["leaves_color_list"] ):
    label.set_color( leaf_color )

print( d["leaves_color_list"] )

Output

Problem
You can see that the left-most label is not the right color in a few cases: 4 and 1 .
Is the nature of my data unsupported for creating a dendrogram?   (My matrices are normally much larger and the non-zero values are usually low-value integers.  But I wanted to create a simple example for the repro here.)
Or might there be a bug in the generation of the leaves_color_list?
If it is a bug, can you tell me where to open a new issue so this can be fixed?
Thanks!

@spackows Thank you for reporting. I'm not sure the behavior is expected or not, but the doc of dendrogram says:
https://github.com/scipy/scipy/blob/4cf21e753cf937d1c6c2d2a0e372fbc1dbbeea81/scipy/cluster/hierarchy.py#L3253-L3254
I think your x[: ,2] is not monotonic

[0 1 0 1 0 0]

Is it expected?

Hi.
Thank you for your reply.

It is expected that the distances in Z[:,2] be monotonic ...

Is it x[ :, 2 ] - the input matrix of data passed to linkage - that needs to be monotonic?
Or is it z[ :, 2] - the result from linkage, the distance matrix passed to dendrogram - that needs to be monotonic?
The docs say "Z".. which makes me think they mean the later.
When I run this code:
x = np.array([[1, 0, 0],
              [0, 0, 1],
              [0, 2, 0],
              [0, 0, 1],
              [0, 1, 0],
              [0, 1, 0]] )

z = linkage( x, "single" )

print( z[ :, 2 ] )

I get this output, which is monotonic:
[ 0.         0.         1.         1.41421356 1.41421356 ]

... otherwise  crossings appear in the dendrogram.

The problem with leaves_color_list isn't that it's out of order (eg. what I might expect to happen if the lines crossed.)

Hypothesis
It looks like an edge case where the distance is zero.
When I change one value in the input data so the distances is not quite zero, the leaf color is correct.
Code
x = np.array([[1, 0, 0],
              [0, 0, 1.1],
              [0, 2, 0],
              [0, 0, 1],
              [0, 1, 0],
              [0, 1, 0]] )

plt.figure( figsize=( 15, 3 ) )
plt.box( False )
z = linkage( x, "single" )
d = dendrogram( z )

for label, leaf_color in zip( plt.gca().get_xmajorticklabels(), d["leaves_color_list"] ):
    label.set_color( leaf_color )

print( d["leaves_color_list"] )

Output

Ah, sorry. I misunderstood. You are right.
Could you please create an issue for your problem?
I will try to check it deeply.

Thank you for your attention to this discussion.
I opened this issue to investigate the zero-distance problem: https://github.com/scipy/scipy/issues/16558
*I have never contributed to scipy before.  But I would be happy to help with this problem, if I can - such as by trying workarounds or testing a fix.
Thank you again!


change nav item logout link from get link to post button

As this is the first time tring to contribute OSS, sorry in advance if any of my procedures are not appropriate.

navbar logout link changed to post form.
Description

example/templates/_base.html fixed to:

<form action="{% url 'logout' %}" method="post" style="display: inline;">
    {% csrf_token %}
    <button type="submit" class="nav-link" style="background: none; border: none; padding-top: 8px; margin: 0; text-decoration: none; cursor: pointer;">
        {% trans "Logout" %}
    </button>
</form>

Motivation and Context

original link below causes 405 response.
{% trans "Logout" %}
How Has This Been Tested?

the test below actually ends with some errors but not differ from origin/master's result.
python .\example\manage.py test --settings=tests.settings

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hi, I fixed it.
Think that 'class="nav-link btn btn-link pt-2 m-0 text-decoration-none"' enough.
I visually confirmed that the nav-link works correctly in both expanded and collapsed states.

So I launched local sever and checked if the class propery works.
Think it's enough.


[FEATURE] Adjust SfBreadcrumbs to small screens

Describe how this feature will help us improve
Currently, SfBreadcrumbs component is not adjusted to small screens, especially when there is a lot of breadcrumbs. When there is no place to display all breadcrumbs, they are are cut off.
Describe the solution you'd considered
In my option, the best solution will be add the possibility to wrap breadcrumbs in a second line, when there is no place. We can achieve it in a very simple way:

Add flex-wrap: wrap; CSS rule to .sf-breadcrumbs__list
Replace before pseudo-element, which is used to display separator, to after pseudo-element
Add bigger line-height

These changes allow us to achieve something like this, where there is no place to display breadcrumbs in one line: https://prnt.sc/sp809s

Multiline breadcrumbs added. I left line height unchanged, because it looks good imo, but it need to be designed @mbaran-dvnt


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.4.126",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=f9e7a38d148eb660dbcfad40bfb60dcc16997b15"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "f9e7a38d148eb660dbcfad40bfb60dcc16997b15",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "wq: handle VM suspension in stall detection\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.126 by commit f9e7a38d148eb660dbcfad40bfb60dcc16997b15. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000888


Inverting Follow / Following displaying on other users

In some cases, when I click on follow, it removes me following the other user and visa versa for clicking on following hence inversion.

should be fixed now


Improving the documentation

We are now working on improving the documentation for Proto.Actor.
What do you think should be improved, in which areas?
Please come with suggestions.

removing obsolete part could be good
I am trying to use simplescheduler but apparently it's obsolete and apis disappeared (example : context.AwaitTask)


Support content:encoded field

For medium, content:encoded is used.
https://medium.com/feed/invironment/tagged/food
https://github.com/rbren/rss-parser/issues/166

Thank you for reporting this, I have added support for content:encoded in version 0.5.5.
The content can be found in the Feed Entry field content.value.
Please try out the fix and let me know if it helped.

@MikaelPorttila
I tried. It's working well. 😄
Thanks you! 👍


Update tokio and futures crates

Fix the vulnerable version of the tokio crate (RUSTSEC-2021-0072).
Update the yanked version of the futures crate.

[x] This PR has been added to CHANGELOG.md (at the top of the list);

Codecov Report

Merging #87 (8583318) into main (49dcb88) will increase coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head 8583318 differs from pull request most recent head 3f9954a. Consider uploading reports for the commit 3f9954a to get more accurate results

@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   78.82%   78.83%
=======================================
  Files          49       49
  Lines        2938     2939    +1
=======================================
+ Hits         2316     2317    +1
  Misses        622      622

Impacted Files
Coverage Δ

datanymizer_engine/src/utils.rs
100.00% <0.00%> (ø)

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 ed318cf...3f9954a. Read the comment docs.


Getting accessToken never returns in latest update causing some bad issues

https://github.com/Innim/flutter_login_facebook/blob/3314600fd3da6643a3070f4aa2aada75a9ed58ef/lib/src/facebook_login.dart#L34
The linked function never returns after updating to plugin version 1.5.0 causing severe issues (e.g. you won't get a result from isLoggedIn).
I see your code changes just imply updating the underlying FB SDK. Did they make changes in the SDK you need to update accordingly to?

Hey! On which platform?

Oops, sorry should've included that info. I found the issue using a Samsung, Android 12 and my doctor output is the following:

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.1, on macOS 12.4 21F79 darwin-arm, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.67.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
• No issues found!

Just checked - all working well.
Do you made previous migration?
Is there any errors in logcat?

That's it. Missed the most recent migration 👍


[DO NOT MERGE] Update AppEngine Samples

Updating the following app.yaml files for AppEngine Samples:

[x]  Analytics Sample
[x]  Cloud SQL
[x]  Datastore
[x]  Endpoints
[x]  Hello World
[x]  Memcached - ignored
[x]  Mailgun
[x]  Metadata Server
[x]  PubSub
[x]  SendGrid
[x]  Static Files (Rails)
[x]  Static Files (Sinatra)
[x]  Storage
[x]  Twilio

Updated App Engine samples
PTAL @remi

PTAL @remi

👍  LGTM

reminder: do not merge 😄


Difference between MapTR and LaneGAP?

Hey guys, thanks for your brilliant work in both MapTR and LaneGAP. After reading this paper, it seems that the perception pipeline for LaneGAP is basically the same with MapTR? And if not, what's the difference? Hope to get your reply soon

LaneGAP and MapTR do share similar network design. LaneGAP extends MapTR to lane graph modeling. We integrade them into one unified framework in MapTRv2 (https://arxiv.org/abs/2308.05736).

LaneGAP enables MapTR percepting lane graph with better performance (superior than pixel-wise, piece_wise) at least cost (without need of post-processing in pixel-wise and extra connectivity prediction in piece-wise).


Attempt to deploy OpenShift router causes stack-create to fail

When an OpenShift stack is created with the deploy_router switch set, the services deployment fails.
heat stack-create oshift -e openshift-on-openstack/env_origin.yaml -f openshift-on-openstack/openshift.yaml -P external_network=public -P flavor=m1.shift -P server_image=centos72 -P master_server_group_policies=affinity -P master_docker_volume_size_gb=6 -P node_docker_volume_size_gb=4 -P deploy_router=True
tail -f /var/log/ansible-services-16767.log
TASK: [Evaluate oo_masters] ***************************************************
fatal: [localhost] => with_items expects a list or a set

FATAL: all hosts have already failed -- aborting

PLAY RECAP ********************************************************************
           to retry, use: --limit @/root/services.retry

localhost                  : ok=0    changed=0    unreachable=0    failed=1

Mar 24 19:20:40 oshift-infra os-collect-config[9501]: + ansible-playbook --inventory /var/lib/ansible/inventory /var/lib/ansible/services.yml
Mar 24 19:20:40 oshift-infra os-collect-config[9501]: [2016-03-24 19:20:40,080] (heat-config) [ERROR] Error running /var/lib/heat-config/heat-config-script/8cbd5259-3e31-4c56-a14e-36d096caed00. [2]

https://github.com/redhat-openstack/openshift-on-openstack/blob/master/fragments/master-ansible.sh#L234-L248

Fixed by https://github.com/redhat-openstack/openshift-on-openstack/pull/85


Updated code

Included code in calculateDifferenceBetweenTimePeriod
Included code in listoperations.py in baisc
@RKRohk please review my code.
Proposed Changes
Add a short description here

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have checked that my PR is not a duplicate

@RKRohk can you please review my changes.


(WIP) Add pending reference resolution when referenced element is not exported

If element is not exported all pending references are left in memory until transformation completes. This causes memory issues when a lot of elements are not exported. To reduce memory usage transformer can resolve pending references as soon as it knows that element won't be exported.

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

is this still WIP as marked in the title? If so can you mark it as a draft?

Sorry, forgot to remove WIP from the title. PR is ready for review.

I've also updated pipeline since tests were facing SegFault issue mentioned here: https://github.com/iTwin/itwinjs-core/issues/5663#issuecomment-1600764489

thinking about this some more, instead of keeping the pending references in memory and only updating them, can we just keep the id of the incomplete element, and after all have been inserted, re-transform all incomplete elements from scratch? So same as what we do now, but rather than update elements as soon as we find their references, wait until the end when everything is inserted, and batch update them.


Best Practice : Use constant String instead of using directly

Changed String literal directly used inside the log method to constant. Declared a private static final string and used inside the log method.

@bentrengrove @pfmaggi @alanv @melix Please review

It is not a good practice to tag people in order to get your PR reviewed. Especially when they have nothing to do with the project. Please be patient and I'm sure the maintainers of this repository will come back to you.


Buttons with an icon and text are not up to spec

The icon needs to be 18x18 and have 8px of space to the right of it, per:
https://material.io/design/components/buttons.html#specs
Here's what I used:
.mat-button-wrapper .mat-icon {
  height: 18px;
  width: 18px;
  line-height: 18px;
  margin-right: 8px;
}

In addition to that we need to reduce the button's left padding to 12px (e.g. by applying a left margin of -3px).
Should the font size of the icon be reduced to 18px?

I was going to create a separate issue for this, but this one seems related.
MatButton with icon & text is not documented on https://material.angular.io/components/button/overview
It's not clear whether developers should use a  around the label text or not and this choice affects the spacing between icon & text. I realize this is an artifact of how HTML works, but it should be documented or corrected by Material CSS.
Spacing Reduced:
In these cases, HTML does not render a space between icon & text.
<button mat-raised-button>
  <mat-icon>favorite</mat-icon>
  Favorite
</button>

<button mat-raised-button>
  <mat-icon>favorite</mat-icon>Favorite
</button>

Space between icon & text:
In these cases, HTML renders the space, which creates spacing between icon & text
<button mat-raised-button>
  <mat-icon>favorite</mat-icon>
  Favorite
</button>

<button mat-raised-button>
  <mat-icon>favorite</mat-icon>
   Favorite
</button>

@calebegg I used your workaround but had to exclude mat-fab, mat-mini-fab and mat-icon-button.
:not(.mat-fab):not(.mat-mini-fab):not(.mat-icon-button) > .mat-button-wrapper .mat-icon,
:not([mat-fab]):not([mat-mini-fab]):not([mat-icon-button]) > .mat-button-wrapper .mat-icon {
	height: 18px;
	width: 18px;
	line-height: 18px;
	margin-right: 8px;
	margin-left: -4px;
}

With the above examples, looks the icon is misaligned vertically compared with the text.


support/github runners on develop

chore: only use github machines on develop
!drop

📝 Description
Use github runners only on develop
❓ Context

Impacted projects: CI
Linked resource(s): []

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Blocked for now, but it seems https://github.com/microsoft/playwright/pull/27513 could be a solution.


Add EESI

The EESI is a french (public) art school in Angoulême and Poitiers https://www.eesi.eu/site/?rubrique162

@nathangir eesi.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Is it possible to define starting coordinates?

I would like the globe starts rotating in Asia. Is it possible to do that?
Thank you.

Just move to asia first.
like this:
GlobeCoordinates centerChina = GlobeCoordinates(20, 102.4074);
earthController.onLoaded = () {
earthController.focusOnCoordinates(centerChina);
};

@louislamme You can focus anywhere you want after FlutterEarthGlobe first initialization using onLoaded and then focusOnCoordinates as @jiezaichan suggested.

Thank you very much for both quick response. It worked like a charm. Thanks for this nice package!


chore: update github actions

Description
The node16 runner is deprecated, I updated all the actions I could here.
Still missing:
~~https://github.com/pnpm/action-setup~~ (removed in favor of corepack, but that needs to be tested)
https://github.com/technote-space/get-diff-action (archived, needs to be replaced)
https://github.com/dependabot/fetch-metadata (has PR: https://github.com/dependabot/fetch-metadata/pull/443)
https://github.com/ts-graphviz/setup-graphviz (has PR: https://github.com/ts-graphviz/setup-graphviz/pull/558)
https://github.com/thomaseizinger/create-pull-request

Will close this as stale.


Adding Google Sheets Refunds functionality to mitxonline

Pre-Flight checklist

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[x] New settings are documented and present in app.json
[x] New settings have reasonable development defaults, if applicable
[ ] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
Fix #669
What's this PR do?
Installs mitol-google-sheets, and mitol-google-sheets-refunds libraries
Adds implementation of the refund process
When google sheet is configured for processing refunds, the following process takes place:

Staff enters refund information in the configured google sheet
An scheduled task in MITx Online picks up the new change and  updates the order status to reflect the refund with a refund transaction
MITx Online calls the CyberSource API to issue the refund to the learner
MITx Online updates the google sheet indicating that the refund has been processed successfully or with an indication of failure

How should this be manually tested?
Follow the instructions here for developer setup.
When you Google Drive is set up correctly, proceed to testing:

Fill out and submit the spreadsheet request form. This should add a row to the first worksheet in the enrollment code request spreadsheet.
Run the management command to process the sheet:
./manage.py process_refund_requests -i "<spreadsheet id>". This should update the "Date Processed" column for the row you added.
Check the status of the request in the spreadsheet.

Codecov Report

Merging #723 (0506f6e) into main (9776fdc) will decrease coverage by 0.00%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main     #723      +/-   ##
==========================================
- Coverage   85.85%   85.84%   -0.01%
==========================================
  Files         124      124
  Lines        5492     5505      +13
  Branches      887      888       +1
==========================================
+ Hits         4715     4726      +11
- Misses        638      639       +1
- Partials      139      140       +1

Impacted Files
Coverage Δ

main/urls.py
81.25% <ø> (ø)

main/views.py
90.00% <75.00%> (-5.46%)
:arrow_down:

cms/models.py
72.15% <100.00%> (+0.11%)
:arrow_up:

main/settings.py
93.20% <100.00%> (+0.13%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Nested CSS font-family not working as expected

Description
Using nuxt fonts to set different fonts for specific CSS classes works perfectly for single-level classes, but when trying to apply a font to nested CSS classes, it does not load as expected.
For example, Lato works fine for .lato, but Nunito fails to apply in nested elements like .nunito p:
<template>

    'Lato works!'

'Nunito does not work :('

</template>

Reproduction steps

Apply a font family using nuxt fonts to a nested CSS class selector.

Note that the font only applies to single-level selectors, not nested ones like .nunito p.

Expected behaviour
Both fonts should be applied according to the CSS structure, with Lato on the outer div and Nunito on the inner
 element.
Reproduction link
Stackblitz repro

I assume that the AST does not run nested CSS correctly here.
If it is ok, I could make a PR if one of the maintainers agrees with this change :)
Thanks for this great module!

I could also add a new test for Nested CSS here

that would be great - thank you 🙏


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for gentle-cat-bc6937 processing.

Name
Link

🔨 Latest commit
b390068ab65dfd8cf05431371e45abcf01735536

🔍 Latest deploy log
https://app.netlify.com/sites/gentle-cat-bc6937/deploys/63315ab1bcbbec0008fc1d7f


Set sci ns to mirror JVM ns

Closes #362.

UI tests are green now but there's still issues e.g. with the slideshow viewer in the static build but that doesn't trigger a ui test failure because the render error doesn't log an error to the console atom.
As we can see in the commits to fix the ui tests, this is currently a breaking change. I'm thinking about keeping it this way though, because I think on the flip side, things are much clearer in the long term because there's no more question about what the v alias points to.

Need to follow up with good docs about this.


Add tests for Dataset repo.

Motivation and context

Code coverage tests for Dataset repo module using mock functions.
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@MashaSS , could you please check warnings from linters?

Looks good for me, but I noticed that test_clone_nonexistent_repo takes a long time, In my opinion, it is better to remove it.

Removed.


The API help links do not work

When I click in the menu of the API help, they do not work.

finished


Enhancement: Overwrite ReadOnly Files

This is really annoying when trying to deploy.
Take a look at the sample code and determine if it can be beefed up to meet this need.
           foreach (string file in Directory.GetFiles("\\"))
           {
               var ft = new FileInfo(file);
               ft.IsReadOnly = false;
           }

One could also look at doing just removing the read only file attribute instead: http://www.csharp-examples.net/file-attributes/

Fixed in d28b77a8acbca0b2a912582602a455632e1ca2de


libc version causes MacOS compilation failure

When trying to build on MacOS, cargo fails with the following error seen here (failed to find TCP_KEEPINTVL and TCP_KEEPCNT in crate libc). Fails both the CI checks and on a local machine. After doing some investigation, it seems that this PR introduced the variables to the libc crate starting with 0.2.71, whereas mio currently requires exactly 0.2.69 on all Unix builds (including MacOS).
Patching our full build with mio:0.7.7 but with libc:0.2.71 (not 0.2.72) allows the local build to pass, so it seems that the only requirement for builds to pass is to upgrade the libc version
Edit:

tested on GitHub Actions using macos-latest (10.15)
tested locally on Catalina (10.15.7 19H114)

PR for v0.7.x: #1437.

PR for v0.7.x: #1437.

Closed by #1437.

Closed by #1437.


[SPARK-47409][SQL] Add support for collation for StringTrim type of functions/expressions

Recreating original PR because code has been reorganized in this PR.
What changes were proposed in this pull request?
This PR is created to add support for collations to StringTrim family of functions/expressions, specifically:

StringTrim
StringTrimBoth
StringTrimLeft
StringTrimRight

Changes:

CollationSupport.java

Add new StringTrim, StringTrimLeft and StringTrimRight classes with corresponding logic.
CollationAwareUTF8String - add new trim, trimLeft and trimRight methods that actually implement trim logic.

UTF8String.java - expose some of the methods publicly.
stringExpressions.scala

Change input types.
Change eval and code gen logic.

CollationTypeCasts.scala - add StringTrim* expressions to CollationTypeCasts rules.

Why are the changes needed?
We are incrementally adding collation support to a built-in string functions in Spark.
Does this PR introduce any user-facing change?
Yes:

User should now be able to use non-default collations in string trim functions.

How was this patch tested?
Already existing tests + new unit/e2e tests.
Was this patch authored or co-authored using generative AI tooling?
No.

@cloud-fan some tests are stale here, but overall I think the changes in this PR are good
however, changes are indeed large and merge conflicts arise daily - could you please review this?

thanks, merging to master!


[ci skip] [pilot] Add more owners

Add more owners
NONE

It kind of defeats the purpose of having this file in pilot, doesn't it?

@andraxylia: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
66c458e6923b731c46128ee919a28aec2015845d
link
/test e2e-suite-rbac-auth

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

@andraxylia: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
a882b212e45ffdfcf6ef5113c61aa3d42a84cb99
link
/test e2e-suite-rbac-auth

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

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

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-auth.sh
a882b212e45ffdfcf6ef5113c61aa3d42a84cb99
link
/test e2e-suite-rbac-auth

prow/e2e-suite-rbac-no_auth.sh
a882b212e45ffdfcf6ef5113c61aa3d42a84cb99
link
/test e2e-suite-rbac-no_auth

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


Temporal codecs

TODO:

[x] interval codec
[ ] integration test on real postgres table

Resolves #42

@tpolecat Ready for review

Conflict to resolve, sorry. I'm trying to get caught up. This looks good but I'm kind of amazed Duration works.

Travis build fails, this seems to be a problem: https://travis-ci.community/t/error-installing-oraclejdk8-expected-feature-release-number-in-range-of-9-to-14-but-got-8/3766/6

rebased and tidied up a bit in #109


Not working on iOS.

Hi, my app runs well on Android but on iOS (iPhone 8) the player opens but doesn't play my video.
Please advise if you're aware of this and if I should change anything. Do you need my package.json contents?

Hi @delanick, I have the same problem, Are you fix it?

@liam-nirvana just managed to get it working on my side. For me the issue was my server is http not https, so you need to include your server in your info.plist file like this:
<dict>
        <key>NSExceptionDomains</key>
        <dict>
            <key>wesignit.co.za</key>
            <dict>
                <key>NSIncludesSubdomains</key>
                <true/>
                <key>NSTemporaryExceptionAllowsInsecureHTTPLoads</key>
                <true/>
                <key>NSTemporaryExceptionMinimumTLSVersion</key>
                <string>TLSv1.1</string>
            </dict>
        </dict>
    </dict>

This tells iOS that bthe server security certificate is not needed and the server can bbe trusted. This worked for me, hope it works for you too!

Thanks for following up @delanick - I had never had an issue with it and the demo worked fine so I was waiting for more information before looking into it 😄 - glad you solved on your own though. That's a good learning point with iOS development and HTTPS vs HTTP urls. I remember coming across that several years ago during Cordova development and the pain it caused forced me to never forget it 😅.

@bradmartin no worries. Yeah I hope that was the problem and solution, it's just what worked for me. I knew iOS had issues with http urls, but didn't think video content would fall into that issue, as it's just streaming/downloading a video.


Transaction linking to wrong URL

Looking at https://opencollective.com/pytorch-ignite-org - there is a single transaction, which is the expense paid from Season of Docs.

However, if you click on that expense title, you get a 404. It's trying to send you to https://opencollective.com/pytorch-ignite-org/expenses/42605 but the correct link is https://opencollective.com/season-of-docs-2021/expenses/42605.

This one was reported in https://opencollective.slack.com/archives/G46PNRCGP/p1623409906075300 and fixed in https://github.com/opencollective/opencollective-frontend/pull/6479, but we haven't deployed the fix yet


error: [Errno 13] Permission denied: 'dist\upfirdn2d-0.0.0-py3.8-win-amd64.egg'

when i run python setup.py install, occurred this kind of error

You need permission to your system disk partition. Using adminastrator mode or reinstall your python environment to other partition.

can you show me easy installation for training   step bby step ? or
ср, 2 авг. 2023 г. в 14:20, Lizhen Wang @.***>:

You need permission to your system disk partition. Using adminastrator
mode or reinstall your python environment to other partition.
—
Reply to this email directly, view it on GitHub
https://github.com/LizhenWangT/StyleAvatar/issues/22#issuecomment-1661511279,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAVLPMDN3AROGS6Z6KLKJ2DXTHPQZANCNFSM6AAAAAA3AWXRNM
.
You are receiving this because you authored the thread.Message ID:
@.***>

i have done but the same error creating 'dist\upfirdn2d-0.0.0-py3.8-win-amd64.egg' and adding 'build\bdist.win-amd64\egg' to it
error: [Errno 13] Permission denied: 'dist\upfirdn2d-0.0.0-py3.8-win-amd64.egg'


docs: add development.md to help new contributors

Add additional detail on the development workflow.
CONTRIBUTING.md reviews high-level process
docs/development.md explains detailed procedures

LGTM.


generate the dart services library as part of the build

Fix #55
@lukechurch

also @wibling

Thanks @devoncarew LGTM!

Merging so I can use it

LGTM


PRJ_COMPLETE_COMMAND_TYPE wrong enum values in API DOC

https://docs.microsoft.com/en-us/windows/desktop/api/projectedfslib/ne-projectedfslib-prj_complete_command_type
Should be
typedef enum PRJ_COMPLETE_COMMAND_TYPE {
  PRJ_COMPLETE_COMMAND_TYPE_NOTIFICATION = 1,
  PRJ_COMPLETE_COMMAND_TYPE_ENUMERATION = 2,
} ;

With the current state of the docs PRJ_COMPLETE_COMMAND_TYPE_NOTIFICATION is 0
insteadof 1.

Similar issues for  (I expect this to even be more)

PRJ_CALLBACK_DATA_FLAGS
PRJ_NOTIFICATION + PRJ_NOTIFY_TYPES (misses values which are a power of 2)

Hi @steffengy -- thanks so much for letting us know about this issue. We'll work on investigating these pages.


Now the applications routes are updated when a new resource is added

Done. Also added new functionallity (When removes an application, its routes also are deleted)


Replay: Add ignorable extensions

Problem
There are some extensions that we may want to outright ignore during replay.  This is especially true if the extension is present on the capture, but not the replay.
One such extension is the VK_EXT_tool_info extension.  This extension is exposed by the capture layer and may not be present on the system after the capture layer is removed.
Solution
Filter out the extension if it is requested when the underlying replay device does not support it.
Testing

Android with capture layer present and then removed on replay

Android instance for UI can grab all listed extensions and enable for replay always.
However, the entrypoints were not used, just the extension name.
Replay worked with message about ignoring the extension.

Linux with capture layer present for VulkanInfo

VulkanInfo can use the extension's entrypoints as well with the --show-tool-props argument.
Captured with and without tool props
Both worked, the only message appearing on the replay when using tool props was the following messages:

[gfxrecon] INFO - Skipping call down to vkGetPhysicalDeviceToolProperties because extension not supported by underlying Vulkan implementation. [gfxrecon] INFO - Replay adjusted the vkGetPhysicalDeviceToolPropertiesEXT array count: capture count = 1, replay count = 0

CI gfxreconstruct build queued with queue ID 35591.

CI gfxreconstruct build # 2505 running.

CI gfxreconstruct build # 2505 passed.

CI gfxreconstruct build queued with queue ID 41445.

CI gfxreconstruct build # 2536 running.

CI gfxreconstruct build # 2536 passed.

CI gfxreconstruct build queued with queue ID 41488.

CI gfxreconstruct build # 2537 running.

CI gfxreconstruct build # 2537 passed.


chore(main): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-07-04)
Features

add first installer version (11a1b27)
add first installer version (246e79a)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.5.0
:sunflower:


Returns wrong trapOid for HP nodes

I am trying to compile HP ILO node MIBs and generate the Json file.
The tool generates wrong OIDs for the traps
generated by Tool::
"name": "cpqiScsiLinkUp",
"oid": "<IP_ADDRESS>.4.1.2320.169001",
Expected :
"name": "cpqiScsiLinkUp",
"oid": "<IP_ADDRESS>.<IP_ADDRESS>.169001",
cpqiscsi.zip

You might read etingof/pysnmp#429 to learn why you shouldn't use this repo, either for bug reports or its out-of-date PyPI package.
This is a known issue and has been fixed in various forks.


Is it possible for this to work with Metal?

This looks like an incredible framework, but I would love to be able to use it with custom 3D objects that I have using Unity as my 3D renderer rather than SceneKit.

@eschos24,
I have perviously received some requests to support Unity; however, I didn't have much time recently to work on it.
Can you please clarify to me how you would want to use ARVideoKit in Unity?
-Ahmed

Basically, the concept is using native iOS controls and design throughout the app, but then use Unity as my renderer to display 3D objects using ARKit. Does that make sense? I do not want to build the entire app using Unity because I want it to look and feel like an iOS app, but I want to take advantage of Unity's tools for rendering 3D objects, which uses Metal rather than SceneKit.