CLOUDSTACK-8639:fixing calculation mistakes in component/test_ss_doma…

test result
Test secondary storage limit of domain and its sub-domains ... === TestName: test_04_create_template_delete_account | Status : SUCCESS ===
ok

Ran 1 test in 175.563s
OK

I browsed the code and found out that this "templateSize = int(int(templates[0].size) / (1024**3))" was already been used in other places. So, your changes does make sense to me and therefore you get a LGTM :+1:
Cheers,
Wilder

LGTM (we could decide to make this a generic macro type of construct so we guarantee unified behavior. on the other hand this is not building a rocket...)


Time entry listing design changes

Switch the vertical locations of description row and project row

Superseded by #1600.


possible to convert this to char[] instead of void* (char*)?

possible to convert this to char[] instead of void* (char*)?
doing so will reduce memory usage by not having to keep keylen.
any reasons why didnt it start with char[] instead? coz str is very limited. (need to use strlen which is not very byte friendly for other data)
great work by the way.

I unfortunately don't understand what you're asking in this issue. To be quite honest, I'm not sure even if you're in the right project. There aren't any character pointers or strings in this codebase.

because there is strlen() which terminates on "0"
i'm writing a golang binding for this and GoBytes has {1,2,3,0,4,5} values and when C sees a 0, it is terminated.
i've gotten it halfway done but i realised you have keylen <- i'm not proficient in C but if we use char[], wont that make keylen unnecessary?

This still does't make sense. Character arrays in C don't have lengths that you can detect like that -- so no, it doesn't. Why don't you use the version of this library that we've written in Go? https://github.com/zmap/zdns/blob/master/cachehash/cachehash.go

@zakird ok i've re-checked all my code, was my issue. i didnt grasp c properly. now i do. it's great piece of software!
really like it a lot. thx for the hard work.


Possible VAE memory leak?

Hi there --
I'm not sure if the issue I'm seeing is in these bindings or in the upstream lib, but I'm observing that when using the high-level API on CUBLAS, that after the __del__ wiring does the fancy stepping to invoke free_sd_ctx when the model gets either garbage collected or explicitly deleted -- not all the VRAM gets released.
After some experimentation, I've noticed that the amount that hangs around is always almost exactly the same amount as gets allocated for the VAE, plus about 100MB. VAE tiling reduced the size of the leak, and doing the VAE phase on the CPU leaves just the 100 or so MB of leftovers.
If I was going to hazard a guess, there's more being allocated in stable-diffusion.cpp's load_from_file() than is getting freed by the free_sd_ctx() call.
If there's anything I can do to help sleuth this out, please don't hesitate to ask.

You're right, I think this might be an issue in stable-diffusion.cpp though. I can recreate the same behaviour in stable-diffusion.cpp by adding a sleep at the end of the CLI code main.cpp file inside examples\cli\main.cpp. Even though it seems everything should have been unloaded, not all the VRAM is released until the code finally exits. This is assuming the CLI code includes everything to properly unload the VRAM but it's also possible I have missed something somewhere.
I will create an issue in the stable-diffusion.cpp repo and see if anyone has any suggestions.


add homebrew installation details

I think we can close this in favor of https://github.com/chainguard-dev/apko/pull/689?

sounds good


Initializing SPI randomly hangs

Hi, I'm seeing random hangs when I initialize the SPI bus. It seems to happen ~10% of the time when running from debug, and a bit less frequent when running from release. It's hanging on this line.
My code:
spi::Config::new(mclk, sercom3, pads, clock.freq())
        .baud(100.mhz()) // Max speed for ADC
        .length::<spi::lengths::U2>() // 16 bit
        .bit_order(spi::BitOrder::MsbFirst)
        .spi_mode(spi::MODE_0) // CPOL = 0, CPHA = 0
        .enable()

and here's the stack backtrace after I halt the app when hung:
stack backtrace:
   0: atsamd_hal::sercom::spi::reg::Registers<S>::set_length
        at C:\Development\Vital\device-sw\deps\atsamd\hal\src\sercom\spi\reg.rs:108:15
   1: atsamd_hal::sercom::spi::Config<P>::default
        at C:\Development\Vital\device-sw\deps\atsamd\hal\src\sercom\spi.rs:660:9
   2: atsamd_hal::sercom::spi::Config<P>::new
        at C:\Development\Vital\device-sw\deps\atsamd\hal\src\sercom\spi.rs:697:9
   3: bsp::detector::init_spi
        at C:\Development\Vital\device-sw\phoenix-firmware\bsp\src\detector.rs:121:5
   4: app::app::init
        at src\main.rs:228:28
   5: app::app::rtic_ext::main::{{closure}}
        at src\main.rs:11:1
   6: app::app::rtic_ext::main::__rtic_init_resources
        at src\main.rs:11:1
   7: cortex_m::asm::nop
        at C:\Users\MarkBremer\.cargo\registry\src\github.com-1ecc6299db9ec823\cortex-m-0.7.4\src\asm.rs:35:5
   8: main
        at src\main.rs:11:1
   9: Reset

Obviously, if I don't use the 32-bit extension it works ok, but the HAL defaults to using it

@sakian, my first guess would be that the order of initialization matters. Check the datasheet and see what it says on initialization order. I know I ran into that issue once before.
If that turns out to be the cause, then we should probably find a way to handle initialization order for users, so you don't need to comb through the datasheet just to set up a SPI bus.
If that's not the cause, then I'm not sure. I'll have to dig more

Thanks, @bradleyharden, I've rearranged the init order to match what they're looking for and will try it out.

@sakian, any luck?

Unfortunately, I've been tied up on other things. I tried reverting my changes and replicating the issue, but it's quite transient so it didn't happen again. I'll have to wait till it shows up again and reorganize the function call order. It does make sense that it's the issue though because I didn't follow the order specified in the datasheet.
Mark Bremer

From: Bradley Harden @.>
Sent: Sunday, May 22, 2022 3:47:01 PM
To: atsamd-rs/atsamd @.>
Cc: Mark Bremer @.>; Mention @.>
Subject: Re: [atsamd-rs/atsamd] Initializing SPI randomly hangs (Issue #609)
@sakianhttps://github.com/sakian, any luck?
—
Reply to this email directly, view it on GitHubhttps://github.com/atsamd-rs/atsamd/issues/609#issuecomment-1133961474, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVKGVEY3IWXS35RHUHLAERDVLKFLLANCNFSM5V4CBZZQ.
You are receiving this because you were mentioned.Message ID: @.***>

I saw the issue again, and after reorganizing the commands to match the order defined in the datasheet (section <IP_ADDRESS>) the issue went away. It's a very random issue, so I'm not sure if the change actually fixed it, but I'll keep this change in place to see if it happens again.


hop-cli: 0.2.53 -> 0.2.54

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hopinc/cli/releases.
meta.description for hop-cli is: Interact with Hop in your terminal
meta.homepage for hop-cli is: https://github.com/hopinc/cli
meta.changelog for hop-cli is: https://github.com/hopinc/cli/releases/tag/v0.2.54
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.54 with grep in /nix/store/0hmlxy32hk9yjvjn3xj18ry98zy9sv62-hop-cli-0.2.54
found 0.2.54 in filename of file in /nix/store/0hmlxy32hk9yjvjn3xj18ry98zy9sv62-hop-cli-0.2.54

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
hop-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0hmlxy32hk9yjvjn3xj18ry98zy9sv62-hop-cli-0.2.54 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/76c3e7af95ee73dd926c5059f5417b576686edd2#hop-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0hmlxy32hk9yjvjn3xj18ry98zy9sv62-hop-cli-0.2.54
ls -la /nix/store/0hmlxy32hk9yjvjn3xj18ry98zy9sv62-hop-cli-0.2.54/bin

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

1 package built:

hop-cli

Maintainer pings
cc @techknowlogick for testing.


Problems with Mobends

Mobends Mod version: 1.2.1-19.12.21
Describe the bug
I looked at the previous iusse about Mobends. In the case of wearing MWC armor, the first-person grip will take effect.
(Only shirt armor is useful)
I would like to ask, is it possible to fix it without wearing armor to work?
In addition, there is no aiming animation in third person

[X] I reproduced this issue with as few other mods as possible installed.
[ ] I am unable to reproduce this issue consistently.

Expected behavior
First-person gun gripping animations do not require armor to take effect.
The third person has aiming animations
Screenshots or/and videos
The situation of not wearing armor in first-person view

In a third-person view, with or without armor. (I'm holding down the right mouse button to aim)

Versions

Modern Warfare Cubed: MWC-0.1-Dev-12
Forge: 1.12.2 - <IP_ADDRESS>60
Java: JDK8
Java JVM: Hotspot

All 3rd person animations have been removed by the Vic4Games team. This is planned to be re-added in the future.

All 3rd person animations have been removed by the Vic4Games team. This is planned to be re-added in the future.

no It's First person animations he's trying to say,
We do not have compatibility with mo bends sorry.


Fix DetachGlobalRef usage in the async example

After the GlobalRef reimplementation in https://github.com/prevoty/jni-rs/pull/67, the example provided in example/mylib stopped working. This pull request fixes the example by removing the calls to the detach() and attach() methods.

@dmitry-timofeev Apparently, they aren't. It would be awesome to fix that. :upside_down_face:

@DarkEld3r, @dmitry-timofeev,
I think it should be enough to add an [[example]] block to the Cargo.toml, e.g.:
[[example]]
bench = false
name = "mylib"

I can do this as a part of this PR if you'd like :slightly_smiling_face:

I can do this as a part of this PR if you'd like

It would be awesome!


New Release 6.11.0

@sammarks  can you setup a new release, with a new tag? I updated the code, but I don't have perms to update the release.

You do now!


Positining toasts relative to a specific control

im having issues to position toasts on  apage. The options presented by ToastPositions are insufficient for me as they seem to be relative to the entire page and not a specific form element.
I tried using ToastrOptions.Target but that doesnt work either. what does that property do?
im trying this on the latest ntoastnotify build with a .net core 2.2  mvc app.

Please refer to the docs of toastr or noty and see if they meet your requirement. This is simply a wrapper around those libraries.

Closing this as this is beyond the scope of this package.


feat(pinia-orm): Composite primary key can now be used with destroy and onDelete

🔗 Linked issue
resolves #1620

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Adding support for composite key destroy because lookup already works with it. Also adapted onDelete to work with it.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.01% :warning:

Comparison is base (a61e359) 99.84% compared to head (b4631d1) 99.84%.

Additional details and impacted files
@@            Coverage Diff             @@
##           v1.7.0    #1671      +/-   ##
==========================================
- Coverage   99.84%   99.84%   -0.01%
==========================================
  Files          86       86
  Lines        5877     5873       -4
  Branches      494      494
==========================================
- Hits         5868     5864       -4
  Misses          6        6
  Partials        3        3

Files Changed
Coverage Δ

packages/pinia-orm/src/model/Model.ts
100.00% <100.00%> (ø)

packages/pinia-orm/src/query/Query.ts
99.51% <100.00%> (-0.01%)
:arrow_down:

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


azd infra synth Improvements

Opening as a draft of my WIP branch while I finish this up - there are some questions that I have.

synth: Take infra section of azure.yaml into account
azd infra synth was not taking the path to the shared infrastructure
folder inor the name of the root module from the infra of
azure.yaml into account, and instead just always generated files
into the infra folder at the root of the project.
Since these were getting generated to the wrong place, they were
ignored during future azd provision runs.
We now take these values into account when generating filenames and
paths.
synth: Warn when AppHost has changed since generation
This changes adds a commented to files generated by azd infra synth
to include a hash of the manifest that was used to generate the
infrastructure. When we use these files later, we compare the hash
recorded in the file to the hash of the current manifest and issue a
warning.

Still some work to be done here - we need to plumb the warning into the provision side of the house as well.
I'm not sure what the output should look like exactly and the copy is also pretty bad.
Here's the output of azd deploy with this change, where I ran azd infra synth:

I'm sure there's something better we can be saying here.  I'm not sure if we want to tell the user to re-run azd infra synth since that would just cause them to discard any changes they had made "by hand".
One frustrating thing to me is that the warning is written at the end of the operation (and only upon success) but really I would like to issue the warning as soon as I notice it (after reading the yaml template) but we don't have a great way to do that today with the console abstraction and the task abstraction.
Something else that is unfortunate right now is that the warning is overly pessimistic.  For example, in the above I had taken the aspire starter project:
var builder = DistributedApplication.CreateBuilder(args);

var apiService = builder.AddProject<Projects.AspireStarterChanges_ApiService>("apiservice");

builder.AddProject<Projects.AspireStarterChanges_Web>("webfrontend")
    .WithExternalHttpEndpoints()
    .WithReference(apiService);

builder.Build().Run();

and then just added a .WithEnvironment("FOO", "BAR")
to the builder.AddProject<Projects.AspireStarterChanges_Web>("webfrontend") chain.
Ideally, we would have not issued a warning for the apiservice project, since there was no change to the parts of the manifest it cares about, but right now manifest hash generation is very simple - we just take the entire document, remove whitespace and sha256 it.
We also aren't taking the contents of any bicep files referenced by the manifest into account - but really we should (normalizing newlines?)
But - opening this up for discussion.
@davidfowl Did you have thoughts about what you wanted the UX to feel like here?

but right now manifest hash generation is very simple - we just take the entire document, remove whitespace and sha256 it.

The fact that the end to end integration tests failed on each type of OS and each of them computed a different hash makes me think we will need to do something more clever.

@davidfowl Did you have thoughts about what you wanted the UX to feel like here?

I was thinking that it would prompt before doing anything much like azd infra synth prompts today before overwriting files. The prompt would offer 2 choices, "Continue without new changes" and "Re-run infra synth and deploy"

The prompt would offer 2 choices, "Continue without new changes" and "Re-run infra synth and deploy"

The thing I am struggling with, @davidfowl, is that I'm not sure that "Re-run infra synth and deploy" is something reasonable to pick?  If someone did this, it would remove all the customizations they had applied by hand after infra synth, like an "accept theirs" in a merge conflict.  Would we expect you'd just want to deploy that without making any changes or even looking at what got generated?
The idea of the prompt during deployment is interesting, it should just work, but we will need to also look at how this feels in VS when using their Aspire Deployment flow.  When we were just printing messages it made sense to me how this would work in VS, but if we throw up a prompt, I am less certain about how it will work.  We do have prior art here for prompting during the up flow but we should walk through the scenario and validate it both works and the UX is reasonable.
In non-interactive scenarios, like CI, I am guessing we would just "Continue without new changes" and log the warning.  Feels better than aborting the whole operation.

The thing I am struggling with, @davidfowl, is that I'm not sure that "Re-run infra synth and deploy" is something reasonable to pick? If someone did this, it would remove all the customizations they had applied by hand after infra synth, like an "accept theirs" in a merge conflict. Would we expect you'd just want to deploy that without making any changes or even looking at what got generated?

I see, I think you're right. I think it should fail then instead of prompt. It's very likely a mistake and you should be forced to take an explicit action like specify a --overwrite flag, instead of showing a warning and deploying anyways.


Error when running script

Thank you for sharing this. I really need this functionality, but when I run the python script I get this:
File "csv_to_md-file.py", line 17
    self.settings:dict = {}
                 ^
SyntaxError: invalid syntax

Can you advise me what to do please?

Make sure to use an up-to-date Python version. (See the Readme, I tested it on 3.9.4, the package that causes the issue is supported since 3.5)

Make sure to use an up-to-date Python version. (See the Readme, I tested it on 3.9.4, the package that causes the issue is supported since 3.5)

Thank you so much for responding. Not being a programmer, it never occurred to me to check the version. I have now got your script working perfectly for my needs. Very grateful indeed for it and for your support. All best wishes. Edward


Adding Rediess Cache for case search for case ID

Hi @radheshjoshi1  Please add the development tasks

added the redis cache for case service. saving case in redis cache while update, create and search
for search in redis cache defaultfields should be false and caseId should be present.

did the sanity checks in dev @Ramu-kandimalla , ready to test in qa

Hi @Ramu-kandimalla,

Env_[dristi-kerala-dev]
for case search testing done_here we we have few scenario's..

(a)search_case_Judge:"criteria": "status":
"PENDING_ADMISSION",
"ADMISSION_HEARING_SCHEDULED",
"CASE_ADMITTED"
url: https://dristi-kerala-dev.pucar.org/case/v1/_search
"limit"- 10 | Total Count-38 | Status Code-200 ok | 1St Time-721 ms | 2nd Time-672 ms
(b)search_case_Judge:"criteria": "status":
"PENDING_ADMISSION",
"ADMISSION_HEARING_SCHEDULED",
"CASE_ADMITTED"
url: https://dristi-kerala-dev.pucar.org/case/v1/_search
"limit"- 50 | Total Count-38 | Status Code-200 ok | 1St Time-2.01 s | 2nd Time-8.76 s

2.search_case_Litigant:criteria":
"filingNumber": "",
"caseType": "NIA S138",
"substage": "",
"tenantId": "kl",
"searchKey": "filingNumber",
"defaultFields": true,
"litigantId": "IND-2024-09-06-000701",
"pagination": {
"limit": 10,
"offSet": 0
url: https://dristi-kerala-dev.pucar.org/case/v1/_search
"limit"- 10 | Total Count-14 | Status Code-200 ok | 1St Time-665 ms | 2nd Time-629 ms
3.search_case_FSO:"criteria":
"status": "UNDER_SCRUTINY"
url: https://dristi-kerala-dev.pucar.org/case/v1/search?=1726232463563
"limit"- 1 | Total Count-5 | Status Code-200 ok | 1St Time-10.40 s | 2nd Time-216 ms
4.search_case_Advocate:"criteria":
"filingNumber": "",
"caseType": "NIA S138",
"substage": "",
"tenantId": "kl",
"searchKey": "filingNumber",
"defaultFields": true,
"advocateId": "c90fbe06-8608-4e82-86e1-787c55aeeb1f",
"pagination":
"limit": 10,
"offSet": 0
url: https://dristi-kerala-dev.pucar.org/case/v1/_search
"limit"- 10 | Total Count-4 | Status Code-200 ok | 1St Time-384 ms | 2nd Time-338 ms


[docs-infra] JSDoc links are broken

Steps to reproduce
Link to live example: https://codesandbox.io/p/sandbox/focused-bardeen-6cw9nz?file=%2Fsrc%2FApp.tsx
Steps:

In the code pane of the CodeSandbox linked above, hover the disabled prop in <ListItem disabled />.
In the info tooltip, click the ListItemButton link in the @deprecated message.

Current behavior
The link points to "/material-ui/api/list-item-button/" i.e. the link is broken.
Expected behavior
The link should point to "https://mui.com/material-ui/api/list-item-button/".
Context
The docs infra uses JSDoc comments to generate documentation and relative URLs are used for links. The correct URLs are generated in the docs by appending the correct base URL but devs end up with broken links in their IDEs because of the relative URLs.
Your environment
No response
Search keywords: jsdoc, urls

@aarongarciah I'd like to take a look at this issue, but it looks like the codesandbox link is broken for me. Can you give more context on how to reproduce this issue?

Would it make sense to make them all use absolute URL's? It looks like some of the links in the file already have them.

Yes, the end goal is to use absolute URLs inside every JSDoc block so devs see the correct URLs in their IDE. But we need to do it in a way that doesn't break URLs that end up in the docs. In the following screenshot, the blue blocks are used to generate part of the docs:

The first blue block ends up on the API page.

The second blue block ends up in the props table.

The yellow block is inserted here automatically and AFAIK it's not used to generate any documentation, so it's safe to use absolute URLs there.

If we use absolute URLs in the blue blocks, anytime the docs are built under other base URLs (e.g. PR previews or https://next.mui.com/), they'll point to another domain, which is not what we want.
So the idea is to use absolute URLs in every JSDoc block and adapt the docs generation to replace the https://mui.com base URL with the correct base URL depending on the. This requires changes in at least:

ComponentApiBuilder.ts
HookApiBuilder.ts

Probably more things need to be adapted, so I'd love someone from @mui/docs-infra to confirm if this approach makes sense before we continue this effort.

Some JSDocs are already using the absolute URL

in material-ui: https://github.com/mui/material-ui/blob/next/packages/mui-material/src/TextField/TextField.d.ts/#L84-L93
in the X components: https://github.com/mui/mui-x/blob/master/packages/x-data-grid/src/models/gridExport.ts/#L93-L97

Having working links seems more important than supporting links in preview.
For the next.mui.com we could do a "replace all" with ](https://mui.com when starting the next branch. Ths fix the docs link but also make sure users are redirected to the up to date documentation if they are using the alpha/beta version

Thanks @alexfauquette! @tonygravell let me know if you're still interested in working on this and if you need any help.

Okay, let me take a look and I'll reach out if I need be. Thanks!

For the next.mui.com we could do a "replace all" with ](https://mui.com when starting the next branch. Ths fix the docs link but also make sure users are redirected to the up to date documentation if they are using the alpha/beta version

@alexfauquette For the replace all, would we do this from the root? Are we trying to do this for all files or should I be adding a filter (*.d.ts, *.ts, *.md, etc.)?

It should only be about the JSDocs.
So I assume searching for packages/**/*.ts and packages/**/*.d.ts
By the way, most of those links are generated by those two lines of code:
https://github.com/mui/material-ui/blob/master/packages/api-docs-builder/ApiBuilders/HookApiBuilder.ts/#L112-L113
https://github.com/mui/material-ui/blob/master/packages/api-docs-builder/ApiBuilders/ComponentApiBuilder.ts/#L151-L154
It would be nice to either modify the constant, or better make it configurable, such that other user of the script can pick a different host when defining settings https://github.com/mui/material-ui/blob/master/packages/api-docs-builder-core/materialUi/projectSettings.ts

@alexfauquette I made this draft PR because I'm aware this doesn't fully solve the issue. Is this what you were thinking for your most recent comment about having a configurable host constant? I'm still not sure I understand the replace all part after looking at the code. This part still isn't clear to me: "For the next.mui.com we could do a "replace all" with ](https://mui.com" Could you try and explain that a little more?

It might help me to understand the context. Is it that next.mui.com is an invalid URL/no longer useful and just needs to be replaced?

About the script modification
When a project is in alpha/beta phase, we have two documentation:

The docs at mui.com is the documentation of the last stable major. WHich code lives on master branch
The docs at next.mui.com is the documentation of the new major which is still in unstable mode, living on the next branch

So the code currently on master will be used to build v5 version of the lib and should direct users to the mui.com docs. And the one on next branch is used to build the v6.alpha which should redirect to next.mui.com
Your PR looks good. You will have to run pnpm proptypes and pnpm docs:api after modifying the host to see its impact.
It should solve most of the wrong links.
About the replace all
Some links are handmade. So we should also modify them by hand. For example this JSDocs comment is the same on master and next
https://github.com/mui/material-ui/blob/4abc1b405ba4a572698d8ed6b575b5fc7751d83a/packages/mui-joy/src/Drawer/Drawer.tsx#L400-L401
THis is more anecdotic than the links your PR fix

@alexfauquette After adding those changes you recommended on my PR, I see a lot of changed files (160 total) in VSCode when running pnpm docs:api. It looks like its just updating the the URL. I'm assuming this is okay because its a relative path and if next.mui.com is correct, this should work now?

If that's true, this should be working.
Would I submit the changed files with my PR or is that for local dev only?

For me looks good, I run it to verify the modification. I added them to your PR since they look ok
I opened another PR (#42528) on top of yours to manually fix the handcrafted descriptions. @DiegoAndai are you ok with the two updates?

@alexfauquette do you know what the argos failing check on my PR is about? Is that something I need to handle or is it optional? I haven't used that tool before.

@tonygravell I approved the changes in Argos. We have a few flaky tests at the moment. All green now.

@DiegoAndai are you ok with the two updates?

@alexfauquette changes look good to me 👌🏼

sooo good !
its clean...

@aarongarciah @DiegoAndai I think this issue can be closed, right?

@siriwatknp yes 👍


Using "Apply formula to Data" in a ChoiceField will clear the choices list ?

Hi,
Strange behavior, here.
Situation before:

Now I click on "Apply formula to Data", without even entering a formula or nothing, choices have been cleared:

Now if I click "Apply" (without changing the default formula):

Any hint ? :)
Yohan

Seems this will also remove configuration of a DateField, so it may be that any field configuration is lost. That's bit crippling.
cc @LouisDelbosc @CamilleLegeron in case it's an easy fix :)

@paulfitz we can close this issue 👍


manual build for amd64 and arm64 can now get custom tag

Codecov Report

Merging #175 (f239396) into master (f5699f8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   85.98%   85.98%
=======================================
  Files          75       75
  Lines        5238     5238
=======================================
  Hits         4504     4504
  Misses        734      734

Flag
Coverage Δ

unittests
85.98% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


compiler: Misc compiler tweaks and improvements

Context: essentially all of this is necessary for new features and bug fixes in PRO
I'd like to start getting reviews ASAP because we gonna need to merge this rapidly

Merged


Support newest base version

Wreaks havoc with many Docker images that have base-<IP_ADDRESS> pre-installed.

Fixed in 3efae28e06c290284cb302749941fe9cc1776ff8 and uploaded to hackage.


Add initial_node_count support to gke-node-pool

Support for initial_node_count added to gke-node-pool module.
Set it to provision the number of desired nodes right away after node-pool creation without the need for manual resize using cloud console.
The setting will be considered only when creating a node-pool and in the next deployments it will be ignored.
Manual tests:

Added the initial_node_count setting to a node-pool in a blueprint
Provisioned the blueprint
Verified the number of provisioned nodes on Cloud Console

@ankitkinra

If we ignore this setting update, then wouldn't our subsequent configurations using this node count be incorrect ?
It's recommended to ignore it for updates because if the node_pool resized manually, this setting will cause node_pool recreation: https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/container_node_pool.html#initial_node_count

Does this setting work with autoscaling ? How does this relate to static_node_count ?

node_count and initial_node_count cannot be set together:
https://github.com/hashicorp/terraform-provider-google/blob/6be57bdad382050af99b07d4b35ede995bfd97fe/google/services/container/resource_container_node_pool.go#L940
I added two precondition to control node_count and min/max total node count in autoscaling.


有没有类似uni-app中的uni.$emit或者wx.navigateTo中events页面间通信功能

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

1.可以通过 store 进行数据通信
https://cmljs.org/doc/logic/store.html
2 组件间可以利用 父子组件通信
https://cmljs.org/doc/view/event.html

嗯嗯，看到了，多谢


Thumbnail returns a null

Environment details

OS: Ubuntu 22.04.1
PHP version: 8.1
Package name and version: v3

Steps to reproduce

Upload a file
Attempt to get a Thumbnail

Code example
        $service = new \Google\Service\Drive($client);
        $driveFile = new \Google\Service\Drive\DriveFile();
        $driveFile->setName($filename);
        $driveFile->setParents([]);
        $driveFile->setMimeType($mime);
        $details = $service->files->create($driveFile, ['data' => $data]);
        $file = $service->files->get($details->getId(), ['fields' => ['hasThumbnail', 'webViewLink', 'thumbnailLink']]);
        if ($file->hasThumbnail):
            var_dump($file->getThumbnailLink());
        endif;

The file uploads on Google Drive and also responds with an ID.

Could be better.
What is the point of having an array key accept list of comma-separated fields?
Guide here : https://developers.google.com/drive/api/guides/fields-parameter is based on List that we do in Python.
Do you think it would be helpful if I send a PR to correct the structure here?


Web App

Closes #38

Also just FYI: iirc we have to sign off our commits e.g. by using the -s flag should you use the CLI.


Symbols tool failed error on xcode upload to itunes connect.

[✓] Flutter (Channel master, v1.8.2-pre.189, on Mac OS X 10.13.6 17G3025, locale
en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.1)
[✓] Android Studio (version 3.4)
[✓] VS Code (version 1.36.1)
[✓] Connected device (4 available)
• No issues found!
This can be fixed downgrading to a lower version.  I backed down to v1.7.8+hotfix.4 and the uploads worked again.  Did not test against any in between versions.

What was the error? Can you include repro steps?

To reproduce,
in mac terminal in your flutter project, type ‘flutter build ios —release
Open the project in Xcode.
Archive the app for submission to iTunes connect.
Once the app is archived, you can press distribute or validate and once it gets to processing frameworks it will fail with the message ‘Symbols tool failed’.
You can uncheck 'Unchecking Include app symbols allows it to upload’ when that option appears and it will not fail.  But the app would not load properly when downloaded from test flight.  It would hang after the splash screen which was most likely the Xcode launch screen.
Cleaning the build before archiving does not make a difference.

On Jul 26, 2019, at 4:47 PM, Jenn Magder<EMAIL_ADDRESS>wrote:
What was the error? Can you include repro steps?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flutter/flutter/issues/37047?email_source=notifications&email_token=AF7MOFB5O3ELDJUJWPJN7TTQBOEIZA5CNFSM4IHHZ4R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD256KXY#issuecomment-515630431, or mute the thread https://github.com/notifications/unsubscribe-auth/AF7MOFFBVHMD73B5LAIHZ43QBOEIZANCNFSM4IHHZ4RQ.

Same issue here
Build failed :|
Unable to export archive:
2019-07-30 07:50:30.674 xcodebuild[4137:23872] [MT] IDEDistribution: -[IDEDistributionLogging _createLoggingBundleAtPath:]: Created bundle at path '/var/folders/r7/d9twdq011sb8d3q1p8f39cdr0000gn/T/Runner_2019-07-30_07-50-30.673.xcdistributionlogs'.
error: exportArchive: Symbols tool failed

Error Domain=IDEFoundationErrorDomain Code=1 "Symbols tool failed" UserInfo={NSLocalizedDescription=Symbols tool failed}

** EXPORT FAILED **

@jodymac Can I get flutter doctor -v?    May be related to https://github.com/flutter/flutter/pull/37217 but I can't tell if you're on a commit than includes https://github.com/flutter/flutter/pull/36471.

This is all I have that is listed above.  [✓] Flutter (Channel master, v1.8.2-pre.189, on Mac OS X 10.13.6 17G3025, locale
en-US)
I’ve backed off to 1.7.8 hot fix 4 so I can upload and test off apple testflight immediately.

On Jul 26, 2019, at 4:59 PM, Jody McAlister<EMAIL_ADDRESS>wrote:
To reproduce,
in mac terminal in your flutter project, type ‘flutter build ios —release
Open the project in Xcode.
Archive the app for submission to iTunes connect.
Once the app is archived, you can press distribute or validate and once it gets to processing frameworks it will fail with the message ‘Symbols tool failed’.
You can uncheck 'Unchecking Include app symbols allows it to upload’ when that option appears and it will not fail.  But the app would not load properly when downloaded from test flight.  It would hang after the splash screen which was most likely the Xcode launch screen.
Cleaning the build before archiving does not make a difference.

On Jul 26, 2019, at 4:47 PM, Jenn Magder<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
What was the error? Can you include repro steps?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/flutter/flutter/issues/37047?email_source=notifications&email_token=AF7MOFB5O3ELDJUJWPJN7TTQBOEIZA5CNFSM4IHHZ4R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD256KXY#issuecomment-515630431, or mute the thread https://github.com/notifications/unsubscribe-auth/AF7MOFFBVHMD73B5LAIHZ43QBOEIZANCNFSM4IHHZ4RQ.

[✓] Flutter (Channel master, v1.8.2-pre.229, on Mac OS X 10.14.4 18E226, locale en-ZA)
    • Flutter version 1.8.2-pre.229 at /Users/calvinmuller/iStreet/Mobile/flutter
    • Framework revision 9357e70dfd (16 hours ago), 2019-07-29 22:11:51 -0700
    • Engine revision 09f8bffb96
    • Dart version 2.5.0 (build 2.5.0-dev.1.0 0ca1582afd)

@calvinmuller Thank you!  You are on Framework revision 9357e70dfd which does include https://github.com/flutter/flutter/pull/36471.
@dnfield @xster  Do you know if this should be resolved with https://github.com/flutter/flutter/pull/37217?  I can't tell from https://github.com/flutter/flutter/pull/36471#discussion_r308521828 if the symptoms would be the same.
Let me try to repro.

It's plausible that #37217 could solve but @goderbauer could perhaps confirm.
#36471 could have retriggered the same issues that https://github.com/flutter/flutter/pull/22540 previously fixed. But since https://github.com/flutter/flutter/pull/22540 had no bug attached, it's unclear what the failure symptoms were. @goderbauer: was the OP's description of the issue the same as what you fixed with https://github.com/flutter/flutter/pull/22540?

Yeah, prior to #22540 xcode would show the exact same failure message. So, I would suspect that #37217 will fix this issue (there is a little uncertainty since "Symbols tool failed" is xcode's catch-all error for any problem it had when invoking the symbols tool...).

Had this issue on v1.8.2-pre.181. flutter upgrade to v1.8.2-pre.242 and 'flutter clean' fixes mine.

I'm on master and the highest I can go is Flutter 1.8.2-pre.229 so not sure how you got to pre.242

Thanks @shenwilly, when I build again on CodeMagic I will switch it back to master and give it a try. btw switching to dev allowed to build and publish.

Confirmed.  [✓] Flutter (Channel master, v1.8.2-pre.242, on Mac OS X 10.13.6 17G3025, locale
en-US)
This fixes the error.  I was able to upload to iTunes connect and run the app successfully from test flight.

On Jul 31, 2019, at 2:49 AM, Calvin<EMAIL_ADDRESS>wrote:
Thanks @shenwilly https://github.com/shenwilly, when I build again on CodeMagic I will switch it back to master and give it a try. btw switching to dev allowed to build and publish.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/flutter/flutter/issues/37047?email_source=notifications&email_token=AF7MOFHM7VEU33V2J7DLJHLQCFN37A5CNFSM4IHHZ4R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3GW3YI#issuecomment-516779489, or mute the thread https://github.com/notifications/unsubscribe-auth/AF7MOFDSBOF6U4MQWD36TT3QCFN37ANCNFSM4IHHZ4RQ.

Hooray!  Sorry about that!


FIX DOUYIN

use playwright

playwright 目前应该没有对 Ubuntu 之外的 Linux 发行版提供支持，能否给出解决方案？
related issue: https://github.com/microsoft/playwright/issues/5780#issuecomment-968091413

npm i 应该不会默认安装 chromium，应在部署流程中明确步骤
比如npx playwright install chromium


64bit-portability-issue

I'm trying to build ext with the latest zephir and I'm geting (for both 32/64bit version):

E: php5-ice 64bit-portability-issue /home/abuild/rpmbuild/BUILD/php5-ice-1.0.37/zephir-master/parser/parser/parser.c:9235

[   97s] + sed '9235q;d' parser.c
[   97s] #define SUCCESS 1

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29643974-64bit-portability-issue?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

@steffengy did you reproduce it? I have problem to bulid phalcon on opensuse:
https://build.opensuse.org/package/live_build_log/home:mruz/php5-phalcon/openSUSE_13.2/i586
https://build.opensuse.org/package/live_build_log/home:mruz/php5-phalcon/openSUSE_13.2/x86_64


[FIX] ComponentAnalyzer: Detect model types from dataSource

Coverage increased (+0.2%) to 84.207% when pulling 3be2bfb6d4d04d91d122c0db31aa8363cfe20744 on fix-ComponentAnalyzer-model-type-dataSource into feb95e41c199d5b455272ba5886cdd79d1502cd7 on master.


Is there a way to run shell commands on the web container, from the playwright one?

Hi @julienloizelet , hope all is well with you.
I wonder if it is time to look at moving either this repo or https://github.com/deviantintegral/ddev-playwright to under the ddev namespace.
In a related topic, I'd like to transfer https://github.com/deviantintegral/playwright-drupal to either the Lullabot or ddev namespaces, and add some colleagues of mine to help maintain it.
I took a look again at the code here (I haven't run it yet), and the one thing I'm not sure is possible is running shell commands on the web container to run test setup steps.
We do a few things in playwright-drupal that run commands on the web container:

We can run arbitrary drush commands against a specific test site. In tests, we use this to conditionally enable modules and other such things, but just for specific test cases.
We also use that to install a base copy of the site with sqlite for tests: https://github.com/deviantintegral/playwright-drupal/blob/main/src/setup/global-setup.ts
We also use this to get the right session cookie data so we can run tests in parallel: https://github.com/deviantintegral/playwright-drupal/blob/main/src/testcase/test.ts#L40

Have you encountered a need for anything like this? The only way I can think of to do this with a separate container is to bind the docker socket in to the playwright container, and I'm not sure that would always work in all environments.
cc @rfay .

Hi @deviantintegral, thanks for your message.
I'm not a drupal user, but as far as I understand, the drush command is a cli tool like bin/magento for Magento 2 or wp for WordPress.
Most of the time, if I have to run such a cli tool, I run it from the web container.
In the event that I need to execute a PHP script during a Playwright test, I've found no better way than to create a simple custom-script.php that executes PHP code (with exec native PHP method or whatever).
To run the script, I browse to with await page.goto('/custom-script') or await page.goto('/custom-script?action=flushCache&other_action=something_else').
It's probably not very clean, but with this type of approach, we can act on the web container.
For information, but I'm not sure it is related to your need,  all the /var/www/html code is also available in the playwright container (thanks to this volume:
https://github.com/julienloizelet/ddev-playwright/blob/main/docker-compose.playwright.yaml#L35).
Thanks again !

I'm all in favor of consolidated maintenance. Anything that moves to the ddev namespace still needs great maintainers like the two of you. Figuring out the best thing for you as maintainers and for users is the key goal.

Thanks for the update, it's appreciated. I'm glad to see there isn't anything we missed.
We aim to avoid writing web-accessible scripts like you've mentioned, as there's been cases in the past where they accidentally get deployed to production and expose sites to serious security issues. The surface area of what you can do with drush is just so large that writing wrappers around it could be pretty complex and time consuming! Running commands directly also lets us more easily stream and route output from stdout and stderr during test execution, without having to bounce through an http request.


Does not support touch or digital pen to select text

Issue Type: Bug
I understand this is still an open issue.   It continues to be a problem that text cannot be selected using a digital pen or with touch.
VSCode is basically usless to use when a user does not have a mouse and keyboard.  I can't select text with the Microsoft Surface digital pen or via touch.
Microsoft Windows 1809 software keyboard has a regression that it will close the software keyboard when the shift+arrow key is pressed to select text.  In the previous release this was an issue but was eventually fixed.
VSCode cannot be limited to mouse and physical keyboard.
VS Code version: Code 1.30.0 (c6e592b2b5770e40a98cb9c2715a8ef89aec3d74, 2018-12-11T22:29:11.253Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Atom(TM) x7-Z8700  CPU @ 1.60GHz (4 x 1600)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
3.92GB (1.45GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Rather than creating a new issue, I am adding a note here that similar issues can be experienced using a Wacom pen / touch tablet.


Register provider via policy For MongoCluster.

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - mongocluster - ci

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

/azp run java - mongocluster - mgmt - tests


Error: Failed installing dependency: http://luarocks.org/repositories/rocks/async-1.1-1.rockspec - Could not satisfy dependency: love ~> 0.9

Error: Failed installing dependency: http://luarocks.org/repositories/rocks/async-1.1-1.rockspec - Could not satisfy dependency: love ~> 0.9
how to  install love

The async package in the luarocks.org repository is a different package than http://github.com/clementfarabet/async
Make sure that you are using the torch rocks repo: http://htmlpreview.github.io/?https://github.com/torch/rocks/blob/master/index.html


Error when uploading (server side)

Hi,
I'm getting this error from the server side while uploading the image:
Exception while invoking method 'c.sign' ReferenceError: check is not defined
Any idea?

Did you update to 1.2? Maybe I need to check for the existence of check before running it, I thought it was part of the core. Look for meteor's check package and add it if you need it to work ASAP, if not I'll patch it up tonight if I get a chance.

Yes, it's 1.2. So, this might be due to the upgrade?

I am able to make it work by adding "check" and "random" packages. Thanks.

Excellent, that will affect all your meteor methods though, make sure to use check in every method you have now. This is good for security anyway if you're planning on taking your app to production someday.

Oh, I never thought it would. Hmm.. might not be a good way to do it since
my codebase is huge already. I'll just wait for your update, then?
On Thu, Sep 24, 2015 at 6:55 AM, Marcelo Reyna<EMAIL_ADDRESS>wrote:

Excellent, that will affect all your meteor methods though, make sure to
use check in every method you have now. This is good for security anyway if
you're planning on taking your app to production someday.
—
Reply to this email directly or view it on GitHub
https://github.com/Lepozepo/cloudinary/issues/44#issuecomment-142754822.

Sounds good
Marcelo Reyna
On Wed, Sep 23, 2015 at 3:57 PM, Arnold Gamboa<EMAIL_ADDRESS>wrote:

Oh, I never thought it would. Hmm.. might not be a good way to do it since
my codebase is huge already. I'll just wait for your update, then?
On Thu, Sep 24, 2015 at 6:55 AM, Marcelo Reyna<EMAIL_ADDRESS>wrote:

Excellent, that will affect all your meteor methods though, make sure to
use check in every method you have now. This is good for security anyway if
you're planning on taking your app to production someday.
—
Reply to this email directly or view it on GitHub
https://github.com/Lepozepo/cloudinary/issues/44#issuecomment-142754822.

Reply to this email directly or view it on GitHub:
https://github.com/Lepozepo/cloudinary/issues/44#issuecomment-142755048

The code is checking for check now ^_^


fixes #80014

@isidorn Could you please take a look? This is my first PR to vscode, so any contributing suggestions are more than welcome!

@dgozman thanks a lot for providing a PR. This looks great and captures the approach I had in mind. I left one minor comment inline in the code.
Did you build vscode with these changes and verifed that #80014 gets fixed?

Looks great, thanks a lot for this PR.
Merging in 🎉

Thank you!


New feature: IPFS support

We do not know of any OCDS users who use IPFS, so I will close this issue for now.


I suggest you mention in your documentation an example of how to specify the path to the "binary" for Windows users. I think Linux users and maybe Mac users throw around the term "binaries" a lot and assume everyone knows what they are talking about.
Being primarily a Windows user, what could have saved me like... 1.5 days of time... was if your documentation had an example of how to point to the Yuglify "binary" in the PIPELINE dictionary.
Link to the documentation I'm talking about.

Command line to execute for the Yuglify program. You will most likely change this to the location of yuglify on your system.
Defaults to '/usr/bin/env yuglify'.

If only you had said:

For Windows users, use the full absolute path.
Example: 'C:/Users/Owner/AppData/Roaming/npm/yuglify.cmd'

I went down the rabbit hole of messing with folder permissions, I tried putting Yuglify in every directory imaginable, I disabled my UAC on my OS... and in the end, it seems so obvious now. I don't know why your default for YUGLIFY_BINARY is '/usr/bin/env yuglify' (why the space?) but to Windows users, this is unfamiliar. I guess that's how you open programs on Linux. Sure, cool little knowledge nugget there, and maybe that will come in handy when I go to deploy...
But when I just want to get started and use a package to see if it's going to work... and Yuglify is the default... and I don't deeply know npm or Javascript best-practices (working on it!), I think your docs could be more clear.
My 10 cents.

Hello @jaradc,
The syntax used in the documentation /usr/bin/env yuglify this referer to virtualenv which you can use on both Linux and Windows. Virtual environments bring a certain number of advantages, I advise you to look at what it is to understand.
If you really want the easier solution, you should have been to use css-html-js-minify compressor, which is based on python package and doesn't require anything more than pip install.
Don't hesitate to tell me if you have questions.
Buky


CouchDB Support

Hi.
Since this library provides out the box MongoDB support, is there any plans to include CouchDB?
Like pouchdb-authentication

Please use the issue templates when creating a new issue.
There are no immediate plans to support CouchDB.
It's not supported by TypeORM or Prisma, so it would need it's own adapter.


Development

MVP Ready. Heroku deployment after merging to Main. If agree, please review and Merge.
Thanks, John.

NOPE. lol.... Let's do some clean up before doing that... Retest in Dev, deploy Dev, and MERGE and battle tested to proceed with Product deployment


Go full Gradle build

I've seen some other Eclipse plugins recently that have managed to not use Maven at all for their build!  So let's see if I can do something similar here because Tycho is kinda complicated, making me forget everything about how to use it between releases :confused:
Links:

https://github.com/de-jcup/eclipse-yaml-editor
https://github.com/diffplug/goomph
https://github.com/diffplug/gradle-and-eclipse-rcp
https://www.youtube.com/watch?v=PIC6YeRkRlo

This is proving a bit tougher w/ Gradle incompatibility issues in the Goomph plugin and me having recently gone Gradle 6 and Java 11 for development.  I'll shelve this for now and revisit it another time.

Have attempted this many times in the last few years, but can never quite get it working 🤷♂  At least it works all good w/ GitHub Actions and the command line now, so I'm less motivated to have this.


Is TLSv1.1 and lower, only available with legacy provider? (v3)

Are protocol versions controllable with providers? Specifically are TLSv1.1 and lower only available via the legacy provider, and not via default or FIPS one?

As I understand it, TLS is not inside the provider. All the
crypto is done inside a provider, TLS just makes use of a
provider.

TLS 1.1. and before will not work with the FIPS provider, they will work with the default provider.
Of course, all the algorithms used need to be present.  The legacy provider has RC4, SEED, IDEA, ... so cipher suites that use these will not work without it loaded.

Providers control what algorithms are available for TLS, they don't current limit TLS versions.


[Conflict] StepupFixer by Aroma1997 stops working with your mod installed

I added your mod to my modpack and, as the title states, StepupFixer stopped working.

Never mind, something else is wrong and just happened to be at the same time as I installed your mod. Issue persisted even though I uninstalled your mod... though nothing else changed.


You shouldn't be able to pan/zoom until the diagram is out of sight

(Another note for me.)

Closing in favour of #141


Bump T1 dependencies

Updated: nix/t1/_sources/generated.json
nix/t1/_sources/generated.nix

Wait for bump at tomorrow.


[packagers] Homebrew dependencies are not mapped from JReleaser config as documented

Task List

[x] Steps to reproduce provided
[ ] Stacktrace (if present) provided
[ ] Example that reproduces the problem (link to git repository is ideal)
[x] Full description of the issue provided (see below)

Steps to Reproduce

Create a Java Maven project with this kind of JReleaser config, similar to the example config provided in the JReleaser reference:

                        <packagers>
                            <brew>
                                <active>ALWAYS</active>
                                <tap>
                                    <name>homebrew-tap</name>
                                </tap>
                                <dependencies>
                                    <dependency>
                                        <key>key2</key>
                                        <value>value2</value>
                                    </dependency>
                                </dependencies>
                            </brew>
                        </packagers>

Run jreleaser:package and check the contents of the brew formula .rb in target/jreleaser/package.

Expected Behaviour
Formula should contain
depends_on "key2" => "value2"

as documented in https://jreleaser.org/guide/latest/reference/packagers/homebrew.html#_configuration
Actual Behaviour
Formula contains
  depends_on "dependency"

I haven't had the time to check if this problem is mirrored in other configuration formats and/or packagers.
Environment Information

Operating System: MacOS Ventura
JReleaser Version: 1.5.1
JDK Version: OpenJDK 17.0.6 (Homebrew build)

Docs have the old notation when JReleaser used to have an explicit Maven model. The correct way to set dependencies is without the <dependency> element passing key/values directly to <dependencies>. This is a doc issue.


Basic unit tests

Needs some very basic unit test implementations.

the original pr has unit testing, what do you need to achieve it here?

It is just a reminder about something TBD when I have some time. You are free to make it and PR if you want. Thanks.


Custom related_name between Organization, OrganizationOwner, and OrganizationUser

I'm trying to add django-organizations to a nascent project and it looks like it should do nearly exactly what I want except for one thing. I'm using a sub-classed OrganizationUser model from OrganizationUserBase. The relation between the org, owner, and user in the database is specified in terms of organization_id, and organization_user_id regardless of the name of my subclassed objects and regardless of what I put for default_related_name in model meta. I poked around this projects code and it looks like there are a few places that might rely on explicitly having these relationships named like this - I'm wondering how necessary is this? Is it something that could be changed? If I manually foreign key my models together using a different name than the one provided how much functionality will break?
Example:
class Lab(OrganizationBase, TimestampBase, UUIDBase):
    name = models.CharField(max_length=255, help_text=_("What is the lab's name?"))

    class Meta(TimestampBase.Meta):
        db_table = "lab"
        default_related_name = "lab"
        verbose_name = _("lab")
        verbose_name_plural = _("labs")

class LabUser(OrganizationUserBase, TimestampBase, UUIDBase):
    email = models.EmailField(
        blank=True, default="", help_text=_("What is the user's email address?")
    )

    class Meta(TimestampBase.Meta):
        db_table = "lab_user"
        default_related_name = "lab_user"
        verbose_name = _("lab user")
        verbose_name_plural = _("lab users")

class LabOwner(OrganizationOwnerBase, TimestampBase, UUIDBase):

    class Meta(TimestampBase.Meta):
        db_table = "lab_owner"
        default_related_name = "lab_owner"
        verbose_name = _("lab owner")

results in a database schema where lab_owner has a foreign key named organization_id and organization_user_id.
This isn't a deal breaker, but it makes it more difficult to use the database directly because my app doesn't embrace the concept of organizations.

I poked around this projects code and it looks like there are a few places that might rely on explicitly having these relationships named like this - I'm wondering how necessary is this?

Strictly speaking, I don't think it's absolutely necessary. We'd just need to be able to introspect the related name.

Is it something that could be changed?

Yes, I do think so. And provided it was not backwards incompatible it would be a welcome addition. Is it something you'd have interest in writing a PR for?

Yes, I do think so. And provided it was not backwards incompatible it would be a welcome addition. Is it something you'd have interest in writing a PR for?

Very much so. I haven't had a chance to make material process on the project that would require this change but I should be (re-)beginning work soon and assuming that this is still useful to me I'll take a crack at making the above change.


“We were unable to load Disqus" error

Hi Robpol86,
I am encountering "We were unable to load Disqus" error when I follow the usage guide. I use Sphinx-bootstrap-theme and you can find my repo for the web here
I also tried to directly copy universal installation code directly into a html page, and it works.
Do you have clue on why my site not working?
Thanks much!

I'm having the same issue.

@moigagoo I would recommend sphinx news feed extension if you want to use "disqus" directive. I have tested on my own. It works perfectly.

@xxks-kkk Thanks! Works like charm!

@moigagoo no problem :smile:

@xxks-kkk Thanks! I spent the whole morning trying to integrate with RFD and Disqus. Your information gave me a great help.

@godleon no problem :)


Fix equipment background corruption removing background cursor feature

Needs a lot of test. I tested just a few times...

I don't know what this refers to and what I should test for. Could you please expand on it.

When you scale the window, the background of equipment panel (where the player's equipped items stay) doesn't scale properly, artifacts from the window before the scale stay below the items.

Commit rebased.

Tested, didn't found bugs.


The SubGroup list is empty when the Group is created.

System Health details
System Information

version
core-2023.7.3

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.11.4

os_name
Linux

os_version
6.1.34

arch
x86_64

timezone
Europe/Kiev

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.32.1

Stage
running

Available Repositories
1277

Downloaded Repositories
37

AccuWeather

can_reach_server
ok

remaining_requests
3

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 10.3

update_channel
stable

supervisor_version
supervisor-2023.07.1

agent_version
1.5.1

docker_version
23.0.6

disk_total
116.7 GB

disk_used
24.2 GB

healthy
true

supported
true

board
generic-x86-64

supervisor_api
ok

version_api
ok

installed_addons
Samba share (10.0.2), Advanced SSH & Web Terminal (15.0.5), Home Assistant Google Drive Backup (0.111.1), HDD Tools (1.1.0), File editor (5.6.0), ESPHome (2023.7.0), VLC (0.2.0), MQTT Explorer (browser-1.0.1), Photoprism (ubuntu-230719-73fa7bbe8)

Dashboards

dashboards
13

resources
20

views
56

mode
storage

Recorder

oldest_recorder_run
18 июля 2023 г. в 10:53

current_recorder_run
28 июля 2023 г. в 16:00

estimated_db_size
2891.92 MiB

database_engine
mysql

database_version
11.0.2

Sonoff

version
3.5.2 (2ad1cd7)

cloud_online
3 / 3

local_online
0 / 0

Xiaomi Miot Auto

component_version
0.7.10

can_reach_server
ok

can_reach_spec
ok

logged_accounts
1

total_devices
34

Checklist

[X] I have read the FAQ to see if there is a known solution to my problem.
[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
The SubGroup list is empty when the Group is created.

Reproduction steps

Add from GUI

Debug logs
Not sure if there is a debug log for the GUI.

Diagnostics dump or YAML config
I have 2 groups in PowerCalc, created in the GUI, for Area.

I see, it's a regression introduced in 1.8.4
I have made a modification to exclude the current group from sub groups when you are editing an existing config entry.
However this change caused the subgroup selector to be empty when creating a new group.
Will work on a fix now.

1.8.4 was not released yet. But it was only an issue on master branch.
Anyway it's fixed now with #1775. Also added a test to verify the behaviour and make sure it won't break anywhere in the future.
Will release 1.8.4 shortly.

Thanks, a create Group, add 2 PowerCalc SubGroup, but

Anything in the logs?

Anything in the logs?

There's nothing.
I restart HA, group empty ... Group settings:

home-assistant.log.zip

Could you paste the group name here. I don't speak ukranian, and don't have these characters on my keyboard, so don't know what to search for in the logs ;-).

Could you paste the group name here.

Groups:

Спальня (transliteration: spalnia)
Кухня  (transliteration: kukhnia)

Main group name: Home

I think I have found the issue and a fix with #1778
Just merged to master. Could you install latest master and let me know your findings. Hopefully it's also fixed for you.

Yes it works. Thank you.


Dark mode for documentation

Closes (ISSUES HERE).
Put your pull request body here!
Checklist before submitting:

[X] Added entry to CHANGELOG.md
[X] Added/updated relevant tests
[X] Added/updated documentation

Coverage remained the same at 94.188% when pulling fda736a17cdace110da3034a6b33931022e650fa on Nickuhhh:master into 380c3d652f41d896d2f5ebcc128aeed99d176184 on Roblox:master.

I have read the CLA Document and I hereby sign the CLA


When suggesting record attributes, the cursor point should be between the quotes/ []

Description:
When suggesting record attributes with string value/ array get the cursor between the quotes/ []

Closing since the fix has been merged


Test

The test we did is on here, can fix it later. The css folder and style.css are also attached to this request.


Add global --json, --auth, && --silent flag support

To better support programatic use cases, this PR adds --json flag support for relevant commands.
This will disable normal CLI logging and only output json for the user to parse and use elsewhere
Example:

Depends on https://github.com/netlify/netlify-dev-plugin/pull/224 & https://github.com/netlify/cli-utils/pull/13

Todo:

[ ] merge https://github.com/netlify/cli-utils/pull/13/files
[ ] bump version here

Example commands:
--json Outputs returned data as JSON for further processing.
netlify sites:list --json | jq '.[0]'
netlify deploy --json | jq '.site_id'
netlify addons:list --json

--silent silences all logs in a command
netlify deploy --silent

--auth allows for the auth flag to be passed into any command.
netlify status --auth otherAccountToken


Test

Pull Request Test Coverage Report for Build 76

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 69.376%

Totals

Change from base Build 73:
0.06%

Covered Lines:
5655

Relevant Lines:
7747

💛  - Coveralls


parseNumber tests failing for out-of-bound numbers on clang

The following tests are failing for parseNumber using clang++ Apple LLVM version 6.0 (clang-600.0.57) (based on LLVM 3.5svn).:
[  FAILED  ] AssemblyContextParseNarrowUnsignedIntegers.Sample
[  FAILED  ] AssemblyContextParseWideUnsignedIntegers.Sample
[  FAILED  ] AssemblyContextParseFloat.Sample
[  FAILED  ] AssemblyContextParseDouble.Sample

This is because clang does not fail for large numbers(i.e. 1e400) when converting text to float using istringstream.
#include <iostream>
#include <iomanip>
#include <sstream>
#include <limits>
using namespace std;

int main() {
  istringstream text_stream("1e400");
  double val = 0;

  text_stream >> std::setbase(0);
  text_stream >> val;

  cout << text_stream.fail() << endl;
  cout << "val: " << val << endl;
}

Output:
clang++ -std=c++11 foo.cpp -o clang_foo && echo clang && ./clang_foo && g++-4.8 -std=c++11 foo.cpp -o gcc_foo && echo gcc && ./gcc_foo
clang
0
val: inf
gcc
1
val: 1.79769e+308

The reliance on the failure from istringstream is used to check for out-of-bounds arguments to things like OpConstant %type , where %type is a known type.
For example if %type is an unsigned 32-bit int, then the istringstream::fail() is used to see if the value is out of bounds for that type.
In short, this is behaviour we do want to rely on, so that the assembler can properly check its inputs.
This looks like a bug in the Apple C++ library.
Behaviour is as specified in http://en.cppreference.com/w/cpp/io/basic_istream/operator_gtgt
In particular "f extraction fails, zero is written to value and failbit is set. If extraction results in the value too large or too small to fit in value, std::numeric_limits::max() or std::numeric_limits::min() is written and failbit flag is set."
Well, we've already worked around a GNU libc++ bug.   We seem to have an Apple one.
Thanks for reporting this.

I investigated two test failures which occur on Mac OSX using Apple's Clang.
There are two failing essential cases:

parsing  "-0" is expected to succeed with value 0 even for unsigned target types.

parsing a too-large floating point value in regular notation (e.g. 1.0e400) is expected to result in an error, but it does not.

Fixed in master with 6bad02c320fa


DOING and CRITICAL treeviews not being updated

I have a TODO file for my project like this:
`
Todo:
Heading 1:
☐ Task 1 @doing
☐ Task 2 @critical
☐ Task 3
`
For "Task 1" and "Task 2" I do not see any items in the DOING and CRITICAL treeview, not even after saving the TODO file.

you need add a Tab space before task item

I tried but this doesn't work. I'm using Ctrl+Enter to create new task item. When I'm restarting VS Code, the treeview will reload and show the right things but not during work.

can you show me a capture image of vscode?
tredition<EMAIL_ADDRESS>周一下午1:43写道：

I tried but this doesn't work. I'm using Ctrl+Enter to create new task
item. When I'm restarting VS Code, the treeview will reload and show the
right things but not during work.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bmpi-dev/vscode-todo-plus-plus/issues/2#issuecomment-619736211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASVKFGR2MAKSMC4HT4NJ33ROULPJANCNFSM4MP27NWQ
.

This is the view after saving the TODO file:

The Files section is updated properly, but not CRITICAL and DOING.

You file extension must be *.todo, this image show you file has not a correct .todo extension.
在 2020年4月28日 +0800 PM2:05，tredition<EMAIL_ADDRESS>
This is the view after saving the TODO file:
The Files section is updated properly, but not CRITICAL and DOING.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Doesn't change anything. Item 5 should be unter DOING but isn't.

contact me use telegram：https://t.me/bmpidev , I will help you look deep in this problem.
2020年4月29日 +0800 PM1:44 tredition<EMAIL_ADDRESS>
Doesn't change anything. Item 5 should be unter DOING but isn't.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I have the same problem. In my case it's not even detecting the file:


Remove frameworks that only work in Python 2.7

[Merge after #1200 ]
There have been frameworks that have only supported Python 2.7.  While these frameworks are no longer supported by New Relic (by virtue of having a last release of 3+ years ago) the convention has been to keep the instrumentation in the code and just remove the test runner.  However, with a lack of Python 2.7 support, these frameworks will not work and will need to be removed.
Frameworks removed:

umemcache
oursql
pywapi
pylons
web2py

Codecov Report
Attention: Patch coverage is 74.75728% with 104 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop-remove-python27@3173305). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

newrelic/core/application.py
16.66%
20 Missing :warning:

newrelic/api/transaction.py
50.00%
16 Missing :warning:

newrelic/core/agent.py
7.69%
12 Missing :warning:

newrelic/core/database_utils.py
88.67%
11 Missing and 1 partial :warning:

newrelic/network/addresses.py
0.00%
4 Missing :warning:

newrelic/bootstrap/sitecustomize.py
0.00%
3 Missing :warning:

newrelic/api/graphql_trace.py
0.00%
2 Missing :warning:

newrelic/common/object_names.py
50.00%
2 Missing :warning:

newrelic/core/config.py
81.81%
2 Missing :warning:

newrelic/core/profile_sessions.py
0.00%
2 Missing :warning:

... and 27 more

Additional details and impacted files
@@                    Coverage Diff                     @@
##             develop-remove-python27    #1205   +/-   ##
==========================================================
  Coverage                           ?   81.17%
==========================================================
  Files                              ?      197
  Lines                              ?    21502
  Branches                           ?     3748
==========================================================
  Hits                               ?    17455
  Misses                             ?     2930
  Partials                           ?     1117

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


ROX-23132: use make tag result in image names on Konflux

Description
Build pipeline produced quay.io/redhat-user-workloads/rh-acs-tenant/acs/central-db:4.4.x-436-gd6f46a5cdd.
I have not updated other pipelines yet, I want to have a discussion first if we should commit to this new image tagging scheme & what happens with image flavors & -fast suffix.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Konflux PR pipeline for central-db passes.

roxctl-on-pull-request-lpd7n failed with a 502 Bad Gateway while pulling registry.access.redhat.com/ubi8/ubi-minimal:latest. There have been successful pipeline runs for this component before, so we can ignore this as a flake.


update example with new syntax

labels -> withLabels

Pull Request Test Coverage Report for Build 1273

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

Totals

Change from base Build 1272:
0.0%

Covered Lines:
12054

Relevant Lines:
16980

💛  - Coveralls


Multiple client_id/relying_party_id pairings for the same adfs server

I have a Django app that already has ADFS ready to go to and working smoothly for a single CLIENT_ID/RELYING_PARTY_ID pairing:
# settings.py

AUTH_ADFS = {
    "SERVER": "adfs.mywebapp.com",

    "CLIENT_ID": "myclient-id-1",
    "RELYING_PARTY_ID": "mywebapp.com",

    # ...

    "CLAIM_MAPPING": {"first_name": "given_name",
                      "last_name": "family_name",
                      "email": "email"},
}

My app, however, has two URLs (the default one created by AWS Elastic Beanstalk and another more concise one created using AWS's DNS service). I have CLIENT_ID/RELYING_PARTY_ID pairings for each of these, too. I can successfully use ADFS for either one of these individually, but I would like my users to be able to use both to reach the ADFS server. Is this possible using Django ADFS?
 "CLIENT_ID": "myclient-id-1",
 "RELYING_PARTY_ID": "mywebapp.com"

 "CLIENT_ID": "myclient-id-2",
 "RELYING_PARTY_ID": "mywebapp.elasticbeanstalk.com"

No, not at the moment.
You can always add more then 1 redirect URI to your ADFS configuration. That way you can use a single Client ID, but allow redirects to multiple locations.

@fgscivittaro have you tried adding an extra redirect URI to your ADFS configuration? If so, did that fix your problem?

Closed due to inactivity


Coverage

Refactoring: Remove method which is not read
Testing: Add case for non-writable PUT
Testing: Add coverage

I set the file permissions to 0444 on the Mac to get the first test addition to err.
The need for this test case was discovered as a result of the first coverage addition.
When noticing that function coverage was not at 100%, I added the HTML reporter, and then discovered a method (in the HEAD code I added) which was not being read so removed it.

Excellent, thanks!


Unable to load Cats vs Dogs dataset

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No. I used the same code as is given in https://www.tensorflow.org/datasets/overview, but changed the dataset from mnist to cats_vs_dogs
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Windows 10.0.18362, 64-bit
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
No. Using a PC
TensorFlow installed from (source or binary):
Binary
TensorFlow version (use command below):
Tensorflow 2.3.0
Tensorflow datasets 3.2.1
Python version:
Python 3.5
Bazel version (if compiling from source):
NA
GCC/Compiler version (if compiling from source):
NA
CUDA/cuDNN version:
CUDA driver version   : 10020
GPU model and memory:
NVIDIA GeForce 940MX 4GB

Describe the current behavior
I am unable to download the cats_vs_dogs dataset through the tensorflow datasets.
Describe the expected behavior
I should be able to load a variable with the dataset.
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
ds = tfds.load('cats_vs_dogs', split = 'train', shuffle_files = True)
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 ds, train = tfds.load('cats_vs_dogs', split = 'train', shuffle_files = True) #download cats vs dogs file for this projectb
~\AppData\Roaming\Python\Python35\site-packages\wrapt\wrappers.py in call(self, *args, **kwargs)
565
566         return self._self_wrapper(self.wrapped, self._self_instance,
--> 567                 args, kwargs)
568
569 class BoundFunctionWrapper(_FunctionWrapperBase):
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\api_utils.py in disallow_positional_args_dec(fn, instance, args, kwargs)
67     _check_no_positional(fn, args, ismethod, allowed=allowed)
68     _check_required(fn, kwargs)
---> 69     return fn(*args, **kwargs)
70
71   return disallow_positional_args_dec(wrapped)  # pylint: disable=no-value-for-parameter
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\registered.py in load(name, split, data_dir, batch_size, shuffle_files, download, as_supervised, decoders, read_config, with_info, builder_kwargs, download_and_prepare_kwargs, as_dataset_kwargs, try_gcs)
369   if download:
370     download_and_prepare_kwargs = download_and_prepare_kwargs or {}
--> 371     dbuilder.download_and_prepare(**download_and_prepare_kwargs)
372
373   if as_dataset_kwargs is None:
~\AppData\Roaming\Python\Python35\site-packages\wrapt\wrappers.py in call(self, *args, **kwargs)
604
605             return self._self_wrapper(self.wrapped, self._self_instance,
--> 606                     args, kwargs)
607
608         else:
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\api_utils.py in disallow_positional_args_dec(fn, instance, args, kwargs)
67     _check_no_positional(fn, args, ismethod, allowed=allowed)
68     _check_required(fn, kwargs)
---> 69     return fn(*args, **kwargs)
70
71   return disallow_positional_args_dec(wrapped)  # pylint: disable=no-value-for-parameter
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\dataset_builder.py in download_and_prepare(self, download_dir, download_config)
374           self._download_and_prepare(
375               dl_manager=dl_manager,
--> 376               download_config=download_config)
377
378           # NOTE: If modifying the lines below to put additional information in
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\dataset_builder.py in _download_and_prepare(self, dl_manager, download_config)
1017     super(GeneratorBasedBuilder, self)._download_and_prepare(
1018         dl_manager=dl_manager,
-> 1019         max_examples_per_split=download_config.max_examples_per_split,
1020     )
1021
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\dataset_builder.py in _download_and_prepare(self, dl_manager, **prepare_split_kwargs)
949
950       # Prepare split will record examples associated to the split
--> 951       self._prepare_split(split_generator, **prepare_split_kwargs)
952
953     # Update the info object with the splits.
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\dataset_builder.py in _prepare_split(self, split_generator, max_examples_per_split)
1032                                      hash_salt=split_generator.name)
1033     for key, record in utils.tqdm(generator, unit=" examples",
-> 1034                                   total=split_info.num_examples, leave=False):
1035       example = self.info.features.encode_example(record)
1036       writer.write(key, example)
C:\Anaconda\lib\site-packages\tqdm\notebook.py in iter(self, *args, **kwargs)
232     def iter(self, *args, **kwargs):
233         try:
--> 234             for obj in super(tqdm_notebook, self).iter(*args, **kwargs):
235                 # return super(tqdm...) will not catch exception
236                 yield obj
C:\Anaconda\lib\site-packages\tqdm\std.py in iter(self)
1163
1164         try:
-> 1165             for obj in iterable:
1166                 yield obj
1167                 # Update and possibly print the progressbar.
C:\Anaconda\lib\site-packages\tensorflow_datasets\image_classification\cats_vs_dogs.py in _generate_examples(self, archive)
87     """Generate Cats vs Dogs images and labels given a directory path."""
88     num_skipped = 0
---> 89     for fname, fobj in archive:
90       res = _NAME_RE.match(fname)
91       if not res:  # README file, ...
C:\Anaconda\lib\site-packages\tensorflow_datasets\core\download\extractor.py in iter_zip(arch_f)
197     for member in z.infolist():
198       extract_file = z.open(member)
--> 199       if member.is_dir():  # Filter directories  # pytype: disable=attribute-error
200         continue
201       path = _normpath(member.filename)
AttributeError: 'ZipInfo' object has no attribute 'is_dir'

I cannot reproduce this.
We don't officially support anaconda here. Can you try again installing TF with official installation guide?

Its working fine for the mnist dataset. Its seems the problem is only with the cats vs dogs dataset. I'll do as you ask tonight and let you know.

@TheNightBaron,
I was able to run the code without any issues, please find the gist of it here.
Could you please check if you're facing the same issue in a Python virtual environment? Thanks!

I have python only on Anaconda. Do I need to install python separately for trying this?

We don't directly support Anaconda and I cannot reproduce this in TF with the official install guide


fix: add withDefaults support

fix issue https://github.com/Tahul/pinceau/issues/39

Hey @SGAMERyu!
Thank you so much for this fix!


Audiracmichelle/issue24

worked on the suggestions, and requested re-review for merging approval! @lindadelacombaz


Maximum update depth exceeded error when container is scrollable and nouse is used

Hello! We get an errors when sortable items container has a scrollbar.
There is example: https://codesandbox.io/s/smoosh-fast-y17ye
We have an error only with mouse.
Just take an element and move it outside of the container.
https://user-images.githubusercontent.com/7955306/131701811-a6411a84-da29-4250-8d11-53825afca776.mov

I believe this is fixed in the next versions of @dnd-kit (npm releases tagged with next). Feel free to re-open if not
https://codesandbox.io/s/sleepy-zhukovsky-53tve

will it be available in v.3? or in v.4 only?

@chestozo<EMAIL_ADDRESS>and up only.


Feedback for "Build and Deploy" - Suggestion

Hello experts!
In the "Build and Deploy" section we have only the MTA development subsection on the matter. Is it intended on remain on that way or can we expand on deploy options?
If is intended on be expanded, please let me know so that I can contribute (if possible).
Thank you very much for your time!
https://help.sap.com/products/SAP Business Application Studio/9d1db9835307451daa8c930fbd9ab264/d5e4d02d976f40feabe9b4c57443b7a5.html

Hi,
Thank you! I would very much like to learn about your deployment options.

https://people.sap.com/rcaziraghi

Thank you @Paolaufer ! Excited to be able to contribute.


[Bug]: figure shortcode is not working for images stored in the same folder as in the markdown blog post for non default languages

What version of Hugo are you using (hugo version)?
Hugo v0.123.8 extended version
Does this issue reproduce with the latest release?
I have a folder structure is as following:
.
└── content
    └── posts
        └── my-first-post
            ├── img.png
            ├── index.en.md
            └── index.zh-cn.md

And my blog has set the default language to English.
Using the figure shortcode, I can see that img.png is properly showing in index.en.md but not index.zh-cn.md
The shorcode I used in both markdown file is simply just {{
}}.
Downgrading Hugo to v0.121.2 works.

A reproducible sample can be my repo at https://github.com/CXwudi/personal-blog-source, change the HUGO_VERSION in .devcontainer\.env to 0.123.8, and run ./test-local.sh in the root directory to bring up my blog. Checkout the Chinese version of the first post and you can see

@CXwudi Thanks for creating this issue. This will be resolved with https://github.com/gohugoio/hugo/issues/12245.
In the interim, assuming that you are using the embedded figure shortcode described here, create your own to override it:

layouts/shortcodes/figure.html

  {{- if .Get "link" -}}

  {{- end -}}

  {{- $u := urls.Parse (.Get "src") -}}
  {{- $src := $u.String -}}
  {{- if not $u.IsAbs -}}
    {{- with or (.Page.Resources.Get $u.Path) (resources.Get $u.Path) -}}
      {{- $src = .RelPermalink -}}
    {{- end -}}
  {{- end -}}

  {{- if .Get "link" }}{{ end -}}
  {{- if or (or (.Get "title") (.Get "caption")) (.Get "attr") -}}

      {{ with (.Get "title") -}}

{{ . }}

      {{- end -}}
      {{- if or (.Get "caption") (.Get "attr") -}}

        {{- .Get "caption" | markdownify -}}
        {{- with .Get "attrlink" }}

        {{- end -}}
        {{- .Get "attr" | markdownify -}}
        {{- if .Get "attrlink" }}{{ end }}

      {{- end }}

  {{- end }}


Remove INTENDED prefix from configuration roles

This is the last of the Configuration Role related changes for 8.2.  This should wait until the following PRs are finished before attempting to merge, to ensure it doesn't cause excessive conflicts:

https://github.com/gradle/gradle/pull/24183
https://github.com/gradle/gradle/pull/24271
https://github.com/gradle/gradle/pull/24309
https://github.com/gradle/gradle/pull/24325
https://github.com/gradle/gradle/pull/24334
https://github.com/gradle/gradle/pull/24346
https://github.com/gradle/gradle/pull/24397 (Copy work - don't worry about this one)
https://github.com/gradle/gradle/pull/24340

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test QFL

I've triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test this

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.

I've triggered a build for you.


Adapt data layer to new Laravel backend

Sub-tasks:

[x] Remove unused treesPlanted function from API protocol and implementations
[x] Create new Supervisor service (ideally async style) to establish pattern for pulling from new Api (i.e. auth, object mapping etc.)
[x] Create new Site service based on Supervisor service
[x] Create new Species service
[ ] Adapt tree upload to new backend
[ ] Add "add site" functionality to Site service

Waiting on https://github.com/protect-earth/protect-earth-api/issues/60

Update on blockers: https://github.com/protect-earth/protect-earth-api/issues/92 is done and https://github.com/protect-earth/protect-earth-api/issues/60 is just being used as a place to track feedback on the new upload endpoint, as the mock PR has been merged already.

Waiting on https://github.com/protect-earth/protect-earth-api/issues/121


[anaylzer-plugin] Feature request: providing a type inference information

Dart currently does not support utility types for functions: something likeReturnType, and Parameters in typescript.
For my custom packages, I need those abilities to improve the DX.
For example
void main() {
  final dynamic wrapped = Wrapper(() => 10);
  print(wrapped());

  final dynamic wrapped2 = Wrapper((int a, int b) => a * b);
  print(wrapped2(10, 20));
}

class Wrapper {
  Wrapper(this.f);
  final Function f;

  @override
  dynamic noSuchMethod(Invocation invocation) {
    final positionals = invocation.positionalArguments;
    final optionals = invocation.namedArguments;
    final builtValue = Function.apply(f, positionals, optionals);
    final computedValue = builtValue * 10;
    return computedValue;
  }
}

I like to provide an analyzer plugin that provides type information. So that when developers type wrapped2(, my custom plugin shows the (int, int) parameter types.
If the analyzer plugin can provide this ability, it will be great.

That said, this sounds like a feature that would be useful, and one that we might already have for VS Code.

@bwilkerson I don't believe we support this. In the code example above, wrapped2 is typed dynamic so I don't think there is enough information to know its arguments.
If I understand correctly, this seems more like a language(?) request, something similar to callable classes, but where the signature of the call method is something that can come from a type argument? For example something like:
class Wrapper<T extends Function> {
  final T f;
  Wrapper(this.f);

  /// This signature wants to be T
  void/* return type of T */ call(/* arguments of T */) {
    return f(/* args */);
  }
}

I'm not sure this is possible today?

If I understand correctly ...

I think you're right, and I missed that aspect of it. This isn't something that can be done in the analyzer without a change to the language, so I'm going to close this issue.
@wurikiji
If you want to propose a change to the language, you can do so at https://github.com/dart-lang/language/issues/new/choose.


LF-20009: Bumping SDK for Twitter video fix

https://livefyre.atlassian.net/browse/LF-20009

Duplicated by: https://github.com/Livefyre/streamhub-wall/pull/105


United Nations Flag

Would love a general catch all flag for international uses
Detailed Description
Either the UN flag or one of these could work https://en.wikipedia.org/wiki/Flag_of_Earth

Made this as a stop gap: https://www.figma.com/file/fh5XywZYumgX18bEzJN5Bc/UN-United-Nations?node-id=0%3A1

Hi @duncangraham. Thanks for thinking along with Flagpack. Currently we're working toward making Flagpack reflect the ISO 3166 standard. Ironically enough, while this standard is in part set by the United Nations, the flag of the UN itself is not part of this standard. For that reason, we unfortunately have no plans to include the flag of the United Nations into Flagpack.


Fix Deprecation Warnings.

Fix deprecation for symfony/config 4.2+
(Greatly inspired by https://github.com/stof/StofDoctrineExtensionsBundle/pull/388/commits)

Thanks

@goetas could you please release new version?

https://github.com/willdurand/BazingaHateoasBundle/releases/tag/2.0.1


Refactor fused MLP + fused attention loading. Fix for fused MLP requiring Triton even when not used.

This is a fix for: https://github.com/PanQiWei/AutoGPTQ/pull/43#issuecomment-1547925665
Changes:

In modeling/_base.py, add new classmethods get_fused_attention_module and get_fused_mlp_module. if called directly they return "this class does not support" warnings.
In modeling/llama.py, add same classmethods, implementing the imports of FusedLlamaMLPForQuantizedModel and FusedLlamaAttentionForQuantizedModel with try except blocks
In modeling/_base.py implement checks for inject_fused_attention and inject_fused_mlp which only call get_fused_mlp_module and get_fused_attention_module if right conditions are met.  In particular, don't call get_fused_mlp_module unless use_triton is True.
Therefore FusedLlamaMLPForQuantizedModel will not be imported unless the user specifies use_triton and inject_fused_mlp
Therefore the user does not need Triton installed and can execute CUDA code without errors.

Testing done:

Inference with: CUDA, CUDA + FA, Triton, Triton + FA, Triton + FM. Triton + FA + FM
Quantisation with CUDA

This patch works. Thanks.

Thank you @TheBloke to create this pr and solved user's problem when try injecting fused module without triton.
But, I think it would be better to fix from the root cause instead of adding new functions as patches for it will make the code more complex.
In my opinion, the better design pattern to keep in "automatic" is keep something to None or use global flag to disable if it's not supported in some specific cases, as those done in flash-attention's block.py and text-generation-inference's layers.py
I will make a new pr to fix the triton problem in a new way, so this pr can be closed.
Once again, I'm really appreciate your contributions and they are truly making this project better and better. ❤🔥

I've made #92 to fix import error when triton is not installed and optimized code to make it more automatic when trying to integrate with triton

Looks good! Will test it in a minute


There's a np2.cfg file in my home dir

There's a np2.cfg file in my home dir, every time I open this core it creates this file at /home/my_user/
It may be at System/BIOS directory I guess (RETRO_ENVIRONMENT_GET_SYSTEM_DIRECTORY)?

I fixed cfg file creation at 'current directory'.
It can be created at System/BIOS directory, so you said.
File creation was operating faster than getting System directory.
Thanks to your report!
2017-11-25 13:06 GMT+09:00 sergio-br2<EMAIL_ADDRESS>
There's a np2.cfg file in my home dir, every time I open this core it
creates this file at /home/my_user/
It may be at System/BIOS directory I guess (RETRO_ENVIRONMENT_GET_SYSTEM_
DIRECTORY)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AZO234/NP2kai/issues/11, or mute the thread
https://github.com/notifications/unsubscribe-auth/AE5BtdPmZvQ0YlSX6855IAtrWoxIY_BLks5s55JHgaJpZM4QqTKK
.

I think that this fix is not UNIX standard. I suggest that the config file locates in ~/.np2/, ~/.config/np2/, or somewhere like that. It's better that one can specify the config file by command parameter like
$ np2 --config-file ~/.np2/np2.cfg
With this function, one can install the excutable binary to /usr/local/bin/.  One can write
alias np2='/usr/local/bin/np2 --config-file ~/.np2/np2.cfg'
in ~/.basrc in order to omit the specification every time.

Saving cfg files in System/BIOS directory is the standard in RetroArch

I don't know much about RetroArch, but System/CONFIG or something like that is standard, isn't it? I think in most settings the two (BIOS and CONFIG) are the same directory. In any case, at least in the case of the SDL version, like the X11 version in which the config file is ~/.np2/np2rc, the config file should be placed in ~/.np2/, ~/.config/np2/ or somewhere like that.
BTW, why is the name of the config file "np2.cfg", while that of the X11 version is "np2rc"?

I changed config/BIOS files location of SDL2 and X11 NP2kai to ~./config.
SDL2 : ~./config/np2kai
X11 : ~./config/xnp2kai
Regard.


Fix bug in Locative logic

An error would occur if a device that wasn't being tracked exited. I have fixed the problem and added a test to check for this case.

:dolphin: can merge when tests pass


Bump to v1.5.1 with fixes

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Removed several patches that have been upstreamed:

https://github.com/orocos/orocos_kinematics_dynamics/pull/334
https://github.com/orocos/orocos_kinematics_dynamics/pull/335
https://github.com/orocos/orocos_kinematics_dynamics/pull/336
https://github.com/orocos/orocos_kinematics_dynamics/pull/337

Happy to merge this in if you think it's ready @traversaro

Fix #14

Happy to merge this in if you think it's ready @traversaro

It is ready!


refactoring : add __all__s to __init__s from pyglet.window modules & submodules

Closes #426
I have not yet tested if this actually silences the warnings, and expect many other changes could (should?) be included in here but i figured i'd better ask you guys' thoughts about this first.
All kinds of feedback appreciated !

Thanks for the PR, and sorry for the delay!
I had a quick look, and some of the classes don't necessarily need to be included in__all__.
I would also say that it's preferrable to keep things to a single line, where they fit.
We generally stick to PEP8 formatting, with the exception of allowing 100 (or occasionally 120) columns instead of 80.
If you have any questions, please let me know.


Can not sign transaction build by legacy package

Transactions build via legacy package ethereumjs-tx use tx.getChainId to get chainId and will return a number.
So when we submit a legacy transaction to keyring will goes in https://github.com/GridPlus/eth-lattice-keyring/blob/master/index.js#L795, but signTransaction use the return value as number.toString('hex') https://github.com/GridPlus/eth-lattice-keyring/blob/master/index.js#L194 this will cause an exception like: RangeError: toString() radix argument must be between 2 and 36

btw actually this._getEthereumJsChainId(tx).toString('hex') is useless since this._getEthereumJsChainId(tx) already revert to a hex string without the 0x prefix, so I think fix as below should be useful
if (typeof tx.getChainId === 'function') {
    return tx.getChainId().toString('16');
}

Thank you for this! Will fix


More robust check for GITHUB_TOKEN

It may be an empty string in PRs from forks when a custom secrete is used.

@tkf @johnnychen94 Is this PR good to be merged?