python3Packages.fontawesomefree: init at 6.4.0

Description of changes
Adds the fontawesomefree python module.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.fontawesomefree
python310Packages.fontawesomefree.dist
python311Packages.fontawesomefree
python311Packages.fontawesomefree.dist


Official Release MUI5

This is just a question about the release date for the non-beta/optimised version of @mui-treasury/layout (so far still in 1.0.1-alpha.4). Most of the third-party MUI libraries have already been updated to support the latest version. Also, will the doc be updated too?

Amazing, thanks @siriwatknp bundle size is particularly important, hope this final release will be light weight. 😅

@siriwatknp are there any plans to update the components and style packages? I use the ParallaxSlide and DotIndicator from components, i've currently got a fork with mui v5 updates but it'd be nice if it was fully updated to move away from the old mui styles!

Is documentation is ready for this ?


Help: Add procecure to exisiting database

Hello fellow community,
I am fairly new to dbeaver so excuse me if this is a trivial issue that I should have solved myself.
I am conntected to an mssql database where I would like to add more procedures.
My issue is that I can't find a way to add more procedures to the database. Is this not implemented or have I completely missed this feature?
Thanks beforehand,
//ST

This depends on database (DBMS). Some (like Oracle, MySQL, PostgreSQL) support procedure creation in UI. For others the only way is to execute SQL script.


NamedContextFactory#createContext maybe throw a NullPointerException

in method: org.springframework.cloud.context.named.NamedContextFactory#createContext
code:
`
if (this.configurations.containsKey(name)) {
for (Class<?> configuration : this.configurations.get(name)
.getConfiguration()) {
context.register(configuration);
}
}
`
configurations.containsKey(name)    return not NULL,but  "this.configurations.get(name).getConfiguration()" maybe Null

Can you please format the code and provide a test case that actually shows that we have an issue?

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


fix: permissions for ubuntu axum server file mount

closing as duplicate of #397. will workaround by using root user on kittynode-server, but should fix long term.


[Version Bump] Refactor set release name

Hey, I just made a Pull Request!
Refactor set release name so that it uses the release line. Currently if you were to tun the version bump using main it would do the correct version bump but the branch and PR would have the details from the current next release which is confusion and not correct.
Note: added the .eslintrc.cjs file to resolve the "Parsing error: The keyword 'import' is reserved eslint" error that the script was hitting when I would try and commit my changes.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Until this is merged it block creating automate version bumps correctly. I'd like to do so for the nomad, periskop, and kafka plugins.


BXMSPROD-944 GA build-chain pull-request flow

https://issues.redhat.com/browse/BXMSPROD-944
related PRs:

https://github.com/kiegroup/kogito-apps/pull/535
https://github.com/kiegroup/kogito-cloud-operator/pull/653
https://github.com/kiegroup/kogito-examples/pull/452
https://github.com/kiegroup/kogito-images/pull/281
https://github.com/kiegroup/kogito-runtimes/pull/887

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

not needed


feat: query with geo bounding box

What's changed:

When the map application is opened(initial load), it will query data with the current map bounds
When user zoom in/out, move the map, resize the window, etc. New queries will be fired with the updated bounding box and then update the map accordingly.

Bug fixes:

fixed wrong maplibre instance passed to addTooltip() function which results in a runtime error
fixed empty onHover popup rendered on layer which has tooltip disabled

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

@ruanyl Looking really good. Thanks for making the changes. Two comments.
Can we add check box (instead of switch) inside Filters section like below to decide whether to apply this filter or not?

I still see tooltip hover when it is disabled.

Hi @VijayanB, thanks for the comments!

For adding the a checkbox to let user control how the map data should be loaded seems to me that we're exposing to many implementation details to the end user.

From the end user's point of view, when they are viewing the map, they may not interested in how the data is loaded and adding this checkbox may confuse the user. But of course, this is just my two cents.

For the empty popup, nice catch! Now I'm able to reproduce it when I disabled the tooltip but not refresh the page. And I investigated the issue, I think the reason is mouseenter event listener added by addTooltip function is not unsubscribed(map.off()) when layer config changed(tooltip config). The empty popup is actually from the old mouseenter event listener.

To fix the issue, we would need to unsubscribe the previous event listener whenever adding the new event listener when layer config changed, we do this for click popup in map_container.tsx.
It seems it's not a trivial task to refactor this part, I'd suggest to fix it in a separate task.

Hi @VijayanB, thanks for the comments!

For adding the a checkbox to let user control how the map data should be loaded seems to me that we're exposing to many implementation details to the end user.

From the end user's point of view, when they are viewing the map, they may not interested in how the data is loaded and adding this checkbox may confuse the user. But of course, this is just my two cents.

Totally understand. However, adding filter implicitly (without an option to disable ) might confuse users too. This option was already available in Coordinate map, so i assume our users are aware of that.
One use case i could think of is, let's say if  i want to view top 100 documents ( in an index irrespective of which part of map i am looking at ) i won't be able to do it unless i am at higher zoom level.

For the empty popup, nice catch! Now I'm able to reproduce it when I disabled the tooltip but not refresh the page. And I investigated the issue, I think the reason is mouseenter event listener added by addTooltip function is not unsubscribed(map.off()) when layer config changed(tooltip config). The empty popup is actually from the old mouseenter event listener.

To fix the issue, we would need to unsubscribe the previous event listener whenever adding the new event listener when layer config changed, we do this for click popup in map_container.tsx.
It seems it's not a trivial task to refactor this part, I'd suggest to fix it in a separate task.

Agree. Let's do it in separate task like your suggestion.

@VijayanB Aha, you're right, thanks for the context, that makes sense to me 👍

@ruanyl Even when i haven't enable the option to use geo filters, i see requests are triggered in network tab and these requests for some reason have old time filter ex: i updated my time filter to 15 weeks, but, i see these requests uses old time filter vale ( 15 m) .

@VijayanB Do you mean the global time filter is not applied? Is it a regression due to this PR?

lobal time filter is not

Ideally no new request should be triggered since useGeofilter is false.
When geofilter is true, time filter is not reflected.

Ideally no new request should be triggered since useGeofilter is false.
When geofilter is true, time filter is not reflected.

@VijayanB the above issue should have been fixed now

@ruanyl can you rebase with latest changes? Thanks

@junqiu-lei @VijayanB  Could you check the PR again?

@junqiu-lei @VijayanB Could you check the PR again?

Sure. Since there is a conflict, can you rebase? Sorry for keeping this PR unmerged

@VijayanB No worry, the PR is updated now :)


Everything blurry in Chrome only

I know this has been said a lot and a few work arounds have been posted,  but it seems that this is in Chrome only (so maybe it's completely browser dependend).
You can even see it on the swiper on the homepage of http://idangero.us/swiper. The texts in the blue bar on top are not sharp. In Firefox, Edge and Safari it's all fine.
So, any recent new ideas how to fix this.

roundLengths : true will fix it :)

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Query Panel ignores {{ env_var() }} jinja in profiles.yml

Expected behavior
The extension should have read the specified environment variable via the env_var function to run a query against the database specified in the profiles file
Actual behavior
Environment variable is not read and the query fails due to wrong credentials
Steps To Reproduce

Specify the database PW in profiles.yml like this:
password: "{{ env_var('DB_PASSWORD') }}"

Run a query in the query panel

Log output/Screenshots
No response
Operating System
MacOS
dbt version
1.8.1
dbt Adapter
dbt-trino
dbt Power User version
v0.39.19
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Did you follow the instructions on docs.myaltimate.com/setup/optConfig#environment-variables-setup? Please reopen the issue if not succesful and add more reproduction details.


Faulty Features (read below)

Replacing a song with a MIDI file larger than the original results in malfunctioning of the song midway through playback.
Maybe re-implementing the size of each song in the Python Patcher can resolve this temporarily.
(See Swap Meet.mid in MIDI.ZIP below)

Song Length in game seems unaffected despite altering values during patching. This results in long silent periods after the notes have ended.
(See Song of Storms.mid in MIDI.ZIP below)

Time Signature (3/4) / (4/4) are mismatched with the percussion if the original song had a different Time Signature. Example: Patching a 3/4 song into a 4/4 song will result in 4/4 percussion remaining and playing out of sync with the rest of the song.

Some songs begin before the Melody Countdown appears on screen. I believe this is due to some songs having the Tutes play before the song begins and will need to be handled on a case by case basis.

Testing Environment:
Windows 10 64-bit Home
Dolphin 5.0-14344
Wii Console [RVL-001] using Homebrew Channel & USB Loader GX
WiiMusicEditor.py - Main and Beta releases
MIDI Files Used:
MIDI.zip
PS. I am fluent in C. If you would like some help on this project that requires the use of C, I am more than willing to assist.

With the example of 2 and 3, your issues are unrelated to the editor yourself. Some songs also behave weirdly in general. Your Swap Meet MIDI also has 5 tracks, not 4. **Please make sure your problem is related to the Editor itself, not MIDIs or the Game. **


Excess precision for small floating point numbers.

The following program gives the following output:
{"value":0.59999999999999998}
This is undesireable. How can I avoid it?
int main() {
JSON_Value* json_val = json_value_init_object();
json_object_set_number(json_object(json_val), "value", 0.6);
char* json_str = json_serialize_to_string(json_val);
printf("%s\n", json_str);
json_free_serialized_string(json_str);
json_value_free(json_val);
}

json_set_float_serialization_format  was added in af848c27b467afff371aa22af9da2f3e1e15692a.

In the majority of case you would want a reasonable default "json_set_float_serialization_format" clearly solves that. That is good! But I do not understand by the default serilization format cannot be just be changed. Becasue everyone will now end up throwing a call too json_set_float_serialization_format in main() to force that reasonable default - or am I misunderstanding something?

Sorry, could you explain it more clearly?

The essence is that some numbers I want to output with just 1 decimal. Other I'd like 3 or 4. It is a per situation decision at which resolution to store each number.

Sorry I won't be able to provide this sort of flexibility.

would you accept a pullrequest containing such a feature using an extra function call fx. "json_object_set_number_prec" taking an additional argument containing the desired number of digits?

No, I don't want to add this to the API. Feel free to fork parson and add it though.

Ok. I'm a bit sad about that, especially because parson great and forking leads nowhere good.

I'd like to reopen this issue. The fix you did does not solve the problems for all normal cases.
In my application I many times want to write a number fx. 0.6. This number cannot be represented using a whole binary double precision floating point number. So I get excess decimals.
Given the fact that 0.6 is stored in a single precision float I set the float format to "%1.7g". This is all well and good and my float is serialized to be "0.6".
But, it also turns out that I in my application also serialize numbers which are in microseconds since epoch. And here the problem arises. Current epoch in microseconds is around<PHONE_NUMBER>957384. When serialized this ends up as 1.639465e+15 .... which is clearly wrong.
I see a couple of solutions that can solve this.

Allow to specify precision per. number.

Add an int64 number type. Other parsers clearly distinguishes between floating point and integer types - possibly because of this type of case.

What do you think?

Sorry, I don't see myself working on it anytime soon. 1 sounds bad, 2 was raised by multiple people so far, so maybe at some point I should give it a try, but I don't see myself working on it anytime soon (yes, reviewing someone else's code is also work, not far off from writing the code myself). So for now I would consider this a wontfix and if you really need this I still encourage you to fork parson.

Ok - your work efforts in the matter aside. If solution 2 where to be implemented - would you welcome that addition to the API?
I will do the implementation if you are willing to accept it.

As I said, reviewing code is also work, so no.

We are seriously misunderstanding each other during these discussions.
My question is:  Would you accept an addition of json_object_set_int64(...)   when you have the time to review it?

As I said, it's a wontfix, please stop creating pull requests related to this issue. I don't have time to work on this issue and please respect that.


Replace the repo contents with another independently developed implementation

Based on https://github.com/Agile86/kaitai_rust/tree/rust_basic_support_v2 (permalink: Agile86 / kaitai_rust > a47ab47). It brings the entire unchanged commit history into the repo (including commit hashes, so they're the same as in the original repos Agile86/kaitai_rust and bspeice/kaitai_rust), which reveals that the commits were authored by @bspeice, @Agile86 and @revitalyr (according to Git).

First of all, we need to solve one problem - no LICENSE file in the repo. As explained at https://choosealicense.com/no-permission/:

When you make a creative work (which includes code), the work is under exclusive copyright by default. Unless you include a license that specifies otherwise, nobody else can copy, distribute, or modify your work without being at risk of take-downs, shake-downs, or litigation. Once the work has other contributors (each a copyright holder), “nobody” starts including you.

Therefore, it is desirable to choose a license. If you're mentioned, please comment whether is it OK to release your changes under the MIT license.
@bspeice, I assume you own the code in your https://github.com/bspeice/kaitai_rust repo, so your consent with the proposed MIT license should be enough. @Agile86 and @revitalyr, please check with your employer whether they agree with releasing your work in https://github.com/Agile86/kaitai_rust/tree/rust_basic_support_v2 under MIT.

@Agile86 and @revitalyr, you have permission to release this under the MIT license. Thank you for your hard work!

I agree to publish the code under the MIT license.

I agree to publish the code under the MIT license.

@Agile86 @revitalyr Thank you for your approval!

The license is fine by me as well.

@bspeice Thanks!


Feat: Support the Snowflake Warehouse model config for dbt projects

Support for https://docs.getdbt.com/reference/resource-configs/snowflake-configs#configuring-virtual-warehouses

Is this a per model config property? Not something that can be inferred from the profile target?

@crericha yes, it is. Updated the description with a reference to relevant docs.


Saka Key does not work after update

How do I determine my current version of SakaKey?   Running Firefox 62.0.3.  When I display SakaKey info  (Tools/Addons/ SakaKey)  it shows at the top 1.25.2.  But, the last changed date (at the bottom) shows Nov. 19 2017.  I have automatic updates turned off, so I don't think I'm running 1.25.2.

I updated to latest version today.  After that, SakaKey does not work AT ALL.  I reset my configuration to the "all defaults" option.   SakaKey then worked fine.

Then I tried to revert to my old version.  By (1) closing Firefox.  (2) replacing the current "extensions" folder in my Firefox profile folder.  (3) restarting firefox.    Then SakaKey still did not work.
a.  Do I have to manually re-apply all my configuration changes to the new version?
b.  Is there somewhere to go to "properly" re-install an OLDER version of SakaKey?
I just knew I should not have updated Saka Key.  If it ain't broke, don't fix it.

my bad.    I did system restore and updated SakaKey.  Now I see that the version no.  was 1.25.2  and is now  1.25.5.  I missed something.  And I think my "quick and dirty" try to regress playing with the Extension FF folder didn't help things
Moderator:  this issue should be DELETED!!!

please delete this issue.


My branch

I apologize for the mistake. This pull request was created by mistake and is being closed to prevent any unintended changes. Please disregard.


Update Mezzanine from Bootstrap 3 to Bootstrap 5 alpha

Changes made so far

Update bootstrap css and js files

Remove jquery files

@hopechan thank you for contributing this! I see for now you are just removing the jQuery dependencies so I've left a review with some more detail on that


Failed while compiling project

Hi I have tried just compile as a standard Android Project but it fails with this exception, I do not know if I should perform some actions before of compiling. Thanks in advance.
4: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':app:configureCMakeDebug[x86_64]'.
> [CXX1405] error when building with cmake using C:\SPACECADET\app\src\main\cpp\CMakeLists.txt: Build command failed.
  Error while executing process C:\ANDROIDSDK\cmake\3.18.1\bin\cmake.exe with arguments {-HC:\SPACECADET\app\src\main\cpp -DCMAKE_SYSTEM_NAME=Android -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DCMAKE_SYSTEM_VERSION=21 -DANDROID_PLATFORM=android-21 -DANDROID_ABI=x86_64 -DCMAKE_ANDROID_ARCH_ABI=x86_64 -DANDROID_NDK=C:\ANDROIDSDK\ndk\21.4.7075529 -DCMAKE_ANDROID_NDK=C:\ANDROIDSDK\ndk\21.4.7075529 -DCMAKE_TOOLCHAIN_FILE=C:\ANDROIDSDK\ndk\21.4.7075529\build\cmake\android.toolchain.cmake -DCMAKE_MAKE_PROGRAM=C:\ANDROIDSDK\cmake\3.18.1\bin\ninja.exe -DCMAKE_LIBRARY_OUTPUT_DIRECTORY=C:\SPACECADET\app\build\intermediates\cxx\Debug\2x3f2t63\obj\x86_64 -DCMAKE_RUNTIME_OUTPUT_DIRECTORY=C:\SPACECADET\app\build\intermediates\cxx\Debug\2x3f2t63\obj\x86_64 -DCMAKE_BUILD_TYPE=Debug -BC:\SPACECADET\app\.cxx\Debug\2x3f2t63\x86_64 -GNinja}
  -- Detecting C compiler ABI info
  -- Detecting C compiler ABI info - done
  -- Check for working C compiler: C:/ANDROIDSDK/ndk/21.4.7075529/toolchains/llvm/prebuilt/windows-x86_64/bin/clang.exe - skipped
  -- Detecting C compile features
  -- Detecting C compile features - done
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: C:/ANDROIDSDK/ndk/21.4.7075529/toolchains/llvm/prebuilt/windows-x86_64/bin/clang++.exe - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- Configuring incomplete, errors occurred!
  See also "C:/SPACECADET/app/.cxx/Debug/2x3f2t63/x86_64/CMakeFiles/CMakeOutput.log".

  CMake Error at CMakeLists.txt:133 (add_subdirectory):
    The source directory

      C:/SPACECADET/app/src/main/cpp/SDL

    does not contain a CMakeLists.txt file.

  CMake Error at CMakeLists.txt:134 (add_subdirectory):
    The source directory

      C:/SPACECADET/app/src/main/cpp/SDL_mixer

    does not contain a CMakeLists.txt file.

Be sure to have correctly pulled the SDL and SDL_mixer repos in the correct directories


ALSA lib pcm.c:8315:(snd_pcm_recover) underrun occurred

Hello, Ankush. When the program runs for a period of time, this error may occur, and then the memory will increase until server crashes. Have you encountered this problem? Thx

It seems like a leak in a library used by pygame. I think I might have encountered this.
A hacky way to get around this is to run your job in "batches" -- i.e., restarting the job after saving a fixed number of samples.

Sorry, i don't understand it. what's the "samples" meaning?

samples = synthetic images generated by the engine.

In case someone is still getting the same errors, I found this solution and after several runs the error did not appear anymore.


Revert "Yank obsolete rules_go and gazelle versions"

Reverts bazelbuild/bazel-central-registry#725
This is breaking bazel's CI: https://buildkite.com/bazel/bazel-bazel-github-presubmit/builds/16408 and all other builds on master since this landed have failed due to a<EMAIL_ADDRESS>-><EMAIL_ADDRESS>chain. The rules_go version may be fluid enough to change for grpc, but since CI guards against this with "Existing modules should not be changed" checks, bazel CI will be broken until a new version of grpc is released to BCR and bazel is made to depend upon it, for which there may not be an upstream version that is compatible without further changes to bazel core.

Since Bzlmod uses Minimum Version Selection, it should suffice to temporarily add a bazel_dep on rules_go in a non-yanked version to Bazel's MODULE.bazel file.
After CI is green again, we can add a new, BCR-only version of grpc (with a .bcr.1 version suffix) that depends on a new version of rules_go, update the grpc pin in Bazel and remove the direct rules_go dep.

Cc @meteorcloudy

rules_go is similarly afflicted with gazelle at the recommended 0.38.1. Trying for 0.39.1 which moves the bar on gazelle to 0.30.0

bazel CI green for bzlmod builds with https://github.com/bazelbuild/bazel/pull/18828<EMAIL_ADDRESS>

Thanks for the report and PR!
Add discussed above, let's fix this forward with https://github.com/bazelbuild/bazel-central-registry/pull/734 and your Bazel PR.


[Bug]: fail to reset the input stream whose size > 2GB

@Yu-Shawn thanks for reaching out.
the method in the sdk-v1 accept an int parameter

so, please reduce the workers and try again, thank you.
Notice:
Because of Announcing end-of-support for AWS SDK for Java v1.x effective December 31, 2025, so the next milestone version of cosbench-sineio (v<IP_ADDRESS> will be released at 2024.8.31) will be changed as follows:

sio renamed to siov1, and will not be maintained.
siov2 renamed to sio(use aws-sdk-java-v2), as a LTS Storage.


flycheck should use the output of lsp-mode

Flycheck shouldn't run an additional process, but use rls output somehow. I think flymake works that way, but I didn't take a look at it.

On this subject, I can't seem to gather the errors at all when using rust-analyzer as the LSP server. Do you know if that's a limitation from the server, lsp-mode, or something we can do in rustic ?

This shouldn't make any difference since the flycheck checkers run cargo without lsp-mode. That's what it used to be. I have to take a look at this.

It seems lsp-ui now adds its own flycheck checker and you can view the errors found by rls with lsp-ui-flycheck-list. But flycheck-list-errors doesn't seem to access these sources.
Can you tell me your value for lsp-rust-clippy-preference and check both error listing flycheck commands with rls und rust-analyzer.
For rust-analyzer clippy isn't configured yet.

lsp-mode now comes with a generic flycheck checker.


Improve Memory Footprint of HNSW Connections

Changes
TODO
Status
Not ready, currently failing the chaos pipeline
Evaluation
Deep 10M
m=64 Before

m=64 After

m=8 Before

m=8 After

SIFT 1M
m=64 Before

m=64 After

m=8 Before

m=8 After

Currently failing the chaos pipeline, so most likely there is an issue with updates/deletes that needs to be fixed.

Chaos Pipeline is now green! 🎉


Update README.md

Add AiLama to the list of community apps in Extensions & Plugins

Possible to rebase?

i updated it and opened a new pull request


[scarb-doc] Inner attributes

@Arcticae I will open a separate PR with new database for scarb-doc, merge it first into main and then rebase this one on top of main. It is due to Cairo already being bumped on main


Still being billed

Hello, I deleted my pro account a couple of months ago. However, I'm still being billed for it. I, of course can't sign in, so I created another account hoping to just link the payment with the new account. That is not working, so I was hoping we could just stop billing for the account that is no longer available. Thanks

Did you cancel the subscription? Also, what was the username you were being billed under so I can investigate?

dear Remy,
Calm down is not the  biggest thing is a oldschool cell...skubido
2016-01-31 21:57 GMT+01:00 Remy Sharp<EMAIL_ADDRESS>
Did you cancel the subscription? Also, what was the username you were
being billed under so I can investigate?
—
Reply to this email directly or view it on GitHub
https://github.com/jsbin/jsbin/issues/2692#issuecomment-177608667.

--
Tisztelettel:
Dian László
Tel.:06/703774257

I am not entirely sure if I canceled the subscription. I thought that was
what I was doing when I canceled the account. :/
The username should have been yahwehisourgod.
I created a new one under this username today.  If there is anyway to
transfer the billing to this new account, I would be ok with that.  I just
didn't want to subscribe to a new pro account when I was already being
billed.  If any of that makes sense. .
On Jan 31, 2016 2:57 PM, "Remy Sharp"<EMAIL_ADDRESS>wrote:

Did you cancel the subscription? Also, what was the username you were
being billed under so I can investigate?
—
Reply to this email directly or view it on GitHub
https://github.com/jsbin/jsbin/issues/2692#issuecomment-177608667.

I'll take a look at this in the morning and get back to you. I suspect I should be able to activate against your new account, but I want to be sure first.

Thank you so much

I've found your account active in stripe (exactly as you said, it's still billing). I've updated your account so that you've got the pro status again. If you can also check jsbin.com/upgrade to make sure you have details of your invoices - that would be perfect.

All as it should be.  Thanks for your help!
On Feb 1, 2016 3:35 AM, "Remy Sharp"<EMAIL_ADDRESS>wrote:

I've found your account active in stripe (exactly as you said, it's still
billing). I've updated your account so that you've got the pro status
again. If you can also check jsbin.com/upgrade to make sure you have
details of your invoices - that would be perfect.
—
Reply to this email directly or view it on GitHub
https://github.com/jsbin/jsbin/issues/2692#issuecomment-177875632.


Replaced python sdk usage with docker cli

Replaced python sdk usage with docker cli in test_fluent_session.py.

Please also remove the docker dependency from requirements\requirements_tests.txt.

Please also remove the docker dependency from requirements\requirements_tests.txt.

Done. Thank you.


qcfractal-server assumes pg_ctl is provided by your linux distro

Describe the bug
qcfractal-server is assuming all Linux distros distribute pg_ctl. That is not the case when using Debian as can be seen here: https://wiki.debian.org/PostgreSql#pg_ctl_replacement. This might be easy to fix by just adding pg_ctlcluster as an alternative to pg_ctl.
To Reproduce
Try to run qcfractal-server start with postgres installed on Debian and you will get:
muammar@jedi ~
  % qcfractal-server start                                               !10004
Starting a QCFractal server.

QCFractal server base folder: /home/muammar/.qca/qcfractal

>>> Examining SSL Certificates...

Autogenerated SSL certificates, clients must use 'verify=False' when connecting.

>>> Checking the PostgreSQL connection...

Could not detect a PostgreSQL from configuration options, starting a PostgreSQL server.

Could not find 'pg_ctl' in the current path. Please install PostgreSQL with 'conda install postgresql'.

Alternatively, you can install a system PostgreSQL manually, please see the following link: https://www.postgresql.org/download/

Expected behavior
qcfractal-server should start operating normally.
Additional context
This is affecting Debian GNU/Linux. I am not sure if derivatives are affected as well but chances are high.

If I read this correctly we should be able to simply switch pg_ctl with pg_ctlcluster? We might try to make this configurable, we can make a simple fix for now.

Yes, something like changing this function https://github.com/MolSSI/QCFractal/blob/5ac50f9a7aad70579d23679ee76cb291c4679737/qcfractal/postgres_harness.py#L48 to check if pg_ctlcluster is also there and use it instead of pg_ctl.

You seem to have a good handle on this and can test out your solution on Debian, would you mind making a PR?
Likely something in the init that looks like:

if shutil.which("pg_ctl"):
    self.pg_ctl = shutil.which("pg_ctl")
elif...
else:
    raise

Hey good place for a Walrus operator in Python 3.8.

You seem to have a good handle on this and can test out your solution on Debian, would you mind making a PR?

I can do that. I will prepare a PR to close this issue :).

Thank you!

@muammar Gentle reminder here, we will likely be putting out a release sometime tomorrow.

@muammar Gentle reminder here, we will likely be putting out a release sometime tomorrow.

I was working on this yesterday, and just did a commit you can see here: https://github.com/muammar/QCFractal/commit/1916e28e418de6bce2fb5cc9ba408d5ab28e6d66. This might not be the best way to solve this because in the Debian documentation it says that pg_ctlcluster has to be used. I don't have experience with postgres (I just had with MySQL and Mongo), so I have to look more into how to do this in a better way.
However, the hack above seems to work butI am running into this issue:
muammar@jedi ~
  % qcfractal-server init                                                                                !10080
Initializing QCFractal configuration.

>>> Settings found:

base_folder: /home/muammar/.qca/qcfractal
database:
  database_name: qcfractal_default
  directory: null
  host: localhost
  logfile: qcfractal_postgres.log
  own: true
  password: '**************'
  port: 5432
  username: null
fractal:
  allow_read: true
  compress_response: true
  geo_file_path: null
  heartbeat_frequency: 1800
  log_apis: false
  logfile: qcfractal_server.log
  max_active_services: 20
  name: QCFractal Server
  port: 7777
  query_limit: 1000
  security: null
  service_frequency: 60

>>> Writing settings...

>>> Setting up PostgreSQL...

Initializing the Postgresql database:
Traceback (most recent call last):
  File "/home/muammar/.local/bin//qcfractal-server", line 11, in <module>
    load_entry_point('qcfractal==0.8.0+87.g1916e28', 'console_scripts', 'qcfractal-server')()
  File "/home/muammar/.local/lib/python3.7/site-packages/qcfractal-0.8.0+87.g1916e28-py3.7.egg/qcfractal/cli/qcfractal_server.py", line 454, in main
  File "/home/muammar/.local/lib/python3.7/site-packages/qcfractal-0.8.0+87.g1916e28-py3.7.egg/qcfractal/cli/qcfractal_server.py", line 215, in server_init
  File "/home/muammar/.local/lib/python3.7/site-packages/qcfractal-0.8.0+87.g1916e28-py3.7.egg/qcfractal/postgres_harness.py", line 306, in initialize_postgres
  File "/home/muammar/.local/lib/python3.7/site-packages/qcfractal-0.8.0+87.g1916e28-py3.7.egg/qcfractal/postgres_harness.py", line 40, in _run
  File "/usr/lib/python3.7/subprocess.py", line 472, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/usr/lib/python3.7/subprocess.py", line 775, in __init__
    restore_signals, start_new_session)
  File "/usr/lib/python3.7/subprocess.py", line 1436, in _execute_child
    executable = os.fsencode(executable)
  File "/usr/lib/python3.7/os.py", line 809, in fsencode
    filename = fspath(filename)  # Does type-checking of `filename`.
TypeError: expected str, bytes or os.PathLike object, not NoneType

and
muammar@jedi ~
  % qcfractal-server start                                                                                !10078
Starting a QCFractal server.

QCFractal server base folder: /home/muammar/.qca/qcfractal

>>> Examining SSL Certificates...

Autogenerated SSL certificates, clients must use 'verify=False' when connecting.

>>> Checking the PostgreSQL connection...

Could not detect a PostgreSQL from configuration options, starting a PostgreSQL server.

Starting the database:
pg_ctl command: ['-l', '/home/muammar/.qca/qcfractal/postgres/qcfractal_postgres.log', 'start']

Could not start the PostgreSQL server. Error below:

pg_ctl: directory "/home/muammar/.qca/qcfractal/postgres" is not a database cluster directory

Do you think this is now related to something else?

I would guess that this is the patch that you posted. shutil.which will return None if nothing is found. You might try to run shutil.which("pg_ctlcluster") to see what is returned since it is a perl script (should be fine, but YMMV).

At the point where I made the change,
diff --git a/qcfractal/postgres_harness.py b/qcfractal/postgres_harness.py
index f47c400..c3eecf9 100644
--- a/qcfractal/postgres_harness.py
+++ b/qcfractal/postgres_harness.py
@@ -166,6 +166,7 @@ Alternatively, you can install a system PostgreSQL manually, please see the foll
                 break

         self.logger(f"pg_ctl command: {cmds}")
+        print(psql_cmd)
         psql_cmd = [psql_cmd, "-D", str(self.config.database_path)]
         return self._run(psql_cmd + cmds)

This prints:
/usr/lib/postgresql/11/bin/pg_ctl

I made some other changes https://github.com/muammar/QCFractal/commit/0f087e0152ec27a0711734fdb45b2df48263a6d6, and now I think is working now:
fixing permissions on existing directory /home/muammar/.qca/qcfractal/postgres ... ok
creating subdirectories ... ok
selecting default max_connections ... 100
selecting default shared_buffers ... 128MB
selecting default timezone ... America/Los_Angeles
selecting dynamic shared memory implementation ... posix
creating configuration files ... ok
running bootstrap script ... ok
performing post-bootstrap initialization ... ok
syncing data to disk ... ok

Success. You can now start the database server using:

    /usr/lib/postgresql/11/bin/pg_ctl -D /home/muammar/.qca/qcfractal/postgres -l logfile start

Starting the database:
pg_ctl command: ['-l', '/home/muammar/.qca/qcfractal/postgres/qcfractal_postgres.log', 'start']
waiting for server to start.... done
server started

PostgreSQL successfully started in a background process, current_status:

pg_ctl: server is running (PID: 26900)
/usr/lib/postgresql/11/bin/postgres "-D" "/home/muammar/.qca/qcfractal/postgres"

Building user information.

Database server successfully started!

>>> Initializing database schema...

Creating tables for database: postgresql://localhost:5432/qcfractal_default

Stamping Database with current version..

>>> Finishing up...

>>> Success! Please run `qcfractal-server start` to boot a FractalServer!

Could you test in your end, too? I will submit a PR with this.

postgres detection is completely changed in v0.50 and I think it handles this use case. If it still doesn't work correctly, please open a new issue. Thanks!


Getting this error after bulding:

my os is debian 9.
here is the error:
cp: cannot stat './bin/evilginx': No such file or directory
make: *** [Makefile:20: install] Error 1

What command gave you that error?


Scheduler pod is not restarted after Airflow DB downtime

What is your question?
I'm using PostgreSQL as a backend DB. At some point DB becomes unavailable and after some downtime it become alive again, but scheduler pod lost connection to it and is not restarted.
What can I check, any suggestion?
I am using version 2.1.2

@dktranzact I have a feeling this is related to https://github.com/airflow-helm/charts/pull/351 (which will be fixed in the next chart release, 8.5.0).
However, I am not clear why the scheduler wouldn't crash if the database becomes unavailable, can you share the logs of the scheduler pod when this happens?

@dktranzact can you confirm if version 8.5.0 of the chart resolves this issue for you?


model summary print instead of logging

Before submitting

[x] Was this discussed/approved via a Github issue?
[x] Did you read the contributor guideline?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #518. Model summary printing should be controlled by trainer argument weights_summary (full, top, None), not logging level. Using the print function should be ok in this case.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

agreed. why can’t we use logging?
if there’s some technical hurdle we can move to print for this part since the summary is the first thing that happens and logging is there to not break progress bar flow

Ok then, I must be misunderstanding how this works. From what I saw in the code, the logger is set to info level but logging.info doesn't print anything, so doesn't seem to work as intended. What if the PL user wants to use logging in their training?  I'll leave it to someone else who has more experience with this package.


remove abs vectorization, add fabs, fixes #437

Submisison Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary:
Remove vectorization on abs, add to fabs and lgamma.
Intended Effect:
Appropriate vectorization support for the math library and the Stan langauge.
How to Verify:
Unit tests.
Side Effects:
No
Reviewer Suggestions:
anyone
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

This depends on the following branch to Stan for the upstream tests to pass:  https://github.com/stan-dev/stan/pull/2120

Jenkins, retest this please.

Thanks.  Looks like it's passing everything
but integration now.

Bob

On Nov 22, 2016, at 3:04 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
Jenkins, retest this please.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Give gold and expirience for killing bosses (to the whole team)

Again, to make more organazied teamfights happen.
I know that expirienced players will go for bosses anyway. But most of the people playing custom games are casuals. So, in my opinion, you need to create additional motivation for team movement.

We have debated this. So far, the major position is that we want cores and gold/exp to be separate resources. It helps to prevent too much snowballing.


container parallel testing

build-depends: 31590
build-depends: https://github.com/dci-labs/bos2-ci-config/pull/148
TestBos2Sno: sno control-plane-parallel

Hi @ansvu could you please rebase the change to pick the fix for the DCI job failure you're facing?

Hi @ansvu could you please rebase the change to pick the fix for the DCI job failure you're facing?

Just got  back from PTO. One question, I did not do PR on gerrit to update tests.yml, this PR is failed to at generic test case and it is not even used my changes yet, right?  It failed because I did not do PR on gerrit then using test-hints to test with depend-on PR309? Thanks.

recheck

Hi @ansvu, I reviewed the PRs and created two more pipelines for the testing.
Here is the complete list of what would be nice to do:

Close this PR and open another one, creating branch redhatci:container-parallel instead of ansvu:container-parallel. That would allow me and @manurodriguez modify the commits in the PR and thus speed-up the work.
Rebase the branch to fix local/check rhel8-build
Remove build-depends description from CR 31590 to avoid circular dependencies
Run three tests:

current preflight tests on your images but with no parallelization, your code is not used: ```build-depends: 31590
build-depends: https://github.com/dci-labs/bos2-ci-config/pull/148
TestBos2Sno: sno control-plane-no-parallel

2. parallelization by batches of 1 image: ```build-depends: 31590
build-depends: https://github.com/dci-labs/bos2-ci-config/pull/148
TestBos2Sno: sno control-plane-1-parallel

parallelization by batches of 3 images: ```build-depends: 31590
build-depends: https://github.com/dci-labs/bos2-ci-config/pull/148
TestBos2Sno: sno control-plane-3-parallel

retest

retest

retest

retest

Moved to the redhatci:container-parallel


Fix the invalid regex problem. There won't be crashes due to invalid regexes now

Fixes #7 the right way.

+1 for the catch block message 😄

Haha 😁


add LPushX and RPushX

and fix a typo also add a error assert

for this issues https://github.com/flower-corp/rosedb/issues/148


Checking do not ask again doesn't save preference

Describe the bug
When moving images, a conformation to to save dates pops up, clicking do not ask again does not doesn't save the dates even though save dates was clicked
To Reproduce
Steps to reproduce the behavior:

Select Image
2.Click move to
3.Choose album
4.Check do not ask again in confirmation dialogue (leave save dates checked)
5.Click Continue

Expected behavior
The moved Images should not change  the date modified tag when save dates and do not ask again are checked together.
Screenshots

System information and logs:
Package: deckers.thibault.aves
Installer: dev.imranr.obtainium
Aves version: 1.11.3-izzy, build 12202
Flutter: stable 3.22.2
Android version: 11, API 30
Android build: RMX1971_11_F.06
Device: realme RMX1971
Support: dynamic colors=false, geocoder=true, HDR=true
Mobile services: ready
Connectivity: mobile
System locales: en_US
Storage volumes: /storage/emulated/0/
Storage grants: /storage/emulated/0/Pictures/, /storage/emulated/0/DCIM/
Error reporting: false

"Save dates" saves the current "file modified date" as a metadata date (typically In Exif, but could be XMP, depending on file format).
The "file modified date" will get modified as a side effect of moving the media, whatever you check. As a gallery app, Aves does not have full control of the file attributes.

The "file modified date" will get modified as a side effect of moving the media, whatever you check. As a gallery app, Aves does not have full control of the file attributes.

Hmm but moving without the do not ask again option works fine like the date modified doesn't change then


Download whole channel - but not all. due to what?

I try to download a channel it has 304 Videos but:
[youtube] user XXXXX: Collected 302 video ids (downloading 302 of them)
I guess two of them are not viewable in my country. Can you then PRINT the URL's so I can have a look on them on my own.

youtube-dl does not filter videos. Please provide the exact command line so we can investigate this issue.

Sorry, we need the command line (or channel name) to reproduce and resolve this issue. I'm closing the issue until either of these is posted.

On 12/15/2011 03:06 PM, Philipp Hagemeister wrote:

Sorry, we need the command line (or channel name) to reproduce and resolve this issue. I'm closing the issue until either of these is posted.

Reply to this email directly or view it on GitHub:
https://github.com/rg3/youtube-dl/issues/243#issuecomment-3161316
hi the channel name is rich2015x

I dont have the command line in history but I think it was:
youtube-dl.py -cw --rate-limit 150k -o
"%(upload_date)s-%(stitle)s.%(ext)s"  http://www.youtube.....
Greetings


ldd: command not found

Hi!
Great work with the drivers! This is exactly what I have been looking for, so keep up the good work :)
I've got one issue though,
When I run
!ldd +sqlite3/private/libsqlite3_.mex*
in the Matlab terminal I get the following response:
/bin/bash: ldd: command not found
Thus, I can't run sqlite3.make without crashing (locked by mexLock API.)
Do you have a solution to this? I've tried starting matlab by copying and pasting the preload line to the terminal but that didn't help.
 You have to use otool -L command instead of ldd on mac.
The other compile error could be related to this. Try restart Matlab first.

Thank you so much!
Now it works :)

On 12 Mar 2017, at 15:39, Kota Yamaguchi<EMAIL_ADDRESS>wrote:
@max614 https://github.com/max614 Are you using macOS? You have to use otool -L command instead of ldd on mac.
The other compile error could be related to this https://www.mathworks.com/matlabcentral/answers/315679-error-compilng-c-mex-file. Try restart Matlab first.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/kyamagu/matlab-sqlite3-driver/issues/12#issuecomment-285928095, or mute the thread https://github.com/notifications/unsubscribe-auth/AQjbgdyCI2uQH2snFKntfavJkEXfVImJks5rk6EygaJpZM4Mad5M.


Environment Configuration

Hi,
Thanks for your excellent work and codes!
I have a question about the environment configuration, I hope you could help me if you have free time.
when I run:
cd hisup/csrc/lib
make
an error occurred. (I'm sure I have cuda available)

could you give me a hand? Also, is it possible to run the training or testing code without doing this step? Thank you

Hi,
Thanks for your interest! Could you please share the information about your system?
Best


Support Ltac Debugger

Should support interactive ltac debugging provided by Set Ltac Debug.

I'm not sure whether you're following CoqPL, but I hear the new visual debugger comes down to 6 XML commands under the hood and it is intended to be supportable in other frontends.
https://coq.inria.fr/refman/practical-tools/coqide.html#coqide-debugger

I've been following the debugger progress, but I haven't seen the CoqPL talk yet. It's definitely something I'd like to support once I have the magical combination of motivation to learn how it works and time to implement it.


Audio Endpoint Implementation

Adds the implementation for the /audio endpoints on the API - transcriptions and translations.
See files for changes.


Release dapr-python v0.5.0b

Describe the proposal
This is the release plan to ship dapr-python v0.5.0b.

Package name: dapr-python

python-sdk includes actor sdk implemenation with the generated proto client so that it makes sense to rename from dapr-clients to dapr-python

Package version: 0.5.0b
Release work items.

[ ] Merge features/actor to master
[ ] Set up CI/CD
[ ] Convert README.md to README.rst which is de-facto python doc format.
[ ] Add more comments to codes which allows sphinx to auto-generate python docs.
[ ] Release dapr-python 0.5.0b

@msfussell @orizohar @yaron2 @amanbha Please review this release plan and give me any feedbacks.
cc/ @mchmarny

A couple of thoughts:

For the naming dapr-python, if this is a python package, is the suffix redundant? should we perhaps just call it dapr or dapr-sdk?
As discussed offline - since actors are not common in the Python world we need to make sure we provide good links to what is the Actor model for python devs who are new to this.

A couple of thoughts:

For the naming dapr-python, if this is a python package, is the suffix redundant? should we perhaps just call it dapr or dapr-sdk?

We can call dapr I will update it on the plan.

As discussed offline - since actors are not common in the Python world we need to make sure we provide good links to what is the Actor model for python devs who are new to this.

yeah https://github.com/dapr/python-sdk/issues/37 will add the link.

@youngbupark dapr LGTM

Merged features/actor to master now.

Every build will be published to dapr-dev. So anyone can install the latest build.
Release build will be dapr. the official build will be uploaded to this repo.


fisheye depth using dist or z-buffer

https://github.com/autonomousvision/kitti360Scripts/blob/9defe218f4d05a1c920924f38ebfa8efc27318da/kitti360scripts/helpers/project.py#L200C22-L201C1
I found that the depth preparation for perspective camera uses z-buffer however the preparation for fisheye using dist. Is there any particular reason for these inconsistant ways?

Hi, the reason is that the fisheye camera model is not a normal perspective camera model. The points are first projected to a unit sphere, see this
documentation.

Good to confirm, thx for the reference.

Hi, is there any code to project the pcd file in world frame to the cameras


Update version for asterisk, ring, shelly, sia, pollenflug adapter

New stable version for following adapters:

Asterisk
Ring
Shelly
SIA
Pollenflug

Thank you!


Auto format skills with levels

Landed in https://github.com/HowProgrammingWorks/SelfAssessment/commit/b08cf3f59e3804733eb6d5bc825b6026d83f20e4, https://github.com/HowProgrammingWorks/SelfAssessment/commit/dd9156e28f89167f5fb4fcd9b7f0d0815accba49, https://github.com/HowProgrammingWorks/SelfAssessment/commit/c03f008da01682aa0ad7e3a5536195a6fdd12327, and https://github.com/HowProgrammingWorks/SelfAssessment/commit/7e4ed044dcf17d72f0ce5acbfbd4123c81f09f24


Enable to compile HLSL to lib profiles

Is your feature request related to a problem? Please describe.
Instead of compiling the whole shader, compile fragments and link them later.
Describe the solution you'd like
Accept the input of HLSL fragments, compile to lib_6_0 profile. Then link them into one DXIL or SPIR-V before converting to other shading languages.
Describe alternatives you've considered
Concatenate HLSL strings to one, then compiling it.

Due to relocatable SPIR-V module is not supported by Dxc (https://github.com/microsoft/DirectXShaderCompiler/issues/2633), we have to postpone the SPIR-V part.


Enabling ActionController to hook into metrics

Environment
Provide at least:

Ruby Version.: ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-darwin17]
Rails Version.: 5.0.6
Semantic Logger Version.: 4.3.0
Rails Semantic Logger Version.: 4.3.1
Other Application/framework names and versions (e.g. Puma, etc.).: Puma
Rails configuration. Only need the settings related to Rails Semantic Logger and Semantic Logger.:

config.rails_semantic_logger.add_appender(metric: :statsd, url: 'udp://<IP_ADDRESS>:8125')

Full Stack Trace, if an exception is being raised. (N/A)

Expected Behavior

Is there any way to make it so the ActionController hooks into a metrics subscriber (like statsd)? It seems like the metrics subscriber only receives events when a metrics: key is provided on the logger.info or logger.error calls. I can see it working when I do something like

logger.info(metric: 'foo')
but I don't see it happening for any of the ActionController log messages even after I configured a custom formatter (which I believe happens after the events are triggered, which makes sense)

logger.info(metric: 'foo') is referred to a "metrics only" log event. By design these metrics are not output to human readable text output destinations. For example they are written to the JSON output appender, which is how we forward these metrics only events to Splunk.
All of the built-in destinations are documented: https://rocketjob.github.io/semantic_logger/appenders.html
Of course, you can create your own appender for any other destination that you want to send events/metrics to.
The most complete metrics only appender currently implemented is for Signalfx since it supports metrics as well as dimensions. It ignores all log events that do not have metrics.

I guess my question is whether or not I can take the existing log subscribers for the action controller that get monkey patched in this library and somehow hook into them and intercept the info/error to add additional information in order to trigger a metric subscriber to fire as well

Using named tags will add any additional data to the action controller log messages. For example:
 config.log_tags = {
    request_id: :request_id,
    ip:         :remote_ip,
    user:       -> request { request.cookie_jar['login'] }
  }

Rails also has a built-in mechanism to add custom data to its payload from any controller by implementing append_info_to_payload. For an example, look for Adding custom data to the Rails Completed log message: https://rocketjob.github.io/semantic_logger/rails.html


AD Cleanup: Remove array & pair-specific logic for dadd and dzero methods.

This should be a quick cleanup. We already have dadd and dzero defined in diff.meta.slang for DifferentialPairType and ArrayType.
There should not be any need to synthesize them again in slang-ir-autodiff-transpose.cpp (emitDZeroOfDiffInstType, emitDAddOfDiffInstType)

Not sure this is currently actionable; Yong & Sai to discuss offline


Improve permissions notice banners

On a number of organization settings panels, we show notification banners that describe configurable permissions settings, e.g.:

We should make the following across-the-board improvements to these banners:

[ ] Make it explicitly clear when permissions settings are configurable by rewording the banners. (No language changes for permissions that are not configurable.)

Current structure: Only organization [X roles] can [do Y] in this organization.
New structure: This organization is configured so that [X roles] can [do Y].

[ ] For users who have the relevant permissions, change the banner color from the yellow warning color to a notice color.

Note that these banners should only be show if either (a) the users is an administrator, and can therefore modify permissions settings, or (b) the user is missing permissions to perform some actions on the panel.
This issue should address the confusion described in #13759.

@zulipbot claim

@zulipbot claim

Unclaiming myself from this issue so that I can start working on other issues. I have opened the PR and would do the changes suggested in review.

@zulipbot abandon

@zulipbot  claim

@alya  kindly allow me to work on this  issue


Clarify presto-admin configuration doc

Can you add in your commit message why external hostnames or IPs are needed ?
Check the travis build failure

Looks good once you address Akshat's comment


NOAA vs AGAGE

Work out how NOAA data differs, or doesn't, from AGAGE. For example, why are there 3 timeseries in Figure 4 here, yet CSIRO's observations are part of both NOAA (https://gml.noaa.gov/dv/site/?program=ccgg) and AGAGE (https://agage.mit.edu/global-network). Is there overlap yet they are somehow different products or are these products truly independent?
Required to close

[ ] answer about how these datasets do or don't differ

Paul Krummel will have good ideas here too. I think the networks are different and use different measurements, so we probably don't need to worry about double counting too much


Fix: catch batch errors

Fixes an app immediate crash as per #3883.
Doesn't fix the root cause of the crash though. Probably related to Infura?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 9 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 36.997%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Transactions/TxList/BatchExecute.tsx
0
9
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
3849

Relevant Lines:
9474

💛  - Coveralls

@francovenica please just test if the Batch Execution still works.

Batch tx still work fine.
LGTM


Fix/helm ingress

PR purpose:
Fix the helm chart ingress definition to reference the entry service as well as update it to account for recent ingress apiVersion removals.
Fixes:
https://github.com/podtato-head/podtato-head/issues/173
Other info:
Also updated the helm chart readme to reflect changes and tweak syntax in a few areas.  Here is the test I ran to validate the ingress works now (using kind with an nginx ingress controller):
helm upgrade --install podtato-head ./ --values - <<EOF
    entry:
      serviceType: ClusterIP
    ingress:
      enabled: true
      className: nginx
      hosts:
        - host: podtato-head.127-0-0-1.nip.io
          paths:
            - path: /
              pathType: ImplementationSpecific
EOF

One should definitely merge this. Without this fix the app is not deployable anymore (at least not on Openshift) because of the outdated ingress api.

Thanks for pointing this out @MxNxPx! Apologies for my delay in reviewing. I just tested this out and LGTM.
In fact I'd never tested with proper ingress for some reason, this motivated me to get it working - on OpenShift in fact!


Beautify sponsors

Use custom CSS after bootstrap so that overriding actually works
Fix some logos
Make lead everywhere
Move YT video outside accordion

@luanyi @hao-fang I cannot reproduce the "focus on rocket chat" error you previously mentioned.


csearch sometimes fails on character classes containing only the same letter in upper and lowercase

What steps will reproduce the problem?
laptop$ cat userids.txt
dgryski
laptop$ cindex .
2012/01/24 14:48:22 index /[XXXXXX]/
2012/01/24 14:48:22 flush index
2012/01/24 14:48:22 merge 0 files + mem
2012/01/24 14:48:22 8 data bytes, 237 index bytes
2012/01/24 14:48:22 done
laptop$ csearch '[g]r'
/[XXXXX]/userids.txt:dgryski
laptop$ csearch '[Hg]r'
/[XXXXX]/userids.txt:dgryski
laptop$ csearch '[Gg]r'
laptop$
laptop$ csearch 'g[Rr]'
/[XXXXX]/userids.txt:dgryski
laptop$ csearch '[Dd]g'
laptop$ csearch '[ZDd]g'
/[XXXXX]/userids.txt:dgryski
laptop$

What is the expected output? What do you see instead?
I expect 'dgryski' to be printed, but instead depending on the regex no lines
are found.

What version of the product are you using? On what operating system?
070ef10ab799 tip.  Darwin 10.8.0

Please provide any additional information below.

Original issue reported on code.google.com by dgryski on 24 Jan 2012 at 1:59

Actually, it looks like it fails when the two-letter-character class is the
first item in the regex.

laptop$ csearch 'd[gG]r'
/[XXXXX]/userids.txt:dgryski

Original comment by dgryski on 24 Jan 2012 at 2:09

This issue was closed by revision 5cd8d184e954.

Original comment by dgryski on 2 May 2012 at 8:05

Changed state: Fixed

I did a bit of investigation last night and it looks like the problem is with
match.go:stepByte().  If we need to fold for this particular instruction, we
uppercase the character 'c', but it doesn't get reset back to the lowercase
version  when we move on to processing the next state -- the character 'c' is
still the modified uppercase version instead of the original lowercase version
that's actually in the string.

I've attached a patch to match.go to fix this, and two test cases to
regexp_test.go.

Original comment by dgryski on 27 Jan 2012 at 8:51
Attachments:

regexp_test.diff
match.diff

Issue 19 has been merged into this issue.

Original comment by dgryski on 2 May 2012 at 8:54


Could you support using a dark-mode code block theme even in light-mode themes?

Just like the webpage below.

Sure, I can do that. Do you think the other case (dark mode theme with light mode code block) is also something anyone would want?

Because svgFile should be a filename not the actual svg, and that file contains the svg ;) Here is the snipper from the RADME:
{
  "languages": [
    {
      "codeblockLanguages": ["language1", "language2"],
      "displayName": "iRule",
      "svgFile": "f5.svg",
      "format": "tcl"
    }
  ]
}


Set WKT_TYPE from Proj Version

Version 3 uses PJ_WKT2_2019 as the formatting type for WKT representations. This only available in 6.3+ and breaks at the build stage for 6.2 versions. This PR defines a macro WKT_TYPE which will use the newest available version. It also expands on the CI to test against multiple Proj versions.

@keithdoggett since it is not failing on mac, maybe we can avoid the continue-on-error there ?


Invoke Go to Definition.

It's expectable if the project is not type checked yet. If it is, then it's a bug. However I found the new Go to definition much slower than what we had in VS 2015. May it be the case that it performed some more smart cacheing?

This not only happens in the newest 4.1 branch.
In VS2015, esp. when a project is not (completely) compilable and is fairly large (100k LOC) then GoToDefinition can take up to a minute or so, and if you meanwhile continue typing, at some point you'll get a popup saying "No definition found" (not sure of the the text).
If the definition is in another project (linked as project reference), GoToDefinition sometimes works, sometimes doesn't. I just tried it again in VS2015 (solution with 12 F# projects). F12 on a method in project X works (after 20 sec), but same method used in project Y doesn't. Both X and Y are part of the same solution and linked as project reference.
I use F12 (GoToDefinition) a lot, but have learned to live with its perks. But if someone is going to have a stab at making it more resilient I'd welcome that a lot.

@abelbraaksma Yes, I've marked it as Urgency-Soon, because I believe GoToDefinition to be one of the top items we need to do very well if we want to match "experience parity" with C#/VB.  I think the perf is definitely worth investigating.
At the very least, we'll need to make it aware of other projects for vs2017 - right now, it only works within the same project, which is a regression (even if it was spotty at times in the past).

@vasily-kirichenko I noticed that after quite a long while, it would be pretty fast.  Perhaps that's the typechecking.  I hope it's not, though, because otherwise we likely would not be able to speed up what I observed.

@cartermp Using a "debug" install of those tools it is likely ~40 seconds before GTD is ready in service.fs.  Using a release build should be ~25 seconds

@dsyme I see.  IIRC I'm using a release build - I'll test it out again and see if I can get decent estimates on the time it takes.  After a while (presumably when everything is typechecked), it's basically instant.

This delay also happens on my project when it is just compiled. I notice a first-time long delay (sometimes > 60s). After that, everything in the linked project seems to be direct. Reopening the project will show red squiggles everywhere (though sometimes F12 works regardless), even though there are no errors. After rebuild and all the linked projects look "normal" again, there's still the first time delay.
After some editing, the delay sometimes comes back (but right now I can't repro that, may be Upd 3 fixed this).
First-time delay seems per-project (i.e. assembly), not per class, and irrespective whether it was just compiled. When the project is not compilable, F12 seems to work on types + members, but not on extension members.
2nd and later times are still slow compared to C# (near instant). On my system it takes about 1-2s, sometimes 2-5s. I only checked self-built projects, as I find F12 with disassembled or reflected types rather useless (albeit better than nothing).

Just a suggestion: could we fix this perhaps by using a timer callback? I.e, suppose it takes longer than 2s, or suppose the type system is still working in the background, show a message in the status bar: "Go to member not yet available, please wait until background type checking is completed.".
I think this will be frustrating, but less frustrating than thinking it doesn't work, you continue your editing work and after a minute the cursor jumps to whatever it was on when you hit F12 a minute ago. Happens to me quite often...

@abelbraaksma Could you edit your comments above to indicate if the observations are with VS2015 , VS2017RC or VS2017(master) please?   Thanks

@dsyme , done. Sorry, thought it was clear from the context. It's all VS2015, Upd 3, F# 4.0, I've edited the comments.

@abelbraaksma I think the GTD implementation has changed sufficiently that we will only be able to investigate reports that reproduce using VS2017RC (master).
It's likely this will still reproduce there  - could you check please?

@dsyme, that may take some time as I haven't downloaded it yet and I may need to set up a VM so as not to potentially taint my development environment (not sure if the VS2017RC can be installed side-by-side with VS2015 risk-free). But I'll try it out.

Closing, as this has been made much better with recent performance improvements to FCS.  It's also really a symptom of classification taking a long time in FCS, particularly for files in VisualFSharp.sln.

@dsyme, that may take some time as I haven't downloaded it yet
I have meanwhile done so and the "go to declaration" functionality has much improved in the new VS2017, it is fast and most of the time accurate (but not so with same-named DU's, but that's a new issue).


ceph-volume problem when updating from nautilus to octopus

Bug Report
What happened:
TASK [scan ceph-disk osds with ceph-volume if deploying nautilus] ********************************************************************************************************
Wednesday 13 May 2020  06:49:40 +0000 (0:00:00.144)       0:10:25.771 *********
fatal: [tosstore01]: FAILED! => changed=true
  cmd:
  - ceph-volume
  - --cluster=ceph
  - simple
  - scan
  delta: '0:00:08.616638'
  end: '2020-05-13 08:49:49.504158'
  msg: non-zero return code
  rc: 1
  start: '2020-05-13 08:49:40.887520'
  stderr: |2-
     stderr: lsblk: /var/lib/ceph/osd/ceph-86: not a block device
     stderr: Unknown device, --name=, --path=, or absolute path in /dev/ or /sys expected.
    Running command: /sbin/cryptsetup status /dev/sdc1
    Traceback (most recent call last):
      File "/usr/sbin/ceph-volume", line 11, in <module>
        load_entry_point('ceph-volume==1.0.0', 'console_scripts', 'ceph-volume')()
      File "/usr/lib/python3/dist-packages/ceph_volume/main.py", line 39, in __init__
        self.main(self.argv)
      File "/usr/lib/python3/dist-packages/ceph_volume/decorators.py", line 59, in newfunc
        return f(*a, **kw)
      File "/usr/lib/python3/dist-packages/ceph_volume/main.py", line 150, in main
        terminal.dispatch(self.mapper, subcommand_args)
      File "/usr/lib/python3/dist-packages/ceph_volume/terminal.py", line 194, in dispatch
        instance.main()
      File "/usr/lib/python3/dist-packages/ceph_volume/devices/simple/main.py", line 33, in main
        terminal.dispatch(self.mapper, self.argv)
      File "/usr/lib/python3/dist-packages/ceph_volume/terminal.py", line 194, in dispatch
        instance.main()
      File "/usr/lib/python3/dist-packages/ceph_volume/devices/simple/scan.py", line 381, in main
        self.scan(args)
      File "/usr/lib/python3/dist-packages/ceph_volume/decorators.py", line 16, in is_root
        return func(*a, **kw)
      File "/usr/lib/python3/dist-packages/ceph_volume/devices/simple/scan.py", line 263, in scan
        '--force was not used and OSD metadata file exists: %s' % json_path
    RuntimeError: --force was not used and OSD metadata file exists: /etc/ceph/osd/86-0d986519-724a-4d90-9ca1-63ad1f3d37be.json
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>

Some parts are still in Version 14.2.9 because of the problem - Mons,mgr and 20 OSDs are on the new Version
What you expected to happen:
The OSD 86 is working fine and I'm not deploying nautilus as the task label is suggesting
How to reproduce it (minimal and precise):
ansible-playbook rolling_update.yml -i ../hosts_ceph
Share your group_vars files, inventory and full ceph-ansibe log
all.yml:
upgrade_ceph_packages: True
ceph_origin: "repository"
ceph_repository: "community"
ceph_stable_release: "octopus"
public_network: "<IP_ADDRESS>/27"
cluster_network: "<IP_ADDRESS>/27"
monitor_interface: "external"
osd_objectstore: "bluestore"
osd_scenario: "lvm"
radosgw_address: <IP_ADDRESS>
radosgw_frontend_port: 80
copy_admin_key: true
devices: []
inventory:
[mons]
tosstore01
tosstore02
tosstore03
[mgrs]
tosstore01
tosstore02
tosctrl01
[osds]
tosstore01
tosstore02
tosstore03
tosstore04
[mdss]
tosstore01
tosctrl01
[rgws]
tosctrl01
Environment:

OS: Ubuntu 18.04.4 LTS
Kernel: 5.3.0-46
Docker version if applicable (e.g. docker version): no
Ansible version: ansible-playbook 2.9.7
ceph-ansible version: stable-5.0
Ceph version: 15.2.1 (Some parts are still in 14.2.9 because of the problem - 20 OSDs are on the new Version)

Which sha1 commit are you using on the stable-5.0 branch ?
Because we've added the --force option via https://github.com/ceph/ceph-ansible/commit/7cc9f93680d84503943d60b2bb950dd68a2259ed back in september.

Thanks. Yes i was using an old version of the Playbook.
I'm still a bit confused because of the "deploying nautilus" thing.. but ok its working now.


Fix callback-example.yaml

This fix adds quotes around {$request.query.callbackUrl}/data to make it valid YAML.
{ is a special character that starts a mapping, so string literals beginning with { need to be escaped.
UPD: Now that 3.0 is released, should the PR target master or OpenAPI.next?

Closing, because there's another PR with the same fix - https://github.com/OAI/OpenAPI-Specification/pull/1228.


🔧 bulma helpers to tailwind

ref #9013

is-clipped => overflow-hidden
is-shadowless => shadow-none
is-unselectable => select-none

also conflicts here


Icons not visible in Dark Mode [UI]

Current Behavior
GItHub Icons are not visible in Dark Mode, icons specifically for dark mode should be, also from an asthetic point of view their alignments are not correct
Desired Behavior
Specific icons should be used for dark mode
Screenshots/Logs

Please assign me this issue

Hi @ranjanmangla1 if anyone is not working on this you can assign this to me

@Vivek-Prajapatii I am working on it!

We don't require this change as this page is going to update with some new content.
More info: https://github.com/meshery/meshery.io/issues/1197


Better support for 'defaultProps' in JSX

This proposal aims to fix problems around React's use of defaultProps, which TypeScript has trouble understanding today.
Background
Today, React has a concept called defaultProps; these are props that an external user of a component may consider optional, but are always present in the component definition.
export class Greeting extends React.Component {
  static defaultProps = {
    name: 'stranger'
  }

  render() {
    return (

Hello, {this.props.name}

    )
  }
}

let myElement1 = <Greeting name="Daniel!"/>; //
Hello, Daniel!

let myElement2 = <Greeting />;               //
Hello, stranger

Unfortunately, TypeScript --strictNullChecks users who want to take advantage of this are in a tough spot.
They can either specify these props as optional and assert that defaulted properties are present internally,
export interface GreetingProps {
  name?: string
}

export class Greeting extends React.Component<GreetingProps> {
  static defaultProps = {
    name: 'stranger'
  }

  render() {
    // Notice the non-null assertion!
    //                           | |
    //                           | |
    //                           \ /
    //                            V
    return (

Hello, {this.props.name!.toUpperCase()}

    )
  }
}

or they can jump through levels of indirection:
export interface GreetingProps {
  name?: string
}

// Notice we are using a 'const' declaration here!
export const Greeting: React.Component<GreetingProps> = class extends React.Component<GreetingProps> {
  static defaultProps = {
    name: 'stranger'
  }

  render() {
    return (

Hello, {this.props.name!.toUpperCase()}

    )
  }
}

Proposals
Using resolution semantics of the JSX factory
Users can already emulate this behavior with React.createElement: https://tinyurl.com/y9jbptt9
// React-like API

export interface Component
 {
    props: P;
}

export interface ComponentClass
 {
    new(props: P): Component
;
    defaultProps: Defaults;
}

export interface Element
 {
    type: string | ComponentClass
;
    props: P;
}

// Some additions

type Defaultize<T, Defaults> =
    & Partial<Pick<T, Extract<keyof T, keyof Defaults>>>
    & Pick<T, Exclude<keyof T, keyof Defaults>>;

declare function createElement<Props, Defaults>(
    type: ComponentClass<Props, Defaults>,
    props: Defaultize<Props, Defaults> | null):
    Element<Props>

// Now comes your code...

class FooComponent {
    props: { hello: string, world: number };
    static defaultProps: { hello: "hello" };
}

// works: 'hello' got defaulted
createElement(FooComponent, { world: 100 });

// error: missing 'world'
createElement(FooComponent, { hello: "hello" });

In theory, just passing the right data to the JSX factory should do the work for us.
However, there are some issues:

The Defaultize type is hard to read.
TypeScript treats the object literal in createElement as an inference site, and when inference fails, users get no completions for props. It's not ideal in the general case, but the goal is to continue to give a good experience in JSX.

Add a new type in the JSX namespace

We could potentially add a new field called ComponentDefaultPropNames in JSX which would either be a union of literal types or possibly an object type whose property names describe the respective names that defaultProps could have:
namespace JSX {
    export type ComponentDefaultPropNames = "defaultProps";
}

When writing out a JSX tag for a component, JSX.ComponentDefaultPropNames is looked up on the type of the tag name to determine which properties should be considered optional.
So for example:
<TagName {...someObject}>

If TagName expects props with the type Props, TypeScript can relate typeof someObject to some combination of Props and (typeof TagName)[JSX.ComponentDefaultPropNames].
Potential issues
Under either proposal, some type is inferred from the type of the component's defaultProps property.
Since React's .d.ts files currently define defaultProps to be a Partial
, where P is the type parameter for the type of props, we need to avoid that from making all props optional.
To avoid issues here, we could say that only required properties on defaultProps satisfy a JSX element's type. So for example:
interface MyProps {
  name: string;
  age: number;
}

class MyComponent extends React.Component<MyProps> {
  render() {
    return
My name is {this.props.name} and I am {this.props.age} years old.
;
  }

  static defaultProps: { name?: string, age: number } = {
    name: "Daniel",
    age: 26,
  }
}

// error: 'name' is still considered required
let x = <MyComponent />

// works! 'age' is optional because it was definitely present in the type of 'defaultProps'.
let y = <MyComponent name="Daniel" />

namespace JSX {
export type ComponentDefaultPropNames = "defaultProps";
}

No. We're better than this now. Instead, for your consideration:
namespace JSX {
    export type LibraryManagedProps<TComponent, TProps> = /*...*/
}

which can be defined as:
namespace JSX {
    export type LibraryManagedProps<TComponent, TProps> =
        TComponent extends { defaultProps: infer D; propTypes: infer P; } ? Defaultize<TProps & InferedPropTypes
, D>
        : TComponent extends { defaultProps: infer D } ?  Defaultize<TProps, D>
        : TComponent extends { propTypes: infer P } ? TProps & InferedPropTypes

        : TProps;
}

where InferedPropTypes is a type the maps from the propTypes shapes to normal types. This could handle defaultProps and propTypes, and is a little more aligned with the current goal of attempting to remove the JSX namespace completely (as these are closer to the mechanics the factory function would need to use). TComponent is the static class or sfc, and TProps is the props taken from that component or SFC (so either the props member or first argument). Technically TProps is derivable from TComponent... but, convenience (also we do this to handle SFC overloads, which are not derivable right now and are the major stopping block from using the factory).
We shouldn't tie functionality to strings or properties anymore - we don't need to (we have generic index types now - the property references were a hack for before we had them). And "Defaultize is ugly" isn't really a great argument against it, because that exact type would need to be hardcoded into the checker to make the property-name thing work (except even uglier because it'd have no alias, so would be represented in declaration emit and quick info with no aliasing and a ton of nested conditionals), except it wouldn't be able to be recoded if/when the JSX library's functionality changes - it would hardcode the compiler to the current version and implementation of react (specifically), which should be an nongoal.

One thing I'd like to mention is that it doesn't support generic components yet, which has nothing to do with LibraryManagedAttributes, but because TypeScript doesn't support high order kinds.
So if your component has Props<T>, the generic type T would become undefined or any(I didn't find the pattern here, but I saw both in error messages) when it's passed to TProps.

typescript version 3.0.3
@types/react version 16.4.13
defaultProps don't work with HOCs  =\
HOCs
import * as React from 'react';
import { Subtract } from 'utility-types';

// services
import { TimeService } from './TimeService';

export type WithServerTimePropsType = {
  serverTime: number;
};

function withServerTime<T extends WithServerTimePropsType>(
    Child: React.ComponentType<T>,
  ): React.ComponentType<Subtract<T, WithServerTimePropsType>> {
    class WithServerTime extends React.Component<Subtract<T, WithServerTimePropsType>, {}> {
      static displayName: string = `WithServerTime(${Child.displayName || Child.name})`;

      render(): React.ReactNode {
        return <Child serverTime={timeService.serverTime} {...this.props} />;
      }
    }
    return WithServerTime;
  };

Component with defaultProps
type PropsType = {
  className?: string;
  timeFormat: TIME_FORMAT;
};

class CountDown extends React.Component<PropsType> {
  static defaultProps: PropsType = {
    timeFormat: 'hms',
  };

  render(): React.ReactNode { ... }
}

Using
const HOCCountDown = withServerTime(CountDown);

// error TS2322: Type '{}' is not assignable to type 'Pick<PropsType, "className" | "timeFormat">'.
  Property 'timeFormat' is missing in type '{}'.
<HOCCountDown />

However I indicated timeFormat is optional.

@inomdzhon You did not actually declare timeFormat as optional for your HOCCountDown; your HoC function returns React.ComponentType<Subtract<T, WithServerTimePropsType>> - that will not implicitly passthru the specific defaultProps from the input type (React.ComponentType only asserts that the defaults are bounded to the props, not what the defaults actually are). In fact, the component you return will happily override whatever defaults the input component specified with no defaults at all. Now, implementation-wise your code is ok, since the inner component's defaults will pickup what your component missed and you otherwise don't use the props, but your type annotations don't claim this in any way! So your input may have defaults, but your HoC wrapper doesn't care about that or inspect that, and your HoC wrapper doesn't claim to persist the defaults of a component it is passed (just the props)! So your types are claiming to construct a component with the same props as the input component uses, but without the same defaults. You need to derive your output defaultProps member type from the input's defaultProps member type to do that (or, alternatively, pull in the defaulted props from the input type into your component instead of the input props by using the LibraryManagedAttributes implementation from react or an equivalent your write yourself). The source of this complexity is a need to be mindful of the difference between the constructor-arguments-props-type and the component-internal-props-type for a component. The generic parameter is the component-internal type, while the public type is derived from that via react internals (represented in typespace by the LibraryManagedAttributes type constructor).
Although it's probably worth mentioning that even if you alter your component to get that right, you may need to be on nightly for it to work, since we partially fixed a related issue in intersected types a few days ago.

@weswigham any chance for a code example of how that could work? :)

@madou woh, it's work 😲...unless this error 🤔

@madou I've encountered the same issue - "solved" it by casting the passed component back to a React.ComponentType. You can see it in action here: withForm.tsx
It works, but it really feels more like an ugly workaround. I'd appreciate it if someone would find a nicer solution!

@weswigham hi) what you think about @maschino solution for "provide" defaultProps for HOC?


readBuff(T*, size_t) never actually reads

Thank you for making this nice implementation!
I tried to use it in a project and discovered that there is small issue in the implementation of readBuff(T*, size_t) — the head is used instead of tail. This always results in available being 0 and thus the read is never performed.
https://github.com/jnk0le/Ring-Buffer/blob/757b4f58c40208a6a8c1702cfc90f87c6963cff3/ringbuffer.hpp#L426
I believe the fix is too small for making a fork, but I can do it if you want.
Thanks!

Thanks for spotting this.


Responsive Images

I can't find any style for .img-responsive in Bootstrap v4. Is this intentional, and if so, does anyone know the reasoning behind it?

@aburi this was changed to .img-fluid — See #17168

Thanks @kkirsche.

Anytime


To Add ERC20 Diamond Foundry Template

this is to add a template for an erc20 foundry contract and tests for this template

@collinsadi please kindly assign me to this task to complete it

@collinsadi done with this implementation with the pull request here https://github.com/collinsadi/diamonds/pull/14


Remove doc html_root_url

The html_root_url currently links to an old version of the docs, so if you run cargo doc --no-deps on a downstream package that depends on<EMAIL_ADDRESS>intra-doc links to indexmap types will resolve to<EMAIL_ADDRESS>on docs.rs.

Oops! We should just remove this altogether, as it's no longer recommended: https://github.com/rust-lang/api-guidelines/pull/230

Okay, I've updated the PR to remove the attribute instead of updating it.


Raising error 50000 throws an exception when attempting to catch specifically

Describe the bug
Microsoft.Data.SqlClient does not seem to like the default error number of 50000 when it is raised by a T-SQL statement, and it throws an exception.
Exception message: Error number 50000 is invalid. The number must be from 13000 through<PHONE_NUMBER> and it cannot be 50000.
Stack trace:
 ---> Microsoft.Data.SqlClient.SqlException (0x80131904): Error number 50000 is invalid. The number must be from 13000 through<PHONE_NUMBER> and it cannot be 50000.
   at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, SqlCommand command, Boolean callerHasConnectionLock, Boolean asyncClose)
   at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at Microsoft.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString, Boolean isInternal, Boolean forDescribeParameterEncryption, Boolean shouldCacheForAlwaysEncrypted)
   at Microsoft.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean isAsync, Int32 timeout, Task& task, Boolean asyncWrite, Boolean inRetry, SqlDataReader ds, Boolean describeParameterEncryptionRequest)
   at Microsoft.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String method)
   at Microsoft.Data.SqlClient.SqlCommand.ExecuteNonQuery()

To reproduce
Include a complete code listing (or project/solution) that we can run to reproduce the issue.
Partial code listings, or multiple fragments of code, will slow down our response or cause us to push the issue back to you to provide code to reproduce the issue.
string sql = "[MySchema].[MyProcName]";

try
{
    using (var sqCon = new SqlConnection(_connectionString))
    {
        DateTime transactionBegin = DateTime.UtcNow;

        sqCon.Open();
        using (var sqCmd = sqCon.CreateCommand())
        {
            sqCmd.CommandType = CommandType.StoredProcedure;
            sqCmd.CommandText = sql;

            sqCmd.Parameters.AddWithValue("@SomeParameter", someId);

            sqCmd.ExecuteNonQuery();
        }

        sqCon.Close();

        DateTime transactionEnd = DateTime.UtcNow;
        TimeSpan tran = transactionEnd - transactionBegin;

        _log.Debug($"{{\"result\": \"success\", \"elapsed\": \"{Math.Round(tran.TotalMilliseconds, 2)}\", \"data\": \"{someId}\"}}");
    }
}
catch (SqlException sqx) when (sqx.Number == 50000)
{
    _log.Warn($"The Specific thing we were looking for happened so we raised error 50000 in the proc: {sqx.Number} {sqx.Message}");
    throw new CustomExceptionForThisCondition(task);
}
catch (Exception ex)
{
    throw new Exception($"Something unexpected went wrong for {someId}", ex);
}

In [MySchema].[MyProcName]
-- detect some condition and decide you wish to raise an error back to the caller:
IF @IsSomethingBad = 1
BEGIN
		RAISERROR(50000, 16,1);
END;

Expected behavior
When RAISEERR is called and raises the default error number, a SQLException should be thrown with the correct error number noted so that it can be caught and handled as intended.
Further technical details
Microsoft.Data.SqlClient version: 5.2.0
.NET target: Net 8
SQL Server version: Azure SQL
Operating system: Windows 2019
Additional context
It seems like this is something the SqlClient library is intentionally looking for and having a problem with, due to the specificity of the exception message, in an attempt to force us to specify an error code other than 50000 which is the default. We should be consistent across the offerings by supporting the ability handle errors with the default error number https://learn.microsoft.com/en-us/sql/t-sql/language-elements/raiserror-transact-sql?view=sql-server-ver16#msg_id and not imposing a requirement in dotnet which does not exist in SQL itself.

After doing a small test which I should have done prior to submitting this bug, I can see this error is coming from the database and not SqlClient. Closing.