Error: Building package boost-preprocessor:x64-windows failed with: BUILD_FAILED

vcpkg version
Vcpkg package management program version 2018.11.23-nohash

vcpkg install boost-test:x64-windows
The following packages will be built and installed:

boost-algorithm[core]:x64-windows

boost-array[core]:x64-windows

boost-assert[core]:x64-windows

boost-bind[core]:x64-windows

boost-build[core]:x64-windows

boost-compatibility[core]:x64-windows

boost-concept-check[core]:x64-windows

boost-config[core]:x64-windows

boost-container[core]:x64-windows

boost-container-hash[core]:x64-windows

boost-conversion[core]:x64-windows

boost-core[core]:x64-windows

boost-detail[core]:x64-windows

boost-exception[core]:x64-windows

boost-function[core]:x64-windows

boost-function-types[core]:x64-windows

boost-functional[core]:x64-windows

boost-fusion[core]:x64-windows

boost-integer[core]:x64-windows

boost-intrusive[core]:x64-windows

boost-io[core]:x64-windows

boost-iterator[core]:x64-windows

boost-modular-build-helper[core]:x64-windows

boost-move[core]:x64-windows

boost-mpl[core]:x64-windows

boost-numeric-conversion[core]:x64-windows

boost-optional[core]:x64-windows

boost-predef[core]:x64-windows

boost-preprocessor[core]:x64-windows

boost-range[core]:x64-windows

boost-regex[core]:x64-windows

boost-smart-ptr[core]:x64-windows

boost-static-assert[core]:x64-windows
boost-test[core]:x64-windows

boost-throw-exception[core]:x64-windows

boost-tuple[core]:x64-windows

boost-type-index[core]:x64-windows

boost-type-traits[core]:x64-windows

boost-typeof[core]:x64-windows

boost-unordered[core]:x64-windows

boost-utility[core]:x64-windows
Additional packages (*) will be modified to complete this operation.
Starting package 1/41: boost-preprocessor:x64-windows
Building package boost-preprocessor[core]:x64-windows...
-- Downloading https://github.com/boostorg/preprocessor/archive/boost-1.70.0.tar.gz...
-- Downloading https://github.com/boostorg/preprocessor/archive/boost-1.70.0.tar.gz... Failed. Status: 7;"Couldn't connect to server"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:172 (message):
Failed to download file.
If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
variables to "https://user:password@your-proxy-ip-address:port/".
Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
scripts/cmake/vcpkg_from_github.cmake:108 (vcpkg_download_distfile)
ports/boost-preprocessor/portfile.cmake:5 (vcpkg_from_github)
scripts/ports.cmake:71 (include)
Error: Building package boost-preprocessor:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: boost-preprocessor:x64-windows
Vcpkg version: 2018.11.23-nohash
Additionally, attach any relevant sections from the log files above.

netsh winhttp show proxy

Current WinHTTP proxy settings:
Proxy Server(s) :  XXX.XX.XXX.XXX:XXXX
Bypass List     :  (none)

Never mind, resolved by setting both HTTP_PROXY and HTTPS_PROXY


[Feature request] Checkpointing in train.py

I've been using train.py as an entry point for testing new agents and environments.
These agents, environments, and associated code are not neccesarily stable. Either in the sense of the physics simulation or in terms of code quality.
Thus is it is not surprising that when training for millions of steps there will be mishaps for example:

An agent becomes physically unstable and the simulator crashes or hangs
An agent enters an undesirable part of policy space and begins behaving erratically
An agent learns to hack the reward function
Some other part of the code crashes or hangs

Currently train.py saves the state of the agent after the specified training run completes.
This ticket is to consider and discuss adding a mechanism by which intermediate checkpoints are produced during training.

This will be addressed in https://github.com/hill-a/stable-baselines/issues/348 normally

The callbacks are there, I will add that feature soon.


Document injected data format.

Hi,
I'd like to use just the client-side part of this. My app is built using Django, and I'd prefer not to add anything that requires javascript evaluation on the server. I can probably generate the json in Python. But in order to do that I'd like to know how the data is structured.

@bigblind if you're going to be rendering this client-side only, you should be able to use the regular react-relay module.

@bigblind naturally, it will become your responsibility as a developer to provide all the correct data that Relay would request if it was rendered on the server, and update the data whenever you update the components, etc. Should not be too hard, just something to keep in mind when you're making changes. I am not sure if I am reading the change logs correctly, but it seems like with 0.8 this is possible just with react-relay as @hellatan suggested?

@thelordoftheboards What API method on relay would I call then to inject the data? I asked about this in the Relay issue tracker, and they directed me here, and it seems pretty clear that I need the injectPreparedData method that isomorphic-relay provides.

@bigblind ok, i think i see what you mean based on your comment in your relay comment. As you mentioned, i would imagine you would use the injectPreparedData method in your client-side entry file and you would be responsible for passing the json to that method.
You'd obviously need to mimic the data structure that that method requires, which looks something like this:
data:  [ { query:
     { calls: [],
       children: [Object],
       directives: [],
       fieldName: 'viewer',
       isDeferred: false,
       kind: 'Query',
       metadata: [Object],
       name: 'CollectionsLandingQueryRoute',
       type: 'Viewer' },
    result: { viewer: [Object] } } ]

Hypothetically it sounds like it could work, but I'm pretty new to isomorphic react (as in started using it the past couple days) so I'm not 100% sure what the inner workings of injectPreparedData is doing.
The reason why there's query.result.viewer is due to this bug in relay.

The data is an array of { query: ConcreteQuery, response: Object }.
The shape of ConcreteQuery is described here. And the response is just a response of GraphQL server for that query. But note that ConcreteQuery is not a part of Relay public API, and can be changed or removed in any release. Also, isomorphic-relay might not use ConcreteQuery in future versions, but take advantage of GraphMode instead (when it is ready).


Fix Robot Melee Animations

Fixed Robot mele Animation
Robot Enemies Polish
Fixed: (Engine only) When loading level 2 editor camera was active instead of game camera.
Fixed: (Engine only) When loading level 2 player appeared far from the start.

Task linked: CU-86bzqvxjj Bug: Fix Robot Melee Attack Animation


getters should save the head of their lists and restart when they are called when the head is null

This issue has been addressed in #9


Support for multi-channel recording on the client?

Hey, I was just wondering if there was a way to only record the local user's voice, so that we can get a local multi-channel recording?

The startAudioRecording method records all the users in the channel...

you can use raw data api. or cloud recording.

OH but in flutter it's not that easy to access raw data api.. would cloud recording help?

Thanks for the quick reply. Cloud recording would help, but I would prefer to do it on the client. Is there any way I could access the raw data api on Flutter?

Cloud recording would help, but I haven't seen any tutorials for getting it to work with Flutter. If there is one, then I would love to see it :)

https://docs.agora.io/en/cloud-recording/landing-page?platform=RESTful check here for docs. it's triggered from your server usually.

I am using cloud functions. (Firebase/Firestore)

Can I have two people join two different channels. On the one channel person B is only able to listen with person A speaking, and on the second channel person A is only able to listen with person B speaking?
The channels operate at the same time. And any one of these people can record the channels and they will only record the people speaking on the channel (which in this case is limited to one person).
Would this work on a p2p voice call?

why do you want B to hear A while they are in different channels? Why not just put them in one channel?

Because how then could you record each user's own voice on the client?
This solution assumes two things.
a) You can be connected to two different channels at once
b) If you record audio in one channel, it will not pick up audio coming from the other channel (even if that audio is coming out of your phone's speaker)

Hey, it's fine, I will just use the cloud recorder for now. But there should be a way of each person recording their own audio streams.


edit: enable Sublime Text project view

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Code very similar to this was removed in c6b6fc678e66668b08d31e7a55950768e0e4d687 because we had, at the time, moved away from the model where the Homebrew prefix was the repository. Now that we've actually moved back to that model, we've re-inherited exactly the same problems that inspired the original code.
In place of the original code, which specified the list of files to open in cmd/edit, this PR adds a Sublime Text project file and opens that with brew edit instead. That limits the code changes significantly, with the majority of the diff being indent-related. I've left a comment at the top of the Sublime project file to indicate how it was authored.
A similar PR to this was previously open in #10710. This version addresses Mike's comments in the original version.

Review period will end on 2021-08-03 at 00:00:00 UTC.

Review period ended.


Tiers should allow custom button text

Currently, tiers force button text ("Become a Sponsor", "Become a Backer", "contribute") text on their button.  Tiers should allow us to customize the text on the button, especially because we are changing the names of "Backer" and "Sponsor" and "contribute" looks really bad for custom tiers.

What would you like to see?
We could let customize the button but I'm cautious about adding too many settings/features that will boggle down the interface and UX.

@xdamman the text of the button should be configurable, just like the description and title.

Any update here?

Ping @piamancini @xdamman

@xdamman those look like just the frontend components of this - how does that interact with the API server/database storage?

I might have a chance to look at this coming up.

If anyone's willing to tackle this issue: we already support this feature - we're only missing an input in tiers edit section for users to customize the value themselves. The field is tier.button.

A $50 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $50 when the Pull Request is reviewed, accepted and merged. More info
This bounty is currently reserved to participants of the OSCA (Open Source Africa) event. If not completed there, it will be later open to everyone.

from #osca ph will like to work on this issue

please I'm confused how do i go about this

@burinious You need to add a new field "Button Text" in the "Tiers" administration interface, in the "Edit Collective" panel.

I'm thinking that it would be great to provide multiple pre-defined options (donate, contribute, join, participate, register...) that collective admin can pick. Pre defined options can easily be translated, it's much harder for custom strings.

I don't think we should assume we know what the pre-defined options should be. Better to let it be customizable then look at the actual data to see what the most common ones are and implement that in a future phase if needed. If a Collective's main community is non-English speaking, then they will make the titles in their ow language anyway.

@znarf I am trying my hands on fixing this issue.
After setting up the development environment, I realized I needed to login when I hit the url I could observe in this comment (https://github.com/opencollective/opencollective/issues/543#issuecomment-508932858) above.
After Login, I get me a message on the page stating You need to be logged in as the creator of this collective or as a core contributor of the Blogmarks collective.
So, is there a way I can get dummy credentials that can give me access to this page, so that I can see the fixes I make as work on this issue?
That will be appreciated.

@jsamchineme You don't have to use this precise Collective, I just took a screenshot because I'm an admin of this Collective. To develop this feature, feel free to create a new Collective and you will be admin of it.

@znarf Thanks for the hint. I have now created a new collective and added a tier to the account.
However, I see that the API doesn't have a field "buttonText" returned with a Tier.
Here is the screenshot

Does this mean I would also have to work on the backend project to enable this feature?

For reading, the buttonfield is already exposed in the GraphQL API. So you don't need to code the support for it, you can just add it in the GraphQL query in the frontend.
For saving, I'm not sure.

I just checked the backend code and it appears saving caters for the button field.

@znarf have updated the description

Thank you @burinious ! It's merged. (Deployment to production is still pending).
To claim the bounty, you can:
a) submit an invoice to https://opencollective.com/engineering
b) or ask for an Open Collective gift card of the same value (to donate to other collectives)


Fix path issues during testing

I corrected a mistake in the naming of the target folder used by @amorehead in some of the tests. Furthermore, I increased the max number of retry attempts to make sure that the tests find a sample. There is still a chance that it won't find one, in which case the test will fail. Should this be an even higher number?
Finally, I also modified the README a bit to indicate that I only worked on the distogram issue 😄

@vandrw, you should be able to specify the sample_only_pdb_ids flag in the weighted PDB sampler and the PDBDataset to have the tests only try to load the particular mmCIF files in data/test.


Use GITHUB_TOKEN to access private repos

Right now, I'm not able to export to pdf from a private repo. Maybe the GITHUB_TOKEN could be used to access private repos.

That would allow any user with access to the instance the ability to view any markdown file in the repo. While technically possible, it's not something that I am interested in supporting personally.

@benbalter Would you be interested in a PR if I do it?
Also, I couldn't make this work using a GITHUB_TOKEN with access to private repos. And, in theory, it should work as is. Do you know if I'm missing something?

@mjlescano I'm curious how this could be used in a way so as not to essentially make a private repo public?


Add support for html in chat

The chat widget currently just support text only, so I decide to add a package to parse HTML string to React Component in safe way.
Thanks for this awesome lib: react-html-parser
Demo

top!

@0toshigami is there a way to also provide custom CSS? does ReactHtmlComponent support CSS
also please mention me so I will be notified, for some reason I don't receive any notifications, need to look into that

@0toshigami is there a way to also provide custom CSS? does ReactHtmlComponent support CSS
also please mention me so I will be notified, for some reason I don't receive any notifications, need to look into that

@yashppawar What is meant by custom CSS? Is it for the chat widget or for the content within the chat widget?
Additionally, I have provided support for Markdown syntax.

@yashppawar What is meant by custom CSS? Is it for the chat widget or for the content within the chat widget?

sorry for not being clear, I meant CSS for the HTML added inside the chat widget, I saw the video element which had a width greater than that of the chat widget... so was asking if its possible to give CSS to make it look better

sorry for not being clear, I meant CSS for the HTML added inside the chat widget, I saw the video element which had a width greater than that of the chat widget... so was asking if its possible to give CSS to make it look better

@yashppawar Oh, I also noticed that. The video element is an iframe element that has width and height attributes. This makes it convenient for us to customize those attributes. Inline CSS will also work.

Please merge the PR, this is what I need now. 👍

Sorry was busy due to exams, I was trying to get this to work on my machine, but the HTML was not rendered... is there anything required for this to work? I did delete node modules and install them again

Sorry was busy due to exams, I was trying to get this to work on my machine, but the HTML was not rendered... is there anything required for this to work? I did delete node modules and install them again

Have you read the updated example in my readme? The message method now has 2 new parameters,
allow_html (default = False) and
is_table (default = False)
st.session_state.setdefault(
    'generated',
    [{
        'type': 'normal', 'data': f''
    }]
message(
    message=st.session_state['generated'][0]['data'],
    key=0,
    allow_html=True,
    is_table=True if st.session_state['generated'][0]['type']=='table' else False
)

yes @0toshigami, I have read it and was trying to execute the same code which you have added in the readme.md

@yashppawar hmmm, that's strange. Have you tried rebuilding the frontend?

yes @0toshigami, I have read it and was trying to execute the same code which you have added in the readme.md

It works on my machine though.

Let me try it again today, @0toshigami how do I rebuild frontend? Purge node modules and build dir, and npm i, then npm start?

how do I rebuild frontend? Purge node modules and build dir, and npm i, then npm start?

@yashppawar Maybe you could try to remove the node_modules and package-lock.json, then npm i, and npm run build

I urgently need this feature, please expedite the PR merge. Thank you!

Looking forward to this PR merge too!

I tried adding an href to an external site and encountering some weird behavior. Is it possible to route the link to open in a new tab pls? @0toshigami

Hi @stephenleo
Because each message is displayed as an , it's good to use the target attribute for  tag. If you want to open the link in a new tab, use "_blank". Or you can use "_parent" if you one to open the link in the parent window.
link text

amazing! Thank you very much

So are we merging this so i can remove the link to requirements.txt instead of just having the library name? Whats are missing to merge?

I've been using this fork fine for a few days now. Will be a great addition to the library!

Really sorry for the inconvenience I am causing, I was not able to look at it for long, and might not be able to do more for a week at least (exams), I'll try what I can do..., is it working for everyone else? somehow I end up with more errors than I started with while rebuilding, might have to try a fresh install of node next

@yashppawar I tried it in a fresh conda environment and it worked great!

I'm really looking forward that this merge request is accepted!

Can you please release this to latest version. this change is not available on pypi

The chat widget currently just support text only, so I decide to add a package to parse HTML string to React Component in safe way.
Thanks for this awesome lib: react-html-parser
Demo

Please merge this PR

This is also something I desperately need!
Please look at merging the PR at earliest convenience, thanks!

Hi @0toshigami ,
Thank you for adding the HTML functionality to the message function. While using it, I came across few use-cases where the current front-end output needs improvement.
Using Collapsed sections with markdown,

Example code
import streamlit as st
from streamlit_chat import message

def on_input_change():
    user_input = st.session_state.user_input
    st.session_state.past.append(user_input)
    st.session_state.generated.append("The messages from Bot\nWith new line")

def on_btn_click():
    del st.session_state.past[:]
    del st.session_state.generated[:]

markdown = "Below is an example of collapsible markdown taken from Wikipedia."
paragraph = "Euclidean geometry is a mathematical system attributed to ancient Greek mathematician Euclid, which he described in his textbook on geometry; Elements. Euclid's approach consists in assuming a small set of intuitively appealing axioms (postulates) and deducing many other propositions (theorems) from these. Although many of Euclid's results had been stated earlier,[1] Euclid was the first to organize these propositions into a logical system in which each result is proved from axioms and previously proved theorems."
markdown += """Toggle me!{paragraph}""".format(paragraph=paragraph)

st.session_state.setdefault('past', ['Show example of collapsible markdown'])
st.session_state.setdefault('generated', [{'type': 'normal', 'data': f'{markdown}'}])

st.title("Example")
with st.container():
    for i in range(len(st.session_state['generated'])):
        message(st.session_state['past'][i], is_user=True, key=f"{i}_user")
        message(
            st.session_state['generated'][i]['data'],
            key=f"{i}",
            allow_html=True,
            is_table=True if st.session_state['generated'][i]['type']=='table' else False
        )

    st.button("Clear message", on_click=on_btn_click)

with st.container():
    st.text_input("User Input:", on_change=on_input_change, key="user_input")

Can you please add functionality to support scrolling with message function?
https://discuss.streamlit.io/t/scrolling-text-containers/26485/3
It will be also useful to have a fixed height container with scrolling for the whole chat functionality. Below is a Gradio example,

@nubgamerz @saurabhgssingh @imsadiku @ketank1000
Until the PR is merged, you can install the PR using the below command,
pip install git+https://github.com/0toshigami/st-chat.git#egg=streamlit-chat

Hello! @0toshigami
Love the enhancement! Hope PR goes through soon.
My chatbot will cite the source which is a url when responding. and the url is automatically made into a hyperlink after installing the PR. I want the hyperlink to open in a new window but am unable to add target="_blank" to open to a new window. Any suggestions/help would be greatly appreciated!
Thanks

Hey it would be great if you could merge this PR.

Great work. Can you merge this @YashVardhan-AI ?

merged, and published at https://pypi.org/project/streamlit-chat/0.1.0/

How can I make hyperlinks work? I just tried it and it did not work

@DonjiKong I have updated it, now the links should open in new tab

I also have the issue that the message bubble does not resize properly with the output when using markdown/html.
Was anyone able to fix that?


fix: evaluated popup value and error for list v2

Description
logBlackList prevented errors from getting to the Evaluation popup, also, the datapath has been changed, so we don't need List1.template.Text1.text. We just access it from the data tree directly. This introduced meta widget appearance in the debugger. This issue would be solved in https://github.com/appsmithorg/appsmith/issues/18382.
Fixes #16576
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

Tested this PR and LGTM

Tested for evaluated value in {{currentItem}}
Regression on List v2


OTSU

最大类间方差

A Threshold Selection Method from Gray-Level Histograms
A Fast Algorithm for Multilevel Thresholding

A Threshold Selection Method from Gray-Level Histograms
A Fast Algorithm for Multilevel Thresholdin


don't quote the filter values #24

fixes #24

This strip the quotes; we have them in current version. But inspecting the actual sql query shows the fq fields are different. In case of author, the lindat-dspace version queries author_keyword which is not tokenized the same way so the quotes are superfluous.
Marking as needs testing...


fix 404: middleware to render "not found" page

Changes

Associated issue
Resolves #161 Custom 404 page no longer working
How to test

Checklist

[ ] I have performed a self-review of my own code
[ ] I have made sure that my PR is easy to review (not too big, includes comments)
[ ] I have made updated relevant documentation files (in project README, docs/, etc)
[ ] I have added a decision log entry if the change affects the architecture or changes a significant technology
[ ] I have notified a reviewer

Closing in favour of #180


Rename customer group to any-authenticated-user

To make the purpose of this group clearer a more explicit naming is favored

Fixed by commit<PHONE_NUMBER>7dc016105a06a179f57b42ac356507


Service URL in CLI

Thank you for that piece of software.
I have a tiny suggestion. At the beginning, I couldn't get the service to work as I was using http://<IP_ADDRESS>:5000 instead of http://<IP_ADDRESS>:5000/reconcile.
I was confused due to the default message flask provide:
> csv-reconcile serve
 * Serving Flask app 'csv-reconcile'
 * Debug mode: off
WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
 * Running on http://<IP_ADDRESS>:5000
Press CTRL+C to quit

Would it be possible to add an additional message in CLI, something like:
> csv-reconcile serve
[...]
 * Reconciliation service at http://<IP_ADDRESS>:5000/reconcile

Press CTRL+C to quit

It’s mentioned in the Quickstart section of the README, but I can sympathize because I was similarly confused using the predecessor to this tool.  I’ll give this a think.


Implement ASP.NET 5 support

Implement ASP.NET 5 support and/or WebListener support

..And since ASP.NET 5 will be supported cross-platform, this may satisfy the need to get Mac support. Support for Linux though will be handled through Issue #23 to work out the Mono details using the HttpListenerNanoServer.


Portfolio View. Use colord token icons in portfolio view for token-centric view

Duplicated by https://github.com/novasamatech/nova-spektr/issues/2546


[Bug]: Location update evnets are delivered in batches and out of order on Android

Please check the following before submitting a new issue.

[X] I have searched the existing issues.
[X] I have carefully read the documentation and verified I have added the required platform specific configuration.

Please select affected platform(s)

[X] Android
[ ] iOS
[ ] Linux
[ ] macOS
[ ] Web
[ ] Windows

Steps to reproduce
simply starts a location stream and make it running long enough in background (with wake lock).
Expected results
The callback is called with location events with the right order. (In batches or not does not matter that much)
Actual results
Events are delivered in batches, (i.e. a while without any updates, then a bunch of events are delivered at ~ same time but out of order).
Here is some data I got. (I set update interval to 1s and not doing much in the callback so it is not a race condition in my code). I also droped actual coordinates.
I failed to find any report that Android may deliver location updates in random order so there might be a race condition in this lib.
out_of_order_events.csv
Code sample
n/a
Screenshots or video
n/a
Version
4.5.4
Flutter Doctor output
n/a

Dear @CaviarChen,
Thanks for filing the issue. In order to investigate the issue could you please tell us something about the clear steps to reproduce or at lease some example code?
Kind regards,


Overly complicated to disable auto-save for a particular form

I would like to disable auto-save while I am learning how to develop powerapp forms.  Why can't you just have a simple checkbox?

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

ID: 9cea415f-d8c4-93a8-e2ed-51164eeb6ca0
Version Independent ID: 54be91e7-1cf8-f19c-6099-2407c4bd5ba4
Content: Disable auto-save in a model-driven app with Power Apps - Power Apps
Content Source: powerapps-docs/maker/model-driven-apps/manage-auto-save.md
Service: powerapps
GitHub Login: @Mattp123
Microsoft Alias: matp

@GBauer1833 -vThanks for your feedback. For product-related feedback, visit https://ideas.powerapps.com.
Closing this issue.

@GBauer1833 -vThanks for your feedback. For product-related feedback, visit https://ideas.powerapps.com.
Closing this issue.


Pull underlying capabilities up to 00-multus.conf

As discussed in issue #544 the nested plugin capabilities are not integrated in the 00-multus.conf file generated by the images/entrypoint.sh file.
I implemented the parsing of the underlying master plugin json configuration file with python, to retrieve the nested capabilities and to integrate them in the top-level section of the 00-multus.conf file (because it is not sufficient to have capabilities declared in the delegates section of the CNI config file.)
The python script for analyzing underlying plugins capabilities is compatible with Python 2 and Python 3, it could be a bit shorter if I used only Python 3 syntax, but I noticed that the Dockerfile is based on a CentOS version that uses Phyton 2.
When no capabilities in the underlying plugins, then nothing gets added to the 00-multus.conf file.

Pull Request Test Coverage Report for Build 195756930

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.997%

Totals

Change from base Build 185831388:
0.0%

Covered Lines:
1082

Relevant Lines:
1524

💛  - Coveralls

Sorry for the multiple push: I introduced a modification that improves the parsing of the capabilities, and first commited a vim file instead of my modification, and when I force pushed a new commit with the correct file I forgot to put a title for the commit 🤦♂😅
Everything should be OK now.

Thank you for your PR, @clementnuss !
It looks good to me. Let's wait a few days for other comments and merge it.

Thanks for merging!
Any why the build failed ? the same commit passed all tests successfully yesterday 🤔


Update .gitignore

更新为常用Android项目忽略规则，避免不必要的文件上传，如local.properties等

 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.


Horrible tensorboard graph:

Hi,
I'm trying to scope certain components of my tensorflow fold model, and I after rending it in tensorboard, it seems that its ignoring the scope in which I construct different blocks. See this http://i.imgur.com/Vtf9OhP.png
The associated source code which generates this graph is:
with tf.variable_scope("context_free_discriminator"):
	with tf.variable_scope("embedding"):
	    pre_process =  (td.InputTransform(lambda tokens: [tokeToVec(s, word2vec, indim) for s in tokens])
	                    >> td.Map(td.Vector(indim)))

	    word_cell = td.ScopedLayer(tf.contrib.rnn.BasicLSTMCell(num_units=300), 'word_cell')
	    stacked_cell =  td.ScopedLayer(tf.contrib.rnn.BasicLSTMCell(num_units=100), 'stacked_cell')

	    # word lstm convers word sequence into an lstm.
	    word_lstm = (pre_process >> td.RNN(word_cell))
	    hierarchical_lstm = (word_lstm
	                         >> td.GetItem(0) #hidden states
	                         >> td.RNN(stacked_cell))
	    embedding =  (hierarchical_lstm
	                                      >> td.GetItem(1) # Final state (out, hidden)
	                                      >> td.GetItem(0) # Final output state.
	                                      >>  td.FC(outdim))

	with tf.variable_scope("model"):
	    logit_output_block = (embedding_block
	              >> td.FC(428)
	              >> td.FC(328)
	              >> td.FC(num_mocs, activation=None))
	    compiler = td.Compiler.create(logit_output_block)
	    logit_tensor = compiler.output_tensors

	with tf.variable_scope("training"):
	    _desired = tf.placeholder(tf.int32)
	    _desired_onehot = tf.one_hot(_desired, num_mocs)
	    loss = cross_entropy = tf.reduce_mean(
	        tf.nn.softmax_cross_entropy_with_logits(labels=_desired, logits=logit_tensor))

	    train_op = tf.train.AdamOptimizer(learning_rate).minimize(cross_entropy)

My understanding is that since td is build over tf, not everything will be accessible as would be the case with "pure" tensorflow, meaning specifically that scopes are probably used by td to deal with the dynamic part of building the graph. I'm not entirely sure that the tf.variable_scope commands are compatible with the whole td.compiler thing. If they were, you wouldn't need a scopedlayer, IMHO.
A good example of that is the results of the 'word_cell' scope, that is "internally" translated to several blocks.
Of course, I'm a random internet dude, not a contributor to the project, so everything I said might be horribly wrong.

Why do you say it's ignoring the scope?  The graph is showing the names of
various blocks, not the fully qualified names of the variables used by
those blocks.  Each Layer you create (e.g. FC or ScopedLayer) will create
its own variable scope, nested in whatever the current variable scope is,
to hold the variables for that layer.
To be honest, what confuses me about the picture is that you have multiple
while loops, and multiple instantiations of your LSTM cells; I don't know
what's causing that.
If you want to see whether your variables have been created appropriately,
call
[v.name for v in tf.global_variables()]; that should show you the fully
qualified names of all variables.
On Fri, Feb 24, 2017 at 2:12 PM, William Guss<EMAIL_ADDRESS>wrote:

Hi,
I'm trying to scope certain components of my tensorflow fold model, and I
after rending it in tensorboard, it seems that its ignoring the scope in
which I construct different blocks. See this
http://i.imgur.com/Vtf9OhP.png
The associated source code which generates this graph is:
with tf.variable_scope("context_free_discriminator"):
with tf.variable_scope("embedding"):
pre_process =  (td.InputTransform(lambda tokens: [tokeToVec(s, word2vec, indim) for s in tokens])
>> td.Map(td.Vector(indim)))
  word_cell = td.ScopedLayer(tf.contrib.rnn.BasicLSTMCell(num_units=300), 'word_cell')
  stacked_cell =  td.ScopedLayer(tf.contrib.rnn.BasicLSTMCell(num_units=100), 'stacked_cell')

  # word lstm convers word sequence into an lstm.
  word_lstm = (pre_process >> td.RNN(word_cell))
  hierarchical_lstm = (word_lstm
                       >> td.GetItem(0) #hidden states
                       >> td.RNN(stacked_cell))
  embedding =  (hierarchical_lstm
                                    >> td.GetItem(1) # Final state (out, hidden)
                                    >> td.GetItem(0) # Final output state.
                                    >>  td.FC(outdim))

with tf.variable_scope("model"):
logit_output_block = (embedding_block
>> td.FC(428)
>> td.FC(328)
>> td.FC(num_mocs, activation=None))
compiler = td.Compiler.create(logit_output_block)
logit_tensor = compiler.output_tensors
with tf.variable_scope("training"):
_desired = tf.placeholder(tf.int32)
_desired_onehot = tf.one_hot(_desired, num_mocs)
loss = cross_entropy = tf.reduce_mean(
tf.nn.softmax_cross_entropy_with_logits(labels=_desired, logits=logit_tensor))
  train_op = tf.train.AdamOptimizer(learning_rate).minimize(cross_entropy)

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/fold/issues/19, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGGbTWb9XH1cEmFARsGoyPrKWN7afbx5ks5rf1XZgaJpZM4MLvGB
.

--
DeLesley Hutchins | Software Engineer |<EMAIL_ADDRESS>|<PHONE_NUMBER>

Yeah, TensorBoard graph view is not very helpful with Fold, unfortunately. Regarding scoping, if you know what you're doing you can pass in a scope as an argument to td.ScopedLayer and it will get used exactly as provided. If you pass in a string we go through some gymnastics to ensure that every ScopedLayer really does correspond to a unique scope. We do this to avoid some breakage and gotchas in the TF variable scoping mechanism. What Delesley said is a very good suggestion for making sure the right variables are being created. If you are observing undersharing (e.g. a single scoped layer that encapsulates variables that are being created more than once) that is bug, please do report it. But the view of the graph in tensorboard is unfortunately not sufficient to figure out what's going on.
Regarding the while loop, I'm confused; Fold certainly only creates one while loop per compiler; maybe you ran Block.eval() somewhere? (eval() is only for messing around at the repl and unit tests; it shouldn't be used in "real" models)


Uncaught TypeError: Cannot convert "[object Object]" to unsigned int when deleting a table

Hi, and thanks for the great library! I am trying to delete a table to free up the memory but what I see printed in the console is the following:
Uncaught TypeError: Cannot convert "[object Object]" to unsigned int
    at Object.toWireType (psp.js:2)
    at Object.unregister_gnode (psp.js:2)
    at _table.delete (perspective.js:678)
    at perspective.js:1092

Digging in the toWireType method, brings me to:
registerType(primitiveType, {
            name: name,
            "fromWireType": fromWireType,
            "toWireType": (function(destructors, value) {
                if (typeof value !== "number" && typeof value !== "boolean") {
                    throw new TypeError('Cannot convert "' + _embind_repr(value) + '" to ' + this.name)
                }

In the CSV there are primitive values such as strings, numbers, booleans and Perspective itself works fine. Any suggestions?

Hi @ivdelchev, what does your code look like to delete the table?

I've added tests to verify this does not happen, at least in trivial cases, almost verbatim as above.
If you find you still run into this, please open a new issue with a bit more detail about how to recreate - a broken test case would be ideal.


'flutter start' should run pub get, or prompt you to do so, if it hasn't been run for your app

Clone the flutter git repo anew, or do a hard git clean, and then try running "flutter start" in "flutter/examples/stocks".

[ ] The problem is that you haven't yet run pub get. We can determine that easily by checking the pubspec.lock file -- it's not there (or is older than the pubspec.yaml).

[ ] We should definitely not ask if you've added "flutter" to the pubspec.yaml since we can trivially verify this for ourselves. Either you have or you haven't; if you haven't, we should say explicitly "this package does not depend on 'flutter'. If it is a flutter application, then..." and explain how to add the dependency (we can even figure out the exact path for you right there).

I get a slightly different message, adding here for searchability:
Error: incompatible sky_engine package; please run 'pub get' to get the correct one.
I feel like this is by far the most common error I get with the flutter tool.  Probably because I surf flutter HEAD and often have stale pub gets in my examples folders.

/me will take a look.


Manage roles draft

Feedback (cc @Valcoholics):

in the intro where we describe what the ThemeRegistry is, we can actually take out the "modular" approach. this actually isnt a modular approach, and the rules regarding access control are baked into the registry contract. we should be linking to this repo instead of the "remote-access-control" one
i think its (only very slightly) confusing that we have the cool graphic of goerli.etherscan, but then are linking to mainnet etherscan. i dont think we need to change anything, but perhaps we could clarify some language about the theme registry being available at the same address on multiple chains? @salieflewis could you chime in here? also we're gonna need to provide support for the new testnet that is coming at some point, just flagging that for us to be aware of (cuz goerli might fully die)
"On the contract's page, click on the "Connect to Web3" button located on the top right corner. This will prompt you to connect your wallet." I dont think most people know you can do this. i think a screenshot here would help
The header "Write Contract" is confusing for more people I assume. I think something more like "prepare transaction" prob is a little easier to digest
this bullet point "You'll need to provide the required input parameters." reads a little random. could we skip/remove this, or maybe if we want to keep it use it as a space to provide info on ethereum function inputs more generally?
i think providing the roles first, then the tuple shape, and THEN providing the example input for them to swap in and out addresses/roles would make more sense. right now u read that first without knowing what the diff roles are or what even the tuple looks like which is confusing i think
more screenshots for the "confirm transaction" bit may be helpful (particurarly in showing what it looks like on etherscan when someones wallet is properly connected, where to press send txn, etc)
"Review the transaction details, and if everything looks correct, confirm the transaction." I actually think this is confusing and we should just remove it. "if everything looks correct" is not helpful to ppl who dont know what looks correct or not


Unable to prevent reading from the Cache

The current implementation will always check the cache regardless of what value is passed in for the ttl.
Currently in the file /src/HTTPCache.ts on line 78 there is no check of ttl before reading from the cache:
const entry = await this.keyValueCache.get(cacheKey);

The ask would be to provide a conditional check on this line to bypass the read from the cache if ttl=0 as it is currently impacting the reported miss rate when trying to only inspect the miss rate for queries/request where the cache is actively used.

Would it be better here instead of setting ttl=0 to instead have a config to disable the cache entirely so then we can just check that instead?

@JoeDevine Feel free to close the issue


vela port-forward not working for webservice component

Describe the bug
I have deployed an app like below:
$ kubectl get app nginx -o yaml
apiVersion: core.oam.dev/v1beta1
kind: Application
metadata:
  name: nginx
  namespace: default
spec:
  components:
  - name: nginx
    type: webservice
    properties:
      image: nginx
      ports:
      - expose: true
        port: 80
        protocol: TCP
    traits:
    - properties:
        replicas: 1
      type: scaler

It has ports specified in component properties.
But the command vela port-forward report an error.
$ vela port-forward nginx
? You have 2 deployed resources in your app. Please choose one: Cluster: local | Namespace: default | Kind: Service | Name: nginx
Error: no port found in app or arguments

To Reproduce
Apply the yaml above and run
vela port-forward nginx

Expected behavior
Hope there can be normal port-forward
Screenshots

KubeVela Version
latest version
Additional context
The reason is the port-forward use hard-code way to check the properties:

But we have upgrade the fields from port to ports for webservice component.
We need to fix the incompatible in CLI .

Correct usage:
$ vela port-forward nginx 8080:80

There is no valid validation of the parameters


Workloads Repo for FluxCD Addon

Describe the feature
Workloads Repo for FluxCD Addon
Use Case
Workloads Repo for FluxCD Addon
Proposed Solution
Workloads Repo for FluxCD Addon
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request

CDK version used
2.72
EKS Blueprints Version
1.7.3
Node.js Version
19.X
Environment details (OS name and version, etc.)
Ubuntu

This is complete. Closing this issue


Remove DisplayVersion from 7zip.7zip version 24.06

DisplayVersion differs from PackageVersion by trailing .0s, which do not play a part in package matching. Removing it to reduce client mapping code and avoid publish pipelines issues that could arise in case of an incorrect DisplayVersion value.
Microsoft Reviewers: Open in CodeFlow

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 897)


Make completion::ESCAPE_CHAR public

This is necessary to correctly use the unescape function on every platform (i.e. to extract the path after it has been completed).

Closed in favour of a proper solution, see #106.


[builder] Ensure space to be gid 1 for COLR fonts

COLRv0 fonts render badly on Edge in Windows 10. The issue disappeared in Windows 11.
The browser appears to be inserting whichever glyph is on GID 1 in places such as colored glyphs, spaces, and also seemingly at random. While the screenshot below shows Arabic, the issue appears identically for Latin text.
The OpenType 1.7 Specs suggests the NULL glyph to be on GID 1, with the additional recommendation "Glyph 1 should have no contours and zero advance width".
However, this section disappeared from the OpenType 1.8 Specs and the issue is therefore to be considered a shaping bug in Windows 10.
I’m creating this issue because I believe that the space character should be put on GID 1 by gftools builder (whenever a COLR table is present) as a workaround, as I consider this to be knowledge that a font engineer (or worse, a type designer) shouldn't need to know, and can be fixed programmatically.
(The fact that it got fixed in Windows 11 suggests that Microsoft knows about the bug and won’t bother fixing it for Windows 10 users).
For reference, nanoemoji already knows about this and will put space on GID 1 when assembling a font from scratch from SVGs, but maximum_color won’t fix the order and @anthrotype said he’d like to keep it that way and have maximum_color only work on the color tables.
Also, I will reference this issue from the fontbakery check that I'm about to write.

on Edge in Windows 10.

i believe also on Chrome, not just Edge, on Windows 10, at least according to the above mentioned comment in nanoemoji code that says "Win 10 Chrome likes a blank gid1 so make gid1 space" (haven't actually tested this)

I think nanoemoji used to add an actual ".null" glyph with no contours and zero advances to appease Win10 initially, but then we figurted that since we already were adding a glyph without contours (space), it may as well be placed as the second glyph (gid1) in the font and that did the trick.

Wouldn't it be more beneficial to move this more upstream to say fontmake?

I don't know, the bug only affects COLRv0 fonts on Windows 10. Are you saying fontmake should move space glyph to gid1 all the time? The only requirement we make is that .notdef is the first glyph (otherwise cmap gets screwed), and we add one at gid0 if missing from the source.
How would we identify the "space" glyph. Can't simply rely on the name, that can be anything. I guess one would have to search the glyphs for one that's mapped to 0x0020 and has no contours. Note that space is not a required glyph either (never mind the legacy spec recommendations). The Win10 rendering bug would be worked around by placing any zero-contour glyph to gid1 (not necessarily "space" 0x0020). But there's also the possibility that the font source contains no zero-contour glyph, in which case you'd have to create one just for this.

i think general purpose tools like fontmake or maximum_color just do just what they are told, whereas more opinionated ones like gftools builder can choose to enforce that space be present and is at glyph index 1 whenever the output is COLR font and justify that with a bug affecting a major implementation.

Are you saying fontmake should move space glyph to gid1 all the time?

Just if we're dealing with a colr v0 font.

How would we identify the "space" glyph.

Something like
if `0x20` in cmap and noContours:
    # move to gid1
else:
    # make blank glyph

One could argue that we're trying to work around software bugs here. Perhaps we should submit a bug to MS since they're still supporting Win 10 till ~2024

They fixed it in Windows 11 and didn't bother for Windows 10...

AFAIK Win 10 still gets updates, https://support.microsoft.com/en-us/topic/september-13-2022-kb5017308-os-builds-19042-2006-19043-2006-and-19044-2006-e4ea187e-28e8-4d4b-808b-2794babdce4c

unlikely that they'll backport that fix.
Also, to know that the output is going to be a COLR v0 and not a COLR v1 font, ufo2ft would have to build the COLR table first and check what the resulting version coming off fontTools.colorLib actually is..

Another reason to move this upstream is that some folks do not use our builder. What about a ufo filter?

What about a ufo filter?

no, that gets too complicated. Would you like to work on a PR for ufo2ft?

I don't mind hurting myself on this. I'll probably opt for the easier option after attempting it ;-).

no need to get hurt! 😄
I agree that it'd be better if COLRv0 built with ufo2ft would just work on windows 10, which is still used widely.
I can help you find a suitable place inside ufo2ft codebase for this workaround. Please file an issue over there as well thanks!

although.. I am a bit wary that having ufo2ft shuffle glyphs around and not respect the explicit UFO public.glyphOrder because some windows bug may not be the right way to go about this. E.g. what if one sets public.glyphOrder and then expects that glyphs are sorted that way in the final font (some other tool may be relying on those explicit glyph indeces for whatever reason) but then ufo2ft decides to unilaterally change it because of that windows bug..
I think I'd be comfortable with having ufo2ft emit a warning message if the second glyph is not empty and the COLR table built is version 0, so the font developer can address that in the source and set the glyphOrder accordingly.
So to summarize, I suggest that ufo2ft does the following:

if UFO has no explicit glyphOrder, then place an empty glyph after .notdef and the rest sort alphabetically; we can safely do that for all the fonts, not just COLRv0. If they don't set an explicit glyphOrder, then we can assume they don't care about it and we can order them as we please
if UFO has an explicit glyphOrder and we produced a COLRv0, emit a warning so the font dev can update their explicit glyphOrder accordingly, iff they care about windows 10, which we can't always guarantee is the case.

Whilst having lunch I realised that inserting a blank glyph may affect vtt hint transfers since VTT relies on glyph index order. I may just add something to the gftools.fix module for the time being.

FWIW, the spec update was in response to https://github.com/MicrosoftDocs/typography-issues/issues/346.

@khaledhosny ah! Thanks Khaled, I knew I had seen this before and could not find that thread

Closing since #610 should fix this issue automatically.


fix: do not persist pending transactions across app sessions

Description
As the title - we currently don't have a way to resolve these pending transactions across app sessions.
Test plan
n/a
Related issues

Fixes RET-894

Backwards compatibility
Y

The only thing I'm concerned with that it could be hard to remember to get rid of it once peding txs can be resolved correctly.

yes indeed, i added an A/C to RET-896 to make sure we don't forget FYI @dievazqu


feat: add more tests for cw2981

Codecov Report

Merging #6 (9207e38) into main (88ba96b) will decrease coverage by 0.71%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   73.38%   72.68%   -0.71%
==========================================
  Files          12        8       -4
  Lines         590      432     -158
==========================================
- Hits          433      314     -119
+ Misses        157      118      -39

Impacted Files
Coverage Δ

contracts/cw2981-royalties/src/lib.rs
100.00% <100.00%> (+3.44%)
:arrow_up:


Rename catalog example to e.g. ckan

catalog example is also using ckan and that's an important point.
v.minor thing to do though ...

Issue fixed in commit 04d750f

This is now FIXED and the ckan example is available here - https://catalog-portal-js.vercel.app/


chart history broke for one account

https://stellar.expert/explorer/public/account/GAKYXYYEOQX2HM6WCHFSIT66NIRJXPJK4VYXO4L35SDJVEGU4EXKKT5G
for MOBI shows flat line at 0 but there has been a lot of in/out

Balance history chart shows aggregated balance at the end of the day. Every time GAKY...KT5G received MOBI, it transferred (distributed?) the whole amount to other accounts immediately. So MOBI balance at the end of the day was always 0. So it's an expected behavior
I'll see what can be done with it.
Thanks for submitting the issue.

ah yes that explains it :) PS love stellar.expert!!

Fixed in v0.9


Modal and Dialog lose styling in iFrame, mount in wrong container

[X] The issue is present in the latest release.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Loading Modals and Dialogs in iFrames lose their style.
Expected Behavior 🤔
iFrame should mount in the body of the iframe, not outside the iFrame container.
Steps to Reproduce 🕹
Using a slightly modified version of material-ui/docs/src/modules/components/DemoSandboxed.js

Create a component with a Modal
Use that component inside of the component prop
Open the modal

This is the same no matter how the iFrame is implemented.

Code sandbox:
https://codesandbox.io/s/dialog-iframe-issue-z0653?file=/src/App.js
Context 🔦
We have a page designer tool that works in a similar way to Storybook, though much simpler.
From a separate library using material UI, we import the components and set their props in this page designer.
Components imported from the separate library with modals lose their styles.

Your Environment 🌎
System:
    OS: macOS 11.2.3
  Binaries:
    Node: 12.18.0 - /usr/local/bin/node
    Yarn: 1.17.3 - /usr/local/bin/yarn
    npm: 7.10.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 90.0.4430.85
    Edge: Not Found
    Firefox: 84.0.1
    Safari: 14.0.3
  npmPackages:
    @emotion/react: ^11.1.5 => 11.1.5
    @material-ui/core: ^4.11.3 => 4.11.3
    @material-ui/lab:  4.0.0-alpha.57
    @material-ui/styles: ^4.11.3 => 4.11.3
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react:  17.0.0
    react: ^16.14.0 => 16.14.0
    react-dom: ^16.14.0 => 16.14.0
    styled-components: ^5.2.3 => 5.2.3

Same as #13625. You need to target the styles to be injected in the iframe.


new console.Console() constructs the wrong thing

Versions
node-inspector version: 0.12.8
node.js version: 4.4.2
OS name and version: OSX 10.11.4
Description
In my app I'm logging by doing:
const logger = new console.Console(outStream, errStream);

Then I export the logger to the rest of my app. This works just fine, until I try to run using node-inspector, and the app crashes because logger.log() and logger.error() are undefined. Setting a breakpoint, my logger is a WRAPPED_BY_NODE_INSPECTOR object which doesn't implement any of the expected console methods.
Screenshot

Workaround
Doing some tests, I tried this
console.log(console.constructor === console.Console);
// running normally, logs `true`
// running with node-inspector, logs `false`

Thus I'm able to workaround the error by changing my code to:
const logger = new console.constructor(outStream, errStream);

But it seems hackish, so I thought I'd create an issue.

Would be fixed in next version.
Fix in master branch now.


[Validator] Fix to NPE occurs when adding a new field to openapi:ServiceInfo annotation

Fix #603
Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Shall we add a test case for open-api validator for this scenario ?

Shall we add a test case for open-api validator for this scenario ?

Yeah sure I tested this manually this fix didn't give NPE but received an "Undefined embed field" error message since this field introducing is with this PR https://github.com/ballerina-platform/ballerina-openapi/pull/599
I will add tests with fixing this issue https://github.com/ballerina-platform/ballerina-openapi/issues/22 :)


operator hive-operator (1.2.4054-8ea4577)

Update Hive community operator channel(s) [['alpha']] to 1.2.4054-8ea4577

/test 4.8-deploy-operator-on-openshift

/merge possible

/merge possible

/merge possible


Add fail_level to inputs

Add fail_level and make fail_on_error deprecated.

Thank you for your contributing.
I understand that -fail-on-error option is deperecated at v0.20.2.
https://github.com/reviewdog/reviewdog/blob/master/CHANGELOG.md#rotating_light-deprecation-warnings

I created v1.7.0-beta for testing.
https://github.com/mongolyy/reviewdog-action-biome/releases/tag/v1.7.0-beta
Once I have confirmed that there are no problems, I'll release v1.7.0.


Enforcing back tenant selection using closest match for namespace prefix

Should close #123.

@MaxFedotov please, take a look at the updated test case: I guess it's safe enough since we're implementing a sort of closest match.

@prometherion LGTM! Thank!


[Feature] Overlay for Videos

Feature Request
At the moment, we can't add an overlay to CldVideoPlayer

Describe the solution you'd like
It would be nice to have overlays/watermarks to videos

hey @lelouchB check out this second example:
https://next-cloudinary-gsjwm4f2y-colbyfayock.vercel.app/components/cldvideoplayer/examples
thats what you had in mind right?
I'm using the transformation parameter for now and just passing it through
id like to eventually build a better API, like have an overlays= prop like CldImage, but for now, i wanted to get something together as i want to put more thought around that
gonna wait to get your eyes on this before i merge it :)

That's exactly what I needed
Thanks so much :)


[FEATURE]: Add all parameters to the 'info' section of the OpenAPI specification

Is your feature request related to a problem
I cannot display the author details, license details, etc. nicely in the automatically generated documentation and in the OpenAPI specification. I want to be able to do this, when I call the app = FastAPI() statement.
The solution you would like
I want to see this additional metadata in the info section of the generated OpenAPI specification. In order to achieve this, I need the FastAPI() call to support additional parameters, like termsOfService, contact and license.
Describe alternatives you've considered
no alternatives.
Additional context
Check OpenAPI specification.

I found the answer to my own question / request. It is possible to extend the generated OpenAPI specification with new fields, if you follow the guideline described in this link: https://fastapi.tiangolo.com/tutorial/extending-openapi/
This ticket can be closed, unless the developer team wants the FastAPI() call to support all the info parameters described here: https://swagger.io/specification/#infoObject

Thanks @ka-zo for reporting back your solution.
I think we could add an additional parameter to the FastAPI class to take the extra OpenAPI data. But I'm not sure yet about names, how many, and structure. And as this has not been commonly requested, it hasn't had a high priority.
I would accept a PR with it if someone was interested in it, though.
With respect to this issue, I think that for now we can take it as a question and close it. And if this comes up later and more people are requesting it and the current method is too cumbersome, we can think of adding it.

Agreed, and the solution described in the aforementioned link (https://fastapi.tiangolo.com/tutorial/extending-openapi/) is already satisfactory.

There's direct support for this now in FastAPI 0.68.0 🔖🎉
https://fastapi.tiangolo.com/tutorial/metadata/


gha: lockfiles labels and auto-pr details

🚀 Pull Request
Description
Replaces #4561

Consult Iris pull request check list

Whoops! Apparently, there was an error in the file https://github.com/SciTools/iris/actions/runs/1824096068/workflow
It's complaining about the with block (I think! I don't find that "Invalid workflow file" error message terribly illuminating). I'm not sure what the problem is though. It could be the labels as I had removed that when testing in my repo

Turns out the problem was with the title not the label. I have proposed a fix in #4579.