(maint) Fixed incorrect heading used in release notes

Description Of Changes
This commit fixes that the release notes incorrectly used a h2 header instead of the expected h3.
Motivation and Context
To use the correct header.
Testing

[ ] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[x] Minor documentation fix (typos etc.).
[ ] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated

Related Issue
N/A

Thanks for getting that fixed up @AdmiringWorm


Use regular URL to generated documentation for converters/json2edn.py

Use regular URL (written in Markdown) to generated documentation for converters/json2edn.py

Fixed in https://github.com/RedHatInsights/insights-results-aggregator-utils/pull/256


fix(actions.copy): disallow copying parent folder into dir

Closes #173
@steve-numeus could you please verify this fixes the issue on your end?

Works, thank you!


QR Code Connection not working

Hello,
I'm trying to connect to the HomeSpan via the QR Code, but I am having some issues:
I'm using a ESP-32 from AzDelivery.
Arduino ESP32 Board Library: ESP32 v1.0.6
HomeSpan: v1.2.1
I upload any example sketch using default settings except Core Debug Level set to "Verbose".
Set a Code Via S Command (example: S 01201012)
Copy the Payload and create a QR Code from it.
Scanning the QR Code with the Home App opens the accessory adding page. When confirming and trying to add it times out after a while. There are no log messages from the ESP32 attempting a Pairing process.

Can someone help?

For Clarification: I am assuming via the QR code, the iPhone will send the ESP32 the Wifi Credentials and then connect to it. Is that not the case?
When adding wifi credentials, the QR code works. However, that adds a new step to the pairing process that i want to avoid.

Thanks for the clarification.  Unfortunately, the QR code can only be used for pairing, not for transmitting WiFi credentials, so the two-step process is required.  There are mechanisms to to use a QR code for both, but this is only available in the commercial MiFi implementation of HomeKit, not the HAP-R2 non-commercial version used by HomeSpan.

I looked it up on the official esp-homekit-sdk and found the same result - makes me sad this is only avaiable for the MiFi programm.
The Strange Wifi Erroros persist, but I will ignore them for now.


Use ttnn::transpose instead of runtime decomposing transpose

Previously, transpose was decomposed in runtime to permute. This PR transfers directly to ttnn::transpose.
I ran transpose tests on forge with this change and everything seems fine.
@vladimirjovanovicTT do you have some insight for why you added this decomposition? Was that a workaround due to some transpose case failing or transpose wan't implemented (less likely)?

Af first, ttnn::transpose wasn't avaiable, then after a metal uplift it became available, but simple use-cases didn't work.
I'm happy to hear that it works well now!


[FLINK-9804][state] KeyedStateBackend.getKeys() does not work on RocksDB MapState

What is the purpose of the change
This PR fixes the bug that the KeyedStateBackend.getKeys() does not work on RocksDB MapState.
Brief change log

Change RocksDBKeyedStateBackend#RocksIteratorForKeysWrapper() to let it support get keys for RocksDB MapState.

Verifying this change

Added StateBackendTestBase#testMapStateGetKeys() to guard the changes

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

No

CC @aljoscha

@aljoscha Thanks for your quick review, will address your comments while merging.


Battery level in ev3dev

Dear friends,
here I am doing a project on ev3 in which I had flashed ev3dev. Now I would like to  monitor the battery level of my brick any one can please help me to face this situation

I am not sure if this bash script is current due to the changing /sys/class/leds/ file names.
But uoy could take a look at checkpower it uses blinkleds that probably has older leds filenames. It depends on the kernel version. But that is easy to fix.

Thank you,
But my requirement is to know what is the current value of the battery level in the "ev3 rechargeable battery" which is come along with ev3kit.

I see that the script is no longer up to date and not yet compatible with the strech image I run.
But I do run an EV3 with rechargable battery and I do use an updated version of checkpower.
But the basic of it is, switch to the battery directory:
cd /sys/devices/platform/battery/power_supply/lego-ev3-battery

Maybe this exact directory name is not available on your level of ev3dev.
You did not report at what level of ev3dev you run.
But it should be easy to find that out. E. g. using a find command like
find /sys/devices/platform/ -name voltage_now

In the lego-ev3-battery folder is a file voltage_now indicating the current delivered by the battery.
At the moment my EV3 reports with
cat voltage_now

8443733 which means 8.443733 Volt (I think it's in microVolt)
Would not be too dufficult for some savvy python programmer to use that info.
What made you conclude that checkpower script is not fit for the ev3 rechargable battery?

I had gone through the root file but in first root I cant go further on battery at platform itself it shows an error that
robot@ev3dev:/sys/devices/platform$ cd battery /
-bash: cd: battery: No such file or directory

As suggested above, you may try to find the voltage_now field using a find command like:
sudo find /sys/devices/platform/ -name voltage_now

At the moment I do nao have access to my EV3 brick due to bad sd-card.

Thank you @BertLindeman  Here I can able to get the battery voltage at /sys/devices/platform/legoev3-battery/power_supply/legoev3-battery$ and now I would like to display the value of the battery level from the above root folder.... could you please help me..

Sure ;-)
The fields as they are now on my ev3-brick running ev3dev at a stretch image:
ls -l /sys/devices/platform/battery/power_supply/lego-ev3-battery/
-r--r--r-- 1 root root 4096 Sep  4 11:35 current_now
lrwxrwxrwx 1 root root    0 Sep  4 11:35 device -> ../../../battery
drwxr-xr-x 2 root root    0 Sep  4 11:35 power
-r--r--r-- 1 root root 4096 Sep  4 11:35 scope
lrwxrwxrwx 1 root root    0 Sep  4 11:35 subsystem -> ../../../../../class/power_supply
-r--r--r-- 1 root root 4096 Sep  4 11:35 technology
-r--r--r-- 1 root root 4096 Sep  4 11:35 type
-rw-r--r-- 1 root root 4096 Sep  4 11:35 uevent
-r--r--r-- 1 root root 4096 Sep  4 11:35 voltage_max_design
-r--r--r-- 1 root root 4096 Sep  4 11:35 voltage_min_design
-r--r--r-- 1 root root 4096 Sep  4 11:35 voltage_now

The field you want is voltage_now.
# cat voltage_now
8474733

As you see you need to divide by a million.
I use as an absolute minimum 6200000 so 6.2 Volt.
If that level is reached the brick will poweroff quite quickly.

yeah thanks here I can able to read the voltage by doing
**cat voltage_now **
$ 7045800
but what I need is, the code to print the value of the battery voltage. which I get in those root folder to screen. but I cant able to get the value from the root folder.

Sorry, I do not use the EV3 screen, too smal for me.
So I can not help you with printing a value to the screen.
Maybe the examples on this web page can help
You may look at the example "Display Text" on that page.

yeah, But my exact requirement is the python code to get the value from the root folder continuously
how to access the value of  root folder


"Jump to the function exit" in lcov report with multiline conditional

This test module contains a very simple multi-line controlling expression for an if statement.
def fun(x):
    if (
        x
    ):
        print("got here")

For some reason, the way the if statement is formatted causes the arc description engine to fail to pick up the name of the function.  In an LCOV report we get these branch records for line 2:
BRDA:2,0,jump to line 5,-
BRDA:2,0,jump to the function exit,-

The smallest change that makes the problem go away is to move the token x to line 2 without changing anything else, i.e.
def fun(x):
    if (x

    ):
        print("got here")

Then the branch records correctly identify the function:
BRDA:2,0,jump to line 5,-
BRDA:2,0,return from function 'fun',-

Probably related to #1874.

This is fixed in commit c85eaba206e1bf98302e0997e32c079e2c231f4b.

This is now released as part of coverage 7.6.10.


Error when selecting date column in dBRunner

Problem
The following error occurred.
strconv.Atoi: parsing “2022-08-30 15:07:26”: invalid syntax

Verified DB was MySQL

refs https://github.com/k1LoW/runn/pull/128#issuecomment-1232302966


CORS response headers not applied when (auth) Filter rejects

Describe the bug
If you set up CORS correctly with ambassador (proper methods, origins, headers), cross-origin requests work fine. Once you add a Filter for authentication to the mix (e.g. OAuth or JWT) you will run into CORS problems quickly, since the CORS response headers are not applied to unauthenticated requests.

From a browser you can't send credentials along with OPTIONS, so you'll get a CORS error there, blocking e.g. DELETEs. This can be circumvented as in #2962.
If you're authenticated and you applied the OPTIONS workaround, it works fine. However, if authentication expires, you will run into the CORS problem for all requests: the browser will display all requests as CORS errors instead of giving the web developer access to the appropriate error code/message (e.g. a 401 Unauthorized), since CORS response headers are missing. (same behavior as #3051, though that's for an external Auth Service).

To Reproduce
Steps to reproduce the behavior:

Set up an ambassador Module with cors definitions
Set up a Filter (tried OAuth2 and JWT) and FilterPolicy for some endpoint
Call the endpoint without being authenticated from an allowed origin
CORS response headers won't be present

Expected behavior
CORS headers should be applied to unauthenticated / rejected requests too, so one can act upon the authentication error messages.
Versions (please complete the following information):

Ambassador: 1.11.1 (https://artifacthub.io/packages/helm/datawire/ambassador/6.5.18)
Kubernetes environment: Azure Kubernetes Service, kubernetes 1.19.3

Additional context
Add any other context about the problem here.

One hack for the JWT Filter is to use the errorResponse to inject the appropriate cors headers:
e.g. something like:
    errorResponse:
      headers:
      - name: "Access-Control-Allow-Origin"
        value: "{{ .httpRequestHeader.Get \"Origin\" }}"
      - name: "Access-Control-Allow-Credentials"
        value: "true"

but then cors settings are in different spots. so less than ideal :confused:

Looked a bit deeper into this, seems there are quite a few related issues:
#3051
#2030
#2962
I had a look at my econf.json and traced it back to ir.json (thanks to https://www.getambassador.io/docs/latest/topics/running/debugging/). Think the problem is that the auth filter comes in front of the cors filter; did some experiments in envoy (cause it's easier just to supply that config) and if I swap them it works as expected: i.e. OPTIONS requests are unauthenticated, all others are, and CORS response headers are applied in all cases. So the #2962 workaround would also no longer be necessary.
Code in ambassador that does this:
https://github.com/datawire/ambassador/blob/9775cb284e51732bc879d216351d30c3f74db88b/python/ambassador/ir/ir.py#L267-L276
(not sure if just swapping cors and auth would fix it; have no working ambassador dev setup, don't know the code base well enough and can live with my JWT Filter workaround I posted above for now)

...(same behavior as #3051, though that's for an external Auth Service)

Hi, author of the above issue here. I agree, I titled my issue the way I did because I couldn't speak to non-external auth service behavior. But seeing your write up gives me confidence that we should advocate for the CORS processing to be applied even if the auth fails (in both the external auth service and the conventional filter cases).
Invalid credentials on an otherwise CORS-compliant request should not result in a CORS error in the browser app.


Key showing up in weird format after readding devices to Tuya

As the device got reset after power outage, I had to re-add the devices to the Tuya App. After re-adding the devices, I ran tinytuya wizard which shows up keys in weird format for these recently added devices.
{
        "name": "Light bulb 1",
        "id": "xxxxxxxxxxxxxxxxxxxx",
        "key": "bYAp`PR-O9qNwLV",
        "mac": "xx:xx:xx:xx:xx:xx",
        "uuid": "xxxxxxxxxxxxxxxxxxxx",
        "sn": "xxxxxxxxxxxxxxxxxxxx",
        "category": "dj",
        "product_name": "Light Bulb",
        "product_id": "xxxxxxxxxxxxxxxxxxxx",
        "biz_type": 0,
        "model": "",
        "sub": false
        .......
}

This key value seems encoded? It's not working with controlling devices on local tuya lan via Homebridge or HomeAssistant.
Has anyone faced similar issue?

They're not encoded, they just started including symbols.  #341 .  Not sure why it isn't working for you, perhaps one of the symbols is getting interpreted as an escape character?

Yes, I think the issue was in the paste of the keys interpretted as different character. As the format of the key looked very different from the previous one, I was convinced that's the issue.
Thanks for your quick response, it worked now!


fix typecheck

Codecov Report

Merging #284 (30c1eee) into main (5f00d10) will increase coverage by 0.13%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #284      +/-   ##
==========================================
+ Coverage   40.04%   40.17%   +0.13%
==========================================
  Files         134      134
  Lines       20797    20728      -69
  Branches     1203     1201       -2
==========================================
  Hits         8328     8328
+ Misses      12469    12400      -69

Impacted Files
Coverage Δ

packages/typecheck/src/diagnostics.ts
0.00% <0.00%> (ø)

...languageservice/src/features/DiagnosticsService.ts
0.00% <0.00%> (ø)

...ageservice/src/services/TypescriptPluginService.ts
0.00% <ø> (ø)

Continue to review full report at Codecov.

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


Vue 3 Compatibility

Last question about compatibility: Feb 4, 2021
Last commit: Apr 22, 2021
Wrote mail: Dec 01, 2022

No response

I guess this repo is dead (locked to vue 2)
This is not a promotional issue
Since i really could need this for Vue 3, i created my own version of it.
Simplified, smaller, no multiple instances, faster
If anyone wants to give it a shot:
VueJS Tour
Edit:
I'll transfer some issues from here too, so the development of this great package can go on.

Hey @GlobalHive!
Thanks for your message!
The project is not dead but we indeed don't have the time and ressources to update it at a proper pace anymore... so thank you for making vue-tour continue to live for Vue 3.
That being said, if you'd like, we could bring back your project to this repository, in a new branch and publish as a new version<EMAIL_ADDRESS>for exemple), so that you can benefit from the popularity of this package?
We could make you maintainer and just keep the approves to merge on master.
Tell us what you think of it!

Thanks for the reply @mmorainville
Well i don't know if you checked my version but i did a whole lot of things different
since i didn't want to copy it 1:1
What means i have no step component, everything is kept inside the VTour component, except styles.
Styles are overridable and scss based.
The next update is going to introduce the VTour options prop, which will hold all "static" options for popper and jump.js
What means: If we combine them, then there are 2, from ground up, complete different packages
What could bring some issues and confusion for the users.

https://www.npmjs.com/package/vue3-tour

Does the vue3-tour package can be used in nuxt 3 ? I tried installing it but not able to get the reference to $tours in nuxt 3 like nuxt 2?
Please look into this issue for reference:
https://github.com/pulsardev/vue-tour/issues/265

@VenkateshDotdash my version of this VueJsTour
has a Nuxt 3 version provided by BayBreezy


LUD-17: Protocol schemes and raw (non bech32-encoded) URLs

Add support for LUD-17.

Need to declare support for URL schemes within Xcode (Info.plist)

Done in a703a158
It will be available in the next (major) app release: v1.6.0


PMM-7861 Create tests for adding Azure MySQL, PostgreSQL

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests/detail/pmm2-ui-tests/8372/tests

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests/detail/pmm2-ui-tests/8570/pipeline

@yurkovychv I am waiting for your re-review.


Don’t throw an exception for missing assets in prod

Instead we’re returning the URL unaltered. We don’t want to bring down the entire site because a single image is missing. Instead the browser should resolve a missing asset path to a 404, if not handled otherwise.
The debug behaviour is unaltered, as in it throws a 404 in the EmbedController whenever it can't resolve an asset.

The throwing in the EmbedController is too late: it will just generate a 404, but leave the actual page untouched – even in debug mode.
That is not desired, as we want to fail hard and early if static assets are missing in debug.
So you need to rethrow the exception in debug:
catch (AssetsException $e)
{
    // fail hard in debug
    if ($this->debug)
    {
        throw $e;
    }

    // Instead of throwing an exception since we can't resolve the asset, we're returning the
    // asset path un-altered so the browser will resolve it to a 404
    return $assetPath;
}

This is a good idea. I've went ahead and implemented your feedback as presented.

Thank you 🎈


📚 - PdfViewer downloads file instead of showing.

What is the affected URL?
https://taiga-ui.dev/components/pdf-viewer
Description
Just to press any button that must show PDF.
Expected: PdfViewver with pdf in it.
Actual: pdf-file download starts.
Firefox 101, no addons. Windows 11 21H2. Google Chrome - no issue.
Screen Record
Which browsers have you used?

[ ] Chrome
[X] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

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

I just found why it is - default action in Firefox was set to download for PDF files. If its can't be handled or alerted I think that it issue can be closed.

Please note that I only indicated Windows. But now I checked on macOS, and I was able to reproduce this problem there.
Also note my first comment.
Reproduce on macOS 12.3, Firefox 101 :

There doesn't seem to be a robust way to detect this setting and since the PDF gets downloaded user is not limited in their ability to view the content. As per the first comment, I think we can close it.


Prepping JOSS submission

More notes on this PR after the tests finish running...

Hi @kvyh, @StuartLittlefair, @bsipocz, @eteq, @jberlanga, @adrn, @wtgee, @stephtdouglas, @aniketp41, @kelle, @larrybradley, @astrofrog, & @ejeschke!
After chatting with @eteq about it at Astro Hack Week, I'm prepping a submission to the Journal of Open Source Software on astroplan so our hard work can be citable. I'm CCing everyone who has contributed to astroplan here, who I intend to list as co-authors.
Please let me know if:

you would not like to be included in the author list (otherwise I will assume you are happy to be included)
I got your affiliation and/or ORCID incorrect – check the list here
you have suggestions for revisions to the short software summary

I'm hoping to submit this week, but welcome revisions and comments at any time. Thanks everyone!

Hi @bmorris3, thanks for the include. Can you update my affiliation to "Macquarie University".
I think the software summary looks good. My only suggestion would be a slight change: "...altitude-azimuth coordinates for targets from observatories anywhere on Earth." Also, maybe just "locations" instead of "observatories". But I don't think either of these changes are critical.

Thanks for including me, but I don't think my contribution to astroplan warrants co-authorship.

Ok @kelle, so long as you know that you're still appreciated! 😀

@bmorris3 - Can JOSS deal with multi affiliations?

Looks good to me! May have time to add comments about the writeup, but don't wait for me...

@bsipocz I'd be happy to find out; what other affiliations should I give you?
@adrn Thanks!

@bmorris3, thanks for doing this, it's a great idea.  After seeing all the orcid references I had to find out what it was, and having done so, acquired one myself: 0000-0001-9586-4482.  I'm still filling in the info, but if you haven't yet submitted, would you mind adding mine?  Thanks!

@ejeschke Of course, you got it.

One question: is there anything in the literature we can cite that discusses Subaru's scheduler? (@ejeschke probably knows?) Might be good to include that as a reference if so, given that it served as inspiration for some of the scheduler stuff.

Hmm, well it's not really a good source for a description of the scheduling (it was 10 pages, but they made me cut it to four because apparently poster presentations only get 4-page papers in the proceedings), but if you are looking for references for the JOSS paper you might cite the most recent paper I just added to my orcid.  Look for the reference for "Queue Mode Software for Subaru Telescope".  One niggle is that it was for ADASS 2016 in Oct and I haven't seen any announcement about the proceedings publication yet, which for ADASS usually follows in the next year (I think it must be running late this year, but "should be" Vol 505, ADASS XXVI).

@ejeschke Thanks, I've added that reference.

Thanks all. Submitted!


Update SP 84 from Warn to Info

CARB approved modification of SP 84 entries as shown in the attached image and table A-1 text.
These changes are not perceived to be material change to the level of effort for Table A-1 changes given in #820, #818, #982, #970, #971, #859, #515

J1939-84-Table-A-1-MY24 15Sept2021.docx

Check SP 84 warning messages provided by the next release

Release 3.1.5 has an updated table A-1.

Completed with table A1 changes.


Disallow the combination of a user-defined schema and include/exclude keys

Description
Disallows adding an S3 sink with an Avro or Parquet codec which uses both a user-defined schema and include/exclude keys. Auto-schema may still use include/exclude keys.
Issues Resolved
Resolves #3253.
Check List

[x] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks good overall but there is some missing test coverage somewhere in data-prepper-api
Execution failed for task ':data-prepper-api:jacocoTestCoverageVerification'.
> Rule violated for bundle data-prepper-api: instructions covered ratio is 0.9, but expected minimum is 1.0


Journal Link Title Url missing in link posts.

Description of bug
Posting a link to a site in the journal posts content about the site, however the link to the site is not correct when you click on it, and the link in journal does not work.
Steps to reproduce
List the steps to reproduce the behavior:

Go to journal
post a link to a website such as www.github.com
Hover over the title (or image) and inspect the link.
link created for the title (and image) is blank

Current result
The link is not being stored in the database.  Link is created but  no URL set in db to produce a link for a title along with the image.
Expected result
Website link posted in the journal is stored in the database to allow the journal to create the link to the website link posted.
Screenshots
If applicable, provide screenshots to help explain your problem.

Additional Context
The URL is not being stored in the database.  The database is storing everything but the URL.  If posting with http://url.com or https://url.com or www.url.com will create a URL dataItem entry in the database.  There will be a check that will test and set the http:// or https:// added to a www only link urls provided when detected.  If only a www it will not be treated properly in DNN so it needs to add an http:// at least to the link for DNN to allow it and it to pass checks.  I believe everything gets the the prefix stripped (http https) and then it gets added again later as it gets passed around.   I believe somewhere along the way DNN sees any Urls as not urls by the time it gets to the test and so it stores an emty string instead.

I can reproduce in 9.4.2

The images are not working as well.  I may have found a solution but I need to figure out how to test faster so I can see if it indeed is a solution in the short days we live in...  Both links are stripped for image source and URL link.  I also have another issue relating to this area I will try to knock em both out while looking here.  I want to fix an issue for the DNN Events calendar.  https://github.com/DNNCommunity/DNN.Events/issues/40
This one has been an issue forever, i feel like just stripping the HTML or figuring out why this is different from other posts.   So I am trying to work on both of these concerns now along with a few others for the events modules hoping I can help out on it.  DNN Events needs some love too :)

An update here so this does not go stale:
The journal is not saving the URL: in the database.  I have read the module source code a few times and believe that I am getting warmer as to where this event happens.  I think I need a better testing environment setup to really tackle things like this one.
Some odd behavior when you manually set the URL: part that is missing in the databse journal item data that I noticed:
If you have a url like www.therestoftheurl.com instead of one with http/https in front of it, the journal tries to load a page from within the site www.dnnsite.com/www.therestoftheurl.com but if you have http:// or https:// added it will link to domains off the website.  I would prefer not to have either http or https and load a new browser tab with just the domain and let the web server that handles the domain put http or https on the url as need if possible?!
Maybe this might help someone come up with a solution as well.  Thank you for any help and support here.

@thabaum I have not dug into the code myself, but a guiding principle across DNN (and most other web apps) is that when storing URL field data, if it is not prefixed by https://, http:// or //, then it is not an absolute URL.  It is instead assumed to be a relative URL.  Therefore, the behavior being reported is actually what I would expect.
As it relates to the URL's "preview image" that is usually provided via a combination of page data and metadata:

Title
Description
og:title
og:description
og:url
og:image
favicon

Again, I am not sure what this code is reliant on in DNN, but it is most likely one or more of the above.  The og: prefixed metadata is based on OpenGraph.
That said, is this really a bug?

The protocol should be part of an url, an url is in fact not even for web, it could even be ftp://somedomain.com/somefolder so yeah, the protocol is definitely a needed part of any url.

@valadas ultimately yes - of course.  😄
I'm just saying that DNN most likely supports relative and absolute URLs.  Whether or not it should in this context is probably where the focus needs to be.  I would think in this context it should only accommodate/support absolute URLs.

Posting a "link" in the journal works when you just type "www.domain.com" and so DNN journal does not know HTTP or HTTPS but still knows it is a link and then it strips things // and prior to the www I believe I seen in the code.  There are a few areas that are of interest I will make references here soon maybe it can help shine light here.
DNN Journal would have to add an HTTP:// or HTTPS:// without knowing which to put I am guessing the HTTP:// would be default.  I was trying to find where it adds this, i found some code that strips it and adds again.  When it grabs images and description of the page it has the URL in memory so need to know why that URL is not getting set in the Database.

@david-poindexter
The bug here is the link does not work, it is blank when you post a link in a journal post.  Image and description of the website works but if you click on the link created for the journal post it goes no where.
To reproduce post a link in the journal and try to click on the url that is created for a link to the website that is entered.  The href="" which in the database it is not being set.   I can manually add the link that is missing in the database and everything works for that post.  This is when I found it the link must have HTTP:// or HTTPS:// to make the link work in the journal because when I set it to just have a domain name without those it would go to a local page on the website creating a page not found error.
This is just a clue to what needs to happen prior to setting the URL in the database is there needs to be HTTP:// added to the front of the website link for DNN Journal to allow it to work AND it is not getting set at all either way so there are two things going on here that must happen for it to work right.

the www.domain.com link needs be http://www.domain.com to work
the domain after it has been properly created in a variable needs to add that to the database when created the post inside the journal module on a dnn page.

Some reason the url is not being posted to the database in any way, it is blank so when you click on the link created in the journal it goes no where.
This is a bug and needs some love.

Keeping this from going stale, I am going to give it a go here soon pick up where I left off in the next couple weeks.   This, social groups and DNN events not stripping the HTML are issues related I will be hoping to address for some version 9 fixes.  Thank you.

Issue looked a little confusing. I updated the issue along with adding some additional context with things I believe may be happening here to create the issue.  If you post a link in the journal the Url is not being saved to the data items in the database.  So then the url is blank when the journal creates post entry to be viewed.  If you add an address with Http:// or Https:// in the database manually the address appears and will work right.
Possibility is the https:// and http;// get checked and if they have anything with a // it is removed.  But I don't think http:// or https:// is ever added back again to the link data and then when it needs to get saved it gets checked to see if it has // but it has been removed when it was cleaned of it therefore a url is never stored it the database.   A www does not have this so it fails as well getting saved.  If I remember right this is what I believe is the current issue best explained I can.
I am going to hopefully figure a way to step through this again and understand what is missing here.  I will be giving this all another go here in the next week.

OK just so this is not stale... my situation is getting a better environment to test for a solution for this issue.  I only have so much time and resources to spend to make something happen and I have done it a few times in the past but it is a process to setup and get things running on my machine which is what has been dragging me behind here. I am a past unix, C and pascal programmer give me a brake :)... I do need to get this completed I need this to function properly.  I did put some notes above talking to myself... I will review after I get down editing this part of the platform, building and debugging it.  Part I have the issue with is getting the site going.
What I believe is a solution here... is steps, videos no longer than 3 minutes long.  Each is a step towards setting up the environment.  Everyone may have an issue with one step but you have to watch 5 20 minute to 1 hour videos and still may not get the answer to making it happen.  Anyone have a link to a good relevant version of setting up DNN so you can debug the platform in a productive way?  I just need an example of making a change to a file saving it and running a site to view the change.  I am guessing otherwise and I think whatever I am missing everyone else takes for granted ;)

Did you read the build instructions here, there are explanations for creating a full release, updating an existing development site or building for debug. Following the build instructions for debug is probably what you want and then you would put a breakpoint and attach to the w3wp process for the site and it should work. If you have problems, please let us know.

Thank you @valadas it has been a real challenge, seems like at one time I had it all right but then just blanked on how to do it again with so much time between attempts not doings this daily...
I will get back with any questions I have along the way.  It would be fun to get into debugging this in a pro-efficient way... and I think I just need to practice doing it for a while a lot.  I am getting warmer to VS and GitHub so that helps not having that frustration on top of this bug :)

https://github.com/dnnsoftware/Dnn.Platform/blob/develop/.github/BUILD.md
That is the link to get environment setup,
I found my problem, I never knew what Cake was... I always wondered and figured it had to do with things that it did but nothing about actually configuring it for my environment.
This unpacks the web files so I can see changes, this was my missing link.
missingLink == Cake;

I am going to do a recap here soon as I set up hoping this week get somewhere on this issue.

OK, its cold again... I am going to give this a go!

Line 422
                if (!IsAllowedLink(ji.ItemData.Url))
                {
                    //ji.ItemData.Url = string.Empty;
                }

I commented out the entire if statement and the URL information was not submitted to the database.
I commented out just the line inside the if statement as shown above and it works!
What should we do here to make this right.  I found a solution, delete this one line.  But the if statement has to stay here or the url is never submitted to the database array information for ItemData
Make Sense?  I want to submit a PR but I dont think I have this 100% yet.   Any input on what should be done here to make it a success!

Line 318
        private static bool IsAllowedLink(string url)
        {
            return !string.IsNullOrEmpty(url) && !url.Contains("//");
        }

If you are posting a Url, Of coarse it will have "//" in an address like http://domain.com.
Another issue is it puts HTTP instead of HTTPS, however most sites forward to HTTPS I entered HTTPS site, What you enter is what should be entered into the database.

what is funny... github makes a link out of a www in places, I feel DNN was going to be a little better having www.

I double tested this solution and seems to resolve things just fine.  I like how it helps the ease of extending future features with journal input as well.  Thank you all for reviewing this.

For what its worth, you can use this javascript/jquery to make the links clickable. More a patch however, If the users was to delete the actual link text before posting it won't work.
	var targetDiv = document.querySelectorAll("[id^='jid-'] .journalitem p:first-child");
	for(var i=0; i<targetDiv.length;i++){

		var sumbittedUrls = $(targetDiv[i]).text();
	  	var urls = sumbittedUrls.match(/(ftp|http|https):\/\/(\w+:{0,1}\w*@)?(\S+)(:[0-9]+)?(\/|\/([\w#!:.?+=&%@!\-\/]))?/);

		if($(targetDiv[i]).parent().find('.jlink a').length !== 0) {
	  		$(targetDiv[i]).parent().find('.jlink a').attr('href', urls[0]);

			$('.journalitem p:contains('+(urls[0])+')').each(function(){
				$(this).html($(this).html().split(urls[0]).join(""));
			});

		}

	}


Support PHPUnit 6 & DBUnit 3

Added proper namespacing for PHPUnit 6/DBUnit 3 and class aliases for backward compatibility for older versions of PHPUnit back to 4.3.

@maxleaver Nice work!


Provider android:authorities is referencing resources, which is not allowed

If you run
apktool d
on the APK from https://www.dropbox.com/s/jht3qsx6ttx13z5/Sunshine.apk?dl=0 apktool produces a manifest with the content
<provider android:authorities="@string/content_authority" android:exported="false" android:name="com.example.android.sunshine.app.data.WeatherProvider" android:syncable="true"/>
The problem is, that android:authorities must not reference a resource, as described in https://stackoverflow.com/questions/6461776/using-string-for-androidauthorities-in-a-contentprovider
Apktool should inline this kind of constant. I do not know what Apktool does to the android:versionCode attribute, but the behaviour should be the same there.

Before
ibotpeaches@raganok:~/Downloads/Apktool/Bug951$ aapt d xmltree Sunshine.apk AndroidManifest.xml | grep 'authorities'
        A: android:authorities(0x01010018)=@0x7f0d0012

After decode & build
ibotpeaches@raganok:~/Downloads/Apktool/Bug951$ cd Sunshine/dist
ibotpeaches@raganok:~/Downloads/Apktool/Bug951/Sunshine/dist$ aapt d xmltree Sunshine.apk AndroidManifest.xml | grep 'authorities'
        A: android:authorities(0x01010018)="com.example.android.sunshine.app" (Raw: "com.example.android.sunshine.app")

I fixed this issue already here - #911

Rebuilding works indeed, but why is the ResXmlPatcher not applied when the AndroidManifest.xml itself gets written out?
If you use decode and check out the AndroidManifest.xml plaintext file, you will see that the value did not get replaced.
I do not want to use apktool to rebuild the apk, but rather use the decoded resources (maybe modify them) and use plain aapt to repackage it (I can copy the dex file from the original apk). This does not work because apktool writes out the xml without the attribute getting replaced by the constant.

Other users reported that changing the value during the decode stage was improper as it didn't reflect the original application. Since apktool promises to decode matching the original nearly 100%, I agreed and did it that way. Thus the change occurs during the build stage.
I think your use case is quite rare and the existing method proves to please the greatest number.

If matching the original app is the goal, isn't match-original the option to use for it?
But what I do not understand is: If in the original app the string in the xml was actually a reference to the string resource, why does Android accept it at runtime? I mean, how could the original app manifest contain a reference to the string resource at that point if it does not get accept at runtime? Shouldn't the original apk manifest have the constant string in there?
You may be right that the use case is rare. I'll probably fork it and patch it to achieve my goals. Let's see what my teammates say :).


List all accounts in the service

As a Operator
I need to be able list an account
So that I can see all active accounts
Details and Assumptions
Acceptance Criteria
Given existing account
When operator need list them
Then make the account list and display it


⚠ GitLab Server has degraded performance

In 1bc921a, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7077 ms

Resolved: GitLab Server performance has improved in d7bc1c2.


Add support for impostors to clue scroll plugin

Add support for object impostors to clue scroll plugin.
Signed-off-by: Tomas Slusny<EMAIL_ADDRESS>

@Adam- added export for ObjectComposition#getId and used that instead of .getImpostorIds


Workshops filter should include inperson & online for delivery options

In an earlier prototype of the workshop search page we see "inperson" and "online" as options for "Filter by Delivery."
In sites we have recently built with the theme, we only see "Online" as an option.  See for example Carpentries workshops and Software Carpentry workshops.

@maneesha thank you for reporting that, a oversight from our recent refactoring of the filter/data parsing. This is fixed on the main branch of the theme. You should update your projects to latest version of the theme.

Looks good.


replacement fee too low error while trying to deploy split module

Generally replacement fee too low happens because we're sending in transaction with the same nonces.
If we were to send 2 transactions back to back, we need to increment the nonce on the second transaction previous nonce + 1, so we don't try to override or replacement the initial transaction.
feel free to ask @nkrishang for more details on how nonce works and how we can prevent it

For this error, we should write a test that properly throws it, and then recover with the nonce change strategy.


feat: add sonner toaster

Description & Technical Solution

feat: add sonner toaster

Checklist

[x] I have commented on my code, particularly in hard-to-understand areas.
[x] Already rebased against the main branch.

Deployment Links To Test
Screenshots
sonner-toater.webm

📝 Changed routes:

/index.page
…dbox/sonner-toast.page

(view full size)
(view full size)

Commit 4427b3100aa7cf88a84e02f5b35b87ad1efe55f5 (https://design-system-moon-3wq9wpzat-ppdbultimate.vercel.app).


Test results.

Have you ever wondered why your code implement is higher than the original paper? Is there any changes on net structure or used some tricks?

I see you say that the classification task is run multiple times with good results, but is the same for part segmentation and semantic segmentation? Just run many times?

I see you say that the classification task is run multiple times with good results, but is the same for part segmentation and semantic segmentation? Just run many times?

I did run multiple times for part segmentation task.
For semantic segmentation, I only run the exp in one time.
Except for runing the exp multiple times, there are no other changes or tricks.

Thanks for your reply.


Add ovarious to latest

Please add my adapter ioBroker.ovarious to latest.
This pull request was created by https://www.iobroker.dev c0726ff.

RE-CHECK!


O sorensen issue 11 1

Pull request based on Issue 11 created by Brian Buffet

Thank you @fernandohandball for spotting the missing training, this is added now in this new pull request

Thank you again @fernandohandball for spotting the mistake, I have tried again


fix(src): Replace React.useLayoutEffect with useIsomorphicLayoutEffect

Closes #367
Closes #351

+1 please merge @emilkowalski

+1 please merge @emilkowalski  It's a need for server side rendering things!

+1 pls merge @emilkowalski

+1 pls merge @emilkowalski

+1 pls merge @emilkowalski 🙏 thx


Debugger: Function Names not Visible in the Call Stack View

Does this issue occur when all extensions are disabled?: N/A The issue is with the bundled call stack view (see attached movie clip), but I have to have the Go extension enabled to demo it with Go code and show the issue.

VS Code Version: 1.77.0
OS Version: macOS Big Sur 11.7.3

Steps to Reproduce:

Load a Go project
Set a breakpoint
Launch the program and have it stop in the debugger

I would prefer if function names were visible in the call stack view without having to make the side bar really really wide.
Demo
https://user-images.githubusercontent.com/158201/229415502-941c4bee-cc7f-4190-8599-58c9f538e08a.mov

Tagging @roblourens since this is close to #178308.

Please file this on the repo for the Go extension. I think that the extension should either change what it is sending as the frame name (I'm not a Go expert so I'm not sure how to interpret what I see in your screenshot) or they can suggest a helpful way for us to truncate stack frame names in the UI that would preserve the most helpful info.

screenshot

Just to avoid any mistakes, this is a 22s video, can you confirm that you saw what the stack trace looked like towards the end, when the Call Stack View was much wider?
Either way, I now passed this on to what I believe might be the right place: https://github.com/go-delve/delve/issues/3323

I see it, but vscode only displays what the extension sends us, and it sent us a really long name

or they can suggest a helpful way for us to truncate stack frame names in the UI that would preserve the most helpful info.

Is there a way in the protocol to do this?


Curly braces inside q// match against real curly braces

In this file:
http://www.shlomifish.org/Files/files/code/vim-perl-issue-curly-braces-inside-q.txt
One cannot match the very first curly brace (at the start of line 2) because the ones inside the q/../ qualify as such. I am using commit b94578ec3e0633592820c3a10c12930b80560193 .
Please look into it.
Regards,
-- Shlomi Fish
P.S: I discovered that one cannot attach files to issues of GitHub's issue tracker:
http://stackoverflow.com/questions/10963205/how-to-attach-file-to-a-github-issue
I guess it can suck a lot less.

Vb


Fix finding packages in lower-case dirs on case-sensitive ..

.. filesystems.
NuGet now restores packages in lower case directory names, but the
build task ResolveNuGetPackageAssets constructs the path from package
name and version, like:
Microsoft.NETCore.Portable.Compatibility/1.0.1
.. instead of
microsoft.netcore.portable.compatibility/1.0.1
This breaks builds on msbuild/mono on case-sensitive filesystems, eg. on
Linux.
The actual on-disk path is available in the lock/assets.json file as:
"libraries": {
"Newtonsoft.Json/10.0.2": {
...
"path": "newtonsoft.json/10.0.2",
...

So, we use that to look for the package. But fallback to the old method if this
"path" is not available.

@radical,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@tmeschter will you take a look also?

@radical any reason for targeting dev15.1? I see this has some merge conflicts with the dev branch, can you resolve those and target dev? I don't think this will be used otherwise.

dev15.1 has more recent commits, so I thought maybe that is the "active" branch. I have opened a new PR #36 .

@radical makes sense, I didn't realize the branching strategy had changed. @tmeschter does using dev15.1 look correct to you?

@emgarten @radical The change should have gone into master rather than dev15.1, which was created specifically for Dev15.1 changes. However, it probably doesn't matter.
I had to create that branch a while back because master already contained changes that weren't approved for 15.1, and so I needed a 15.1-specific branch based on what we shipped for 15.0 RTM. Ideally what we would do is merge dev15.1 back into master, and for the time being use master for changes that are going to go into the next release (15.4 or 15.5).


Handle invalid email in GitHub backend

Automated message from Dropbox CLA bot
@umairwaheed, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott, please review.

Merged, thanks @umairwaheed!
I think we probably still need to do some work on telling the user what's going on in the error handling -- if we redirect them back to login because they don't have an account, it'd be really good to have that login page tell the user why.

@timabbott, I think I have figured out the problem with our GitHub backend while I was trying to make the GitHub auth work in dev. Basically the authenticate method of GitHub backend is a catch-all method. The algorithm that Django uses, tries all authenticate methods against the arguments and if any method accepts those arguments then Django will use that. So if our GitHub backend is above other backends, it's authenticate method would run.


ARROW-5660: [GLib][CI] Use Xcode 10.2

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4641   +/-   ##
=======================================
  Coverage   88.49%   88.49%
=======================================
  Files         890      890
  Lines      110406   110406
  Branches     1418     1418
=======================================
  Hits        97703    97703
  Misses      12422    12422
  Partials      281      281

Impacted Files
Coverage Δ

cpp/src/arrow/io/readahead.cc
95.91% <0%> (-1.03%)
:arrow_down:

cpp/src/plasma/thirdparty/ae/ae.c
70.75% <0%> (-0.95%)
:arrow_down:

cpp/src/arrow/csv/column-builder.cc
97.05% <0%> (+1.76%)
:arrow_up:

Continue to review full report at Codecov.

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


Animations override opacity settings

Hey I apologize if this is an easy one but I setup a fadeIn animation on my top navigation bar and it works fine except I lost the ability to have it be semi-transparent. Is there any way to override this or am I stuck with one or the other.

As you can see in the CSS file, the fadeIn is simple to edit
@-webkit-keyframes fadeIn {
  0% {
    opacity: 0;
  }

  100% {
    opacity: 1;
  }
}

@keyframes fadeIn {
  0% {
    opacity: 0;
  }

  100% {
    opacity: 1;
  }
}

.fadeIn {
  -webkit-animation-name: fadeIn;
  animation-name: fadeIn;
}

better would be another animation for your needs, e.g. for 50% transparency:
@-webkit-keyframes fadeInNavigation {
  0% {
    opacity: 0;
  }

  100% {
    opacity: 0.5;
  }
}

@keyframes fadeInNavigation {
  0% {
    opacity: 0;
  }

  100% {
    opacity: 0.5;
  }
}

.fadeIn {
  -webkit-animation-name: fadeInNavigation;
  animation-name: fadeInNavigation;
}

now you could add fadeInNavigaion as your desired animation

perfect, thank you so much

On Apr 16, 2015, at 2:50 PM, Sundypha<EMAIL_ADDRESS>wrote:
As you can see in the CSS file, the fadeIn is simple to edit
@-webkit-keyframes fadeIn {
0% {
opacity: 0;
}
100% {
opacity: 1;
}
}
@keyframes fadeIn {
0% {
opacity: 0;
}
100% {
opacity: 1;
}
}
.fadeIn {
-webkit-animation-name: fadeIn;
animation-name: fadeIn;
}
better would be another animation for your needs, e.g. for 50% transparency:
@-webkit-keyframes fadeInNavigation {
0% {
opacity: 0;
}
100% {
opacity: 0.5;
}
}
@keyframes fadeInNavigation {
0% {
opacity: 0;
}
100% {
opacity: 0.5;
}
}
.fadeIn {
-webkit-animation-name: fadeInNavigation;
animation-name: fadeInNavigation;
}
now you could add fadeInNavigaion as your desired animation
—
Reply to this email directly or view it on GitHub https://github.com/daneden/animate.css/issues/386#issuecomment-93813176.


STM32 Compile Issues

Hi, I cannot get a simple DCS-BIOS sketch to compile on my STM32F103C8 blue pill.
I can however compile and upload the simple Blink sketch and another sketch which reads potentiometer outputs etc with no issues.
I used the STM32duino board definitions.
I have installed your Arduino library version 3.6  from here: https://github.com/talbotmcinnis/dcs-bios-arduino-library/releases/download/0.3.6/dcs-bios-arduino-library-0.3.6.zip
I am using the Flight Panels DCS BIOS code.
Hers is the sketch code:
#define DCSBIOS_DEFAULT_SERIAL
#define DCSBIOS_FOR_STM32
#include "DcsBios.h"
/* Declare a Master Caution Reset button on pin 10 /
DcsBios::Switch2Pos ufcMasterCaution("UFC_MASTER_CAUTION", PB4);
/ Make the LED connected to pin 13 into a Master Caution Light */
DcsBios::LED masterCaution(0x1012, 0x0800, PC13);
void setup() {
DcsBios::setup();
}
void loop() {
DcsBios::loop();
}
I get the following errors on compile:
In file included from C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/DcsBios.h:130,
from C:\Users\Peter\AppData\Local\Temp\arduino_modified_sketch_800786\MasterCaution.ino:6:
C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/internal/RotarySwitch.h: In member function 'void DcsBios::RotarySwitchT<pollIntervalMs, stepsPerDetent>::pollInput()':
C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/internal/RotarySwitch.h:48:54: error: no matching function for call to 'min(int, signed char&)'
48 |     switchValue_ = min(switchValue_+1, maxSwichValue_);
|                                                      ^
In file included from c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\algorithm:62,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring_constants.h:26,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring.h:34,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/Arduino.h:36,
from sketch\MasterCaution.ino.cpp:1:
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algo.h:3474:5: note: candidate: 'template<class _Tp, class _Compare> constexpr _Tp std::min(std::initializer_list<_Tp>, _Compare)'
3474 |     min(initializer_list<_Tp> __l, _Compare comp)
|     ^~~
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algo.h:3474:5: note:   template argument deduction/substitution failed:
In file included from C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/DcsBios.h:130,
from C:\Users\Peter\AppData\Local\Temp\arduino_modified_sketch_800786\MasterCaution.ino:6:
C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/internal/RotarySwitch.h:48:54: note:   mismatched types 'std::initializer_list<Tp>' and 'int'
48 |     switchValue = min(switchValue+1, maxSwichValue);
|                                                      ^
In file included from c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\algorithm:62,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring_constants.h:26,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring.h:34,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/Arduino.h:36,
from sketch\MasterCaution.ino.cpp:1:
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algo.h:3468:5: note: candidate: 'template constexpr _Tp std::min(std::initializer_list<_Tp>)'
3468 |     min(initializer_list<_Tp> l)
|     ^~~
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algo.h:3468:5: note:   template argument deduction/substitution failed:
In file included from C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/DcsBios.h:130,
from C:\Users\Peter\AppData\Local\Temp\arduino_modified_sketch_800786\MasterCaution.ino:6:
C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/internal/RotarySwitch.h:48:54: note:   mismatched types 'std::initializer_list<Tp>' and 'int'
48 |     switchValue = min(switchValue+1, maxSwichValue);
|                                                      ^
In file included from c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\algorithm:61,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring_constants.h:26,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring.h:34,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/Arduino.h:36,
from sketch\MasterCaution.ino.cpp:1:
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algobase.h:278:5: note: candidate: 'template<class _Tp, class _Compare> constexpr const _Tp& std::min(const _Tp&, const _Tp&, _Compare)'
278 |     min(const _Tp& __a, const _Tp& __b, _Compare comp)
|     ^~~
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algobase.h:278:5: note:   template argument deduction/substitution failed:
In file included from C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/DcsBios.h:130,
from C:\Users\Peter\AppData\Local\Temp\arduino_modified_sketch_800786\MasterCaution.ino:6:
C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/internal/RotarySwitch.h:48:54: note:   deduced conflicting types for parameter 'const Tp' ('int' and 'signed char')
48 |     switchValue = min(switchValue+1, maxSwichValue);
|                                                      ^
In file included from c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\algorithm:61,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring_constants.h:26,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/wiring.h:34,
from C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\cores\arduino/Arduino.h:36,
from sketch\MasterCaution.ino.cpp:1:
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algobase.h:230:5: note: candidate: 'template constexpr const _Tp& std::min(const _Tp&, const _Tp&)'
230 |     min(const _Tp& __a, const _Tp& b)
|     ^~~
c:\users\peter\appdata\local\arduino15\packages\stmicroelectronics\tools\xpack-arm-none-eabi-gcc\10.2.1-1.1\arm-none-eabi\include\c++\10.2.1\bits\stl_algobase.h:230:5: note:   template argument deduction/substitution failed:
In file included from C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/DcsBios.h:130,
from C:\Users\Peter\AppData\Local\Temp\arduino_modified_sketch_800786\MasterCaution.ino:6:
C:\Users\Peter\Documents\Arduino\libraries\dcs-bios-arduino-library-0.3.6\src/internal/RotarySwitch.h:48:54: note:   deduced conflicting types for parameter 'const Tp' ('int' and 'signed char')
48 |     switchValue = min(switchValue+1, maxSwichValue);
|                                                      ^
Multiple libraries were found for "Servo.h"
Used: C:\Users\Peter\AppData\Local\Arduino15\packages\STMicroelectronics\hardware\stm32\2.1.0\libraries\Servo
Not used: C:\Program Files (x86)\Arduino\libraries\Servo
exit status 1
Error compiling for board Generic STM32F1 series.

I think this will be fixed on master as of a few days ago.  Can you try to pull that down?

Hi, I don't know what you mean.. Sorry.
Are you referring to the Arduino library ?   Can you send me a link ?
Thanks.

https://github.com/talbotmcinnis/dcs-bios-arduino-library/tree/master/src
Download the code from that link, and place it in your documents/arduino/library/dcs-bios... (make sure the files overwrite existing ones to know you are in the right place).
Alternatively I can work on making a new release zip but probably not able until the weekend.

Thanks. I will try that tomorrow when I can and get back to you.

Hi, that worked perfectly for compile and upload.....Thanks.
However, when I use the connect-serial-port.cmd I cannot send switch commands to the aircraft, but it does work with LED light status.
So, it seems that I can't send commands to the sim but the sim can send led light status info back to the board (I'm using the built-in led to test).
Now, here is the really weird part:  When I use the DCS-BIOS Hub executable to connect to the sim, everything works !!!
Any idea why this may be so ?
Also, is there a way to synchronise the lights in the sim to the physical led lights when we first enter the cockpit?
For example, my built-in LED is on when the Master Caution light in the cockpit if off and vice versa.
I need a way to read the status of all the lights and then set the physical LED lights to that same status.

Hi Talbot, have you had. chance to consider this problem re connection?

Hey.  Very sorry for ghosting you for so long.  Just making sure I understand you:
It sounds like communication from DCS to your board works.  You are saying an LED on your board reacts to DCS?  But a switch on the board does not cause a change in DCS?
I'd start by confirming the serial port can receive.  If you upload a sketch and use the arduino serial monitor (or any serial terminal program), you should be able to toggle the switch and see a human readable line every time the switch state changes.
Next I'd be curious to know if the Chrome Controls Ref works.  You can launch the controls ref, then run DCS and try issuing commands from the control ref and see if they have an effect in DCS.  If not, review your export.lua installation and specifically the files that should be copied along with it.
In DCS review control bindings for the control you are testing to make sure its not bound to a joystick button that is holding the state in DCS.
You mentioned firewall... are you sending this between PCs or is DCS running on the same PC that the microcontroller is connected to?
Let me know and I'll try to get back to you faster next time!!

Thanks for replying Talbot.

It sounds like communication from DCS to your board works. You are saying an LED on your board reacts to DCS? But a switch on the board does not cause a change in DCS?

That is correct.

If you upload a sketch and use the arduino serial monitor (or any serial terminal program), you should be able to toggle the switch and see a human readable line every time the switch state changes.

Does the sketch have to have serial.print() statements in it or can it be any sketch and just have  the serial monitor open and flick a switch connected to the board ?

In DCS review control bindings for the control you are testing to make sure its not bound to a joystick button that is holding the state in DCS.

I have no controllers attached during testing with DCS BIOS (except fro rudders).

You mentioned firewall... are you sending this between PCs or is DCS running on the same PC that the microcontroller is connected to?

DCS is running on the same PC that the microcontroller is connected to.  This is not a networked situation.
Why would it work flawlessly with the HUB executable and not the connect-serial-port.cmd ?
I am away from my computer at the moment flying, so I hope to get back to you ASAP with the rest of the testing you asked for.
Regards,  Pete

Great report. It sounds like you have done everything right. My specialty pretty much ends at the microcontrollers so you might need to raise the question back in discord with the dcs-bios side of things.  I have no theories about why hub would work I haven't used it in years.  The original dev ?Ian?  Has been pretty unhelpful about releasing or supporting it afaik but I have never asked him personally.
One idea might be to use portman or performance monitor to check for any conflicting processes using the dcs-bios ports.  You'll see the required ports if you open the script file pointed to by your export lua.  One for out, one for in.  Or maybe increment both sides (socat command and the lua) by the same amount and retry?

Thanks. I’ll try and work out the Portman thing but I think it’s beyond my pay grade.

Portman is beyond what you need actually.  Window resource monitor will show you what you need: http://support.moonpoint.com/os/windows/network/monitoring/resmon/NIS/
Just look for anything on port 7778.

Could Windows 11 be the issue ?

Interesting.  Definitely possible.  Have you tried running socat elevated?
Once socat is running you should definitely see it in listening on 7778.  If not, then something has blocked it.

On Nov 19, 2021, at 08:23, speedbird5 @.***> wrote:

Could Windows 11 be the issue ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sorry, socot sends to 7778 so it’ll be some dcs-bios or dcs itself listening on port 7778.

On Nov 19, 2021, at 08:23, speedbird5 @.***> wrote:

Could Windows 11 be the issue ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yes I have tried running socat  elevated and I get an error. See my earlier post and the error I get.

Talbot, I you have access to a Windows 11 machine, is it possible for you to test ?  thanks

Hi, sorry for resurrecting this, is RS485 supported? Its complaining about <libmaple/usart.h>


Cant install realsense-viewer?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{D435i }

Firmware Version
(

Operating System & Version
Ubuntu 22.04.1 LTS

Kernel Version (Linux Only)
5.15.0-46-generic

Platform
PC

SDK Version
{ legacy / 2.. }

Language
{python }

Segment
{others }

Issue Description
After upgrading to ubuntu 22.04.1 LTS i dont seem to have realsense-viewer installed anymore and cant do so either? Also when trying to run any realsense related code im getting the following error
RuntimeError: Failed to open scan_element /sys/devices/pci0000:00/0000:00:14.0/usb4/4-2/4-2:1.5/0003:8086:0B3A.0005/HID-SENSOR-200076.4.auto/iio:device1/scan_elements/in_anglvel_x_en Last Error: Permission denied
What do?

Hi @MyOtherNamesWereTaken  The librealsense SDK does not currently have official support for Ubuntu 22.04 (Jammy) or kernel 5.15.  An unofficial test patch for kernel 5.13 and 5.15 support and a test package is available at https://github.com/IntelRealSense/librealsense/issues/10439#issuecomment-1171965106
https://github.com/IntelRealSense/librealsense/issues/10792#issuecomment-1220320263 provides advice for installing the test package.

When will the official support be released?

Ubuntu 22.04 Jammy support is being worked on by Intel, though I do not have a time schedule to provide for formal implementation at the time of writing this.  I do apologize.

Hi @MyOtherNamesWereTaken  Do you require further assistance with this case, please?  Thanks!


cloud-nuke: 0.1.27 -> 0.1.28

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/gruntwork-io/cloud-nuke/releases.
meta.description for cloud-nuke is: "A tool for cleaning up your cloud accounts by nuking (deleting) all resources within it"
meta.homepage for cloud-nuke is: "https://github.com/gruntwork-io/cloud-nuke"
meta.changelog for cloud-nuke is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/bvkzi0xd968hvavflqgw5pxw1br86d07-cloud-nuke-0.1.28/bin/cloud-nuke passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 0.1.28 with grep in /nix/store/bvkzi0xd968hvavflqgw5pxw1br86d07-cloud-nuke-0.1.28

found 0.1.28 in filename of file in /nix/store/bvkzi0xd968hvavflqgw5pxw1br86d07-cloud-nuke-0.1.28

directory tree listing: https://gist.github.com/0fc8ea0f7762c8ca8b90a5599df513f1

du listing: https://gist.github.com/039c25ee24e67b01b9cbf605ad82280d

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
cloud-nuke

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bvkzi0xd968hvavflqgw5pxw1br86d07-cloud-nuke-0.1.28 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cloud-nuke https://github.com/r-ryantm/nixpkgs/archive/e8a7bf4e2db69b8004d911b8cbab0f136d23d9c8.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bvkzi0xd968hvavflqgw5pxw1br86d07-cloud-nuke-0.1.28
ls -la /nix/store/bvkzi0xd968hvavflqgw5pxw1br86d07-cloud-nuke-0.1.28/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cloud-nuke

Maintainer pings
cc @marsam @kalbasit for testing.

Result of nixpkgs-review pr 118972 run on x86_64-darwin 1

1 package built:

cloud-nuke


Update scalaVersion to 2.13

Affects Version(s): <Spring for Apache Kafka version>

The Kafka library is built with Scala. Scala is known to be binary incompatible between 'minor' versions. This means that a library built with Scala 2.12 cannot be replaced by a library built with Scala 2.13.
Scala 2.13 is released over a year ago (https://scala-lang.org/news/2.13.0) and a Kafka library built with Scala 2.13 is released since Kafka 2.4.0+ (https://www.confluent.io/blog/apache-kafka-2-4-latest-version-updates).
I would like to be able to have my project use Scala 2.13 artifacts (performance improvements, longer support, etc..) and also use Spring Kafka.
I propose to change the scalaVersion of Spring Kafka to 2.13.
Pros:

Projects are unblocked to use Scala 2.13 artifacts

Cons:

Projects that use artifacts that are not available for Scala 2.13 (legacy, unmaintained, ..) cannot use the releases of Spring Kafka that have the scalaVersion set to 2.13.

The Kafka downloads page still recommends 2.12.

We build for multiple versions of Scala. This only matters if you are using Scala and you want a version built for the same Scala version you use. Otherwise any version should work (2.12 is recommended).

So we need to stick with that until it changes.
Furthermore, the runtime does not use scala at all; we only use the scala libraries in the spring-kafka-test jar (for the embedded Kafka server).
You can always override the versions in your project; instructions here.

Thanks for the instructions! Adding the following to my POM does indeed help me :)
        <dependency>
            <groupId>org.apache.kafka</groupId>
            <artifactId>kafka_2.13</artifactId>
            <version>${kafka.version}</version>
            <scope>test</scope>
        </dependency>

        <dependency>
            <groupId>org.apache.kafka</groupId>
            <artifactId>kafka_2.13</artifactId>
            <version>${kafka.version}</version>
            <classifier>test</classifier>
            <scope>test</scope>
        </dependency>

You are completely correct, this is only an issue when you have spring-kafka-test as a dependency on your classpath.
I think it makes sense to follow the kafka guideline then in upgrading, since there is a workaround in place!

FYI, as of the recent 2.6 release; they are now recommending 2.13 - so our next release (also 2.6 coincidentally) will move to that version.
https://kafka.apache.org/downloads


Remove option to customize installation

in #3904 this was missed as there's only one component available to install now we can remove the options  to customize the installation
fixes #3909

Wondering if the tray gets automatically removed on upgrades now that it's removed from installers?

I checked on macOS and not it doesn't uninstall the tray on upgrade
on windows i haven't checked yet (don't have the windows machine with me today) but there the upgrade process is, first uninstall existing then install new one, so there most likely the installed tray will be removed during upgrade
for macOS we can add rm -rf /Applications/Red Hat OpenShift Local.app to the postinstall script and that'd make the behavior consistent for both the platforms

Wondering if the tray gets automatically removed on upgrades now that it's removed from installers?

I checked on macOS and not it doesn't uninstall the tray on upgrade
on windows i haven't checked yet (don't have the windows machine with me today) but there the upgrade process is, first uninstall existing then install new one, so there most likely the installed tray will be removed during upgrade
for macOS we can add rm -rf /Applications/Red Hat OpenShift Local.app to the postinstall script and that'd make the behavior consistent for both the platforms

something to stop the tray if it's running? This can be handled in a followup issue.


on_create_client | session established | client_key: 0xAABBCCDD, address: ... Segmentation fault (core dumped)

Ubuntu 18.04 LTS, (ESP32 with ros2arduino):
When I first run: MicroXRCEAgent udp -p 2018, the client connected well.
Then I close the Agent,not use 'q' ,but with 'Ctrl + C'
After I started the Agent and client again,
It shows: on_create_client | session established | client_key: 0xAABBCCDD, address: ...

I found this issue: Not follow https://micro-xrce-dds.readthedocs.io/en/latest/installation.html#installing-the-agent-stand-alone
but follow https://micro-xrce-dds.readthedocs.io/en/latest/installation.html#installing-agent-and-client,
When I Install Agent stand-alone the issue disappeared.


New release

Hi, guys. Thanks for the great job!
I'd like to know the new release date. Since this change https://github.com/devise-security/devise-security/commit/e280f147dcd43a70e3fecbe8e12c09911fd17539#diff-80cfb2fb1849d9a2fd2fbad3821ff00c7d0d22e3ea1a095f9d9acf59c1ef2db8
is really important for Ruby 3.0

I'll take a look and see if things are ready. There were some changes I wanted to get into the next release but they possibly can wait so that this fix goes out.

Once we get https://github.com/devise-security/devise-security/pull/291 and https://github.com/devise-security/devise-security/pull/300 in we should be able to do a release.

Seconding the OP - thanks for the great work, but also my app requires Ruby 3.0, and so also requires this new release.
Many thanks

v0.16.0 is out.


Mismatch between code and equation given in the paper

Hello. I am training ClimODE to other tasks for comparison, and have found some mismatch of the code.

Code to generate the initial velocity in optimize_vel
vel_loss = nn.MSELoss()(delta_u,out.squeeze(dim=1)) + 0.0000001*(final_x + final_y)
The code wants to generate the initial velocity according to the continuity function described in the paper, but it minizes the $||du/dt - \text{advection}(\text{velocity})||_2^2$, which is different from the equation (10) $\text{argmin} ||du/dt + \text{advection}(\text{velocity})||_2^2$.
However, as a result, the wrong loss function vel_loss will generate a velocity fits the numerical differentiation of any variable at current time, which is also useful for predicting the future state.

Code to update the state of variable in Climate_encoder_free_uncertain
The function self.pde will generate the tuple $(d \ \text{velocity}/dt,  \text{advection}=\text{adv}1+\text{adv}2 )$, which is used to numerical integration in
pde_rhs  = lambda t,vs: self.pde(t,vs)
final_result = odeint(pde_rhs,final_data,t,method=self.method,atol=atol,rtol=rtol)
It indicates that $du/dt = \text{advection}$, which is contradictory to the equation (2) $du/dt = -\text{advection}$ in the paper.

The two mismatch mentioned above would still generate the true result, but the true velocity should be the negative of the velocity trained in the model. However, I don't know whether it would produce new problems in other parts of the code.

Hi,
In the code, we are fitting and learning -v instead of v, which modifies the advection equation as $du/dt = advection$. This equation defines the dynamics for the initial velocity, as you showed, also in the    pde function in the model class to evolve the state variables u.
The advection equation is used only in these two model components, and the emission model is independent of the advection equation (only taking the evolved states).  Hence, the learning of -v will not affect any other part of the model, and since we use the same equation everywhere throughout the model, the dynamics learned are still advection. We have empirically verified that learning -v does not affect via validating the mass preserving nature of advection (See Appendix H).
If you plot the velocity field, kindly use -v as it will show the true velocity, as learning -v is just a negative direction of v and is consistent with the advection equation used in the model. The advection equation in the paper and used in the code is conceptually the same; it's just a matter of choice that we learn -v instead of v, but the state still follows advection.
I will also add a note in the repo mentioning that to clarify it.

Hi,
Due to inactivity, I assume the issue has been resolved and am closing it. If you have any other questions, feel free to open another issue.

Hi,
It was an oversight on my part that I forgot to close the issue. Thank you very much for your answer.

------------------ Original ------------------
From: @.>;
Date:  Wed, Aug 28, 2024 10:22 PM
To: @.>;
Cc: @.>; @.>;
Subject:  Re: [Aalto-QuML/ClimODE] Mismatch between code and equation given in the paper (Issue #2)

Hi,
Due to inactivity, I assume the issue has been resolved and am closing it. If you have any other questions, feel free to open another issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Does TTL work real-timely ?

When a key expires, would it be removed immediately, or ardb doing this until it be scaned ?

it would be deleted while it be scaned. It's too expensive to remove expired key immediately.

Does it have a background process to do this periodically ?

There is a background thread  to do this work, but it's only work well if only a few keys were setting expired time. If all keys have expire time,  the thread would be very slow to do this work.
in 'rocksdb-only' branch, i'm using rocksdb's compaction filter to do this work, the expired keys would only be deleted in compaction job.

Thanks.


[SPARK-35943][PYTHON] Introduce Axis type alias.

What changes were proposed in this pull request?
Introduces Axis type alias for axis argument to be consistent.
Why are the changes needed?
There are many places to use axis argument. We should define Axis type alias and reuse it to be consistent.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.

Test build #140410 has started for PR 33144 at commit 63530c4.

Test build #140410 has finished for PR 33144 at commit 63530c4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140410/

Merged to master.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44925/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44925/


Redirect previous roadmap location to current one

The document roadmap.md was linked-to several times then moved as part of restructuring work. In order to retain the working link, add the sphinx-reredirects module and tell it to redirect the old roadmap.md location to the new one.

Note for reviewers: The result on this PR's build of the docs is that the old location:
https://mbed-tls--90.org.readthedocs.build/en/90/roadmap/
Is redirected to the new location:
https://mbed-tls--90.org.readthedocs.build/en/90/project/roadmap/

So far I've been adding redirect to the readthedocs config - which produces HTTP 301 moved redirects rather than HTML <meta> redirects like sphinx-reredirects. The downside is that the list of redirects lives outside this repo, but it is more compatible with non-browser clients.

The relevant docs: https://docs.readthedocs.io/en/stable/user-defined-redirects.html

FYI: Readthedocs feature request for managing redirects in-tree, with potential workarounds listed: https://github.com/readthedocs/readthedocs.org/issues/4221

So far I've been adding redirects to the readthedocs config - which produces HTTP 301 moved redirects rather than HTML <meta> redirects like sphinx-reredirects. The downside is that the list of redirects lives outside this repo, but it is more compatible with non-browser clients.

Ah okay, that makes more sense then. I'll close this PR as superfluous.


enhencement: supplement the comment for SchemaActionType

What problem does this PR solve?
Issue Number: ref #4862
Problem Summary:
What is changed and how it works?
Supplement the comment for SchemaActionType.
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

we need more comments on these lines
https://github.com/pingcap/tiflash/blob/f4c2e012fc3067193412aca91c6b26e30173c722/dbms/src/TiDB/Schema/SchemaBuilder.cpp#L373-L374
/cc @lidezhu for https://github.com/pingcap/tiflash/pull/5044

/merge

/run-all-tests

/merge


Change collection underlying NodeSet into a List

From the start of SDA, a NodeSet has been extending a CopyOnWriteArraySet. Is this really the best choice when use as a parent nodes' child set?
A Set cannot contain duplicates, which makes sense for a child nodeset so at the time it seemed appropriate. Also, it is thread-safe which felt important. However:
A node that is already a child cannot be added to the same (or another) parent anyway so we do not require a Set to safe-guard against adding a duplicate.
Adding new nodes, is almost always done at the end, so an ArrayList is a good choice performance wise. Removing a node maybe costly but removals are rare anyway.
Concurrent writes to a nodeset are usually rare and if required synchronisation can be added.
In A List we get some methods for free (get, indexOf).

See branch issue5 for a POC (it takes only 1 line of code to change NodeSet).
Some testing suggests that the performance gain when using ArrayList is rather limited for my testcases. Also, I actually get a concurrent write exception in the SDS core lib and I'm not sure where it is coming from. It seems the CopyOnWrite collection was not a bad choice after all. At the very least we would still require a CopyOnWriteArrayList.

I changed the underlying collection into a CopyOnWriteArrayList rather than a Set for the following reasons:

There seems to be a modest performance gain, or at least no obvious degradation.
NodeSet no longer has to provide implementations of get() and indexOf().
In rare cases I would get errors in SDT core as Jaxen looks for a List rather than a Set.

Closed of sda-core 2.1.0.


Custom template not working

I'm not getting a custom template to work (Grunt Iconizr 0.2.3).
First is a tiny thing that could be fixed:
If I overwrite the existing template in node_modules/grunt-iconizr/node_modules/iconizr/tmpl/sprite.scss and don't delete the existing sprite sheets in the destination folder, the new template changes are not picked up. This made me think it wasn't working altogether. Once I deleted the folder contents it started to get picked up again.
However, my main issue is that I don't seem able to get a custom template picked up. For example, if I create a 'template.scss' file and set an object like this in the Gruntfile.js:
splashv2: {
				src: 'source/components/splash/icons-source',
				dest: 'source/components/splash/icons-sprited',
				options: {
					padding: "10",
					layout: "vertical",
					prefix: "Cat-id",
					spritedir: "",
					sprite: "sprite-v2",
					verbose: "11",
					render: {
						css: false,
						scss: {
							template    : 'source/components/splash/template.scss',
							dest        : 'source/components/splash/_icon-template'
						}
					}
				}
			},

I get no error message but the template isn't picked up and used. What do I have wrong here? I've tried all manner of relative/absolute paths but just don't seem able to get the template picked up.
For example, If I had a structure like this:
node_modules (Iconizr live in here)
source (my own project)
  /components
    /splash
      template.scss
      _icon-template.scss

How should I be linking that inside the object?
Or is this functionality only available in svg-sprite at present?

Sorry, finally figured this out.

@benfrain No problem, Ben.
Currently grunt-/node-iconizr is still based on the "old" svg-sprite base, as I'm so totally overloaded these days (moving my office next Friday ...) and don't manage to finalise the next generation somehow. However, there's already a new version of (grunt-)iconizr here on GitHub — if you're interested, please check the "dev" branches of node-iconizr and grunt-iconizr. Unfortunately there's no proper documentation yet (that's why I didn't release it), but we already do use it in production. For now, this would be your best bet as a manual.


Actually update the module to dnd5e 3.0.4

The main page on https://foundryvtt.com/packages/items-with-spells-5e shows that version 11.1.2 supports up to dnd5e version 3.0.4... but it can't be enabled because this line wasn't changed. This fixes the issue and we can actually enable it in our game now.

The package page actually just shows the system's latest version, not the latest version the module is compatible with. Module authors have no way of setting this property anywhere but in the manifest.
Regardless, there aren't any plans to update IWS for v3 of the dnd5e system.


Feature Request: Add support for more chains

I'd especially be interested in:

Arbitrum
Optimism

Optionally:

Avalanche
Gnosis

Sounds good! I'll put it on the backlog. If you are interested, you can also implement this yourself by adding these chains's info in
https://github.com/fuzzland/ityfuzz/blob/master/proxy/main.py#L49-L177
https://github.com/fuzzland/ityfuzz/blob/master/src/evm/uniswap/mod.rs#L390-L415
https://github.com/fuzzland/ityfuzz/blob/master/src/evm/onchain/endpoints.rs#L19-L119

A few new chains are supported in #130, but without flashloan support, which we are still working on


Use Default Import Instead of Named Import

This is to fix #4.

Codecov Report

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

@@          Coverage Diff           @@
##           master      #5   +/-   ##
======================================
  Coverage    90.9%   90.9%
======================================
  Files           1       1
  Lines          22      22
  Branches        3       3
======================================
  Hits           20      20
  Misses          1       1
  Partials        1       1

Continue to review full report at Codecov.

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

Not needed. The documentation is out-of-sync with the released code.


audit: pin glibc version to 2.23

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

Review period will end on 2021-02-16 at 00:00:00 UTC.

Initially I wanted to write a rubocop for this, but as I need formula.version I think this is right place for this check.

Review period ended.

Thanks, Michka!


changelog?

Where can I find a history of changes made to the latest release?

https://github.com/bitbeans/SimpleDnsCrypt/releases/tag/0.4.3


lsd release v1.0.0

modify config.yaml;
rename dracula.yaml to colors.yaml;
parameters theme should be modified to custom;
lsd v1.0.0 add some new features;

Can confirm that with the current instructions lsd issues the following:
Warning: the 'themes' directory is deprecated, use 'colors.yaml' instead.

I came to see the repository to fix this, and had the idea to check the PRs beforehand.


git command fails but build doesn't fail

Details
I noticed this while reporting #5032

Read the Docs project URL: https://readthedocs.org/projects/test-builds/builds/
Build URL (if applicable): https://readthedocs.org/projects/test-builds/builds/8319150/
Read the Docs username (if applicable): humitos

Expected Result
Fail the build completely.
Actual Result
git command fails, but the build continues and passes. This means that the version built is invalid and we end up serving inconsistent docs.

I saw this when svn was failing some weeks ago, we need to verify the exit code and raise the proper exception.


Update ansiblels.lua: fix completion issue

Fix for issue: https://github.com/ansible/ansible-language-server/issues/391

see https://github.com/neovim/nvim-lspconfig/pull/1464


babel support for the node component loader

A follow up of https://github.com/noflo/noflo/pull/294
Huh, it seems running npm install babel-core --save sorted the dependencies of the package.json in alphabetical order for some reason... did it always work like that?

Thanks!


Why doesn't EQ operator work?

As indicated by the "Search Tips" box in the UI:
GT, LT, GE, LE all work for both answers and votes.
Why doesn't EQ work? Wouldn't this be standard along with the other comparison operators? Or did we customize the others and just not expose EQ (yet)?

No customization here, no.
I guess there's no EQ because that's the default.  If you use  constraint:value, then it's looking for things equal to value

OH! Great thanks for the tip, missed that.


Update py-publish.yml to generate build provenance attestations

ref to #28. update .github/workflows/py-publish.yml will allow all released Python packages in cpp-linter org can support generating attestations. then users can verify artifact attestations with GitHub CLI if they need to.

But that is just for the static binaries. This workflow is for python publishing. Am I still misunderstanding attestations?

Oh, ok. I didn't know you plan to use gh-cli to verify. According to the gh attestation verify docs:

By default, the verify command will attempt to fetch attestations associated with the provided artifact from the GitHub API. If you would prefer to verify the artifact using attestations stored on disk (c.f. the download command), provide a path to the --bundle flag.

I was expecting we'd have to keep track of the attestations locally. I did not know what tool you planned to use (or the tool's behavior).
Given the use gh-cli, I guess it isn't necessary to save the attestation as downloadable artifacts. Still I'm unsure of where exactly the gh-cli tool is fetching the appropriate attestation. Obviously, I'll learn as I follow your progress 😉

yep. LGTM


expandColumnOptions does not work

Hi :)
I was trying expandColumnOptions to pass into BootstrapTable a callback for expandColumnComponent, as described here literally:
<BootstrapTable //... data, options, tableHeaderClass and tableBodyClass passed into and working
  expandableRow={() => {return true;}}
  expandComponent={this.getExpandableContent.bind(this)}
  expandColumnOptions={{
      expandColumnVisible: true,
      expandColumnComponent: expandColumnComponent,
      columnWidth: 100
  }}>
   {/* ... */}
</BootstrapTable>

having:
const expandColumnComponent = ({isExpandableRow, isExpanded}) => {
   debugger;
   let content = '';
   if (isExpandableRow) {
      //as in docs
   }
};

This option is not having any effect on the rendered table. No errors in the console, and debugger is never reached.
Any idea on this? Thanks a lot!

@susannakosic sorry for lately reply, is it solved?

Upgrading to last stable sorted it.


arp-scan: 1.9 -> 2017-07-17

Motivation for this change

Change upstream to GitHub
There is no release in recent years but development is undergoing on GitHub by the original author.
Add missing dependency
get-oui and get-iab need some Perl modules, but they are lacked.

cc @bjornfor
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Please ask upstream to make a new release.
If packaging unreleased software we invert QA responsibility between upstream/downstream.

That was my lack of understanding. This commit includes a point of minor release update. Do you think that it's unsatisfactory for QA yet? If not, I'll change rev to https://github.com/royhills/arp-scan/commit/5d2d1f7feedbebf0fcbfc8af71b1aee861ce72e2.

Please see https://github.com/royhills/arp-scan/issues/28.

Should we just recognise bfee8899993fbcca74de44c594903e0827da10b5 as the 1.9.5 release? Looks like negotiations with upstream about a git tag are not succeeding…

Since the project is abandoned, if the update is needed I'd set rev = "96d195093de46e67ea2bda0f00700e37dd11df3f" (the last commit on master) and merge this.

@mikoim: Upstream finally tagged 1.9.5! Would you mind refreshing the PR? Also, if the missing dependencies exist in the current version, please put that commit first. (Fix bugs first, then add features.)

@bjornfor, 1.9.5 is an ancient tag.

@orivej it is the latest version that got a version bump in README, and it is much newer than 1.9

OK, but it was made before this PR, and @mikoim initially wrote "There is no release in recent years but development is undergoing on GitHub by the original author", so they were aware of 1.9.5.

Right, it's an old tag, but it is the latest release. I think we should stick with releases and let upstream decide when the code is ready for downstream.

It is an old revision, but it is newly tagged.

@matthewbauer now using tagged release


Added requestedTheme option into terminal settings

Added the possibility to change the theme (dark/light/default) from the terminal settings

 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.:x: Gabboxl sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

The most common scenarios should be there in the GUI, with a button in the settings screen to open the commented JSON file with advanced settings

This change just exposes that setting in the default profiles.json, right? Without this change, if you add "requestedTheme": ... it will still work, you just won't know the setting exists.

@DHowett-MSFT yes, this change just exposes the setting to change the theme in the json file.

That makes sense. Thanks!

I found what was blocking the serialization of the requestedTheme value in the json file. Now it works.