is get_release_date() functional?

Hi,
I'm trying this library to retrieve some information about my last.fm top albums and their release date. My code looks like
import pylast
network = pylast.LastFMNetwork(api_key = API_KEY, api_secret = API_SECRET,
        username = username, password_hash = password_hash)

user = network.get_authenticated_user()
top_albums = user.get_top_albums()
for album in top_albums:
    print album.item.get_artist()
    print album.item.get_release_date()

I can get top_albums and for each item get_artist but get_release_date seems broken.
I installed from pypi and then installed the git version.

Thanks for the report. Unfortunately Last.fm redesigned ("reimagined") their website over a year ago and also broke their API. They've been slowly adding things back, but there still remains lots of broken stuff. They've also removed a lot from their API docs page, and we don't know if they're coming back.
Anyway, pylast's get_release_date calls Last.fm's album.getInfo:
https://github.com/pylast/pylast/blob/54c4f9c093e8e51a3425ca104ac249d0303f943e/pylast/__init__.py#L1840-L1844
Checking their docs at http://www.last.fm/api/show/album.getInfo shows a sample response containing releasedate:
<album>
  <name>Believe</name>
  <artist>Cher</artist>
  <id>2026126</id>
  <mbid>61bf0388-b8a9-48f4-81d1-7eb02706dfb0</mbid>
  <url>http://www.last.fm/music/Cher/Believe</url>
  <releasedate>6 Apr 1999, 00:00</releasedate>
  <image size="small">...</image>
  <image size="medium">...</image>
  <image size="large">...</image>
  <listeners>47602</listeners>
  <playcount>212991</playcount>
  <toptags>
    <tag>
      <name>pop</name>
      <url>http://www.last.fm/tag/pop</url>
    </tag>
    ...
  </toptags>
  <tracks>
    <track rank="1">
      <name>Believe</name>
      <duration>239</duration>
      <mbid/>
      <url>http://www.last.fm/music/Cher/_/Believe</url>
      <streamable fulltrack="0">1</streamable>
      <artist>
        <name>Cher</name>
        <mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
        <url>http://www.last.fm/music/Cher</url>
      </artist>
    </track>
    ...
  </tracks>
</album>

But clicking the example URL for XML shows no releasedate:
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<lfm status="ok">
<album>
<name>Believe</name>
<artist>Cher</artist>
<mbid>63b3a8ca-26f2-4e2b-b867-647a6ec2bebd</mbid>
<url>https://www.last.fm/music/Cher/Believe</url>
<image size="small">
https://lastfm-img2.akamaized.net/i/u/34s/b0c2311a9af7f0edbc8b99450944ca1b.png
</image>
<image size="medium">
https://lastfm-img2.akamaized.net/i/u/64s/b0c2311a9af7f0edbc8b99450944ca1b.png
</image>
<image size="large">
https://lastfm-img2.akamaized.net/i/u/174s/b0c2311a9af7f0edbc8b99450944ca1b.png
</image>
<image size="extralarge">
https://lastfm-img2.akamaized.net/i/u/300x300/b0c2311a9af7f0edbc8b99450944ca1b.png
</image>
<image size="mega">
https://lastfm-img2.akamaized.net/i/u/b0c2311a9af7f0edbc8b99450944ca1b.png
</image>
<image size="">
https://lastfm-img2.akamaized.net/i/u/arQ/b0c2311a9af7f0edbc8b99450944ca1b.png
</image>
<listeners>293982</listeners>
<playcount>1821554</playcount>
<tracks>
<track rank="1">
<name>Believe</name>
<url>https://www.last.fm/music/Cher/_/Believe</url>
<duration>238</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="2">
<name>The Power</name>
<url>https://www.last.fm/music/Cher/_/The+Power</url>
<duration>236</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="3">
<name>Runaway</name>
<url>https://www.last.fm/music/Cher/_/Runaway</url>
<duration>286</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="4">
<name>All or Nothing</name>
<url>https://www.last.fm/music/Cher/_/All+or+Nothing</url>
<duration>238</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="5">
<name>Strong Enough</name>
<url>https://www.last.fm/music/Cher/_/Strong+Enough</url>
<duration>223</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="6">
<name>Dov'e L'amore</name>
<url>https://www.last.fm/music/Cher/_/Dov%27e+L%27amore</url>
<duration>258</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="7">
<name>Takin' Back My Heart</name>
<url>
https://www.last.fm/music/Cher/_/Takin%27+Back+My+Heart
</url>
<duration>272</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="8">
<name>Taxi Taxi</name>
<url>https://www.last.fm/music/Cher/_/Taxi+Taxi</url>
<duration>304</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="9">
<name>Love Is the Groove</name>
<url>
https://www.last.fm/music/Cher/_/Love+Is+the+Groove
</url>
<duration>271</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
<track rank="10">
<name>We All Sleep Alone</name>
<url>
https://www.last.fm/music/Cher/_/We+All+Sleep+Alone
</url>
<duration>208</duration>
<streamable fulltrack="0">0</streamable>
<artist>
<name>Cher</name>
<mbid>bfcc6d75-a6a5-4bc6-8282-47aec8531818</mbid>
<url>https://www.last.fm/music/Cher</url>
</artist>
</track>
</tracks>
<tags>
<tag>
<name>pop</name>
<url>https://www.last.fm/tag/pop</url>
</tag>
<tag>
<name>90s</name>
<url>https://www.last.fm/tag/90s</url>
</tag>
<tag>
<name>dance</name>
<url>https://www.last.fm/tag/dance</url>
</tag>
<tag>
<name>cher</name>
<url>https://www.last.fm/tag/cher</url>
</tag>
<tag>
<name>albums I own</name>
<url>https://www.last.fm/tag/albums+I+own</url>
</tag>
</tags>
<wiki>
<published>27 Jul 2008, 15:55</published>

Believe is the twenty-third studio album by American singer-actress Cher, released on November 10, 1998 by Warner Bros. Records. The RIAA certified it Quadruple Platinum on December 23, 1999, recognizing four million shipments in the United States; Worldwide, the album has sold more than 20 million copies, making it the biggest-selling album of her career. In 1999 the album received three Grammy Awards nominations including "Record of the Year", "Best Pop Album" and winning "Best Dance Recording" for the single "Believe". Read more on Last.fm.

<content>
Believe is the twenty-third studio album by American singer-actress Cher, released on November 10, 1998 by Warner Bros. Records. The RIAA certified it Quadruple Platinum on December 23, 1999, recognizing four million shipments in the United States; Worldwide, the album has sold more than 20 million copies, making it the biggest-selling album of her career. In 1999 the album received three Grammy Awards nominations including "Record of the Year", "Best Pop Album" and winning "Best Dance Recording" for the single "Believe". It was released by Warner Bros. Records at the end of 1998. The album was executive produced by Rob Dickens. Upon its debut, critical reception was generally positive. Believe became Cher's most commercially-successful release, reached number one and Top 10 all over the world. In the United States, the album was released on November 10, 1998, and reached number four on the Billboard 200 chart, where it was certified four times platinum. The album featured a change in Cher's music; in addition, Believe presented a vocally stronger Cher and a massive use of vocoder and Auto-Tune. In 1999, the album received 3 Grammy Awards nominations for "Record of the Year", "Best Pop Album" and winning "Best Dance Recording". Throughout 1999 and into 2000 Cher was nominated and winning many awards for the album including a Billboard Music Award for "Female Vocalist of the Year", Lifelong Contribution Awards and a Star on the Walk of Fame shared with former Sonny Bono. The boost in Cher's popularity led to a very successful Do You Believe? Tour. The album was dedicated to Sonny Bono, Cher's former husband who died earlier that year from a skiing accident. Cher also recorded a cover version of "Love Is in the Air" during early sessions for this album. Although never officially released, the song has leaked on file sharing networks. Singles "Believe" "Strong Enough" "All or Nothing" "Dov'è L'Amore" Read more on Last.fm. User-contributed text is available under the Creative Commons By-SA License; additional terms may apply.
</content>
</wiki>
</album>
</lfm>

I've reported this to Last.fm:
https://getsatisfaction.com/lastfm/topics/album-getinfo-is-missing-releasedate
But to be honest, I don't expect them to fix it anytime soon, if ever. Sorry about that.
(If pylast can be made more robust to handle errors better, PRs are welcome, but it shouldn't break Libre.fm support.)

okay, thanks for your answer.

Last.fm still haven't fixed this, so it'll be removed from the next pylast release.

Should someone be still interested, I recovered the release year of albums with the following ugly code:
import pylast
import requests
from bs4 import BeautifulSoup

def scrape_release_date(url):
    year = None
    try:
        response = requests.get(url)
        assert response.ok

        # Sample html:
        #

        #

        #
Length

        #

        #             6 tracks, 48:35
        #

        #
Release Date

        #
25 April 1975

        #

        #

        html = BeautifulSoup(response.content.decode(), features="lxml")
        block = html.find("dl", {"class": "catalogue-metadata"})
        dt = [e for e in block.find_all('dt') if e.text=='Release Date'][0]
        dl = dt.findNextSibling()

        # Sample text:
        # - 25 April 1975
        # - October 2002
        # - 1970

        #print(dl.text)
        tokens = dl.text.split(' ')
        year = int(tokens[-1])
        #print(year)
    except Exception as e:
        import ipdb; ipdb.set_trace()
        year = None

    return year

FWIW, I 'revived' a support ticket about this on Last.fm Support Forums: https://support.last.fm/t/get-album-release-date-from-album-getinfo/56159/1


B, Nt, E = q.shape ValueError: too many values to unpack (expected 3)

Hi, I ran the code using "torchrun train.py --config configs/demo.yaml"
For some reason, I got the Value error:
B, Nt, E = q.shape
ValueError: too many values to unpack (expected 3)
has anyone gotten this error? I'm using paired-image-folders to compile dataset.

Hello, we tried to solve the issue.
This is what we did:
Modified the forward method of the Attention class to handle cases where the input tensor shape is not as expected. Added a check for the input shape and reshaped the tensor if necessary.
You can review changes in this commit: https://github.com/mnmnk43434/tianrun-chen-SAM-Adapter-PyTorch-103/commit/8001893498d2131409a8eec1ac360b69b85c322e.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.

Kindly want to confirm which model are you using? For SAM2, we are using https://dl.fbaipublicfiles.com/segment_anything_2/092824/sam2.1_hiera_large.pt


simple new spline limits

Currently this does not work as intended, because there are image where during decoding in QuantizedSpline::Decode only one control point appears and this leaves the manhattan_distance zero, although the width might be very large.
Here is the image I'm testing with:
https://jxl-art.surma.technology/?zcode=rZJJTgMxEEXXrlP4Am3V5GmDBBKIFQJlwTqiG2JEOhFxNpye6g7iAuCSbJXf9y9Pz23sOy-McD-1t133WhBgc_xo8_R03s69fW17O8zX4_v51PfT3P0vB28tIvo_xerC_-Iy1D_bkISkjoNQLS6HqEhuwJAyZqc2SFGHQS2jUEuqC2RRrjUWEkMx5YJCGheF2CRp0uLQUBLJiFwN2Sot5ok5YXHkrLQJuLDlymXhZEXI6ovRIDVWUo1Mq5OkmCNj5Likl7g4ZBQ0VNdd2yGS2APlhdhuWdBcFjFQQl8ugwkAbufx51HhpvVxOtqvIIS7z-1-ejyc7G4G0wK0V__irzzB4B_WfjN1zxHhGw

rebased


list nested objects but not need a nested list

Hi
I need to make a list from a geojson data I use in open layer. Data can't be changed.
I tried in many way to do it using list.js, that looks me be perfect for my need. However I can not be able to do it.
My data is something like that:
var json_2 = [ {"properties": { "zona": "Italia", "nome": "A"} }, {"properties": { "zona": "Italia", "nome": "I" } } ]
I should like to have a list with:
Italia - A
Italia - I
Is it possible in some way without parse the data?

Thanks for this issue. I'm not the library maintainer but I don't think that the library will ever support a native format for custom JavaScript objects like zona: <something> or nome: <something> really.
Adapting data is not generally bad, expecially in these situations. I mean: you can keep the original data exactly as-is, while it's possible to just adapt the data on-the-fly into a new desired data structure, and put that as input of the library.


Problem with relative paths in train_eval.py and log variables in utils.py

Very interesting work and thanks for providing a code base with the paper.
When launching python code/train_and_eval.py on the recommended dataset, the attached error is raised. [I am running this code outside of docker but with all the needed requirements in place.]

I solved the issue by creating the features folder. Shouldn't it be specified somewhere in the readme to create an empty features folder at the right location?
After fixing that, more issues are raised due to undefined variables in code/utils.py (e.g., log_dir).

As-is, the train_eval.py script does not complete running even for a single epoch.

Thank you for reporting this problem. I will look into it as soon as possible, but the current coronavirus situation will slow down my intervention. I will notify you as soon as I commit the new version.
In the spirit of helping you get the code running as quickly as possible, I would suggest you comment line 578 of code/train_and_eval.py. Not calling log_file() should solve your issue.

With the last commits, I removed these lines since they were mostly for debugging purposes. It should run without errors now.
Thanks again for reporting the issue!
I will close it now.


No way to specify parameters to KIVY_VIDEO provider

We have a case where we're using ffpyplayer, and we want to ensure it uses a specific hardware decoder vs choosing the default h264 software decoder.
What's the best way to parameterize this information?  Should this be an issue on ffpyplayer instead?

Related ffpyplayer issue: https://github.com/matham/ffpyplayer/issues/60

I think it's too specific. I'm not against a patch that pass some information from the env to the providers, but even if we would have a way, we need to know what exactly we want to pass.
You showed what you wanted in the related issue, therefore if you provide a patch for that, i'll be happy to merge.
So easier is to have env variable working on ffpyplayer project itself IMO.
Or provide a way to parse argument in KIVY_VIDEO, and then use it to map some argument to the ffpyplayer implementation.

An environmental variable is not appropriate for ffpyplayer because the vcodec is passed into init. It would be like adding an environmental variable to control the number of rows in a grid layout.
A more appropriate solution would be to have a config options duct on the video class which each backend provider can use as needed. Then ffpyplayer can use it for the vcodec parameter. What do you think @tito? It should be easy to add!?

I think that's reasonable - the config options dict would be specific to the provider, and that would be OK considering the provider is likely to be specified as well.


Update minimum Go version in CONTRIBUTING.md doc to 1.18

Summary

Updates minimum Go version in CONTRIBUTING.md doc to 1.18
Adds Jetbrains IDE files to gitignore

Related issue(s)
Fixes #1650

Thank you!
/lgtm
/approve

/ok-to-test


Can't install sparrow wallet on parrot os

Has anyone had any luck installing sparrow on parrot? I can install it on tails but nothing on parrot.

There is record of another user running Sparrow on Parrot - perhaps provide more detail on where the install is failing.

Maybe I'm not installing it right. I have downloaded the sparrow pgp manifest.txt file
And sparrow 1.8.1 amd64.deb I tried installing it through the terminal no luck.

Maybe I'm not installing it right. I have downloaded the sparrow pgp manifest.txt file And sparrow 1.8.1 amd64.deb I tried installing it through the terminal no luck.

Once you verified the .deb installer ( see instructions on the download page) open terminal - cd where sparrow is  sudo dpkg -i (your file)

about to try this and I will get back to you. Thanks.

Maybe I'm not installing it right. I have downloaded the sparrow pgp manifest.txt file And sparrow 1.8.1 amd64.deb I tried installing it through the terminal no luck.

Once you verified the .deb installer ( see instructions on the download page) open terminal - cd where sparrow is  sudo dpkg -i (your file)

Can you expanded on what you mean by (your file)? I'm kinda new to coding so I'm barely learning how to send codes from to the terminal. I'm stuck at this part

Maybe I'm not installing it right. I have downloaded the sparrow pgp manifest.txt file And sparrow 1.8.1 amd64.deb I tried installing it through the terminal no luck.

Once you verified the .deb installer ( see instructions on the download page) open terminal - cd where sparrow is  sudo dpkg -i (your file)

Ok I got it!! Now I get this error. I'm going to restart it image

All good now! I got it installed thanks for the help guys. @craigraw you can close this topic now.


⚠ Bofrost.at has degraded performance

In 49d6f32, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 999 ms

Resolved: Bofrost.at performance has improved in 31574fa.


Opcode name fence

While working on https://github.com/lowRISC/ibex/issues/134 I just saw that we now have
https://github.com/lowRISC/ibex/blob/b203a0f7c70be7c7dbdbb8d6abb97671323e2023/rtl/ibex_defines.sv#L33
as opposed to OPCODE_MISC_MEM. I understand that we only have FENCE and not FENCE.I so this is the only time the opcode is used. Just want to check if we want to keep OPCODE_FENCE?

Good catch, our general policy is that these enums should match the spec wherever sensible to allow easy cross-referencing and comparison. So OPCODE_MISC_MEM as that major opcode is called in the current spec is more appropriate.

Thanks @asb for the clarification, I will create a PR.

Very minor: Unless I'm missing some logic behind it, the ordering in that enum seems pretty arbitrary too. I'd personally prefer it if the order reflected the table 25.1 opcode map in the RISC-V ISA manual. Again, this makes it easier to cross-check the two quickly.

Will reorder the enum, but just to make it clear, it should be sorted by keeping the higher bits stable? So row after row.

In the LLVM backend we did LOAD, LOAD_FP, MISC_MEM, ...
I don't feel strongly - anything that's logical.


Multi Lingual website

Can this plugin generate sitemap for a multiple lingual website? If so, how?

Hi @xiki808 this is not possible for now. I will work on it as soon as possible. If you need this feature quickly, feel free too PR with an i18n config. You can find some inspiration here => sitemap-ts

@xiki808 you can now use v1.7.1 using i18n config!

Actually I'm not sure how I managed to build the first few times as I'm always getting the following error now:
TypeError: [vite-plugin-pages] Could not load virtual:vite-plugin-pages/generated-pages?id=~pages (imported by node_modules/@horizon/frontend/src/lib/router.js): Invalid URL
I guess it's some conflict as I'm actually importing import routes from '~pages' inside a custom NPM package that I have which provides basic components and initiates things such as the router itself

Well, i think you can use dynamicRoutes to load your sitemap.
onRoutesGenerated: async (routes) => {
    const languages = ['en', 'it']
    const dynamicRoutes = languages.map(lang => `/${lang}/test`)
    generateSitemap({ routes: [...routes, ...dynamicRoutes] })
},

I think your TypeError is not depending on this plugin but directly vite-plugin-pages. You can maybe add "skipLibCheck": true in your tsconfig.json or add --skipLibCheck in your tsc command.

Thanks will try it out with the dynamicRoutes.
As for the TypeError, indeed that's what the error is pointing to, but if I just comment out the i18n config line  the projects builds and I also get all the routes.

Yes sure, feel free to PR a solution for your situation. I will add this to my roadmap but will not get any time for now to develop this feature asap.

As for the TypeError, that's was a silly mistake from my side.. was not returning the routes, vite-plugin-pages's onRoutesGenerated function

As for the TypeError, the issue was not returning an asynced function to vite-plugin-pages's onRoutesGenerated function


[hmac,doc] Align documentation with HW changes

This implements minor fixes to the HMAC documentation: minor wording edits and alignment with changes to the IP and its SW usage.
This should close https://github.com/lowRISC/opentitan/issues/23455 once merged.

Thanks @ballifatih and @andreaskurth for all your comments, I think I addressed them all in my last force push!


Is CI borked

What was wrong?
How was it fixed?
To-Do

[ ] OPTIONAL: Update Spec if applicable

[ ] Add entry to the release notes (may forgo for trivial changes)

[ ] Clean up commit history

Codecov Report

Merging #446 (84cabfa) into master (f399aeb) will decrease coverage by 85.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #446       +/-   ##
==========================================
- Coverage   86.47%   1.47%   -85.01%
==========================================
  Files          71      21       -50
  Lines        4859    1493     -3366
==========================================
- Hits         4202      22     -4180
- Misses        657    1471      +814

Impacted Files
Coverage Δ

common/src/utils/keccak.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

compiler/src/lowering/utils.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

analyzer/src/namespace/scopes.rs
0.00% <0.00%> (-95.48%)
:arrow_down:

analyzer/src/namespace/events.rs
0.00% <0.00%> (-94.74%)
:arrow_down:

common/src/utils/humanize.rs
0.00% <0.00%> (-93.75%)
:arrow_down:

analyzer/src/context.rs
0.00% <0.00%> (-91.50%)
:arrow_down:

parser/src/lexer.rs
0.00% <0.00%> (-86.21%)
:arrow_down:

compiler/src/abi/elements.rs
0.00% <0.00%> (-84.62%)
:arrow_down:

test-utils/src/lib.rs
0.00% <0.00%> (-83.81%)
:arrow_down:

analyzer/src/namespace/types.rs
0.00% <0.00%> (-83.07%)
:arrow_down:

... and 61 more

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 f399aeb...84cabfa. Read the comment docs.

I will merge this without review as soon as it goes green since it is so minor and fixes the build


Missing latest charges

Describe the bug
I'm missing the latest charges.
The charges endpoint limits the number of results. In my case it returns the first 28 objects starting from 1970-01-01. Always querying with start=1970-01-01 seems not to be the best solution.
To Reproduce
Steps to reproduce the behavior:

Have a car with > 28 charges
Click on objects view in ioBroker
Select charges folder
Do not see latest charges

Expected behavior
See all, or at least the last x charges.
Versions:

Adapter version: 0.0.8
JS-Controller version: 3.3.22
Node version: v14.18.2
Operating system: Docker image buanet/iobroker:v6.0.0 on raspbian 64Bit (debian 11 based)

Additional context
I would be happy with a configuration option to set the number of past days for which the charges are then retrieved. start could then easily be curDate - number of configured days.

It would be more elaborate but not as efficient to first query the number of charges with charge-history in order to determine suitable intervals and then call up the charges accordingly.
Incidentally, charge-history can also be called with type=month&start=1970-01&end=2022-01 (end must not be in the future) to narrow down the intervals better.

Can you also test the endpoint without parameter
/commerce/v1/accounts/{account_id}/kamereon/kca/car-adapter/v1/cars/{vin}/charges

No parameters results in http 400 with
{
    "type": "FUNCTIONAL",
    "messages": [
        {
            "code": "err.func.wired.missingRequestParam",
            "message": "must be present",
            "parameters": {
                "type": "Country",
                "name": "country"
            },
            "propertyPath": "country"
        }
    ],
    "errors": [
        {
            "errorCode": "err.func.wired.missingRequestParam",
            "errorMessage": "must be present"
        }
    ],
    "error_reference": "FUNCTIONAL"
}

With country but without start and end results in http 400 with
{
    "type": "FUNCTIONAL",
    "messages": [
        {
            "code": "err.func.400",
            "message": "{\"errors\":[{\"status\":\"Bad Request\",\"code\":\"400\",\"title\":\"Required LocalDate parameter 'start' is not present\",\"detail\":\"start is missing\"}]}"
        }
    ],
    "errors": [
        {
            "errorCode": "err.func.400",
            "errorMessage": "{\"errors\":[{\"status\":\"Bad Request\",\"code\":\"400\",\"title\":\"Required LocalDate parameter 'start' is not present\",\"detail\":\"start is missing\"}]}"
        }
    ],
    "error_reference": "FUNCTIONAL"
}

Ok then will I add a option to receive the last xx days default value 14


Update DG for DeleteTags and some minor changes

Closes #258
Closes #254

Thanks @yanghengtang for pointing out. Fixed.


NSCache implementation missing

NSCache is currently unimplemented.

If its okay, I'll go ahead with the implementation.
R

@RG-J Thanks so much for offering.  Will let you know by end of day.

@RG-J this is urgently required to unblock a customer, so we will do this one internally.  If you are interested, i could tag you on a few issues where we can use your help.  Thanks again for your contributions!

Since the reference documentation imposes no restrictions on the backing store for NSCache, apart from that it bear dictionary semantics, we have a bit of agency in how we implement it. I intend to start with a simple LRU cache.
Details
Using a doubly-linked list (for quick element replacement and removal) and a hash table (for quick lookups) seems like it will be the right choice here. Since we're a primarily C++-on-the-inside codebase, we'll use std::list<_NSCacheEntry> and std::unordered_map<id, std::list<_NSCacheEntry>::iterator>, with specializations for std::hash<id> and std::equal_to<id>.
std::list was chosen for its stable iterators and efficient reverse traversal. We'll keep the cost on _NSCacheEntry, even if there is no cost.
Operations
Insertion - O(1) amortized

Prepend a new cache entry {key, value, cost} to the backing list.
Insert a new entry mapping key to the iterator hash table.
Evict all objects over the cache limit -- including, possibly, the newly-added one.

Removal - O(1) amortized

Find the iterator corresponding to key.
Remove the iterator from the backing list.
Remove the found iterator from the iterator hash table.

Lookup - O(1)

Find the iterator corresponding to key.
Promote it to the head of the backing list.

Eviction/Cost Balancing - O(n) worst case

Starting at the end of the backing list, remove items.

If we have a cost limit, only remove items with a nonzero cost.
If we have a count limit, remove items regardless of their costs.
If we have both, remove items following the above rules until both limits are met.

Eviction is the only O(n) operation for the LRU cache: in the worst case, we must remove every item from the backing list. With sane defaults, we shouldn't hit the pathological case.
Compatibility
NSDiscardableContent objects will be discarded during eviction, but only removed from the cache if evictsObjectsWithDiscardedContent is enabled. If they are not evicted, they will be considered to have 0 cost.


Catch awkward typos

e.g. in the route definition:
          -
            beta.example.com:
              tls-amce: 'false'

We should just check the annotations against a known list using https://github.com/agnivade/levenshtein. Would also need to check against a corpus of existing .lagoon.yml to determine how breaking such a change would be.


[Bug]: Receiving error message "User is not authorised to perform: securityhub:BatchImportFindings"

Hi @toniblyx,
Please can you help me with the following:
1 - I am running a Prowler scan locally, using the following script (image 1) in one account, across 5 different regions.
2 - The Prowler scan for all regions, except for us-east-1 runs smoothly. However, with us-east-1 I get the attached error message (image 2) for many findings.
3 - Looking in AWS CloudTrail I also see the following (image 3) for BatchImportFindings event types.
4 - Whether I aggregate Security Hub findings in us-east-1 or one of the other regions, I still receive these errors in the us-east-1 region.
5 - Security Hub is enabled in all of the regions I am using and I am using a full admin role with all of the necessary permissions.
I am running Prowler from:

Resource: Workstation
OS: Mac, M1 Chip
AWS-CLI Version: aws-cli/2.7.21
Prowler Version: Prowler 2.11.0-21July2022

Please can you help me to understand what the issue is and how I can resolve this?
Many thanks,
Maeve
Images:
Image 1:

Please note that 'region-1' etc are placeholders for the other regions that I'm using.
Image 2:

Image 3:

Hi @MaeveScarry, thank you for reaching out! Please make sure that you don't have any SCP blocking that region from sending findings to SecurityHub, if that is not the case, could you send us the logs by running prowler with bash -x ./prowler ... in the us-east-1 region? thanks.

Hi @sergargar - massive apologies for my delayed response.
Please see the attached images of the logs. I've had to redact sensitive information. There were lots of logs, but I've chosen to send a few that I think would be most helpful.
I've also checked and there are no SCPs blocking the region.
Many thanks

Hi @maevescarrymp, thank you for the logs. It seems that you are receiving the findings in Security-Hub (us-east-1) despite the errors, right?
I think that is failing when archiving previous fails in SecurityHub that are no longer fails. Could you test it by removing first all SHub findings in us-east-1 region?

Hi @MaeveScarry, feel free to reopen this issue if you still have problems related to it. Thank you!


More minor edits to docs

Don't use _ and Out[]
Clarifying points.
Listing all mark_ methods and aggregations.

Fixes #142 and #143

Now rebased on top of work of @willingc !


Missing Fonts when showing Chinese and Japanese Characters

Randomly testing some page and found Chinese and Japanese Characters cannot show properly

This should be a matter of installing fonts that support Chinese and Japanese characters to the Linux system running browservice. On my Ubuntu 20.04 testing VM, I was able to reproduce the issue, and running
sudo apt-get install font-noto-cjk

made the characters visible. This only installs Google Noto fonts; you might want to install other font packages as well.

Thanks,it works


compatibility issue with plugin yankring

vim-scripts/YankRing.vim is a plugin to manage clipboard within neovim. it works correctly with neovim and some GUI's like fvim, neoray and neovide.
Windows 10. Neovim 0.10.1. goneovim 0.6.8.
I think maybe b/c of goneovim's clipboard feature. does not help with "Clipboard = false" in setting.toml.
this error will lead to errors in other plugins, such as deoplete, which works fine without yankring.

commenting out these two links in my init.vim does not help,
set clipboard=unnamed "win32yank available
set clipboard+=unnamedplus

part of the code of the plugin,
function! s:YRCheckClipboard()
    if g:yankring_clipboard_monitor == 1
        " If the clipboard has changed record it inside the yankring
        " echomsg "YRCheckClipboard[".len(@*)."][".@*.']['.s:yr_prev_clipboard_star.']'
        if has('unnamedplus') && &clipboard =~ '\<unnamedplus\>'
            if !empty(@+) && @+ != s:yr_prev_clipboard_plus
                let elem    = s:YRMRUElemFormat(
                            \   getreg('+')
                            \ , getregtype('+')
                            \ )
                let found   = s:YRMRUHas('s:yr_history_list', elem)

                " Only add the item to the "top" of the ring if it is
                " not in the ring already.
                if found == -1
                    call YRRecord3("+")
                endif
                let s:yr_prev_clipboard_plus = @+
            endif
        else
            if !empty(@*) && @* != s:yr_prev_clipboard_star
                let elem    = s:YRMRUElemFormat(
                            \   getreg('*')
                            \ , getregtype('*')
                            \ )
                let found   = s:YRMRUHas('s:yr_history_list', elem)

                " Only add the item to the "top" of the ring if it is
                " not in the ring already.
                if found == -1
                    call YRRecord3("*")
                endif
                let s:yr_prev_clipboard_star = @*
            endif
        endif
    endif
endfunction

@bon-ami
Thank you for reporting the issue.
I have fixed the initialization process code, which I believe is related to the problem you reported.
You can obtain the updated binary from the link below, so could you please try it out?
https://github.com/akiyosi/goneovim/actions/runs/10966376589

@bon-ami Thank you for reporting the issue. I have fixed the initialization process code, which I believe is related to the problem you reported. You can obtain the updated binary from the link below, so could you please try it out?
https://github.com/akiyosi/goneovim/actions/runs/10966376589

I confirm it's solved. Thanks.


How to build DataFusion python wheel

The latest python datafusion release 0.2.0 (from PyPi) was Dec 9, 2020. The DataFusion 4.0.0 version released but no official DataFusion release for Python. I am trying to build a wheel from the source but no setup.py or instructions how to build a wheel package. Please, can someone provide the steps or instructions how to build a wheel package? Thanks.

We will hopefully be able to release 0.3.0 this week. In the mean time you can follow the instructions at https://github.com/apache/arrow-datafusion/tree/master/python#how-to-develop. maturin is the command you want to use. maturin develop to build and install the binding into local python virtualenv. maturin release to build a release wheel that can be published to pypi index.


GitHub chart-releaser action failed

Description
The chart-releaser action failed - I believe this was because some of the releases already exist.
The error was:

Some of the releases were being created, but the index.yaml file in the GitHub pages branch was not being updated.
I believe a fix is to set the skip_existing option to true in the GitHub action workflow.
I temporarily fixed by manually deleting the release causing the issue and re-running the GitHub action job.

This was an error in recreating a chart that had not been changed.


If there is an initialTestResultReceivedTimestamp  it may be the bug.

If a user removed a test prior to 1.11, they may have an old timestamp that should have been deleted, this tries to detect that and delete the timestamp.
If there is also a registration value, but no devicePairingSuccessfulAt timestamp, then we are being called from asyncRegisterDeviceViaGUID or asyncRegisterDeviceViaTAN, and it's a new test and initialTestResultReceivedTimestamp should be null.
Also see #2637

Called from 3 places:
    private suspend fun fetchTestResult(registrationToken: String): DeviceUIState = try {
        val testResult = submissionService.asyncRequestTestResult(registrationToken)
        updateTestResult(testResult)
        deriveUiState(testResult)
    } catch (err: NoRegistrationTokenSetException) {
        DeviceUIState.UNPAIRED
    }

    suspend fun asyncRegisterDeviceViaGUID(guid: String): TestResult {
        analyticsKeySubmissionCollector.reset()
        val registrationData = submissionService.asyncRegisterDeviceViaGUID(guid)
        submissionSettings.registrationToken.update {
            registrationData.registrationToken
        }
        updateTestResult(registrationData.testResult)
        submissionSettings.devicePairingSuccessfulAt = timeStamper.nowUTC
        backgroundNoise.scheduleDummyPattern()
        analyticsKeySubmissionCollector.reportTestRegistered()
        return registrationData.testResult
    }

    suspend fun asyncRegisterDeviceViaTAN(tan: String) {
        analyticsKeySubmissionCollector.reset()
        val registrationData = submissionService.asyncRegisterDeviceViaTAN(tan)
        submissionSettings.registrationToken.update {
            registrationData.registrationToken
        }
        updateTestResult(registrationData.testResult)
        submissionSettings.devicePairingSuccessfulAt = timeStamper.nowUTC
        backgroundNoise.scheduleDummyPattern()
        analyticsKeySubmissionCollector.reportTestRegistered()
        analyticsKeySubmissionCollector.reportRegisteredWithTeleTAN()
    }

        if (
            // We have a timestamp, but is it the bug or a legitimate timestamp
            initialTestResultReceivedTimestamp != null &&
            // This get's set directly before we are being called from  asyncRegisterDeviceViaGUID or asyncRegisterDeviceViaTAN
            submissionSettings.registrationToken.value != null &&
            // Now we need to make sure that we are not being called as part of a fetchTestResult
            // and this value gets set after we are called
            submissionSettings.devicePairingSuccessfulAt == null
            // All of these values are nulled when a test is removed on 1.15
        ) {
            Timber.tag(TAG).w("User has stale initialTestResultReceivedAt, fixing EXPOSUREAPP-4484.")
            submissionSettings.initialTestResultReceivedAt = null
        }


Failed to create terminal for task

Paste the tasks from #70774 into the vscode repo's tasks.json file
Start the watch task
Kill the terminal before the build is totally done
Get an error popup Failed to create terminal for task

A bad merge deleted the fix for the original issue 😓

duplicate of #70557

Actually not a bad merge. Fixing again.


Seems bot to work on IE11 or Edge

not

It can be fixed with mixed results, for example rc.path did not work, no Path2D support?
// I was able to add or install dev dependencies babel-cli, ... listed below and then the babel-polyfill
 "devDependencies":
    "babel-cli": "^6.26.0",
    "babel-preset-es2015": "^6.24.1",
    "babel-preset-es2017": "^6.24.1",
    "uglify-js": "^3.3.16"

  "dependencies":
    "babel-polyfill": "^6.26.0"

And you need a suitable babelrc
{
  "presets": [ "es2017", "es2015" ]
}

And then a way to run babel and uglify, for example I added to package.json
    "babel-es5": "babel dist/rough.js -o dist/rough.es5.js",
    "minify-u": "uglifyjs dist/rough.es5.js -o dist/rough.min.js",

It can be fixed with mixed results, for example rc.path did not work, no Path2D support?

IE11 does not support Path2D. Edge does, but there is an issue with SVG path data.
https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8438884/

Path2D is only used when filling with hachure styles in paths. Not in issue for other shape constructs.
I want to move away from Path2D anyway because it will solve some other minor issues as well. Working on some good algorithms that may work as an alternative

The build now includes an ES5 version.

Cool

I tried the ES5 version and it does not work on IE11.
Not when the keyword async is present. It's a ES2017 keyword.

@Nalca you're right
I need to run this through babel to fix those. May need a Promise polyfill as well.
PRs welcome. Not sure when I'll get a chance.

I used the babel-polyfill to fix the missing "async" after running into it. Simple add to my project build.

Removed Path2D dependency.


DOC: add to README so example works under Python 3.6

What does this PR implement?
It adds decode_responses=True in the example in README.rst as mentioned in https://github.com/RedisLabs/rejson-py/issues/8#issuecomment-321514267 (which took me too long to find after copy/pasting the example with Python 3.6.5 and rejson-py 0.3.0).

Coverage remained the same at 100.0% when pulling cd0f92c5ee7ea7d91ff8d15f3c5d1e3d8f4b0779 on stsievert:patch-1 into ba4adf5fdcaa63bf27e2ed43baf2fd81a39eb665 on RedisLabs:master.

I am another victim of this minor detail.... please review and merge maintainers.


BulkInsertOrUpdate fails with Primary Key as Int default value (0)

Hi,
I got an issue when I'd like to insert my entities to my Database.
I'm using EFCore (SqlServer) 3.1.2 & EFCore.BulkExtensions 3.1.0
Here is the entity :
public class BCardEntity : IIntEntity
{
        [Key]
        [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
        public int Id { get; set; }

        public byte SubType { get; set; }

        public byte Type { get; set; }
}

As you can see, I did setup the [Key] attribute and the DatabaseGenerated attribute to Identity in order to have an identity on the primary key.
Here is the "InsertOrUpdate" code I implemented :
        public async Task<IEnumerable<TDto>> SaveAsync(IEnumerable<TDto> objs)
        {
            try
            {
                await using DbContext context = _contextFactory.CreateContext();
                IEnumerable<TEntity> entities = _mapper.Map(objs);

                List<TEntity> entitiesList = entities.ToList();
                await using (IDbContextTransaction transaction = await context.Database.BeginTransactionAsync())
                {
                    try
                    {
                        await context.BulkInsertOrUpdateAsync(entitiesList, new BulkConfig()
                        {
                            PreserveInsertOrder = true,
                            SetOutputIdentity = true,
                            SqlBulkCopyOptions = SqlBulkCopyOptions.KeepIdentity | SqlBulkCopyOptions.CheckConstraints
                        });
                        await transaction.CommitAsync();
                    }
                    catch (Exception e)
                    {
                        Log.Error("SaveAsync.Transaction", e);
                        await transaction.RollbackAsync();
                    }
                }

                await context.SaveChangesAsync();
                return _mapper.Map(entitiesList);
            }
            catch (Exception e)
            {
                Log.Error("SaveAsync", e);
                return Array.Empty<TDto>();
                // log exception
            }
        }

However, here is the exception being thrown :

It happens to all my entities that are inserted with its Pk at 0.
Thanks by advance,
Blowa.

Fixed, I was trying since several hours on the wrong implementation.


Development

Primeira versão estável, as implementações até aqui foram monitoradas e não apresentaram nenhum erro!

Pull Request Test Coverage Report for Build 476

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

Totals

Change from base Build 474:
0.0%

Covered Lines:
1290

Relevant Lines:
1354

💛  - Coveralls


Add new release message

Maybe also add the 'spell checking scripts'. For example with: Improve the quality of documentation

@Prasanna-Ponnusamy you need to add an approving review to merge this review ;)


UI Unit Tests via CSS: Automatically show errors in code

The basic premise is based on the article UI Unit Tests via CSS: Automatically catch errors in your code, in production.
It would be useful to include and work upon a tests.css stylesheet that is included in development environments and automatically catches and reports errors via the UI.

@aleemb isn't that basically just Bootlint?

Neat idea, but not something we'll put in v3 and maybe not even v4 given our Bootlint bot as @kkirsche mentions. We'll keep it in mind though!

@kkirsche wasn't aware of Bootlint, makes sense and fits the bill. I suppose this is a different approach to tackle the same problem with the benefit that it can run on a dynamic and growing dataset, runs in the client's context and requires no further tooling after initial setup except updating the CSS.

xamdo93qwerty
Sent from my iPhone

On 05-Aug-2015, at 2:06 pm, aleemb<EMAIL_ADDRESS>wrote:
The basic premise is based on the article UI Unit Tests via CSS: Automatically catch errors in your code, in production.
It would be useful to include and work upon a tests.css stylesheet that is included in development environments and automatically catches and reports errors via the UI.
—
Reply to this email directly or view it on GitHub.


[crypto] Read/Write files as small chunks

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change (select or add applied and delete the others)

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] API change with a documentation update
[ ] Additional test coverage
[ ] Code cleanup or just sync with upstream third-party crates

How has this been tested?
CI: http://ci.mesalock-linux.org/qinkunbao/incubator-teaclave/66
Checklist

[ ] Fork the repo and create your branch from master.
[ ] If you've added code that should be tested, add tests.
[ ] If you've changed APIs, update the documentation.
[ ] Ensure the tests pass (see CI results).
[ ] Make sure your code lints/format.

CI passed: http://ci.mesalock-linux.org/qinkunbao/incubator-teaclave/66


Webpart provisioning for newly provisioned List Instance fails

Upon including a webpart in the PnP:File or PnP:Page, if the List is a newly provisioned List from the collection of Instances and not an OOTB list provisioned via a Feature I receive the following standard error "The page you selected contains a list that does not exist. It may have been deleted by another user."    If i run the ApplyTemplate a 2nd time the webpart is added correctly.  This is only on the first application of the template that it fails.  I've added a post event process not an ExtensibilityHandler.   The webpart addition during a ExtensibilityHandler would also fail with the same message.   The TokenParser Web Context appears to contain the existing provisioned web context which could not find the "new" List Instance.

Updating comment to focus on the schema versus the provisioning engine functionality.    Is there a portion of the schema which will request "a-new" the collection of Tokens once the object handlers have processed a specific portion of the work?

This issue was moved to SharePoint/PnP-Sites-Core#1483


python312Packages.pysml: 0.1.3 -> 0.1.4

Diff: https://github.com/mtdcr/pysml/compare/refs/tags/0.1.3...0.1.4

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 347299 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.edl21
python311Packages.pysml
python311Packages.pysml.dist
python312Packages.pysml
python312Packages.pysml.dist


About the epoch of training

How many epochs have been trained for the model in this paper?
Thanks~

As configured in train_adaattn.sh, there are 2 epochs with full learning rate and another 3 with learning rate decay.


Build failed

command:  RUSTFLAGS="-C link-dead-code" cargo test -vv --no-run
$ rustc --version
rustc 1.25.0-nightly (da569fa9d 2018-01-16)
$ cargo --version
cargo 0.25.0-nightly (a88fbace4 2017-12-29)

log
error[E0275]: overflow evaluating the requirement `Inner: std::panic::RefUnwindSafe`
  |
  = help: consider adding a `#![recursion_limit="128"]` attribute to your crate
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `std::marker::PhantomData<ScheduledTimer>`
  = note: required because it appears within the type `arc_list::ArcList<ScheduledTimer>`
  = note: required because it appears within the type `Inner`
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `std::marker::PhantomData<ScheduledTimer>`
  = note: required because it appears within the type `arc_list::ArcList<ScheduledTimer>`
  = note: required because it appears within the type `Inner`
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `std::marker::PhantomData<ScheduledTimer>`
  = note: required because it appears within the type `arc_list::ArcList<ScheduledTimer>`
  = note: required because it appears within the type `Inner`
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `std::marker::PhantomData<ScheduledTimer>`
  = note: required because it appears within the type `arc_list::ArcList<ScheduledTimer>`
  = note: required because it appears within the type `Inner`
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `std::marker::PhantomData<ScheduledTimer>`
  = note: required because it appears within the type `arc_list::ArcList<ScheduledTimer>`
  = note: required because it appears within the type `Inner`
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `std::marker::PhantomData<ScheduledTimer>`
  = note: required because it appears within the type `arc_list::ArcList<ScheduledTimer>`
  = note: required because it appears within the type `Inner`
  = note: required because it appears within the type `alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `*const alloc::arc::ArcInner<Inner>`
  = note: required because it appears within the type `core::nonzero::NonZero<*const alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::ptr::Shared<alloc::arc::ArcInner<Inner>>`
  = note: required because it appears within the type `std::sync::Weak<Inner>`
  = note: required because it appears within the type `ScheduledTimer`
  = note: required because it appears within the type `arc_list::Node<ScheduledTimer>`
  = note: required because of the requirements on the impl of `std::panic::UnwindSafe` for `std::sync::Arc<arc_list::Node<ScheduledTimer>>`
  = note: required because it appears within the type `std::option::Option<std::sync::Arc<arc_list::Node<ScheduledTimer>>>`
  = note: required because it appears within the type `sleep::Sleep`

error: aborting due to previous error

error: Could not compile `futures-timer`.

Caused by:
  process didn't exit successfully: `rustc --crate-name futures_timer /Users/kam/.cargo/git/checkouts/futures-timer-6c8164b2e0647973/9715aa6/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=8fdf150a955f4a2c -C extra-filename=-8fdf150a955f4a2c --out-dir /Users/kam/Workspace/RustProjects/cswitch/target/debug/deps -L dependency=/Users/kam/Workspace/RustProjects/cswitch/target/debug/deps --extern futures=/Users/kam/Workspace/RustProjects/cswitch/target/debug/deps/libfutures-bcd0e102b4818b5f.rlib --cap-lints warn -C link-dead-code` (exit code: 101)

I've got this too when running my project against tarpaulin, the coverage tool. More specifically, on the xd009642/tarpaulin:develop-nightly Docker image in GitLab CI.
I have no clue whether this statement may help you find what the problem is, but don't hesitate to ask me for more data... if you remotely know where I should look.

@rubdos oh this is unfortunately not an issue in this crate I believe but rather an upstream rust-lang/rust issue. I think the closest issue is https://github.com/rust-lang/rust/issues/45629, so I'm going to close this in favor of that.


Expose Integration Flow Endpoint as API and Test the Flow

Tutorials: https://developers.sap.com/tutorials/cp-starter-isuite-api-management.html
On Step 2: Define the API
At 11. Choose RequestProductDetails and then the Resources tab.
the first line should be: choose Edit > Edit in API Designer.
( otherwise, the green POST button is not visible )

Hi @FernanDoylet
Thanks for noticing this. I'm looking into this feedback.

This is in Step 2 @Karunaharan
Right after "11. Choose RequestProductDetails and then the Resources tab."
please add "Choose edit > Edit in API Designer"
because the previous 'Save' hides the green POST button shown on the graphic.


Overlapping but not on screenshot

it 'require accept terms and can download it' do
    user = c :user, terms_agreed_at: nil
    login user
    screenshot_and_open_image
    page.should have_content 'Terms of Use'
    click_on 'Terms of Use'
    page.response_headers['Content-Type'].should == "application/pdf"
end

Failures:

  1) Users require accept terms and can download it
     Failure/Error: click_on 'Terms of Use'
     Capybara::Poltergeist::MouseEventFailed:
       Firing a click at co-ordinates [343, 731.5] failed. Poltergeist detected another element with CSS selector 'html.js.no-flexbox.flexboxlegacy.canvas.canvastext.no-webgl.touch.no-geolocation.postmessage.websqldatabase.no-indexeddb.hashchange.history.draganddrop.websockets.rgba.hsla.multiplebgs.backgroundsize.borderimage.borderradius.boxshadow.textshadow.opacity.cssanimations.csscolumns.cssgradients.cssreflections.csstransforms.no-csstransforms3d.csstransitions.fontface.generatedcontent.no-video.no-audio.localstorage.sessionstorage.webworkers.applicationcache.svg.inlinesvg.smil.svgclippaths body.users section#cookie-policy div.inner p' at this position. It may be overlapping the element you are trying to interact with. If you don't care about overlapping elements, try using node.trigger('click').
     # ./spec/requests/users_spec.rb:24:in `block (2 levels) in <top (required)>'

http://i.imgur.com/LFB1COB.png
If I disable cookie-policy template test passes
http://i.imgur.com/aTI9O4e.png
Then I use trigger(:click)
it 'require accept terms and can download it' do
    user = c :user, terms_agreed_at: nil
    login user
    screenshot_and_open_image
    page.should have_content 'Terms of Use'
    page.find('label[for=user_terms_agreed_at] a').trigger(:click)
    # click_on 'Terms of Use'
    page.response_headers['Content-Type'].should == "application/pdf"
  end

Fails with
Failures:

  1) Users require accept terms and can download it
     Failure/Error: page.response_headers['Content-Type'].should == "application/pdf"
       expected: "application/pdf"
            got: nil (using ==)
     # ./spec/requests/users_spec.rb:25:in `block (2 levels) in <top (required)>'

Finished in 4.57 seconds
1 example, 1 failure

@yaauie Please refer this https://github.com/goodwill/capybara-select2/issues/14

Closing this until the info requested by @yaaule is provided

minimal, complete reproduction steps (e.g., remove your application logic & give the minimal HTML/CSS/JavaScript & a failing test case I can run without any knowledge of your application stack)
expected behaviour
observed behaviour

If that is provided we can reopen

Having the same issue:
Here's a screenshot, that poltergeist provides me with. There's a dropdown with actions, and it cannot scroll to the appropriate position and click it.


Fix FPGA DFL Release/Assign ioctl numbers

Using now correct IOCTLs for upstream DFL drivers.

Tested on the host with Arria10 card
/lgtm


Add Pareas

For fun I've added the GPU-accelerated compiler which I wrote for my thesis. The language it compiles is very simple, but is just about enough for this benchmark, and I thought it would be interesting to compare.
Here are some results with some compilers that I had laying around with ./benchmark --function-count=200 --function-depth=200 --run-count=1 --languages=pareas,c:tcc,Zig,rust,D:dmd --ops=Check,Build. CPU is a Ryzen R7 3700X, GPU is a Radeon RX 580X (using the Futhark OpenCL backend), OS is Void Linux with kernel 5.16.

Lang-uage
Temp-lated
Check Time [us/fn]
Compile Time [us/fn]
Build Time [us/fn]
Run Time [us/fn]
Check RSS [kB/fn]
Build RSS [kB/fn]
Exec Version
Exec Path

D
No
9.4 (3.4x)
N/A
N/A
N/A
4.7 (10.2x)
10.3 (22.4x)
v2.098.0
dmd

D
Yes
17.7 (6.3x)
N/A
N/A
N/A
13.4 (29.1x)
19.8 (43.1x)
v2.098.0
dmd

C
No
2.8 (best)
N/A
2.9 (best)
15 (best)
0.5 (best)
0.5 (best)
0.9.27
tcc

Zig
No
59.8 (21.4x)
N/A
252.7 (87.2x)
111 (7.3x)
21.9 (47.6x)
35.1 (76.5x)
0.10.0-dev.1068+ca97caab8
zig

Zig
Yes
72.9 (26.1x)
N/A
266.0 (91.8x)
109 (7.2x)
29.9 (64.9x)
41.0 (89.4x)
0.10.0-dev.1068+ca97caab8
zig

Rust
No
101.6 (36.3x)
N/A
296.8 (102.4x)
1544 (102.2x)
15.1 (32.9x)
33.3 (72.5x)
1.61.0-nightly
rustc

Rust
Yes
115.8 (41.4x)
N/A
195.2 (67.3x)
1551 (102.6x)
17.7 (38.5x)
21.8 (47.6x)
1.61.0-nightly
rustc

Pareas
No
13.2 (4.7x)
N/A
433.8 (149.6x)
N/A
5.3 (11.5x)
5.4 (11.8x)
unknown
pareas

(Note that dmd couldn't compile some files).
My particular interest here is the Check time, which is the specific part that I worked on in this compiler. In the above example, generated with the same parameters as in the project readme, it gets relatively close but still seems to suffer from some overhead. The story changes when we increase the size of the source files (and disable codegen):
| Lang-uage | Temp-lated | Check Time [us/fn] | Compile Time [us/fn] | Build Time [us/fn] | Run Time [us/fn] | Check RSS [kB/fn] | Build RSS [kB/fn] | Exec Version | Exec Path |
| :-------: | ---------- | :----------------: | :------------------: | :----------------: | :--------------: | :---------------: | :---------------: | :----------: | :-------: |
| C         | No         |    7.8 (3.3x)      | N/A                  | N/A                | N/A              |    0.5 (1.6x)     | N/A               | 0.9.27       | tcc       |
| Pareas    | No         |    2.4 (best)      | N/A                  | N/A                | N/A              |    0.3 (best)     | N/A               | unknown      | pareas    |

(./benchmark --function-count=1000 --function-depth=1000 --run-count=1 --languages=C:tcc,Pareas --ops=Check)
Note that pareas needs to be at least version ead2de65619a0706ffc98d1afcc77a57bb9a66f6. Furthermore, the compiler suffers from a large startup overhead because the runtime has to compile the OpenCL/CUDA kernels. For this reason I've added a caching system to my compiler, such that the first run of the compiler on a particular system saves the compiled kernels for later use. I believe this is fair since its a one-time setup cost which does not depend on the program to be compiled. For reference, compiling those kernels takes about 7 seconds on my system.

I'm gonna trust you on this one. :)
Thanks!


Support dot syntax

Session.send(request)
    .success { response in
        // Type of `response` is `[Repository]`,
        print(response)
    }
    .failure { error in
        self.printError(error)
    }
}

It can remove a lot of switch ... case...,I think this syntax is much more clarity.

Anyway, i like this open proj, it's amazing.

Hi @alfredcc,
Thank you for the suggestion!
Result<T, E> lets call-site handle both success and failure cases, because switch statement must have exhaustive cases. That's why I chose Result<T, E> and switch statement rather than dot syntax.


Update metadata handling of application packages in resource catalogue

We need to make sure we use the latest pygeometa version for this to work.

This commit is needed: https://github.com/geopython/pygeometa/commit/129c53316911d1b2e22da56b5f75ddb8ccf56369

It is now safe to merge, pygeometa just released 0.10.0
https://github.com/geopython/pygeometa/releases/tag/0.10.0


dev-cmd/unbottled: improve output.

Handle a few more edge cases beyond https://github.com/Homebrew/brew/pull/10110:

improve the unbottled output heading so it makes sense when we're outputting bottles that are ready to bottle
output already bottled formulae if named
handle Linux/Mac requiring formulae and output if named
output bottle unneeded/disabled formulae if named
use green/yellow/red for bottles ready to be built now/later/never

Review period will end on 2020-12-25 at 14:08:16 UTC.

Looks good, thanks @MikeMcQuaid

Review period ended.


Example for Uno WiFi

Hi! Thank you so much for this amazing library! I am trying to use it with the UNO R4 WiFi, and I wonder: should I use the Uno version of the code as you point out? In that case, there is only an Ethernet-based example. Could you tell me how should I use this with WiFi too? Or do you think that with the larger memory of the R4 I would be good using publishers? Thank you so much!

For Uno R4, please use OscWifi.ino.

Thank you!!


test37_noninteger_stride fails with numpy<1.23

Problem description
This test could be skipped for numpy<1.23. It fails with:
    @needs_numpy
    def test37_noninteger_stride():
        a = np.array([[1, 2, 3, 4, 0, 0], [5, 6, 7, 8, 0, 0]], dtype=np.float32)
        s = a[:, 0:4]  # slice
        t.pass_float32(s)
        assert t.get_stride(s, 0) == 6;
        assert t.get_stride(s, 1) == 1;
>       v = s.view(np.complex64)
E       ValueError: To change to a dtype of a different size, the array must be C-contiguous

test_ndarray.py:689: ValueError

It was changed in https://github.com/numpy/numpy/commit/1a009e0d061df490fa91df8684768ed3217704db
Reproducible example code
No response

Hi @wojdyr,
Thank you for filing this issue and clearly documenting when numpy added the functionality on which this test relies.
If you have time, please check whether PR #659 fixes this issue for you.   (The fix is simple--it may be easiest to just cut-and-paste the change into your local copy of test_ndarray.py.)
Regards,
Paul

Hi Paul, thanks, your fix works.
When I was opening this issue I was actually wondering if I should open a PR instead. But since this test can be skipped in more than one way (one could, alternatively, check np.__version__) and I couldn't decide which way is better, I ended up just opening the issue. Thanks for picking it up.

Closed via #659


[dataprotection] Support for PostgreSQLFlexibleServer and MySQL and code cleanup

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

Added dataprotection support for PostgreSQLFlexibleServer and MySQL workloads: new manifests, code cleanup.
az dataprotection backup-instance update-msi-permissions: New parameter --target-storage-account-id for Restore, support Restore for new workloads, code cleanup.

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

dataprotection

⚠ Release Suggestions
Module: dataprotection

Update version to 1.0.0b2 in setup.py
Set azext.isPreview to true in azext_dataprotection/azext_metadata.json
Remove azext.isExperimental: true in azext_dataprotection/azext_metadata.json

Notes

Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
For more info about extension versioning, please refer to Extension version schema

Hi @zubairabid please follow the release suggestions here for your module version config according to cli extension's version schema: https://github.com/Azure/azure-cli/blob/release/doc/extensions/versioning_guidelines.md

⚠ Release Suggestions
Module: dataprotection

Update version to 1.0.0b2 in setup.py
Set azext.isPreview to true in azext_dataprotection/azext_metadata.json
Remove azext.isExperimental: true in azext_dataprotection/azext_metadata.json

Notes

Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
For more info about extension versioning, please refer to Extension version schema

Hi @zubairabid please follow the release suggestions here for your module version config according to cli extension's version schema: https://github.com/Azure/azure-cli/blob/release/doc/extensions/versioning_guidelines.md

Hi @AllyW I removed the isExperimental: true. We want this to be a stable release, are there any other changes I need to make.

⚠ Release Suggestions
Module: dataprotection

Update version to 1.0.0 in setup.py
Remove azext.isExperimental: true in azext_dataprotection/azext_metadata.json

Notes

Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
For more info about extension versioning, please refer to Extension version schema

@zubairabid new version has been calculated as stable release. Please adjust the code accordingly

@AllyW the current version is already 1.0.0, just the isExperimental tag had not been removed.


Add some solutions

kamyu104 is probably not a nice person and wouldn't add your PR.

@kamyu104 If it's not appropriate to make a PR like this, please let me know. Anyway thanks for this project.


ENH: add a SequenceCollection.count_ambiguous_per_seq method, fixes #2005

Summary by Sourcery
Add a new method to count ambiguous characters per sequence and enhance sequence filtering by introducing an ambig_fraction parameter. Update tests to cover these new functionalities.
New Features:

Introduce a method count_ambiguous_per_seq in SequenceCollection to count ambiguous characters per sequence.

Enhancements:

Add ambig_fraction parameter to omit_bad_seqs function to allow filtering sequences based on ambiguous character content.

Tests:

Add tests for the new count_ambiguous_per_seq method in SequenceCollection.
Add tests for the ambig_fraction parameter in omit_bad_seqs function.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

24 of 24 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 92.362%

Totals

Change from base Build<PHONE_NUMBER>1:
0.01%

Covered Lines:
34042

Relevant Lines:
36857

💛  - Coveralls


Take the water visibility QA in consideration in the pipeline_node

Take the water visibility QA in consideration in the pipeline_node.
E.g.: If water visibility is 1.5 the robot should not detect the pipeline if its altitude is higher than 1.5

Closed with PR #57


Config visual rewrite

Im gonna leave a link to pastebin when im done, i left some controversial things to discuss right after "!!!" just like comms.

Here we go!
https://pastebin.com/bBRtkveg

Thank you for this work. However, the problem. I have with changing some of the default values is that it kind of makes the maxframes config useless, no?

Well, I suppose I just recommended updating some values to make the config more perfomance based,. Still, I haven't done anything to maxframes.cfg so the values may be the same in some parts. These two configs just need to be compared

Yes now I see some values are just the same except water and sound sections

Just let me know If you don't want these changes and I edit maxframes.cfg as I did with the main config. It will be just easier to add these options to the bottom and comment them, pretty much just like Comanglia has their config with Disabled shadows and option to enable them just by uncommenting the same cvars with different values for eg

// Add new renderables and compute blends for added renderables asynchronously
cl_threaded_client_leaf_system 1 // optimization for 4 core+ CPUs
//cl_threaded_client_leaf_system 0 // use traditional behavior for CPUs with less cores

// Force multithreaded mode for the material system queue
mat_queue_mode 2 // optimization for 4 core+ CPUs
//mat_queue_mode -1 // let the engine determine the optimal mode otherwise

// Set up bones on animated entities asynchronously
r_threaded_renderables 1 // optimization for 4 core+ CPUs
//r_threaded_renderables 0 // use traditional behavior for CPUs with less cores

// Queue decals to material system
r_queued_decals 0 // Possible crashes and little benefit when decal limit is low
//r_queued_decals 1 // Possible performance improvement if you don't crash

Here's how I'm rewriting my config. Any suggestions? I feel like moving all of the comments to the side gives me less room to explain things and won't let me add customization options.

//These settings for quard-core CPUs only, settings for <4-core CPUs are listed below

cl_threaded_client_leaf_system 1 // Add new renderables and compute blends for added renderables asynchronously
mat_queue_mode 2  // Force multithreaded mode for the material system queue
r_threaded_renderables 1 // Set up bones on animated entities asynchronously
r_queued_decals 0 // Queue decals to material system

//cl_threaded_client_leaf_system 0 // use traditional behavior for CPUs with less cores
//mat_queue_mode -1 // let the engine determine the optimal mode otherwise
//r_threaded_renderables 0 // use traditional behavior for CPUs with less cores
//r_queued_decals 1 // Possible performance improvement if you don't crash

This is just my vision of the situation

Hre's the solution if you experiencing room problems:
command  x  //blabla
                    //blabla
                    //bla

Why shouldn't the same CVars be together? How did you choose to group them like that?

Im sorry if this confusing you. Grouping up things like this just gives a cleaner look. Every commented command in the section clearly visible and can be easily fund and uncommented.
// --- Specific section here ---
Here at the top we've got the most compatible values

//Here at the bottom we've got more personalized commands, commented.

This is just makes the config user friendly, and if you run it w/o editing, it would be okay to use.
Like I have to change some cvars every time the config being updated because some cvars "are not for every user" (im talking about threading as a 2 core CPU user)
Actually If you you want the config to be more compatible, use threading cvars for 2 core CPUs.
For example Shadows section may look like this:

// --- Shadows ---

r_shadows 0					// disable shadows, big performance hit if you enable
r_shadowrendertotexture 0	// render shadows using two materials for each texture, makes shadows look better but uses a lot of RAM
cl_blobbyshadows 1			// don't do a second shadow render pass for sharper shadows
r_threaded_client_shadow_manager 1	// threaded shadow manager
r_flashlightdepthres 32		// the resolution of the shadow depth texture, this only works if depth texturing is enabled
							// this makes shadows more sharp and detailed the higher it is, shadows are fine at 32 or 64 though
r_flashlightdepthtexture 0	// depth texturing for shadows: these shadow calcuations are done mostly. It has a big performance hit on CPU
r_flashlightmodels 0		// disable shading/lighting useless performance hit
r_flashlightrender 0
r_flashlightdrawfrustum 0
nb_shadow_dist 300			// don't render shadows far away (nextbot system)
r_shadowmaxrendered 17		// don't render more shadows than needed

//The same commands to turn on shadows here

It's easy to uncomment things rather than edit values and remember the previous ones
Customization only for gibs, shadows, sprays (turn on and off by uncommenting) and threading would be enough for now
Again, im really sorry for wasting your time with these lil things

Oh, I just replied to the other issues which is about the same topic but yeah I agree with @JackStanley

I wasn't asking about grouping according to type, but why not have different options for the CVars grouped by CVar?
For example:
cl_threaded_client_leaf_system 1 // Asynchronously add new renderables and compute blends for added
                                 // renderables
//cl_threaded_client_leaf_system 0 // Use traditional behavior for CPUs with less than 4 cores

r_threaded_renderables 1 // Asynchronously set up bones on animated entities
//r_threaded_renderables 0 // Use traditional behavior for CPUs with less than 4 cores

As opposed to:
cl_threaded_client_leaf_system 1 // Asynchronously add new renderables and compute blends for added
                                 // renderables
r_threaded_renderables 1 // Asynchronously set up bones on animated entities

//cl_threaded_client_leaf_system 0 // Use traditional behavior for CPUs with less than 4 cores
//r_threaded_renderables 0 // Use traditional behavior for CPUs with less than 4 cores

If you want an easier time setting up the config, I think the better solution is presets (fewcores.cfg, maxframes.cfg, slowcpu.cfg, etc). That way you don't have to uncomment settings you would uncomment anyway.

Oh wait I remember your config had ~20 .cfg files, it's no way D: Just do as you like, or start staw poll in tf.tv thread :)

Well, this approach would be different to that. It would basically be a few overrides for a certain user.
In the old system, it was just every category had its own cfg, but in this new system, it will just be for quickly switching settings that you would change anyway if you needed it.

So are the comments right after cvars will be implemented? W/o any categorizing

Working on it: https://gist.github.com/mastercoms/44be52f6c070de1a7c2e8a0430e0b257

See https://github.com/mastercoms/tf2cfg/tree/4.0.0

Well a lot of work nicely done here!


Typo "@option"→"@option"

Italic with escape characters
https://docs.microsoft.com/en-us/sql/relational-databases/system-functions/getfilenamespacepath-transact-sql?view=sql-server-2017

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 Thank you!


Update PR template to include doc issue ref

Description
Initiative to ensure docs are in line to current feature set.
@yaron2 as discussed, pls review

@msfussell


Incorrect instructions for building

We need to include the correct instructions for Swifty-LLVM (which are currently wrong)

Hi!  This is related enough to not open a separate issue.  I succeeded in building valc with a homebrew-installed llvm, which I think is what most people use.  It would be nice to put this in one form or another into README:
$ git clone https://github.com/val-lang/Swifty-LLVM.git
$ cd Swifty-LLVM
$ brew install llvm@15
$ ./Tools/make-pkgconfig.sh llvm.pc
$ cd ..
$ git clone https://github.com/val-lang/val.git
$ cd val
$ cp ../Swifty-LLVM/llvm.pc ./
$ export PKG_CONFIG_PATH=$PWD
$ swift build -c release

Produces valc at .build/arm64-apple-macosx/release/valc
Just for reference, here’s my llvm.pc:
Name: LLVM
Description: Low-level Virtual Machine compiler framework
Version: 15.0.7
URL: http://www.llvm.org/
Libs: -lc++<EMAIL_ADDRESS>-lLLVM-15
Cflags<EMAIL_ADDRESS>

Good idea, @adaszko. It's nice to have the option not to put the .pc file in some system path.  I didn't want to have to set an environment variable, and wanted to have the option to use XCode, so mine is in $HOMEBREW_PREFIX/lib/pkgconfig, which we should also list as an option.

@adaszko you inspired this change, getting sudo out of our CI process.

@adaszko you inspired this change, getting sudo out of our CI process.

Great.  Now I wish I’d inspire something high impact, like a type system improvement for example 😄

Great. Now I wish I’d inspire something high impact, like a type system improvement for example 😄

That's not out of reach.
Do you have any idea? 😉


Xcode 14 - Warnings related to iOS 11 support (Swift Package Manager)

Checklist

[x] I've checked that all new and existing tests pass
[x] I've updated the documentation if necessary
[x] I've added an entry in the CHANGELOG to credit myself

Description
Hi,
I would like to contribute fixing a recent warning.
The new Xcode 14 uses next SDKs as base: iOS 11.0, macOS 10.13, watchOS 4.0 and tvOS 11.0. Compiling in this Xcode shows a warning.
Handle this restrictions relying on swift version (which is also vinculated to each Xcode version).
Regards and comments welcomed
Motivation and Context

Fixes a warning

Run project tests and compile on each platform.

I think as half of the changes are already included in https://github.com/AliSoftware/OHHTTPStubs/pull/352 this PR could be discarded when the other is merged.

Thank you for your contribution!
Since #361 was merged I am closing this. If some of these changes are still required, please do not hesitate to open another PR!


Add wireguard support

This adds wireguard schema and it's tests.
Signed-off-by : Shreya Gupta<EMAIL_ADDRESS>

Hello contributor, thanks for submitting a PR for this project!
I am the bot who triggers "standard-CI" builds for this project.
As a security measure, I will not run automated tests on PRs that are not from white-listed contributors.
In order to allow automated tests to run, please ask one of the project maintainers to review the code and then do one of the following:

Type ci test please on this PR to trigger automated tests for it.
Type ci add to whitelist on this PR to trigger automated tests for it and also add you to the contributor white-list so that your future PRs will be tested automatically. ( keep in mind this list might be overwritten if the job XML is refreshed, for permanent whitelisting, please follow #3 option )
If you are planning to contribute to more than one project, maybe it's better to ask them to add you to the project organization, so you'll be able to run tests for all the organization's projects.

We are not child project of NetworkManager, please don't copy whatever NetworkManager provides to our schema. Nmstate has plugin design which allows other backends.
And nmstate schema should not depend on implementation of certain backend, so you should look on wireguard options itself instead of what NM provides.

While I do agree on this, I see an issue here. NetworkManager provides certain options and some of them are wrappers that does not exist on wireguard. I will look at them and help @shreyagupta30 on this :-)

The ipv4-auto-default-route, peer-routes and etc are just convenient stuffs created by NM which can be done via existing nmstate route and route rule schema.
You also missed a lot options:
* Peer

* KeepAlive

* Allowed IP

We are going to collect them and add these.

We will not merge schema-only PR without implementation/test code showing it actual works.

We are not planning on merging them now. This a PR for initial feedback which is nice.

We seems don't have intention to support wireguard server, please also mention that in commit message indication this is only for wireguard client.

Yep. That is missing.
By the way, if we want to support other options than the supported from NetworkManager we are going to need WIreguard support on Nispor and therefore, generic netlink support on rtnetlink. I will investigate this.
Thank you! @cathay4t @shreyagupta30

Not stale :-)

Not stale :-)

Not stale :)

Not stale :.-)

Not stale

not stale

Not stale

This was not stale, we are looking at it. Sorry for the inconvenience.


Does not work with oculus quest

I tried http://code-panda.com/pages/projects/v_0_3_0/a_sky_forge_lunar_eclipse_example this in oculus quest native browser and firefox.
result >> grayisch sky, really laggy
I guess the issue is memory/performance related.. on my desktop(firefox) it works well.

That's a high likelihood. Are you in tethered or untethered mode? Having additional power often allows mobile devices to switch to higher graphics settings that are normally unavailable at lower settings to reduce the power load.
I've noticed that the sky tends to lag on my macbook air, but works well on my RX580 machine (It was developed using a GTX 1070). I do not have a Quest to verify this with, however. I believe most of the computational cost comes from the extensive computational load that the sky puts on the GPU. V 0.3.0 (currently master) does a good job of improving performance, but it's still not a perfect solution. I'm working out an entire redo of the project, slated for V 1.0.0 in the future, but that's a ways off while I'm still figuring out the technology and waiting for some of it to mature. Even then, I'm uncertain if this will truly improve performance enough to bridge the gap. Thank you for giving it a shot, however.

i tried tethered mode... same problem. The quest browser isn't that great a performance monitor is missing so i cant pinpoint the problem. However thanks for your help.


Brackets auto-closes tags even when setting "whenOpening" to false

Prerequisites

[X] Can you reproduce the problem with Debug -> Reload Without Extensions?
[X] Did you perform a cursory search to see if your bug or enhancement is already reported?
[X] Did you read the Troubleshooting guide?

For more information on how to write a good bug report read here
For more information on how to contribute read here
Description
When running Brackets v.1.12 (and also checked v1.11) for MacOS 10.10.5, and setting whenOpening to false, the tag auto-completion feature still completes the tag as soon as the opening tag is typed.  This is a nuisance when editing code vs. writing code from scratch as I always have to delete the second tag that is auto-created every time I type in a tag.
[Description of the bug or feature]
Steps to Reproduce

Install Brackets v1.12 for MacOS.
Start Brackets and click Brackets > Preferences.
Scroll down to the closeTags section and change "whenOpening" to false.
Save the preferences.
Close Brackets.
Re-open Brackets.
Open any HTML file.
Type  and notice that the closing tag  is auto-generated.  It seems to be true for all other tags as well.

This is a hassle because every time I type a tag, I get a closing tag and when I'm editing HTML, this is not the way to go.  I have to delete the closing tag all the time.
Another time, I also deleted the following files to clear any cache after saving the changes to defaultPreferences.json from the same folder and restarting Brackets:
brackets.json
state.json
/cef_data/
I also have tried launching Brackets with Debug > Reload Without Extensions

Expected behavior: [What you expected to happen]
When editing the defaultPreferences.json as described in the steps above, the expectation is that a tag can be typed without a closing tag being auto-generated.
Actual behavior: [What actually happened]
Closing tags are auto-generated every time an opening tag is typed.
Versions
Seen with versions 1.11 and 1.12 for MacOS v10.10.5.
More...
Release 1.12 build 1.12.0-17621 (release d3b783b99)
build timestamp: Thu Feb 01 2018 03:32:03 GMT-0800
Please include the OS and what version of the OS you're running.
Please include the version of Brackets. You can find it under Help -> About Brackets (Windows and Linux) or Brackets -> About Brackets (macOS)

Working perfectly fine for me. Please check you have the same entries in brackets.json
"closeTags": {
        "whenOpening": false,
        "whenClosing": false
    },

Please make sure there is no parsing error in brackets.json and this "closeTags" is same as mentioned here.

Hi Navch,
Thanks for your reply.  I tried, but no avail.  I have attached the preferences file and a couple of screen shots to show the tags still auto-complete.  Would you mind checking?
Thanks,
Chris

[defaultPreferences.json.zip](https://github.com/adobe/brackets/files/1826686/defaultPreferences.json.zip)

Hi @csauvageau
It looks like you are updating defaultPreferenc.json, I think you need to update brackets.json file. On clicking  Brackets > Preference, it will open both files. And you need to update in brackets.json. Brackets pick preference from brackets.json, defaultPreference.json is there just for reference.
Please try this and let me know.

Hi Naveen,
Just tried modifying brackets.son and got this error message.  Looks like it doesn't recognize the properties.  I also restarted Brackets as well.  If you have any other suggestions, please do let me know.  I appreciate your help.
See screen shot below.

Thanks,
Chris

you missed comma (,) on the 6th line. Add that and then press enter. and paste the code written here
"closeTags": {         "whenOpening": false,         "whenClosing": false     },
It will work like charm :)

Great!  Thank you!  Yes, works like a charm!  I wonder why it was not configured this way out of the box, though.  Still, it works and is much more usable now.  Thanks so much, Naveen!
Chris

Hello there! I am starting to learn HTML, really a newby here... When I have read these explanation I though I would resolve my issue, which is exactly the same. However, I'm still getting error messages when I try to change my brackets.json file. Here is how I wrote it:

I think I did not get the comme thing on the 6th line. Could someone explain it for me, or even show me the entire code?
Thank you kindly!

I guess no need of comma at the end of 6th line

what is exactly you want to do? If you want to get rid of this. Then just delete %appdata%/Brackets/brackets.json
It will auto generate this file on next launch. Then just do Debug > Open preference file
It will open two file one is brackets.json other is defaultPreference.json. Whatever setting you want to do copy that piece of code from defaultPreference file and paste in brackets. And modify accordingly. Make sure you put comma at right place. And you can use some online json validator to validate your json.

Ok, thank you, mate. I will try it now...
I am trying to eliminate the autoclosing tags default.

It seems like there was that extra comma at the end of 6th line! It is working now.
Still learning to get smart at these details!
Thank you for this very fast helping hand! :)

@leolaps Enjoy learning 👍 Go through brackets feature. This editor has amazing features like multi cursor, quick edit, hints and many more. This editor has a lot of wonderful extensions as well. Then you will enjoy learning more :)

It looks like you have the same problem as above -- a comma at the end of the "closeTags" line. In JSON, commas are separators, not terminators.
There's a JSONLint extension that will catch this for you.

Thanks that helped fix the problem

Thank you, it helped me out.

what is wrong

looks like comma missing on 8th line and extra comma on 9th line. You could catch this by code colouring on 9th line colour of key is different.

Hi.
I´m trying to change the bracket.json as instructed above, but I get the following error:

Any ideas what I´m doing wrong?

Hi!
Is my first week using brackets and I am having the same issue than above: Not being able to remove the automatic closing of the tags.
I checked many times the preferences and try different ways of writing them but the tags are still closing automatically.
Here is how my actual brackets.json file looks like:

Hi escapeedo,
I just downloaded the latest Brackets (Release 1.13 build 1.13.0-17696 (release 49d29a8bc)).
Auto-close still seems to be off, which is the behavior I want.  Here is my brackets.json file:
{
"brackets-eslint.gutterMarks": true,
"brackets-eslint.useLocalESLint": false,
"fonts.fontSize": "12px",
"fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
"closeBrackets": false,
"closeTags": { "whenOpening": false, "whenClosing": false },
"closeOthers.below": true,
"closeOthers.others": true
}
Good luck!
Chris

Thanks @csauvageau
I think the issue was in the comma on my line 9, which it was redundant with the comma in line 10.
I could realize that checking your settings.
Thank you very much, Chris! 🤘🤘🤘

I am having the same issues as all others, i keep getting my tags closed when i dont want to.
I tried copying @csauvageau info, but still having the issue. In my image exaple i show that  auto closed when i put the end bracket, i do not want that!

I am not getting the auto close brackets or hints please help me with this
i tried changing the preference, uninstalled and re-installed, but nothing worked


[Feature Request] Include common junk items in standard .gitignore

Some files created by OS specific programs might clutter folders in the ARC, adding unnecessary additional overhead to all steps included (Like syncing with the git remote).
Files like this might include the Desktop.ini.
The ArcCommander already has a standard .gitignore, that can optionally be created when initializing the ARC. Maybe this .gitignore might be extended with some additional rules for catching these unwanted files.
@dvs-frbg @muehlhaus @chgarth

Good idea. We can borrow from this repo, which has ignore rules for macOS, Windows, Linux, and MS Office.