Fixed spm  integration on regular Xcode project

without this change project itself can be built as spm package, but failed to do so when you integrate it as a dependency via Xcode.

Will appreciate a new release if possible :) There is a similar PR fo the https://github.com/pinterest/PINCache/pull/301
In general, spm/cocopods/xcode/carthage support is a little mind-blowing :)
cc @garrettmoon

Thanks for putting this up! I hate to ask, but would you mind updating the CI test to ensure that SPM works when integrating into a project?

Yes sir! Will add it to my list.

Can we get a release with this in it, please? @garrettmoon

Going to ask as well, can we please get a release for these fixes? @garrettmoon


How long until transfers show up in new wallet?

I created a new wallet and then sent 2 U.S. dollars worth of ETC over to it as a test transfer from my Coinbase account.  After about 5 minutes or so the transfer cleared from the Coinbase account.  But it's now been 30 minutes and the ETC amount has not shown up yet in my new ETC Wallet.
Is this normal or is there a problem?
I also have noticed (I am new here) that the link to find my transaction history did not work.  I then tried to go to the main site URL and it shows as down right now with 503 Service Unavailable:   http://etherhub.io/

the balance should update within a minute. try gastracker.io if the etherwallet node is not updating

Hi Elaine, how are you?
Thanks for getting back to me.  I think I figured out the issue about 2 hours later...maybe you can tell me if I'm correct.
So I kept looking at my Coinbase account and the "test" 2 dollar "ETH" transfer to my etherwallet definitely went through, but was still not showing up on the other end.
I eventually found after all kinds of digging around on the 'ol interwebs ;-)  that Coinbase made an announcement at the end of 2016 that they would no longer perform transactions in "ETC"......"ah hah!", I thought.  haha
So it seems I was confused by this etherwallet and didn't understand that there are 2 types of ether (tell me if I'm wrong), either ETC (classic) or ETH (the current "thing" haha).
I think for some reason Coinbase transferred the 2 dollars of ETH outbound (or maybe took it for themselves??? haha) and/or it just disappeared, because it still hasn't shown up in my new "ETC" wallet which isn't "ETH".
Lemme know if that's sounds about right to you or if I'm way off.  Obviously I'm fairly new at all this even though I first started out buying very small amounts of BTC 3 years ago when I created a Coinbase account and a Blockchain wallet.  I had no clue what I was doing and 3 years later after lots of reading and listening to 100's of interviews and podcasts about all this I still feel like I can't explain it very well or convincingly enough to friends and family.  None of whom have yet to open accounts anywhere and buy some BTC or ETH or whatever yet.
That's how I know this stuff isn't in a "bubble" or going away.  Absolutely everyone I know up to this point don't know anything about cryptos and don't have accounts or "own" any online of offline.  And my inability to explain it technically or properly at a layman's level (or even high level! haha) is probably keeping them from getting in and remaining skeptical about it.
I feel they are missing the train as it leaves the station at this point.
Anyways...long story longer :)  I'm just really glad I didn't do worse than losing 2 dollars worth.  I was so close to transferring half of my ETH on the first try before I stopped myself at the last second and just did 2 dollars instead.
Wish I woulda done just 1 dollar though.  hahaha
Well, thanks much again for getting back to me.  If you have any idears or wanna tell me if I'm kinda right or not about what may have happened, please do.  I would like to know if possible and learn more so if I am talking to anyone again about this I can at least inform them of something like this so they don't make the same mistake that I did.
Take care,
Ian Curran
Tampa, FL
-----Original Message-----
From: "Elaine Ou"<EMAIL_ADDRESS>Sent: Tuesday, June 13, 2017 2:29am
To: "ethereumproject/etherwallet"<EMAIL_ADDRESS>Cc: "mtbiker98"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Subject: Re: [ethereumproject/etherwallet] How long until transfers show up in new wallet? (#43)
the balance should update within a minute. try gastracker.io if the etherwallet node is not updating
—
You are receiving this because you authored the thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .

Hi Ian,
I actually live in Tampa as well and I do help people in the area become more familiar with crypto currencies.
Yes, there are two Ethereums. Ethereum and Ethereum Classic. It's a long story. Luckily for you, the address you made exists on both blockchains. It also has the same key. So if you sent ETH to it instead of ETC, you probably have a high chance of recovering it..
Go to https://myetherwallet.com.  Upload the key. You should see the ETH.  https://classicetherwallet.com is for ETC.
If you would like to get into contact for me to learn more. Please feel free to call or text at<PHONE_NUMBER>.
Carlos Graterol
Marketing Manager at ETC Dev Team


Fix or disable clang format checks

As part of our CI we run clang-format to ensure that code is properly formatted. Sadly IDEs use different clang-format versions. Visual Stuido 2022 e.g. uses an newer version than we do. For people using MSVC 2022 (like me), this makes it pretty much impossible to write code that doesn't fail CI. We somehow need to do something about this urgently.

I'm aware of that option. But tbh. I'd prefer if we had a way where no changes to a IDE setup would be required. I don't want to adjust a global setting just for a single project.

Does #939 help resolve this for you?
Out of the options we tried the only way we can fix this for MSVC out of the box is to update the clang format version in the CI. Which in turn will just break again in the futur when MSVC updates its dependencies
With #939 all users should have the same experience after installing pre-commit. It also will allow us to automate the copyright checks in the future too

Fixed via #939.

Reopening again. I can't get one of my PRs to pass clang-format checks. Been trying for hours. See https://github.com/KhronosGroup/Vulkan-Samples/actions/runs/8381317875/job/22952552513?pr=887
It's both infuriating and frustrating.
Can we add some way to forcefully skip this check?

Any idea where to get the clang version we use in the CI (15 afaik?). I downloaded LLVM for Win64 and changed MSVC to use that, but it seems that version is even newer than the one from MSVC (17).

It's getting worse. So I downloaded LLVM and told VS to use clang-format from that instead (as in Bradley's screenshot). It did format a few things differently, and now I get even more clang format errors in the CI.

https://github.com/KhronosGroup/Vulkan-Samples/actions/runs/8381317875/workflow?pr=887
The CI output is utterly useless. It's a gigantic one liner that tells you roughly where the problem is, but not not WHAT it actually is.

It looks like either the output of clang-format (steps.clang-diff.outputs.changes) is being collected up word-wise into the variable or is being evaluated word-wise and thus collapsing whitespace.
Someone with good enough YAML-fu could take a look to see how this could be changed to output the diff verbatim.  My YAML ability is non-existent and a quick search didn't reveal anything.  Last people to touch that workflow file at that step are @charles-lunarg and @TomAtkinsonArm

I've been thinking about this.  Maybe a solution is instead of having the CI notice differences to have the CI save the changes to a new file and commit it back while failing the build, so all you have to do is sanity check that file, move it over to the offending one and then you'll pass CI?
That way at least there's a quick path towards solving the under-laying issue that clang-format isn't meant to have any backwards compatibility.  The only real way is to have one clang-format version that the whole project uses; which doesn't bode well when people update their IDEs (something we want to encourage).  The only other solution is to not enforce clang-format.

Another alternative is just to add the nicely formatted log of the change file in the CI. In the worst case scenario you will be able to read and manually fix the discrepancies.
I ran pre-commit manually on Saschas PR and it seems to have fixed the files. I dropped a comment on that PR with an example command to run
The final alternative would be to bump our clang tooling to the latest supported by MSVC. Fairly easy to do. It may remove some barrier to entry
I do think that pre-commit is the most sensible solution as it pulls the correct versions and is IDE agnostic. But I am bias, I use this at work across multiple repositories and we practically automate all tests, lints, formats etc with it across multiple tech stacks (Web, native c++, unreal engine, go, I think there's some Java in there too but I stay clear from those repos)
Let me know if we want a version bump and I'll sort out the docker images

Personally, I'd favor pre-commits as well.  However, we have to think of maintainers as customers/end users.  We can't mandate that everyone have a particular tool or setup if we want to maximize the number of people that can contribute to the project.  So pre-commits are great for us on platforms that are setup for it, but less great for people who either have never used it or run into issues with it.  Meeting them in the middle with a path to get out of hours of trying to get past CI seems like a good compromise.
What I did was create a very quick PR which allows the CI to save the artifact that is generated via the clang-format job.  That way if you make a PR and it fails, you can download the diff and hopefully that will make it easier to work with.  Might have to reformat the diff so it's easier to consume, or just save off the new files and let those be downloaded?  But goal is to meet people in the middle so when they have a build issue we're at least doing a small step to give them a quick path out of the issue.  Just download the artifact run patch and continue on.

We have clang-format in https://github.com/LunarG/gfxreconstruct CI GH Actions and I regularly cut-and-paste the diff reported in our CI (e.g. https://github.com/LunarG/gfxreconstruct/actions/runs/8354729132/job/22868539456) as a patch to fix clang-format on my machine.  It's lazy, but in the absence of a readily-reproducible clang-format that works everywhere it does the job.  I like #996, and even better because it's a downloadable artifact.
I agree something that just reformats it on the way in or provides an actionable commit is preferable.

If the problem is that some people have a different version of Clang on their system, wouldn't it be simpler to just include an additional script (or an additional command line option for the existing script) that will use a docker container to execute the correct clang-format version against the files?

It's not unreasonable to say that version X of clang-format is in use on the developer machine.  Nor is it unreasonable to say that we have a dockerfile which they could install/use.  Nor is it unreasonable to say that there's pre-commit scripting support.
However, in the unlikely event those don't get used, it's not a bad thing for us to provide a path out of having this issue.  I personally don't use either pre-commit nor the dockerfile as I primarily do my dev in CLion, which supports using pre-commit, or any version of clang-format directly.  However, every now and again even I run into problems; not that I claim to be a competent programmer; I think it's a small thing to offer another path towards getting people to work with our repo.
Also, something else to bare in mind.  This project is meant to be able to work on platforms that might not even have support for LLVM.

I don't mean to suggest that we shouldn't do any particular improvement that's been suggested.  Simply that providing the option of using a container based clang-format might be an easy to accomplish way to solve the original problem.

This project is meant to be able to work on platforms that might not even have support for LLVM.

Do you mean "work" in the sense of "people can build and run the examples" or in the sense of "people are actively writing code and submitting PRs"?  Because I assume that the issue of LLVM availability is really only a concern to the latter.  Most of my experience developing on platforms with sparse options for tooling have been embedded stuff that ultimately requires some kind of host system to work with, but I haven't spent a lot of time working on GPU stuff beyond the desktop environment.

The way I think of it is, if Vulkan can work on a given platform, then it'd be nice if samples could work on that same platform.


[UX/UI] Tooltips needed on media nav buttons

The buttons should have hover over text showing what the buttons do.
For instance, https://archive.org/details/birdbookillustra00reedrich/mode/2up?view=theater has active tooltips.
From testing:
The navigation buttons that appear with individual items don't have “text” when you hover over them. It isn't always clear what each button does.
Please yes, pre-launch for sure. This is a good catch by them and should absolutely be included in the UI to help UX. Icons alone are not ideal (though somewhat unavoidable on mobile). This is a feature the original reader has. https://archive.org/details/birdbookillustra00reedrich/mode/2up?view=theater

Design Specs

Style tooltips similar to AE, but in brand color #022851
Tooltips show on default view and fullscreen modes on all the round blue buttons
Verbiage for labels pictured below (obviously these are not positioned correctly, just showing which buttons the words belong to)

Default View

Fullscreen View

@wrenaria @EricNebeker this should be deployed to dev now.

I see a couple adjustments to make --
(1) When "share" options are showing, the button should remain the dark hover color and the tooltip should not show. Currently both the sharing options and tooltip show as stacked on top of each other.

Current
Desired

(2) In fullscreen view the default "Search Inside" button is the wrong color. Should be aggie-blue-80 #13639E like the other round buttons. It is also missing the active coloration that appears in the default view when the yellow search box is open.
(3) When "Search Inside" is open/active, can we change the label to say "Hide Search Box" ?
Screenshot of current active state:


Error when running run.sh

Hello,
I am currently getting the following error when running run.sh
root@raspberrypi:/home/pi/vr-test# sh run.sh
run.sh: 3: run.sh: run.sh: 5: run.sh: devilspie: not found
./head-tracking.py: Permission denied
Error: Failed to open I2C device /dev/i2c-0: No such file or directory (system:2)
[0218/210551.611639:ERROR:chrome_main_delegate.cc(627)] Web security may only be disabled if '--user-data-dir' is also specified.
[2281:2281:0218/210552.045145:ERROR:zygote_host_impl_linux.cc(89)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.
run.sh: 8: kill: No such process
run.sh: 9: kill: No such process
run.sh: 10: kill: No such process
Can you please let me know what the issue could be.
Kind Regards,
Jason

So i have figured out part of it but i now need to know where you got the modified minimu9-ahrs to bypas the no gyro error.
4) Install modified minimu9-ahrs (bypass "No gyro found" error).
Please can you help

Jason,
Apologies for not responding to your question right away.
I got it by downloading the source from GitHub and changing it myself. In minimu9-ahrs.cpp there's a function main_with_exceptions that fails if there are missing sensors:
  if (missing)
  {
    if (missing.mag)
    {
      std::cerr << "Error: No magnetometer found." << std::endl;
    }
    if (missing.acc)
    {
      std::cerr << "Error: No accelerometer found." << std::endl;
    }
    if (missing.gyro)
    {
      std::cerr << "Error: No gyro found." << std::endl;
    }
    std::cerr << "Error: Needed sensors are missing." << std::endl;
    return 1;
  }

I'm pretty sure I just commented out the return 1; This whole thing is just a simple demo for a project video, so it's a bit klugey.

Hello Andy,
I have tried that and got further but it seems there is more to this that meets the eye as i am now getting
Error: Failed to parse calibration file ~/.minimu9-ahrs-cal
I have touched this file to see if that would fix it but it has not.
I will have to sit down when i have some free time to figure it out.
Thanks for replying

Ah, yes! You need to run minimu9-ahrs-calibrate and follow the directions. That should generate the file.

Hello Andy,
I have run that but still, get errors about the gyro
pi@raspberrypi:~ $ minimu9-ahrs-calibrate
To calibrate the magnetometer, start rotating the IMU through
as many different orientations as possible.
Error: No gyro to read.
You can stop rotating the IMU now.
Warning: Only 0 readings were provided.
Traceback (most recent call last):
File "/usr/bin/minimu9-ahrs-calibrator", line 103, in
run()
File "/usr/bin/minimu9-ahrs-calibrator", line 79, in run
cal = simple_minmax_calibration(raw_readings)
File "/usr/bin/minimu9-ahrs-calibrator", line 98, in simple_minmax_calibration
guess.extend([min(values), max(values)])
ValueError: min() arg is an empty sequence

Hello,
I believe there is something within minimu9.cpp that needs to be disabled as well as this seems to be where the error is coming from that i am getting


MPTweakInline.h unavailable to tvOS

Integration Method: Carthage
Xcode Version: 8.3.2
Library Version: 3.1.8
Platform:  tvOS
Language: Swift / Objective-C
Description:
MPTweakInline.h is unavailable to tvOS, so I am unable to create a MPTweakValue for A/B testing on tvOS. I noticed the Target Membership for the file is not enabled for tvOS. Is this intentional?
Expected Behavior:
MPTweakInline.h together with a list of other headers should be available to the tvOS target.
For reference:
Headers under tvOS

Headers under iOS

Hi @makangus, we actually don't have A/B testing support on tvOS, so there would be no way for tweaks to be used.


Documentation ! help

I have built my own ngrok server, is it possible to integrate my own ngrok service with this ngrok wrapper, it seems that this assembly still needs the ngrok client, can you provide some documentationon this matter ? much appreciated.

Hello @StevenLiSiwei,
Glad to hear you have an ngrok server, though I'll like to know how it differs to ngrok.exe (functionality, launching, etc)?

It seems that this assembly still needs the ngrok client
If I get your point, yes, this library is usable ONLY when ngrok.exe is running.  It connects to ngrok.exe on http://localhost:4040 (or whatever port the service is running on)


Regenerate without special generation of enums with a single member

@tmiasko Thanks.
👍

cc @GuillaumeGomez

Thanks!


fixreturn original expr if cast to decimal literal overflow

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.01 seconds
stream load tsv:          450 seconds loaded<PHONE_NUMBER>9 Bytes, about 158 MB/s
stream load json:         22 seconds loaded<PHONE_NUMBER> Bytes, about 102 MB/s
stream load orc:          58 seconds loaded<PHONE_NUMBER> Bytes, about 18 MB/s
stream load parquet:          28 seconds loaded 861443392 Bytes, about 29 MB/s
insert into select:          69.3 seconds inserted 10000000 Rows, about 144K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230626110318_clickbench_pr_167937.html


youtube-dl.py ytsearch, gvsearch and ybsearch...  hm... and dmsearch, mvsearch? :-)

Hi!
According to http://rg3.github.com/youtube-dl/documentation.html , it is possible to search video sites and automatically download the videos found.
As far as i can see, ytsearch (for youtube), gvsearch (for google video) and ybsearch (for yahoo video) are implemented.
But i can only get ytsearch working. The other boths dont work for me.
Example:
A batch file "test.bat" with the content:

e:\programme\python26\python youtube-dl.py -U
e:\programme\python26\python youtube-dl.py -i -w "ytsearch2:funny cats"
e:\programme\python26\python youtube-dl.py -i -w "gvsearch2:funny dogs"
e:\programme\python26\python youtube-dl.py -i -w "ybsearch2:funny animals"

produces the following output:

c:\test>test
c:\test>e:\programme\python26\python youtube-dl.py -U
Updating to latest version...
youtube-dl is up-to-date (2012.01.08b)
c:\test>e:\programme\python26\python youtube-dl.py -i -w "ytsearch2:funny cats"
[youtube] Setting language
[youtube] query "funny cats": Downloading page 1
[youtube] ElzPZNSdd5w: Downloading video webpage
[youtube] ElzPZNSdd5w: Downloading video info webpage
[youtube] ElzPZNSdd5w: Extracting video information
[download] Destination: ElzPZNSdd5w.mp4
[download] 100.0% of 43.72M at  999.03k/s ETA 00:00
[youtube] w0ffwDYo00Q: Downloading video webpage
[youtube] w0ffwDYo00Q: Downloading video info webpage
[youtube] w0ffwDYo00Q: Extracting video information
[download] Destination: w0ffwDYo00Q.flv
[download] 100.0% of 4.84M at  105.97k/s ETA 00:00
c:\test>e:\programme\python26\python youtube-dl.py -i -w "gvsearch2:funny dogs"
[video.google] query "funny dogs": Downloading page 1
c:\test>e:\programme\python26\python youtube-dl.py -i -w "ybsearch2:funny animals"
WARNING: Falling back on generic information extractor.
[generic] ybsearch2:funny animals: Downloading webpage
ERROR: Unable to retrieve video webpage:
c:\test>

So, from youtube there are two videos downloaded, as intended, very fine :-)
But via google and yahoo none.
Am i doing or understanding something wrong, or is there something broken?
And... unfortunately i am no programmer, so i can not implement it by myself, but i suspect that i am not the only one who would appreciate it very much, if such a search option would be implemented for other video download sites too, e.g. dailymotion or myvideo :-)
Thanks in advance, best regards :-)

Can reproduce the same thing here.

Same problem!

I'll have a look into it. Since I'm currently busy writing my Master's thesis, this may take a while. Expect updates in April. But of course, patches are always welcome ;)

Yahoo video is completely broken. See #300
Issued the pull request for gvsearch #301


extensions: datadog_trace make sure it throws when using unsupported PHP versions

Version 0.75.0 is the last version I can build using PHP 5.6.

That's what we already do.
This snippet:
if lib.versionOlder prev.php.version "7" then
        prev.extensions.datadog_trace.overrideAttrs (attrs: {
          name = "datadog_trace-0.75.0";
          version = "0.75.0";
          src = pkgs.fetchurl {
            url = "http://pecl.php.net/get/datadog_trace-0.75.0.tgz";
            hash = "sha256-xgymzG9QBhLO/sBaEAfAVwPDB8PswB5Yq6yqXbyvAvw=";
          };
        })

Will make sure to use version 0.75.0 only for version of PHP < 7. Since the only version below 7 that we are maintaining is 5.6, it's good! :)
On the contrary, on Nixpkgs, I just pushed an update to get the latest version in, at: https://github.com/NixOS/nixpkgs/pull/242166

Sorry for the noise. I simply misread the code >.<


GUICont2 to Staging - Forgot Password Page

@noodles-1 SOS need help with the password hashing, as well as validation that new pass != existing password.

conflicts resolved.


Types not found - robodux

Failing module

GitHub: https://github.com/neurosnap/robodux
npm: https://npmjs.com/package/robodux

import type {
  LoadingItemState,
} from "https://esm.sh/robodux@15.0.0?pin=v121";

Error message
After running deno run I got this:
error: TS2305 [ERROR]: Module '"https://esm.sh/robodux@15.0.0?pin=v121"' has no exported member 'LoadingItemState'.
  LoadingItemState,
  ~~~~~~~~~~~~~~~~

Additional info

esm.sh version: v121
Deno version: deno 1.33.0 (release, x86_64-unknown-linux-gnu)

CURL
curl -v<EMAIL_ADDRESS>< x-typescript-types<EMAIL_ADDRESS>< report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=tyvOKbUkk%2F2j4si%2F9diJkeMgEp3OpFiTwAFPMZE84HapX6UkgFXH2J2CmfoZgk59xBL5p7%2B61ey4U%2BGrLSPs09qinXSISKJ147q16OUBMh8z4XDX2TdBJos%3D"}],"group":"cf-nel","max_age":604800}
< nel: {"success_fraction":0,"report_to":"cf-nel","max_age":604800}
< server: cloudflare
< cf-ray: 7ca5a76b5ce061c8-ORD
< alt-svc: h3=":443"; ma=86400, h3-29=":443"; ma=86400
<
/* esm.sh -<EMAIL_ADDRESS>*/
export * from<EMAIL_ADDRESS>* Connection #0 to host esm.sh left intact

curl<EMAIL_ADDRESS>Types not found%

@neurosnap deployed, can you please try again? thank


Preparation of meeting and guide for modelling of physical systems with the use of Python programming language

The following actions are to do, in order to prepare the meeting and the guide:

[x] #1016

[x] #1017

[x] #1018

All done.


Feature Request: Clear QR Code history

In the documentation is written:
"It is recommended to configure your app to ignore QR codes older than a specific timestamp. Currently, the API does not support clearing QR code history."
I would like to clear the QR code history, since currently the QR codes are persistent not only over sessions of the same app, but also over sessions of different apps.

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

ID: 6572ea07-1ad5-ef4e-72de-e3f5f7e2b086
Version Independent ID: f8a7b6db-5cd3-b895-0e5f-b079b60b8394
Content: QR code tracking - Mixed Reality
Content Source: mixed-reality-docs/qr-code-tracking.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @dorreneb
Microsoft Alias: dobrown

I second this.

+1


Remove unnecessary use of read_link

Perf improvement for symlink detection based on https://github.com/Peltoche/lsd/issues/538

TODO

[x] Use cargo fmt
[x] Add necessary tests
[ ] ~Add changelog entry~

Codecov Report

Merging #539 (7730544) into master (70ec1d6) will decrease coverage by 0.26%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #539      +/-   ##
==========================================
- Coverage   88.48%   88.22%   -0.27%
==========================================
  Files          36       36
  Lines        3535     3567      +32
==========================================
+ Hits         3128     3147      +19
- Misses        407      420      +13

Impacted Files
Coverage Δ

src/meta/mod.rs
38.33% <25.00%> (-1.15%)
:arrow_down:

tests/integration.rs
100.00% <100.00%> (ø)

src/meta/symlink.rs
65.30% <0.00%> (-4.09%)
:arrow_down:

src/main.rs
7.69% <0.00%> (-2.31%)
:arrow_down:

src/meta/filetype.rs
80.00% <0.00%> (-1.06%)
:arrow_down:

src/meta/size.rs
94.15% <0.00%> (-0.59%)
:arrow_down:

src/flags/sorting.rs
99.06% <0.00%> (-0.47%)
:arrow_down:

src/display.rs
72.83% <0.00%> (-0.11%)
:arrow_down:

src/app.rs
95.83% <0.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


Gardenlet deployment using helm fails

How to categorize this issue?

/area control-plane
/kind bug
What happened:
While deploying the Gardener v1.52.0 gardenlet deployment fails with the following error:
Error: kind PriorityClass with the name "gardener-system-critical" already exists in the cluster and wasn't defined in the previous release. Before upgrading, please either delete the resource from the cluster or remove it from the chart
Error: UPGRADE FAILED: kind PriorityClass with the name "gardener-system-critical" already exists in the cluster and wasn't defined in the previous release. Before upgrading, please either delete the resource from the cluster or remove it from the chart

But the gardenlet images gets upgraded to the latest version. This removes the priority-class from managed resources and the priority class gets garbage collected.
So if one deploy the gardenelet again it will succeed without error.
What you expected to happen:
The gardenlet deployment should succeed without error.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Gardener version: v1.52.0
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

/cc @kris94 @ialidzhikov @timebertt

I see there is a mechanism in helm 3 for adopting existing resources if they are correctly labeled: https://github.com/helm/helm/pull/7649
However, leveraging this mechanism in gardenlet will get quite complicated.
Also, it will not help our internal case because we are still not using helm 3 (😞).

But the gardenlet images gets upgraded to the latest version. This removes the priority-class from managed resources and the priority class gets garbage collected.
So if one deploy the gardenelet again it will succeed without error.

If that's true and it will succeed after one failure, I don't mind living with this bug, considering that workarounds/fixes will be highly complicated.
We could even edit the release notes to explain this to gardener operators.

My suggestion is to use the garden priority class for v1.52 and let MR clean the old priority class garden-system-critical. Later In in v1.53, we can switch back again to  garden-system-critical. Because deploying 3-4 on canary and live doesn't seem right to me.

My suggestion is to use the gardenlet priority class for v1.52 and let MR clean the old priority class garden-system-critical. Later In in v1.53, we can switch back again to garden-system-critical. Because deploying 3-4 times on canary and live doesn't seem right to me.
Is this reasonable @timebertt @ialidzhikov?

xref https://github.com/gardener/gardener/issues/5634

Sounds good to me. This might be the simplest solution we can come up with
This also means we need to partially revert https://github.com/gardener/gardener/pull/6235 for now.

I couldn't follow all the details but if (partial) revert of the PR that introduces the issue (https://github.com/gardener/gardener/pull/6235) does the trick, then fine from my side. 👍

I couldn't follow all the details but if (partial) revert of the PR that introduces the issue (#6235) does the trick, then fine from my side. 👍

#6430 partially reverts #6235.

Hm, I am not quite sure that this approach will work. I see that in https://github.com/gardener/gardener/pull/6257 components like istiod and gardener-resource-manager are adapted to use the garden-system-critical. With #6430 I don't see who creates this garden-system-critical PriorityClass. Or do I overlook something?

Or do I overlook something?
No, I also just checked that #6430 is merged.

Will it work if we completely revert #6235? IIUC now https://github.com/gardener/gardener/pull/6430 is partial revert of #6235 (without the changes that deploy gardener-system-critical as part of the Seed bootstrap). And to me it seems that the issue is that in #6430 we don't deploy the gardener-system-critical PriorityClass.

As https://github.com/gardener/gardener/issues/6257 is meanwhile merged (and there are seed components that rely on the gardener-system-critical PriorityClass), with @acumino we rather tend to completely revert https://github.com/gardener/gardener/pull/6235. It can be followed up separately later on how to migrate the gardenlet Deployment to the new PriorityClass.

I had a call with @kris94 , he is also fine with the revert of https://github.com/gardener/gardener/pull/6235 and then cherry-picking it to v1.52.
Then we can follow a 2 release migration process for switching gardenlet back to the gardener-system-critical PriorityClass.

I had a call with @kris94 , he is also fine with the revert of #6235 and then cherry-picking it to v1.52. Then we can follow a 2 release migration process for switching gardenlet back to the gardener-system-critical PriorityClass.

Since #6257 is merged not only gradenlet but everything which is using gardener-system-crititcal priority class needs to be switched to some different priority class as the first step and later in second step e can clean that temporary priority class and let helm deploy the gardener-system-critical priority class.


Implemented the local notifications in the application

Description
I've implemented the local notifications into the application along with all of it's necessary methods in the new notifications.dart file. I've also pre-initialized and configured the application with the notifications package. Now the notifications can be utilized as and when required.
Fixes #365
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@avinashkranjan please review this PR


CLI create command still try to create even when the help flag is used

Which version of kubefirst are you using?
v2.2.17
Which cloud provider?
k3d (local)
Which installation type?
CLI
Which distributed Git provider?
None specific
What is the issue?
The CLI create, even when used with the --help flag, is trying (or create) a cluster. When the help flag is used, it shouldn't create the cluster, and just display the help information.

It even asked for root access, not sure why, but here's the error I had after since I already had a cluster created, and running.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It's not an issue with kubefirst 🤦♂
I did have an alias to do extra stuff after creating a new cluster (like connection the argo CLI, updating KUBECONFIG paths...), and I didn't manage the help flag, so the error showing after are for my other commands...

It's not an issue with kubefirst 🤦♂
I did have an alias to do extra stuff after creating a new cluster (like connection the argo CLI, updating KUBECONFIG paths...), and I didn't manage the help flag, so the error showing after are for my other commands...
Again... 🤦♂


Grouping support

Hi Eliot. Have you worked with grouping in the Doxygen flow (https://www.stack.nl/~dimitri/doxygen/manual/grouping.html)? Our current doxygen-to-dita flow is a customization of the HTML generation program, which builds the navigation in a way that considers "grouping" instructions. The XML output does not seem to respect grouping information explicitly. We are analyzing the XML to determine whether we can infer grouping information and rebuild the DITA map taking into account this grouping information. If you've done some analysis here, I would appreciate your guidance.

I have not done anything with grouping instructions (didn't know those were a thing). If the information is not in the XML then I don't have it available.
For the Oculus-driven work I'm doing I simply emulating the organization of what they've publicly published. I have no idea how generally applicable that code will be.
If you can point me to specifications or examples I can see what I figure out.

Thanks, Eliot. Tushar is doing a first round of analysis and we'll publish the results.

Hi All,
Grouping is great way to categorize information generated by Doxygen.
However, grouping is optional and it is therefore, you do not see any group
information generated when you process the index.xml file. It is important
to note that if you have groups defined in your source files, you should
see 'group__.xml' files generated in your output folder. You may
want to look into the .xsd files to see if you can include the group
information there. Similar to the DoxygenLayout.xml file that control the
appearance of the HTML/PDF output formats.
Eliot, I observed the Oculus developer documentation and you're right
groups are not defined in the source files.
I have always been looking for a similar solution and am already excited!!!
-Aman Talwar
On Tue, May 10, 2016 at 8:24 PM, sethpark<EMAIL_ADDRESS>wrote:

Thanks, Eliot. Tushar is doing a first round of analysis and we'll publish
the results.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/dita-community/doxygen2dita/issues/3#issuecomment-218183161


📄 Add CODE_OF_CONDUCT.md file

📄 Add CODE_OF_CONDUCT.md file
Add a CODE_OF_CONDUCT.md file to the root of the project.

Hello, could you assign this issue to me ?

Hello @PrerakMathur20 .
I would like to work on this issue. Could you please assign it to me.

@Nincompup Assigned to you!
@KanishkDadhich Since the Issues are FCFS, I can't assign you this one. Feel free to look around for more issues you're comfortable with.


Cannot create application setting starting with "IDENTITY_"

When creating application settings starting with IDENTITY_ they don't show up in the SWA.
Steps to reproduce:
az staticwebapp appsettings set --name <YOUR_SWA_NAME> --setting-names IDENTITY_DB_NAME=test IDENTITY_DB=test IDENTITY_ABC=test test=test
Results in these app settings:

@annaji-msft

Hey @itpropro, these are some of the app settings that are not allowed. If you'd like to use them you need to use bring your own functions, more details here:
https://docs.microsoft.com/en-us/azure/static-web-apps/apis#constraints

Thanks for documenting them @miwebst I will close the issue now :)


Pass options to getters and setters

@apotonick as per our email conversation.
This PR allows to do the following:
class SongDecorator < Representable::Decorator
  property :title, exec_context: :decorator

  def title(user_options:, **)
    user_options[:upcase_title] ? represented.title.upcase : represented.title
  end
end

The same can be done with setters:
class SongDecorator < Representable::Decorator
  property :title, exec_context: :decorator

  def title=(input, user_options:, **)
    super(user_options[:upcase_title] ? input.upcase : input)
  end
end

In addition to accepting the user_options keyword argument, the methods can also accept options, if the full options hash is needed.
I am not sure if there's a better approach to it.
Also not sure if a test would be required or if the existing tests are fine. I've run into an issue writing a test for it because the test representer (#representer!) seems to disregard the exec_context option for properties and always call getters and setters on the decorated object instead.

@apotonick bump?


Meteors now leave behind a bit of ore

About the PR

Made meteors leave behind asteroid rocks after the explosion
Why / Balance

idk, an admin told me to do it
flavor?
Technical details

Created WeightedSpawnEntityBehavior that takes a WeightedRandomEntityPrototype and spawns a number of the same entity between a given min and max, at a random offset from the final position of the entity.
Spawning can be delayed if, for example, the entity has an explosive behavior that would destroy your newly spawned entity. If this happens, a temporary entity is created to act as a spawner.
Created a spawn table of possible asteroid ores that the meteors will now leave behind with values chosen at whatever I randomly decided was a number
Arbitrarily chosen spawn values that seemed appropriate for the size of the meteors

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

KABOOM VIDEO
https://www.youtube.com/watch?v=tgqEwkIMV7o
Breaking changes

Changelog

:cl:

add: Meteors now leave behind asteroid rocks on impact.

This is actually very cool, woah..

Maybe salvage will finally stay on station!

TryStopNukeOpsFromConstantlyFailing my NEMESIS


Updating model does not update selectize

Continuing from https://github.com/machineboy2045/angular-selectize/issues/47
Adding an item to the model used for options in Selectize does not change the options.
JS
$scope.people = [{id: 1, name: "Don"}, {id: 2, name: "John"}];

$scope.myConfig = {
    options: $scope.people,
    valueField: 'id',
    labelField: 'name',
    create: true,
    sortField: 'text',
    maxItems: 1
  }

$scope.addPerson = function(){
  $scope.people.push({ id: 3, name: "Melissa" });
}

HTML
<selectize ng-model="person" config="myConfig" placeholder='Select a person...'></selectize>

<button ng-click="addPerson()">Add Melissa</button>

$watch has been updated to $watchCollection in my version.

Options that are passed via config are not bound to the scope of the directive and can thereby not be watched. Try passing the options via HTML attribute.
<selectize ng-model="person" options="people" config="myConfig" placeholder='Select a person...'></selectize>

Passing the options via HTML attribute does not seem to work in my case. I load options from the server. The options are bound to $scope in an http callback. The options list is empty.

My problem was fixed by loading the list in the http callback in which options are loaded.

I load the list on my service callback and options are not loaded. Can you provide some code to see the difference?

@mariowise I finally found my problem was the $scope. I have one selectize in each item in a list, and I'm updating the options from another scope which is far away. I end up updating the selectize using a lot of $scope.$parent.

I thing I misspoke. I have a similar problem to yours. And I can't solve it... my selectize stays with an [] (empty Array) and the options are not updated when my service (promise) resolves. Any ideas?

@mariowise things get really complicated when the $scope come to play. I end up solving my problem with the following code:
<selectize config="indexColumnConfig" options='$parent.tableColumns'
                                           data-ng-model="tableIndex.columnName"></selectize>

oh right! was my mistake.. I was also passing the variable throw a $parent controller... now everything is working... Thanks!


AttributeError: 'NoneType' object has no attribute 'shape'

where I run tools/test.py  ，I have the following problems：
for i, data in enumerate(data_loader):
File "/home/zjr/anaconda3/envs/AerialDetection/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 582, in next
return self._process_next_batch(batch)
File "/home/zjr/anaconda3/envs/AerialDetection/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 608, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
AttributeError: Traceback (most recent call last):
File "/home/zjr/anaconda3/envs/AerialDetection/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/zjr/anaconda3/envs/AerialDetection/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/media/sdc/zjr/code/AerialDetection/mmdet/datasets/custom.py", line 197, in getitem
return self.prepare_test_img(idx)
File "/media/sdc/zjr/code/AerialDetection/mmdet/datasets/custom.py", line 378, in prepare_test_img
img, scale, False, proposal)
File "/media/sdc/zjr/code/AerialDetection/mmdet/datasets/custom.py", line 332, in prepare_single
img, scale, flip, keep_ratio=self.resize_keep_ratio)
File "/media/sdc/zjr/code/AerialDetection/mmdet/datasets/transforms.py", line 33, in call
img, scale_factor = mmcv.imrescale(img, scale, return_scale=True)
File "/home/zjr/anaconda3/envs/AerialDetection/lib/python3.7/site-packages/mmcv/image/resize.py", line 132, in imrescale
h, w = img.shape[:2]
AttributeError: 'NoneType' object has no attribute 'shape'
I'm sure the image exists and no Chinese path.
If you have already identified the reason, you can provide the information here. Thanks

Some problem. Have you solved it?

Some problem. Have you solved it?


refactor(landing): migrate index to tsx

This commit refactors client/src/components/landing/index.js component into a React.FC component. The file extension was changes from js to .tsx.
Related issue: #43778

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Tested this locally and the site builds without issues. As far as using tsx instead of ts for the file extension, I made a judgement call. I personally like using jsx/tsx whenever there's HTML alongside the JS in a React file.
One quick-question. Unsure as to what the displayName property is. Is that needed? I can check line 42 back in, I just wasn't sure what that was in reference to.
Thanks and looking forward to your comments and feedback 😄 🍂

Hi @luqven, I wouldn't normally recommend changing code in a TS migration, but since this is relatively small change, I think this is fine.
By the way, the tests will need to be updated if we use function Landing, but if you run npx jest it should be reasonably clear.
Give me a shout if you need any more feedback.

Thanks for the feedback @ojeytonwilliams !
TIL about not using FC , good callout. Apols for the late turn around on this, addressing your comments now and will push up soon.

Hey @luqven do you think you'll have a moment to make those changes?

Hey @luqven do you think you'll have a moment to make those changes?

Hey @ojeytonwilliams I meant to ping you a while back. I've been stuck on updating the tests
expect(view.type.displayName === 'Landing').toBeTruthy();
Took me an embarrasingly long time to realize I was missing this line
Landing.displayName = 'Landing'; 🤦🏼
Just pushed up some changes.

Took me an embarrasingly long time to realize I was missing this line

No worries!  It's totally cool to ping people if you're confused about something, though.  We're happy to help where we can.


fix: CSS Variables LSP default settings

Without providing default values via the lua config, you get an error similar to this:
TypeError: Cannot read properties of null (reading 'lookupFiles')
  at CSSVariableManager.parseAndSyncVariables (/Users/gegoune/.pnpm-global/5/.pnpm/css-variables-language-server@2.7.0/node_modules/css-variables-language-server/dist/index.js:250:54)
  at /Users/gegoune/.pnpm-global/5/.pnpm/css-variables-language-server@2.7.0/node_modules/css-variables-language-server/dist/index.js:328:22\n\nNode.js v21.6.2\n"

The LSP provides a default (https://github.com/vunguyentuan/vscode-css-variables/blob/763a564df763f17aceb5f3d6070e0b444a2f47ff/packages/css-variables-language-server/src/CSSVariableManager.ts#L31-L50) but it doesn't seem to work when run in Neovim. I'm not sure of the best way to resolve this (does something need to be addressed in the language server?) but an easy fix is to hardcode the settings in the lspconfig.

Can confirm it solves the issue for me, thanks!


Handling of "'1" expression

from https://github.com/chipsalliance/Surelog/issues/3375
Reading this Verilog with UHDM then Yosys has the wrong values.
typedef struct packed {logic [3:0] a;} my_struct_packed_t;

module unsized_single_bit_1 (
    output wire [3:0] out1,
    output wire out2,
    output my_struct_packed_t out3,
    output wire out4
);

  // out1 should be 4'b1111 --> OK
  assign out1   = '1;

  // out2 should be 1'b1 --> OK
  assign out2   = (out1 == 4'b1111);

  // out3 should be 4'b1111 --> NOT OK
  assign out3.a = '1;

  // out4 should be 1'b1 --> OK
  assign out4   = (out3 == '1);

endmodule : unsized_single_bit_1

FYI @QuantamHD @hzeller

@grotival This should be fixed when using yosys-f4pga-plugins after: https://github.com/chipsalliance/yosys-f4pga-plugins/pull/519. Could you please confirm?

All good, thanks!


Laravel 4.2 App::finish doesn't run after response is returned.

I was trying to do some lengthy processing on the backend after a response is returned to the client using App::finish but apparently what happened is that it actually wait for the processing to finish and then the response only get sent to the client.
Tried to put a dd('test) inside the App::finish, and in fact I received the response followed by the "test" string.
The laravel 4.2 docs say "The finish event is called after the response from your application has been sent back to the client"
Hope someone can shed some light regarding this. Thanks

4.2 isn't supported anymore. The forums might be a good place to ask though. :)

Sorry about that. Will head over to the forum for this matter. Thanks


add security writeups and dev tools link

Looks good, add an entry to the Table of Contents. Maybe there are a few other historical writeups that everyone should be familiar with. Maybe an old Hacking Distributed writeup on the DAO hack and similar stuff?

Added toc. There are definitely other good ones but I don't have links handy. Can do them in a later commit.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Infinite loop on large or invalid path request

I have the same issue as in this issue:
https://github.com/recastnavigation/recastnavigation/issues/45
https://github.com/recastnavigation/recastnavigation/issues/178
I'm running with a slitghly modified version of RecastNavigation but, it does includes all the fixes for "Maxnodes".
Let me know if I can do any interesting logging.
I can provide a patch file of all my modifications  privatly if necessary.
Note: In the target game distance, "yards".
I can generate path up to 13000 yards without issue but yet it would infinite loop on path  of ~4000yards sometimes.
Also: Does Recast/Detour can works in a multi-threaded environment if 2 path are requested from 2 differents threads at the same time or that would be a major issue and probably the source of my problem ?

Recast and Detour is not thread safe; you should not use dtNavMeshQuery simultaneously from different threads. You can create multiple instances and attach them to the same dtNavMesh instance - this is safe. dtNavMesh is also safe to use from multiple threads, but only for reading (i.e. no addTile calls and the likes).


target.txt keep only the command of the last scanned url for a subdomain

Scanning multiple urls for the same subdomain, in case injections points are detected on more urls, the file target.txt contains only the ghauri command related to one url.
Steps to reproduce the behavior:

scan a url for a subdomain where ghauri can find an injection point in GET parameter (POST may have the bug too)
scan another url for the same domain, ghauri should find another injection point in GET parameter
check file $HOME/.ghauri//target.txt. You'll have only one command.

Expected behavior
Having the commands related to all urls scanned
Desktop (please complete the following information):

OS: Debian and Kali
Ghauri version which is causing the error (should be latest): 1.3
Type of SQL injection which cause the error: [e.g. boolean/time/error based]
Phase where error occur: data

Additional context
I did a bash script with a loop to scan multiple urls with ghauri. Using files saved in the .ghauri/ folder, reviewing the results is not easy.
In log, we don't have the url. In session.lite, there's not a column with the url with parameters, we should assemble it using the endpoint, he subdomain and the parameters columns and the target.txt has the mentioned bug.

will check this thanks for the issue.

@DylanTiger sorry for the very delayed response but i have checked it now and it is intended response this is how it should work, the targer.txt just saves the current command ran not the history for it even you can try sqlmap it does the same if you try it the same way you suggested in the issue.


Download button

I started a small project with Pode.Web for searching Exchange with Get-MessageTrackingLog https://github.com/ili101/ExchangeMessageTrackingLogWeb.
I want to add an Excel download button to the WebTable like the embedded "Export" (CSV) one.
I made a WebSection with a button and moved the data to it with Get-PodeState but I don't know if it's the best way to do it or what to do from here.
Something like save the file to some temporary folder and link the button to it and then delete it?
Can you help? (If you like to test it the code by default doesn't need Exchange, it use dummy data for tests)

That's actually a neat way of doing it! As it stands at the moment I can't really think of another way.
The only other idea is that in 0.3.0 (#8), tables are now their own element and the refresh/export buttons belong to each table. Maybe we could have a New-PodeWebTableButton which allows you to add new meta buttons to a table 🤔 And that button's scriptblock is sent the table's current viewable data.
To download the file, via your current button or the above idea, you should be able to do Set-PodeResponseAttachment in the button's scriptblock, and point to the file created by Export-Excel - later deleting the file at some point (not straight away).

Tried but didn't manage to get it working from the button, File is not downloaded. Not sure what is missing.
https://github.com/ili101/ExchangeMessageTrackingLogWeb/commit/696441dedd5766f0f5a788581cbf101f21d9e843
Also tried Move-PodeResponseUrl I think the browser downloads it but not showing the download popup.

Two things on this one, first I've fixed why both Set-PodeResponseAttachment and Move-PodeResponseUrl weren't working in the develop branch.
Also, in your script when saving/exporting the Excel file, the path will need to be joined onto the server path:
$path = Join-Path (Get-PodeServerPath) '.\Storage\test.xlsx'
Export-Excel -InputObject $global:Results -WorksheetName 'Log' -TableName 'Log' -AutoSize -Path $path

^ and that should do it 😄

Two things on this one, first I've fixed why both Set-PodeResponseAttachment and Move-PodeResponseUrl weren't working in the develop branch.
Also, in your script when saving/exporting the Excel file, the path will need to be joined onto the server path:
$path = Join-Path (Get-PodeServerPath) '.\Storage\test.xlsx'
Export-Excel -InputObject $global:Results -WorksheetName 'Log' -TableName 'Log' -AutoSize -Path $path

^ and that should do it 😄

It works with csv file but when using xlsx the file get corrupted.
On disk it's saved ok, browsing to the URL manually in the browser download correctly but downloaded copy by clicking the button is re-encoded or something.
https://github.com/ili101/ExchangeMessageTrackingLogWeb/blob/6ca939ab5cc2a7f9252efd8edb7dc1ac3f755863/Home.ps1#L95-L96

It works with csv file but when using xlsx the file get corrupted.
On disk it's saved ok, browsing to the URL manually in the browser download correctly but downloaded copy by clicking the button is re-encoded or something.
https://github.com/ili101/ExchangeMessageTrackingLogWeb/blob/6ca939ab5cc2a7f9252efd8edb7dc1ac3f755863/Home.ps1#L95-L96

Managed to reproduce it, what's in develop now lets me download excel files.

Managed to reproduce it, what's in develop now lets me download excel files.


STY: Align all build_ext definitions the same way.

No functional change.
Done to improve diff with Q2.0 which adds more modules, and re-aligned the two modules changed here.

thanks


model_dump(mode="python") returns tuple instead of namedtuple

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
After migration to pydantic V2 model_dump() now returns a tuple instead of a namedtuple when the model uses a namedtuple
returning a simple tuple makes it impossible to access the tuple items by name.
While it makes sense to get rid of namedtuple for JSON format, the python format should keep it as before.
The migration documentation was only talking about the JSON schema:
The JSON schema no longer preserves namedtuples as namedtuples.
Example Code
import typing

from pydantic import BaseModel as BaseModelV2
from pydantic.v1 import BaseModel as BaseModelV1

class Foo(typing.NamedTuple):
    url: str

class ModelV1(BaseModelV1):
    foo: Foo

class ModelV2(BaseModelV2):
    foo: Foo

objV1 = ModelV1(foo=Foo(url="https://example.com"))
print(type(dict(objV1).get("foo")))
# print: <class '__main__.Foo'>
print(type(objV1.dict().get("foo")))
# print: <class '__main__.Foo'>

objV2 = ModelV2(foo=Foo(url="https://example.com"))
print(type(dict(objV2).get("foo")))
# print: <class '__main__.Foo'>
print(type(objV2.dict().get("foo"))
# print: <class 'tuple'> (and a deprecation warning)
print(type(objV2.model_dump().get("foo")))
# print: <class 'tuple'>

print(objV1.dict())
# print: {'foo': Foo(url='https://example.com')}
print(objV2.model_dump())
# print: {'foo': ('https://example.com',)}

print(objV1.dict()["foo"].url)
# print: https://example.com

print(objV2.model_dump()["foo"].url)
# Traceback (most recent call last):
#   File "/Users/sileht/workspace/mergify/engine1/toto.py", line 35, in <module>
#     print(objV2.model_dump()["foo"].url)
#           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
# AttributeError: 'tuple' object has no attribute 'url'

Python, Pydantic & OS Version
pydantic version: 2.3.0
        pydantic-core version: 2.6.3
          pydantic-core build: profile=release pgo=false
                 install path: /Users/sileht/workspace/mergify/engine5/.venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.5 (main, Aug 24 2023, 15:09:45) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
                     platform: macOS-14.0-arm64-arm-64bit
     optional deps. installed: ['typing-extensions']

This is the intended behavior. The result of model_dump() should be "plain python" objects that more or less can be passed to json.dumps. For example BaseModel instances get converted to a Python dict. I'd consider this a bug in v1.


Update dependencies

Maybe it's a good thing to also update the dependencies in the /example/pubspec.yaml app folder.

Maybe it's a good thing to also update the dependencies in the /example/pubspec.yaml app folder.
Maybe also address this:
This app is using a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to migrate this app to the V2 embedding.
Take a look at the docs for migrating an app: https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects
The plugin `gallery_saver` requires your app to be migrated to the Android embedding v2. Follow the steps on the migration doc above and re-run

I think that's fixed by #196

Any updates on this?

As @Zazo032 mentioned, I created this PR #196
where I updated plugin dependencies, specifically http package and migrated Android to use  flutterEmbedding V2.


Add esy

This PR reduce the gaps between native/rescript by allowing native user to use the same syntax. It is very low overhead. Ideally you have to bump the version number on update. or because this project is "private": true we will have to consume with github hash so version number is useless and no overhead.
I can't expose benchmarkings since it use macos specific code.
Since rescript copy rescript files during his compilation process I haven't set up esy release that allow to install only the rescript binary with npm.

Got stale and we've decided to go to a different direction as you probably know, sorry.
Thanks for the contribution though


[BEAM-119] Integrate DisplayData into DataflowPipelineRunner

Translate registered display data into the job steps used by DataflowPipelineRunner.

R: @bjchambers @kennknowles

LGTM.
@kennknowles in your court

LGTM


RBAC Network filter Stat Naming

When using the RBAC network filter, the following statistics are output:
{prefix}rbac.allowed: 0
{prefix}rbac.denied: 0
{prefix}rbac.shadow_allowed: 2
{prefix}rbac.shadow_denied: 2

Where {prefix} is the stat_prefix set here: https://www.envoyproxy.io/docs/envoy/v1.14.2/api-v2/config/filter/network/rbac/v2/rbac.proto#config-filter-network-rbac-v2-rbac
This formatting does not seem follow the formatting of other statics e.g. cluster.*, htttp.*, listener.*; this can be problematic for existing metric integrations e.g. DataDog. Would it be possible to align the RBAC network filter stat reporting with other Envoy metrics? e.g. rbac.{stat_prefix}.*

cc @kyessenov @yangminzhu


imp: add auto retry for 'invalidchunklength' and 'cancelled' failures

It would be nice if we could add an automatic job re-run trigger for the following types of GHA failures:
requests.exceptions.ChunkedEncodingError: ("Connection broken: InvalidChunkLength(got length b'', 0 bytes read)", InvalidChunkLength(got length b'', 0 bytes read))

CondaHTTPError: HTTP 530 CONNECTION FAILED for url <https://conda.anaconda.org/conda-forge/osx-64/cryptography-39.0.0-py38ha6c3189_0.conda>
Elapsed: 00:09.195701
CF-RAY: 79d1a4b35f567ecb-LAX

An HTTP error occurred when trying to retrieve this URL.
HTTP errors are often intermittent, and a simple retry will get you on your way.

discussed w/Evan, we're going to hold off on doing this - juice isn't worth the squeeze with the new CI.


Replace requestId with requestID

In order to make gofmt happy

I run it locally with syntastic, yes. We can add it to travis, but there are currently so many failures that we'd be hours of work to fix them.

Ah ok, I thought this was the only one 😂

It's the only one in that package 😉


Error:  different prefix: "/"

Hi there !
Thanks for this plugin.
At the moment, I am configuring multi language for our Jekyll based project, so I cloned this example repo and tried to launch on my windows local.
After bundle install, when I run bundle exec jekyll serve, it says the following errors.
$ bundle exec jekyll serve Configuration file: E:/projects/jekyll/jekyll-language-plugin/example/_config.yml             Source: E:/projects/jekyll/jekyll-language-plugin/example        Destination: E:/projects/jekyll/jekyll-language-plugin/example/_site  Incremental build: disabled. Enable with --incremental       Generating... jekyll 3.2.0 | Error:  different prefix: "/" and "E:/projects/jekyll/jekyll-language-plugin/example"
I would be appreciated for any help !
Thanks.

Try the current version of Jekyll.


Please add support for Address Information

I need to be able to access the user's address, and Amazon only added support for this recently in April 2017. https://developer.amazon.com/public/solutions/alexa/alexa-skills-kit/docs/device-address-api
Namely, I need a way to access both device ID and consent Token in a launch request.
Please let me know if there's a way to do this because I can't find any such implementation in the project.
As stated, I need access, so please let me know if there is no implementation and no plans for implementation so I can migrate libraries.
Thank you

Hi @EgMan, I plan to add support for the device address API.

That's great to hear.  Do you have an idea about the timeline of when this could be available? Or are you too busy to be able to say? Thanks

Just for note, you can easily derive your own class from Card like below:
`public class AskForPermissionsConsentCard : Card
{
public override string Type {
get { return "AskForPermissionsConsent"; }
}
    public virtual IEnumerable<string> Permissions {
        get;
        set;
    }
}`

and provide it in the response.

Should be fixed by PR #53


How to test erasing concept of NSFW finetuned models?

Thank you for your interesting papers. I'm really impressed by your research.
I want to test the generation result of NSFW-nudity prompts with the provided finetuned model weights. But, when I sample the images with the prompts 'nudity', the model generate the nude images!
How can I generate images which 'nudity' is erased?

Thank you! I am not sure about your inference setup. Could you try and use our generation code provided in the eval-scripts folder? https://github.com/rohitgandikota/erasing/blob/main/eval-scripts/generate-images.py
If this still persists, please do let us know

closing this issue because of inactivity. Feel free to open it if the issue still persists


Why neuralangelo trains and convergence much slower thant neus?

As mentioned in the title, I do these experiment in mipnerf360 bicycle. I know that the numerical gradient will slow down the training, but why the convergence is slow too?
here is the cmd i run for neuralangelo and neus. Limited by VRAM, I set --pipeline.model.sdf-field.hash-features-per-level 2  --pipeline.model.sdf-field.log2-hashmap-size 19 for neuralangelo and for fair comparision, the neus is set the same.
# neuralangelo
ns-train neuralangelo --pipeline.model.sdf-field.use-grid-feature True \
--pipeline.model.sdf-field.hidden-dim 256 \
--pipeline.model.sdf-field.num-layers 2 \
--pipeline.model.sdf-field.num-layers-color 2 \
--pipeline.model.sdf-field.use-appearance-embedding False \
--pipeline.model.sdf-field.geometric-init True \
--pipeline.model.sdf-field.inside-outside False \
--pipeline.model.sdf-field.bias 0.5 \
--pipeline.model.sdf-field.beta-init 0.3 \
--pipeline.model.sdf-field.hash-features-per-level 2 \
--pipeline.model.sdf-field.log2-hashmap-size 19 \
--trainer.steps-per-eval-image 1000 \
--pipeline.datamanager.train-num-rays-per-batch 1024 \
--trainer.steps_per_save 10000 --trainer.max_num_iterations 50001 \
--pipeline.model.background-model mlp \
--vis wandb --experiment-name debug mipnerf360-data \
--data data/nerfstudio-data-mipnerf360/bicycle

# neus
ns-train neus --pipeline.model.sdf-field.use-grid-feature True \
--pipeline.model.sdf-field.hidden-dim 256 \
--pipeline.model.sdf-field.num-layers 2 \
--pipeline.model.sdf-field.num-layers-color 2 \
--pipeline.model.sdf-field.use-appearance-embedding False \
--pipeline.model.sdf-field.geometric-init True \
--pipeline.model.sdf-field.inside-outside False \
--pipeline.model.sdf-field.bias 0.5 \
--pipeline.model.sdf-field.beta-init 0.3 \
--pipeline.model.sdf-field.hash-features-per-level 2 \
--pipeline.model.sdf-field.log2-hashmap-size 19 \
--trainer.steps-per-eval-image 1000 \
--pipeline.datamanager.train-num-rays-per-batch 1024 \
--trainer.steps_per_save 10000 --trainer.max_num_iterations 50001 \
--pipeline.model.background-model mlp \
--vis wandb --experiment-name debug \
--machine.num-gpus 2 mipnerf360-data \
--data data/nerfstudio-data-mipnerf360/bicycle

The result of neus in 14k

The result of neuralangelo in 14k

The result of neuralangelo in 26k maybe comparable, but not so good enough

Hi, that's because neuralangelo use a coarse to fine strategy to gradually activate higher level of feature grids so that only the coarse grids are used at the beginning of training.
Btw, I suggest using bakedangelo since it is much faster and has better background modelling.

Thank you for reminding! I ' ll try your advice and thank you for your reply!

@niujinshuchong  Sorry for bothering again. I tried bakedangelo as you advised, and i set the scene_scale in mipnerf 360 to make it learn the background mesh. And it only trained for 180k iters because of the the instability of my machine.
ns-train bakedangelo --pipeline.model.sdf-field.use-grid-feature True \
--pipeline.model.sdf-field.hidden-dim 256 \
--pipeline.model.sdf-field.num-layers 2 \
--pipeline.model.sdf-field.num-layers-color 2 \
--pipeline.model.sdf-field.use-appearance-embedding True \
--pipeline.model.sdf-field.geometric-init True \
--pipeline.model.sdf-field.inside-outside False \
--pipeline.model.sdf-field.bias 0.5 \
--pipeline.model.sdf-field.beta-init 0.1 \
--pipeline.model.sdf-field.hash-features-per-level 2 \
--pipeline.model.sdf-field.log2-hashmap-size 19 \
--pipeline.model.level-init 4 \
--trainer.steps-per-eval-image 1000 \
--pipeline.datamanager.train-num-rays-per-batch 2048 \
--trainer.steps_per_save 10000 --trainer.max_num_iterations 500001 \
--pipeline.model.background-model grid \
--vis wandb --experiment-name bakedangelo-bicycle \
--machine.num-gpus 2 mipnerf360-data \
--data data/nerfstudio-data-mipnerf360/bicycle

for mesh extraction, i set --bounding-box-min -2.0 -2.0 -2.0 --bounding-box-max 2.0 2.0 2.0 --resolution 4096 --simplify-mesh True
I notice that the forground is much better. Shown in the figure below, the wire for bicycle wheels can be reconstructed which is difficult for others methods.

however, the background mesh is still miss, even though i set --bounding-box-min -4.0 -4.0 -4.0 --bounding-box-max 4.0 4.0 4.0 while the background tree and bush appears in the rendered depth map.
I am wondering why and how i can get the forground and background mesh at the same time. Thank you!

Also the ground reconstructed mesh has large voids, perhaps because of the network capacity？

You could try with --pipeline.model.background-color white.


Crash Report 144079836

Link:
https://wowsims.github.io/wotlk/mage/#eJztUj1IW1EUft/J43l7tXiNHW5DFH2TCMLNewR0yqtTBoeAi6NFdHIQt3YKRRA72GJLodHB0cVS2iVVDEXsEIookTpI/IsODtJWXLLZ814mNwfd+g2Xc77zne9c7rkyKagfWSxjq30F+Az6DuwCJ6ALYFAKaPQgwAjiZJDYs1/ByU0+fzE+LaBiia+QrermC+nLDtf5sUmlIklb5b+RbFFn66RrHW7sirkm9Yuzd52srnFQZ/VyOVJLtcDEUtyFJR+rY47fJ137oEgWW+ysketwHApb1SkXZ7mTs+vI81OJ9IckD5zbCMe+KYXn751w+OJP7qwWqRoJL9dIz3dKi8tVthQCpjlnj6L3LVn/cQsvHV4sTWEwLoWyg5ask6O4lbAF6bHX6PV8Y0w65fspY3zfpNLG9zj20ibte33G7zMe0yxK1aDF4jGpP9ukb7ap67xCPScVMvUj/ln2XzQVYB3AKpDlHtKjaHZ+Imhr3OJJ8LTwMUQ5M9BgdjPdExH2M94qdQ1r8eDLmwnuwyVf3roXn7ug8VZDh8GzRl7PZDGKf/p5dlw=
RNG Seed:<PHONE_NUMBER>
runtime error: invalid memory address or nil pointer dereference
Stack Trace:
goroutine 11 [running]:
runtime/debug.Stack()
/opt/hostedtoolcache/go/1.18.7/x64/src/runtime/debug/stack.go:24 +0x6
github.com/wowsims/wotlk/sim/core.runSim.func1()
/home/runner/work/wotlk/wotlk/sim/core/sim.go:59 +0x10
panic({0xdf020, 0x1122690})
/opt/hostedtoolcache/go/1.18.7/x64/src/runtime/panic.go:838 +0x29
github.com/wowsims/wotlk/sim/core.(*Aura).Activate(0x0, 0x1ca53f0)
/home/runner/work/wotlk/wotlk/sim/core/aura.go:494 +0x2
github.com/wowsims/wotlk/sim/mage.(*Mage).registerMirrorImageCD.func1(0x1ca53f0, 0x14ca000, 0x14dc400)
/home/runner/work/wotlk/wotlk/sim/mage/mirror_image.go:36 +0x8
github.com/wowsims/wotlk/sim/core.(*Spell).applyEffects(0x14dc400, 0x1ca53f0, 0x14ca000)
/home/runner/work/wotlk/wotlk/sim/core/spell.go:423 +0x23
github.com/wowsims/wotlk/sim/core.(*Spell).makeCastFuncWait.func1(0x1ca53f0, 0x14ca000)
/home/runner/work/wotlk

Sorry about that, fixed by #1859

Thank you

From: James Tanner @.>
Sent: Sunday, November 20, 2022 10:19 AM
To: wowsims/wotlk @.>
Cc: rynoman14 @.>; Author @.>
Subject: Re: [wowsims/wotlk] Crash Report 144079836 (Issue #1794)
Closed #1794https://github.com/wowsims/wotlk/issues/1794 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/wowsims/wotlk/issues/1794#event-7851956639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4JJTHUTIV6FGSZFJLIVRDTWJJTURANCNFSM6AAAAAASFX5XMI.
You are receiving this because you authored the thread.Message ID: @.***>


Fix signature verification on fork boundaries

fixes #6951
Documentation

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

Changelog

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

Makes sense to me, essentially the state was not updated because of the missing blocks in the beginning of the fork?

yes.. I think there were some inconsistency during signature verification. I think that ForkInfo should be always be derived from the slot taken from the object we are validating and not from our head\state

Looks great!
How did we pass https://github.com/ConsenSys/teku/blob/06efe1053107e5f09030dc5d2fc1ef6e5ed47623/ethereum/statetransition/src/integration-test/java/tech/pegasys/teku/statetransition/attestation/AttestationManagerIntegrationTest.java#L183 before this?
So, SignedContributionAndProofValidator is tested for changes, it would be good to test AggregateAttestationValidator changes too


Fix Ruby 3 compatibility issues

Closes #79
Previously, anyone using the airrecord gem, couldn't upgrade their codebase to ruby 3 without errors. This adapts the Faraday Middleware (FaradayRateLimiter) initializer to just use the double splat, and updated some of the table code and tests to only use kwargs, instead of a mix. I didn't modify the travis.yml file, but I can if needed.

Was surprised to see all the tests change:
-    tea = Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"])
+    tea = Tea.new(fields: {"Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]})

Is that new syntax required? Is this a big breaking change?

Was surprised to see all the tests change:
-    tea = Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"])
+    tea = Tea.new(fields: {"Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]})

Is that new syntax required? Is this a big breaking change?

I can rework it but the table objects (tea, etc) have mixed keyword args and normal args and that’s not allowed so I converted them to just use kwargs. If you want, I can send them in as double splats as well. The tests failed in ruby 3, so one way or another they’ll have to change but the question is which approach you guys prefer I take.

I can rework it but the table objects (tea, etc) have mixed keyword args and normal args and that’s not allowed so I converted them to just use kwargs. If you want, I can send them in as double splats as well. The tests failed in ruby 3, so one way or another they’ll have to change but the question is which approach you guys prefer I take.

Yeah I'm not opposed to this necessarily -- I'm just trying to figure out if this is an "all your existing code will break, need to do a migration" situation or if we can retain the same interface.

Yeah I'm not opposed to this necessarily -- I'm just trying to figure out if this is an "all your existing code will break, need to do a migration" situation or if we can retain the same interface.

For anyone else digging, this is the relevant change in ruby 3: https://www.ruby-lang.org/en/news/2019/12/12/separation-of-positional-and-keyword-arguments-in-ruby-3-0/

For anyone else digging, this is the relevant change in ruby 3: https://www.ruby-lang.org/en/news/2019/12/12/separation-of-positional-and-keyword-arguments-in-ruby-3-0/

@Meekohi Right... either we move people to forcing them to send some of their params in the new initializer in a hash, or we force them to add fields:
I'm open to other suggestions and will modify as necessary.

@Meekohi Right... either we move people to forcing them to send some of their params in the new initializer in a hash, or we force them to add fields:
I'm open to other suggestions and will modify as necessary.

Thanks, I understand this better now. I'll let others chime in, but I do think we should stick to the ActiveRecord conventions where possible, so
Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"])
should do what you expect, and not need the explicit {}.
Proposal: I'm wondering what if we just make a separate option with one argument  def initialize(fields) that handles the "normal flow" I imagine 90% of people are using and all the docs show, and then we update the id/created_at syntax to require the explicit {}:
Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]) # ok
Tea.new({"Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]}) # ok
Tea.new({"Name" => "Dong Ding", "Brews" => ["rec2", "rec1"])}, id: 'id123', created_at: Time.now) # ok
Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]), id: 'id123', created_at: Time.now) # Wrong Number of Arguments

This still causes a breaking change for anyone using the second syntax, but I don't suspect that is super common probably?
P.S.
I also thought some about if we could support ruby 3 without any breaking change in the interface... I guess you could do something like keep the 3 argument version and include a catch-all version:
def initialize(obj)
      @id = obj.id
      self.created_at = obj.created_at
      self.fields = obj.reject{|k,v| k.include? "created_at", "id"}
    end

but this will break if you have a field named created_at in your Airtable table, which seems like it might happen with some frequency.

Thanks, I understand this better now. I'll let others chime in, but I do think we should stick to the ActiveRecord conventions where possible, so
Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"])
should do what you expect, and not need the explicit {}.
Proposal: I'm wondering what if we just make a separate option with one argument  def initialize(fields) that handles the "normal flow" I imagine 90% of people are using and all the docs show, and then we update the id/created_at syntax to require the explicit {}:
Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]) # ok
Tea.new({"Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]}) # ok
Tea.new({"Name" => "Dong Ding", "Brews" => ["rec2", "rec1"])}, id: 'id123', created_at: Time.now) # ok
Tea.new("Name" => "Dong Ding", "Brews" => ["rec2", "rec1"]), id: 'id123', created_at: Time.now) # Wrong Number of Arguments

This still causes a breaking change for anyone using the second syntax, but I don't suspect that is super common probably?
P.S.
I also thought some about if we could support ruby 3 without any breaking change in the interface... I guess you could do something like keep the 3 argument version and include a catch-all version:
def initialize(obj)
      @id = obj.id
      self.created_at = obj.created_at
      self.fields = obj.reject{|k,v| k.include? "created_at", "id"}
    end

but this will break if you have a field named created_at in your Airtable table, which seems like it might happen with some frequency.

This still causes a breaking change for anyone using the second syntax, but I don't suspect that is super common probably?

Agreed, I don't think that's common; and proper documentation around continuing to support that functionality would be trivial.

but this will break if you have a field named created_at in your Airtable table, which seems like it might happen with some frequency.

We actually do use Airtable like that so that would break some of our stuff ... and I'm not sure if I could guess as to how common that is. 🤷
I'm fine with tackling/providing support/discussing whatever you guys feel is best.

Wow, I haven't dug much into Ruby 3 -- that change is going to be a nightmare for upgrading code-bases. 😱 Either way, we don't have to make upgrading Airrecord painful. It's possible to remain fully backwards compatible. I went ahead and did this in master, releasing it as a new patch version 1.0.6 which you'll find now on Rubygems.
Thanks a lot for the discussion in here and the relevant links, made it quick to fix this.
https://github.com/sirupsen/airrecord/commit/646869268f22d28070e186b49a2135bf8f72fc39


core-image-ros-world: do_rootfs fails with run-postinsts returned -11

Since https://github.com/openembedded/openembedded-core/commit/4c2af72f51a7bf187615fc0b3a229d25c3e191e9 ("python3: Improve logging capabilities for do_create_manifest"), bitbake core-image-ros-world -k fails on my local machine with:
NOTE: Installing the following packages: packagegroup-core-boot packagegroup-ros-world run-postinsts
NOTE: /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/recipe-sysroot-native/usr/bin/opkg --volatile-cache -f /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/opkg.conf -t /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/temp/ipktemp/ -o /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/rootfs  --force_postinstall --prefer-arch-to-version   install packagegroup-core-boot packagegroup-ros-world run-postinsts
ERROR: Unable to install packages. Command '/home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/recipe-sysroot-native/usr/bin/opkg --volatile-cache -f /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/opkg.conf -t /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/temp/ipktemp/ -o /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/rootfs  --force_postinstall --prefer-arch-to-version   install packagegroup-core-boot packagegroup-ros-world run-postinsts' returned -11:
[...]
ERROR: core-image-ros-world-1.0-r0 do_rootfs: Function failed: do_rootfs
ERROR: Logfile of failure stored in: /home/lukas/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/qemux86-oe-linux/core-image-ros-world/1.0-r0/temp/log.do_rootfs.20376
ERROR: Task (/home/lukas/repositories/github.com/bmwcarit/meta-ros/recipes-core/images/core-image-ros-world.bb:do_rootfs) failed with exit code '1'

I do not understand how the failure is caused by the commit above, but this was what git bisect identified and by re-checking the commit before and the first bad commit multiple times, I can come to no other conclusion at the moment that this commit is the first bad commit.
After an update of openembedded-core to https://github.com/openembedded/openembedded-core/commit/c1573cb7faeb296fe7077a60d02443d5ed5bded0 ("nativesdk-glibc: Split glibc and libcrypt to use libxcrypt instead"), the error does not appear; so it probably just was some short-time glitch in openembedded-core.
I will not further investigate this error for now.

No longer occurs in the [thud-draft] branch. @bulwahn < Can this issue be closed?

"Silence implies consent" (see http://wiki.c2.com/?SilenceImpliesConsent).


Confused about version specifications

I don't know exactly how to interpret this message, resulting from a dep ensure:
 v2.1.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint ^2.1.1 from project bitbucket.org/team/proj.

projdep is being used directly by proj; there are no indirect dependencies on projdep.

Is it because of the semantic versioning syntax (all of my bookmarks start with v)?
Is it because I'm using Mercurial bookmarks instead of tags?
Am I specifying constraints incorrectly?
I've edited Gopkg.toml and added version = "2.1.1", and I get the same behavior.
I've deleted Gopkg.lock and run ensure
In no case is dep updating the dependency to the most recent version (v2.1.1).  It's stuck at v2.1.0 -- I've checked a file in the dep project under vendor.

1097)team/proj % dep ensure<EMAIL_ADDRESS>1098)team/proj % dep ensure<EMAIL_ADDRESS>ensure Solve(): No versions of bitbucket.org/team/projdep met constraints:
        default: Could not introduce bitbucket.org/team/projdep@default, as it is not allowed by constraint ~2.1.1 from project bitbucket.org/team/proj.
        v2.1.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint ~2.1.1 from project bitbucket.org/team/proj.
        v2.1.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint ~2.1.1 from project bitbucket.org/team/proj.
1099)team/proj % dep ensure<EMAIL_ADDRESS>1100)team/proj % dep ensure<EMAIL_ADDRESS>ensure Solve(): No versions of bitbucket.org/team/projdep met constraints:
        default: Could not introduce bitbucket.org/team/projdep@default, as it is not allowed by constraint ~2.1.1 from project bitbucket.org/team/proj.
        v2.1.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint ~2.1.1 from project bitbucket.org/team/proj.
        v2.1.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint ~2.1.1 from project bitbucket.org/team/proj.

I'll answer my own question: I think dep doesn't know about Mercurial bookmarks.  What eventually worked to get the newest version of my dependency was deleting Godep.* and vendor, and running dep init again.
If somebody confirms that dep doesn't support Mercurial bookmarks, I'll close tihs.

@serussell I think it should work if you constraint on branch = "v2.1.1" instead of version.
I don't know hg as well as I might like, but when I wrote the implementation, this description of bookmarks suggested that they bear more resemblance to what dep defines to be a branch than a tag - it is very much expected that bookmarks should move. The taxonomy we follow is this:

Revisions are immutable identifiers, and by construction cannot change the underlying snapshot of code to which they refer
Tags are mutable identifiers, and while they may change the snapshot to which they refer, it is generally considered poor practice for them to do so
Branches are mutable identifiers, and it is fully expected and intended that they will change the snapshot to which they refer

While it is certainly possible, as is the case here, that a branch name could be valid semver, we still don't count it as a "semver version" in the system, and only semver versions can be matched by the range constraint you specified. This is all about making "movement" of these versions consistent - whereas branches are designed to move via their own internal mechanisms,
we restrict semver to tags because they're not supposed to move individually. Rather, the user achieves "movement" through semver by specifying a range, and selecting from one of a set of versions. We want to keep these two movement models separate.
This is definitely something worth documenting, though. We also need to improve the output of these messages to make clear that there's a type mismatch.

this description of bookmarks suggested that they bear more resemblance to what dep defines to be a branch than a tag - it is very much expected that bookmarks should move
I think you're right about that.  However:

There can be only one active bookmark on a branch at a time, and only active bookmarks move.  Inactive bookmarks must be manually moved, often with a --force keyword.
A newly set bookmark becomes the active bookmark, and the previous bookmark is set inactive.  This means:

➜  ~ cd foo
➜  foo hg init
➜  foo echo 0 > a.txt
➜  foo hg add a.txt
➜  foo hg bookmark A
➜  foo hg ci -m 0
➜  foo echo 1 > a.txt
➜  foo hg bookmark B
➜  foo hg ci -m 1
➜  foo hg log
changeset:   1:49ef9a8a464e
bookmark:    B
tag:         tip
user:        Sean E. Russell<EMAIL_ADDRESS>date:        Thu Jun 01 17:52:47 2017 -0400
summary:     1

changeset:   0:b37d7fe3bc7a
bookmark:    A
user:        Sean E. Russell<EMAIL_ADDRESS>date:        Thu Jun 01 17:52:36 2017 -0400
summary:     0

In the above example, B will move with tip until a new bookmark is set, or B is manually marked as inactive (hg bookmark -i B)
In the above example, A will sit with that rev 0 until or unless somebody forces it to move.
The above rules mean that bookmarks are useful for tagging, and indeed Bitbucket uses bookmarks instead of tags for Mercurial repositories in their Pipelines.  In other words, if a user uses Bitbucket and wants to use Pipelines, they have to use bookmarks instead of tags.

I can sympathize with the philosophy; I would propose an alternative view: just like we don't want to be too strict with version restrictions (dep itself recommends against =), developers using bookmarks "float" them until they're ready to deliver a release candidate, at which point they mark them inactive -- fixating the bookmark and saying "done, let's move on to the next version."  It's the difference between "crap, I tagged this as 1.0.1, but it's broken, so now I have to tag my next commit as 1.0.2," and "this will be the 1.0.1 candidate; when it passes and is ready for QA, I'll fixate it."

IIIIINTERESTING. 🤔
And we're able to determine if a bookmark is marked as active or inactive, right? If so, I think it might work well to, within gps' taxonomy of these concepts, treat active bookmarks as branches, and inactive bookmarks as versions. This might make for some gotchas when a bookmark is switched from active to inactive, but...still, might be a good approach.

Hi,
I notice only now that in all of my editing, I cut out the part at the top of my reply where I said "You're absolutely right: bookmarks move."  Sorry; my reply came across more strident than I meant it to.
Yes, it is easy to tell which bookmark is active.  I haven't checked to see how you're interacting with Mercurial repositories; however:
From hg help:
    Creating or updating to a bookmark causes it to be marked as 'active'. The
    active bookmark is indicated with a '*'. When a commit is made, the active
    bookmark will advance to the new commit. A plain 'hg update' will also
    advance an active bookmark, if possible. Updating away from a bookmark
    will cause it to be deactivated.

And a couple of examples from the sample repo I created in the previous comment:
➜ hg bookmarks
   A                         0:ca2585030ab3
 * B                         1:671d37de1a07

The asterisk denotes the active bookmark.  Using Mercurial templates to get the active bookmark:
➜ hg log -r . --template "{activebookmark}\n"
B

As I mentioned, Bitbucket Pipelines uses bookmarks where you would expect them to use tags -- to trigger asset-producing builds.

I notice only now that in all of my editing, I cut out the part at the top of my reply where I said "You're absolutely right: bookmarks move." Sorry; my reply came across more strident than I meant it to.

No problem - honestly, I didn't even notice 😄

Yes, it is easy to tell which bookmark is active. I haven't checked to see how you're interacting with Mercurial repositories; however:

Cool, then this might be a good idea - though the more I think about the aforementioned gotcha, the more it concerns me.
The spot where we make these decisions is in the hg-specific implementation of source.listVersions(). Have a look, and we can discuss more about what to do.


A -p meta to the contig start search to handle small circular contigs.

When running rotary on an expanded dataset I am getting the following error in the rule search_contig_start
  GNU nano 2.9.3                                                                                STR27d/logs/circularize/search_contig_start.log

### Predict genes ###
-------------------------------------
PRODIGAL v2.6.3 [February, 2016]
Univ of Tenn / Oak Ridge National Lab
Doug Hyatt, Loren Hauser, et al.
-------------------------------------
Request:  Single Genome, Phase:  Training
Reading in the sequence(s) to train...

Error:  Sequence must be 20000 characters (only 6580 read).
(Consider running with the -p meta option or finding more contigs from the same genome.)

### Find HMM hits ###

Error: Sequence file STR27d/circularize/identify/STR27d_circular.faa is empty or misformatted

### Done. ###

This error happens when a very small contig is predicted to be circular, but the larger chromosomes are too fragmented to be circularized. Perhaps these tiny circular elements are plasmids <20K bp?.
Proposed Solution:
As the error suggests, we run prodigal in meta mode using the -p meta flag.
https://github.com/hyattpd/prodigal/wiki/Gene-Prediction-Modes
@jmtsuji Would there be any disadvantages to running prodigal in meta mode for this use case?

@jmtsuji Testing the meta flag now.

May be of useful review as well: https://github.com/hyattpd/prodigal/wiki/Advice-by-Input-Type#plasmids-phages-viruses-and-other-short-sequences


ID1 ID3 ID4

confirmed:ID1
account1:4oMmEuDL8kEpdxUXDzWtxxx16MWGcV5P8p4hVbNmKeVMQPd7yb
account2:4dt8Z8UNHZF9LpHdDYVWNBCdoMkVBGYwjgXUrFLYRX7emMGNZF

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:verified

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:verified

pending:ID4
AccountAddress:4oMmEuDL8kEpdxUXDzWtxxx16MWGcV5P8p4hVbNmKeVMQPd7yb
DeviceDetails:verified
We cannot fully verify this submission.It looks like you are trying to submit ID4, however, it seems like we cannot find the transaction between your account and Concordium Test Account. Please assist us by providing the missing credentials.
@sharppanda

pending:ID4
AccountAddress:4oMmEuDL8kEpdxUXDzWtxxx16MWGcV5P8p4hVbNmKeVMQPd7yb
DeviceDetails:verified
We cannot fully verify this submission.It looks like you are trying to submit ID4, however, it seems like we cannot find the transaction between your account and Concordium Test Account. Please assist us by providing the missing credentials.
@sharppanda

I thought it might be a transaction between any account using a QR code. So I used the QR code posted in discord by someone.
This is the transaction hash: 195904da358775986fabbc4b118ab53a2b58cf8a29e7d292a df15761Oec3c23d

I thought it might be a transaction between any account using a QR code. So I used the QR code posted in discord by someone.
This is the transaction hash: 195904da358775986fabbc4b118ab53a2b58cf8a29e7d292a df15761Oec3c23d

We are clearly state here ( https://github.com/Concordium/Testnet3-Challenges/issues/38 ) that the first part of the mission is to scan QR code from our Concordium test account here.

We are clearly state here ( https://github.com/Concordium/Testnet3-Challenges/issues/38 ) that the first part of the mission is to scan QR code from our Concordium test account here.

confirmed:ID4
AccountAddress:4oMmEuDL8kEpdxUXDzWtxxx16MWGcV5P8p4hVbNmKeVMQPd7yb
DeviceDetails:verified

confirmed:ID4
AccountAddress:4oMmEuDL8kEpdxUXDzWtxxx16MWGcV5P8p4hVbNmKeVMQPd7yb
DeviceDetails:verified


[Auth] Mark *Credential classes as Sendable

These classes have no mutable state so they should meet the semantic requirements of Sendable. In Firebase 12, we should refactor to checked  Sendable. We can't currently do so without a breaking change because of the fact that the classes are open (Sendable classes may not be open).
Resolves:

Because the APIs return the AuthCredential superclass, AuthCredential needs to be marked unchecked Sendable. All of it's subclasses need to then be marked unchecked Sendable or a warning will occur that the sendability of the superclass means the subclass should also explicitly mark itself as unchecked Sendable.
I added some API improvement ideas to the Firebase 12 gdoc.
#no-changelog

These classes have no mutable state so they should meet the semantic requirements of Sendable. In Firebase 12, we should refactor to checked  Sendable. We can't currently do so without a breaking change because of the fact that the classes are open (Sendable classes may not be open).

~Only AuthCredential and OAuthCredential, PhoneAuthCredential are open.~ Ah, all are subclasses of an open class!
LGTM.

Yes, I think one approach to consider in the future is making them structs that conform to a common protocol (that conforms to Sendable). And for ObjC compat, make an ObjC only wrapper to preserve the current API.


Move SIMD code into arch/avx2.c and arch/sse2.c

The Makefile treats these files specially: avx2.c compiles with -mavx2 and sse2.c with -msse2. This solves two problems.
First problem: The SSE2 code was actually compiling with AVX instructions because of '-mavx2'. This would cause a crash on hardware that does not support AVX or AVX2 such as an Atom CPU. (We missed this testing on the Sandy Bridge lab servers because they do support AVX even though they do not support AVX2.) Caught by Justin Cormack: https://groups.google.com/d/msg/snabb-devel/I_3dJvfl7gY/YHgHVzjMq0kJ
Second problem: The Makefile had overlapping rules for building the SIMD code and this causes warnings to be printed. That is because the default build rule applied to these files but then we also had specific rules. This is solved now because the arch/ directory is excluded from the default rules.

Confirmed that is working correctly on Atom:
selftest: checksum
no avx2
sse2: 1000/1000
selftest: ok

@eugeneia Do you think this is a real error or a spurious one?

It looks unrelated to this patch, but hard to tell what the issue is...

I'll wait for a recheck before merge just to be safe.


IIS Web App Management Version 2 does not properly apply the password of a custom account to the application pool

I am consistently having an issue with IIS Web App Management (Version 2) does not properly apply the password of a custom account to the application pool.  I am able to do this with Version 1 task without an issue.

We are unable to repro this issue. Closing this issue. Please share debug logs if you are still facing this issue.
Debug logs can be enabled by adding release variable system.debug with value true.

@Ajay-MS => I'm able to reproduce this error with a password with a $ character.

@GregoryOtt Thanks for sharing the scenario. I will try it out and will update the thread.

@rajatagrawal-dev  Can you please look into this issue

This is a known issue. Fix for this is currently in PR and will roll out in a couple of weeks. Until then you can try and escape the dollar character by changing "$" to "`$"


app:ci_gravity set to left or right cutting off current element indicator

In using app:ci_gravity="left" or "right", the current element indicator is cut off

Decided to do a
layout width and height as "wrap_content"
with
app:ci_gravity="center"
in order to get the layout I wanted.


Bump Dependencies #1451495584

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1451495584

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Included with https://github.com/wso2/product-is/pull/12783