Missing BOM tag (UTF-8 support incomplete)

Hi,
I created an playlist with this library and enabled the Encoding.UTF8 flag. This playlist works fine in Winamp, but Foobar isn't able to play the UTF8 paths.
After searching on the internet I found a topic talking about the BOM tag (first three bytes of the file: EF BB BF), I tested this with Foobar and it actually works!
Anyway, Am i missing some configuration option or is this missing from the library?
For reference: http://www.hydrogenaud.io/forums/index.php?showtopic=93905
Edit: I temporarily solved it using:
OutputStream outputStream = new FileOutputStream(fullPlayListPath);
PlaylistWriter writer = new PlaylistWriter();
outputStream.write(new byte[] {(byte) 0xEF, (byte) 0xBB, (byte) 0xBF});
writer.write(outputStream, playlist, Format.EXT_M3U, Encoding.UTF_8);

Would ofcourse be nicer if the library would take care of this.

Thank you for bringing this to my attention! There is currently no support for BOM parsing / writing in this library.
After briefly educating myself on the BOM character, I think it is suitable for this library to provide optional support where the default is to assume it is not present. The UTF-8 specification says that a BOM should be optional and its use is not recommended since byte order is irrelevant to UTF-8. The BOM character seems to be used as a marker that the stream is UTF-8 rather than indicating byte order. This is confusing and not something that should be supported by default IMHO.
I will add optional support for reading / writing and let you know when it is ready.

@maikeldus I prepared PR #19 if you want to create your own snapshot build from the branch. I can't say exactly when a release will go out to Maven Central.

Thanks for the response! The issue is not critical so I will just wait for whenever the maven library gets updated. I have checked the merge request and the addition look good, I also prefer the builder option.
I do still have another small question, do you know if it's possible to create m3u files (non-UTF8) that reference an UTF8 filename (with for example unicode usage in the filenames)? The default Android media implementation does not seem to support m3u8.
Thanks for sharing the code as a library btw ;)

Are you asking if you can create windows-1252 encoded files with the .m3u8 extension? You can explicitly set the encoding you want this library to use when parsing (utf-8 or windows-1252) so the filename's extension is not taken into account.

I just sent out a release for the 0.1.2 version. It may take a couple hours before it is publicly available. Please see if that works for you and then I can close this issue.

Sorry for the late response, it's working fine now. Thanks!
My question was about the default Android media implementation failing to parse UTF-8 playlist files, but I found a music player with a custom implementation.


New ABAP cleaner release not found by Eclipse

Hello,
I am using "Eclipse IDE for Java Developers (includes Incubating components)" Version: 2024-03 (4.31.0) Build id: 20240307-1437 and unfortunately the ABAP cleaner release 1.16.1 is not found when I try to update it via Help->Check for Updates.

I already edited the eclipse.ini as mentioned here https://github.com/SAP/abap-cleaner/blob/main/docs/installation-issues.md#known-installation-issues but this has no effect.
What do I have to do to be able to update to the latest ABAP cleaner release (currently 1.16.1)?
Kind Regards,
Kai

Hi Kai,
that's odd – generally, ABAP cleaner 1.16.1 seems to be found, at least I have not yet heard of another case where it wasn't. I'll have to ask a colleague about it.
Kind regards,
Jörg-Michael

Hi Kai,
I just released version 1.16.2, maybe you could try again whether this is now found.
Kind regards,
Jörg-Michael

Hi  Jörg-Michael,
unfortunately, the new version is still not found.
Kind Regards,
Kai


Add result struct to activity function signature

closes #800

I made the recommend changes in the new commits. I also added some comments with my thoughts as a whole.

Can you rebase? :pray:


batch users query

Slight improvement while we're at it - could both CI steps in the scripts folder be bumped to Juno v4.0.0


How to download ILSVRC2015-VID?

I have met some trouble in downloading the ILSVRC2015-VID dataset from the command line on Ubuntu. I have found 85.7G of the link from the Internet. Is it really so large? Can someone share how you downloaded ILSVRC2015-VID? I would be grateful if you could answer me.

是的，就是那么大。使用命令下载容易出错，建议使用迅雷或者浏览器下载。

I have met some trouble in downloading the ILSVRC2015-VID dataset from the command line on Ubuntu. I have found 85.7G of the link from the Internet. Is it really so large? Can someone share how you downloaded ILSVRC2015-VID? I would be grateful if you could answer me.

Can you share the VID dataset? The link to download it is dead.

https://github.com/StrangerZhang/SiamFC-PyTorch/issues/32#issuecomment-1429650603


:boom: nhsuk/frontend implementation

Couldn't easily resolve the issues with shrinkwrap. So created a new branch to fix this issues. Closing this now.


Note for debug MixList.php package not active

After registering my theme i couldn't compile because not active. It was automatically added to the ignored package in package.json

What caused it to be added to the list of ignored packages?

AFAICT, the only way it would be added to ignored package would be if they mistakingly said no to adding it to package.json in the first place, so I'll close this for now, unless a bug can be demonstrated.


Support .NET Framework 3.5

Oh you just downgrade it to byte array. That's great! Yeah you shouldn't have any length problems. Here's some extension methods I use with your class, I really enjoy them. Keep up the great work.
        public static Stream ToStream(this string str)
        {
            MemoryStream stream = new MemoryStream();
            StreamWriter writer = new StreamWriter(stream, Encoding.Unicode);
            writer.Write(str);
            writer.Flush();
            stream.Position = 0;
            return stream;
        }

        public static Stream ToStream(this byte[] bin)
        {
            MemoryStream stream = new MemoryStream(bin);
            return stream;
        }
        public static string ToBase32k(this byte[] bin)
        {
            return Base32k.Encode(bin);
        }

        public static string ToBase32k(this Stream bin)
        {
            return Base32k.Encode(bin);
        }

        public static byte[] FromBase32k(this string str)
        {
            return Base32k.Decode(str);
        }

I'm sidetrack on trying to keep root and and get a working recovery on an Android phone that's poorly documented right now and if you have experience with that you know how draining it is. Otherwise I would be motivated to work with your code more.

Thanks, I hope you will take advantage of this project.
Although it is not directly related to the subject, I recommend that you specify the language name for syntax highlighting in your code block.
    ```csharp
        public static byte[] FromBase32k(this string str)
        {
            return Base32k.Decode(str);
        }
    ```

The above code block is displayed as below.
        public static byte[] FromBase32k(this string str)
        {
            return Base32k.Decode(str);
        }

Didn't know you could do that, thanks!
On 11/12/2021 12:39 PM, kzrnm wrote:

Thanks, I hope you will take advantage of this project.
Although it is not directly related to the subject, I recommend that
you specify the language name for syntax highlighting in your code
block.
public static byte[] FromBase32k(this string str)
{
return Base32k.Decode(str);
}

The above code block is displayed as below.
public static byte[] FromBase32k(this string str)
{
return Base32k.Decode(str);
}
--
You are receiving this because you commented.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Triage notifications on the go with GitHub Mobile for iOS [3] or
Android [4].

Links:
[1] https://github.com/kzrnm/Base32768/pull/10#issuecomment-967296251
[2]
https://github.com/notifications/unsubscribe-auth/AA5OOMVWRJGUV4FZENZU22TULVGO3ANCNFSM5H5JYW7A
[3]
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
[4]
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub


images smaller than content width are not centered

I think this is fixed, @thatbudakguy would you review?

 Does that mean it's working?

@thatbudakguy I don't know of any current examples - it came up when Mary was working on the PPA post she did after DH2018, but I think we worked around it. You can either set the size smaller manually or add/upload a small image to check.

Uploaded a small image to check and it was correctly centered! 🌮


DEYE SUN-8K-G05 yaml file

Hi I have a deye sun-8k-g05 inverter and I can't read the power on PV1 and PV2, what are the addresses to read the power ?. Does anyone have a yaml file for this inverter where all the parameters can be read ?.

Hi @folny, so any of the already present deye_* profiles does not work?

Enable debug logging and attach the file here.

Okay, I'll add the debug file tomorrow.

Hi
So I played around a bit today and it seems the data is now correct but I would like to know how I can read the values from the smart meter connected to the inverter so I can find out the household consumption and the energy that goes from the grid and into the grid.

Try it with deye_hybrid.yaml profile then. String and Hybrid profiles are essentially the same but the String has fewer registers read.

I have already tried this option, but the data I need is incorrect.

What about 203, 204?

Yes, it should be those registers, but I didn't have time to try it today, I'll try it tomorrow and let you know.

So today I added more registers to the file and it seems that everything is working as it should. So I'm adding my file here and I hope it helps someone in the future.
deye_sun_g05_power_flow_card.zip

I think that Load Power should be unsigned.
feat: Load & Grid Power for Deye String

There are more irrelevant values but I left them there.


Remove slightly dangerous optimization

If there happens to be a file at the location passed into the ManifestLoader, we were using that to compile the manifest, instead of the actual contents. This is dangerous, because the public API of ManifestLoader` accepts a custom filesystem parameter that is not taken into account at all here, so the file being used might not be the right one, it just has the same path on the local filesystem.
We could have done some more work to preserve the optimization in the case where the used filesystem is the local one and the correct file is being used, but that doesn't seem worth it since the vast majority of loads are being done for remote dependencies (this we may load multiple times as we are resolving) which are never using the local filesystem.

@swift-ci please smoke test

Looks like there are various tests asserting the current behavior, so converting to draft for now until I get to fixing them.

@swift-ci please smoke test

@neonichu do we want to take this one?

Yah, definitely, I need to come back to it and implement your suggestions.

@swift-ci please smoke test

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test

Fixed this so that we pass the original path to the compiler, this makes the VFS actually be used. Still seeing one test failure in testPackageContextDirectory.

Looks like this is maybe breaking manifest.displayName as it looks empty during testPackageContextDirectory with my changes.

Ah no, the problem is that the test was only working due to the bug where we would present the temporary file as the "real" manifest.
Prior to my change:
(lldb) po packageDirectory
"/var/folders/jc/zkdxhl4j0d35ydwqzd758l_w0000gn/T"

After my change:
(lldb) po packageDirectory
"/"

Since the manifest we're being told to load is /Package.swift, it seems like the result after my changes is the actual correct one.

@swift-ci please smoke test

Forgot to actually push my changes before re-running 🤦

@swift-ci please smoke test


Why wasnt seaweefs made to handle large files?

I know it can handle large files by chunking them, but whats the reason for not handling whole large files?

the storage is writing the whole file as one consecutive disk space. This speeds up reading also, but not able to handle large file. the chunking needs to be handled at different level.


test: hide and reshow documentation preserves colors

This is pending a new release from blink. The tests depend on a fix that was merged in https://github.com/Saghen/blink.cmp/pull/768


How to make access log stored into a custom file.

Refer to this article: https://istio.io/latest/docs/tasks/observability/logs/access-log/ ,e.g.
$ kubectl logs -l app=sleep -c istio-proxy
[2020-11-25T21:26:18.409Z] "GET /status/418 HTTP/1.1" 418 - via_upstream - "-" 0 135 4 4 "-" "curl/7.73.0-DEV" "84961386-6d84-929d-98bd-c5aee93b5c88" "httpbin:8000" "<IP_ADDRESS>:80" outbound|8000||httpbin.foo.svc.cluster.local <IP_ADDRESS>:37652 <IP_ADDRESS>:8000 <IP_ADDRESS>:46520 - default

We can get the access log easily, but it printed to the standard output by default (And can be found at /var/log/pods/$pod/.....json file)
Can we configure the access log to a custom file of the istio-proxy container while support standard output as well?

I added some changes(add --log-path to envoy) to istio/proxyv2 image
So i added  --log-path /var/lib/istio/data/envoy.log to the envoy process, but the access log didn't print to envoy.log.
btw, have any method of add blow configuration to all dynamic_resources and static_resources?
  custom-envoy.yaml: |-
    filter_chains:
    - filters:
      - name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          access_log:
          - name: envoy.access_loggers.file
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog
              path: /var/lib/istio/data/access_log.log

You can set accesslog path before install, or update the istio configmap directly. And then restart all sidecars.

https://istio.io/docs/reference/config/istio.mesh.v1alpha1/ see accessLogFile.
BTW, writing to stdout is standard kubernetes best practice. I would not recommended it personally - but you are free to do it if you please.

Also it is dynamically updatded, no restart required.


Bug in FormatParameterMismatch cop?

@edmz
With the new version 0.33.0, I am seeing FormatParamterMismatch complain about correct usage of String#% (which I happen to prefer over format):
My test.rb
# encoding: utf-8
puts '%s' % 42

My RucoCop output:
Inspecting 1 file
W

Offenses:

test.rb:2:11: W: Number arguments (0) to String#% mismatches expected fields (1).
puts '%s' % 42
          ^
test.rb:2:11: C: Favor format over String#%.
puts '%s' % 42
          ^

1 file inspected, 2 offenses detected

I know how to configure my way out of “Favor format over String#%”, but the first message looks like a bug.

Yep, that's definitely a bug.

I created PR #2112 that should fix this.


eslint-config-office-addins 1.0.16 is missing the lib folder

Prerequisites
Please answer the following questions before submitting an issue.
YOU MAY DELETE THE PREREQUISITES SECTION.

[x] I am running the latest version of Node and the tools
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed

Expected behavior
by running "npm install eslint-config-office-addins" the latest version of the npm module is installed and works
(all seems to work with version<EMAIL_ADDRESS>Current behavior
after I run the lint comment via the "npm run lint" command I get this:

<EMAIL_ADDRESS>lint:office [EDITED_PATH] office-addin-lint check

Oops! Something went wrong! :(
ESLint: 6.8.0.
ESLint couldn't find the config "eslint-config-office-addins" to extend from. Please check that the name of the config is correct.
The config "eslint-config-office-addins" was referenced from the config file in "[EDITED_PATH]/.eslintrc.json".
If you still have problems, please stop by https://gitter.im/eslint/eslint to chat with the team.

Steps to Reproduce
follow the instruction in the README.md of this repo: https://github.com/OfficeDev/Office-Addin-Scripts/blob/master/packages/office-addin-lint/README.md
Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Operating System: OSX Catalina (10.15.6)
Node version: v10.17.0
Office version: not relevant
Tool version: eslint-config-office-addins 1.0.16

eslint-plugin-office-addins 0.1.7 have this issue too.
0.1.6 is ok.

@hyacinthus Please try 1.0.19. This should fix the issue, Please let us know if that's not the case.
Thanks,
Courtney

@TCourtneyOwen  my project is start from react demo. It do not use eslint-config-office-addins. but use eslint-plugin-office-addins . I'm not see a new version for this package.

@TCourtneyOwen still borked, see #333.

Can you please update to v.1.0.19.  It's the latest version.
Thanks,
Courtney

@TCourneyOwen
Check npm. See my comment on #333
Latest version for config is as you say but latest version for plugin is v0.1.7.

I just published<EMAIL_ADDRESS> Please try that out
Thanks,
Courtney


Generate UUIDs starting with a letter

Is your feature request related to a problem? Please describe.
Hello,
for specific needs, would it be possible to generate UUIDs with at least one letter at the start?
Describe the solution you'd like
For example:
1 ) e1b86008-fef5-41e8-9c14-f817e1a5fd45
2) a2b84546-f443-4515-89d2-593d6e1e448d
3) ffd718f7-14ae-431f-b1c1-59c572f6ebba
and so on...
Additional context
I'm using the uuidv4() function;
Thank you so much for your support!

This library generates UUIDs according to RFC 4122.
UUID v4 are random, so there is no control over how they start. If this is a requirement, then UUIDs may not be the right choice for your use case.


Config support for fallthrough attribute

Adds a feature test for C++17 [[fallthrough]] and removes gcc 7.1 warnings complaining about a possible fallthrough.

FWIW: this describes how things are implemented in gcc: https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/


Cosmetic endline fix for --version

Description
This minor, cosmetic fix ensures the output of --version ends with a newline even when there's no runtime installed. Perhaps there is a better way, like ending L26 with a newline and chomping the output of daprd --version.
Issue reference
Seeing this is a minor fix in a somehow obscure use case (I hit this bug when dapr init failed on me, so I still had no runtime version output) I didn't want to waste the team's time with an issue.
Checklist
I built this with REL_VERSION=foo make release with no warnings and confirmed the built binary outputs an endline when daprd is not present in PATH.
Apologies for writing more in this text than in the actual code.

Thanks!


Command line option for parsing UTC time to slot number

Description
This PR implements command line with slot number calculation for https://github.com/input-output-hk/cardano-node/issues/4928
Sample output:
https://cardanoscan.io/block/5385759 - Allegra era (1h TZ difference)
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2021-02-24T21:48:45Z'
22637034

https://cardanoscan.io/block/8687402 - Babbage era (2h TZ difference)
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2023-04-24T09:12:42Z'
90761271

https://cardanoscan.io/block/21567 - Byron era (2h TZ difference)
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2017-09-28T21:38:11Z'
21580

https://cardanoscan.io/block/0 - Byron era (2h TZ difference)
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2017-09-23T21:44:51Z'
0

one second before Genesis
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2017-09-23T21:44:50Z'
Command failed: query slot-number  Error: Past horizon: PastHorizon {pastHorizonCallStack = [("runQuery",SrcLoc {srcLocPackage = "ouroboros-consensus-<IP_ADDRESS>-b918dc6058e2288b1a799e94c8b13ecbad48fbefc1d681b1ef06a70562e74bd8", srcLocModule = "Ouroboros.Consensus.HardFork.History.Qry", srcLocFile = "src/Ouroboros/Consensus/HardFork/History/Qry.hs", srcLocStartLine = 428, srcLocStartCol = 44, srcLocEndLine = 428, srcLocEndCol = 64}),("interpretQuery",SrcLoc {srcLocPackage = "cardano-api-8.0.0-inplace", srcLocModule = "Cardano.Api.Query", srcLocFile = "src/Cardano/Api/Query.hs", srcLocStartLine = 199, srcLocStartCol = 21, srcLocEndLine = 199, srcLocEndCol = 51})], pastHorizonExpression = Some (ELet (ERelTimeToSlot (EAbsToRelTime (ELit (RelativeTime -1s)))) (\x0 -> ELet (ERelToAbsSlot (EVar x0)) (\x1 -> EPair (ESnd (EVar x0)) (EPair (EVar x1) (ESlotLength (EVar x1)))))), pastHorizonSummary = [EraSummary {eraStart = Bound {boundTime = RelativeTime 0s, boundSlot = SlotNo 0, boundEpoch = EpochNo 0}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 89856000s, boundSlot = SlotNo 4492800, boundEpoch = EpochNo 208}), eraParams = EraParams {eraEpochSize = EpochSize 21600, eraSlotLength = SlotLength 20s, eraSafeZone = StandardSafeZone 4320}},EraSummary {eraStart = Bound {boundTime = RelativeTime 89856000s, boundSlot = SlotNo 4492800, boundEpoch = EpochNo 208}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 101952000s, boundSlot = SlotNo 16588800, boundEpoch = EpochNo 236}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 101952000s, boundSlot = SlotNo 16588800, boundEpoch = EpochNo 236}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 108432000s, boundSlot = SlotNo 23068800, boundEpoch = EpochNo 251}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 108432000s, boundSlot = SlotNo 23068800, boundEpoch = EpochNo 251}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 125280000s, boundSlot = SlotNo 39916800, boundEpoch = EpochNo 290}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 125280000s, boundSlot = SlotNo 39916800, boundEpoch = EpochNo 290}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 157680000s, boundSlot = SlotNo 72316800, boundEpoch = EpochNo 365}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 157680000s, boundSlot = SlotNo 72316800, boundEpoch = EpochNo 365}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 176688000s, boundSlot = SlotNo 91324800, boundEpoch = EpochNo 409}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}}]}

end of N+1 epoch 5 days ahead:
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2023-04-29T22:00:00Z'
91239309

beginning of N+2 epoch - 6 days ahead:
cabal exec cardano-cli -- query slot-number --mainnet --socket-path ../network-mainnet/node.socket '2023-04-30T22:00:00Z'
Command failed: query slot-number  Error: Past horizon: PastHorizon {pastHorizonCallStack = [("runQuery",SrcLoc {srcLocPackage = "ouroboros-consensus-<IP_ADDRESS>-b918dc6058e2288b1a799e94c8b13ecbad48fbefc1d681b1ef06a70562e74bd8", srcLocModule = "Ouroboros.Consensus.HardFork.History.Qry", srcLocFile = "src/Ouroboros/Consensus/HardFork/History/Qry.hs", srcLocStartLine = 428, srcLocStartCol = 44, srcLocEndLine = 428, srcLocEndCol = 64}),("interpretQuery",SrcLoc {srcLocPackage = "cardano-api-8.0.0-inplace", srcLocModule = "Cardano.Api.Query", srcLocFile = "src/Cardano/Api/Query.hs", srcLocStartLine = 199, srcLocStartCol = 21, srcLocEndLine = 199, srcLocEndCol = 51})], pastHorizonExpression = Some (ELet (ERelTimeToSlot (EAbsToRelTime (ELit (RelativeTime 176688909s)))) (\x0 -> ELet (ERelToAbsSlot (EVar x0)) (\x1 -> EPair (ESnd (EVar x0)) (EPair (EVar x1) (ESlotLength (EVar x1)))))), pastHorizonSummary = [EraSummary {eraStart = Bound {boundTime = RelativeTime 0s, boundSlot = SlotNo 0, boundEpoch = EpochNo 0}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 89856000s, boundSlot = SlotNo 4492800, boundEpoch = EpochNo 208}), eraParams = EraParams {eraEpochSize = EpochSize 21600, eraSlotLength = SlotLength 20s, eraSafeZone = StandardSafeZone 4320}},EraSummary {eraStart = Bound {boundTime = RelativeTime 89856000s, boundSlot = SlotNo 4492800, boundEpoch = EpochNo 208}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 101952000s, boundSlot = SlotNo 16588800, boundEpoch = EpochNo 236}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 101952000s, boundSlot = SlotNo 16588800, boundEpoch = EpochNo 236}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 108432000s, boundSlot = SlotNo 23068800, boundEpoch = EpochNo 251}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 108432000s, boundSlot = SlotNo 23068800, boundEpoch = EpochNo 251}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 125280000s, boundSlot = SlotNo 39916800, boundEpoch = EpochNo 290}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 125280000s, boundSlot = SlotNo 39916800, boundEpoch = EpochNo 290}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 157680000s, boundSlot = SlotNo 72316800, boundEpoch = EpochNo 365}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}},EraSummary {eraStart = Bound {boundTime = RelativeTime 157680000s, boundSlot = SlotNo 72316800, boundEpoch = EpochNo 365}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 176688000s, boundSlot = SlotNo 91324800, boundEpoch = EpochNo 409}), eraParams = EraParams {eraEpochSize = EpochSize 432000, eraSlotLength = SlotLength 1s, eraSafeZone = StandardSafeZone 129600}}]}

Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated.  These may include:

golden tests
property tests
roundtrip tests
integration tests
See Runnings tests for more details

[ ] Any changes are noted in the CHANGELOG.md for affected package
[ ] The version bounds in .cabal files are updated
[ ] Code is linted with hlint.  See .github/workflows/check-hlint.yml for to get the hlint version
[ ] Code is formatted with stylish-haskell.  See .github/workflows/stylish-haskell.yml for to get the stylish-haskell version
[ ] Self-reviewed the diff

Needs a rebase.


Support "ApplicationName" connection parameter

With JDBC I can pass the ApplicationName parameter as part of the connection string - jdbc:postgresql://localhost:5435/DBNAME?ApplicationName=MyApp - and have "MyApp" show up in various statistics on the server. It would be useful to be able to do this on finagle-postgres as well.

Thanks, @agenticarus. I would happily ship this change in case you have time to work on this. If you don't have the time it can take a while since we right now don't have a lot of people maintaining this project, and there are some big upgrades and fixes with more priority in the backlog.

I'll take a look!


Unable to use Multi Projects with API Key token created on different Project.

Even Though I am using a different project token to create resources on Gridscale, terraform is creating resources on default project only.
Please do fix this problem.
Terraform version: 0.13.4
GridScale Terraform Provider version: v1.7.4

Thanks for reporting the issue. We will take a look at it as soon as possibile.

@tmallikarjuna According to gridscale multi-project, you will need to source the .envrc file (or use a tool called "direnv" to easily switch between environment variables) before applying. The command to source the .envrc is source .envrc.

@tmallikarjuna btw, I will update the README file to highlight "source the .envrc file" part.

Another way to deal with the multiple project is to define a block as following in each .tf file:
provider "gridscale" {
  uuid = "uuid" ## This can be discarded; if you already set it in an environment variable, and the projects using the same user UUID
  token = "project-a-token"
}

For more info: https://registry.terraform.io/providers/gridscale/gridscale/latest/docs

Hi Tom,
Thanks for the information. I was sourcing the .envrc file and the reason I don't know why it was overriding my default project's API key and maybe due to a global sourcing file was interrupting. Anyways, thanks for the info and the update of the guide.
Best regards,
Tejesh


Fix icon name in conda windows build script.

This fix is necessary to make a conda build on windows. You could maybe consider to also build the beta releases on conda-forge :)

Thanks for the fix! I'll create Windows packages later today :-)

Packages will be available in our Anaconda channel: spyder-ide.

Oh. Btw. Spyder froze on startup unless I had IPython 4.2 installed.


Cannot find module 'virtual:uno.css"

UnoCSS version
0.53.6
Describe the bug
Unocss example doesn't work in Codesandbox.
Reproduction
Try to open the SvelteKit Codesandbox example on the UnoCSS page.
System Info
Edge, Sveltekit
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I don't know if it is related, but Edge also crashes when installing unocss (using npm like "npm install -D unocss") on Stackblitz and gives an error of "This page is having a problem".

@Laxmanan-Krishnapillai Did you resolve this in the end?  I'm getting the same problem.


Decorator microbatch

For adaptively changing grad_accum, it seems we need to wrap multiple additional things:

training eval metrics
AFTER_DATALOADER event because of seq-length-warmup
[in future] eval code
I think it's cleaner to rewrite this as a decorator, so I'm refactoring, but I want input before we commit to this

Note that this is merging into https://github.com/mosaicml/composer/pull/485 and not dev

Yay look it now works with seq_length_warmup
Epoch 0:   0%|          | 0/9000 [00:00<?, ?it/s]
1 -> 2
Epoch 0:   0%|          | 0/9000 [00:05<?, ?it/s]
2 -> 4
Epoch 0:   0%|          | 0/9000 [00:05<?, ?it/s]
4 -> 8
Epoch 0:   0%|          | 0/9000 [00:05<?, ?it/s]
 8 -> 16
Epoch 0:   0%|          | 0/9000 [00:05<?, ?it/s]
 16 -> 32
Epoch 0:   0%|          | 0/9000 [00:05<?, ?it/s]
32 -> 64
Epoch 0:   0%|          | 40/9000 [01:26<4:59:54,  2.01s

Not bothering about lint etc for now, will clean it all up together in https://github.com/mosaicml/composer/pull/485

Also, feel free to merge this whenever - any and all comments can just be addressed on the other PR.

Super helpful feedback! Thanks everyone for chiming in. It seems like there's a few takeaways:

Decorators here aren't great b/c of state changes. Agree with this
Decompose the trainer loop into more fns
I like Ravi's recursive structure -- will switch to that instead of while loop
Keep it a little less general: wrap the core training loop but not engine calls
I like Ajay's suggestion: lets do dummy forward/backward passes on init to start the process (avoids some edge cases like seq warmup) and spit out a warning

Sadly it looks like this won't be as general as originally hoped :'( but hopefully still useful


Support more devices - HackRF One

This is by far one of the best SDR Spectrum apps, and the only one that offers background subtraction. It would be nice though if it could be used with more powerful devices like the HackRF one.

Yes please


File upload appears to hold everything in memory until the end

Describe the bug

For large files, memory upload consumes significant memory and it grows even more when writing out the file iobuffer.
Upload is very slow and consumes significant CPU when everything is on the same host (both reflex and the browser). Uploading files shouldn't really consume much CPU at all...

To Reproduce
Steps to reproduce the behavior:

Just use the standard upload example and upload a multi-gigabyte file.

Expected behavior
Very little memory should be used because the data should be incrementally written to a file.
To avoid re-writing the file after upload, I'd expect to be able to specify the target or if you choose to write to a temporary file I'd expect to be able to rename the temporary file to it's final home.
Specifics (please complete the following information):

Python Version: 3.12.4
Reflex Version: 0.5.4
OS: WSL Ubuntu 22.04.4 LTS  on Windows 11 pro
Browser (Optional): Brave 1.67.116 ( Chromium 126.0.6478.71)

Another odd behavior for upload is the on_upload_progress which claims the upload is 100% done but then spends another 10% of the time before returning from rx.upload_files and passing the result on to the handle_upload function.

can i work on these issuse

@jaypatidar14 just assigned you! Let us know if you need any help

Hello! can I work on this issue if it is still open?

I should point out that the very large memory consumption only happens at the end of the upload when the file write is happening. It seems fairly moderate during most of the upload.

I have one more observation that seems to point the root of the issue at gunicorn. I decided to see if it was any faster to upload when running reflex in production mode and I see now that instead of python consuming 100% CPU, it's the gunicorn process.


Fix AccummulatorCompiler to work with type-specific stack representation

Fix aggregation compiler to work fine if input function is defined for
Type which uses non standard stack representation. E.g. TIMESTAMP(p>6)
uses LongTimestamp on stack.

rebase

@findepi  test added


⚠ Secret Website 3 has degraded performance

In 1281c8b, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2592 ms

Resolved: Secret Website 3 performance has improved in 13a1518.


Really bad FPS

So I have a custom built PC, Geforce 1060 6gb/i5 processor/plenty of ram. I can run battlefield, skyrim remastered, witcher 3 all in ultra perfectly fine and smooth but I cannot surpass 10fps on the RuneLite client. Ive tried prioritizing the app through the task manager but that helps very slight (1/3fps at most) and its so unnoticeable. Its definitely not a problem with my rig and I've tried resetting it multiple times for the same thing to occur. A fix or tips how to would be greatly appreciated

Does this happen on the vanilla client?

Does this happen on the vanilla client?

It does when its on expanded to fit mode. The fixed, static size runs perfectly fine and I am totally confused as to why

Then there is nothing we can do about it. You may want to turn on the Stretched Fixed Mode plugin and turn on vanilla fixed mode.

Then there is nothing we can do about it. You may want to turn on the Stretched Fixed Mode plugin and turn on vanilla fixed mode.

I will try that, but any clue on why I'm running into this problem?

typically its because your window is too large for your cpu's speed


Introduce script and action step to build documentation

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Related to #2232
Description
This PR contains x2 Commit. You may want to review and merge #2459 separately, first? Or both together here - your call.
I'm intentionally raising small changes; this alone doesn't "do" all that much, yet; my plan is to gradually build upon this.
Type
Choose one: Builds
Checklist

[X] I have read and acknowledged the Code of conduct.
[X] I have read the Contributing page.
[X] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[X] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[ ] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

@santosh-pingle or @jingtang10 do you want me to "self merge" this (and future related) changes, or do you want to review?

@santosh-pingle or @jingtang10 do you want me to "self merge" this (and future related) changes, or do you want to review?

prefer to review :)

Kokoro: Build and Device Tests — Kokoro build failed. Check logs for details.

@jingtang10 looks like this ^^^ is due to:
┌─────────┬──────────────────────┬────────────────────────────┬───────────────────────────────────┬───────────────────────────────────┬────────────────────────────────┐
│ OUTCOME │      MATRIX ID       │          APP NAME          │          TEST FILE NAME           │          TEST AXIS VALUE          │          TEST DETAILS          │
├─────────┼──────────────────────┼────────────────────────────┼───────────────────────────────────┼───────────────────────────────────┼────────────────────────────────┤
│ success │ matrix-22nv06xa30fdw │ demo-debug-androidTest.apk │ datacapture-debug-androidTest.apk │ Nexus6P-24-en_US-portrait         │ 91 test cases passed           │
│ failure │ matrix-22nv06xa30fdw │ demo-debug-androidTest.apk │ datacapture-debug-androidTest.apk │ MediumPhone.arm-33-en_US-portrait │ 1 test cases failed, 92 passed │
└─────────┴──────────────────────┴────────────────────────────┴───────────────────────────────────┴───────────────────────────────────┴────────────────────────────────┘

which is very very likely completely unrelated to this PR? How do you want to handle this? Do you want to just merge this as-is anyway?


Akmods key generation on arm64 mac

generate-akmods-key pull an amd64 image only. I've tested a number of different Silverblue images unsuccessfully. My current workaround is to pull registry.fedoraproject.org/fedora and install the missing packages with dnf. This takes forever. Anyone can suggest an image to use? I would gladly submit a fix for it if I knew one.

I think your PR links to this issue: https://github.com/ublue-os/nvidia/pull/140

I'd like to consider closing this as we no long have a generate-akmods-key script in this repo... We don't even have it in akmods repo either.


Minesweeper game

Related Issue
Closes: #1113
Describe the changes you've made
Minesweeper game using html css and javascript added
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Screenshots

Looks Great. Good work

Thanks for this awesome contribution. Keep Contributing.


Initial Robot Name

Good first iteration.  Take a look at the comments, refactor where needed and resubmit.


cli script hangs

Type of issue

[x] Bug

Description
On my main (and brand new) computer, when I run
pcs -t remotemonitoring -s basic -r dotnet
the script accepts my solution name and my subscription choice, but it then hangs
indefinitely after it accepts the subscription. No further keystrokes are responded to at all.
I have to close the command window to get rid of it.
I do not have this problem creating the same solution on one of my other machines (with a whole bunch
of other npm packaged installed).
I've included a file showing the session. It shows a fresh npm install -g iot-solutions output followed
by the problematic login/command sequence.
pcs session.txt
Context and Environment

Operating System: Windows 10 pro 64 Version 10.0.17134.345
GitHub branch: N/A
Node.js Runtime: v10.13.0 (64 bit)

This hanging behavior continues in the recent 2.0.0 commit. I've discovered that the failure happens with nodejs version 10.13.0 but not with nodejs version 8.3.0

I have tried it with 10.13 node version and I'm able to run the command without issues. The latest released version for CLI is 1.0.13. Can you try again?

I did try again; on my older box which was able to run successfully using the npm published iot-solutions, I was able to reproduce this failure by upgrading to node 10.13.0 with no other changes. That makes two machines to repro the problem.

Also reported by #431

This has been closed, so I presume a solution exists. Can you point me to it? the dup you mentioned doesn't have the solution either, but it's also closed.

Not sure what you mean, this issue is still open. I closed the other issue.

I get same issue with node 10.13.0

The cause is identified and fix is on the way.

Yes this has been fixed here https://github.com/Azure/pcs-cli/pull/438 though we haven't published a newer version of the package yet. If you are building the packages locally (npm install & npm start & npm link) then you can just grab the latest code from master.
If not, please let me know and I'll find what our timeline is on getting an update pushed out.

Will be in the next release.


Easy way to block search engine crawling in robots.txt

For people who don't want their datasets to be crawled by search engines.

I'm going to close this as WONTFIX for the moment. Once Plugins #14 grows the ability to add extra URL paths and views someone who needs this could build it as a plugin instead.


cherry-pick for 0.8.0 release to master

What type of PR is this?
/kind chore
Why do we need this PR?
This PR adds in a cherry-picked PR for the changes made for the 0.8.0 release of argocd-operator

Hi @reginapizza , why was control-plane label updated in 0.8? Looks like this has caused some regression(#1087).
- control-plane: argocd-operator
+ control-plane: controller-manager

@iam-veeramalla had told me to update it to that, I'm not exactly sure why it was changed originally?


502: Bad Gateway on upload when Provides-Dist metadata is invalid instead of error messages

I am getting bad gateway errors the past 14 hours, only from my GitHub actions runs. Wondering if this could be related to the cause of #9151 since everything else seems to be working. My local CI works with the same token on a different repo so maybe this is some sort of regional proxy service outage or something.
Edit 1: Now it is only for this repo but not for my other repos with the same GitHub workflow and token setup. I have had the same result for GitHub actions runners 12,13, and 14. Does not seem to be related to which runner I use and seems to be isolated to this package repo.
Edit 2: I have figured out the issue the core metadata specification for Provides-Dist shows a version format that will not work for warehouse as it uses packaging.requirements.Requirement to validate this header. I have opened a PR over on the spec to change the example as well https://github.com/pypa/packaging.python.org/pull/1586
Uploading distributions to https://upload.pypi.org/legacy/
Uploading
blastpipe-2024.11.32-cp310-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
WARNING  Received "502: Bad Gateway"
         Package upload appears to have failed. Retry 1 of 5.
Uploading
blastpipe-2024.11.32-cp310-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
WARNING  Received "502: Bad Gateway"
         Package upload appears to have failed. Retry 2 of 5.
Uploading
blastpipe-2024.11.32-cp310-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
WARNING  Received "502: Bad Gateway"
         Package upload appears to have failed. Retry 3 of 5.
Uploading
blastpipe-2024.11.32-cp310-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
WARNING  Received "502: Bad Gateway"
         Package upload appears to have failed. Retry 4 of 5.
Uploading
blastpipe-2024.11.32-cp310-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
WARNING  Received "502: Bad Gateway"
         Package upload appears to have failed. Retry 5 of 5.
WARNING  Error during upload. Retry with the --verbose option for more details.
ERROR    HTTPError: 502 Bad Gateway from https://upload.pypi.org/legacy/
         Bad Gateway

I've asked @DarkaMaul to take a look at this 🙂


Add support for metadata template deletion via the API to the CLI

Add support for metadata template deletion via the API to the CLI. This is now available via the Box Content API
https://developer.box.com/v2.0/reference#delete-metadata-template

@schwartzMS We're currently in the process of a major update of the CLI, and adding support for all currently-unsupported API endpoints will happen as a part of that effort.  We'll keep you updated when the new version is released!

@schwartzMS The latest v2.0.0 release of the CLI includes support for deleting metadata templates, under the box metadata-templates:delete command.  You can find the latest installers at https://github.com/box/boxcli/releases/tag/v2.0.0 — please try it out and let us know if you run into any issues!


JSON config

Configuration is done through src/config.json file. An example is in src/config.json.sample. The version will be automatically injected.

LGTM.
I even tested it. It also copies config.json.sample to dist, which probably was not intended but is harmless anyways (in fact, we have a complete app tree that is probably useless there, so one more file...) :-)

@ancorgs Oh, I added the sample file at the end. I'll fix it before merging.

Fixed and merged.


Use makefile to avoid potential deduplication of build.rs

https://github.com/sagiegurari/cargo-make is pretty cool as well.


Add detailed description and metadata to schema examples

Updated JSON, PHP, and YAML schema examples to include a detailed description and name metadata. Also added a full_clean.yml file with the comment lines removed. Updated the ErrorSuite.php, Schema.php, Utils.php, ValidateCsvTest.php classes to reflect these changes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
920

Relevant Lines:
965

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
920

Relevant Lines:
965

💛  - Coveralls


chore(deps): revert fullcalendar

Probably it's just the @fullcalendar/vue update, but I'm reverting all three just to be sure. We need to figure out how to use the other syle-loader later

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


QUANTIFY_SALMON:SALMON_TXIMPORT fails with Fatal error: cannot create 'R_TempDir'

$ nextflow run -profile test,docker main.nf
...
-[nf-core/rnaseq] Pipeline completed with errors-
Error executing process > 'NFCORE_RNASEQ:RNASEQ:QUANTIFY_SALMON:SALMON_TXIMPORT'

Caused by:
  Process `NFCORE_RNASEQ:RNASEQ:QUANTIFY_SALMON:SALMON_TXIMPORT` terminated with an error exit status (2)

Command executed:

  salmon_tximport.r \
      NULL \
      salmon \
      salmon.merged

  cat <<-END_VERSIONS > versions.yml
  SALMON_TXIMPORT:
      r-base: $(echo $(R --version 2>&1) | sed 's/^.*R version //; s/ .*$//')
      bioconductor-tximeta: $(Rscript -e "library(tximeta); cat(as.character(packageVersion('tximeta')))")
  END_VERSIONS

Command exit status:
  2

Command output:
  (empty)

Command error:
  Fatal error: cannot create 'R_TempDir'

Work dir:
  /working/nf-core-rnaseq/work/40/f45cae46f1d2a5deb898e2d55b6dbb

Tip: you can try to figure out what's wrong by changing to the process work dir and showing the script
file named `.command.sh`

Check Documentation
I have checked the following places for your error:

[ ] nf-core website: troubleshooting
[ ] nf-core/rnaseq pipeline documentation

Description of the bug

Steps to reproduce
Steps to reproduce the behaviour:

Command line:
See error:

Expected behaviour

Log files
Have you provided the following extra information/files:

[ ] The command used to run the pipeline
[ ] The .nextflow.log file

System

Hardware:
Executor:
OS:
Version

Nextflow Installation

Version:

Container engine

Engine:
version:

Additional context

Can confirm it is a Docker-on-Mac issue, everything runs fine on a fresh linux EC2 instance with Docker.

There's a scale issue there. The EC2 instance likely has a bigger /tmp than a Mac laptop. This may not be a Docker-on-Mac thing.

I have more than 100G free in /tmp, and now also see this with a different R-based docker image in nf-core/viralrecon.

Did you manage to fix this @heuermh ? Is it a pipeline related issue?

I'll give this another look after the most recent template pull requests are merged

Will close this for now @heuermh ! But please feel to re-open if the issue persists. Thanks!


pls give me the information

Which OAuth2 provider or API service?

External references, such as API reference guide?

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 18 May 2015 at 4:50

Smart


robot server start --config-repo XYZ changes previously set jumpscale repo globally

Issue migrated from https://api.github.com/repos/zero-os/0-robot/issues/236, opened by @yveskerwyn
When starting a zero-robot like this:
config_repo='ssh://git@docs.greenitglobe.com:10022/yves/my_jsconfig'
zrobot server start --listen :6600 --template-repo $zos_templates --data-repo $data_repo --config-repo $config_repo

the path of the config repo is changed for the whole system (globally):
j.tools.configmanager.path
-> '/Users/yves/code/docs/yves/my_jsconfig'

This while existing JumpScale interactive shells still use the previously set config repo:
j.tools.configmanager.path
->  '/Users/yves/code/docs.grid.tf/yves/jsconfig'  ## NOTE THE DIFFERENCE

Expected behavior was that the robot would use a sandboxed config repo, instead of changing it globally.

commented by @zaibon
@yveskerwyn This behavior is the expected behavior. I don't know what a sandboxed config repo is. That sounds like something that could be used here indeed to help development. But in production this is not a real problem anyhow.


Support "github.io" group id for jitpack.io version badge

Currently, the jitpack.io version badge only supports the "com.github" group id for GitHub projects. This pull request adds support for "io.github" as group id. On the jitpack.io side, support was added in jitpack/jitpack.io#1102. The version badge now uses the "io.github" variant if selected. If just "github" is selected, "com.github" is used as the group id (which is compatible with existing badges).
The pull request also enables options that contain dots in the drop-down list of the web frontend.

Warnings

:warning:
This PR modified service code for jitpack but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @pweisenburger!

  Generated by :no_entry_sign: dangerJS against e9b862b532b6210e8a8f30525312e11c5b3832c5

Thank you for submitting a PR! However, I think I'd prefer to see a different approach that doesn't require modification of the patterns, much in the same way that we didn't do so for the com.github group id. I think adding a new option to the route param like githubio, iogithub, github-io, etc. should suffice, which we can then map accordingly.
Also, we should add a new test that has the io variant of the group id

Thanks! However, it's unclear to me what a new option (like githubio) would mean when the vcs part of the pattern is set to, say, bitbucket. It seems to me that both the new option and vcs are not orthogonal concepts but describe, in fact, the exact same parameter. But maybe I got the suggestion wrong.

It seems to me that both the new option and vcs are not orthogonal concepts but describe, in fact, the exact same parameter. But maybe I got the suggestion wrong.

Yes, I think you've misunderstood my suggestion of adding a new option/variant to the existing route parameter as meaning a new route parameter should be added. I was suggesting the former, and we definitely do not want to do the latter.
The more I've thought about this though, the less inclined I am to have us continue to surface the group id via a "version control system" parameter which we then attempt to map to an actual group id. All that does is save the user from typing a couple keys and creating blocking problems for new group ids as in the case you reported here.
Would you have any concerns if we pivoted to allowing the user to just provide the group id directly via a single route parameter?
This would be an adjustment to the badge route, but we have a widely used mechanism to allow us to adjust routes without breaking existing badges via our redirector's, one of which is already in place for our JitPack badge so it would be pretty easy to add another entry

@pweisenburger have you had a chance to read through @calebcartwright's latest comment? It would be great to get this landed. 😉

Given that there hasn't been any activity for a while, I'm going to close this pull request. Feel free to reopen it if you get a chance to look into it again. 😉
If someone else wants to pick this up, you can simply fetch the commits by running git fetch origin pull/7281/head:pr-7281. Address the pending review comments and open a new pull request with a co-authored trailer included in the commit message to give attribution to the original author.


Weblate translating issue

Some content isn't translatable currently, as it says it's in maintenance mode.

Thank you. It's unlocked now. I am still learning how Weblate integrates with GitHub! It's quite complex. Very likely there will be more hiccups... sorry in advance. :)


Fix Crossfade ids

Codecov Report

Merging #722 (c69fe90) into main (026b0a2) will decrease coverage by 0.03%.
The diff coverage is 62.50%.

:exclamation: Current head c69fe90 differs from pull request most recent head 683545f. Consider uploading reports for the commit 683545f to get more accurate results

@@            Coverage Diff             @@
##             main     #722      +/-   ##
==========================================
- Coverage   69.94%   69.91%   -0.04%
==========================================
  Files         113      113
  Lines        5544     5554      +10
  Branches     1105     1109       +4
==========================================
+ Hits         3878     3883       +5
- Misses        906      909       +3
- Partials      760      762       +2

Impacted Files
Coverage Δ

...lin/com/alexvanyo/composelife/ui/ComposeLifeApp.kt
67.92% <62.50%> (+0.16%)
:arrow_up:

...o/composelife/donotkeepprocess/DoNotKeepProcess.kt
13.63% <0.00%> (-2.16%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Pricing period not specified

This page indicates Each active committer to at least one repository with Advanced Security enabled consumes one license and costs $49. Is that a one off fee of $49, a monthly fee of $49, an annual fee of $49 or something else?

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

ID: cf1a4901-ad2d-e6d5-6d69-95d19c26e112
Version Independent ID: cf1a4901-ad2d-e6d5-6d69-95d19c26e112
Content: Billing for GitHub Advanced Security for Azure DevOps - Azure Repos
Content Source: docs/repos/security/github-advanced-security-billing.md
Service: azure-devops
Sub-service: azure-devops-integration
GitHub Login: @juliakm
Microsoft Alias: jukullam

@MatthewSteeples -- Matthew, thank you for your question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@juliakm -- Julia, please look into this issue.

Thanks, we are updating this shortly.


metadata.json

I have downloaded the DFDC dataset, but metadata.json  wasn't  found, can you share the download link of this file, thank you very much

same problem

It should be available if you download the dataset from Kaggle https://www.kaggle.com/c/deepfake-detection-challenge/data


Deprecated version of the Android embedding

Describe the bug
The plugin countdown_flutter uses a deprecated version of the Android embedding.
To avoid unexpected runtime failures or future build failures, try to see if this plugin supports the Android V2 embedding. Otherwise, consider removing it since a future release of Flutter will remove these deprecated APIs.
If you are the plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.
To Reproduce
Steps to reproduce the behavior:

Add the plugin to pubspec.yaml
Run flutter pub get
See error

Expected behavior
Should run pub get without warning.
Screenshots

Additional context
If you are the plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.

Same issue
this plugin need to be migrated to Android Embedding V2


Observation count in user profile cached but not directly invalidated when updated

Describe the bug
The observation count in the user profile is cached and not directly updated when new observations are added.
To Reproduce
Steps to reproduce the behavior:

Go to your profile page
Check observation count
Add observations (in my case it was 2)
Check observation count again

Expected behavior
The observation count in the profile should be updated with the new observations.
Screenshots

Context

OS: NixOS 22.11.4588.93fddcf640c
Browser: Google Chrome 114.0.5735.133

Additional context
I had 658 observations, added 2, then it was 659.
The page with my own observations had the correct value.
After 10 minutes, it was still not updated to the correct value.
After 17 minutes it was. Maybe the cache TTL is 15 minutes?
The cache should be invalidated when the number of observations (or any other number displayed) changes. Or don't cache at all. A database can also cache.
Related: https://github.com/inaturalist/inaturalist/issues/2835
cc @kueda

Duplicate of #1114, but this definitely has a lot of good additional context, so thanks for that, @davidak.


[BEAM-11936] Fix errorprone UnusedVariable in Runners

Fixed errorprone warning listed here:
UnusedVariable: The presence of an unused variable may indicate a bug. This check highlights private fields and parameters which are unused and can be safely removed without considering the impact on other source files.
For reference https://errorprone.info/bugpatterns
Split from #15742

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

Run Java PreCommit

R: @kennknowles do you have time to review this?

Run Java PreCommit

Thank you both!

Thanks Kenn!


VM import for VMware has no helpful error handling

Version Info
docker build 4.4.2019
Describe the bug
If an error occurs during import, the subscription remains in status pending. There is no error description. A stack overflow is logged in the oscm-app log file. Was tested e.g. with wrong Subscription Id, Instance ID, not suitable organizationID
How to Reproduce
Steps to reproduce the behavior:

Go to Marketplace and select a service
Click on "Get it now"
set Import of existing VM`s
choose a VM instance in vsphere from another organization
type "Subscription name" of  this VM
type "Name of VM" of this VM Instance
click next and subscribe

Observed behavior
Subscription status is pending.
oscm-app log file:
Caused by: java.lang.StackOverflowError
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
at org.oscm.app.vmware.business.statemachine.CreateActions.get(CreateActions.java:73)
Expected behavior
I expect a usable error message like : no acces rights or Instance not found if the name is not correct.

@cworf91
Thanks for fixing the stackoverflow error. The error message is logged correctly now.
It would be good, if an error notification is given in the marketplace portal or in email to the customer. The subscription interface is designed to handle this.
Please check why the abortAsyncSubscription  is not invoked.

https://github.com/servicecatalog/oscm/pull/284

Dear user,
The current operation on subscription 'xxx' for customer 'PLATFORM_OPERATOR' has been suspended (Instance: '1556007920907').
Reported problem: null
The error message is still not helpful, if the subscriptionID is not valid. I would expect a message like:
The provisioning process for subscription 'xxx' for customer 'PLATFORM_OPERATOR' has been aborted (instance: '1556007505565'), because the corresponding virtual system could not be found in the backend system. Please ignore this message, if the subscription is already terminated.
Reported problem: Failed to find the VM "xxx". Therefore it can not be imported.

https://github.com/servicecatalog/oscm/pull/293
https://github.com/servicecatalog/oscm/pull/295

Throws now AbortException with the errormessage "Failed to determine the access info for VM instance"


meta: fix stage numbering in bootstrapping docs

The stage numbering was recently changed from 1...3 to 0...2, but a couple places were missed.

Hello!
One or more of the commit messages in this PR do not match the Jakt code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

Commit message has been fixed!


zoom-us: 5.11.{1.8356,3.3882} -> 5.11.9.{4300,10046}

Description of changes
Zoom started demanding I upgrade to >=5.11.9 the other day, so here's a bump. Someone will need to test this on x86_64-darwin and aarch64-darwin since I don't have access to those platforms - I just downloaded the appropriate files from the website and swapped in their hashes. The x86_64-linux version seems to work fine.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

BTW are you aware you can use the passthru.updateScript with:

Unfortunately the update script doesn't work because Zoom has changed their page layout; for the Linux download page, the version number is not printed onto the screen until you click the dropdown menu and select "Other Linux". I figured I'd at least update the versions in a quick PR and fix the update script later. Also the update script seems to only update the version for my own platform (Linux) and not for other platforms. I figured Mac users would like an update too :)

Ah good, it looks like they have a simple REST endpoint where we can get all the info we need ([1], [2]) without HTML-munging let alone JS-munging or Selenium or whatever... fixing update.sh should be easy too.
Should I make it update versions and hashes for all platforms, or is there a reason it's only supposed to do an update for the platform it's being run on?

Should I make it update versions and hashes for all platforms, or is there a reason it's only supposed to do an update for the platform it's being run on?

Making the update script update all platforms at once would be great, but we can live without this feature. In any case, I still would like to make sure the update script works properly in this PR, at least for a single version.

So the update script worked as expected? Without any change?

No, I need to fix it. I will push an update to the PR tomorrow. Thanks for the feedback so far :)

Agreed, I was already planning to do that when I saw your comment in fact :)
I've pushed an update which hopefully simplifies this satisfactorily. update.sh now updates all three platforms and can be run either directly or via the update.nix incantation @doronbehar mentioned, from any directory. In the meanwhile, Zoom has released yet another new version on Linux, so I bumped to that as well (<IP_ADDRESS>00).

Thanks for working on the update script.


add support for metricflow

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (4271c40) 80.88% compared to head (e5a047c) 80.87%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
- Coverage   80.88%   80.87%   -0.02%
==========================================
  Files          46       47       +1
  Lines        5630     5646      +16
  Branches     1384     1386       +2
==========================================
+ Hits         4554     4566      +12
- Misses        828      830       +2
- Partials      248      250       +2

Files
Coverage Δ

metrics_layer/core/parse/__init__.py
100.00% <100.00%> (ø)

...rics_layer/core/parse/project_reader_metricflow.py
73.33% <73.33%> (ø)

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


SLP tokens still unsendable

Tested again after the fix, on chrome + firefox
Sufficient BCH and SLP tokens but gives following error:
"ALERT: Request failed with status code 400"
I suspect the bug is due to failed transactions (red colored in the log).
When you click open transaction
Ie: bitcoincash:qr0mll0xvr2q9gp9m5h6lvd6ayndp8xtqq3ls2czyy to simpleledger:qzg5y7qtzjqfmm3wefxp7q2rq9asm9q6qshrc3rd8a
link opens to: https://explorer.bitcoin.com/bch/tx/undefined
BCH sending from wallet still works fine, same as other SLP tokens, just affected token with red offending transaction

Thanks for the details @tweetyjade. I'm preparing to wade through error logs and I'll search for those addresses and error codes.

Cheers
Still having status 400 error post Nov 29th extension update @christroutner

looking into the Firefox browser console, i see this error. hoping this helps the debugging
"XML Parsing Error: no root element found
Location: https://rest.bitcoin.com/v2/rawtransactions/sendRawTransacti…6d22c3757c74e3049decc11df93f3c014e3457afeba3c2695fa8b4175d9f"


Questions: is there any ways to compile to wasm with emscripten?

I want to simulate some code with jerryscript using wasm through emscripten. but I found build options without config for emscripten. so there is any ideas to build with emscripten?

Quite some time ago (in 2016, yikes!), I managed to build JerryScript for emscripten. Here are the notes from those days:

Environment:

OS X 10.11.6 (Darwin 15.6.0)
Emscripten 1.35.0 with Node.js v4.1.1 (from: http://kripken.github.io/emscripten-site/docs/getting_started/downloads.html)
JerryScript commit 9915307 (Tue Oct 4 11:51:03 2016 +0200)

Commands:
cmake -H. -Bbuild -DJERRY_LIBC=OFF -DJERRY_LIBM=OFF -DCMAKE_C_COMPILER=emcc -DENABLE_LTO=OFF -DENABLE_ALL_IN_ONE=ON -DENABLE_STRIP=OFF -DCMAKE_EXECUTABLE_SUFFIX_C=.js
make -C build
cd build/bin && cat ../../tests/hello.js | node jerry.js --no-prompt

Note:

Only REPL works. Passing JS files as arguments (e.g., node jerry.js ../../tests/hello.js) fails with "Error: failed to open file".

Since then, quite some things have changed: sources, build system, emcc, node, ... The above will not work out of the box for sure. I'd start the experiment with the following build configuration:
rm -rf build/
cmake -H. -Bbuild -DCMAKE_C_COMPILER=emcc -DENABLE_LTO=OFF -DENABLE_AMALGAM=ON -DENABLE_STRIP=OFF -DCMAKE_EXECUTABLE_SUFFIX_C=.js
make -C build VERBOSE=1

The above commands get me quite far, but linking fails as I don't have emsdk installed, and the Ubuntu-provided emcc has llvm triplet issues. I hope that someone can pick it up from here.

there is any way to make a toolchain to  build? I make a toolchain file and config in command line in build.py, but when I build, it always use windows build tools, here is my toolchain code
set(CMAKE_SYSTEM_NAME emscripten)
set(CMAKE_SYSTEM_PROCESSOR wasm)

set(CMAKE_C_COMPILER emcc)

and I found any other toolchain in windows don't work either。

More details are needed to find the root of the problem. How did you invoke build.py? What was the output?
Moreover, finding the correct configuration options for an emscripten build is independent from using a toolchain file. So, I suggest to try and configure cmake directly and when the build finally works, move the cmake command line options into a toolchain file.

currently I can successfully use emscripten sdk to build lib, here is my code
emcmake cmake -Bbuild -DJERRY_DEBUGGER=ON -DJERRY_LINE_INFO=ON  -DJERRY_CPOINTER_32_BIT=ON -DJERRY_SYSTEM_ALLOCATOR=ON
emmake make -C build

but I have not tested yet.


Requirements

[ ] User Interface

[ ] Display a cursor on x-y coordinate plane
[ ] Playback file browsing
[ ] Settings activity

[ ] Signal Processing

[ ] Access microphone / headset / earbuds for input stream
[ ] Compute FFT on input signal
[ ] Extract R1, R2, R3 and pitch from input stream

[ ] Data storage

[ ] Evaluate importance of high fidelity for recordings - what bit rate is necessary for useful analysis?
[ ] File system access
[ ] Encoding / Decoding

Stretch Goals

[ ] Ability to play back input audio on configurable delay

[ ] Thorough unit testing
[ ] Github actions testing + passing
[ ] Documentation


Add example to documentation for spitfire

I noticed in the documentation it says nvim has a spitfire flag but I can't seem to find the right place to put it. Could you share an example lazy config with spitfire turned on?

According to this line https://github.com/elixir-tools/elixir-tools.nvim/blob/b465f6aff50257fa466de3886fc3e7de2dcff0de/lua/elixir/nextls/init.lua#L40 and the note on https://www.elixir-tools.dev/docs/next-ls/spitfire/
it should be sufficient to use it with:
elixir.setup {
  nextls = {
    enable = true, -- defaults to false
    spitfire = 1
    init_options = {
      ...


Explicitly mention bower in the dev instructions

As a response to #73 I added the bower install step explicitly to the dev instructions in the README. I think it will help JS newcomers.

Thanks!


Some tests fail randomly when using the trio runner

test_open_proc_SIGINT_can_be_ignored, test_open_proc_SIGINT_while_running and test_open_proc_with_trio_KeyboardInterrupt_while_running all fail randomly when using the open_in_process_with_trio() runner. That happens because sometimes the runner raises a RuntimeError like shown below
<function test_open_proc_SIGINT_while_running.<locals>.do_sleep_forever at 0x7f214a0c10d0> raised an unexpected exception: Cancel scope stack corrupted: attempted to exit <trio.CancelScope at 0x7f214a0f6280, active, cancelled> in <Task '__main__._do_async_fn' at 0x7f214a0f28e0> that's still within its child <trio.CancelScope at 0x7f214a0f6460, active, cancelled>

This is probably a bug in your code, that has caused Trio's internal state to
become corrupted. We'll do our best to recover, but from now on there are
no guarantees.

Typically this is caused by one of the following:
  - yielding within a generator or async generator that's opened a cancel
    scope or nursery (unless the generator is a @contextmanager or
    @asynccontextmanager); see https://github.com/python-trio/trio/issues/638
  - manually calling __enter__ or __exit__ on a trio.CancelScope, or
    __aenter__ or __aexit__ on the object returned by trio.open_nursery();
    doing so correctly is difficult and you should use @[async]contextmanager
    instead, or maybe [Async]ExitStack
  - using [Async]ExitStack to interleave the entries/exits of cancel scopes
    and/or nurseries in a way that couldn't be achieved by some nesting of
    'with' and 'async with' blocks
  - using the low-level coroutine object protocol to execute some parts of
    an async function in a different cancel scope/nursery context than
    other parts
If you don't believe you're doing any of these things, please file a bug:
https://github.com/python-trio/trio/issues/new

Ick.  I've run into this error from trio elsewhere and I never figured out how to properly address it...

Yeah, I've had to deal the same issue in https://github.com/ethereum/trinity/blob/d80a3b1fe5661d5108c9f10ddd077312ad4661a9/p2p/discovery.py#L1349-L1368
Just need to find what's causing it here

I can't seem to find out what is causing this. At first I thought it was the signal asynciterator in https://github.com/ethereum/asyncio-run-in-process/blob/d0c0d16fc33fbf28e329df54811f37fbc66e0f5c/asyncio_run_in_process/_child_trio.py#L45
but even after removing that it still happens

One thing that makes the failure disappear is changing the sleep(0) below to, say, a sleep(0.01): https://github.com/ethereum/asyncio-run-in-process/blob/d0c0d16fc33fbf28e329df54811f37fbc66e0f5c/tests/core/test_open_in_process.py#L93-L97
I've no idea why, though, but it looks like another race condition that's only triggered by our tests that run not-very-realistic functions in the subprocess and kill the subprocess immediately after they're started


azure-cli: 2.55.0 -> 2.56.0

Description of changes
azure-cli is currently broken, likely after
https://github.com/NixOS/nixpkgs/pull/279138
https://github.com/NixOS/nixpkgs/pull/279143
(somewhat suspicious that it built on the PR?)
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.05 Release Notes (or backporting 23.05 and 23.11 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.

Currently tests fail:
=========================== short test summary info ============================
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_cloud_is_registered - AssertionError: False is not true
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_get_active_cloud_name_default - AssertionError: 'AzureCloud' != 'TestCloud'
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_get_clouds_concurrent - azure.cli.core.cloud.CloudNotRegisteredException: The cloud 'AzureCloud' is...
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_get_known_clouds - azure.cli.core.cloud.CloudNotRegisteredException: The cloud 'AzureCloud' is...
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_modify_known_cloud - azure.cli.core.cloud.CloudNotRegisteredException: The cloud 'AzureCloud' is...
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_remove_known_cloud - azure.cli.core.cloud.CloudNotRegisteredException: The cloud 'AzureCloud' is...
FAILED azure/cli/core/tests/test_cloud.py::TestCloud::test_switch_active_cloud - azure.cli.core.cloud.CloudNotRegisteredException: The cloud 'AzureGermanClo...
FAILED azure/cli/core/tests/test_extension.py::TestExtensions::test_add_extension_with_preview - AttributeError: 'NoneType' object has no attribute 'startswith'
FAILED azure/cli/core/tests/test_extension.py::TestExtensions::test_add_extension_without_preview - AttributeError: 'NoneType' object has no attribute 'startswith'
FAILED azure/cli/core/tests/test_extension.py::TestExtensions::test_add_list_show_preview_extension - AttributeError: 'NoneType' object has no attribute 'startswith'
FAILED azure/cli/core/tests/test_extension.py::TestExtensions::test_add_preview_extension_by_default - AttributeError: 'NoneType' object has no attribute 'startswith'
FAILED azure/cli/core/tests/test_extension.py::TestExtensions::test_update_extension_with_preview - AttributeError: 'NoneType' object has no attribute 'startswith'
========== 12 failed, 236 passed, 3 deselected, 22 warnings in 3.67s ===========

I'm aware of these test issue however I haven't managed to find the root cause yet, reverting both PRs you mentioned allow the test to pass ?
I'll try that in order to check.

I'm aware of these test issue however I haven't managed to find the root cause yet, reverting both PRs you mentioned allow the test to pass ? I'll try that in order to check.

No, test failure seems to be caused the update.
I'v reverted the PRs on master: builds
I'v cherry-picked the update commit on top: test failure

@katexochen have you noticed that on master currently 2.55.0 does not build anymore ?
It seems some runtime dependencies are missing for some azure-mgmt packages.
Notably msrest. I think that's one issue.
But it's not related to the failing tests.

@katexochen have you noticed that on master currently 2.55.0 does not build anymore ?

Yes, that's the failure I mentioned in the description od this PR,  caused by the 2 mentioned PRs and fixed by the first commit of this PR.

@katexochen have you noticed that on master currently 2.55.0 does not build anymore ?

Yes, that's the failure I mentioned in the description od this PR,  caused by the 2 mentioned PRs and fixed by the first commit of this PR.

I can't seem to figure out the issue that is making the test failing, right now, it seems so weird.
For the same dependencies, 2.55.0 tests are passing but 2.56.0 is not, both tests are the same, how the heck ? And I doubt azure-cli would get released with broken tests.
I believe the azure-cli tests have been passing test for this release, however I can't access the detailed build logs : https://github.com/Azure/azure-cli/pull/28121/checks?check_run_id=20112571131 so I'm really not sure what's changing the behavior's result for this release.

I disabled the python pytests and the CLI does build and the self-test works, should we disable them for now and wait for an anwser upstream as I fear that upstream might not rely on their test but most likely the az self-test output.
But I need to investigate further.
What do you think about this @katexochen ?

I rebased to include the changes from https://github.com/NixOS/nixpkgs/pull/281213 and added ignores for the two test files that previously failed.
azure-cli-core seems to build now, but the azure-cli self-check fails:
Running CLI self-test.

Loading all commands and arguments...
Error occurred loading commands!

The command failed with an unexpected error. Here is the traceback:
No module named 'azure.mgmt.compute.v2023_09_01'
Traceback (most recent call last):
  File "/nix/store/2kx3ab6hfqk5rbhk76bnliygmin7isvn-python3.11-knack-0.11.0/lib/python3.11/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
             ^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
           ^^^^^^^^^^^^^^^^^^
  File "/build/azure-cli-2.56.0-src/src/azure-cli/azure/cli/command_modules/profile/custom.py", line 183, in check_cli
    raise ex
  File "/build/azure-cli-2.56.0-src/src/azure-cli/azure/cli/command_modules/profile/custom.py", line 178, in check_cli
    create_invoker_and_load_cmds_and_args(cmd.cli_ctx)
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/file_util.py", line 74, in create_invoker_and_load_cmds_and_args
    invoker.commands_loader.load_arguments()
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/__init__.py", line 516, in load_arguments
    cmd.load_arguments()  # this loads the arguments via reflection
    ^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 318, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/nix/store/2kx3ab6hfqk5rbhk76bnliygmin7isvn-python3.11-knack-0.11.0/lib/python3.11/site-packages/knack/commands.py", line 104, in load_arguments
    cmd_args = self.arguments_loader()
               ^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py", line 282, in arguments_loader
    arguments = self.load_getter_op_arguments(self.getter_op_path)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py", line 70, in load_getter_op_arguments
    op = self.get_op_handler(getter_op_path)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/kj31f6d9d7swbl8f6981wniabmcjf4r4-python3.11-azure-cli-core-2.56.0/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py", line 59, in get_op_handler
    handler = import_module(mod_to_import)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/asiphbpiy2gmidfm3xbwcikayhs66289-python3-3.11.7/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1126, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1140, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'azure.mgmt.compute.v2023_09_01'

I disabled the python pytests and the CLI does build and the self-test works

@AkechiShiro did you just disable the pytests of azure-cli-core?

No I disabled the whole pytests for the package but we should try and disable only those the azure-cli-core and see if that still builds (it should) I think we can move forward that way, while waiting some clarification on what's going on upstream.

you're missing a bump for the az-mgmt-compute dependency that's why the self-test of the cli are failing.

Thanks, I indeed missed that (and another bump). Should work now.

Something must be wrong with how pytests are being run.

Yes, but I still don't have a clue what's wrong with them.

Did you check this comment : https://github.com/Azure/azure-cli/issues/28191#issuecomment-1902879710 from upstream ?
It seems something is wrong with the pytest in Python.

Did you check this comment : https://github.com/Azure/azure-cli/issues/28191#issuecomment-1902879710 from upstream?

Saw it, but I don't get it.

@AkechiShiro are those tests a blocker, or can we proceed with this PR and fix the tests when we figured it out?

@AkechiShiro do you have time for a review, so we can get this merged?

@katexochen I'll try and find time to review it, this week-end, if I cannot, I'll let you know as soon as possible
So far, my understanding is that testing this PR shows :

self-test of the CLI passes
pytest are also green besides the ones that were added to be ignored.

Do you agree ?

I'll try and find time to review it, this week-end, if I cannot, I'll let you know as soon as possible

Thanks! :)

So far, my understanding is that testing this PR shows :

self-test of the CLI passes
pytest are also green besides the ones that were added to be ignored.

Do you agree ?

Yes, exactly.

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

2 packages built:

azure-cli
azure-cli.dist


Default WebKit Accept header causes problems with some web services/servers

<EMAIL_ADDRESS>commented:

What steps will reproduce the problem?

Run the following script:

if (phantom.state.length == 0) {
phantom.state = '1';
phantom.open( 'http://gdata.youtube.com/feeds/api/videos?author=geologicrecords&v=2&prettyprint=true&start-index=1&max-results=5' );
} else {
console.log( phantom.content );
phantom.exit();
}

Fetch the same URL using another tool such as WGET or a recent build of Firefox to see correct results.

What is the expected output? What do you see instead?
Should show a pretty-printed ATOM feed of the data about 5 videos on YouTube.  Instead, the data is all HTML-escaped for literal display and is surrounded by  and  tags.
What version of the product are you using? On what operating system?
PhantomJS 1.1 Mac OS/X 10.6.7
Please provide any additional information below.
I was able to isolate the problem to the Accept: header being sent by WebKit.  If you Google WebKit Accept header you will see there has been considerable complaints that WebKit is using a bad default, taken from an old build of Firefox.  WGET works correctly because it sends Accept: /.
WebKit is prioritizing text/plain too high (and making other errors), so in this case Google's servers sends the result back as "text/plain" and WebKit tries to escape it for easy display.
At the very least, we should update the default for the Accept: header that is used by default in PhantomJS.  If <a href="http://code.google.com/p/phantomjs/issues/detail?id=77">Issue 77 is resolved in a way that allows setting the Accept: header, it should fix this bug as well.  (But I think it would be best to change the default as well).

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #112.
:star2:   2 people had starred this issue at the time of migration.

PhantomJS 2.0's Accept: header for top-level documents is consistent with that generated by Chromium and Firefox.


Generate Routes programmatically

Hi!
Are there any way of generate programmaticaly based on route name?
I've noticed that  component don't render a  element , and for some seo constraints i need render it with href on server side rendering. Currently we have a component wrapped by  element and we have to fill href property.
Thanks a lot

I have identified than if you put an <a> element inside <Link> object, next-routes inject href. I think that this is enough for our case, although it could be interesting have service to generate programmatically route href.

Also came up with this one and I really don't want to write an own function for compiling routes to a simple string ...

@adrianalonso I found a way which is working after digging the source code and trying things out. It isn't the prettiest syntax - but the more important thing - not documented (cc @fridays @elliottsj) ... Here a short example:
import routes from 'next-routes';

export default routes()
  .add('test', '/:lang(en|de)/test/:step', 'TestPage');

import routes from './my-routes-above';

routes.findByName('test').toPath({
  lang: 'en',
  step: 'first-step'
});


Why is Psbt::sighash_ecdsa public?

The method takes &mut SighashCache which doesn't enforce that the transaction is equal to unsigned_tx which looks broken. Judging from other methods around it should've been private.
Note: I have some idea how to avoid cloning the transaction during signing but this method being public significantly restricts the design.

I've worked on that code so I took a look, I don't see any reason that it should be public. IMO it would be fine to make it private. FWIW I'm re-writing the psbt stuff at the moment, I have not gotten to the signing API yet though. It will likely need some work to add taproot support.

OK, let's make it private. I think my upcoming PR might help with signing and other things.


Refactor scanning and parsing

This is still work in progress.
As it stands at the moment, this is ABI breaking and therefore would be for qpdf 12.

I added the abi label as you see. You can always remove it if it makes sense to do so in the future. I haven't scrutinized this yet. BTW, I think if you rebase, your change to enable C++17 will disappear since I pushed that up separately. It may be a couple of weeks before I have significant time for qpdf. This weekend and next weekend are quite full. If I have qpdf time, it will be just to take care of small issues. Thanks as always for your efforts.

I am not sure whether it is clear from the commits where I am going. If not, let me know and I'll summarize. In any case, this will need more documentation comments before it gets finalized.
I can see the possibility that there will be a case for API breaking changes, in which case this could be modified to go into 11.2.

Btw, is there a way to turn of sanitizer overflow checking for specific sections of code?

Btw, is there a way to turn off sanitizer overflow checking for specific sections of code?

I don't think so, but why would you want to?

Because whether or not the variable overflows makes no difference in this case. I (have to)  detect the risk of an overflow and handle it. Actually preventing the overflow adds overhead with no benefit

The sanitizers only complain if there is an actual overflow, and if they complain, then there is an actual overflow. I'm not sure I understand exactly what situation there could be where it makes no difference if the variable overflows. Anyway, even if it really did make no difference, I would start getting bug reports all over the place and rejections from places. For example, both Red Hat and Google wanted at() to be used instead of operator[] for vector because of static analyzers. People are willing to pay a performance price for overflow checks.