TypeError: write() only accepts bytes, unicode, and dict objects

(mediadata) bob@hostname:~/project/media_soe$ jupyter notebook --port 5000 --no-browser
[I 12:01:24.641 NotebookApp] [jupyter_nbextensions_configurator] enabled 0.4.1
[INFO 2020-03-31 12:01:24.642 TabNine]:  install dir: /home/bob/.virtualenvs/mediadata/lib/python3.6/site-packages/jupyter_tabnine
[INFO 2020-03-31 12:01:24.643 TabNine]: TabNine binary already exists in /home/bob/.virtualenvs/mediadata/lib/python3.6/site-packages/jupyter_tabnine/binaries/2.5.1/x86_64-unknown-linux-gnu/TabNine ignore downloading
[WARNING 2020-03-31 12:01:27.313 TabNine]:  wrong response of turning on semantic completion
[I 12:01:27.315 NotebookApp] Serving notebooks from local directory: /home/bob/project/media_soe
[I 12:01:27.315 NotebookApp] The Jupyter Notebook is running at:
[I 12:01:27.315 NotebookApp] http://localhost:5000/?token=0bb320619291829c2f74580820b382e36233bfec349e36e6
[I 12:01:27.315 NotebookApp]  or http://<IP_ADDRESS>:5000/?token=0bb320619291829c2f74580820b382e36233bfec349e36e6
[I 12:01:27.315 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[C 12:01:27.325 NotebookApp]

    To access the notebook, open this file in a browser:
        file:///home/bob/.local/share/jupyter/runtime/nbserver-7983-open.html
    Or copy and paste one of these URLs:
        http://localhost:5000/?token=0bb320619291829c2f74580820b382e36233bfec349e36e6
     or http://<IP_ADDRESS>:5000/?token=0bb320619291829c2f74580820b382e36233bfec349e36e6
[W 12:01:32.510 NotebookApp] 404 GET /api/kernels/20be7e9c-d6a9-4a1e-a457-e147e5f9bc48/channels?session_id=e52034af673c417f9fd8db20a0c906bf (<IP_ADDRESS>): Kernel does not exist: 20be7e9c-d6a9-4a1e-a457-e147e5f9bc48
[W 12:01:32.549 NotebookApp] 404 GET /api/kernels/20be7e9c-d6a9-4a1e-a457-e147e5f9bc48/channels?session_id=e52034af673c417f9fd8db20a0c906bf (<IP_ADDRESS>) 45.41ms referer=None
[W 12:01:48.582 NotebookApp] Replacing stale connection: 20be7e9c-d6a9-4a1e-a457-e147e5f9bc48:e52034af673c417f9fd8db20a0c906bf
[I 12:01:52.360 NotebookApp] 302 GET / (<IP_ADDRESS>) 0.91ms
[I 12:02:04.074 NotebookApp] Kernel started: 5d2f6a7a-b8b9-4ff0-9988-86dd3126e0ca
[E 12:02:16.210 NotebookApp] Uncaught exception GET /tabnine?data=%7B%22version%22%3A%221.0.7%22%2C%22request%22%3A%7B%22Autocomplete%22%3A%7B%22filename%22%3A%2201-print%2Fnotebook%2Flists-of-prints.py%22%2C%22before%22%3A%22%23%25config%20IPCompleter.greedy%3DTrue%5Cnimport%20pandas%20as%20pd%5Cnimport%20numpy%20as%20np%5Cndf%20%3D%20pd.P%22%2C%22after%22%3A%22%5Cndf%20%3D%20pd.DataFrame()%5Cnpath_data%20%3D%20%5C%22..%2Fdata%2F2019-10-22_Toutes_les_fiches_techniques_Print.xls%5C%22%5Cndf%20%3D%20pd.read_excel(path_data)%5Cndf.head()%5Cndf.columns%5Cndf.drop(%5B%27Unnamed%3A%200%27%5D%2C%20axis%3D1%2C%20inplace%3DTrue)%5Cndf.head()%5Cndf.columns%5Cnlcol%20%3D%20%5B%27number%27%2C%20%27wbs%27%2C%20%27name%27%2C%20%27pm_name%27%2C%27status%27%2C%27finished%27%2C%27recording_type%27%2C%27univers%27%2C%27notes%27%2C%27client%27%2C%27product%27%2C%27industrie%27%2C%27gfk_dep%27%2C%27collect%27%5D%5Cndf.columns%20%3D%20lcol%5Cndf.head()%5Cnimport%20numpy%20as%20np%5Cnimport%20pandas%20as%20pd%5Cnfrom%20pandas_profiling%20import%20ProfileReport%5Cnprofile%20%3D%20ProfileReport(df%2C%20title%3D%27Pandas%20Profiling%20Report%27%2C%20html%3D%7B%27style%27%3A%7B%27full_width%27%3ATrue%7D%7D)%5Cnprofile%5Cn%23df.number%20%3D%20df.number.str.replace(%27_001%27%2C%20%27%27).map(int)%5Cndf.number%20%3D%20df.number.astype(str).str%5B0%3A6%5D.map(int)%5Cndf.head()%5Cn%23%20Number%20of%20unique%20surveys%5Cnlen(df)%5Cnlen(df.number.unique())%5Cn**142**%5Cn%25%25bash%5Cnls%20~%2Fmnt%2FWorkflow%2FCrossteam%2FSurveys%2F*%2F%5Cnimport%20glob%5Cnimport%20os%5Cnlsurvey%20%3D%20glob.glob(os.environ%5B%27HOME%27%5D%2B%27%2Fmnt%2FWorkflow%2FCrossteam%2FSurveys%2F*%2F*%27)%5Cnlidfr%20%3D%20%5B%20int(i.split(%27%2F%27)%5B8%5D.split(%27%20-%20%27)%5B0%5D.replace(%27%20%27%2C%27%27))%20for%20i%20in%20lsurvey%5D%5Cnlidfr%5Cndf.number.tolist()%5Cnset(df.number.tolist())%20%26%20set(lidfr)%5Cnfind%20%2F%20-type%20d%20-name%20%5C%22dir-name-here%5C%22%22%2C%22region_includes_beginning%22%3Atrue%2C%22region_includes_end%22%3Atrue%2C%22max_num_results%22%3A10%7D%7D%7D (<IP_ADDRESS>)
    HTTPServerRequest(protocol='http', host='localhost:5000', method='GET', uri='/tabnine?data=%7B%22version%22%3A%221.0.7%22%2C%22request%22%3A%7B%22Autocomplete%22%3A%7B%22filename%22%3A%2201-print%2Fnotebook%2Flists-of-prints.py%22%2C%22before%22%3A%22%23%25config%20IPCompleter.greedy%3DTrue%5Cnimport%20pandas%20as%20pd%5Cnimport%20numpy%20as%20np%5Cndf%20%3D%20pd.P%22%2C%22after%22%3A%22%5Cndf%20%3D%20pd.DataFrame()%5Cnpath_data%20%3D%20%5C%22..%2Fdata%2F2019-10-22_Toutes_les_fiches_techniques_Print.xls%5C%22%5Cndf%20%3D%20pd.read_excel(path_data)%5Cndf.head()%5Cndf.columns%5Cndf.drop(%5B%27Unnamed%3A%200%27%5D%2C%20axis%3D1%2C%20inplace%3DTrue)%5Cndf.head()%5Cndf.columns%5Cnlcol%20%3D%20%5B%27number%27%2C%20%27wbs%27%2C%20%27name%27%2C%20%27pm_name%27%2C%27status%27%2C%27finished%27%2C%27recording_type%27%2C%27univers%27%2C%27notes%27%2C%27client%27%2C%27product%27%2C%27industrie%27%2C%27gfk_dep%27%2C%27collect%27%5D%5Cndf.columns%20%3D%20lcol%5Cndf.head()%5Cnimport%20numpy%20as%20np%5Cnimport%20pandas%20as%20pd%5Cnfrom%20pandas_profiling%20import%20ProfileReport%5Cnprofile%20%3D%20ProfileReport(df%2C%20title%3D%27Pandas%20Profiling%20Report%27%2C%20html%3D%7B%27style%27%3A%7B%27full_width%27%3ATrue%7D%7D)%5Cnprofile%5Cn%23df.number%20%3D%20df.number.str.replace(%27_001%27%2C%20%27%27).map(int)%5Cndf.number%20%3D%20df.number.astype(str).str%5B0%3A6%5D.map(int)%5Cndf.head()%5Cn%23%20Number%20of%20unique%20surveys%5Cnlen(df)%5Cnlen(df.number.unique())%5Cn**142**%5Cn%25%25bash%5Cnls%20~%2Fmnt%2FWorkflow%2FCrossteam%2FSurveys%2F*%2F%5Cnimport%20glob%5Cnimport%20os%5Cnlsurvey%20%3D%20glob.glob(os.environ%5B%27HOME%27%5D%2B%27%2Fmnt%2FWorkflow%2FCrossteam%2FSurveys%2F*%2F*%27)%5Cnlidfr%20%3D%20%5B%20int(i.split(%27%2F%27)%5B8%5D.split(%27%20-%20%27)%5B0%5D.replace(%27%20%27%2C%27%27))%20for%20i%20in%20lsurvey%5D%5Cnlidfr%5Cndf.number.tolist()%5Cnset(df.number.tolist())%20%26%20set(lidfr)%5Cnfind%20%2F%20-type%20d%20-name%20%5C%22dir-name-here%5C%22%22%2C%22region_includes_beginning%22%3Atrue%2C%22region_includes_end%22%3Atrue%2C%22max_num_results%22%3A10%7D%7D%7D', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
    Traceback (most recent call last):
      File "/home/bob/.virtualenvs/mediadata/lib/python3.6/site-packages/tornado/web.py", line 1703, in _execute
        result = await result
      File "/home/bob/.virtualenvs/mediadata/lib/python3.6/site-packages/jupyter_tabnine/handler.py", line 14, in get
        self.write(response)
      File "/home/bob/.virtualenvs/mediadata/lib/python3.6/site-packages/tornado/web.py", line 834, in write
        raise TypeError(message)
    TypeError: write() only accepts bytes, unicode, and dict objects
[E 12:02:16.214 NotebookApp] {
      "Host": "localhost:5000",
      "User-Agent": "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:74.0) Gecko/20100101 Firefox/74.0",
      "Accept": "*/*",
      "Accept-Language": "en-US,en;q=0.5",
      "Accept-Encoding": "gzip, deflate",
      "X-Requested-With": "XMLHttpRequest",
      "Connection": "keep-alive",
      "Referer": "http://localhost:5000/notebooks/01-print/notebook/lists-of-prints.ipynb",
      "Cookie": "username-localhost-5000=\"2|1:0|10:1585646585|23:username-localhost-5000|44:YjcwMmI0NjJhMDg1NDBkY2JhOTNlY2JmOGU0MDE0NTM=|b0dca45e70ea81d989d8a8f32cba9415d7a79358fb8975812ed7a4f43ae1f845\"; _xsrf=2|a827a59c|8bd2453659d2d4eb00d65b78172de365|1583854181; csrftoken=GEFigwPHZbf42XpAMDdrKSt1XNMuzWNz13UGtWil7j4cBKe3LkQwPgGp423VYjM3"
    }
[E 12:02:16.214 NotebookApp] 500 GET /tabnine?data=%7B%22version%22%3A%221.0.7%22%2C%22request%22%3A%7B%22Autocomplete%22%3A%7B%22filename%22%3A%2201-print%2Fnotebook%2Flists-of-prints.py%22%2C%22before%22%3A%22%23%25config%20IPCompleter.greedy%3DTrue%5Cnimport%20pandas%20as%20pd%5Cnimport%20numpy%20as%20np%5Cndf%20%3D%20pd.P%22%2C%22after%22%3A%22%5Cndf%20%3D%20pd.DataFrame()%5Cnpath_data%20%3D%20%5C%22..%2Fdata%2F2019-10-22_Toutes_les_fiches_techniques_Print.xls%5C%22%5Cndf%20%3D%20pd.read_excel(path_data)%5Cndf.head()%5Cndf.columns%5Cndf.drop(%5B%27Unnamed%3A%200%27%5D%2C%20axis%3D1%2C%20inplace%3DTrue)%5Cndf.head()%5Cndf.columns%5Cnlcol%20%3D%20%5B%27number%27%2C%20%27wbs%27%2C%20%27name%27%2C%20%27pm_name%27%2C%27status%27%2C%27finished%27%2C%27recording_type%27%2C%27univers%27%2C%27notes%27%2C%27client%27%2C%27product%27%2C%27industrie%27%2C%27gfk_dep%27%2C%27collect%27%5D%5Cndf.columns%20%3D%20lcol%5Cndf.head()%5Cnimport%20numpy%20as%20np%5Cnimport%20pandas%20as%20pd%5Cnfrom%20pandas_profiling%20import%20ProfileReport%5Cnprofile%20%3D%20ProfileReport(df%2C%20title%3D%27Pandas%20Profiling%20Report%27%2C%20html%3D%7B%27style%27%3A%7B%27full_width%27%3ATrue%7D%7D)%5Cnprofile%5Cn%23df.number%20%3D%20df.number.str.replace(%27_001%27%2C%20%27%27).map(int)%5Cndf.number%20%3D%20df.number.astype(str).str%5B0%3A6%5D.map(int)%5Cndf.head()%5Cn%23%20Number%20of%20unique%20surveys%5Cnlen(df)%5Cnlen(df.number.unique())%5Cn**142**%5Cn%25%25bash%5Cnls%20~%2Fmnt%2FWorkflow%2FCrossteam%2FSurveys%2F*%2F%5Cnimport%20glob%5Cnimport%20os%5Cnlsurvey%20%3D%20glob.glob(os.environ%5B%27HOME%27%5D%2B%27%2Fmnt%2FWorkflow%2FCrossteam%2FSurveys%2F*%2F*%27)%5Cnlidfr%20%3D%20%5B%20int(i.split(%27%2F%27)%5B8%5D.split(%27%20-%20%27)%5B0%5D.replace(%27%20%27%2C%27%27))%20for%20i%20in%20lsurvey%5D%5Cnlidfr%5Cndf.number.tolist()%5Cnset(df.number.tolist())%20%26%20set(lidfr)%5Cnfind%20%2F%20-type%20d%20-name%20%5C%22dir-name-here%5C%22%22%2C%22region_includes_beginning%22%3Atrue%2C%22region_includes_end%22%3Atrue%2C%22max_num_results%22%3A10%7D%7D%7D (<IP_ADDRESS>) 6.58ms referer=http://localhost:5000/notebooks/01-print/notebook/lists-of-prints.ipynb

This seems more related to the jupyter notebook plugin.
Can you open an issue there?


[BUG] Tapping any tab in Google search redirects back to Web tab when top URL includes Google search engine

Root cause:
Top of url included support url. Redirect / reset of url is done always without checking for additional or modifier
Sets to reproduce:

Set top of url to - https://google.com/search?udm=14&q= (this is a redirect to the new “web” tab in google search).
Set default search engine as - Google - both in CSE and Safari Settings
Attempt to make a search
Try changing to images tab or shopping tab or back to all tab
You get redirected back to web tab

Issue isn’t reproducible if DuckDuckGo is set as the default search engine both in CSE and Safari Settings.
Thank you in advance

This is a known bug, but I have not found a way to fix it so far.
This bug occurs in Google, 360Search, and Yandex.


Let PrinterMonitor find the Octoprint IP

When Octoprint installs you always have to give it a name as it appears on the network, so I figured why not use it instead of a hard coded, sometimes prone to change, IP?
Let Printer Monitor do a scan at the startup, find the matching name that was used for the Octoprint install, then set the IP itself. Regardless of the router changes, printer monitor will always find Octoprint's IP.
Mel
Settings.h
String OctoPrintServer = "x.x.x.x"; // IP or Address of your OctoPrint Server (DO NOT include http://)
String OctoHostName = "DROOLER";// name of the octoprint install
printermonitor
void setup() {
findMDNS();// insert as very last line in setup(), go find Octoprint
}
void findMDNS() {
// We now query our network for 'web servers' service
// over tcp, and get the number of available devices
int n = MDNS.queryService("http", "tcp");
if (n == 0) {
Serial.println("no services found");
}
else {
for (int i = 0; i < n; ++i) {
// Going through every available service,
// we're searching for the one whose hostname
// matches what we want, and then get its IP
//Serial.println(MDNS.hostname(i));
//Serial.println(MDNS.IP(i));
//Serial.println(MDNS.port(i));
if (MDNS.hostname(i) == OctoHostName) {
OctoPrintServer = String(MDNS.IP(i));// shove in the IP of the octoprint instance
}
}
}
}

This will be a nice addition.  I will look at it closer here this week.

Thanks Mel, I will start testing it later this week when I get some time.

Mel, I have updated the dev branch of 1.5 with the MDNS -- it also includes the web ui configuration for this as well.  It seems to be working well for me.  I also have it check for the IP after saving the configuration if the user changes the hostname.    When you get time try it out:  https://github.com/Qrome/printer-monitor/tree/1.5

I'll check it out tomorrow and let you know.

Working perfectly here David!


Added password input for firmware 1.5.0_0027

I've upgraded to firmware 1.5.0_0027 before using Home Assistant, now the login asks for a password, 'admin' works fine.
I changed it using passwd to another one, doesn't seem to be a problem for the xiaomi app.
Data of all my miaomiaoce.sensor_ht.h1 and miaomiaoce.sensor_ht.t2 sensors are now also received in Home Assistant.
Zigbee is used in my case in Zigbee Home Automation (ZHA) mode to connect to Megos LIGHTWAY bulb and remote (TuYa-based), which also works fine.

1.5.0_0027 is no more actual. v1.5.0_0102 is latest firmware without password bug


fix(github-pull-requests-board): filter out github accounts that have…

… been deleted
Pull request that contain interactions with user accounts that have been
deleted are null in the response from the graphql API. This change prevents
deleted accounts from breaking the rendering of the PR.
Signed-off-by: Niall McCullagh<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hmm, so just to clarify, basically, PR's that are by user accounts that have been deleted cause this plugin to crash?
Shouldn't we just default the user to an empty user or guest user or something rather than not showing the PR at all? Feels a little unfortunate that the PR doesn't disappear when the user does, so why treat them the same?

The plugin crash not only when the PR is opened by the account that was deleted but if the deleted user has interacted with the PR at all (comment, reviewed). The UserHeaderList tries to assign login and  avatarUrl by destructuring assignment but the user involved is null this throws an exception that bubbles up. For example,

This change still shows the PR but does not display the deleted user's details. That is, if the creator of the PR is a deleted account then no avatar is displayed.
I did think about replacing the null with a stubbed Ghost user but decided against hardcoding the username/avatar of the Ghost user as Github could change this at any time. If you think that this would actually be a better approach I'm open to change.

@niallmccullagh yeah, I think we have a problem here that we're handling bad data wrong, and we should be catching for these use cases. I think we can just skip any of the users that are null from the list and try not to destructure the avatarUrl at all in these cases.

I think we can just skip any of the users that are null from the list and try not to destructure the avatarUrl at all in these cases.

That's what the change does @benjdlambert. Is there anything you would like me to change in the PR?


Tag GA sessions with metadata to indicate whether a user is an hei scitt or sd user

Context
We'd like to be able to slice GA data by provider type.

Changes proposed in this pull request

Adds a method to determine the primary provider type, this will be either university or scitt or school_direct based on the types of the providers the user belongs to.
Sets the primary provider type on the custom GA dimension (index 2 because that's the next one available).

Guidance to review
We have this dimension set on our Manage Test GA account, verify it's working as expected before further deployment.

Link to Trello card
https://trello.com/c/QzpVg0M5/3511-tag-ga-sessions-with-metadata-to-indicate-whether-a-user-is-an-hei-scitt-or-sd-user

Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] This code does not rely on the addition/removal of Azure config environment variables in the same Pull Request
[x] New environment variables have been added to the Azure config

@cwrw I had to make a slight tweak here https://github.com/DFE-Digital/apply-for-teacher-training/pull/4506/commits/3ca96d7a29ae93119f329ec51626d5cf03264441#diff-5183cb85f5b7e77048cc9ae789796cfa9ae0378ac7743b6f7a8294a4bf6fa35fR37 as current_provider_user will return false if not present and this will raise undefined method 'primary_provider_type' for false:FalseClass if we use the .& safe navigation operator.


Setting up airbyte to use vault as its secret store.

Closes #962
Description
Sets up airbyte to use vault as its secret store rather than storing secrets in plaintext within postgres.
Requires redeploying the environment from scratch. Airbyte doesn't know to fall back to postgres for old secrets after making this change.
Motivation and Context
Be more secure.
How Has This Been Tested?
Redployed the QA environment with this new config.
Types of changes

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

Checklist:

[X] My code follows the code style of this project. (Did you install and run the pre-commit hooks?)
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

:+1:


Dummy change to check if CI runs

cc @dimitropoulos

 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.


AI Request

I think you should add some AI features to your website

The AI feature has been built


Make web app mobile friendly

Error solved by: <meta name="viewport" content="width=device-width, initial-scale=1.0"/>


glib::closure! macro and signal connector should use more descriptive assertions

E.g. currently when the number of arguments does not match one simply gets an assertion that e.g. 2 != 5 without any hints about the actual problem.
CC @jf2048 @bilelmoussaoui

I think we can try with another shot at this one again: https://github.com/gtk-rs/gtk-rs-core/pull/567 now that it is looking like GATs will make it into Rust 1.65?

Yeah worth trying again with 1.65 :) Not sure if there's enough GAT to make it work though.


Device which python datatype to add unit tests for and inform course assistant which to add

We should choose one python datatype, between List, Dictionary, ByteArray, Bytes, and OrderedDict, to add unit tests for. We should inform the course assistant which datatype we chose.

We have decided to test list. @Melkster has mailed this to raphaela


Comparator materialdata

Adds mechanism for adjusting the switch for comparators and a tag for what it's current state is.

Noting one of your commits ordering them alphabetically, I put it in the right place and ordered it as well


Local volumes does not work for cluster snapshot

To make this work, we have to create a Job on the fly that can mount the local volume and take backup.

This works. Needs to be mounted in kubed.


cache TDA calls

the reason this did not work so well is because the input are numpy arrays which are mutable

I think the original goal was to avoid re-computation if we compute fit first and then later featurize


docs(securitycenter): Fix conflict tag introduced in PR 1957

Last One hopefully!
Fixing a conflicting tag introduced in #1957. I overlooked this and added the same tag twice, sorry..

Please prioritize this small PR so I can submit doc-side updates to Security Command Center. Thank you!


[Mev Boost\Builder] Makes proposer public key available for getHeader

PayloadAttributesCalculator becomes ProposersDataManager which is now also responsible of forwarding  registration data to builder api. This data will be also used to determine if a proposer has been configured to use builder api. "prepare_beacon_proposer" and register_validator are not currently not supporting the option of disabling builder api. if we want to I think we may add a new attribute in "prepare_beacon_proposer" (like a teku object holding our custom config) which may contain a disable\enable flag.
enrich PayloadAttributes (now PayloadBuildingAttributes) with ValidatorRegistration information
various refactoring

Seems to me that in ValidatorApiHandler we need to remove ForkChoiceNotifier and add  ProposersDataManager
which will handle prepare and register apis. A subscription should notify the ForkChoiceNotifier when prepared proposer has been called.
Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[ ] I thought about adding a changelog entry, and added one if I deemed necessary.

if we want to I think we may add a new attribute in "prepare_beacon_proposer" (like a teku object holding our custom config) which may contain a disable\enable flag.

I think generally the model is that the beacon node controls whether a builder is used and which one. If there's a requirement to decide that per-validator we should probably get it added to the standard validator APIs (probably as an optional section).  I'm not keen to have teku specific extensions there if we can avoid it.

I think generally the model is that the beacon node controls whether a builder is used and which one.

ah that makes sense. So activating\deactivating mev for a given validator turn to be moving from\to a VC that is using a BN with builder enabled\disabled.


Picking up search results embedded in "People also ask"

The results are included in the navigation even when the questions are collapsed, so you have to press down several times to get to the first real result. The HTML structure of these embedded results on this particular page is similar to the main search results. I have come across other "People also ask" boxes that were not included in the navigable result set.

I've been unable to replicate this. It only seems to happen when the extension is reloaded with the search results already open. I assume this means that the "People also ask" box is added dynamically after the extension runs under normal circumstances.

Thanks! I also ran into this a few times and I definitely want to fix it.
Let's keep it open for now and I'll try to make progress when I have more time.


Is it possible to boot up only the Intercom messanger?

Hello,
Is it possible to open the Intercom messenger without the Intercom Icon. I've see a few sites, upon request helping, boots up Intercom without the Intercom Icon. Take for example dYdX. If you click on the Help Chat icon in the upper left corner, it opens the Intercom messenger without the Intercom icon.

Hi, thanks for creating this issue.
Yes there is you can set hideDefaultLauncher to false in both the boot and update function from useIntercom.

Awesome, that worked! Thanks for the help 👍!


chore: qs-jx-v3 to 0.0.10

chore: Promote qs-jx-v3 to version 0.0.10

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


resource-types: schema.org improvements

Description
Improvements to the Schema.org properties on Resource Types. There are three changes in this PR:

publication-dissertation: schema.org/thesis (lowercase t) is a 404, use schema.org/Thesis instead
event: schema.org/event is an event property not event Thing, should be schema.org/Event instead
publication-thesis: use schema.org/Thesis like the publication-dissertation instead of https://schema.org/ScholarlyArticle

Aside, I did ./run-tests.sh afterwards (I also updated these resource types values in a testing fixture), but there seem to be several modules missing for the tests script to actually work. I had to pipenv run pip install docker-services-cli check_manifest sphinx before they would complete.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[x] I'm aware of the code of conduct.
[x] I've created logical separate commits and followed the commit message format.
[x] I've added relevant test cases.
[x] I've added relevant documentation.
[x] I've marked translation strings (for relevant code).
[x] I've followed the CSS/JS and React guidelines (for relevant code).
[x] I've followed the web accessibility guidelines (for relevant code).
[x] I've followed the user interface guidelines (for relevant code).
[x] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[x] I've NOT included third-party code (copy/pasted source code or new dependencies).

Thanks a lot for this fix!


converts image considering format provided

Solves issue #150 . The pdf2img script previously accepted both jpg and png as output format arguments, but always converted to png regardless of the argument provided. This PR fixes issue by correctly converting the PDF to the specified format.

 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.


fix: title in sidebar

cw3-flex-multisig was shown as cw3-fixed-multisig in sidebar

@orkunkl does this auto-deploy on merge or do you need to manually update the docs site?

@orkunkl does this auto-deploy on merge or do you need to manually update the docs site?

It auto deploys (https://github.com/CosmWasm/docs/blob/master/.github/workflows/main.yml#L31)

Thank you so much.


BDS-138 : Remove Scope field from definition.json file

Removed scope field from definition file and updated ReadMe

Test this please.


Token approvals in ContractWrite

Enable users to handle token approvals when they use the ContractWrite component.

Done you can see it in action:
https://github.com/verynifty/fastdapp/blob/main/examples/aave_pool.md?plain=1#L182


OrganizeImports produces non-sensical imports

OrganizeImports produced non-sensical imports, like
// test-tests/shared/src/test/scala/zio/test/environment/RandomSpec.scala

import scala.util.Random.random.Random

// test/shared/src/main/scala/zio/test/package.scala

import zio.test.PlatformSpecific.TestEnvironment.collection.immutable.SortedSet
...
import scala.collection.immutable.SortedSet.duration.Duration

Discovered in PR https://github.com/zio/zio/pull/4449/
Full commit with all the changes OrganizeImports made is here: https://github.com/zio/zio/pull/4449/commits/495623ed9168a0646ecd5ef73526a43ad8d986ac
These non-sensical imports had to be manually fixed by this commit https://github.com/zio/zio/pull/4449/commits/689c20576eeb156c972addff4b0141182b1fd883
The original snapshot of the codebase is here https://github.com/zio/zio/tree/1816c4232ee16f8e50ef4291b7810ebdea4e8529 + you have to adjust the OrganizeImports settings to
OrganizeImports {
  expandRelative = true
  groupedImports = Merge
  # IntelliJ IDEA's order so that they don't fight each other
  groups = [
    "java."
    "*"
    "scala."
  ]
}

RemoveUnused {
  imports = false // handled by OrganizeImports
}

It should be 100% reproducible, just run fix from sbt
The version of OrganizeImports used is 0.4.4 https://github.com/zio/zio/blob/1816c4232ee16f8e50ef4291b7810ebdea4e8529/project/BuildHelper.scala#L236
Are there any other details that would help you debug this?

@sideeffffect, thanks for reporting! I was trying to reproduce this issue, but running either sbt fix or sbt test/scalafixAll hits GC issues and sbt kept crashing. I checked ZIO CircleCI settings, but I didn't find anything related to JVM memory settings. Does ZIO has certain JVM/GC requirements to build?

Another thing I tried was invoking the "organize imports" code action (which is powered by OrganizeImports) in Vim with Metals 0.9.7 just on those two problematic files, and I got the correct results for both of them. Metals 0.9.7 uses OrganizeImports 0.4.2, could you please help to confirm whether 0.4.2 works for you? If there's still an issue, I suspect it's due to certain unexpected interactions between different Scalafix rules.
BTW, my previous sbt test/scalafixAll run finally ended with an OOM error :(

Maybe running with more memory (-Xmx4092M) could help. Full build on my computer takes cca 10 mins.

I've tried to run it again on the same snapshot with both 0.4.2 and 0.4.4, and fix finished with different outcomes. OrganizeImports always produced nonsensical imports in different two files. It indeed seems like some weird interaction between Scalafix rules.
It's not a huge deal, it's easy to fix manually, because it's only in few files.
But it still would be great to get to the bottom of the problem. Perhaps you could use ZIO as a stress test for OrganizeImports, but you'll have to give sbt a lot of memory (as I've shown above).

@sideeffffect, thanks for the follow-up! Using ZIO as a stress test is a great idea. I did want to add a few automated tests to check out certain repos and run OrganizeImports against them for correctness test and maybe also perf tests.
Unfortunately, I don't have enough cycles recently, but will try to find some time to root cause this issue later.


Document Channels in eventing-contrib

From @mattmoor :
Context: I've been trying to consolidate our install docs, and something that popped out of that is trying to incorporate the stuff documented here, which includes a random subset of sources: https://knative.dev/docs/install/knative-custom-install/
Ask: Here's what I'd like to see for every source that lives in Knative in some capacity:

a regular release with install instructions
a sample that we can link from the install block (bonus: tests!)
for everything not satisfying the above: a feature track project board, which clearly calls out who owns it and what work is left.

/assign

The project has two channel implementations,
[] NATSS: A little bit of info in here. But could be improved. @Abd4llA any means to update this?
[] Apache Kafka:  Documentation and install instructions are here.

For Kafka we added it to the knative-with-any-k8s.md :  https://github.com/knative/docs/pull/2200

NATS PR -> https://github.com/knative/docs/pull/2200

The original ask quoted was for sources, I think we should have both tracked, one for channels and one for sources.

Sources are here, @vaikas   https://github.com/knative/eventing-contrib/issues/894

sorry :) the quote pasted to the issue definition talks about sources :)

I think I cross-posted the same thing to channels as a "We should do this for channels too" so probably my bad :)

Last missing piece is https://github.com/knative/docs/pull/2214  from @nachocano
@nachocano once your PR merges, can you close this ?

Last missing piece is knative/docs#2214 from @nachocano
@nachocano once your PR merges, can you close this ?

Sure!


[New] Measure distance in scene

Description
This PR implements Measure distance in scene in Analysis category.
URL to README: https://github.com/Esri/arcgis-maps-sdk-swift-samples/tree/Caleb/New-MeasureDistanceInScene/Shared/Samples/Measure distance in scene
Linked Issue(s)

swift/issues/4158

How To Test

Tap on the map to set the location distance measurement start location.
Tap again to set the location distance measurement end location.
Tap on "Imperial" to change the location distance measurement unit system.

Screenshots

To Discuss

As of PR creation, this sample does not implement any drag functionality.

@pgruenler I figured out the issue with points and added drag functionality so it should me good to go now. Let me know if you have any more problems.

@yo1995 Can you let me know if you think there is a better way to use the task blocks and if you think that this sample needs a view model?

Can you let me know if you think there is a better way to use the task blocks

Will comment when I review.

if you think that this sample needs a view model

This is a tough call. 4 objects (scene, analysis overlay, formatter, measurement) typically drive me towards a data model, but since the sample is only 150 lines I'll leave it up to you.

Sounds good. I ended up adding a view model.


Azure Identity => ERROR in getToken() call for scopes []: Managed Identity authentication is not available.

I am using using azure-identity library with version 1.12.0.
I have followed all the steps to enabled workload-identity as mentioned in https://learn.microsoft.com/en-us/azure/aks/workload-identity-deploy-cluster.
Following is the snippet of code I am using -
DefaultAzureCredential managedIdentityCredentialUserAssigned = new DefaultAzureCredentialBuilder()
.managedIdentityClientId("bd947a20-baf1-4009-ab9a-c8aa361527a6").build();
    AccessToken accessToken = managedIdentityCredentialUserAssigned
            .getToken(new TokenRequestContext().setTenantId(tenantId)).block();
    String token = accessToken.getToken();
    logger.info("token : {}", token);

Here bd947a20-baf1-4009-ab9a-c8aa361527a6 is clientId corresponding to the managed Identity.
managedIdentityCredentialUserAssigned.getToken() is throwing following error -
2024-05-09 06:07:41.760 [main] [DEBUG] com.azure.core.implementation.ReflectionUtils - Attempting to use java.lang.invoke package to handle reflection.
2024-05-09 06:07:41.763 [main] [DEBUG] com.azure.core.implementation.ReflectionUtils - Successfully used java.lang.invoke package to handle reflection.
2024-05-09 06:07:41.771 [main] [DEBUG] com.azure.identity.EnvironmentCredential - Azure Identity => Found the following environment variables: AZURE_CLIENT_ID, AZURE_TENANT_ID
2024-05-09 06:07:41.772 [main] [DEBUG] com.azure.identity.EnvironmentCredential - Azure Identity => ERROR in EnvironmentCredential: Failed to create a ClientSecretCredential or ClientCertificateCredential. Missing required environment variable either AZURE_CLIENT_SECRET or AZURE_CLIENT_CERTIFICATE_PATH
2024-05-09 06:07:41.773 [main] [DEBUG] com.azure.identity.EnvironmentCredential - Azure Identity => ERROR in EnvironmentCredential: Failed to determine an authentication scheme based on the available environment variables. Please specify AZURE_TENANT_ID and AZURE_CLIENT_SECRET to authenticate through a ClientSecretCredential; AZURE_TENANT_ID and AZURE_CLIENT_CERTIFICATE_PATH to authenticate through a ClientCertificateCredential; or AZURE_USERNAME and AZURE_PASSWORD to authenticate through a UserPasswordCredential.
2024-05-09 06:07:41.909 [main] [DEBUG] com.azure.identity.ManagedIdentityCredential - Azure Identity => Found the following environment variables: AZURE_CLIENT_ID, AZURE_TENANT_ID
2024-05-09 06:07:41.910 [main] [DEBUG] com.azure.identity.SharedTokenCacheCredential - Azure Identity => Found the following environment variables: AZURE_CLIENT_ID, AZURE_TENANT_ID
[DEBUG] com.azure.identity.ManagedIdentityCredential - Azure Identity => ERROR in getToken() call for scopes []: Managed Identity authentication is not available.
2024-05-09 06:07:42.012 [main] [INFO] com.azure.identity.ChainedTokenCredential - Azure Identity => Attempted credential EnvironmentCredential is unavailable.
2024-05-09 06:07:42.038 [main] [DEBUG] com.azure.core.implementation.util.Providers - Using com.azure.core.http.netty.NettyAsyncHttpClientProvider as the default com.azure.core.http.HttpClientProvider.
2024-05-09 06:07:42.117 [main] [WARN] com.azure.core.http.netty.implementation.Utility - The following Netty dependencies have versions that do not match the versions specified in the azure-core-http-netty pom.xml file. This may result in unexpected behavior. If your application runs without issue this message can be ignored, otherwise please update the Netty dependencies to match the versions specified in the pom.xml file. Versions found in runtime: 'io.netty:netty-codec' version: 4.1.100.Final (expected: 4.1.101.Final)
2024-05-09 06:07:43.361 [ForkJoinPool.commonPool-worker-1] [INFO] com.azure.identity.ChainedTokenCredential - Azure Identity => Attempted credential WorkloadIdentityCredential is unavailable.
2024-05-09 06:07:43.370 [ForkJoinPool.commonPool-worker-1] [DEBUG] com.azure.identity.ManagedIdentityCredential - Azure Identity => ERROR in getToken() call for scopes []: Managed Identity authentication is not available.
2024-05-09 06:07:43.370 [ForkJoinPool.commonPool-worker-1] [INFO] com.azure.identity.ChainedTokenCredential - Azure Identity => Attempted credential ManagedIdentityCredential is unavailable.
2024-05-09 06:07:43.375 [ForkJoinPool.commonPool-worker-2] [DEBUG] com.azure.identity.implementation.IdentityClient - SharedTokenCacheCredential authentication unavailable. No accounts were found in the cache.

Changed the code like this -
TokenCredential managedIdentityCredential = (new ManagedIdentityCredentialBuilder()).clientId(clientId)
.build();
String accessToken = ((AccessToken) managedIdentityCredential.getToken((new TokenRequestContext())
.addScopes(new String[] { "https:///.default" })).block())
.getToken();
Still getting error like this -
2024-05-09 12:21:05.499 [ForkJoinPool.commonPool-worker-1] [ERROR] com.azure.identity.ManagedIdentityCredential - Azure Identity => ERROR in getToken() call for scopes [https:///.default]: Managed Identity authentication is not available.
Can you pls confirm if this is a bug in the SDK that needs to be fixed. Is there an alternate way to fetch AAD Token for workload identity?

At the moment, this is blocking me to implement workload identity.

Hello! Can you help me understand the scenario? Generally these credentials are used in the context of one of our service clients (such as KeyVaultClient.) Is that also failing, and you are simplifying the repro here? Can you try a scope like https://vault.azure.net or https://management.azure.com?

Hi,
I need to use the workload identity flow with Azure database for MySQL
single server. I don't need to use the vault as it is not a requirement for
us.
I have followed the steps as listed in this microsoft documentation
https://learn.microsoft.com/en-us/azure/aks/workload-identity-deploy-cluster
.
Following is the code I am using to fetch AAD token -
Map<String, String> env = System.getenv();
String tenantId = env.get("AZURE_TENANT_ID");
String clientId = env.get("AZURE_CLIENT_ID");
String tokenFile = env.get("AZURE_FEDERATED_TOKEN_FILE");
TokenCredential managedIdentityCredential = (new
ManagedIdentityCredentialBuilder()).clientId(clientId)
                .build();

String accessToken = ((AccessToken) managedIdentityCredential
                .getToken((new TokenRequestContext()).addScopes(new

String[] { "https://ssp-demo-db8.mysql.database.azure.com/.default" })).
block()).getToken();
But when I am running this code as K8S pod, I am getting this error -
2024-05-10 07:44:08.990 [main] [DEBUG]
com.azure.identity.ManagedIdentityCredential - Azure Identity => Found the
following environment variables: AZURE_CLIENT_ID, AZURE_TENANT_ID
2024-05-10 07:44:09.131 [main] [DEBUG]
com.azure.core.implementation.util.Providers - Using
com.azure.core.http.netty.NettyAsyncHttpClientProvider as the default
com.azure.core.http.HttpClientProvider.
2024-05-10 07:44:10.636 [ForkJoinPool.commonPool-worker-1] [ERROR]
com.azure.identity.ManagedIdentityCredential - Azure Identity => ERROR in
getToken() call for scopes [
https://ssp-demo-db8.mysql.database.azure.com/.default]: Managed Identity
authentication is not available.
CredentialUnavailableException: Managed Identity authentication is not
available.},
[com.azure.identity.implementation.IdentityClient.lambda$authenticateWithManagedIdentityConfidentialClient$25(IdentityClient.java:563),
reactor.core.publisher.Mono.lambda$onErrorMap$28(Mono.java:3783),
reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onError(FluxOnErrorResume.java:94),
reactor.core.publisher.MonoFlatMap$FlatMapMain.secondError(MonoFlatMap.java:241),
reactor.core.publisher.MonoFlatMap$FlatMapInner.onError(MonoFlatMap.java:315),
reactor.core.publisher.MonoCompletionStage$MonoCompletionStageSubscription.apply(MonoCompletionStage.java:119),
reactor.core.publisher.MonoCompletionStage$MonoCompletionStageSubscription.apply(MonoCompletionStage.java:71),
java.base/java.util.concurrent.CompletableFuture.uniHandle(CompletableFuture.java:934),
java.base/java.util.concurrent.CompletableFuture$UniHandle.tryFire(CompletableFuture.java:911),
java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510),
java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1773),
java.base/java.util.concurrent.CompletableFuture$AsyncSupply.exec(CompletableFuture.java:1760),
java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:373),
java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1182),
java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1655),
java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1622),
java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:165)]
Can you please help to correct or fix this issue as I have been blocked for
the last 2 weeks?
I have already opened an official case (Case<PHONE_NUMBER>001925) with
Microsoft for this issue and there has been no progress till now.
Regards,
Yashpal
On Fri, May 24, 2024 at 4:17 AM Bill Wert @.***> wrote:

Hello! Can you help me understand the scenario? Generally these
credentials are used in the context of one of our service clients (such as
KeyVaultClient.) Is that also failing, and you are simplifying the repro
here? Can you try a scope like https://vault.azure.net or
https://management.azure.com?
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-java/issues/40090#issuecomment-2128156664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATLYJAXECR7LDQSGOWGSS6DZDZWYBAVCNFSM6AAAAABHOHTNZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRYGE2TMNRWGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I had the same issue, with debugging I figured out that the workload identity did not have the appropriate rights for the requested scope.
The token exchange API (https://login.microsoftonline.com//oauth2/v2.0/token) does return the following JSON body:
{     "error": "invalid_grant",     "error_description": "AADSTS501051: Application '<censored app id>'(<censored app name>) is not assigned to a role for the application 'api://<censored api id>'(<censored api name>). Trace ID: <censored trace id> Correlation ID: <censored correlation id> Timestamp: <censored timestamp>",     "error_codes": [         501051     ],     "timestamp": "<censored timestamp>",     "trace_id": "<censored trace id>",     "correlation_id": "<censored correlation id>",     "error_uri": "https: //login.microsoftonline.com/error?code=501051" }
But the azure identity library (I am using 1.12.2) does not print any error message and the response body is swallowed because the status code returned is 400, and the HttpURLConnection class, which executes this call ignores the body for this response code.

I had the same issue, with debugging I figured out that the workload identity did not have the appropriate rights for the requested scope.
The token exchange API (https://login.microsoftonline.com/<tenantId>/oauth2/v2.0/token) does return the following JSON body:
{
    "error": "invalid_grant",
    "error_description": "AADSTS501051: Application '<censored app id>'(<censored app name>) is not assigned to a role for the application 'api://<censored api id>'(<censored api name>). Trace ID: <censored trace id> Correlation ID: <censored correlation id> Timestamp: <censored timestamp>",
    "error_codes": [
        501051
    ],
    "timestamp": "<censored timestamp>",
    "trace_id": "<censored trace id>",
    "correlation_id": "<censored correlation id>",
    "error_uri": "https: //login.microsoftonline.com/error?code=501051"
}

But the azure identity library (I am using 1.12.2) does not print any error message and the response body is swallowed because the status code returned is 400, and the HttpURLConnection class, which executes this call ignores the body for this response code.

Thank you for pointing this out.
We will ensure, the underlying exception flows through in the exception message.
@yashpalslathia21 this is likely a config/permissions issue.
Let us know, if any assistance on API end is still needed,

Hi @roumn. How did you manage to get the response body? As you say, the exception with the body doesn't bubble up and I couldn't find loggers that would allow me to get access to it. Thanks.

Hey @roumn - can you try setting AZURE_HTTP_LOG_DETAIL_LEVEL=BODY_AND_HEADERS and see if that gives you what you're looking for? ref doc.

Hi @billwert, thanks for the message. I tried the env var you mentioned to no luck. I also tried to configure it by code (snippet below).
I'm wondering if these mechanisms apply to Azure Identity and msal4j. I would have liked to enable wire logging for the HTTP requests but there doesn't seem to be such thing for HttpURLConnection.
Any idea of what else can I try to troubleshoot this?
Thanks.
HttpLogOptions options = new HttpLogOptions();
options.setLogLevel(HttpLogDetailLevel.BODY_AND_HEADERS);
options.setPrettyPrintBody(true);
builder.credential(new WorkloadIdentityCredentialBuilder().build());
builder.httpLogOptions(options);

Caused by: java.io.IOException: Server returned HTTP response code: 401 for URL: https://login.microsoftonline.com/<Redacted-AZURE_TENANT_ID>/oauth2/v2.0/token
	at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:2021)
	at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1610)
	at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:224)
	at com.azure.identity.implementation.IdentityClientBase.authenticateWithExchangeTokenHelper(IdentityClientBase.java:812)
	at com.azure.identity.implementation.IdentityClient.lambda$authenticateWithExchangeToken$59(IdentityClient.java:1040)
	at reactor.core.publisher.MonoCallable.call(MonoCallable.java:92)
	at reactor.core.publisher.MonoFlatMap$FlatMapMain.onNext(MonoFlatMap.java:139)
	at reactor.core.publisher.Operators$MonoSubscriber.complete(Operators.java:1839)
	at reactor.core.publisher.MonoCacheTime$CoordinatorSubscriber.signalCached(MonoCacheTime.java:337)
	at reactor.core.publisher.MonoCacheTime$CoordinatorSubscriber.onNext(MonoCacheTime.java:354)
	at reactor.core.publisher.Operators$MonoSubscriber.complete(Operators.java:1839)
	at reactor.core.publisher.MonoCallable.subscribe(MonoCallable.java:62)
	at reactor.core.publisher.MonoCacheTime.subscribeOrReturn(MonoCacheTime.java:143)
	at reactor.core.publisher.Mono.subscribe(Mono.java:4476)
	at reactor.core.publisher.Mono.subscribeWith(Mono.java:4606)
	at reactor.core.publisher.Mono.toFuture(Mono.java:5011)
	at com.azure.identity.implementation.IdentityClient.lambda$getWorkloadIdentityTokenProvider$67(IdentityClient.java:1403)
	at com.microsoft.aad.msal4j.AcquireTokenByAppProviderSupplier.fetchTokenUsingAppTokenProvider(AcquireTokenByAppProviderSupplier.java:75)

Ah. I didn't realize you were talking about WorkloadIdentityCredential here. Unfortunately, this path doesn't use the HTTP pipeline that our normal loggers use (which is what BODY_AND_HEADERS controls.) In the short term I think you're down to some sort of proxy-inspector (fiddler, wireshark, etc.)
We did recently (last week) add some further logging to this path for exceptions, but nothing that would dump the response body directly. We'll reconsider this specific area and see if we can improve this further. Tracking that in #41876

Hi @roumn. How did you manage to get the response body? As you say, the exception with the body doesn't bubble up and I couldn't find loggers that would allow me to get access to it. Thanks.

Hi @arikogan, as you mentioned I could also not find any loggers for this, so I had to step through the code with the debugger to figure this out.

The status to improve logging in this area, can be tracked at https://github.com/Azure/azure-sdk-for-java/issues/41876
Closing this issue.


Possible bug in "Get the user context" sample code

I think
if (loginHint) {
config.extraQueryParameter = "scope=openid+profile&login_hint=" + encodeURIComponent(loginHint);
...
should actually be
if (context.loginHint) {
config.extraQueryParameter = "scope=openid+profile&login_hint=" + encodeURIComponent(context.loginHint);
...
(note the addition of the "context." in two places)

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

ID: be314181-4800-4d02-cf62-fb276dc7cdd2
Version Independent ID: 259e41a3-0607-c3f8-f635-f6bce767ad2b
Content: Silent authentication and SSO - Teams
Content Source: msteams-platform/concepts/authentication/auth-silent-AAD.md
Product: msteams
GitHub Login: @o365devx
Microsoft Alias: o365devx

@HiltonGiesenow - These code snippets are copied from the Microsoft Teams Authentication Sample which has complete code. In documentation we have kept only relevant snippets for better understanding.
Similar issue.


No way to conveniently display custom labels using Select2ListView

There is a very convenient method when I use Select2QuerySetView (using get_result_label), why isn't there a similar thing in ListViews ? There is only a way to change the value, but I can't find any to change the displayed labels, please let me know if I missed something.

Did you figure this out? Would you like to make a pull request?

This is what I did:
`class ColorAutocompleteView(autocomplete.Select2ListView):
def get_result_with_label(self, choice):
    new_choice = (choice[0], format_html('
{}
',
                                           choice[0], choice[1]))
    return new_choice

def get_list(self):
    return [self.get_result_with_label(choice) for choice in TailwindColors.choices]`

This iw what I did:
`class ColorAutocompleteView(autocomplete.Select2ListView):
def get_result_with_label(self, choice):
    new_choice = (choice[0], format_html('
{}
',
                                           choice[0], choice[1]))
    return new_choice

def get_list(self):
    return [self.get_result_with_label(choice) for choice in TailwindColors.choices]`

This is what I did
class ColorAutocompleteView(autocomplete.Select2ListView):
def get_result_with_label(self, choice):
    new_choice = (choice[0],
                  format_html('
{}
',
                                           choice[0], choice[1]))
    return new_choice

def get_list(self):
    return [self.get_result_with_label(choice) for choice in TailwindColors.choices]


Indicator

Closes #53

Shot Type
Count
Status

Added
12
🟡

Deleted
0
❌

Regression
2
❗

🟡 storybook/components-indicator--default__[w1440px] (click to expand)

Image

🟡 storybook/components-indicator--default__[w1920px] (click to expand)

Image

🟡 storybook/components-indicator--default__[w390px] (click to expand)

Image

🟡 storybook/components-indicator--default__[w900px] (click to expand)

Image

🟡 storybook/components-indicator--right-side__[w1440px] (click to expand)

Image

🟡 storybook/components-indicator--right-side__[w1920px] (click to expand)

Image

🟡 storybook/components-indicator--right-side__[w390px] (click to expand)

Image

🟡 storybook/components-indicator--right-side__[w900px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w1440px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w1920px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w390px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w900px] (click to expand)

Image

❗ storybook/components-card--icon-header__[w1920px]:

Before
After

❗ storybook/components-card--item-card-avatar__[w390px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
6
❗

❗ custom/widgets-heart-rate--hint-opened__[w1440px] (click to expand)

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w1920px] (click to expand)

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w390px] (click to expand)

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w900px] (click to expand)

Before
After

❗ storybook/widgets-meal-nutrients--default__[w900px] (click to expand)

Before
After

❗ storybook/widgets-progress-info--default__[w390px] (click to expand)

Before
After

Shot Type
Count
Status

Added
6
🟡

Deleted
0
❌

Regression
3
❗

🟡 storybook/components-indicator--default__[w390px] (click to expand)

Image

🟡 storybook/components-indicator--right-side__[w390px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w1440px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w1920px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w390px] (click to expand)

Image

🟡 storybook/components-indicator--with-content__[w900px] (click to expand)

Image

❗ storybook/components-card--icon-header__[w1920px]:

Before
After

❗ storybook/components-card--item-card__[w1920px]:

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1920px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
6
❗

❗ custom/widgets-heart-rate--hint-opened__[w1440px] (click to expand)

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w1920px] (click to expand)

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w390px] (click to expand)

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w900px] (click to expand)

Before
After

❗ storybook/widgets-meal-nutrients--default__[w900px] (click to expand)

Before
After

❗ storybook/widgets-progress-info--default__[w390px] (click to expand)

Before
After

Shot Type
Count
Status

Added
2
🟡

Deleted
0
❌

Regression
12
❗

🟡 storybook/components-indicator--default__[w390px]:

Image

🟡 storybook/components-indicator--right-side__[w390px]:

Image

❗ storybook/components-card--item-card__[w1920px] (click to expand)

Before
After

❗ storybook/components-indicator--default__[w1440px] (click to expand)

Before
After

❗ storybook/components-indicator--default__[w1920px] (click to expand)

Before
After

❗ storybook/components-indicator--default__[w900px] (click to expand)

Before
After

❗ storybook/components-indicator--right-side__[w1440px] (click to expand)

Before
After

❗ storybook/components-indicator--right-side__[w1920px] (click to expand)

Before
After

❗ storybook/components-indicator--right-side__[w900px] (click to expand)

Before
After

❗ storybook/components-indicator--with-content__[w1440px] (click to expand)

Before
After

❗ storybook/components-indicator--with-content__[w1920px] (click to expand)

Before
After

❗ storybook/components-indicator--with-content__[w390px] (click to expand)

Before
After

❗ storybook/components-indicator--with-content__[w900px] (click to expand)

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1920px] (click to expand)

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
4
❗

❗ custom/widgets-heart-rate--hint-opened__[w1440px]:

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w390px]:

Before
After

❗ custom/widgets-heart-rate--hint-opened__[w900px]:

Before
After

❗ storybook/widgets-heart-rate--default__[w1920px]:

Before
After


Bump rabbitmq

This also fixes a unit test that probably never was passing before (which honestly has me confused as to why it was there)

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

@jordansissel just pushed up the changes we discussed!

LGTM


Add support for maven-compiler-plugin configuration

I thought for completeness sake we should also add support for the version config of maven-compiler-plugin.
<plugins>
    <plugin>
        <artifactId>maven-compiler-plugin</artifactId>
        <configuration>
            <source>1.8</source>
            <target>1.8</target>
        </configuration>
    </plugin>
</plugins>

Also added a test for this.
(I hope this is okay with you, I thought instead of asking you to implement this I can quickly do it myself)

Hello @tim-we, thanks for the commit (and sorry for the delay), could you also commit the javascript generated files?

Sorry, forgot about those. Done :)


Allow API keys to be set using environment variables

Some users, like those on Elastic Beanstalk, will package a configuration file into an immutable package designed to be pushed to multiple machines. In certain cases, this means that the same configuration file must be usable for multiple environments without modification.
In order to allow this, we want to allow API keys to be specified using environment variables. For example:
[[files]]
path = "/var/log/nginx/ngingx.log"
apiKey = "env:TIMBER_NGINX_API_KEY"

[[files]]
path = "/var/log/node/node.log"
apiKey = "env:TIMBER_NODE_API_KEY"

Supersedes #32

It may be helpful here, instead of using our own notation, to make the apiKey a template with text/template and allow a user to pass something like {{env \"TIMBER_NODE_API_KEY\"}}


[runtime] Add missing stubs

These were uncovered while testing Windows support.
cc @compnerd

I don’t think this is correct; rather, I think this is the issue discussed at https://forums.swift.org/t/windows-cross-module-protocol-conformance/9777/3. If you demangle the symbols using swift-demangle you’ll see that they’re protocol descriptors and type descriptors, which currently can only be statically linked in on Windows.

Oh yup, I must have missed this from my conversation with @compnerd. This is what I'm using locally to get the tests to compile, but we should find a solution that isn't just a workaround


lm studio support

Congratulations for this great Extension!!
Have you planned the possibility of using it with lm studio? because when we put “ip:1234/v1” we get the message “Ollama is running” but it does not load the models.
Thx !

Currently, only Ollama API  is supported. Ialready have plans to support other local providers soon :)


Symlink cargo build artifacts instead of copying them

Motivation
Avoid copying big cargo build artifacts from the nix store, when using cargoArtifacts.
This works by just symlinking everything in target/release/build and target/release/deps. and uses rsync to copy other files. AFAIK the big files are all contained in these folders and are content-addressed by hash, so this should be safe.
What I have noticed (even before this change), is that cargo tarpaulin generally seems to rebuild the cargo artifacts, ignoring previous artifacts, even with --skip-clean, this fails as it tries to regenerate already built artifacts and tries to write into the build folders.
This should probably be investigated, so that cargo tarpaulin reuses the previously built/cached dependencies.
I have for now just removed --skip-clean in cargo tarpaulin, as it doesn't seem to have the intended effect anyway?
Without that flag, all tests run through.
Btw. I have noticed the (now removed) comment, but I have not experienced any issues other than the cargo tarpaulin above.
Checklist

[ ] added tests to verify new behavior
[ ] added an example template or updated an existing one
[ ] updated docs/API.md (or general documentation) with changes
[ ] updated CHANGELOG.md

Have you seen #76 ?

I gave it a spin, but realized it can't help me as I'm running with:
installCargoArtifactsMode = "use-zstd";

for reasons explained in #76.
I turned it off, but then discovered that after building dependencies Nix does bunch of shrinking ... on lots of files, and then it causes building the workspace to rebuild all the dependencies again. :/
I'm not sure if master has some error, or my setup is triggering some issues (works OK with use-zstd).
So I can't tell how much does it improve things.

It seems like the problem is not there in v0.12.1, so I rebased your commit on top of v0.12.1:
9ca63fb40f4ff761a6cbb7d5c07f5fe4c98782b9 (HEAD -> symlink-artifacts, origin/symlink-artifacts) Symlink cargo build artifacts instead of copying them
445a3d222947632b5593112bb817850e8a9cf737 (tag: v0.12.1) Update CHANGELOG

and again I'm seeing rebuilds of dependencies. So either your change is causing it too, or I messed up something somewhere.

I tried again (to double check) just v0.12.1 and it doesn't have the problem.

I'm using fenix toolchain. I reported https://github.com/ipetkov/crane/issues/323 , maybe it will shed some light.

Hi @Philipp-M thank you for the PR! When I originally explored symlinking all artifacts (instead of copying them deeply) cargo would fail with some errors about trying to write to a read-only file system (i.e. the nix store), but I hadn't considered that this might not apply to all artifacts, so kudos to spotting that!

What I have noticed (even before this change), is that cargo tarpaulin generally seems to rebuild the cargo artifacts, ignoring previous artifacts

I don't use cargo-tarpaulin much these days, but I do remember it insisting on cleaning the target directory (to recompile all deps with the right compiler flags IIRC). If it is ignoring --skip-clean it deserves some investigation, but it could be an unrelated issue

Noting @dpc 's comment on #323 (please correct me if I'm misunderstanding or misrepresenting anything)

Edit: Just checked and it seems that v0.12.1 behaves correctly. It also does the shrinking but when buildWorkspace starts it only builds local crates as expected.

Sounds like these changes (as they stand currently) are leading to rebuilds. My first guess is that cargo may be getting confused by looking at artifacts whose timestamp is older than the sources (sources in the Nix store have a timestamp in 1970) and it is resulting in rebuilding them. I wonder if the artifacts should be hardlinked (not symlinked) so that they get their own timestamp instead of inheriting the previous one...

@Philipp-M I pushed a small fix to resolve the build errors. Looks like a lot of the tests are passing but some are still failing (perhaps we shouldn't link build script output directories since scripts might need to be re-run and they expect to be able to write files? Alternatively perhaps we can take a shortcut by linking any *.rlib files or similar heuristics rather than guessing which directories are for build scripts and which ones aren't)

Thanks.
Sorry haven't had time to investigate this further (since it's also a little bit hard to debug, as on my machines everything seems to be working). According to the tests only OSX has problems (though I guess this has to be tested thoroughly on different machines anyway I think). I guess to be sure, one has to deep dive into the source of how cargo etc. is deciding whether stuff has to be rebuilt on different machines. Leaving build script dirs out of the linking process, is probably not a bad idea:

When rebuilding a package, Cargo does not necessarily know if the build script needs to be run again. By default, it takes a conservative approach of always re-running the build script if any file within the package is changed
...
The rerun-if-changed instruction tells Cargo to re-run the build script if the file at the given path has changed. Currently, Cargo only uses the filesystem last-modified “mtime” timestamp to determine if the file has changed. It compares against an internal cached timestamp of when the build script last ran.
If the path points to a directory, it will scan the entire directory for any modifications.

@dpc, can you provide more info, what exactly is going wrong (like hardware/OS, the exact issue)?

Just noticed #344 (thanks for further investigating btw.), I guess I can close this for now?
One thing I noticed while checking the deps folder, is that I think .so files can be safely symlinked as well.

@Philipp-M It's up to you! I decided to pick the low hanging fruit so we could make incremental updates.
Feel free to explore the .so linking optimization in this PR or a fresh one as you prefer

I tried the current master with #334 and just so you're aware: in a larger project, just using zstd is still waaay better than not using, even with current optimizations.
In our our project zstd produces: 828M for deps, and 1.5G for workspace build. no-zstd&symlinks: 4.6G and 4.1G.

@dpc fwiw this PR's symlinking only happens when inheriting existing artifacts to perform a build, but is entirely unrelated with how the resulting artifacts are organized in /nix/store


Added Shoot Target Game

It solves issue #1533
Its working video,
https://user-images.githubusercontent.com/75881393/120095765-4cc7d180-c145-11eb-8bc6-8aa8e1ec3cf0.mp4

Please @Vishal-raj-1 @ridsuteri @991rajat @urvashi-code1255 review this PR.


acapella default voice changed to a german one.

Main project is thinking about to remove acapella, so this is not necessary anymore:
https://github.com/kalliope-project/kalliope/issues/265

Acapella TTS has been updated and back working on dev branch. Enjoy ;)

@gahujipo: can you please reopen the pr, as long as its supported by kalliope, the pr is useful.

thanks for pr


Request: a better documentation for replace

Can wee see a motivating use case for replace?
The description in the docs:
https://tzlaine.github.io/parser/doc/html/boost_parser__proposed_/tutorial/algorithms_and_views_that_use_parsers.html#boost_parser__proposed_.tutorial.algorithms_and_views_that_use_parsers._globalname_alt__boost__parser__replace___code__phrase_role__identifier__boost__phrase__phrase_role__special______phrase__phrase_role__identifier__parser__phrase__phrase_role__special______phrase__phrase_role__identifier__replace__phrase___code___globalname_
Doesn't give a hint why anyone would need this feature for.
The example with bp::lit("XYZ") isn't very convincing. It looks like replace for strings would be better.
Is there a real life use case for this that we could see?

You really want me to justify the utility of string replacement?

The docs need to give an example anyway. I think demonstrating a parser that is a bp::lit is not the best choice: there are simpler ways to replace one string literal with another.
Maybe instead an example that erases the card numbers:
auto card_number = bp::repeat(4)[bp::int_] >> bp::repeat(3)['-' >> bp::repeat(4)[bp::int_]];
auto card_number_wipe = bp::replace(card_number , "XXXX-XXXX-XXXX-XXXX");

Ok, I see.  Yeah, that's a much better example.  I'll change this.


remvoing the top 1em from budget-table

Github Issue (if applicable): #620
Explanation of Bugfix/Feature/Enhancement:
There was a top: 1em !important on the budget table for slim/slim-font styles.
I've removed it and now the top row is no longer cut off.
Recommended Release Notes:
Fixed a bug where the top row of your budget table was cut off when using the slim rows options.

Thanks!


Refactor AWS SDK Client Metrics Collection

Adds a parent abstract class to PrestoS3FileSystemMetricsCollector so that other SDK clients can share the metrics collector support.
Adds reporting for client retry pause time indicating how long the thread was asleep between request retries in the client itself.
Fixes the reporting client timings. Previously, when the client retried a request only the first request timings would be recorded in the stats. Now, all request timings are reported individually.
Fixes PrestoS3SelectClient stats reporting. Previously, stats were collected in a PrestoS3FileSystemStats instance that wasn't actually registered with JMX.

Test failure appears unrelated.

Thanks!


fix(calendar): added period range case handler

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 5 (20.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 84.504%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/calendar/src/components/period-slider/utils.ts
1
5
20.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
9364

Relevant Lines:
10252

💛  - Coveralls


[20508] TCP first message loss

Description

With the related PR, the tcp_negotiation_timeout Transport Descriptor will be used to wait until logical port negotiation is finished prior sending data.

@Mergifyio backport 2.13.x 2.12.x 2.10.x 2.6.x

Related implementation PR:

eProsima/Fast-DDS#4454

Contributor Checklist

[x] Commit messages follow the project guidelines.
N/A Code snippets related to the added documentation have been provided.
[x] Documentation tests pass locally.
[x] Applicable backports have been included in the description.

Reviewer Checklist

[x] The PR has a milestone assigned.
[x] The title and description correctly express the PR's purpose.
[ ] Check contributor checklist is correct.
[ ] CI passes without warnings or errors.

@richiprosima Please test docs

@richiprosima Please test this


Parallelize (maybe?) gps.WriteDepTree

gps.WriteDepTree() is the global entry point that actually creates a vendor/ tree. Right now, it operates entirely in serial, writing out dependency directories one at a time. I don't know exactly how much we'd be able to get out of parallelizing this, as most of the bottleneck here is likely to be I/O. However, it seems worth at least trying to run these operations concurrently; my naive expectation would be that 2x parallelization would at least be able to earn some speedups; the kernel ought to be able to schedule the vcs subprocesses sanely so that read/CPU work segments in one process alternate with writing in the other process, helping to ensure we're always saturating disk writes.
If someone could do an experimental implementation of this, then work up some benchmarks to validate the hypothesis, that'd be great.

Hey, is someone working on this, does it need to be claimed?

@tonto nobody's working on it, so it's all yours if you want it! 🎉 🎉

@sdboyer cool, I'll have a go then, fingers crossed :)

great! please don't hesitate to ask questions, whether here or in slack.

@sdboyer Without making a PR I pasted the code for WriteDepTree with some ExportProject error handling related questions in the comments...
func WriteDepTree(basedir string, l Lock, sm SourceManager, sv bool) error {
	if l == nil {
		return fmt.Errorf("must provide non-nil Lock to WriteDepTree")
	}

	err := os.MkdirAll(basedir, 0777)
	if err != nil {
		return err
	}

	wg := sync.WaitGroup{}
	// Related to Q1
	var e error

	for _, p := range l.Projects() {
		go func(p LockedProject) {
			wg.Add(1)
			defer wg.Done()

			to := filepath.FromSlash(filepath.Join(basedir, string(p.Ident().ProjectRoot)))

			err = sm.ExportProject(p.Ident(), p.Version(), to)
			if err != nil {
				// Q1 - Since any goroutine can error out, what do we do with err's
				// (e here is temporary just to catch any error)
				// Since the function previously returned immediately after an error and called `removeAll`
				// does this mean that we should stop all other goroutines
				// or just record an error like this (or use `pkg/errors` to wrap them) and wait out
				// the rest of the goroutines
				e = fmt.Errorf("error while exporting %s: %s", p.Ident().ProjectRoot, err)
				log.Println("project error: ", to, e)
				return
			}

			if sv {
				filepath.Walk(to, stripVendor)
			}
			// TODO(sdboyer) dump version metadata file
		}(p)
	}

	wg.Wait()

	// Related to Q1
	if e != nil {
		removeAll(basedir)
		return err
	}

	return nil
}

I know that this was just supposed to be a proof of concept bench mark and that maybe the question above is misplaced, so you don't have to answer it right now, but the real problem is that I can't really run the benchmark for WriteDepTree since I get an error from ExportProject:
--- FAIL: BenchmarkCreateVendorTree-4
        result_test.go:149: unexpected error after 29 iterations: /var/folders/5_/6g14f8cd4h5d0k43fsmrltgr0000gn/T/vsolvtest/export/github.com/sdboyer/testrepo/README already exists, no checkout
                /var/folders/5_/6g14f8cd4h5d0k43fsmrltgr0000gn/T/vsolvtest/export/github.com/sdboyer/testrepo/tmp2 already exists, no checkout
                : exit status 128

Probably complaining because of conflicting work between goroutines...
I see that the benchmark uses the built-in SourceMgr - does this imply that it is not thread-safe ?!
Sorry if I got things wrong, your feedback would be helpful.

@tonto no worries! like i said, questions are totally fine 😄
that you'd be getting an error like that is surprising - it would seem to suggest that there's a possible scenario in which WriteDepTree() is exiting before all of its subprocesses terminate. that definitely should not be possible.
more generally, though...yep! this is the kind of challenge you have here - a sync.WaitGroup may or may not be the best way of accomplishing this, as you need to communicate progress and potentially errors back from each individual goroutine.
i might instead explore something more like a dispatcher/controller goroutine plus a configurable number of worker goroutines.
does that make sense?

@sdboyer Yes, was thinking along the same lines concerning controller - worker groutines, so that's clear, thanks :)
Regarding the error I'm getting not really but I will investigate further

great!
also, a key note about the domain model here...
there's an assumption that projects cannot be logical descendants of one another - only siblings, or entirely unrelated. that is, we assume that somedomain.com/project and somdomain.com/project/subprojectcannot both exist asProjectRoots in a single Lock`. (they probably shouldn't exist globally either, but that's a different scope of problem). however, this assumption isn't really clearly enforced anywhere. it's probably impossible for it to arise normally, but we'll eventually need to defend against it.
i'm raising it now just to say that, for the purposes of this issue, you should ignore it. we'll work on it in a follow-up 😄

done now


When will python 3.9 be supported?

In the cvxpy documentation under Install, it is specified that it is only compatible with 3.5, 3.6, or 3.7. Is there a plan or timeline for making it compatible with 3.9?

conda-forge supports python 3.9 already;
conda config --add channels conda-forge
conda create --name my_env cvxpy python=3.9

@jackalack Python 3.9 should work without problems, the CI tests are already including it. I think it's just a matter of updating the docs, or do you get any specific error when installing with 3.9?

@phschiele and @h-vetinari thank you for your responses. @phschiele to answer your question, I was indeed asking due to the documentation specifying compatibility up to 3.7. I was unsure if there were known issues with 3.8 and 3.9

I changed the documentation.


Implement Email Service through contact page.

Currently the Send button in Contact page just logs the form content.

Do this after #69 is merged.

@gaurovgiri
Do you have anything planned for how to approach this?
Is using mailto fine?

@BISHALTWR I think the best for right now is to implement mailto. We can implement third party services for email but the issue with the client side application is that the secrets of the 3rd party app will be publicly visible. So yeah best thing to do right now is to implement mail to. Also could you check if we can send subject and body as well in the parameters.

Alright, i will look into it.


Updated downtime status panel

Issue: https://github.com/open-cluster-management/backlog/issues/16691
Signed-off-by: dislbenn<EMAIL_ADDRESS>

/retest

/approve
/lgtm

/approve
/lgtm


Pic 4199 - Add IsYouth to the CHER

It would be great to see an integration test for this to prove that isYouth gets added to the message which is put onto the topic
Maybe expanding on this?
https://github.com/ministryofjustice/court-hearing-event-receiver/blob/c3a83a7f025babe2c005b95b0da06cd883fde065/src/test/kotlin/uk/gov/justice/digital/hmpps/courthearingeventreceiver/controller/EventControllerIntTest.kt#L57


Remove unnecessary dependencies and align local setup and quickstart scripts

refactor: remove unnecessary dependencies

argocd
external-dns
redis
webhook
skupper and submariner
deploy-sample-applicationset

refactor: Align local-setup and quick-start deployment paths
Updates local-setup to use the same kustomizations as the quick start scripts. Installation and configuration of hub and spoke dependencies is now consistent across both deployment paths.
The only difference between quick start and local setup now is that metrics is enabled by default with local-setup, managedzones and dns providers are created using local environment files instead of environment variables and the MGC gateway controller is not running on the hub cluster.
Note: The skupper POC docs are listed in our docs site and would need to be removed before the next release if we merge this as is. Personally i see no advantage to leaving these here.

I think it is reasonable to remove the skupper and submariner docs at this point. We don't ever test them and really if you use gateway API and use an e/w technology that either support multi-cluster service or a regular service we will be able to work with it. Changes look fine.
Will run a local-setup to verify

local setup completed so happy to merge
/lgtm


Annotation in new version

In previous version were option to draw annotations, how to draw annotations in new version.

We now use the c3.js api to do these kind of things. In their API the annotations are called grids.
See this grafiddle for a working example.


chore: updated to focal beta

Updated Travis build dist to Focal beta, to resolve build errors.

Recently updated the Node version to 18.0.0 here - https://github.com/IBM/networking-go-sdk/pull/120

Follow issue for more details - https://github.com/IBM/networking-go-sdk/issues/121

:tada: This PR is included in version 0.37.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Check for empty vectors before trying to access a pointer to the first element in Octree<T, use_pairs, AL>::cull_convex().

Fixes #38457.

Thanks!

Cherry-picked for 3.2.2.


[examples] add shard tools and example on aishell

This is the shard tools and example on aishell to support the unify io, see https://github.com/wenet-e2e/wenet/pull/620

This is the aishell example of unified IO, see https://github.com/wenet-e2e/wenet/pull/627.


Support conversion between spark timestamp and ch datetime64

Changelog category (leave one):

Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Support datetime64  for clickhouse backend

Can one of the admins verify this patch?

test this please

I'm still working, no need for review for now, thanks!

I'm still working, no need for review for now, thanks!

It is done.

Does anyone have time to review this PR, thanks!

test this please

test this please


Building binaries for Arduino fails, fatal error: PDM.h

@tensorflow/micro
System information

Ubuntu 20.04 x86_64
TensorFlow installed from source:
Tensorflow version : 389405a77946410400ed410246e4cc7257802dde
Target platform: Arduino Nano 33

Documentation here under Build binaries heading suggest that a binary build can be initiated with a call as follows:
make -f tensorflow/lite/micro/tools/make/Makefile TARGET=<target> <project_name>_bin

Therefore a simple call as follows should work or need specific architecture to run to success.
make -f tensorflow/lite/micro/tools/make/Makefile TARGET=arduino micro_speech_bin

However, as the required PDM.h Arduino library is not in the source tree, when compiling micro_speech for Arduino, this fails with the error.
tensorflow/lite/micro/examples/micro_speech/arduino/audio_provider.cc:39:10: fatal error: PDM.h: No such file or directory
   39 | #include "PDM.h"
      |          ^~~~~~~

Supported targets and target architectures are not documented anywhere I have found.
Furthermore, the Makefile.inc in the arduino folder tensorflow/lite/micro/examples/micro_speech/arduino contains a reference to 'sparkfun_edge' instead of 'arduino' the following:
ifeq ($(TARGET),$(filter $(TARGET),arduino))

MICRO_SPEECH_SRCS += \
	tensorflow/lite/micro/examples/micro_speech/sparkfun_edge/audio_provider.cc \
	tensorflow/lite/micro/examples/micro_speech/sparkfun_edge/command_responder.cc

endif

@victorromeo Could you please let us know if the issue still persists ? Thanks!


Docs: Reactive streams clarify cold publishers

JAVA-3961

This is a difference to the pre reactor implementation. However, it's determined by how Mono's/Flux's work internally.

This is a difference to the pre reactor implementation. However, it's determined by how Mono's/Flux's work internally.

@rozza Thanks for doing the change!

@rozza Thanks for doing the change!


Connect switcher text to version switcher

The switcher text currently is not connected to the switcher and doesn't do anything. This fixes that.

Thanks, Im going to close this PR temporarily to set up a proper contributing workflow with a proper test deployment :+1:

Actually, this can be merged, misunderstood what it did :sweat_smile:


Requesting redraw in the sample C program

How do we handle
https://github.com/rust-windowing/winit/blob/master/examples/window.rs#L29
in the sample C program?
A suggestion: Make a copy of the winit_event_loop_run_return function with the following tweak:
#[no_mangle]
pub fn winit_event_loop_run_return_with_data(
    event_loop_ptr: *mut ValueBox<WinitEventLoop>,
    data: *mut c_void,
    callback: extern "C" fn(*mut c_void, *mut WinitEvent) -> WinitControlFlow,
) {
    if event_loop_ptr.is_null() {
        eprintln!("[winit_events_loop_run_return] _ptr_events_loop is null");
        return;
    }

    let mut event_processor = EventProcessor::new();

    event_loop_ptr.with_not_null(|event_loop| {
        event_loop.run_return(
            |event,
             _events_loop: &EventLoopWindowTarget<WinitUserEvent>,
             control_flow: &mut ControlFlow| {
                *control_flow = ControlFlow::Poll;
                let mut c_event: WinitEvent = Default::default();
                let processed = event_processor.process(event, &mut c_event);
                if processed {
                    let c_event_ptr = Box::into_raw(Box::new(c_event));
                    let c_control_flow = callback(data, c_event_ptr);
                    unsafe { Box::from_raw(c_event_ptr) };
                    match c_control_flow {
                        WinitControlFlow::Poll => *control_flow = ControlFlow::Poll,
                        WinitControlFlow::Wait => {
                            *control_flow = ControlFlow::WaitUntil(
                                time::Instant::now() + time::Duration::new(0, 50 * 1000000),
                            )
                        }
                        WinitControlFlow::Exit => *control_flow = ControlFlow::Exit,
                    }
                }
            },
        );
    });
}

We use the data parameter to pass pointer to a user defined struct:
typedef struct UserData {
  Window* window;
} UserData;

then in main()
  struct UserData udata;
  udata.window = window;

  winit_event_loop_run_return_with_data(event_loop, (void*)&udata, process_event);

in process_event
WinitControlFlow process_event(void* data, WinitEvent* event) {
    // rust owns `event`, no need to drop
    printf("Something\n");

    if (event -> event_type == WinitEventType_WindowEventCloseRequested) {
        return WinitControlFlow_Exit;
    }  else if (event->event_type == WinitEventType_MainEventsCleared) {
    winit_window_request_redraw(udata->window);
    }
    if (event -> event_type == WinitEventType_WindowEventResized) {
        printf("Window resized to: (%d x %d)\n", event -> window_resized.width, event -> window_resized.height);
    }

    if (event -> event_type == WinitEventType_WindowEventCursorMoved) {
        printf("Cursor moved to: (%f @ %f)\n", event -> cursor_moved.x, event -> cursor_moved.y);
    }

    if (event -> event_type == WinitEventType_WindowEventScaleFactorChanged) {
        printf("Scale factor changed to: %f. New physical size: (%d x %d)\n", event -> scale_factor.scale_factor, event -> scale_factor.width, event -> scale_factor.height);
    }

    return WinitControlFlow_Wait;
}

An issue with this is, running the binary takes 8% of the CPU compared to the Rust example, which I guess is not acceptable.

@syrel Any ideas here? :sweat_smile:

Nice idea! I have added winit_event_loop_run_return_data and winit_event_loop_run_data.
These new methods don't use the control flow as provided by the user and should not consume more CPU 👍

@syrel Does it make sense to ask for another function which is identical to winit_event_loop_return_data except, it doesn't take the data parameter?
Basically a version of winit_event_loop_run_return which doesn't constantly wake the event loop!
Use case being, some languages allow defining functions within functions, which would erase the need for defining a data.
While I can also just use winit_event_loop_return_data and use a pointer to a dummy variable to pass for data, (It works fine!) having a separate function would be a lot more elegant! :)
Here's probably what it'd look like:
#[no_mangle]
pub extern "C" fn winit_event_loop_run_return_alt(
    event_loop: *mut ValueBox<WinitEventLoop>,
    callback: extern "C" fn(*mut WinitEvent) -> WinitControlFlow,
) {
    event_loop
        .with_mut(|event_loop| {
            let mut event_processor = EventProcessor::new();
            event_loop.run_return(
                |event,
                 _events_loop: &EventLoopWindowTarget<WinitUserEvent>,
                 control_flow: &mut ControlFlow| {
                    control_flow.set_wait();

                    let mut c_event: WinitEvent = Default::default();
                    let processed = event_processor.process(event, &mut c_event);
                    if processed {
                        let c_event_ptr = Box::into_raw(Box::new(c_event));
                        let c_control_flow = callback(c_event_ptr);
                        unsafe { Box::from_raw(c_event_ptr) };
                        *control_flow = c_control_flow.into();
                    }
                },
            );
        })
        .log();
}


Port based reverse Proxy / Base Domain Config

Hey there,
we try to run Stirling as Docker behind a port based reverse proxy.
Like https://my.domain.com:28080. The app itself runs on http://localhost:8080.
With login enabled this does not seem to work as calling the app always redirects to https://my.domain.com/login. The expected behaviour should be redirecting to https://my.domain.com:28080/login.
Is there any way I missed to configure something like a SYSTEM_ROOTURI? The only thing that is close but not just it is the SYSTEM_ROOTURIPATH ENV var.
Appreciate any help :-)
Thanks and best regards,
Jan

Modify Nginx reverse proxy configuration:
location / {
proxy_pass http://localhost:8080;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header Host $host:28080; #Here's the key point: configure it this way to avoid losing the port.
}

Hey @chenjuefei,
thank you very much for your feedback and proposed solution.
 Customizability of the actual web server and reverse proxy configs is limited in this setup. I do not even know if Synology run nginx or Apache under the hood.
I added the Host header anyway to the response header config - not knowing what they make of it - and it did not change the behavior of the app. It still forwards to the login page without the port. Noteworthy: It forwards without any host. Not even the one it uses locally.
Using http://<IP>:8080 works fine and as expected, btw. Port remains set even with the redirects.
I opened this ticket as this is one of the first apps we came along (Docker or native web server) that does not offer an ENV var / config to solve this kind of issue where the public host is different from the internal host.

Same here 👍

image

Hi, I have the same with Nginx Proxy Manager and set Stirling as subfolder.
Regards.

location /pdf {
    proxy_pass http://***.***.*.***:8080;
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header Host $host; #Here's the key point: configure it this way to avoid losing the port.
}

Hello, follow your example of a subfolder reverse proxy and I have the page with this look, can you recommend any adjustments to make this PDF editor work? Thank you! I hid my local IP *
image

My experience is you need an additional / behind /pdf
location /pdf/ {
# YOUR OTHER CONFIGS
}

Me too.  Using /pdf or /pdf/ is the same behaviour.  Other apps (e.g. radarr) seem to have a URL base path defined in their configs or use 100% relative references.

having the same issue here on a synology nas! @JanThiel @caio1007 @indigodelta  not sure if you were able to find a solution guys?

@nunuxx Setup Stirling on a custom hostname on port 80 or 443 or without login does "work".

ohh how to deploy it "without login" sir? do i just lead the username and
password blank in the deployment script?
On Mon, May 27, 2024 at 12:53 PM Jan Thiel @.***> wrote:

@nunuxx https://github.com/nunuxx Setup Stirling on a custom hostname
on port 80 or 443 or without login does "work".
—
Reply to this email directly, view it on GitHub
https://github.com/Stirling-Tools/Stirling-PDF/issues/674#issuecomment-2132692521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG742YA3DRXB4PANCZYBNFLZELC3ZAVCNFSM6AAAAABBR45W5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSGY4TENJSGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@nunuxx It's all in the docs:
https://stirlingtools.com/docs/Advanced Configuration/How to add configurations/
SECURITY_ENABLELOGIN

thank you so much @JanThiel i confirm that setting SECURITY_ENABLELOGIN to FALSE then makes the custom hostname over port 443 work on a synology nas!


Dashboard - Gridview - Contextual Menu

Feature Description
As an editor I want an easy way to access all the options for managing stories
Feature Brief
Design

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

While hovering below the card, show the more icon to open the contextual menu
The menu should have the following (see design for separation):
3. Open in editor
3. Preview
3. See #768 for Rename, Duplicate, Delete
3. Create Template (Copies the current story into a new template, and navigates to the template section.)

QA Instructions

From Sam w.r.t. the position of the menu: If so we should spawn the menu on the top or bottom depending on the available space.


Ruby version

Comment:
Hello,
Installing the package installs ruby version  2.6. Is there a possiblity to also support ruby version to >=3 ?.
I initially thought that the downloaded ruby version is due to my OS RHEL-7.9 , so i tried to
conda install ruby

and got ruby 3.1.2 installed
ruby --version
ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [x86_64-linux]

Inside the .ci_support folder i see that there are only yml files for linux_64_ruby2.5.yaml and linux_64_ruby2.6.yaml,  could you please consider adding ruby 3 too?
Thanks in advance!

@awvwgk , i see that you are set as the maintainer of this package.  Could you please have a look. Thanks!

Might need to rebuild the package to get a newer Ruby version. But could also be an issue with the global pinning of Ruby in cf. There are unfortunately not many gems packaged on cf so this part of the ecosystem is not very active.

Could you please try rebuilding to see if the new ruby version is then available?. The ruby feedstock  has version 3.1.2 available since 10 Months.
If it doesn't work and it's related to Ruby version being pinned by conda forge itself, how could we communicate to the corresponding parties?
Thanks in advance!

Looks like there is an migration for Ruby 3.1 started already, but since you asked nicely, I'll manually migrate the feedstock (see #10).


Add Giza Engineering Institute ( GEI )

1 - Official website: https://gei.edu.eg
2 - IT course page: https://gei.edu.eg/index.php/ar/scientific-depts/communications-engineering

@ahmedasal14 First stage of verification passed, official domain is gei.edu.eg as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@ahmedasal14 Pull request merged. Thank you.


Updating Storage API Autosharding documentation to include that it doesn't work on Runner V2

Found when looking at a number of Runner V2 pipelines that stream to BQ with Storage Write API (including #28168)

R: @naireen

may check #27316 (which added documentation that STORAGE_WRITE_API supports autosharding) and change in the according places


Create PostgreSQL adapter

The July 2020 release of CAP does provide a New cds.Service Base Class. That should make it possible to implement the PostgreSQL adapter. The npm package for PostreSQL can be found here: pg.

A first version of cds-pg - PostgreSQL adapter for SAP CDS (CAP) is published. Please help to improve it and contribute.


No option to save model; saving in main directory

There seems to be no explicit option to save a model.
Instead of saving the model checkpoint files in the main directory, it would be better to do so inside a new directory named checkpoints or so

I'll try to fix this

Should I use argparse to create a separate option when running the code itself?

No need to add argparse now in my opinion. It can be done in the same way we're handling tensorboard. The algo class can take in 2 arguments: save_model=None (str) and save_interval. (int)
if save_model is None, then no need to save, else, save it in the location specified by it.
save_interval just takes in a number saying how many iterations/episodes between saves, as has already been done.
Does this make sense?

Yeah, will do.

Yes, we'd like to avoid argparse for now, especially at the core of the library since it does not make sense. Maybe we can have it examples.
On another note, we should shift the following code to examples as well:
if __name__ == "__main__":
    algo = Algo()
    algo.learn()
    etc

Closed via #63


Refactor npm scripts & add new Live

Replaced gulp with npm scripts. Added new live

Added serve on watch (I usually want watch only when serving)

Fixed 2 bugs:

Hours to seconds was not converting correctly. Dates in string are in format 12 AM/PM, so hours >12 were not converted correctly.  Replaced to hour * 3600 + minute *60
Schedule.json was being cached between requests in the browser. To avoid that I added a get parameter in the path with the current time in epoch.

Quick request:

Could we remove the notify node? It's filling my notification tray when I'm on watch. Or keep it, it's not a big deal. @dasilvacontin  thoughts on this?

\o/

@casassg I personally ignore my notification tray. It doesn't bother me, but I guess a 'completion ping' would be nicer, and notif only if error, with message.

@dasilvacontin I removed the notif for dist finally (it was breaking the built on the server). Kept the watch notif.


feat(ui): donation information card refresh

Brief description. What is this change?

Update Calendar Card Component
We're updating the information shown across the public calendar, donor dashboard, and admin calendar to be more consistent with the new designs.
Please see the figma for updated designs for cards.
Public Calendar

Admin Calendar

Donor Dashboard

Steps to test

Create some schedules from different donors
Verify that the three views look correct on desktop and mobile

Everything looks good to me! I think on my computer the mobile view looks a little strange - is it possible to reduce the margin?

should be fixed!


rename Datasets Server to dataset viewer

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[notification-hubs] Migrate @azure/arm-notificationhubs to ESM/vitest

Packages impacted by this PR

@azure/arm-notificationhubs

Issues associated with this PR
Describe the problem that is addressed by this PR
Migrates to ESM/vitest using automation.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.