Conventional commits usage and further definition

This issue follows up on #1773 and recently merged #5342 for further defining usage of conventional commit structure specifically for our Lodestar monorepo.
Reference comment: https://github.com/ChainSafe/lodestar/pull/5419#pullrequestreview-1403626336

Add structure: type(scope?): subject and brief explanations for each
Define when to use each type
Should we advocate for using scope in every PR title? If yes, how? The github actions checker currently does not enforce this usage.
Modify examples of PR titles with bullet points instead of quoting
Define when and how to indicate breaking changes on commit level and further discuss how to display those in the Changelog which should be targeted towards non-developer user/operator of Lodestar. (see comment)

What's the purpose of (scope)? Having to pick a scope every time sounds annoying to me

What's the purpose of (scope)?

I think scope could be an easy way to give the commit more context, especially because Lodestar monorepo has several packages and subcategories

gives more clarity on what part of the code was modified, meaning the commit message itself does not need to do that and can focus on the change being done
makes commit easier filterable/searchable, would in work if this patterns is consistently applied
depending on how we generate changelogs, most tools group by scope, this makes changelog easier to read
generally, it should be easy to define a scope for most commits based on

package modified
hard fork
EIP

The scope is not required by the PR title checker as discussed here https://github.com/ChainSafe/lodestar/pull/5342#discussion_r1158126144, so usage would be based on personal preference. For me, it sometimes makes it easier to convey all required information in the commit message.

Completed by #5342


[QUESTION] Is matter rust ready for production ready controller implmenetation?

I am planning to build matter controller hubs with control bridge is this ready.

The "device" aspect is close, but - no, it is not completely ready.
There are still gaps that we need to fill-in. I would expect - for devices - the implementation to be ready in several months. Say - by EOW.
The "controller" aspect is not even started yet, and I don't have any ETA when it might be ready, as I actually don't have any use case for this yet.
I also don't really understand what you are trying to build. What does a "matter controller hub with control bridge" mean?

A hub like aqara m3 that is a matter controller and also control bridge used for connecting non matter devices to fabric

You don't need the "matter controller" functionality to connect non-Matter devices to a fabric. You only need the "Matter Bridge" functionality to do that.

I want matter controller for a stand alone hub also

As I said, controller is not possible with rs-matter currently, and unclear when it would be possible.
You might have a better luck using the Matter C++ SDK instead.
I'm closing this now, as it was just a question, but if you would like to keep an issue along the lines of "Support controller functionality in rs-matter", feel free to open one. I'm just not sure when (or if) that would be implemented, without people willing to contribute it.

Another alternative for mature controller support is https://github.com/project-chip/matter.js


Doesn't work on touchwiz lollipop rom.

hi,
this module doesn't remove the notification on my note 3 running lollipop 5.0 touchwiz.

Can you screenshot this "notification" you are talking about? This module removes a specific line of text in the quick toggles dropdown (see issue #1); it doesn't touch any notifications.

It doesn't work for me either, I'm on stock marshmallow rom (HTC M8 Google play edition)


Add support for Long256 type

https://questdb.io/docs/reference/api/ilp/columnset-types#long256

we should also add cases to test json

Hey, if no one is working on it, then I would like to work on it!

@JammUtkarsh that's great. Assigned you to this one. Thanks!


The accuracy of deeplab

I retrain the deeplab part and the result is 61% for baseline while 66% for dcn. This is a lot below the official result. I only changed the gpu to 2 and remove the multiprocess in loader. Anyone know why it came to be that?
the config file for dcn
---
MXNET_VERSION: "mxnet"
output_path: "./output/voc12"
symbol: resnet_v1_101_deeplab_dcn
gpus: '0,1'
SCALES:
- 360
- 600
default:
  frequent: 10
  kvstore: device
dataset:
  NUM_CLASSES: 21
  dataset: PascalVOC
  dataset_path: "./data/VOCdevkit2012/"
  image_set: 2012_train_seg
  root_path: "./data/"
  test_image_set: 2012_val_seg
network:
  FIXED_PARAMS:
  - conv1
  - bn_conv1
  - res2
  - bn2
  - gamma
  - beta
  FIXED_PARAMS_SHARED:
  - conv1
  - bn_conv1
  - res2
  - bn2
  - res3
  - bn3
  - res4
  - bn4
  - gamma
  - beta
  IMAGE_STRIDE: 0
  PIXEL_MEANS:
  - 103.06
  - 115.90
  - 123.15
  pretrained: "./model/pretrained_model/resnet_v1_101"
  pretrained_epoch: 0
TRAIN:
  warmup: false
  warmup_lr: 0.00005
  warmup_step: 1000
  begin_epoch: 0
  end_epoch: 12
  lr: 0.0005
  lr_step: '8'
  model_prefix: "deeplab_resnet_v1_101_voc12_segmentation_dcn"
  FLIP: true
  BATCH_IMAGES: 1
  ENABLE_CROP: False
  CROP_HEIGHT: 768
  CROP_WIDTH: 1024
  RESUME: false
  SHUFFLE: true
TEST:
  BATCH_IMAGES: 1
  test_epoch: 12

@Mendel1   have you retrain other part, for exampe ,rfcn and faster rcnn?   what is the  performace of  other parts?  thank you.


Add link to Roslyn breaking changes for .NET 5.0 and C# 9.0

I can't find a parallel article for Visual Basic to make sure it's not missing any links from https://github.com/dotnet/roslyn/tree/master/docs/compilers/Visual Basic

@BillWagner Fixed the conflict and added the ms.custom.


all-of-tests.eo:27-30: I assumed that we need to add more...

The puzzle 14-989b9606 from #14 has to be resolved:
https://github.com/objectionary/home/blob/6947074c54cbf0e3a2d366c45230f29eb491c136/tests/org/eolang/hamcrest/all-of-tests.eo#L27-L30
The puzzle was created by graur on 19-May-22.
Estimate: 45 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 14-989b9606 has disappeared from the source code, that's why I closed this issue.


Added MyPy types to zilencer/views.py

Automated message from Dropbox CLA bot
@pxhanus, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

I posted a few comments @pxhanus, otherwise this looks great!
Let me know when you've pushed a revised version correctly those little issues.

@timabbott, just updated the branch with your comments.

Automated message from Dropbox CLA bot
@pxhanus, thanks for signing the CLA!

Looks like you mispelled Realm as Realms; I just went ahead and fixed that and merged this, since the rest looked great.  Thanks @pxhanus!


Not an issue (sorry) but can someone help me understand this

I'm trying to customise the responsive image for content, but struggling to wrap my head around this
	// Full width page template
	if ( is_page_template( 'page-templates/page-full-width.php' ) ) {
		if ( 1200 < $width ) {
			$sizes = '(max-width: 1199px) 98vw, 1200px';
		} else {
			$sizes = '(max-width: 1199px) 98vw, ' . $width . 'px';
		}

So its saying If the window size is greater than 1200px then the image size is 98% of 1200px, but if the window size is less than 1200px the image size is 98% the width of the window?
Is that right?

@nitrokevin this is sort of confusing because we're talking about the width of the image AND the width of the viewport. The $width variable refers to the width of the image, not the viewport.
So the first part of the conditional is saying if the image is wider than 1200px, at the media query of (max-width: 1199px) the image should be 98% of the viewport width. If it doesn't match that media query, the width of the image will be set to 1200px.
The else part of the conditional is the same as above, except instead of setting the width to 1200px when it doesn't match the media query, the size of the image is set to the actual width of the image.
This article explains it pretty well:
https://viastudio.com/optimizing-your-theme-for-wordpress-4-4s-responsive-images/

Closing this out, feel free to reopen if you need more help.


Creating multiple RDSs with create-role-monitor it will throw an exception that the role exists cause it's used with hardcoded name in the code, the name should be depend to some variable

Description
Please provide a clear and concise description of the issue you are encountering, your current setup, and what steps led up to the issue. If you can provide a reproduction, that will help tremendously.
⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Terraform:

Provider(s):

Module:

Reproduction

Steps to reproduce the behavior:

Code Snippet to Reproduce

Expected behavior

Actual behavior

Terminal Output Screenshot(s)

Additional context

there is a variable for the role name https://github.com/terraform-aws-modules/terraform-aws-rds/blob/e38aaced17278567973dad4980fae8757d6a1265/modules/db_instance/main.tf#L216


PdfDocument have support for Windows Ink

PdfDocument have support for Windows Ink

This is a feature request for the platform, not the sample in this repo. Platform feature requests can be made in the Project Reunion project.


pig-latin: regenerate tests

Regenerate tests based on updated canonical data.
BookKeeping version incremented:
Additional tests for handling words containing 'y' added.

Need to update the example solution to pass the new tests.


OCI error on Intel CPU type

Describe the bug
Intel CPU based error with OCI when try to use
docker exec -it <container_id> /bin/bash
To Reproduce

Create machine with Ubuntu 22.04 LTS (Jammy) and Intel CPU type.

Install docker in this machines with Install Docker Engine on Ubuntu

Create docker-compose.yml file

Try to create a Seq container. Code as below.

services:
  seq:
    container_name: seq
    restart: unless-stopped
    volumes:
      - ./backup/seq:/data
    ports:
      - 5341:80
    environment:
      - "ACCEPT_EULA=Y"
      - "SEQ_FIRSTRUN_ADMINPASSWORD=Aa123456@"
    image: datalust/seq

Success Created container

When execute command as below
docker exec -it seq /bin/bash

Error of OCI

This is same to Azure-Sql-Edge container.

FYI:
But when I try to use same Ubuntu 22.04 LTS with Apple CPU type it will be able to run the No.6 command.
Expected behavior
Expected to be able to use docker exec command to access container directory.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.8.2
Commit: 44cf8269710cd29802bfee176a7cec65b55cc822 (v1.8.2)
System info:
macOS: 14.6 (23G80)
CPU: arm64, 8 cores
CPU model: Apple M3
Model: Mac15,12
Memory: 16 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-11-28T08-28-05.008792Z.zip

Screenshots and additional context (optional)
No response

Duplicate of #1522

Hi @kdrag0n, I don't think my post is duplicated. Since my container is build/created successfully. Just when I am trying to exec to the command to the container folder and it failed. And I can access the Seq service thru the address.


feat: add home view artists rail

This PR resolves ONYX-1167
Description
This PR adds the artists generic rail to Section. This rail's goal is to render ArtistsRailHomeViewSection.
https://github.com/user-attachments/assets/f35cda56-9c0b-4083-9478-4ca6e4269723

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against a788a025da34303482c4d1941190a678bb8f4909

Let's merge this and move to hooks later - @olerichter00 thanks for the fixes you made.


Allow configuring severity levels

This feature will allow configuring severity levels.
Related issue https://github.com/prontolabs/pronto-flay/issues/19

Released to RubyGems as 0.11.1 🎉


How to send Audio to IP Camera?

I would like to send vocal message to camera, so camera will play it.
How can I do this?

https://stackoverflow.com/questions/59465932/how-to-send-audio-data-on-rtsp-backchannel


feat: Add keyboards and pointing devices support

This pull request adds support of these devices:

VZUSBKeyboardConfiguration: https://developer.apple.com/documentation/virtualization/vzusbkeyboardconfiguration?language=objc
VZMacTrackpadConfiguration: https://developer.apple.com/documentation/virtualization/vzmactrackpadconfiguration?language=objc
VZUSBScreenCoordinatePointingDeviceConfiguration: https://developer.apple.com/documentation/virtualization/vzusbscreencoordinatepointingdeviceconfiguration?language=objc

@siketyan
Thanks you for support keyboards and pointing devices. It looks fine, so I have merged it.


[BUG] Attempt to set font using execCommand is erroring Flutter Web

Description
When trying to set the default font in the editor in the example application running on a Chrome browser, the font in the editor is not updated. Errors are seen in the IDE and Chrome Inspector.
To Reproduce
Steps to reproduce the behaviour:

Clone the repository.
Update pubspec.yaml of the example to add a dependency override - flutter_colorpicker: ^0.5.0
Update main.dart to add - controller.execCommand('fontName', argument: 'Courier New'); in the onInit of the Callbacks.
Run the application with Chrome (Web) as the target.
Once the page loads, enter any character in the editor.
Observe error in IDE & Chrome Inspector.

Expected behaviour
As per issue #93, if the default font is set for the editor, the editor should set the default font. I have tried the code changes with sans-serif, Times New Roman but no luck; I am seeing the same issue.
Screenshots
IDE -

Chrome Inspector -

Log Traces
IDE -
💪 Running with sound null safety 💪
Debug service listening on ws://<IP_ADDRESS>:64372/09CKqth2IIQ=/ws
Height of Platform View type: [9lPxiIN7KK12vg==] may not be set. Defaulting to `height: 100%`.
Set `style.height` to any appropriate value to stop this message.
Width of Platform View type: [9lPxiIN7KK12vg==] may not be set. Defaulting to `width: 100%`.
Set `style.width` to any appropriate value to stop this message.
done
Error: Unexpected null value.
    at Object.throw_ [as throw] (http://localhost:64325/dart_sdk.js:5061:11)
    at Object.nullCheck (http://localhost:64325/dart_sdk.js:5388:30)
    at http://localhost:64325/packages/html_editor_enhanced/src/html_editor_controller_unsupported.dart.lib.js:8408:18
    at Object._checkAndCall (http://localhost:64325/dart_sdk.js:5268:16)
    at Object.dcall (http://localhost:64325/dart_sdk.js:5273:17)
    at http://localhost:64325/dart_sdk.js:101824:100

Chrome Inspector -
Uncaught Error: Unexpected null value.
    at Object.throw_ [as throw] (errors.dart:251)
    at Object.nullCheck (operations.dart:528)
    at html_editor_widget_web.dart:661
    at Object._checkAndCall (operations.dart:334)
    at Object.dcall (operations.dart:339)
    at html_dart2js.dart:37308

Environment
Here is the output from flutter doctor -
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 2.5.0-5.3.pre, on macOS 11.5.2 20G95 darwin-x64, locale en-AU)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.1)
[✓] VS Code (version 1.60.0)
[✓] Connected device (2 available)

Additional context
I need to set the line height and the default font in the editor instead of offering the user Style options.

To everyone in the html-editor-enhanced - Thank you so much for your work on this package. Without, I couldn't have thought of building a Web application using Flutter where a Rich Text Editor is definitely needed.

Hi there, I am pushing an update soon. The way to do this is to use the onFocus callback:
onFocus: () {
                if (controller.characterCount == 0) {
                  controller.execCommand('fontName', argument: 'Courier New');
                }
                print('editor focused');
              }

Note: this will not work until the 2.3.0 release is pushed out. Expect it within the next hour or so :)

Thank you @tneotia 🙏

Hey @tneotia ,
Dont think this:
onFocus: () {
                if (controller.characterCount == 0) {
                  controller.execCommand('fontName', argument: 'Courier New');
                }
                print('editor focused');
              }

Works anymore


Feature requests for handhelds

I use fzy regularly on my handheld device with file manager nnn (I am the author of the tool). I'm really happy with the performance on a mid-range Android device.
From months of usage, I believe the following enhancements would improve the user experience on smaller form-factors greatly:

Most of the time I search files like documents of music. If I haven't missed it, currently there is no way to match only the filename instead of the full path. If such an option is vailable, that would reduce the number results and show more accurate ones in this use case.
On smaller screens, path folding would help a lot. Most of the time I can't see the file name, only the path and if there are more than one matches in the path, it's kind of a blind guess. In case it helps, here's an example path folding logic in nnn: https://github.com/jarun/nnn/blob/v2.8.1/src/nnn.c#L4045-L4066

Once again, thanks for the great little utility!

@jarun say:

On smaller screens, path folding would help a lot. Most of the time I can't see the file name, only the path and if there are more than one matches in the path, it's kind of a blind guess. In case it helps, here's an example path folding logic in nnn: https://github.com/jarun/nnn/blob/v2.8.1/src/nnn.c#L4045-L4066

Or https://github.com/motersen/pwdxz

@zvezdochiot very useful for us devs and I am having it in right away. The problem is, compilation turns off end users. That's why I am looking for something that's built in fzy.
Also, we still need the first one, to search only files (excluding paths). So both the requests kind go hand in hand.

Ignore this. nnn has dropped fzy support so this feature request isn't relevant to the project anymore.


Rewrite Cargo.toml when packaging crates

This commit is an implementation of rewriting TOML manifests when we publish
them to the registry. The rationale for doing this is to provide a guarantee
that downloaded tarballs from crates.io can be built with cargo build
(literally). This in turn eases a number of other possible consumers of crates
from crates.io

Vendored sources can now be more easily modified/checked as cargo build should
work and they're standalone crates that suffice for path dependencies
Tools like cargobomb/crater no longer need to edit the manifest and can
instead perform regression testing on the literal tarballs they download
Other systems such as packaging Rust code may be able to take advantage of
this, but this is a less clear benefit.

Overall I'm hesitatnt about this, unfortunately. This is a silent translation
happening on publish, a rare operation, that's difficult to inspect before it
flies up to crates.io. I wrote a script to run this transformation over all
crates.io crates and found a surprisingly large number of discrepancies. The
transformation basically just downloaded all crates at all versions,
regenerated the manifest, and then tested if the two manifests were (in memory)
the same.
Unfortunately historical Cargo had a critical bug which I think made this
exercise not too useful. Cargo used to not recreate tarballs if one already
existed, which I believe led to situations such as:

cargo publish
Cargo generates an error about a dependency. This could be that there's a
version not present in a path dependency, there could be a git
dependency, etc.
Errors are fixed.
cargo publish
Publish is successful

In step 4 above historical Cargo would not recreate the tarball. This means
that the contents of the index (what was published) aren't guaranteed to match
with the tarball's Cargo.toml. When building from crates.io this is ok as the
index is the source of truth for dependency information, but it means that any
transformation to rewrite Cargo.toml is impossible to verify against all crates
on crates.io (due to historical bugs like these).
I strove to read as many errors as possible regardless, attempting to suss out
bugs in the implementation here. To further guard against surprises I've updated
the verification step of packaging to work "normally" in these sense that it's
not rewriting dependencies itself or changing summaries. I'm hoping that this
serves as a good last-ditch effort that what we're about to publish will indeed
build as expected when uploaded to crates.io
Overall I'm probably 70% confident in this change. I think it's necessary to
make progress, but I think there are going to be very painful bugs that arise
from this feature. I'm open to ideas to help weed out these bugs ahead of time!
I've done what I can but I fear it may not be entirely enough.
Closes #4027

r? @matklad

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

I'll need some time to digest it, but one thought I have right now is that perhaps it would be prudent to add original manifest to archive as well? I think having unmodified manifests could help us if something goes terribly wrong.

That's probably not a bad idea, yeah. Something like Cargo.toml.orig perhaps?

That's probably not a bad idea, yeah. Something like Cargo.toml.orig perhaps?

Any name will do as long as it is mentioned in the comment of the rewritten file.
However, mechanically rewriting a file which was created manually by the user does not look super great to me. An alternative would be to store modifications in a separate file, Cargo.patch, analogous to Cargo.lock, and then merge Cargo.toml and Cargo.patch during workspace loading. This second solution seems more clean to me, but I don't like it anyway because it's more complicated and introduces additional logic :)

Unfortunately the key feature here is that the Cargo.toml is overridden. in the sense that a "normal" cargo build will work in the crate, and the crate otherwise operates as a normal path dependency when the source is downloaded. Now that doesn't meant that we couldn't support a "patch" format of some kind (maybe not literally a patch file), but it means that it'd become a first-class feature of Cargo that everyone could use everywhere, which I'm not sure we want to do just yet...

Am I correct that this rewriting should not affect anything besides the literal "cargo build downloaded crate" use case, because we are rewriting only the summary of the manifest, and we ignore summary from Cargo.toml and use the one from the index anyway?
On the one hand, this is good, because it means that rewriting Cargo.toml is unlikely to break anything significant. On the other hand, this means that this feature will be rarely tested in the wild, and some bugs may remain uncovered.

This is a silent translation happening on publish, a rare operation, that's difficult to inspect before it
flies up to crates.io

I think because publish is a rare operation, we can make it much noisier than usual. Like, we can copy all the files we are going to publish to a temp directory, together with rewritten toml, show the listing of files we are going to publish (were there some complaints that cargo accidently published stuff that wasn't supposed to be published and vice verse?), print some information about rewriting path deps to usual deps, and ask a conformation?

Now that doesn't meant that we couldn't support a "patch" format of some kind (maybe not literally a patch file), but it means that it'd become a first-class feature of Cargo that everyone could use everywhere, which I'm not sure we want to do just yet...

One more alternative is to add a single very specific boolean option to Cargo.toml (like, appending published = true to the end of the file), which basically says: "ignore path deps when reading this file".

On the other hand, this means that this feature will be rarely tested in the wild, and some bugs may remain uncovered.

Ah, I missed the part that now we verify unmodified package! That should help a lot.
Overall, I think we should definitely do this! The only thing that makes me nervous is implementaion: there's a lot of code there, and most of the code is identity function. Perhaps we can operate on untyped toml Table directly and modify only the path we are interested in. That way, we wouldn't need to change TOML frobbing code when adding new keys to the manifest.

Am I correct that this rewriting should not affect anything besides the literal "cargo build downloaded crate" use case

Not 100%. We do indeed ignore the Summary portion of Cargo.toml, but we still parse Cargo.toml from the registry downloads (for normal crates.io builds) to learn about targets. Stuff like "is this a proc macro?" or "where is the library entry point file?" In that sense the targets are critical to preserve, the dependencies less so for normal operation.

I think because publish is a rare operation, we can make it much noisier than usual

I definitely agree with this, we've got a lot of leeway here. I'm wary to add interactivity which might break scripts, but I think a saving grace of ours is the sanity check of "cargo build" just before you publish. The modifications here make this almost a cargo build modulo how the workspace is handled, but it should in theory be a good check that the crate at least builds.
Do you think that's enough? Or do you think we should add something else? I'm wary to say "please look at this file and make sure it's right" because it's a pretty ugly file (not a pretty toml, a fully explicit toml) so it will look alien to most users.

One more alternative is to add a single very specific boolean option to Cargo.toml (like, appending published = true to the end of the file), which basically says: "ignore path deps when reading this file".

Hm that's an interesting idea yeah. Something that basically makes it as uninvasive as possible. I'm still a little wary of doing this though as it's a new public-facing feature unlike this implementation which is entirely an internal detail (in theory).
One thought that's also been rolling around in the back of my head is that there may not always be an obvious and/or canonical desugaring for a manifest. Right now we drop [workspace] and path dependencies, but the path dependencies are specifically rewritten to crates.io and more generally the registry being published to.
Basically I'm thinking that with multi-registry support Cargo may not know via simply a boolean flag where a path dependency is supposed to point, it may or may not be crates.io.

only thing that makes me nervous is implementaion: there's a lot of code there, and most of the code is identity function. Perhaps we can operate on untyped toml Table directly and modify only the path we are interested in?

My hope here was to use the literall 100% same exact structures for serialization and deserialization. That at least provides the guarantee that we won't forget to implement any key we add support for or something like that. It definitely means it's a bit trickier, however, because a lot of the deserialization business is done mostly for ergonomics (lots of options, multi variants, etc).
The downside is definitely that it's a relatively large wad of code, and some of the translation is non-obvious. Primarily the inference from a list of Target structures back to the [lib] etc sections is pretty scary...

Oh I've also pushed an update which writes in Cargo.toml.orig with the exact contents of the original manifest.

Primarily the inference from a list of Target structures back to the [lib] etc sections is pretty scary...

Yeah. Why  from_package_for_registry has a signature Package -> TomlManifest. Can we make it TomlManifest -> TomlManifest?

Oh I don't mind either way, we need to go from a Package to TomlManifest somewhere, and there's not really a lot of usage in having an ephemeral TomlManifest, so I figured we'd short-circuit.

Could you run into cases where the Package is less complete than the raw toml?  e.g. missing a field for some forward-compatible new feature, which probably shouldn't just be dropped when an older cargo does the rewriting.

Yes that's true I think. Basically if you use an older Cargo to publish a newer crate, it'd drop fields it didn't know about.
The alternative, though, I see as sort of just as bad. If we just trawl through the TOML representation deleting, for example, path keys and workspace keys, then we (a) may delete too much or (b) may forget to delete something crucial. For example if we add a new [foo-dependencies] section it seems very easy to forget to delete path keys from that section, whereas today it'd be a compiler error.
Does that make sense? I think I'd slightly prefer the strategy with compiler errors and basically recommend you publish w/ recent versions of Cargo, which typically is what you're doing anyway as you likely just ran cargo test before or some time before publication.

Oh I don't mind either way, we need to go from a Package to TomlManifest somewhere,

Can you just reparse toml file into TomlManifest? It seems to me that Package -> Toml is actually the most laborious step here, and not the actual rewrite of stuff. And because TomlManifest is typed, you'd still get compile time errors.
What do you think about adding a test that rewrites a Cargo.toml and checks the resulting file literally? I understand that this should already be integration-tested by our packaging tests, but having a before-after style test would probably help with checking some edge cases, should they become apparent.

@matklad hm I'm not 100% sure what you're asking for actually, mind elaborating? I'm not sure what you're thinking is transitioning to what?
I wouldn't mind though, yeah, adding a byte-for-byte check somewhere of the manifests that are generated.

@matklad hm I'm not 100% sure what you're asking for actually, mind elaborating?

Sure!
We have an original Package and want to get a rewritten TomlManifest. This can be represented as a composition of two functions: to_toml_manifest: Package -> TomlManifest and rewrite: TomlManifest -> TomlManifest. In the current implementation in from_package_for_registry we do both steps simultaneously.
Looks like it's to_toml_manifest part that is the most difficult! And in some sense it seems not really necessary: we've got a Package from TomlManifest originally, so why do we have to write the conversion in the opposite direction, if we can reuse the exact original manifest?
I think we can trivialize to_toml_manifest, if we don't try to recreate TomlManifest from Package literally, as we do now, but instead use the same code we already have for creating TomlManifests, namely, parsing a Cargo.toml file. In pseudorust:
fn to_registry_toml(pkg: Pacakge) -> String {
    let toml_text = fs::File(pkg.manifest_path).read_to_string();
    let toml_value: toml::Value = parse(toml_text);
    let toml: TomlManifest = serde::deserialize(toml_value); // here we are getting `TomlManifest` for free
    rewrite(toml).to_string()
}

Hm, now that I've spelled this out I see one more way to get TomlManifest for free: we can add a original_manifest field to the Package:
struct Package {
  original_manifest: TomlManifest
}

Anyways, this is only an implementation detail and it does not matter much if tested thoroughly :) If you think the current implementation is sufficiently best, let's add a byte-for-byte check and merge this.

Ok I think I like that strategy more. It still doesn't handle @cuviper's concerns about how it drops unknown keys from the manifest (e.g. ones added in future Cargos) but it makes me more comfortable by preserving the original keys and such.

@matklad ok I've updated with said strategy

@alexcrichton looks great now!
@bors r+

:pushpin: Commit 57db8bd has been approved by matklad

:hourglass: Testing commit 57db8bd with merge 3973598...

:sunny: Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 397359840ecad02d5fe69b2a0cf328e98235ffea to master...


Conway's Game of Life for CP/M-65

Implementation of Conway's game of life using the screen driver. Tested on my own computer build which I am porting CP/M-65 to, and on emulated BBC Micro.

Excellent, thank you!
Just a few notes on looking at the code:

constructing each line in a buffer and using screen_putstring should be faster as you don't need to go through the overhead of the screen dispatch routine for every character. I don't know if it is faster, but it should be, and would be interesting to find out.
there are supposed to be some constants defined for reading things like cursor keys. Apparently I never implemented that...

I thought about using screen_putstring as well, but I got some odd results when trying to use it from C, and I did not have time to investigate it further. It seems like the pointer does not make it to the BIOS in the expected fashion, but I could be mistaken as I have not looked too much at it.
Constants for cursor keys would be useful!


Instructions for known contaminating sample

Could you please post instructions running CleanCall when the source of contamination is known?  I can't seem to find the anywhere.

Sorry for the delayed response. When you run "cleanCall genotype", you need
to provide --col-mixid option to indicate the column that contains the IDs
of known contaminating sample.
Sorry that it was not included in the README.md but it was documented in
cleanCall genotype software tool itself. I hope that this is helpful, but
let me know if you have further questions.
Hyun.
Hyun Min Kang, Ph.D.
Associate Professor of Biostatistics
University of Michigan, Ann Arbor
Email<EMAIL_ADDRESS>On Fri, Jun 14, 2019 at 6:41 PM Elena Nabieva<EMAIL_ADDRESS>wrote:

Could you please post instructions running CleanCall when the source of
contamination is known? I can't seem to find the anywhere.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hyunminkang/cleancall/issues/8?email_source=notifications&email_token=ABPY5OKLTLS3YUB3APSI3CLP2NRV3A5CNFSM4HYGG7S2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZQMQSA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPY5OPMKDLJFEXRDY2Y27TP2NRV3ANCNFSM4HYGG7SQ
.

Thank you Hyun.  I have seen the command-line option, but I still cannot figure out how exactly to apply it. Could you please give an example of an index file for the contamination sample case?

Hi Elena,
You need to add additional column in your PED file. For example, header could be
#FAM_ID IND_ID DAD_ID MOM_ID SEX MPU CONTAM MIX_ID
And include the identity of contaminating sample in MIX_ID, and use --col-mixid MIX_ID when you run the tool. I hope this is clear.

Thank you very much for your reply.  Can I run my pipeline by you to make sure I'm running it correctly:
The setup is that I have an abortus sample and a mother sample, and I want to cleancall the abortus given that it may be contaminated with the mother.
The index file looks like this:
index.txt:
abortus abortus.bam
mother mother.bam
I run it through cleancall-pileup:
$CLEANCALL/cctools pileup --loci $HAPMAP --index index.txt --out out --ref $REF --run 6
(where $HAPMAP refers to the hapmap 3 file from GATK resource bundle)
it creates a  a ped file that looks like this:
#FAM_ID IND_ID  FAT_ID  MOT_ID  SEX     MPU
abortus abortus 0	0	0	out.abortus.txt.gz
mother  mother  0	0	0	out.mother.txt.gz
(and the corresponding pileups, i.e., out*.txt.gz files that are mentioned in the ped file)
Then I run cleancall-verify as:
$CLEANCALL/cctools verify --index out.ped --out out.verify --vcf $HAPMAP --run 6
 giving it the ped file from above
As a result it gives me a out.verify.ped file that looks like this:
#FAM_ID IND_ID  FAT_ID  MOT_ID  SEX     MPU     FREEMIX CHIPMIX
abortus abortus 0       0     0       out.abortus.txt.gz   0.17626 NA
mother mother  0       0       0	out.mother.txt.gz    0.00114 NA
Which I edit, as per my understanding of what you have written (and correcting the pedigree info):
#FAM_ID IND_ID  FAT_ID  MOT_ID  SEX     MPU     FREEMIX CHIPMIX MIX_ID
abortus abortus 0       mother  0  out.abortus.txt.gz    0.17626 NA mother
abortus mother  0       0       0	out.mother.txt.gz     0.00114 NA NA
And then I run it (just on chr1, since otherwise I get an error) as
#$CLEANCALL/cctools genotype --invcf $HAPMAP  --ped out.verify.ped --out out.genotype.vcf.gz --ref $REF --run 1 --col-pair MIX_ID --chr chr1
Is this correct?
Thank you very much for the help!


Make 'locate yourself' actual text instead of CSS

Issue Overview

It'd be easier to work with in general if it were actual text, not just CSS content.
Describe your environment

Steps to Reproduce

Expected Behavior

Current Behavior

Possible Solution

Screenshots / Video

Related Issues and/or PRs

I'm working on this one as part of set: #29 #32 #28 .

I'm working on this one as part of set: #29 #32 #28 .


Make it clear this is a React component, not an IDE

We should also update this description to be something like
"Code editor as a React component with unlimited AI autocomplete by Codeium and TypeScript support.
@khou22


Upgrade pkg to 4.1.0

Stack traces are shorter, removing info of where it was pkged.
/snapshot/home/travis/build/zeit/now-cli/lib/index.js becomes
/snapshot/now-cli/lib/index.js

That's wonderful!


ReferenceError: GhostText is not defined

I'm using Chrome Version 53.0.2785.89 m, with the GhostText Extension for chrome 1.2.1 and the Sublime Text 3 package GhostText 1.0.13, running on Windows 7 Professional x64.
Whenever I try to activate GhostText the chrome console produces the following output:
Got message: Object {action: "select-field", tabId: 38}
GhostText: selectField()
followed by this error:
Error in event handler for runtime.onMessage: ReferenceError: GhostText is not defined
Sublime Text itself seems to be able to successfully start GhostText, however does not seem to log anything thereafter.
Any help would be very much appreciated,
cheers.

The same problem

I think it related to  the last update. #81

Sorry about this, I'm looking into it.

1.2.2 should work

Thank you for uploading new version to Chrome Web Store.
For make it ASAP for everyone

Navigate to: chrome://extensions
click on Developer mode
click Update extensions now


Some examples fail validation

https://ampbyexample.com/components/amp-form - https://validator.ampproject.org/#url=https%3A%2F%2Fampbyexample.com%2Fcomponents%2Famp-form
https://ampbyexample.com/components/amp-fx-flying-carpet - https://validator.ampproject.org/#url=https%3A%2F%2Fampbyexample.com%2Fcomponents%2Famp-fx-flying-carpet
https://ampbyexample.com/components/amp-live-list - https://validator.ampproject.org/#url=https%3A%2F%2Fampbyexample.com%2Fcomponents%2Famp-fx-flying-carpet
https://ampbyexample.com/components/amp-sidebar - https://validator.ampproject.org/#url=https%3A%2F%2Fampbyexample.com%2Fcomponents%2Famp-sidebar
https://ampbyexample.com/samples_templates/housing/preview/ - https://validator.ampproject.org/#url=https%3A%2F%2Fampbyexample.com%2Fsamples_templates%2Fhousing%2Fpreview%2F

Hi,
when samples contain experimental component, it's normal that they don't validate.
amp-form is experimental, so all the examples containing it will fail the validation, such us amp-form and housing.
amp-fix-flying-carpet is experimental, see doc
amp-live-list is experimental, see doc
amp-side-bar should work, but probably the problem is that we are using that twice in the page (for the left menu and the example itself). Please create an issue just for this.
Hope it helps.

The amp-sidebar sample is tricky to get to validate. There are two problems:

amp-sidebar needs to be the first element after the body tag (see #167)
only one sidebar is allowed per page and we have two together with the ABE navigation menu.

Would it be possible to use an amp-iframe for the script tag (experimental mode)?
This would also allow to have the amp-sidebar as a child of  for example in this page: https://ampbyexample.com/samples_templates/product/preview/

We can't use an iframe for triggering experimental mode -> experimental flag is stored via cookie  which needs to be set for the document host.
Fixing samples with amp-sidebar needs to be handled during the example compilation step.


Add-bde-token

[ ] I will not ping the Discord about this pull request.

Please provide the following information for your token.
Please include change to the src/lib/.json and in the PR
At minimum each entry should have

Token Address: H5gczCNbrtso6BqGKihF97RaWaxpUEZnFuFUKK4YX3s2"
Token Name: Big Defi Energy
Token Symbol: BDE
Logo: https://raw.githubusercontent.com/bitcoinoverdose/bitcoinoverdose/main/bde.png
Link to the official homepage of token: BigDefiEnergy.com
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

If there's issue with the pull request let me know

@bitcoinoverdose Please resolve conflicts on this branch and this should be good to go.

https://github.com/solana-labs/token-list/pull/216


New Charts in v2.5.12

System charts (v1) rancher/system-charts:dev-v2.5:

Name
Version
Backend Dev(s)

Feature charts (v2) rancher/charts:dev-v2.5:

Name
Version
Note
Backend Dev(s)
rancher-max of prev chart

Longhorn
1.2.300+up1.2.3

@innobead

Longhorn
1.1.300+up1.1.3

@innobead

rancher-alerting-drivers
1.0.301

@jiaqiluo

rancher-backup
1.2.100+up1.2.1

@superseb
<2.5.99-0

rancher-istio
1.10.400

@brendarearden
>= 2.5.0-0 <= 2.5.99-0

rancher-kiali-server
1.35.100+up1.35.0

@brendarearden

rancher-logging
3.15.0

@paynejacob@Anna Blendermann
<2.5.99-0

rancher-monitoring
16.6.1+up16.6.0

@jiaqliuo

rancher-vsphere-csi
2.3.1

@pennyscissors
>= 2.5.0-0 <= 2.5.99-0

rancher-webhook
0.1.300+up0.1.3

@aiyengar2

Verified the following charts on 2.5.12-rc4:

Name
expected version
actual version
expected rancher
actual rancher
result

Longhorn
1.2.300+up1.2.3
1.2.300+up1.2.3

<2.5.99-0
MATCH

Longhorn
1.1.300+up1.1.3
1.1.300+up1.1.3

<2.5.99-0
MATCH

rancher-alerting-drivers
1.0.301
1.0.301

>= 2.5.0-0 <= 2.5.99-0
MATCH

rancher-backup
1.2.100+up1.2.1
1.2.100-rc1+up1.2.1-rc1
<2.5.99-0
<2.5.99-0
MATCH (!) rc needs to be removed

rancher-istio
1.10.400
1.10.400
>= 2.5.0-0 <= 2.5.99-0
>= 2.5.0-0 <= 2.5.99-0
MATCH

rancher-kiali-server
1.35.100+up1.35.0
1.35.100+up1.35.0

MATCH

rancher-logging
3.15.0
3.15.0
<2.5.99-0
<2.5.99-0
MATCH

rancher-monitoring
16.6.1+up16.6.0
16.6.1+up16.6.0

MATCH

rancher-monitoring-crd
16.6.1+up16.6.0
16.6.1+up16.6.0

MATCH

rancher-vsphere-csi
2.3.1
2.3.1
>= 2.5.0-0 <= 2.5.99-0
>= 2.5.0-0 <= 2.5.99-0
MATCH

rancher-webhook
0.1.300+up0.1.3
0.1.300-rc1+up0.1.3-rc1

MATCH (!) rc needs to be removed

@sgapanovich - rancher-webhook and rancher-backup-restore-operator have been unRC'd.  Can you please verify that they're still good?

Available to test in v2.5.12-rc7

check webhook and rancher-backup
both apps are ready to be releases
rancher-images.txt has no issues

@brendarearden checked that charts are unRC'd. please tag a final RC


Override json encoder for individual field

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
I would like to be able to override JSON encoder for individual fields, instead for the whole model.
The following example is not possible, or I could not realize how to achieve that based on documentation.
Example:
from pydantic import BaseModel
from decimal import Decimal
from functools import partial

class Foo(BaseModel):
    price: Decimal = Field(encode=partial(round, ndigits=2))
    size: Decimal = Field(encode=partial(round, ndigits=1))

Foo(price=Decimal('1.11111111'), size=Decimal('2.222222')).json()  # want: '{"price": 1.11, "size": 2.2}'

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[x] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

You can define a custom DecimalEncoder class that inherits from the json.JSONEncoder class and overrides the default() method to handle Decimal objects by converting them to floats.
Then, in the json method of the Foo class, we pass an instance of DecimalEncoder as the cls argument to json.dumps() to use the custom encoder.
Example
import json

class DecimalEncoder(json.JSONEncoder):
    """
    Custom JSON encoder that can handle Decimal objects.
    """
    def default(self, obj: Any) -> Any:
        if isinstance(obj, Decimal):
            return float(obj)
        return super().default(obj)

Understanding how to override JSON encoder for individual fields
In this part, you need to define a custom  json method that first calls self.dict() to get a dictionary of the model's fields and values.
Then, for each field, you can check if a custom encoder function has been defined using the encoding parameter of the Field() class. If so, you apply the encoder to the field's value.
Finally, you serialize the modified dictionary to a JSON formatted string using json.dumps().
import json
from typing import Any
from pydantic import BaseModel, Field
from decimal import Decimal
from functools import partial

class Foo(BaseModel):
    price: Decimal = Field(encode=partial(round, ndigits=2))
    size: Decimal = Field(encode=partial(round, ndigits=1))

    def json(self, *args: Any, **kwargs: Any) -> str:
        """
        Serialize the model to a JSON formatted string.
        """
        data = self.dict()
        for field_name, field in self.__fields__.items():
            if field_name in data:
                # Apply the custom encoder to the field
                encode_func = field.field_info.extra.get('encode')
                if encode_func:
                    data[field_name] = encode_func(data[field_name])

        return json.dumps(data, cls=DecimalEncoder, *args, **kwargs)

you can define the price and size fields with the encode parameter set to partial(round, ndigits=X), where X is the desired number of digits to round to.
Then define the custom DecimalEncoder class and the json method of the Foo() class in the same way as before.
Now you can run your example around and get the desired result:
foo = Foo(price=Decimal('1.11111111'), size=Decimal('2.222222')).json()

print(foo) # {"price": 1.11, "size": 2.2}

Hope this can help you

Thanks, your solution will actually work.
I wish this is a feature of the framework, maybe in the planned Pydantic V2.
What's your opinion on that?

This is already possible in pydantic-core and is already partially implemented in main.
There are two solutions, both shown here.
def custom_int_serialising(value: int, info):
    return f'custom serializer 1: {value}'

# this makes CustomInt reusable,
# but you could also put this code directly into the model on the "x: " line
CustomInt = Annoated[int, CustomSerializer(custom_int_serialising)]

class MyModel(BaseModel):
    x: CustomInt
    y: int

    @serializer('y')
    def custom_serializer(self, info):
        return f'custom serializer 2: {self.y}'

Both are WIP, but should be fully supported  when we release V2.
I'll close this since you have a fix for now.


docs(kubernetes): suggest the use of an init container to download mods which disallow project distribution

Enhancement Type
Improve an existing feature
Describe the enhancement
Some mods disallow project distribution and have to be download manually. This can be tricky for containerised environments, especially for Kubernetes. It might be nice to document the use of an init container to download these assets automatically. See what I've done as an example here.
It might also be possible to write something quick for Docker compose too?
minecraft-cf-atm8-0 minecraft-server [mc-image-helper] 12:20:47.139 WARN  : The authors of the mod 'Fix Experience Bug (now with FABRIC and FORGE versions)' have disallowed project distribution. Manually download the file 'ExperienceBugFix-1.19-<IP_ADDRESS>.jar' from https://www.curseforge.com/minecraft/mc-mods/fix-experience-bug and supply the mod file separately.
minecraft-cf-atm8-0 minecraft-server [mc-image-helper] 12:20:45.911 WARN  : The authors of the mod 'More Overlays Updated' have disallowed project distribution. Manually download the file '[1.19.2] MoreOverlays 1.21.5' from https://www.curseforge.com/minecraft/mc-mods/more-overlays-updated and supply the mod file separately.
minecraft-cf-atm8-0 minecraft-server [mc-image-helper] 12:20:47.165 WARN  : The authors of the mod 'Hexerei' have disallowed project distribution. Manually download the file 'hexerei-0.3.0.jar' from https://www.curseforge.com/minecraft/mc-mods/hexerei and supply the mod file separately.

The linked example rendered as yaml:
initContainers:
  - name: download
    image: curlimages/curl:8.1.1
    args:
      - -C
      - '-'
      - -LOf
      - '{https://mediafilez.forgecdn.net/files/4178/188/ExperienceBugFix-1.19-<IP_ADDRESS>.jar,https://mediafilez.forgecdn.net/files/4322/445/moreoverlays-1.21.5-mc1.19.2.jar,https://mediafilez.forgecdn.net/files/4466/686/hexerei-0.3.0.jar}'
    workingDir: /downloads
    resources:
      limits:
        cpu: "1"
        memory: 1Gi
    volumeMounts:
      - name: downloads
        mountPath: /downloads
    imagePullPolicy: IfNotPresent
    securityContext:
      capabilities:
        drop:
          - ALL
      readOnlyRootFilesystem: true
      allowPrivilegeEscalation: false

Great suggestion. It would be awesome if you could PR a new entry in this docs section: https://github.com/itzg/docker-minecraft-server/tree/master/docs/misc/deployment

@uhthomas You might want to take a look at what @tacomonkey11 has built in this repo:
https://github.com/TacoMonkey11/curseforge-initContainer

On another node. Have you had any issues with connection timeouts to the game? I am especially having it with ATM9


using #id rather than #/id with history API

Hi, I just heard about impress.js the other day and I'm truly impressed. Many thanks! 8-)
impress.js uses #/id to avoid the browser scrolling to the anchor, but the recommended way of linking into the presentation is #id. I've tried using the history API as follows to remove the slashes, and it works for me, but I'm no HTML5 wizard so I don't know what limitations this might be introducing. Anyway, maybe it helps.
Patch:
diff --git a/js/impress.js b/js/impress.js
index dedcefa..caa95fe 100644
--- a/js/impress.js
+++ b/js/impress.js
@@ -609,7 +609,10 @@
             // makes transtion laggy.
             // BUG: http://code.google.com/p/chromium/issues/detail?id=62820
             root.addEventListener("impress:stepenter", function (event) {
-                window.location.hash = lastHash = "#/" + event.target.id;
+                lastHash = "#" + event.target.id;
+                if (window.location.hash !== lastHash) {
+                    history.pushState(null, null, lastHash);
+                }
             }, false);

             window.addEventListener("hashchange", function () {

Hi Jacek
When you say

impress.js uses #/id to avoid the browser scrolling to the anchor, but the recommended way of linking into the presentation is #id

What exactly do you mean by "recommended"? It seems to me having slashes in the anchor is quite common, and may indeed be used to describe "state" of more complex javascript apps, rather than just linking to an element id.
Or to put it more simply, linking to something like: http://impress.github.io/impress.js/#/step-2 works just fine, so what is the problem this issue is trying to fix?


256 color mode/Colorschemes won't work on Raspbian Correctly

~/micro -version
Version: 2.0.3
Commit hash: f594686
Compiled on April 10, 2020

cat /etc/os-release
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Issue
256 color mode/Colorschemes wont work on Raspbian Correctly

✔ img - cmc-16
❌ img - ZenBurn
⚠ ...

+Info

✔ ColorTest

This may be caused by your terminal emulator not supporting 256 color, or not enabling it. You can set the TERM shell variable to xterm-256color to enable it (echo $TERM will give xterm if not enabled).

Thanks, it worked!


Show highest APY boost when there are multiple

This isn't an ideal solution but should work fine just now for v1.
If there are multiple active boosts on a vault, the APY and green tag will display the boost with the highest APY.
The 'Boost' button however will go to the /stake page - rather than straight to the boost page when there are multiple boosts.
state.vault.vaultLaunchpools now stores a list of boost ids sorted from highest apy to lowest; or an empty array if no boosts for that vault
state.vault.vaultLaunchpool now stores the id of the highest boost; or null if no boosts for that vault

this is awesome!


Fix a bug of path or it will raise ImportError: No module named pycoc…

Fix a bug of path or it will raise ImportError: No module named pycocotools.coco

interesting, i thought _init_paths.py already has data/coco included?

For me it was sufficient to make after downloading the coco API:
cd data
git clone https://github.com/pdollar/coco.git
cd coco/PythonAPI
make

thanks! closing


star pattern using for loop

Example for printing star pattern for pyramid

#include<stdio.h>
int
main ()
{
  int i, j;
  for (i = 1; i <= 5; i++)
    {

      for (j = i; j < 5; j++)
	{
	  printf (" ");
	}

      for (j = 1; j <= (2 * i - 1); j++)
	{
	  printf ("*");
	}

      printf ("\n");
    }

  return 0;
}

Output:
     *
    ***
   *****
  *******
 *********

``

<!-- Please follow this checklist and put an x in each of the boxes, like this: [x]. It will ensure that our team takes your pull request seriously. -->

- [ ] I have read [freeCodeCamp's contribution guidelines](https://github.com/freeCodeCamp/freeCodeCamp/blob/master/CONTRIBUTING.md).
- [ ] My pull request has a descriptive title (not a vague title like `Update index.md`)
- [ ] My pull request targets the `master` branch of freeCodeCamp.
- [ ] None of my changes are plagiarized from another source without proper attribution.
- [ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.

Closes #XXXXX

Looks good to me, and congratulations on your first Pull Request!  🎉🎉🎉
We look forward to future contributions!


City select widget improvements

[x] Is it possible to change some colors?

#FFFFFF: Unsupported app, can not be selected
#F4F4E9: Supported app, can be selected (not changed, this is the default color in Mapael, I guess)
#5BC4BF: App that is selected
#A3E135: When hovering (not changed)

[x] Add a legend that explains these color codes

#5BC4BF:

EN: You have selected this city app
NL: Je hebt deze stad app geselecteerd

#F4F4E9:

EN: They own our city app
NL: Zij hebben onze stad app

#FFFFFF:

EN: Will they be next to own our city app?
NL: Zijn zij de volgende met onze stad app?

[x] I changed the modal size to fit the entire screen. But when zooming the map, the full potential height isn't used yet. I think you can achieve this by changing the height of #map_container .map svg to 100%

For this one

I changed the modal size to fit the entire screen. But when zooming the map, the full potential height isn't used yet. I think you can achieve this by changing the height of #map_container .map svg to 100%

Did you mean the full potential height of the modal or the map? i only extended the modal body so that the footer should be always at the bottom.
The data of the map itself define a fixed width and height.


How to format on save (like it works with all styled. functions)?

I may be missing something really obvious but is there a way to make vscode auto format the template string as css just like it does with styled-components or @emotion/styled?
I'm getting syntax highlighting and auto completion and I have vscode-styled-components extension installed as well.

Formatting is not the main feature in this extension, people usually choose prettier when doing document formatting.
But I don't know how to write a prettier plugin about embedded lang, it is hard coded.

Close due to inactivity.


Performance in Cora

Hey, I run your code without modifying anything, but obtaining bad performances under default setting(Cora dataset).

Epoch 99/10000
1/1 [==============================] - 0s 78ms/step - loss: 0.1020 - acc: 0.0569 - val_loss: 0.3604 - val_acc: 0.0395y with val_weighted_acc available, skipping.
Epoch 100/10000
1/1 [==============================] - 0s 80ms/step - loss: 0.1021 - acc: 0.0480 - val_loss: 0.3605 - val_acc: 0.0343y with val_weighted_acc available, skipping.
Epoch 101/10000
1/1 [==============================] - 0s 74ms/step - loss: 0.1021 - acc: 0.0679 - val_loss: 0.3605 - val_acc: 0.0273y with val_weighted_acc available, skipping.
Epoch 102/10000
1/1 [==============================] - 0s 82ms/step - loss: 0.1018 - acc: 0.0731 - val_loss: 0.3605 - val_acc: 0.0218y with val_weighted_acc available, skipping.
Done.
Test loss: 0.7192684412002563
Test accuracy: 0.046159528195858

My environment is TensorFlow2.3-gpu and Keras 2.4.3.

Hey,
sorry, this is an old implementation based on TF 1.15 and Keras < 2.3.
Many things changed between TF1 and TF2, and the results may not be reproducible using different versions (I know, weird, but you can try for yourself).
I have added a deprecation warning on the README, and I suggest you look at the up-to-date, more modern Spektral library which also features a GAT example.
Cheers

Thanks for your nice work. I have done some simple modifications on this code and obtain expected results.


fix: add tty only for backend

Closes RVTEE-494

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#327
#326  👈
#323
#322
#320
#325
#324
#318
#317
#316
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AngelOnFira and the rest of your teammates on  Graphite


User-initiated zoom events?

Hello,
It seems that leaflet has no specific event, which is triggered when the user zooms the map, but not when the zoom level was changed programmatically.
Maybe dragstart should be triggered when pinch-zooming because this usually changes the map position too.
I use the leaflet-locatecontrol and I need to stop tracking the user movements when he drags or pinch-zooms the map.

While creating a custom layer I found myself having to hack the lib to have access to zoom events.
I needed to know to which zoom level the map was animating, zoom property is updated only on zoomend event, which is too late if you want to animate anything custom at the same time as the map animation.
If anyone is interested by my solution, what I did is firing a zoom event:
this.fire('zoom', {zoom: zoom}); in setZoom() and setZoomAround() functions from L.Map.
Can be related to https://github.com/Leaflet/Leaflet/issues/3269

I have the same situation, when I need to prevent from invoking fitBounds while user interacts with the map. Looks like it could be fixed while https://github.com/Leaflet/Leaflet/pull/3278, but it's already was merged. I believe it could be named something like triggerSource with values user and program, or just a boolean triggeredByUser.

Additional information, but related to the google maps: http://stackoverflow.com/questions/10097647/how-to-listen-for-user-generated-zoom-in-google-maps

Just for anyone, who stuck with the same issue, here are the approach I've used in the project:
  onDragEnd() {
    this.onUserChange();
  },
  onZoomEnd() {
    if (this.isUser) {
      this.onUserChange();
    }
  },
  onMoveEnd() {
    this.isUser = true;
  },

bindings:
    this.map.on('dragend', this.onDragEnd);
    this.map.on('zoomend', this.onZoomEnd);
    this.map.on('moveend', this.onMoveEnd);

Would be nice to replace onMoveEnd to idle (https://github.com/Leaflet/Leaflet/issues/3178), but for my case that's enough.

Any update on when this might be added?

#2267 and #2532 are open issues making the same request

The fastest way of getting this into core is a pull request ;)
We are now working on 1.0.0-rc1 release, and we have frozen the features perimeter of the 1.0, so we will basically only work on bug fixes ourselves. But any pull request will be considered.
So, to answer your question, unless someone provide a pull request, this will not be in Leaflet before 1.1 or later.

Didn't do any PR back in time as I saw you were rebuilding the whole things with 1.0.
Gonna have a look to that to implement the fix I did above.

Yes @yohanboniface, doing stuff is always the fastest way to get stuff done :)
I would like some direction as to which solution is best before I take on implementation as the issue doesn't look very simple.  I noticed an event muting option for programmatic changes to map state via the API was suggested in https://github.com/Leaflet/Leaflet/issues/3088, this may be a solution as well.

It would be awesome if I could pass arbitrary data on the zoom function in the options, and get access to those options in the zoomstart and zoomend events.
This way I you could detect it yourself because you've passed something ;-) And also it would open a lot more possibilities.
@dmitry Did you find a better way in the mean time?

How can we make it better then?


How to customize the bot input field?

When I have the "Wikipedia Search" bot/app installed and when I enter "@wikipedia" and hit return/enter, then the input field is being modified, whereas at the beginning the wikipedia logo occurs and the text "Wikipedia Search > Browse Wikipedia" and is following by an input field with a placeholder "Enter a search term".
It tried to find some documentation in order to do this for my own bot/app, but did not find any documentation. How can this be done?
Thanks for your help in advance!

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

ID: 87d66f12-3a20-8d6a-ed4c-90bc26f90591
Version Independent ID: 3725c141-86f1-b815-6568-16758d948fac
Content: Designing your bot - Teams
Content Source: msteams-platform/bots/design/bots.md
Product: msteams
GitHub Login: @heath-hamilton
Microsoft Alias: lajanuar

@michaelwechner - Taking this for internal discussion with team, I will update you with resolution.

@michaelwechner - Could you please share screen recording of your issue so that we can understand more clearly.

@michaelwechner - Could you please share screen recording of your issue so that we can understand more clearly.

please see the following screenshots:

@michaelwechner - For this you can use Search Messaging Extension. Search commands allow the users to search an external system for information either manually through a search box, or by pasting a link to a monitored domain into the compose message area, and insert the results of the search into a message. Please check this sample  and documentation.

Thanks for your feedback and pointers, I will give it a try :-)

@michaelwechner - Could you please confirm if your issue has resolved with above suggestion or still looking for any help?

I have not been able to try it yet, but I assume that it will work as you explain above

We are closing this issue for now, Please feel free to reopen it, if required.


Add references support for instance variables

Build on top of #2632 to add support for finding instance variable references.

Hi there!
Still digging but I'd love to give this one a try!

Hello! Awesome. If you have any doubts or get blocked, let us know and we can help!

Hi @vinistock
Sorry for the delay! I got completely distracted by the Factorio: Space Age 😅
I'm not certain if this functionality should narrow down the instance variable's context the way Go to Definition does. It took me some time to figure out how Go to Definition works.
Based on the LSP specification,

The references request is sent from the client to the server to resolve project-wide references for the symbol denoted by the given text document position.

I suppose finding all matched instance variables beyond context should be OK?
I’ve opened the PR and would appreciate your review! https://github.com/Shopify/ruby-lsp/pull/2787

It might be worth checking what other language servers do, but I suspect we shouldn't be matching on instance variables in other contexts, since they may be named co-incidentally, whereas for constants and class/module names, we can (generally) determine that they refer to the same thing.


[DO NOT MERGE] [1455] Remove databricks prefix from delta configs

DO NOT MERGE. We should think of a solution for #1384 (Delta SQL confs) at the same time.
Description
Resolves #1455
This PR removes the databricks prefix from Delta Configs (e.g. spark.delta.properties.defaults.$config instead of spark.databricks.delta.properties.defaults.$config). If both config keys are used, and with different values, an exception is thrown.
How was this patch tested?
New unit test, and existing tests pass.
Does this PR introduce any user-facing changes?
Yes. Users should now use prefix spark.delta.properties.defaults instead of spark.databricks.delta.properties.defaults when setting global default delta config values.

cc @Clcanny this built off of your PR https://github.com/delta-io/delta/pull/1469


include HTML tag for lists into the list snippet

Coverage remained the same at 90.476% when pulling df0a7834e7688e3e319e1bb07dc5e82c04923023 on Zack-83-patch-2 into c6a930f6adfaa0fc5a3ce3fbd4fbac3c85cea2a1 on master.


Support compatible extension "btree_gist test case"

兼容模式下，btree_gist 插件的回归测试用例需要通过
In compatibility mode, the regression test cases of the btree_gist plugin need to pass
cd contrib/btree_gist
make check and make oracle-check should pass

#443


I am getting error when installing package using pipenv.

I am trying to install coala-bears in a virtual environment using pipenv, I am getting error when using pipenv but I am able to install coala-bears using simple pip.
Describe your environment

OS Type - Linux (Fedora - 26)
Python version: Python 3.6.2
Pipenv version: version 8.2.6

Expected result
coala-bears  is installed with all its dependancies.
Actual result
Traceback (most recent call last):
  File "/usr/bin/pipenv", line 11, in <module>
    load_entry_point('pipenv==8.2.6', 'console_scripts', 'pipenv')()
  File "/usr/lib/python2.7/site-packages/pipenv/vendor/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/pipenv/vendor/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python2.7/site-packages/pipenv/vendor/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python2.7/site-packages/pipenv/vendor/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python2.7/site-packages/pipenv/vendor/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/pipenv/cli.py", line 1852, in install
    do_lock(system=system, pre=pre)
  File "/usr/lib/python2.7/site-packages/pipenv/cli.py", line 1083, in do_lock
    pre=pre
  File "/usr/lib/python2.7/site-packages/pipenv/utils.py", line 421, in resolve_deps
    resolved_tree.update(resolver.resolve(max_rounds=PIPENV_MAX_ROUNDS))
  File "/usr/lib/python2.7/site-packages/pipenv/patched/piptools/resolver.py", line 101, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/usr/lib/python2.7/site-packages/pipenv/patched/piptools/resolver.py", line 199, in _resolve_one_round
    for dep in self._iter_dependencies(best_match):
  File "/usr/lib/python2.7/site-packages/pipenv/patched/piptools/resolver.py", line 278, in _iter_dependencies
    for dependency in self.repository.get_dependencies(ireq):
  File "/usr/lib/python2.7/site-packages/pipenv/patched/piptools/repositories/pypi.py", line 171, in get_dependencies
    result = reqset._prepare_file(self.finder, ireq)
  File "/usr/lib/python2.7/site-packages/pipenv/patched/pip/req/req_set.py", line 639, in _prepare_file
    abstract_dist.prep_for_dist()
  File "/usr/lib/python2.7/site-packages/pipenv/patched/pip/req/req_set.py", line 134, in prep_for_dist
    self.req_to_install.run_egg_info()
  File "/usr/lib/python2.7/site-packages/pipenv/patched/pip/req/req_install.py", line 438, in run_egg_info
    command_desc='python setup.py egg_info')
  File "/usr/lib/python2.7/site-packages/pipenv/patched/pip/utils/__init__.py", line 707, in call_subprocess
    % (command_desc, proc.returncode, cwd))
pip.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /tmp/tmpIGoBs3build/functools32/

Can you upgrade to the latest release of pipenv and let us know if this is still an issue?

I tried upgrading my pipenv using sudo pip install -U pipenv but I am getting following error:
Exception:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/site-packages/pip/commands/install.py", line 365, in run
    strip_file_prefix=options.strip_file_prefix,
  File "/usr/lib/python2.7/site-packages/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/usr/lib/python2.7/site-packages/pip/req/req_install.py", line 854, in install
    strip_file_prefix=strip_file_prefix
  File "/usr/lib/python2.7/site-packages/pip/req/req_install.py", line 1069, in move_wheel_files
    strip_file_prefix=strip_file_prefix,
  File "/usr/lib/python2.7/site-packages/pip/wheel.py", line 247, in move_wheel_files
    prefix=prefix,
  File "/usr/lib/python2.7/site-packages/pip/locations.py", line 140, in distutils_scheme
    d = Distribution(dist_args)
  File "/usr/lib/python2.7/site-packages/setuptools/dist.py", line 365, in __init__
    self._finalize_requires()
  File "/usr/lib/python2.7/site-packages/setuptools/dist.py", line 372, in _finalize_requires
    if not self.install_requires:
AttributeError: Distribution instance has no attribute 'install_requires'

So I removed the pipenv and reinstalled it agian now I am using **pipenv**, version 8.3.2 and now I am unable to install any package using pipenv.

Based on the previous error message it seems like your system pip
installation is pretty screwed up.  Pipenv relies on a working copy of pip
to handle installation.  Beyond that you'll need to supply logs, command
output, configuration, and the relevant pipfile in order for us to do any
troubleshooting.
On Tue, Nov 7, 2017 at 12:46 AM Ratan Kulshreshtha<EMAIL_ADDRESS>wrote:

So I removed the pipenv and reinstalled it agian now I am using pipenv,
version 8.3.2 and now I am unable to install any package using pipenv.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kennethreitz/pipenv/issues/1035#issuecomment-342382030,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6PxtVKIQ11dfqDBvhJ4pKjseiP6DNVks5sz-6cgaJpZM4QS6wl
.

Also hitting this:
Traceback (most recent call last):
  File "/usr/local/bin/pipenv", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/vendor/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/cli.py", line 1866, in install
    do_init(dev=dev, allow_global=system, ignore_pipfile=ignore_pipfile, system=system, skip_lock=skip_lock, verbose=verbose, concurrent=concurrent, deploy=deploy, pre=pre)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/cli.py", line 1330, in do_init
    do_lock(system=system, pre=pre)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/cli.py", line 1121, in do_lock
    pre=pre
  File "/usr/local/lib/python2.7/dist-packages/pipenv/utils.py", line 461, in resolve_deps
    resolved_tree.update(resolver.resolve(max_rounds=PIPENV_MAX_ROUNDS))
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/piptools/resolver.py", line 102, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/piptools/resolver.py", line 200, in _resolve_one_round
    for dep in self._iter_dependencies(best_match):
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/piptools/resolver.py", line 296, in _iter_dependencies
    dependencies = self.repository.get_dependencies(ireq)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/piptools/repositories/pypi.py", line 153, in get_dependencies
    result = reqset._prepare_file(self.finder, ireq)
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/pip/req/req_set.py", line 639, in _prepare_file
    abstract_dist.prep_for_dist()
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/pip/req/req_set.py", line 134, in prep_for_dist
    self.req_to_install.run_egg_info()
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/pip/req/req_install.py", line 438, in run_egg_info
    command_desc='python setup.py egg_info')
  File "/usr/local/lib/python2.7/dist-packages/pipenv/patched/pip/utils/__init__.py", line 707, in call_subprocess
    % (command_desc, proc.returncode, cwd))
pip.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /tmp/tmpaKwDy5build/bcolz/

Pipfile
[[source]]

url = "https://pypi.python.org/simple"
verify_ssl = true
name = "pypi"

[dev-packages]

[packages]

bcolz = ">=1.1.2"
bleach = ">=2.0.0"
certifi = ">=2016.2.28"
Cycler = ">=0.10.0"
decorator = ">=4.1.2"
entrypoints = ">=0.2.3"
"html5lib" = ">=0.999999999"
ipykernel = ">=4.6.1"
ipython = ">=6.2.0"
ipython_genutils = ">=0.2.0"
ipywidgets = ">=7.0.1"
jedi = ">=0.10.2"
"Jinja2" = ">=2.9.6"
jsonschema = ">=2.6.0"
jupyter = ">=1.0.0"
jupyter_client = ">=5.1.0"
jupyter_console = ">=5.2.0"
jupyter_core = ">=4.3.0"
MarkupSafe = ">=1.0"
matplotlib = ">=2.0.2"
mistune = ">=0.7.4"
nbconvert = ">=5.3.1"
nbformat = ">=4.4.0"
notebook = ">=5.1.0"
numpy = ">=1.13.1"
olefile = ">=0.44"
opencv-python = ">=<IP_ADDRESS>"
pandas = ">=0.20.3"
pandocfilters = ">=1.4.2"
pexpect = ">=4.2.1"
pickleshare = ">=0.7.4"
Pillow = ">=4.2.1"
prompt_toolkit = ">=1.0.15"
ptyprocess = ">=0.5.2"
pygments = ">=2.2.0"
pyparsing = ">=2.2.0"
python-dateutil = ">=2.6.1"
pytz = ">=2017.2"
PyYAML = ">=3.12"
pyzmq = ">=16.0.2"
qtconsole = ">=4.3.1"
scipy = ">=0.19.1"
seaborn = ">=0.8.1"
simplegeneric = ">=0.8.1"
terminado = ">=0.6"
testpath = ">=0.3.1"
tornado = ">=4.5.2"
tqdm = ">=4.15.0"
traitlets = ">=4.3.2"
wcwidth = ">=0.1.7"
webencodings = ">=0.5.1"
widgetsnbextension = ">=3.0.3"

[requires]

python_version = "3.4"

Note the above Pipfile was automatically created by pipenv --three in a repo with this requirements.txt
bcolz>=1.1.2
bleach>=2.0.0
certifi>=2016.2.28
cycler>=0.10.0
decorator>=4.1.2
entrypoints>=0.2.3
html5lib>=0.999999999
ipykernel>=4.6.1
ipython>=6.2.0
ipython-genutils>=0.2.0
ipywidgets>=7.0.1
jedi>=0.10.2
Jinja2>=2.9.6
jsonschema>=2.6.0
jupyter>=1.0.0
jupyter-client>=5.1.0
jupyter-console>=5.2.0
jupyter-core>=4.3.0
MarkupSafe>=1.0
matplotlib>=2.0.2
mistune>=0.7.4
nbconvert>=5.3.1
nbformat>=4.4.0
notebook>=5.1.0
numpy>=1.13.1
olefile>=0.44
opencv-python>=<IP_ADDRESS>
pandas>=0.20.3
pandocfilters>=1.4.2
pexpect>=4.2.1
pickleshare>=0.7.4
Pillow>=4.2.1
prompt-toolkit>=1.0.15
ptyprocess>=0.5.2
Pygments>=2.2.0
pyparsing>=2.2.0
python-dateutil>=2.6.1
pytz>=2017.2
PyYAML>=3.12
pyzmq>=16.0.2
qtconsole>=4.3.1
scipy>=0.19.1
seaborn>=0.8.1
simplegeneric>=0.8.1
six>=1.11.0
terminado>=0.6
testpath>=0.3.1
tornado>=4.5.2
tqdm>=4.15.0
traitlets>=4.3.2
wcwidth>=0.1.7
webencodings>=0.5.1
widgetsnbextension>=3.0.3

More info:
lsb_release -a
lsb_release -a
LSB Version:	core-2.0-amd64:core-2.0-noarch:core-3.0-amd64:core-3.0-noarch:core-3.1-amd64:core-3.1-noarch:core-3.2-amd64:core-3.2-noarch:core-4.0-amd64:core-4.0-noarch:core-4.1-amd64:core-4.1-noarch:security-4.0-amd64:security-4.0-noarch:security-4.1-amd64:security-4.1-noarch
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.5 LTS
Release:	14.04
Codename:	trusty

python --version
Python 2.7.6

python3 --version
Python 3.4.3

pipenv --version
pipenv, version 8.3.2

pip --version (external to env)
pip 9.0.1 from /usr/local/lib/python2.7/dist-packages (python 2.7)

pop --version (within env)
pip 9.0.1 from /home/idanforth/.local/share/virtualenvs/fastai-eJJZx2IL/lib/python3.4/site-packages

Ok I've resolved this on two of my machines and have an idea what's happening.
Fix:
sudo apt-get install python3.4-dev

Why this may be a fix:
When installing from a Pipfile a deeper error with installing bcolz is swallowed. That error points to missing Python header files.
Is it possible that that error leaves a tmp directory in a locked state which gives rise to the error code 1 aka permissions error?

@iandanforth anything is possible although passing in --verbose to your install command might generate more output.  You can also make  a new virtualenv and install with pip directly and debug from there, but I'm going to go ahead and close this issue since it's not an error with pipenv.  If and when we get to the bottom of the cause we can see about bubbling up a more informative error

I had the same error with pipenv and finally found the problem. When installing pipenv I used pip instead of pip3 so pipenv was installed using Python 2.7 and not 3.6.
Steps to reproduce:
pip install --user pipenv
pipenv --python 3.6
pipenv install connexion
# bails out with an error :-(

The error does not occur when installing pipenv using pip3:
pip uninstall pipenv
pip3 install --user pipenv
pipenv --python 3.6
pipenv install connexion
# no errors, horray! :-)

Maybe pipenv should produce a warning when it is running using Python 2.7 and tries to install packages for a Python 3.x environment?

@yonasyifter , I think you need to sudo. You're getting a permission Error.
NOTE: I was getting this:
:stderr: Exception:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/site-packages/pip/commands/install.py", line 307, in run
    requirement_set.prepare_files(finder)
  File "/usr/lib/python2.7/site-packages/pip/req/req_set.py", line 370, in prepare_files
    ignore_dependencies=self.ignore_dependencies))
  File "/usr/lib/python2.7/site-packages/pip/req/req_set.py", line 587, in _prepare_file

.
.
.
File "/usr/lib/python2.7/site-packages/pip/_vendor/cachecontrol/serialize.py", line 87, in dumps
).encode("utf8"),
MemoryError
You are using pip version 8.1.2, however version 20.2.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Which looks similar to some of the other errors here. Except for the "MemoryError" at the bottom. I was testing in AWS on a t2.micro instance which has about 1GB RAM. Moving to a larger ec2 instance fixed the problem.


Window layout different

I opened nvim followed by :LBQuestions followed by selecting any question.
On my nvim, the input.txt opened on the left and right side is divided into executable on top and code window on the bottom.
Do I need to set something explicitly?
My init file has:
    use {
      'Dhanus3133/Leetbuddy.nvim',
      requires = {
        'nvim-lua/plenary.nvim',
        'nvim-telescope/telescope.nvim',
      },
      config = function()
        require('leetbuddy').setup({
          domain = "com",
          language = "cpp",
        })
      end,
    }

Thanks

This is due to the default way of splitting windows in Vim. By changing the options splitright and splitbelow to true, you can help resolve the problem.


Removing the prefix “m_” from data members.

Fixes # .
Did you remember to?

[ ] Add test case(s)
[ ] Update CHANGES.txt

@cbeust and @juherr - Please let me know if this can be merged.  If not, I will skip the rest of it.

LGTM! 👍 IMO, it's a step forward and it will stop to burn my eyes :p
BTW, I'm not sure @cbeust will approve too ;p

@juherr - My humble apologies. I thought I had already ran the unit tests locally before I raised the PR. Guess I didnt... embarrassed
Please give me sometime. Will take care of this once I reach home.
Once again, my apologies for the confusion.

@krmahadevan No prob, don't worry!

Closing this PR.


[feature] Primitives for tracing

Provides simple wrappers around tracing.

LGTM from what I understand about the tracing magic (which isn't that much TBH 😉). I'll approve it to avoid blocking you, but it would be great if you could have a look at the enabled/disabled stuff below and the micro-mimimi so we don't add / do stuff where it's not needed (which will probably most of the use cases outside "our" environment)... 🙏

Thanks for the review! Any idea why the pipeline is grossly acting up? What's interesting is that goQuery builds from the recent commits 🤷

LGTM from what I understand about the tracing magic (which isn't that much TBH 😉). I'll approve it to avoid blocking you, but it would be great if you could have a look at the enabled/disabled stuff below and the micro-mimimi so we don't add / do stuff where it's not needed (which will probably most of the use cases outside "our" environment)... 🙏

Thanks for the review! Any idea why the pipeline is grossly acting up? What's interesting is that goQuery builds from the recent commits 🤷

Mh, looks a bit like a dependency issue to me: https://github.com/els0r/telemetry/actions/runs/6683668660/job/18160101847#step:4:6815
Maybe a go mod tidy could do the trick? 🤔


support developing durable functions with a local Azure Functions

Sometimes the local Azure Functions host will set messagettl to −1 instead of -1 (the first numer uses unicode letter U+2212 instead of the regular '-' character). This causes Azurite to, understandiby, return a 400 error and the durable function to not start. By adding the code here this is mitigated by replacing '−' with '-' in messagettl query paramters before they are attemted to be used and thus avoiding retuinijng 400.
Thanks for contribution! Please go through following checklist before sending PR.
PR Branch Destination

For Azurite V3, please send PR to main branch.
For legacy Azurite V2, please send PR to legacy-dev branch.

Always Add Test Cases
Make sure test cases are added to cover the code change.
QueueServiceClient does not allow adding strings as messagettl. Could you please advice on how to create the test case in this instance? :)
Add Change Log
Add change log for the code change in Upcoming Release section in ChangeLog.md.
Development Guideline
Please go to CONTRIBUTION.md for steps about setting up development environment and recommended Visual Studio Code extensions.

@microsoft-github-policy-service agree

@kentis
I just tested on Azure product Storage server, and it looks not work with "−1" but only work with "-1".
Can you pass test with "−1" on product Azure?
If not, we should not take this PR change, but the durable function should be fixed?

request fail with "−1" on product Azure

POST https://[accountName].queue.core.windows.net/weitest/messages?sv=2019-07-07&sig=[hidden]&st=2023-02-25T16:00:00Z&se=2024-03-02T16:00:00Z&sp=rap&api-version=2019-07-07&messagettl=%E2%88%921 HTTP/1.1
User-Agent: Mozilla/5.0 (Windows NT; Windows NT 10.0; zh-CN) WindowsPowerShell/5.1.19041.2364
Content-Type: application/x-www-form-urlencoded
Host: [accountName].queue.core.windows.net
Content-Length: 118
Expect: 100-continue

<?xml version="1.0" encoding="utf-8"?><QueueMessage><MessageText>VGhpcyBpcyBtZXNzYWdlIDE=</MessageText></QueueMessage>
HTTP/1.1 400 Value for one of the query parameters specified in the request URI is invalid.
Content-Length: 413
Content-Type: application/xml
Server: Windows-Azure-Queue/1.0 Microsoft-HTTPAPI/2.0
x-ms-request-id: d05102bd-4003-0047-0e84-4ab199000000
x-ms-version: 2019-07-07
x-ms-error-code: InvalidQueryParameterValue
Date: Mon, 27 Feb 2023 08:21:22 GMT

<?xml version="1.0" encoding="utf-8"?><Error>InvalidQueryParameterValue<Message>Value for one of the query parameters specified in the request URI is invalid.
RequestId:d05102bd-4003-0047-0e84-4ab199000000
Time:2023-02-27T08:21:22.4939252Z</Message><QueryParameterName>messagettl</QueryParameterName><QueryParameterValue>−1</QueryParameterValue><Reason>Must be an integer value.</Reason></Error>

request pass with "-1" on product Azure

POST https://[accountName].queue.core.windows.net/weitest/messages?sv=2019-07-07&sig=[hidden]&st=2023-02-25T16%3A00%3A00Z&se=2024-03-02T16%3A00%3A00Z&sp=rap&api-version=2019-07-07&messagettl=-1 HTTP/1.1
User-Agent: Mozilla/5.0 (Windows NT; Windows NT 10.0; zh-CN) WindowsPowerShell/5.1.19041.2364
Content-Type: application/x-www-form-urlencoded
Host: [accountName].queue.core.windows.net
Content-Length: 118
Expect: 100-continue

<?xml version="1.0" encoding="utf-8"?><QueueMessage><MessageText>VGhpcyBpcyBtZXNzYWdlIDE=</MessageText></QueueMessage>
HTTP/1.1 201 Created
Transfer-Encoding: chunked
Content-Type: application/xml
Server: Windows-Azure-Queue/1.0 Microsoft-HTTPAPI/2.0
x-ms-request-id: 183c0fe6-5003-005b-2c84-4ae3f9000000
x-ms-version: 2019-07-07
Date: Mon, 27 Feb 2023 08:20:15 GMT

197
<?xml version="1.0" encoding="utf-8"?><QueueMessagesList><QueueMessage><MessageId>39d8651d-c83c-48a0-9ce1-cc18f148a0d1</MessageId><InsertionTime>Mon, 27 Feb 2023 08:20:16 GMT</InsertionTime><ExpirationTime>Fri, 31 Dec 9999 23:59:59 GMT</ExpirationTime><PopReceipt>AgAAAAMAAAAAAAAAMm4+UoRK2QE=</PopReceipt><TimeNextVisible>Mon, 27 Feb 2023 08:20:16 GMT</TimeNextVisible></QueueMessage></QueueMessagesList>
0

This means that devs will have to locally patch versions of Azurite, or add code only for local testing, since DurableTask also seems reluctant to fix trhe issue ref: https://github.com/Azure/durabletask/issues/809
This is much more of an issue when running locally than in Azure product since Culture typically varies way more on developer machines than it does in production.

"−1" can‘t pass test on product Azure, so we should not take this change.

@blueww  What if we hid the functionality behind a command line argument? Would that make the change more acceptable?

@kentis
Since product Azure doesn't accept "−1", if Azurite accept it, there might be some customer whose program with "−1" can pass with Azurite, but fail with product Azure.
And for easy customer usage, we will try to keep the parameter count as low as possible, and only add parameter when it's really necessary.
For this case, it's actually a bug of Azure Function (I don't see it repro on any other SDK, please correct me if not.).
The correct way to resolve it is: Fix the bug in Azure functions.
And before the issue is fixed in Azure Function, a workaround is patch Azurite locally, and run with your local built Azurite. (Or patch Azure Function?)

@kentis
I will close this PR.
Please fix Azure Function for this issue, as it will also repro on product Azure.


False positive null parameter using default namespace in Scala XML

XML like
<DeclareSalary xmlns="http://www.swissdec.ch/schema/sd/20051002/SalaryDeclarationServiceTypes" xmlns:sd="http://www.swissdec.ch/schema/sd/20051002/SalaryDeclaration"
causes  the following warning/error:
Null is used as a method parameter: new scala.xml.NamespaceBinding(null, "http://www.swissdec.ch/schema/sd/20051002/SalaryDeclarationServiceTypes", $tmpscope)
Reason is that a default namespace is in fact using a null parameter:
https://github.com/scala/scala-xml/blob/master/src/main/scala/scala/xml/NamespaceBinding.scala
Thus one might argue that is actually not a false positive because the parameter null is actually used, but from an end-user perspective there is no workaround, that is just how the Scala API is.

Sorry, wrong project


Added Chris UI to run using podman

@jennydaman Please review

I renamed the YAML file in accordance to the conventions described here, which saves us the effort of having to modify minichris.sh
https://github.com/FNNDSC/miniChRIS-k8s/blob/main/podman/kube/README.md


[Rancher2] [rke2 install tutorial] Update to mention 1.20 channel during install

Summary:

We should update https://rancher.com/docs/rancher/v2.5/en/installation/resources/k8s-tutorials/ha-rke2/ such that the curl install example includes the necessary environment variable and value to pull down the 1.20 line, not 1.21. This is for Rancher 2.5 docs.
This is because only Rancher 2.6 will be supporting Kubernetes 1.21 and up -- Rancher 2.5 should not use 1.21. The K3s/RKE2 team will be marking v1.21.2+rke2r1 as stable on July 7th (assuming there are no serious issues that should prevent this). When that happens, 1.21 will be stable and thus the default version used when installing (if following this docs page as it is written today).
Details:

What we should do is:
Modify this section to be like:
curl -sfL https://get.rke2.io | INSTALL_RKE2_CHANNEL=v1.20 sh -
systemctl enable rke2-server.service
systemctl start rke2-server.service

We can publish this now, rather than waiting later until 1.21 release. But it should get published before July 7th.

@ShylajaDevadiga @rancher-max question for you two:
For RHEL-like OS'es do we need to specify anything else here? Or is just INSTALL_RKE2_CHANNEL=v1.20 enough?

@davidnuzik Passing just INSTALL_RKE2_CHANNEL=v1.20 is enough

Closing because https://github.com/rancher/docs/pull/3362 was merged


Remove Service hairpin table ServiceHairpinTable and HairpinSNATTable

Pipeline modifications:

Remove table serviceHairpinTable 23.
Remove table HairpinSNATTable 108.
Add table hairpinMarkTable 81 after table
l2ForwardingCalcTable 80.

When a local Endpoint is referenced by a Service, then a
flow that matches the packet whose input and output
interfaces are the same and makes mark HairpinRegMark,
will be installed on table hairpinMarkTable. Packets with
mark HairpinRegMark will be performed SNAT with Antrea
gateway IP on table snatConntrackCommitTable.
Signed-off-by: Hongliang Liu<EMAIL_ADDRESS>

Codecov Report

Merging #2842 (1c9c3a9) into main (e880406) will increase coverage by 0.00%.
The diff coverage is 10.60%.

@@           Coverage Diff           @@
##             main    #2842   +/-   ##
=======================================
  Coverage   40.90%   40.90%
=======================================
  Files         158      158
  Lines       19539    19548    +9
=======================================
+ Hits         7992     7996    +4
- Misses      10798    10801    +3
- Partials      749      751    +2

Flag
Coverage Δ

unit-tests
40.90% <10.60%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../agent/flowexporter/connections/conntrack_linux.go
59.78% <0.00%> (ø)

...kg/agent/flowexporter/connections/conntrack_ovs.go
72.72% <0.00%> (ø)

pkg/agent/openflow/client.go
24.31% <0.00%> (+0.21%)
:arrow_up:

pkg/agent/openflow/pipeline.go
21.97% <10.23%> (-0.21%)
:arrow_down:

.../flowexporter/connections/conntrack_connections.go
58.82% <50.00%> (ø)

...gent/controller/noderoute/node_route_controller.go
45.22% <0.00%> (+1.12%)
:arrow_up:

Could you provide information about this change in the commit message & PR description? I have no context for this change and there is no linked issue.

Updated.

I think Antonin means to explain why we make the changes.
In the commit message:

flow that matches the packet whose input and output
interfaces are the same and makes mark HairpinRegMark,
will be installed on table hairpinMarkTable.
a flow will be installed into hairpinMarkTable, which sets HairpinRegMark for the packets whose input and output ports are the same.

mark HairpinRegMark will be performed SNAT with Antrea
Packets with HairpinRegMark set will be SNAT'd

gateway IP on table snatConntrackCommitTable.
in snatConntrackCommitTable

@jianjuns, Thanks. Since a SNAT ct zone is added in PR #2599, hairpin Service traffic can make use of the SNAT ct zone instead of current stateless SNAT by modifying source and destination IPs. By removing hairpin table ServiceHairpinTable and HairpinSNATTable, the OVS pipeline can be simpler.


Update Facebook

Updated to use the new Facebook pixel. Not sure if this should replace the current Facebook (obviously not backward-compatible), maybe I should pull it out into a new module. Also playing with adding a method to application_controller to make tracking even easier:
def track_facebook_event(type, options={})
  tracker do |t|
    t.facebook :track, { type: type, options: options}
  end
end

Also updated the README with some examples of how it works.
I'm happy to continue updating this as necessary, let me know if there are any changes you would like to see.
edit: lol forgot the specs. well, let me know if this is useful before I change them  ¯_(ツ)_/¯

@Omnihuron thanks for the contribution! as the new one isn't compatible, i like the idea of maybe having a separate module, do you know something about the end-of-live period for the "old" pixel?
specs are always welcome ;)

According to this page they are going to remove the old conversion pixel soon
https://www.facebook.com/business/help/1686199411616919
But yeah, until then I can see the value in creating a separate module. I'll do that this week and fix the specs and re-PR. Thanks!

@amclelland : Great stuff. have you managed to make the module?

We will merge #75 which has moved the facebook pixel into a separate handler. I think this makes more sense as the facebook pixel API is very different.  And the current handler must actually be deprecated anyway since facebook no longer supports. :)
What do you think? does that work for you? As we are currently not using the facebook pixel tracking any feedback would be super helpful.
thanks again for the pull request!
we will name the next rack-tracker release "was lange währt wird endlich gut" as it took too long to update. :D


Shouldn't kill the flow in all cases

I called GetObject and the key was not found. The flow terminates. That's a bit unfriendly, IMO. Perhaps some functions should terminate the flow, but I don't think things that may be dynamically set during runtime (eg filenames/Keys) should handle error states by terminating the flow.

This would be a breaking change for many people. However I see the value. How about an option (per node) to have either the current behaviour (default) or return the error object.
This would need to be done in the build script and applied to all nodes.

Yes, that would be a good enhancement. I'm happy to help implement that if you'd like?

Another thought this morning. This could also be handled by having a second output port on each node that would only be used in the event of an error.
This would mean users wouldn’t need a seperate switch node to determine the logic to use.  I’m leaning this way at present but happy to discuss.
If you want to implement this, happy for you to do so, please just submit the pull request with only changes to the build scripts (not all nodes updated) in the first instance.

Awesome, will do. Thanks. Sorry I wasn't any help in fixing it myself, a bit too much on at the moment!